Prod beta --- > Prod Stable

Prod beta --- > Prod Stable

Codecov Report

Merging #119 (c42e031) into prod-stable (9a84d2c) will increase coverage by 7.48%.
 Consider uploading reports for the commit 2f4b043 to get more accurate results

@@               Coverage Diff               @@
##           prod-stable     #119      +/-   ##
===============================================
+ Coverage        13.03%   20.52%   +7.48%
===============================================
  Files               31       37       +6
  Lines              514      692     +178
  Branches           101      129      +28
===============================================
+ Hits                67      142      +75
- Misses             415      513      +98
- Partials            32       37       +5

Impacted Files
Coverage Δ

src/Components/Reports/DownloadReport.js
0.00% <0.00%> (ø)

src/Components/Reports/SystemsPDFReport.js
0.00% <0.00%> (ø)

src/Routes/RosPage/RosPage.js
0.00% <0.00%> (ø)

src/Utilities/api.js
0.00% <0.00%> (ø)

src/Components/RosTable/RenderColumn.js
18.75% <25.00%> (+2.08%)
:arrow_up:

src/constants.js
72.41% <87.50%> (+5.74%)
:arrow_up:

src/Components/Reports/Util.js
92.30% <92.30%> (ø)

src/Components/Reports/Common/SystemsFirstPage.js
100.00% <100.00%> (ø)

src/Components/Reports/Common/SystemsTablePage.js
100.00% <100.00%> (ø)

src/Components/Reports/UtilTestData.js
100.00% <100.00%> (ø)

... and 1 more

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 9a84d2c...2f4b043. Read the comment docs.

Should we just include #118 or all the commits + #118 ? @PreetiW @kgaikwad

Should we just include #118 or all the commits + #118 ? @PreetiW @kgaikwad

@r14chandra till now we create merge commit which included all the commits+118 so I would say lets do the same while merging the PR. For reference check here: https://github.com/RedHatInsights/ros-frontend/commits/prod-stable


Feature Request: MFA support

I'm getting VMAIL_ERROR:  Invalid credentials (Failure), but I think it's because I have MFA set up on my gmail account.
I'm going to look into PR'ing this myself, but I'm not sure the Net::IMAP class support that,sso this might be tricky.

Solved: You need to create an app password for this to work.
https://support.google.com/mail/answer/185833?hl=en&visit_id=1-636298093628599059-604302321&rd=1


Error while adding convolution layer

Hello, this is my network:
from pathlib import Path
import pandas as pd
from keras.models import Sequential
from keras.layers import Embedding, Dense, Dropout, Activation, Conv1D, GlobalMaxPooling1D,MaxPooling1D,Flatten,LSTM,Reshape
from keras.preprocessing.text import Tokenizer, text_to_word_sequence
from keras import optimizers
import matplotlib.pyplot as plt
from keras.callbacks import EarlyStopping, ModelCheckpoint
import numpy as np
from keras.datasets import imdb
import csv

def get_data(path):
    p = Path(path)
    train = pd.concat([pd.read_csv(f, sep='\t', index_col=0)
                       for f in p.glob('train_???.tsv*')],
                      ignore_index=True)
    test = pd.concat([pd.read_csv(f, sep='\t', index_col=0)
                      for f in p.glob('test_???.tsv*')],
                      ignore_index=True)
    return ((train['Text'], train['Sentiment']),
           (test['Text'], test['Sentiment']))

# read data
(X_train, y_train), (X_test, y_test) = get_data(r'C:\NLP1')

# build vocabulary
t = Tokenizer(num_words=10000)
t.fit_on_texts(X_train)
t.fit_on_texts(X_test)

early_stopping = EarlyStopping(monitor='val_acc', # go through epochs as long as accuracy on validation set increases
                               patience=2,
                               mode='max')
# build a model
model = Sequential()
model.add(Dense(100, activation='relu', input_dim=10000))
model.add(Dropout(0.25))
model.add(Conv1D(filters = 100,
              kernel_size = 5,
              activation = 'relu',
              ))
model.add(Dense(50,activation = 'relu'))
model.add(Dense(1, activation='sigmoid'))
optimiz = optimizers.Adam(lr=0.0001, beta_1=0.9, beta_2=0.999, epsilon=1e-08, decay=0.0)
model.compile(loss = 'binary_crossentropy',optimizer = optimiz ,metrics = ['accuracy'])

# convert texts to TF-IDF matrices
X_train = t.texts_to_matrix(X_train, mode='tfidf')
X_test = t.texts_to_matrix(X_test, mode='tfidf')

# fit the model
hist = model.fit(X_train,y_train,validation_data = (X_test,y_test),epochs = 5,batch_size = 32,callbacks=[early_stopping])

On line ```
model.add(Conv1D(filters = 100,
kernel_size = 5,
activation = 'relu',
))

Got error
` Input 0 is incompatible with layer conv1d_1: expected ndim=3, found ndim=2`

How to fix this?

problem is with the input dimensions. Can you provide/check dimensions of X_train, y_train, X_test, y_test

As stated in the documentation: https://keras.io/layers/convolutional/#conv1d
Conv1d requires an input_shape=(batch_size, steps, input_dim)
Dense will outputs : (batch_size,100)

@Dref360 how about the solution ? how one satisfies the steps in the conv1d argument ?

Hello!
I'm not sure I understand what is steps in this case. Are you refering to the stride?


Scheduler Amends

Re-order to make it a little easier to read
Prevent overlapping on jobs likely to overlap (e.g. those running every minute)

:tada: This PR is included in version 5.5.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


grt: avoid segments out of layer range in congested routing

Fix https://github.com/The-OpenROAD-Project/OpenROAD/issues/6234

needs a format


File encoding in shebang lines

At the moment, all python files in the repo have encoding shebang line for Vim like this:
# vim:fileencoding=utf-8
This format is described in Python docs
To be precise with Vim modeline, that shebang line should look like this:
# vim: set fileencoding=utf-8 :
And this format is mentioned in PEP 263.
The issue is to find out if the existing line is correct for Vim (i.e. it is a correct Vim modeline string) and Vim actually performs it. Otherwise, change it to the line described in PEP 263.

According to description of the term "Shebang" in Wikipedia, shebang lines start with #!, so those lines with encoding setting are not really "shebang".


India map delhi is not available

Show result by click on map. Delhi is not availbe.

What exactly do you mean should happen? Please elaborate.

There is a map on page screen where we can interact with to get state wise result.
Delhi is not there on the map.
On 25 Mar 2020 2:58 am, Somesh Kar<EMAIL_ADDRESS>wrote:
What exactly do you mean should happen? Please elaborate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/someshkar/covid19india-network/issues/37#issuecomment-603514342, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKUQPQXOYRZMSL6H5YKISATRJEQXRANCNFSM4LS4VPMQ.

@dhruvaggarwal2709 Please raise this issue in the respective repo https://github.com/covid19india/covid19india-react


Update notification inside Portainer

The user should be informed that a new version of Portainer is available. An information message should be displayed within the UI when the used Portainer version is not the latest.

After more thoughts on this, I think that this check should be done in the frontend only and should be executed right after login (we don't want to check this every time the view change, as it might spam Github).

Closed in favor of #1649


Broken on 4.6.0

Fix remains to be seen.

Make sure to update the minimum system version after this.


Feature/fix module

Replace azure with azure-storage.
Replace createBlockBlobFromFile with createBlockBlobFromLocalFile

Apparently this project is not maintained


Record fix -- LogReaderSpec and SimpleLogSpec (LogWriterSpec)

when you say testfile.data -- wouldn't I have to fill that with my string still?

would I do
val input = new ArrayBuffer (testfile.data)
input.writeString(str)
testfile.data would consist of the str already right?

In the case of LogReaderSpec, you're doing:
var input = PagedBuffer(12)
input.writeString (str)
input.writeByte(0)
testfile.flush (input, 0).expectPass()

You can just do
val buf = new ArrayBuffer (testfile.data)
buf.writeString (str)
buf.writeByte(0)

A similar shortcut can be taken in SimpleLogSpec for reading items from the test file.

So the shortcut worked for SimpleLogSpec but for LogReaderSpec
I tried:
var testfile = StubFile(new ArrayByte, 0)
var buf = ArrayBuffer(testfile.data)
buf.writeString (str)   // this line is an error: Buffer overflow, 1 required, 0 available
buf.writeByte(0)
var logreader = new LogReader(testfile, DriveGeometry(10, 10, 16384))

The buffer overflow seems to occur because there is not any available space to write. Comes from the method at com.treode.buffer.ArrayBuffer.requireWritable(ArrayBuffer.scala:61)

Yeah, the ArrayBuffer does not grow space, so you need to give yourself some space first.
var testfile = StubFile(1 << 16, 0)
var buf = ArrayBuffer(testfile.data)
buf.writeString (str) // no more error
buf.writeByte(0)

hmm...the error still shows up

I've changed the constructor methods for ArrayBuffer. Now there's readable and writable. You can use each as needed, rather than needing to fiddle with writePos.

Thanks that worked!


Copy clBLAS header files upon install

Projects that only depend on the clBLAS routines implemented by
ISAAC do not have any other clBLAS dependency than ISAAC. However, they
need the clBLAS header files installed. This fix makes it so that such
projects only need to install ISAAC.

Sorry for the delay. Actually clBlas should be in driver/external/CL. I'm a bit wary of installing CL files by default because it may pollute systems that use CUDA only.

I agree with you actually. Thanks for the PR and sorry again for the delay.

👍 Thanks!


more zxcvbn properties

Hi @Loupehope
Thanks for your excellent work! I got more properties from zxcvbn because I need them in my project.
Maybe someone else need them too. Thank you again!

Thanks for your comments. Let me try.


add cryptomove permission

Description
Add permission to publish the Jenkins Plugin CryptoMove, which can hopefully be listed on the Jenkins web page. https://github.com/jenkinsci/cryptomove-plugin

Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[x] Add link to resolved HOSTING issue in description above

For a new permissions file only

[x] Make sure the file is created in permissions/ directory
[x] artifactId (pom.xml) is used for name (permissions YAML file).
[x] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[x] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

Please add a link to the hosting issue that was completed. This helps us keep track of things.

@slide here you go https://issues.jenkins-ci.org/browse/HOSTING-789?jql=text ~ "cryptomove" thank you!

@slide - can you please take a look at this screenshot? I thought this PR allowed me to have the proper permissions to publish versions?
Or do the versions get automatically published in CI? The docs aren't very clear on this.
Thank you,

You use the mvn release:prepare release:perform command to do releases after setting up your settings.xml, you shouldn't be trying to upload manually. Did you follow this information on how to release? https://jenkins.io/doc/developer/publishing/releasing/
It also looks like your path is different for your plugin, something I did not catch when merging the PR. The yml file you submitted has org/jenkins-ci/plugins/ for the path, but it looks like your pom.xml has io.jenkins.plugins, these need to match up. You can submit a PR to the yml file to fix the path. Sorry for not catching that.

Yes I tried with that step and received an access denied error from the
Jenkins maven server so I tried with the UI.
The path mishap seems to be the cause of my issue. I'll submit a pr soon,
thank you.
El mié., 19 de junio de 2019 08:56, Alex Earl<EMAIL_ADDRESS>escribió:

You use the mvn release:prepare release:perform command to do releases
after setting up your settings.xml, you shouldn't be trying to upload
manually. Did you follow this information on how to release?
https://jenkins.io/doc/developer/publishing/releasing/
It also looks like your path is different for your plugin, something I did
not catch when merging the PR. The yml file you submitted has
org/jenkins-ci/plugins/ for the path, but it looks like your pom.xml has
io.jenkins.plugins, these need to match up. You can submit a PR to the yml
file to fix the path. Sorry for not catching that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jenkins-infra/repository-permissions-updater/pull/1181?email_source=notifications&email_token=ALE73US6ILKVGKEMFFQOI73P3JJMPA5CNFSM4HZEKYAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYCKRNQ#issuecomment-503621814,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALE73UTAUOF3OGE2XKTOB73P3JJMPANCNFSM4HZEKYAA
.


Unable to find template file

Here is my cookbook at chef/cookbooks/mage/recipes/default.rb:
#
# Cookbook Name:: mage
# Recipe:: default
#

magento_vhost 'www.copiousinc.com' do
    docroot       '/vagrant'
    fpm_location  'unix:/var/run/php7.0-fpm.sock'
    version       2
    action        :create
end

Here is the error I am getting:
==> web: ================================================================================
==> web:
==> web: Error executing action `create` on resource 'magento_vhost[www.copiousinc.com]'
==> web:     ================================================================================
==> web:
==> web:     Chef::Exceptions::FileNotFound
==> web:     ------------------------------
==> web:     template[/etc/nginx/sites-available/www.copiousinc.com] (/var/chef/cache/cookbooks/cop_nginx/resources/magento_vhost.rb line 14) had an error: Chef::Exceptions::FileNotFound: Cookbook 'mage' (0.1.0) does not contain a file at any of these locations:
==> web:       templates/ubuntu-16.04/magento/mage2.vhost.conf.erb
==> web:       templates/ubuntu/magento/mage2.vhost.conf.erb
==> web:
==> web:   templates/default/magento/mage2.vhost.conf.erb
==> web:       templates/magento/mage2.vhost.conf.erb
==> web:

It looks like it is only checking my mage cookbook for the template. Maybe you need to set a "cookbook" value here?

Fixed in https://github.com/copious-cookbooks/nginx/commit/d2768e16c81ce34efc7c912906dbf1ab5d01e8de, thanks @dmatthew!


Author of this code is Chuck Norris! Please change the author!

I honestly belive that such powerful project can make only Chuck Norris. Chuck Norris can write nocode that will compile and work on every platform. 0 strings but ∞ capabilities. This is Chuck Norris style!

also chuck norris is the only one who can write this framework without a computer !

Actually, nocode is Chuck Norris in code form, so awesome that no code has to be involved, it does anything.

I'd worry Chuck is coming after the people creating issues


js-beautify inserts space in hexadecimal bigint literals

Description
The input is parsed as a BigInt, the output is parsed as a Number and an identifier.
Input
The code looked like this before beautification:
0xFFn

Expected Output
The code should have looked like this after beautification:
0xFFn

Actual Output
The  code actually looked like this after beautification:
0xFF n

Steps to Reproduce
$ echo "0xFFn" | js-beautify

Environment
OS: Ubuntu 20.04.2 LTS
Settings
I'm not sure, but i think it's with the default settings

sure 🙂


iSCSI for PXE diskless

RancherOS Version: (ros os version)
v1.5.1
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
baremetal, diskless ipxe booted server nodes.
Hello! First off, thanks to all who have worked on this. It's fantastic.
I work at a university (CS department), and would like to make use of RancherOS as well as rancher for our Kubernetes experiments (probably becoming a "production" env later on).
I have some servers I'd like to boot via iPXE (working) that should persist their state to iSCSI disks.
The problems I am facing are probably similar to most trying to do this. I'm not clear on where the capabilities lie currently, and what to do to get RancherOS to mount the remote iSCSI storage.
I have played around a fair amount already, so feel free to ask me to do some testing and so on.  I am also happy to help with any feedback needed for issue #2636

Unfortunately, booting ROS via iPXE currently only supports local storage.

Thanks for the reply. I will then use Ubuntu for the nodes with no disks. They only have 8 Gig ram, so having ROS in memory and running loads might end up making them not very useful. Also, persistence is helpful for node reboots. I don't want to have to remove and add nodes each time a reboot is needed.
This might be a really nice feature though. Especially for those who want to use infrastructure for a few different things. Like at the university, we could shut down a lab over night to run a massively parallel work load, simply by booting ROS from the network with some iSCSI persistence partitions.
Also, it seems to me that Ubuntu being the base OS, vs booting PXE ROS, is a little faster/more stable in terms of deploying containers. Does that sounds right? Or should ROS be significantly faster, experientially?

So I discovered how to get the node to "persist" by keeping it's name the same. This seems to work well enough. I will therefore close this, but if it becomes a feature request for future, that wouldn't be a terrible thing.


Recycler Overhaul

About the PR

does a bunch of things

Adds new directional sprites for the recycler
Makes the recycler invincible to prevent permanent removal
the recycler now returns the full amount of materials when recycling objects.
Properly implements breakage, allowing it to be repaired and adding custom sprites
Allows the emag to be undone when breaking a recycler
Removes the material reclaimer
Fixes bug where the recycler could extract solutions it wasn't meant to (fiber from clothing, etc)

fixes #30500
Why / Balance

These changes are primarily to facilitate salvage's usage of the recycler in the future. The material reclaimer was really underwhelming and, though it was meant for salvaging originally, just came off as a far less flavorful version of the regular recycler. On top of that, a smaller version that is better for shuttles isn't going to be necessary once the shuttle is removed in the future.
The changes to making it invincible and the emag repairable largely just serve to not let it get trivially griefed. As its now the sole way of extracting materials, you shouldn't just be able to smack it until it breaks and gimp salvage.
Media

Requirements

[x] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: The recycler can now be broken down and repaired.
add: Breaking the recycler now removes the EMAG effect.
tweak: The material-reclaiming efficiency of the recycler has been increased.
fix: Fixed a bug where inaccessible solutions could be extracted from entities via the recycler.
remove: Removed the material reclaimer.

based!! Finally janitors have proper reason to use the janitor other than ocd

I think that emagged recycler being instant gib is awful all around. If you'd bother to change that!

It's a easily avoided instant gib (just don't go down disposals)

I think that emagged recycler being instant gib is awful all around. If you'd bother to change that!

after the first gib or so people pretty quickly realise it's emagged, + it takes tc for syndie to buy emag

The material reclaimer getting removed probably facilitates a breaking changes entry. There's a chance that some downstreams might use it as an important part of their gameplay loop.

agreed. Recycler being actually useful now is pog but removing the recycler could break things downstream.

Not to mention the reclaimer is pretty helpful for extracting reagents from certain items. Is there anything that mimics that function upstream?

It kinda makes cargos job more annoying and forces QM to disposals to reclaim said materials? I dunno if I agree with the removal of the material reclaimer.

it's more a janitor's job
to reclaim junk and give materials to those who need them

the recycler should be mapped into salvage at some point. I'll probably put out a mapping ping once this is done just to get things rolling.

the recycler should be mapped into salvage at some point. I'll probably put out a mapping ping once this is done just to get things rolling.

replacing disposals or?

replacing disposals or?

i imagine you could either put disposals next to salv and make it a combo or just have a second recycler inside of salvage itself

replacing disposals or?

i imagine you could either put disposals next to salv and make it a combo or just have a second recycler inside of salvage itself

Damn, I love being the guardian of disposals as a Jani, always saw it as apart of service.

i imagine you could either put disposals next to salv and make it a combo or just have a second recycler inside of salvage itself

for some maps like core its not feasible to entirely move dispo near salv unless causing many more major issues, so i might as well add a second one then, thanks!

what if an emagged recycler, instead of gibbing the target, dealt 500 slash damage and made the target scream? would be funny

what if an emagged recycler, instead of gibbing the target, dealt 500 slash damage and made the target scream? would be funny and also somewhat recoverable

out of this pr scope

material reclaimer removal needs breaking change section filled out


Documentation: missing default values in jsdoc

jsdoc (all options) is missing the default values we have in our documentation.

Very nice how the values and types were added :D


The capture action does not keep mob's nbt data anymore

API: Paper 1.20.6-149
Magic: 10.9.2
In previous versions the capture action would keep the nbt data of the mob it was used on but now it seems to give spawn eggs with random nbt data.

Thanks for the report!
This is fixed in the latest dev build here: https://jenkins.elmakers.com/job/MagicPlugin/4302/


How to prevent list scroll while swiping an item

Describe the bug
List scrolls everytime i swipe an item in the list. this is happening every time i swipe any item in the list
To Reproduce
run the swipe list view and try to swipe any item in the list
Environment :
OS: Android
RNSLV Version: "^3.2.4"
RN Version: "0.63.2"

Hi @Yandamuri the SwipeListView is locked to scrolling when an item is swiped so I'm not sure how this could be happening? Could you show me a gif of the issue and some reproducible sample code? I can't currently reproduce on my side, thanks!

@jemise111 I am not if it happened in my device only i found only once and i am unable to reproduce it now. Anyway thank you for your response. If i can reproduce in the future i will come up with code .


fix(lint): disable selector-type-no-unknown

Prevent errors on custom elements in #346, see https://github.com/catppuccin/userstyles/pull/346#issuecomment-1820213474.

I really liked the added typo protection that selector-type-no-unknown added, and I think it's rare enough where a workaround for each userstyle isn't that much effort. I'll push what I came up with.

Lichess lint passes for me when I create ./styles/lichess/.stylelintrc.json with:
{
  "rules": {
    "selector-type-no-unknown": [
      true,
      { "ignore": ["signal", "index", "fill"] }
    ]
  }
}

8ed27aa works but this really isn't getting any more maintainable.
Luckily it turned out that turning on nodeModulesDir and not passing in config into stylelint.lint does everything we want for us.
The config file will have to include an extends to the repo root, compared to the previous impl:
{
  "extends": "../../.stylelintrc.js",
  "rules": {
    "selector-type-no-unknown": [true, { "ignore": ["signal", "index", "fill"] }]
  }
}

That's way better, thanks!


Use of principled shader wrapper

Blender has a wrapper util for principled shader.
See https://developer.blender.org/T70666
Can be great to use it and enhance it with our needs too.
Link to #462

Is this still relevant? What is the node_shader_utils.py utility and its applicability here?

Closing this ticket, as we never implented it, and now we have develop a system that retrieve all needed info about the Principled Shader


Move aws-cli ansible logic

https://www.pivotaltracker.com/story/show/156348355

Adds ansible logic for installing aws-cli - this is being extracted from checkbot https://github.com/codeship/checkbot/pull/1084

@LarsFronius actually we don't need the S3 implementation. The new implementation for codeship_aws, when it doesn't find a script with the given name, fallbacks to aws $@ which is exactly what the current version (in checkbot) is doing. The reason why we don't have those scripts in the first place (for S3) is because we're calling aws s3 directly.

Given it's such straight forward simple one-liner, the only reason for an "implementation" as a wrapper script would be because we want to have it follow the same "call path". It's definitely no necessity to make things work.


[snet-farmers] Strategy

This strategy returns the yield-farming pool balances of the voters for the SingularityNET project.

@ChaituVR
All checks have been passed. could you merge this PR?


netease-cloud-music: init at 1.2.1

nixos/doc/manual/release-notes/rl-2009.section.md said netease-cloud-music depends on deepin desktop environment and because deepin desktop environment had been removed, netease-cloud-music is also removed.
However, it seems that netease-cloud-music can work without deepin desktop environment. So I package it again.
https://github.com/NixOS/nixpkgs/issues/94870 removed DDE
https://github.com/NixOS-CN/flakes/pull/51 recommend to push this package to NixOS/nixpkgs
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

CC: @linyinfeng @romildo @worldofpeace @jtojnar @ttuegel

Remove some libraries.

Now, this is result:
$ tree -L 3 result/
 result
├──  bin
│  └──  netease-cloud-music
├──  lib
│  └──  netease-cloud-music
│     ├──  libdouble-conversion.so.1
│     ├──  libjpeg.so.8
│     ├──  libqcef.so.1 -> libqcef.so.1.1.4
│     ├──  libqcef.so.1.1.4
│     ├──  libQt5Core.so.5
│     ├──  libQt5DBus.so.5
│     ├──  libQt5EglFSDeviceIntegration.so.5
│     ├──  libQt5Gui.so.5
│     ├──  libQt5Network.so.5
│     ├──  libQt5Qml.so.5
│     ├──  libQt5Svg.so.5
│     ├──  libQt5WebChannel.so.5
│     ├──  libQt5Widgets.so.5
│     ├──  libQt5X11Extras.so.5
│     ├──  libQt5XcbQpa.so.5
│     ├──  libQt5Xml.so.5
│     ├──  libudev.so.1
│     ├──  libXtst.so.6
│     ├──  plugins
│     └──  qcef
└──  share
   ├──  applications
   │  └──  netease-cloud-music.desktop
   ├──  doc
   │  └──  netease-cloud-music
   └──  icons
      └──  hicolor

QApplication: invalid style override passed, ignoring it.
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function TLS_client_method
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function SSL_CTX_new
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function OPENSSL_init_ssl
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function ERR_get_error
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function ERR_get_error
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function TLS_client_method
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function SSL_CTX_new
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function OPENSSL_init_ssl
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function ERR_get_error
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function ERR_get_error
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function TLS_client_method
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function SSL_CTX_new
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function OPENSSL_init_ssl
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function ERR_get_error
No appenders assotiated with category qt.network.ssl
[Warning] <> QSslSocket: cannot call unresolved function ERR_get_error

I have not met these warnings. But for me, ibus cannot work:

(netease-cloud-music:722574): IBUS-WARNING **: 00:12:55.544: Unable to connect to ibus: The given address is empty
(netease-cloud-music:722574): IBUS-WARNING **: 00:13:02.329: Events queue growing too big, will start to drop.

NetEase Cloud Music for Linux is no longer officially maintained, and it has been taken down from the official website (https://music.163.com/#/download).
I suggest pushing it to the Nix User Repository

I suggest pushing it to the Nix User Repository

Yes, before this PR is created, I use it in my NUR.

Any other point about this PR? Now some conflict have been occured :cry:


Web GUI not working any more

With the latest changes and a fresh install of all the Web GUI does no longer work - lighttp is unable to start:
pi@allsky:~ $ sudo systemctl status lighttpd.service
● lighttpd.service - Lighttpd Daemon
   Loaded: loaded (/lib/systemd/system/lighttpd.service; enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Wed 2020-08-26 09:01:55 CEST; 17s ago
  Process: 1564 ExecStartPre=/usr/sbin/lighttpd -tt -f /etc/lighttpd/lighttpd.conf (code=exited, status=255/EXCEPTION)

Aug 26 09:01:55 allsky systemd[1]: lighttpd.service: Service RestartSec=100ms expired, scheduling restart.
Aug 26 09:01:55 allsky systemd[1]: lighttpd.service: Scheduled restart job, restart counter is at 5.
Aug 26 09:01:55 allsky systemd[1]: Stopped Lighttpd Daemon.
Aug 26 09:01:55 allsky systemd[1]: lighttpd.service: Start request repeated too quickly.
Aug 26 09:01:55 allsky systemd[1]: lighttpd.service: Failed with result 'exit-code'.
Aug 26 09:01:55 allsky systemd[1]: Failed to start Lighttpd Daemon.

Check the logs /var/log/lighttpd/error.log and /var/log/allsky.log

Hey, I'm still having problems to set up everything...I now get the Message ERR_CONNECTION_REFUSED when I try to access  my local IP http://<IP_ADDRESS>/ or http://<IP_ADDRESS>/public.php or http://allsky.local
can someone help me, the raspberry is totally new to my so I don't know where to look for permissions ect...

so my sd card died and I had to reload my Raspberry Pi.  I am unable to get the gui to load also.
/var/log/lighttp/error.log
2020-08-26 16:08:11: (server.c.1464) server started (lighttpd/1.4.53)
2020-08-26 16:08:26: (server.c.2059) server stopped by UID = 0 PID = 1
/var/log/allsky.log
Aug 26 16:03:34 allsky allsky.sh[499]: can't get debug descriptor: Resource temporarily unavailable
Aug 26 20:18:50 allsky allsky.sh[578]: can't get debug descriptor: Resource temporarily unavailable
Aug 26 20:26:06 allsky allsky.sh[581]: can't get debug descriptor: Resource temporarily unavailable
Aug 26 20:40:56 allsky allsky.sh[571]: can't get debug descriptor: Resource temporarily unavailable

Same problem, when I tried to check the log. It is telling me the lighttpd folder is owned by www-data

I just removed a line from lighttpd.conf that was added recently through a pull request. Try running sudo gui/install.sh again and that should fix it.

Just tried it and still not loading
pi@allsky:~ $  sudo systemctl status lighttpd.service
● lighttpd.service - Lighttpd Daemon
Loaded: loaded (/lib/systemd/system/lighttpd.service; enabled; vendor preset: enabled)
Active: failed (Result: exit-code) since Thu 2020-08-27 14:40:22 EDT; 1min 35s ago
Process: 636 ExecStartPre=/usr/sbin/lighttpd -tt -f /etc/lighttpd/lighttpd.conf (code=exited, status=255/EXCEPTION)
Aug 27 14:40:22 allsky systemd[1]: lighttpd.service: Service RestartSec=100ms expired, scheduling restart.
Aug 27 14:40:22 allsky systemd[1]: lighttpd.service: Scheduled restart job, restart counter is at 5.
Aug 27 14:40:22 allsky systemd[1]: Stopped Lighttpd Daemon.
Aug 27 14:40:22 allsky systemd[1]: lighttpd.service: Start request repeated too quickly.
Aug 27 14:40:22 allsky systemd[1]: lighttpd.service: Failed with result 'exit-code'.
Aug 27 14:40:22 allsky systemd[1]: Failed to start Lighttpd Daemon.
/var/log/lighttpd/error.log
2020-08-27 14:39:33: (server.c.1464) server started (lighttpd/1.4.53)
2020-08-27 14:39:49: (server.c.2059) server stopped by UID = 0 PID = 1

Oh sorry, there's another step to do before running the installer: Open gui/lighttpd.conf and remove this line mimetype.assign             = (".css" => "text/css", )

Thank you Thomas, that worked for me.

Also works now for me.
Thanks!!!!

Hey @thomasjacquin
Thanks man! Awesome! It works for me too! Awesome project, thanks for sharing and doing great support! I just bought you the next cup of coffee!;)
Greetings from Germany!

Thanks @Respi84. It's much appreciated!


build and inspect

build and inspect
this commits extends the UI with the following flag:

--bin
--example
--lib
--release

if one of --bin, --example or --lib is used then cargo-$tool will first
build the Cargo project and then proceed to run the llvm tool on the output
artifact -- that is the output artifact will be the first argument passed to the
llvm tool.
--release controls whether the project is compiled with optimizations or not.
if none of these three flags is passed then the old behavior is used. No path to
an artifact is passed to the llvm tool and the user is expected to provide it.
this new logic supports workspaces and the CARGO_TARGET_DIR env variable.
closes #15
r? @rust-embedded/tools (anyone)
This builds on top on #19 so that should be merged first

Sample outputs:
$ cargo new --lib foo && cd $_

$ cat >src/lib.rs <<'EOF'
#![no_std]
#[no_mangle]
pub fn foo(x: u32, y: u32) -> u32 {
    x + y
}
EOF

$ cargo objdump --lib --release -- -disassemble 2>/dev/null

/home/japaric/tmp/foo/target/release/libfoo.rlib(foo-136cd7af94a84b98.foo.awvzjwiu-cgu.0.rcgu.o):       file format ELF64-x86-64

Disassembly of section .text.foo:
foo:
       0:       8d 04 37        leal    (%rdi,%rsi), %eax
       3:       c3      retq

$ cargo objdump --target aarch64-unknown-linux-gnu --lib --release -- -disassemble 2>/dev/null

/home/japaric/tmp/foo/target/aarch64-unknown-linux-gnu/release/libfoo.rlib(foo-05ebc283581f4ba3.foo.9kf7hsz7-cgu.0.rcgu.o):     file format ELF64-aarch64-little

Disassembly of section .text.foo:
foo:
       0:       20 00 00 0b     add     w0, w1, w0
       4:       c0 03 5f d6     ret

$ cargo objdump --target riscv32imac-unknown-none-elf --lib --release -- -disassemble 2>/dev/null

/home/japaric/tmp/foo/target/riscv32imac-unknown-none-elf/release/libfoo.rlib(foo-05ebc283581f4ba3.foo.9kf7hsz7-cgu.0.rcgu.o):  file format ELF32-riscv

Disassembly of section .text.foo:
foo:
       0:       33 85 a5 00     add     a0, a1, a0
       4:       67 80 00 00     ret

$ cargo objdump --target thumbv7m-none-eabi --lib --release -- -disassemble 2>/dev/null

/home/japaric/tmp/foo/target/thumbv7m-none-eabi/release/libfoo.rlib(foo-05ebc283581f4ba3.foo.9kf7hsz7-cgu.0.rcgu.o):    file format ELF32-arm-little

Disassembly of section .text.foo:
foo:
       0:       08 44   add     r0, r1
       2:       70 47   bx      lr

Help text:
$ cargo nm -h
cargo-nm 0.1.2
Proxy for the `llvm-nm` tool shipped with the Rust toolchain.

USAGE:
    cargo-nm [FLAGS] [OPTIONS] [--] [args]...

FLAGS:
    -h, --help       Prints help information
        --lib        Build only this package's library
        --release    Build artifacts in release mode, with optimizations
    -V, --version    Prints version information
    -v, --verbose    Use verbose output

OPTIONS:
        --bin <NAME>         Build only the specified binary
        --example <NAME>     Build only the specified example
        --target <TRIPLE>    Target triple for which the code is compiled

ARGS:
    <args>...

The specified <args>... will all be passed to the final tool invocation.

ping @ryankurte @therealprof anyone of you up for reviewing this?

Other than the ergonomics, LGTM.

I fixed the clippy warnings and added some examples to the help text. It now looks like this:
$ cargo nm -h
cargo-nm 0.1.2
Proxy for the `llvm-nm` tool shipped with the Rust toolchain.

USAGE:
    cargo-nm [FLAGS] [OPTIONS] [--] [args]...

FLAGS:
    -h, --help       Prints help information
        --lib        Build only this package's library
        --release    Build artifacts in release mode, with optimizations
    -V, --version    Prints version information
    -v, --verbose    Use verbose output

OPTIONS:
        --bin <NAME>         Build only the specified binary
        --example <NAME>     Build only the specified example
        --target <TRIPLE>    Target triple for which the code is compiled

ARGS:
    <args>...

The specified <args>... will all be passed to the final tool invocation.

To see all the flags the proxied tool accepts run `cargo-nm -- -help`.

EXAMPLES

`cargo nm --lib`                           - lists all symbols
`cargo nm --lib -- -print-size -size-sort` - lists all symbols sorted by size (smallest first)

I also fixed the postprocessing logic applied to the output of size; it was postprocessing the output in Berkely format when it shouldn't.

The behaviour is still very weird, if I specify an unexpected parameter I get a useless message:
error: Found argument '--disassemble' which wasn't expected, or isn't valid in this context

USAGE:
    cargo-objdump <binary-name> --release

For more information try --help

But if I use --help I get a useful message like the one posted above. Why can't I get the same useful message in both cases?

Fixed the clippy warning and tweaked the help text to make clearer that the arguments for the tool must be passed after the --.

Why can't I get the same useful message in both cases?

That's how clap behaves. I'm not sure if there's a way to improve the error messages. You get similar error messages if you misuse cargo-rustc:
$ cargo rustc --release  --example i2c_hal_ssd1306alphabeter --emit=obj
cargo rustc --release  --example i2c_hal_ssd1306alphabeter --emit=obj
error: Found argument '--emit' which wasn't expected, or isn't valid in this context

USAGE:
    cargo rustc --example <NAME>... --release

For more information try --help

Hm, we should take it up with clap then. It's certainly a usability deficiency.

bors r+


Airflow dag execution error

My environment is Airflow running in Docker. Python 3.6 downgraded from 3.7 and Airflow 10.7 ->upgraded from 10.4 to try and solve this issue. Here is the error on docker logs airflow.  No dags are getting executed or scheduled or triggered although there are being correctly picked up and displayed on the web UI.
[2020-01-04 03:30:03,454] {{dagbag.py:90}} INFO - Filling up the DagBag from /usr/local/airflow/dags
[2020-01-04 03:30:06,753] {{dagbag.py:405}} ERROR - [Errno 22] Invalid argument
Traceback (most recent call last):
File "/usr/local/airflow/.local/lib/python3.6/site-packages/airflow/models/dagbag.py", line 389, in collect_dags
safe_mode=safe_mode)
File "/usr/local/airflow/.local/lib/python3.6/site-packages/airflow/models/dagbag.py", line 210, in process_file
zip_file = zipfile.ZipFile(filepath)
File "/usr/local/lib/python3.6/zipfile.py", line 1131, in init
self._RealGetContents()
File "/usr/local/lib/python3.6/zipfile.py", line 1216, in _RealGetContents
fp.seek(self.start_dir, 0)
OSError: [Errno 22] Invalid argument
Thank you anyone who has encountered this issue before.

@mangoman18 do you have an init.py file at the root of your zip? If yes, make sure to build the zip without it.
Try something like this:
zip -x __init__.py --include \*.py -r dags.zip *


WalletConnect. QR code for pairing isn't shown in Brave browser

Steps to reproduce:

Open Nova Spektr web onboarding screen
Select Nova Wallet or WalletConnect

Expected result:

QR code for pairing is shown

Actual result:

QR code for pairing isn't shown

More info:
If to add a WalletConnect, Nova Wallet from the wallet tab, then it works well.
https://github.com/user-attachments/assets/94e7ad20-3e41-4796-af95-17aae666226d

Hello @pgolovkin  A support ticket has been opened for you. Kindly Talk to Support on the Live support page so that your issue can be looked into. CLICK HERE

I've made some research and found problem with Brave Shield. It tries to prevent fingerprinting and one of the "feature" is a limit for opened WebSocket (now it's 30 concurrent connection). That means Wallet Connect socket usually doesn't start and user can't connect wallet on onboarding step. Current workaround it to disable fingerprinting protection for our application.


"manage_config" boolean to disable config.json creation

Small tweak that allows the admin to disable config.json creation in favor of having the module only handle installation.  This can be valuable when leveraging tools like consul-template to dynamically build configs vs the more static puppet approach.

Hey @Justin-DynamicD, thanks for the PR! Can you please rebase?

Hi @Justin-DynamicD, there are still merge conflicts. Let me know if you need some help in fixing them. You can always join the IRC channel #voxpupuli on freenode. That's the channel from the puppet community, they are always happy to help out with questions about git and puppet and related tooling.


Draggable buttons not read properly by screenreader

Reproduction
https://formkit.link/1ec7860cf08f8ebd8a6c2dfc10b95720
Describe the bug
It reads "button list item". If i go to the html to the span with Move down etc. and remove all the classes and put sr-only instead it works like it is supposed to.
Environment
• OS: [e.g. iOS]
• Browser [e.g. chrome, safari]
• Version [e.g. 22]

@LindaMeijering Just to be clear, did you mean to include the prop draggable on the repeater in your playground example? Or are you referring to another element?

@sashamilenkovic  My mistake, i mean the up down and delete buttons on the right side of the element. Not the draggable prop.

@LindaMeijering This is another issue that seems fine when using the native screen reader (Apple Voiceover). If you inspect the element, you can see the span contains the inner text "Move Down". Not sure we want to make any sort of change just because this extension is not working properly (which is scheduled to be deprecated) :  https://chromewebstore.google.com/detail/screen-reader/kgejglhpjiefppelpmljglcjbhoiplfn?hl=en-US&utm_source=ext_sidebar,

@sashamilenkovic Microsoft narrator is indeed fine 👍

Same for this one, Microsoft narrator works well.


Deleted this Unwanted File

@king-slayer-00, The pr looks good, but next time give a more elaborate title. Using "this unwanted file" is not appreaciated. You can mention in the description too about the pr you did. Also create issues first and then mention the issue in the pr.


Make instructions on usage in the browser clearer

I have been spending some time with this already, but I can't get it to work. I can't be the only one.
The readme has basic usage instructions for Node.js and browser, but it is never explained how to include the Javascipt files in your project. I have tried

adding index.js and math.js via <script> tag
using ESM syntax with import from
letting Browserify create a compiled javascript file

None of it worked. I'd have given up, but it does work on NPM Runkit, so I know there is a way to do it, which is frustrating. I tried looking for a CDN version of the code, but it seems it does not yet exist either.
I don't know whether it makes sense to provide a compiled, single-file release version for the browser, but it sure would have made including this on a website easier.

Yes, that would be very helpful I think. I have built my own version of it in the meantime:
https://github.com/oelna/signed-qr-codes/blob/main/noble-bls.js
It works as an ES6 module in the browser (can be used via import { nobleBLS as bls } from './noble-bls.js';)
Disclaimer: I know nothing about any of this! I fiddled and deleted until something worked, but this should really be done by someone who knows what they are doing.

See if it's clear enough right now.


Default NPC's to be unlinked and add number to end

A system setting to set these checkboxes (checked) as default.

Not really needed


Fixed more typos in the docs

Go ahead and call me a Grammar Nazi, but everyone knows that Grammar Romans are superior.

I found some errors in the property values and types (I didn't look at those yet), I'll fix them tomorrow.


Failed to update registry credential 1c37, Resource type [registryCredential] is not updatable

Rancher V1.2
objectpartners/rancher-ecr-credentials:1.1.0
Got a failure shown as follows,
Updating ECR Credentials
Failed to update registry credential 1c37, Resource type [registryCredential] is not updatable
I am not sure if i was missing something...

I've seen this reported before - https://github.com/objectpartners/rancher-ecr-credentials/issues/5.
Can you verify that you have the appropriate lables configured on the service? https://github.com/rancher/community-catalog/blob/master/templates/ecr/2/docker-compose.yml#L8-L9
Otherwise you have to set the CATTLE_ envvars per the README.

ECRCredentials:
environment:
AWS_REGION: ${AWS_REGION}
AWS_ACCESS_KEY_ID: ${AWS_ACCESS_KEY_ID}
AWS_SECRET_ACCESS_KEY: ${AWS_SECRET_ACCESS_KEY}
CATTLE_URL: ${CATTLE_URL}
CATTLE_ACCESS_KEY: ${CATTLE_ACCESS_KEY}
CATTLE_SECRET_KEY: ${CATTLE_SECRET_KEY}
labels:
io.rancher.container.pull_image: always
io.rancher.container.create_agent: 'true'
io.rancher.container.agent.role: environment
tty: true
image: objectpartners/rancher-ecr-credentials
stdin_open: true
The above was my scripts. I followed the steps shown in http://rancher.com/using-amazon-container-registry-service/, plus CATTLE_URL, CATTLE_ACCESS_KEY and CATTLE_SECRET. I dont think I missed out something...

And you've define the registry in Rancher for the environment that this container is running in?

i actually directly type in my key and secret in the scripts instead of  using ${}.

yes i've defined my ecr already

@OniReimu ok, i think I know the problem here - are you passing global credentials to your container?
If so, that won't work unless you change the CATTLE_URL to point to a specific environment (http://my-rancher.domain.com/v1/project/)

and if you're running on the rancher cluster, you dont' need to explictly pass the CATTLE_ envvars to the container. Rancher will auto assign them for the environment that it is running in.

thank you, it works when i replaced the global key with an env key.


feat: Replaced epydoc with pydoctor

I, @mljoshi, and @Michael-Kowata have worked on replacing the API documentation tool epydoc with pydoctor, addressing the issue #1364.
Changes that were made:

Removed the docs/epy folder and the old HTML files inside the folder that were generated with epydoc
Created a new folder called pydoc and regenerated the appropriate HTML files with pydoctor inside the folder
Updated the associated links in some of the markdown files
Updated the command for the generation of the HTML files in the build.sh file

Files that were updated:

docs/README.md
docs/batch.md
docs/build
docs/media.md
docs/mocks.md
docs/start.md

Also fixes #2300.

@tristanlatr or @vchudnov-g -- Is it possible for you to take a look at this PR sometime this week?  The authors of this PR are students at the Computing Talent Initiative (CTI), and they are presenting their work this Friday.  It would be nice if this PR were to get approved by then or at least give them some feedback which they can talk about during their presentation.   Thank you.


Implement niches pages

Goal

Quickly direct users to the specific benefits they should expect to receive from this site.
This goes back to Doug Hunter's (plan reviewer from our Monday update list) suggestion that Lattice Semi's solution to enabling quick narrowing to the needs of who was using the site was through the menu going to specific needs.

Proposed Changes

[x] Replace "features benefits pricing" links in Header with "niches".
[ ] "Niches" link in Header opens to dropdown menu with 4 choices: "UX design, digital marketing, machine learning, other", which lead to brief pages for the benefits of Marmalade to each type of freelancer.
[ ] Use the ALT text also for captions for images.

How to Add Markdown Image Captions in Gatsby. 2021-07-21.
See the gatsby-remark-images plugin for adding captions to images referenced in a Markdown file.

Content

See Marmalade > website > Niches pages for the images and text content.

Full-sentence tags are included for all images.

I pulled in some of the information from the Description field of Issue #94 into this issue and then closed that issue as a duplicate of this issue.

Yay !
On Sat, Aug 7, 2021, 3:28 PM Jim Tyhurst @.***> wrote:

I pulled in some of the information from the Description field of Issue
#94 https://github.com/Marmalade-ai/company-website/issues/94 into this
issue and then closed that issue as a duplicate of this issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Marmalade-ai/company-website/issues/55#issuecomment-894713431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAZH6RNMDTIRLQMIFR44RTT3WXQBANCNFSM5AU4PZ5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


Adding a docker build

Ref.: https://github.com/rickbergfalk/sqlpad/issues/90
Just test it using: docker-compose up
You'll need docker & docker-compose ;).

Seems fine, but I'm not at all experienced with docker, so I'll let someone else decide on this :)

Next step would be create some organisation on hub.docker.com then push the resulting image :).
This container will probably need improvements, but that's a first step ^^.

Thanks @marcmillien. I'm not experienced with docker either, but I like this being an option. Docker has been one of those technologies on my backlog to learn...
@marcmillien would you like to be part of the SqlPad org on docker? No pressure if you'd rather not.
Otherwise I'll try and learn this stuff later tonight :)


Add message when generating report

Report generation can take some time

added full loading screen and message - will need to get translated at some point


Improve result of 'reset' sync when data supplied is from an older version peer

Scenario: a new version is released, a truant client upgrades and connects. It is told to reset to server state, but the server state is from an older client with a lower version.
This truant client receives a bunch of invalid data. It prunes it, leaving not a lot left. This looks like lost data.
Desired outcome: truant client receives data, resets to the appropriate version of the db, then migrates it up and syncs it back.
Open question: how will the new replica know for sure what version the data it receives comes from? Do we need at least one touch op per migration to ensure there is an op from the latest version? Or is it logically consistent that if a migration doesn't create any operations, there is no invalid data?

Clients now receive all available historical schemas in context, which can be used to reset the database and remigrate up to latest.
I'm going to say for the moment that any client resetting to some data should go ahead and reset to the latest version number encoded in an operation/baseline timestamp and migrate up.
Dropping the queryable db isn't necessary. The migration can all happen in-memory and it will land on the latest version anyway, at which point data can be safely inserted/replaced in the queryable db.
Receive data -> materialize and migrate documents to newest version -> write documents to db, which is already at newest version since the store was intialized.

This relates highly to backups, too. Same principle.

First bug in the wild due to this, #413

Main difficulty in implementation here is system organization. Today, the reset sync message is handled at the EntityStore layer, which is too late to do migration.
The client also just isn't really designed to migrate after initialization.
If actually migrating IDB versions isn't necessary (see above), this is still hopefully possible without too much refactoring. But that comment disregarded migration queries, which need the queryable db available at the current migration version.
I think the actual procedure will be...

Receive old reset sync
Open a temporary queryable db at reset version
Load in reset sync to meta db
Run migration up to current version
Copy data from temp queryable to real queryable

Temp is only necessary to avoid hot-swapping the live queryable db with a new db at an older version. If that can be done, it isn't needed.
Hot swapping live queryable db was the roadblock last time I tried this as I recall.


Friends

Closes #363

@ellenkorbes Is there a way to keep Kubicorn related documentation separate and kubicorn.io files (like _include, _layout folders) separate may be in a sub-directory ? What I want to keep is docs directory cleaner, so that if someone is visiting the project for the first time, it doesn't get confusing which files to look.

@prateekgogia As far as I understood, I think we want to make user use kubicorn.io instead of docs on GitHub. That could make things easier and better for user. We also need to work on #339, which will show user where she/he should read the docs.

@prateekgogia I don't think so. GitHub Pages isn't very flexible, the website files can either be at / or at /docs. What we can do is have /docs/README.md do a better job at explaining how things work.
And as @xmudrii said, I was under the impression we wanted to website to be the main place to read documentation, not the repo.

There's also the option of having a separate repo for the website, à la https://github.com/gohugoio/hugoDocs

@ellenkorbes this could be an good idea, but I wouldn't do it at this point. We still didn't reached some stability point, API is changed with every other PR: new variables are added, etc.
At this point, it would just add an additional step for contributors as they'll have to update two repos.
Once we reach some point when changes are less breaking, this could be an awesome idea.

Sounds good to me.


Checkbox Component styling

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

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

What is the current behavior?

Issue Number: GOOD-181
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
https://github.com/reaviz/reablocks/assets/11332713/6dae8b9e-1f1d-4c8a-865c-a5f9bcc6e042
https://github.com/reaviz/reablocks/assets/11332713/bd468a85-cd4d-48ef-a2c4-4cc1c74719f3


Create .prow.yaml

Sample PROW job config file to get started.

@gvbalaji that's just to get started to use prow to merge, as for code validation those repositories will need each a different set, packages will need some kpt linter/syntax checker, doc repository probably markdown linter and spellchecker maybe etc. This is up for owners/contributors to provide as ultimately it's there to make their lives easier, so I don't want to enforce anything from start.


Update Settings “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for nuxt-test-20241121 processing.

Name
Link

🔨 Latest commit
a85b977db6719e4871f5b60bfc77bb80e6f16c7e

🔍 Latest deploy log
https://app.netlify.com/sites/nuxt-test-20241121/deploys/673f3203d9c389000805c9d5


Filter price is not working for default theme

Bug report
Title
Filter price is not working for the default theme
Issue Description
Filter price is not working for the default theme
Preconditions
1. framework Version- master

Steps to reproduce
1. step1- Open shop view
2. step2- filter product based on price(issue)

Expected result

should filter all product based on price range

Actual result

Fixed

https://prnt.sc/rjfd18

This issue is still exist, please fix


Add commonPrefix(with:), commonSuffix(with:)

Find the common prefix/suffix of two sequences or collections.

commonPrefix(with:)
commonSuffix(with:)
endsOfCommonPrefix(with:): get a pair of indices that mark the end of the common prefix of two collections
startssOfCommonSuffix(with:): get a pair of indices that mark the start of the common suffix of two collections

extension Sequence {
  public func commonPrefix<Other: Sequence>(
    with other: Other,
    by areEquivalent: (Element, Other.Element) throws -> Bool
  ) rethrows -> [Element]
}

extension Sequence where Element: Equatable {
  public func commonPrefix<Other: Sequence>(
    with other: Other
  ) -> CommonPrefix<Self, Other> where Other.Element == Element
}

extension LazySequenceProtocol {
  public func commonPrefix<Other: Sequence>(
    with other: Other,
    by areEquivalent: @escaping (Element, Other.Element) -> Bool
  ) -> CommonPrefix<Self, Other>
}

extension Collection {
  public func commonPrefix<Other: Sequence>(
    with other: Other,
    by areEquivalent: (Element, Other.Element) throws -> Bool
  ) rethrows -> SubSequence
}

extension Collection where Element: Equatable {
  public func commonPrefix<Other: Sequence>(
    with other: Other
  ) -> SubSequence where Other.Element == Element
}

extension LazyCollectionProtocol {
  public func commonPrefix<Other: Sequence>(
    with other: Other,
    by areEquivalent: @escaping (Element, Other.Element) -> Bool
  ) -> CommonPrefix<Self, Other>
}

extension LazyCollectionProtocol where Element: Equatable {
  public func commonPrefix<Other: Sequence>(
    with other: Other
  ) -> CommonPrefix<Self, Other> where Other.Element == Element
}

extension BidirectionalCollection {
  public func commonSuffix<Other: BidirectionalCollection>(
    with other: Other,
    by areEquivalent: (Element, Other.Element) throws -> Bool
  ) rethrows -> SubSequence
}

extension BidirectionalCollection where Element: Equatable {
  public func commonSuffix<Other: BidirectionalCollection>(
    with other: Other
  ) -> SubSequence where Other.Element == Element
}

extension Collection {
  public func endsOfCommonPrefix<Other: Collection>(
    with other: Other,
    by areEquivalent: (Element, Other.Element) throws -> Bool
  ) rethrows -> (Index, Other.Index)
}

extension Collection where Element: Equatable {
  public func endsOfCommonPrefix<Other: Collection>(
    with other: Other
  ) -> (Index, Other.Index) where Other.Element == Element
}

extension BidirectionalCollection {
  public func startsOfCommonSuffix<Other: BidirectionalCollection>(
    with other: Other,
    by areEquivalent: (Element, Other.Element) throws -> Bool
  ) rethrows -> (Index, Other.Index)
}

extension BidirectionalCollection where Element: Equatable {
  public func startsOfCommonSuffix<Other: BidirectionalCollection>(
    with other: Other
  ) -> (Index, Other.Index) where Other.Element == Element
}

Checklist

[ ] I've added at least one test that validates that my change is working, if appropriate
[x] I've followed the code style of the rest of the project
[x] I've read the Contribution Guidelines
[ ] I've updated the documentation if necessary

I think it's more natural to not pluralize of start and end, i.e. just have endOfCommonPrefix, startOfCommonSuffix

I think it's more natural to not pluralize of start and end, i.e. just have endOfCommonPrefix, startOfCommonSuffix

Fair point. I think I went with this because we already have endOfPrefix(where:) and startOfSuffix(where:) which both return a single index, but that's probably fine.

Don't endsOfCommonPrefix(with:) and startssOfCommonSuffix(with:) overlap with diverges(from:) and converges(with:) from #37 ?


Duplicate (but case-different) proxy environment variable causes crash

I run in a mixed environment where I have Linux tools and mixed environment variables. For proxy variables (http_proxy, https_proxy, no_proxy) I have applications that use uppercase variables and other that use lowercase variables, so I have to define them both. However, SyncTrayzor crashes because of NO_PROXY and no_proxy both existing among the environment variables.
2020-09-17 09:59:08.5162 #16720 [Error] SyncTrayzor.Bootstrapper: An unhandled AppDomain exception occurred. Terminating: True System.ArgumentException L'elemento è già stato aggiunto. Chiave nel dizionario: 'NO_PROXY'. Chiave aggiunta: 'no_proxy' Void Insert(System.Object, System.Object, Boolean)    in System.Collections.Hashtable.Insert(Object key, Object nvalue, Boolean add)
   in System.Diagnostics.ProcessStartInfo.get_EnvironmentVariables()
   in SyncTrayzor.Syncthing.SyncthingProcessRunner.StartInternal(Boolean isRestart) in C:\projects\synctrayzor\src\SyncTrayzor\Syncthing\SyncthingProcessRunner.cs:riga 130
   in SyncTrayzor.Syncthing.SyncthingManager.<StartAsync>d__118.MoveNext() in C:\projects\synctrayzor\src\SyncTrayzor\Syncthing\SyncthingManager.cs:riga 192

(sorry for the Italian exception message, basically it says "duplicate key in hash table")
The issue can be worked around by removing the uppercase variables. I created a wrapper script to do that, but I thought this would be worth a bug report.
I'm not an expert on Windows systems, but if you need more information, just ask.

Thanks!

Unfortunately it looks like this is an issue with the .NET Framework -- this happens when fetching the set of environmental variables. I'm afraid I don't think there's anything I can do, sorry!

I actually expierenced this with other .NET applications too, that's unbelievable!
Thank you for trying.


Change end_time to utc

Description of changes:
Changing end_time parameter to use UTC time aswell (same as start_time).
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-sDRE8Pq0duHT
Commit ID: fe4e6397fde55bd00084bbd436a1c5dd4e56e8af
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Histogram CT Zero ingestion

Fixes #13384
Fundamentally it follows the same logic used in AppendCTZeroSample(), used to append zeros for usual samples, but for histograms, we're using Go's zero value for histogram.Histogram{} and histogram.FloatHistogram{}
I am opening it as a draft because I'm seeing some weird failures in tests that I couldn't understand yet
Could you take a look @bwplotka @thespiritXiii @Maniktherana?

Alright, only 1 test is failing now: TestHeadAppender_AppendHistogramCTZeroSample.
What this test does is ingest Histogram samples with created timestamps in several different scenarios. Some of those scenarios should reject the sample and others should ingested. After that, we query the ingested samples and verify that a histogram zero sample and the normal histogram sample are retrieved as expected.
The test is failing right now because the histogram zero samples are being modified after ingestion with a non-zero CounterResetHint. @beorn7 @krajorama, I'm not familiar with this part. Is it expected that histograms are modified during ingestion? Should I ignore this field when comparing the ingested/queried histogram?

Alright, only 1 test is failing now: TestHeadAppender_AppendHistogramCTZeroSample.
What this test does is ingest Histogram samples with created timestamps in several different scenarios. Some of those scenarios should reject the sample and others should ingested. After that, we query the ingested samples and verify that a histogram zero sample and the normal histogram sample are retrieved as expected.
The test is failing right now because the histogram zero samples are being modified after ingestion with a non-zero CounterResetHint. @beorn7 @krajorama, I'm not familiar with this part. Is it expected that histograms are modified during ingestion? Should I ignore this field when comparing the ingested/queried histogram?

relevant @bwplotka https://github.com/prometheus/prometheus/issues/13384#issuecomment-2306538815

Alright, only 1 test is failing now: TestHeadAppender_AppendHistogramCTZeroSample.
What this test does is ingest Histogram samples with created timestamps in several different scenarios. Some of those scenarios should reject the sample and others should ingested. After that, we query the ingested samples and verify that a histogram zero sample and the normal histogram sample are retrieved as expected.
The test is failing right now because the histogram zero samples are being modified after ingestion with a non-zero CounterResetHint. @beorn7 @krajorama, I'm not familiar with this part. Is it expected that histograms are modified during ingestion? Should I ignore this field when comparing the ingested/queried histogram?

Yes, the appended histogram as well as the resulting histogram in TSDB can be modified by Append.
The counter reset hint is set to help the PromQL engine in counter reset detection. If the count reset hint is left 0, then the PromQL engine runs the detection every time , which is super costly for native histograms, so the Appender tries to set the counter reset to either 1 or 2 (detected reset, detected no reset) to help PromQL. So reading back different hint is totally expected. Also if I understand correctly, the zero histogram is getting written into TSDB as an extra sample. That will induce a counter reset, so to see Counter resets is totally expected.
There's two strategies in tests around this:

If we are not interested in the counter resets, in which case you'd set the counter reset hint to 0 in the actual samples before making the comparison with the expected samples.
If we are interested in the counter resets, in which case you have to know/calculate what the expected hint should be.

If AppendHistogramCTZeroSample was just simply calling AppendHistogram I would say go with 1. , but it's more complicated and might get even more complicated due to out-of-order (#14546 ) so I think we need to go for 2 and encode the expected counter resets.

Appender tries to set the counter reset to either 1 or 2 (detected reset, detected no reset) to help PromQL.

Also if I understand correctly, the zero histogram is getting written into TSDB as an extra sample. That will induce a counter reset, so to see Counter resets is totally expected.

Ah ok that makes sense then. Yes, we should always see a CounterResetHint==2 when ingesting a zero sample histogram.
But that leaves me with another question: If we're scraping a histogram for the first time and this histogram has set CT during exposition. Should the zero sample also contain ResetHint==2? Theoretically there was no reset since it's the first sample ever ingested for the series

Yep, you got it. If there's no previous sample, we encode 0 (unknown reset) because later when the chunk is read back there might be some previous sample loaded from disk for example and we cannot say counter reset or no reset with any confidence, so we encode "unknown".

Converted to draft while I work on the failing test in head_appender

This PR will conflict with #14546 once merged. I don't mind resolving the conflicts here, so it's up to you which PR gets merged first @krajorama!

Converting to draft as I work through the conflicts

Converting to draft as I work through the conflicts

Many thanks for taking the hit, the OOO NH PR was months in the making.

Many thanks for taking the hit, the OOO NH PR was months in the making.

No problems :)
PR should be ready for review now

It should be ready for another review

Just did a quick experiment with an app exposing native histograms, I can see the first one being ingested as a 0 🥳

Taking the liberty to merge!


is this supposed to work anymore

im getting err file_get_contents(https://api.telegram.org/bot5164225693:AAHG2Bf1sjVbkU_6eGSNS4YFYNt_cyCuXIA/sendMessage?

I got it to work. i was just confused about whose id should be in 'Telegram chat_id'
its supposed to be your regular user id number, not the bots'


Added conditional validation example

That's a great addition, thanks @igor-alexandrov - the problem is only, do we actually want to push validation groups or rather high-level rules in dry-v?
Also, I'm contemplating rewriting most parts of the Reform docs. Are you interesting in helping? We could start with discussing structure and examples, and then go for it!?!

@apotonick sure I want to help. Also I like the idea of how DRY keeps their documentation: they don't have anything in README and all documentation is on their site: http://dry-rb.org/gems/dry-validation https://github.com/dry-rb/dry-validation. This is easier to maintain and keeps everything in one place.

As for me, validation groups are more understandable than high-level rules.


Fixing profile workflow

Description
This PR:

Replaces the ubuntu_latest with ubuntu-latest, so it should actually work.
Renames the job from test to a more accurate profiling.
 Partly to reuse the existing code, partly to give a more robust test. Although, everything coming into develop or main should have been through workflow already.

Fixes #210
Type of change

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

Ha - looks like I renamed qa to test and test to profiling. That seems ok.


Fix alignment in generated file

I don't know Ruby well enough, but it looks like the commit message may be incorrect: how does this fix alignment? It looks more like it just removes a named match from the regex. ?

It fixes the alignment by ensuring that all lines start with the same indent.
The existing code only replaces whatever happens to match the first leading spaces.
rubys can confirm if necessary

Thx. yeah... I'm gonna defer to another with Ruby skillz...

Logic before: find the first line with indentation, and then replace that amount of indentation in all lines with two spaces.
Logic after: replace all indentation with two spaces.
The code before understood that some of the data had nested indents.  (Things like <LocationMatch> and the like).

okay... so, variant indents are not preserved: every indent gets switched to two spaces. ... Losing the variance doesn't sound helpful.

At the moment the generated output looks like this:
<Directory /x1/srv/whimsy/www/test/roster>
AuthType Basic
  AuthName "ASF Committers"
  AuthBasicProvider ldap
  AuthLDAPUrl "ldaps://ldap2-us-west.apache.org:636 ldap1-us-west.apache.org:636/ou=people,dc=apache,dc=org?uid"
  AuthLDAPGroupAttribute memberUid
  AuthLDAPGroupAttributeIsDN off
  Require ldap-group cn=committers,ou=groups,dc=apache,dc=org
</Directory>

<LocationMatch ^/incubator/moderators>
AuthType Basic
  AuthName "ASF Members and Incubator PMC"
  AuthBasicProvider ldap
  AuthLDAPUrl "ldaps://ldap2-us-west.apache.org:636 ldap1-us-west.apache.org:636/ou=people,dc=apache,dc=org?uid"
  AuthLDAPGroupAttribute memberUid
  AuthLDAPGroupAttributeIsDN off
  Require ldap-group cn=committers,ou=groups,dc=apache,dc=org
</LocationMatch>

Which is harder to read.

I agree that this could be improved.  I suggest that the problem is in the call to strip which removes the whitespace from the first line.  A better replacement:
content = content[1..-2].gsub /^#{content[/^\s+/]}/, '  '

:+1:

Unfortunately it hasn't worked properly.
Although the individual elements now have the same alignment, it is too deep.
And there are spurious trailing blank lines.
For example, the sample above has become:
<Directory /x1/srv/whimsy/www/test/roster>
            AuthType Basic
            AuthName "ASF Committers"
            AuthBasicProvider ldap
            AuthLDAPUrl "ldaps://ldap2-us-west.apache.org:636 ldap1-us-west.apache.org:636/ou=people,dc=apache,dc=org?uid"
            AuthLDAPGroupAttribute memberUid
            AuthLDAPGroupAttributeIsDN off
            Require ldap-group cn=committers,ou=groups,dc=apache,dc=org

</Directory>

<LocationMatch ^/incubator/moderators>
            AuthType Basic
            AuthName "ASF Members and Incubator PMC"
            AuthBasicProvider ldap
            AuthLDAPUrl "ldaps://ldap2-us-west.apache.org:636 ldap1-us-west.apache.org:636/ou=people,dc=apache,dc=org?uid"
            AuthLDAPGroupAttribute memberUid
            AuthLDAPGroupAttributeIsDN off
            Require ldap-group cn=committers,ou=groups,dc=apache,dc=org

</LocationMatch>

As a result of the increased indentation, nested alignment is broken:
<LocationMatch ^/board/publish_minutes>
            AuthType Basic
            AuthName "ASF Secretarial Team"
            AuthBasicProvider ldap
            AuthLDAPUrl "ldaps://ldap2-us-west.apache.org:636 ldap1-us-west.apache.org:636/ou=people,dc=apache,dc=org?uid"
            AuthLDAPGroupAttribute member
            AuthLDAPGroupAttributeIsDN on
            <RequireAny>
    Require ldap-group cn=asf-secretary,ou=groups,ou=services,dc=apache,dc=org
    Require ldap-group cn=apldap,ou=groups,ou=services,dc=apache,dc=org
  </RequireAny>

</LocationMatch>

I've made a pull request with a fix: https://github.com/apache/infrastructure-puppet/pull/270/commits/f0f1524e22d44c6d03edda1550309be7b1be54e8
I've also added test scaffolding which will be useful in the future: https://github.com/apache/whimsy/blob/master/tools/vhosttest.rb


[feature-request] Clock Icon

is it possible to add the possibility to toggle the icon on/off and it's position?

is it possible to add the possibility to toggle the icon on/off and it's position?

Any updates on this? Would be nice to easily hide the icon


Remove un-necessary ITuple constraint

While working on type forwarding in CoreFx branch, I realized this constraint is actually creating some source-level back compatibility issue. Since it can be removed, I might as well.
Relates to CoreCLR PR dotnet/coreclr#8787

Hi @jcouv, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


Support HTML/CSS renderers for print

Hi!
Would caniuse be interested in getting support for HTML/CSS renderers for print?
There are many tools that can generate PDFs and support paged media, such as WeasyPrint (disclaimer: I’m one of the maintainers), PagedJS, Vivliostyle, Prince… They don’t appear in websites visits, because they’re rarely used on public websites, but they’re "quite popular": for example, WeasyPrint is downloaded more than 1 million times a month, and it’s probably not the most widely used of the list 😄.
This topic has also been discussed a bit in #1510.
If you think that it would be nice, don’t hesitate to ping me, I can find some time to work on this!
If you think that it’s not worth it, no problem, we’ll find another way to advertise our supported features.
(And thanks for this amazing tool 💜)

Sounds like a great thing to track! But indeed out of scope for caniuse.com
Perhaps if someone were to create a similar project (or like caniemail.com) specific to print renderers that would be great and I'd be happy to link there!


Publish 7.3-20210925230640+0000

Publish 7.3-20210925230640+0000

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Generated dbus code doesn't match org.bluez.xml

I'm attempting some modifications to add newer services like LEAdvertisement1.  I noticed when modifying the existing org.bluez.xml file, then generating the code via gdbus-codegen --interface-prefix org.bluez --generate-c-code generated-code --c-generate-object-manager org.bluez.xml that the code generated was slightly different in many places.  The most common occurence was gchar being generated instead of the GBytes in the tinyb generated-code.h and generated-code.c.  This obviously breaks tinyb as its code is built on top of this generated code.  I tried several variations on the gdbus-codgen command to no avail.
Any thoughts or could someone share what gdbus-codegen command they used?  My version is 2.48.2 just like the one in the repo.

Yes. I used a modified version of gdbus which I didn't get to upstream, as it needed some extra changes to not break compatibility of other programs. At the time I found a bug report in which the gdbus maintainers rejected a complaint that gchar * is not a good format, so I put in patch on the backlog and never got around to doing it. I don't have the patch anymore, but it was not difficult to hack it. If you need any help let me know and I'll look into it.

I'm actually working on it a little in my spare time right now.  Having a bit of issue getting GDBus.Error:org.bluez.Error.InvalidArguments: Invalid Length when attempting to call  gatt_characteristic1_call_write_value_sync() from BluetoothGattCharacteristic::write_value.  Not sure why it's being such a pain as it seemed to be nearly a drop in replacement of GBytes to gchar.  I'll clean it up on the branch I'm working on and commit so you can see it.

See my fork here

gchar* doesn't work because BlueZ transfers binary data over DBus, which might contain zeroes. Since gchar * doesn't actually contain length information, the transfer function stops when it finds the first zero, and you will end up with this error: GDBus.Error:org.bluez.Error.InvalidArguments: Invalid Length.
Instead, you have to use GBytes and in generated function itself, you'll need blocks that marshall the GBytes from/into DBus byte arrays, and not strings.
I suggest you just manually merge the generated-code.c you get with the old one, keeping the use of GBytes.

Bummer!  I did find that I could add the argument --annotate "org.bluez.GattCharacteristic1.WriteValue()[value]" org.gtk.GDBus.C.ForceGVariant true to the code generation and it would actually change the troublesome gchar* value into a GVariant*.  I'm trying to see if I can leverage that to somehow keep the generated code stable.

I did end up having success with this!  Added annotations to the dbus profile xml file <annotation name="org.gtk.GDBus.C.ForceGVariant" value="true"/> to force GVariants and replaced the code.  Now the generated code works!  I'll still need to make the changes to GattDescriptor but glad to have solved it.  Thanks for your help!

Good stuff @impala454, I never got to this, would that mean that it is possible to extend tinyb for this?
https://github.com/intel-iot-devkit/tinyb/issues/80

BTW, do you have this as well?
https://github.com/intel-iot-devkit/tinyb/issues/69
Something tells me that there is something wrong with org.bluez.xml

Hey @vkolotov yep on the Adapter.SetDiscoveryFilter.  The reason I wanted the code generator to work is because I wanted to update the interface xml to the latest BlueZ.  I've also tinkered with OpenHAB so would love to help them out.  I will remove the generated code and have CMake generate it automatically when it's done.  Hopefully I'll have a PR in by the end of the week.
As far as the java stuff, not really sure.  I'll reply on #69 with what little I know about it.

@impala454 Awesome find. Should make it a lot easier to update TinyB, and it should be possible to remove the generated-code.c altogether if this works reliably.

@impala454 @petreeftime good stuff guys, the plugin for openhab will be ready soon, SetDiscoveryFilter would help a lot.

@petreeftime is it worth doing a PR on the completed C++ side, or would you rather wait until the corresponding java code is written by someone else?  For what it's worth, I'll likely be helping out a lot with the C++ side in the near future.  What are your thoughts on splitting this library out into a C++ only library, and someone else building the Java side which depends on it?  That may get better traction to bring in more help.  I notice someone else just submitted a PR for functionality only on the java side.

I am fine with having C++ only functionality.

Sorry I've been a little slow on this but likely over the next couple of months I'll be working on this.  I will probably pear it back to C++ only.  If it needs to be a separate fork that's ok just let me know.

Hey @impala454 I'm happy to do java part if you are still keen on doing dbus-codegen and some c++ parts of it. Pls guide me what to do as I'm not a big expert in c++. Particularly, what command did you use to run dbus-codegen?

I'm really sorry I've just been swamped at my day job and the project I thought would be using this got put off.  I may have time at some point but for now, I believe my fork as is incorporates the automatic dbus code generation.

Hi @impala454  and @vkolotov - thank you for your discussion and work.
I have merged your work here https://github.com/sgothel/tinyb
Some of the unref/free I have added, but I am not a glib expert,
so I might have missed some parts?
Just in case you still work with this library, maybe you like to review and merge back.
I may also review lbt4 and some other binding tools later,
but so far I like the low footprint and no dependencies here.
Thank you.


Fix cursor moving back and missing characters

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1914
Cause of the issue: The EditText temporarily contains an extra character. The TextWatcher gets removed (here) before it can process this extra character, leaving the saved text state (in QuestionnaireViewItem) inconsistent with the actual EditText content. This inconsistency triggers a UI update using EditText#setText (here), which unintentionally resets the cursor position to the beginning.
EditText#append moves the cursor to the last position.
Read comments in rest of the PR for more details.
Description
Clear and concise code change description.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: (Bug fix | Feature | Documentation | Testing | Code health | Builds | Releases | Other)
Screenshots (if applicable)
Checklist

[ ] I have read and acknowledged the Code of conduct.
[ ] I have read the Contributing page.
[ ] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[ ] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[ ] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[ ] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

This, however, has a side effect (which existed earlier as well but was overlooked because of cursor resetting) of missing some characters when typing too fast.
Reason: EditText's extra character is skipped when using the above solution.
Solving this will require more effort. I can think of a solution which will involve detecting "user has stopped typing". Something like this - stackoverflow.
Let me know if you want to fix this as well. Will raise a separate issue for this.
Alternative: Use RxBindings or do some backpressure solutioning to handle all character inputs from user. But this will need a more complex handling - like "not updating the EditText until upstream processing (handleInput) is complete".

@jingtang10 what are your thoughts on this?

I made an attempt at solving the "side effect" as well. Seems good to me but Idk if this behaviour - "detect user has stopped typing" can have any other side effects.
Only difficulty is to test this as racing conditions are always a pain to test.
How I tested it - Ran my finger over "QWERTY" tabs on keyboard back and forth really fast. Before the last commit I could see some missed characters. After the last commit, I didn't see any missed character in quite some attempts trying this test.
Can you try testing this PR ?
@pld

Thanks for all your work on this @MJ1998. @FikriMilano will test the PR tomorrow and we'll get back to you.

Just FYI an elaboration on the how I am testing this:-
I slide my finger and keep it pressed over "QWERTY" keys back and forth. So the input I generate is "qwertytrewqwertytrewq...".
You can observe some characters being missed before the last commit if you test like this. @FikriMilano
In actual case the upstream function can take more time to finish (for example, resolving fhir query)  - which makes characters missing clearly visible.

Regarding your question on RxBinding, I think its a much bigger effort as we dont use simple TextView or EditText views, rather its a EditTextInputLayout which offers more features like validation, header views along with it.
So moving from EditTextInputLayout to RxBinding views will be difficult.
@FikriMilano

@MJ1998 this fix you shared is working great. Thank you for prioritising this and getting it resolved so quickly!

@MJ1998 upon further testing, the issue of the missing characters is significantly affecting the data entry.
https://github.com/google/android-fhir/assets/8417097/b9775530-db3a-4cc6-a9ae-352ccf68e2af
Can we please go back to the drawing board on this?

Is this issue happening on this PR?

@MJ1998 yes it is.

Can you try increasing the DELAY TIME to 1 second or 2 second and see if this is happening ?
Not the best solution but I am trying to figure out what is causing the delay in the upstream function (that validates the input string and sends it to the QuestionnaireViewModel for updating the RecyclerView) if there is no dependent items also.
To increase delay time search for this line: textInputEditText.afterTextChangedDelayed(500)

Can you try this solution in  @f-odhiambo device and confirm if it works ?
I am not sure because that if condition is what we are anyway doing in updateUI.

Context: User input is sent back to QuestionnaireViewModel for validation, updating dependent items and then building the RecyclerView list -> which then comes back to the EditViewHolderFactory.
The problem is happening when user input speed exceeds the speed at which this loop finishes.
So here's what we are implementing:
We will detect new user input and when loop comes back to EditViewHolderFactor we will cut off the loop to prevent updating UI when THERE IS new user input present!
@jingtang10 @FikriMilano

yay, awesome, thank you @MJ1998 🎉 !


Add a changelog

It would be very useful to have a change log for every minor and major change to external developers have a chance to see what has changed.

Fully agreed.


Mulitple security fixes

This has been merged in manually to the 9.0rc branch, but it has problems. See a separate task coming shortly.


SDN-4549: [release-4.14]Backport subnet configs

Clean backport of transit and join subnets to 4.14

/jira refresh

/approve

base on https://github.com/openshift/cluster-network-operator/pull/2473#issuecomment-2289573368
/label qe-approve

@JoelSpeed https://github.com/openshift/cluster-network-operator/pull/2473#issuecomment-2289573368 here is a link to a comment describing qe (@asood-rh) running regression / feature tests. We currently do not plan on backporting the day-2 masquerade subnet changes so all cases have been tested and passed

/lgtm
/label backport-risk-assessed

/label cherry-pick-approved

/jira refresh

/jira refresh

/jira refresh

/jira refresh


Fix a typo in the README

See default value here: https://github.com/borgbase/ansible-role-borgbackup/blob/master/defaults/main.yml#L4

Good find. Borgmatic also uses config.yaml more often.


Add TrueJob

Hey @sampl,
This is a repeat of #28, but since there were merge conflicts and @JangoSteve was the one who submitted originally, I figured It would be easier to put in a new pull request with my account (in case other things need to be updated later on) and of course I rebased it up to master's level for ease of pulling in.
Let me know if you have any questions.

thanks @scouttyg!


TTNN Op Interface

We should get the ball rolling to help define what the op interface with TTNN looks like and how it interacts with the compiler and runtime.  This issue is intended to act as a working document / common place to discuss and put meeting notes.
Stakeholders:

@nobradovictt: tt-mlir optimizer
@rjakovljevicTT / @mbezuljTT: TTNN Kernels
@kmabeeTT / @tapspatel: Device runtime / flatbuffer schema
@xanderchin / @cfjchu: TTNN point of contacts

Initial things these ops need to model:

L1 usage
Op Constraints

Tensor Layout (Row major vs Tile)
Parallelization Constraints (Legal grids, width/height/block shard strategy support, shard orientation)

@nsmithtt @mbezuljTT @derdeljanTT and I agreed to split this issue into the following work streams and PoCs:

[ ] L1 usage, and
[ ] Op Constraints.

The potential third work stream is "Op performance modeling". However, as we know it would be a lot of work to provide and maintains "Op perf" API with Ops being actively developed and optimized, we should also explore potential alternatives in terms of how Optimizer would make performance related decisions. This is to be discussed at least with @s-jovic, @pavlepopovic, @nobradovictt.
I am going to open a thread for each topic below, please use it for all related discussions to make it more readable and easier to follow.


How to change font and increase line height?

Is there anyway i can change font-family and line-height? Thanks.

i also  want to know


Async.Request has unnecessary optional files

Perhaps it would be better to make greater use of enums with associated types?

I refactored RequestType to use an associated type enum a couple of days ago - what "optional files" are you referring to?

I missed that. Will take a look. 👌🏻

Based on Tom's comments, I'm going to close this as won't fix. We can make tweaks if we see anything that actually needs changing.


Inner_join not working though column headers exist and data overlap

Inner_join not working though column headers exist and data overlap (see attached csv_files.zip)
See the following outputs:
>cat Dingo_BioSamples_table.csv
Acc,BioSample,SampleName,**SRS**,Breed
3366709,SAMN03366709,RKW13760,SRS1025425,Dingo
8873190,SAMN08873190,NewGuineaSingingDog01,SRS3259288,New Guinea Singing Dog
4851104,SAMN04851104,1756,SRS1397565,Village Dog Sub-Saharan Africa
4851101,SAMN04851101,PT49,SRS1397563,Village Dog
3366710,SAMN03366710,RKW13759,SRS1025423,Israeli wolf
> head Canine_SRA_table.csv
SRA,SRX,SRP,**SRS**,SRR
SRA699439,SRX4041934,SRP144493,SRS3259403,SRR7120307
SRA699439,SRX4041933,SRP144493,SRS3259404,SRR7120306
SRA699439,SRX4041932,SRP144493,SRS3259402,SRR7120305
SRA699439,SRX4041931,SRP144493,SRS3259401,SRR7120304
SRA699439,SRX4041930,SRP144493,SRS3259400,SRR7120303
SRA699439,SRX4041929,SRP144493,SRS3259399,SRR7120302
SRA699439,SRX4041928,SRP144493,SRS3259398,SRR7120301
SRA699439,SRX4041927,SRP144493,SRS3259397,SRR7120300
SRA699439,SRX4041926,SRP144493,SRS3259396,SRR7120299
> cat Dingo_BioSamples_table.csv | dplyr inner_join Canine_SRA_table.csv
Warning message:
package ‘dplyr’ was built under R version 4.0.3
/30days/ibar/dplyr-cli-ae532e85493b.rds

There's definitely overlap in the values of SRS in the data, based upon the join should be made.
 grep SRS1397565 Canine_SRA_table.csv
SRA699439,SRX4036182,SRP144493,SRS1397565,SRR7107945

I don't have bandwidth to look into this.  PRs welcome.


[enhancement] Drop support python2 #81

Removed Python2 and it's supported structures.

Simplified Super()
Removed mock
Upgraded openwisp-utils to 0.4.0
Removed six

Closes #81

Please share if anything is to look out for :smile:

And about nose2 as well :smile:


containers: fix KubernetesEventMonitor creation

During a recent refactoring (authentication) api_version become a class attribute and in the introduction of ContainerProviderMixin it has been moved to kubernetes_version.
This patch reconciles the two changes in the kubernetes event catcher fixing a problem that prevented the event monitor from starting.
/cc @Fryguy @blomquisg @abonas

@miq-bot add_label providers/containers

@Fryguy can you check if now it's ok?

:+1:


Enhance the beginner experience with simple steps on getting started

Overview
As more people join the community, it's good to be able to get them engaged as soon as possible. Whether that's by adding a demo video or overview showing how things work, or by creating short tutorials with simulated pieces (like using Paho to send MQTT messages that are later shown on their own DMS Lite, or by showing how to receive and interpret a message). The goal would be for them to be able to use the technology or contribute to it very quickly.
Additional context
After Call for Code submissions end on July 31, we'd like to provide IBM Developer visitors to go explore and contribute to previous projects that are now open source. They should be able to use the tech, or clearly see how they might open an issue or submit a pull request for basic improvements.

We are currently working hard on rewriting the Quick Start to get started with the CDP. I and @knouse1344 are working on an instruction video that should be a good addition to how to build a Duck.

This could be an option?
ESP32 Simulator

Would love your feedback on what can be done to improve the experience @Evilgeniusnerd @trentbrown13 and others.

Maybe we can make some beginner projects? Build your own weather station?

Quick start guide video of a powerpoint presentation
Make a list of project ideas available on the website
More youtube videos

We have two scripts and documentation covering this one https://github.com/amirna2/ClusterDuck-Docs/blob/master/cdp-install-osx.sh. and another https://github.com/Code-and-Response/ClusterDuck-Protocol/wiki/Linux-CDP-Dev-Setup

videos drafted, will share videos in next couple days

working thru an improved variant of this

Looks like there's been a lot of great documentation added to the wiki!


Games not shown

When I get into the web interface, I only see the .json and the other file.
Also, when I get into the shop from switch, I see those 2 files, but all the games are at "New Game" section.
the problem is that I want to install the update files but I can't see them..

Yes you will only see only the shop files from homepage, as the tinfoil app has some limit for files count in main page as described in the README.md
To avoid TinfoilApp freezes, the file listing was disabled, but you can open copying links from /shop.json
Send the config that are you using and photos from the switch listing to allow us help you more precisely.

the JSON is empty, when I look at it from linux.
If I go from the switch, I can seed it in hex, but not something that I can copy/paste and use.
are you familiar with the file naming on TinFoil?
to show as:
New Game
Update
DLC
?

can you tell what's the correct folder structure? file naming?
thanks.

In my setup the roms folder are structured like this:

And generate a structure like that:

how can you tell the id of the game and the id of the update file?..
if I do
cat shop.json, it's empty..

yes, but how can I install the update files if they are not shown?..
I can only install games.
But the update files are not listed so the game won't load.
So how can I see those update files?
Also, how can I know the id of the game update?


queue registration should be updated based on latest fixes

in 4.0.52 we delivered improved abort logic so there is no way to keep tests in "queued" state anymore. As result our logic with queue registration + testcase removal is broken again:

we successfully executed and registered 100 tests per suite
during refactoring 30 of them we removed
right after removal next run will contain 70 pass and 30 queued which are translated to 'aborted" on finish
for next run queue registration again register 100 as expected number and we can't correct it until disable queue registration

please update logic and don't copy "queued" and "aborted" tests during queue registration

Works as expected, current situation is caused by 2 events happened at the same time (tests removal and abort by timeout), which is usually unlikely and will be resolved when run is finished without abort.