Python minor performance improvements and simplifications

report more detailed performance info (used here to measure the rest of the changes)
run: install everything from requirements.txt (in case more libraries will be used in the future);
run: also remove venv in cleanup;
python: use defaultdict. Just convenience, no performance impact;
python: remove heapq. Counter can find top-k itself. Just convenience, no performance impact;
both: use ujson as a transparent replacement for json. 90 ms faster output;
numpy: generate unique tags using python set. 20 ms faster than filling an ndarray and then calling .unique();
numpy: iterate over post.tags dict instead of iterating over all possible tag ids in the inner loop. 20ms faster;
use uint8 as matrix dtype: 20ms faster than uint16;

Interesting result in c9414160a56d5122a8ccb738a8675d394fdfa987: using np.argpartition() is 30ms slower than np.argsort(), even though partition does less (it does not have to fully sort the array). I think the key is kind="stable". There's no algorithm selection for argpartition yet, and stablesort seems to be very fast on this specific dataset. Are related posts almost-sorted by relatedness?

Ooh, "stable" argsort is probably using radix sort in this case, which should be super fast given relatedness has very few possible values. https://numpy.org/doc/stable/reference/generated/numpy.argsort.html#numpy.argsort

Hey, thanks for your efforts but we need the timing to be compatible with the rest of the solutions because the benchmark results are automatically generated. The rest of your changes looks good.
Let me know if you're going to modify the pull request or I can do it.

Oh, I didn't know. What does the benchmark actually collect, the "Processing time (w/o IO): ..." lines?

Yes. I am also trying to limit the run time of the workflow because the free tier is limited.
link to regex

Changed. Now it only prints the detailed table if env DETAILED_PYTHON_PERF="1", otherwise behaves as usual.
Also now using time functions that return int nanoseconds instead of float seconds (it will not actually improve precision given the current scale of ~ second per run, but why not).

Processing times is a bit slower on my machine. ujson sped up I/O though.  Strange.

I think I know why I didn't see a difference before: version differences!
The latest result, where it's 120ms slower are in a py=3.9 venv, but the jupyter notebook with performance plots I generated in py=3.11. Probably either counter or defaultdict got optimized between 3.9 and 3.11.

I will merge and see how it performs in the VM

Yup, plot in 3.9:

Plot in 3.11:

I have 3.11.5 on my machine. It's slower on the vm too
https://github.com/jinyus/related_post_gen/pull/59

I will have to revert for now.

The script does (a): for p in heapq.nlargest(5, relation_count, key=relation_count.get)
Internally the counter does (b): for p, _n in heapq.nlargest(n, relation_count.items(), key=itemgetter(1))
If I replace (a) with (b) in the script, it has the same performance as just calling relation_count.most_common(), which makes sense.
It seems (b) is measurably slower on almost all of my systems too. I wonder if it's because in (a) the heapq only stores the keys and gets counts from the counter, but in (b) the heapq stores the item-count pairs, so it uses 2x the memory.
Checked on a PC. (b) is slower than (a) on both Linux and Windows (both in python 3.11). Performance is the same only on M1 Mac in 3.11. I wonder how.
Win10 results (1: and 2: don't have the slower top5 from counter):
commit				import 		compile 	input 		process 		output 		total
0: install all pytho… 	11.90±0.18 	0.00±0 	10.42±0.26 	2744±18 	480.9±1.8 	3248±19
0: try counter most_… 	7.09±0.18 	0.00±0 	10.17±0.17 	2810±16 	54.28±0.99 	2882±16
1: minor pythonizati… 	12.08±0.28 	0.00±0 	10.27±0.26 	2729±13 	482.0±1.9 	3233±14
2: use ujson for IO…… 	7.12±0.23 	0.00±0 	10.21±0.17 	2718±14 	54.3±1.3 	2789±14

So this convenience change is definitely out.
By the way, on Windows the uint8 numpy matrix actually runs slower than uint16:
commit				import 		compile 	input 			process 		output 		total
0: minor pythonizati… 	115.92±0.72 	0.00±0 	10.35±0.21 		768.8±3.9 	481.6±1.5 	1376.7±4.0
1: use ujson for IO…… 	117.47±0.82 	0.00±0 	10.06±0.16 		770.8±3.4 	53.09±0.59 	951.4±4.0
2: np: use python se… 	118.21±0.91 	0.00±0 	10.031±0.098 	701.8±4.1 	52.74±0.63 	882.7±4.4
3: np: use smaller m… 	117.74±0.82 	0.00±0 	9.99±0.15 		743.1±2.1 	53.10±0.60 	923.9±2.7
4: np: use dict of t… 	118.3±1.3 	0.00±0 	10.03±0.14 		696.2±5.8 	53.15±0.61 	877.8±6.0
5: try to use numpy … 	117.56±0.87 	0.00±0 	9.95±0.17 		737.8±1.8 	53.03±0.68 	918.4±1.9

Is that a factor at all for you, or can performance differences on Windows be safely ignored?


Add support for servers that don't support token based authentication

This changes causes the login process to fall back to authenticating directly with the user's password if the token method doesn't work.
This enables the application to connect to servers that don't support token-based authentication.
For context:
I'm running a self-hosted supysonic as the backend and it doesn't (and won't ever) support the token-based authentication since that methods requires the server to know the user's plaintext password.

I'd really appreciate it if you could use eslint to auto-fix the code based on the project's rules.

Will do, sorry. I've just installed a VS Code extension that should do this going forward. Would it make sense to add an extension like that to the recommended extensions list?
ie:
diff --git a/.vscode/extensions.json b/.vscode/extensions.json
index 24d7cc6..a9629ad 100644
--- a/.vscode/extensions.json
+++ b/.vscode/extensions.json
@@ -1,3 +1,3 @@
 {
-  "recommendations": ["tauri-apps.tauri-vscode", "rust-lang.rust-analyzer"]
+  "recommendations": ["tauri-apps.tauri-vscode", "rust-lang.rust-analyzer", "dbaeumer.vscode-eslint"],
 }

Would it make sense to add an extension like that to the recommended extensions list?

Yeah, totally make sense. I'll include the extension to recommendations.


integrations flow

After refactor, both - js and ruby apis are prepared for modular integrations. It can be extracted into external gems. For now we have only react-router, Alt is almost ready, redeux is in progress. What do you think about such extraction?
I personally think that we sohuld keep core integrations (router, most popular flux stuff) as part of gem and add description + example showing how other integrations can be done.

Good idea :)

It's really good idea. I love namespaces, we should not be selfish and populate global namespace too much :)
But I don't want to type too much while using core methods.
What do you think about keeping core methods in global namspace but plugins specific ones under in theirs namespace?
example:
RWR.getComponent()
RWR.alt.getStore()

Merged in https://github.com/netguru/react_webpack_rails/pull/31


Updated the build scripts to enable Multi-Arch Builds

Ready to Go
I revamped the build scripts to enable building of images, tags, and manifests. There are many new added features but here are some of the high level ones:

Scans 8 and 11 folders in the repo to get all Dockefiles to be processed
Publishes verbose tags with additional specific tags
Allows users to utilize the scripts locally or in a CI/CD fashion
Auto generates all needed tags and can be easily fine tuned to produce more tags
Will detect incompatible images (ie s390x on centos or arm64 on alpine) and filter them out of the build matrix
Easily changeable environment variables to allow users to swap out configs (For example, you can set an environment variable to change the default image repository you want to push too.)

Where I need help
Need help adapting some of the parallel code in #998 to the Jenkinsfile. Currently no modifications have been made to the Jenkinsfile, thus the automatica testing will fail. Once the Jenkinsfile has been modified to be parallel and to point at the new build scripts the Jenkins team should be able to start producing multi-arch images.
If you want to help out there is some good documentation on how to use the new build scripts in the .ci folder.
Additionally, all tags currently supported by Jenkins are still being produced by the new build scripts expect slim. I changed slim to just be a subtype of debian. So you should be able to pull down debian-busterslim but not slim. This can be change very easily if the want/need exists for the slim tag.
 I modeled a lot of the new build scripts off some ideas he had in his scripts.
Fixes #1001

Custom DockerHub repo where I been using images: https://hub.docker.com/r/jamescrowleyibm/jenkins @slide

We're going with a different approach now of using docker buildx bake, which means we don't need any shell scripting, the tool supports all we need.
If you want to re-submit tag change proposals feel free to submit a PR to:
https://github.com/jenkinsci/docker/blob/master/docker-bake.hcl


chat cli

This is a first draft of a chat CLI
Try with:
python examples/scripts/chat.py  --model Qwen/Qwen1.5-0.5B-Chat --device mps

cc @younesbelkada

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


[feature] style "Finish your profile" dialog links as purple

closes #582
What changed? Why?
This small PR changes the style of the "Finish your profile" dialog links to be purple. Why? So that it is more obvious that those specific words are clickable and would show a dialog. However, we do not underline because underlines are usually recognized as links that lead to a new page.
UI changes
OLD:
https://github.com/Stand-With-Crypto/swc-web/assets/135282747/746b102a-328e-4022-b31e-425ed24c4e28
NEW:
https://github.com/Stand-With-Crypto/swc-web/assets/135282747/6230c269-7ded-4812-ae85-646f640b35f7
PlanetScale Deploy Request
No PlanetScale schema changes.
Notes to reviewers
No specific notes.
How has it been tested?

[X] Locally
[X] Vercel Preview Branch
[ ] Unit test
[ ] Functional test

Change management
type=routine
risk=low
impact=sev5

I think this was actually on purpose :) @coinbasemar thoughts?

These should be purple if they're clickable links

They're not really links, but they are clickable... what's the verdict here @travisbloom-cb? Okay to change, or should we leave as is?


Mouse wheel produces left and right clicks

Someone else try to reproduce, please.
Scrolling up produces left (red) clicks, scrolling down produces right (yellow) clicks.

Yes, same for me. The issue is (I think) that the normal Squeak VM actually translates mousewheel into Alt+Down or something like that. @marceltaeumel knows this better.

Then for me the question is whether, in addition to the obligatory workaround mapping of the wheel events, Squeak images want to learn about the existence of a Mouse Wheel at some time. :-)

Actually, why not, if we're reporting the mouse wheel right, we should be able to just adapt the Smalltalk code to distinguish it from the buttons. I'll dig into that.

Since we still have time to the Squeak 4.6 release, we might be able to get @marceltaeumel to push that change through for us :)

Related: http://forum.world.st/Unique-Mouse-Wheel-Events-td4814473.html
Also: wat?

We now generate MouseWheelEvents, and with a recent trunk image this works.


[Bug]: view sys.index_columns reponds with a incorrect value in field column_id

What happened?
Using DBeaver to browse the database connecting with and SQL Server driver to an AWS Aurora instance with Babelfish enabled, I noticed that columns in indexed are always shown as the first columns in the table.
I finaly found that this is due to the query of the view sys.index_columns which use the attnum field from the table pg_attribute. But this table joined like it is currently give no information of the column id (or position) inside the table.
I added a extra join to pg_attribute to fix this (pull request will follow...)
Version
BABEL_3_X_DEV (Default)
Extension
None
Which flavor of Linux are you using when you see the bug?
No response
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Could you please include a short reproduction of the issue?

Create a table with indices in columns other than first ones:
create table t1 (
  c1 int
, c2 int not null
, c3 int
, c4 int
, constraint t1_pk primary key (c2)
);
create index t1_i1 on t1 (c3, c1);
create index t1_i2 on t1 (c4);

Then query the sys tables:
select i.name as index_name
     , ic.key_ordinal
     , ic.column_id
     , c.name as column_name
  from sys.indexes i
  join sys.index_columns ic on i.object_id = ic.object_id and i.index_id = ic.index_id
  join sys.columns c ON ic.object_id = c.object_id and ic.column_id = c.column_id
  join sys.tables t ON i.object_id = t.object_id
 where t.name = 't1'
;

Expected result:

index_name
key_ordinal
column_id
column_name

t1_pk
2
1
c2

t1_i1
3
1
c3

t1_i1
1
2
c1

t1_i2
4
1
c4

Writing this made me realize that index_id in sys.index_columns is incorrect for columns with key_ordinal > 1.
select name
         , index_id
  from sys.indexes
where name like 't1%';

name
index_id

t1_pkt1ad527cedc91b740e0f35d5161fdc69e3
2

t1_i1t1fc3ad6f1506b2a13371857879dfdbad8
3

t1_i2t104a836e77ad57814e46a4ebb6606df0e
4

=> ok

select ic.index_id
     , ic.key_ordinal
     , ic.column_id
     , c.name as column_name
  from sys.index_columns ic
  join sys.columns c ON ic.object_id = c.object_id and ic.column_id = c.column_id
  join sys.tables t ON c.object_id = t.object_id
 where t.name = 't1'
;

index_id
key_ordinal
column_id
column_name

5
2
2
c1

3
1
3
c2

4
1
1
c3

2
1
4
c4

=> ko: there no index_id 5

I had not this issue with babelfish 2.3.0 (with aws aurora pg 14.6)
I have it now with babelfish 3.2.0 (with aws aurora pg 15.3
I'll look in to it...

Hi
I pushed an extra commit in my branch to fix index_id.
Now I get the expexted result from the query above.


publishing today's changes to docs.docker.com

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit e1034b60ed67a72cfcca3642cdfc5774a0e753ae
https://deploy-preview-9523--docsdocker.netlify.com


Issue with inferring types from pipes.

Using code that is almost verbatim lifted from docs:
const forceNumber = z.union([z.string(), z.number()]).pipe(z.coerce.number())

Idea being to accept a number or string, and coerse to string or fail. The function works, unit tests are all fine, but type inference falls over like so:
const forceNumber = z.union([z.string(), z.number()]).pipe(z.coerce.number())

const dataSchema = z.object({
  numberOrString: forceNumber,
})

type Data = z.infer<typeof dataSchema>

// parse any data into schema or fail
const parseSchema = <U>(data: any, schema: z.ZodType<U>): U => {
  return schema.parse(data)
}

// log Data
const logSchemaData = (data: Data) => {
  console.log(data)
}

function processDataAndLog(data: any) {
  const parsedData = parseSchema(data, dataSchema)
  logSchemaData(parsedData)
}

this gives us the following error:

I'd recommend checking out this guide for some pointers on writing generic functions that accept Zod schemas: https://github.com/colinhacks/zod?tab=readme-ov-file#writing-generic-functions

Thanks @colinhacks, although having to infer the return type of a generic parsing function feels a bit icky. Wonder if there is a way to avoid that

As it means that we're actually short circuiting the return type when we return the value of schema.parse, which could lead to human error that is not caught by TS

I have found that
const parseSchema = <Input, Output>(
  data: any,
  schema: z.ZodType<Output, ZodTypeDef, Input>,
): Output => {
  return schema.parse(data)
}

Fixes this without having to resort into type casting.

This is the recommended way to write that function:
// parse any data into schema or fail
const parseSchema = <U extends z.ZodType>(data: any, schema: U): z.infer<U> => {
  return schema.parse(data);
};

This is the recommended way to write that function:
// parse any data into schema or fail
const parseSchema = <U extends z.ZodType>(data: any, schema: U): z.infer<U> => {
  return schema.parse(data);
};

Can I ask why is that recommended over what I've done? Just for my understanding


Commit link for stack incorrectly keeps .git as extension

Fix an invalid commit link in the Stack details screen that can occur if the original copy/pasted Git URL of the stack is a GitHub repo URL with an extension of .git.

Also PR:
https://github.com/portainer/portainer/pull/10778


Nxp - The number of jobs with the Bucharest location in solr.peviitor.ro(117) is not the same as the number of jobs on the company's website (120)

ENVIRONMENT: production
URL: https://solr.peviitor.ro
Browser: Microsoft Edge
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open the URL in the browser
Execute query for Nxp company with Bucharest city
Check the number of jobs
Open https://nxp.wd3.myworkdayjobs.com/en-US/careers?locations
Check the number of jobs with Bucharest as the location

EXPECTED RESULTS:
The number of jobs is the same
ACTUAL RESULTS:
The number of jobs with the Bucharest location in solr.peviitor.ro(117) is not the same as the number of jobs on the company's website (120)

UPDATE 11.10.2023
ACTUAL RESULTS:
The number of jobs with the Bucharest location on solr.peviitor.ro(118) is not the same as the number of jobs on the company's website (121)

Fixed!

The number of jobs in solr.peviitor.ro is  the same as the number of jobs on the company's website


Properties as attributes on Project element

Hey friends,
With all the great strides that are happening with the *proj format, wouldn't it be neat if you could set property values as attributes on the root project element?
f.e, before:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp1.0</TargetFramework>
  </PropertyGroup>
</Project>

after
<Project Sdk="Microsoft.NET.Sdk" OutputType="Exe" TargetFramework="netcoreapp1.0"/>
This would only be a short hand and most people would set most of their properties the old way, but it would get some of those properties you always set out of the way.
In my mind this would be for properties only, not Items, so the names would not be ambiguous in that sense. They'd then be pre processed into regular property groups (if I remember correctly there is a preprosessor step that transforms the sdk property already correct?)
What do you think?

ping?
Obviously this would be a post vs2017 change (unless its super easy) but I think it would be very useful.
I'd love to take a stab at it myself but i'm not quite sure where to start, so if you'd have any pointers i'd be very grateful :)

So now that 2017 has shipped, will you consider this for the 2.0 timeframe? I think it would be very handy

IMHO, This is too much. Most of the time, especially for real world projects, you would most certainly have more than 10 lines of properties/items definitions.
Is there any scenario where you think, this is useful?


Replace PropTypes from React with form prop-types package

See https://facebook.github.io/react/blog/2017/04/07/react-v15.5.0.html#migrating-from-react.proptypes
Also seems to be breaking on<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Thanks for the contribution @sabbyt54 - this all looks good to me.


cmd/vet: add unkeyed field literal check to 'go test' set

#2794 proposes requiring keyed (tagged) literals for any imported struct.
go vet has a test for this already.
But the test is only on in go vet not the automatic vet during go test.
I suggest we enable it during go test at the start of the Go 1.14 cycle
as an experiment. If it works well there then maybe it would make sense
to promote the restriction to the language itself. And if not, then not.
Or maybe vet alone will be enough. Time will tell.
/cc @robpike @ianlancetaylor @griesemer

Looks like this can be done in Go 1.15 cycle.


fix for issue#48: created post_save receiver after profile is created

Issue: 48
Changes proposed in this pull request and/or Screenshots of changes:

created post_save receiver after profile is created

LGTM . Thank you


Crash when multiple lines are entered in "Repo stars"

Crash when multiple lines are entered in "Repo stars"
Log:
22634-22634/com.github.pavlospt.refactoredumbrella E/actoredumbrell: Unknown bits set in runtime_flags: 0x8000
2020-06-02 13:38:11.251 22634-22634/? E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.github.pavlospt.refactoredumbrella, PID: 22634
java.lang.NumberFormatException: For input string: "4
4"
at java.lang.Integer.parseInt(Integer.java:615)
Solution:
make EditText as single line
and
or make EditText input as number only

@PraveenTalari Hello! This is a pet project. Its usage is not fine grained and it will not cater for every edge case scenario 😄


Refactor address and asset types

@matthiasgeihs This PR is read for review. However, I have not rebased it on the latest development branch. Because, this PR modifies the pkg/io package, and this package has been in moved out in the current dev branch.
As we had discussed, we could move the pkg/io package back into the go-perun repository, because it is specific to perun. After that, we could rebase this branch on to develop.

Close and reopen as an attempt to restart the tests.

@matthiasgeihs Yes, I will have to rebase. But, I think we should move back the pkg/io into go-perun before rebasing.
I will submit a PR for that.

@matthiasgeihs I have opened #271 for moving back pkg/io into go-perun.
We could rebase this PR after that one is merged.

Partially closes #257 (tasks 1,2 of 4)


multi-gpu ddp calls validation loop too many times

When using ddp with multiple gpus, each validation and test loop is called with the entire validation dataset for each gpu.
Expected behavior is that the dataset is divided appropriately across the gpus.
I am using current master (cloned Mar 14), Ubuntu 19.10, Cuda 10.1, python 3.7.5, pytorch 1.4, venv environment.
The problem appears to be in auto_add_sampler() in data_loading.py. It does not create a DistributedSampler for validation or test datasets.

I'll make a pr if the above seems right. it is working for me now, but need to run full testing.

Latest pull - 1 hour ago, no longer this behavior. Closing.

Sorry - this issue still exists in some configurations. My proposed fix is not the total picture. Still investigating - will provide reproducible example.

Testing underway. Will make PR tomorrow.

Dont want to clutter up PR world if no one is interested in this. Let me know ...

that sounds a good contribution to me... mind send a PR?
Any suggestion @PyTorchLightning/core-contributors?
in a technical note when you refer some master state pls use coit hash as there can be multiple commits each day...

will do


[MODEL EVALUATION REQUEST] ibm-granite/granite-8b-code-base

Model ID
ibm-granite/granite-8b-code-base
Model type
Decoder model (e.g., GPT)
Model languages

[X] Danish
[X] Swedish
[X] Norwegian (Bokmål or Nynorsk)
[X] Icelandic
[X] Faroese
[X] German
[X] Dutch
[X] English

Merged model
Not a merged model

Benchmarked on Danish.

Benchmarked on English now too!


  word wrap  Issue

Bug reports:

Specifications:

html2canvas version tested with:  1.0.0
Browser & version:  Chrome  91.0.4472.114

请问有什么解决方法吗

解决了吗

请问是怎么解决的

Any solution to this problem?

"Fixed" the problem with this pull request: https://github.com/niklasvh/html2canvas/pull/3092
(May possibly introduce side effects)
Install the fixed html2canvas with: npm i @deepnotes/html2canvas
Fork: https://github.com/DeepNotesApp/html2canvas


[BEAM-2371] Add WriteFiles translation

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

The Java DirectRunner needs WriteFiles to be well-defined with a payload in order to detect (and provide) runner-specified sharding.

Coverage increased (+0.02%) to 70.683% when pulling 1841803e45b875f5cb5379305b4c6498fca256c4 on kennknowles:WriteFiles-translation into 2df9dbd24f6fd3bcfb14316859f4b38c38d95eca on apache:master.


No error message when hash is not specified

Bug reproduces when running replicate prepare/start/stop command without specifying either blue or green hash: deepify replicate [prepare/start/stop] --blue --green --resources resource_name
Cannot read property 'split' of null error appears. Please display error message that blue/green hash(or both) is required.
Note: bug also reproduces when --resources value is not specified.

Bug is still reproducing with the same behavior as in task description

Bug is reproducing with the same behavior as in task description.
Please refer to attachment for more details:


Change MASP spec title and file name

Title it and change the filename to "multi-asset shielded pool" in some form.

Done; new spec link is https://github.com/metastatedev/masp/blob/main/docs/multi-asset-shielded-pool.pdf


Custom command passing a string literal

Describe the bug
Running an ad hoc docker run command in the terminal works. Passing the same command to lazydocker as either a scalar (folded/literal) or inline fails.
To Reproduce
Steps to reproduce the behavior:

Create a custom command in config.yml- name: run-meetup
  attach: true
  command: >
    docker run -it --name meetup_bot
    -p 80:3000
    -e PORT="3000"
    -v "$(pwd)":/home/appuser/app
    --env-file .env
    meetup_bot
    bash

or- name: run-meetup
  attach: true
  command: 'docker run -it --name meetup_bot -p 80:3000 -e PORT="3000" -v "$(pwd)":/home/appuser/app --env-file .env meetup_bot bash'

Open lazydocker
Run the command
Raises error:+ docker run -it --name meetup_bot -p 80:3000 -e PORT=3000 -v $(pwd):/home/appuser/app --env-file .env meetup_bot bash

docker: Error response from daemon: create $(pwd): "$(pwd)" includes invalid characters for a local volume name, only "[a-zA-Z0-9][a-zA-Z0-9_.-]" are allowed. If you intended to pass a host directory, use absolute path.
See 'docker run --help'.

Expected behaviour
It should interpolate/expand the pwd command or $PWD built-in variable in the command. Only able to hard-code the absolute path to work around the issue. Less than ideal, of course.
Screenshots
N/A
Desktop (please complete the following information):

OS: macOS 12.4 (21F79)
Lazydocker Version: v0.18.1
The last commit id if you built project from sources: N/A

Additional context
Using the new docker mount syntax got the same results.

$(pwd) have to run in the shell, but lazydocker seems like run command directly.
I think this is not bug, but run commands through shell will be more convenient I think.

What would you suggest the workflow should be if this isn't a bug @skanehira?

I agree, we ought to run all custom commands in a shell so that things like $(pwd) can be evaluated. We do that currently with lazygit

The place we could update is here. In lazygit we have a NewShell function which takes a command and wraps it in a shell invocation (see here) so we can copy that approach in lazydocker

I'll tackle this one.

go ahead @gusandrioli :)

Appreciate the follow-up! 🙏

@pythoninthegrass PR was merged :)


CTHUB - GER: remove incorrect warnings

Describe the task
Remove incorrect Phone Number and Email warnings in GER dataset.
When Phone Number and Email have empty values, CTHub produces the following error:
    Warning Name: Possible Errors in Email Addresses
    Expected value: Verify email addresses are valid

    Warning Name: Phone Number Appears Incorrect
    Expected value: Ensure phone numbers match the Canadian format (XXX-XXX-XXXX)

These warnings should only apply to values that are not blank. In this case no warning should have come up for blank cells in Phone Number and Email fields since these fields are allowed to have blank values. This issue may arise from the order in which the checks are being done.
This issue can be replicated using the test_file_missing_allowed file  on Teams ( GER_SUVI > test_files).
Purpose
Ensure that incorrect warnings do not come up for users in Phone Number and Email columns.
Acceptance Criteria

[ ] No Possible Errors in Email Addresses warning comes up when Email is blank
[ ] No Phone Number Appears Incorrect warning comes up when Phone Number is blank

[x]  No Possible Errors in Email Addresses warning comes up when Email is blank
[x]  No Phone Number Appears Incorrect warning comes up when Phone Number is blank

It all works as intended! 👍
I also noticed a UX issue. Warnings and errors remain from the first attempt. I will follow up on that with a ticket.


USDM - JSON Validation doesn't work with Parquet

Note that the newly created USDMDataService.to_parquet function will not work as currently written - it also seems to assume that there will be a single USDM JSON file for each dataset, which is not the case.

This issue was first noted as a comment in #840.  See step 2 of the suggested changes for #840 for a fix for a comparable problem where get_dataset also doesn't work for USDM because it assumes a single dataset per file.


NODE_SIZE: 5 -> 3

Codecov Report
Patch coverage has no change and project coverage change: -0.05 :warning:

Comparison is base (255cc6f) 46.33% compared to head (f7fc496) 46.28%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop      #99      +/-   ##
=============================================
- Coverage      46.33%   46.28%   -0.05%
+ Complexity       996      994       -2
=============================================
  Files            164      164
  Lines           6732     6734       +2
  Branches         678      678
=============================================
- Hits            3119     3117       -2
- Misses          3378     3382       +4
  Partials         235      235

Impacted Files
Coverage Δ

...ch/ydb/core/impl/pool/EndpointPriorityFactory.java
82.14% <ø> (ø)

... and 5 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


0.12.1

Signed-off-by: Stephen Curran<EMAIL_ADDRESS>

@jamshale -- could you please look at the integration test error? Presumably intermittent, but not one I recognize.

The integration test got a duplicate record error setting up endorser roles. I've never seen this before. I'd like to have a look at this closer and figure out if it's a blocking issue or not.

I'm going to look into that error from the integration tests and make a ticket. I'm sure it's intermittent and already existed, but still, it shouldn't be able to happen.
Doesn't affect the release.

FYI on the failed integration test. Seems like, somehow the test created a race condition, by duplicating a /set-endorser-role request, with these lines.
7hAcme.agent | Updating endorser role for connection: d6283a4d-aa71-49b7-9e80-aa58f69cd8f7, TRANSACTION_ENDORSER
7hAcme.agent | Updating endorser role for connection: d6283a4d-aa71-49b7-9e80-aa58f69cd8f7, TRANSACTION_ENDORSER

creating a duplicate record. I tried to recreate the problem but haven't been able to. Because I've never seen this error before and seems very unlikely to happen on this endpoint in practice, I'd say we don't have to worry about it.


[CI] Update GitHub Action to push docker images

Background
Updates the GitHub Actions CI to push docker images due to the deprecation of set-output commands in Github Action (source: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/).
The functionality's still the same.
Suggested Design
Update GitHub action logic
Rename it to push-docker-images.yaml
Acceptance Criteria

all existing and new tests should pass

great idea


Hub to Hub Association

[Enter feedback here]
Is this available for the GCC Tenant?

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

ID: c98d42c3-a275-25a1-efbf-03a383c511c1
Version Independent ID: 7de4ec5f-4a1d-651f-1ace-6d3f62cbf89a
Content: Associate a hub site to another hub site in the SharePoint admin center - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/hub-to-hub-association.md
Service: sharepoint-online
GitHub Login: @MattEEvans
Microsoft Alias: matteva

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Thanks, Manuela! Great answer, and thanks for handling this!
From: Manuela Scanu @.>
Sent: Thursday, March 10, 2022 8:38 PM
To: MicrosoftDocs/OfficeDocs-SharePoint @.>
Cc: Matt Evans (CONTENT) @.>; Mention @.>
Subject: Re: [MicrosoftDocs/OfficeDocs-SharePoint] Hub to Hub Association (Issue #3210)
Closed #3210https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SharePoint%2Fissues%2F3210&data=04|01|Matt.Evans.OFFICE%40microsoft.com|e7aa2d40d1454a1f42de08da0318daf7|72f988bf86f141af91ab2d7cd011db47|0|0|637825702546167325|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=OkNxJra7I9uhgstwvsKgQhVh76mQwq3o%2FGVgkiz4bEY%3D&reserved=0.

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SharePoint%2Fissues%2F3210%23event-6222339988&data=04|01|Matt.Evans.OFFICE%40microsoft.com|e7aa2d40d1454a1f42de08da0318daf7|72f988bf86f141af91ab2d7cd011db47|0|0|637825702546217328|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=u6nngtu8pL%2BLgEIvrEpLgbbqwzsmSPKc8XAZ7qObeVQ%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQZH3D5ASOHLIAGZ6XUZUHDU7LEYVANCNFSM5QNNGXGA&data=04|01|Matt.Evans.OFFICE%40microsoft.com|e7aa2d40d1454a1f42de08da0318daf7|72f988bf86f141af91ab2d7cd011db47|0|0|637825702546217328|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=7vzC%2BeSS80GEXEFUuLc65oBSHtDPnV0twofQrLVY1c8%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|Matt.Evans.OFFICE%40microsoft.com|e7aa2d40d1454a1f42de08da0318daf7|72f988bf86f141af91ab2d7cd011db47|0|0|637825702546217328|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=UaNJN6Pih8m%2Fim1eHgjCVqhjBuASu0mv%2F1qlTIMcI6U%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|Matt.Evans.OFFICE%40microsoft.com|e7aa2d40d1454a1f42de08da0318daf7|72f988bf86f141af91ab2d7cd011db47|0|0|637825702546217328|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=ah1LzdXwwgNzoG3jpeiXmG1PimURmBthQKSfI6U6apk%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Freeze column footer

Hi guys!
I have probelm with frozen for 3 columns, for example, in my footer table, on link above sugested change my css, unfortily doen's work...
Freeze column footer #1541
https://github.com/jspreadsheet/ce/issues/1541
Another idea?
Thx

Hi,
How you want freeze, horizontale or verticale direction ?
If it's when in verticale move of all footer , you can use this CSS fix :
.jexcel > tfoot > tr > td {
  position: sticky;
  left: 0px;
  z-index: 3 !important;
}

on horizontale move :
.jexcel > tfoot > tr > td {
  position: sticky;
  bottom: 0px;
  z-index: 3 !important;
}

After you can select number of colums with css property :nth-child like this
.jexcel > tfoot > tr > td:nth-child(1), .jexcel > tfoot > tr > td:nth-child(2) {
  position: sticky;
  left: 0px;
  z-index: 3 !important;
}

otherwise, better, you can add with JS, class jexcel_freezed on column in footer you want freeze (Example with nesterHeader : #1539 )


Staging site docs are indexed by Google

I noticed when using Google to search the docs that some results were coming from the staging site - we shouldn't let search engines index this for duplicate content reasons, but it could also be confusing to the reader if the docs on staging are different from those on prod and they find them in search results.

This also applies to the marketing site: https://github.com/balena-io/resin-site/issues/877

I've had the same issue, I have typed manual BalenaOS releases and got this page https://www.balena-staging.io/os/docs/raspberrypi3/getting-started/

@chrisys @georgiats it was discussed with @alexandrosm that we should let staging results be there. Not able to find the Thread for it now :/

Normal practice is to use canonical links in these cases : https://en.wikipedia.org/wiki/Canonical_link_element

@iamsolankiamit Any definitive reason as to why that was the case so the issue can be closed?


things to check before prod

[x] air is not diggable
[x] grass_block and dirt have effective tools
[x] dont add repairWith if it's empty

eventually would be nice to add cake food/sat to foods.json

[ ] 1.15's lootContextBuilder should add .put(LootContextParameters.BLOCK_STATE, blockState) ...etc

it's filterLight NOT filtersLight

[ ] so many versions have maxDurability wrong, it should be item#getMaxDamage

[ ] rename ClientSideAnnoyances to be lowercase

[ ] most versions dont need MiningToolAccessor, there is a item.getEffectiveBlocks()...

[ ] add attributes.json to all versions

[ ] enchantment.isDifferent in EnchantmentsDataGenerator should have a not before it, ie:

            if (!enchantment.isDifferent(other) && other != enchantment) {

[ ] make sure getMaximumPower is properly used on all enchants, since it is overridden

-        int b = getMaximumPower(enchantment,0);
-        int a = getMaximumPower(enchantment,1) - b;
+        int b = enchantment.getMaximumPower(0);
+        int a = enchantment.getMaximumPower(1) - b;

[ ] remove SkyColors from 1.11.2

[ ] remove stackSize from blocks.json in all versions

[ ] remove this comment

        // FIXME: ENTITY ID IS WRONG

[ ] vanishable should not be in the enchantCategories for all items in items.json...

[ ] make sure BlockDataDumper's emitLight is correct for all versions

[ ] replace DGU.getCurrentlyRunningServer().stopServer(); with System.exit(0) everywhere

@AlexProgrammerDE any thoughts on those ?

I think this is basically done


Need a single command to create a local cert chain, optionally for mTLS

This should include:

A single root CA and key
A single server cert and key
Optionally a client cert and key if the "--mtls" option is passed to the command

Current state is running these commands... not ideal.
# Generating CA cert
certcli root-ca ca.localhost
# Generating server CSR
certcli csr --san localhost --ip <IP_ADDRESS> localserver
# Signing server cert with local CA
certcli sign localserver.csr ca.localhost.cer ca.localhost.key
# Formatting information for Go use
certcli format --from-der --to-pem cert localserver.cer localserver.pem
certcli format --from-der --to-pem key localserver.key localserver.key.pem
certcli format --from-der --to-pem cert ca.localhost.cer ca.localhost.pem
echo Done!


⚠ Mobitopup has degraded performance

In 9b655e9, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 5355 ms

Resolved: Mobitopup performance has improved in 2828ca4.


Fix build on MinGW 64 by providing localtime() instead of localtime_r() for MinGW

Signed-off-by: Kirill Leyfer<EMAIL_ADDRESS>

Thanks, I solved this by providing a wrapper for localtime_r() in a header file.


Updated symfony/process constraint to allow 5.0

Missed this one in my previous PR

How about including symfony/event-dispatcher as well?

@biggianteye good idea, done!

To make symfony/event-dispatcher work, I had to do some more fixes, which breaks BC with <5.0. So I removed those version numbers from composer.json now.

Any plans to merge this?

Ho @peterjaap , any update on this ?

@kezek I'm not working with this lib anymore, feel free to contribute!


How can we benefit from repo?

This is what powers Flutter's notable commits https://github.com/flutter/flutter/issues/121415. Most folks should subscribe to that issue instead of this repo.
You can use this repo to do more advanced digging:

You can improve the scoring heuristic to better detect notable commits: https://github.com/loic-sharma/flutter-changelog/blob/a1a97121cbaee74ccfda92c3c8a4cb6088d7c88f/main.dart#L93
There's a list of pull requests that are missing reviewers: https://github.com/loic-sharma/flutter-changelog/blob/main/pulls.md

really  I don't understand 😕
please make videos for outputs or screenshot
not all people will understand


[fix](jdbc catalog) fix jdbc mysql client match jsonb type

When using mysql catalog to connect to Doris, some lower versions of Doris's Json type will be displayed as JsonB, and we need to be compatible with it

run buildall

run buildall


[BUG] Unable to connect to postgresql with azure ad

Unable to connect to the Azure PostgreSQL Single server database with Managed Identity in a Spring Boot simple setup.
Exception
2022-10-31 22:28:07.939  INFO [,,] 25480 --- [  restartedMain] org.hibernate.dialect.Dialect            : HHH000400: Using dialect: org.hibernate.dialect.PostgreSQLDialect
2022-10-31 22:28:08.541  INFO [,,] 25480 --- [  restartedMain] com.zaxxer.hikari.HikariDataSource       : HikariPool-1 - Starting...
Disconnected from the target VM, address: '<IP_ADDRESS>:53157', transport: 'socket'
2022-10-31 22:28:09.832 ERROR [,,] 25480 --- [  restartedMain] com.zaxxer.hikari.pool.HikariPool        : HikariPool-1 - Exception during pool initialization.

org.postgresql.util.PSQLException: The server requested password-based authentication, but no password was provided by plugin null
	at org.postgresql.core.v3.AuthenticationPluginManager.lambda$withEncodedPassword$0(AuthenticationPluginManager.java:110) ~[postgresql-42.3.6.jar:42.3.6]
	at org.postgresql.core.v3.AuthenticationPluginManager.withPassword(AuthenticationPluginManager.java:81) ~[postgresql-42.3.6.jar:42.3.6]
	at org.postgresql.core.v3.AuthenticationPluginManager.withEncodedPassword(AuthenticationPluginManager.java:107) ~[postgresql-42.3.6.jar:42.3.6]
	at org.postgresql.core.v3.ConnectionFactoryImpl.doAuthentication(ConnectionFactoryImpl.java:691) ~[postgresql-42.3.6.jar:42.3.6]
	at org.postgresql.core.v3.ConnectionFactoryImpl.tryConnect(ConnectionFactoryImpl.java:180) ~[postgresql-42.3.6.jar:42.3.6]
	at org.postgresql.core.v3.ConnectionFactoryImpl.openConnectionImpl(ConnectionFactoryImpl.java:235) ~[postgresql-42.3.6.jar:42.3.6]
	at org.postgresql.core.ConnectionFactory.openConnection(ConnectionFactory.java:49) ~[postgresql-42.3.6.jar:42.3.6]
	at org.postgresql.jdbc.PgConnection.<init>(PgConnection.java:223) ~[postgresql-42.3.6.jar:42.3.6]
	at org.postgresql.Driver.makeConnection(Driver.java:402) ~[postgresql-42.3.6.jar:42.3.6]
	at org.postgresql.Driver.connect(Driver.java:261) ~[postgresql-42.3.6.jar:42.3.6]
	at com.zaxxer.hikari.util.DriverDataSource.getConnection(DriverDataSource.java:138) ~[HikariCP-4.0.3.jar:na]
	at com.zaxxer.hikari.pool.PoolBase.newConnection(PoolBase.java:364) ~[HikariCP-4.0.3.jar:na]
	at com.zaxxer.hikari.pool.PoolBase.newPoolEntry(PoolBase.java:206) ~[HikariCP-4.0.3.jar:na]
	at com.zaxxer.hikari.pool.HikariPool.createPoolEntry(HikariPool.java:476) ~[HikariCP-4.0.3.jar:na]

To Reproduce
Simple Spring Boot app with using the library
        <dependency>
            <groupId>com.azure.spring</groupId>
            <artifactId>spring-cloud-azure-starter-jdbc-postgresql</artifactId>
            <version>4.5.0-beta.1</version>
        </dependency>

Followed the guide as defined here
https://learn.microsoft.com/en-us/azure/developer/java/spring-framework/configure-spring-data-jpa-with-azure-postgresql?toc=%2Fazure%2Fpostgresql%2Ftoc.json&bc=%2Fazure%2Fbread%2Ftoc.json&tabs=passwordless
also set the following properties
  datasource:
    url: jdbc:postgresql://<myDB>.postgres.database.azure.com:5432/postgres?sslmode=require
    username: <myRole>@<myDB>
    azure:
      passwordless-enabled: true

Code Snippet
After the properties, the application fails at the start while trying to create connection pool.
Expected behavior
The connection pool should have been setup without any errors.
Setup (please complete the following information):

OS: Windows
IDE: IntelliJ
Library/Libraries: 4.5.0-beta.1
Java version: 17
App Server/Environment: Tomcat
Frameworks: Spring Boot

Additional context
The example in the link considers System Managed Identity, but User Managed Identity is not explained on how to use or setup. I have defined the default identity using the following properties but not sure if this is the right way.
spring:
  cloud:
    azure:
      credential:
        managed-identity-enabled: true
        client-id: *clientID
        client-secret: *clientSecret
      profile:
        tenant-id: *tenantID
        cloud-type: azure

Hi @vtapadia, thanks for reaching out, I will look into this issue.

Hi @vtapadia
Could you try configurations like below and also check this documentation.
:

spring:
  datasource:
    url: jdbc:postgresql://<myDB>.postgres.database.azure.com:5432/db
    username: <myRole>@<myDB>
    azure:
      passwordlessEnabled: true
      credential:
        managed-identity-enabled: true

I tried using managed  identity within Azure Spring Apps, and it worked.

Hi @backwind1233
Thanks for the quick reply.

I am still not able to see that the fields are identified, it is not able to detect even client-id when I give it actual value and not reference. So it seems I have to wait for the next drop to see that those values are detected in the Intellij.
I am testing it locally only. I guess this is the issue maybe, because we do not use a service to deploy this. We host it ourselves (Cloud Foundry) and use the ClientSecretCredential to use the User Managed Identity to connect to other services. We use the same Client Secret to connect to Key Vault and other services and they work jsut fine.
Yes, and that is what I actually wanted to use, because we use User Managed Identity.

I thought that if it supports AD, then however we get the accesstoken, it should be able to use that to connect. But maybe I am expecting something that is still not supported. Can you confirm?

@backwind1233 with regard to this

but the other two managed-identity-enabled and cloud-type should able be detected, it works in my local IDE.

could you check the generated metadata from released artifact?

Hi @stliu
I checked the generated metadata and confirmed it contains the metadata.

I downloaded the jar file spring-cloud-azure-autoconfigure-4.5.0-beta.1.jar from maven central  spring-cloud-azure-autoconfigure/4.5.0-beta.1, unzip the jar and checked the metadata.

I also checked the PR  and file changes.

Hi,
Thanks and I will check this out and see if I can find the issue. In the meantime, we also deployed a azre spring cloud instance to see if this works there, but running into java version issue. For that I will have to raise another issue at the portal side..  :(

@vtapadia Hi any updates on this issue?


CC0039  False Positive? concatenating to loop variable propery/field

This should not raise a diagnostic:
	class MyObject
	{
		public string MyObjectString;
	}
	class MyClass
	{
		private readonly List<MyObject> _items = new List<MyObject>
		{
				new MyObject {MyObjectString = "ItemA"},
				new MyObject {MyObjectString = "ItemB"},
				new MyObject {MyObjectString = "ItemC"}
		};
		private void MyMethod(string suffix)
		{
			foreach (MyObject o in _items)
			{
				o.MyObjectString += suffix; // CC0039 fires on this line
				o.MyObjectString = o.MyObjectString + suffix; // but not on this one
			}
		}
	}

The code fix makes no sense at all in this context.
The same issue occurs in a VB equivalent situation
The analyzer should check if the symbol being used is in context only within the foreach. Otherwise, it makes no sense.
The PR should go into the v1.0.x branch.

@RachelDavids Bug confirmed. I have added some more info to the description.

Closed on #826.


Missing metadata field in ProvisionedServiceSpec

Hi,
According to OSB API, we should be allowed to return the metadata field in a provisioning response.
Do you plan to update ProvisionedServiceSpec struct in the near future?
If no, then we can make a contribution, but to be honest, we really need this fast so my second question will be: there is a chance all processes from your side about the review and merge will be quick?

Hi @jasiu001
This is a property introduced in OSBAPI 2.16 and we haven't started working on that yet.
WE are not doing much active development here at the movement but we would be happy to review your PR and merge it asap 🙂
As a side note, it would be interesting to know what is your use case for returning metadata on provision and what platform would be using that?
Thanks,
Felisia

Hi @FelisiaM
That is great, I made PR with implementation.
We use your library in our open-source Kyma project, to be more specific in the system which installs clusters with Kyma.
External services trigger provisioning using OSB API and your library is really helpful to handle this :)

Thank you for your contribution @jasiu001
I have now merged the PR and released version 7.5.0 with the change


500 errors from user profile page.

https://try.gitea.io/Leila/SudokuSolver
feedback from the general channel of gopher slack.
cc @tboerger

#1258

@ethantkoenig Good Job 👍 waiting Drone build success.


Cache app context mechanism

This PR introduces a cache mechanism for the application context. By default, the memory cache layer is enabled, although users have the option to disable this feature.
Changes Made

App Hooks: Added a new cache hook within the App Hooks. This hook initializes a new instance of the in-memory cache engine with default configuration. Consideration should be given to moving this configuration into a centralized location.
Cache Feature: The cache feature introduces a cache field under AppContext, allowing for the creation of specific cache implementations. The cache_inmem feature, enabled by default, includes both the cache feature and the moka library.

Discovery cach in memory libs:

https://crates.io/crates/moka
https://crates.io/crates/cached

Steps

[ ] Created a trait for a cache implementation to support different cache engines.
[ ] Memory caching is enabled by default (cache_inmem).
[ ] Consider adding more engines I.e Redis
[ ]  Added the cache feature, providing a cache field within AppContext and enabling the creation of specific cache implementations.
[ ]  Added the cache hook.
[ ] Tests.
[ ]  Consider refactoring the cache hook configuration into a centralized location for better organization.
[ ]  Enhance documentation for improved clarity and usability.
[ ] Think of strategy like ttl when insert value

looks good
i think if the storage isn't optional, cache shouldn't be optional (we can toggle the cache drivers based on feature, but leave all the abstraction to be always inside)

@jondot,
I have removed the feature flag and created a null driver for cache and storage like we spoke.
Redis implementation will be in a different PR


Make TestOneDevDownloadRepo work again & more resistant

It's a mistake that the OneDev tests still use the "normal" assert.EqualValues methods. I added a PR which provides multiple assert methods in https://github.com/go-gitea/gitea/blob/main/modules/migrations/main_test.go for the tests. All other providers use these methods, so OneDev should too. Then you don't have to worry about the time fields.


Identify some of dual licenses and skip built-in packages' licenses

This PR adds a pattern to identify dual-license and skip resolving built-in package's licenses

Just a minor polish, we won't highlight dependencies' license check in 0.1.0.
BTW, hopefully I can start the release process within this weekend.

Just a minor polish, we won't highlight dependencies' license check in 0.1.0.
BTW, hopefully I can start the release process within this weekend.


Implemented dll and changed exe output

Here is a dll version of the current existing code for exe.
I also changed the exe output to return 1 before the actual version in case it fail or give 0 for some reasons.
I also added the exit code to readme.

thank you LGTM


Exception documentation out of date in README

The exception section of the README seems to be out of date since 5cf103c4b6da368f19f5285a628797b8427a8033 where more meaningful errors were added.

This is fixed by #64. Thanks again, @hughbien!


Remove echo client

As echo workload can be tested using generic client, removing echo client
helps ease of maintenance

Signed-off-by: manju956<EMAIL_ADDRESS>

Looks okay.
Just one remnant mention you need to update from sdk/TestingContracts.md.

Updated sdk/TestingContracts.md.


Update mapVBVD.py

I love to see, that python gains more traction in the mri space.
This is just a really minimal change:
If tqdm gets imported from tqdm.auto, it uses the html display in jupyter notebooks.

Hi @p-sodmann thanks for submitting this request. I'll merge this now but I probably won't update the pypi/conda versions for a couple of days until your other issue is resolved.


Delete sms domain when having a v2 mention

Fixed Issues
$ GH_LINK https://github.com/Expensify/App/issues/40969
Tests

What unit/integration tests cover your change? What autoQA tests cover your change?
What tests did you perform that validates your changed worked?

QA

What does QA need to do to validate your changes?
What areas to they need to test for regressions?

@fnwbr 🙏

😘

damn I mentioned wrong person 😆, sorry for this blooper.

@flodnv can you give me review? it's really small 😄

😂 👍


electron-prompt does not work in electron V10

When i try opening electron prompt with even the simplest of parameters i'm getting strange error in console.
This is the sample code
    prompt({
      title: 'test',
      label: 'test',
      value: 'test',
      type: 'input'
    });

This is the error
(node:90465) UnhandledPromiseRejectionWarning: Error: Image could not be created from
    at /private/var/www/activecollab-desktop/node_modules/electron-prompt/lib/index.js:41:22
    at new Promise (<anonymous>)
    at electronPrompt (/private/var/www/activecollab-desktop/node_modules/electron-prompt/lib/index.js:12:9)
    at click (/private/var/www/activecollab-desktop/utils/menu.js:338:5)
    at MenuItem.click (electron/js2c/browser_init.js:1481:9)
    at Function.executeCommand (electron/js2c/browser_init.js:1742:13)
(node:90465) UnhandledPromiseRejectionWarning: Error: Image could not be created from
    at /private/var/www/activecollab-desktop/node_modules/electron-prompt/lib/index.js:41:22
    at new Promise (<anonymous>)
    at electronPrompt (/private/var/www/activecollab-desktop/node_modules/electron-prompt/lib/index.js:12:9)
    at click (/private/var/www/activecollab-desktop/utils/menu.js:338:5)
    at MenuItem.click (electron/js2c/browser_init.js:1481:9)
    at Function.executeCommand (electron/js2c/browser_init.js:1742:13)
(node:90465) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:90465) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)

Thanks for the heads up, it looks like electron treats the icon in BrowserOptions as set even when null, triggering that error

That was blazingly fast response and fix. Cheers!


Google readers integrations doc and drive updates

Description

Add missing README.md for all readers folder lost during the last migration from llamahub
Add query_string in Google Drive Reader
Some others minor fixes

Type of Change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] I stared at the code and made sure it makes sense

Suggested Checklist:

[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[x] I ran make format; make lint to appease the lint gods

looks amazing!

Thanks, I was not able to lint / build locally, let me check at the errors... :)

@hatianzhang it looks like the toml part failed, might be due to me adding:
[tool.llamahub.class_contributors]
GoogleDriveReader = ["metanov"]

should I remove that part ?

@hatianzhang it looks like the toml part failed, might be due to me adding:
[tool.llamahub.class_contributors]
GoogleDriveReader = ["metanov"]

should I remove that part ?

please do not remove that part, otherwise it won't show up in our llamahub.ai webpage.
Please instead do the following:
make lint
git add .
git commit -m "lint"
git push origin <your-branch-name>

thx @logan-markewich for the updates!


GitHub Copilot Reference Links Disrupt NVDA Screen Reader

Type: Bug
When there is a reference to GitHub in the Copilot explanation, it is difficult to understand the link to the reference, and it disrupts reading with NVDA.
See an example below.
Explanation
Context:
This code snippet appears to be part of a method for constructing or configuring an object, possibly using a design pattern like Builder.
Field tpHrio:
.tpHrio(...): It is configuring the field tpHrio of the object being constructed.
VS Code version: Code 1.94.2 (384ff7382de624fb94dbaf6da11977bba1ecd427, 2024-10-09T16:08:44.566Z)
OS version: Windows_NT x64 10.0.22631
Modes: Restricted

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8365U CPU @ 1.60GHz (8 x 1896)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
31.72GB (20.14GB free)

Process Argv
--crash-reporter-id 5391933d-2a30-4ca5-a5e4-b4571630ea33

Screen Reader
yes

VM
0%

Extensions (11)

Extension
Author (truncated)
Version

gitlens
eam
15.6.1

gc-excelviewer
Gra
4.2.62

azure-pipelines
ms-
1.247.2

python
ms-
2024.16.1

vscode-pylance
ms-
2024.10.1

remote-containers
ms-
0.389.0

remote-wsl
ms-
0.88.4

java
red
1.35.1

vscode-yaml
red
1.15.0

vscodeintellicode
Vis
1.3.1

vscode-maven
vsc
0.44.0

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
vscaat:30438848
c4g48928:30535728
azure-dev_surveyonecf:30548226
962ge761:30959799
pythongtdpath:30769146
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
accentitlementsc:30995553
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
724cj586:31013169
a69g1124:31058053
dvdeprecation:31068756
dwnewjupytercf:31046870
newcmakeconfigv2:31071590
impr_priority:31102340
nativerepl2:31139839
refactort:31108082
pythonrstrctxt:31112756
wkspc-onlycs-t:31132770
wkspc-ranged-t:31151552
cf971741:31144450
autoexpandse:31146404
iacca1:31156133
notype1:31157159
5fd0e150:31155592
iconenabled:31158251

@jeanbrazcosta a gif would be helpful to understand where your cursor is and what your screen reader is saying at that time.

@meganrogge Sorry, I don't think there was an example of how the reader is reading when there is a reference link in the answer:
"command:_github.copilot.openSymbolFromReferences?%5B%22%22%2C%5B%7B%22uri%22%3A%7B%22scheme%22%3A%"
Note: In all links the reference appears as I mentioned in quotation marks above and it has been very difficult to read the responses:
To reproduce, open a code with several classes and ask a question, where the copilot references it with a link to facilitate the line in which he is explaining, the reader will read these characters and there are several of them for each link that is created.


Research removal of InternalPool completely.

Per Garbage Collection observations made in #463 -- research the ramifications of removing InternalPool entirely. If the refactor can be performed safely and in reasonably good time, benchmark the performance of Celluloid without InternalPool and with it, and then retest #463 without it.
Of course, researching this truly and completely will result in an experimental prototype of Celluloid with no InternalPool ... if that branch outperforms the existing form and solves the Garbage Collection dilemmas we're experiencing polish that experimental version and remove InternalPool going forward.

See comments on #535 regarding .group_class ... about to push first run at this, working fine, all tests running stupid fast, but I need to retroactively heal InternalPool ( or not )

Closed in #545


[Featured Card group]:  Determine the name of the new DDS component

Overview
This component was designed and code submitted behind a feature flag by AEM.  They named it Promo Group, but this may not be the best component name within the Carbon for ibm.com repo.
Design spec from Jina:
https://ibm.box.com/s/yh1r97n9qr9d6rd5jhz79c7vxqx7uhan
Reference build from HC:
https://pages.github.ibm.com/MSC-Cloud/hc_fractal/components/detail/8b--cool-gray-10.html
Acceptance Criteria

[ ] Work the DDS team members to define the component name

@photodow @shixiedesign I noticed that the issues for this Featured Card group were not created.  I am creating them now.  James, the notes in the original issues says that you are aware of this work and started on it.  I have assigned it to this sprint, 21-04.

New card decorator: Card in card
Modify Cards group to take a featured Card in card

Add example page to storybook's Cards group story with Cards group — with card in card

Considerations for the future.

Add grid gutter types to Cards group.

collapsed (1px border)
condensed (2px gutter)
hanging (left hangs, right 16px)
default (32px no hang)

Add column count to Cards group. (e.g. 2, 3, 4 to start with)
Replace Card with Cards group in Content group — with cards.

Added Card in card
Added Card group example of card group with feature card to Box folder
*might need a little clean up...


Freeing pointer in CRYPTO_realloc if num == 0 degrades security (makes double-frees possible)

This is a common pattern, both within OpenSSL and in other software:
#include <openssl/crypto.h>
int main(void)
{
    void* q, *p = OPENSSL_malloc(10);
    if ( p == NULL ) {
        goto end;
    }
    q = OPENSSL_realloc(p, 20);
    if ( q == NULL ) {
        goto end;
    }
    p = q;
end:
    OPENSSL_free(p);
    return 0;
}

This works as long as the amount of bytes requested to be realloc'ed (here: 20) is positive. However, since this patch: https://github.com/openssl/openssl/commit/bbd86bf5424a611cb6b77a3a17fc522931c4dcb8#diff-0ad35796f3b4040e66893ac4cb4b6b28R172 CRYPTO_realloc will free the pointer if num == 0.
If you replace '20' with '0' in the example, a double-free will occur for current versions of OpenSSL (1.1.0/master) but not for older versions.
I don't believe that OpenSSL currently contains any code where it is possible to realloc zero bytes in any of its internal functions. However, other software sometimes also uses OpenSSL's allocation functions in parts of their code where heap memory is required. This new feature actually introduces a new attack vector since an attacker needs to coerce the application into reallocating zero bytes. This may happen through an array allocation in the form
OPENSSL_realloc(p, n * sizeof(element));

or through overflows.
With the new feature, this may lead to double-frees where the application will free p if OPENSSL_realloc fails (like in my example code) whereas it would operate fine with the old OPENSSL_realloc.
So the new feature is bound to degrade security some place, some time (especially as more and more projects will be linked against OpenSSL 1.1.0), and I think that it's better to leave it out.

I disagree.  I think it is better, and easier to understand, if OPENSSL_realloc semantics were exactly like realloc.

Ok


add UserManager.getUserCount for versions with signature permissions

should solve issue #58

Why close this?

I just need time to properly test and review changes like this and it has to be mirrored for AttestationServer. It can't be done only for the Auditor app.
This is changing the binary format of the attestation message so it will break compatibility unless it's incorporated into an increase of the protocol version which needs to be carefully done and tested.


Language is not reflected.

Confirmed version: mysql-681852b
The following bug was found.

:bug: Bug

Account list
Change Password

We have confirmed that the same problem occurs in all added languages.

Thanks for reporting. Will fix.

Fixed in latest update.


IPLD Explore page revamp

TODO:

[ ] start page with intro and examples - WIP https://github.com/ipfs-shipyard/ipfs-webui/pull/690
[x] replace CID buffers with stringified CID in object explorer
[x] truncate large file buffers in object explorer - trying to show the entire byte buffer of a large file in the UI isn't good.
[ ] extract node type info for links and pass around rather than guessing from CID shape.
[ ] loading states
[ ] error states

CIDs as strings and truncation of large buffers - https://github.com/ipfs-shipyard/ipfs-webui/pull/707

Done, and deployed as a standalone app https://explore.ipld.io and as part of https://webui.ipfs.io


Enable turning on echo for scripts

As per feedback: https://github.com/dotnet/roslyn-project-system/pull/1811.

tag @dotnet/project-system @tannergooding


Chore/reproduce package problem

When we have nested message definitions, grpcurl is unable to resolve some symbols.  I have found that by injecting a different package name it will resolve fine, so there may be an issue in how we calculate the package names of modules.
This PR demonstrates this in the example app.

regenerate the example protos ./generate_protos.sh
run the example app iex -S mix
some messages resolve❯ grpcurl -plaintext <IP_ADDRESS>:50051 describe example.helloworld.HelloSpec
example.helloworld.HelloSpec is a message:
message HelloSpec {
  string message = 1;
  .example.helloworld.HelloSpec.Spec spec = 2;
  message Spec {
    int32 major = 1;
    int32 minor = 2;
    int32 patch = 3;
    string notes = 4;
  }
}

nested message does not resolve (when you comment out the package overwrite code in server.ex)❯ grpcurl -plaintext <IP_ADDRESS>:50051 describe example.helloworld.HelloSpec.Spec
Failed to resolve symbol "example.helloworld.HelloSpec.Spec": Symbol not found: example.helloworld.HelloSpec.Spec

This may be related to https://github.com/elixir-grpc/grpc-reflection/issues/20

this PR is not needed


Remove vestigial webpack-dev-server config

We don't use webpack-dev-server anymore so this configuration is not used by anything.

!merge


Update the web demo app to use the latest config provider

Updated the app to use ASP.NET Core 3.0
Updated the app to use v3 of AppConfig provider

May I have a code review from anyone?


Invalid docker CMD for the latest master branch.

I am trying to build grobid docker image with the latest master branch.
The problem of dockerfile is the new config is located in grobid-home/config/grobid.yaml, not grobid-service/config/config.yaml.
CMD ["./grobid-service/bin/grobid-service", "server", "grobid-service/config/config.yaml"]

02:17:13.968 [main] INFO org.grobid.service.main.GrobidServiceApplication - Configuration file: /opt/grobid/grobid-service/config/config.yaml
java.io.FileNotFoundException: File grobid-service/config/config.yaml not found
        at io.dropwizard.configuration.FileConfigurationSourceProvider.open(FileConfigurationSourceProvider.java:18)
        at io.dropwizard.configuration.BaseConfigurationFactory.build(BaseConfigurationFactory.java:80)
        at io.dropwizard.cli.ConfiguredCommand.parseConfiguration(ConfiguredCommand.java:126)
        at io.dropwizard.cli.ConfiguredCommand.run(ConfiguredCommand.java:74)
        at io.dropwizard.cli.Cli.run(Cli.java:78)
        at io.dropwizard.Application.run(Application.java:93)
        at org.grobid.service.main.GrobidServiceApplication.main(GrobidServiceApplication.java:126)

PS: I think we shouldn't use hardcode arguments.

Oh yes sorry I've completely forgotten that in the last update. I never liked that command line at all, and yes we should not use hardcoded arguments, so I removed both the config.yaml and the ./grobid-service/bin/grobid-service.
In the docker image, it should be now simply:
./gradlew grobid-service:run

instead of the bad looking:
CMD ["./grobid-service/bin/grobid-service", "server", "grobid-service/config/config.yaml"]

Oh yes sorry I've completely forgotten that in the last update. I never liked that command line at all, and yes we should not use hardcoded arguments, so I removed both the config.yaml and the ./grobid-service/bin/grobid-service.
In the docker image, it should be now simply:
CMD [./gradlew grobid-service:run]

instead of the bad looking:
CMD ["./grobid-service/bin/grobid-service", "server", "grobid-service/config/config.yaml"]

Ready? Why we need gradle for a released layer?

I also found that libfontconfig is required for pdfalto
ERROR [2021-06-29 07:11:43,574] org.grobid.core.process.ProcessPdfToXml: pdfalto process finished with error code: 127. [/opt/grobid/grobid-home/pdfalto/lin-64/pdfalto_server, -fullFontName, -noLineNumbers, -noImage, -annot
ation, -filesLimit, 2000, /opt/grobid/grobid-home/tmp/origin9823192860162901769.pdf, /opt/grobid/grobid-home/tmp/gfmWEw2Bxc.lxml]
ERROR [2021-06-29 07:11:43,574] org.grobid.core.process.ProcessPdfToXml: pdfalto return message:
/opt/grobid/grobid-home/pdfalto/lin-64/pdfalto: error while loading shared libraries: libfontconfig.so.1: cannot open shared object file: No such file or directory

ERROR [2021-06-29 07:11:43,577] org.grobid.service.process.GrobidRestProcessFiles: An unexpected exception occurs.
! org.grobid.core.exceptions.GrobidException: [PDFALTO_CONVERSION_FAILURE] PDF to XML conversion failed. pdfalto cannot be executed correctly. Has libxml2 been installed in the system? More information can be found in the l
ogs.

Ready? Why we need gradle for a released layer?

To get rid of the GrobidServiceApplication.java ? So simplify a bit... But maybe a bad idea, do we have an overhead with gradle?

The dockerfile uses multi-stage builds to minimized image size, so the runtime stage does not have Gradle and source code.

I honestly don't think having the "hardcoded" path inside the docker image is a bad thing, because we build the image and the internal docker structure is actually settle by us.
This small bug was due to the fact that a) we don't have a docker CI build and b) the latest modification were a substantial re-design of the configuration files

I honestly don't think having the "hardcoded" path inside the docker image is a bad thing, because we build the image and the internal docker structure is actually settle by us.
This small bug was due to the fact that a) we don't have a docker CI build and b) the latest modification were a substantial re-design of the configuration files

Yes! we need docker CI build!

Yes you're right gradle is not in the runtime image!
We need to update GrobidServiceApplication.java to simplify it.

We need to update GrobidServiceApplication.java to simplify it.

mmm no too fast, GrobidServiceApplication.java  was updated and it is fine.


DB connection params documentation

Following the installation guide, the database connection cannot be established:
ERROR 2018-11-19T23:24:23Z: r2d2: database error: FATAL: password authentication failed for user "hecate"
It does work however with user postgres for the db writer role. For example, this will work:
cargo run -- --database_read "hecate_read:123456@localhost:5432/hecate" --database "postgres:123456@localhost:5432/hecate"
This doesn't work:
cargo run -- --database_read "hecate_read:123456@localhost:5432/hecate" --database "hecate:123456@localhost:5432/hecate"
(I changed passwords prior to running the example above in order to rule out empty password config issues).

@benstadin thanks a ton for opening an issue! Let's rule out the low lying fruit.

Did you run the schema file located here? https://github.com/mapbox/Hecate/blob/master/src/schema.sql

Once you create a new database it can be loaded with psql -U postgres -f src/schema.sql <DATABASE_NAME>

Can you confirm the hecate and hecate_read users were created successfully?

echo "\du" | psql -U postgres will output a list of users

  Role name  |                         Attributes                         | Member of
-------------+------------------------------------------------------------+-----------
 hecate      | No inheritance                                             | {}
 hecate_read | No inheritance                                             | {}
 postgres    | Superuser, Create role, Create DB, Replication, Bypass RLS | {}

Do all users have local access enabled? If you are running ubuntu I've had trouble with this one before when initially trying to get set up.

echo "show hba_file;" | psql -U postgres will print the path location
If you can take a look and ensure local connections are trusted that would be great!
I typically use the following on my dev machine

local all postgres trust
local all all trust
host all all <IP_ADDRESS>/32 trust
host all all ::1/128 trust
host replication postgres samenet trust

Let me know if any of these fix the problem, if so I'd love to update the docs with some troubleshooting help.

@benstadin thanks a ton for opening an issue! Let's rule out the low lying fruit.

Did you run the schema file located here? https://github.com/mapbox/Hecate/blob/master/src/schema.sql

Yes. I used the same DB name, commands and users as in the Build example.

Once you create a new database it can be loaded with psql -U postgres -f src/schema.sql <DATABASE_NAME>
Can you confirm the hecate and hecate_read users were created successfully?

Yes.

echo "\du" | psql -U postgres will output a list of users

  Role name  |                         Attributes                         | Member of
-------------+------------------------------------------------------------+-----------
 hecate      | No inheritance                                             | {}
 hecate_read | No inheritance                                             | {}
 postgres    | Superuser, Create role, Create DB, Replication, Bypass RLS | {}

  Role name  |                         Attributes                         | Member of
-------------+------------------------------------------------------------+-----------
 hecate      | No inheritance                                             | {}
 hecate_read | No inheritance                                             | {}
 postgres    | Superuser, Create role, Create DB, Replication, Bypass RLS | {}

Do all users have local access enabled? If you are running ubuntu I've had trouble with this one before when initially trying to get set up.

echo "show hba_file;" | psql -U postgres will print the path location
If you can take a look and ensure local connections are trusted that would be great!
I typically use the following on my dev machine

local all postgres trust
local all all trust
host all all <IP_ADDRESS>/32 trust
host all all ::1/128 trust
host replication postgres samenet trust

Let me know if any of these fix the problem, if so I'd love to update the docs with some troubleshooting help.

This is what it's showing for me:
# TYPE  DATABASE        USER            ADDRESS                 METHOD

# "local" is for Unix domain socket connections only
local   all             all                                     md5
# IPv4 local connections:
host    all             all             <IP_ADDRESS>/32            md5
# IPv6 local connections:
host    all             all             ::1/128                 md5
# Allow replication connections from localhost, by a user with the
# replication privilege.
local   replication     all                                     md5
host    replication     all             <IP_ADDRESS>/32            md5
host    replication     all             ::1/128                 md5

My environment:

OS X 10.13.6
postgres 11.1
rust nightly as per build instruction

Update
I updated the postgres config and it works now. Thanks!
The error message ("password authentication failed for user ...") is misleading. It's probably from the driver, though it should rather say the connection failed for that user.

@benstadin yeah this is an error passed via the driver. I have a ticket here: https://github.com/mapbox/Hecate/issues/90 to improve initial database connection errors.
Going to close this one out and add the hba fix to the setup docs. Thanks a ton for reporting this, look forward to more bug reports if you find any!

I'm following all the instructions for the installation but i keep getting this error
ERROR: Connection unable obtain postgres version using (READ/WRITE) hecate@localhost:5432/hecate
however i can connect to the database using the hecate user from pgadmin and everything looks fine from the command line.
any thoughts?

@mapsgeek What version of Postgres/Postgis are you using?

v14, postgis 3.1
should i downgrade?!


Add cloud-sdk-js-odata-batch-changeset

Thanks for the tutorial. Some issues:

The tutorial references "a previous tutorial" but doesn't mention the previous tutorial. Perhaps this tutorial will be added to an existing series? Either way, we should discuss your plans.
You can use second person ("do this", "you can now see ...") but not first person ("I will show you", "We will do this").
The question of what version is supported seems like such a small-detail question, that everyone would simply have to scroll up to check this out. A better question might be "why do we use OData batch" or what are the limitations or consequences.
rules.vr files do not go into production (how did you include it since we have it in gitignore?)

I will close this pull request and I suggest we have a short call.


[FEATURE] Option to use playername instead of displayname for team chat

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Add a way to stop the prefixing, currently, the players display name is used, add an internal placeholder for just players name
ie %name% which can be used for that message in messages.yml

This feature has been implemented and will be included in the next update.

This feature has been implemented and will be included in the next update.


Investigate performance of demo instance when multiple users try to create submissions at the same time

Once we are running fcrepo in it's own ECS Service, we'll be able to monitor performance and respond.


Optimize imports

Optimize imports
Fixes #3201 [GCI Ideas] Remove unused imports from classes
I've optimized imports and removed unused imports with the Code > Optimize imports option
Tests performed
Gradle build finished successfully with default settings.

Codecov Report

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

@@          Coverage Diff           @@
##           master   #3272   +/-   ##
======================================
  Coverage    6.91%   6.91%
======================================
  Files         270     270
  Lines       11635   11635
  Branches      907     907
======================================
  Hits          805     805
  Misses      10770   10770
  Partials       60      60

Impacted Files
Coverage Δ

...fr/free/nrw/commons/filepicker/UploadableFile.java
1.66% <ø> (ø)
:arrow_up:

...va/fr/free/nrw/commons/campaigns/CampaignView.java
0% <ø> (ø)
:arrow_up:

.../fr/free/nrw/commons/upload/ThumbnailsAdapter.java
0% <ø> (ø)
:arrow_up:

...fr/free/nrw/commons/media/MediaDetailFragment.java
0% <ø> (ø)
:arrow_up:

...in/java/fr/free/nrw/commons/media/MediaClient.java
74.54% <ø> (ø)
:arrow_up:

...commons/upload/categories/CategoriesPresenter.java
76.27% <ø> (ø)
:arrow_up:

...java/fr/free/nrw/commons/category/CategoryDao.java
0% <ø> (ø)
:arrow_up:

...c/main/java/fr/free/nrw/commons/utils/UiUtils.java
0% <ø> (ø)
:arrow_up:

...free/nrw/commons/theme/NavigationBaseActivity.java
0% <ø> (ø)
:arrow_up:

...fr/free/nrw/commons/mwapi/OkHttpJsonApiClient.java
0% <ø> (ø)
:arrow_up:

... and 111 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 f27749f...b91b879. Read the comment docs.

@m-alzam Thanks!
Are you a CGI student?

@nicolas-raoul Yes, they are, cf https://codein.withgoogle.com/dashboard/task-instances/4981019858436096/

@m-alzam if you could remove the conflicts, that would be great.
Rest looks good :+1:

@nicolas-raoul should I approve the task?

if you could remove the conflicts, that would be great.
Rest looks good 👍

Are there any conflicts? I fixed them with a second commit.

It shows three conflicting files here.

Sorry, I was using my mobile and it's not showed in mobile. I'm going to fix it

@m-alzam great :rocket:

It should be fixed

Better merge this quickly, because such large refactorings become conflicting very quickly when more commits are merged.


Microsoft.Azure.SignalR.Management dependency on Newtonsoft System.IO.FileLoadException

Describe the bug
Azure SignalR Management sample fails with System.IO.FileLoadException while running on .NET Core 3.1
Unhandled exception. System.AggregateException: One or more errors occurred. (Could not load file or assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed'. The located assembly's manifest definition does not match the assembly reference. (0x80131040))
 ---> System.IO.FileLoadException: Could not load file or assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed'. The located assembly's manifest definition does not match the assembly reference. (0x80131040)
File name: 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed'

Any reason why Newtonsoft.Json is still being used instead of System.Text.Json?

To Reproduce
Link for the sample

Exceptions (if any)
System.IO.FileLoadException

Further technical details
Version details in the screenshot attached.

Your Azure SignalR SDK version
Your Server ASPNETCORE version or Assembly version of Microsoft.AspNetCore.SignalR
Your SignalR Client SDK version

Hi @sukesh-ak , I can run the message publisher with <TargetFramework>netcoreapp3.1</TargetFramework>.
My dotnet version is 5.0.100-preview.5.20279.10. Could you tell me yours?
When you get this problem? On building the sample or running some function? Could you provide more details?

My Newtonsoft.Json is v11.0.2, have you install other packages that reference Newtonsoft.Json 10.0.1 whichi cause the problem.

I am getting this error while running the app on .NET Core 3.1 and not on compile time.
If I add the NewtonSoft.Json package manually added to the project it works.
I would guess its time to move to System.Text.Json since that's the official route for .NET APIs

I used the same version of Management SDK. But the inside Newtonsoft.Json is v11.0.2. I neve ad Newtonsoft.Json or other packages manually.
@vicancy About System.Text.Json, what do you think?

If you look at the screenshot in the original post above the dependency is coming from
System.Identitymodel.Tokens.Jwt which has a version of 5.5.0
but the latest version on NuGet of Jwt is 6.7.1.

move to System.Text.Json

the entire hierarchy of dependency needs an update for .NET Core 3.1

agree to both


Getting error: "Connection failed"

I installed gsheetsdb package and tried to use it in superset and in python script but I am getting connection failed error. What should be the full uri for gsheets?
Kindly help in this issue.

I have Installed package 'gsheetsdb' (version=0.1.12) and other dependencies from the requirement.txt file. Then, I have run the following code in PyCharm:
from sqlalchemy import *
from sqlalchemy.engine import create_engine
from sqlalchemy.schema import *
# Connection and authentication
service_account_file = "service_account_file.json"    # your service account credential file
engine = create_engine('gsheets://', service_account_file=service_account_file<EMAIL_ADDRESS>inspector = inspect(engine)
url = "google_spread_sheet_url"

# Processing
table = Table(
    url,
    MetaData(bind=engine),
    autoload=True)

query = select([func.count(table.columns.cnt)], from_obj=table)
print(query.scalar())

Note: This works for me. You can try using a python script.

I have Installed package 'gsheetsdb' (version=0.1.12) and other dependencies from the requirement.txt file. Then, I have run the following code in PyCharm:
from sqlalchemy import *
from sqlalchemy.engine import create_engine
from sqlalchemy.schema import *
# Connection and authentication
service_account_file = "service_account_file.json"    # your service account credential file
engine = create_engine('gsheets://', service_account_file=service_account_file<EMAIL_ADDRESS>inspector = inspect(engine)
url = "google_spread_sheet_url"

# Processing
table = Table(
    url,
    MetaData(bind=engine),
    autoload=True)

query = select([func.count(table.columns.cnt)], from_obj=table)
print(query.scalar())

Note: This works for me. You can try using a python script.

Thanks @JagritiG for the comment. I have started using a different approach but this will be helpful in future.

Thanks @JagritiG for the comment. I have started using a different approach but this will be helpful in future.


On completion Sudo fail: sudo: PAM authentication error: Conversation error

Firstly this was (is?) brilliant, and extremely useful.
But - I have now just built 3 seperate Ubuntu VMs (two via Azure Portal, and the last via the above az cli) They all fail with "sudo: PAM authentication error: Conversation error" once successful sss -l … has logged me in. The usual next step is to sudo to apply the apps required, firewall etc. I tried su - and get offered a login, but have no idea what password to use (the plan was to try and fix sudo).
I have a few 2- 3 months older Ubuntu VMs - done exactly the same way which work fine.
Has the code been changed?
This is frustrating and preventing me from building a QA server (I am going to have to return to native login for now).

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

ID: bb8e2623-1f79-aee3-bd97-8b5f647b6980
Version Independent ID: 6346b54d-cc8d-0b79-a6b8-f9144cfa2459
Content: Log in to a Linux VM with Azure Active Directory credentials
Content Source: articles/virtual-machines/linux/login-using-aad.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

@JerryMWeeks thanks for reaching out. I believe there is a current issue with this feature that could be causing this.
See #28822
Also, this feature is in preview so it should be only used for testing at this point in time. Not for production workloads.
@jakaruna-MSFT can look further into it as he is in contact with the team who owns this extension.

@JerryMWeeks Thanks for reporting.
Your issue is the same as the issue which @MicahMcKittrick-MSFT  mentioned in the above comment.
If you login with the username and password used while creating the VM and go to the file /var/log/auth.log, You can find the below error messages.
Apr 25 12:00:51 ubuntu-1 sshd[3408]: pam_aad(sshd:auth): Version: 1.0.008370001; CorrelationId: xxxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxx
Apr 25 12:00:51 ubuntu-1 sshd[3408]: pam_aad(sshd:auth): Cannot find the cloud instance for location SouthIndia

We have informed the Product team about this issue. Will post the updates on the issue which Micah referrenced in the previous comment.

@JerryMWeeks I tried creating the server in South India which gave the error as shown in the previous comment.
When I created a server in Central US, I am getting the same error as yours. I will report this tot he product team. Thanks.

Same issue, I gave myself the Admin Login permissions and I'm able to login successful. However I cannot run any sudo command, as it's giving me the error "sudo: PAM authentication error: Conversation error"

There is update released for that feature.  I will try that and let you know the status

This issue is resolved now. I verified with Ubuntu and it works well..

This issue is resolved now. I verified with Ubuntu and it works well..

Many thanks I'll give this a test again over the next few days and respond back.

Hello, I'm seeing the same  issue on a RHEL 7.6 server... does the update work  on that distribution as well ?

There is a workaround described here: https://github.com/Azure/azure-linux-extensions/issues/791  that disables password authentication
Changing the line in /etc/sudoers.d/aad_admins from:
%aad_admins ALL=(ALL) ALL
to
%aad_admins ALL=(ALL) NOPASSWD: ALL
With this change an Azure Active Directory account can become root on the test server without entering the user password, however there is not yet a solution to have sudo work with password authentication enabled.

@JerryMWeeks AAD login works well with Redhat 7.6. I verified just now.

@carlosporter That is not a workaround. That can be used when the user don't want to login again with AAD when using sudo.
The default behavior is to ask for password when the user users sudo.
I will reply in that issue as well.

@carlosporter
I will close this out for now. If you need additional help please let me know and we can reopen and continue.


Xamarin.Google.Android.Places version alignment

Places SDK for Android shows the SDK version number like this:
dependencies {
  implementation 'com.google.android.libraries.places:places:2.2.0'
}

The NuGet package Xamarin.Google.Android.Places, however, has version <IP_ADDRESS>.
Does the NuGet package match the version of Google's SDK? How can I tell if the NuGet package is up to date?

@breyed
That nuget is for group com.google.android.libraries.places artifacts places and places-compat and yes it needs updates, because there are newer versions 2.x.y,
https://github.com/xamarin/XamarinComponents/blob/master/Android/GooglePlaces/build.cake