Force no split in make_rng

Discussed in https://github.com/google/flax/discussions/3113

Originally posted by zaccharieramzi May 24, 2023
I have the following situation: I am using a Dropout layer multiple times without a nn.scan or nn.while_loop, therefore I cannot use split_rngs={"dropout": False}.
However, I would still like to use the same dropout mask twice.
Is it possible to specify "no split" to make rng for certain collections?
If I just take the original dropout example I would like to do something like:
# Setup.
import jax
import jax.numpy as jnp
import flax.linen as nn

# Randomness.
seed = 0
root_key = jax.random.PRNGKey(seed=seed)
main_key, params_key, dropout_key = jax.random.split(key=root_key, num=3)

# A simple network.
class MyModel(nn.Module):
  num_neurons: int
  training: bool
  @nn.compact
  def __call__(self, x):
    x = nn.Dense(self.num_neurons)(x)
    # Set the dropout layer with a rate of 50% .
    # When the `deterministic` flag is `True`, dropout is turned off.
    x = nn.Dropout(rate=0.5, deterministic=not self.training)(x)
    x = nn.Dropout(rate=0.5, deterministic=not self.training)(x)
    return x

# Instantiate `MyModel` (you don't need to set `training=True` to
# avoid performing the forward pass computation).
my_model = MyModel(num_neurons=3, training=False)

x = jax.random.uniform(key=main_key, shape=(3, 4, 4))

# Initialize with `flax.linen.init()`.
# The `params_key` is equivalent to a dictionary of PRNGs.
# (Here, you are providing only one PRNG key.)
variables = my_model.init(params_key, x)

# Perform the forward pass with `flax.linen.apply()`.
my_model.training = True
y = my_model.apply(variables, x, rngs={'dropout': dropout_key})

and still have jnp.sum(y == 0.) / (3*4*3) == 0.5 approx.
For more context I am actually trying to implement Deep Equilibrium Models using jaxopt and flax, where the fixed point defining function uses dropout.
I also tried to see if the split_rngs functionality could be extended to jaxopt but I think it's going to be difficult.

Hey @zaccharieramzi, I've converted the discussion into and issue as it seems something that we should improve.
I've created #3114, which would allow you to optionally specify the rng key for each Dropout layer, e.g:
# A simple network.
class MyModel(nn.Module):
  num_neurons: int
  training: bool
  @nn.compact
  def __call__(self, x):
    x = nn.Dense(self.num_neurons)(x)
    # Set the dropout layer with a rate of 50% .
    # When the `deterministic` flag is `True`, dropout is turned off.
    key = self.make_rng('dropout')
    x = nn.Dropout(rate=0.5, deterministic=not self.training)(x, rng=key)
    x = nn.Dropout(rate=0.5, deterministic=not self.training)(x, rng=key)
    return

This way both layers will produce the same mask.

ofc I understand it might be way more complex to do, so it's really just a question

@cgarciae I see that this was closed so maybe you missed my earlier question. Typically in modules like dot_product_attention the dropout is hardcoded without the possibility to set the rng.
Do you think it's best then to reimplement all these modules with the possibility to pass the rng?

FYI @zaccharieramzi, I added a dropout_arg to nn.MultiHeadDotProductAttention in #3384 so you can get the same dropout mask


Avoid adding diagnostic for array and dictionary when the closing bracket is on a new line

as mentioned in the comment: https://github.com/apple/swift-syntax/pull/1573#discussion_r1201066026

@swift-ci please test


Fails completely if rendering into an environment without a DOM

I'm adding snapshot testing from storyshots (conceptually from Jest) which is failing due to not being in an environment with a DOM. In my instance it's failing because ReactDOM.findDOMNode is not a function inside of componentDidMount.
While I don't expect this to work in a non-DOM environment I would expect that it wouldn't fail and break my tests if in an environment it doesn't understand.

why not? As a very specifically DOM-grounded hoc, that's one thing I absolutely would expect. PRs welcome to fix that, but expecting it to work without a DOM from the outset doesn't jive =)

Yeah it's not ideal, but right now my only way to have tests is to conditionally wrap my component in the hoc if I'm running tests, which feels kinda hacky.
And again, I don't expect it to functionally work, I would just like some manner to prevent it from a hard failure in that scenario. I'd imagine shallow rendering with testutils or Enzyme would also encounter the same thing.

Cool, if you're fine with a try/catch or check for it then I'm happy to put together a PR.

yeah that's fine by me, or even an if (!ReactDOM.findDOMNode) { return early  }. As long as it does not impair the code from doing what it needs to do when there is a dom to work with, I'm open to solutions.

if that's the only way we can be sure we don't have a DOM, then that sounds fine - I'm a proponent of "solve a problem, then refine the approach when someone runs into edge cases with the solution"

v5.7.1 released


Question: How to resolve all instances implementing a common protocol?

At the moment, in a separate step, we resolve a specific set of protocols so they can perform additional configuration steps.
In the past when I developed on Windows I would just give all these services a common interface (protocol) that they would have to implement, and then I just had to get all instances implementing that interface. How would I go on reproducing such behaviour using Dip?

@mylemans in early days there was an idea to support multi-injection but we dropped it as the same result can be achieved with tags and map function (see https://github.com/AliSoftware/Dip/pull/39#issuecomment-177610238)


feat: support FS injection into the scaffold config

Description
This feature allows users to supply their own FS as part of the CLI configuration. This makes it easy for programmatic use of the scaffold generation tool.
scaffoldFS is of course not exposed as a CLI argument. It takes precedence over scaffoldDir, but since it cannot be populated via CLI we should be OK.
Open to suggestions on the implementation here, please review when you can!

@hay-kot  please take a look

Can you explain a little bit more on the problem you're trying to solve here? An example or two would be really helpful

Can you explain a little bit more on the problem you're trying to solve here? An example or two would be really helpful.

@hay-kot  so I'm actually wrapping this with a more opinionated CLI that will generate scaffolding based on a project structure I've embedded as part of the CLI binary itself, using the standard embed library. In this scenario, I'm not shelling out at all and I'd like to just be able to pass the FS handle to the underlying scaffolding code. The scaffolding templates do not exist on the local filesystem, they are part of the binary.

Ahh, I see. It sounds like from your comment https://github.com/hay-kot/scaffold/issues/48 you're actually using the controller directly which I don't support right now. I don't intend to guarantee any APIs there and will likely refactor them a few more times as I restructure this repository and clean things up.
Given that, I'm going to close this PR. I think if you're wanting to consume the controller directly and need modifications to support that you'll have to maintain a fork. Alternatively, you may be able to just dump your embedded contents to a temporary location on the file system and pass that to the controller and use it without modification. Like I said though, you may experience breakages with updates.


Display a warning for unsupported boards

Display a warning rather than an error when the target board is
unsupported. This is not a fatal error, since pyOCD can still connect
and be used to debug. It just cannot flash the target.

@mbed-bot: TEST


Update rgb lib

This is WIP because it seems rgb-lib is also WIP.

Test failed due to inconsistency in a database. This will be fixed by https://github.com/RGB-Tools/rgb-lib/pull/28


Pichu issues

Hi,
First of all this site is awesome! Great job!
Pichu has some issues:

Sheild missplaced.
Missing projectiles
Missing Thunder animiation

Thanks!

Hi thanks for the report. These are known issues that I mostly fixed for top tiers and mostly haven't done for low tiers yet.


crash when play rtsp live stream(UE4.16)

hi, we are using VlcMedia as a project plugin
it crashed when play rtsp stream,
[log]
LogVlcMedia:Error: mmdevice: cannot initialize COM (error 0x80010106)
LogVlcMedia:Warning: faad: decoded zero sample
LogVlcMedia:Warning: Possible loss of audio quality due to sample rate != 44100 Hz
LogVlcMedia:Warning: freetype: DirectWrite initialization failed. Falling back to GDI/Uniscribe

LogVlcMedia:Error: vmem: video format setup failure (no pictures)
LogVlcMedia:Error: core: video output creation failed
LogVlcMedia:Error: core: failed to create video output

LogVlcMedia:Warning: freetype: DirectWrite initialization failed. Falling back to GDI/Uniscribe
LogVlcMedia:Error: vmem: video format setup failure (no pictures)
LogVlcMedia:Error: core: video output creation failed
LogVlcMedia:Error: core: failed to create video output

LogVlcMedia:Warning: freetype: DirectWrite initialization failed. Falling back to GDI/Uniscribe
**LogVlcMedia:Error: swscale: invalid scaling: 1280x720 -> 1280x0
LogVlcMedia:Error: swscale: invalid scaling: 1280x720 -> 1280x0**
LogVlcMedia:Error: core: Failed to change zoom
LogVlcMedia:Error: core: Failed to set on top
LogVlcMedia:Error: core: Failed to change source AR
LogVlcMedia:Error: core: Failed to change Viewpoint

LogVlcMedia:Warning: freetype: DirectWrite initialization failed. Falling back to GDI/Uniscribe
LogVlcMedia:Warning: avcodec: plane 0 not aligned: disabling direct rendering
LogVlcMedia:Error: core: buffer deadlock prevented

do you have any ideas why this would be happening?

it worked after remove "libscale_plugin.dll"

Interesting... thank you very much for tracking it down! You might want to file an issue https://github.com/videolan/vlc


v0.4.8 is not uploading to pip

Bug Report
Installation Method
pip
Environment

Open WebUI Version: 0.4.7

Ollama (if applicable): 0.4.6

**Operating System:**Arch Linux

Browser (if applicable): Firefox

Confirmation:

[* ] I have read and followed all the instructions provided in the README.md.
[ ] I am on the latest version of both Open WebUI and Ollama.
[* ] I have included the browser console logs.
[* ] I have included the Docker container logs.
[* ] I have provided the exact steps to reproduce the bug in the "Steps to Reproduce" section below.

Expected Behavior:
install v0.4.8 on pip
and the action job work well
Actual Behavior:
Because in the Actions,the pip is not uploading[says Open-Webui reached the pip package 100MB limit],so unable to install it in pip
Description
Bug Summary:
project reached pip size limit 100MB
Reproduction Details
Steps to Reproduce:
You can try to upload it to pip
Logs and Screenshots
Browser Console Logs:
no
Docker Container Logs:
no
Screenshots/Screen Recordings (if applicable):
no
Additional Information
nothing

Workaround if you still want to install it, download the open-webui-0.4.8.tar.gz file from the releases page and install it via pip (make sure you have nodejs/npm installed)
pip install open-webui-0.4.8.tar.gz

Workaround if you still want to install it, download the open-webui-0.4.8.tar.gz file from the releases page and install it via pip (make sure you have nodejs/npm installed)
pip install open-webui-0.4.8.tar.gz

This is from the logs:

Transparency log entry created with index: 153947321
Uploading distributions to https://upload.pypi.org/legacy/
Uploading open_webui-0.4.8-py3-none-any.whl

WARNING  Error during upload. Retry with the --verbose option for more details.
ERROR    HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/
File too large. Limit for project 'open-webui' is 100 MB. See
https://pypi.org/help/#file-size-limit for more information.

can anyone please try to build a pip package? because i'm in china,the network issue make me unable to download onnxruntime-node from npm

https://github.com/pypi/support/issues/5250

Same here; Upgrading pip doesn't go to version v0.4.8 (remains at v0.4.7).

It's the same for me, under macOS 15.2:
"Requirement already satisfied: open-webui in /Applications/Python/open-webui/lib/python3.11/site-packages (0.4.7)"

See the issue above. Until pypi approves the file size limit increase for open webui 0.4.8 cannot be uploaded.

If we want this to be expedited, comments should be left here: https://github.com/pypi/support/issues/5250
There's nothing we can do on our end.

Download the open-webui-0.4.8.tar.gz file, and unzip the package
cd ./open-webui-0.4.8
run python -m build

hint: NodeJS npm is required for building Open Webui

pypi updated to allow 200MB
https://github.com/pypi/support/issues/5250#issuecomment-2543232065
Can 0.48 be uploaded to pypi so we may update using pip - thanks

Resolved!


'cleanupIDs' plugin bug due to preserved IDs/prefixes

It is possible for the 'cleanupIDs' plugin to change an ID to one that is preserved, or one that matches a prefix to preserve, when minifying an ID. Therefore, it is possible for the resultant SVG to have two elements with the same ID.
For example, with the following input:
<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" viewBox="0 0 230 120">
    <defs>
        <circle id="a" fill="red" cx="60" cy="60" r="50"/>
        <rect id="rect" fill="blue" x="120" y="10" width="100" height="100"/>
    </defs>
    <use xlink:href="#a"/>
    <use href="#rect"/>
</svg>

if the ID a is preserved, the output will be:
<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" viewBox="0 0 230 120">
    <defs>
        <circle id="a" fill="red" cx="60" cy="60" r="50"/>
        <rect id="a" fill="blue" x="120" y="10" width="100" height="100"/>
    </defs>
    <use xlink:href="#a"/>
    <use href="#a"/>
</svg>

v1.3.1 with the fix is just released.


While getting Login with github profile credentials, It's throws an Error ?

Hi,
Even Through I providing right credentials it's throws an error
How to make it correct

Thank You,
SUGGU SANDEEP.

did you check any of these closed issues https://github.com/abapGit/abapGit/issues?q=is%3Aissue+401 ?

https://docs.abapgit.org/guide-authentication.html

Thank You @larshp & @mbtools
Problem Got Solved
As Need To Mention Token Which got generated the GitHub Settings>Developer Settings


Build an executable jar from QueryRunner for performance testing.

Build executable jar from QueryRunner for easy deployment to
different environments for perofrmance testing.

This is very useful. Can we take it one step further and invoke PinotToolLauncher instead of QueryRunner. We can add QueryRunner as subcommand pf ToolLauncher

Looks like tabs got in. Fixed it

Current coverage is 64.43% (diff: 100%)

No coverage report found for master at 0fded25.

Powered by Codecov. Last update 0fded25...037e9ae


1.22.0 release notes and documentation

Proposed changes
Added release notes and documentation for the upcoming 1.22.0 release of Compose
Unreleased project version (optional)
1.22.0

cc @vdemeester


Missing dist folder again

Using this package via Bower. Once again the package is missing the dist folder.
So it breaks building of existing system. i.e. it is breaking change.
Please please please put it back.

The dist folder is not missing, see: https://github.com/jzaefferer/jquery-validation/tree/1.15.1.
You should not use the master version but a released one.

So I have updated to this version and one person on my team is still not getting dist folder. Any ideas?

@jrock2004 I've just checked and it's there:
$ bower install<EMAIL_ADDRESS>$ ls bower_components/jquery-validation/dist
additional-methods.js     additional-methods.min.js jquery.validate.js        jquery.validate.min.js

I just followed your suggestion. I used bower install<EMAIL_ADDRESS>but I can't find the dist folder. Is there something else that must be done to see it?

@smithaitufe Make sure you use the latest bower (currently 1.8.0) and clear its cache by bower cache clean. You may also remove bower_components to a the clear state.
In any case, the dist folder is definitely in the package. If you don't have it, this is an issue with bower, not with the jquery-validation plugin.

@mgol
Thanks. I did that and got it working.


Issue with partially mocking module that is also used within same file by another module

Hello.
I think I've found an issue. I'm having trouble with partially mocking deeply nested module that is also used in same file. Example below. I'm on Mocha and Node 17.4.0.
const supertestModule = await esmock.px('../modules/supertest.js', null, {
  '../../../../server/troublesomeFile.js': {
    funcOne: async () => {
      console.log('Ran')
    }
  }
})

troublesomeFile.js
export async function funcOne() {
  console.log('Should not be called')
}

export async function funcTwoSameFile() {
   const test = await funcOne()

  // some more code

  return test
}

supertest.js (might not be relevant)
// imports app.js...
export default async () => {
     // More code...

     return supertest(app)
      .post(`/v1/${endpoint}`)
      .set({ ...thingsToSet })
      .send({ ...requestBody })
  }
}

When funcTwoSameFile is called somewhere deeply nested in app.js, original version of funcOne is called and not the mocked one. Is this some kind of limitation or am i missing something? My other (full-module, not .px mocks) are working fine in other tests.
Thanks for your help!

https://github.com/iambumblehead/esmock/blob/master/src/esmockModule.js#L97-L98
@vueme esmock currently does something like this paraphrased example,
const originalFile = import( pathtooriginalfile );
const mockedFile = Object.assign({}, originalFile, mockDefinitions);

return mockedFile

and so the partial mock won't work for your use-case (sorry)
is it OK with you if this ticket is closed?

Thanks for quick answer!
You do not happen to have any workaround recommendation?
Thanks anyways!

@vueme these may not be great ideas but here they are :) One idea is to define both mock functions at the same time eg,
const mock = {};
mock.funcTwo = () => 'mocked func two';
mock.funcOne = () => mock.funcTwo();

const supertestModule = await esmock.px('../modules/supertest.js', null, {
  '../../../../server/troublesomeFile.js': mock
})

another idea is to use 'this' or some other namespace creatively like this
originalfile.js
export const obj = {};

obj.funcOne = async function () {
  return 'this.funcOne'
};
obj.funcTwo = async function () {
  return await this.funcOne();
};

originalfile-test.js
const supertestModule = await esmock.px('../modules/supertest.js', null, {
  '../../../../server/troublesomeFile.js': {
     obj: {
       ...troublesomefile.obj,
       funcOne: () => 'mocked value'
     }
   }
})

I didn't test these but hope their intention is expressed

Thanks for the ideas and this great js package! I'll look into this.


Run the error in the trin_net_rcnn.py

Hello, I encountered errors in the picture during the process of running the program. Excuse me, is Troi in the picture a module in Python? But after installing the error in the picture, I hope you can help to point out.

Hi 7linlin,
Thanks for reporting the bug. The line of code should be
from rcnn.my_fast_rcnn_output import fast_rcnn_inference_single_image
We have fixed it.
Thanks!

thank you for your reply


Attribute to apply transparent wrappers to fields

Most rkyv functionality can be implemented in terms of wrapper types around some data. For example:
pub struct Compress<T>(T);

// implement archive, serialize, and deserialize

#[derive(Archive, Serialize, Deserialize)]
pub struct Example {
    name: Compress<String>,
}

This is a great way to take advantage of type extensions to quickly leverage other libraries. However, we're still left with a Wrapper<T> which is just noise for the unarchived field. A better option would be to have the derive macros automatically perform this transformation for us. This would ideally lead to some syntax like:
#[derive(Archive, Serialize, Deserialize)]
pub struct Example {
    #[with(Compress)]
    name: String,
}

that would wrap the field &T in a Wrapper<T> and archive the &Wrapper<&T>. During deserialization, it would deserialize it into the original type. Applying multiple wrappers should also be easy:
#[derive(Archive, Serialize, Deserialize)]
pub struct Example {
    #[with(Intern, Compress)]
    name: String,
}

with wrappers being applied left to right (i.e. Intern<&Compress<&T>>).
Wrapper types should already be expressible using the traits currently available:
pub struct Wrap<'a, T>(&'a T);

pub struct ArchivedWrap<T>(T);

pub struct WrapResolver<T: Archive>(T::Resolver);

impl<'a, T> Archive for Wrap<'a, T> {
    type Archived = ArchivedWrap<T::Archived>;
    type Resolver = WrapResolver<T>;

    fn resolve(&self, pos: usize, resolver: Self::Resolver, out: &mut MaybeUninit<Self::Archived>) {
        self.0.resolve(pos, resolver.0, project_struct!(out: Self::Archived => 0);
    }
}

impl<'a, T: Serialize<S>, S: Fallible + ?Sized> Serialize<S> for Wrap<'a, T> {
    fn serialize(&self, serializer: &mut S) -> Result<Self::Resolver, S::Error> {
        Ok(WrapResolver(self.0.serialize(serializer)?))
    }
}

impl<'a, T: Archive, D: Fallible + ?Sized> Deserialize<T, D> for ArchivedWrap<T::Archived>
where
    T::Archived: Deserialize<T, D>,
{
    fn deserialize(&self, deserializer: &mut D) -> Result<T, D::Error> {
        Ok(self.0.deserialize(deserializer))
    }
}

but the bound T: Archive<Archived = Self> may need to be lifted from Deserialize.

Need to make sure we can support enum variants, e.g.
enum Segment<&'a T> {
    Primary(&'a T),
    Secondary(&'a T),
}

// wrapper implementation

#[derive(Archive, Serialize, Deserialize)]
pub struct Example {
    #[with(Segment::Secondary)]
    name: String,
}

I think it should be okay as long as the arguments are treated as constructors.

Fixed by cd320234423b076b3edd7c29e31eb66964e02377


Ticket 63963: update jackson dependency to <IP_ADDRESS>

Description
This patch updates jackson databind to newer version <IP_ADDRESS>
Motivation and Context
Current version 2.9.10. has some security vulnerabilities as described in the Jmeter bug ticket
How Has This Been Tested?

gradlew check passes
running own version of jmeter for some days with updated library without problems

Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.

Codecov Report

Merging #546 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #546      +/-   ##
============================================
+ Coverage     56.11%   56.11%   +<.01%
- Complexity    10020    10021       +1
============================================
  Files          1024     1024
  Lines         62909    62909
  Branches       7067     7067
============================================
+ Hits          35301    35302       +1
+ Misses        25131    25130       -1
  Partials       2477     2477

Impacted Files
Coverage Δ
Complexity Δ

...n/java/org/apache/jmeter/reporters/Summariser.java
86.25% <0%> (+0.76%)
19% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Support manifest embedding in user-app rpms & fixes

First commit adds support for embedding application manifests
into the user-application containers for use with microshift/ostree
Second commit fixes uid/gid setup and file capabilities
rpm requires the file attribute uid/gid to be symbolic
and not numeric. Additionally tar didn't store the caps
but cpio doesn't keep those inside rpm either.
So we create a list of file caps during tar creation, and then
we populate the %files metadata with %caps attributes so those
are re-installed during rpm unpack.
This is tested and working.

/lgtm


SMHE-2299: Update protocol adapter to CXF version defined in super pom

https://jenkins.fdp.osgp.cloud/view/SMHE-cucumber/job/gxf-smhe-cucumber-build/301/


How to extract protein position for amino acid change from GEMINI database?

Hi GEMINI developers!
Thanks for the excellent framework.
Is there a way to extract protein position for amino acid change from GEMINI database
(VEP annotation) I don't see this field in the variants or variant_impacts tables.
All I want is to have the variant effect for protein-coding genes reported in common F234L format.
Thanks.
Sergey

Hi, for historical reasons, this is stored in aa_length in the database.

Thanks Brent.
Could you please describe this in the documentation?
http://gemini.readthedocs.io/en/latest/content/database_schema.html#the-variants-table
Also this field seems to work with VEP.
SN

Yes, meanwhile, a pull-request would be appreciated.


Archive.org can't display scraped franklin websites

Expected Behaviour
Archive.org shows the website as it was.
Actual Behaviour
The website appears blank and the console shows: “Uncaught SyntaxError: Unexpected token ‘export’“, for lib-franklin and scripts respectively.
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Access either of these URLs

https://web.archive.org/web/20230201084316/https://www.netcentric.biz/
https://web.archive.org/web/20230102083751/https://www.theplayers.com/

Logs taken while reproducing problem
“Uncaught SyntaxError: Uneexpected token ‘export’“

This seems to be a problem with archive.org and should be reported there.


cannot have code completion for node.js typings in a nodeclipse project with palantir typescript builder

Hi,
I am a java developer trying to set up an dev environment to do some development using nodeclipse and typescript and angular2.
I installed nodeclipse and then eclipse-typescript.
Created a simple helloworld typescript nodeclipse project.
Then selected the typescript builder.
I create an empty file and then typed.
var http = require("http");
then if I type http.  you should expect a menu with list of options available, I don't get any.
But if type
console.   then I get a menu with list of options available.
For some reason the code completion for node.js functionality is not working.
I tried including node.d.ts into the projects, but it resulted in error that there is duplicate copy of require.
Not sure how to make the code completion for node.js to work in an nodeclipse project which uses typescript builder.
Please help me to fix this issue.
thanks in advance.
Johnson Abraham

Hey johnson,
In node.d.ts, require extends the following signature:
interface NodeRequireFunction {
    (id: string): any;
}

This means, in particular, that http gets declared as a var with type any. If I do
import * as http from "http";

then I get the autocompletions. Does this help?


getReady() should return a list of PublicSignKey's instead of hashcodes

It will just make things easier when working with Peerbit because then you can directly interact with users publickeys.

The reason why this was the behaviour previously was that it is faster to return the representation used by pubsub directly
In the future the anonymity aspect might be imprortant


Remove hard-coding of RAPIDS version where possible

Read VERSION file from CMake
Read ucxx.__version__ from docs build
Read VERSION file from shell scripts
Use variable substitution in Doxyfile
Update pyproject.toml to use dynamic version
Update ci/release/update-version.sh

Issue: https://github.com/rapidsai/build-planning/issues/15

Yes, that was taken into consideration, and it's why I didn't use the rapids-version and rapids-version-major-minor commands (among other deviations from the other repos.)

Think we may need to add some symlinks to the top-level VERSION file into Python projects

Something seems to have broken ucxx wheel builds. It's failing with a patchelf error.
Fixed-up wheel written to /__w/ucxx/ucxx/python/final_dist/ucxx_cu11-0.37.0a36-cp310-cp310-manylinux_2_28_aarch64.whl
Archive:  /__w/ucxx/ucxx/python/final_dist/ucxx_cu11-0.37.0a36-cp310-cp310-manylinux_2_28_aarch64.whl
  inflating: repair_dist/ucxx_cu11.libs/libcudart-be0b6e85.so.11.8.89
  inflating: repair_dist/ucxx_cu11.libs/libucp-d240e863.so.0.0.0
  inflating: repair_dist/ucxx_cu11.libs/libucs-8ef1b6eb.so.0.0.0
  inflating: repair_dist/ucxx_cu11.libs/libucm-ed183686.so.0.0.0
  inflating: repair_dist/ucxx_cu11.libs/libuct-d5cf5765.so.0.0.0
/__w/ucxx/ucxx/repair_dist/ucxx_cu11.libs /__w/ucxx/ucxx
/__w/ucxx/ucxx
/__w/ucxx/ucxx/repair_dist/ucxx_cu11.libs/ucx /__w/ucxx/ucxx
patchelf: missing filename

https://github.com/rapidsai/ucxx/actions/runs/8254849219/job/22581770392?pr=201#step:9:2121

See https://github.com/rapidsai/ucxx/pull/203 for a possible fix.

Now that #203 has landed, updating this PR to include that fix


Class generic extends class generic will not be evaluated even though the condition is always false

Bug Report
🔎 Search Terms
conditional type narrow class generic
🕗 Version & Regression Information
This is the behavior in every version I tried, and I reviewed the FAQ for entries.
⏯ Playground Link
Playground link with relevant code
💻 Code

type StoreEntry = {
  id: string
}

// A basic function with a return type that depends on its input parameters.
function get<
  Entry extends StoreEntry,
  Arg extends Entry['id'] | Entry['id'][] | undefined = undefined
>(
  idOrIds?: Arg
) : Arg extends Entry['id'] ? Entry | null
  : Record<string, Entry> {
}

function test<Entry extends StoreEntry>(entryIds: Entry['id'][]) {
  const result = get(entryIds);
  console.log(typeof result['1']); // Ok
}

class Store<Entry extends StoreEntry> {
  public get1<Arg extends Entry['id'] | Entry['id'][] | undefined = undefined>(
    idOrIds?: Arg
  ) : Arg extends Entry['id'] ? Entry | null
    : Record<string, Entry> {
  }

  public test1(entryIds: Entry['id'][]) {

    const result = this.get1(entryIds);
    // Entry["id"][] extends Entry["id"] ? Entry | null : Record<string, Entry>

    console.log(typeof result['1']); // (bug) Result is possibly null
  }

  public get2<Arg extends Entry['id'] | Entry['id'][] | undefined = undefined>(
    idOrIds?: Arg
  ) : Arg extends string ? Entry | null // Replacing Entry['id'] with string
    : Record<string, Entry> {
  }

  public test2(entryIds: Entry['id'][]) {

    const result = this.get2(entryIds); // Record<string, Entry>

    console.log(typeof result['1']); // Ok
  }

  public test3(entryIds: Entry['id'][]) {
    type ok = Entry['id'][] extends string ? Entry | null : Record<string, Entry>;
    // ok: { [x: string]: Entry}

    // All of the below conditions will never be met but it's not computed

    type unresolved1 = Entry['id'][] extends Entry["id"] ? Entry | null : Record<string, Entry>;
    // Entry["id"][] extends Entry["id"] ? Entry | null : Record<string, Entry>

    type unresolved2 = Entry['id'] extends Entry ? Entry | null : Record<string, Entry>;
    // Entry["id"] extends Entry ? Entry | null : Record<string, Entry>

    type unresolved3 = Entry[] extends Entry ? Entry | null : Record<string, Entry>;
    // Entry[] extends Entry ? Entry | null : Record<string, Entry>
  }
}

🙁 Actual behavior
A conditional type that compares a class generic against a class generic will not be resolved, even though the condition is false no matter what the value of the generic is.
🙂 Expected behavior
The conditional should be properly resolved since whatever  Entry['id'] is, there is no scenario where Entry['id'][] would extend Entry['id'] which must be string. I cannot define a type that extends StoreEntry where the id property would be anything but a string. The conditional should be resolved as false in each case.

@MartinJohns
The previous issue occurred in functions, this one only occurs in classes. If it's the same design limitation - why does this work in functions?

The conditional should be properly resolved since whatever Entry['id'] is, there is no scenario where Entry['id'][] would extend Entry['id'] which must be string.

In general, it's wrong, extremely difficult, or impossible to prove that for a given T, a type constrained to U cannot possibly extend it. Trivially, T & U by definition does, and whether or not T & U is necessarily never involves a lot of counterfactual reasoning that can generally be pushed upstream into "You shouldn't have written the type that way".

Alright, thanks. I did not encounter the issue for anything else and in this scenario it's not a problem since the type is already constrained earlier by the extends keyword.


Fix tests

Currently just a temporary workaround. Needs proper fixing to merge.
See #171 (closes #171)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
3111

Relevant Lines:
3121

💛  - Coveralls


[api] we must be able to override from where assemblies are loaded

The current code assume they will be with the executable. That's not gonna work for all scenarios, e.g. plugins.  PR#42 ran into this issue while adding Xcode-based unit tests. There's a band-aid API to move things forward but it should be revisited to cover more scenarios*

creating a .framework will also require a different location, c.c. @rolfbjarne

reference: https://github.com/mono/Embeddinator-4000/pull/42

We got APIs for this now in the runtime


CLEANUP ember docs and refactor

Issue: #4319
What I did

Update docs to be more in line with init
Working on https://github.com/gabrielcsapo/ember-storybook to streamline ember onboarding

relevant errors from CI:
[08:57:48]	[Step 2/2] in directory: /mnt/agent/work/a1796458d49a7849
[08:57:48]	[Step 2/2] + cd examples/ember-cli
[08:57:48]	[Step 2/2] + yarn build-storybook
[08:57:49]	[Step 2/2] yarn run v1.9.4
[08:57:49]	[Step 2/2] $ yarn build && cp -r public/* dist; build-storybook -s dist, public
[08:57:49]	[Step 2/2] $ ember build
[08:57:53]	[Step 2/2] Could not start watchman
[08:57:53]	[Step 2/2] Visit https://ember-cli.com/user-guide/#watchman for more info.
[08:57:53]	[Step 2/2] Building
[08:57:53]	[Step 2/2] DEPRECATION: Ember will stop including jQuery by default in an upcoming version. If you wish keep using jQuery in your application explicitly add `@ember/jquery` to your package.json
[08:57:54]	[Step 2/2] Environment: development
[08:57:58]	[Step 2/2] cleaning up
[08:57:58]	[Step 2/2] cleaning up...
[08:57:58]	[Step 2/2] Built project successfully. Stored in "dist/".
[08:57:59]	[Step 2/2] info @storybook/ember v4.0.0-alpha.24
[08:57:59]	[Step 2/2] info
[08:58:00]	[Step 2/2] info => Loading presets
[08:58:00]	[Step 2/2] info => Loading custom addons config.
[08:58:00]	[Step 2/2] info => Loading custom webpack config (full-control mode).
[08:58:00]	[Step 2/2] info => Copying static files from: dist
[08:58:00]	[Step 2/2] ERR! Error: no such directory to load static files:
[08:58:00]	[Step 2/2] error Command failed with exit code 255.
[08:58:00]	[Step 2/2] info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
[08:58:00]	[Step 2/2] Process exited with code 1
[08:58:00]	[Step 2/2] Process exited with code 1 (Step: build (Command Line))
[08:58:03]	[Step 2/2] Step build (Command Line) failed

@ndelangen the issue was with build-storybook -s dist, public turning into ['dist', ''] I think it might be with static build resolution where we always add public, so it being redundant might break things, trying to debug now


add LD+JSON decoder

solves: #90

I think that this will cover a lot of cases. But because of the possible complexity of schema.org I won't give a 100% guarantee that it will work with all cases in the current state. But I think that we won't find these cases without releasing it!?


Facing problem in authentication

Describe the bug
User is not getting logged. Everything is done according to the documentation https://adminbro.com/tutorial-rbac.html
Installed libraries and their versions

<EMAIL_ADDRESS><EMAIL_ADDRESS>

Can you show your code where you add adminjs router to your express app?

Closing because of lack of response


sdxl version doesn't work on non-square image

Man, Can you provide your resolution ? I will reproduce and fix it

@xiao2mo I've faced the same problem. I am using https://huggingface.co/RunDiffusion/Juggernaut-XL-v9 model on the recommended resolution of 1216x832. But it works only for square resolution.
Thank you for your work!

Now HiDiffusion supports it. Please update HiDiffusion to the latest version
pip3 install hidiffusion -U
Here is an example of  2432x1664 resolution (2x height and width of 1216x832)


fixed empty THE_FILE_LIST

avoding wrong paths concat when AWSLogs is doubling

Thanks for submitting, taking a look at this now and validating.

This broke normal operation, probably a better option would be to update the 'S3_BUCKET_NAME' setting.  I'm assuming the doubling you're speaking about is for example:
s3://loadbalancer-logs/AWSLogs/service/AWSLogs/etc
In that case you, it would be best to set 'S3_BUCKET_NAME' to:
S3_BUCKET_NAME="loadbalancer-logs/AWSLogs"


Error with API Joining Space

Using the code to join a space.  Pass the GuestToken, spaceId and set Destination Type to SpaceId, get following result.  Spinning wheel then get the Conversation Not Found error.  Here is the console message:
Uncaught (in promise) t: Conversation not found
undefined https://conv-a.wbx2.com/conversation/api/v1/conversations/63986c80-bd6d-11ea-9512-678f137a325f
WEBEX_TRACKING_ID: webex-js-sdk_bbcade39-a9d8-4a2d-b924-98d9374d1567_10

@gmendiol-cisco, are you passing an access token or a JWT token? The widgets only understand the former

You are correct in that I was passing the JWT Token, when I added the exchange and used the Access Token, it worked great.

I have another question regarding video.  I've tried initiating video 1:1 or 1:Space from both directions and keep running into this error:
engine.js:252 Uncaught (in promise) TypeError: Cannot read property 'getUserMedia' of undefined
at Function.value (engine.js:252)
at engine.js:896
I'm using the production bundle.js and have also tried latest.  I appreciate any guidance you can provide.

@gmendiol-cisco, I hope you've figured it out by now 😅  but that error means that you're trying to load the widget in an insecure page. To get access to camera and microphone the page need to be served securely, e.g. HTTPS.


enable systemd to reload gitea config

Really? I don't know that Gitea's config could be reloaded.

Really? I don't know that Gitea's config could be reloaded.

The title is inaccurate. It's just reloading Gitea in the systemd configuration.

There is a SIGHUP handler here:
https://github.com/go-gitea/gitea/blob/09dd4b42c7afe3f474d900b9d5856af2929a12ea/modules/graceful/manager_unix.go#L134-L136
If I understand correctly, it is a complete process restart, certainly not a config reload like this PR says.

Unless restart the Gitea process, at the moment, there is no feasible approach to make GItea fully reload the config dynamically.
It's the hardest part in Gitea, more difficult than the Template/Route refactoring / Queue rewrting / Logger system refactoring. The idea/implementation is not that hard, but it needs to rewrite a lot of code.

Hi @jnahmias , thank you for the contribution. According to the comments above, it seems that it's better not to add ExecReload=/bin/kill -HUP $MAINPID to the systemd config, because Gitea doesn't support the "config reloading" that way.


Can Create a tech profile and Sign in.... Why ?

Issue is im trying to sign in and create a tech profile, nothing is happenning...
Get to https://www.microsoft.com/en-us/learning/training.aspx using my microsoft account...

Click the start learning button taken to https://docs.microsoft.com/en-gb/learn/  but now asked to sign in..

click sign in, taken to https://techprofile.microsoft.com/en-GB/register?replyUrl=https://docs.microsoft.com/en-gb/learn/&  where i fill out, hit register and nothing happens...

Expected behavior
I'k like to create a tech profile, and enter the site, access courses and track progress.  I do have a profile with the same email address on  https://academy.microsoft.com/en-us/dashboard  so i dont know if that needs to be linked at all...
Desktop (please complete the following information):

Windows 10
firefox, but have tried it in private window and on chrome and on safari on my mac, same issue

I have exactly the same issue, except in my browser (Chrome) I get a message in red after I click the 'Register' button, which states: "Something went wrong. please try again later."

Try setting country to US as per this comment: https://github.com/MicrosoftDocs/feedback/issues/1234#issuecomment-469759236

No Works even I selected US. Same message: "Something went wrong. please try again later."

Yep..   Same here selecting US doesnt fix, i get the "something wrong" message in chrome and on Safari but dont get it on firefox.

Same problem, have been going around in circles on your new partner site for a couple of weeks. - no message, just does nothing Using Edge on Windows 10

@jialge please take a look and review the feedback from this customer. Thanks.

I am getting the same
Examining traffic I see
`
Request URL: https://docs.microsoft.com/api/profiles
Request method: OPTIONS
Remote address: <IP_ADDRESS>:443
Status code: 403 Forbidden
Version: HTTP/2.0
Referrer Policy: no-referrer-when-downgrade
REQUEST HEADERS
Accept: text/html,application/xhtml+xm…ml;q=0.9,image/webp,/;q=0.8
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.5
Access-Control-Request-Headers: content-type
Access-Control-Request-Method: POST
Connection: keep-alive
DNT: 1
Host: docs.microsoft.com
Origin: https://techprofile.microsoft.com
Referer: https://techprofile.microsoft.…&activate-azure-sandbox=true&&
TE: Trailers
User-Agent: Mozilla/5.0 (Windows NT 6.1; W…) Gecko/20100101 Firefox/65.0
`

with Firefox is worked for me without check the "I agree" box......

can confirm this worked for me too.
Leaving country blank did not work. Tested on both edge and chrome.

@dastaffo Adding you here as well to take a look.


Cumulative update

Update gosdk dependence
Update API meta data

Codecov Report

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

@@           Coverage Diff           @@
##           master     #178   +/-   ##
=======================================
  Coverage   86.64%   86.64%
=======================================
  Files          50       50
  Lines        3003     3003
=======================================
  Hits         2602     2602
  Misses        286      286
  Partials      115      115

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 b905465...41ee38d. Read the comment docs.


add pod_subnet e2e case

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/approve

/lgtm


Add links for contract events and logs tutorial

Problem
Add links to tutorial when published
Solution
see above
Alternatives
No response

https://docs.hedera.com/guides/resources/tutorials/get-event-information-from-hedera-smart-contracts


Remove the cleanup logic that has now cleaned up orphaned pods

This reverts commit 39e6b2c985540ecfe7203833fdbdbf8b8e9e4c39, the main part of #3192. This was something intented to be reverted once we had let it execute in all hubs - it was just a one off cleanup operation.

Thanks, your most welcome to merge it!
The PR adding it had a 100% success rate so I think it could work out without any issues to merge this as well.


i cannot run QuestPatcher-mac.dmg

I cannot run QuestPatcher-mac.dmg. plz help i really want to mod Gorilla tag.

You are both being extremely vague, you need to be specific as to what your problem actually is if you want me to fix it.


optimize: throw Exception when no index in the table

Ⅰ. Describe what this PR did
git some tips when no index in the table cause NPE in TableMetaCache.java.
Ⅱ. Does this pull request fix one issue?

#1529
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1530 into develop will decrease coverage by 0.04%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             develop    #1530      +/-   ##
=============================================
- Coverage      46.43%   46.38%   -0.05%
  Complexity      1691     1691
=============================================
  Files            347      347
  Lines          12713    12717       +4
  Branches        1588     1590       +2
=============================================
- Hits            5903     5899       -4
- Misses          6172     6177       +5
- Partials         638      641       +3

Impacted Files
Coverage Δ
Complexity Δ

...rm/datasource/sql/struct/TableMetaCacheOracle.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...seata/rm/datasource/sql/struct/TableMetaCache.java
51.21% <0%> (-0.64%)
13 <0> (ø)

...server/store/file/FileTransactionStoreManager.java
45.99% <0%> (-1.4%)
19% <0%> (ø)

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 70b04d3...07ac955. Read the comment docs.


No directive found with exportAs 'ngxTouchKeyboard'

I'm trying to use this really great lib in my Angular 16 application, but even if I follow the read.me instructions, I can't create the
This is my app.module:
import { NgxTouchKeyboardModule }  from 'ngx-touch-keyboard';
...
@NgModule({
    declarations: [...],
    providers: [...],
    bootstrap: [AppComponent],
    imports: [..., NgxTouchKeyboardModule]
})

This is my html code:
<input type="text" ngxTouchKeyboard #touchKeyboard="ngxTouchKeyboard" (focus)="touchKeyboard.openPanel()"/>
And this the error that I receive: No directive found with exportAs 'ngxTouchKeyboard'
Image:

Hi @guirms
I'd like to thank you for your attention.
Please import "NgxTouchKeyboardModule" to Module of declaration "ProductionTimesDialogComponent".

Thanks @mohsen77sk, this works! I was using a standalone component, so all I need do to was import this module properly at my component class


Bug - Canvas has 10 times the size of the original photo

Hi,
When i upload an image less than 1MB it doesnt change the file size but, when i upload an image that has More than 1MB is multiplied by 10 times the size, because of this, i cant upload the image to my server.
Im trying to see where is located this "bug" on my files.
I use Windows 7 Proffesional
Google Chrome version : 55.0.2883.87 m
Thank you.

The same situation on my server. Hope we can find the way to fix it.

If you are using toDataURL, try to add toDataURL('image/jpeg'), default output seems to be 'image/png.'

thanks, it works.

The thing is that im actually working with toDataURL('image/jpeg') but its like it doesnt work.
Here is the piece of the code that i use to get the data of the canvas.
var canvas = document.getElementById('canvas');
var canvasData = canvas.toDataURL('image/jpeg');

I really dont know whats happening.

If your original photo is a JPEG image, the call toDataURL('image/jpeg'), if your original photo is a PNG image, the call toDataURL('image/png'), etc.


[SPARK-21811][SQL] Fix the inconsistency behavior when finding the widest common type

What changes were proposed in this pull request?
Currently we find the wider common type by comparing the two types from left to right, this can be a problem when you have two data types which don't have a common type but each can be promoted to StringType.
For instance, if you have a table with the schema:
[c1: date, c2: string, c3: int]
The following succeeds:
SELECT coalesce(c1, c2, c3) FROM table
While the following produces an exception:
SELECT coalesce(c1, c3, c2) FROM table
This is only a issue when the seq of dataTypes contains StringType and all the types can do string promotion.
close #19033
How was this patch tested?
Add test in TypeCoercionSuite

Test build #89376 has started for PR 21074 at commit 803a6a4.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2332/
Test PASSed.

Merged build finished. Test PASSed.

cc @HyukjinKwon @gatorsmile @cloud-fan

Test build #89463 has started for PR 21074 at commit d77f213.

Test build #89459 has finished for PR 21074 at commit 571912f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89459/
Test PASSed.

Test build #89478 has started for PR 21074 at commit 4ce5081.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2409/
Test PASSed.

Test build #89508 has started for PR 21074 at commit 55eefe0.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2434/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2441/
Test PASSed.

Test build #89519 has started for PR 21074 at commit ff514af.

Test build #89508 has finished for PR 21074 at commit 55eefe0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89508/
Test PASSed.

Test build #89519 has finished for PR 21074 at commit ff514af.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89519/
Test FAILed.

Test build #89516 has finished for PR 21074 at commit c2abce2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89516/
Test PASSed.

thanks, merging to master!

@HyukjinKwon thanks! I encountered some network issue and forgot to come back and merge this...


test[DO NOT MERGE]: show custom inflation func usage

Description

Motivation and Context

[ ] I have raised an issue to propose this change (required)
[ ] My issue has received approval from the maintainers or lead with the design/approved label

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I've read the CONTRIBUTION guide
[ ] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

app/export.go:232:7: G404: Use of weak random number generator (math/rand instead of crypto/rand) (gosec)
	r := rand.New(rand.NewSource(seed))

This lint failure can't be fixed. The function from cosmos-sdk that we need to use, accepts only math/rand and not crypto/rand. So, we have no choice. Also, it is used only in tests. Doesn't affect security in any sensible way.

app/export.go:232:7: G404: Use of weak random number generator (math/rand instead of crypto/rand) (gosec)
	r := rand.New(rand.NewSource(seed))

This lint failure can't be fixed. The function from cosmos-sdk that we need to use, accepts only math/rand and not crypto/rand. So, we have no choice. Also, it is used only in tests. Doesn't affect security in any sensible way.

you can disable the check for that line with // nolint: gosec like here

Please rebase against master.

you can disable the check for that line with // nolint: gosec like here

Thanks, Done.

Please rebase against master.

Done.

This seems to have genesis time as a configurable option? Wouldn't this create all sorts of issues if one validator was using a different time?

This seems to have genesis time as a configurable option? Wouldn't this create all sorts of issues if one validator was using a different time?

The option is used to to supply the genesis time, only to make unit tests pass. Actual validators would have the genesis time in the config file, and that should be same for all validators, I think.

This seems to have genesis time as a configurable option? Wouldn't this create all sorts of issues if one validator was using a different time?

The option is used to to supply the genesis time, only to make unit tests pass. Actual validators would have the genesis time in the config file, and that should be same for all validators, I think.

Can't the tests just assume some random arbitrary genesis time then? Just pick a time that makes sense, i.e. sometime in 2018 or something?

@arijitAD, this looks promising.

There's also this, which looks like it's more actively maintained: https://github.com/ericlagergren/decimal

Can't the tests just assume some random arbitrary genesis time then? Just pick a time that makes sense, i.e. sometime in 2018 or something?

No, having a random genesis time doesn't work with tests. They start failing in that case. Seems, there are some parts in tests which depend on the genesis time being correctly set.

Can't the tests just assume some random arbitrary genesis time then? Just pick a time that makes sense, i.e. sometime in 2018 or something?

No, having a random genesis time doesn't work with tests. They start failing in that case. Seems, there are some parts in tests which depend on the genesis time being correctly set as per the test seed.

If that is the case then can OptsWithGenesisTime be moved into a _test.go file and in the regular codepath we continue to use simapp.EmptyAppOptions{} ?

If that is the case then can OptsWithGenesisTime be moved into a _test.go file and in the regular codepath we continue to use simapp.EmptyAppOptions{} ?

No, it doesn't seem feasible. There are existing test utility functions in non _test.go files, which have to use OptsWithGenesisTime.

The e2e tests are failing with
E[2021-11-22|13:45:28.281] applying statefulSet                         module=provider-cluster-kube err="StatefulSet.apps \"web\" is invalid: spec: Forbidden: updates to statefulset spec for fields other than 'replicas', 'template', and 'updateStrategy' are forbidden" lease=akash1v7u8mq7npa6g7pcncnqgshc5dw5cf5m40gww26/100/1/1/akash1ra7cz7jsljstqk3230426949s79j079h8ptjg7 service=web
I[2021-11-22|13:45:28.284] declaring hostname

@troian is this something added with storage?

@hydrogen18 correct, i’ll give it a check why it updated multiple times


How do you get this to work with more than 5 fonts?

I am trying to train a model that recognizes many fonts, however it appears to not work with more than 5 fonts.
I have replaced the conv_label function with
font_list = {}
def conv_label(label):
    if label in font_list:
        return font_list[label]
    else:
        font_id = len(font_list)
        font_list[label] = font_id
        return font_id

and this allows me to automatically give each font a unique id. However upon generating training data and then trying to train the model I get the error IndexError: index 6 is out of bounds for axis 1 with size 5.
I then change
trainY = to_categorical(trainY, num_classes=5)
testY = to_categorical(testY, num_classes=5)

to
trainY = to_categorical(trainY, num_classes=len(font_list))
testY = to_categorical(testY, num_classes=len(font_list))

and run it again. However, this time I get the error ValueError: Dimensions must be equal, but are 5 and 6 for '{{node mean_squared_error/SquaredDifference}} = SquaredDifference[T=DT_FLOAT](sequential_2/dense_11/Softmax, IteratorGetNext:1)' with input shapes: [?,5], [?,6].

you have not happen to solve this problem, have you ?

Looks like you need to change the model creation method (line 103 of train.py). The last layer is hard-coded to 5, so you need to replace that with your desired number of classes. Simplest way is probably to pass num_classes to the model_create method and use that number to determine the size of the last layer output (the one using softmax)


Test behavior of APPLICATION_JSON String return types

This change is

Yep, I know.  You said that you had made it so that returning String is
possible with APPLICATION_JSON though?
On May 10, 2017 3:15 PM, "Mark Elliot"<EMAIL_ADDRESS>wrote:
@markelliot commented on this pull request.
In jaxrs-clients/src/test/java/feign/TextDelegateDecoderTest.java
https://github.com/palantir/http-remoting/pull/421#discussion_r115736148:

@@ -78,6 +79,13 @@ public void testUsesStringDecoderWithTextPlain() throws Exception {
}

 @Test

public void testCannotReturnStringWithMediaTypeJson() throws Exception {

   assertThatThrownBy(() -> service.getJsonString("foo"))

(Feign throwing the exception means that the content foo came back over the
wire)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/palantir/http-remoting/pull/421#discussion_r115736148,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGOdwf_o5NuMLx8Ft5Ep_21NxGRPMxDTks5r4bhNgaJpZM4NWo-t
.


Fixed URL Issue

Description
Fixed URL Issue
This PR fixes #4681
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit 829922579a7abdce54a299ff3447b72aedd50758 at: https://61a2c49ec6e229f5e6d6225b--meshery-docs-preview.netlify.app

@shubham-tyagi Can you please check this PR and approve this.

great stuff @sahilsuman933, but there is a DCO issue while automatic merging. On a side note you don't have to merge your branch in your master branch it's safe to keep your master branch away from your local commits. Just submit a PR through the branch you made changes in. See this for reference https://docs.meshery.io/project/contributing/contributing-gitflow.

@warunicorn19 can you tell us exactly why there is a DCO issue here?

@shubham-tyagi, it's because of the missing sign-off, @sahilsuman933 are you sure you have a sign-off configured? Like eother switch it on from vscode or add it to your .gitconfig

Maybe because I edited the doc directly from the GitHub.

Oh yeah it's because of that. Please follow the link that @shubham-tyagi shared.

@warunicorn19 yeah because I tried to fix this issue using the github directly and I didn't got any way to insert signoff in github


Marvel Example: Override dioProvider instead of repositoryProvider

Fixes https://github.com/rrousselGit/river_pod/issues/438

Thanks


Add ability to set current_dir

Some tests need to run in certain directories but if you call std::env::set_current_dir within your test, you can hit race conditions with other tests running at the same time.  This allows you to set the current_dir only on the child process.

Nice, thanks! Can you rebase this on master now that #24 has landed?

btw this is how I resolved the unused_doc issues in another project
https://github.com/cobalt-org/cobalt.rs/commit/bd5d3b732a0bb50ec6e150956dcca9f3fb1b0914

Heh, thanks for the tip. Maybe updating error-chain would help as well ;)
Ed Page<EMAIL_ADDRESS>schrieb am Fr. 8. Sep. 2017 um 15:18:

btw this is how I resolved the unused_doc issues in another project
cobalt-org/cobalt.rs@bd5d3b7
https://github.com/cobalt-org/cobalt.rs/commit/bd5d3b732a0bb50ec6e150956dcca9f3fb1b0914
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/killercup/assert_cli/pull/33#issuecomment-328100791,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABOX4PkjfMfiQ7APLPLnx1LL0swP8W1ks5sgT6PgaJpZM4PQpAL
.

Thanks for this whole batch of PRs! I'm at a conference currently, so I'm only now able to review it. Looks great, though!
bors r+

bors r+
bors[bot]<EMAIL_ADDRESS>schrieb am Mo. 11. Sep. 2017 um 12:20:

Timed out
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/killercup/assert_cli/pull/33#issuecomment-328487518,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABOX_RzR0OXENc7bht1-kskU5_hrujEks5shQmAgaJpZM4PQpAL
.

Huh, not sure what's bothering bors here. Maybe #33 and #35 can't be merged together?

https://github.com/killercup/assert_cli/blob/master/.travis.yml#L27-L29

You should probably allow Travis to build the staging branch.

Weird. How did it ever work before? Anyway, I'll try to fix it later when I
have some time
Michael Howell<EMAIL_ADDRESS>schrieb am Mo. 11. Sep. 2017 um
19:01:

You should probably allow Travis to build the staging branch.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/killercup/assert_cli/pull/33#issuecomment-328592950,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABOX8A6iZ3boWYwl63jd3sb6MXIWBYtks5shWdkgaJpZM4PQpAL
.

Okay, let's give it another try!
bors r+


[BUG] OpenSSLTest is not using the OpenSSL Provider

OpenSSLTest is a subclass of SSLTest and ensures that the OpenSSL Provider from netty tcnative can be used to provide SSL for the cluster.  See the output of OpenSSLTest.testHttpsAndNodeSSLKeyPass below and see that the test is actually using the JDK SSL provider:
---------------- Starting JUnit-test: OpenSSLTest testHttpsAndNodeSSLKeyPass ----------------
tcpClusterManagerPorts: [7130]/tcpAllPorts: [7130, 8115, 9070]/httpPorts: [9302, 9470, 9615] for (6024-11023) fork 1
[2022-10-24T13:42:35,917][WARN ][org.opensearch.node.Node] version [3.0.0-SNAPSHOT] is a pre-release version of OpenSearch and is not suitable for production
[2022-10-24T13:42:35,918][WARN ][org.opensearch.security.ssl.util.SSLCertificateHelper] Certificate chain for alias node-0 contains a root certificate
[2022-10-24T13:42:35,968][WARN ][org.opensearch.security.ssl.util.SSLCertificateHelper] Certificate chain for alias node-0 contains a root certificate
[2022-10-24T13:42:35,996][WARN ][org.opensearch.security.OpenSearchSecurityPlugin] OpenSearch Security plugin run in ssl only mode. No authentication or authorization is performed
[2022-10-24T13:42:36,020][WARN ][org.opensearch.gateway.DanglingIndicesState] gateway.auto_import_dangling_indices is disabled, dangling indices will not be automatically detected or imported and must be managed manually
{"client.type":"node","cluster.initial_cluster_manager_nodes":["<IP_ADDRESS>:7130"],"cluster.name":"utest_n287_fnull_t1691768839757","cluster.routing.allocation.disk.threshold_enabled":"false","discovery.initial_state_timeout":"8s","discovery.seed_hosts":["<IP_ADDRESS>:7130"],"http.compression":"false","http.cors.enabled":"true","http.port":"9302","http.type":"org.opensearch.security.ssl.http.netty.SecuritySSLNettyHttpServerTransport","http.type.default":"netty4","node.attr.shard_indexing_pressure_enabled":"true","node.max_local_storage_nodes":"3","node.name":"node_utest_n287_fnull_t1691768839757_num3","node.roles":["cluster_manager"],"path.data":["/home/runner/work/security/security/build/testrun/test/target/data/utest_n287_fnull_t1691768839757/data"],"path.home":"/home/runner/work/security/security/build/testrun/test/target","path.logs":"/home/runner/work/security/security/build/testrun/test/target/data/utest_n287_fnull_t1691768839757/logs","plugins.security.ssl.http.clientauth_mode":"REQUIRE","plugins.security.ssl.http.enable_openssl_if_available":"true","plugins.security.ssl.http.enabled":"true","plugins.security.ssl.http.keystore_alias":"node-0","plugins.security.ssl.http.keystore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/node-0-keystore.jks","plugins.security.ssl.http.keystore_keypassword":"changeit","plugins.security.ssl.http.truststore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/truststore.jks","plugins.security.ssl.transport.enable_openssl_if_available":"true","plugins.security.ssl.transport.enabled":"true","plugins.security.ssl.transport.enforce_hostname_verification":"false","plugins.security.ssl.transport.keystore_alias":"node-0","plugins.security.ssl.transport.keystore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/node-0-keystore.jks","plugins.security.ssl.transport.keystore_keypassword":"changeit","plugins.security.ssl.transport.resolve_hostname":"false","plugins.security.ssl.transport.truststore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/truststore.jks","plugins.security.ssl_only":"true","transport.tcp.port":"7130","transport.type":"org.opensearch.security.ssl.http.netty.SecuritySSLNettyTransport","transport.type.default":"netty4"}
[2022-10-24T13:42:36,029][WARN ][org.opensearch.node.Node] version [3.0.0-SNAPSHOT] is a pre-release version of OpenSearch and is not suitable for production
[2022-10-24T13:42:36,030][WARN ][org.opensearch.security.ssl.util.SSLCertificateHelper] Certificate chain for alias node-0 contains a root certificate
[2022-10-24T13:42:36,094][WARN ][org.opensearch.security.ssl.util.SSLCertificateHelper] Certificate chain for alias node-0 contains a root certificate
[2022-10-24T13:42:36,121][WARN ][org.opensearch.security.OpenSearchSecurityPlugin] OpenSearch Security plugin run in ssl only mode. No authentication or authorization is performed
[2022-10-24T13:42:36,145][WARN ][org.opensearch.gateway.DanglingIndicesState] gateway.auto_import_dangling_indices is disabled, dangling indices will not be automatically detected or imported and must be managed manually
{"client.type":"node","cluster.initial_cluster_manager_nodes":["<IP_ADDRESS>:7130"],"cluster.name":"utest_n287_fnull_t1691768839757","cluster.routing.allocation.disk.threshold_enabled":"false","discovery.initial_state_timeout":"8s","discovery.seed_hosts":["<IP_ADDRESS>:7130"],"http.compression":"false","http.cors.enabled":"true","http.port":"9470","http.type":"org.opensearch.security.ssl.http.netty.SecuritySSLNettyHttpServerTransport","http.type.default":"netty4","node.attr.shard_indexing_pressure_enabled":"true","node.max_local_storage_nodes":"3","node.name":"node_utest_n287_fnull_t1691768839757_num2","node.roles":["data"],"path.data":["/home/runner/work/security/security/build/testrun/test/target/data/utest_n287_fnull_t1691768839757/data"],"path.home":"/home/runner/work/security/security/build/testrun/test/target","path.logs":"/home/runner/work/security/security/build/testrun/test/target/data/utest_n287_fnull_t1691768839757/logs","plugins.security.ssl.http.clientauth_mode":"REQUIRE","plugins.security.ssl.http.enable_openssl_if_available":"true","plugins.security.ssl.http.enabled":"true","plugins.security.ssl.http.keystore_alias":"node-0","plugins.security.ssl.http.keystore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/node-0-keystore.jks","plugins.security.ssl.http.keystore_keypassword":"changeit","plugins.security.ssl.http.truststore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/truststore.jks","plugins.security.ssl.transport.enable_openssl_if_available":"true","plugins.security.ssl.transport.enabled":"true","plugins.security.ssl.transport.enforce_hostname_verification":"false","plugins.security.ssl.transport.keystore_alias":"node-0","plugins.security.ssl.transport.keystore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/node-0-keystore.jks","plugins.security.ssl.transport.keystore_keypassword":"changeit","plugins.security.ssl.transport.resolve_hostname":"false","plugins.security.ssl.transport.truststore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/truststore.jks","plugins.security.ssl_only":"true","transport.tcp.port":"8115","transport.type":"org.opensearch.security.ssl.http.netty.SecuritySSLNettyTransport","transport.type.default":"netty4"}
[2022-10-24T13:42:36,154][WARN ][org.opensearch.node.Node] version [3.0.0-SNAPSHOT] is a pre-release version of OpenSearch and is not suitable for production
[2022-10-24T13:42:36,155][WARN ][org.opensearch.security.ssl.util.SSLCertificateHelper] Certificate chain for alias node-0 contains a root certificate
[2022-10-24T13:42:36,221][WARN ][org.opensearch.security.ssl.util.SSLCertificateHelper] Certificate chain for alias node-0 contains a root certificate
[2022-10-24T13:42:36,247][WARN ][org.opensearch.security.OpenSearchSecurityPlugin] OpenSearch Security plugin run in ssl only mode. No authentication or authorization is performed
[2022-10-24T13:42:36,265][WARN ][org.opensearch.gateway.DanglingIndicesState] gateway.auto_import_dangling_indices is disabled, dangling indices will not be automatically detected or imported and must be managed manually
{"client.type":"node","cluster.initial_cluster_manager_nodes":["<IP_ADDRESS>:7130"],"cluster.name":"utest_n287_fnull_t1691768839757","cluster.routing.allocation.disk.threshold_enabled":"false","discovery.initial_state_timeout":"8s","discovery.seed_hosts":["<IP_ADDRESS>:7130"],"http.compression":"false","http.cors.enabled":"true","http.port":"9615","http.type":"org.opensearch.security.ssl.http.netty.SecuritySSLNettyHttpServerTransport","http.type.default":"netty4","node.attr.shard_indexing_pressure_enabled":"true","node.max_local_storage_nodes":"3","node.name":"node_utest_n287_fnull_t1691768839757_num1","node.roles":["data"],"path.data":["/home/runner/work/security/security/build/testrun/test/target/data/utest_n287_fnull_t1691768839757/data"],"path.home":"/home/runner/work/security/security/build/testrun/test/target","path.logs":"/home/runner/work/security/security/build/testrun/test/target/data/utest_n287_fnull_t1691768839757/logs","plugins.security.ssl.http.clientauth_mode":"REQUIRE","plugins.security.ssl.http.enable_openssl_if_available":"true","plugins.security.ssl.http.enabled":"true","plugins.security.ssl.http.keystore_alias":"node-0","plugins.security.ssl.http.keystore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/node-0-keystore.jks","plugins.security.ssl.http.keystore_keypassword":"changeit","plugins.security.ssl.http.truststore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/truststore.jks","plugins.security.ssl.transport.enable_openssl_if_available":"true","plugins.security.ssl.transport.enabled":"true","plugins.security.ssl.transport.enforce_hostname_verification":"false","plugins.security.ssl.transport.keystore_alias":"node-0","plugins.security.ssl.transport.keystore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/node-0-keystore.jks","plugins.security.ssl.transport.keystore_keypassword":"changeit","plugins.security.ssl.transport.resolve_hostname":"false","plugins.security.ssl.transport.truststore_filepath":"/home/runner/work/security/security/build/resources/test/ssl/truststore.jks","plugins.security.ssl_only":"true","transport.tcp.port":"9070","transport.type":"org.opensearch.security.ssl.http.netty.SecuritySSLNettyTransport","transport.type.default":"netty4"}
{
  "principal" : "CN=node-0.example.com,OU=SSL,O=Test,L=Test,C=DE",
  "peer_certificates" : "3",
  "ssl_protocol" : "TLSv1.2",
  "ssl_cipher" : "TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384",
  "ssl_openssl_available" : false,
  "ssl_openssl_version" : -1,
  "ssl_openssl_version_string" : null,
  "ssl_openssl_non_available_cause" : "java.lang.ClassNotFoundException: io.netty.internal.tcnative.SSLContext",
  "ssl_openssl_supports_key_manager_factory" : false,
  "ssl_openssl_supports_hostname_validation" : false,
  "ssl_provider_http" : "JDK",
  "ssl_provider_transport_server" : "JDK",
  "ssl_provider_transport_client" : "JDK"
}

These 2 PRs may be related: https://github.com/opensearch-project/security/pull/422 and https://github.com/opensearch-project/security/pull/1649 - since tcnative is not available on the classpath at runtime it will pick the built in JDK provider.
I believe the test is working because this block will return the JDK provider instead of the OpenSSL provider so that cluster is still able to setup SSL: https://github.com/opensearch-project/security/blob/main/src/main/java/org/opensearch/security/ssl/DefaultSecurityKeyStore.java#L161-L169
There is no specific assertion in the test to ensure it was brought up with the OpenSSL provider.

[Triage] @reta Looks like the OpenSSL feature is not currently working and the tests do not capture this. Would you mind taking a look?


Approximation of the finite line source

This issue is to implement the approximation of the finite line source solution introduced in [1]. An option will be added to choose to evaluate the FLS using the current quadrature method or the approximation.

[1] Cimmino, M., 2021. An approximation of the finite line source solution to model thermal interactions between geothermal boreholes. International Communications in Heat and Mass Transfer, 127, p.105496.

Would it be possible to get this into master for the v2.1.0 release? Given that Cimmino (1) mentions a 1000x speed increase, this should provide a nice increase to calculating g-functions with accuracy.
I'd be happy to help make this happen. If you'd want to open a draft PR, I can work on it.

equivalent solver : Rectangular fields up to 400 boreholes
Results labeled with 'PR-189' use the approximation of the FLS.

similarities solver : Rectangular fields up to 400 boreholes
Results labeled with 'PR-189' use the approximation of the FLS and nFLS=10 terms. We can see that the calculation time is only decreased for smaller bore fields.


jellyfin-web: 10.8.4 -> 10.8.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jellyfin-web is: "Web Client for Jellyfin"
meta.homepage for jellyfin-web is: "https://jellyfin.org/"
meta.changelog for jellyfin-web is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 10.8.5 with grep in /nix/store/27nd9318qvg8klfvyxngrqj70ds6wchb-jellyfin-web-10.8.5
found 10.8.5 in filename of file in /nix/store/27nd9318qvg8klfvyxngrqj70ds6wchb-jellyfin-web-10.8.5

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
jellyfin
jellyfin-media-player
jellyfin-web

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/27nd9318qvg8klfvyxngrqj70ds6wchb-jellyfin-web-10.8.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jellyfin-web https://github.com/r-ryantm/nixpkgs/archive/9d0df3ead7bb61588b9fb754e30f706ca4c51d1a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/27nd9318qvg8klfvyxngrqj70ds6wchb-jellyfin-web-10.8.5
ls -la /nix/store/27nd9318qvg8klfvyxngrqj70ds6wchb-jellyfin-web-10.8.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

jellyfin
jellyfin-media-player
jellyfin-web

Maintainer pings
cc @nyanloutre @minijackson @purcell @jojosch for testing.

Result of nixpkgs-review pr 193007 run on x86_64-linux 1

3 packages built:

jellyfin
jellyfin-media-player
jellyfin-web


Auto Submit for Previous Weeks?

Hi,
This is more a feature request, currently when someone tries to Submit a week but missed a week it comes up with the warning:
"Please add previous weeks to approve".
When people have a couple of weeks off on holiday, they have to go back and do previous weeks. Is it possible to get it to auto submit any previous weeks by default? Like a round up? Wasnt sure if thats possible?
Thanks.
Mike

Hi Mike,
I think such a functionality might quite often be ticked also accidently. I would recommend to always check-in every week for approval after review, otherwise a week might be accidently submitted without even checking. Personally I think the risk is higher than the benefit. For this I close this issue, but I will include this into the "Discussions" sections, as if more find such a thing useful, it might become implemented.


[ re #473 ] Takahashi's complete development

Regarding #473, this is my first try in reformulating the diamond property (triangle property, actually) using Takahashi's approach. I am not a very good storyteller, so the informal development might need to be polished further. Please feel free to improve my wording.
Thanks to Andrea @iwilare for the naming and the triangle diagram, I implemented them altogether.
For the record, here are the details of my changes.

There are more high-level descriptions but less Agda code obviously. The main difference in terms of proofs between these two properties is that a single induction on M ⇛ N suffices to prove the triangle property instead of a double induction to prove the diamond property. The latter involves actually a number of cases eliminated implicitly by unification in Agda.

The diamond property for the parallel reduction in this setting becomes optional. The triangle property is now used to prove the strip lemma without extracting the confluent term and the required parallel reduction explicitly, i.e. this step

    with par-diamond mn mm'
... | ⟨ L , ⟨ nl , m'l ⟩ ⟩

I let the original proof be an exercise, so the reader might appreciate Takahashi's approach.

Oh, the last one,  the codepoint for the triple arrow ⇛ should be U+21DB instead of U+3015.

I think we can remove that statement now.
-Jeremy

On Jul 14, 2020, at 12:35 PM, Wen Kokke<EMAIL_ADDRESS>wrote:

@wenkokke commented on this pull request.
In src/plfa/part2/Confluence.lagda.md:

Berghofer's mechanization in Isabelle, which is based on an earlier
-article by Nipkow (JAR 1996).  We opted not to use the "complete
-developments" approach of Takahashi (1995) because we felt that the
-proof was simple enough based solely on parallel reduction.  There are
-many more mechanizations of the Church-Rosser theorem that we have not
-yet had the time to read, including Shankar's (J. ACM 1988) and
-Homeier's (TPHOLs 2001).
+article by Nipkow (JAR 1996).  There are many more mechanizations of
+the Church-Rosser theorem that we have not yet had the time to read,
Not sure if this admission is warranted in a textbook?

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or unsubscribe.

@jsiek are you happy to merge this?

Yes, thanks!

On Jul 24, 2020, at 1:46 PM, Wen Kokke<EMAIL_ADDRESS>wrote:
@jsiek https://github.com/jsiek are you happy to merge this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/plfa/plfa.github.io/pull/493#issuecomment-663655834, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA5IGPG5JPD47F4UQJSQVWDR5HCHBANCNFSM4OZRSFNQ.


TAB not working

Personally I do not have this problem, but one thing is that there is a bug with the display of mods in the background that are bugged with the display of the font.
Try to see if the key is well assigned and if there is no error in the console.
I/!\ In single player mode the tab is not set by default but I'll add it.
I'm sorry for the delay in answering, everything is explained in the readme


fix(server): Update context find logic in context-registry

If an OpenApi spec has multiple paths that share the same root, but different child paths, if those parent paths are a mixture of cases, the context registry stores the contexts for the paths once (which is correct), but the .find operation in context-registry expects the map key to match the path.
For example, suppose a spec has paths of myPets/vaccinations as well as mypets/visits. Now suppose the user has created a _.context.ts file at routes/myPets/_.context.ts. When a request comes in for myPets/vaccinations it will find the custom context, but when the request comes in for mypets/visits it will fail to find the custom context because it will look for a parent path of mypets and nothing exists in the ContextRegistry.entries for that key.
This PR fixes the path find logic.

Great find and fix! Can you add a unit test?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

14 of 14 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 87.296%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
3241

Relevant Lines:
3747

💛  - Coveralls


Update versions and release artifact

@gvolpe
Hi!
Are there plans to maintain the project?
I wanted to use it but it appeared that there is no released artifact. Once I just sourced from github dependencies were too outdated.
Thanks

At the moment, no plans to work on this project as I'm focused on Scala 3 and most of the type-level machinery only works in Scala 2. Maintainers always welcome.


[Bug]: Conduit Power doesn't prevent drowning

Mod Loader (Required)
Forge
Minecraft Version (Required)
1.19.2
Mod Version (Required)
Latest
Notes (Required)
https://github.com/LunaPixelStudios/SteamPunk/issues/5
latest.log (Optional)
No response

Fixed in v4.0.1 and v8.1.0.


Display node list for failing jobs even if its retry has passed

Sometimes tests fail but their retry passes.
It would be nice if ReFrame could display the node list of the failed tests even if their retry passes.
The optimal would be to have also the node list be printed for the passing retries.
This would allow to easily see if the failing tests have always used the same node list compared to the passing ones.

This seems to be related to #1093.


Add members to the project, and the members that are not added can also operate the item

In GitLab by @sokril on Oct 23, 2017, 16:07

In GitLab by @yhua123 on Nov 8, 2019, 07:39
by design

In GitLab by @sokril on Nov 8, 2019, 08:13
closed

In GitLab by @sokril on Nov 8, 2019, 08:13
assigned to @wknet123


pkgsMusl.go gccgo bootstrap

Description of changes
Partially fixes #196111 (pkgsMusl.go, but not pkgsStatic.go)
depends on #207562 and #207670
The official golang binary distribution that we use for bootstrapping go is only available for glibc.
We could consider switching to gccgo bootstrap in general (other than darwin, which is not supported in gccgo yet), to improve support for more platforms.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

go 1.20 requires go 1.17.13 to bootstrap, is this going to work for that?

gccgo 12.2.0 has functionality equivalent to go version 1.18, so yes 🙂

gccgo 12.2.0 has functionality equivalent to go version 1.18

How sustainable will this be going forward when go upstream starts bumping the bootstrap version? IIRC they are planning to switch to bootstrap x from x -2,  e.g 1.22 requires a 1.20 bootstrap.

In case gccgo falls behind too much to build the latest go version, I see multiple ways to handle it:

Just let it break; As long as we're just using it for native musl, anything is an improvement
Backport patches to make the latest version build. These will appear rather quickly, since other distributions are using gccgo for bootstrapping go (Alpine, spack)
A go compiler build only takes few minutes, not hours. We could build gccgo 12 -> go 1.20 -> go 1.22. And I doubt that gccgo will fall behind by more than 4 versions
Switch back to some other bootstrap option, like back to the upstream binaries (they can be used for musl with gcompat if needed)

Backport patches to make the latest version build. These will appear rather quickly ...

This at least seems promising but if they don't appear we probably will "just let it break".

Keep the latest version that can be built with gccgo around and lock pkgsMusl.go to that version.

Old versions of go are already a hassle, don't want to keep them around just for musl. Also this won't work with buildGoModule packages if their vendor hashes change between go versions. I wouldn't be surprised if had other breakage with buildGoModule as well.

We could build gccgo 12 -> go 1.20 -> go 1.22. And I doubt that gccgo will fall behind by more than 4 versions

Same again, don't want to keep old versions around.

Switch back to some other bootstrap option, like back to the upstream binaries (they can be used for musl with gcompat if needed)

I'd suggest looking into this as an fallback in case the backport patches don't work out for whatever reason.

Okay, agreed. If the gccgo bootstrap breaks, we look into running the upstream binaries via gcompat for musl.

go_1_20 (1.20rc1) is in master now so it can be included in this PR.

Rebased, added changes for go 1.20, pkgsMusl.go_1_20 builds

With this PR, pkgsMusl.go_1_20, pkgsMusl.go_1_19, pkgsMusl.go_1_18, builds fine for me.
But ldd ./result/bin/go, returns:

./result/bin/go: error while loading shared libraries: /nix/store/ayfr5l52xkqqjn3n4h9jfacgnchz1z7s-glibc-2.35-224/lib/libc.so: invalid ELF header

Is it expected?

You are probably using the wrong ldd

[yuka@yuyu:~/proj/nixpkgs]$ nix build .#pkgsMusl.go_1_20 -o go

[yuka@yuyu:~/proj/nixpkgs]$ ldd go/bin/go
go/bin/go: error while loading shared libraries: /nix/store/ayfr5l52xkqqjn3n4h9jfacgnchz1z7s-glibc-2.35-224/lib/libc.so: invalid ELF header

[yuka@yuyu:~/proj/nixpkgs]$ readlink $(which ldd)
/nix/store/s7ip867mrpnnjlppbnxlcsq10gv13x2x-glibc-2.35-224-bin/bin/ldd

[yuka@yuyu:~/proj/nixpkgs]$ nix build .#musl

[yuka@yuyu:~/proj/nixpkgs]$ result/bin/ldd go/bin/go
	/nix/store/w47181p6h6vchwq98yqk9sc3z4s9pxvq-musl-1.2.3/lib/ld-musl-x86_64.so.1 (0x7fb7f5f2e000)
	libc.so => /nix/store/w47181p6h6vchwq98yqk9sc3z4s9pxvq-musl-1.2.3/lib/ld-musl-x86_64.so.1 (0x7fb7f5f2e000)

[yuka@yuyu:~/proj/nixpkgs]$

result/bin/ldd go/bin/go

Thanks. You are always saving me from myself. :-)

@yu-re-ka Why is this still a draft? Is any further changes necessary?

It depends on https://github.com/NixOS/nixpkgs/pull/207670 which has not been reviewed yet

It depends on #207670 which has not been reviewed yet

But this PR already includes those commits.

But this PR already includes those commits. That PR could be closed in favor of this, no?

The content of that should still be reviewed by someone qualified to judge changes to the cc-wrapper.

Feel free to rebase and merge this after https://github.com/NixOS/nixpkgs/pull/207670 is in.


Patch 2 doesnt work

Hi is it just me or this mod doesnt work on new game patch my friend joins with discord but he isnt in hub and even if we change characters.

I can't even get to the part where I invite my friend lol


Extension causes high cpu load

Issue Type: Performance
Extension Name: vsliveshare-audio
Extension Version: 0.1.48
OS Version: Windows_NT x64 10.0.17134
VSCode version: 1.32.3

:warning: Make sure to attach this file from your home-directory: C:\Users\shubham\ms-vsliveshare.vsliveshare-audio-unresponsive.cpuprofile.txt :warning:
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Dupe of #1824


Polyfill parseInt() for radix 10 or switch to + or Number()

Apparently there's still a need to support really old js where parseInt() requires the radix parameter to work correctly.  See: Issue #14
The cleanest fix is probably just to switch to using Number().

This change probably just requires swapping out all occurrences of parseInt(..., 10) with Number(...).
For example, change:
const checksum = parseInt(clabeNum.substring(17, 18), 10);

to:
const checksum = Number(clabeNum.substring(17, 18));

And be sure to run $ npm test to verify all the test cases still pass.


Allow a panesActiveItem show a saved Dialog

Currently if you want to create a new file type eg:

writes and reads over ssh
writes and reads to a remote server
writes and reads via ftp

This functionality can be elegantly achieved by:
-inheriting from File from pathwatcher and over-riding read/write methods and nativeEvent Listeners.
-making a few minor changes to an extended text-buffer.
The only way of providing saveAs functionality is by overriding the activePane. or shim atom.showSaveDialogSync, both are quite large changes to the atom for such a small requirement.

cc @atom/feedback

cc @nathansobo is this something we want to add?

not sure what the test naming conventions are. Let me know if i got it wrong :smile:

Sorry, I should have thought of this sooner, but I'm a bit worried to have a synchronous design on this. Presumably you'll want to be able to show save dialogs that are rendered in HTML? If we do things synchronously, we won't be able to present it in a modal way. What's your use case for a synchronous API?

@nathansobo good catch. i'm building my package on this branch so i don't miss anything else :smile:. I will make the required changes

We need to document all the optional interface methods on pane items, including this one. We could deal with that in a separate PR though.

Ping me again when you've addressed my comments.

@billybonks have all the changes been made?

@benogle  sorry for the late reply. i still need to change one more test. i am having trouble rebuilding atom. i know it uses apm, but i upgraded npm to v3 and then probably upgrade node-gyp, which may be causing this error

going to close this for now, i don't need the api at the moment and i am sure the code has changed quite a bit.


UofT | Python Assignment 2 (with branch-name corrected)

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
I submitted my assignment 2 on Saturday, but today I realized that the branch name needs to be "assignment-2" rather than "assignment_2".
So I created a new branch "assignment-2" and creating this new Pull Request.
What did you learn from the changes you have made?
I learned that I can rename branch-name in Github, but that will also close any open Pull request.
So I went to the closed pull request and 'restored' that old branch.
I saw that I now have 2 branches related to assignment 2: old "assignment_2" and new "assignment-2".
So I am now creating this new Pull Request in new branch "assignment-2". But I will also leave the other Pull Request open - as proof to marker that I did indeed submit my assignment on time.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
See above.
How were these changes tested?
See above.
A reference to a related issue in your repository (if applicable)
N/A
Checklist
[x] I can confirm that my changes are working as intended

Thank you Pedram for having confirmed on Slack that my in-line comments in the initial submission are good. I will close the Pull Request now.


bubble up error from MAPL_getHorzIJindex function

Types of change(s)

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Trivial change (affects only documentation or cleanup)

Checklist

[ ] Tested this change with a run of GEOSgcm
[ ] Ran the Unit Tests (make tests)

Description
Stop the program when the grid is not a normal CS grid but computes the IJindex. It partially addresses issue #2850 . Eventually, I think the Grid class should have information about stretch and the ij index is computed according to the info the grid provides
Related Issue

The main problem is, if we commit this, then we can't run the stretched grid with GOCART, right? And, well, I'm not sure if that's desired.
We might need to support some sort of "slower"/"dumber" IJ method if running stretched grid.

I've drafted this PR until @tclune can have a think on it as well.

I can confirm this definitely works! I ran a stretched case and failed as expected.

Hmm. GNU is not happy:
/root/project/GEOSgcm/src/Shared/@MAPL/base/Base/Base_Base_implementation.F90:3458:33:

 3458 |        latRe = half_pi*sign(1.0, Zz)
      |                                 1
Error: 'b' argument of 'sign' intrinsic at (1) must be the same type and kind as 'a'
make[2]: *** [src/Shared/@MAPL/base/CMakeFiles/MAPL.base.dir/build.make:1094<EMAIL_ADDRESS>Error 1

The function for reverse Schmidt transform is added and the indices can be calculated now.  But I have not run the gcm with stretched grid. How can I do the setup and run? @mathomp4

I just tried a C270 run and:
 accurate_lon:    5.32325421858271
 accurate_lat:  -0.483265549030082
 edge_lat    :   0.350599277377354
 edge point  :           30
 Error: It could be
   1)Grid is NOT gnomonic_ed;
   2)lats lons from MAPL_GridGetCorners are NOT accurate (single precision from
 ESMF)

and I also saw:
 Error: Grid should have pi/18 shift

This PR also addresses issue #3060

Update: maybe this is on me. My Debug build worked. I'm going to nuke my release build and build from scratch.

Okay. It was me. I can run C270 now without errors, etc.

Oh. I guess I should test with v11. I just did it with v12.
Also, ping @mmanyin that the stretch grid might be okay again...

For posterity, I ran GEOS at C270 before and after this fix. This is the output of SO2EMVN before the fix:

and then after the fix:


How to use them in html ?

How to use them in html?

Just replace className with class and replace svg attributes like strokeWidth with stroke-width and copy the code in the react fragment and use that in html. 👍


Rename fabric

Fabric was a bad name choice on my part.
It's very common and conflicts with other "similar" projects that use it for different reasons.
Alternatives:

[ ] Net
[ ] Firmament
[ ] ... open to suggestions

/cc @jimeh @superdecimal


Add missing slot props in each-loop

Added missing slot properties to <slot> in {#each} block

Thank you!


feat: update rules as in Oisy Wallet

Motivation
The rules we had in this repo were not in-sync with what we have in Oisy Wallet main. Therefore this update which basically copy-paste the rules (basically because one rule is split in a separate file for svelte).
Notes
I did not copied the usage of @dfinity/utils at this point felt like we do not necessary needs the dependency for a single comparison.

I'm removing it because it's no in main in Oisy Wallet.
Once we got the lib used in Oisy, the rule can be moved and developed here.


Custom viewport column count

Problem Description
We are trying to customize different column counts for different viewports using CSS.
Having read following article, we know that, for the blog module, this works for desktop: https://intercom.help/elegantthemes/en/articles/2310880-change-blog-module-columns-number
What we want to achieve:
Three columns of the blurb module normally become only one row with a resolution under 980px. We would like to add additional viewports for modules containing rows.
For example, we want two columns between 1120px and 980px instead of three. (Above this we want to keep three or more columns.)

@justinbernard-dev, You have to write custom CSS to get a custom viewport.
Let me know if you have further questions.
Thanks
Mehedi

Hello @code-with-mehedi,
the problem is, that we don't know which classes to use in order to achieve custom viewports for columns or if there is any documentation on this. It seems quite complicated to achieve this and we don't find anything on the internet.
Thanks in advance for your help!
Justin

@justinbernard-dev, To do this you have to write a custom class in the container. Then you can write your media query. For that you need good knowledge of CSS media query.

@code-with-mehedi,
we have decent knowledge of CSS and, as we style our pages manually, also have custom classes in the container. We tried using the following code for the divi blog module, but it didn't work quite responsively and did not change the columns for our defined viewports.
Code we used
@media (min-width: 980px) and (max-width: 1120px) {
  .cu-blog .et_pb_salvattore_content[data-columns]::before {
	  content: '4 .column.size-1of4' !important;
  }

  .cu-blog .column.size-1of4 {
	  width: 24%!important;
	  margin-right: 1%;
  }
}