Perfectly correct pipeline is rejected "resource not used"

Hi,
How do you check for resource not used. I have resource that is used only in on_failier, but still used. When I try to create pipeline with this resource it is rejected. Note that the same pipeline is accepted from concourse 2.7.0 directly.

This uses the go-concourse API library, but the version that is vendored in is quite old. Additionally, at the time of vendoring in, this library required copying some internal components of the go-concourse library, e.g. https://github.com/robdimsdale/concourse-pipeline-resource/blob/master/out/helpers/setpipelinehelper.go
It doesn't seem surprising that the newer API and this vendored/copied client library might be somewhat out of sync. Fixing this might be as simple as updating the vendored package and the associated copied code, or it might be quite a bit more involved if the difference is large.
I probably won't get chance to get to this soon, but if you're willing to try this out a PR would be more than welcome.

I think it is because the job on_failier and on_success are quite new. I actually found a workaround, I am not under the gun for the fix.

Actually, I just noticed it does not just reject correct pipelines, it literally drops the job on_failier and on_success steps. By the way, why the resource needs to preprocess the pipelines, can it simply pass through it as is?

It's not pre-processing as such, it's more that it duplicates the steps fly takes when creating a pipeline config - merging credentials etc. The resource actually used to embed a copy of the fly binary and shell out to it, but that was cumbersome compared with using the concourse golang library.
It's quite likely that new syntax is not supported because the code was copied and vendored from fly some months ago.
Let me know if you're interested in submitting a PR and we can go through in more detail what might need to change.


Icelandic update+

Added in new strings missing, and fixed one grammatical error.
(Shoutout to kazu0617 for helping with conflict before!)

Thank you for the update!


InnerStartIndex is equal to 0 for  tag

Hi. I'm having a problem with InnerStartIndex is equal to 0 for p tags on my end
HtmlAgilityPack version: <IP_ADDRESS>
OS: Windows 10
.NET Framework version: 4.6.1
Here's the code to reproduce the problem:
string htmlText = "
Test
";
HtmlDocument document = new HtmlDocument();
document.LoadHtml(htmlText);
var nodes = document.DocumentNode.SelectNodes("//p");
var node = nodes.ElementAt(0);
var start = node.InnerStartIndex;
Console.WriteLine(start); // Outputs "0"

Hello @vforvoid ,
The InnerStartIndex is only populated for Text and Comment node (Don't ask me why!)
For example var textStart = node.ChildNodes[0].InnerStartIndex;
I'm not sure of the historical reason of this but by looking at the code, it might cause some side impact to make it work ;(
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus •  LinqToSql Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Well I will probably take a closer look at this next week as I need this functionality on my end.

Hello @vforvoid ,
This issue will be closed since as we said until we code a next major version, we prefer not fixing it to avoid side impact.


Release 0.1

I hope to have the first release when this gets merged! @ViralBShah @shashi
Todo:

[ ] Graph joins
[ ] Graph condensation
[ ] Phase out NullModule
[ ] Rename package to Networks.jl
[ ] Get LightGraphs.jl stuff working on 0.5
[ ] Cleaner queries
[ ] Improve Test coverage to 90+
[ ] Documentation
[ ] Sort out Dependencies and Tag!

Current coverage is 73.16% (diff: 25.53%)

Merging #22 into master will decrease coverage by 1.59%

@@             master        #22   diff @@
==========================================
  Files            25         27     +2
  Lines          1327       1360    +33
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            992        995     +3
- Misses          335        365    +30
  Partials          0          0

Powered by Codecov. Last update 0ae3dec...e59bbea

Before tagging this, let's pick the new name.

Looks like I'm going with Graft.jl

:fireworks:

I've sent a request to register and tag Graft.


Dump not working

I tried to dump my competition from 2 test competitions:
https://www.codabench.org/competitions/197/
https://www.codabench.org/competitions/198/
I can get the original bundle, but not a modified version from the editor.

I close another dump issue. But not sure whether this other one was addressed:
https://github.com/codalab/competitions-v2/issues/415

This problem appeared from a refactor, I just pushed a fix for this and tested on 197. I'll add an issue (#590) for writing a comprehensive dump test so we don't run into this again as easily.


net/http: TestTransportBodyReadError flake on OpenBSD

Seen from trybot:
https://storage.googleapis.com/go-build-log/dba99eff/openbsd-amd64-60_9893d709.log
--- FAIL: TestTransportBodyReadError (0.01s)
	client_test.go:1821: Got error: net/http: HTTP/1.x transport connection broken: some body read error; want Request.Body read error: some body read error
FAIL
FAIL	net/http	4.860s

(Others: feel free to debug this before I get to it.)


Automatic error report

I cannot open and visualize my data.
I simply created randomly a dataset:
data = np.random.randn(50000)  * 20 + 20
AND THIS ERROR APPEARED

What error?

This should be fixed in our latest Spyder 3 version. Please update by opening the Anaconda Prompt and running there
conda install spyder=3.3.6


Documentation

Dear,
I see you have a very nice MARL library for RL.
Currently I am reviewing (MA)RL libraries and I can't find a separate documentation website. Is this true? Maybe I overlook it somewhere.
However the GitHub readme suggests there is one.
Hope you can guide me into the right direction!
Best,
Chris

Hi! We currently do not have a separate documentation website. The "documentation" in the readme refers to the detailed pydoc-compatible documentation for our classes and functions within the source code. In particular, the classes and functions in pantheonrl/common detail the interface and assumptions that we make.

Perfect, thank you!


Check env variables in job launcher

Description
Clean everything around env variables

[x] Remove unused env variables from env example
[x] Use env validator like here
[x] Use constants for the env variable names

Updated Nest. Many libraries were not compatible with each other, all versions were updated. @portuu3


Kanai's Cube

The "Corrupted Ashbringer" cannot be cubed in diablo yet the planner allows it

It's in the not yet implemented list anyways, so this is just a UI issue.

The i have a suggestion to add instead :)
Could you move the effects from the Kansai's Cube items to Skills & Effects like the other equipped items ? it seems a bit odd as many items have enable or stack count which affect damage values havng to jump forth and back all the time :)

@dominiklohmann: he means you can't cube it at all in the game (because its effect replaces your weapon with Ashbringer, which probably causes animation issues when you have other weapons equipped).
I could make it un-cubable but I don't think its a big issue anyway.
@wow-config: Yeah I'm not too happy with how cube items are separate, I used to have it in skills & effects before but too many people complained that it was too "hidden" and many people just couldn't find it.
I think the best solution would be to duplicate it so it appears in both places at the same time.

because its effect replaces your weapon with Ashbringer, which probably causes animation issues when you have other weapons equipped

I get your point, just never considered that as a reason. You can't even see the effect in Archon for example.

because its effect replaces your weapon with Ashbringer, which probably causes animation issues when you have other weapons equipped.

What i meant was its presence could confuse newer players but as he explained it is just a UI issue.


[v2] useFormik returns undefined values

🐛 Bug report
Current Behavior
useFormik is returning undefined for the values object:
const formik = useFormik({
  initialValues: { user: 'test' }
});

console.log(formik.values); //> logs undefined

But for some reason, it is not happening when this code is running in the server (SSR). When running on the server, the function returns the correct value but as soon as the code runs in the browser, values is set to undefined again.

This was tested on CRA 3.0.1 and Next 8.1.0. I cannot reproduce this on CodeSandbox.
Expected behavior
formik.values should be consistent and output { user: 'test' }.
Reproducible example
Live demo: https://formik-nextjs-jgkhfsskwe.now.sh/ (refresh the page and see the value coming from the server and then being overridden by the client)
Code: https://zeit.co/rafaelalmeidatk/formik-nextjs/jgkhfsskwe/source?f=pages/index.js&host=formik-nextjs-jgkhfsskwe.now.sh
Suggested solution(s)
This started on version 2.0.1-alpha.51, I suspect it was due to 902a40aeaab7de4f41d20e485f96bf1e215caf48. Version 2.0.1-alpha.5 works.
Maybe there is something going on the production build of the lib because it works fine if I try to run it locally in development mode or use the dev files instead of the production ones.
Your environment

Software
Version(s)

Formik
2.0.1-rc.0

React
8.1.0

TypeScript
-

Browser
Chrome 69.0.3497.100

npm/Yarn
yarn 1.15.2

Operating System
69.0.3497.100

Oops duplicate of #1523, sorry 😅
I will add my info there

@rafaelalmeidatk Sorry for a sloppy question, but is there a place with 2.0 docs, where useFormik is described?

@klimashkin not yet, as far as I know. I gathered some pieces of code from the comments here #1063, read the source code and tried different things until it worked. I don't know everything about v2 but I may try to create some documentation if I find some time.

same thing bugging me.


Examples of using the BLAS functions

Apologies for this novice question.  I'm not an expert Lisp programmer but I'm trying to use magicl to do some linear algebra and other matrix operations and I'm getting a bit bogged down.  I was wondering if there were any examples of using the BLAS functions from Lisp (e.g., MAGICL.BLAS-CFFI:%DASUM) that anyone could point me to?  I've used some of the high level functions but I want to explore the extent of BLAS.  I've searched through the files on the Rigetti GitHub pages without any success.  If anyone has some examples I could investigate I'd be very grateful.  Many thanks.

This was a bit more transparent under previous versions. On 0.6.5, two natural places to look are examples.lisp and high-level.lisp (e.g. scale and multiply-complex-matrices). Note that in examples.lisp the magicl.blas-cffi package is aliased to blas.

That's very helpful - thank you.


The IO/clock constraint does not get honored at certain corner cases

For a design where IO is followed by a Flipflop, the FF is pulled into the IO and registered IO is inferred.
In this case if user wants to constraint the IO clk to some specific clock then the constraint is not honored

@kkumar23 Could you please provide more details like eg. in which of the test designs you see this issue?
I couldn't have reproduced it so far. I took the io_reg test design and constrainted clk to io.clk[3]. Then, I verified the repacked netlist (.net) and confirmed that indeed the net clk enters the io block through io.clk[3].
I've also verified the output FASM file. It contains the only line responsible for the clock input mux:
fpga_top.grid_io_bottom_bottom_5__0_.logical_tile_io_mode_io__7.mem_iopad_0_clk_0.mem_out[1]

which corresponds to the choice of clk[3] input.
I did the test for the current master branch (90fd7572).

@mkurc-ant : I reinstalled the package and verified. I see the changes in the fasm file . It was for the io_reg design.

@mkurc-ant : can you add the make support for the json. We are using toolchain wrapper as of now.

@kkumar23 Of course, I will be looking into that.


Use divergent colorscheme for choropleth maps

Currently, the choropleth maps displayed on analyse pages are coloured using a categorical scale: rather than colors varying continuously, all CCGs in the same quintile are assigned the same color.
The maps would be easier to interpret if a divergent color scheme were used, highlighting values that deviate significantly from the mean in either direction.
Example
Consider two antipsychotics, Zuclopenthixol Acetate and Pericyazine, both of which are used almost exclusively in a small number of CCGs.
Current color scheme
With the current color scheme, it is hard to tell anything about how prescriptions for either drug are distributed geographically.
It's difficult to answer the question of how many CCGs prescribe pericyazine at a high level, as many CCGs are assigned the color corresponding to the broad range 2.29-247.36.
The map for zuclopenthixol acetate has no color variation at all, in the color scale equivalent of the radius scale bug #1.

Divergent color scheme
Re-plottting in R with a divergent color scheme makes the actual pattern of use clear: pericyazine is restricted to a cluster around Norfolk, whereas zuclopenthixol acetate is used at a few isolated CCGs. For convenience, these plots show raw prescription counts (without normalising by dividing by total antipschotic use), but the pattern is broadly the same.

Colour scheme
I agree that a divergent colour scheme may be an improvement.
Here's what the Pericyazine one would look like:

Category discrimination
The issue about the Pericyazine cluster in Norwich not being apparent is not related to the colour scheme.  It's because we classify the values using quintiles.
Upping this to deciles improves discrimination somewhat:

Using equidistant bins, as you did, brings out the anomaly in Norwich:

However, this is at the cost of losing some discrimination in the mid-levels. In this example, where the distribution of values is highly skewed, the mid-level discrimination is meaningless, but in more normal distributions, using quantiles has the effect of increasing the discrimination across all categories.
Looking at such an example (the Rosuvastatin example linked from the analyse page), it seems to me that using quantiles doesn't offer any great advantage over equidistant bins (we can also use k-means to cluster which gives similar results):
Using quantiles:

Using equidistant limits:

Using k-means clustering:

Two-bin anomaly
As you noted, zuclopenthixol acetate example is a different bug affecting cases where there are only two classes of data, which we can easily fix.

I meant to suggest using a colorscheme that is both diverging and continuous, rather than the current scheme, which is both sequential and semgmented/quantized/binned.
You are correct that the invisibility of the Norwich pericyazine cluser is due to the binning: I should have been more precise and referred to the 'colormap', to also include the binning process.
As you show, increasing the number of bins helps to improve discrimination. An alternative is to make the color scale continuous. This has the potential disadvantage that it might make it harder for people to read the value for a particular CCG off the scale, but they can get this directly from the tooltip.
If you stick with a binned colormap, I think that constant-sized bins are preferable to variably-sized bins (whether determined by quantiles or k-means):

in very skewed cases like Norwich pericyazine they show the anomaly much more clearly than do quantile based bins
the increase in mid-level discrimination in more typical cases seems to be only slight (based on the Rousavastatin example you show), and small differences from the mean are generally relatively uninteresting anyway
they have less potential for confusion, as viewers can't be misled by assuming that bins are constant-width

I see what you mean. Yes, I think that could make sense. Here are some before/after screenshots.


Support Reading PageIndex with ParquetRecordBatchStream

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

ParquetRecordBatchStream does not contain the necessary logic to read the offset index
Describe the solution you'd like

Add support for reading the column index to ParquetRecordBatchStream
Describe alternatives you've considered

Additional context

I can take this one

I think this will involve reading the necessary data in ParquetRecordBatchStreamBuilder::new if the metadata returned by AsyncFileReader::get_metadata doesn't already have it loaded. I think it is important to handle the case where the AsyncFileReader is able to provide this information, e.g. if there is some metadata caching, the data is stored in a catalog, etc...

Was tinkering around with this and we can go one of two ways:

Fetch all the index metadata when constructing the stream.
Fetch the index on demand as needed.

The latter approach, we could add something to AsyncFileReader like
fn get_page_locations(&mut self, metadata: &ParquetMetadata, mask: &ProjectionMask) -> BoxFuture<'_, Result<Vec<Vec<PageLocation>>>

to only fetch what we need for a given projection. But that may be overkill given the amount of data....

I think do the first option for now, the data should be small and relatively cheap to decode. We can always revisit if this simple approach proves insufficient


Submission of oep Scaleout phase 1 proposal

Hi there! We are officially moving all Halo Grant proposals to our new community forum page and will no longer be accepting proposals on GitHub. Please submit your proposal to the community page according to the provided template: https://community.iotex.io/c/halo-grants/61
Thank you!


Simulations

Reference Issues/PRs
Fixes #144
Fixes #198
What does this implement/fix? Explain your changes.
Part of ER simulations is replaced by SBM
Modifying the omni test
Any other comments?

Codecov Report

Merging #200 into master will decrease coverage by 0.11%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #200      +/-   ##
==========================================
- Coverage   86.03%   85.91%   -0.12%
==========================================
  Files          30       30
  Lines        1840     1825      -15
==========================================
- Hits         1583     1568      -15
  Misses        257      257

Impacted Files
Coverage Δ

graspy/simulations/simulations.py
95.6% <100%> (-0.25%)
:arrow_down:

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 9ebdcb0...864b959. Read the comment docs.


Wiki page redirect

In the wiki, when I click on the 5th page titled "Install the Fairy-wallet application", I get redirected to the "Home" page
https://github.com/tarassh/fairy-wallet/wiki/Install-the-Fairy-wallet-application
redirects to
https://github.com/tarassh/fairy-wallet/wiki

Wow, someone just fixed it, good job!


Add a Gulp task to build SCSS version

In the most cases, I think SCSS users only want to import the actual SCSS files they needed (in favor to reduce the CSS file size).
I added a task to copy the source to SCSS dir and rename them into Sass partials.
You can use it as:
@import "base";
@import "bouncing_entrances/bounceInLeft";

In another side, it would be great to rename the source dir directly and compile a CSS version from it.

Please fix the issue first @KayLeung so I can review it thanks :)

Hey @KayLeung, thanks for contributing! I recently closed the old Sass PR #277 under the impression that what we'd actually want from a major rewrite wouldn't necessarily require SCSS. I'd rather not add SCSS output to this library.


[DO NOT MERGE] BH withdraw release notes

Updates to guidance content to go out with the release of Withdrawal features on Thursday 7 Feb 2019

Closing this PR as the work has been cherry picked onto this - https://github.com/alphagov/content-publisher/pull/767


Incorrect 'No overloads for "apply" match the provided arguments' on the pandas apply function

Incorrect No overloads for "apply" match the provided arguments on the pandas apply function
The problem seems caused by the "args" argument. But only in some cases. See the two nearly identical lines in the snippit below.
This may be similar to #2547, but sees different enough to warrant a new issue
Environment data

Language Server version: v2022.10.20
OS and version: mac os Monterey 12.3.1
Python version (& distribution if applicable, e.g. Anaconda): 3.10.4
pandas==1.5.0
numpy==1.23.2

Code Snippet
import pandas as pd

def gethead(s, y):
    return s.head(y)

df = pd.DataFrame({"A": [1, 2, 3, 4, 5, 6], "B": [11, 12, 13, 14, 15, 16]})

df2 = df.apply(gethead,  args=tuple([4]))
df3 = df.apply(gethead, axis=0, args=tuple([4]))

print(df2)
print(df3)

Repro Steps

Open the code above in VS Code
Run it to confirm that the code works

Expected behavior
There should be no warnings / errors
Actual behavior
Error:
No overloads for "apply" match the provided arguments
Argument types: ((s: Unknown, y: Unknown) -> Unknown, list[int])PylancereportGeneralTypeIssues
This happens on the first call to df.apply but not the second
The only difference is that one of the optional parameters to df.apply is explicitly included in the second case.
Note that when you run the code, both lines produce identical output as expected.
Logs

Log output attached
pylance_output.txt

there are 2 overloads on apply, try right click Got declaration to see them.
see https://github.com/microsoft/pylance-release/issues/3029#issuecomment-1182029725

if you think your code example is correct please create an issue in the pandas stub repo. https://github.com/pandas-dev/pandas-stubs/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc

Specifically I think the problem is that the second overload of apply doesn't have a default value for the axis parameter:
    @overload
    def apply(self, f: Callable) -> Series: ...
    @overload
    def apply(
        self,
        f: Callable,
        axis: AxisType,
        raw: _bool = ...,
        result_type: _str | None = ...,
        args=...,
        **kwargs,
    ) -> DataFrame: ...

but the implementation does:
    def apply(
        self,
        func: AggFuncType,
        axis: Axis = 0,
        raw: bool = False,
        result_type=None,
        args=(),
        **kwargs,
    ):

So that overload should be:
    @overload
    def apply(
        self,
        f: Callable,
        axis: AxisType = ...,
        raw: _bool = ...,
        result_type: _str | None = ...,
        args=...,
        **kwargs,
    ) -> DataFrame: ...

The error goes away after I make that change (add  = ...) in frame.pyi.

Pyright complains about the fix I proposed: Overload 1 for "apply" overlaps overload 2 and returns an incompatible type. Basically once axis has a default value, there's no way to tell which overload should be used and therefore which return type should be inferred.
So I opened a bug here and tagged you @jonmooser: https://github.com/pandas-dev/pandas-stubs/issues/393


Adding more test cases

I've created 6 more test cases, based off the options a user can specify in the /etc/sysconfig/docker-storage-setup file. Not all cases have been covered, but I wanted to do a PR to check progress before creating more test cases/improving these files. Some things that still need to be considered are GROWPART and ROOT_SIZE.
Note: Test 11 and Test 12 take a bit longer since I have them simulate pulling docker images and saturating the LV pool, which of course can be lots of data.

@rhvgoyal @rhatdan could you take a look at this?

bot, add author to whitelist

LGTM, But @rhvgoyal is the expert here.  @shishir-a412ed PTAL

Can we reduce the number of commits ? and update commit messages to more meaningful messages.
If I am reading the git log "tweaked", "type", "comments" don't tell me much about that commit :)
Shishir

I'd git rebase -i origin/master and squash them all together.

okay I will rebase and squash

Please check PR #141 , I this one was too messy. Sorry for any confusion.

@stephen679 You don't need to open new PRs to squash them.  You can just do a
git push --force
after you squash.


onClose() running constantly (!) Potential memory  leak

found these lines of code, are constantly running in case you trigger the login and do not redirect onCode,
(0, _react.useEffect)(function () {
    if (externalWindow) {
      intervalRef.current = window.setInterval(function () {
        try {
          var currentUrl = externalWindow.location.href;
          var _params = new URL(currentUrl).searchParams;

          var _code = _params.get('code');

          if (!_code) {
            return;
          }

          onCode(_code, _params);
          clearTimer();
        } catch (error) {// eslint-ignore-line
        } finally {
          if (!externalWindow || externalWindow.closed) {
            onClose();
            clearTimer();
          }
        }
      }, 700);
    }

might be better to trigger that effect only once?
or do something like:
(0, _react.useEffect)(function () {
    if (!externalWindow.closed) {
      intervalRef.current = window.setInterval(function () {
        try {
          var currentUrl = externalWindow.location.href;
          var _params = new URL(currentUrl).searchParams;

          var _code = _params.get('code');

          if (!_code) {
            return;
          }

          onCode(_code, _params);
          clearTimer();
        } catch (error) {// eslint-ignore-line
        } finally {
          if (!externalWindow || externalWindow.closed) {
            onClose();
            clearTimer();
          }
        }
      }, 700);
    }

there must be a more neat way to expect this window to close down..seems there is a memory leak, at least with these timers in this location

@kapekost release v1.0.3 is out! 🥳

@kapekost release v1.0.3 is out! 🥳

awesome! thank you!

awesome! thank you!

Reopening this issue.
Double/rapid clicking on the button triggers the infinite onClose events.


static: use modern getJSON callbacks

Use done() and fail() instead of complete() and success() that got
removed in jquery 3.0.

Thanks, you need to regenerate the minified js https://docs.readthedocs.io/en/latest/development/standards.html#getting-started

@stsewd i've not regerated them on purpose to avoid issues with review / merge latency. If you are fine with the PR i can build the static files.

Those are deprecated in version 2? We still use 2.0.3
https://github.com/rtfd/readthedocs.org/blob/110347355921fe82aee57ef04160a7a3477dedef/bower.json#L17-L17

Are deprecated since 1.5 or 1.6 can't remember.

Updated branch with rebuilt static files

Updated PR to fix merge conflict

New build conflicts, what about starting to open PR with only the source changed and have a commiter rebuild the static files?

I was thinking about that too, I don't remember what the core team said about that

Tested this locally and it looks good. 👍


Add README to Motion correction

add README to motion correction
add arch param to select the corresponding architecture (vae-mononet or vae-mocounet)
remove loss_network param

please add also a separate config file named param_mc.yml with the required settings.


[wip] move in-memory implementations of database to sqlite for dev

Currently, dex's state is represented by "repo" interfaces. For testing (and --no-db mode) we implement an in-memory version of all these interfaces so we don't have to rely on postgres for testing See the in memory user repo for an example.
These are becoming a pain for a couple reasons.

They're a large portion of the code base.
Maps and slices don't map well to the SQL data structures, so adding additional columns to the database can result in having to completely restructure the in memory version.
In memory repos can't reference data types from other repos. JOIN statements have no
analog.
We don't test our SQL code when we use them in tests.

Goals of this PR are:

Move away from implementing our own in-memory implementations
Begin to experiment with supporting other SQL flavors.
Have --no-db mode use SQLite.

NOTE:
This PR DOES NOT add support for SQLite3 to dex. Only tests and --no-db mode.
TODO:

[x] Add sqlite3 implementation of the database backend.
[x] Move in memory representations to an in memory DB, including --no-db mode.
[x] Disable sqlite3 support unless dex specifies --no-db mode.

@bobbyrullo This is ready for a review. Large changes are

Deleted all the "in memory" implementations of repos.
dex now uses sqlite for tests and --no-db mode (functional tests still use postgres).
Added a db/translate package to translate from Postgres queries to sqlite so we only have to write one kind of queries.
Moved SessionManager to session/manager package (so it can import the db package for tests).
A ton of small fixes to database queries and tests.
No user facing changes.

On thing to note is that the SQL implementation of ClientIdentityRepo expects client secrets to be base64 encoded. Because of this, client secrets explicitly passed to dex during --no-db mode must be base64 encoded. See example here

@bobbyrullo this has been updated (see the last two commits).

Cool I'll take a look later on.
On Tue, Feb 16, 2016 at 6:27 PM Eric Chiang<EMAIL_ADDRESS>wrote:

@bobbyrullo https://github.com/bobbyrullo this has been updated (see
the last two commits).
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/dex/pull/304#issuecomment-184981307.

LGTM - nice work!


proposal: net: add QueryType to *DNSError

Go version
go version go1.21.6 linux/amd64
Output of go env in your module/workspace:
GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/home/scripts/.cache/go-build'
GOENV='/home/scripts/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='local'
GOTOOLDIR='/usr/local/go/pkg/tool/linux_amd64'
GOVCS=''
GOVERSION='go1.21.6'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='0'
GOMOD='/home/scripts/src/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -fno-caret-diagnostics -Qunused-arguments -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build1931870782=/tmp/go-build -gno-record-gcc-switches'

What did you do?
https://go.dev/play/p/oN0gVQA4nm2
Please Note: On goplay lookups don't work. You have to run it locally.
What did you see happen?
Host Lookup
[<IP_ADDRESS>]

TXT  Lookup
[]
lookup mail10476.emails3.rightmove.co.uk on <IP_ADDRESS>:53: no such host
What did you expect to see?
Instead of "no such host" I would hav expected "no TXT record"
See also
https://stackoverflow.com/questions/77880685/golang-net-lookuptxt-returns-no-such-host

After a quick look at the code, a simple and ugly fix looks like this.
--- a/src/net/lookup.go
+++ b/src/net/lookup.go
@@ -856,6 +856,11 @@ func (r *Resolver) goLookupNS(ctx context.Context, name string) ([]*NS, error) {
 func (r *Resolver) goLookupTXT(ctx context.Context, name string) ([]string, error) {
        p, server, err := r.lookup(ctx, name, dnsmessage.TypeTXT, nil)
        if err != nil {
+               if e, ok := err.(*DNSError); ok {
+                       if e.Err == errNoSuchHost.Error() {
+                               e.Err = "no TXT record"
+                       }
+               }
                return nil, err
        }
        var txts []string

That wouldn't work. Please replace the hostname in my example with one that doesn't exist, e.g. put an "x" in front of the name.
Your "fix" would say that there is no such TXT record. While it technically is true, it's not consisternt with the host command:
$ host -t TXT xmail10476.emails3.rightmove.co.uk
Host xmail10476.emails3.rightmove.co.uk not found: 3(NXDOMAIN)

$ host -t TXT mail10476.emails3.rightmove.co.uk
mail10476.emails3.rightmove.co.uk has no TXT record

Is there any way to get the underlying DNS errors?

We currently return "no such host" for NXDOMAIN and NODATA conditions. Maybe we should change that and return "no such host" (or "no such domain") and "no records found" ?

We currently return "no such host" for NXDOMAIN and NODATA conditions. Maybe we should change that and return "no such host" (or "no such domain") and "no records found"?
I think, this is a good idea.

In my current project I'm using Perl for the lookups I have to do. But I want to switch to go. Unfortunately I currently report the DNS errors to my customers and with the net module I will have to change the results. I don't think it's a big issue  in my case, but at least an inconvenience.

We might also introduce a QueryType field into the *net.DNSError, so that we can show the queried record type in the error message.
This would be perfect.

I would first make sure what @Skeeve's original wish regarding the stated problem was.
If it was purely about how errors of such class look when printed, — that is one thing; if, instead, they would like to be able to programmatically differentiate between NXDOMAIN and missing records of the queried class, — that's another story.
I have one more concern (maybe misguided): I would not say that having zero TXT records for an existing domain is an error to begin with. In the OP's case I would expect net.LookupTXT to return nil, nil or []string{}, nil which genuinely means "there was no errors, and the authoritative DNS server returned zero records of the requested type.
IOW, I'd say it's a client's issue to present this situation to a user as an error, if desired.

I have one more concern (maybe misguided): I would not say that having zero TXT records for an existing domain is an error to begin with. In the OP's case I would expect net.LookupTXT to return nil, nil or []string{}, nil which genuinely means "there was no errors, and the authoritative DNS server returned zero records of the requested type".
IOW, I'd say it's a client's issue to present this situation to a user as an error, if desired.

I don't think that this kind of change is backwards compatible at this point.

I would first make sure what @Skeeve's original wish regarding the stated problem was. If it is purely about how errors of such class look when printed, — that is one thing; if, instead, they would like to be able to programmatically differentiate between NXDOMAIN and missing records of the queried class, — that's another story.

First of all I thought that "no such host" for a missing TXT record is plain wrong. That's where I came from.
So regarding your other statement:

I have one more concern (maybe misguided): I would not say that having zero TXT records for an existing domain is an error to begin with.

I fully agree. A missing TXT record is not an error. An empty slice or a nil for the slice would perfectly indicate the "missingness" of the record

if, instead, they would like to be able to programmatically differentiate between NXDOMAIN and missing records of the queried class, — that's another story.

That would be a nice-to have. So being able to drill down without going to another module which allows "bare-bones" DNS access would be very helpful, at least to me.

I don't think that this kind of change is backwards compatible at this point.

That's for sure. Maybe providing another function for that purpose would help here?

I have one more concern (maybe misguided): I would not say that having zero TXT records for an existing domain is an error to begin with.
That would be a nice-to have. So being able to drill down without going to another module which allows "bare-bones" DNS access would be very helpful, at least to me.

This can be implemented, but i would block this on #63116, so that we don't pollute the DNSError with more fields (we already have IsNotFound). Then we can define a global errors:
var (
    ErrNoSuchDomain = errors.New("no such domain") // DNS NXDOMAIN
    ErrNoRecordsFound = errors.New("no records found") // DNS NODATA
)

But then question remains on how to handle /etc/hosts (LookupHost) (is this NXDOMAIN/NODATA or something else?).
Personally I think that if you need this kind of access you should use a DNS library directly like miekg/dns.

This can be implemented, but i would block this on #63116, so that we don't pollute the DNSError with more fields (we already have IsNotFound).

Oh, that is super cool: I, for one, was puzzled when I've discovered it's impossible to somehow "fetch" a non-string reason from a *net.DNSError.

Then we can define a global errors:
var (
    ErrNoSuchDomain = errors.New("no such domain") // DNS NXDOMAIN
    ErrNoRecordsFound = errors.New("no records found") // DNS NODATA
)

While I would prefer a nil, nil, or []string{}, nil variation, these look sensible.

But then question remains on how to handle /etc/hosts (LookupHost) (is this NXDOMAIN/NODATA or something else?).

I would treat a missing /etc/hosts or a missing host in it as NXDOMAIN because we failed to find the host.
Also–again, may be my guess is misguided,–but I thought it's only posisble to have equivalents of A and AAAA records in /etc/hosts, so, basically, any kind of request which is not of these two types can be failed right away if it were about to be attempted agains that file, and that would amount to a "hard" error: a failure to resolve the domain/host name, which happens before an attempt to obtain the records of its zone.

But then question remains on how to handle /etc/hosts (LookupHost) (is this NXDOMAIN/NODATA or something else?).

I'd go with what digand host would give, which is NXDOMAIN.
Or did I misunderstand?

Personally I think that if you need this kind of access you should use a DNS library directly like miekg/dns.

I thought about switching to it. It's just a question of: Is it worth it? After all: I think, and I might be wrong here, the information is already available when using net.LookupXXX. Why not make it accessible?

I don't agree that /etc/hosts should return a NXDOMAIN-kind of error, because when receiving a NXDOMAIN we can assume that no other record exists on that domain name. With hosts that might not hold true  (and probably with other nss modules (with cgo resolver)).
LookupHost("not-in-hosts.example.com") // might return NXDOMAIN when nsswitch.conf is `dns: files`
LookupTXT("not-in-hosts.example.com") // returns an answer even though LookupHost returned NXDOMAIN.

For sure I'm not too deeply involved in this, so what exactly do you mean with /etc/hosts? Looking up ONLY in /etc/hosts? So explicitly limiting lookup to /etc/hosts? In that case a NXDOMAIN would be the perfect fit to my understanding as the requestet domain/hosts is not defined in any of the locations the lookup was allowed to look into. But this is just my uneducated idea about it.

NXDOMAIN response means that the requested domain does not contain any records, but depending on the system configuration that might not hold true, like in the example:
LookupHost("not-in-hosts.example.com") // might return NXDOMAIN when nsswitch.conf is `dns: files`
LookupTXT("not-in-hosts.example.com") // returns an answer even though LookupHost returned NXDOMAIN

Looking up ONLY in /etc/hosts? So explicitly limiting lookup to /etc/hosts? In that case a NXDOMAIN would be the perfect fit to my understanding as the requestet domain/hosts is not defined in any of the locations the lookup was allowed to look into.

Yes, in case where the system is configured to only use /etc/hosts, but it is not perfect, because every other lookup method might resolve that name, even when LookupHost returned NXDOMAIN (other lookup methods directly do DNS without any file lookup).
This is why I am fine with returning strings "no such domain" "no records found", but don't like the idea of separate exported errors because of cases like that. And who knows what the getaddrinfo (through nss modules) and windows report in similar cases.

But then, why would you want to distinguish NXDOMAIN/NODATA errors in the first place? Maybe i am missing something. I've only seen it used to early abort multiple DNS queries, when the first one returns NXDOMAIN (then we are sure that next lookups would return the same response (NXDOMAIN) for the same name).


[BUG] Missing NLB metrics after upgrade to 0.38-alpha

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Upgraded yet-another-cloudwatch-exporter from 0.32 to 0.38 and found NLB metrics that were previously being scraped missing. No changes to the NLB resources itself and no indication of errors in YACE logs.
Attached prometheus screengrab shows no NLB metrics being returned after upgrading YACE on 8th Augus

t
Expected Behavior
NLB metrics being scraped as before.
Steps To Reproduce
YACE version 0.38.0-alpha running in kubernetes cluster.
Config for NLB which has stayed the same throughout the upgrade:
- type: nlb
regions:
- eu-west-1
metrics:
- name: UnHealthyHostCount
statistics:
- Maximum
period: 300
length: 600
Anything else?
No response

I'm running into something similar. aws_*_info metrics for any services are not being returned if those jobs don't have any metrics. For me this is common for NLB metrics, because NLB metrics are absent when there's no data. In your case, if there are no unhealthy hosts in your NLB then there will be no datapoints for UnHealthyHostCount, and no aws_nlb_info metrics will be returned.
If any resource in your job/region returns at least 1 metric, the aws_*_info metrics reappear.
This seems to have been a side-effect of #606, where resources and metrics are only recorded if there's at least one resource and at least one metric.


New install: Do goog accounts work at all now?

hi,
i just installed L4M (lineageos for microg). self-checks everything green. added a google account; i got the standard security alert via email. installed the gmail app, gave it all permissions. when i open it, it doesn't see the account configured on the device. if i use it to add a new account, the add works, but then the gmail app continues to not see any account.
i got my GSF ID and added an exception for it on goog. but nothing changed.
too much is happening lately with play certification and i couldn't really keep up.
is this the dreaded #510 ?
also, i installed yalp store, and it cannot log in with the bundled credentials. i did not want to try me real credentials. yalp store on my other device, which is gapps-enabled and where it used to work fine, now says "no network connection".
any pointers? thanks!

The problem of yalp store is unrelated, see here: https://github.com/yeriomin/YalpStore/issues/447

thanks! ok, i got fdroid's version, i need to wait for the fix. (waiting for fdroid builds gets me on my nerves :) .)
and the gmail issue?

what i want to know is: does gmail fails for everyone these days? or only for me?

Is it time to create a chat channel at Matrix.org or gitter or IRC?
ср, 23 мая 2018 г., 8:36 Lanchon<EMAIL_ADDRESS>
what i want to know is: does gmail fails for everyone these days? or only
for me?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/microg/android_packages_apps_GmsCore/issues/548#issuecomment-391201595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAfmeKMVtj97xpEgIORdXeESShNqVHMbks5t1MslgaJpZM4UINnT
.

--
Ildar Mulyukov,
(ΙΧΘΥΣ) child of God
email<EMAIL_ADDRESS>matrix: @ildar:matrix.org
GoogleTalk<EMAIL_ADDRESS>blog: http://johan-notes.blogspot.com/

(@ale5000-git, f-droid updated yalp store and now it works. gmail issue, i'm still lost about.)

That would be good to have irc channel. It would clean up a bit the issues at github from unexperienced users.
So vote up for IRC +1
On May 23, 2018 3:23:21 AM UTC, ildar<EMAIL_ADDRESS>wrote:

Is it time to create a chat channel at Matrix.org or gitter or IRC?
ср, 23 мая 2018 г., 8:36 Lanchon<EMAIL_ADDRESS>
what i want to know is: does gmail fails for everyone these days? or
only
for me?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub

https://github.com/microg/android_packages_apps_GmsCore/issues/548#issuecomment-391201595,

or mute the thread

https://github.com/notifications/unsubscribe-auth/AAfmeKMVtj97xpEgIORdXeESShNqVHMbks5t1MslgaJpZM4UINnT

.

Though beware that IRC is very outdated hence on 3rd place after Gitter and
Matrix.

@Lanchon
Gmail is working for me but I logged in the account before the "certification thing" so I'm not sure.
Have you tried to manually gran contact permission to Gmail?
Have you tried an old version of Gmail?

hola ale,
thx! as i stated in OP all gmail apk permissions are granted. i havent tried an older app. i dont really have time to investigate or randomly poke at stuff at this time unfortunately. this time around if i cant get stuff to work on a recipe ill have to go gapps again and leave microg for next phone change... :(

@Lanchon does the solution from https://github.com/microg/android_packages_apps_GmsCore/issues/505#issuecomment-386449145 help you with the gmail issue?

Currently I don't have time to test a clean flash but I will do some tests next week.
PS: Did you tried to install microG using my installer?
I have put a lot of code to automatically fix problems, in most cases it does work.
I have created the installer because there are a number of things that can cause problems that checking them at hand everytime I flash a new ROM would be annoying.

i didnt. but im using microg's lineage build and i expect it should work or to know whats wrong. plus i want to keep as L4M-stock as possible, so that updates just work and problems i find down the line are shared by others.

FYI it's broken on playstore for me as well. It's constantly showing the "An app on your device is trying to sign into a Google account" prompt, even if I'm signed in. Pressing "sign in" causes a login prompt to show up, which fails because the account already exists on the device.
@Catfriend1 doesn't resolve the issue for me.

For some people the access problem is fixed by enabling access to less secure apps here.

i've been accessing email via imap for years now, so this is already set in my accounts:

Allow less secure apps: ON

thx!

I can confirm playstore showing constantly the ""An app on your device is trying to sign into a Google account" prompt. Having access to less secure apps "ON" does not help.

@ale5000-git not an option for me because I have 2fa enabled. also, the I can see the phone as signed in recently (I had to log in after deleting the android accounts database), so I'm pretty sure it's not a sign-in issue.

Maybe resetting the account files on the phone will help, I will add an option in my tool to do it soon.

Yes, I haven't yet tried but maybe it help someone.

im running microg <IP_ADDRESS>-.....b48. is this what i should be running?
i continue to have this issue and unfortunately it looks like ill be back to gapps until i swap phones again next year. i can't really do the migration unless i have a new phone to work with; i can't afford to be disconnected for an unknown length of time while doing the work.

I too have the issue with play not recognizing the login.

we'd better have all the "me too"-s in a chat :(
@nailyk , adding +1 to the top post would indicate your "too" allright

I am also having issues now after updating microG core to some unofficial version. Somehow my Google account got lost (while others remained) and I can't re-add it but with different error:
I can succeed each step (mail address, password, policy) but at the end it says there is a problem connecting to Google servers. Also with firewall off it does not work.
Read about deleting some account files but I guess that does delete all accounts? As my problem seems different I'm not sure if I should try this while there is risk of need to re-add all other accounts, too.

deleting account files doesnt work anyway. nothing works.

@ildar @AreYouLoco we have an IRC channel, even though it's not used very much (clearly because not many know about it). It is, perhaps unsurprisingly, #microg on freenode.

The problem should be fixed in the new version of microG GmsCore released yesterday, please try it and report back.


Don't show System Drive and Source Drive badges at the same time

From https://github.com/resin-io/etcher/issues/1136

Which one should we prefer though? @konmouz ?

I think there's a case to be made for keeping both: Doing so reinforces the meaning of "source drive" (or whatever you rename it to) so that the user has a reasonable means of understanding that label.

That's a good point as well. Up to @konmouz I guess.

@jviotti I am not sure if I am the better to answer this, it is a bit technical. Is source drive a boot drive? We need to find a name that represents this drive and fits in one label.

@konmouz The source drive is the drive that contains the image that you previously selected (not necessarily a system drive)

@jviotti, ok I was completely out of topic. Now it makes sense why you might need two. So, since a 'source' drive is only one at a time, it should have a different color label (for instance orange). Also, may worth renaming to 'image source' and of course the drive would be greyed out. Maybe in that sense, we don't need the second label even if the drive is system or boot drive.

Perfect, I'll close this issue then!

Now it makes sense why you might need two. (...)
Maybe (...) we don't need the second label even if the drive is system or boot drive.

I'm not quite sure I follow. Which "second label" are you suggesting we omit @konmouz ?

Re-opening this issue...


[Gekidou] Add Latex support

Summary
This is a port of the recently included Latex support in v1 https://github.com/mattermost/mattermost-mobile/pull/5651
Ticket Link
https://mattermost.atlassian.net/browse/MM-43510
NONE

LGTM, blocking just because of the amount of comments.
I also took a look to the components/markdown/markdown.tsx file and I see potential improvements in some renderers (passing styles as list that will re-render, and so on), but I would prefer to do a more in depth analysis later if we ever find any performance with this.

Are we sure we are not over optimizing?

Are we sure we are not over optimizing?

I would like to look deeper into it, just that. There is no need to do anything now, but I would like at some point see if the parent memo is doing good enough job, and there is no need to optimize anything in the insides (which would lead to remove many memos and callbacks we might be using) or if the parent memo is not good enough, and it requires some tweaking. Not my objective to overoptimize, just to make sure the current optimizations are good enough (or even needed).

Yes, that makes sense.. I'm going to run some profiling on this


Leaving a Post

When viewing a post, after hitting the search icon over the image and then pressing cancel, the entire top bar disappears so the only way to go back to the previous page is by swiping right.

Fixed


accept-language header overriding quey parameter and other methods to set the local

Getting www.test.com?lang=en with the header accept-language="fr" will be displayed in "fr".
I don't know if this is intended, but overriding user preference seems odd to me.
It seems related to #36

Why would ?lang=en be a user preference ?
We should think of a ranking system, but it seems weird to have a user adding himself ?lang=en in his bar. You could use cookie or another way to do that

The term "user preference" is poorly choosen and the way to set this preference has nothing to do with the issue.
The issue is that the result of GET http://www.test.com?lang=en may be in an other language than specified by the url.
This could be really confusing. Visible settings should have the priority on invisible ones.

I agree with that but I'm struggling with time :/.


Cohort Founders Workshop

As part of extended onboarding for cohort founders, this workshop's objective would be for volunteers to understand how to be effective Cohort Founders and understand the primary objectives of CYF and the programme.
Question: would this workshop occur before, during or after CF selection/sign-up?

I think this is a @Dedekind561 and @MarianaVazquezCYF project, amirite?

This isn't to say we necessarily accept the premise here and just execute, but understand this as a request or description of a problem (and you may have a different solution)

@Dedekind561, @LaraHuzjan and I did a bit of a brainstorming on this which is on this Miro board + screenshot.
This workshop should be an in-person (or virtual) to set the tone for cohort founders.
Each box needs to be detailed in further tickets.

What is the priority on this?
Who should be the owner of this?

Tagging @lilianabermudes too since she had done a lot of cohort work before, if I remember right.


Increase lifetime from 2h to 4h for compatibility with Aura.Auth

The lifetime value interferes with the default value in Aura\Auth\Session\Timer. It will throw an exception.

Thanks for your PR. Although I'm not sure we need to change the default value. Configuration allows you to do this and there's no direct link between this middleware and Aura.Auth.

It was an issue I was running into sometimes and I had trouble tracking it down. ("Why is the default value configured in php.ini suddenly different?") The reason is the unconditional setting of cookie parameters which are incompatible to Aura.Auth's.
I would argue that some kind of change is necessary, espacially since this middleware is mentioned in the official Expressive documentation.
Perhaps a note in the documentation? Or the values get set to the values in php.ini?

You can fully customise the default options by creating a session.global.php config file under <project_root>/config/autoload i.e. /<project_root>/config/autoload/session.global.php
All of the default options can be overridden via;
<?php

return [
    'session' => [
        'name'     => 'PHPSESSID',
        'lifetime' => 7200,
        'path'     => null,
        'domain'   => null,
        'secure'   => false,
        'httponly' => true,
    ],
];

If you wish to use the session.cookie_lifetime from php.ini, consider the following;
<?php

return [
    'session' => [
        'name'     => 'PHPSESSID',
        'lifetime' => ini_get('session.cookie_lifetime'),
        'path'     => null,
        'domain'   => null,
        'secure'   => false,
        'httponly' => true,
    ],
];


Android Test version 2.3.00b

Hi,
I created a first test build with the current scope of 2.3, and a first crashlytics integration.
Crashlytics is a tool that will report crash logs when a crash happen.
It works only for Android ... unfortunately. (I found other crash report system cross platform but they were not for free).
So if you are an android user, please download and try to crash the app to see if it works as expected :)
You can also test the bug fixes and the Horrific journey integration while you are it.
Here is the link to the build: https://we.tl/t-J6LNh8uEuZ
Scope:

Add crashlytics (with no default configuration)
#1112  Trigger = DefeatCustomMonster during Horror Phase
Update to Unity 2018.2.21f1
Synopsis bug: languages change not applied #1106
MoM - Monsters being set to Spirits when Physical (and visa versa) #1057
#1101 Add french translation for HJ
Trapdoor token + Bechman fix + Edna Hughes fix (#1099)
Horrific Journeys integration (#1101)

Windows versions and other are coming soon (hopefully this week end).

I will test this today hopefully

One issue right off the bat!

So you can call me as soon as you find a bug :)
I think you can deny it and it will still works.
I don't know why Crashlytics would require that though.

I did deny it and it did work, but might make people paranoid. Does crashlytics have an SMS feature? There has to be a way to turn off call app permissions.
Import took just under 3 minutes on my Galaxy S7 edge btw. So will a little slow but not the end of the world.
I would recommend putting a warning on the import screen along the lines of 'please keep you screen active during import". As the import can hang if you tab out or let your phone screen sleep.

READ_PHONE permissions are required to read device serials, that probably the reason Crashlytics needs them.

After a bit of digging, it's probably because Carshlytics does not set a minSDK.
https://issuetracker.unity3d.com/issues/android-read-phone-state-permission-is-added-in-the-androidmanifest-dot-xml-file-using-latest-26-dot-0-2-sdk

How are we getting along here?
I've been testing my scenarios on the android version and all is good from what I can see (other than the phone permissions topic i raised).
Any bugs introduced that are being worked on or any progress with the crashlytics integration?
Just curious if we can push this to live, since it's a relatively small update.

I don't receive much information yet.
I made some changes already to be able to get the Valkyrie logs with the crash report.
I will now add some logs in most import actions, to know where the crash is happening more or less.
Nothing really blocking an official version release though so I fix we can go live as soon as I'm done with this.
Maybe we should just take one day or two to test the windows version too.

We also have #1102 where I've done no change yet.

No worries @BenRQ appreciate you working on it! I'm just getting over excited about incorporating some of the HJ tiles into my scenarios.

Problem with Android permissions for phone calls is now fixed, it was related to SDK version not set for Crashlytics.

I will close this issue as I have enough information to prepare next version.


Cyber_ Visualizer cannot display the point cloud view correctly

When Cyber_ visualizer displays the point clouds (using apollo_2.0_camera_sample.record transformed form apollo_2.0_camera_sample.bag), it shows a white line, and there is no response to the mouse dragging and zooming. How to solve this problem?
Besides, how to display the obstacle detection box bound? (like the previous offline visualization tool)
Thank you!
System information

OS Platform and Distribution (inux Ubuntu 18.04):
Apollo installed from (source ):
Apollo version (6.0):
Output of apollo.sh config if on master branch**:

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:

Supporting materials (screenshots, command lines, code/script snippets):

I also have the same question

您好，请问您的问题解决了吗？

您好，请问您的问题解决了吗？

Hello. I got the same problem today.  Have you solved it?

Hello. I got the same problem today. Have you solved it?

I solved it by restarting the computer.


Update tapir-core, tapir-http4s-server, ... to 1.10.14

About this PR
📦 Updates

com.softwaremill.sttp.tapir:tapir-core
com.softwaremill.sttp.tapir:tapir-http4s-server
com.softwaremill.sttp.tapir:tapir-json-circe
com.softwaremill.sttp.tapir:tapir-openapi-docs
com.softwaremill.sttp.tapir:tapir-swagger-ui
com.softwaremill.sttp.tapir:tapir-swagger-ui-bundle

from 1.10.13 to 1.10.14
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.tapir" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.softwaremill.sttp.tapir" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #1009.


Fixed pointer dereference to long long warning.

See http://stackoverflow.com/questions/9251102/warning-cast-to-pointer-from-integer-of-different-size-wint-to-pointer-cast for context and explanation of why this solution was chosen.


输入第一条指令的时候显示warning怎么解决

WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1125)'))': /simple/opencv-python/
WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1125)'))': /simple/opencv-python/
WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1125)'))': /simple/opencv-python/
WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1125)'))': /simple/opencv-python/
WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1125)'))': /simple/opencv-python/
Could not fetch URL https://pypi.org/simple/opencv-python/: There was a problem confirming the ssl certificate: HTTPSConnectionPool(host='pypi.org', port=443): Max retries exceeded with url: /simple/opencv-python/ (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1125)'))) - skipping
ERROR: Could not find a version that satisfies the requirement opencv-python
ERROR: No matching distribution found for opencv-python
Could not fetch URL https://pypi.org/simple/pip/: There was a problem confirming the ssl certificate: HTTPSConnectionPool(host='pypi.org', port=443): Max retries exceeded with url: /simple/pip/ (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1125)'))) - skipping

https://pypi.org/simple/opencv-python/ 这个链接能打开么？

知道了 忘记关VPN了


Issue with recent react version ?

If anyone is having issue with react version they may use this package over npm.
https://github.com/KunwarVSuryavanshi/ReactToPDF

v1 has been released supporting react > 16.8.


stargate-mongoose performance testing

@amorton / @kathirsvn can you add some description?

@vkarpov15 Are there existing performance tests for Mongoose that we could leverage? Or feel free to add other recommendations on approach.
The basic goal is to validate that our driver isn't adding a lot of overhead on top of the underlying API.

@jeffreyscarpenter Mongoose's benchmarks are fairly minimal: https://github.com/Automattic/mongoose/blob/master/benchmarks/mapOfSubdocs.js . We haven't prioritized building out a suite of benchmarks yet.

@vkarpov15 do you want to add links to the tests you performed? We can probably close this ticket after that.

Sure, here's the github gist containing source files and results: https://gist.github.com/vkarpov15/70b4a87b4d2d369699ddf04f202547ab


It stopped working on firefox

It just stopped working, reinstalling, restarting, etc. doesn't help

I literally just made an account to post about this, also my "show more" and "share" buttons disappeared before this too. Is this just Google trying to get rid of plugins like this or does this need to be updated?

Same issue 🗡

Yep, same here. Hopefully there will be an update that fixes this because the new layout is garbage. Also, before it broke, "show more" button broke beforehand.

Can confirm, new YouTube is annoying :(

New layout actually crashes my tab in Subscriptions section, need solution.

Don't blame this extension, blame Youtube/Google. The company did warn us the old design was going away at some point. If you thought Classic-Youtube was somehow recreating the classic Youtube experience, that's a wrong assumption, it was merely fooling the Youtube server in to delivering the old design to your browser.

I installed this addon:
https://addons.mozilla.org/en-US/firefox/addon/uaswitcher/
And overrode my user-agent for *.youtube.com to that of Pale Moon (which is based on an ancient version of Firefox) as such:

Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:25.8) Gecko/20151126 Firefox/31.9 PaleMoon/25.8.1

Works now.

Setting the User-Agent to any version of FF of 43 or below seems to work for now, albeit displaying a banner:

On Firefox versions under 71, this can be accomplished easily with about:config flag general.useragent.override.youtube.com=Mozilla/5.0 (Windows NT 6.1; Win64; rv:43.0) Gecko/20100101 Firefox/43.0
Can't wait for Google to force the 5x slower Polymer down the throats of those who need XUL/XPCOM extensions :)))))))

It stopped working on Chrome as well.

@ReimuHakurei @o5k Thanks for the UAS spoofing tip. That workaround works on my end only while being logged-out, though, while being logged-in it still displays the new Layout. Do you have the same problem, or is there a workaround for this too?

It works when logged for me, so not sure.

With the spoofing extension and Youtube Classic extension activated it still displays the new layout and it crashes all the time, too. Super annoying.
Now comes the really strange thing.
Without any Youtube Classic/UAS spoof I can switch into another channel on the same email and it will show me the old layout again. How does this happen?
main account with new layout: http://puu.sh/FWPhC/2d397606fc.jpg
alt account with old layout: http://puu.sh/FWPim/29356f7dc4.jpg
So if you don't mind not having your playlists available, try switching to another channel if you got multiple.

This can be fixed (for now) by doing the following:

Enable cookies on YouTube (if you don't already)
Open Developer Tools (F12)
Go to Storage > Cookies
Look for the PREF setting and add "&f6=8"

I suspect this functionality will eventually be removed though.

@hejops this also does not work for me, i'm on Firefox 66.

Hello, it has been updated to 1.5.4, but it's time to let it go at this point. Old layout will be gone soon.

Basically, it means that the module seems to be corrupted.

@xxzefgh
Thank you for the effort and great Add-On over time.
Unfortunately the update it only works when I use Youtube without Account.
As soon I Log-In, I get the new Layout.
So I tried Internet Explorer which loads the old Layout by default. As soon I Log-In, I get the New Layout which is not even made for IE.
Like they store now the information in Account that the user is capable to use the New Layout.
Nothing we can do I think :(

The issues while being logged-in only occur with my primary channel, with my other ones the old layout persists also while logged-in. Spoofing the UAS unfortunately deactivates the Chat box (Live + replay) and search recommendations underneath the search bar.

@xxzefgh The latest update has fixed it for me, so thanks for the fix!
However, why you want to let it go? The YouTube has switched to this layout more than 2 years ago, and since we know, how bad YouTube are at these things, maybe it will be another six months, until they remove it completely.
Also, please release the fix for the Chromium browsers as well, since there I still have the new layout, and I mostly use Ungoogled/Iridium for the YouTube.

@xxzefgh
Thank you for the effort and great Add-On over time.
Unfortunately the update only works when I use Youtube without Account.
As soon I Log-In, I get the new Layout.
So I tried Internet Explorer which loads the old Layout by default. As soon I Log-In, I get the New Layout which is not even made for IE.
Like they store now the information in Account that the user is capable to use the New Layout.
Nothing we can do I think :(
(Firefox 77.0.1 x64, Windows 10 x64 2004)

It also doesn't work for me and I thought they might put a cookie or something in your account that forces the new layout if you had it once.

For everyone having bizarre issues, try clearing your cookies for Youtube and updating your Youtube Classic add-on.


Unit 7 - protocol buffers issue

in ubuntu environment : ubuntu 20.04.5 LTS (Release : 20.04, Codename: focal)
the following error  appears when executing  mlagents-learn
the same error
=============
mlagents-learn ./config/poca/SoccerTwos.yaml --env=./training-envs-executables/SoccerTwos.x86_64 --run-id="SoccerTwos" --no-graphics
Traceback (most recent call last):
File "/home/isaac/anaconda3/envs/rl/bin/mlagents-learn", line 33, in
sys.exit(load_entry_point('mlagents', 'console_scripts', 'mlagents-learn')())
File "/home/isaac/anaconda3/envs/rl/bin/mlagents-learn", line 25, in importlib_load_entry_point
return next(matches).load()
File "/home/isaac/anaconda3/envs/rl/lib/python3.9/importlib/metadata.py", line 86, in load
module = import_module(match.group('module'))
File "/home/isaac/anaconda3/envs/rl/lib/python3.9/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1030, in _gcd_import
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "", line 850, in exec_module
File "", line 228, in _call_with_frames_removed
File "/home/isaac/ml-agents/ml-agents/mlagents/trainers/learn.py", line 2, in
from mlagents import torch_utils
File "/home/isaac/ml-agents/ml-agents/mlagents/torch_utils/init.py", line 1, in
from mlagents.torch_utils.torch import torch as torch  # noqa
File "/home/isaac/ml-agents/ml-agents/mlagents/torch_utils/torch.py", line 6, in
from mlagents.trainers.settings import TorchSettings
File "/home/isaac/ml-agents/ml-agents/mlagents/trainers/settings.py", line 25, in
from mlagents.trainers.cli_utils import StoreConfigFile, DetectDefault, parser
File "/home/isaac/ml-agents/ml-agents/mlagents/trainers/cli_utils.py", line 5, in
from mlagents_envs.environment import UnityEnvironment
File "/home/isaac/ml-agents/ml-agents-envs/mlagents_envs/environment.py", line 12, in
from mlagents_envs.side_channel.side_channel import SideChannel
File "/home/isaac/ml-agents/ml-agents-envs/mlagents_envs/side_channel/init.py", line 5, in
from mlagents_envs.side_channel.default_training_analytics_side_channel import (  # noqa
File "/home/isaac/ml-agents/ml-agents-envs/mlagents_envs/side_channel/default_training_analytics_side_channel.py", line 7, in
from mlagents_envs.communicator_objects.training_analytics_pb2 import (
File "/home/isaac/ml-agents/ml-agents-envs/mlagents_envs/communicator_objects/training_analytics_pb2.py", line 35, in
_descriptor.FieldDescriptor(
File "/home/isaac/anaconda3/envs/rl/lib/python3.9/site-packages/google/protobuf/descriptor.py", line 561, in new
_message.Message._CheckCalledFromGeneratedFile()
TypeError: Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:

Downgrade the protobuf package to 3.20.x or lower.
Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates
=============
And if i launch mlagents-learn without any parameters, i get also :
Traceback (most recent call last):
File "/home/isaac/anaconda3/envs/rl/bin/mlagents-learn", line 33, in
sys.exit(load_entry_point('mlagents', 'console_scripts', 'mlagents-learn')())
File "/home/isaac/anaconda3/envs/rl/bin/mlagents-learn", line 25, in importlib_load_entry_point
return next(matches).load()
File "/home/isaac/anaconda3/envs/rl/lib/python3.9/importlib/metadata.py", line 86, in load
module = import_module(match.group('module'))
File "/home/isaac/anaconda3/envs/rl/lib/python3.9/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1030, in _gcd_import
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "", line 850, in exec_module
File "", line 228, in _call_with_frames_removed
File "/home/isaac/ml-agents/ml-agents/mlagents/trainers/learn.py", line 2, in
from mlagents import torch_utils
File "/home/isaac/ml-agents/ml-agents/mlagents/torch_utils/init.py", line 1, in
from mlagents.torch_utils.torch import torch as torch  # noqa
File "/home/isaac/ml-agents/ml-agents/mlagents/torch_utils/torch.py", line 6, in
from mlagents.trainers.settings import TorchSettings
File "/home/isaac/ml-agents/ml-agents/mlagents/trainers/settings.py", line 25, in
from mlagents.trainers.cli_utils import StoreConfigFile, DetectDefault, parser
File "/home/isaac/ml-agents/ml-agents/mlagents/trainers/cli_utils.py", line 5, in
from mlagents_envs.environment import UnityEnvironment
File "/home/isaac/ml-agents/ml-agents-envs/mlagents_envs/environment.py", line 12, in
from mlagents_envs.side_channel.side_channel import SideChannel
File "/home/isaac/ml-agents/ml-agents-envs/mlagents_envs/side_channel/init.py", line 5, in
from mlagents_envs.side_channel.default_training_analytics_side_channel import (  # noqa
File "/home/isaac/ml-agents/ml-agents-envs/mlagents_envs/side_channel/default_training_analytics_side_channel.py", line 7, in
from mlagents_envs.communicator_objects.training_analytics_pb2 import (
File "/home/isaac/ml-agents/ml-agents-envs/mlagents_envs/communicator_objects/training_analytics_pb2.py", line 35, in
_descriptor.FieldDescriptor(
File "/home/isaac/anaconda3/envs/rl/lib/python3.9/site-packages/google/protobuf/descriptor.py", line 561, in new
_message.Message._CheckCalledFromGeneratedFile()
TypeError: Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:

Downgrade the protobuf package to 3.20.x or lower.
Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates

You can have a look at:
https://stackoverflow.com/questions/72441758/typeerror-descriptors-cannot-not-be-created-directly
downgrade protobuf:
pip install protobuf==3.20.*

I'm closing this issue since the MLAgents update resolved this


Push to OCI insecure registry

What this PR does / why we need it:
This PR implement push chart to OCI registry, or pull chart from OCI registry.
Fixes: https://github.com/helm/helm/issues/6324
Usage:
# push
$ ./bin/helm push consul-3.9.5.tgz oci://docker.registry:30001/library --insecure-skip-tls-verify

# pull
$ ./bin/helm pull oci://docker.registry:30001/library/consul --version 3.9.5 --insecure-skip-tls-verify

# install
$ ./bin/helm install consul oci://docker.registry:30001/library/consul --version 3.9.5 --insecure-skip-tls-verify

# upgrade
$ ./bin/helm upgrade --install consul oci://docker.registry:30001/library/consul --version 3.9.6 --insecure-skip-tls-verify

Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[x] this PR has been tested for backwards compatibility

Update push/pull chart to oci insecure registry from the latest code, because oci feature move out experimental.

Waiting for this fix to implement OCI based image and helm repository using Harbor.

I will commit helm install/upgrade code when use insecure OCI registry or self-certificate registry on the next week, there are some problems with the code need to be update in my local develop branch.

Now this  pr supports  helm install/upgrade/pull/push with self-signed certificate OCI registry.
@jdolitsky please review the code if you have time, thanks.

@pytimer - can you please fix the DCO error?

I believe this is in conflict with https://github.com/helm/helm/pull/10638 ...
@antgamdia - once @pytimer fixes DCO, would you be able to push on top of this PR/fork? I think ClientOptResolver if preferable over the WithResolver proposed here.
@pytimer - let us know if this sounds OK to you

@pytimer - can you please fix the DCO error?

I will fix it today.

@antgamdia - once @pytimer fixes DCO, would you be able to push on top of this PR/fork? I think ClientOptResolver if preferable over the WithResolver proposed here.

OK. If need me to do anything, please tell me, thanks.

.antgamdia - once .pytimer fixes DCO, would you be able to push on top of this PR/fork? I think ClientOptResolver if preferable over the WithResolver proposed here.

Sure! Will do during next week

.antgamdia - once .pytimer fixes DCO, would you be able to push on top of this PR/fork? I think ClientOptResolver if preferable over the WithResolver proposed here.

Sure! Will do during next week

Done! See my comments at https://github.com/helm/helm/pull/10638#issuecomment-1059426505

/cc @jdolitsky @scottrigby this looks good for review again

The helm pull use insecure-skip-tls-verify when use chart repository, so use this flag to be consistent in helm push/pull

@jdolitsky can you please review the changes to this PR?

come on!

Nice to see this PR finally approved! Do you think you can also squeeze this one https://github.com/helm/helm/pull/11129? cc @jdolitsky 🙏

cant wait for this fix anymore......please merge asap

helm 3.10.0 and after version

Upload a chart to a registry.
If the chart has an associated provenance file,
it will also be uploaded.
Usage:
helm push [chart] [remote] [flags]
Flags:
-h, --help   help for push
Global Flags:
--burst-limit int                 client-side default throttling limit (default 100)
--debug                           enable verbose output
--kube-apiserver string           the address and the port for the Kubernetes API server
--kube-as-group stringArray       group to impersonate for the operation, this flag can be repeated to specify multiple groups.
--kube-as-user string             username to impersonate for the operation
--kube-ca-file string             the certificate authority file for the Kubernetes API server connection
--kube-context string             name of the kubeconfig context to use
--kube-insecure-skip-tls-verify   if true, the Kubernetes API server's certificate will not be checked for validity. This will make your HTTPS connections insecure
--kube-tls-server-name string     server name to use for Kubernetes API server certificate validation. If it is not provided, the hostname used to contact the server is used
--kube-token string               bearer token used for authentication
--kubeconfig string               path to the kubeconfig file
-n, --namespace string                namespace scope for this request
--registry-config string          path to the registry config file (default "/root/.config/helm/registry/config.json")
--repository-cache string         path to the file containing cached repository indexes (default "/root/.cache/helm/repository")
--repository-config string        path to the file containing repository names and URLs (default "/root/.config/helm/repositories.yaml")
but not find   -insecure-skip-tls-verify

3.10.3 version， helm push not find -insecure-skip-tls-verify

@dreamerkr this PR was not merged yet so it did not make it into the 3.10 release

@dreamerkr this PR was not merged yet so it did not make it into the 3.10 release

now，How to solve this problem? the best solution use https? but I only want try it

@dreamerkr currently, yes only an HTTPS expository is supported. As mentioned, there are ongoing efforts to bring the plain HTTP based integrations

Can we get a release with this PR merged please?

Could we get this feature merged? It has been here for a long time. We need this feature. Thanks.

Tested v3.11.3
helm registry works with --insecure
helm push/pull works with --insecure-skip-tls-verify

Tested v3.11.3 helm registry works with --insecure helm push/pull works with --insecure-skip-tls-verify

still not work in v3.11.3
root@ubuntu:~# helm version
version.BuildInfo{Version:"v3.11.3", GitCommit:"323249351482b3bbfc9f5004f65d400aa70f9ae7", GitTreeState:"clean", GoVersion:"go1.20.3"}

login
root@ubuntu:~# helm registry login --insecure sealos.hub:5000 -u admin -p passw0rd
WARNING: Using --password via the CLI is insecure. Use --password-stdin.
INFO[0000] Error logging in to endpoint, trying next endpoint  error="Get \"https://sealos.hub:5000/v2/\": http: server gave HTTP response to HTTPS client"
Login Succeeded
root@ubuntu:~#
root@ubuntu:~#
root@ubuntu:~# helm push nginx-13.2.34.tgz oci://sealos.hub:5000/library --insecure-skip-tls-verify
Error: failed to do request: Head "https://sealos.hub:5000/v2/library/nginx/blobs/sha256:d5f440f331d6ce10a5af6c0ce82633dc49217b8f04531d5a27143ff51afe07a2": http: server gave HTTP response to HTTPS client
root@ubuntu:~#

Tested v3.11.3 helm registry works with --insecure helm push/pull works with --insecure-skip-tls-verify

still not work in v3.11.3
root@ubuntu:~# helm version
version.BuildInfo{Version:"v3.11.3", GitCommit:"323249351482b3bbfc9f5004f65d400aa70f9ae7", GitTreeState:"clean", GoVersion:"go1.20.3"}

login
root@ubuntu:~# helm registry login --insecure sealos.hub:5000 -u admin -p passw0rd
WARNING: Using --password via the CLI is insecure. Use --password-stdin.
INFO[0000] Error logging in to endpoint, trying next endpoint  error="Get \"https://sealos.hub:5000/v2/\": http: server gave HTTP response to HTTPS client"
Login Succeeded
root@ubuntu:~#
root@ubuntu:~#
root@ubuntu:~# helm push nginx-13.2.34.tgz oci://sealos.hub:5000/library --insecure-skip-tls-verify
Error: failed to do request: Head "https://sealos.hub:5000/v2/library/nginx/blobs/sha256:d5f440f331d6ce10a5af6c0ce82633dc49217b8f04531d5a27143ff51afe07a2": http: server gave HTTP response to HTTPS client
root@ubuntu:~#

Ah, yep you're correct.  I confused this with a server using a self-signed certificate

root@ubuntu:/home/cyxinda/workspace/test# helm version
version.BuildInfo{Version:"v3.12.0", GitCommit:"c9f554d75773799f72ceef38c51210f1842a1dea", GitTreeState:"clean", GoVersion:"go1.20.3"}
root@ubuntu:/home/cyxinda/workspace# helm push test-0.1.0.tgz oci://harbor.<IP_ADDRESS>.nip.io/my --ca-file /etc/containerd/certs.d/harbor.<IP_ADDRESS>.nip.io/ca.crt --cert-file  /etc/containerd/certs.d/harbor.<IP_ADDRESS>.nip.io/harbor.<IP_ADDRESS>.nip.io.cert --key-file  /etc/containerd/certs.d/harbor.<IP_ADDRESS>.nip.io/harbor.<IP_ADDRESS>.nip.io.key
Pushed: harbor.<IP_ADDRESS>.nip.io/my/test:0.1.0
Digest: sha256:15244ea1d01d8d7f4617252687c8c143ef543d61c9e13657ef30139258b8944d

root@ubuntu:/home/cyxinda/workspace/test# helm pull oci://harbor.<IP_ADDRESS>.nip.io/my/test --version 0.1.0 --ca-file /etc/containerd/certs.d/harbor.<IP_ADDRESS>.nip.io/ca.crt --cert-file  /etc/containerd/certs.d/harbor.<IP_ADDRESS>.nip.io/harbor.<IP_ADDRESS>.nip.io.cert --key-file  /etc/containerd/certs.d/harbor.<IP_ADDRESS>.nip.io/harbor.<IP_ADDRESS>.nip.io.key
Pulled: harbor.<IP_ADDRESS>.nip.io/my/test:0.1.0
Digest: sha256:15244ea1d01d8d7f4617252687c8c143ef543d61c9e13657ef30139258b8944d
root@ubuntu:/home/cyxinda/workspace/test# ls
test-0.1.0.tgz

@pytimer Can you please rebase?

When will this be merged? We actually need this badly.

@BesartSulejmani Capabilities should now be available
https://github.com/helm/helm/pull/12128
We should be good to close this PR as I believe that everything has been covered? @mattfarina @joejulian @gjenkins8


Regen with new model flatten logic

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check for Azure.ResourceManager.Resources
API changes are not detected in this pull request for Azure.ResourceManager.Resources

API change check for Azure.ResourceManager.DnsResolver
API changes have been detected in Azure.ResourceManager.DnsResolver. You can review API changes here
API changes
-         public ResourceIdentifier VirtualNetworkLinkId { get; set; }
+         public ResourceIdentifier VirtualNetworkLinkId { get; }

API change check for Azure.ResourceManager.EventHubs
API changes are not detected in this pull request for Azure.ResourceManager.EventHubs

API change check for Azure.ResourceManager.CosmosDB
API changes are not detected in this pull request for Azure.ResourceManager.CosmosDB

API change check for Azure.ResourceManager.Compute
API changes have been detected in Azure.ResourceManager.Compute. You can review API changes here

API change check for Azure.ResourceManager.ServiceBus
API changes are not detected in this pull request for Azure.ResourceManager.ServiceBus

API change check for Azure.ResourceManager.AppConfiguration
API changes have been detected in Azure.ResourceManager.AppConfiguration. You can review API changes here
API changes
-         public ResourceIdentifier PrivateEndpointId { get; set; }
+         public ResourceIdentifier PrivateEndpointId { get; }

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

API change check for Azure.ResourceManager.KeyVault
API changes are not detected in this pull request for Azure.ResourceManager.KeyVault

API change check for Azure.ResourceManager.ConnectedVMwarevSphere
API changes are not detected in this pull request for Azure.ResourceManager.ConnectedVMwarevSphere

API change check for Azure.ResourceManager.Monitor
API changes are not detected in this pull request for Azure.ResourceManager.Monitor

API change check for Azure.ResourceManager.EdgeOrder
API changes have been detected in Azure.ResourceManager.EdgeOrder. You can review API changes here
API changes
-         public TransportShipmentTypes TransportPreferencesPreferredShipmentType { get; set; }
+         public TransportShipmentTypes? TransportPreferencesPreferredShipmentType { get; set; }

API change check for Azure.ResourceManager.Cdn
API changes are not detected in this pull request for Azure.ResourceManager.Cdn

API change check for Azure.ResourceManager.Sql
API changes have been detected in Azure.ResourceManager.Sql. You can review API changes here
API changes
-         public ResourceIdentifier PrivateEndpointId { get; set; }
+         public ResourceIdentifier PrivateEndpointId { get; }
-         public ResourceIdentifier PrivateEndpointId { get; set; }
+         public ResourceIdentifier PrivateEndpointId { get; }

API change check for Azure.ResourceManager.WebPubSub
API changes are not detected in this pull request for Azure.ResourceManager.WebPubSub

API change check for Azure.ResourceManager.AppService
API changes are not detected in this pull request for Azure.ResourceManager.AppService

API change check for Azure.ResourceManager.Network
API changes have been detected in Azure.ResourceManager.Network. You can review API changes here

API change check for Azure.ResourceManager.Storage
API changes have been detected in Azure.ResourceManager.Storage. You can review API changes here

API change check for Azure.ResourceManager.IotCentral
API changes have been detected in Azure.ResourceManager.IotCentral. You can review API changes here
API changes
-         public AppSku SkuName { get; set; }
+         public AppSku? SkuName { get; set; }
-         public AppSku SkuName { get; set; }
+         public AppSku? SkuName { get; set; }

API change check for Azure.ResourceManager.ArcScVmm
API changes are not detected in this pull request for Azure.ResourceManager.ArcScVmm

API change check for Azure.ResourceManager.Reservations
API changes have been detected in Azure.ResourceManager.Reservations. You can review API changes here
API changes
-         public string SkuName { get; set; }
+         public string SkuName { get; }

API change check for Azure.ResourceManager.ServiceLinker
API changes are not detected in this pull request for Azure.ResourceManager.ServiceLinker

API change check for Azure.ResourceManager.ElasticSan
API changes are not detected in this pull request for Azure.ResourceManager.ElasticSan


Include imports for Request configuration classes  and model classes for snippets that make use of the same

When a snippet makes use of a model class, or a request configuration class, not having these in in an include line in the snippets throws an error.
Customers have to go through the painful experience of manually looking for the appropriate path and including or relying on IDE hints to fix the errors until all needed imports are included:

[ ] Include path to the model class or namespace to solve the missing class error.
[ ] Include path to request configuration class for models that use request configuration  to remove missing class error and enhance copy  and paste experience.
[ ] Change Raptor import mechanism  so that Raptor tests rely on imports within the snippets

@shemogumbe Thanks for the offer, could you please align with Kenneth on fixing it this Sprint if possible?

@shemogumbe to close this issue and link it to the closed PR.


make it compile as a C++ program

currently uses some C-isms, but minor changes should make this compile with a C++ compiler too.

Can I try this?

Yes feel free to give this a try.
The big issue right now is void * casts and a few other type errors because C++ is stricter than C with these. Other than that, most of it should be fine as I've tried to avoid most  C specific stuff.
One C specific feature I have used is flexible array members. These are offered as an extension to C++ on gcc and clang (i believe) so it's fine for now if they are still used.
You can work off of the v0.4.3 branch and do a PR to that branch once it's done.

Perhaps should be closed?

I'll close this once I've finished the tests for this, I haven't forgotten the issue.

I'm gonna close this an open another for the testing acrtually.


Aspect runs on all dependencies and not only built target

One issue I encounter in our codebase is we have a lot of generated code (e.g. protobufs) and third party code that we frankly don't want to run clang-tidy on. While the lints it catches are legitimate, we won't have time to go through all the dependencies and fix their clang-tidy bugs. We need to focus on our own codebase. However, when I build a target using the flags specified, I get errors on all dependencies, not just on the one that is being built.
Having the aspect only run on the specific target being built makes it a lot easier to be selective about what you want to run clang tidy on in CI. This is how the rust clippy aspect. Could we change the behaviour to work that way?

Thanks for the report. I think into this point:
https://github.com/erenon/bazel_clang_tidy/blob/master/clang_tidy/clang_tidy.bzl#L132
The following should be inserted:
    # Ignore external targets
    if target.label.workspace_root.startswith("external"):
        return None

https://github.com/bazelbuild/rules_rust/blob/main/rust/private/clippy.bzl#L59C1-L61C20
This would ignore the external deps. Can you make a PR?
I don't know yet how to ignore locally generated protobufs.

Thanks @erenon! Let me look into how to do it. The more I've looked into this, the more I realise it might also be because it's running on the generated header files. Do you have a mechanism for filtering out header files?

Should be fixed by #47, please reopen if not.


Sampler minor fix: no reason to send all exec_flags to all processes

As the title suggests, we can just send each process its own execution flag separately, rather than sending the whole list.
Very nice repo BTW :)

You're right. Thanks!


colVis with ColReorder example is broken

https://datatables.net/extensions/colreorder/examples/integration/colvis.html
Move columns and then hide one of the columns that you moved.
The column that you click is not the one that hides/shows.
Please fix this :D
Thanks!

It already is. The fix will be released as Buttons 1.2.2 in the next week or two. Until then use the nightly version.


Fix minor errors in AKS Fleet resource propagation doc

@phealy

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

PR review team emailed author today.

PR review team email author today

@phealy
Can you review this old PR and determine whether it needs to be closed or merged?
@MicrosoftDocs/public-repo-pr-review-team

@shashankbarsin
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@shashankbarsin
Can you respond to the comment above?
@MicrosoftDocs/public-repo-pr-review-team

@nojnhuh,
Thank you for contributing to this Azure documentation. Unfortunately, changes to Kubernetes Fleet content can't be accepted this week. The Skilling Content group is migrating this content to a new repository: azure-management-docs.
In the meantime, we are closing this PR and tagging it so that when the content is available in the new repo, we can follow up with new location information.
#label:"migrate-followup"
#please-close

@nojnhuh
The migration has been paused, and changes to Kubernetes Fleet content can be accepted for now. Thanks for your patience!
#remove-label:"migrate-followup"

The content migration has re-started.
#label:"migrate-followup"
#please-close

@nojnhuh
We’re happy to share that contributions to [folder/service] are once again possible at azure-management-docs. We appreciated your patience during this migration.
Find the article in the new repository and make the changes there (or copy/paste the raw markdown from your changed file) and open a new PR.
#remove-label:"migrate-followup"


⚠ Bofrost.be has degraded performance

In 5bf6ddf, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1314 ms

Resolved: Bofrost.be performance has improved in dfcec06 after 6 minutes.


stopRecorder not working

stopRecorder not working it shows already stopped working.
Version: 3.1.2
@hyochan

Same here

stopRecorder Function is working properly.
Outside the component, add const audioRecorderPlayer = new AudioRecorderPlayer(); to make this work. As a result, the component will appear like this.
for Functional Component:
const audioRecorderPlayer = new AudioRecorderPlayer();
const App = () => {
const [isLoggingIn, setIsLoggingIn] = useState(false);
return(
...
)
}
For Class component:
inside the constructor
constructor(props) {
super(props);
this.state = {
};
this.audioRecorderPlayer = new AudioRecorderPlayer();
}

@Billalkhn Thanks for the reply, it's working now. I have added this line
const audioRecorderPlayer = new AudioRecorderPlayer();
inside the function.

Same. I just copy from documentation

Please refer the link to get the solution audio https://stackoverflow.com/a/77155779/15204127


don't pin theme to a specific version

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@89da58a). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #6239   +/-   ##
=========================================
  Coverage          ?   83.28%
  Complexity        ?    12531
=========================================
  Files             ?     1451
  Lines             ?    47382
  Branches          ?     5806
=========================================
  Hits              ?    39462
  Misses            ?     5224
  Partials          ?     2696

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 89da58a...f736a3f. Read the comment docs.


feat: Credentials Authentication

This PR adds email / password authentication to create-t3-turbo using the NextAuth Credentials Provider.
Although NextJS discourages password authentication, for companies with an existing user base with existing credentials, it is difficult to not support password authentication. NextAuth also attempts to disable database sessions if using the Credentials Provider.
This PR demonstrates how to use the NextAuth Credential Provider to create JWT tokens and Database sessions.
The authorize method validates the email, hashes the password with bcrypt, compares password hashes, creates a database session, and returns the user object and sessionToken.
The jwt callback is used to add the sessionToken to the token object used to generate the JWT, tying the JWT to the database session, and validates the current JWT token against the database.
A signOut event is used delete the session from the database if the sessionToken is present in the JWT.
This should work with additional providers such as discord.

Sorry for the late response, but we don't wanna provide a credentials provider by default. You should know what pitfalls there are when adding password auth and providing a default isn't our responsibility.


How to do archchtecture search on supernet?

Hi, in your paper, you say "After training the supernet, we randomly sample 1,000 architectures from the supernet and finetune each of them for 3 epochs", how to do this? Constructing a network according to a randomly generated genotype_list and evaluate this network?

I believe you have got the correct understanding of it.

The released train codes are not for training super-net, right? I find it is hard to train super-net without detail setting. Could you please release the codes or configs for training super-net?

right, btw, how you split the validation and training set during architecture search? @gmh14

right, btw, how you split the validation and training set during architecture search? @gmh14

@815961618 @gmh14 Is there a way to obtain the trained super-net on any dataset?


Update README.md with additional command necessary for installation

When installing pocketbase with go version 'go1.22.0 linux/amd64' you get an error about not being able to find packages.  'go mod init xxxxx' is required before the go get command.
I chose pocketbase as the default 'go mod init' but it may make sense to let the reader know they could use something else, like 'example.com/pocketbase' or 'go mod init $MODULENAME'.

Thank you but I don't think this is necessary.
Actually the "Installation" section can be removed and replaced with the content from https://pocketbase.io/docs/go-overview/ (I'll consider updating it sometime later).