Optimize the installation of fermitools

Hi all,
 we need to have multiple versions of the fermitools installed on a dedicated place,

so comparisons can be made between the different results for each version.
 I tried to use the --prefix option for conda create, writing such command :

   conda create fermitools=1.0.0 --prefix=$NFS_ZONE/fermitools/1.0.0

and it generates the fermitools package on this place, occupying 3.2GB

This directory includes also all  needed packages such as astropy for example,
which occupies at least 900MB  when installed alone.
We would like to optimize the disk usage for our different local installations
by taking advantage that many underlying packages use the same version
(as GLAST_EXT did in the past)
Would it be possible, during installation, to refer to a previous existing directory,
and copy only the differing versions ?
This kind of incremental update would save a lot of disk space.
Thanks in advance,
David Landriu
PS:   I tried to install first astropy alone, activate this astropy environment and
launch again the conda create, but it did not take care of this existing astropy,
and replicated it in the fermitools installation, even when I tried the --use-local option.

Hi @landriu,  I've tried a couple of different methods to implement this in Conda but frankly the Conda system is in no way designed for sharing external dependencies (indeed, the way conda enrvironments are set up it explicitly tries to prevent this from happening.)
We are going to discuss this further at the FSSC but for now your best bet (if saving disk space is critical) would be to compile the tools on your own outside of the conda system.  You could use conda to install all of the dependencies in one location using the
conda install --only-deps -c conda-forge -c fermi fermitools
command and then point SCons at the install location.  The commands that conda uses to package the tools are in this file (it simply executes this bash script):
https://github.com/fermi-lat/Fermitools-conda/blob/master/build.sh
To do this you will need to install 'fermi-repoman' (to check out the packages) in addition to SCons.  This can be installed via pip.  You may also find some of the information on this page useful:
https://github.com/fermi-lat/Fermitools-conda/wiki/Contributing-to-the-Fermitools

It would only be possible to build a fermitools package with a shared source activate dependencies if Fermitools is built outside conda using Joe's suggested method.

and could it be possible for FSSC to provide such a short version of fermitools, to be associated
locally by the user with the needed dependencies as another delivery ?
Dependencies should vary slower than fermitools, I guess, and many versions could work with
same external packages ?

In a sense this short version is available currently in the github source repositories using Joe's suggested method.
To the best of my knowledge it is not possible to provide this in Conda.
Note this is not a use case the FSSC has historically supported for a number of reasons. Dependencies do vary (quickly, more quickly than the fermitools will) and we can only guarantee a version of the tools will work with the all of the specified dependencies. This is why Conda is so powerful; it ensures exact dependencies are always properly managed.
Using a shared dependency model is possible. If the user wishes to build from source via Joe's suggested method. This is internally how we build the tools for Conda, and it relies on the older model used by the internal LAT-team ScienceTools. An existing, local GLAST_EXT directory points to the many shared externals. Version and dependency management is entirely the responsibility of the user.
This method works for site installation, and is still viable, but not the goal of the FermiTools or Conda.

Do you have another package / dependency manager system you're aware of that you prefer? One that has the kind of model you're describing, with shared dependencies in addition to segregated environments? If so that could certainly be something we would expand support to in the future. We have tentative plans to bring the FermiTools to pip for example.

What is your system currently using? An OS manager like yum or apt-get? An HPC manager like SPACK or Environment Modules (or the older 'tap'), or Conan? These are the ones I've heard of.

Hi Alex,
in fact I was using the installer.py from Tom Stephens at SLAC, that I modified to check

if dependencies were already available on local GLAST_EXT, and just downloaded the necessary
package provided by SLAC, adapted to our system in Saclay.
So it was really simple to deal with deliveries on our side, with the benefit of sparing disk space.

I think I did a ftp a long time ago :
I can give you the sources here :
*** Original installer.py
installer.py.txt
*** Modified script :

Saclay_local_installer.py.txt

I should note: the user should only have to compile once.  This should generate more portable binaries.

Hi ,
I have trained for some months to use the conda environments, which are all stored
in the conda/envs directory, and it seems if I am not mistaking, that anaconda is doing
a kind of releases management, not reinstalling the components when already defined.
We could then consider using conda environments is a solution to this problem .

Hi David,
I've been working on a system to do exactly this, making development easier for local builds without relying on conda-build. It should be viable for all future releases, as well as our current beta releases 1.3.6 and 1.9.13
https://github.com/fermi-lat/fermi_dev_scripts.

This issue is resolved with the new fermitools 2.2.0 version.


Client side failover reading configuration from cloud config servers

[suggestion for enhancement/discussion]
We're using a cloud config server. In order to be resilient to failure of the server, we have multiple instance behind a load balancer. However, in case the load-balancer fails, we'd need some IP-Level failover setup to guarantee access to the config servers.
We were wondering, if it would be easier to allow multiple endpoints for configuration via
spring.cloud.config.uri or a similar property, so that the client can try to configure itself with a secondary uri if the first fails.
Is this something considered of value? if so, we'd like to work on it with a little help.

I think resilient load balancers are free or cheap in pretty much any cloud environment, so I wouldn't make this a high priority. But if you have some code you want to contribute we can definitely take a look. It would be most likely to be successful if it stayed orthogonal to the existing code, so we don't risk making what is now quite simple into something complicated. Maybe the LoadBalancer abstraction from spring cloud would be useful?

I've had others ask for this functionality as well. It's similar to how eureka works too.

Just wondering: If we would implement this, how exactly would one USE this?
Provide a comma separated List to "spring.cloud.config.uri" or have a secondary "spring.cloud.config.uris" that has a list?

I'd start with a comma separated list. Would be simple to change if needed.

Thanks for your feedback!
After talking to my colleague @uweschaefer we would start working on this.

I signed the Individual Contributor Agreement, my confirmation number is<PHONE_NUMBER>16100217.


Create or delete a catalog item on update

When updating an existing AnsiblePlaybook catalog item - add the ability to add a new retirement playbook if one didn't previously exist before: Case A
Also, allow deleting of an existing job_template if the update does not pass in a :playbook_id:
Case B
Further more:  If you create a new catalog item, delete it, and then create a new one with the same name as the old one you will no longer receive an error: Case C
Testing notes:
Case A

Create new Ansible Playbook Catalog item without a retirement playbook
Edit the above Catalog Item and add a retirement playbook.

Case B

Create new Ansible Playbook Catalog item without a retirement playbook
Edit the above Catalog Item and add a retirement playbook.
Edit the above Catalog Item again and delete the retirement playbook.

Case C

Create a new catalog item
Delete that catalog item
Create a new catalog item with the same name as in part 1.

https://www.pivotaltracker.com/story/show/141868401
https://www.pivotaltracker.com/story/show/141265431
https://bugzilla.redhat.com/show_bug.cgi?id=1438839
https://bugzilla.redhat.com/show_bug.cgi?id=1442006

@miq-bot assign @bzwei
@miq-bot add_label bug, providers/ansible_tower, services
cc - @gmcculloug

@bzwei - I believe I have addressed your feedback.
cc - @gmcculloug

@miq-bot add_label blocker

Fine backport details:
$ git log -1
commit b82c87a1f49f94afcc94570eb8db5f11c651b924
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Tue Apr 25 17:30:30 2017 -0400

    Merge pull request #14830 from syncrou/deux_1438839_create_or_delete_catalog_item_on_update

    Create or delete a catalog item on update
    (cherry picked from commit 258dd189206418143b9f948bbc93aefac972ee66)

    https://bugzilla.redhat.com/show_bug.cgi?id=1445894
    https://bugzilla.redhat.com/show_bug.cgi?id=1445942


Error while installation

During pip installation got error as below, tried on python 2.6.6 and 2.7.6 on CentOS
Downloading/unpacking tweepy
Downloading tweepy-3.3.0.tar.gz
Running setup.py egg_info for package tweepy
Traceback (most recent call last):
File "", line 16, in
File "/tmp/pip-build-root/tweepy/setup.py", line 17, in
install_reqs = parse_requirements('requirements.txt', session=uuid.uuid1                                        ())
TypeError: parse_requirements() got an unexpected keyword argument 'session'
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 16, in
File "/tmp/pip-build-root/tweepy/setup.py", line 17, in
install_reqs = parse_requirements('requirements.txt', session=uuid.uuid1())

TypeError: parse_requirements() got an unexpected keyword argument 'session'

Command python setup.py egg_info failed with error code 1 in /tmp/pip-build-root                                        /tweepy
Storing complete log in /root/.pip/pip.log

Sorry about this issue, my mistake
didnt go through the older issue.
pip install --upgrade pip had done the trick


ACM repo initialization takes forever

Here is the log:
╷
│ Error: local-exec provisioner error
│
│   with null_resource.init_acm_repository,
│   on tenant-configuration.tf line 30, in resource "null_resource" "init_acm_repository":
│   30:   provisioner "local-exec" {
│
│ Error running command
│ '"/home/lgrangeau/github.com/googlecloudplatform/federated-learning/terraform/scripts/init-acm-repository.sh"
│   "./acm-repo-path"
│   "https://source.developers.google.com/p/lgu-demos/r/fl-configsync"
│   "main"
│ ': signal: interrupt. Output: Cloning into './acm-repo-path'...
I had to interrupt it because it has been running for like 15mn before interrupting

Ok, it seems that it's related to some king of popup I have in VSCode asking for username and password I didn't see (or didn't show ?) the first time

/shrug
Please, reopen if it happens again.


Add the latest static Irish translations

What is the context of this PR?
Adds the latest fixes from crowdin to the Irish static messages file.
How to review
Check the translated strings match crowdin.
Checklist
New static content marked up for translation
Newly defined schema content included in eq-translations repo

 Load Irish PO files for NISRA

 Load Irish PO files for NISRA

Spacing issues fixed. Thanks guys!

Spacing issues fixed. Thanks guys!


[All] Chart merging is blocked - PRs do not get Contribution Allowed label added

Since Monday 2019-Nov-4 no PRs are getting the Contribution Allowed label added to them and so cannot be merged.

Thanks for calling this out. The addition of the label has been fixed for any new commits pushed.
We can close this after we have gone through the backlog of those missing the label and adding it.

This should now be fixed. The backlog of those missing the label has been processed.


Does piscope supports noise filter?

piscope does not have any noise filters.
However piscope takes its data from pigpio which does have glitch and noise filters.  If the filters are set with pigs they will affect the data fed to piscope.
http://abyz.me.uk/rpi/pigpio/pigs.html#FG
http://abyz.me.uk/rpi/pigpio/pigs.html#FN

piscope does not have any noise filters.
However piscope takes its data from pigpio which does have glitch and noise filters.  If the filters are set with pigs they will affect the data fed to piscope.
http://abyz.me.uk/rpi/pigpio/pigs.html#FG
http://abyz.me.uk/rpi/pigpio/pigs.html#FN

Great, thanks your guide!

Great, thanks your guide!

@joan2937 Why I get socket connect failed?

@joan2937 Why I get socket connect failed?


Jacobian

Codecov Report

Merging #48 into master will not impact coverage.

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

Coverage remained the same at 100.0% when pulling 2ac9a91331b3b9ddf6f724822e568bcec9328344 on jacobian into 4d30c8afd2070832a5c6aa5cea83272f63f39550 on master.


Temporary workaround for #555 team-removal and side-kick removal

Summary
This still has issues, to avoid using MapInstance in Team.cpp there is a condition where removing a sidekick can result in one member of the sidekick duo having a "sidekick" state still. This really wont be resolved until #642 is complete.

The goal of this PR is to prevent the hard crashes when someone removes a team member, or zones while on a team.
This isn't pretty, but it's supposed to be temporary.

It is too, pretty!


WIP: Added model, cft and example apigateway resource

Issue #86
Opening this work in progress PR for visibility as I'm on vacation for 10 days.
This PR adds the ModelDefinition (and associated examples) for creating an API Gateway Reverse Proxy for a Kubernetes service
The APIGateway resources manages the VPCLink and RestAPI for a single stage which proxies all traffic to an HTTP service exposed via a Kubernetes service with type "LoadBalancer" and with the appropriate annotations for creating an NLB.  (k8s 1.9+)
The APIGateway resource requires the NLB DNS name and ARN - however it would be ideal to just provide the DNSName since it's already available through the k8s service resource.
I'm gonna have to keep this PR open for a minute (which might give us a chance to figure out if this is a sane path - at least as a first pass) since I'm heading to Japan tomorrow -> Oct 13th 2018.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Also, I should have read your description ;) enjoy your vacation!

Also, sorry I should have read your description ;) Enjoy your vacation!


src/commands: support schema plan

atlas migrate lint on migrations

Status
Step
Result

No migration files detected

ERD and visual diff generated
View Visualization
No issues found
View Report
Read the full linting report on Atlas Cloud

atlas migrate lint on sqlite-destructive

Status
Step
Result

1 new migration file detected
20230925192914.sql
ERD and visual diff generated
View Visualization
Analyze<PHONE_NUMBER>2914.sql1 reports were found in analysis
Destructive changes detectedDropping table "t1" (DS102)
Read the full linting report on Atlas Cloud

Atlas detected changes to the desired schema
SQL Plan-- Create "t1" table
CREATE TABLE "public"."t1" (
  "c1" integer NOT NULL
);
Atlas lint results

Status
Step
Result

Detect schema changes1 new statement detected
No issues found

📝 To edit this plan, follow these steps

Run atlas schema plan pull --url "atlas://circleci-schema-test/plans/pr-22-E+w50mxU" > pr-22-E+w50mxU.plan.hcl
Modify the plan.
Push the plan using atlas schema plan push --pending --env  --file pr-22-E+w50mxU.plan.hcl


Login on official instance (EU) does not work.

After reading some other issues and trying some things, I am out of ideas. As I signed up on vault.bitwarden.eu I used the following link to get the urls:
https://vault.bitwarden.eu/api/config
But setting that up did now work, it returned:
$ flatpak run --command=/app/bin/goldwarden com.quexten.Goldwarden vault login --email<EMAIL_ADDRESS>Login failed: Could not login: could not pre-login: Not Found:

The normal login (via GUI) opens a popup to enter the password but also does not work, on the first try it gave an error with HTML code, but I can't reproduce this. Login via client id and secret did not work as well.
I'd appreciate any help!

Can you post the output of flatpak run --command=/app/bin/goldwarden com.quexten.Goldwarden config get-environment?

Of course, hold on.
$ flatpak run --command=/app/bin/goldwarden com.quexten.Goldwarden config get-environment
{"api":"https://vault.bitwarden.com/api","identity":"https://vault.bitwarden.com/identity","notifications":"https://notifications.bitwarden.com","vault":"https://vault.bitwarden.com"}

This was after a clean install, then I got the error message
Could not login via password: Bad Request:
{"message":"Traffic from your network looks unusual.
Connect to a different network or try again later. [Error Code 7]"}

so I tried the id/secret thing again.
$ flatpak run --command=/app/bin/goldwarden com.quexten.Goldwarden config set-api-url https://api.bitwarden.eu/
Done
$ flatpak run --command=/app/bin/goldwarden com.quexten.Goldwarden config set-identity-url https://identity.bitwarden.eu/
Done
8:~$ flatpak run --command=/app/bin/goldwarden com.quexten.Goldwarden config set-notifications-url https://notifications.bitwarden.eu/
Done

Then I got the Could not pre-login: Not Found: again.

With vault.bitwarden.eu this worked in the latest main branch version for me. I think the bug is most likely just the client id / client secret, which should be fixed in #212

Oh... my bad. Fix incoming soon, actually it was never updating the url properly when setting from the GUI.

0.3.2 is out now, should be on FlatHub within a few hours. Please try again there.

Thank you! Will try asap.

Works perfect now, thanks!


Replace etcd deployment to bitnami etcd chart

Background
The greptimedb-etcd chart is not production-ready and is only for testing. It's very limited when we use it in multiple deployment scenarios.
Enhancement
For more convenience, we should use bitnami-etcd, which is more functional and recommended by etcd community.

Updated in README.


Icon pack resolution breaks when dependencies not aligned

Summary
Icon pack resolving svg path doesn't work well with basedir: this.project.root when dependecies aren't aligned
Expected Behavior

Should've resolved to addon's dependency

Actual Behavior

Always resolves to projects top level dependency version

Possible Solution

Align dependency version

Steps to Reproduce

Have<EMAIL_ADDRESS>Nested have<EMAIL_ADDRESS>
will see ENOENT: no such file or directory, lstat '/workspace/mcp-ui/node_modules/ember-frost-core/frost-icon-svgs'
Since 7.0.0 stores svgs in svg folder, and 8.2.5 stores in frost-icon-svgs

This is not an issue that needs to be addressed via changes to this codebase.  What needs to happen is for packages depending on older versions of this addon, which might be used alongside newer versions of the same, to be updated so that all code and dependencies are in sync.

While @notmessenger has a point in that the root of the problem is not being fixed, I still think this issue being address in this code base is still valid. It ensures that the icons are being fetched from the right repo vs whatever winner comes out of the npm alignment issue.
At least with this work, we can limp along until we can address the huge fallout of of multiple versions in our repo.

This change requires the following scenarios to be tested:

Test that each addon that uses this addon to make its svg packs continues to function as expected.

For those that are using the same code to do so it should be a "rubber-stamp" review but there is no guarantee that they are all being done the same.

An addon that consumes another addon that makes its own svg pack needs to be tested
The above combinations then also need to be tested when consumed in an application.
The above combinations then also need to be tested in an addon like frost-foundation that is consumed by an application.
For good measure test that all of this works against existing apps as well, such as MCP-UI.

This code is very intertwined with how Ember CLI works and the complicated relationship we have between all of our addons and these are all of the testing steps/scenarios we had to employ when making the last round of refactors.  Without this testing there is no guarantee that something does not become broken.
When we mentioned this to @EWhite613 he said that he did not have the time to do all of this testing.
This comment (thread) serves as my due diligence in laying out the concerns with this approach so if @cstolli says to merge https://github.com/ciena-frost/ember-frost-core/pull/579 then we should proceed.

I know that the level of testing that @notmessenger outlined was conducted and verified that functionality was working correctly prior to this change: https://github.com/ciena-frost/ember-frost-core/pull/579

Great points guys. I think it's fair that we do at least that much testing to ensure this doesn't break anything else. I would prefer that @cstolli not merge this without that testing because I'm not familiar with the code to be confident enough to give my approval. We're not dead in the water because there is a work around.


fs: add fs_heap_strndup, replace strndup with fs_heap_strndup

Summary
fs: add fs_heap_strndup, replace strndup with fs_heap_strndup
fix malloc/free inconsistency, stdndup and fs_heap_free mismatch
Impact
Testing
sim

[Experimental Bot, please feedback here]
No, this PR does not fully meet the NuttX requirements.
Here's why and how to improve it:
Missing Information

Summary:

Why is this change necessary? Clearly state the problem caused by the malloc/free inconsistency. Explain why strndup and fs_heap_free mismatch.
How does fs_heap_strndup solve this?  Provide a concise explanation of the new function's behavior.

Impact:

You need to analyze and address all impact points. Even if an area is not affected, state "NO" explicitly. Don't leave sections blank.
Examples of potential impact (you need to confirm these):

Impact on user: Will any applications using strndup need to be modified?
Impact on build:  Were any new dependencies introduced by fs_heap_strndup?
Impact on hardware: Likely NO, but confirm this.
Impact on documentation: Does the addition of fs_heap_strndup require documentation updates?

Testing:

Insufficient Detail: "sim" is not enough information.
Provide:

Specific simulator used:  e.g.,  qemu-system-arm, qemu-system-riscv64, etc.
NuttX configuration: Which board configuration within the simulator (sim:nsh, sim:last, etc.)?

Testing logs:  You must include relevant log snippets showing:

The issue before the change.
The issue resolved after the change.

How to Improve the PR Description

Expand the Summary:  Provide a clear and concise explanation of the problem and the solution.
Complete the Impact Assessment: Address all impact categories, even if they are not affected (state "NO").
Provide Detailed Testing Information: Specify the simulator, configuration, and include relevant before/after log snippets.

By providing this missing information, you'll make it much easier for reviewers to understand and evaluate your PR.


fix: use hasProtocol rather than checking if url starts with http

resolves https://github.com/unjs/ipx/issues/79
Thanks @eggsy.

wow 🤯 － this was fast!

Issue here is that localhost:3000 is thought to be a protocol -> https://github.com/unjs/ufo/issues/70


One-liner description uses deprecated "should" syntax

Subject of the issue
When you use format --documentation against a spec expressed in the one-liner syntax, the example description is formed using "should".
This is contrary to modern usage where a spec's description is encouraged to be a statement about what the subject is not what it should be.
Your environment

Ruby version: ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-darwin17]
rspec-core version: rspec-core 3.8.0

Steps to reproduce
Use the following script as described in the bug reporting template
# frozen_string_literal: true

begin
  require "bundler/inline"
rescue LoadError => e
  $stderr.puts "Bundler version 1.10 or later is required. Please update your Bundler"
  raise e
end

gemfile(true) do
  source "https://rubygems.org"

  gem "rspec", "3.8.0" # Activate the gem and version you are reporting the issue against.
end

puts "Ruby version is: #{RUBY_VERSION}"
require 'rspec/autorun'

RSpec.describe 'one-liner description uses deprecated "should" syntax' do
  subject { 1 + 1 }

  it { is_expected.to eq(2) } # => "  should eq 2"
end

Expected behavior
I expect the following output:
one-liner description uses deprecated "should" syntax
  is expected to eq 2

or even better
one-liner description uses deprecated "should" syntax
  is 2

Actual behavior
one-liner description uses deprecated "should" syntax
  should eq 2

Thanks. I did search the outstanding issues before I raised this one. Clearly I missed that one!
Not sure I understand your CI workflow; is it bad that you got Travis failures when the fix was merged to master?
Anyway, hope this can get released soon. Will there be another 3.x release before 4.0?

Not sure I understand your CI workflow; is it bad that you got Travis failures when the fix was merged to master?

Yes if they are real failures, they weren't they got restarted and passed. You'll see an all green history.

Anyway, hope this can get released soon. Will there be another 3.x release before 4.0?

Most certainly, RSpec 4 is a long way off.


Add @with_terminal

I added a macro @with_terminal to simplify usage of with_terminal

Hi @fonsp any changes? Or should I choose of you don't want to merge?

Closing due to lack of interest


HDFS-16439.Makes calculating maxNodesPerRack simpler.

Description of PR
It should be simpler to compute numOfReplicas when BlockPlacementPolicyDefault#getMaxNodesPerRack() is executed. This is the purpose of this pr.
Details: HDFS-16439
How was this patch tested?
For testing, there is not much pressure.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 40s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
32m 22s

trunk passed

+1 :green_heart:
compile
1m 27s

trunk passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
1m 18s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m  1s

trunk passed

+1 :green_heart:
mvnsite
1m 29s

trunk passed

+1 :green_heart:
javadoc
1m  2s

trunk passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 33s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m  9s

trunk passed

+1 :green_heart:
shadedclient
22m 19s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 15s

the patch passed

+1 :green_heart:
compile
1m 18s

the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
1m 18s

the patch passed

+1 :green_heart:
compile
1m 13s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
1m 13s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 52s

the patch passed

+1 :green_heart:
mvnsite
1m 19s

the patch passed

+1 :green_heart:
javadoc
0m 49s

the patch passed with JDK Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 22s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 17s

the patch passed

+1 :green_heart:
shadedclient
22m 19s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
230m 33s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 47s

The patch does not generate ASF License warnings.

329m 17s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3937/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3937

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux b88178c3d165 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / a1e510065ec31953f455469af3dbd66dfdf4355b

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.13+8-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3937/1/testReport/

Max. process+thread count
3155 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3937/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Would you like to help review this PR, @ayushtkn @tomscut .
Thank you very much.

Thanks @ayushtkn  for the comment and review.
This happens when more replicas are required, say the number of all DataNodes is 5 and the number of replicas required for the file is 7. At this time, the code execution here is usually triggered, because the number of copies of the file cannot exceed the number of nodes in the cluster.
Here, there will be 2 benefits:

When resetting the new number of file copies, it will be more concise and easier to understand, (clusterSize - numOfChosen). This is the main purpose.
It can reduce some calculation steps, (clusterSize - numOfChosen) only needs to perform 1 step.


Update flyway-core to 7.7.1

Updates org.flywaydb:flyway-core from 6.5.7 to 7.7.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.flywaydb", artifactId = "flyway-core" } ]

labels: library-update, semver-major

Superseded by #1248.


add libcurl dependency

This fixes GH action for model ensemble failing with the following error:
Configuration failed because libcurl was not found. Try installing:
 * deb: libcurl4-openssl-dev (Debian, Ubuntu, etc)
 * rpm: libcurl-devel (Fedora, CentOS, RHEL)

LGTM. We should add cacheing to this workflow but that can be a separate PR.


DigestAuth functions aren't thread-safe (Race conditions)

go build -race, will add instrumentation to catch race conditions.
It caught problem within DigestAuth.clients (map)
I'm attaching a copy of digest.go.
I changed the mutex to be a read/write mutex, so you could have finer grain control over it.
this copy doesn't cause problems with -race
Also I started to fix golint problems  https://github.com/golang/lint. But stopped, because it was getting out of hand. (Sorry for dirtying up the answer)
digest.txt
Thanks for your awesome work on the repo!!
digest.txt is really digest.go!

#53

Not only it's a theoretical problem catched by Go race detector, it actually occasionally leads to crashes.


mrpt_navigation: 2.2.1-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository mrpt_navigation to 2.2.1-1:

upstream repository: https://github.com/mrpt-ros-pkg/mrpt_navigation.git
release repository: https://github.com/ros2-gbp/mrpt_navigation-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 2.2.0-1

mrpt_map_server

No changes

mrpt_msgs_bridge

No changes

mrpt_nav_interfaces

No changes

mrpt_navigation

No changes

mrpt_pf_localization

No changes

mrpt_pointcloud_pipeline

No changes

mrpt_rawlog

No changes

mrpt_reactivenav2d
* Update for MRPT 2.14.3 (new RNAV weight for target heading)
* Support speed_ratio property for Waypoints
* Improve astar navigation demo
* Contributors: Jose Luis Blanco-Claraco

mrpt_tps_astar_planner
* Update demo for astar
* Publish costmaps to ROS for visualization too apart of the custom MRPT GUI
* Improve astar navigation demo
* astar planner: add refine() step and parameter to optionally disable it
* astar node: add two more launch args: problem_world_bbox_margin and problem_world_bbox_ignore_obstacles
* astar params: add more comments and tune for speed
* PTGs .ini: Add docs on how to enable backward motions
* Contributors: Jose Luis Blanco-Claraco

mrpt_tutorials
* Update demo for astar
* Publish costmaps to ROS for visualization too apart of the custom MRPT GUI
* rviz layout: show rnav selected path
* Improve astar navigation demo
* astar planner: add refine() step and parameter to optionally disable it
* astar node: add two more launch args: problem_world_bbox_margin and problem_world_bbox_ignore_obstacles
* Contributors: Jose Luis Blanco-Claraco

Hey @jlblancoc, this seems to have created a regression in the buildfarm.
Do you want to make another release fixing it or should I revert this? I'm hoping to announce a sync in the next day.

Here's the PR (I hit back after clicking revert so the revert PR wasn't created cleanly).
https://github.com/ros/rosdistro/pull/43179


Documentation

It would be good to have some documentation, or at least a basic 'getting started' guide. Referring to the source, RbNaCl and libsodium docs is ok, but not super user friendly.

@alias1 for getting started, please read the libsodium documentation. Also, contributions to the docs are welcome.


Better way to manage parameter_list, maybe without NamedEmbedding

First I want to thank you for making this pykg2vec project! Its comprehensive inclusion of many SOTA algorithms along the development history of knowledge graph embedding is really impressive!
I am trying to revise an existing model by adding a new linear layer, and of course I just added the new layer into the parameter_list. All training went through pretty well, but an exception occurred in export_embedding(), where every parameter in the list need to be have a name, or is assumed to be a NamedEmbedding.
https://github.com/Sujit-O/pykg2vec/blob/492807b627574f95b0db9e7cb9f090c3c45a030a/pykg2vec/utils/trainer.py#L462-L471
I read the code and realized that pykg2vec is using NamedEmbedding for every parameter that is even not embedding at all. For example these parameters in SME model have to be NamedEmbeddings, though they are not embeddings at all.
https://github.com/Sujit-O/pykg2vec/blob/492807b627574f95b0db9e7cb9f090c3c45a030a/pykg2vec/models/pairwise.py#L573-L578
And because of this unnecessary detour to NamedEmbedding, they are later used in a little obscured way like this
https://github.com/Sujit-O/pykg2vec/blob/492807b627574f95b0db9e7cb9f090c3c45a030a/pykg2vec/models/pairwise.py#L627-L629
I am sure you must have a reason to devise such a mechanism in pykg2vec, would you please elaborate, or just use pytorch's way to handle parameters with register_parameter() ? I assume it may simplify the code base, and make it easier to understand / extend?
@baxtree @louisccc

Hi, @dany-nonstop, Thanks for the suggestion and I think you are right in say parameter_list is not the torch way and can be superseded by register_parameter(). The torch version of pykg2ves was "translated" straightly based off its tensorflow version and at then, there was probably no concept of named embedding in torch hence the NamedEmbedding. Do you mind submitting a PR, assuming you may have done some exploratory change and the tests are still passing?

Hi, @baxtree thanks for shedding light on the origin of the code. while going through the code I noticed that parameter_list is only saved in export_embedding but was never reused elsewhere. Even in during inference in pykg2vec-infer the Trainer is directly loading the saved state_dict instead of using the saved embeddings in tsv files.
I'm not sure how useful those tsv files are, and am wondering if the code related to it should be completely removed instead. I'd love to contribute to pykg2vec, but need to make sure I know what I'm doing before making any changes. 😆

Hi, @louisccc . Can you pls share some history about the saved embeddings as mentioned above and guide @dany-nonstop through the contributing process?


Reaction icons are now broken and misplaced

Today one of Mastodon main branch updates broke the reaction icons under a post. Now they're all misplaced and gathered at the right corner. This doesn't happen with an open post but all previous toots and replies are affected.
Global timeline:

Home timeline:

Single column:

The footer menu in the mobile theme is also broken.


auto dial not working

Hello everyone
The auto-translation of the program does not work
When I log in to the game, I translate by pressing alt + u. There is no automatic translation. How can we fix this problem?

Hi, I have the same problem. Have you found a solution?


Question: how to disable default mapping

Hey there,
and thank you for the awesome plug which makes enjoyable editing lisp code in neovim.
Found some issue regrading overwriting of some existing user keymapping with conjure plugin's ones. Skimming documentation didn't help me (maybe overlooked this), but I want to give a full control of making my own mapping, however changing every conjure's keymap is tedious and fragile (I'm happy with some of them such as, e.., but not all of them).
So my workaround is something like vim.g["conjure#mapping#prefix"] = "" but not sure it is the right way doing a such thing. And maybe it's worth to mention somewhere in docs how to turn off default mapping completely.
Thank you.

Hey, that's done by setting them to v:false which is the VimL way of
writing false. If you're writing Lua you can set them to false. I can't use
nil because there's no way to distinguish nil from no user configuration,
so nil looks like you haven't set anything so I should default it. False on
the other hand is detectable and should turn the mapping off. This should
be in :h conjure, if not that's an issue.
On Wed, 21 Dec 2022 at 10:47, Illia Danko @.***> wrote:

Hey there,
and thank you for the awesome plug which makes enjoyable editing lisp code
in neovim.
Found some issue regrading overwriting of some existing user keymapping
with conjure plugin's ones. Skimming documentation didn't help me (maybe
overlooked this), but I want to give a full control of making my own
mapping, however changing every conjure's keymap is tedious and fragile
(I'm happy with some of them such as, e.., but not all of them).
So my workaround is something like vim.g["conjure#mapping#prefix"] = ""
but not sure it is the right way doing a such thing. And maybe it's worth
to mention somewhere in docs how to turn off default mapping completely.
Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/Olical/conjure/issues/448, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACM6XLU5RRK4RJ2CNY2WN3WOLN2JANCNFSM6AAAAAATFPMWOY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


is there charset code that support  Arabic text

need to print invoice in Arabic...thank you

windows-1256 worked with me

please provide gist of the code ....

please provide gist of the code ....here is mine
 String voucherNameWithValueMessage = 'ليبيانا 5 دينار'; blueThermalPrinter.printCustom( voucherNameWithValueMessage, _onlyBoldText, _alignCenterText, charset: 'windows-1256');

me too it's not working for me
did you solve it ? if so
I really need help

windows-1256 worked with me

how did you did it ?!

still nothing....

still nothing....

actually I find a way but it's really twisted  🤣

I used canvas textpainter and then convert it to an image
then I used the function printImageBytes
It works but there was a problem the function printimage has a fixed height for the image [200px - 250px]. I will try to make it more than one image on sunday.

please provide gist of the code ....here is mine
 String voucherNameWithValueMessage = 'ليبيانا 5 دينار'; blueThermalPrinter.printCustom( voucherNameWithValueMessage, _onlyBoldText, _alignCenterText, charset: 'windows-1256');

me too it's not working for me
did you solve it ? if so
I really need help

still nothing....

actually I find a way but it's really twisted 🤣

I used canvas textpainter and then convert it to an image
then I used the function printImageBytes
It works but there was a problem the function printimage has a fixed height for the image [200px - 250px]. I will try to make it more than one image on sunday.

not an efficient solution ....i thought  about make it an pdf then printing it as an image before.....still not really a solution 😂...so still waiting on @kreemov to show me if it really works.

What's the result of using utf-8 as the charset?

What's the result of using utf-8 as the charset?

this is the output

.

Can I ask the make/model of the printer?

that is the problem we don’t have constraint on what printer the merchant
uses, we only provide the connection and printing functionality....if we
did we could have used the printer's sdk...😁
On Fri, Dec 3, 2021 at 9:37 PM Mike Diarmid @.***>
wrote:

Can I ask the make/model of the printer?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kakzaki/blue_thermal_printer/issues/82#issuecomment-985778251,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIIKSN6BKN3RMY5SLVIYBLLUPEL75ANCNFSM45PBJ7DA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I see 😅 the only other thing I can suggest is try print through a quick Node.js script with something like https://www.npmjs.com/package/node-thermal-printer that's known to support Arabic - if it works with that then it confirms it's an issue with this plugin and not the printer

Thanks ...and also thanks a lot for the support ✌ ... i appreciate the time .......can’t with for the next space ...it’s seems my issue with the "multi package"  was not fully understood because of its complexity and with out having visual representation 😅😭

Solution inspired from @fatimaown93.
You can convert the widget into an image (using Screenshot) and print it:

Here is the Widget:
Screenshot(
    controller: screenshotController,
    child: Container(
      width: 190,
      decoration: BoxDecoration(
          border: Border.all(color: Colors.black),
        color: Colors.white,
      ),
      // color: Colors.white,
      child: Column(
        children: [
          Row(
            children: [
              Text(
                'أهلاً وسهلاً',
                style: TextStyle(
                  color: Colors.black,
                ),
                textAlign: TextAlign.right,
              ),
              Spacer(),
              Text(
                'شكو ماكو؟',
                style: TextStyle(
                  color: Colors.black,
                ),
              ),
            ],
          ),

        ],
      ),
    ),
  ),

The widget is converted into image of Uint8List:
var imageTextRawUint8List = await screenshotController.capture() ?? Uint8List(0);

Then, print the image bytes:
 bluetooth.printImageBytes(imageTextRawUint8List);

Thanks so much @bluemix  ,but I have already thought about it and implemented it if you have seen the conversion between me and @fatimaown93, but I want is an efficient solution , I  wish if there was a way to convert codepage to utf-8 command as binary message to the thermal printer.  Im still trying to test some things , I will keep you posted if I came with a better solution (I don't want to depend on a package not made for this problem and can change over time ) .

How do you get the screenshit without building it ? I've been trying but it's not working.

windows-1256

how? can u share the code please
or correct if i am wrong
Uint8List encoded =
await CharsetConverter.encode('windows-1256', message.toString());
printer.textEncoded(encoded);

you have to go the printer settings menu (Android settings) and set the encoding to utf-i or windows-1256

Changing the printer settings (can be found in the android settings) worked for me, I set it to UTF-8 and now it's printing Arabic characters!
Thanks @bluemix 👏

Can you share the path of file(to) to edit?

Can you share the path of file(to) to edit?

Changing the printer settings (can be found in the android settings) worked for me, I set it to UTF-8 and now it's printing Arabic characters! Thanks @bluemix 👏

Can you share the path of file(to) to edit?

@blitzlepe It's in the system settings, there must be a section called Printer where you can change the encoding to UTF-8


⚠ Bio Links has degraded performance

In 4f0b1f2, Bio Links (https://bio.rheinklang.events/links) experienced degraded performance:

HTTP code: 200
Response time: 1914 ms

Resolved: Bio Links performance has improved in 891285d after 25 minutes.


Updated Powershell script

Hi, the script in the original page was failing when you would delete a trigger. It would attempt deleting the trigger after it has been enabled. Therefore, I moved the enabling of triggers at the end after the cleanup steps.
Also, I added logging for objects that are being deleted.
Lastly, I added logging for the enabling/disabling triggers.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: patpicos sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@patpicos : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

Hi Doug,
I guess I should remove the default values for the parameters and make them mandatory to ensure the user passes in the values to drive expected behavior.
I removed the $rootFolder variable since I expect the user to pass in the full path of the armTemplate file.
See how I have it configured in DevOps. Let me know how you'd like me to proceed

@patpicos I prefer to limit the scope of changes, although I understand your reasoning. Let's put back $rootFolder and I'll sign off on this pull request.

@patpicos : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

2nd commit made to add the first parameter ($rootFolder)

#sign-off

when do the changes get reflected on the web site?

@patpicos I assume you now see your changes on the page. After I sign off on your PR, it's not merged immediately, and then we only publish twice a day. But typically that should mean within 24 hours.

I see it now!


Steps dont match the portal.

I started lab 1 with creating a KQL database and ingesting the data. But the fabric portal is changed, so the steps in the lab do not match anymore

pr #14 merged.
Thx!


Write blog-post about Pull request vs merge request.

I wrote this quick explanation of my thoughts on slack. The sentiment was that it was worth expanding to a blogpost:
Depends on what the PR is for. I am beginning to distinguish lingually between “Pull Request” and “Merge Request”. A PR is a request for someone to look at your code (online or “pulling" the branch if needed). A PR can be on very unfinished code and is a good tool for collaboration. This might still make perfect sense even in a Pip world. On the other hand, a Merge Request is the more classical sense indicating “I have this code, I really believe it is done done, and maybe we have already discussed it at length." In a fork-model this would indicate "Would you please accept it in your repo”. In a normal central repo model, this would more be a role-based thing or motivated by a security policy? .

You, Bo, Thierry and I worked out some thoughts earlier around those lines.... maybe you can link to that and close that old issue?


JedisException Could not return the resource to the pool when using Jedis cluster

Expected behavior
Jedis should not throw such exception in any scenario.
Actual behavior
I'm connecting to Azure redis, which is cluster mode with 1 shards and two nodes are master and slave. When reboot master node, the exception will be thrown sometimes.
Exception in thread "main" redis.clients.jedis.exceptions.JedisException: Could not return the resource to the pool
        at redis.clients.util.Pool.returnBrokenResourceObject(Pool.java:94)
        at redis.clients.jedis.JedisPool.returnBrokenResource(JedisPool.java:235)
        at redis.clients.jedis.Jedis.close(Jedis.java:3430)
        at redis.clients.jedis.JedisClusterCommand.releaseConnection(JedisClusterCommand.java:172)
        at redis.clients.jedis.JedisClusterCommand.runWithRetries(JedisClusterCommand.java:166)
        at redis.clients.jedis.JedisClusterCommand.run(JedisClusterCommand.java:33)
        at redis.clients.jedis.JedisCluster.get(JedisCluster.java:169)
        at com.microsoft.azure.redis.jedis.benchmark.ReconnectBenchmark.simulateWorkload(ReconnectBenchmark.java:64)
        at com.microsoft.azure.redis.jedis.benchmark.ReconnectBenchmark.simulateClusterWorkload(ReconnectBenchmark.java:78)
        at com.microsoft.azure.redis.jedis.benchmark.ReconnectBenchmark.runTest(ReconnectBenchmark.java:50)
        at com.microsoft.azure.redis.jedis.benchmark.ReconnectBenchmark.main(ReconnectBenchmark.java:38)
Caused by: java.lang.IllegalStateException: Invalidated object not currently part of this pool
        at org.apache.commons.pool2.impl.GenericObjectPool.invalidateObject(GenericObjectPool.java:640)
        at redis.clients.util.Pool.returnBrokenResourceObject(Pool.java:92)

Steps to reproduce:
This exception can't be stably reproduced:

Create a redis cluster with 1 shard and 2 nodes are master and slave.

I didn't explicitly return any object. The code is something like this

        try(Jedis jedis = jedisPool.getResource()){
            // perform some redis command
        } catch (JedisConnectionException e){
            // log
        }

I believe jedis is implicitly returned by try clause. This looks strange.

@warrenzhu25 I said about closing, not returning.
However, try-with-resources is implicitly calling jedis.close(). Is it, somehow, being called again?

I'm petty sure that I haven't called it ever. Maybe this is common pool issue, I'll look into it.

I have tried like bellow,
try {
jedis = jedisPool.getResource();
RateRow[] cbaserates = gson.fromJson(jedis.get("WLBaseRates"), RateRow[].class);
} catch (IllegalStateException ex) {
ex.printStackTrace();
} finally {
jedis.close();
}
But still I have an issue sometime showing Exception in thread "Timer-0" redis.clients.jedis.exceptions.JedisException: Could not return the resource to the pool at redis.clients.util.Pool.returnBrokenResourceObject


pull request for e2e test at 02/03/2017 20:26:07

:white_check_mark: Validation status: passed
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.


Return NaNs when parameter sweep hits bad value

@olgadoronina this should be good to merge, just give it a once over to make sure it's got the right elements from PR commit c86eed6


Update docs

[ ] Regenerate doxygen documentation
[ ] Update https://libpointmatcher.readthedocs.io

Also publish the internal doc from the headers (module name, parameter ranges, etc.)
See ./examples/list_modules

The doxygen documentation is now available here :
https://norlab.ulaval.ca/research/libpointmatcher/#documentation

Both stable and latest documentations were regenerated on readthedocs.io: https://libpointmatcher.readthedocs.io


Update requirements.txt

Thanks for your great work ! That's awesome. I followed your latest YouTube video to create a landing page and it works! But when I use it, I noticed that selenium need be add to the requirements.txt

Thanks! I don't want to add Selenium to the global requirements, as not all use cases need it. We now have requirements for each agent separately in their source files.


Note the Magento Cart2Quote extension RCE vulnerability

Either accept it or not, I don't mind (: It just came up; looking for a place to log Magento extension vulns.

Can I just say, I hadn't anticipated this would get attention let alone the super helpful comments that you've put on it. 10/10 would report again; I will amend those notes and fix up the PR. Will squash the commit so it's tidier.

@andrewhowdencom FYI, you can run php validator.php locally to ensure that the structure of your advisory is correct (instead of doing back-and-forth waiting for the CI to run each time you try to fix it). Some of the issues we reported are also detected by the validator (not all of them, as it cannot ensure that the content of the advisory matches the needed one of course)

PR Updated to the best of my current knowledge.

Is there anything else I can do to make this PR better? I'm looking to establish precedent for submitting Magento extensions; I'll create new PRs as I come across other extension (or core) vulns -- I also want to encourage others in the community to do so also.

Let's merge! Thanks for your contribution.

Thanks man! I appreciate it. There's such a need for a project such as this, and this one is well structured and already provides the tooling!


Handle network errors during page refresh

Steps to recreate:

enter main VM Portal page with VMs/pools
disable networking (i.e. using dev tools)
trigger manual refresh
Expected result: error notifications are displayed
Actual result: VM Portal crashes

Fixes:

error.status == 0  - if network is down the reported status code is
zero which bypasses some of our checks. Fixed by returning
the entire error object.
temporary network error causes VMs/pools to be removed from
the dashboard. Fixed by removing only if the server reported 404.
use common filtering routine to process VMs and pools retrieved
during page refresh

Unfortunately, I'm still experiencing this issue. The pool is still visible in the VM portal, but it does not happen always. Once a new pool is created in the Admin portal, it "replaces" the already removed pool in the VM portal.

@isaranova

Unfortunately, I'm still experiencing this issue. The pool is still visible in the VM portal, but it does not happen always.

Problem can be re-created if there is a single pool in the system and that pool will be deleted.

The problem will be tracked under #1573

@isaranova @rszwajko can we close this original issue as done since Radek opened a new one?

The problem will be tracked under #1573

@isaranova @rszwajko can we close this original issue as done since Radek opened a new one?

Yes


Add dipole B1 3.2 mrad source point markers

@xresende could you please update version in this PR?

@xresende could you please update version in this PR?

done, @anacso17


Barrier: Some actor does not collect barrier so that epoch can not commit

Describe the bug
A clear and concise description of what the bug is.
Run ./scripts/launch_risedev_bench.sh  in https://github.com/singularity-data/tpch-bench and I found that compute-node all stop processing data in a few minutes. And the metrics barrier-send-latency does not show anything.
I add more log in compute-node and meta-node and I found that some actor does not run LocalBarrierManager::collect so that the rpc in StreamService::barrier_complete does not return result to meta-service and meta-service could not commit this epoch. It seems that some actor is blocked so that it can not do anything.
To Reproduce
deploy a new risingwave cluster in AWS EC2 as https://singularity-data.quip.com/EbTfA4xUdJ2z/Benchmark-RisingWave-Three-Easy-Steps (We need at least three compute-node) and run ./scripts/launch_risedev_bench.sh. Then you can observe you cluster are blocked.
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

Which revision does this bug occur on?

Which revision does this bug occur on?

The new Main

Which revision does this bug occur on?

I found it in new Main branch yesterday

Which revision does this bug occur on?

I found it in new Main branch yesterday

Do you have an exact commit hash?

Which revision does this bug occur on?

I found it in new Main branch yesterday

Do you have an exact commit hash?

8ad51838db0f4bc013ad0f05a92421ba8f0200df

I'm also encountering deadlock of "epoch can not commit" when running nexmark q5 with very large parallelism (not related to #4354). May be related to this.

@BugenZhao  I find that this bug may be caused by https://github.com/singularity-data/risingwave/pull/4045 with binary-search.

@BugenZhao  I find that this bug may be caused by https://github.com/singularity-data/risingwave/pull/4045 with binary-search.

Oops. Any further investigations?

@BugenZhao  I find that this bug may be caused by https://github.com/singularity-data/risingwave/pull/4045 with binary-search.

I've reviewed the implementation of remote input and found that creating client with moka might be suspicious, considering the bug we found in our LruCache last time. I've not checked how moka dedups inflight requests yet.
https://github.com/singularity-data/risingwave/blob/f5ab9b4e561ba2a2e8aaff062db9dc0fde44ecb4/src/stream/src/executor/exchange/input.rs#L125
A quick PoC might be creating the client outside of the stream and check whether there're still problems.


Changes done to acknowledgment statement

The previous line was not eye catching. I have made changes to the acknowledgment statement to make it grammatically correct. Kindly check and merge the change

Coverage remained the same at 84.728% when pulling 85784800dbea2a31f434ae2e4e0523032e63fa47 on mouli-16:master into 9ad168b9cc0ad0be1873decf505f5765413ad94f on CircuitVerse:master.

Coverage remained the same at 84.728% when pulling 85784800dbea2a31f434ae2e4e0523032e63fa47 on mouli-16:master into 9ad168b9cc0ad0be1873decf505f5765413ad94f on CircuitVerse:master.


Mouse right-click create file/folder does not claim window focus

Issue Type: Bug
Set window focus to any other window (Safari?), right click on a folder in VS Code and choose to create a new folder.
Start typing a name for the folder - your keystrokes are still sent to the other window.
Expected behavior:
As soon as VS Code creates a text-entry for the new folder, it should grab window focus so the keystrokes will be captured.
VS Code version: Code 1.45.0 (d69a79b73808559a91206d73d7717ff5f798f23c, 2020-05-07T15:57:33.467Z)
OS version: Darwin x64 19.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
32.00GB (0.36GB free)

Process Argv

Screen Reader
no

VM
10%

Extensions (3)

Extension
Author (truncated)
Version

dart-code
Dar
3.10.1

flutter
Dar
3.10.1

vscode-icons
vsc
10.1.1

https://github.com/microsoft/vscode/issues/97408


ScatterPlotList: Fix GetLegendItems()

Purpose:
The legend item of ScatterPlotList cannot display because of the GetLegendItems() function doesn't set properties correctly.
New Functionality:
None

Good catch @Maoyao233, thanks!


NDARRAY doesn't support floats

https://github.com/georgia-tech-db/eva/blob/85977ecd2d3c483273a8069124fd5bc11f243f0c/src/catalog/schema_utils.py#L62

Fixed!


Bugged Center point marker left over since update. Causing area to crash.

I can't delete this center point marker! Why has it stayed there? Why is it crashing my game and how can I delete it without reversing hours of work? Problem started after recent plan build update.

Hey, can you check if that fixes it?
PlanBuild.zip

Hey, can you check if that fixes it? PlanBuild.zip

That has resolved the issue! Thank you for that.

Thanks for testing, going to release a fixed version soon


additional sudo cmds needed for non-root user

Signed-off-by: Mike Lammon<EMAIL_ADDRESS>

Is this still needed since #199 landed?
I'm guessing not as IRONIC_DATA_DIR="$WORKING_DIR/ironic" and the user should own WORKING_DIR?

Build SUCCESS, see build http://<IP_ADDRESS>:8080/job/dev-tools/368/

Thanks, good catch


ValueError: operands could not be broadcast together with shapes (32,) (6,) (32,)

Hi, except NNCLR, other models faced this error ValueError: operands could not be broadcast together with shapes (32,) (6,) (32,)
Here is the log:

/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/training.py in _method_wrapper(self, *args, **kwargs)
    106   def _method_wrapper(self, *args, **kwargs):
    107     if not self._in_multi_worker_mode():  # pylint: disable=protected-access
--> 108       return method(self, *args, **kwargs)
    109
    110     # Running inside `run_distribute_coordinator` already.

/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/training.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, validation_batch_size, validation_freq, max_queue_size, workers, use_multiprocessing)
   1101               logs = tmp_logs  # No error, now safe to assign to logs.
   1102               end_step = step + data_handler.step_increment
-> 1103               callbacks.on_train_batch_end(end_step, logs)
   1104         epoch_logs = copy.copy(logs)
   1105

/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/callbacks.py in on_train_batch_end(self, batch, logs)
    438     """
    439     if self._should_call_train_batch_hooks:
--> 440       self._call_batch_hook(ModeKeys.TRAIN, 'end', batch, logs=logs)
    441
    442   def on_test_batch_begin(self, batch, logs=None):

/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/callbacks.py in _call_batch_hook(self, mode, hook, batch, logs)
    287       self._call_batch_begin_hook(mode, batch, logs)
    288     elif hook == 'end':
--> 289       self._call_batch_end_hook(mode, batch, logs)
    290     else:
    291       raise ValueError('Unrecognized hook: {}'.format(hook))

/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/callbacks.py in _call_batch_end_hook(self, mode, batch, logs)
    307       batch_time = time.time() - self._batch_start_time
    308
--> 309     self._call_batch_hook_helper(hook_name, batch, logs)
    310
    311     if self._check_timing:

/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/callbacks.py in _call_batch_hook_helper(self, hook_name, batch, logs)
    340       hook = getattr(callback, hook_name)
    341       if getattr(callback, '_supports_tf_logs', False):
--> 342         hook(batch, logs)
    343       else:
    344         if numpy_logs is None:  # Only convert once.

/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/callbacks.py in on_train_batch_end(self, batch, logs)
    959
    960   def on_train_batch_end(self, batch, logs=None):
--> 961     self._batch_update_progbar(batch, logs)
    962
    963   def on_test_batch_end(self, batch, logs=None):

/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/callbacks.py in _batch_update_progbar(self, batch, logs)
   1015       # Only block async when verbose = 1.
   1016       logs = tf_utils.to_numpy_or_python_type(logs)
-> 1017       self.progbar.update(self.seen, list(logs.items()), finalize=False)
   1018
   1019   def _finalize_progbar(self, logs):

/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/utils/generic_utils.py in update(self, current, values, finalize)
    556           self._values[k] = [v * value_base, value_base]
    557         else:
--> 558           self._values[k][0] += v * value_base
    559           self._values[k][1] += value_base
    560       else:

ValueError: operands could not be broadcast together with shapes (32,) (6,) (32,)

please help @beresandras

Yeah, all except NNCLR. I didn't change anything but my dataset. The thing is I already made sure my dataset format is similar to the original.

Okay, but there are no hyperparameters with the value of either 32 or 6 in the code (which is in the error message). What is your image shape? What is the model's output size? How many classes do you have? Do you train on one GPU? Is NNCLR giving decent results?
Since the original code runs without an error, one of the changes must be causing it.

What is your image shape? -> 64x64
What is the model's output size? -> dim=128
How many classes do you have? -> 248 classes
Do you train on one GPU? -> I tried both on one GPU and multiple GPUs, and both produced the same error.
Is NNCLR giving decent results? -> not so good, kinda bad TBH

Try to find what the error message refers to, what has the shape of (32,) and (6,).


Remove workaround for ISPN-4969

Linux with security manager Build 1060 is now running using a merge of 95336b27b2e1f133bec42670aac40b6c98231c35

Linux Build 6083 is now running using a merge of 95336b27b2e1f133bec42670aac40b6c98231c35

Windows Build 1196 is now running using a merge of 95336b27b2e1f133bec42670aac40b6c98231c35

Linux with security manager Build 1060 outcome was SUCCESS using a merge of 95336b27b2e1f133bec42670aac40b6c98231c35
Summary: Tests passed: 846, ignored: 319 Build time: 0:06:15

Windows Build 1196 outcome was SUCCESS using a merge of 95336b27b2e1f133bec42670aac40b6c98231c35
Summary: Tests passed: 2927, ignored: 311, muted: 2 Build time: 0:50:32

Linux Build 6083 outcome was SUCCESS using a merge of 95336b27b2e1f133bec42670aac40b6c98231c35
Summary: Tests passed: 2928, ignored: 311, muted: 1 Build time: 0:51:37

This looks good to merge.


License?

Thanks for making this great package!
The repository has no LICENSE file, so I'm not sure if I'm allowed to modify or redistribute your code. By default publicly available but unlicensed work is all rights reserved.
Could you add a license to the repository please? (hopefully something permissive like MIT 😅)

Thank you for your interest in FRAPOSA. The MIT license was just added.


[2.x] Supports Laravel 11 & PHP 8.2+

This will be a major update - breaking changes.

Supports PHP 8.2 & PHP 8.3
Supports Laravel 11
Bumps PHPUnit to 10.x
Adds the in-house doctrine manager 😎 since Laravel dropped Doctrine. This makes the codebase a bit nicer.
Makes DBAL become a required dependency and gets rid of the installDependencies part. EloquentDocs needs DBAL so it doesn't really make sense to put it in require-dev lol

For Laravel 11 users, can install the latest version:
composer require sethsandaru/eloquent-docs --dev

For Laravel 9 & 10 users:
composer require sethsandaru/eloquent-docs ^1 --dev

Notes

EloquentDocs doesn't support the DB URI connection at the moment. You need to specify the host, port, username, password, and database.
SQLite users only need to specify the DB_DATABASE, use the full path.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.96%. Comparing base (dd8ddc6) to head (2046f03).

Additional details and impacted files

@@             Coverage Diff              @@
##               main      #20      +/-   ##
============================================
+ Coverage     93.83%   98.96%   +5.12%
+ Complexity       68       67       -1
============================================
  Files             7        8       +1
  Lines           227      289      +62
============================================
+ Hits            213      286      +73
+ Misses           14        3      -11

Files
Coverage Δ

src/Commands/BulkEloquentDocsGeneratorCommand.php
100.00% <ø> (+10.86%)
:arrow_up:

src/Commands/EloquentDocsGeneratorCommand.php
100.00% <ø> (+12.19%)
:arrow_up:

src/Services/DoctrineManager.php
100.00% <100.00%> (ø)

src/Services/Generators/ColumnsGenerator.php
95.74% <100.00%> (+3.43%)
:arrow_up:

... and 3 files with indirect coverage changes


grizzly-ls: ModuleNotFound gevent

Describe the bug
grizzly-ls tries to noop gevent.monkey.patch_all, but gevent is not an dependency of grizzly-ls (only in grizzly context).
To Reproduce
Install grizzly-loadtester-ls in a project that does not have gevent as an requirement.
Expected behavior
It should work in projects that does not have gevent as an requirement.

fixed in Biometria-se/grizzly-lsp#51.


Don't Merge! PR real time test for webhook case

Docs Build status updates of commit adc200e:
:clock10: Pending: waiting for processors (29 builds ahead of you)
:warning: Docs Build is busy, currently there are 29 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit adc200e:
:clock10: Full build: the average full build time is 7 min(s) 19 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit adc200e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:warning:Warning
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)
Details

docs-ref-conceptual/testing.md

[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.

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.


Right Click Only

I want to use just to detect a rightclick/contextmenu and handle.
I have no menu items and have disabled defaults, but when I right click I still see a white rectangle. Is there a way to stop this?
Thanks

Then I guess you'd should implement you own right click logic and handle it.
It's not that hard.


Replace SBTableView keyboard handling with selector bound to shortcuts

Why have a delegate for all this if it can be handled by an NSMenuItem?

Actually overlaps with #50 too, since menuForEvent seems to be handled by SBTableViewDelegate.


Reorganize fixtures for TPC-H

Previously we had a split system to accommodate the single-machine systems with coiled.function.  Now we make those slightly less convenient in order to clean up the fixture infrastructure.

Merging tomorrow morning my time if no one gets to it first.


Forward-headers and provided headers.

Hi, first of all, this package is AWESOME.
However, we have a problem to make it works with AWS AppSync.
The problem is, that AppSync will deny all connections without Authorization header. So, we created a small script, which will obtain a JWT from AWS and then starts graphql-faker with that token.
Until this, everything is ok. Then when our web client tries to access a graphql-faker's endpoint, it also provides a different JWT as you can be logged in as any user. However, faker will access an AppSync with a JWT provided at start.
In another words:
Is it possible to provide a --header "Authorization: IDTOKEN" alongside with --forward-headers and force graphql-faker to use a provided Authorization header or forward Authorization header when it is provided by client.
We are using this command to start it:
graphql-faker --forward-headers Authorization --header "Authorization: IDTOKEN" --extend https://abc.amazonaws.com/graphql ./ext-dev.graphql

@deftomat Headers are compared in lower-case so following command should work:
graphql-faker --forward-headers authorization --header "authorization: IDTOKEN" --extend https://abc.amazonaws.com/graphql ./ext-dev.graphql

Can you please test it?
If it works I will fix Faker to do this automatically.

@deftomat Change was really small so I just published 1.7.5 now your original command should work.
Can you please test it?

@IvanGoncharov
AWESOME, 1.7.5 is working.
Thanks.

@deftomat Can you share you little script to do the trick? From what I read here it's not in the repo right?

@ebisbe Hi, it's a long time ago and we are not using it anymore. We switched from AppSync to Apollo in lambdas, which already includes the faker/mocks as AppSync was a paint for us 😞


[improvement](routine load) fe metadata add routine load error reason

Proposed changes
Add routine load pause reason and cancel reason to fe metadata to prevent the loss of these messages after restarting the FE node. which will make auto resume routine load job faild, and users may be confused about the error reason of the routine load job.
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...

you should add some regression test case


Check for incomplete fluxObjectivs

ModelPolisher must deal with cases, in which input files lack an objective function. For instance, they might only define an empty list of flux objectives or only an objective without any such list at all:
<fbc:listOfObjectives fbc:activeObjective="obj">
   <fbc:objective fbc:id="obj" fbc:type="maximize">
    <fbc:listOfFluxObjectives/> <!-- This list is sometimes also missing -->
  </fbc:objective>
</fbc:listOfObjectives>

One way to deal with such an input would be trying to insert a biomass function. This requires an additional user option for the coefficient with a default value of 1.0. Since there could be multiple target reactions, this needs to be an array of coefficients.
However, this has some problems attached:

This is based on the assumption that the biomass reaction is the target function, but even if such a reaction is present, a different reaction could be the optimization target.
Some models don't have a biomass function, such as model iAT_PLT_636.xml, but still contain an empty flux objective.

The solution would be to create another parameter to specify the identifiers of those reactions that are the optimization targets.

This bug has been fixed with the release of version 1.2. There are now two new command-line arguments that are explained in the release notes. In these cases, ModelPolisher tries to find and use by default the biomass reaction with coefficient 1.0, unless otherwise specified by the user.


Nesting external inputs in an inline block should leave room for inputs

It's very unclear here that there are open slots to connect blocks to the & here.  Ideally, the layout would add a little bit of extra space for the missing blocks.

Duping to #325.  I think we should just disallow this entirely: when a child of an inline block is made external, it should change the parent to external as well, and vice versa


Update CNI config versions to 0.3.0

We've been seeing conflicts with other CNI consumers where CRI-O's configuration files are causing the CNI plugins to fail to start because their versions are too low. Upgrading the plugin versions should resolve this conflict, and not cause any adverse effect to a typical CRI-O install.

Don't you have to update the plugins as well?

the long term fix though matt is to go to cni right ?

@baude I think these configs need to stay here even if we start shipping a packaged config - we need this for people who build from source and potentially local tests

@mrunalp The CNI and Plugin versions we have in code are plenty new enough to deal with a v0.3.0 config (they should be good up to v0.3.1), but I could revendor and use more recent versions in our Dockerfiles if you want

/test all

@rajatchopra @dcbw PTAL

/test integration_rhel

LGTM
/test integration_rhel

/test integration_rhel

/test integration_rhel

All Green,
Merging.


FamousEngine->removeScene does not recursively remove all child and grandchild nodes

It is expected that removeScene would do a depth-first search and remove all child and grandchild nodes but it doesn't seem to do this. The child and grandchild nodes continue to remain.

I'm running into the same issue when doing dismount() on my root node and then try to mount new nodes.
It basically boils down to this:
rootNode.dismount();
famousScene.addChild(new Node());

I'm getting an error in the PathStore. Seems like the memo is correctly cleared, but according to the paths array all the children of the rootNode are still there.
Doing a manual recursive depth-first dismount() solves the issue for me.


Update to AGP 8.1.0

Task/Issue URL: https://app.asana.com/0/488551667048375/1205249419591212/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3553   👈

This comment was auto-generated by Graphite.