CACIB wish to run a sealer node (Andina sealer)

Company

Full legal name: Credit Agricole CIB
Company website: www.ca-cib.com
Representative contact name: Guénolé de Cadoudal
Representative contact email<EMAIL_ADDRESS>
I hereby recognize that our company does not already have a Node in this network
Sealer Node

Sealer address: <0xF4358290290E67bDB8ae3457BEEa659302EaE358> (ensure you have the ownership and safekeeping of the private key)
TBD

Motivations
CACIB, with SEB, are launching this Proof of Carbon Reduction network as a proposal to establish a public ethereum based network that incentivize virtuous environmental behaviour

Hello @rghoshfinaxys, I have allowed your this node as sealer

@rghoshfinaxys I have removed this node because it didn't synchronize and therefore the consensus stale

@guenoledc it is synchronizing now, could you add it back again?
I have taken the liberty of proposing it though clique from the cachapa node

I can't anymore
The bootstrap node is out of the consensus as planned

@guenoledc @rghoshfinaxys I have placed a vote for it from our sealer already in the network. Should be approved now.


suit sensor / device network performance issue

On the recent stress test suit sensors is stupidly expensive (over half of tick time was on it!) and scales poorly for many players. Not entirely sure why it's happening but Mirror was investigating.
Ideally suit sensors would only send payloads whenever anything besides map pos changed (instead of an update loop) + the other funnies going on is found out (e.g. if it's broadcasting on the same frequency as everything else or something).

AFAICT one of the problem is that ConnectionsForFrequency will iterate over all devices on a network (e.g. all wired or all wireless devices) every time any message is sent. Instead of a FindAll(connection => connection.Frequency == frequency) it should probably be a dictionary or something.
But the bigger problem is probably that they do all just use the default frequency, so the cost of the updates goes like N^2, because every device on the network will raise at least two events at every other device on the network, which also results in many wireless range checks.
Given that nothing stops a device from sending on a frequency that differs from the one it is receiving on, the medical monitors should probably just be put on a separate frequency from the sensors. And to prevent stuff like this in future, the default frequency (0) should probably just be considered invalid? Would also make it easy to disable receive capability, which (so far) is completely unused by things like suit sensors. That would also prevent people from killing the server by finding and pinging the sensor frequency, or the frequency of whatever other high-quantity send-only devices get added.

I'll implement your suggestions @ElectroJr.
Preventing entities from receiving any device messages when you only want to send is a good idea.


called Option::unwrap() on a None value

Hi Prisma Team! My Prisma Client just crashed. This is the report:
Versions

Name
Version

Node
v16.15.1

OS
windows

Prisma Client
4.2.1

Query Engine
2920a97877e12e055c1333079b8d19cee7f33826

Database
sqlserver

Logs
t dirname Cnode_modules\.prisma\client
prisma:client relativePath ..\..\..\prisma
prisma:client cwd C:\prisma
prisma:client clientVersion 4.2.1
prisma:client clientEngineType library
prisma:client:libraryEngine internalSetup
prisma:tryLoadEnv Environment variables loaded \.env
prisma:client clientVersion 4.2.1
prisma:client clientEngineType library
prisma:client:libraryEngine internalSetup

prisma:client clientVersion 4.2.1
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true

Client Snippet
// PLEASE FILL YOUR CODE SNIPPET HERE

Schema
// PLEASE ADD YOUR SCHEMA HERE IF POSSIBLE

Prisma Engine Query
{"X":{}}

@jdmelendez thanks for the error. Could you provide the query and the schema you used to create this panic. That will help us solve this issue.

Ping @jdmelendez

Can you help us out here @jdmelendez? With the current information we unfortunately will need to close the issue as it is not enough to understand what is going on and reproduce it. Thanks!

Closing then until someone provides more information. Please comment or open a new issue. Thanks!


Consider implementing RISE/Reveal slides in notebooks

In a course I teach, for several years, I've used the RISE/Reveal Jupyter extension to turn my Jupyter notebooks into slides. Perhaps you'd want to consider doing something similar here? Basically, it makes it so you can present cell(s) from your Jupyter notebook as slides (big screen, big font) while running live Python/etc on the slides, showing and rotating molecules, whatever.
This solves the problem of "this Jupyter notebook is too small to read on when projected on a screen in this classroom so I have to zoom way in"; basically, you get the normal Jupyter view when you DON'T activate RISE and it switches to slide mode whenver you do. FOr me it's been magical/a game-changer.
(If you've already tried it and ruled it out, feel free to close this issue, but it works great for me.)

@davidlmobley thanks for pointing us to this. I think it is a great feature, we just never ended up including it!
@hamzaibrahim21 can you please incorporate this.


fix JsonToStringStyle

except number types and boolean type , all other types in java (char, String, Date etc.) should be appendAsString.
add method appendDetail(StringBuffer buffer, String fieldName, char value) to JsonToStringStyle
modify appendDetail(StringBuffer buffer, String fieldName, Object value)

Coverage decreased (-0.01%) to 93.83% when pulling ed0fea128ad83d6c07367eea7bf2e24116fa5723 on jacktan1991:branch-3.4-fix-json-style into 4bf7ca0a56b5868ca6dc151404d15a7f7ce24e26 on apache:trunk.

I just modified two java files, I don't konw why Failed.
https://github.com/apache/commons-lang/pull/67/files

@jacktan1991 Commons Lang has been migrated from svn to git. Because of this the main branch has been renamed to 'master'. Can you please rebase your work against the master branch? Thank you!


[Question] I want to build a Homelab with harvester and TinyMiniMicro PCs, anything worth mentioning?

I'd get couple TinyMiniMicro PCs ( https://www.servethehome.com/introducing-project-tinyminimicro-home-lab-revolution/ ) and install  harvester on them.
How should I go about it? just download iso and install?
Is it possible to remotely shutdown or put these PC nodes to sleep and wake or boot them? Would I need Intel vPro for it?
---edit
currently working with TrueNAS Scale and Proxmoxx. Future looks like Harvester and unraid (for tiered Cached and HDD spin down)

harvester hardware requirements: https://github.com/harvester/harvester#hardware-requirements

"8 cores" is bit difficult for TinyMiniMicro PCs.
Used WOL on Proxmox, was hoping for something native on harvester


program too large with commit 12d0d

i just rebased on head of the repo (currently at 83689) and found out my program no longer fits the max size due to commit 12d0d. has anyone any recommendation on how to adjust this commit or a local change i could use to overcome this issue and still be able to stay aligned with the latest and greatest version?

Have you tried adding -Os to CFLAGS?

i just dit, with this :
CFLAGS:=$(filter-out -O2,$(CFLAGS))
CFLAGS += -Os
however, the error is still reported:
section .text' will not fit in region iram1_0_seg'
On Sep 30, 2016 5:30 AM, "Vlad Ivanov"<EMAIL_ADDRESS>wrote:

Have you tried adding -Os to CFLAGS?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SuperHouse/esp-open-rtos/issues/225#issuecomment-250732475,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATZ3zOPQHi7OIO-PPTK_PaQPHB645G7zks5qvQDmgaJpZM4KBFd1
.

Most functions should be placed into flash by default, do you explicitly specify any function attributes in your code to force functions into IRAM?

Nope. All functions run in flash. What puzzles me is what in this commit
(12d0d) is causing this error. Once reverted, it builds fine
On Oct 1, 2016 3:51 AM, "Vlad Ivanov"<EMAIL_ADDRESS>wrote:

Most functions should be placed into flash by default, do you explicitly
specify any function attributes in your code to force functions into IRAM?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SuperHouse/esp-open-rtos/issues/225#issuecomment-250905981,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATZ3zHxXEA0oBF4L30TCSSJCTZuzitjiks5qvjsZgaJpZM4KBFd1
.

This commit increases data memory usage by a few bytes, maybe that's the reason? Try building any of example apps (with Os), and maybe make clean in your app to rebuild everything with optimization.

Out of curiosity, did you figure it out?

No, I didn't. I went back to the code changes on that commit, and can't
really spot what would cause such an error.
I had try to build the examples , but that didn't help me with the issue.
I'll conduct some more experiment next week and will post the results.


[Bug] cursorWordPart[Left/Right] navigations doesn't work for text like foo1Bar

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: v1.60.0-insider
OS Version: Windows 10

Steps to Reproduce:

Enter foo1Bar in the editor.
Put cursor at the beginning.
Use cursorWordPartRight.
Cursor goes to the end.

Expected behavior:
The cursor stops just before Bar.

I don't know the semantics of cursorWordPartRight - why do you think this is a bug?
I think this feature should be documented (see #131429).

Fixed via https://github.com/microsoft/vscode/pull/147074


Minimize to tray\Minimize to tray instead of closing feature

I don't rememder how but previous version did this (probably with 3rd party soft), native would be good

Do you mean this option? This will make it go in the tray instead of closing

Do you mean this option? This will make it go in the tray instead of closing
Yes thanks, thought it would be in the general settings


[ci] Reduce GPU mem_util for KVCache for DBRX model

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Should be fixed by https://github.com/deepjavalibrary/djl-serving/pull/1949


Can't customize visualizer project!

I installed the paraview-5.4.1 in my windows10 system and then
I pulled the visualizer github repository and just trying to update the header name or removing it in the src\MainView.js#163. and build with npm run build /  npm run build:release commands
and copy paste into \share\paraview-5.4\web\visualizer\www directory and then launch the launcher.bat then hit the  http://paraview/?data=can.ex2 but the changes won't be reflected.

You should just use the latest ParaView binary with both the /server part of your Visualizer repo and /dist as www.

Thanks for the Answer, one thing i want to know i'm using paraview 5.4.1 and visualizer latest (master branch) will this versions may causing ?

Using 5.4 will definitely be an issue with Visualizer/master.
You can see the compatibility list here

I tried visualizer-2.1.4 with paraview 5.4 , i cloned visualizer-2.1.4 version of repo and try to run npm install command but it failing with error :
You need to install the latest version of Visual Studio
including the "Desktop development with C++" workload.
For more information consult the documentation at:
https://github.com/nodejs/node-gyp#on-windows .
so, as maintained link , i ran the  npm install --global windows-build-tools command . This is also failing with error :
Could not install Visual Studio Build Tools.
i have installed microsoft visual c++ 2015-19 redistributable package (x64) 14.28.29325.
my question is if i have above redistributable  installed then why npm install failing?


wont install

windows 10
install from package control
reloading settings Packages/User/Package Control.sublime-settings
Package Control: The dependency 'bs4' is not currently installed; installing...
Package Control: The dependency 'bs4' is not available
Package Control: The dependency 'bs4' could not be installed or updated
reloading settings Packages/User/Preferences.sublime-settings

shrug

It should work now, could you try again?


Don't use Durations for calculating relative times

This PR closes https://github.com/uutils/coreutils/issues/6629.
There were two problems with parse_relative_time, which have been fixed by this PR.
One problem was that parse_relative_time() got the current date using Utc::now().date_naive() while parse_relative_time_at_date() got the current date using Local::now().date_naive(). This means that the date in the end result will be 1 day off if the date in your timezone is different from the date in UTC.
Another problem was that parse_relative_time_at_date() created a chrono::Duration. Unfortunately, Duration doesn't understand months and years, only seconds and nanoseconds. Therefore, the old code would assume every month was 30 days and every year was 365 days. This meant stuff like touch -d "month" wouldn't work in February and touch -d "year" wouldn't work in 2024.
I've now made it so that parse_relative_time_at_date returns a DateTime rather than a Duration. Within that function, checked_(add/sub)_(days/months/signed) is used to add days, months, years, whatever while being aware of the fact that months and years can be different sizes. parse_datetime_at_date() in lib.rs now directly calls this function, so there's no need to calculate a Duration at all.
The old parse_relative_time function has now been relegated to being a helper for the relative time parsing tests.
Testing
I haven't run all the uutils/coreutils tests yet, but I did run the following locally (I ran it between UTC midnight and my midnight). The yesterday bug mentioned in https://github.com/uutils/coreutils/issues/6629 were not triggered.
> cargo run -p uu_touch -- -d "yesterday" foobarbaz.txt
...(cargo output omitted, there was a deprecation warning)
> stat foobarbaz.txt
  File: foobarbaz.txt
  Size: 0               Blocks: 0          IO Block: 4096   regular empty file
Device: 8,32    Inode: 317960      Links: 1
Access: (0644/-rw-r--r--)  Uid: ( 1000/ysthakur)   Gid: ( 1000/ysthakur)
Access: 2024-08-26 22:20:47.096649101 -0400
Modify: 2024-08-26 22:20:47.096649101 -0400
Change: 2024-08-27 22:20:47.092271897 -0400
 Birth: 2024-08-27 21:51:57.632730345 -0400
> touch -d yesterday foobarbaz.txt
> stat foobarbaz.txt
  File: foobarbaz.txt
  Size: 0               Blocks: 0          IO Block: 4096   regular empty file
Device: 8,32    Inode: 317960      Links: 1
Access: (0644/-rw-r--r--)  Uid: ( 1000/ysthakur)   Gid: ( 1000/ysthakur)
Access: 2024-08-26 22:20:57.214983299 -0400
Modify: 2024-08-26 22:20:57.214983299 -0400
Change: 2024-08-27 22:20:57.202269430 -0400
 Birth: 2024-08-27 21:51:57.632730345 -0400

And here it is for -d "month":
> cargo run -p uu_touch -- -d "month" foobarbaz.txt
...(cargo output omitted again)
> stat foobarbaz.txt
  File: foobarbaz.txt
  Size: 0               Blocks: 0          IO Block: 4096   regular empty file
Device: 8,32    Inode: 317960      Links: 1
Access: (0644/-rw-r--r--)  Uid: ( 1000/ysthakur)   Gid: ( 1000/ysthakur)
Access: 2024-09-27 22:22:21.913646930 -0400
Modify: 2024-09-27 22:22:21.913646930 -0400
Change: 2024-08-27 22:22:21.912248547 -0400
 Birth: 2024-08-27 21:51:57.632730345 -0400
> touch -d month foobarbaz.txt
> stat foobarbaz.txt
  File: foobarbaz.txt
  Size: 0               Blocks: 0          IO Block: 4096   regular empty file
Device: 8,32    Inode: 317960      Links: 1
Access: (0644/-rw-r--r--)  Uid: ( 1000/ysthakur)   Gid: ( 1000/ysthakur)
Access: 2024-09-27 22:22:33.164133827 -0400
Modify: 2024-09-27 22:22:33.164133827 -0400
Change: 2024-08-27 22:22:33.152245574 -0400
 Birth: 2024-08-27 21:51:57.632730345 -0400

@cakebaker Could you take a look at this if you have time?

Good work, thanks for your PR!


Add try-excepts around fragile openai code

Overview
In order to provide a safety-net for our us and our customers, add try-except around fragile parsing code that could cause a crash.
Note this is based on https://github.com/newrelic/newrelic-python-agent/pull/1096 which should be reviewed and merged first.

Codecov Report
Attention: Patch coverage is 80.12821% with 31 lines in your changes are missing coverage. Please review.

Project coverage is 81.80%. Comparing base (b950070) to head (df73941).
Report is 20 commits behind head on develop-ai-limited-preview-3.

Files
Patch %
Lines

newrelic/hooks/mlmodel_openai.py
80.12%
22 Missing and 9 partials :warning:

Additional details and impacted files
@@                       Coverage Diff                        @@
##           develop-ai-limited-preview-3    #1101      +/-   ##
================================================================
- Coverage                         81.86%   81.80%   -0.06%
================================================================
  Files                               192      192
  Lines                             21253    21294      +41
  Branches                           3711     3711
================================================================
+ Hits                              17398    17419      +21
- Misses                             2783     2798      +15
- Partials                           1072     1077       +5

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

Pre-approving this PR!


No matching distribution found for mxnet when install mxnet everywhere

Description
(Brief description of the problem in no more than 2 sentences.)
I use pip install mxnet -v but everywhere it can't be install
every whl package have the same message: it is not compatible with this Python
Environment info (Required)
My computer:
----------Python Info----------
Version      : 3.6.5
Compiler     : GCC 4.2.1 Compatible Apple LLVM 9.1.0 (clang-9<IP_ADDRESS>)
Build        : ('default', 'May 11 2018 11:40:06')
Arch         : ('64bit', '')
------------Pip Info-----------
Version      : 10.0.1
Directory    : /Users/zhutou/Documents/数据/搜索数据分析/.env/lib/python3.6/site-packages/pip
----------MXNet Info-----------
No MXNet installed.
----------System Info----------
Platform     : Darwin-17.5.0-x86_64-i386-64bit
system       : Darwin
node         : zhutoudeMacBook-Pro.local
release      : 17.5.0
version      : Darwin Kernel Version 17.5.0: Fri Apr 13 19:32:32 PDT 2018; root:xnu-4570.51.2~1/RELEASE_X86_64
----------Hardware Info----------
machine      : x86_64
processor    : i386
My server:
----------Python Info----------
('Version      :', '2.7.12')
('Compiler     :', 'GCC 5.4.0 20160609')
('Build        :', ('default', 'Dec  4 2017 14:50:18'))
('Arch         :', ('32bit', 'ELF'))
------------Pip Info-----------
('Version      :', '10.0.1')
('Directory    :', '/home/zhutou/work/testmx/.env/local/lib/python2.7/site-packages/pip')
----------MXNet Info-----------
No MXNet installed.
----------System Info----------
('Platform     :', 'Linux-4.4.0-125-generic-i686-with-Ubuntu-16.04-xenial')
('system       :', 'Linux')
('node         :', 'iZ25785d3n3Z')
('release      :', '4.4.0-125-generic')
('version      :', '#150-Ubuntu SMP Wed May 9 10:54:03 UTC 2018')
----------Hardware Info----------
('machine      :', 'i686')
('processor    :', 'i686')
What to do:
1. Download the diagnosis script from https://raw.githubusercontent.com/apache/incubator-mxnet/master/tools/diagnose.py
2. Run the script using `python diagnose.py` and paste its output here.

Package used (Python/R/Scala/Julia):
(I'm using ...)
Python
Steps to reproduce
(Paste the commands you ran that produced the error.)

follow MXNet install introduce
pip install mxnet

What have you tried to solve it?

delete python virtualenv,and retry

I got an error when build from source:
ERROR: macOS version 10.13 NOT supported.
why not support this version?

@zjjott thanks for reporting this issue. @sandeep-krishnamurthy could you add label "Feature Request" , "Installation" to this?
I found a similar issue on https://discuss.mxnet.io - https://discuss.mxnet.io/t/installing-on-os-x-high-sierra-10-13/223
Would recommend trying the work-around in the meanwhile.

@kalyc I try to modify install-mxnet-osx-python.sh script,but it install python2 package in system,I want to install mxnet in my virtualenv and with python3.6
I follow this discuss link ,and try pip install mxnet-mkl -U --pre but it also doesn't work,the same error.

I installed mxnet thru pip install mxnet , but I keep downloading the 1.8 version, and when I try installing the pre-requisites for open vino I keep getting
ERROR: Could not find a version that satisfies the requirement mxnet<=1.5.1,>=1.0.0
ERROR: No matching distribution found for mxnet<=1.5.1,>=1.0.0
Can you help me?
----------Python Info----------
Version      : 3.6.10
Compiler     : GCC 4.2.1 Compatible Clang 4.0.1 (tags/RELEASE_401/final)
Build        : ('default', 'Mar 23 2020 17:45:12')
Arch         : ('64bit', '')
------------Pip Info-----------
Version      : 21.0.1
Directory    : /Users/vireshsahay/anaconda3/lib/python3.6/site-packages/pip
----------MXNet Info-----------
No MXNet installed.
----------System Info----------
Platform     : Darwin-18.2.0-x86_64-i386-64bit
system       : Darwin
node         : Maheshs-MacBook-Pro.local
release      : 18.2.0
version      : Darwin Kernel Version 18.2.0: Fri Oct  5 19:41:49 PDT 2018; root:xnu-4903.221.2~2/RELEASE_X86_64
----------Hardware Info----------
machine      : x86_64
processor    : i386
b'machdep.cpu.brand_string: Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz'
b'machdep.cpu.features: FPU VME DE PSE TSC MSR PAE MCE CX8 APIC SEP MTRR PGE MCA CMOV PAT PSE36 CLFSH DS ACPI MMX FXSR SSE SSE2 SS HTT TM PBE SSE3 PCLMULQDQ DTES64 MON DSCPL VMX SMX EST TM2 SSSE3 FMA CX16 TPR PDCM SSE4.1 SSE4.2 x2APIC MOVBE POPCNT AES PCID XSAVE OSXSAVE SEGLIM64 TSCTMR AVX1.0 RDRAND F16C'
b'machdep.cpu.leaf7_features: SMEP ERMS RDWRFSGS TSC_THREAD_OFFSET BMI1 AVX2 BMI2 INVPCID FPU_CSDS'
b'machdep.cpu.extfeatures: SYSCALL XD 1GBPAGE EM64T LAHF LZCNT RDTSCP TSCI'
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0128 sec, LOAD: 0.8395 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.0413 sec, LOAD: 0.6384 sec.
Error open Gluon Tutorial(cn): https://zh.gluon.ai, <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:852)>, DNS finished in 0.38120508193969727 sec.
Timing for FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0554 sec, LOAD: 0.4926 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.1052 sec, LOAD: 1.4829 sec.
Error open Conda: https://repo.continuum.io/pkgs/free/, HTTP Error 403: Forbidden, DNS finished in 0.0895528793334961 sec.

ERROR: No matching distribution found for mxnet==1.9.1

ERROR: No matching distribution found for mxnet==1.9.1

ERROR: Could not find a version that satisfies the requirement mxnet==1.9.* (from versions: 0.11.0, 0.12.0, 0.12.1, 1.0.0, 1.0.0.post1, 1.0.0.post3, 1.0.0.post4, 1.1.0.post0, 1.2.0, 1.2.1, 1.2.1.post1, 1.3.0, 1.3.1, 1.4.0, 1.4.0.post0, 1.4.1, 1.5.0, 1.6.0, 1.7.0.post1, 1.7.0.post2)
ERROR: No matching distribution found for mxnet==1.9.*
Anyone has any ideas? I've been trying to solve/understand for the past days now but can't get it to work.

It's because mxnet hasn't made a wheel for Windows since 2019.  Very confusing.  So you need to downgrade Python to 3.7.

Im in python 3.7 but still the same error:
(python3.7) PS C:\Users\Pc> pip install --force-reinstall mxnet==1.8.0
ERROR: Could not find a version that satisfies the requirement mxnet==1.8.0 (from versions: 0.11.0, 0.12.0, 0.12.1, 1.0.0, 1.0.0.post1, 1.0.0.post3, 1.0.0.post4, 1.1.0.post0, 1.2.0, 1.2.1, 1.2.1.post1, 1.3.0, 1.3.1, 1.4.0, 1.4.0.post0, 1.4.1, 1.5.0, 1.6.0, 1.7.0.post1, 1.7.0.post2)
ERROR: No matching distribution found for mxnet==1.8.0


Bug in _read() function example

Regarding the example given in documentation for _read() function.
int main( void )
{
  ...
  unsigned int nbytes = 60000, bytesread;

  ...

  /* Read in input: */
  if( ( bytesread = _read( fh, buffer, nbytes ) ) <= 0 )
    perror( "Problem reading file" );
  ...
}

Since bytesread is an unsigned int the if-statement checking for error value -1 will not work. So I believe the example is having a bug.

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

ID: 8d1d4511-a144-242e-8c51-f3543246da51
Version Independent ID: cde3b84b-c9fd-f246-b9e5-55bbceea6ebc
Content: _read
Content Source: docs/c-runtime-library/reference/read.md
Product: visual-cpp
GitHub Login: @corob-msft
Microsoft Alias: corob

All MS examples having ton of bugs, actually cross-compiling into windows is a nightmare.

The sample code has been updated with variables of the correct types.

Shouldn't then the printf( "Read %u bytes from file\n", bytesread ); be updated as well to using %i?

@misuo Probably. I'm partial to %d, myself.


arguments in csproj file

Is there a way to do use an argument in the project file for the argument to pass to the docker build?  (or some way to do that within visual studio without needing to do it via the command line?
ie:
$timestamp = [Xml.XmlConvert]::ToString([DateTime]::UtcNow, [Xml.XmlDateTimeSerializationMode]::Utc)
--build-arg BUILD_DATE=$timestamp

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

ID: 84f968ae-2fa3-06af-06c0-74e122071ce5
Version Independent ID: af37447f-bb94-abac-c4cc-cac649575d2c
Content: Visual Studio Container Tools build properties - Visual Studio (Windows)
Content Source: docs/containers/container-msbuild-properties.md
Product: visual-studio-windows
Technology: vs-container-tools
GitHub Login: @ghogen
Microsoft Alias: ghogen

@davidbIQ -- thank you for your questions.  Please consider these resources and let me know if you find an answer:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

cc @ghogen for awareness


Cannot pull parent image from insecure registry

$ sudo /usr/local/bin/buildah build-using-dockerfile .
STEP 1: FROM my-insecure-registry.example.com:8888/rhel7:7-released
error building: error creating build container: error pulling image "my-insecure-registry.example.com:8888/rhel7:7-released": Error initializing image from source docker://my-insecure-registry.example.com:8888/rhel7:7-released: pinging docker registry returned: Get https://my-insecure-registry.example.com:8888/v2/: http: server gave HTTP response to HTTPS client

Pulling of the image via docker daemon is fine:
$ sudo docker pull my-insecure-registry.example.com:8888/rhel7:7-released
Trying to pull repository my-insecure-registry.example.com:8888/rhel7 ...
sha256:87046178b950978a423047d58c11f822f8b0b3c30adf9441d42c779701a8a09f: Pulling from my-insecure-registry.example.com:8888/rhel7
26e5ed6899db: Already exists
66dbe984a319: Already exists
Digest: sha256:87046178b950978a423047d58c11f822f8b0b3c30adf9441d42c779701a8a09f
Status: Image is up to date for my-insecure-registry.example.com:8888/rhel7:7-released

Note: Output has been modified to rename internal server name to my-insecure-registry.example.com:8888.

As a test/workaround.  Can you try:
buildah from --tls-verify=false my-insecure-registry.example.com:8888/rhel7:7-released

Then try the builldah bud again.  Hopefully it will pick up the image that you downloaded in the from.  If that works, then we probably have a tls-verification change that we need to do for the build-using-dockerfile command.

@lcarva Sorry! Forgot to include your name in the last one, and more importantly forgot to thank you for the report.

That fixes it!
$ sudo /usr/local/bin/buildah from --tls-verify=false my-insecure-registry.example.com:8888/rhel7:7-released
Getting image source signatures
Copying blob sha256:26e5ed6899dbf4b1e93e0898255e8aaf43465cecd3a24910f26edb5d43dafa3c
 71.40 MiB / 71.40 MiB [====================================================] 9s
Copying blob sha256:66dbe984a319ca6d40dc10c2c561821128a0bd8967e0cbd8cc2a302736041ffb
 1.21 KiB / 1.21 KiB [======================================================] 0s
Copying config sha256:549b1c5d7a448e697f6689001f70ee504aa6ef853c658782810117fe8a67c310
 6.22 KiB / 6.22 KiB [======================================================] 0s
Writing manifest to image destination
Storing signatures
rhel7-working-container

Which causes the buildah build-using-dockerfile command to work just fine.
Any chance the  --tls-verify=false parameter can be added to build-using-dockerfile? It would be a nice to have.

@lcarva Thanks for the test and confirmation.  Glad to hear that worked.  I'll look into adding the tls-verify flag to the buildah bud command.  Will send you a pointer to the PR once put together, hopefully early next week.

Yes @TomSweeneyRedHat Open A PR it should definitely have it.

PR created: #297

I'm closing ticket as #297 fixes this problem.


[Qt-Wallet] Transaction failed error messages need to be updated.

Description:
Tried TTL Blocks with lesser values and observed Transaction failed in the receiver side with the below message,

Expected Result:
Error messages should be simple and easy to understand. Current error messages need to be updated with some word rephrasing.

New message

Note, error massage mapping is done on QT wallet side.

Verified with the latest build, hence closing this ticket.


Relock and black

Signed-off-by: Jacob Hayes<EMAIL_ADDRESS>

Codecov Report

Merging #308 (aeb7a4f) into golden (57fe0a7) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##            golden      #308   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           42        42
  Lines         2599      2599
  Branches       654       654
=========================================
  Hits          2599      2599

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Support base URL overrides

For when UI and API are hosted at different URL paths, for example serving the built HTML/CSS/JS static content from a CDN, but serving the API from behind some API gateway.
I'd be happy to add docs if maintainer(s) like this feature.

Thanks for the contribution @masch712! I'd be happy to pull this in if you can add some documentation? Maybe a short new section in docs/configuration.md since this config is kind of a special case. Doesn't need to be much. A short blurb on why someone would set this like you have here in PR description would be fine.

Thanks @rickbergfalk !  I left some details in docs/configuration.md.  After further testing, and in the process of writing the docs, I realized I oversold this feature in my initial PR description :-).  This feature doesn't really support hosting the static content in a CDN, it's more for a Backend-For-Frontend situation where the BFF handles user authentication and serves the sqlpad single-page app, but the API is hosted in a separate domain (behind an API Gateway, for example, which enforces authorization) and the BFF relays requests to the API.  Again, sort of a custom situation for my own setup, but this code change works for me.

Hey @masch712 - Wanted to give you a heads up that I merged but then had to revert this. The dev flow of using the config.dev.env was no longer allowing user login due to the changes.
Would happily merge again once that is solved for.

@rickbergfalk yikes, sorry about the breaking chnage!  Fix is straightfoward; I created a new PR for it: https://github.com/sqlpad/sqlpad/pull/1141

No worries! It didn't go out in a release or anything and I should have caught it in review :)


cmd: update cobra command option to silence usage and error

Signed-off-by: Allen Sun<EMAIL_ADDRESS>Describe what this PR does / why we need it
Every time when we support end user, I found that there are always command usage information and duplicated error. It makes information less explicit. We make the cobra not to print usage info and just print one time of error.
Does this pull request fix one issue?
none

Describe how you did it
none
Describe how to verify it
none
Special notes for reviews
none

Codecov Report

Merging #1523 (a2444ca) into main (2e5c537) will increase coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1523      +/-   ##
==========================================
+ Coverage   10.59%   10.68%   +0.08%
==========================================
  Files          84       84
  Lines        7629     7640      +11
==========================================
+ Hits          808      816       +8
- Misses       6719     6721       +2
- Partials      102      103       +1

Impacted Files
Coverage Δ

pkg/env/env.go
57.83% <0.00%> (+2.27%)
:arrow_up:

Continue to review full report at Codecov.

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


Add support build image

https://github.com/docker/swarm/issues/632 swarm need this Interface
test code
package main

import (
        "archive/tar"
        "bytes"
        "fmt"
        "github.com/samalba/dockerclient"
        "io/ioutil"
)

func main() {
        // Init the client
        docker, _ := dockerclient.NewDockerClient("unix:///var/run/docker.sock", nil)
        files := make(map[string]string)
        files["Dockerfile"] = "FROM busybox"
        buf := new(bytes.Buffer)
        tw := tar.NewWriter(buf)
        for k, v := range files {
                hdr := &tar.Header{
                        Name: k,
                        Size: int64(len(v)),
                }
                err := tw.WriteHeader(hdr)
                if err != nil {
                        return
                }
                _, err = tw.Write([]byte(v))
                if err != nil {
                        return
                }
        }
        image := dockerclient.BuildImage{
                Config:         nil,
                DockerfileName: "",
                Context:        buf,
                RemoteURL:      "",
                RepoName:       "testimage:ok",
                SuppressOutput: false,
                NoCache:        false,
                Remove:         false,
                ForceRemove:    false,
                Pull:           true,
                Memory:         0,
                MemorySwap:     0,
                CpuShares:      0,
                CpuPeriod:      0,
                CpuQuota:       0,
                CpuSetCpus:     "",
                CpuSetMems:     "",
                CgroupParent:   "",
        }
        reader, err := docker.BuildImage(image)
        if err != nil {
                fmt.Printf("%s", err.Error())
        }
        data, _ := ioutil.ReadAll(reader)
        fmt.Printf("%s", data)
}

Signed-off-by: Xian Chaobo<EMAIL_ADDRESS>

ping @donhcd @vieux @aluzzardi @ehazlett

github hid this but it's still relevant:

thoughts @aluzzardi @vieux @samalba ?

Any update on this?  I think it would be cool to have build capabilities.

@ehazlett
Yeah, swarm 0.4 also need this.
Anyway, I have compared the latest docker code, find out that more fields is added in build. so I will update this recently.
Also, any thought about AuthConfig that @donhcd says?

imo we can remove the X-Registry-Auth header and the Auth field in BuildImage
looks like X-Registry-Config has been used since 1.0 https://github.com/docker/docker/blob/v1.0.0/api/server/server.go#L888-L891

ping @donhcd @ehazlett
Updated and check again. Thanks.

ping @vieux @aluzzardi  @donhcd
Need code review again, Thanks

LGTM

ping @ehazlett

LGTM


'az network application-gateway create' doesn't work when associating WAF policy with AppGW.

az feedback auto-generates most of the information requested below, as of CLI version 2.0.75

Describe the bug
'az network application-gateway create' doesn't work when associating WAF policy with AppGW.
we might have succeeded in creating AppGW associated with WAF policy when Azure CLI version was 2.0.70.
To Reproduce
You can repro when using the following commands.

$lacation = "japaneast";
$vnetName = "AppGW-VNet";
$subnetName = "AppGW-Subnet";
$rgpVnetName = "AppGwGroup";
$rgpAppName = "AppGwGroup";
$apgwName = "Test-AppGW";
$wp_name = "${apgwName}-waf-policy01";
$sbntInfoId = $(az network vnet subnet show --name "${subnetName}" --vnet-name "${vnetName}" --resource-group "${rgpVnetName}" --query id --output tsv)
$publicip_name = $apgwName + "-ip";
az network public-ip create --name "${publicip_name}" --allocation-method "Static" --sku "Standard" --resource-group "${rgpVnetName}";
$publicIpId = $(az network public-ip show --name "${publicip_name}" --resource-group "${rgpVnetName}" --query id --output tsv)
az network application-gateway create     --name "${apgwName}"
--location "${lacation}"     --resource-group "${rgpAppName}"
--subnet "${sbntInfoId}"     --min-capacity 2
--max-capacity 10     --sku "WAF_v2"
--zones 1 2 3     --http-settings-cookie-based-affinity Disabled
--frontend-port 80     --http-settings-port 80
--http-settings-protocol "Http"     --public-ip-address "${publicIpId}"
--waf-policy "${wp_name}"

And then, you would encounter the following error message.

Deployment failed. Correlation ID: c136dab2-a462-4268-8267-0f92d08cdb75. {
"error": {-capacity 2 `
"code": "ApplicationGatewayWithoutWafConfigurationCannotBeAddedWithCustomRuleOnlyFirewallPolicy",
"message": "CustomRule only firewall policy /subscriptions/49dac445-53b8-451a-b1a0-e47c9fdc93d3/resourceGroups/AppGwGroup/providers/Microsoft.Network/ApplicationGatewayWebApplicationFir
ewallPolicies/Test-AppGW-waf-policy01 cannot be applied to an Application Gateway  /subscriptions/49dac445-53b8-451a-b1a0-e47c9fdc93d3/resourceGroups/AppGwGroup/providers/Microsoft.Network/
applicationGateways/Test-AppGW which doesnt have any WebApplicationFirewallConfiguration.",

As per the error message it says "ApplicationGatewayWithoutWafConfigurationCannotBeAddedWithCustomRuleOnlyFirewallPolicy"
However, we are unable to set WAF mode enable because Azure CLI command can't set WAF mode as per the following document.
https://docs.microsoft.com/en-us/cli/azure/network/application-gateway?view=azure-cli-latest#az-network-application-gateway-create
Expected behavior
succeed in creating AppGW associated with WAF policy
Environment summary
Cloud shell / CLI version 2.0.75 / OS version / Shell Type : bash or PS

Thanks for the feedback! We will make a plan for it.

Once the feature App Gateway WAF Ignite features release, waf-policy have the capality to config those things and can apply among not only different application-gateways. Please have a try.
Feel free to reopen. If you have any futher question.

Once 2.0.77 ( 20/11/2019 ) release, you can use
az network applicate-gateway waf-policy  to config a waf-policy. And waf-policy can work among multi applicatin-gateways.
You can:

Add pre-defined managed rules by az network applicate-gateway waf-policy managed-rule rule-set, lke SQL Injection pretection.
Add custom rule you would like to filter on your requests: az network applicate-gateway waf-policy managed-rule custom-rule
You can also configure this waf policy by az network applicate-gateway waf-policy policy-setting, configurable properties include: state, mode, request_body_check, max_request_body_size_in_kb and file_upload_limit_in_mb.

Sorry for the inconvenience and confusion.
Feel reopen to issue bug or feature if you have any futher question.


Add Warn field to additionalPrinterColumns, remove scoreWarning default

This commit adds the "Warn" field from status.summary as an additionalPrinterColumn.
It also removes the default value for scoreWarning. This is because the CRDs are of version
apiextensions.k8s.io/v1beta1. Setting a default in this version is not allowed. Although the
file crd.go specifies version as "v1", wrangler is generating CRD with version "apiextensions.k8s.io/v1beta1".
This could depend on the available versions in the cluster and can be investigated later.
For now in order to avoid any install issues, the default of "pass" is being removed. The logic to make the
scan fail checks the field specifically for "fail" value, so it won't be affected.

https://github.com/rancher/security-scan/issues/32


Selenium 3.1 is wrongly recognized as Selenium 2

I think that this line in StartGrid.java:
boolean isSelenium3 = RuntimeConfig.getConfig().getWebdriver().getVersion().startsWith("3.0");
should actually be:
boolean isSelenium3 = RuntimeConfig.getConfig().getWebdriver().getVersion().startsWith("3.");
because now with webdriver version 3.1 out, and automatic webdriver update, the SGE stops converting the node config file to match the Selenium 3 version...
The issue is then visible in the log files as:
command: /usr/lib/jvm/java-8-oracle/jre/bin/java -Dwebdriver.chrome.driver=/tmp/webdriver/chromedriver/chromedriver_2.27_64bit -Dwebdriver.gecko.driver=/tmp/webdriver/geckodriver/geckodriver_0.14.0 -cp /home/user/Desktop/SeleniumGrid/SeleniumGridExtras-1.12.10-SNAPSHOT-jar-with-dependencies.jar:/tmp/webdriver/3.1.0.jar org.openqa.grid.selenium.GridLauncherV3 -role node -nodeConfig node_5555.json -log log/node_5555.log
exit code: 1
standard error:
Exception in thread "main" org.openqa.grid.common.exception.GridConfigurationException: Error with the JSON of the config : Deprecated -nodeConfig file encountered. Please update the file to work with Selenium 3. See https://github.com/SeleniumHQ/selenium/wiki/Grid2#configuring-the-nodes-by-json for more details.
at org.openqa.grid.internal.utils.configuration.GridNodeConfiguration.loadFromJSON(GridNodeConfiguration.java:444)
at org.openqa.grid.internal.utils.configuration.GridNodeConfiguration.loadFromJSON(GridNodeConfiguration.java:421)
at org.openqa.grid.selenium.GridLauncherV3$3.setConfiguration(GridLauncherV3.java:271)
at org.openqa.grid.selenium.GridLauncherV3.buildLauncher(GridLauncherV3.java:155)
at org.openqa.grid.selenium.GridLauncherV3.main(GridLauncherV3.java:75)
Caused by: org.openqa.grid.common.exception.GridConfigurationException: Deprecated -nodeConfig file encountered. Please update the file to work with Selenium 3. See https://github.com/SeleniumHQ/selenium/wiki/Grid2#configuring-the-nodes-by-json for more details.
at org.openqa.grid.internal.utils.configuration.GridNodeConfiguration.loadFromJSON(GridNodeConfiguration.java:436)
... 4 more
Temporary workaround  is to disable the automatic driver update and set the webdriver version to below 3.1, e.g. 3.0.0

+1
Workaround: set auto update drivers to 0, and use webdriver 3.0.1.

+1
Thanks for the workaround, it's working great for me!

Thank you ! I will make the fix now

Fixed it here : https://github.com/groupon/Selenium-Grid-Extras/commit/a6c3cc29ff07d1f7ff889c6b3a132fc3472d34d4

I am prepping for a release, but not sure how much testing I can do anytime soon. @zsucic @travisneedham @mstankovic86 If you can grab the latest to test with, that would be helpful.

I can probably do it on Monday.  I need my environment to be stable this weekend.

Same here...
On Fri, Feb 17, 2017, 17:10 travisneedham<EMAIL_ADDRESS>wrote:

I can probably do it on Monday. I need my environment to be stable this
weekend.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/groupon/Selenium-Grid-Extras/issues/309#issuecomment-280692056,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARE5lA4pEFVf_YJtFV7bwh0x8Mbs40Jeks5rdcZ4gaJpZM4MEGYb
.

When y do bugfix?

did someone test it?

I'd be happy to test it with my small configuration (Hub & 3 Node VMs) if there are test bits available.

@mstankovic86 when i try build github version app with maven build failed and don`t build

[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building Selenium Grid Extras 1.0-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] SeleniumGridExtras ................................. FAILURE [ 41.067 s]
[INFO] Selenium Grid Extras ............................... SUCCESS [  0.002 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 41.280 s
[INFO] Finished at: 2017-02-24T12:21:12+03:00
[INFO] Final Memory: 31M/358M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.12.4:test (default-test) on project SeleniumGridExtras: There are test failures.
[ERROR]
[ERROR] Please refer to /home/alexey/IdeaProjects/Selenium-Grid-Extras/SeleniumGridExtras/target/surefire-reports for the individual test results.
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[INFO] Build failures were ignored.

Binaries look to be posted (thanks devs!) but GitHub is choking on file downloads atm.

 Please let me know if this is fixed. I tested it and I believe its fixed, but would like to here from someone who had the issue.
Thank you

v1.12.11 seems to recognize Selenium version correctly now, however because of this issue:
java.util.concurrent.ExecutionException: java.lang.NoSuchMethodError: com.google.common.base.Preconditions.checkState(ZLjava/lang/String;Ljava/lang/Object;)V
I'm unable to use 3.1 or 3.2
with 3.0 it seems to work ok so workaround is still valid.

My experience matches zsucic, I rolled back to 3.0.1 and all is fine.

@zsucic @smattt I made the fix in 1.12.12 (for Precondition). I bumped the guava dependency.

v1.12.12 is working great for us! Thanks again!

Same here. v1.12.12 is working great for us! Thank you!

v1.12.12 worked for me briefly when using webdriver 3.2.0.
but now webdriver 3.3.0 is out, it's not working again.
As a work around I've set webdriver version to 3.2.0 and disabled auto updating of drivers, this has got my nodes working again for now.

I tried a fresh installation on a Linux VM (ubuntu 14.04) for evaluation purposes and failed with errors like described above. Tried different settings: Autoupdate drivers turned on and off, using webdriver version 3.2.0, setting iedriver to 2.53.1, which works on an older version of selenium-grid-extras.
Nevertheless, installation always stops with following exception:
Download IE Driver Executable 2.53.1
Exception in thread "main" java.lang.UnsupportedClassVersionError: org/apache/tools/ant/taskdefs/Expand : Unsupported major.minor version 52.0
at java.lang.ClassLoader.defineClass1(Native Method)
at java.lang.ClassLoader.defineClass(ClassLoader.java:803)
at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142)
at java.net.URLClassLoader.defineClass(URLClassLoader.java:442)
at java.net.URLClassLoader.access$100(URLClassLoader.java:64)
at java.net.URLClassLoader$1.run(URLClassLoader.java:354)
at java.net.URLClassLoader$1.run(URLClassLoader.java:348)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(URLClassLoader.java:347)
at java.lang.ClassLoader.loadClass(ClassLoader.java:425)
at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:308)
at java.lang.ClassLoader.loadClass(ClassLoader.java:358)
at java.lang.ClassLoader.defineClass1(Native Method)
at java.lang.ClassLoader.defineClass(ClassLoader.java:803)
at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142)
at java.net.URLClassLoader.defineClass(URLClassLoader.java:442)
at java.net.URLClassLoader.access$100(URLClassLoader.java:64)
at java.net.URLClassLoader$1.run(URLClassLoader.java:354)
at java.net.URLClassLoader$1.run(URLClassLoader.java:348)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(URLClassLoader.java:347)
at java.lang.ClassLoader.loadClass(ClassLoader.java:425)
at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:308)
at java.lang.ClassLoader.loadClass(ClassLoader.java:358)
at java.lang.ClassLoader.defineClass1(Native Method)
at java.lang.ClassLoader.defineClass(ClassLoader.java:803)
at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142)
at java.net.URLClassLoader.defineClass(URLClassLoader.java:442)
at java.net.URLClassLoader.access$100(URLClassLoader.java:64)
at java.net.URLClassLoader$1.run(URLClassLoader.java:354)
at java.net.URLClassLoader$1.run(URLClassLoader.java:348)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(URLClassLoader.java:347)
at java.lang.ClassLoader.loadClass(ClassLoader.java:425)
at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:308)
at java.lang.ClassLoader.loadClass(ClassLoader.java:358)
at com.groupon.seleniumgridextras.downloader.Unzipper.unzip(Unzipper.java:62)
at com.groupon.seleniumgridextras.downloader.IEDownloader.download(IEDownloader.java:105)
at com.groupon.seleniumgridextras.tasks.DownloadIEDriver.execute(DownloadIEDriver.java:117)
at com.groupon.seleniumgridextras.tasks.DownloadIEDriver.execute(DownloadIEDriver.java:88)
at com.groupon.seleniumgridextras.tasks.DownloadIEDriver.initialize(DownloadIEDriver.java:153)
at com.groupon.seleniumgridextras.SeleniumGridExtras.main(SeleniumGridExtras.java:88)

@beratomi
Try to install newest java for ubuntu. It looks like you try to run jar with older version than it was compiled.
Details: http://stackoverflow.com/questions/22489398/unsupported-major-minor-version-52-0

Thx, you made my day :-)


Equivalent Java application implemented

Hi Scott Hoover,
Good day! Hope you're well.
Java implementation has been completed and tested on real environment (as I wrote a similar version for my job in production server). It's slightly different from Scala version + some additional features. README have been updated.
I am wondering if the jars mentioned for flume setup are also needed from Java version because I included necessary dependency in pom.xml. What do you think?
Please review the pull request and let me know your opinion.
Regards,
Kaidul

@kaidul I would suspect that including that dependency in your pom should be sufficient.

@githoov Thanks for merging :) I've sent another pull request with README modification and other little changes.


try logging client

https://github.com/keboola/storage-api-php-client/pull/804

Replaced by https://github.com/keboola/job-runner/pull/122 and https://github.com/keboola/job-runner/pull/120


Indicate extensive functionality

Indicate that get_forecast has extensive functionality like get_observation

Coverage remained the same at ?% when pulling 0c38ac298c0a9f12bb4830ef2332fca782bdc55b on meichthys:patch-1 into a0c563d6e7fd94c8a9d9695cbe10d82b6e7def24 on paulokuong:master.

@meichthys thanks


issue50: Fixing some warnings during npm install

Using the correct apache licence identifier in package.json, see: https://spdx.org/licenses/
gulp-clean deprecated in favor of del, see here
Update dependencies using caret(^) in version
(Bonus) adding dependency badges from https://david-dm.org/

I made a mess with the previous pull request so I'm sending another one...

Just like my previous comment - you need to bump project version. Actually for every merge request - we need to bump the last number (x.y.z so we need to bump z). This way every single change to the project will have its own number so you always know if the version of the app you are currently working is the latest or not. Bump the version number and I will merge.

Done...

Merged! Thanks for contributing!


Configuration profiles for NON-SDK packages do not seem to import correctly

XRTK - Mixed Reality Toolkit Bug Report
Describe the bug
If you install the Core + SDK, the asset GUID regeneration for the assets in the "XRTK.Generated" folder seem to work fine, However.
For the ancillary packages, Oculus, WMR, Lumin, the guide regeneration / realignment does not seem to go deep enough and the input actions configuration for each controller still maps to the GUID's for the original assets
To Reproduce

Create a new XRTK project
Import XRTK and SDK (0.2 Preview packages)
Check controller configuration is as expected (see image 1 below)
Import another package (Oculus)
Accept prompt for importing the config
Check controller configuration (see image 2)

On inspection:

The GUID's for the SDK assets are different, checking the controller action mapping asset for an OpenVR controller confirms the GUID's are different (regeneration successful)

The GUID's for the Oculus assets have NOT been regenerated, checking the controller action mapping asset for an OpenVR controller confirms the GUID's are IDENTICAL

Expected behavior
Guids in the "copied" assets should have new GUID's and have the lookups reflected in the assets they are used, as happens with the SDK assets
com.xrtk.sdk - OpenVRControllerDataProvider - controllerMappingProfiles -> XRTK.Generated - OpenVRControllerDataProvider - controllerMappingProfiles (working)
GUID's for controller mapping profiles different
Actual behavior

com.xrtk.oculus - LeftOculusTouchControllerDataProvider - controllerMappingProfiles -> XRTK.Generated - LeftOculusTouchControllerDataProvider - controllerMappingProfiles (NOT working)
GUID's for controller mapping profiles identical
Your Setup

Operation system Windows
Unity Version 2019.1.14f0
XRTK Version 0.2.0-preview

Additional context

Try installing the sdk last. I think it might be because the guid regenerator is expecting the old guid references but they're technically invalid because the old references have already been rewritten.

That seems to make the opposite of sense @StephenHodgson , as why would the SDK effect when a different package is installed?
The SDK installs one as it's GUID's are regenerated correctly.
My initial suspicion is that the GUID regeneration isn't going deep enough, seeing it does seem to update the higher levels (, Input Actions, Speech, Controller data provider, etc)  but the lowest level isn't

It's based on the root folder we provide it.
It makes sense if you're installing step by step and not all at once.

I think the root of the problem is that the packages aren't being installed all at once.
If you install one package, then re-generate the guids, when you go to install the next package, the guid references are already changed, and there is a mismatch.

OK, issue resolved or at least diagnosed.
Seems when the profiles were stripped and moved to their own packages, some of them (namely WMR and Oculus) were still sharing the mappings from the OpenVR implementation and hence did not have their own input mappings.
Hence when the profiles that WERE in the packages were copied, they worked.
The simple solution for now is to create/copy the input mapping profiles to the respective packages.  The new GUID regen system isn't up to "sharing" config across packages atm.

Added copies of the original assets to the respective packages.
Guid's should not conflict as they are regenerated when they are copied to the users XRTK.Generated folder.  Tested by removing the XRTK.Generated folder and forcing a reimport.


Implement data encryption between client and server

commit 2ac662a8c8ea14fe4477f34a4cc0db901e12f47d
Implemented the encrypt / decrypt functions that uses the RSA algorithm and the Fernet and base64 modules.


Quick question> How to use the mfe remote component in the shell html

I have a MFE that exposes a module. This module exports and declare componentX. I want to use this component as  in the shell/host Html. I cannot use router because this is a component that always shows in the app.component in the shell MFE. How can I do this?
best
HM

I have a MFE that exposes a module. This module exports and declare componentX. I want to use this component as in the shell/host Html. I cannot use router because this is a component that always shows in the app.component in the shell MFE. How can I do this?
best HM

Hi, now  i'm working on it. Creating lib to load componentX in HTML template by structural directive.
Example how it would be:
<ng-container *mfeOutlet="'login/entry'"></ng-container>
When i'm done i'll tell you!
P. S. with angular v12 and angular-architects/module-federation-plugin v12 everything worked, but then i'm migrate to angular v13 and angular-architects/module-federation-plugin v14 does not(
Has this error:

I have a MFE that exposes a module. This module exports and declare componentX. I want to use this component as in the shell/host Html. I cannot use router because this is a component that always shows in the app.component in the shell MFE. How can I do this?
best HM

Hi, I have a solution how to load an MFE component directly in HTML, this library https://www.npmjs.com/package/ngx-mfe will help you with this. Use the MfeOutletDirective directive from this library.
What you need is called plugin-based approach, I hope i helped you.
P. S.
If you have any questions, then I will be happy to help you, since I am the developer of this library.

Cool! Thanks dkhrunov

In my usecase, I want to load more than one MFE plugins in my home component using module federation. Is that supported and is there a sample for that?

In my usecase, I want to load more than one MFE plugins in my home component using module federation. Is that supported and is there a sample for that?

Hi, yes you can do that! You just need use two times MfeOutletDirective in your template (HTLM).
It should be look like this:

An example above i use *mfeOutlet two times, one for display EntryComponent from "client-dashboard-mfe" app, and second for display NotFoundComponent from "client-fallback-mfe".

Awesome. Let me try that.

Awesome. Let me try that.

Ok, lets me know if you get some problems. Also, if you get to solve your problem, also write, please!

Sure... I didn't get a chance to try that yet. Hoping to do it today.


Azure Backup Recovery Vault for Generic Subscription

Overview/Summary
Added the recovery vault capabilities for the backup of Virtual Machines in the generic subscription Archetypes. The capability is enabled or disabled through a Boolean flag
This PR fixes/adds/changes/removes

backupRecoveryVault.json Schema type
BackupRecoveryVaultIsFalse.json test file
BackupRecoveryVaultIsTrue test file

Breaking Changes
N/A
Testing Evidence
.\test-all.ps1 -TestFolder . -SchemaFolder ../../schemas/latest/landingzones/ ==> Passed
As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant GitHub Issues
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Updated relevant and associated documentation.

Please update the image in docs/archetypes/generic-subscription.md to include backup recovery vault.

Please update schemas/latest/readme.md with the change you made.  This file contains the changelog organized by date.

Updated requested changes


Remove Agent's dependency on proxy to access Antrea Service

We add Endpoint resolution to the AntreaClientProvider, so that when
running in-cluster, accessing the Antrea Service (i.e., accessing the
Antrea Controller API) no longer depends on the ClusterIP functionality
provided by the K8s proxy, whether it is kube-proxy or AntreaProxy.
This gives us more flexibility during Agent initialization. For example,
when kube-proxy is removed and ProxyAll is enable for AntreaProxy,
accessing the Antrea Service no longer requires any routes or OVS flows
installed by the Antrea Agent.
To implement this functionality, we add a controller (EndpointResolver),
to watch the Antrea Service and the corresponding Endpoints
resource. For every relevant update, the Endpoint is resolved and the
new URL is sent to the AntreaClientProvider. This is a similar model as
the one we already use for CA bundle updates.
Note that when the Service stops being available, we clear the Endpoint
URL and notify listeners. This means that GetAntreaClient() can now
return an error even if a previous call was successful.

@tnqn This is related to the idea you brought up in https://github.com/antrea-io/antrea/pull/6342#issuecomment-2123701833.
I considered 2 solutions:

Use a controller to maintain an up-to-date endpoint URL (implemented in this PR)
Resolve the endpoint by calling proxy.ResolveEndpoint synchronously every time GetAntreaClient() is called

I don't really have a strong preference between the 2 and I can go either way.
One issue with the current approach is the transient error logs when the Agent is started before the Controller, but maybe it's pretty minor:
I0522 22:27:14.957139       1 endpoint_resolver.go:159] "Starting controller" name="ServiceEndpointResolver-kube-system/antrea"
E0522 22:27:15.064154       1 endpoint_resolver.go:187] "Failed to resolve Service Endpoint, requeuing" err="no endpoints available for service \"antrea\"" key="kube-system/antrea"
E0522 22:27:15.176660       1 endpoint_resolver.go:187] "Failed to resolve Service Endpoint, requeuing" err="no endpoints available for service \"antrea\"" key="kube-system/antrea"
E0522 22:27:15.386246       1 endpoint_resolver.go:187] "Failed to resolve Service Endpoint, requeuing" err="no endpoints available for service \"antrea\"" key="kube-system/antrea"
E0522 22:27:15.789781       1 endpoint_resolver.go:187] "Failed to resolve Service Endpoint, requeuing" err="no endpoints available for service \"antrea\"" key="kube-system/antrea"
E0522 22:27:16.597705       1 endpoint_resolver.go:187] "Failed to resolve Service Endpoint, requeuing" err="no endpoints available for service \"antrea\"" key="kube-system/antrea"
E0522 22:27:18.203027       1 endpoint_resolver.go:187] "Failed to resolve Service Endpoint, requeuing" err="no endpoints available for service \"antrea\"" key="kube-system/antrea"
I0522 22:27:21.409353       1 endpoint_resolver.go:204] "Selected Endpoint has changed for Antrea Service" url="https://<IP_ADDRESS>:10349"

If we resolve every time GetAntreaClient is called, we may incur a small extra cost, but it is not significant as we just lookup the Service and Endpoints resource from the informer cache.
Let me know if you have a preference between the 2.
The current solution is tested and works, but I would not mind switching to the other solution.

@tnqn I addressed comments and did some improvements. I also added a few unit tests for the new code. PTAL.

With the latest code, the new logs look much better because we avoid retries when unnecessary.
Agent starting before Controller:
I0523 22:08:26.117956       1 endpoint_resolver.go:194] "Starting controller" name="ServiceEndpointResolver-kube-system/antrea"
E0523 22:08:26.224238       1 endpoint_resolver.go:240] "Cannot resolve endpoint because Service is unavailable" err="no endpoints available for service \"antrea\"" service="kube-system/antrea"
I0523 22:08:33.546174       1 endpoint_resolver.go:276] "Selected Endpoint has changed for Service, notifying listeners" service="kube-system/antrea" url="https://<IP_ADDRESS>:10349"

Controller restart (Pod deletion, new Pod scheduled on different Node):
I0523 22:09:30.868733       1 endpoint_resolver.go:194] "Starting controller" name="ServiceEndpointResolver-kube-system/antrea"
I0523 22:09:30.969815       1 endpoint_resolver.go:276] "Selected Endpoint has changed for Service, notifying listeners" service="kube-system/antrea" url="https://<IP_ADDRESS>:10349"
E0523 22:09:49.622553       1 endpoint_resolver.go:240] "Cannot resolve endpoint because Service is unavailable" err="no endpoints available for service \"antrea\"" service="kube-system/antrea"
I0523 22:09:49.625716       1 endpoint_resolver.go:278] "No more selected Endpoint for Service, notifying listeners" service="kube-system/antrea"
E0523 22:09:49.843244       1 endpoint_resolver.go:240] "Cannot resolve endpoint because Service is unavailable" err="no endpoints available for service \"antrea\"" service="kube-system/antrea"
I0523 22:10:00.721816       1 endpoint_resolver.go:276] "Selected Endpoint has changed for Service, notifying listeners" service="kube-system/antrea" url="https://<IP_ADDRESS>:10349"

/test-all

/test-all

I suppose this change is invisible to users and no need to be in release log?

I don't think very strongly either way, but if it were me I would mention it. I will add the action/release-note label, but feel free to remove it if you disagree.

Linux e2e tests currently failing because of missing script:
./ci/jenkins/docker_login.sh: No such file or directory

/test-all

/test-all

/test-vm-e2e


It would be great if we can generate one word random outputs with this library

This is a feature request:
I can get the max benefit of this library if there is a way to generate only one-word output.
eg: Apple
Banana
The problem is the inability of forcing the library not to generate random fruit name with multiple words. basically to avoid "SUGAR BABY WATERMELON", "KING COCONUT" like fruit names because they have more than 1 word.
I am expecting something like
getRandomFruitsName({maxWords: 1}); which will ensure it will only return one word fruit name.
Thank you.!

@dulara1994 Hello.
Sorry for the late reply.
That's an interesting idea!
I am going to consider whether it is feasible.
Thanks for the great ideas!

@dulara1994  I've come up with an implementation based on your ideas.
I think the function design would look like this.
(The maxWords would be an option in the second argument.)
getRandomFruitsName(lang, option)

Example:
getRandomFruitsName('en', {maxWords: 1})

@dulara1994 Added maxWords option.
I've released it on npm, so you could install the latest version and give it a try.
https://www.npmjs.com/package/random-fruits-name#option

This issue is close.
If you have any requests or issues, please create a new issue.
Thanks for the great idea!

This issue is close.
If you have any requests or issues, please create a new issue.
Thanks for the great idea!


EVP_EncryptInit.pod: update

Mention CTR, GCM, CCM, OCB, WRAP and XTS modes.

[ x] documentation is added or updated

Merged thanks.


[Bug] Setup failed for custom integration 'lunar_phase': Requirements for lunar_phase not found: ['ephem==4.1.5'].

Short summary 📝
Setup failed for custom integration 'lunar_phase': Requirements for lunar_phase not found: ['ephem==4.1.5'].
Detailed description 📋
After updating to HA 2024.12.0 the integration does not load anymore with the above error message.
Version 📌
0.2.0
Environment 🌐
No response
Logs and screenshots 📂
No response

Hi @nemoBE, update the integration to latest release v0.2.2

Thanks, that worked


Catch error thrown from Joplin API when getting the parent tile of a …

this PR is the outcome of the discussion in #9, although I don't know if it fixes the original issue ;-)

Thank you! You're notes on the troubleshooting are very helpful and this PR even more so. Let's hope this closes out the issue for everyone. At the very least, I learned something new about js/ts :)
I will consider this to fix #9 and re-open if something else comes up.

I've just released v1.6.1 with your changes (and a couple more .catch calls to be safe). Please let me know if the plugin works for you now.

This appears to have fixed my problem. I now have a summary note that is populated with real todos.


Kubedock liveness / readiness probe?

Hi there,
More of a question than an issue...
Does kubedock provide a liveness / readiness endpoint?
I would like to run kubedock as a standalone deployment in k8s, so I'm wondering how to configure the k8s liveness / readiness probes.
Thanks!

The /_ping endpoint can be used for that. This will return OK when the api is up and running.
Not sure if it makes sense to add another specific one, as this is one is part of the official docker api already.


StartHost() and StartServer() should return success or failure for error handling

Is your feature request related to a problem? Please describe.
There is no way to react on possible failures to start a server.
Describe the solution you'd like
These methods should return a bool or some enum for fine-grained error handling control.

NetworkingManager.Singleton.StartHost()
NetworkingManager.Singleton.StartServer()

The MLAPI can't really fail these two methods. What exact failure are you looking for? You can always catch the exceptions that are thrown by the Transport.

I want to catch an error where the user tries to start a server where the port is already in use.
Cannot open socket on ip {*} and port {7777}; check please your network, most probably port has been already occupied
UnityEngine.Networking.NetworkTransport:AddHost(HostTopology, Int32)
MLAPI.Transports.UNET.RelayTransport:AddHost(HostTopology, Int32, Boolean)
MLAPI.Transports.UNET.UnetTransport:StartServer()
MLAPI.NetworkingManager:StartHost(Nullable`1, Nullable`1, Nullable`1, Stream)

There is no exception thrown in this case. I tried to catch any exception but that doesn't work.

Well, this is just simply the fault of the transport. It's nothing the MLAPI can do anything about. Switch transport is my only advice. UNET is being replaced anyway.

OK


0.20.9 ignores .withIdleTimeout on BlazeServerBuilder

Since upgrading from 0.20.6 to 0.20.9 I've noticed that BlazeServerBuilder ignores the .withIdleTimeout setting.
Even with the following, I get timeouts after the default 30 seconds
    BlazeServerBuilder[F]
      .bindHttp(midwayConfig.listenPort, midwayConfig.listenAddress)
      .withHttpApp(finalHttpApp)
      .withIdleTimeout(5 minutes)
      .serve
      .drain

Hi @kbreidenbach - could you please give a full example, e.g. github repo w/ a few curl's for reproducing the problem?

Even with the following, I get timeouts after the default 30 seconds

Which timeout do you get after 30 seconds?

@kbreidenbach
The default timeouts have been reworked in 0.20.8. Most likely you're hitting the responseHeaderTimeout which now defaults to 30 seconds.
As far as I know, even in 0.20.6 the code sample you've provided won't result in 5 minutes long timeout. Instead you would hit the responseHeaderTimeout after 1 minute (the old default).
The defaults for idleTimeout and responseHeaderTimeout have been altered, to have responseHeaderTimeout < idleTimeout, as it results in better behaviour: sending a 5xx error response on timeout, instead of abruptly terminating TCP connection.

Can this one be closed then, @RafalSumislawski @rossabaker, per https://github.com/http4s/http4s/issues/2796#issuecomment-525922712?


Fix Docker build

node-sass requirements changed and now build fails because of no longer supported node version.
The solution is to change the base image to a compatible one.

Another thing to mention is that I'm curious why docker CI action won't get triggered for PR or any of your commits

The workflow will only execute on push to the main branch @MohamedElashri

There were some issues with the workflow file as well. Fixed em

Oh I understand the problem now, sorry I didn't notice that the repo still using master.


Added VERIFY_SSL variable to OctoPrint component

Added configuration variable of VERIFY_SSL defaulted to True to OctoPrint for self-signed certs on HTTPS connection.
Description:
Related issue (if applicable): fixes #
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#11629
Example entry for configuration.yaml (if applicable):
octoprint:
  - host: xxx.xxx.xxx.xxx
    port: 443
    api_key: xxxxxxxxxxx
    ssl: true
    verify_ssl: false
    name: My Self Signed Printer

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @fanuch,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

I'll close this PR until the new PyPI package is available.


profile loop

when many profile match the network status (e.g. when computer is connected both Ethernet and wifi) the program loop switching.

Go to Settings and select "Stop activating networks after one is detected". If the first profile matches, Proxy Switcher will stop trying to detect the next network. You can order your network profiles by drag&drop.


App languages and system texts in Studio

@arjan-h , please review this PR. I have decided to keep all information in one document not to force users to switch between documents. I find it easier that the part on how to translate texts is kept together with the explanation of the default language and how to manage app languages and system texts.

Closing the PR as no longer relevant.


Fix COUNT for SQL queries that use DISTINCT

Codecov Report

Merging #319 (0c0aa6c) into main (feac44a) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #319   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          231       231
  Lines        12542     12546    +4
=========================================
+ Hits         12542     12546    +4

Impacted Files
Coverage Δ

pkg/database/filter.go
100.00% <ø> (ø)

internal/database/sqlcommon/sqlcommon.go
100.00% <100.00%> (ø)

internal/database/sqlcommon/tokenbalance_sql.go
100.00% <100.00%> (ø)

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 feac44a...0c0aa6c. Read the comment docs.


add release action

Adds a release.yml github actions workflow that builds the server binary and extension for every target listed in https://code.visualstudio.com/api/working-with-extensions/publishing-extension#platformspecific-extensions.
The action runs on new git tags, and creates a release with all the binaries and vsix, and calls vsce publish.
You'll need to add a VSCE_PAT github secret (how to get access token) and add a publisher field to the package.json.

add png version of icon
change extension.ts to look for bundled executable first, falls back to typst-lsp[.exe] (could also be configurable via vscode settings)
add LICENSE file and package.json properties to addons/vscode
add esbuild for bundling and change package.json scripts

npm run package builds .vsix, bundles using esbuild minified
npm run compile bundles using esbuild with source maps
npm run check runs tsc to check typescript
npm run lint/lint-fix as before

If you'd like to have anything changed let me know :)

I don't have time to fully review this right now, but this looks great! I'll get to it as soon as I can. My biggest concern is actually the name of the project; I'd like to make sure it's okay to call this "Typst LSP" and not "LSP for Typst" or something similar (to avoid looking official when we're not).

Suggesting adding the following to speed up builds:
Incremental builds end up being slower in CI because there is overhead and there wasn't a previous build to base upon.
env:
  CARGO_REGISTRIES_CRATES_IO_PROTOCOL: sparse
  CARGO_INCREMENTAL: '0'

Oh awesome, I didn't know! Never mind my comment then :)

This looks good.

I think the alpine build fails because @vscode/vsce is not installed


feat(opendataset): add dataloader for COCO2017 dataset

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.503%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6127

Relevant Lines:
7083

💛  - Coveralls

Duplicate of: https://github.com/Graviti-AI/tensorbay-python-sdk/pull/981


#P04.2. Only display Hackathons where the org is CodeInstitute or where the user.org == hackathon.org

Description
Added an if statement to check if the organisation is Code Institute or if the user.organisation == hackathon.organisation

Pull request type

#P04.2. Only display Hackathons where the org is CodeInstitute or where the user.org == hackathon.org
Testing
I created 3 Hackathons, one has organisation= "Code Institute", second has it blank and the third one has the organisation="Some organisation".

I changed my user organisation to "Some Organisation", so the Hackathons that were displayed were the one with organisation= "Code Institute" and organisation="Some organisation".

Does this introduce a breaking change

[ ] Yes
[x] No

LGTM


podman.spec: recommend slirp4netns

Fixes: #1234
Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>

@rhatdan PTAL. I picked up the low hanging fruit from https://github.com/containers/libpod/issues/1234#issuecomment-411519194. As slirp4netns is already packaged for Fedora, I assume the change is okay.

thanks for working on this.  LGTM

Does SUSE support Recommends yet?

Does SUSE support Recommends yet?

It does but we're not using this .spec for packaging.

Perhaps as a separate PR, we should consider enhancing the packaging so that it can be tested in COPR against Fedora, EL7, openSUSE Leap 15.0, and openSUSE Tumbleweed.

THis should be
%if 0%{?fedora}
Recommends: container-selinux
Recommends: slirp4netns
%else
Requires: container-selinux
Requires: slirp4netns
%endif

I'd flip it to the following:
%if 0%{?rhel} && 0%{?rhel} < 8
Requires: container-selinux
Requires: slirp4netns
%else
Recommends: container-selinux
Recommends: slirp4netns
%endif

I agree, the rhel will handle centos as well.

Updated, thanks!

bot, retest this please

@rh-atomic-bot r+

@rh-atomic-bot retry

I am just going to merge since this passes all tests and Humu seems to not like it.


Create separate file for unreleased 3.0 changes

Contributors and maintainers frequently put changelog entries in the wrong section. It's an easy mistake to make! Given that making changes intended for the next minor release is the norm, this change optimizes that process by making CHANGELOG.md contain only entries for the next minor. Truly breaking changes intended for the next major are kept in a separate file. Credit to @msfroh for the idea.
Note that the CHANGELOG-3.0.md content is not actually correct, because it contains mistakes intended to be fixed by this change going forward! That content can be cleaned up by a follow up PR.
Slack thread for additional context: https://opensearch.slack.com/archives/C05L60S4UBT/p1712011157190119
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[x] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@andrross totally understand the pain, could we (if it makes sense) follow this convention, each branch has own CHANGELOG, fe:

CHANGELOG-3.x
CHANGELOG-2.x
CHANGELOG-1.x

The main will keep:

CHANGELOG-3.x
CHANGELOG-2.x

The 2.x will keep:

CHANGELOG-2.x
CHANGELOG-1.x

I think it would be more straightforward to manage, what do you think? @peternied  @dblock ?

could we (if it makes sense) follow this convention, each branch has own CHANGELOG, fe:
CHANGELOG-3.x
CHANGELOG-2.x
CHANGELOG-1.x

I second @reta suggestion, I would also say that it could be used as a way to automation and account for changed that have not been backported.

This will need changes to changelog verifier workflow for verification.

The approach that @andrross has taken here doesn't break changelog verifier for ~98% of changes. Since things that get backported to 2.x are still in CHANGELOG.md, the changelog verifier will be happy with them (but contributors are less likely to put things in the wrong section because CHANGELOG.md no longer has the "Unreleased 3.0" section).
For the ~2% of changes that are targeted for 3.0, the changelog verifier will break, but a maintainer can manually confirm a) it's a breaking change that shouldn't be backported, b) there's an entry in CHANGELOG-3.0.md. Assuming that's the case, they can add the skip-changelog label and everything works.

What I really like about the approach in this PR is that it gives us immediate benefit for the vast majority of changes (i.e. things that go into main then get backported to 2.x), while we still have an easy workaround (skip-changelog) for the handful of changes that target 3.0.

We would have to have the same thing for 2.x branch from where we backport changes to 1.x. The confusion I personally have is this: CHANGELOG,md has to correspond to the current working branch (main or 2.x or 1.x), this is the most logical expectation in my opinion. This approach breaks that and the alternative is to eliminate the confusion altogether (hence no CHANGELOG,md but branch specific ones)

Updated the workflow to work with this new structure: https://github.com/opensearch-project/OpenSearch/pull/13072

@andrross totally understand the pain, could we (if it makes sense) follow this convention, each branch has own CHANGELOG, fe:

CHANGELOG-3.x
CHANGELOG-2.x
CHANGELOG-1.x

I think it would be more straightforward to manage, what do you think? @peternied @dblock ?

Thanks @reta. I want to dig into this a bit more because I think it's pretty subtle. The 2.x branch having a CHANGELOG-2.x and CHANGELOG-1.x doesn't actually make sense because 1.x changes don't flow through the 2.x branch (Changes start on main and are backported directly to 2.x and 1.x if necessary. Sometimes changes to 1.x happen directly there because the security fix or whatever is only relevant to the now very old code on 1.x.) With our branching model, the main branch is kind of special in that most changes that are destined for the next minor release will never actually be unreleased from the perspective of main (because they will be released before any actual release comes out of main).
After talking through this, I think I still like the code that I have here. Let me walk through the scenarios:
As a user/casual observer of the development in OpenSearch:

I open the repository to the default branch (main) and want to know what changes are in flight and will land in the next release, where do I look? -> CHANGELOG.md
I open the repository to the 2.x branch and want to know what changes will land in the next 2.x release, where do I look? -> CHANGELOG.md
I open the repository to the 1.3 branch and want to know what changes will land in the next 1.3.x release, where do I look? -> CHANGELOG.md
I open the repository to main and want to see the breaking changes intended for the next major release (3.0), where do I look? -> CHANGELOG-3.0.md

As a contributor/maintainer:

I make a change to the default branch intended for the next release, where does the entry go? -> CHANGELOG.md
I make a security fix directly to 1.3 intended for the next patch release, where does the entry go? -> CHANGELOG.md
I make a forward looking, breaking change to main intended for 3.0, where does the entry go? -> CHANGELOG-3.0.md

@reta @kotwanikunal what do you think?

Basically, to summarize the justification in my previous comment, the upshot is that in all the 90+% use cases (for both contributors and observers), CHANGELOG.md is the file that you should be looking at or modifying. Even when we have a plan to release 3.0, once we cut the last minor release on 2.x everything will increment and main will contain CHANGELOG.md and CHANGELOG-4.0.md and all changes will be intended for 3.0 will go in CHANGELOG.md. The only scenario that requires thinking harder is when you make a breaking change intended for 3.0 and we have not yet cut the last planned minor release on 2.x (this scenario is definitely the outlier).

After talking through this, I think I still like the code that I have here. Let me walk through the scenarios:

Thanks @andrross , totally not against that, I think this change makes sense (and my suggestion is not better), I personally feel confused about treating main in a special way  (CHANGELOG.md + CHANGELOG-3.0.md) whereas all other release branches will have only (CHANGELOG.md). But I can live with that, certainly.

I personally feel confused about treating main in a special way (CHANGELOG.md + CHANGELOG-3.0.md)

@reta I understand, but we are already treating main in a special way today as it is the only branch in which a CHANGELOG has multiple sections. This commit is actually kind of just cosmetic but it should mostly eliminate a class of common errors with regard to the changelog.

❌ Gradle check result for 1c32252: FAILURE
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

#13046

❌ Gradle check result for 1c32252: FAILURE

#5329

@andrross Should this be backported, seems like the answer is yes, but it should be modified.  Can you take this on?

@peternied I can take care of it.


Missing Release notes for v1.6.2

The v1.6.2 is out on nuget, but I cannot find the release notes at the usual link https://learn.microsoft.com/en-us/windows/apps/windows-app-sdk/stable-channel
Where can I find it?

You should wait.
New version uploaded in nuget but not announced yet in github repo.

I know... and a week is passed since the release.

The rel notes are now live: https://learn.microsoft.com/en-us/windows/apps/windows-app-sdk/stable-channel#version-162-16241106002

I know... and a week is passed since the release.

A week? Unless an interesting time paradox has taken place, NuGet is only reporting that a day has elapsed. A day is normal for the time it takes between documentation being added to the Microsoft back end and when it shows up in the actual documentation.

To be honest, I didn't checked the 'date published' on Nuget, but only package version, that is: "1.6.241106002", and I have thought "241106" it was the date of publishing.


What is the format of the labels generated ?

Search before asking

[X] I have searched the Autodistill issues and found no similar feature requests.

Question
What is the format of the labels generated using autodistill. For example i have an image which contains 2 boxes. These 2 boxes have been identified but the labels generated contain more than 2 rows. What does that mean. Also the first column should signify the class number which should be 0 but it contains values from 0-8.
Additional
new_image2.txt

I have attached the label file - new_image2.txt for this image

Hello @charuu! What model are you using? If there are more than four values, it means the result is a polygon. Polygons can be transformed into xyxy coordinates if required using the supervision polygon_to_xyxy function: https://supervision.roboflow.com/detection/utils/#polygon_to_xyxy
With that said, if you are using a segmentation/detection combination model to get coordinates, you may only need the detection part. For instance, if you are using Grounded SAM, you may only need Grounding DINO.


Use weakref to deal with GC issues in .join(), rather than trying to clean up ourselves

This prevents the error messages in #650.
It also:

Fixes a bug where coro.join() returns the result the first time, but None on subsequent times
Replaces four ways to spell join (coro._join, coro.join(), _Join(coro), Join(coro)) with two (coro.join(), Join(coro))

I like the solution, looks great to me!

Thanks @eric-wieser for this really nice contribution! It's good to have someone on board that knows the less used features of Python really well. And to celebrate that, git chose the rather nice commit hash ffff6ad for you :-)


Revert "usage: retrieve usage per userId  (#156)"

This reverts commit 1731f13fef575d0db69006ba9dbef85c04ea019f.

What happened?


HealSpell buff

Healing spells have been strengthened to match ONE D&D.
CureWounds buffed content increases the recovery dice from 1 to 2, and the dice when upcasting also increases from 1 to 2.
HealingWord buffed content increases the recovery dice from 1 to 2, and the dice when upcasting also increases from 1 to 2.
PrayerOfHealing buffed content increases the recovery dice from 2 to 3, and the dice when upcasting also increases from 1 to 2.
MassHealingWord buffed content increases the recovery dice from 1 to 3, and the dice when upcasting also increases from 1 to 2.
MassCureWounds buffed content increases the recovery dice from 3 to 5, and the dice when upcasting also increases from 1 to 2.

incomplete. added on my end.


What is required to support the upcoming Monero fork to the V7 Algo?

@oliverw
Have you seen this?
https://getmonero.org/2018/02/11/PoW-change-and-key-reuse.html
https://github.com/xmrig/xmrig/issues/434
http://xmr.noctism.com/
Have you started working on this? Want some help?

@Gadflyii Not yet started but following the Monero Dev Blob. Required changes will most likely be in LibMultihash sub-project.

Would appreciate if support for the new the Monero Cryptonight V7 algo and previous versions will be easy to switch since many cryptonote coins exist and may either take time to adopt or not at all.
Something like (in CoinMetaData):
{ CoinType.XMR, (coin, alg)=> CryptonightMoneroV7 },
{ CoinType.ETN, (coin, alg)=> CryptonightMoneroV6 },

Don't worry. I'm planning to implement it exactly like that.

Want some help @oliverw  ?

@Gadflyii Absolutely. Just make sure that backwards compatibility is ensured.

@oliverw  I saw you added v7 in release 21,  Is there anything else to be done?


Re-create data table component by columns instead of by row

Building Figma tables based on columns instead of rows allows easier resizing of columns
https://www.untitledui.com/blog/create-tables-in-figma?utm_source=joeyabanks&utm_medium=email&utm_campaign=baseline-17-typography-variables-in-figma-are-here#build-table-components-with-columns-instead-of-rows


Fix agent ID card unlimited range

About the PR

Apparently agent ID card did not check range/obstruction.
Fixes #32446
Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Changelog
:cl:

fix: Agent ID card can now only copy access within interaction range.

Thanks for the fix!


specify type for variable 'tag' in modules/secure-bucket and modules/cloudtrail-baseline

This pull request fixes an issue (for me at least) with terraform 0.13.4 whereby terraform will refuse to create buckets as it is wrongly interpreting the tag variable (even with the default):
Error: Incorrect attribute value type

  on main.tf line 51, in resource "aws_s3_bucket" "access_log":
  51:   tags = var.tags
    |----------------
    | var.tags is "{\"Terraform\":true}"

Inappropriate value for attribute "tags": map of string required.

Two commits, the first (a28e9a3) fixes this bug, the second (162d6e4) adds types to the other vars and fixes a leading whitespace.

actually this seems like it might be happening in alot of places, seen it now in cloudtrail-baseline

There's one more instance of this in the root of the module but I'm closing this as at the moment he whole module has a problem akin to this one https://github.com/hashicorp/terraform/issues/26312 and whilst I spent a couple of hours working on this I can't any more


Remove redundant temperature sensor ID for CPU cores

Removed sensors:
CORE12, IPMI_SENSOR_ID 0xA9
CORE24, IPMI_SENSOR_ID 0xAE
CORE36, IPMI_SENSOR_ID 0xAF
Signed-off-by: Artem Senichev<EMAIL_ADDRESS>

It's just a mistyping. All other sensors of CPU cores have been already removed by MSI.


Add back-to-back-stem-and-leaf plots

Fulfills issue #1

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

Looks good. Two things:

Will you please make back-to-back a keyword option in stem-and-leaf. This way the usage would be something like: (stem-and-leaf foo :back-to-back t).  Same for split-stems (if you do that one), (stem-and-leaf x :split-stems t).  The keyword defaults for those should be nil
Squash all commits with a single commit message like "implement back to back stem plots".  See how to write a good commit message for some guidelines.

I've just edited my comment. I had split brain syndrome :-) because initially we were discussing split-stems and this PR implements back-to-back.  We really shouldn't mix &optional and &keys parameters, so back-to-back does need to be separate since it takes both x and y.  split-stems we can signal with a keyword to stem-and-leaf.
Thanks for the contribution!

Thanks for the review!

initially we were discussing split-stems and this PR implements back-to-back.

Ah, I think you're confusing me for @jg20019. I have not participated in the discussion for the other issue.

Done

Will you please make split-stems a keyword option ...

Done, plus I added it to the docstring.

Well done, thanks!


[Bug](topn opt) disable topn 2 phase read when storage policy is not …

…emtpy
Proposed changes
Issue Number: close #xxx

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

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 53.62 seconds
stream load tsv:          502 seconds loaded<PHONE_NUMBER>9 Bytes, about 142 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>4 Bytes


merge styles from get involved into main

This branch tries to reduce the size of get-involved.css by merging styles into main.css and removing obsolete and redundant styles. It also tries to fix some code-style blunders like spaces instead of tabs, new lines after selectors, etc.
Watch out for regressions in the alignment/resizing of columns and their contents on desktop and mobile. Also check to make sure regressions weren't introduced to header styles on the homepage and installation page.
fixes #255

There seems to be some serious changes in  header styles on get involved. Were these intentional?

Yes. I think if we feel like we need to change the header styles that should happen in main.css so we have consistent typography across the site. This branch should more or less keep the same sizes, but it most likely introduces changes in weight and color