Framework is not consulted on results provided by tmt-report-result

By a mistake, even after loading results generated by tmt-report-result script the test framework was called to provide a test result. This led to a duplication, extra results were generated.
The patch makes tmt-report-result results similar to custom results file: if results exist, they are loaded and passed to the execute plugin, and the test framework is not consulted.
Fixes https://github.com/teemtee/tmt/issues/2389
Pull Request Checklist

[x] implement the feature
[ ] write the documentation
[ ] extend the test coverage
[ ] update the specification
[ ] adjust plugin docstring
[ ] modify the json schema
[ ] mention the version

@psss @lukaszachy I'm looking for some guidance on documentation. I discovered two places where tmt-report-result is mentioned:

https://github.com/teemtee/tmt/blob/main/spec/plans/execute.fmf#L185
https://github.com/teemtee/tmt/blob/main/stories/features/report-result.fmf#L6

Both probably deserve an update, but I find some statements puzzling, namely the result aggregation. Is this something done by tmt-report-result script? I think so, but I'd like to confirm this. Python code definitely does no sorting, the test outcome is the very first result loaded from the file (which is, I guess, also the only result in the file?)

@psss @lukaszachy I'm looking for some guidance on documentation. I discovered two places where tmt-report-result is mentioned:
* https://github.com/teemtee/tmt/blob/main/spec/plans/execute.fmf#L185

* https://github.com/teemtee/tmt/blob/main/stories/features/report-result.fmf#L6

Yes, probably both should be updated. It could make sense to move all the details into stories and provide just links from the plans/execute/tmt spec.

Both probably deserve an update, but I find some statements puzzling, namely the result aggregation. Is this something done by tmt-report-result script? I think so, but I'd like to confirm this. Python code definitely does no sorting, the test outcome is the very first result loaded from the file (which is, I guess, also the only result in the file?)

The thing is that currently tmt-report-result just overwrites the file with the worst result it ran into. Because of this individual phase results are lost. My idea was to teach tmt-report-result to create a custom.yaml according to our results spec and then just load the results.

@psss @lukaszachy I'm looking for some guidance on documentation. I discovered two places where tmt-report-result is mentioned:
* https://github.com/teemtee/tmt/blob/main/spec/plans/execute.fmf#L185

* https://github.com/teemtee/tmt/blob/main/stories/features/report-result.fmf#L6

Yes, probably both should be updated. It could make sense to move all the details into stories and provide just links from the plans/execute/tmt spec.

Understood, I’ll update it this way then.

Both probably deserve an update, but I find some statements puzzling, namely the result aggregation. Is this something done by tmt-report-result script? I think so, but I'd like to confirm this. Python code definitely does no sorting, the test outcome is the very first result loaded from the file (which is, I guess, also the only result in the file?)

The thing is that currently tmt-report-result just overwrites the file with the worst result it ran into. Because of this individual phase results are lost. My idea was to teach tmt-report-result to create a custom.yaml according to our results spec and then just load the results.

I see, thank you. This was the missing link, so to speak!


Be more explicit with the relays that have received the comment

A user found that the feedback when not being able to send the event to all relays was insufficient, reported here https://github.com/verbiricha/habla.news/issues/160

Done! @verbiricha can you update habla.news to zapthreads 0.5.1? it has a bunch of fixes and on the latest nostr-tools

Thanks Fran! Works perfectly.


The cookbook has a maintainer_email specified as part of its metadata.

pseudocode still lists the name of the rule as "has a maintainer", but other than that :+1:


feat(transformer): finish 2016 exponentiation operator

The bulk of the code for review: https://github.com/web-infra-dev/oxc/pull/996/files#diff-4950080c26a449198bc5b125fd72a73afe42cddde9943bc7280fa1cbb7310ced
It took me a while to understand what babel was doing ...
As a bonus, I found swc producing the wrong code for the following snippet:
let x = {};
let y = 0;
let z = 0;
x[z++] **= y;

Gets transformed into
let x = {};
let y = 0;
let z = 0;
var ref = x[z++];
x[z++] = Math.pow(ref, y);

Notice how z++ is evaluated twice.

Which means I should ...

[ ] add more unit tests for this PR

And I also noticed that the benchmark for transformer is wrong ...

[ ] Fix transformer benchmark because it is transforming the already transformed AST

I'll work on the TODOs today.


Support for localtunnel.me

I am testing gotty with ngrok and I didn't find any problem but with localtunnel, http://localtunnel.me/ I can receive the data from my host.
What is the problem?
using DevTools in Chrome I don't receive any problem.
Many thanks in advance
Juan Antonio

I gave localtunnel.me a try and succeeded to open Gotty on Chrome. Probably you have something wrong in your network.

Closing as no recent activities. Please feel free to reopen if needed.


Adding Ramat-Gan Armonim School

@michronin I am sorry, the domain ramat-gan.info belongs to a city, not a college, nor a university.
I have to close the request without merging. Please ensure that you are familiar with this repository rules located at the bottom of this page before submitting a new request.
If you do not have an email which your school provided you with, please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
If you have any questions or concerns please send an email to<EMAIL_ADDRESS>Thank you for the understanding!


Add an org.osbuild.librepo source

Currently a draft for comments until I get the corresponding osbuild-composer part added.
The current SCHEMA works, but when I use a v2 pipeline with it, it doesn't pass and I cannot figure out why.
It works fine using v1 in the pipeline.
Running the tests with python3 -m pytest --pyargs test.run.test_sources --rootdir=. -v works. Luckily the metalink and mirror list server support can be simulated using static files.

CI needs to have python3-librepo included in order to run all the tests - https://github.com/osbuild/containers/pull/53

Great stuff. Is this ready to be finalised?

I think this part is pretty solid, but I'm working through some issues in osbuild-composer with the cloudapi using it, so I don't want to merge until I'm sure no further changes will be needed.

Still planning on getting this in, just isn't the current highest priority.

Ran into this issue here: https://github.com/osbuild/bootc-image-builder/issues/404
So looks like if you build an image... and one of the mirrors go down, then you are unable to build it with bootc-image-builder.
Any way we could add this to post-summit tasks / have an issue opened for it?
@bcl @achilleas-k


Switching audio languages clears buffer and stalls playback for a second.

Have you read the Tutorials? yes
Have you read the FAQ and checked for duplicate open issues? yes
What version of Shaka Player are you using?
3.3.2
Please ask your question
When I switch between audio languages, the player seems to remove all buffer and tries to immediately switch to the selected streams.
Example can be seen on the official web page: https://shaka-player-demo.appspot.com/demo/#audiolang=en-US;textlang=en-US;uilang=en-US;asset=https://storage.googleapis.com/shaka-demo-assets/angel-one/dash.mpd;panel=HOME;build=compiled
Is there a way to make the switch more fluid, e.g. after the new stream has buffered for a second?

As an example of different UX, the dash.js player switches streams on language change seamlessly after about 2 seconds in this example: https://reference.dashif.org/dash.js/latest/samples/multi-audio/multi-audio.html
which is also not ideal, as 2 seconds are quite long.
Is there a way to preload other audio tracks in some way or run them in parallel perhaps to make switching audio instantaneous?

Unfortunately, you really can't have it both ways.  Either we buffer the new thing first and delay switching (as dashjs seems to), or we clear the buffer and switch immediately.  Currently, we switch immediately.
The thinking is that if I need to change audio languages, it's because I can't understand the current one.  That should be an immediate change.
We have audio language preferences in the configuration, so that we can select the best language upfront.  If you know your user's preferred languages, you can set that.  If you don't have user settings for language in your application, you can use navigator.language to fill the player configuration.
Buffering other languages all the time during playback, just in case the user wants to switch to one later, would be wasteful of bandwidth.  I wouldn't be interested to merge something like that.
Does this answer your question?

Thank you for the thorough response!
I understand the reasoning and yes, from the UX point of view, if a user selects something critical and the change needs 2+ seconds to propagate, it would just feel unresponsive.
My use case is a bit special, I'm looking to create language learning exercises based on voluntary (and fast) switching between languages. Basically, once the user is tired of listening to a film scene in a foreign language, they can move the mouse to a different field and listen in a language they understand better. That way, people would be able to pick up some context when they need it, and would be less frustrated when overwhelmed for a moment.
So, now I'm either looking up into solutions that would sync up different media files - like http://webtiming.github.io/timingsrc/doc/exp_mediasync.html
or into getting Shaka Player to buffer the other audio streams all the time. I like Shaka Player for how well it handles many things, but I just started looking into the code base yesterday. If you could point me in the direction on which code parts I should focus in order to enable buffering all audio streams, that would be greatly appreciated!

You can't buffer all audio streams at once.  That's not how the underlying MediaSource API works as far as I know.
You could modify Shaka Player to allow you to treat language changes specially.  Instead of clearing the buffer, you could clear the buffer some small distance ahead of the playhead.
The issue with that is seeking.  If the user seeks back, they are getting the old language again.  If you really want to buffer everything all the time, and just dynamically change which language is rendered... that's not a thing we've ever considered, and I'm not sure how best you would achieve it.
From a pure audio point of view, ignoring the way Shaka and DASH/HLS work, you could imagine having multiple synchronized audio elements, and separately toggle the muted status of them based on some custom UI.  Then everything buffers in parallel, independently, and you don't have to think about what language you buffered in any given part of the presentation.
Would this work?  I think I could guide you on synchronizing multiple media elements.  I've done it recently in another project.
You could still use Shaka Player for adaptive video resolution/bitrate if you want, and handle audio separately with direct mp4 playback.
Or if you want to do DRM or ABR for audio, you could still use multiple instances of Shaka Player for the video and the various languages, each with their own audio element synced to the video element.
Or if you don't care about ABR or DRM for any of it, you wouldn't really need Shaka to begin with.  Just plain <video> and <audio> elements and some JS to sync them.
Let me know if this helps, or if you'd like to see a code snippet to sync multiple media elements.

Okay, here's a sketch based on a side-project I have yet to release.  Google already released copyright of that side-project to me, and you (all) have my permission to use this as you see fit:
mainMediaElement.addEventListener('timeupdate', () => {
  for (const secondaryMediaElement of secondaryMediaElements) {
    secondaryMediaElement.playbackRate = computePlaybackRate(
        mainMediaElement, secondaryMediaElement);
  }
});

function computePlaybackRate(mainMediaElement, secondaryMediaElement) {
  const seeking = this.mediaElement.seeking || extraElement.seeking;
  const ended = this.mediaElement.ended || extraElement.ended;
  if (seeking || ended) {
    return 0;
  }

  const diff = mainMediaElement.currentTime - secondaryMediaElement.currentTime;
  if (diff > 1) {
    // Seek to catch up.
    secondaryMediaElement.currentTime = mainMediaElement.currentTime;
    return 0;
  } else if (diff > 0.2) {
    return 1.02;
  } else if (diff > 0.1) {
    return 1.01;
  } else if (diff < -1) {
    // Seek to catch up.
    secondaryMediaElement.currentTime = mainMediaElement.currentTime;
    return 0;
  } else if (diff < -0.2) {
    return 0.98;
  } else if (diff < -0.1) {
    return 0.99;
  }
  return 1;
}

You will also want to listen to some other events on mainMediaElement and use them to synchronize state to the secondary elements:

play
pause
waiting (treat like pause)
canplay (play secondary if main not paused)
seeking (sync time and paused state)
ratechange (if the main element's rate can be changed, multiply with the computed secondary rate)
volumechange (if the main element has audio)

I hope this helps!  It has been working well for me so far in my project.

Thank you Joey for the code and the permission to use it.
That's a great answer to my question!


clear input when stop signup

Location
account complete
Expected behavior
when aborting the account create flow, all input is deleted (name, avatar, password)
Actual behavior
the inputfields are filled with the previous information when creating again, after aborting the first round "create account"
Steps to reproduce the problem
create account - username and avatar - password - account complete - stop signup - create account - username and avatar (is prefilled now with past info) - password (prefilled with past password)
Operating system / hardware
macOS Sierra

@kikipluche Should be fixed now

check


XLA: softmax with numerically masked inputs does not match its non-XLA counterpart on CPU

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
2.8
Custom Code
No
OS Platform and Distribution
Linux Ubuntu 20.04.3 LTS
Mobile device
No response
Python version
3.8.10
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
11.4
GPU model and memory
T4 (16 GB)
Current Behaviour?
TL;DR if a function contains a softmax after a numerically masked input (i.e. after adding a very large negative penalty), and that function is compiled with `tf.function(jit_compile=True)`, its CPU output is very different from its non compiled counterpart. On GPU, the two outputs are very close.

Long description:
On NLP models, like `T5`, it is common to have multiple inputs in the same batch with different lengths. To handle them without `RaggedTensors`, we pad the shortest entries in the batch to the longest length, and send the padded tokens along an attention mask (containing 0s where padding happened, 1s otherwise). Internally, in the model, we convert this binary attention mask to a numerical one, containing a large penalty (e.g. `-1e9` in Hugging Face's `T5`) that will be added to the input of the attention layers' softmax, such that no attention is given to padding. As such, this numerically masked softmax is a common operation in large language models.

As part of our efforts to speed up text generation at Hugging Face, we struggled to reproduce forward passes of some models when they were compiled with `tf.function(jit_compile=True)` (https://github.com/huggingface/transformers/issues/16838). Upon further inspection, we noticed that:
- The problematic behavior was CPU-only;
- The issue came from the softmax operation, but only when masking was present;
- It is present even when the attention mask contains all 1s, meaning that the large negative penalty doesn't get applied at all to the inputs;
- Reducing the large penalty to a not so large penalty, like `-100`, still results in noticable mismatches.

The snippet below gives a simple example where the problem described above can be seen -- it passes on GPU but fails on CPU. Because it is easily reproducible, I'm not including the XLA compilation files (as suggested [here](https://www.tensorflow.org/xla#reproducible_bug_reports)), but feel free to request them :)

Standlone code to reproduce the issue
import tensorflow as tf

# same outcome for values smaller than -1e3
LARGE_PENALTY = -1e9

def simple_softmax(x):
    return tf.nn.softmax(x)

def masked_softmax(x, boolean_mask):
    numerical_mask = (1. - tf.cast(boolean_mask, dtype=tf.float32)) * LARGE_PENALTY
    masked_x = x + numerical_mask
    return tf.nn.softmax(masked_x)

xla_masked_softmax = tf.function(masked_softmax, jit_compile=True)
xla_simple_softmax = tf.function(simple_softmax, jit_compile=True)
x = tf.random.normal((1, 10))

# same outcome regardless of the boolean mask here
boolean_mask = tf.convert_to_tensor([[1] * 9 + [0] * 1], dtype=tf.int32)

# masks input outside of the compiled softmax -> works correctly on CPU and GPU
numerical_mask = (1. - tf.cast(boolean_mask, dtype=tf.float32)) * LARGE_PENALTY
masked_x = x + numerical_mask
xla_out = xla_simple_softmax(masked_x)
out = simple_softmax(masked_x)
print(tf.math.reduce_max(tf.math.abs(xla_out - out)).numpy())
assert tf.experimental.numpy.allclose(xla_out, out)

# masked_softmax -> fails regardless of the mask on CPU, works correctly on GPU
xla_out = xla_masked_softmax(x, boolean_mask)
out = masked_softmax(x, boolean_mask)
print(tf.math.reduce_max(tf.math.abs(xla_out - out)).numpy())
assert tf.experimental.numpy.allclose(xla_out, out)

Relevant log output
No response

cc @Rocketknight1
Related issue: https://github.com/huggingface/transformers/issues/16838

I forgot to add an important detail to this issue: if we add a constant to the input of the XLA-compiled softmax, which should result in the exact same output, then CPU+XLA behaves correctly. (cc @ydshieh)
As an example, the following script runs well on CPU and GPU. The only difference is a +1 in the softmax. Adding +1 to the input before the XLA function yields the same CPU-related problem.
import tensorflow as tf

# same outcome for values smaller than -1e3
LARGE_PENALTY = -1e9

def simple_softmax(x):
    return tf.nn.softmax(x)

def masked_softmax(x, boolean_mask):
    numerical_mask = (1. - tf.cast(boolean_mask, dtype=tf.float32)) * LARGE_PENALTY
    masked_x = x + numerical_mask
    return tf.nn.softmax(masked_x + 1)

xla_masked_softmax = tf.function(masked_softmax, jit_compile=True)
xla_simple_softmax = tf.function(simple_softmax, jit_compile=True)
x = tf.random.normal((1, 10))

# same outcome regardless of the boolean mask here
boolean_mask = tf.convert_to_tensor([[1] * 9 + [0] * 1], dtype=tf.int32)

# masks input outside of the compiled softmax -> works correctly on CPU and GPU
numerical_mask = (1. - tf.cast(boolean_mask, dtype=tf.float32)) * LARGE_PENALTY
masked_x = x + numerical_mask
xla_out = xla_simple_softmax(masked_x)
out = simple_softmax(masked_x)
print(tf.math.reduce_max(tf.math.abs(xla_out - out)).numpy())
assert tf.experimental.numpy.allclose(xla_out, out)

# masked_softmax -> fails regardless of the mask on CPU, works correctly on GPU
xla_out = xla_masked_softmax(x, boolean_mask)
out = masked_softmax(x, boolean_mask)
print(tf.math.reduce_max(tf.math.abs(xla_out - out)).numpy())
assert tf.experimental.numpy.allclose(xla_out, out)

@gadagashwini  I was able to replicate the issue on colab using TF v2.8.0 and tf-nightly on both cpu and gpu runtime.Please refer to the attached  gists.
Thanks!

@gante,
I tried to execute the mentioned code on the latest tensorflow v2.14 and observed that the code was providing the output which was matching on both GPU and CPU environments. Kindly find the attached gist. Thank you!


What you expected to happen:
Not that.
How to reproduce it (as minimally and precisely as possible):

Use really any kubectl command, such as kubectl get namespace. On my Mac, I can't even run terminal commands in other Terminal.app tabs until the kubectl command terminates. I get a spinning beachball for my mouse cursor. My computer becomes unusable for several minutes.
Anything else we need to know?:
Environment:

Kubernetes client and server versions (use kubectl version):

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.4", GitCommit:"e87da0bd6e03ec3fea7933c4b5263d151aafd07c", GitTreeState:"clean", BuildDate:"2021-02-21T20:21:49Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"19+", GitVersion:"v1.19.8-eks-96780e", GitCommit:"96780e1b30acbf0a52c38b6030d7853e575bcdf3", GitTreeState:"clean", BuildDate:"2021-03-10T21:32:29Z", GoVersion:"go1.15.8", Compiler:"gc", Platform:"linux/amd64"}

Cloud provider or hardware configuration: EKS
OS (e.g: cat /etc/os-release):

macOS 11.3 Big Sur

Possible duplicate of https://github.com/kubernetes/kubectl/issues/1053
@mcandre does it happen always, or just frequently but random?
Also, do you have an M1 Mac?
Anything in system logs during that time?

Let's track this on the other issue.
@mcandre please respond to open questions there to help us debug.
/close
/triage duplicate


Cannot see Redshift Spectrum Tables under the Tables Node in the Navigator

Discussed in https://github.com/dbeaver/dbeaver/discussions/14037

Originally posted by ANTGOMEZ September 28, 2021
I have the latest version of Community Edition DBeaver, I also have the very latest AWS JDBC driver, from here: https://docs.aws.amazon.com/redshift/latest/mgmt/jdbc20-download-driver.html
I  create a new Redshift SSO connection, expand a DB down to the Spectrum schema and expand the Tables node, no tables show up. The tables do exist, I can view them in another client navigator. Also, others that use DBeaver Community here can see the tables with same driver and version.
Any advice appreciated.
Thanks!

Seeing the same issues here. However, the tables show up after the drivers and connections are initially setup. Then, suddenly, out of the blue, the tables won't show anymore. Using the latest version of DBeaver and JDBC drivers. We are also using IAM authentication with browser based plugin (SSO / MFA).
For me, I noticed the problem occur after adding a second connection to the same Redshift cluster but a different database.
This also occurred for another user on the previous version of DBeaver (21.2.0). When we cleaned everything up (deleted appdata), installed the newest version of Dbeaver and tested again, it worked, again.
There are no errors/warnings in the DBeaver log. Driver debug logging looks mostly "normal" to me. I can see the queries built and sent to the database when I refresh my table views. However, I do see a repeating error after these queries, "Sep 28 15:54:27.382 ERROR  [21 DBeaver: Refresh navigator object(s)] com.amazon.redshift.jdbc.RedshiftStatementImpl.cancel: statementState is StatementCancelState.IDLE"

@appleoddity Can you please verify that you have all query types enabled in query manager?

@Matvey16
I have confirmed the problem persists whether I login through an IAM / SSO role, or directly through a database user. Even when using the cluster admin user.
I have turned on the Redshift driver debug logging. During that time, I delete both the driver configuration and the connection and re-added it. One time it displayed the external spectrum tables and then immediately quit working again. So, I was able to capture the logs for both events. Yet, I am still unable to make heads or tails of why it is behaving the way it is.
My previous statement about queries sent to the database was not accurate. From the logs I can see additional queries sent to the database, but for some reason Redshift doesn't register those in svl_statementtext.
I do see a difference in the queries sent when it works, and when it does not work, but everything else looks the same in the logs. Namely, when it works, DBeaver sends the query SELECT * FROM pg_catalog.svv_external_tables WHERE schemaname='<external schema name>';
The following queries are sent when I try to expand the tables in the external schema within the UI.
The exact query flow when it works is:
SELECT current_schema(),session_user

SHOW search_path

SELECT db.oid,db.* FROM pg_catalog.pg_database db WHERE datname='<My DB Name>'

select * from pg_catalog.pg_settings where name='standard_conforming_strings'

SELECT version()

SELECT 1 FROM pg_catalog.pg_attribute s
JOIN pg_catalog.pg_class p ON s.attrelid = p.oid
JOIN pg_catalog.pg_namespace n ON p.relnamespace = n.oid
WHERE p.relname = 'pg_type'
AND n.nspname = 'pg_catalog'
AND s.attname = 'typcategory'

SELECT t.oid,t.*,c.relkind,format_type(nullif(t.typbasetype, 0), t.typtypmod) as base_type_name, d.description
FROM pg_catalog.pg_type t
LEFT OUTER JOIN pg_catalog.pg_class c ON c.oid=t.typrelid
LEFT OUTER JOIN pg_catalog.pg_description d ON t.oid=d.objoid
WHERE t.typname IS NOT NULL
AND (c.relkind IS NULL OR c.relkind = 'c')

SELECT * FROM pg_catalog.svv_external_tables WHERE schemaname='<My External Schema>';

The exact query flow when it doesn't work is this:
SET application_name = 'DBeaver 21.2.4 - Metadata <My DB Name>'

SELECT current_schema(),session_user

SHOW search_path

SELECT db.oid,db.* FROM pg_catalog.pg_database db WHERE datname='My DB Name'

select * from pg_catalog.pg_settings where name='standard_conforming_strings'

SELECT version()

SELECT 1 FROM pg_catalog.pg_attribute s
JOIN pg_catalog.pg_class p ON s.attrelid = p.oid
JOIN pg_catalog.pg_namespace n ON p.relnamespace = n.oid
WHERE p.relname = 'pg_type'
AND n.nspname = 'pg_catalog'
AND s.attname = 'typcategory'

SELECT t.oid,t.*,c.relkind,format_type(nullif(t.typbasetype, 0), t.typtypmod) as base_type_name, d.description
FROM pg_catalog.pg_type t
LEFT OUTER JOIN pg_catalog.pg_class c ON c.oid=t.typrelid
LEFT OUTER JOIN pg_catalog.pg_description d ON t.oid=d.objoid
WHERE t.typname IS NOT NULL
AND (c.relkind IS NULL OR c.relkind = 'c')

Notice that DBeaver has added a set statement in the beginning, and neglected to SELECT * FROM pg_catalog.svv_external_tables in the end when it does not work. This makes sense, as it is that final query that provides the list of external tables.
What else can I do to help get this fixed?

@Matvey16
I started to dig in to the source code for this issue.
I don't fully understand the details behind the DBPDriver.GetDriverParameter function. But, I have discovered that DBeaver is not getting the proper "serverType" property from the driver.
In the DBeaver debug log, I can see DBeaver writes:
"Initiate connection to PostgreSQL database..." - when it doesn't work.
"Initiate connection to Redshift database..." - when it does work.
It seems to work one time after creating a new copy of the Redshift driver and new database connection. Then, the next time we connect to the database (perhaps after closing and opening DBeaver) it no longer recognizes the proper serverType.
I still have not reverse engineered this code enough to determine where the driver parameter "serverType" is coming from. Seems like it might be an artifact of the DBeaver plugin.xml for org.jkiss.dbeaver.postgresql.serverType. But, I'm also wondering if this is in the driver package itself, somewhere.
FYI, I am running the Redshift <IP_ADDRESS> JDBC driver. I have tried manually downloading it and using it, and also allowing DBeaver to download it within the driver manager.

@Matvey16
Ok. I have discovered the issue and how to reproduce.
If I go in to Driver Manager and I select the existing "Redshift" driver and then Copy that driver, it seems like DBeaver will no longer recognize the new drivers as serverType = Redshift. Instead, it changes to "PostgreSQL." I don't need to modify anything in the copied driver. Just simply copying it, breaks it.
I suspect another indication of that is the changed icon:

However, if I create the copy of the driver, and then create a new connection to Redshift using that driver, DBeaver will recognize this as a Redshift serverType on the first connection and work properly. If I close DBeaver and re-open it, and try to connect, all subsequent connections will use the "PostgreSQL," serverType.
If I, instead, modify the settings of the original "Redshift" driver, it will continue to work fine. I have to modify the parameters because the JDBC URL template needs to be changed in my use case. But, I wanted the flexibility of having two "Redshift" drivers. One with the original settings, and one with the alternate settings.
Is this a misunderstanding of the driver copy functionality? Or is this a bug? Should I create a new issue with this detail?

@appleoddity Thank you for the investigation. It is indeed a bug, although it is reproducible only in Community version. You don't need to open a separate issue, we will work on that in scope of this issue and your information will be very helpful.

Verified


Session error handler not working as expected (or at all?)

To start with, here are the libraries I am using:

Ember Inspector
3.6.0

Ember
3.8.1

Ember Data
3.8.0

jQuery
3.3.1

Ember Bootstrap
2.6.1

Ember Simple Auth
1.8.2

I am attempting to implement a basic Oauth2 ember app using the docs and the dummy application as a guide.
I have noticed that the error handler of a failed authentication does not seem to be working as expected (or at all?)...
Note: I am using Mirage to simulate the /token endpoint as follows:

webapp/mirage/config.js

import Mirage from 'ember-cli-mirage';

export default function() {
  this.urlPrefix = 'http://localhost:1337';

  function formEncodedToJson(encoded) {
    var result = {};
    encoded.split("&").forEach(function(part) {
      var item = part.split("=");
      result[item[0]] = decodeURIComponent(item[1]);
    });
    return result;
  }

  this.post('/token', (schema, request) => {
    var params = formEncodedToJson(request.requestBody);
    console.info(params);

    if (params.password === "success") {
      return {
        "access_token": "PA$$WORD",
        "token_type": "bearer"
      };
    } else {
      var body = {
        "error": "invalid_request",
        "error_description": "Email or password is invalid"
      };
      return new Mirage.Response(400, { "Content-Type": "application/json" }, body);
    }
  });

  this.passthrough('http://localhost:9000/**'); // Passthrough Sentry (if being used)
}

webapp/app/components/login-form.js:

import { inject as service } from '@ember/service';
import Component from '@ember/component';

export default Component.extend({
  session: service('session'),
  actions: {
    authenticateWithOAuth2() {
      let { identification, password } = this.getProperties('email', 'password');
      console.info("Attempting login for " + identification);
      this.get('session').authenticate('authenticator:oauth2', identification, password)
        .then(() => {
          this.get('rememberMe') && this.set('session.store.cookieExpirationTime', 60 * 60 * 24 * 14);
        })
        .catch((reason) => {
          console.error("Authentication Error!")
          console.error(reason);
          this.set('errorMessage', reason.error);
        });
    }
  }
});

Based on the console output, I can see that Mirage is successfully generating the error and its structured based on the outline here: https://tools.ietf.org/html/rfc6749#section-5.2 (which is also mentioned in the API Docs)...
The issue is, I cannot understand how ember-simple-auth is supposed to be handling the response (assuming that it does?)....
Am I expected to handle the response and output the error_description in my own fancy way? Based on the example here: https://github.com/simplabs/ember-simple-auth/blob/master/tests/dummy/app/templates/components/login-form.hbs, it looks like the session object is supposed to be handling the response and returning an actual formatted error? Is that a correct assumption?
The response I see in the console (referenced variable: reason) is this:
Response {type: "default", status: 400, ok: false, statusText: "Bad Request", headers: Headers, …}
bodyUsed: true
headers: Headers {map: {…}}
ok: false
responseJSON: {error: "invalid_request", error_description: "Email or password is invalid"}
status: 400
statusText: "Bad Request"
type: "default"
url: ""
_bodyInit: "{"error":"invalid_request","error_description":"Email or password is invalid"}"
_bodyText: "{"error":"invalid_request","error_description":"Email or password is invalid"}"
__proto__: Object

The referenced variable: reason.error (as modeled after dummy app) is this:
undefined
Thanks for any insight you can provide (and let me know if I forgot some code reference you need)!

Okay.... so I do believe this was mostly my doing... I was trying to troubleshoot this error and inadvertently forgot to turn off rejectWithResponse which when false, seems to process the response.
I guess now that was cleared up, I now need to interpret the error/description and make things clear for the end user.
Thanks for reading!

Yes, there is no automatic error handling in ESA. The session would simply not be authenticated and the promise returned from sesison.authenticate rejects. Be aware though that
let { identification, password } = this.getProperties('email', 'password');

is not going to work - you'll explicitly have to alias email to identification:
let { email: identification, password } = this.getProperties('email', 'password');


Add MD parsing support and fix 1.16 building

Thanks, @ElijahPepe it look beutiful!

Thanks, @ElijahPepe it look beautiful!


[Perf] [UI] Added Subscription for performance Test

Description

Added Graphql Subscription for listening to performance profiles changes and to results.

This PR fixes #4607
https://user-images.githubusercontent.com/51405571/150200868-9380c692-26a0-4397-a3b5-fa0417d84532.mp4
Notes for Reviewers

The resolver sends data whenever there is a test completion or the profile is being saved.

Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #4963 (8a7ae7b) into master (edb9265) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4963      +/-   ##
==========================================
+ Coverage   32.43%   32.47%   +0.03%
==========================================
  Files          61       61
  Lines        5068     5028      -40
==========================================
- Hits         1644     1633      -11
+ Misses       3014     2991      -23
+ Partials      410      404       -6

Flag
Coverage Δ

e2etests
∅ <ø> (?)

gointegrationtests
9.36% <ø> (+0.09%)
:arrow_up:

integrationtests
∅ <ø> (?)

unittests
25.87% <ø> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/perf/profile.go
45.71% <0.00%> (-4.29%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
24.05% <0.00%> (-0.10%)
:arrow_down:

mesheryctl/pkg/utils/helpers.go
66.10% <0.00%> (ø)

mesheryctl/internal/cli/root/system/start.go
56.73% <0.00%> (+0.14%)
:arrow_up:

mesheryctl/internal/cli/root/perf/result.go
68.90% <0.00%> (+1.95%)
:arrow_up:

mesheryctl/internal/cli/root/perf/apply.go
42.04% <0.00%> (+3.81%)
:arrow_up:

Continue to review full report at Codecov.

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

@manav1403 heads-up - https://github.com/meshery/meshery/runs/4872971781?check_suite_focus=true#step:3:75


Add ability to arbitrarily reset page numbering

I would like support to reset page numbers from an arbitrary position in the document. I would also like to be able to use roman numerals as page numbers.
for example, my document has an introduction that I would like to give a separate numbering scheme, so the final page numbers should be something like
i, ii, iii, iv, 1, 2, 3... 20, 21

I was looking for exactly that. (roman numeral section, followed by the normal section where I choose which one is "page 1")

I need something similar. The page numbering must be like:
1, 2, 3.. 20, 21, i, ii, iii, iv
where the roman numbering is for the appendices.

Adding my voice to the choir, although I'm not sure if my requirement is slightly different.
I have a single document that includes other documents to make a book. I would like each includeed part to have its own page numbering in the footer. Something like {chapter-number} - {chapter page number}. From experimentation, modifying a custom {chapter-number} attribute throughout doesn't work because the footer appears to happen last, and so only uses the final value of custom attributes.
Perhaps better would be to have a "progress indicator" like {chapter-number} - {chapter page number}/{total number of pages in chapter}
Let me know if this makes sense.

If we focus on the initial request of allowing the arabic page numbering to start beyond the first page of the body, I think we can achieve it. Something like:
page:
  numbering:
    start_at: 4

The value is assumed to be a physical body number. In other words, the keyword body is 1, and here the 4th physical body page will be virtual page number 1. All pages before are in roman numerals.

1, 2, 3.. 20, 21, i, ii, iii, iv

This is outside the scope of the design of this converter.
@chuck-confluent what you are asking about would be an entirely different request. I have already been thinking about setting an implicit attribute in the running content to hold the chapter number. There may be room to store the page-number-of-chapter as well, but I'm not sure.

From experimentation, modifying a custom {chapter-number} attribute throughout doesn't work

No, it would not. The running content should only be accessing implicit attributes or attributes defined in the document header.


Multiple ratings are not displayed in a single html webpage

There are multiple search results in my webpage. But the stars for the rating are being displayed only for the first search result. The remaining results have no stars to give the rating. How can I display the stars after each search result in the same page?
I have created a div in the results section of the html page  :-

and written the following code in the js file :-

What corrections need to be done in the above codes to get the desired result?

@Swati-Saluja , Please do not use id, instead you can give a class like

 for all the rating elements , and do $(".rate-yo").rateYo() just like how i did in docs( rateyo.fundoocode.ninja ) .


Add dotnet tool: nbgv

As a high-level summary of the new nbgv dotnet console tool, here is the top-level usage text:
usage: nbgv <command> [<args>]

    install        Prepares a project to have version stamps applied
                   using Nerdbank.GitVersioning.
    get-version    Gets the version information for a project.
    set-version    Updates the version stamp that is applied to a
                   project.
    tag            Creates a git tag to mark a version.
    get-commits    Gets the commit(s) that match a given version.
    cloud          Communicates with the ambient cloud build to set the
                   build number and/or other cloud build variables.

Additional help is available for each individual command.
Closes #181

CC: @onovotny


Launch with nextflow tower

The nextflow core site has a launch with mechanism that looks an awful lot like what we ended up with for WDL/CWL launch-with.
Let's see if we can use it!
Example at https://nf-co.re/launch?pipeline=mag&release=1.2.0 (pick ampliseq, pick a few parameters)
Note that the browser goes to https://tower.nf/launch?pipeline=https:%2F%2Fgithub.com%2Fnf-core%2Fampliseq&revision=1.2.0&paramsText={
    "FW_primer": "foo",
    "RV_primer": "foo"
}&resume=false
However https://tower.nf/launch?pipeline=https:%2F%2Fgithub.com%2Fnf-core%2Fampliseq&revision=1.2.0 also works in lieu of us implementing #4039
This could be a nice way of filling this feature gap without much development
┆Issue is synchronized with this Jira Story
┆Issue Number: DOCK-1708
┆Sprint: Sprint 55- Dusky shark
┆Issue Type: Story

Isn't NextFlow Tower going to be like Galaxy, where people can spin up their own? If so, we may eventually need to bring in the DNAstack/Galaxy UI where users can specify an endpoint.

The community version of Tower is available from this repository. It can be deployed in a user's own environment and has features for single users to monitor their Nextflow pipelines, deployed anywhere.

The fully-featured enterprise version of Tower is available from Seqera Labs. It can be deployed in a customers own on-premise or cloud environment and includes advanced workflow management, resource optimization and enterprise-grade support.

Yup

Hello, that would be welcome! Just one note, the Tower community distribution linked above currently does not include the launch feature.

@pditommaso  Thanks for the heads-up!
So to clarify, https://tower.nf/ is running the Tower enterprise distribution. https://github.com/seqeralabs/nf-tower contains the source code for the community edition but not the enterprise edition. If this is correct, we'll note this in our documentation for launch with.
Out of curiosity/keeping up, is there a page that lists the differences between the two versions?

Yes, exactly.

Out of curiosity/keeping up, is there a page that lists the differences between the two versions?

We should have a new web site in a few days, with a comparing matrix. I'll link it here. But in a nutshell, the community just monitoring. The enterprise version adds pipelines launch and multi-cloud supports.


Separate graffiti per validator

For opsec reasons, I believe a seperate graffiti option per validator is a great idea.

Although this is an alpha feature, we have included this feature with a proposerSettingsFile in our unstable branch to allow this. Please see https://github.com/ChainSafe/lodestar/pull/4494 and leave us any feedback if there's issues!


Additional resource for linux in Devops roadmap

Description
Additional resource for linux in Devops roadmap
Screenshots
No response
Additional information
No response

hi
can you assigned me this issue?
which kind of resource is better or recommended (gap of resource from your consideration)?

Can I contribute to this issue and add a few links for Linux resources?

I would like to work on this issue,i have a few links for this

I would like to work on this ,  I would like to add  few server management videos and some few others Linux videos like important commands and stuff .

@BharatSharma29 I don't think it's a good idea to add more resources in the Linux Section. It is already looking bloated & overwhelming too. Try to keep it short & simple, basic knowledge of Linux is way more than enough to get started.
Thank You


IM encoding spec rename/update several IB, needs to update the corresponding code in app/MessageDef

Problem
IM encoding spec rename/update several IB, needs to update the corresponding code in app/MessageDef

Mentioned here: https://github.com/project-chip/connectedhomeip/pull/9510#discussion_r706314636
Among other things, SubscriptionID needs to be changed from 64bit to 32bit.  Likewise, the special 'invalid' subscription ID used in the PR, UINT64_MAX, needs handling and a #define or constexpr.

yep, agree. thanks

All have been completed.


feature: upgrade l10n system and update files

Description
This PR will merge the whole "l10n upgrades" feature branch at once into main. This includes:

A new version of l10nCheck.js that automatically fixes most of the problems it finds and reports back to the user about possible issues that need human attention
Updated localization files

Task Item
#minor

Coverage remained the same at 50.376% when pulling 80d879973d81a1f95867ed970b52263c56b169f2 on beyackle/l10nUpgrades into 1b9d6699e6476b8856caa9bc30471f4375c06cf5 on main.


Problem Querying ailed, [1] retry attempts left

Hello!
I'm using the latest version and I'm having the same problem as shown. Please help me fix it.

ha core rebuild

ha core rebuild

This fixed it for me


Revert unwise mina version change

This reverts commit 4ca825e0984c37a225d0ece77049911ba21bd71d, reversing changes made to 58e286bfd0276be52127dca573cd1301a2adcb95.
Some problems were found with the PR #15638 , so it's best to revert it for now.

!ci-build-me


Update issue templates

Add the default Github Issue templates well adding our personal tags.

You can view the Issue Templates here -> https://github.com/CannonLock/pelican/issues/new/choose

@CannonLock I saw there seems to be a bunch of unrelated commits in this PR. Could you confirm you pushed to the correct branch?


Refurbish the site?

If you are prepared to do the work, you can make it any colour you like :-)
As you've probably seen, the site is built on Jekyll from a mix of Markdown and HTML.
I've got a large collection of ugly scripts that help me keep the docs up-to-date and, importantly, correctly linked.
I like the split / organisation of the pages so far, but I'm open to suggestions.

Why don't we write those pages in markdown? eg. news.html, index.html
I guess it will be a lot easier to maintain if we use markdown.
And WHAT!!? 246 KB for just the main site is already overdoing it. I can help to shrink it to ~10KB
Substituting rss image with svg would be a nice idea.
But what is the jquery for? Is it really that important? (it took ⅓ of the size of the site)
I can't even imaging that this site uses bootstrap (by it's appearance), maybe we could void that too by using normalize.css with some custom css.
Maybe those js files isn't required but I am not sure about it.
http://shivamverma.info/2016/04/22/github-corner-text/ Maybe that too or maybe I can write it myself
It would be great too to see the site looks better on mobile (My site looks horrible on mobile too, maybe I could use this chance to improve it)
Maybe we could based it with papercolor like my site too? https://github.com/NLKNguyen/papercolor-theme

OK, to all your suggestions.
The jquery?  I don't know.  I needed a site and the template came with it.

bootstrap ... normalize.css ... custom css.

OK
FlatCap enjoys the wooshing noise as things fly over his head :-)

GitHub corner?

Looks nice.

site looks better on mobile

Yes, that would be an important change

papercolor-theme

OK, lovely.
Where do you want to do the development?
I'd rather the site wasn't offline at all.

@flatcap I can do the development at my own computer?

I'd rather the site wasn't offline at all.

I don't understand what that supposed to means. I guess that you are hosting it yourself (not 24/7)?

I can do the development at my own computer

ok, great

I'd rather the site wasn't offline at all.

I don't understand what that supposed to means

I just wanted to be clear that you are not editing the current website, but a copy :-)
(The site is hosted by GitHub and some CDN)
Any questions, let me know.
(I'm also in our IRC channel quite a lot: #neomutt on irc.freenode.net)

Okay!

@pickfire is this still relevant for you?

@toogley Wow, it looks a lot better now. Sorry that I can't help out much, have stuff to be done.

@pickfire no problem :)


Could not import mongo_connector.doc_managers.elastic_doc_manager

I have elastic_doc_manager installed, but I am still getting this error:

root@server:~/elastic2-doc-manager# pip install elastic2-doc-manager
Requirement already satisfied (use --upgrade to upgrade): elastic2-doc-manager in /usr/local/lib/python2.7/dist-packages/elastic2_doc_manager-0.1.0-py2.7.egg
Requirement already satisfied (use --upgrade to upgrade): mongo-connector>=2.3.0 in /usr/local/lib/python2.7/dist-packages (from elastic2-doc-manager)
Requirement already satisfied (use --upgrade to upgrade): elasticsearch>=2.0.0,<3.0.0 in /usr/local/lib/python2.7/dist-packages (from elastic2-doc-manager)
Cleaning up...

root@server:~/elastic2-doc-manager# mongo-connector -m localhost:27017 -t localhost:9200 -d elastic_doc_manager
No handlers could be found for logger "mongo_connector.util"
Traceback (most recent call last):
File "/usr/local/bin/mongo-connector", line 9, in
load_entry_point('mongo-connector==2.3', 'console_scripts', 'mongo-connector')()
File "/usr/local/lib/python2.7/dist-packages/mongo_connector/util.py", line 85, in wrapped
func(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/mongo_connector/connector.py", line 1041, in main
conf.parse_args()
File "/usr/local/lib/python2.7/dist-packages/mongo_connector/config.py", line 118, in parse_args
option, dict((k, values.get(k)) for k in option.cli_names))
File "/usr/local/lib/python2.7/dist-packages/mongo_connector/connector.py", line 824, in apply_doc_managers
module = import_dm_by_name(dm['docManager'])
File "/usr/local/lib/python2.7/dist-packages/mongo_connector/connector.py", line 814, in import_dm_by_name
"vailable doc managers." % full_name)
mongo_connector.errors.InvalidConfiguration: Could not import mongo_connector.doc_managers.elastic_doc_manager. It could be that this doc manager has been moved out of this project and is maintained elsewhere. Make sure that you have the doc manager installed alongside mongo-connector. Check the README for a list of available doc managers.

Replica set is enabled.
OS: Ubuntu 14.04 Server
Python: 2.7.6

I've found the solution:
Instead of:
mongo-connector -m localhost:27017 -t localhost:9200 -d elastic_doc_manager
we should use:
mongo-connector -m localhost:27017 -t localhost:9200 -d elastic2_doc_manager
Would be great, if you can update wiki.

Edited the mongo-connector wiki to show both usages.


Various Small Fixes

Fixes Various Minigames

Zombiquarium
Beghouled
Beghouled Plant Flash & Crater Tutorial Arrow Position
Font Drawing
Tallnut Having Wallnut Head
Bungee In Almanac
Garlic Fix
Rain Fix
Many Other Small Fixes (etc. typos)

[15/128] Building CXX object CMakeFiles/re-plants-vs-zombies.dir/Lawn/Challenge.cpp.obj
FAILED: CMakeFiles/re-plants-vs-zombies.dir/Lawn/Challenge.cpp.obj
D:\a\_temp\msys64\mingw32\bin\c++.exe -DWINDOWS -D_DEBUG -ID:/a/re-plants-vs-zombies/re-plants-vs-zombies/SexyAppFramework -ID:/a/re-plants-vs-zombies/re-plants-vs-zombies/SexyAppFramework/sound/SDL-Mixer-X/include -ID:/a/re-plants-vs-zombies/re-plants-vs-zombies/SexyAppFramework/platform/pc -isystem D:/a/_temp/msys64/mingw32/include/SDL2  -MD -MT CMakeFiles/re-plants-vs-zombies.dir/Lawn/Challenge.cpp.obj -MF CMakeFiles\re-plants-vs-zombies.dir\Lawn\Challenge.cpp.obj.d -o CMakeFiles/re-plants-vs-zombies.dir/Lawn/Challenge.cpp.obj -c D:/a/re-plants-vs-zombies/re-plants-vs-zombies/Lawn/Challenge.cpp
D:/a/re-plants-vs-zombies/re-plants-vs-zombies/Lawn/Challenge.cpp:3111:7: error: redefinition of 'void Challenge::DrawRain(Sexy::Graphics*)'
 3111 | }void Challenge::DrawRain(Graphics* g)
      |       ^~~~~~~~~
D:/a/re-plants-vs-zombies/re-plants-vs-zombies/Lawn/Challenge.cpp:3068:6: note: 'void Challenge::DrawRain(Sexy::Graphics*)' previously defined here
 3068 | void Challenge::DrawRain(Graphics* g)
      |      ^~~~~~~~~

i cant seem to test on windows. theres alot of posix functions unavaliable for windows like unistd and pthreads.
but i just fixed this

you might be using visual studio. i use MSYS2 MinGW to compile on windows


Adding the RA MTU option

Implemented the MTU option for Router Advertisement Message (RA) described in RFC4681.
With the addition of this option, it is possible to publicize RA containing MTU from config.

One more, please remove this TBD.
https://github.com/YutaroHayakawa/go-ra/blob/main/README.md?plain=1#L19

Now all looks good! Thanks! Merging...


immutable.js update all the states

When I change the name, all the state in redux will be updated, it will cause all the components which connect to redux render again. Just like the following picture, you will see it when you check the Trace React Updates in react devtool, all the components were rendered again. Is that reasonable?

@mxstbr I don't think it's a good practice to use react and redux in this way. It will contribute to  react virtual DOM deep compare inside. If there are tons of states in redux, react will compare all of the element to decide wether the DOM element should rerender. Is that will not cause app slow? Why not just return a new object relate the component and combine shouldComponentUpdate to reduce react compare inside?

@mxstbr Maybe I have some misunderstanding in immutable.js, I should pass the map object instead of the toJS() object to react, and then shallow compare the map object in shouldComponentUpdate. Thanks!😀

That's exactly it @tinybug, sorry for not being more explicit!

Have a look at react-immutable-render-mixin as FB's shallowEqual doesn't compare well immutables from immutablejs and Immutable.is() won't work with plain obj.
import shallowEqual from 'fbjs/lib/shallowEqual';
import Immutable, { fromJS } from 'immutable';
import { shallowEqualImmutable } from 'react-immutable-render-mixin';

const objA = { a: 1 };
const objB = { a: 1 };
const immA = fromJS(objA);
const immB = fromJS(objB);

console.log('shallowEqual', shallowEqual(objA, objB), shallowEqual(immA, immB));
// ==> shallowEqual true false
console.log('Immutable', Immutable.is(objA, objB), Immutable.is(immA, immB));
// ==> Immutable false true
console.log('shallowEqualImmutable', shallowEqualImmutable(objA, objB), shallowEqualImmutable(immA, immB));
// ==> shallowEqualImmutable true true


[Bug] FreeTAKServer register page bugs

Hi,
I noticed a small bug where when the exception for a user which exists is triggered the left navigation drawer menu is exposed with clickable buttons.
Also when attempting to create an account on the FreeTAKServer , clicking the register button presents what looks like a SQL error. See the images below:
User exists bug:

Create an account error:

register button has been removed


[AUTOPATCHER] Patches CVE-2021-43267 for kernel

        Patches CVE-2021-43267 for kernel.
        This patch and PR was automatically generated and needs human verification before merging.

Closing- fixed by #1639


feat: if there are multiple education entries, as a user, i can see those education entries ranked in reverse chronological order

this is a subtask of #8
#15 will enhance this feature to give users more control

conditions

[x] if all entries have an end date, then the one that ended (graduated) most recently is displayed
[ ] entries without end date should be pushed to the top (as we assume they are still going with it)
[ ] between entries without end dates, the start dates should determine rank, in reverse chronological order (the one that was started later is further up)

this is proving difficult to capture all the situations where user enters end date but not start date, vice versa, both, or neither... I'm almost tempted to just enable sorting by end dates when they are provided, and if they are not provided, they are just added to the top of the list, and the user can just reorder it themselves (issue #15)

moved into waiting to plan next steps

consider making dates compulsory to avoid having to deal with entries without it, and how to display without them

consider making dates compulsory to avoid having to deal with entries without it, and how to display without them

i have begun doing this, however, there are some challenges with styling the month inputs on change of the value


Update requirements.txt

checked and updated new versions of python_jwt and jwcrypto

Hi, thanks for your contribution! Unfortunately, I don't see how this PR improves our project. The versions specified in requirements.txt use the compatible release clause, so pip will automatically use the most recent patch version of the specified minor version. Pinning the version to the patch version is not necessary imo and will only needlessly increase the number of Dependabot alerts for our repository (cf. https://github.com/timewarrior-synchronize/timew-sync-client/pull/38).


[VL] The compression codec does not effect in Insert into command.

Describe the bug
spark sql compression configuration in InsertIntoCommand is not working
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

fixed by #2328


PDF link not working

Fantastic extension! Thanks!
Any idea why https://images-na.ssl-images-amazon.com/images/G/01/opt/general/test/KDP_W8BENa._V160201652_.pdf does not get prefixed?
My rule is match RegExp . (for ALL pages) and prefix aaa {title}
Is RegExp . the best way to match all?
(Also tried match contains :)

Hi @dzg,
Tab Modifier can only change title on an HTML page.
Although the PDF has a tab title, this is not an HTML page. So the Javascript API can't access to the title. Look at this check.
Best

OK, thanks anyway!


ISLE: Stabilize the overlap error order

We were finding rules with the most overlap errors by traversing a hash map, but weren't doing anything to prevent the unstable order of the hash map elements from affecting the error output order. This PR includes the rule id when looking for rules that have the largest number of overlaps, to break ties predictably.

I had a patch something like this. I only dropped it because I wasn't sure it was important. But apparently it is, so let's do it!
In that patch I also sorted the RuleIDs of the rest of the group. I don't think that's super important but it is another source of unstable ordering.

Ah, good point! I'll make that change as well if I continue to notice unstable ordering :+1:


ClientV2 ignoring schema inference hints and infer integer settings

Describe your feedback
We run the following query through the clickhouse java client:
CREATE TABLE {tmpTableName:Identifier}  ENGINE=Log() AS SELECT * FROM s3('...file.csv', 'CSVWithNames')

With the following QuerySettings:
date_time_input_format=best_effort,
input_format_try_infer_integers=0,
schema_inference_hints='timestamp Nullable(Float64)'
input_format_try_infer_exponent_floats=1,
precise_float_parsing=1,
input_format_try_infer_dates=1,
input_format_try_infer_datetimes=1,
schema_inference_use_cache_for_s3=0,
schema_inference_make_columns_nullable=1

In ClientV1, the created table would have Nullable(Float64) for the timestamp column. ClientV2 seems to ignore the inference hints (and the "input_format_try_infer_integers=0" setting), as the column is Nullable(Int64) instead:
TableSchema{tableName='tmp_csv_0be155a3_920d_4496_87a7_ee2facee2efc', databaseName='nominal', columns=[timestamp Nullable(Int64), a Nullable(Float64), b Nullable(String)], metadata={a={type=Nullable(Float64)}, b={type=Nullable(String)}, timestamp={type=Nullable(Int64)}}, colIndex={a=1, b=2, timestamp=0}, hasDefaults=false}

@alxhill thank you for reporting!
will look into it shortly.

Okay, looks like the culprit is that ClientV1 appends all QuerySettings to the URI, while ClientV2 only appends server settings to the URI.
ClientV1
https://github.com/ClickHouse/clickhouse-java/blob/d9cbaba2c5d4022974787e0691a455374bb8f17a/clickhouse-http-client/src/main/java/com/clickhouse/client/http/ClickHouseHttpConnection.java#L164-L170
ClientV2
https://github.com/ClickHouse/clickhouse-java/blob/d9cbaba2c5d4022974787e0691a455374bb8f17a/client-v2/src/main/java/com/clickhouse/client/api/internal/HttpAPIClientHelper.java#L511-L515
Changing from settings.setOption to settings.serverSetting fixed my tests. Seems like a break worth resolving as they're silently being ignored.

@alxhill
sorry - my bad - I would need to document this thing.
Is the issue resolved and only documentation should be changed?
Thanks!

We are unblocked, but this does seem like a break in the client & not something I would expect to see upgrading between minor versions

@alxhill sorry about that - my bad. This is consequences of using two different clients under the hood (old and new) .
We always keep in mind that changes may be breaking and so we need to handle them properly - why many fixes are getting feature flags.