Keyboard not fount

Are you using the live version that could be found here ?

[ ] Yes, absolutely
[ ] No, I might try it out

What type of issue is it ?

[ ] Graphical issue (UX, UI, Graphical Glitch)
[ ] Feature issue (Command not/bad implemented, Non functionnal feature)
[ ] Crash (The buttons clicks does nothing at all, Error in console)
[ ] Question (Any type about the project :D)

Here are relevant errors/infos outputed by the code !

Describe your problem here :D
Arduino: 1.8.10 (Windows 10), Board: "Arduino/Genuino Uno"
In file included from C:\Users\hohoh\Documents\Arduino\sketch_oct24a\sketch_oct24a.ino:5:0:
C:\Program Files (x86)\Arduino\libraries\Keyboard\src/Keyboard.h:29:2: warning: #warning "Using legacy HID core (non pluggable)" [-Wcpp]
#warning "Using legacy HID core (non pluggable)"
^~~~~~~
C:\Users\hohoh\Documents\Arduino\sketch_oct24a\sketch_oct24a.ino: In function 'void typeKey(int)':
sketch_oct24a:9:3: error: 'Keyboard' not found. Does your sketch include the line '#include <Keyboard.h>'?
Keyboard.press(key);
^~~~~~~~
C:\Users\hohoh\Documents\Arduino\sketch_oct24a\sketch_oct24a.ino: In function 'void setup()':
sketch_oct24a:18:3: error: 'Keyboard' not found. Does your sketch include the line '#include <Keyboard.h>'?
Keyboard.begin();
^~~~~~~~
sketch_oct24a:25:18: error: 'KEY_LEFT_CTRL' was not declared in this scope
Keyboard.press(KEY_LEFT_CTRL);
              ^~~~~~~~~~~~~

sketch_oct24a:26:18: error: 'KEY_LEFT_ESC' was not declared in this scope
Keyboard.press(KEY_LEFT_ESC);
              ^~~~~~~~~~~~

sketch_oct24a:35:11: error: 'KEY_RETURN' was not declared in this scope
typeKey(KEY_RETURN);
       ^~~~~~~~~~

sketch_oct24a:39:11: error: 'KEY_TAB' was not declared in this scope
typeKey(KEY_TAB);
       ^~~~~~~

sketch_oct24a:59:18: error: 'KEY_LEFT_ALT' was not declared in this scope
Keyboard.press(KEY_LEFT_ALT);
              ^~~~~~~~~~~~

Multiple libraries were found for "Keyboard.h"
Used: C:\Program
Multiple libraries were found for "HID.h"
Used: C:\Program
exit status 1
'Keyboard' not found. Does your sketch include the line '#include <Keyboard.h>'?
This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.

Could you provide the .zip Duckuino produced ?

Also you can't use Arduino Uno with this project !!

Okay i set the wrong Arduino


BAEL-6835, PrintWriter vs FileWriter in Java

1- The test class, CharsTreamsUnitTest.java doesn't pass the checks when the FileNotFoundExceptions aren't handled manually .
2- The BeforeEach method doesn't pass the checks either when the IOException is not manually handled.
3- Using the try-with-resources syntax doesn't work either.
4- Passing the file path strings directly in the File constructors generates errors because I am using the files created in the BeforeEach method as containers for the return values of my methods:  pwFile = CharStreams.writingStringToFilePrintWriter(TODO_LIST_PW_FILE , TODO.toString());
5- For consistency purposes, I prefer to pass file paths as Strings to all my writer methods' arguments, due to their constructors' second parameters used in some of the CharStreams.java methods.

1- The test class, CharsTreamsUnitTest.java doesn't pass the checks when the FileNotFoundExceptions aren't handled manually .
2- The BeforeEach method doesn't pass the checks either when the IOException is not manually handled.

As discussed, there is a bug in the test, fix the bug and then the extra try-catch blocks can be removed.

3- Using the try-with-resources syntax doesn't work either.

Please give an example of when this does not work for you and I can help you to fix it.

4- Passing the file path strings directly in the File constructors generates errors because I am using the files created in the BeforeEach method as containers for the return values of my methods:  pwFile = CharStreams.writingStringToFilePrintWriter(TODO_LIST_PW_FILE , TODO.toString());

I do not understand what you mean here, but one overarching issue is that you should not have these methods at all. Please inline the methods into the main test body. Tests are written with the code under test clearly visible in the test method. Having a layer of indirection with the CharStreams methods makes it much harder to see what is going on in these tests.
In addition, you should not be assigning to a field that is supposed to be a constant. We can use the same constant for both the constructor and for the assertion. This is preferred to having multiple variables that do the same job (single responsibility principle).

5- For consistency purposes, I prefer to pass file paths as Strings to all my writer methods' arguments, due to their constructors' second parameters used in some of the CharStreams.java methods.

Once you have inlined the CharStreams.java methods, it will become more clear that there are redundant new File creations that can be replaced by the constant fields you have defined.

6- I use the flush() methods in the article intentionally for teaching purposes, since the PrintWriter can automatically flush as opposed to the FileWriter, it'll be more appropriate to omit the flush() method only when demonstrating the auto-flushing feature of the PrintWriter.

Makes sense, understood.

I made the changes as you required but now the checks are failing.

All the checks have passed.

All the checks have passed.

Please scroll up and review all the unresolved issues, I think you might have accidentally missed them by only seeing the most recent ones. Thank you.

Resolved all the issues, but the checks are failing.

Resolved all the issues, but the checks are failing.

Can you please confirm that you are able to compile and run your code locally? From your commit history, it looks as if you might be using the GitHub Web UI to make changes, rather than actually running the code on your local machine
Which IDE are you using?

I use IntelliJ IDEA.
And yes, the recent edits from 5 hours ago were made with GitHub web, before that I was working in my IDE and all the tests were passing.

I use IntelliJ IDEA. And yes, the recent edits from 5 hours ago were made with GitHub web, before that I was working in my IDE and all the tests were passing.

Please use your IDE to compile the code and run the tests. It will tell you where there are compilation errors, and if a test is failing, you can use the debugger to investigate why.
Do you need any help with either of these things?

Okay, no thanks.

Please I need some assistance with this: implement the whenAutoFlushEqualTrue_thenPrintWriterFlushesAutomatically

Outstanding issues from my perspective:

use try-with-resources
flush in every test apart from the auto flush test
implement the whenAutoFlushEqualTrue_thenPrintWriterFlushesAutomatically
move cleanup file delete logic into before method
use a string constant for the first two tests about text data

See #14840 (comment) for my suggestions on how to address these, you are welcome to use this code directly, it already works well

I did all these but the checks are still failing.

Everything works well locally.


PHOENIX-6432 Add support for additional load generators

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 10s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 25 new or modified test files.

_ master Compile Tests _

+1 :green_heart:
mvninstall
17m  4s
master passed

+0
hbaserecompile
24m 33s
HBase recompiled.

+1 :green_heart:
compile
0m 32s
master passed

+1 :green_heart:
checkstyle
0m 31s
master passed

+1 :green_heart:
javadoc
0m 21s
master passed

+0 :ok:
spotbugs
0m 55s
phoenix-pherf in master has 51 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
9m 58s
the patch passed

+0
hbaserecompile
18m  0s
HBase recompiled.

+1 :green_heart:
compile
0m 34s
the patch passed

+1 :green_heart:
javac
0m 34s
the patch passed

-1 :x:
checkstyle
0m 36s
phoenix-pherf: The patch generated 533 new + 757 unchanged - 285 fixed = 1290 total (was 1042)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m 10s
The patch has no ill-formed XML file.

-1 :x:
javadoc
0m 19s
phoenix-pherf generated 25 new + 33 unchanged - 24 fixed = 58 total (was 57)

-1 :x:
spotbugs
1m  6s
phoenix-pherf generated 12 new + 42 unchanged - 9 fixed = 54 total (was 51)

_ Other Tests _

+1 :green_heart:
unit
9m 44s
phoenix-pherf in the patch passed.

+1 :green_heart:
asflicense
0m 13s
The patch does not generate ASF License warnings.

59m  4s

Reason
Tests

FindBugs
module:phoenix-pherf

Possible null pointer dereference in org.apache.phoenix.pherf.workload.mt.handlers.RendezvousingWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at RendezvousingWorkHandler.java:org.apache.phoenix.pherf.workload.mt.handlers.RendezvousingWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at RendezvousingWorkHandler.java:[line 76]

Return value of org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.getPhoenixUtil() ignored, but method has no side effect  At RendezvousingWorkHandler.java:but method has no side effect  At RendezvousingWorkHandler.java:[line 83]

Possible null pointer dereference in org.apache.phoenix.pherf.workload.mt.handlers.TenantOperationWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at TenantOperationWorkHandler.java:org.apache.phoenix.pherf.workload.mt.handlers.TenantOperationWorkHandler.onEvent(BaseLoadEventGenerator$TenantOperationEvent) due to return value of called method  Dereferenced at TenantOperationWorkHandler.java:[line 65]

Return value of org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.getPhoenixUtil() ignored, but method has no side effect  At TenantOperationWorkHandler.java:but method has no side effect  At TenantOperationWorkHandler.java:[line 67]

input must be non-null but is marked as nullable  At IdleTimeOperationSupplier.java:is marked as nullable  At IdleTimeOperationSupplier.java:[lines 51-74]

input must be non-null but is marked as nullable  At PreScenarioOperationSupplier.java:is marked as nullable  At PreScenarioOperationSupplier.java:[lines 51-85]

A prepared statement is generated from a nonconstant String in org.apache.phoenix.pherf.workload.mt.operations.PreScenarioOperationSupplier$1.apply(TenantOperationInfo)  At PreScenarioOperationSupplier.java:from a nonconstant String in org.apache.phoenix.pherf.workload.mt.operations.PreScenarioOperationSupplier$1.apply(TenantOperationInfo)  At PreScenarioOperationSupplier.java:[line 74]

input must be non-null but is marked as nullable  At QueryOperationSupplier.java:is marked as nullable  At QueryOperationSupplier.java:[lines 54-94]

Impossible cast from RuntimeException to java.sql.SQLException in org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.executePreRunOpsForTenant(TenantOperationFactory$TenantView, TenantOperationInfo)  At TenantOperationFactory.java:java.sql.SQLException in org.apache.phoenix.pherf.workload.mt.operations.TenantOperationFactory.executePreRunOpsForTenant(TenantOperationFactory$TenantView, TenantOperationInfo)  At TenantOperationFactory.java:[line 354]

input must be non-null but is marked as nullable  At UpsertOperationSupplier.java:is marked as nullable  At UpsertOperationSupplier.java:[lines 58-161]

org.apache.phoenix.pherf.workload.mt.operations.UpsertOperationSupplier$1.apply(TenantOperationInfo) may fail to clean up java.sql.Statement  Obligation to clean up resource created at UpsertOperationSupplier.java:up java.sql.Statement  Obligation to clean up resource created at UpsertOperationSupplier.java:[line 107] is not discharged

input must be non-null but is marked as nullable  At UserDefinedOperationSupplier.java:is marked as nullable  At UserDefinedOperationSupplier.java:[lines 43-47]

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1226/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1226

Optional Tests
dupname asflicense javac javadoc unit xml compile spotbugs hbaserebuild hbaseanti checkstyle

uname
Linux dd475f21660b 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 5265164

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1226/2/artifact/yetus-general-check/output/diff-checkstyle-phoenix-pherf.txt

javadoc
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1226/2/artifact/yetus-general-check/output/diff-javadoc-javadoc-phoenix-pherf.txt

spotbugs
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1226/2/artifact/yetus-general-check/output/new-spotbugs-phoenix-pherf.html

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1226/2/testReport/

Max. process+thread count
2332 (vs. ulimit of 30000)

modules
C: phoenix-pherf U: phoenix-pherf

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1226/2/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


add prettier + basic voting component

In this PR:

added prettier support so that we get consistent formatting (make sure to install esbenp.prettier-vscode and dbaeumer.vscode-eslint vscode extensions)
added a basic voting component UI (as an exercise of using Angular)

TODO:

preserving voting results into dataService

NICE! One more TODO: We'll want to show the profile of the user (avatar image, preferred pronouns, and name) too...


Bad matches for grammars that require backtracking

This lexical grammar is ambiguous with only one character lookahead and no backtracking:
#[derive(Logos, Debug)]
enum Token {
    #[error]
    Error,

    #[token("a")]
    A,

    #[token("axb")]
    B,

    #[regex("ax[bc]")]
    Word,
}

It lexes ax as A:
fn main() {
    let mut lex = Token::lexer("ax");
    while let Some(t) = lex.next() {
        println!("{:?}: <{}>", t, lex.slice());
    }
}
// output:
// A: <ax>

After encountering ax, it could either be A, Error, the start of B, or the start of Word.  Logos accepts the x, looking for either B or Word.  When it fails to find a B or a Word, it's unable to backtrack the x, and so commits to ax having been A.
This is a super hard problem to solve because there is no DFA for this grammar.  I think the most reasonable thing to do in situations like this would be for Logos to give an error. Edit: it has a DFA, just not one with a single terminal state.  This should probably be handled by remembering the longest match, and backtracking to it if it can't continue.

Simpler example:
#[derive(Logos, Debug)]
enum Token {
    #[error]
    Error,

    #[regex(r"a(xb)?")]
    Word,

    // Only necessary to get it to make a bytewise DFA for /a(xb)?/
    // Otherwise it checks for "xb" in one step
    #[token("axyz")]
    Other,
}

Results in Word: <ax>.
It seems the issue is with DFAs with multiple terminal states separated by non-terminal states. Here's this one. From state C, it needs to be able to revert to how the input was at state B if it can't proceed to D.
This might be related to #180; I'm not sure.

#[derive(Logos, Debug, PartialEq)]
enum Token {
    #[regex(r"[\n]", logos::skip)]
    #[error]
    Error,

    #[token("@", priority = 100)]
    Email,

    #[regex(r"[a-z]@", priority = 1)]
    Misc,
}

The stringa@ has Misc variant instead of Email. Any chance of a fix?

@saskenuba Looking at your grammar, a@ matches the regex [a-z]@ for Misc, and it is not the literal token @ for Email.  In this case, Logos is doing the right thing.  If you want a@ to result in Email, your code is not correct.
What do you want your code to do?  To help you fix it, we'll need to understand the problem you're trying to solve.

Is there an alternate way for me to write my regex so that it will work?


(PUP-10148) Include X-Puppet-Profiling Header

When Puppet[:profile] is true, we should always add
'X-Puppet-Profiling' => 'true' to all requests made by
Puppet::HTTP::Client. By default, this header should be omitted.
This header is used to tell the server that it may log
profiling data for that agent run. See
https://github.com/puppetlabs/puppet/blob/90f42e963a2f2147a6153977e7ef3cbacad5a0ef/docs/profiling.md

Currently puppet only includes the header when using the indirector to make puppet-y REST calls, but not general HTTP connections to the forge for example. I'm thinking we should keep the same behavior, which I think means Puppet::HTTP::Service::* instances need to add the header when calling back into client.get for example

CLA signed by all contributors.

I can't figure out what's going on with appveyor.
openssl (default: 2.1.2)
OpenSSL Version - OpenSSL 1.1.1b  26 Feb 2019
OpenSSL Library Version - OpenSSL 1.1.1b  26 Feb 2019
bundle : The dependency rdoc (~> 6.0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for x64-mingw32 but the dependency is only for ruby. To add those platforms to the bundle, run `bundle
lock --add-platform ruby`.
At line:9 char:1
+ bundle install --jobs 4 --retry 2 --without packaging documentation
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (The dependency ...platform ruby`.:String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

The dependency rubocop (~> 0.49) will be unused by any of the platforms Bundler is installing for. Bundler is installing for x64-mingw32 but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock
--add-platform ruby`.
The dependency rubocop-i18n (~> 1.2.0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for x64-mingw32 but the dependency is only for ruby. To add those platforms to the bundle, run
`bundle lock --add-platform ruby`.
The dependency memory_profiler (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for x64-mingw32 but the dependency is only for ruby. To add those platforms to the bundle, run `bundle
lock --add-platform ruby`.
The dependency pry (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for x64-mingw32 but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock
--add-platform ruby`.
The dependency racc (= 1.4.9) will be unused by any of the platforms Bundler is installing for. Bundler is installing for x64-mingw32 but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock
--add-platform ruby`.
The dependency gettext-setup (~> 0.28) will be unused by any of the platforms Bundler is installing for. Bundler is installing for x64-mingw32 but the dependency is only for ruby. To add those platforms to the bundle, run
`bundle lock --add-platform ruby`.
The dependency ronn (~> 0.7.3) will be unused by any of the platforms Bundler is installing for. Bundler is installing for x64-mingw32 but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock
--add-platform ruby`.
Fetching gem metadata from https://rubygems.org/.........

bundle : C:/projects/puppet/lib/puppet/util/windows.rb:18: warning: Win32API is deprecated after Ruby 1.9.1; use fiddle directly instead
At line:14 char:1
+ bundle exec rake parallel:spec[2]
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (C:/projects/pup...irectly instead:String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

C:/projects/puppet/lib/puppet/util/windows/principal.rb:115:
 [BUG] Segmentation fault
ruby 2.6.3p62 (2019-04-16 revision 67580) [x64-mingw32]

[BUG] Segmentation fault

There's a bug in our windows code that sometimes triggers a ruby segfault. We haven't been able to determine why. I've only been able to reproduce once outside of Appveyor...

I've updated this PR based on Josh's comments


AssetManager-Better Wrapper

AssetManager is still in a brute code form.
try category the logic of code into atomic form and wrapper with internal function.

wrong repo


Fix CMake configuration with PAPI on

Fixes master. I broke configuration with PAPI turned on with #3954. This attempts to fix it. There's a build running on rostam here: http://rostam.cct.lsu.edu/builders/hpx_gcc_6_boost_1_64_centos_x86_64_debug/builds/373.

And another try: http://rostam.cct.lsu.edu/builders/hpx_gcc_6_boost_1_64_centos_x86_64_debug/builds/374. PAPI_LIBRARIES was also missing as a dependency from the PAPI component.

Hopefully the last try: http://rostam.cct.lsu.edu/builders/hpx_gcc_6_boost_1_64_centos_x86_64_debug/builds/375.


Use Netatalk 2.x v220101

@rdmark I don't have an old Mac (or any Mac actually), so I suspect I have no proper way to test this completely. But I have tried to compile the branch and it exited with a zeroconf installation not found error.

I'm suspecting that an apt install libavahi-client-dev is missing somewhere in easyinstall.sh. Other than that, I can only say that it it compiles and papd+timelord are running on my pi4 ;-) So I'm not sure whether it makes sense for me to approve the old mac specific pull requests. I can basically only compile and dry-look at the code.

@bzeiss That's a good catch! For some reason, I was assuming that libcups-dev pulls in libavahi-client-dev, which seems to have been incorrect. I updated the list of packages to install now!
Did you see the atalkd, cnid and afpd daemons get installed and run as well? These three are the core file sharing enabling services.

@rdmark Ah, yes, atalkd, cnid_metad and afpd are running. I wasn't aware that these are installed as well. I was just looking at the services mentioned in the install script.

@bzeiss Good point. I've added a systemctl status command to make it clear which services have been added.

Converting to draft to add a few more steps to automate papd configuration (CUPS)

The script will now install and configure CUPS as well, which should make the printer server work out of the box with most CUPS compatible printers.


AliasLoader load Method isset Parameter is Value instead of Key

Laravel Version
9.52.5
PHP Version
8.1.16
Database Driver & Version
No response
Description
The alias is: "Illuminate\Support\Facades\Mail"
The $this->aliases is:
Array(47)
App:"Illuminate\Support\Facades\App"
Arr:"Illuminate\Support\Arr"
Artisan:"Illuminate\Support\Facades\Artisan"
Auth:"Illuminate\Support\Facades\Auth"
Blade:"Illuminate\Support\Facades\Blade"
...
Mail:"Illuminate\Support\Facades\Mail"
...
The code that seems incorrect is:
if (isset($this->aliases[$alias])) {
return class_alias($this->aliases[$alias], $alias);
}
Obviously, it will fail here. Did you want: array_search ($alias, $this->aliases)?
Steps To Reproduce
Not sure reproducing is needed here - the code is simply incorrect - not sure why there are not more impacts. Maybe I am missing something.

Heya, I'm not sure I see an actual bug in this report. If you can, please submit a PR with a failing test that demonstrates the bug. Thanks


Fix Forest docker image

Summary of changes
Changes introduced in this pull request:

Fixed docker image for Forest + made it more usable,
basic Docker CI and packaging (no release tags yet)
build will no longer fail, even if run outside of the git context (e.g. release tarballs).

Reference issue to close (if applicable)

Resolves https://github.com/ChainSafe/forest/issues/1384
Closes https://github.com/ChainSafe/forest/pull/1442
Other information and links

Co-authoring @bodo-hugo-barwich as he had some good ideas and made the first push towards proper containerisation. Thank you @bodo-hugo-barwich!

How much storage do we have at ghcr? We're the free plan, right? So, half a gig?

How much storage do we have at ghcr? We're the free plan, right? So, half a gig?

Given it's a public repository, I believe our limit is some abuse limit. 500MB limit applies to private repositories.


Library docstrings

Adds docstrings for top-level job-queue and database manager objects.

Two separate commits because two separate changes.


CDPAM-1467 | Read CCMv2 agent params from a config file.

In this PR I have made changes to create a .toml file and add the params required for running the ccmv2 agent on a node. This way we do not expose params when a non root user tries to read the command which might contain sensitive info like proxy username and passwords.
I have tested the changes manually on a freeIPA node and it works as expected.
I also started a build with the branch in http://build.eng.hortonworks.com:8080/job/cloudbreak-image-burn-salt-v3/16278/ which succeeded.

does the current version, which is on the images, support this config file?

does the current version, which is on the images, support this config file?

Thanks. I had misssed it :( . I have updated the image version and its SHA sum in the commit.

Triggered http://build.eng.hortonworks.com:8080/job/cloudbreak-image-burn-salt-v3/16285/ with the changes.


work around empty database when slack is setup

This is quite an edge case but something that came up during my AWS migration testing.

Codecov Report
Attention: Patch coverage is 45.45455% with 6 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

apps/slack/apps.py
45.45%
6 Missing :warning:

Additional details and impacted files

:loudspeaker: Thoughts on this report? Let us know!


AL-119: Format domain event occurred at as an offset ISO string in line with other services

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

Project coverage is 96.09%. Comparing base (2aa8fde) to head (554333d).

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #105      +/-   ##
============================================
- Coverage     96.10%   96.09%   -0.02%
+ Complexity      590      588       -2
============================================
  Files            92       91       -1
  Lines          1949     1944       -5
  Branches         86       86
============================================
- Hits           1873     1868       -5
  Misses           59       59
  Partials         17       17

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Task 183 add rhyming task

Thanks! Can you check why the tests are failing? You can click on "details" above.

@ethankim00, Seems like there is a test failure. Can you address?
File "src/test_all.py", line 45, in
assert type(i) == str, f'the output is not a string'
AssertionError: the output is not a string

@ethankim00, Commenting to make sure that you have seen my earlier comment.

Looks good to me!


fix: provide solution that satisfies requirement

The specification says we have to return something that implements
Animal, but can't tell which at compile time.  The existing solution
only returns one concrete type Sheep, while the intent was to return
either Sheep or Cow.  This requires type erasure that can only be done
with a reference to dyn trait.

After reviewing the surrounding text it's clear this isn't the right solution, but then the exercise asks for something that can't be done: use impl Trait to return values for which the type can't be determined at compile time.


Add dynamic sizing Width/Height calculation feature

In looking at the old barcodelib embedded in our unshareable project code, it looks like we rely on the the old image sizing behavior documented in the ChangeLog at codeproject:

December 3, 2008

Can now specify image size before encoding.

Complete overhaul of the drawing function to allow for dynamic drawing size. Now calculates the bar width based on the specified image size. Removed resize functions due to inaccuracy of the resizing method. The encoding functions have been revamped to reflect these changes (removed some of the overloaded functions referencing the resize functions). Also removed a majority of the case statements from Generate_Image() to simplify it. Updated the article to reflect the changes to the Encode functions and the changes to the drawing functions.

There are a few things about how it looks that the current API works that are not nice here. First, to generate a barcode for an arbitrary string, the library’s user must be able to somehow predict the number of pixels necessary to render that barcode. This is really the biggest issue for me. I would like to be able to render a string and let the library automatically set BarcodeLib.Barcode.Width to a high-enough value for me instead of getting an exception “EGENERATE_IMAGE-2: Image size specified not large enough to draw image. (Bar size determined to be less than 1 pixel)”.
To remedy this, I would like it to be possible to opt into the old behavior with an API like this:
/// If non-null, sets the width of a bar. <see cref="Width"/> is ignored and calculated automatically.
public int? BarWidth { get; set; }

/// If non-null, <see cref="Height"/> is ignored and set to <see cref="Width"/> divided by this value rounded down.
/// <remarks><para>As longer barcodes may be more difficult to align a scanner gun with,
/// growing the height based on the width automatically allows the gun to be rotated the
/// same amount regardless of how wide the barcode is. A recommended value is 2.
/// </para></remarks>
public double? AspectRatio { get; set; }

The idea is that you could choose to dynamically calculate the Width, Height, both, or neither. (Though “dynamically” calculating the Height would basically be statically calculating it in the end ;-)). Does this API sound good? I think I’ll start working on a pull request, but thought I’d put this out here in case I get lost in some rabbit hole or others have comments/suggestions on this approach.

very good write up ... let me ponder this for a bit and read it again when I get home.  THIS is what this project was open sourced for.  Improvements and expansion of the original idea.  Finally someone that understands it and doesnt berate me on the coding. :) (yes I have a person that is berating me on the coding and pretty relentlessly I might add)

sounds good.  If you want to put a pull request together Ill anxiously await it so I can review it!!!  Thanks for the interest and I hope more people find this useful and worth contributing to.

pull is #6.

since we pulled that other code in are you ok with closing this issue?


Testing PR

Testing PR

Looks like carson has reacted to its own comment :)

... he's growing more powerful...


[1.12] Unsmeltable tool parts

I'm attempting to make an armor set from a mod a material.
The metal that the armor set is made of has no associated ingot, block, nugget or gem.
So I registered the sword as the ingot instead.
The parts are all showing up in Creative Mode and in JEI, but I am unable to actually smelt any items in the set, even the sword that I registered as ingot.
The liquid does exist, but cheating it into my furnace still leaves be unable to cast any parts from it.
I tried not having an ingot set at all, but that has the same result.
Is there anything I can do to make it work without a "real" ingot?

{
"name": "Ice",
"make": ["fluid"],
"textColor": "#66B8DA",
"whitelist": {
"oredict": [],
"ingots": [],
"nuggets": [],
"dusts": [],
"gems": [],
"blocks": [],
"ores": []
},
"blacklist": {
"oredict": [],
"ores": []
},
"fluidData": {
"color": "#66B8DA",
"density": 2000,
"viscosity": 10000,
"temperature": 100,
"luminosity": 10,
"rarity": "UNCOMMON"
},
"canCraftToolForge": false,
"toolData": {
"durability": 28,
"harvestLevel": 4,
"extraDurability": 400,
"miningSpeed": 8,
"attack": 7.5,
"handleModifier": 1.25,
"handleDurability": 0,
"color": "#66B8DA",
"temperature": 0,
"projectiles": false,
"drawSpeed": 1.0,
"range": 1.0,
"bonusDamage": 0.0,
"stringModifier": 0.0,
"shaftModifier": 1.0,
"bonusAmmo": 0.0,
"accuracy": 1.0,
"fletchingModifier": 1.0,
"traits": ["freezing"],
"headTraits": ["freezing"],
"handleTraits": [],
"extraTraits": [],
"bowTraits": [],
"stringTraits": [],
"projectileTraits": [],
"shaftTraits": [],
"fletchingTraits": []
},
"localizations": [{
"locale": "en_US",
"solid": "Ice",
"liquid": "Molten Ice",
"bucket": "Ice Bucket"
}]
}

I don't think this is going to work....
However, what object will your players throw into the smeltry to melt into your fluid?

Oh I use Minetweaker to set those.
However, setting an item (eg Packed Ice) as an ingot does not work either.

Here is another JSON file that doesn't work as an example. The Json is valid and the liquid and the parts are all generated, but they cannot be made in any way.

{
"name": "Creativium",
"make": ["fluid"],
"textColor": "#D2F200",
"whitelist": {
"oredict": [],
"ingots": [],
"nuggets": [],
"dusts": [],
"gems": [],
"blocks": ["tconstruct:materials:16"],
"ores": []
},
"blacklist": {
"oredict": [],
"ores": []
},
"fluidData": {
"color": "#D2F200",
"density": 2000,
"viscosity": 10000,
"temperature": 1000,
"luminosity": 10,
"rarity": "RARE"
},
"__comment": "If you do not want to be able to craft the Tool Forge (using blocks of this material) set the following to false.",
"canCraftToolForge": false,
"toolData": {
"durability": 1600,
"harvestLevel": 8,
"extraDurability": 150,
"miningSpeed": 9,
"attack": 8.25,
"handleModifier": 1.2,
"handleDurability": 0,
"color": "#D2F200",
"temperature": 0,
"projectiles": false,
"drawSpeed": 1.0,
"range": 1.0,
"bonusDamage": 0.0,
"stringModifier": 0.0,
"shaftModifier": 1.0,
"bonusAmmo": 0.0,
"accuracy": 0.0,
"fletchingModifier": 0.0,
"traits": ["ecological"],
"headTraits": ["alien","insatiable"],
"handleTraits": ["cheap", "petramor"],
"extraTraits": ["crumbling", "lightweight"],
"bowTraits": [],
"stringTraits": [],
"projectileTraits": [],
"shaftTraits": [],
"fletchingTraits": []
},
"localizations": [{
"locale": "en_US",
"solid": "Creativium",
"liquid": "Molten Creativium",
"bucket": "Sanctum Creativium"
}]
}

I am having the exact same problem with some materials I'm making! I'll see what I can dig up on what might be the problem, but since I'm kind of an amateur here I feel like the dev can do better work at than than I can.

I think I found something! I believe the problem may lie in using custom items (meaning not from vanilla MC) for the items in the json file. I tested one of my materials, and then tested the standard diamond example, and the diamond smelted properly and everything. So maybe that is the problem.

I think found a way to make it work! I set items in both blocks and ingots, and it started working properly. Pending on using custom items instead of vanilla ones, but it's looking good so far. I understand this issue is over a year old but I'm still going to post any updates I have here.

Yeah, that got it working! I added the items from my mod into both the ingots and blocks categories of my [material].json and the liquid was usable. I would suggest trying that if you're still paying attention to this issue and you still have that problem.

Thank you for the feedback.  Also, there is a new version released for MC1.12.2.


feat: Pull notes from PD to create internal service log

Blocked by #2.
Often, SREs will add notes to PagerDuty incidents as they work. Once done, it can be helpful to convert that into an internal service log. Essentially, this ticket is to convert the following Bash script to Go code.
#!/usr/bin/env bash
set -euo pipefail

# Generate internal service logs in Markdown from PagerDuty incident notes
# Example execution: `sl-from-pd Q0KCQ42ZYTYDKL 'This is a test' 52b9eb2b-0f45-43a6-a0c2-c5e5b2931bac`
# Defaults to use $CLUSTER_ID: `sl-from-pd Q0KCQ42ZYTYDKL 'This is a test'`

pd_incident=${1}
title=${2}
cluster=${3:-$CLUSTER_ID}

pd_notes=$(pd incident notes -i ${pd_incident} -x --output json | jq -r '.[] | "### \(.created) from  \(.added_by)\n\n\(.content)\n" | @text')

sl_message=$(echo "# ${title}

[PD Incident](https://redhat.pagerduty.com/incidents/${pd_incident})

## PD Notes

${pd_notes}
" | vipe)

echo "$sl_message"
echo
osdctl servicelog post $cluster --internal -p MESSAGE="${sl_message}"

Following #2, prefill the form elements and allow the user to make edits.
Acceptance Criteria:

Add option --pagerduty/-p option to servicelogger internal command.
Pre-fill title and description form fields as specified above.
Send internal service log.

Tabling this for mow. PagerDuty might be outside the scope of what servicelogger should do.


Visibility Through Walls

Is it possible to turn off the visibility through walls for these holograms? I'm trying to make a shop with them using the floating items, but making it hard when i have 100+ lines showing through the wall. Thank you.

This is not an issue the plugin can solve due to the way names on armor stands work


Apollo Tracing Support

https://github.com/apollographql/apollo-tracing

private static long nanoTime() {
   long nano = 10000L * Stopwatch.GetTimestamp();
   nano /= TimeSpan.TicksPerMillisecond;
   nano *= 100L;
   return nano;
}

source

The nanoTime implementation above is dangerous, because it does not consider Stopwatch.Frequency which may differ from hardware to hardware. The following implementation is technically correct.
private static long GetTimestampInNanoseconds()
{
    const long nanosecondsPerSecond =<PHONE_NUMBER>;

    return Stopwatch.GetTimestamp() *
        (nanosecondsPerSecond / Stopwatch.Frequency);
}

#459


OS-Climate - Establish minimum versions of tools / packages for Dev Cluster

xref https://github.com/os-climate/os_c_data_commons/issues/98
High-level question: if we use conda as our base installation system, users can install shell-level packages such as ghostscript without needing to beg for special install help.  With pip/pipenv, we are entirely limited to Python.  What, really, is the best choice here?

[ ] python packages

[ ] pandas 1.5.3
[ ] numpy 1.24.0
[ ] SqlAlchemy 1.4.46 (2.0.3 released, removes Python2 legacy code, but not compatible with dbt-core==1.3.2)
[ ] Trino Python Client 0.321.0
[ ] Pydantic 1.10.2 (Python 3.11 update)
[ ] Pint 0.20.1 (with uncertainties support)
[ ] Pint-Pandas 0.3 (with uncertainties support)
[ ] python-pachyderm 7.3.3 (forces dbt-core==1.3.2)
[ ] multidict 6.0.2 (Badly-behaved python-pachyderm looks for this version rather than current latest 6.0.4)
[ ] pyyaml 5.4.1
[ ] uncertainties 3.1.7
[ ] boto3 (latest version)
[ ] osc-ingest-tools (>=0.4.3; requires some fixes to work with SQLAlchemy 2.0.x)
[ ] python-dotenv (latest version)
[ ] pyarrow (latest version)
[ ] fastparquet (?)
[ ] iam-units (latest version)
[ ] openscm-units (latest version)
[ ] openpyxl (latest version)
[ ] dash (latest version)
[ ] dash_bootstrap_components (latest version)
[ ] matplotlib (latest version)
[ ] ydata_profiling (latest version; "pandas-profiling[notebook]" is going to be deprecated April 1st)
[ ] pygithub (latest version)
[ ] xlrd (latest version)
[ ] camelot-py or camelot-fork
[ ] consensus correct one of four options for installing opencv-python (see https://pypi.org/project/opencv-python/)

[ ] rpm/yum packages

[ ] ghostscript (needed for camelot)
[ ] tkinter (needed for camelot)

[ ] [dev-packages]

[ ] flake8 = "*"
[ ] coverage = "*"
[ ] Sphinx = "*"
[ ] What else is needed for general pre-commit consistency (black formatting?) and CI/CD happiness (pip audit, setup tools, pytest unit tests, etc)?
[ ] Can we protect AWS keys from finding their way to GitHub via a pre-commit process that forbids credentials.env files from being committed, and/or any files that have well-known secrets labels or content.  If users cannot commit bad changes, they cannot push bad commits.

[ ] ODH sub-packages

[ ] jupyterlab 3.5.0
[ ] Trino 406 (several Hive/Iceberg and also BigQuery fixes since currently installed 398 version)
[ ] TileDB 2.17 (expected April 2023) updates from C++17 to C++20, with lots of breaking changes.  Check with @joemoorhouse  how TileDB may play into the 2023 PhysRisk platform.
[ ] Elyra 0.2.X (with fix for Custom Elyra Notebooks) vs. 0.3.13 or greater?
[ ] Superset 2.0.1
[ ] Python 3.10 (need CI/CD for 3.9 and 3.11)
[ ] Pachyderm 2.4 (does this belong here or in software libraries bundle below?)

[ ] Finalize list of required software libraries, packages for dev cluster (these are best guesses as of 2022-12-22)

[ ] openmetadata-ingestion <IP_ADDRESS>
[ ] OpenShift X.X
[ ] Fybrik X.X
[ ] Inception 25.4
[ ] Datasette?
[ ] dbt-core 1.3.2 (allows python-pachyderm==7.3.3, but prevents sqlalchemy==2.0.2)
[ ] dbt-trino (depends on dbt-core)
[ ] trino-utils 0.3.0 (companion to dbt-utils 1.0.0)
[ ] dbt-utils 1.0.0 (these are installed via dbt, not PyPi nor conda)

[ ] Baseline a default JupyterLab Notebook (update required libraries, remove unnecessary config info, etc.)

[ ] Ensure documentation is accurate for data ingestion pipeline processes

[ ] creation of pipelines for new data sources
[ ] updates to pipelines for existing data sources (remove, add, change)
[ ] creation of new metadata
[ ] updates to existing metadata (remove, add, change)

[ ] Update the OS-Climate Data Commons Developer Guide

remove highlander.  Categorize and group, note installed version.  Create a standard config for notebooks (default for users)

[] Architecture diagram of interdependencies/layers: call out dependencies and inheritance
-[] Phase 1:  open metadata, dbt
[] Python-related
[] Individual packages not under ODH: e.g. fybrik
[] ODH-related: (go back to default operator config - @redmikhail to check with @HumairAK to see what problems will occur if we leverage default)

[] Need to determine validation methodolgy, who will perform UAT (e.g. Trino connectors)
[] Test cases for each upgrade (e.g. smoke test demo with expected results) (Heather to create new issues for each workstream)

main reason for separation - capacity concerns, limited functionality - how ODH treats and updates subcomponents needed updating.  Need to understand where ODH going - treat trino, superset separately as this is also being done within the ODH community.
@redmikhail to consolidate approvers - create separate team to perform PR merge; regain control

Add LF team members to maintainer status:  Apps, Support (https://github.com/operate-first/support) plus others (@redmikhail to add others and provide PR):  6-Dec
Consider a separate branch of Operate First for LF OS-Climate?  To be discussed in upcoming meeting with Marcel and team.

@redmikhail to update list this week
Meeting held with Marcel.  Operate First shrinking - consider use of managed service from Redhat / AWS (Open Shift specifically - SREs); stable cluster is a better candidate for that but not all svcs are covered under managed svc - e.g. GPU usage split.
In January,

Need to meet and discuss pros/cons on moving to ODH as primary implementation (so can get the latest Jupyter release) with overlays for specific packages (e.g. Open Metadata) versus Operate First (where certain releases lag behind latest ones).
Need plan for Open Shift operational support.
Need to understand any custom configurations required by OS-C

Complexity of dealing with a platform (OS-Climate) on top of a platform (ODH) on top of a platform (Operate First)

How do users create their own Data Commons?  What is goal overall - keeping data private as well as accessing data via data exchange

@HeatherAck to schedule meeting week of 9-Jan to align on pros/cons and discuss path forward.

As an open source software project, OS-Climate provides the raw materials for users to contribute to and/or fork project elements as they see fit.  If users have their own ideas about what it means to run the Data Commons within their own local environment, it should be those users doing the legwork of what that actually means, and committing the resources necessary to push patches they want to see into the upstream source code (which OS-Climate should review and potentially accept).  But I don't think the OS-Climate project should try too hard to imagine and prototype those use cases itself.  Rather it should help guide users to do that work for themselves.

Need to determine pace and sizing / priorities for each element on 10-Jan, consistent developer process/implementation

@redmikhail @ryanaslett @MightyNerdEric @erikerlandson to focus on upgrading system level software (see ODH sub packages above:  e.g. Trino, Jupyter, Python)

@caldeirav - do we need to use Elyra pipeline features?
@erikerlandson / @redmikhail - to define standard developer notebook SW configs needed - libraries - Heather to schedule mtg with @erikerlandson - Guillome Moutier may be able to provide support

openmetadata team has been busily walking their versions forward.  <IP_ADDRESS> just released (see https://github.com/open-metadata/OpenMetadata/releases for info about 0.13.2).

@ryanaslett to start investigating latest ODH version compared to installed.  Prep work - manifest storage, look at operate first manifest. (week of 6-Feb)

@ryanaslett - trying to define what ODH contributions is OS-C going to make, but still want to move forward with core component upgrade.  No easy way to upgrade.  Need to install from scratch and migrate functionality (e.g., notebooks, etc.) and access/authentication.  May require diff packages - such as superset.  Start with CL1, eliminate old ODH and install new one.  Get feedback on new version / verify functioning as expected.
Open question:  @caldeirav will ODH be a stable version that we will use long term?  please confirm that Red Hat team will contribute to ODH going forward.

note:  fork it to OS-Climate (not operate first). keep as separate repo.  See if ODH supports SQL Alchemy 2.0


Consider using podman instead docker

It seems that docker is getting abandoned in fedora in favor to podman so currently without passing some kernel args docker is even unusable in fedora-32.
I just tried installing it, 'sudo dnf install podman' and replacing 'docker' by 'podman' in 'build-with-docker.sh' and it worked!:
tar cjf mirage-firewall.tar.bz2 -C _build --mtime=./build-with-docker.sh mirage-firewall
SHA2 of build:   0991877acc49b1c55a57dfa50df157fab7cc6c2fce5727f605e64a0bc73d322e  qubes_firewall.xen
SHA2 last known: d68d2a8d2337b8c1a78995e1acbb4f72082076c73be45bf40dd6268c87b2353e

Notes:

Run with sudo or it will fail due permissions errors
Extend root partition or it ends due out of space.

Thanks for your suggestion, and sorry for the delay. I so far have no issues with docker and f37. Using podman is currently not planed.

On Wed, Jul 05, 2023 at 10:11:04AM -0700, Pierre Alain wrote:

Thanks for your suggestion, and sorry for the delay. I so far
have no issues with docker and f37. Using podman is currently not planed.

sad face :(
--
cheers,
Holger
⢀⣴⠾⠻⢶⣦
⣾⠁⢠⠒⠀⣿⡁  holger@(debian|reproducible-builds|layer-acht).org
⢿⡄⠘⠷⠚⠋⠀  OpenPGP: B8BF54137B09D35CF026FE9D 091AB856069AAA1C
⠈⠳⣄
The law, in its majestic equality, forbids the rich as well as the poor to
sleep under bridges, to beg in the streets, and to steal bread. (Anatole France)

I'm sure, now with "debian 12" and the packages from notset.fr being in use, this can be achieved easily. So, a PR ./build-with-podman.sh (optimally reproducing the same hash) and a CI job that executes it would be very welcome. It is just our development resources on qubes-mirage-firewall won't cover to use podman, but we're happy to review pull requests that contribute it (and to avoid bitrot, the CI job is pretty important).

On Thu, Jul 06, 2023 at 01:56:22AM -0700, Hannes Mehnert wrote:

I'm sure, now with "debian 12" and the packages from notset.fr being in use,
this can be achieved easily.

sadly snapshot.notset.fr is down (and will not come back) and
snapshot.reproducible-builds.org is not yet up. :/
--
cheers,
Holger
⢀⣴⠾⠻⢶⣦
⣾⠁⢠⠒⠀⣿⡁  holger@(debian|reproducible-builds|layer-acht).org
⢿⡄⠘⠷⠚⠋⠀  OpenPGP: B8BF54137B09D35CF026FE9D 091AB856069AAA1C
⠈⠳⣄
🔥 - this is fine.


Diff tiles from Mapbox Studio with our own

This is based on the cool idea of @manuelroth to use visual regression testing for our tiles.
Its crazy how well those diffs work.
For every commit we compare our own OSM-Bright tiles with Mapbox OSM-Bright tiles.
The diff is uploaded to the gh-pages branch below diffs for every build.
Perhaps there is a smarter idea for that.

It can also be used manually and generates diffs and make comparing easy.

The diffs are so cool.
Mapbox Streets:

Openstreets:

Diff:

If we really want to take it to the next level we could actually use the tl utility to

scrape some tiles from mapbox with tl
scrape some tiles from our server with til
make the diffs of all of them
use tl again to package the diff into a new raster mbtiles
and have a crazy raster map!!! of the changes

It is visually appealing when done on raster tiles...
It makes much more sense to make the diffs on the vector tiles, because on raster you don't see half of the information and structure.
The diffs (ideally in Travis) were proposed in https://github.com/geometalab/osm2vectortiles/issues/25#issuecomment-147366671
We need to compare structure and information included in the sample of vector tiles.
Each vector tile can be dumped into JSON or other textual format - so diffs are then easier and more meaningful (but maybe not that visual appealing as your raster diffs). Dump does not need to contain shapes of the vectors - but all the structural metadata (layers, features, and values inside of the features).
In #25 was suggested to use the C++ "tileinfo" utility, but if you are not capable of compiling, feel free to use another parser (https://github.com/mapbox/awesome-vector-tiles#parsers--generators) in JavaScript or Python, ...
The purpose of the suggested debug viewer #12 made purely in JavaScript GL was exactly the visual tool  for exploring and comparing of the information present in the whole dataset (layers from TileJSON), and features in individual tiles (picking by tile or picking by feature) - without extra styling.
A tool which helps to visually see the differences between open-streets-v5 and mapbox-streets-v5 - or later on open-streets-v6 and mapbox-streets-v6.

Yes we should actually work on the vector tiles not the visual representation for QA in our project.
But I guess the visual representation is more helpful for other project, e.g. to ensure when switching to our source that the map still looks the same.
What is much better than scraping the tiles is generating a raster map.
I wrote it all up in a blog post because I guess it is helpful to other people as well.
http://lukasmartinelli.ch/gis/2015/10/16/creating-diff-maps-for-regression-testing.html
I will close down this pull request because everything is decribed in the blog post and
we can use the script from there to generate a webmap when we want to do QA.
For CI integration textual diffs are much more helpful.

BTW a similar raster verification is used in CI of OpenLayers (ol3) - for testing client side rendering differences between WebGL renderer and HTML Canvas renderer. You can check the Travis of OL3.


⚠ GitLab Server has degraded performance

In 5ba7fc8, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 13967 ms

Resolved: GitLab Server performance has improved in 733de5c.


Add install instructions for MSI and offline

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

Docs Build status updates of commit 7fd527b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-1.8.0/install-az-ps-msi.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-1.8.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-1.8.0/toc.yml
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-1.8.0/uninstall-az-ps.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-2.8.0/install-az-ps-msi.md
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-2.8.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-2.8.0/toc.yml
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-2.8.0/uninstall-az-ps.md
:white_check_mark:Succeeded
View (azps-2.8.0)

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.

Docs Build status updates of commit 026288f:
:clock10: Full build: the average full build time is 22 min(s) 27 sec(s), this is based on the last 241 build(s) for this repository.

Docs Build status updates of commit 026288f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-1.8.0/install-az-ps-msi.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-1.8.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-1.8.0/toc.yml
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-1.8.0/uninstall-az-ps.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-2.8.0/install-az-ps-msi.md
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-2.8.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-2.8.0/toc.yml
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-2.8.0/uninstall-az-ps.md
:white_check_mark:Succeeded
View (azps-2.8.0)

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.

Docs Build status updates of commit 7837900:
:clock10: Full build: the average full build time is 22 min(s) 26 sec(s), this is based on the last 242 build(s) for this repository.

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

File
Status
Preview URL
Details

docs-conceptual/azps-1.8.0/install-az-ps-msi.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-1.8.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-1.8.0/toc.yml
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-1.8.0/uninstall-az-ps.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-2.8.0/install-az-ps-msi.md
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-2.8.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-2.8.0/toc.yml
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-2.8.0/uninstall-az-ps.md
:white_check_mark:Succeeded
View (azps-2.8.0)

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.


Good Suggest

Name: Split rigid Smoother = 1
(0: Smooth split rigid speed than vanilla. (it slow when restore than 1), 1: Agar.io Vanilla rigid speed when restore)
Older Vanilla Agar.io have some more smoother rigid than now. (See the april version agar.io split run movie)
So, i suggest you.

No just no...

What do you mean by "rigid"?

It's not so clear what you're talking about, and you didn't provided more details, so I'm closing the issue


Sidebar Line History Box stopped working 9.6.0 -> 9.6.1

GitLens Version: 9.6.1 -> current
VSCode Version: 1.35.0
OS Version: Mac Mojave 10.14.5

Steps to Reproduce:

Clone any repo from github (I cloned mine)
With default settings the Line History box is always:

No line history could be found.

Hi! First of all thank you for such an amazing extension!
I just noticed that Line History box is not working in current version. Its not an annotation line history but the box. I reinstalled to previous versions one by one and I tracked that on 9.6.0 it is working but on 9.6.1 is not working any longer. Please take a look at screenshots and debug dumps. Suspicious is that on 9.6.0 git show :<FILE> command is executed but on 9.6.1 not..
Cheers!
Screenshots:

Debug Dumps:
9.6.0
Looking for git in: /usr/local/bin/git
Using git 2.22.0 from /usr/local/bin/git
> git rev-parse --show-toplevel
> git config --get commit.template
Open repository: /Users/kuba/projects/reactoricus.jakubniewczas.pl
> git status -z -u
> git symbolic-ref --short HEAD
> git rev-parse master
> git rev-parse --symbolic-full-name master@{u}
> git rev-list --left-right master...refs/remotes/origin/master
> git for-each-ref --format %(refname) %(objectname) --sort -committerdate
> git remote --verbose
> git check-ignore -v -z --stdin
> git show :src/frontend/lib/deep-copy.js

9.6.1
Looking for git in: /usr/local/bin/git
Using git 2.22.0 from /usr/local/bin/git
> git rev-parse --show-toplevel
> git config --get commit.template
Open repository: /Users/kuba/projects/reactoricus.jakubniewczas.pl
> git status -z -u
> git symbolic-ref --short HEAD
> git rev-parse master
> git rev-parse --symbolic-full-name master@{u}
> git rev-list --left-right master...refs/remotes/origin/master
> git for-each-ref --format %(refname) %(objectname) --sort -committerdate
> git remote --verbose
> git show :src/frontend/lib/deep-copy.js
> git check-ignore -v -z --stdin

@kubenstein You are only seeing this behavior when you have the last line in the file selected correct? If so, and if the last line in your file is just a newline, then this is expected behavior because git doesn't treat that last line as an actual line at all. It was technically a bug in 9.60 that was causing it to show up.

@eamodio screenshots are quite unfortunate because of new line indeed, sorry for that. However if I hover over other lines the box is still empty.
Take a look at this screenshot, its from other file, because this file has more commits. We can see that file history as well as inline line history work but not the sidebar box:

Hrm, can you send me full logs? I can't reproduce this:

Hi, I noticed a new version (9.8.2) was released yesterday and it fixes the issue!
so I think this issue can be closed.
thank you very much!


Does this work?

If so how?

Not working for now, refactor/fix required.

Traceback (most recent call last):
  File "/home/lix/.local/lib/python3.9/site-packages/requests/models.py", line 910, in json
    return complexjson.loads(self.text, **kwargs)
  File "/home/lix/.local/lib/python3.9/site-packages/simplejson/__init__.py", line 525, in loads
    return _default_decoder.decode(s)
  File "/home/lix/.local/lib/python3.9/site-packages/simplejson/decoder.py", line 370, in decode
    obj, end = self.raw_decode(s)
  File "/home/lix/.local/lib/python3.9/site-packages/simplejson/decoder.py", line 400, in raw_decode
    return self.scan_once(s, idx=_w(s, idx).end())
simplejson.errors.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/lix/anaconda3/lib/python3.9/site-packages/twitter_scraper/modules/tweets.py", line 170, in get_tweets
    yield from gen_tweets(pages)
  File "/home/lix/anaconda3/lib/python3.9/site-packages/twitter_scraper/modules/tweets.py", line 37, in gen_tweets
    html=r.json()["items_html"], url="bunk", default_encoding="utf-8"
  File "/home/lix/.local/lib/python3.9/site-packages/requests/models.py", line 917, in json
    raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: [Errno Expecting value] : 0

Still not working @nsde . I'm not sure will I fix it. PRs are welcome.

Still not working :(

You can try this https://github.com/mahrtayyab/tweety

do you know where the issue is? to get us started and save us time

@FarisHijazi I'm going to look into this a bit today. Any chance you found the start of the issue? The error linked by @nsde is the same one I'm experiencing.

Give it a try https://github.com/mahrtayyab/tweety

@mahrtayyab works for me - thanks!

I guess the first one, but I didn't debug it yet. I'm just predicting @ege-del

not working for me


SelectBehavior.insert merges completion with a following text

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Announcement

[X] I have checked Breaking change announcement.

Minimal reproducible full config
if has('vim_starting')
  set encoding=utf-8
endif
scriptencoding utf-8

if &compatible
  set nocompatible
endif

let s:plug_dir = expand('/tmp/plugged/vim-plug')
if !filereadable(s:plug_dir .. '/plug.vim')
  execute printf('!curl -fLo %s/autoload/plug.vim --create-dirs https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim', s:plug_dir)
end

execute 'set runtimepath+=' . s:plug_dir
call plug#begin(s:plug_dir)
Plug 'hrsh7th/nvim-cmp'
Plug 'hrsh7th/cmp-buffer'
Plug 'hrsh7th/cmp-nvim-lsp'
Plug 'hrsh7th/vim-vsnip'
Plug 'neovim/nvim-lspconfig'
call plug#end()
PlugInstall | quit

" Setup global configuration. More on configuration below.
lua << EOF
local cmp = require "cmp"
cmp.setup {
  snippet = {
    expand = function(args)
      vim.fn["vsnip#anonymous"](args.body)
    end,
  },

  mapping = {
    ['<Tab>'] = cmp.mapping.select_next_item({ behavior = cmp.SelectBehavior.insert }),
    ['<CR>'] = cmp.mapping.confirm({ select = true, behavior = cmp.ConfirmBehavior.insert })
  },

  sources = cmp.config.sources({
    { name = "nvim_lsp" },
    { name = "buffer" },
  }),
}
EOF

lua << EOF
local capabilities = require('cmp_nvim_lsp').default_capabilities()

require'lspconfig'.cssls.setup {
  capabilities = capabilities,
}
EOF

Description
When selecting an item with cmp.mapping.select_next_item({ behavior = cmp.SelectBehavior.insert }) and there is another word after the cursor position, some number of characters at the end of the selected item will not be inserted if the word after the cursor starts with these characters.
This is a problem for me, since I typically select a completion item without confirming it, and this 'merging' behavior creates a problem when e.g. adding a new argument to a call, and the completion item name ends in the same word as the next argument.
Steps to reproduce
nvim -u ~/cmp-repro.vim
iaaab<cr>
bbbb<cr>
bb<esc>0
ia<tab>

Expected behavior
The third line in the file should be 'aaab|bb' where | is the cursor position.
Actual behavior
The third line in the file is 'aaa|bb'.
Additional context
SelectBehavior.insert differs from ConfirmBehavior.insert in that when the completion is confirmed, the text after the cursor is not modified.
So this would produce the expected behavior:
nvim -u ~/cmp-repro.vim
iaaab<cr>
bbbb<cr>
bb<esc>0
ia<cr>

The code responsible for removing the trailing characters:
https://github.com/hrsh7th/nvim-cmp/blob/8f3c541407e691af6163e2447f3af1bd6e17f9a3/lua/cmp/entry.lua#L274-L282
And it was added in ef27b62.

I've actually noticed the same behavior when confirming with:
cmp.mapping.confirm({ select = false, behavior = cmp.ConfirmBehavior.insert })
Any solutions or workarounds for this?

Any solutions or workarounds for this?

You can manually remove this code: https://github.com/hrsh7th/nvim-cmp/blob/8f3c541407e691af6163e2447f3af1bd6e17f9a3/lua/cmp/entry.lua#L274-L282

Doesn't seem to fix my issue... Guess I'll need to make a self contained example to troubleshoot it.

I think my issue was due to: https://github.com/hrsh7th/nvim-cmp/issues/706


Add report task that shows bytecode coverage

Sometimes, it is very difficult to determine why a line is either partially covered or not covered at all
This is usually due to the fact that Kover reports on bytecode-level coverage, but then displays the coverage on top of the source code
With the ability to view which lines of bytecode are covered / not covered, it should be substantially easier to determine which paths through the code are actually untested
e.g. I'm currently trying to determine how the following coverage occurs:
when (response) { // fully covered
  is Success -> { // fully covered
    foo()         // fully covered
  }               // partially covered
  is Failure -> { // partially covered
    bar()         // fully covered
  }               // fully covered
}

It could be a bug with Kover, or it could be a weird path through the code that's been mapped strangely to the source code, or it could be something else - the ability to see bytecode coverage would allow this sort of thing to be diagnosed

Relates https://github.com/Kotlin/kotlinx-kover/issues/272


Error on clicking menu button: "Failed to load message bundle. See console for details."

When I'm live previewing my code and click on the menu button to see the options that should appear there, I instead get the error: "Failed to load message bundle. See console for details."

Version: 1.76.1

same problem with live preview 0.4.7 installed. Console shows:

Does this happen on the newest version of Live Preview?


Initiating roles and permissions

Hey,
on L5 where would you put this initiating roles and permissions file?
Do you create some database/rbac/rbac_init.php file for it?
I understand that later for new users I would integrate this to users management but for initial start do you create them in some init file?

Hello.
Well, sorry, but I don't understand you.
When you are starting, create you first roles and permissions through database seeds and attach permissions for example to a administrator role.
Then create administration panel, where you can attach (or detach of course) roles and permissions to a new users.

Ok, forgot about db seeds. Thanks for hint!


Bitdefender is missing in the right click context menu

Description
Hello,
It seems that my application is not present in the right click context menu.
default context menu:

Files context menu:

Steps To Reproduce
Use Bitdenfender total security
Steps to fix

eg. Fix crash that occurs when...

eg. Correct action happens when clicking...

Solution to add it

Files Version
Version : <IP_ADDRESS> OS Version: 10.0.19044.2364
Windows Version
Windows 10 21H2 19044.2364
Relevant Assets/Logs
debug.log

Can you please upload the debug.log file directly to GitHub (email attachments don't work) . You can get it in Files settings > About > Open log folder.

I add it

Also #2431, issue specific to bitdefender.

What are they doing differently?

@dahall do you have any ideas on fixing this?

Maybe the same as #8251?

Nope because the report is about Windows 10.

Maybe the same as https://github.com/files-community/Files/issues/8251?

Even though the Windows versions are different, I believe this all has to do with how the context menu is registered and how Files is pulling those from the registry.

A good way of checking is looking at explorer++. If they are able to show it then we should be able as well.

@UNDERNAUT can you check this out?

Hello,
It seems that explorer++ isn't able to show Bitdefender context menu.
Thank you for your help and sorry for waiting.

Sorry for the long delay, it appears that this issue is with Bitdefender and not Files. It's possible that they did this by design but it's hard to know for sure.


Update README for cross-platform Django project setup

This pull request addresses issue #512  by updating the README to include platform-specific instructions for creating a Django project in Docker.

https://github.com/docker/awesome-compose/issues/512#issuecomment-2558237102

It may be cleaner to remove sudo from the command entirely. The other documentation samples do not include it, nor do the other instances of docker compose [up/down] in that document.


Saudi Arabia

This is a Bug Report

Problem:
Proposed Solution:
Page to Update:
https://kubernetes.io/...

Hi @Abdualkreim,
Can you provide us with more information on the issue you're experiencing?
/triage needs-information

/close


Generate storageinsights

https://github.com/googleapis/googleapis/blob/f978bc05225b6c658980e9755021d08daddaac80/google/cloud/storageinsights/v1/storageinsights.proto#L39

This is now unblocked, the service is GA:
https://cloud.google.com/storage/docs/insights/inventory-reports


Add missing FileWidget import...

Coverage remained the same at 57.666% when pulling a56adf4ca99410d0a55cce689e5db17690462ac0 on addmissingfilewidgetimport into 671049e9c9a5f81aa39b204be70182606802cfdb on master.


Kestrel works very slow when it is operating under IIS

We are facing a strange problem with using Kestrel under IIS in which some operations (e.g. getting access CA- CertEnrolllib ) take about 100 sec processing time to be complete.
Please note that everything works well with using IIS Express and also http.sys (instead of IIS) where every operation takes maximum 5 seconds to be complete.

Here is the code for an action that works on the aforementioned library CertEnrolllib for certificate enrollment purpose.
    public class TestController : Controller
    {
        [HttpGet]
        public IActionResult Index()
        {
            var s = Stopwatch.StartNew();
            var type = Type.GetTypeFromProgID("X509Enrollment.CX509CertificateRequestPkcs10");
            var instance = Activator.CreateInstance(type);
            type.InvokeMember("InitializeFromTemplateName",
                BindingFlags.InvokeMethod,
                null,
                instance,
                new object[] { 1, "CertEnrollment" });

            s.Stop();
            return Content(s.ElapsedMilliseconds.ToString());
        }
    }

This issue was moved to aspnet/Home#3188


docker: Switch to inheriting from go.bbclass

Inherit from the Go class rather than depending on it as this breaks
the Pyro build.
Change-type: patch
Changelog-entry: Switch docker to inherit from Go class
Signed-off-by: Will Newton<EMAIL_ADDRESS>

What is the problem this PR should fix? How are inheriting and depending interchangeable?

@floion originally in my Pyro PR I had switched to inheriting the go.bbclass which amongst other things DEPENDS on the correct package to get a Go toolchain. However the hostapps PR would not build with that setup as we can see here:
https://jenkins.dev.resin.io/job/yocto-odroid-ux3/549/console
To workaround that the hostapps PR switched back to just a DEPENDS relationship. This then breaks the Pyro support. This PR brings back the inherit as I think it is the correct direction in the future but it is not mergeable until we get a fix for the build failure.
I opened a PR with a possible solution here:
https://github.com/resin-os/oe-meta-go/pull/1
All this stuff is way too ugly for my tastes but it doesn't seem like there are many alternatives.

So the error at https://jenkins.dev.resin.io/job/yocto-odroid-ux3/549/console is when using yocto pyro and using the go.bbclass from poky rather from oe-meta-go?

@floion that error is from using Jethro in that particular case. There is a go.bbclass in oe-meta-go but we didn't use it until I made a change recently. The hostapps PR undid that change in order to fix the error in https://jenkins.dev.resin.io/job/yocto-odroid-ux3/549/console which was introduced because there is now a docker-native which in turn wants a go-native and the go.bbclass from oe-meta-go is deficient in this area.

@agherzan I redid this patch in a way that seems to fix the problem but doesn't need any changes to oe-meta-go.


Parser for Plutus IR

Nobody is really supposed to write PIR textually, but there are two main exceptions to that:

Testing: it's helpful to be able to write examples literally rather than tediously constructing the AST
Debugging: if you print out the PIR for something it's nice to be able to feed that back in via a parser

Then if we did this we could also have a pir executable like the plc one that exposes parsing/typechecking/compiling which would be nice.

Done.


Use data-theme attribute for themes on root element

Use CSS color-scheme property
This sets light/dark theme on scrollbars and form elements.
Remove switch statement and directly apply theme in setTheme function
This change allows additional themes added by a plugin to work without attaching a new listener to #theme element; also prevents rechecking of localStorage to add the custom theme to body.classList.

This seems reasonable, but I'm not sure it's sufficient. If you add a new dark theme option, but the user's OS theme is set to light, code blocks in the rendered page will be rendered with the light theme when selecting your dark theme...
Is there any major benefit to using a data-theme attribute rather than keeping with classes? Doing this switch is a breaking change that doesn't seem necessary.

Based on my testing, the code blocks will match the selected theme. https://typedoc-theme-test-shane4368.surge.sh/classes/Bird.html#beginFlight
Using classes for themes would require additional logic to replace the current theme with the selected theme.
<html class="default no-js not-mobile toggle-visibilityprivate toggle-inherited dark">

Using the above as reference, if you wanted to change theme, you'd have to do html.classList.remove("dark") then html.classList.add("selected-theme"). But "dark" could be any value; you'd have to track the current theme or html.classList.remove(localStorage.getItem("tsd-theme")).
body.light { color-scheme: light; } also doesn't override if it's set on the root element, resulting in dark scrollbar on light theme if OS theme is dark.
If it's a breaking change, you may add this in 0.23.

Tracking the current theme doesn't seem too bad to me - at least until 0.23, where the data attribute is probably better going forward. I didn't know about the colored scrollbars possibility - that's neat, and a really compelling reason to move to :root
Just a note: The hardcoded hl-1 through hl-7 might work today, but there's no guarantee that it works in the future. Those classes are completely dependent on the themes the user chooses. If someone writes a theme that has 50 different colors, it'll behave badly. I'd really rather give plugins like yours a way to handle this nicely. Maybe themes in the dropdown could specify two values?

The hl colours being out of my control was an oversight.

Maybe themes in the dropdown could specify two values?

Like setting a custom hl theme before the html gets written to file, if it's supported? I'm okay with that.

I was thinking of making values contain two items, e.g. hl-light light and hl-dark dark, then your themes would specify hl-light custom for example.
Unfortunately supporting custom highlighter themes for each scheme is expensive, adding ~400ms per 1000 lines of code highlighted for each theme (https://github.com/shikijs/shiki/issues/234)

Oh. That should be fine.


Migrate all Core libraries to new Checkstyle and Spotbugs design

Description
Migrates all Core libraries to the new Checkstyle and Spotbugs design, and re-enables Spotbugs.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Add cupy_backends/cuda/libs/cutensor.pxd

This PR adds cupy_backends/cuda/libs/cutensor.pxd and cythonizes cupy.cutensor.
Further performance improvement will be worked in another PR: #3700

@kmaehashi PTAL!

Jenkins, test this please.

Jenkins CI test (for commit 1590d3031e53c08200ace9f1f386ea08349a5beb, target branch master) succeeded!

LGTM except comment.

LGTM!
pfnCI, test this please.

Jenkins CI test (for commit eb60ce89bbf1942ff4a17912673b47d1be29e5ab, target branch master) succeeded!


ci: update token config for security

:rocket: PR was released in<EMAIL_ADDRESS>:rocket:


Fix crash in allfam10c report

The report allfam10c crashes with the following error
message if it encounters a family with an unknown
husband and/or wife:
Runtime Error at line 54: fullname: the arg #1 must be
a person.
The attached patch fixes this problem by not calling
fullname() for missing individuals.
Reported by: akirschbaum
Original Ticket: "patches/24":https://sourceforge.net/p/lifelines/patches/24/

I decide to make fullname() more robust and handle being called in situations like fullname(husb(fam)) where the FAM record does not have a HUSB node.  This is treated the same as if an INDI doesn't have a NAME record, or the NAME record is empty, and produces an empty string for output.

Fixed via https://github.com/MarcNo/lifelines/commit/d50132c01c349f71ef21102d0a63199d080df3b8


Escaping path params in docs

Escaping some unescaped path params in API docs that caused the paths to be rendered incorrectly (the params were missing).

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"


Add "cons of waterfall process".

agile is contrast to waterfall

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

good luck

@PatiPavanSai Thanks for this pull request (PR).
Unfortunately, we are marking this PR invalid for hacktoberfest. We believe it does not follow the recommended guidelines:

Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we have provided examples of the quality >standards we encourage. This applies mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove whitespace / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs. helping.
Something that's clearly an attempt to simply +1 your PR count for October.

Learn more at https://hacktoberfest.digitalocean.com/details


Feature Request - Customizable IP address (VPN)

Love the new features in 2.2.
Noticed that when deploying that new Network InfoItem when deployed to a Mac that is using a VPN client it is showing the localized address versus the VPN Client Address. We have a zsh command that can pull the accurate IP address every time. There does not seem to be an InfoItemType preference to allow for a command. Requesting a way to use said command to display the IP address we'd want in that ItemInfo panel.
Thank you again!

Thanks @chastinker!
Interesting use case! We have put in on the backlog and will look into it

Thanks for the great app!
We would also appreciate the VPN IP address view. Thank you very much.

Is there a release date here yet? :)

Yes, you'll be able to do this in the upcoming release. We're release this in the next month(s).

Yes, you'll be able to do this in the upcoming release. We're release this in the next month(s).

That sounds great! We think the support app is totally awesome and would like to see the feature requests already made. Can you tell us which requests you would like to implement? Thank you very much!

Please look into the new Support App Extension possibilities to show any information you can get using scripts. The new SupportHelper may be helpful here to fetch new info on every appearance of the app. We'll mark this issue as closed.


Move VC++ Build Tools to Build Tools landing page

We've (finally!) created an actual landing page for the VC++ Build Tools. We'd like your readme to refer people to the landing page rather than directly to the download. This change moves the target of that link.

I get a 404 when I open http://landinghub.visualstudio.com/visual-cpp-build-tool.

I deleted an 's' at the end of the URL. It's "tools". Sorry, I'll fix that in an hour or so.

The new URL seems correct. LGTM

Landed in af35b2a, thanks Andrew.


Add benchmark

current benchark results for merging 1500x1500 non-conflicting and conflicting splices
The number rapidly becomes unbearable, so this is somewhat of an upper limit. The real upper limits on browsers and such may be much lower but this does give confidence that low hundreds of changes can be merged relatively fast.
goos: darwin
goarch: amd64
pkg: github.com/dotchain/dot
BenchmarkMergeNonConflicting-4<PHONE_NUMBER>	         0.25 ns/op
BenchmarkMergeConflicting-4<PHONE_NUMBER>	         0.33 ns/op
PASS
ok  	github.com/dotchain/dot	29.179s

Codecov Report

Merging #291 into master will not change coverage.
The diff coverage is n/a.

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


fixing title, fixing url, hiding page, fixing broken md

Fixes
Checklist

[ ] I have made a material change to the repo (functionality, testing, spelling, grammar)
[ ] I have read the [Contribution Guide] and my PR follows them.
[ ] I updated my branch with the master branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation about the functionality in the appropriate .md file
[ ] I have added in line documentation to the code I modified

Short description of what this PR does:

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

Hello @ksigler7,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Change camera backgorund color

Camera background color to RGBA 0,0,0,0. But "Apply HoloLens Scene Settings" is black(0,0,0,1)
It was changed from black to clear.
https://developer.microsoft.com/en-us/windows/holographic/unity_development_overview#configuring_a_unity_project_for_hololens

Hi @kaorun55, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


Support git blame on rendered markdown

Description
This would be helpful when I notice an unexpected line while reading the rendered markdown. If I click on the Blame button then I'm switched to the code mode. It breaks the flow as I need to find the position of the unexpected line again, which is made hard by the fact that it's hard to visually navigate in markdown code compared to rendered markdown.
Example URLs
https://github.com/qwerty-fr/qwerty-fr/blame/master/README.md

There's no easy way to match the DOM to a line in Markdown, especially because one Markdown line could have several source lines, each of which could have its own commit.
Unfortunately this isn't the kind of complexity Refined GitHub can tackle.


nixos/nginx: should ensure /var/cache/nginx and /var/log/nginx

NixOS nginx module does not create /var/cache/nginx and /var/log/nginx directories, but when the directories do not exist, nginx service fails to start
nginx: [emerg] mkdir() "/var/cache/nginx/client_body" failed (2: No such file or directory)

Test suite explicitly creates the directories using mkdir -p
https://github.com/NixOS/nixpkgs/blob/11c18faa4e4ab42bcb926e45b89811eb74be411d/nixos/tests/service-runner.nix#L26
and so every nginx user has to do
It would be nice to of nixos module ensures the directories before starting nginx (using mkdir -p or systemd.tmpfiles.rules)

Maybe there is some kind of regression in systemd?
It should create those directories:
https://github.com/NixOS/nixpkgs/blob/22e8ada3b3c5fe6d33da9bf8e4d73e3be9fdc993/nixos/modules/services/web-servers/nginx/default.nix#L721
https://github.com/NixOS/nixpkgs/blob/22e8ada3b3c5fe6d33da9bf8e4d73e3be9fdc993/nixos/modules/services/web-servers/nginx/default.nix#L724

Probably I have systemd from 20.03 (running since last reboot) and
nginx module after the change.
On 9/6/20, misuzu<EMAIL_ADDRESS>wrote:

Maybe there is some kind of regression in systemd?
It should create those directories:
https://github.com/NixOS/nixpkgs/blob/22e8ada3b3c5fe6d33da9bf8e4d73e3be9fdc993/nixos/modules/services/web-servers/nginx/default.nix#L721
https://github.com/NixOS/nixpkgs/blob/22e8ada3b3c5fe6d33da9bf8e4d73e3be9fdc993/nixos/modules/services/web-servers/nginx/default.nix#L724
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixpkgs/issues/96052#issuecomment-687741414


Update http4s-client, http4s-dsl, ... to 0.21.15

Updates

org.http4s:http4s-client
org.http4s:http4s-dsl
org.http4s:http4s-okhttp-client

from 0.21.7 to 0.21.15.
GitHub Release Notes - Version Diff
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.http4s" } ]

labels: library-update, semver-patch

Codecov Report

Merging #180 (19120b9) into master (e31fc88) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #180   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          24      24
  Lines         234     234
  Branches        9       9
======================================
  Misses        234     234

Continue to review full report at Codecov.

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

Codecov Report

Merging #180 (19120b9) into master (e31fc88) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #180   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          24      24
  Lines         234     234
  Branches        9       9
======================================
  Misses        234     234

Continue to review full report at Codecov.

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

Superseded by #181.

Superseded by #181.


Switch proxy from Traefik to Caddy

This is feature complete with Traefik, sans the dashboard that Traefik comes with. Caddy does not have a dashboard, but it does expose Prometheus metrics.
Caddy's admin api that allows configuration is not enabled by default for security reasons.
Documentation has been updated for the new Caddy fields in place of the Traefik ones.
Signed-off-by: Robert Detjens<EMAIL_ADDRESS>

This was supposed to stay on our fork, whoops! Please ignore,


schema-gen() returns different results for xml and json when an entity type has duplicate property name

This issue was found when testing #212 .
Sample doc: Attached a zip file
Query:
import module namespace es = 'http://marklogic.com/entity-services' at '/MarkLogic/entity-services/entity-services.xqy';
es:schema-generate( es:model-validate(doc('invalid-bug43212.json'))),
es:schema-generate( es:model-from-xml( fn:doc( 'invalid-bug43212.xml')))
Test Input file.zip
Test Input file.zip
Issue:
With JSON input file schema-gen() returns an additional comment as below:
<!--
XSD schemas prohibit duplicate element names. This element is commented out because it conflicts with another of the same name.
 <xs:element name="OrderID" type="xs:string" xmlns:xs="http://www.w3.org/2001/XMLSchema"/>
-->

NOTE to self: Uncomment schema-gen() test for json input file once bug is fixed.

Duplicate properties are confusing not just for entity services -- I don't think we should try to do anything here.
I would have thought so, but try this:
let $a := json:object()=>map:with("a", "b")=>map:with("a", "c")

let $n1 := xdmp:unquote('{"a":"b", "a": "c"}')
let $n2 := object-node { "a" : "b", "a": "c"}
let $b := xdmp:from-json($n1)
let $c := xdmp:from-json($n2)

return ($a, $b, $c, $n1, $n2)

It seems JSON nodes can have duplicate keys which is fine, but if you make a json:object out of them, then each key will ever only have one value, even if it looks like there are two items in the map.

not entirely clear what to do for this bug, and it's impact is minimal, so set to 9.0-2 to provide stability for release target.

I discussed this issue with some members of the team and we agree it's in the 'garbage in garbage out' class of bugs.  We cannot account for the foibles of map/json implementation on the server.


[FEATURE] Hide/Obfuscate IP in logs

Issue:
Hey there! 👋 I'm currently running favonia/cloudflare-ddns:1.11.0 container on my publicly accessible ArgoCD instance, and the logs display IP addresses. To amp up privacy (especially since it's behind a Cloudflare proxy), I'm suggesting a feature to hide/obfuscate those IPs.
Details:
I'm thinking a simple config option in settings would do the trick. An environment variable like HIDE_IP. Maybe offer methods like replacing the last octet with "XXX" or hashing the IP. Personally, I think IP hashing is preferable in order to maintain observability of IP changes.
🌐 Detected the IPvX address: <SHA_256_IP_HASH>

Note:
Currently loving the tool! 🙌 Thanks for maintaining it. Your work is much appreciated! 🚀

@joaquinrovira Hi, I'm sorry if my intimidating (?) long comment accidentally shut down the conversation. Please feel free to add anything that you might find useful. Thanks! I am eager to figure out what should be changed to the tool (and implement them probably this summer :star_struck:).

Sorry for the late response.

I do want to understand more about the use cases, though, because IP addresses are not very "private," so to speak.

I deploy stuff in my homelab using ArgoCD.  My servers are behind Cloudflare so my IP is AFAIK not trivially exposed. The issue is that ArgoCD dashboard also gives access to the cloudflare-ddns pod logs. Not terrible, but no ideal. So I was wondering if we could obfuscate those IPs somehow.

The other part is, even if your IP is hidden, there are numerous bots scanning vulnerable servers, proxied or not. Overall, the internet is never designed to keep your IP a secret.

Feel free to close the issue if you deem this is to be unnecessary. The idea is to make it slightly harder to get the IP, not impossible.

[...] easy to invert the hashing by enumerating plausible IPs, that is, a dictionary attack. In the case of IPv4, you can enumerate all possible addresses in no time.

This could be solved by salting before hashing. Maybe another env var or just random bytes.

The last thing I want to point out is that I wonder if you want HIDE_IP to affect messages sent to Healthchecks, Uptime Kuma, and/or the shoutrrr support I am currently adding (still a work in progress).

I have not looked into the effects outside my use case. No clue regarding this.
I do not have the disposable time right now but if I do I could try to push a proposal PR.

Just wanted to add that the program has been running flawlessly for several months now. Thanks again for the effort of building and maintaining this project.

I deploy stuff in my homelab using ArgoCD.  My servers are behind Cloudflare so my IP is AFAIK not trivially exposed. The issue is that ArgoCD dashboard also gives access to the cloudflare-ddns pod logs. Not terrible, but no ideal. So I was wondering if we could obfuscate those IPs somehow.

I see... should we just remove the IPs from the logging, then?

Either one would work. Hashing allows to easily see when the underlying value changes. However, simply hiding would be enough. I would not want to add more complexity than needed.

@joaquinrovira Should we hide Cloudflare record IDs as well?

Current design

LOG_OBFUSCATION=1
Meaning: whether the logging is obfuscated enough so that the log can be viewed by the public without fear.
Actual effects: zone IDs, record IDs, custom URLs, IPs are hidden
Known bugs: Go standard library will sometimes expose the full URLs that should be secret
Exceptions: This does not affect messages to Healthchecks.io, Uptime Kuma, and shoutrrr (still being implemented)

The more refined version would be
LOG_OBFUSCATION=ip,timezone

This proposal would be much more extensive the initial scope. It certainly covers my needs and probably anyone else public logs. (If there is anyone... 😅).

Update: I think the proposal should probably be called LOG_REDACTION because we are planning to hide, not obfuscate, private information.

@joaquinrovira I am thinking about these five kinds of "private" information:

Tokens (token)
IPs (ip)
Domains (domain)
UUID (of records and zones) (uuid)
Timezone (timezone)

By default only tokens are hidden. I think you want token,ip in your case. The special value none (or maybe min) shows everything and max hides all.

Will do! Thank you very much. 😃
It will take me a couple of days at least as I'm AFK this week.

@joaquinrovira I haven't implemented the feature yet! Subscribe to #785 for my progress.
PS: I felt the timezone information might be too difficult to hide due to various side channels, giving up hiding it :upside_down_face:

@joaquinrovira I have a design problem now: none of the libraries I use (including the Go standard library) were designed with obfuscation in mind, and they often generate very detailed error messages containing "private" information. To 100% block the leakage of IP addresses or other information, error messages outside my control cannot be shown at all, which could make debugging very difficult if something goes wrong.
I thus wonder if there's another way to solve your problem. For example, what if you redirect the logging into a file?

I am okay with errors showing this kind of information. I have not seen any errors while running the application yet.
Furthermore, if errors are logged to stderr one can always redirect the output with2>/var/log/ddns.log.

@joaquinrovira I'm currently going back to the drawing board because implementing this is more challenging than I thought---partially because of the elaborate system to generate various messages. I have a counterproposal: would it be easier to redirect everything into a file with a new configuration? The current coding makes it trivial to redirect logging (originally for testing message formation). Such as
LOG_OUTPUT=/path/to/log

where the special value - means the standard output. I understand that the downside is you could not view the log directly in the ArgoCD dashboard.
By the way, it might not make much sense to send only errors to stderr because there is no inherent difference between errors and non-errors. The separation is perhaps more meaningful when a program is part of a pipeline to handle data; its standard output would be the input to the next program.