Modify packet process

PacketFilter takes request time as nanos (i64)
Modify ingest packet

Fix: #256

Codecov Report
Base: 77.33% // Head: 77.85% // Increases project coverage by +0.52% :tada:

Coverage data is based on head (7a98b43) compared to base (b420741).
Patch coverage: 100.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #257      +/-   ##
==========================================
+ Coverage   77.33%   77.85%   +0.52%
==========================================
  Files          27       27
  Lines        3706     3735      +29
==========================================
+ Hits         2866     2908      +42
+ Misses        840      827      -13

Impacted Files
Coverage Δ

src/graphql/packet.rs
92.30% <100.00%> (ø)

src/ingest.rs
83.33% <100.00%> (+4.42%)
:arrow_up:

src/ingest/tests.rs
98.66% <100.00%> (+0.08%)
:arrow_up:

src/publish/tests.rs
99.41% <0.00%> (-0.01%)
:arrow_down:

src/publish.rs
71.42% <0.00%> (ø)

src/publish/implement.rs
77.21% <0.00%> (+1.21%)
:arrow_up:

giganto-client/src/publish.rs
78.89% <0.00%> (+2.42%)
:arrow_up:

giganto-client/src/publish/stream.rs
50.00% <0.00%> (+7.69%)
:arrow_up:

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

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


Use -sname epoch_xx@localhost + lager update

See https://www.pivotaltracker.com/story/show/154763964
This will leave node name unchange even if hostname changes.
Also, switched to an updated lager in order to avoid some intermittent failures.

I think this will require changes in the api docs too

I took the liberty or removing reference to lager in PR title - as it was split in separate PR.


Add handling for iso639-3 to valid values (yue not recognized)

This needs a couple of steps:

If the code is not in FOLIO, Stripes will not be able to show it, so we need to add any extra code in to the controlled vocabularies in FOLIO. I have asked about how to achieve this in slack today.
The code needs to be able to know what codes are legal and not. This needs to be added. The risk is otherwise that Stripes will crash.
The code needs to handle scenarios when new language lists are being used by a library and either act upon that or report it. I will start adding the reporting part now.

Setting it to blocked while waiting for the community to get back


crash

i can run with architechtury and other mods that use it, however biome makeover crashes on launch

Thanks for the report. I can't help without a crash report.
You can find them in the crash-reports folder in your Minecraft installation.

No crash reports were generated for the crash

Thanks for the report. I can't help without a crash report.
You can find them in the crash-reports folder in your Minecraft installation.

so idk what to do about that

You haven't given me any information such as BM version, Architectury version, MC Version, Fabric/Forge, why you think BM is causing the crash or anything else that could help identify the issue.
Your latest.log file might hold some answers.

BM 1.5.9, Architectury v5.10.33, mc 1.19, fabric, I am using terralith however it crashes on launch with or without it. AAH thanks for mentioning the latest.log thing, I found the issue. Dual Riders and Biome Makeover are NOT compatible for me for whatever reason

No logs provided. can't fix.


GeoIP

GeoIP is resolving to Perl repo https://github.com/maxmind/MaxMind-DB
c.f. https://github.com/jayvdb/pypidb/issues/12

Worth noting maxmind has lots of timeouts
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /minfraud/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f4880320df0>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /minfraud/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f4880320df0>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /minfraud/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f4880320df0>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /minfraud/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f4880320df0>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /minfraud/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f4880320df0>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /minfraud/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f4880320df0>: Failed to establish a new connection: -11'))
WARNING  pypidb._pypi:_pypi.py:459 https://dev.maxmind.com/minfraud/: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /minfraud/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f4880320df0>: Failed to establish a new connection: -11'))
...
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='blog.maxmind.com', port=443): Max retries exceeded with url: /2020/03/20/maxmind-statement-on-coronavirus-covid-19/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0b50>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='blog.maxmind.com', port=443): Max retries exceeded with url: /2020/03/20/maxmind-statement-on-coronavirus-covid-19/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0b50>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='blog.maxmind.com', port=443): Max retries exceeded with url: /2020/03/20/maxmind-statement-on-coronavirus-covid-19/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0b50>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='blog.maxmind.com', port=443): Max retries exceeded with url: /2020/03/20/maxmind-statement-on-coronavirus-covid-19/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0b50>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='blog.maxmind.com', port=443): Max retries exceeded with url: /2020/03/20/maxmind-statement-on-coronavirus-covid-19/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0b50>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='blog.maxmind.com', port=443): Max retries exceeded with url: /2020/03/20/maxmind-statement-on-coronavirus-covid-19/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0b50>: Failed to establish a new connection: -11'))
WARNING  pypidb._pypi:_pypi.py:459 https://blog.maxmind.com/2020/03/20/maxmind-statement-on-coronavirus-covid-19/: HTTPSConnectionPool(host='blog.maxmind.com', port=443): Max retries exceeded with url: /2020/03/20/maxmind-statement-on-coronavirus-covid-19/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0b50>: Failed to establish a new connection: -11'))
...
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /geoip/geoip2/geolite2/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e866670>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /geoip/geoip2/geolite2/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e866670>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /geoip/geoip2/geolite2/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e866670>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /geoip/geoip2/geolite2/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e866670>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /geoip/geoip2/geolite2/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e866670>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /geoip/geoip2/geolite2/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e866670>: Failed to establish a new connection: -11'))
WARNING  pypidb._pypi:_pypi.py:459 https://dev.maxmind.com/geoip/geoip2/geolite2: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /geoip/geoip2/geolite2/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e866670>: Failed to establish a new connection: -11'))
...
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /faq/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e89abb0>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /faq/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e89abb0>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /faq/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e89abb0>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /faq/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e89abb0>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /faq/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e89abb0>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /faq/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e89abb0>: Failed to establish a new connection: -11'))
WARNING  pypidb._pypi:_pypi.py:459 https://dev.maxmind.com/faq/: HTTPSConnectionPool(host='dev.maxmind.com', port=443): Max retries exceeded with url: /faq/ (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f487e89abb0>: Failed to establish a new connection: -11'))
...
DEBUG    cachecontrol.controller:controller.py:126 Looking up "https://static.olark.com/jsclient/loader.js" in the cache
DEBUG    cachecontrol.controller:controller.py:141 No cache entry available
DEBUG    urllib3.connectionpool:connectionpool.py:955 Starting new HTTPS connection (1): static.olark.com:443
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='static.olark.com', port=443): Max retries exceeded with url: /jsclient/loader.js (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0370>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='static.olark.com', port=443): Max retries exceeded with url: /jsclient/loader.js (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0370>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='static.olark.com', port=443): Max retries exceeded with url: /jsclient/loader.js (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0370>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='static.olark.com', port=443): Max retries exceeded with url: /jsclient/loader.js (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0370>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='static.olark.com', port=443): Max retries exceeded with url: /jsclient/loader.js (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0370>: Failed to establish a new connection: -11'))
ERROR    https_everywhere.adapter:adapter.py:124 handle_error requests.exceptions.ConnectionError: HTTPSConnectionPool(host='static.olark.com', port=443): Max retries exceeded with url: /jsclient/loader.js (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0370>: Failed to establish a new connection: -11'))
WARNING  pypidb._pypi:_pypi.py:459 http://static.olark.com/jsclient/loader.js: HTTPSConnectionPool(host='static.olark.com', port=443): Max retries exceeded with url: /jsclient/loader.js (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f48803a0370>: Failed to establish a new connection: -11'))
...
DEBUG    pypidb._pypi:_pypi.py:313 processing Webpage: https://support.maxmind.com/geoip-data-correction-request/
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
...
DEBUG    pypidb._pypi:_pypi.py:313 processing Webpage: https://support.maxmind.com/
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
INFO     urlextract:urlextract_core.py:567 Unknown exception during gethostbyname(26vddl1ry78464rf7e94z1ee-wpengine.netdna-ssl.com) gaierror(-11)
...


Revert "Added Ai suppported chatbot"

Reverts Manishkr1007/WordWeaver#108

@Manishkr1007 why it's get revert is there something missing??


described correct behavior of _.result in docs

_.result({foo: undefined}, 'foo', 'bar') //bar
The behavior described in the _.result docs specifies that the defaultValue (if provided) will be returned if an objects property does not exists. Though the implementation will also return the default value when the property exists but has the value undefined. Changed docs to reflect this.

Instead of changing the documentation to match the implementation, shouldn't we be changing the implementation to match the documentation?

Agreed this is an improvement

I think we can open another PR if we want to change the implementation (and now, documentation). For now, I agree that this is an improvement.


Add other translation providers

Add provides other than Bollslife to MyBible.
Implementation details can be hashed out here.
@Mallioch

test

BTW, I haven't forgotten about doing this. Expect updates soon.


chore(main): release 2.4.3

:robot: I have created a release beep boop
2.4.3 (2024-08-08)
Bug Fixes

Deno: fix process interoperability after v2.4.2 (#664) (0eb5b9f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wellwelwel/poku/releases/tag/v2.4.3 :sunflower:


Feature request: add Segment support

It would be good to have Segment as an out-of-the-box option.

Can we please have this?


Crons are not being executed.

It took me some time to debug this issue.
We are using your script to schedule raw commands using CronExpressions like ->at("0 3 * * *"). Some of our commands are with doNotOverlap() others are not. We have a total of 20 commands scheduled.
I have observed that when doNotOverlap() is set you, always set the job to run in foreground even if I haven't set the explicitly on the job.
The problem is that the CronExpression isDue() calculates the execution time always using strtotime("now").  Because the execution could take more than one minute and it runs in foreground, the next job picked from the list won't be executed because the isDue will return false, it's not on time anymore because isDue will use now() and not the time the job was scheduled.
Solutions:
1.) I don't understand why you force nonOverlapping jobs to run in foreground. You should remove that line because it doesn't make much sense to me.
2.) You should rewrite your isDue function to use the time when the job was scheduled and not current time which could be several minutes later because of the non-overlapping jobs.
  /**
   * Check if the job is due to run
   *
   * @return bool
   */
  public function isDue()
  {
    return $this->execution->isDue(date("c", $this->time)) && $this->truthTest === true;
  }

I think both 1.) and 2.) solutions should be applied on your script.

Hi @GlobalExperts
I'm forcing the job to run in foreground because I am creating a lock file and deleting it after the scheduled script is being executed. If I run the job in background I won't have a way to remove the lock file and the next cron job will never run until you manually delete the lock file.
That was the solution I found when I developed that feature, but if you have a better idea we can discuss it.
About the second issue you're absolutely right and I'm happy to merge the changes.
Thanks!

Hello @peppeocchi,
I have posted the issue by mistake from the company account.
It doesn't matter if you are running the script in foreground or background because in both cases you are removing the lock file with rm after execution. Because every job has a unique lock file it doesn't really make sense to make them wait after each other.
Because I am not a unix guru I have made a test on our CentOS server to confirm the statement above:
[coderoot@server test]$ touch test.lock
[coderoot@server test]$ php -v;rm test.lock > /dev/null 2>&1
PHP 5.6.30 (cli) (built: Jan 19 2017 07:57:06)
Copyright (c) 1997-2016 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2016 Zend Technologies
[coderoot@server test]$ ls -l
total 0
[coderoot@server test]$ touch test.lock
[coderoot@server test]$ ls -l
total 0
-rw-rw-r-- 1 coderoot coderoot 0 Feb 17 22:00 test.lock
[coderoot@server test]$ php -v;rm test.lock > /dev/null 2>&1 &
PHP 5.6.30 (cli) (built: Jan 19 2017 07:57:06)
Copyright (c) 1997-2016 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2016 Zend Technologies
[1] 11725
[coderoot@server test]$ ls -l
total 0
[1]+  Done                    rm test.lock > /dev/null 2>&1
[coderoot@server test]$ ls -l
total 0
[coderoot@server test]$

The test above confirms that it's safe to modify your $job->run function to this:
public function run()
  {
    $output = [];

    // First reorder the cronjobs
    $this->jobsInBackgroundFirst();

    foreach ($this->jobs as $job) {
      $job->setup($this->config);

      // Check if job is due and if it should prevent overlapping
      if ($job->isDue()) {

        if ($job->preventOverlap() !== false) {

          // If overlapping, skip job execution
          if (! $this->jobCanRun($job)) {
            continue;
          }

          // Create lock file for this job to prevent overlap
          $lockFile = implode('/', [$this->getTempDir(), md5($job->getCommand()) . '.lock']);
          Filesystem::write('', $lockFile);
          // Sets the file to remove after the execution
          $job->removeLockAfterExec($lockFile);
        }

        $output[] = $job->exec();
        $this->executed[] = $job;
      }
    }

    return $output;
  }

@csaba-meszaros you're right, there is no need to force the job to run in foreground.
I'll remove that line, thanks!


Cannot set value in ini file

I am trying to create an online image without anything special. From the readme I tried New-WindowsOnlineImage -ConfigFilePath $ConfigFilePath and I am getting this error. Any ideas how to fix that?
PS C:\Users\Administrator\Desktop\windows-openstack-imaging-tools> New-WindowsOnlineImage -ConfigFilePath $ConfigFilePath
6/25/2018 3:58:31 PM - Windows online image generation started.
6/25/2018 3:58:35 PM - Cannot set value in ini file: 3
Cannot set value in ini file: 3
At C:\Users\Administrator\Desktop\windows-openstack-imaging-tools\UnattendResources\ini.psm1:128 char:13
+             throw "Cannot set value in ini file: " + [PSCloudbase.Win ...
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Cannot set value in ini file: 3:String) [], RuntimeException
    + FullyQualifiedErrorId : Cannot set value in ini file: 3

I think it's some kind of permission issue but I cannot figure out what exactly. It does create the config.ini.offile file. Powershell is running as Adminstrator. I tried "Run as Administrator" on cmd, the started powershell from there and I don't see the error.

Hello,
The image generation is required to be run as Administrator. Have you solved the issue by running as Administrator?
Thank you,
Adrian

Yea, I realize that. However the issue is that I see the problem when running Powershell as Administrator. I do not see the problem when I run cmd as Administrator, then starting Powershell from there.

Is "$ConfigFilePath" a relative or absolute path?
ini powershell module does not work with relative paths.

Hello,
I will create a patch to convert the relative path to an absolute path in the ini.psm1 module, to make sure this never happens again. Thanks for the bug report.

Can you try this fix: https://github.com/cloudbase/windows-openstack-imaging-tools/pull/210 ? It should solve this issue.

Hi, thanks for the patch. I tested it and it works.


Add placeholder for Claims in the KafkaSource status

I'm adding a placeholder for Claims field of the KafkaSource.
I'd like to discuss if we really need this feature which is quite
complex and I'm not sure if it's worth the effort.
Part of #312

Proposed Changes

Add placeholder for Claims in the KafkaSource status

Release Note

None

Docs

None

I'd vote to close this /cc @devguyio


Replace remarshal with yq

Remarshal doesn’t seem to be maintained anymore.
It is also affected by upstream bugs like https://github.com/yaml/pyyaml/issues/89 which affects e.g. prometheus-operator CRDs.
Thanks @adrian-gierakowski  for suggesting this!
This would be a replacement for #25

I won't be able to test for the next 2 or 3 days but still fine to go ahead with the merge.

The merge isn't urgent for me. From my side it can wait after you had a chance to test.


Feature Request: Ability to use a picture icon

Currently the icon only supports "icons" but it would be nice to be able to use custom pictures similar to the mushroom template card I can specify a picture by providing the file path and it uses that over an icon if it's defined.

This is indeed something that is missing, I add that at in my to-do list 👌

a 'show_entity_picture' option would be nice; like in custom:button-card

This is now fixed in the new beta :
https://github.com/Clooos/Bubble-Card/releases/tag/v0.0.1-beta.8


Added NB-IoT support

Added and tested the NB-IoT network functionality based on the open nb-iot branch and the async version

Thanks for testing!
Looks good.
Next time though in the changelog if you add new things, just call the section 'Unreleased' instead of '0.1.x'.
In this case this is a breaking change so it'll have to be 0.2 anyways.


nitcorn: fix bug with binary files

Downloading binary files from a nitcorn server works as expected once again.
The API should be enough for most users but there is a few features lacking. Namely alternating custom text with files and writing custom data. We could merge the body and files attributes into a sequence of Bytables to send, with support for Bytes.
You can test the result on xymus.net, either by downloading the latest WBTW at http://xymus.net/pub/wbtw-v0.4-110-g668b261.tar.gz or by playing a clone of Super Hexagon (mostly made by @ablondin) at http://xymus.net/pub/hex/.

+1

Rerolled to remove the file_exist check.


It doesn't work with butterKnife 8.5.0

It doesn't work with butterKnife 8.5.0

yep, I've seen the same problem. Android Studio doesn't show "Generate ButterKnife Injections" on the Generate menu.

Works normally with AS 2.3.1 and ButterKnife 8.5.1.
Make sure the plugin is installed and ButterKnife is on the project classpath (the project is synced with AS).


Question: problem with word boundary in help buffer

Steps to reproduce
When I :help autocmd and place cursor on the text autocmd_add ,
then use star * command, the whole autocmd_add({replace: will
be highlighted. It's same to autocmd_add(#{replace:.
But when I directly open the file
$ vi /usr/share/vim/vim90/doc/autocmd.txt

then press * on the text autocmd_add, only autocmd_add is
highlighted. It's same to autocmd_add(#{replace:.
It is the builtin star *, i do not do any shortcut map.
Because of this, I had to turn off my script to highlight cursor word in :help buffer.
"autocmd CursorMoved,CursorMovedI * execute 'match Visual' (getline('.')[col('.')-1] =~# '\w' ? '/\<' . escape(expand('<cword>'), '/\.*$^~[]') . '\>/' : '//')

autocmd CursorMoved,CursorMovedI * if &filetype != 'help' | execute 'match Visual' (getline('.')[col('.')-1] =~# '\w' ? '/\<' . escape(expand('<cword>'), '/\.*$^~[]') . '\>/' : '//') | endif

Expected behaviour
Word boundary should be respected in :help buffer.
Version of Vim
9.0 (2022 Jun 28, compiled Oct 27 2023 01:00:56)
Environment
OS: macOS 12.7.3 (21H1015),
Terminal: macOS builtin Terminal Version 2.12.7 (445.1),
TERM: xterm-256color,

Logs and stack traces
No response

Help files use hard-coded values for the isk setting. I strongly recommend not to try to mess with those

Help files use hard-coded values for the isk setting. I strongly recommend not to try to mess with those

Hi Chris,
It does not mean my script is wrong and this difference is acceptable, right?
The purpose of the script is to highlight occurrences of a word when cursor is inside the word, and cancel highlight when cursor is outside the word.
autocmd CursorMoved,CursorMovedI * execute 'match Visual' (getline('.')[col('.')-1] =~# '\w' ? '/\<' . escape(expand('<cword>'), '/\.*$^~[]') . '\>/' : '//')

Thanks

You can get fancy and save and restore the isk option value before/after you call the expand(<cword>)  thing.
But usage questions belong rather to vi.stackexchange.com or reddit or the vim-use ml.

It's better now.
Thanks.
let save_isk=&isk | set isk=@,48-57,_,192-255 |
  \ autocmd CursorMoved,CursorMovedI * execute 'match Visual'
  \ (getline('.')[col('.')-1] =~# '\w' ?
  \ '/\<' . escape(expand('<cword>'), '/\.*$^~[]') . '\>/' : '//') |
  \ let &isk=save_isk

let's close it then.


Explore REST api provided by Google Cloud

Exploring Google Cloud's REST API (https://cloud.google.com/sql/docs/mysql/admin-api) to possibly use for the cloud crawler

I recreated the table Jocelyn made and enabled the Cloud SQL Admin API for it. It looks like we would need to use the OAuth 2.0 protocol with a service account to authorize requests.
More details:
OAuth 2.0: (documentation: https://developers.google.com/identity/protocols/oauth2)

According to https://cloud.google.com/sql/docs/mysql/admin-api#enable_the_api, “Every request your application sends to the Cloud SQL Admin API must include an authorization token,” and you can only use the OAuth 2.0 protocol to authorize requests.

Service Account: (documentation:  https://developers.google.com/identity/protocols/oauth2/service-account)

Since we have a server to server application (no user interaction), I think we need to use the OAuth 2.0 protocol with a service account.

In the documentation, they recommend that people use client libraries (offered for Java and Python) to avoid having to create and cryptographically sign JSON Web Tokens yourself.

There is a client library for JavaScript (https://developers.google.com/identity/oauth2/web/guides/migration-to-gis#implicit_flow_examples), but it is for applications with user interaction (not a service account). We will likely need to use either the Java or Python client library, unless I could figure out a way to just leave out the user interaction.

As discussed, what @katehausladen found out seems a bit complicated. A few thoughts:
First, a Stackoverflow comment claims that "it is not possible to directly connect Javascript to Cloud SQL." This may be true, but it may also have changed since the comment is from 2016.
We discussed earlier that Firebase (Firestore Database) is not an option. @Jocelyn0830 mentioned that we are using certain APIs in our Firefox extension that makes using Firebase difficult. But wouldn't these APIs only relate to the crawling and not the storage to the database? If so, we could use a similar implementation that we are using for the GPC Privacy Choice extension. So, please check the readme and ask @ekuller about this.
Here is a tutorial on Google Cloud and SQL. Maybe, not too helpful.
@sophieeng also gets involved in this once the testing implementation is done.

First, a Stackoverflow comment claims that "it is not possible to directly connect Javascript to Cloud SQL." This may be true, but it may also have changed since the comment is from 2016.

I think it is now possible to connect Google Cloud MySQL to as I found many successful cases online. For example, here is a tutorial written this year that may be helpful to us.

We discussed earlier that Firebase (Firestore Database) is not an option. @Jocelyn0830 mentioned that we are using certain APIs in our Firefox extension that makes using Firebase difficult. But wouldn't these APIs only relate to the crawling and not the storage to the database? If so, we could use a similar implementation that we are using for the GPC Privacy Choice extension.

I believe the problem of Firebase is that Firebase does not support any Firefox addon. Firebase is created for Google Chrome. Details could be found in this stackoverflow post. It is posted in 2021 so it is relatively updated.
My personal opinion is that we should stick to Google Cloud  MySQL.

Yes, if you submit your work by this Friday, 8 am, we won't deduct any points for lateness.

OK, sounds good, @Jocelyn0830!

We choose to develop our own rest api. This may be open later if needed.


ENH: Better warning message on partial fills

Even went with 'canceled' instead of 'cancelled' :us:
@lianga888 can you take a look?

lgtm :)


Fix usdt example in CI

It broke with the ubuntu upgrade in #845.
Debian usdt probes for python3:
ivan@vm:~/projects/ebpf_exporter$ sudo tplist-bpfcc -vv -l python3
python:audit [sema 0xa5f0fe]
  location #1 /usr/bin/python3 0x4728b8
    argument #1 8 unsigned bytes @ regs[22]
    argument #2 8 unsigned bytes @ regs[21]
python:gc__done [sema 0xa5f0f8]
  location #1 /usr/bin/python3 0x48637c
    argument #1 8 signed   bytes @ regs[28]
python:gc__start [sema 0xa5f0f6]
  location #1 /usr/bin/python3 0x486494
    argument #1 4 signed   bytes @ *(sp + 152)
python:line [sema 0xa5f0f4]
  location #1 /usr/bin/python3 0x4b88dc
    argument #1 8 unsigned bytes @ regs[26]
    argument #2 8 unsigned bytes @ regs[0]
    argument #3 4 signed   bytes @ regs[28]
python:import__find__load__start [sema 0xa5f0fa]
  location #1 /usr/bin/python3 0x50ec90
    argument #1 8 unsigned bytes @ regs[0]
python:import__find__load__done [sema 0xa5f0fc]
  location #1 /usr/bin/python3 0x50eca8
    argument #1 8 unsigned bytes @ regs[0]
    argument #2 4 signed   bytes @ regs[1]
python:function__entry [sema 0xa5f0e8]
  location #1 /usr/bin/python3 0x469fcc
    argument #1 8 unsigned bytes @ regs[20]
    argument #2 8 unsigned bytes @ regs[21]
    argument #3 4 signed   bytes @ regs[0]
python:function__return [sema 0xa5f0ea]
  location #1 /usr/bin/python3 0x46a01c
    argument #1 8 unsigned bytes @ regs[20]
    argument #2 8 unsigned bytes @ regs[21]
    argument #3 4 signed   bytes @ regs[0]

Ubuntu 24.04 probes:
ivan@vm:~/projects/ebpf_exporter$ sudo tplist-bpfcc -vv -l ./python3
python:import__find__load__done [sema 0xb8a3ac]
  location #1 ./python3 0x58c604
    argument #1 8 unsigned bytes @ regs[0]
    argument #2 4 signed   bytes @ regs[1]
python:import__find__load__start [sema 0xb8a3aa]
  location #1 ./python3 0x58c624
    argument #1 8 unsigned bytes @ regs[0]
python:audit [sema 0xb8a3ae]
  location #1 ./python3 0x5a220c
    argument #1 8 unsigned bytes @ regs[23]
    argument #2 8 unsigned bytes @ regs[21]
python:gc__start [sema 0xb8a3a6]
  location #1 ./python3 0x5b1540
    argument #1 4 signed   bytes @ *(sp + 72)
python:gc__done [sema 0xb8a3a8]
  location #1 ./python3 0x5b16f4
    argument #1 8 signed   bytes @ regs[21]

Currently we're using python:function__entry and naturally it doesn't work in Ubuntu.

See https://github.com/python/cpython/issues/104280

Done in #416.


A game piece of the respective color is placed at the start of the board.

Resolved with commit:
players can now be moved on the board... on client


About the decoding speed of the ELIC model

Thank you so much for your reimplementation of ELIC. However, in ELIC's original paper, they reported that the decoding latency of ELIC is much less than 100ms. But when I test with ELIC from CompressAI, the latency is about 130ms. May I ask why there is a speed gap?
Looking forward to your reply.

The measured decoding latency may depend on the CPU, GPU, or other conditions.
Their setup is mentioned here:

Supplementary Material
2. Detailed experimental settings
We implement, train, and evaluate all learning-based models on PyTorch 1.8.1. We use NVIDIA TITANXP to test both RD performance and inference speed. To test the speeds, we reproduce previously proposed models and evaluate them under the same running conditions for fair comparison. Since most of the models adopt reparameterization techniques, we fix the reparameterized weights before testing the speed. We follow a common protocol to test the latency with GPU synchronization. When testing each model, we drop the latency results (we do not drop them when evaluating RD performance) of the first 6 images to get rid of the influence of device warm-up, and average the running time of remained images to get the precious speed results.
We do not enable the deterministic inference mode (e.g. torch.backends.cudnn.deterministic) when testing the model speeds for two reasons. First, we tend to believe that the deterministic issue can be well solved with engineering efforts, such as using integer-only inference. Thus, the deterministic floating-point inference is unnecessary. Second, the deterministic mode extremely slows down the speed of specific operators, like transposed convolutions which are adopted by ELIC and earlier baseline models (Ball´e et al. and Minnen et al.), making the comparison somewhat unfair.
arXiv:2203.10886v2 [cs.CV] 29 Mar 2022

Relative measurements may be more worthwhile on different machines. Looking at their chart, it looks like:

ELIC is 20% faster than Minnen2020
ELIC is 300% slower than Balle2018
ELIC-small is 55% faster than Minnen2020
ELIC-small is 120% slower than Balle2018

[!NOTE]
The above comment still applies.

I took another look at the paper, and it says:

The architecture of $g_{\text{ch}}$ network is frankly sketched from Minnen et al. (2020).

That means instead of using a sequential_channel_ramp, the $g_{\text{ch}}$ should actually be something like:
        # In [He2022], this is labeled "g_ch^(k)".
        channel_context = {
            f"y{k}": nn.Sequential(
                conv(sum(self.groups[:k]), 224, kernel_size=5, stride=1),
                nn.ReLU(inplace=True),
                conv(224, 128, kernel_size=5, stride=1),
                nn.ReLU(inplace=True),
                conv(128, self.groups[k] * 2, kernel_size=5, stride=1),
            )
            for k in range(1, len(self.groups))
        }

The parameter aggregation network linearly reduces the dimensions to $2 M^{(k)}$.

So the current param_aggregation correctly uses the linear sequential_channel_ramp.


Output Fİle Size

My input file size is 254MB (Vİdeo bitrate: 910 kbps)
I run auto editor with the default settings as "auto-editor file.mp4"
the output image quality is the same but the file size is approx. 5 times bigger than the input size!
when I manually set the output bit rate same as the inputs', image quality decreases a lot.
how can I solve this problem?

See https://auto-editor.com/docs/file-size


Make Apache optional but keep as default; Closes dj-wasabi/ansible-zabbix-web#58

Description of PR
Keeping default functionality but splitting the Apache tasks into switchable include_tasks.
Abstracting the zabbix_apache_servername variable but setting it for the Apache tasks.
Remove the hardcode role dependency for making it optional. Update README about it.
Type of change
Feature Pull Request
Fixes an issue
https://github.com/dj-wasabi/ansible-zabbix-web/issues/58

Thanks! 👍


added S3 Bucket restriction

This assures that your S3 path is properly constructed.
Should help avoid confusion when your s3 file won't deploy because of a checksum error.

Please confirm this contribution is under the terms of the Apache 2.0 license. Thanks.

Confirmed.


Cannot find module 'stream-buffers' while installing chaincode on local fabric.

Hello. I am following the tutorial in the readme to run the marbles v3 app, but I am getting stuck when it is time to install and instantiate the chaincode.
According to https://github.com/IBM-Blockchain/marbles/blob/v3.0/docs/install_chaincode_locally.md , all that is needed is to run the commands node install_chaincode.js and then node instantiate_chaincode.js. The first time I ran install_chaincode, it seemed to do so successfully, but when I ran the second command, it gave me an Error: Cannot find module 'stream-buffers'. I thought I had messed up somewhere, so I came back and restarted fabric ( sudo docker-compose -f docker-compose-marblesv3.yaml down and then follow the steps to get it up again) and this time it is giving me the error since the first step, the chain code install ( node install_chaincode.js ).
This is the stacktrace:

user@computer:~/Desktop/ibmblockchain/marbles/scripts$ node install_chaincode.js
info: Loaded config file /home/chris/Desktop/ibmblockchain/marbles/config/marbles3.json
info: Loaded creds file /home/chris/Desktop/ibmblockchain/marbles/config/blockchain_creds_docker.json
info: Returning a new winston logger with default configurations
module.js:471
throw err;
^

Error: Cannot find module 'stream-buffers'
at Function.Module._resolveFilename (module.js:469:15)
at Function.Module._load (module.js:417:25)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/home/chris/Desktop/ibmblockchain/marbles/node_modules/fabric-client/lib/packager/Golang.js:23:12)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)

What could be causing this error?

I think you are running this on a Mac.
Can you run npm install stream-buffers and run those commands again?

I am using Ubuntu 16.04 on a x64 machine, but the command did solve the issue, thank you!


Issue with Page

How do I get all the results as opposed to just 20? When I retrieve results it says I have 1404, but I only get 20 of the articles in the articles section of the dictionary. I know it says to use the page parameter but that does not seem to rectify the issue.

Hi @kushal3553 ,
The 1404 is the number of results. It is not possible to retrieve all of these results at once. You can set the pageCount to a max of 100. The best solution is paginate through the results until you reach the amount of records (1404).
Thanks,
Matt


Fatal error occurs if nominatim database already exists

Describe the bug
I have an instance of PostgreSQL running on my host server and I'm running docker with --network="host" and -e PGHOSTADDR=<IP_ADDRESS>. If the nominatim database does not exist, the container starts up. If it does, then I get this fatal error.
createdb: error: database creation failed: ERROR:  database "nominatim" already exists
FATAL: Creating new database failed.

This requires me to drop the database in PostgreSQL before starting the container. Can this error be treated as non-fatal?
To Reproduce
Steps to reproduce the behavior:

Install PostgreSQL on the host machine and start it
Start a nominatim container and use host networking mode with PGHOSTADDR set to <IP_ADDRESS>
Stop the container after it has created the nominatim database
Attempt to start another container

Expected behavior
The container should attempt to use the existing database.
Desktop / Server (please complete the following information):

OS & Version: RHEL 8.4
Docker Version: 20.10.18
Nominatim Version: 4.1

Same problem here.

Same problem here.


Wfly 4870 Component upgrade Hibernate ORM to 5.0.0.CR2

Also removed Hibernate 3.x integration (we now include Hibernate integration for releases {4.1.x, 4.3.x, 5.0}).

Windows Build 1994 is now running using a merge of 5e2359bda41a6c41b5762029776b436b38e87892

Linux Build 6919 is now running using a merge of 5e2359bda41a6c41b5762029776b436b38e87892

Linux with security manager Build 1817 is now running using a merge of 5e2359bda41a6c41b5762029776b436b38e87892

Linux with security manager Build 1817 outcome was SUCCESS using a merge of 5e2359bda41a6c41b5762029776b436b38e87892
Summary: Tests passed: 864, ignored: 311 Build time: 0:06:32

https://github.com/wildfly/wildfly/pull/7707 addresses the HibernateSearchJPATestCasetest failure

Linux Build 6919 outcome was SUCCESS using a merge of 5e2359bda41a6c41b5762029776b436b38e87892
Summary: Tests passed: 2907, ignored: 372, muted: 1 Build time: 0:48:24

Windows Build 1994 outcome was FAILURE using a merge of 5e2359bda41a6c41b5762029776b436b38e87892
Summary: Tests failed: 1 (1 new), passed: 2907, ignored: 372 Build time: 0:48:30
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.jpa.hibernate.classfiletransformertest.ClassFileTransformerTestCase: <no details avaliable>

hmm, org.jboss.as.test.integration.jpa.hibernate.classfiletransformertest.ClassFileTransformerTestCase passed for me.  I'll run that test a few times locally.

org.jboss.as.test.integration.jpa.hibernate.classfiletransformertest.ClassFileTransformerTestCase passes locally.

I saved a copy of the org.jboss.as.test.integration.jpa.hibernate.classfiletransformertest.ClassFileTransformerTestCase deployment time exception here https://gist.github.com/scottmarlow/c56cfe07f2ea514354aa

This could be a legitimate failure

interesting that the failure only occurred on Windows.

retest this please so we can see if the error recreates on Windows.

Linux with security manager Build 1820 is now running using a merge of 5e2359bda41a6c41b5762029776b436b38e87892

Linux Build 6922 is now running using a merge of 5e2359bda41a6c41b5762029776b436b38e87892

Windows Build 1997 is now running using a merge of 5e2359bda41a6c41b5762029776b436b38e87892

Linux with security manager Build 1820 outcome was SUCCESS using a merge of 5e2359bda41a6c41b5762029776b436b38e87892
Summary: Tests passed: 864, ignored: 311 Build time: 0:06:10

Windows Build 1997 outcome was FAILURE using a merge of 5e2359bda41a6c41b5762029776b436b38e87892
Summary: Tests failed: 1, passed: 2907, ignored: 372 Build time: 0:48:10
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.jpa.hibernate.classfiletransformertest.ClassFileTransformerTestCase: <no details avaliable>

Linux Build 6922 outcome was FAILURE using a merge of 5e2359bda41a6c41b5762029776b436b38e87892
Summary: Tests failed: 1 (1 new), passed: 2906, ignored: 372, muted: 1 Build time: 0:50:06
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptTestCase.encryptedAndSignedRequest: <no details avaliable>

Windows Build 2009 is now running using a merge of ad73938202f93eef6960c9f0eca8f4f34b889949

Linux with security manager Build 1832 is now running using a merge of ad73938202f93eef6960c9f0eca8f4f34b889949

Linux Build 6934 is now running using a merge of ad73938202f93eef6960c9f0eca8f4f34b889949

Linux with security manager Build 1832 outcome was SUCCESS using a merge of ad73938202f93eef6960c9f0eca8f4f34b889949
Summary: Tests passed: 864, ignored: 311 Build time: 0:07:40

Yes let's disable that test temporarily, we badly need a WF tag with ORM 5 to move on with many other improvements to come.

Test is disabled and will later be enabled when HHH-9907 is fixed in ORM CR3.  WFLY-4895 is for enabling the ClassFileTransformerTestCase test again.  Thanks for the thumbs up @Sanne

Linux Build 6934 outcome was SUCCESS using a merge of ad73938202f93eef6960c9f0eca8f4f34b889949
Summary: Tests passed: 2905, ignored: 373, muted: 2 Build time: 0:53:09

Windows Build 2009 outcome was SUCCESS using a merge of ad73938202f93eef6960c9f0eca8f4f34b889949
Summary: Tests passed: 2907, ignored: 373 Build time: 0:53:41


Update Get-CsVoicePolicy.md

We are removing this as per Roy (that owns voice scenarios). These should not be available for online.

Docs Build status updates of commit 1b11b7a:
:clock10: Full build: the average full build time is 11 min(s) 47 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 1b11b7a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsVoicePolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Outlookhttps://aka.ms/qtex0l for iOS

From: opbld34<EMAIL_ADDRESS>Sent: Thursday, November 7, 2019 3:36:54 PM
To: MicrosoftDocs/office-docs-powershell<EMAIL_ADDRESS>Cc: Amaresh Bikki<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [MicrosoftDocs/office-docs-powershell] Update Get-CsVoicePolicy.md (#4338)
Docs Build status updates of commit 1b11b7ahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FamareshbAtMicrosoft%2Foffice-docs-powershell%2Fcommits%2F1b11b7a14eb2f05957d860ac7a117150fbbfac22&data=02|01|amareshb%40microsoft.com|396ca1f1354041f7340d08d763db5f8f|72f988bf86f141af91ab2d7cd011db47|1|0|637087666219038991&sdata=9VKATuWXeUcGdxhfd6ca0qZxDr%2FDIvfR%2Bqj66%2Fy%2FOS0%3D&reserved=0:
✅ Validation status: passed
File    Status  Preview URL     Details
skype/skype-ps/skype/Get-CsVoicePolicy.mdhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FamareshbAtMicrosoft%2Foffice-docs-powershell%2Fblob%2Fpatch-1%2Fskype%2Fskype-ps%2Fskype%2FGet-CsVoicePolicy.md&data=02|01|amareshb%40microsoft.com|396ca1f1354041f7340d08d763db5f8f|72f988bf86f141af91ab2d7cd011db47|1|0|637087666219038991&sdata=SqWmb9HAhJ7234bQHFfpIMHXaMz6STGts6wfyveNND8%3D&reserved=0    ✅Succeeded      Viewhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Freview.docs.microsoft.com%2Fen-us%2Fpowershell%2Fmodule%2Fskype%2Fget-csvoicepolicy%3Fview%3Dskype-ps%26branch%3Dpr-en-us-4338&data=02|01|amareshb%40microsoft.com|396ca1f1354041f7340d08d763db5f8f|72f988bf86f141af91ab2d7cd011db47|1|0|637087666219048948&sdata=trJoBUbxXWQqZXuVUxQbbemwuuZDCCZOXy5W%2FevTxYY%3D&reserved=0 (skype-ps)
For more details, please refer to the build reporthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fopbuildstorageprod.blob.core.windows.net%2Freport%2F2019%255C11%255C7%255C3603fd28-efce-f87f-e986-fc8a694fc312%255CPullRequest%255C201911072324089995-4338%255Cworkflow_report.html%3Fsv%3D2016-05-31%26sr%3Db%26sig%3DRqiydp4OQf6bQFTrbjYc5RSy%252BFF8JvDFFiSIlRjC9Ok%253D%26st%3D2019-11-07T23%253A31%253A49Z%26se%3D2019-12-08T23%253A36%253A49Z%26sp%3Dr&data=02|01|amareshb%40microsoft.com|396ca1f1354041f7340d08d763db5f8f|72f988bf86f141af91ab2d7cd011db47|1|0|637087666219048948&sdata=DSzUYmuV326uAU5UbiTqjlJz1owBMGW2J327aulFbQU%3D&reserved=0.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Foffice-docs-powershell%2Fpull%2F4338%3Femail_source%3Dnotifications%26email_token%3DAGDU55XPTE4GA3RQ4PWRLJTQSSRBNA5CNFSM4JKPRZBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDOGS2Y%23issuecomment-551315819&data=02|01|amareshb%40microsoft.com|396ca1f1354041f7340d08d763db5f8f|72f988bf86f141af91ab2d7cd011db47|1|0|637087666219048948&sdata=WnPG3GHdKwv7mxhiiCy%2BWfH%2B8jJImSdpJUY%2FFHJeZcA%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGDU55WYV6URO47ISIFM2IDQSSRBNANCNFSM4JKPRZBA&data=02|01|amareshb%40microsoft.com|396ca1f1354041f7340d08d763db5f8f|72f988bf86f141af91ab2d7cd011db47|1|0|637087666219058906&sdata=bMOib%2BKdmL2GlNmSfKuJiEW017tyRXS%2B5HChJFe%2Bsq4%3D&reserved=0.


Rename type and object on Like model, Inbox

Cleaning up the models to match the spec.
Renamed _type and _object to type and object for the Like Model.
Renamed _type to type for Inbox and corrected the value to 'inbox'.

LGTM


fixing issues with drive letter changes

Purpose

Customers using this approach to use mounted managed disk as data path for the Service Fabric VM extension. In some case the drive letter changed, as a result the cluster will fail on nodes.

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

#sign-off

#sign-off


[5.2.32] specific attribute as input names causing instance to call methods

Laravel Version: 5.2.32
PHP Version: 7.0.8
Database Driver & Version: MySQL 5.6

Description:
This is something that happened as soon as we upgraded to Laravel 5.2.32 from 5.2.31 so the breaking change code is the following:
Model.php 5.2.31:
/**
     * Determine if an attribute exists on the model.
     *
     * @param  string  $key
     * @return bool
     */
    public function __isset($key)
    {
        return (isset($this->attributes[$key]) || isset($this->relations[$key])) ||
                ($this->hasGetMutator($key) && ! is_null($this->getAttributeValue($key)));
    }

Model.php 5.2.32
    /**
     * Determine if an attribute or relation exists on the model.
     *
     * @param  string  $key
     * @return bool
     */
    public function __isset($key)
    {
        if (isset($this->attributes[$key]) || isset($this->relations[$key])) {
            return true;
        }

        if (method_exists($this, $key) && $this->$key && isset($this->relations[$key])) {
            return true;
        }

        return $this->hasGetMutator($key) && ! is_null($this->getAttributeValue($key));
    }

What happens is, if we add an input, lets say a checkbox field
{{ Form::model((new \App\Article), [], ['class' => 'form-horizontal']) }}
{{ Form::checkbox('delete[1]', 1, false) }}
{{ Form::close() }}

This will only work when Form::model is used. if we use Form::open it wont happen.
And we have the following model as an example:
<?php

namespace App;
use Illuminate\Database\Eloquent\Model;

class Article extends Model
{
  /**
   * The table associated with the model.
   *
   * @var string
   */
  protected $table = 'article';
}

The above code when executed will result in the following error:
ErrorException in Model.php line 2691:
Relationship method must return an object of type Illuminate\Database\Eloquent\Relations\Relation (View: /Users/vincentgabriel/Sites/app/resources/views/welcome.blade.php)

I kept trying and trying to see if this was something specific to our setup, but i just installed a new project, followed the above steps and was able to reproduce. it seems the change to the model to call methods if the attribute matches a method causing this issue.
If you change the checkbox name to 'find' 'get' etc. it'll do the same.
Can anyone confirm?

This was changed again in a later version.

@GrahamCampbell do you know which one? is there a change log between versions somewhere?

https://github.com/laravel/framework/blob/5.3/CHANGELOG-5.2.md

https://github.com/laravel/framework/commits/5.2/src/Illuminate/Database/Eloquent/Model.php

@GrahamCampbell Thank You. However, i can still confirm this is happening on 5.2.45 (anything above 5.2.32) it does not happen on 5.3.28 (latest) since there is a return statement under getAttribute method if the method exists on the class itself.
This is still a bug.

@GrahamCampbell This is the change that fixes it in 5.3.x
https://github.com/laravel/framework/commit/7ce97bec546d607ca207082722ceecabfbcc11a9#diff-d8e24a5815a26f7211e66db787be647f

Well 5.2 is actually not supported any more, please upgrade to 5.3.

@themsaid last 5.2 branch release was in late August. where can we see the branch support timeline?

We dropped support for 5.2 shortly after 5.3 was released.

For general releases, bug fixes are provided for 6 months and security fixes are provided for 1 year.

5.2 was released on 21 Dec 2015 so this week is the last one for security fixes if we follow that strictly, more info on https://laravel.com/docs/5.3/releases#support-policy

@themsaid I see. Thanks.


Revamp Grid Pages

Datagrids do not work well. Revamp those pages: Instead of a datagrid that lets the user navigate to details pages, introduce one page with a tree view on the left that servers as an overview of all objects, and lets the user select which one's details are displayed to the right in a details view (instead of going to a details page).

Mostly done, see follow-up issues #105, #108, #109


Created Request::obscure() method

When dumping entire requests using Illuminate\Http\Request::all() to log files, they can often include Personally identifiable information (PII) - such as email addresses and names.
This PR creates a new method called obscure() which, like only() and except() takes an argument(s) of keys, and will then return the same as all() would, but with the relevant keys obscured:

$request = Illuminate\Http\Request::create('/', 'GET', [
    'email' => $email,
    'foo' => 'bar',
]);

$result = $request->all();

/*
array:2 [
  "email" =><EMAIL_ADDRESS>  "foo" => "bar"
]
*/

$result = $request->obscure('email');

/*
array:2 [
  "email" => "*******************"
  "foo" => "bar"
]
*/

Can we make this configurable, same as $dontFlash array in exception handler?
https://github.com/laravel/framework/blob/621d91d802016ab4a64acc5c65f81cb9f5e5f779/src/Illuminate/Foundation/Exceptions/Handler.php#L72-L75

I believe the entire premise is wrong. You shouldn't blacklist things, that will sooner or later become out-of-sync with the actual implementation, and PII will once again be stored in the log files when someone adds another parameter but doesn't modify the logging in all the places the logging exists.
Instead, whitelist the parameters you need to log. You can use the $request->only(...) for this.

@sisve Each to to their own 🤷♂
We're using in this on an old API with success so just thought I'd share with the community so thought I'd share.
Have a good one.

Feel free to macro this into your own projects or make a package for it. I don't think I will include it in the core for now.


fix Fix table sample not take effect

Proposed changes
Modify computeSampleTabletIds location
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

add some test case please

table sample not take effect because selectedIndexId=-1 in computeSampleTabletIds, the previous run FE UT had special processing, so FE UT did not discover the BUG in time.
if (FeConstants.runningUnitTest && selectedIndexId == -1) {
     selectedIndexId = olapTable.getBaseIndexId();
}


Mergeup to master

CLA signed by all contributors.

The PA automation should merge up automatically unless there’s a conflict that needs resolving?

@joshcooper There were conflicts outside of the .pot files.

doh, missed that comment in the channel 👍


Error while trying v2.0.0-rc.3

[core@fsicentk8s01 ~]$ cat /etc/centos-release
CentOS Linux release 7.5.1804 (Core)

Cluster is currently running Kontea Pharos version 1.3.3 and will be upgraded to 2.0.0-rc.3

Continue? (Y/n)

Error:
    [fsicentk8s01] got error (Pharos::SSH::RemoteCommand::ExecError): SSH exec failed with code 1: upgrade-kubeadm.sh
+ . /usr/local/share/pharos/util.sh
+ set -ex
+ '[' -x /usr/local/bin/pharos-kubeadm-1.11.4 ']'
++ mktemp -d
+ tmpdir=/tmp/tmp.SP456NcNxg
+ mkdir -p /tmp/tmp.SP456NcNxg
+ yum install kubeadm-1.11.4 -y --downloadonly --downloaddir=/tmp/tmp.SP456NcNxg
Loaded plugins: fastestmirror, versionlock
Loading mirror speeds from cached hostfile
 * base: mirror.cisp.com
 * extras: mirror.cisp.com
 * updates: mirror.cisp.com
No package kubeadm-1.11.4 available.
Error: Nothing to do
    [fsicentk8s01] retrying after 8 seconds ...

Updated successfully to 2.0.0-rc.1 for now.

@Timer thanks for reporting this!

It appears the upgrade went smoothly, but was a little verbose:
==> Configure master @ fsicentk8s01 fsicentk8s02 fsicentk8s03
    [fsicentk8s01] Checking if Kubernetes control plane is already initialized ...
    [fsicentk8s01] Reconfiguring control plane (v1.11.4)...
    [fsicentk8s01] got error (Pharos::SSH::RemoteCommand::ExecError): SSH exec failed with code 1: kubeadm-reconfigure.sh
+ set -eu
+ unset http_proxy HTTP_PROXY HTTPS_PROXY
+ kubeadm alpha phase certs apiserver --config /tmp/kubeadm.cfg.4c16b2e47ba0f42d91f27bbc1838d3d8
[certificates] Using the existing apiserver certificate and key.
+ kubeadm alpha phase controlplane all --config /tmp/kubeadm.cfg.4c16b2e47ba0f42d91f27bbc1838d3d8
[controlplane] Adding extra host path mount "pharos" to "kube-apiserver"
[controlplane] Adding extra host path mount "k8s-audit-file" to "kube-apiserver"
[controlplane] Adding extra host path mount "k8s-audit-webhook" to "kube-apiserver"
[controlplane] Adding extra host path mount "k8s-secrets-config" to "kube-apiserver"
[controlplane] Adding extra host path mount "pharos" to "kube-controller-manager"
[controlplane] wrote Static Pod manifest for component kube-apiserver to "/etc/kubernetes/manifests/kube-apiserver.yaml"
[controlplane] wrote Static Pod manifest for component kube-controller-manager to "/etc/kubernetes/manifests/kube-controller-manager.yaml"
[controlplane] wrote Static Pod manifest for component kube-scheduler to "/etc/kubernetes/manifests/kube-scheduler.yaml"
+ kubeadm alpha phase mark-master --config /tmp/kubeadm.cfg.4c16b2e47ba0f42d91f27bbc1838d3d8
[markmaster] Marking the node fsicentk8s01 as master by adding the label "node-role.kubernetes.io/master=''"
[markmaster] Marking the node fsicentk8s01 as master by adding the taints [node-role.kubernetes.io/master:NoSchedule]
timed out waiting for the condition
    [fsicentk8s01] retrying after 1 seconds ...
    [fsicentk8s01] Checking if Kubernetes control plane is already initialized ...
    [fsicentk8s01] Reconfiguring control plane (v1.11.4)...
    [fsicentk8s01] got error (Pharos::SSH::RemoteCommand::ExecError): SSH exec failed with code 1: kubeadm-reconfigure.sh
+ set -eu
+ unset http_proxy HTTP_PROXY HTTPS_PROXY
+ kubeadm alpha phase certs apiserver --config /tmp/kubeadm.cfg.e6d94f10436012a01ca53f38da85178c
[certificates] Using the existing apiserver certificate and key.
+ kubeadm alpha phase controlplane all --config /tmp/kubeadm.cfg.e6d94f10436012a01ca53f38da85178c
[controlplane] Adding extra host path mount "pharos" to "kube-apiserver"
[controlplane] Adding extra host path mount "k8s-audit-file" to "kube-apiserver"
[controlplane] Adding extra host path mount "k8s-audit-webhook" to "kube-apiserver"
[controlplane] Adding extra host path mount "k8s-secrets-config" to "kube-apiserver"
[controlplane] Adding extra host path mount "pharos" to "kube-controller-manager"
[controlplane] wrote Static Pod manifest for component kube-apiserver to "/etc/kubernetes/manifests/kube-apiserver.yaml"
[controlplane] wrote Static Pod manifest for component kube-controller-manager to "/etc/kubernetes/manifests/kube-controller-manager.yaml"
[controlplane] wrote Static Pod manifest for component kube-scheduler to "/etc/kubernetes/manifests/kube-scheduler.yaml"
+ kubeadm alpha phase mark-master --config /tmp/kubeadm.cfg.e6d94f10436012a01ca53f38da85178c
[markmaster] Marking the node fsicentk8s01 as master by adding the label "node-role.kubernetes.io/master=''"
[markmaster] Marking the node fsicentk8s01 as master by adding the taints [node-role.kubernetes.io/master:NoSchedule]
timed out waiting for the condition
    [fsicentk8s01] retrying after 2 seconds ...
    [fsicentk8s01] Checking if Kubernetes control plane is already initialized ...
    [fsicentk8s01] Reconfiguring control plane (v1.11.4)...
    [fsicentk8s02] Checking if Kubernetes control plane is already initialized ...
    [fsicentk8s02] Reconfiguring control plane (v1.11.4)...

<repeated for each master>


cli: bump to jest 27

Hey, I just made a Pull Request!
This bumps jest to 27, as well as implements configuration detection based on package role according to #8729
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Dependencies for this has been flipped around, we need to roll out #9254 along with migration to package roles across the board before this can move forward. It's due to the removals of certain APIs in the jsdom environment in Jest 27, meaning that once we move to Jest 27 the environment needs to be set to either jsdom or node depending on the package. Not having it set correctly is likely to lead to breakages on both ends.
We could switch to Jest 27 directly and require that the environment is set in config, but we'd rather roll out package roles and make that the primary option instead, to reduce the number of migrations needed.

#10064 enables us to ship this. But still want to wait a bit longer to not force people to migrate too soon

Missing Changesets
The following package(s) are changed by this PR but do not have a changeset:

@backstage/backend-tasks
@backstage/backend-test-utils
@backstage/plugin-tech-radar

See CONTRIBUTION.md for more information about how to add changesets.
Changed Packages

Package Name
Package Path
Changeset Bump
Current Version

@backstage/backend-common
packages/backend-common
patch
v0.13.2-next.0

@backstage/backend-tasks
packages/backend-tasks
none
v0.2.2-next.0

@backstage/backend-test-utils
packages/backend-test-utils
none
v0.1.23-next.0

@backstage/cli
packages/cli
minor
v0.16.1-next.0

@backstage/plugin-tech-radar
plugins/tech-radar
none
v0.5.11-next.0

Missing Changesets
The following package(s) are changed by this PR but do not have a changeset:

@backstage/backend-tasks
@backstage/backend-test-utils
@backstage/plugin-kubernetes-backend
@backstage/plugin-tech-radar

See CONTRIBUTION.md for more information about how to add changesets.
Changed Packages

Package Name
Package Path
Changeset Bump
Current Version

@backstage/backend-common
packages/backend-common
patch
v0.13.2-next.0

@backstage/backend-tasks
packages/backend-tasks
none
v0.2.2-next.0

@backstage/backend-test-utils
packages/backend-test-utils
none
v0.1.23-next.0

@backstage/cli
packages/cli
minor
v0.16.1-next.0

@backstage/plugin-kubernetes-backend
plugins/kubernetes-backend
none
v0.4.14-next.0

@backstage/plugin-tech-radar
plugins/tech-radar
none
v0.5.11-next.0


GLM does not seem to have files data source options for (datafile, labelfile)

Hello,
I am trying to run the GLM model with a files data source. However, there does not seem to be an option for specifying a files data source with two file patterns. As an example, the Random Forest class has this line:
def learner(datafile:String, labelfile:String):(Learner, FsOpts) = learner(List(FileSource.simpleEnum(datafile, 0, 1), FileSource.simpleEnum(labelfile, 0, 1)))

which will combine the data file and label file string patterns and then allow a learner to run. This way, I can call:
val (nn1,opts1) = RandomForest.learner("data_%02d.fmat.lz4", "label_%02d.fmat.lz4");

But there does not seem to be a comparable option; this is not allowed:
val (nn1,opts1) = GLM.learner("data_%02d.fmat.lz4", "label_%02d.fmat.lz4");

And this is verified by a look at the GLM.scala source code. Would it be possible for GLM to have this option? In addition, GLM would also need to have "0" through "3" to indicate the four different GLM learners that are currently implemented.
Thanks,
Daniel

Since GLM has other options beyond data and labels for the datasource, it takes a list of file name patterns. So
val (nn,opts) = GLM.learner(List("data_%02d.fmat.lz4", "label_%02d.fmat.lz4"))
should do what you want.

PS, BIDMach has a modular architecture and its very easy to put together a Learner using whatever combination of source, model, updater etc, that you want. You can use those convenience functions (e.g. GLM.learner) as a template for your own designs.

For the benefit of anyone else who's reading this: John's suggestion works.


How do set height  of  row

I know  like  this  of setting :  ‘’sheet.column(1).width(30).hidden(false)”， but I dont know  how do set  the height of row

I'm not sure I understand, setting the height of a row is very similar.
sheet.row(1).height(30)
Row has a setter and getter for height, and Column has a setter and getter for width.


[Hub Generated] Review request for Microsoft.AlertsManagement to add version preview/2023-01-01-preview

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

@dw511214992
This is a new API that is not yet deployed in ARM - there's currently zero traffic to it.
Therefore - there are no real breaking changes here.
About the change:
I wrongly typed the disaplay metadata as an object.
Since these metadata items and dynamic and can be varying relatively quickly - a dictionary is more suitable.
To clarify - these metadata items are for UI presentation.

Created a work item:
https://msazure.visualstudio.com/One/_workitems/edit/17141531
Clarifying again - this is not a real breaking change because the API not in use

Created a work item:
https://msazure.visualstudio.com/One/_workitems/edit/17141531
Clarifying again - this is not a real breaking change because the API not in use


Merge the same translation words

Just search "code" in: https://github.com/casbin/casdoor/blob/master/web/src/locales/en.json
You will find 3 different versions under "signup", "user", "forget". We don't need these duplicates. Please merge them all into one section.

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Modified to update plugin md file for WebKitBrowserPlugin.

Closing older PRs. Pls bring to latest sprint or topic branch.


null dereference running wavpack with "-y poc.wav" option

Forwarding another bug report (https://bugs.debian.org/890014):
Running 'wavpack -y poc.wav' with the attached file raises null dereference
which may allow a remote attacker to cause denial-of-service attack
I expected the program to terminate without segfault, but the program crashes as follow
june@june:~/temp/report/wavpack/null$ ASAN_OPTIONS=allocator_may_return_null=1 ../../binary/wavpack-5.1.0/cli/.libs/wavpack -y poc.wav

 WAVPACK  Hybrid Lossless Audio Compressor  Linux Version 5.1.0
 Copyright (c) 1998 - 2017 David Bryant.  All Rights Reserved.

creating poc.wv,==4142==WARNING: AddressSanitizer failed to allocate 0xffffffff82828282 bytes
ASAN:DEADLYSIGNAL
=================================================================
==4142==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc 0x7ffff65f3e7f bp 0x000000000014 sp 0x7fffffffb148 T0)
    #0 0x7ffff65f3e7e in memcpy (/lib/x86_64-linux-gnu/libc.so.6+0x83e7e)
    #1 0x7ffff65e2b0d  (/lib/x86_64-linux-gnu/libc.so.6+0x72b0d)
    #2 0x7ffff65d7aa8 in fread (/lib/x86_64-linux-gnu/libc.so.6+0x67aa8)
    #3 0x55555557ec68 in DoReadFile /home/june/temp/report/binary/wavpack-5.1.0/cli/utils.c:618
    #4 0x5555555793ef in ParseCaffHeaderConfig /home/june/temp/report/binary/wavpack-5.1.0/cli/caff.c:506
    #5 0x555555567c3a in pack_file /home/june/temp/report/binary/wavpack-5.1.0/cli/wavpack.c:1774
    #6 0x555555565e5e in main /home/june/temp/report/binary/wavpack-5.1.0/cli/wavpack.c:1270
    #7 0x7ffff65902b0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x202b0)
    #8 0x5555555609a9 in _start (/home/june/temp/report/binary/wavpack-5.1.0/cli/.libs/wavpack+0xc9a9)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV (/lib/x86_64-linux-gnu/libc.so.6+0x83e7e) in memcpy
==4142==ABORTING

This bug was found with a fuzzer developed by 'SoftSec' group at KAIST.
poc.wav is available at https://bugs.debian.org/cgi-bin/bugreport.cgi?att=1;bug=890014;filename=poc.wav;msg=5.

Thanks again! This was actually already fixed with 8e3fe45a7bac31d9a3b558ae0079e2d92a04799e, so I'm closing.


about Table 3 in the paper

Hello, thanks for the good work.
Just about the Cross-domain generalization evaluation of PSMNet in Table 3.
In the Table 3, the KITTI2015 D1_all of PSMNet trained on Scene Flow datatest  is 16.3, while we got the 28.7, which is far from that reported in your paper. And the pre-trained model from github performances 28.
Wondering the reason about it.
Thanks.

Hello,
Thank you for your concerns about our work. The d1_all of PSMNet we reported in Table 3 is from the Table 3 of DSMNet[1]. Actually, the PSMNet trained by ourselves also can not achieve such performance. As we afraid this is caused by our own improper implementation, we select to report the better result. The open source URL for DSMNet is  https://github.com/feihuzhang/DSMNet. Maybe you can ask them the rationale?
[1] Zhang, Feihu, et al. "Domain-invariant stereo matching networks." European Conference on Computer Vision. Springer, Cham, 2020.


SMES arent mapped on almayer

Description

Reproduction

Screenshots

Additional context

SMES aren't implemented yet but yes correct


Add RFC: approx-asserts

add assert_approx macro to libstd prelude
add assert_tol macro to libstd prelude

Rendered

// You could do something different with macros to differentiate the
// relative and absolute. I'm not sure if `=` works with macros...expressions
// usually just evaluate...not really experience with this type...
assert_tol!(1.003e8f32, 1e8f32, rel = abs = 1e-2f32);
assert_tol!(1.003e8f32, 1e8f32, rel = .01, abs = .1);

I’d prefer just having an approx_eq method on floats, and doing assert!(1.0.approx_eq(1.0001)); as an assertion. Having only a special assert_eq!-like macro seems very strange. I don’t think assert_eq! should exist at all, but that’s not strictly relevant to this RFC and not something I expect to change any time soon.
I think that if we do get a macro like this, we should get a method on floats like it as well. Having a macro to assert something but not a simple method to test it would be pretty ridiculous.

@P1start do you say this out of simple reasoning or experience? I'm not saying you're wrong, but the only time I have wanted approximate equality outside of an assertion is when error-handling user input of the form "I require a list of numbers whose sum must be 1", at which point it is usually clearer to write something like if !(0.999 < sum && sum < 0.001) {...} than bothering with a function.
Other than that, your suggestion has two issues: (minor) it's longer to type and (major) poor error messages:
use std::num::Float;
/// A simple approx-equals function
fn is_approx<T: Float>(x: T, y: T, atol: T) -> bool { (x - y).abs() < atol }
/// Geometric series
fn geom(r: f64, n: u32) -> f64 { if n > 1 { r + r * geom(r, n-1) } else { r } }
fn main() {
    assert!(is_approx(geom(0.5, 10), 1.0, 1e-6));
}

panics with message
thread '<main>' panicked at 'assertion failed: is_approx(geom(0.5, 10), 1.0, 1e-6)', <anon>:7

but what's the value of geom(0.5, 10)?

@mdinger thanks for the hint, will consider

Thanks for the PR!
In general, we're working right now to cut down the standard library to a conservative core that we're happy to commit to. While these macros seem useful, there's not in my mind a strong reason to introduce them into the standard library and prelude right this second -- in general, it's better for this kind of thing to begin its life in the crates.io ecosystem and gradually migrate into std only if needed for standardization or widespread use.

Sounds like good advice.


also download cover

if possible, download convers and put it into the chapter folders
so it will be like
\books\chapter x\images.png
\book\cover.png

Thanks @Shikitsuna for a good idea.
If I understood you correctly, you mean
graph TD;
  Book-->Chapter1;
  Book-->ChapterX;
  Book-->Cover.png;

  Chapter1-->Image1;
  Chapter1-->ImageX;

  ChapterX-->image1;
  ChapterX-->imageX;

Implemented in release v0.1.1


created check for door texture

Does it, and do we want to make declaring the door mandatory ?

If there is a door on the map, then in my opinion a texture must be provided


Re-implement association unloading in elastic_index!

https://github.com/inaturalist/inaturalist/blob/034d73cb02a0d1410696d56037542ca39d1d55f6/lib/elastic_model/acts_as_elastic_model.rb#L305-L325 tries to track what associations have been preloaded before indexing begins and then unloads them when indexing is complete to avoid memory leaks. It relies on an association_cache attribute that was made private in https://github.com/rails/rails/pull/16989, so it's not working in Rails 5. Assuming this degree of memory management is still necessary, we need to re-implement this without association_cache.

Discussed with Patrick and he thinks we should just leave this out and revisit if we start noticing memory problems again.


Make pax/praxis editable again

The rosetta build relies on pax/praxis being installed editable, so this re-enables the editable install in the vanilla pax build

LGTM

As the pax nightly are currently broken, can we merge today this PR?


New max-limit seatbelt

This is to solve #176 for Risecamp @shivaram @ooq  could you guys take a quick look

LGTM. Should we make it possible to pass the parameter in map()?

@ooq No I want it to be hard for them to override
@shivaram I'll change it to scheduler!
On Sun, Sep 3, 2017 at 12:04 AM, Qifan Pu<EMAIL_ADDRESS>wrote:

LGTM. Should we make it possible to pass the parameter in map()?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pywren/pywren/pull/177#issuecomment-326788259, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACPEtAMjwMxzmFC6uQ4qZ_MXbBVyC4sks5sek-agaJpZM4PLD1b
.


Implement Extension Loader

Extensions are now scanned and loaded on demand.
Extensions found are cached into a file that is used to load them (which is also exported).
Editor will ask to restart when an extension requires core functionality.
Editor will attempt to load extensions always before importing or loading scenes. This ensures extensions can register the relevant types.

Looks good to me code wise. Will take some time hands on to see if it covers all bases.


Failed with Termsig=11 if string's length less then 8

I'm trying to use it on CentOS 7 with PHP 7.2.10 and seems like it cannont work with input strings with length < 8.
hash_string.log
hash_string.phpt.txt

Fixed in 00e70a5, thank you!


aac(6')-Iaa excluded

Hi,
I am wondering why the gene aac(6')-Iaa_1_NC_003197 is excluded.
Thanks!

Hello @Andrea-SS ,
If I recall correctly, it's because a number of genomic samples were being predicted to be resistant to gentamicin due to the presence of this one gene but a large portion of them were not actually resistant to gentamicin (https://github.com/phac-nml/staramr/issues/27). However, you can keep aac(6')-Iaa_NC_003197 in your results by using --no-exclude-genes.
Aaron

Thank you!


[BugFix] Only remove partition vision infos when mv's partition has been dropped

Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:disappointed: fail : 49 / 83 (59.04%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/common/SyncPartitionUtils.java
21
52
40.38%
[585, 586, 587, 588, 590, 591, 592, 594, 595, 596, 597, 598, 601, 603, 613, 615, 616, 623, 655, 656, 657, 658, 659, 660, 662, 663, 664, 668, 670, 674, 677]

:large_blue_circle:
com/starrocks/scheduler/PartitionBasedMvRefreshProcessor.java
11
13
84.62%
[457, 459]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/MvUtils.java
14
15
93.33%
[962]

:large_blue_circle:
com/starrocks/catalog/MaterializedView.java
3
3
100.00%
[]


bug: A memory error is occurring in standalone mode.

Steps to Reproduce

using standalone mode in react editor
When re-rendering the editor, the 'indioWorker' is re-initialized each time.

Actual behavior
Runtimeerror: memory access out of bounds.
It appears to be an error that occurs only in specific environments
Expected behavior
Errors should not occur.
Screenshots

Desktop (please complete the following information):

OS: macos
Browser: chrome
Version: 117

Ketcher version [e.g. v2.4.2].
"ketcher-core": "^2.12.0",
"ketcher-react": "^2.12.0",
"ketcher-standalone": "^2.12.0"

We are using the 'ketcher-standalone' module with a patch. When it's initialized only once, the error disappears, and performance improves
var _init = null;

var StandaloneStructServiceProvider = function () {
    function StandaloneStructServiceProvider() {
        _classCallCheck(this, StandaloneStructServiceProvider);
        _defineProperty(this, "mode", 'standalone');
    }
    _createClass(StandaloneStructServiceProvider, [{
        key: "createStructService",
        value: function createStructService(options) {
            if (_init) {
                return _init;
            }

            _init = new IndigoService(options);

            return _init;
        }
    }]);
    return StandaloneStructServiceProvider;
}();

We are observing this same crash too on Macs using Chrome/Brave. Seemed to appear with v117 of Chromium.

FWIW we're no longer seeing this crash and suspect the issue was fixed in a later version of Chromium 🤞

I'm not getting any errors anymore either!~!


Sync using shared (not published) link

This would be a great feature for syncing without it having to be published on the web  :-)

I'm don't sure I understand the motivation for using the share link instead of the published one. Maybe if you could expand upon it
The main issue is that the html format for a shared page is a lot more complicated then the published page. I also think it would have to be a public link anyway

Hi Conor,
I thought that this might be the case but thought I would ask anyway  :-)
I am thinking along the lines of having an option for private flashcards is all.  This would mostly be for flashcards containing sensitive information that they do not want published to the web.

Hey ohn8,
So the really issue is that you would like a way to have a private doc that you can share between a small group?
I doubt  that the public pages are indexed but very hard to know tbh.
The probably solution would be to allow some kind of download from the normal edit page and allow you to manually sync with that?

Hi Connor,
I can't speak for others but in my case I couldn't see a need to share the doc.
If what you described is possible it sounds like that would be a good solution  ;-)

Hey,
So the main reason for sharing the document it that you have a publicly accessible link that you can scrape from. I don't think the link is index so it probably is quite hard to find without someone specifically sharing it with you.
If you are more just thinking of something that is all local then I have another add-on https://ankiweb.net/shared/info/1029306148 that might be a better fit to your usecase?

So I have a development plan for half tested but it will be a while until I can get around to completing it unfortunately.
I will update the issue once I have started working on it though


Stepper component has hard coded lock glyph

Current Behavior
The lock emoji used for "locked" Stepper/Steps cannot be changed. I'm guessing there are users where that look will not mesh with their site design.
It would be nice to have a <slot/> or something for extensibility.
Steps To Reproduce

Using Stepper/Step
Configure the Step to be locked.
Notice the Lock Icon displayed.

Anything else?
No response

FYI this will now use an embedded SVG icon that will auto-adapt the to theme settings using the fill-token design token.

Excellent. That was fast!


API doesn't return full list of incidents

Describe the bug
I am trying to get all closed incidents from a certain period of time, for example the last 7 days. However, my script doesn't return a complete list of incidents and some are omitted. My code is as follows:
import demisto_client.demisto_api
from demisto_client.demisto_api.rest import ApiException
import urllib3
from datetime import datetime
from datetime import timedelta
import dateutil.parser as dparser
import pytz

# Get start and end of report
reportEndDate = pytz.UTC.localize(datetime.today()).replace(hour=0, minute=0, second=0, microsecond=0)
reportStartDate = pytz.UTC.localize((datetime.today() - timedelta(days=7)).replace(hour=0, minute=0, second=0, microsecond=0))

# Configure connection-related parameters and create an API instance
urllib3.disable_warnings()
api_key = '...'
host = 'https://...'
DEMISTO_VERIFY_SSL = False
api_instance = demisto_client.configure(base_url=host, api_key=api_key, debug=False, verify_ssl=DEMISTO_VERIFY_SSL)

incidentsClosedThisWeek = []
lastIncidentCloseTime = reportStartDate

# Since the API returns only 25 incidents, create new requests once per every 25 incidents. To create a new request, get the closed date of the incident closed last. Append all results to a list
iterate = True
while iterate:
	print ("Starting to iterate from datetime:", lastIncidentCloseTime)
	filter = demisto_client.demisto_api.SearchIncidentsData()
	inc_filter = demisto_client.demisto_api.IncidentFilter()
	inc_filter.from_closed_date = lastIncidentCloseTime
	inc_filter.sort = [{"asc": True, "field": "id"}]
	filter.filter = inc_filter
	api_response = api_instance.search_incidents(filter=filter)
	data = api_response["data"]
	for i in range(len(data)):
		incidentsClosedThisWeek.append(data[i])
	if len(data) == 25:
		for i in range(25):
			if lastIncidentCloseTime < dparser.parse(data[i]["closed"]):
				lastIncidentCloseTime = dparser.parse(data[i]["closed"])
		data = None
	else:
		iterate = False

# Fill a list of IDs from each incident (this is to demonstrate the issue, the rest of the code is from my actual script)
incidentIDS = []
for incident in incidentsClosedThisWeek:
    if incident["id"] not in incidentIDS:
        incidentIDS.append(incident["id"])
print(incidentIDS)

To Reproduce
Steps to reproduce the behavior:

Copy the code snippet above
Modify the connection parameters
Run the script and check for missing incidents

Expected behavior
All closed incidents should appear but instead some (most) are missing.
Environment

Python Version: Windows - 3.7.8, Linux - 3.6.8
OS: Irrelevant, tested on Windows and Linux
Demisto Version: 5.5.0 70066
Content Version: 20.7.0 (61242)

Additional context
For some reason, if I just get the closed time of the last incident of the batch, e.g.:
	if len(data) == 25:
		lastIncidentCloseTime = dparser.parse(data[24]["closed"])

Then more incidents appear in the list which makes me think that's a bug with the API. Furthermore, a similar issue appears when I remove the sort filter - then there are less incidents coming back for some reason (even though the sort shouldn't affect the total number of incidents returned by Demisto).

The issue here is that you are sorting on the id field but using the closed field to advance the iteration. This causes you to skip date ranges. You should probably sort on the closed field.

Hi @glicht,
Thank you for your quick reply. That makes sense and it does seem to work as expected.  I can now see the flaw in the logic (you could get the date/time of the last closed incident in the batch as I do in the script above, but that doesn't guarantee that an incident with a higher ID wasn't closed earlier, thus skipping the incident).
Thanks for your help and sorry for the pointless issue.
Best regards :)


undefined method `deep_symbolize_keys' for false:FalseClass

/usr/local/lib/ruby/gems/2.4.0/gems/fir-cli-1.6.0/lib/fir/util/config.rb:21:in config' /usr/local/lib/ruby/gems/2.4.0/gems/fir-cli-1.6.0/lib/fir/util/login.rb:11:in login'
/usr/local/lib/ruby/gems/2.4.0/gems/fir-cli-1.6.0/lib/fir/cli.rb:125:in login' /usr/local/lib/ruby/gems/2.4.0/gems/thor-0.19.4/lib/thor/command.rb:27:in run'
/usr/local/lib/ruby/gems/2.4.0/gems/thor-0.19.4/lib/thor/invocation.rb:126:in invoke_command' /usr/local/lib/ruby/gems/2.4.0/gems/fir-cli-1.6.0/lib/fir/cli.rb:181:in invoke_command'
/usr/local/lib/ruby/gems/2.4.0/gems/thor-0.19.4/lib/thor.rb:369:in dispatch' /usr/local/lib/ruby/gems/2.4.0/gems/thor-0.19.4/lib/thor/base.rb:444:in start'
/usr/local/lib/ruby/gems/2.4.0/gems/fir-cli-1.6.0/bin/fir:8:in <top (required)>' /usr/local/bin/fir:22:in load'
/usr/local/bin/fir:22:in `'

I  got the same issue .Is there anyone figure it out?
There are two macbook difference in ruby version ,one is 2.4.0 got successful when fun 'fir login xxx' ,the other one get this error has ruby with version 2.0.0.
Is it the key point ? I am not sure.

I also got the same issue. Is there anyone figure it out？

I solve this problem.
Solution:
Delete the file ~/.fir-cli, and login angain.

@linet
Where are the file ~/.fir-cli?I didn't find it. Please give advice or comments


does this support HTTPS - Let's Encrypt?

Better to have Let's Encrypt

Hi there 👋
I'm not sure I understand the rationale behind using Let's Encrypt on an alpine Docker image?

Hi,
Thank you for the reply, Why I mentioned Let's Encrypt is then, using this docker image we can host HTTPS-enabled Laravel app easily.

Yes, I understand this but this is an alpine image which does not include php-fpm, nginx or anything that would actually serve your application. Thus, support for HTTPS should be implemented at a different level, e.g. on a different container.
I hope this helps.