Can't find icon that exists in icons

I have this code:
let icon_path = freedesktop_icons::lookup("text-x-script")
            .with_cache()
            .find()
            .unwrap();

And a path to an icon I know exists:
/usr/share/icons/AdwaitaLegacy/48x48/mimetypes/text-x-script.png
But I get an Option::unwrap()' on a 'None' value error
What am I missing here?

You need to specify a theme, there is no global icon lookup in this crate.
        let icon_path = lookup("text-x-script")
            .with_theme("AdwaitaLegacy")
            .with_cache()
            .find()
            .unwrap();

Ah, that's unfortunate, but probably for performance reasons might make sense.
Perhaps a way to specify searching in all themes would be nice

I don't see why you would want to do that.

Maybe I'm using this crate wrong, or what I'm asking for is accomplished in a different way.
I want to access symbolic icons (for example, network-wireless-connected-75-symbolic), and multiple themes may have those icons. I don't want to specify a theme because I want it to be able to fall back to other themes if needed.
I can try to make this functionality myself within my program, but I don't know if this is the expected way to work with symbolic icons in the first place

The whole point of having icon themes is to have a set of icons that share a common look and feel. I understand the temptation of falling back to any theme when an icon is not found. But this is not the way this is intended to work.
Either your base theme has the icons you are looking for, or we fallback to the parent theme, in last resort we use the generic icons from "hicolor" theme.
If you really need to find icons regardless of the theme, you don't need the freedesktop icon implementation at all.

Thanks, that's actually really informative!


Slider inside iframe problem

First of all thank you for your wonderful slider - it really rocks.
We are building WordPress themes and we are using your slider a lot but there is one issue that I noticed in several themes, not only ours.
When I try to load site via iframe on iOS devices (iPhone/iPad) everything breaks...
Here are several examples:
http://themeforest.net/item/words-personal-blog-theme/full_screen_preview/9642505
http://themeforest.net/item/gridstone-elegantly-built-for-blogger/full_screen_preview/9413936
http://demo.mekshq.com/?theme=voice
Sorry for links, I don't have intention of promoting these themes, I just want to solve this problem together.
Thank you
Meks Team

Please upgrade to the newest version of Owl Carousel, released today, and re-open this issue if it persists.


Update task kind with get cached instance task kind

Changed

Updated TaskKind with GetCachedInstanceTaskKind.
Updated CreateInstanceTaskLazyNode to implement Node<Task<TaskKind>>.
Updated CreateInstanceTaskGraphExpandOperationContext maps to store dependencies instead of nodes.

Codecov Report

Merging #132 (7e98c81) into master (50e8adf) will increase coverage by 2.81%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
+ Coverage   96.21%   99.02%   +2.81%
==========================================
  Files          99       22      -77
  Lines        1768      411    -1357
  Branches      237       55     -182
==========================================
- Hits         1701      407    -1294
+ Misses         64        4      -60
+ Partials        3        0       -3

Flag
Coverage Δ

cuaktask
99.25% <ø> (ø)

iocuak
?

porygon
98.58% <ø> (ø)

porygon-typeorm
?

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

Impacted Files
Coverage Δ

...odules/CreateInstancesTaskDependenciesOperation.ts

...tanceTask/modules/TaskBuilderWithNoDependencies.ts

...cuak/src/createInstanceTask/modules/TaskKindSet.ts

...ask/modules/cuaktask/CreateInstanceTaskLazyNode.ts

...lectMetadata/utils/domain/updateReflectMetadata.ts

...utils/typeorm/findManyOptionsToFindOptionsWhere.ts

...ask/models/cuaktask/CreateInstanceDependentTask.ts

...tainerModule/utils/api/convertToContainerModule.ts

.../crud/modules/iocuak/TypeOrmReadContainerModule.ts

...rc/binding/utils/api/convertBindingToBindingApi.ts

... and 67 more

Continue to review full report at Codecov.

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


error: can't expand macros compiled by previous versions of Scala

When i use bazel to build my project, there is an error "can't expand macros compiled by previous versions of Scala", is there any solution which i dont need to modify my code ?

Please search issues for macros or see the README for some discussion. Macro jars need special handling.

I think it is the scala version that cause the problem. Bazel uses scala 2.11, but the jars I used is 2.10.4.   Did you have problems like this before?

Oh. The rules currently only support 2.11. We have a ticket to support
2.12. Supporting 2.10 is not a high priority.
You can't mix scala versions in jars.
On Thu, Feb 2, 2017 at 21:32 Huang Xinxin<EMAIL_ADDRESS>wrote:

I think it is the scala version that cause the problem. Bazel uses scala
2.11, but the jars I used is 2.10.4. Did you have problems like this before?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/127#issuecomment-277182748,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJdmAeZrH5_OmIr1ZK-syxuDPgc1odks5rYtfxgaJpZM4LtQqo
.


Coverage collection fails with 2.10.0-1.0.dev

package:coverage tests have been failing at head with the dev SDK for a couple days now. See: https://travis-ci.org/github/dart-lang/coverage/jobs/712777426
It appears the test application does not exit as this never returns here:
https://github.com/dart-lang/coverage/blob/7e18328a0fcb9353d2ad68748b66f5f5cb7c253c/test/lcov_test.dart#L184
Running the test application directly works as expected so something very odd is happening. To make things even more peculiar, my machine will eventually run out of memory due to a rogue Dart process, presumably the one started by the coverage test.

This might be a duplicate of https://github.com/dart-lang/sdk/issues/42819, are you able to reproduce this on bleeding edge ?
//cc @bkonyi

Yes, this is likely a duplicate of #42819 and should be fixed on bleeding edge.

Great. Thanks!


Seamless looping animations stutter at the last frame before looping when using AnimationTree

Tested versions
Godot 4.2.2 stable
System information
11th Gen Intel(R) Core(TM) i5-11400H @ 2.70GHz  RTX 3050 laptop 16 gb ram Windows 10 64bit
Issue description
https://github.com/godotengine/godot/assets/170141752/d2ed8aeb-c9f2-42b4-83df-d0bcac505e94
the animation works noramlly when not using blend or add nodes or when animated in animmator individually
Steps to reproduce
make looping animation, try blending or adding two of them.
Minimal reproduction project (MRP)
dont have

Please try 4.2.2, bugs might have been fixed since 4.2 was released (or specify that you're using 4.2.2 if you are)

Please try 4.2.2, bugs might have been fixed since 4.2 was released (or specify that you're using 4.2.2 if you are)
Also please specify your operating system

After testing this further i realised that the problem occurs when animationtree is not set to deterministic. however setting it to deterministic offsets some model parts - this can be fixed by filtering the animations for anyone finding this in the future.

Although there is no MRP, so I can't be sure, but I guess this could happen if the Left rotation animation has a Right static animation, and the Right rotation animation has a Left static animation. In that case, I would say this is not a bug.
If this is the case, it is recommended to use a filter or export from the DCC software in the first place. Alternatively, additive blending with NodeAdd2 may solve this problem.


UDF.get_method() used to choose implementation in UDFPartRunner._run_tile

Closes #1508

Shifts the string identifiers "frame", "tile", "partition" onto libertem.common.udf.UDFMethodEnum
Default UDF.get_method() performs the runtime protocol check previously in UDFPartRunner._run_tile and returns an enum member that UDFPartRunner matches against
Additional checking added to verify UDF.get_method() return is valid (both is an enum member, and actually implements the desired Protocol)
Additional test cases for invalid UDF.get_method() implementations

Notes:

Returning an enum member from get_method is technically a breaking change, because UDFMethodEnum.FRAME == 'frame' is False, if we want to be completely backwards compatible we would need to use a StrEnum (either from the PyPi package or as a backport from 3.11). I think the API is/was not public though so no real requirement to do this!
I had to add explicit assert isinstance(udf, UDFFrameMixin) inside _run_tile for each method in order to make mypy happy, this is just additional runtime overhead!
The current implementation (nor the previous implementation) doesn't store the processing mode that the UDF wants to use, which adds a little bit of overhead, but also more importantly doesn't guarantee the processing mode can't change in the time between tiling scheme negotiation and actually processing tiles. On the one hand this would be a 'strange' thing for a UDF to do, but could be a hard to track down bug.
I haven't added any documentation yet, would have to decide if this should be public facing?

Contributor Checklist:

[x] I have added or updated my entry in the creators.json file
[ ] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[x] I have added/updated test cases
[ ] I have included the rebuilt production build of the client (only if changes were made to the GUI)

Reviewer Checklist:

[ ] /azp run libertem.libertem-data passed
[ ] No import of GPL code from MIT code

/azp run libertem.libertem-data

/azp run libertem.libertem-data

/azp run libertem.libertem-data

/azp run libertem.libertem-data

/azp run libertem.libertem-data

CI failure is unrelated, I'm re-triggering failed jobs now

Looks good! CI is still failing, somewhere in pyxem → pyFAI → numexpr. As far as I can see, a new numexpr release should come soon (they are aware of the breakage), so I'm inclined to just go ahead and merge now.

I did also see some logs like "No space left on device" during tests on failed CI runs ?
Thanks for all the reviews and feedback!

Looks good! CI is still failing, somewhere in pyxem → pyFAI → numexpr. As far as I can see, a new numexpr release should come soon (they are aware of the breakage), so I'm inclined to just go ahead and merge now.

I did also see some logs like "No space left on device" during tests on failed CI runs ?

Yeah, that's a curious issue with our ci setup - after some time, /dev/shm in some containers runs out of space - didn't manage to 	find time to track down the reason for that, and re-creating the containers is a quick workaround...

Thanks for all the reviews and feedback!


Assertion Strategy should allow any param in claim set

Related to #197, allow any param for build_assertion method.
Let me know if this need something else to get merged.

This will be fixed by #220 .
Thanks for your work on this @fabn


web: updated initial link of code coverage

It's trivial. This commit is to update an initial web-link of code coverage.
Signed-off-by: Geunsik Lim<EMAIL_ADDRESS>
[Template] PR Description
In general, github system automatically copies your commit message for your convenience.
Please remove unused part of the template after writing your own PR description with this template.
$ git commit -s filename1 filename2 ... [enter]

Summarize changes in around 50 characters or less

More detailed explanatory text, if necessary. Wrap it to about 72
characters or so. In some contexts, the first line is treated as the
subject of the commit and the rest of the text as the body. The
blank line separating the summary from the body is critical;
various tools like `log`, `shortlog` and `rebase` can get confused
if you run the two together.

Further paragraphs come after blank lines.

**Changes proposed in this PR:**
- Bullet points are okay, too
- Typically a hyphen or asterisk is used for the bullet, preceded
  by a single space, with blank lines in between, but conventions vary here.

Resolves: #123
See also: #456, #789

**Self evaluation:**
1. Build test: [ ]Passed [ ]Failed [*]Skipped
2. Run test: [ ]Passed [ ]Failed [* ]Skipped

**How to evaluate:**
1. Describe how to evaluate in order to be reproduced by reviewer(s).

Add signed-off message automatically by running **$git commit -s ...** command.

$ git push origin <your_branch_name>

:memo: Version: 1.20180820. Thank you for submitting PR #231. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://invain.mooo.com/.


pgbouncer=true flag not working as expected

Bug description
Related thread
https://github.com/prisma/prisma-client-js/issues/651#issuecomment-652439460
Using DigitalOcean Connection Pools

Connection String
postgresql://xxxxxxx:xxxxxxx@xxxxxxx:25061/Pool?sslmode=require&pgbouncer=true&schema=xxxxx

How to reproduce

Create new DO managed postgres db
Deploy Prisma schema (without pgbouncer)
Add connection pool
Connect using the connection pool string above (with pgbouncer)
Run app/prisma studio - execute 3 or 4 actions

Results in this error:
Error: Error occurred during query execution:
ConnectorError(ConnectorError { user_facing_error: None, kind: QueryError(Error { kind: Db, cause: Some(DbError { severity: "ERROR", parsed_severity: Some(Error), code: SqlState("26000"), message: "prepared statement \"s162\" does not exist", detail: None, hint: None, position: None, where_: None, schema: None, table: None, column: None, datatype: None, constraint: None, file: Some("prepare.c"), line: Some(512), routine: Some("FetchPreparedStatement") }) }) })
   at PrismaClientFetcher.request (/Users/aaron/.cache/prisma/studio/Users-aaron-Desktop-Ember/runtime/index.js:1:185866)
   at processTicksAndRejections (internal/process/task_queues.js:97:5)

Prisma information
Fairly large schema (can't share specifics)
Using nexus/schema (although reproducible in prisma studio also)
Environment & setup

OS: Mac OS
Database: PostgreSQL
Prisma version: 2.1.3
Node.js version: 12.16.3

Thanks for the information here.
We have e2e tests for DO's PgBouncer, so we are sure it works fundamentally: https://github.com/prisma/e2e-tests/tree/dev/databases/digitalocean-pgbouncer
The first and only difference that pops up for me is that our database connection is not using sslmode=require and does not specify a schema via &schema - but that should not make a difference.
We will look into this.

I can confirm I'm experiencing the same behaviour without the sslmode and schema query params - also reverted to 2.0.0 and had the same issue

@aarshaw Did you re-generate the client after adding pgbouncer=true, I am unable to reproduce this issue.

Yeah, deleted node_modules, regenerated client etc. - the connection string is provided by an env var is that an issue?

What I'm seeing in the tests is that you're executing one query with and without the param - this works for me. Where it isn't working is after a few seconds usage - presumably, after it executes more than one query - have attached a quick screen recording to illustrate -
recording.mov.zip

Thanks, in my local reproduction, I did try multiple queries (even some load testing). Thanks for the video, I will try to give it another go.

Thanks - let me know if you need any further info relating to the environment/schema to replicate

@aarshaw Are you able to reproduce this without using studio? By hitting a script/endpoint multiple times?

Yeah - I'm able to reproduce using:
import { PrismaClient } from "@prisma/client";

const prisma = new PrismaClient();

const run = async () => {
  await prisma.xxxxx.findMany({
    take: 100,
    select: {
      ....
    }
  });

  await prisma.yyyyy.findMany({
    take: 100,
    select: {
      ....
    }
  });

  await prisma.zzzzz.findMany({
    take: 100,
    select: {
      ....
    }
  });

  await prisma.aaaaa.findMany({
    take: 100,
    select: {
      ....
    }
  });
};

run();

Where it crashes on the third execution (zzzzz)

Thank you, is it possible for you to share your schema and some seed data at<EMAIL_ADDRESS>We are happy to sign an NDA if you want. With that, a minimal reproduction would be much easier for us.

Thank you @aarshaw for sharing the schema based on the information we were able to create an isolated minimal reproduction: https://github.com/divyendu-test/prisma-2907
This has something to do with a relation and enums on both sides of the relations used in conjunction with pgbouncer.

I can also reproduce this

Update: The underlying issue is fixed and will be released with 2.4.0 (next week). If you want to try it out now, you can use npm install -g @prisma/cli@dev


Fixed issue no.655 Login/signup page not properly aligned

Pull Request Checklist

[x] I have added screenshots and videos to show before and after the working of my code.
[ ] I have ensured that the screen size is set to 100% while making the video.
[x] I have synced the latest fork with my local repository and resolved any conflicts.
[x] I have mentioned the issue number which I created before making this PR .(format to mention issue number is : fixes #(issue number) )
[x] I understand that if any the above conditions are not met , my PR will not be MERGED .

Before work

After Work

fixes #655


SMART Status of Host with smartctl?

Version
Version 10.0.19042.1645]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.10.16
Distro Version
latest Fedora Remix
Other Software
smartmontools --> smartctl
Repro Steps
Hello,
i want to achieve the following: Query the SMART-Status via SNMP from Windows. Ultimately, this doesn't seem to work without a extended agent on windows - and therefor needs unmanaged code to do so.
Anyway, i try to hack my way around that a bit and have set up a WSL2 with snmpd and extended it with a script. With that script i can query via "smartctl -H /dev/sda".
This setup seems to work, but i am not sure if the returned Health-Status is plausible and display the REAL health status.
smartctl --all crashes due to
">> Terminate command early due to bad response to IEC mode page
A mandatory SMART command failed: exiting. To continue, add one or more '-T permissive' options."
Does someone have any insights on this and how i could test the SMART output?
Expected Behavior
Displaying actual information about SMART Status
Actual Behavior
Probably not showing actual SMART Status
Diagnostic Logs
No response

The /dev/sda in WSL doesn't represent a real device, it is a virtual device.

Sad to hear, thanks though.
That calls for even more hackery!


Plugin fails to get the compileJava task when the apply() method is called

This plugin causes the following error when applied to my project:
 Failed to apply plugin [id 'org.jayware.osgi-ds']
   > Task with path 'compileJava' not found in root project 'custom-java-command'.

That's because in the apply() method, this Plugin assumes the java plugin already added the compileJava task to the project, but that is not always the case as the order in which plugins are applied cannot be assumed by Gradle plugins.
The correct way to "use" a task from a different plugin is to use Gradle's callback mechanism, something like this (in Groovy):
project.tasks.withType( AbstractCompile ) { AbstractCompile compileTask ->
    task.inputs.files( compileTask )
    task.mustRunAfter( compileTask )
}

Notice that this will allow this plugin to support other languages in the JVM, including Kotlin, Scala, Ceylon and Groovy.

Hello Renato, i apologize for the late feedback. I have changed the apply() operation as you suggested and released a new version (0.5.4) of the plugin.
Please let me know whether it solves your issue.


Update office-365-groups-ww.md

Updated reports in the security and compliance center link

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7093

@kwekuako Please merge this PR. Thanks


Add changelog for 2.2.2

I forgot to do that before creating the release!

Also https://github.com/sabre-io/xml/blob/master/lib/Version.php

@staabm should I quickly delete the "release 2.2.2" that I did?
Or bump changelog and Version.php again to 2.2.3 then do a 2.2.3 release?

@staabm should I quickly delete the "release 2.2.2" that I did?
Or bump changelog and Version.php again to 2.2.3 then do a 2.2.3 release?

Feel free to re-tag .. its just a minor thing missing.. wont be a big problem for people which already have the "bogus 2.2.2" tag


Can't add collaborators

Command attempted:

What happened:
When I tried to add another people to my repo, the targets didn't show up from the search engine. Several other friends have the same problem. Can this be fixed as soon as possible?
More info:

Can someone have a look at this AS SOON AS POSSIBLE? It's been a few days already...

Ok, one of the "add people" button is fake...


using Julia reserved names for Python keyword arguments

I would like to call the scipy function scipy.interpolate.Rbf with the argument function = 'linear'. But it gives the error ERROR: LoadError: syntax: unexpected "=". I think this is caused by the term function. Is there another way I can call this function with the keyword argument?
@pyimport scipy.interpolate as sp
rbf = sp.Rbf(x, y, c, function = "linear")

Oh, good question, I didn't think about how to use reserved names for keywords.   Probably we should add an additional optional argument to the low-level pycall function (https://github.com/stevengj/PyCall.jl/blob/fcbe0be562d347843d107617ce178cfa70ed1a54/src/PyCall.jl#L343-L368) to specify this: just a dictionary of names=>values.

julia> sp.Rbf(x, y, c; :function=>"linear")

Ah, thanks @yuyichao, I didn't realize that this syntax existed in Julia.

Thank you


306 sign in and modal (new)

Dev Summary

Mocks

Test Plan

repro steps:

localhost:3000
click X button
expected:

Resources

stackoverflow is cool

https://github.com/Nonprofit-Exchange-Hub/web-app/issues/306 Previous Issue

@esteban-gs Ready for review

New commit, followed all of the comments. Left the Modal.tsx comments for console.log so Dennis or I can finish with the handling of clicking outside of the modal.

@esteban-gs fixed the above
I couldn't replicate the "... error when closing the modal on my local dev instance."


[WIP] require_* doxygen and moving the wiki

Summary
I really don't like the wiki. I find it hard to remember where is what and things are not terribly well sorted. While I was building the doxygen module for the new require_* type traits I figured we have a math site so why not structure it a bit and put stuff over there.
You can run make doxygen to see the new site (You'll need the graphvis package installed for making the call graphs).
There's now a tab on the LHS that groups a few of the wiki pages together. Under the modules tab you'll see functions and classes grouped together. Still a WIP, I'll tag people soon so they can make sure their wiki stuff is correct
I haven't finished fiddling with the CSS stuff but I'll add a little readme to the doxygen so you can see what each weird thing is for
Checklist

[x] Math issue #1396

[x] Copyright holder: Steve Bronder
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[ ] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[ ] the code is written in idiomatic C++ and changes are documented in the doxygen

[ ] the new changes are tested

@seantalts can we install graphvis on wherever Jenkins is? Would be nice to have the call and callee graphs

Sure! What's the exact steps you need me or Nic to do?

Is the OS linux or mac?
For mac
brew install graphviz

apt install graphviz

Hmm, can you find a docker image that has that plus whatever other stuff you need and use that? Jenkins makes that pretty easy especially if the docker image exists already. Is this part of doxygen?

Happy to install it on the hardware machines we have and then we could restrict the build to those machines as well, but the docker thing would let it run in more places.

This docker file seems to have the right stuff?
https://github.com/BrainTwister/docker-devel-env/tree/master/doxygen-1.8.14
Do we have a repo of dockerfiles available somewhere?

Cuz I can probably just slap that stuff on top of our current dockerfile

@wds15 if you get some time next week can you check out the .md files for threading I grabbed from the wiki? Just to make sure they are both up to date and are titled right. If you want to make any changes feel free to push directly to the branch
https://github.com/stan-dev/math/pull/1397/files?file-filters[]=.md#diff-d6281a65fbec195375fbb1a0544e74b1
https://github.com/stan-dev/math/pull/1397/files?file-filters[]=.md#diff-d251a2d144d438d362397278cdb2aa7c

@syclik @bob-carpenter not sure who to tag on review for this. You can run make doxygen to see the site locally.
@charlesm93 This might also be a nice place to have docs for the ODE solvers. You can add a \defgroup like I have here for the probability distributions then tag the solvers docs with \ingroup your_ode_solver_tag
https://github.com/stan-dev/math/pull/1397/files#diff-86921eaf1325ba012f545f449fc93cfcR4
@rok-cesnovar if you have anytime next week can you add some stuff for the section on OpenCL? You can just either push directly or make a branch no worries either way.
You can add the text directly where we make the opencl /defgroup
https://github.com/stan-dev/math/pull/1397/files#diff-7be8a76b90a41c3a23528e2b98e3a2a0R6

Nice work Steve from what I skimmed through. Yes will definitely add some OpenCL docs. Assigning myself.

Cuz I can probably just slap that stuff on top of our current dockerfile

We don't really have a dockerfile for doxygen stuff (or for anything outside of stanc3, pretty much? There might be one in CmdStan for generating the manual). It'd be really easy to add if you can find a published image on docker hub or we can just add a Dockerfile in the repo here; it might be something simple. I might try with this image first (since building images is slower): nnadeau/docker-doxygen (from https://hub.docker.com/r/nnadeau/docker-doxygen/) and see if that works. I'd add it to Jenkins here but there's a merge conflict; to add it you should just be able to add a top level stage like this:
        stage("Doxygen") {
            agent { docker { image 'nnadeau/docker-doxygen'}}
            steps {
                sh "make doxygen"
            }
        }

Cool! I'm afk atm but will fix that merge when I get a chance.
Would it be useful for our users (not math but stan language users) if we had a repo of stan docker files? Could also throw them up on dockerhub

I think the typical thing is just to make docker images and upload them to
docker hub (though we could make a repo that automatically creates and tags
new versions and uploads them to docker hub). I'm not really sure what the
size of the audience for those is given most folks who use Stan do it
through RStudio and the cmdstan users I know wouldn't use it through
docker. My calculus here has been that if it was useful to someone they'd
do it and hopefully share it.
On Sat, Oct 12, 2019 at 10:52 Steve Bronder<EMAIL_ADDRESS>wrote:

Cool! I'm afk atm but will fix that merge when I get a chance.
Would it be useful for our users (not math but stan language users) if we
had a repo of stan docker files? Could also throw them up on dockerhub
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/1397?email_source=notifications&email_token=AAGET3HF6AITWSILWOT67R3QOHQEJA5CNFSM4I7QO3Q2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBCBDDI#issuecomment-541331853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGET3ADMSMXPFXQ5STFFOLQOHQEJANCNFSM4I7QO3QQ
.

@seantalts do you know how I unstash inside of the docker container for Jenkins? I looked up some of the sidecar stuff and couldn't figure it out. Gave the below a go but can't sort out the right command
        stage("Doxygen") {
          agent { docker {image 'nnadeau/docker-doxygen'}}
          steps {
              deleteDir()
              unstash 'MathSetup'
              sh "mkdir -p doc/api"
            	sh "doxygen doxygen/doxygen.cfg"
          }
          post { always { deleteDir() } }
        }

Looks like the unstash claims to be executing successfully but for some reason mkdir -p is failing in a way I didn't think was possible...

Hmm, I can't immediately figure out what's wrong but I'll have more time next week after the release! Sorry about that.

Oh! Forgot about release time, yeah we can totally put this on the backburner for now

@seantalts would you have a chance tmrw to take a peek at this?

There are merge conflicts btw. What do you want me to look at? I don't see any docker stuff in the Jenkinsfile...

@seantalts whups my bad thought I fixed that
Do you know how I unstash inside of the docker container for Jenkins? I looked up some of the sidecar stuff and couldn't figure it out. Gave the below a go but can't sort out the right command
        stage("Doxygen") {
          agent { docker {image 'nnadeau/docker-doxygen'}}
          steps {
              deleteDir()
              unstash 'MathSetup'
              sh "mkdir -p doc/api"
            	sh "doxygen doxygen/doxygen.cfg"
          }
          post { always { deleteDir() } }
        }

Otherwise we can just slap graphviz on the mac and restrict it to build there

Oh right. That's how you do it. I'm not sure why your mkdir -p command is failing. Do you actually need the call to mkdir?

You might also try not deleting the dir and not unstashing - Jenkins is supposed to handle that stuff for us hypothetically. I really wish there was a better way to test this stuff locally, but I think it involves running a Jenkins docker image and is not the easiest. You can also play around more quickly with the replay feature: https://www.cloudbees.com/blog/replay-pipeline

The reason we have the unstashes in all the other stages is because github would spuriously fail every now and then.

Wait, nevermind - the whole pipeline is set up not to do a checkout, so you need the unstash :/

re: pros and cons of getting docker to work: I don't have a strong opinion, I'll just tell you the stuff I'd take into consideration evaluating the tradeoff. First, if it uses a docker image, it can run on any machine with docker and therefore builds are less likely to queue up with a single bottleneck. It also means we have less manual sys admin dependency hell to deal with w.r.t. keeping each individual machine up to date; eventually it could be a goal to move all of the linux builds to docker (if only we could do the same with Windows and Mac OS X and thus script those dependencies as well). Docker images should make it easier to debug locally as well, though that obviously depends on Jenkins have a normal-ish Docker environment. It tends to run all docker builds as a specific user, which often doesn't have permissions inside the docker image.
Oh, that's probably what it is :P Try
                    agent {
                        docker {
                            image  'whatever'
                            args '-u root'
                        }
                    }

args '-u root'
LoL, lemme give it a shot!

Haha I thought make just vanished from the machines xD
You can build your own image like here if you need to extend it See the below Dockerfile.
FROM alpine:3.6

RUN apk update && apk add \
  doxygen \
  graphviz \
  ttf-ubuntu-font-family \
  make

Ty!! Yeah I have to put on my big boy pants and make my own image haha

Would you be interested in making sure this works with the CMake Doxygen build target? I can get you a script that will build it with CMake. If not, nbd, I'll add it to my to-do list.

That would be cool! What is the timeline till we can move over to the monorepo? I think as long as the computer has graphviz (and cmake can detect that) then it should be fine

You'd be surprised how often I'd thought: eh, should be fine. For example, currently the math documentation doesn't build on Windows with MikTex. So more testing the merrier, hah. I'll prep the script for you tonight or tomorrow, just plop it in the base source directory and run CMake.

Actually, all you have to do is copy and paste the following into a directory:
find_package(Doxygen REQUIRED dot)
file(GLOB_RECURSE stan_public_headers ${PROJECT_SOURCE_DIR}/src/*.hpp)

# un-comment to include these in the built docs. these should be in the same folder relative to here
# set(PROJECT_LOGO stanlogo-main.png)
# set(HTML_FOOTER doxygen-footer.html)
set(DOXYGEN_GENERATE_LATEX NO)
set(DOXYGEN_GENERATE_HTML YES)

doxygen_add_docs(stan_documentation
        ${stan_public_headers}
        COMMENT "Generating documentation...")

Will ^ install dot if it does not already exist?

No, it just assumes you have Dot installed somewhere where it can find it (usually on $PATH). For a dependency like that I would handle it in Conan, probably by asking it to shell out to the system package manager / Brew / (scoop | chocolatey | msys2)

Neat! I'll make the dockerfile for jenkins tonight and look this week to make a PR for the cmake stuff in the monorepo

@serban-nicusor-toptal dumb Q, I'm trying to do what you are doing with the dockerfile in stanc3 (dockerfile and jenkinsfile below)
https://github.com/stan-dev/stanc3/blob/master/docker/debian/Dockerfile
https://github.com/stan-dev/stanc3/blob/master/Jenkinsfile
I'm getting an error that Jenkins can't find the dockerfile. Do you know how to make this work? If not then we can just ditch the graphs in the doxygen and build the old way
Jenkins error:
https://jenkins.mc-stan.org/job/Math Pipeline/job/PR-1397/43/console
java.nio.file.NoSuchFileException: /home/jenkins-slave/jenkins-slave-files/workspace/Math_Pipeline_PR-1397/doxygen/docker/alpine/Dockerfile

Hey, your code looks fine at first look, I would make sure that Jenkins pulls the correct files ( git status or ls -lart doxygen  ) from this step here. Take a look, when I get home later I will be able to debug.
You can also use our docker registry here if everything else fails.

I hope people don't mind but I wanted to show everyone the new site so I slammed up it up live. I think it looks pretty nice!
http://mc-stan.org/math/

Nice! Does anyone want to look this over / approve? I think mostly the thing to look at is the Jenkinsfile changes

Am I doing something wrong? The only change I see is that it now updates the Math library documentation whenever we merge to develop?

Oh lol forgot I removed stuff.
What do you think is the best way to review this? essentially it just has a bunch of docs and stuff and makes some groups for doxygen modules

@seantalts after I fix this conflict can we merge this? I have to keep updating it otherwise

@rok-cesnovar @bob-carpenter can we merge this once it passes? It touches a good number of comments so I have to keep dealing with little merge conflicts
Site looks nice tho'!
http://mc-stan.org/math/

Hey yeah sorry, I meant to reply - I think @bob-carpenter or @syclik are the resident doxygen experts. It all looks good to me but I honestly don't know what our policies on doc are at this level.

Just took a peek. It looks good. I'll second @seantalts's comment (in his edit): moving doc to GitHub would be great! I know the wiki is supposed to be a git repository, but the interface there is different from the code and not chasing doc in two different places is a good thing.
@SteveBronder, one major comment: the build instructions in the README.md are incorrect. (see https://github.com/stan-dev/math#installation) It's missing a critical part indicating that the TBB must be built, not just have the sources available. It doesn't look like this is something you did. Let me know if you want me to split it out into a different issue or if you wanted to tackle it since you've got all the doc changes happening at once.

@SteveBronder, one major comment: the build instructions in the README.md are incorrect. (see https://github.com/stan-dev/math#installation) It's missing a critical part indicating that the TBB must be built, not just have the sources available. It doesn't look like this is something you did. Let me know if you want me to split it out into a different issue or if you wanted to tackle it since you've got all the doc changes happening at once.

We could solve it here but that's not really something I know about. It may be easier to have @wds15  do that in a separate PR

Looks great to me!

Good to go?

Much appreciated!
Apologies to the folks that have to do merge conflicts because I touched your comment docs


Not working with support design 26

Not working with support design 26. And why it's deprecated now?

I have the same problem too . its not working with support design V26

I can suggest you another lib. Pm me
On Aug 30, 2017 5:15 PM, "emirmg"<EMAIL_ADDRESS>wrote:

I have the same problem too . its not working with support design V26
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/henrytao-me/smooth-app-bar-layout/issues/210#issuecomment-325972200,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJT0dclaeJ6F7TLsD54ACQXSj3tQbmvTks5sdVJSgaJpZM4PDwGG
.

I have the same problem too .


Uk comparison data

Adds plots of the three UK data sources for each region.

Uses forecast package to give a 7-day moving average for the plotted data
Replaces Rt plot function with something more generic used for both Rt and data for all regions
Updated fig captions

Partial screenshot attached

Other than small comments above this looks really good :)

Thanks for the quick review (sorry this took ages for me to get to - my github hated this repo and I took a while to sort it out). Updated to resolve comments though see above re file locations.

Ah that will be because of my purge of the history. You need to delete anything local and use the clone instructions in the readme (as this avoids pulling the full history of the covid-rt-estimates repo.

Also I think this is the first use of the forecast package in this repo. Can you add it to the DESCRIPTION so it will get installed elsewhere.

What does that plot look like now?

last comment from me: using scales::comma might be nice for the y axis of the log scaled count plots.

Do we know why the plots are more jagged than the other ones on the UK page?

vs.

(not passing judgement on which looks better, just curious)

I think it is because they are made with the summarised csv that is rounded for end users and so only goes up to 0.01 whereas the plots I am making internally aren't rounded (I think).

Sorry, forgot to post what this looks like now - attached.

Looks good to me. Could we standardise to showing every week label?

Thanks for the comments both, updated screenshot below.

Looks great! Last minute thought: shall we have a different colour scheme for this so as not to confuse with the estimate / partial estimate / forecast distinction? I guess it's pretty obvious that they're not this but perhaps this could make it slightly clearer that these plots are a bit separate from the other ones?

Ah good call Seb. I used this by default as it's the scheme we used in the paper, but agree it probably makes sense to differentiate from the estimate/partial/forecast categories when they're all on the same page.
I've tried with green, blue, red (different though not very pretty) - but open to suggestions.

Reading through this again there might need to be some explanation of what the reference dates are for each dataset and why the estimates are lagged (both in these summary plots and then estimates presented in the regional breakdown below). Its now the only page where this is the case.

Might be nice having the data source and lag explanation in another PR.


Cannot select individual tokens in a squad for combat attacks

This is either a feature request or a bug. And I understand if it is simply not achievable.
The scenario is that I have a "squad" of tokens, where a group are following a leader. I would like to be able to select each token in the squad in order to initiate combat for that token. (For example, to have that token use a weapon in the macro bar, I need to select the token, then select the weapon macro to attack a target.) But because the tokens are following the leader, I am only able to select the leader. The only way I see to select an individual token to use for combat is to first remove the token from the group. Further, I am unable to open the character sheet corresponding to any given follower token.
This is, of course, less than desirable if one has a lot of squads in a major battle scene. The moving around by squads is great, but having to de-construct the squad and re-construct it each round of combat is less than ideal. But maybe I am missing a simple way to achieve this!
Again, I understand if this is simply beyond what the module can achieve. But if it is possible, it would be a great improvement!
(Incidentally, the same is not true of say, lights attached to a group. I can attach a light and then, while it is part of the group, right-click to toggle the light on and off, and double-click to open the light settings panel.)

See the readme Enable selection of an attached element by selecting the element and then pressing the unlock button. To prevent the selection again use the lock button.. If you can't get it to work you can get in touch over discord.

Thanks!


Signed xattrs

Currently, there is no end-to-end authentication on xattr names and values.  A compromised MS could equivocate between gateways about them.  Include a per-attribute signature, so even getxattr() can verify the authenticity of the name and value.

This will be addressed in the following way:

gateways sign each metadata entry the replicate to the MS (i.e. the MS is no longer a trusted source of metadata; only a publicly-routable cache for metadata from a volume's coordinators).
coordinators always host the set of xattrs
coordinators replicate the (signed) hash of the xattrs alongside the (signed) xattr_nonce
the xattr_nonce becomes a monotonically-increasing version number, which reader gateways cache locally
other gateways send SETXATTR and REMOVEXATTR messages to the coordinator, which then generates and uploads a new hash and new signed metadata entry

This is possibly fixed in my local tree.

Fixed in master.


Mono support

This adds basic mono support and XBuild support.
For scripts that have defined classes, properties, or methods, you will need to wrap them in #region "Extras" code regions to be properly processed.  This is a current limitation of the mono scripting and should eventually be fixed.
You can invoke XBuild to build projects and solutions.  There is also a MSOrXBuild alias which will route to MSBuild on windows and XBuild on linux/mac.  You can still explicitly use XBuild on windows if you like.
There are still some limitations in locating tools used in some aliases, but the basic scripting itself should be functional :+1:

Merged.

woot! :+1:


jotta-cli: 0.6.21799 -> 0.6.24251

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/jotta-cli/versions.
meta.description for jotta-cli is: "Jottacloud CLI"
meta.homepage for jotta-cli is: "https://www.jottacloud.com/"
Updates performed:

Version update

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.6.24251 with grep in /nix/store/xzi40cc7y4wg9vji9qc0ynss05czkmfc-jotta-cli-0.6.24251

directory tree listing: https://gist.github.com/b0b393ac4b680c510e84b14fda23f00d

du listing: https://gist.github.com/96b703732201721b1797b877bd0a2469

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

1 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
jotta-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xzi40cc7y4wg9vji9qc0ynss05czkmfc-jotta-cli-0.6.24251 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's 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 jotta-cli https://github.com/r-ryantm/nixpkgs/archive/41b1f0202764bfff03fdc6f988d567bcf452bef8.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/xzi40cc7y4wg9vji9qc0ynss05czkmfc-jotta-cli-0.6.24251
ls -la /nix/store/xzi40cc7y4wg9vji9qc0ynss05czkmfc-jotta-cli-0.6.24251/bin

cc @evenbrenden for testing.

Tested and verified jotta-cli and jottad binaries.


UsersStoryPlaceholder: Add delete endpoint and seed field

-Delete endpoint to /player that takes a playerId.
-Session has new seed field that contains a random UUID which is converted to a string.
-Delete endpoint tests also added

@Twhining I will look at it later today, ok?

@C0DECYCLE yeah thats fine


Taverna 2's workflow running feature

I have implemented Taverna 2's Workflow running functionality

@therajanmaurya
Please review it.
Tell me if there is any suggestion or change .
Thanks

Let wait till travis build will pass.

I've tested it and it is working well. A few comments:

Taverna player portal seems to need valid email address and not username. Maybe change the hints.
The first time you try to run a workflow the app asks for user and password even if you have set them using the settings menu.
The app says "Downloading workflow" when it should probably be "Uploading workflow"
Workflow run button doesn't seem to work for "Favourite" workflows
Although it tells you that you can only run t2 workflows (which is a great feature by the way) maybe it should just grey out or not even show the run button instead.
Overall, really good. I like the progress bar at the top. Later on it would be good to use native android UI instead of embedding the portal for inputs and results but this is great progress to get to this point.

Is this otherwise good to go, @ianwdunlop ?
Perhaps we can accept this and make Jira issues for your comments?

@stain
I have added 1st, 4th and 5th comment by @ianwdunlop.  Remaining I will add soon


Add a Gitter chat badge to README.md

Yelp/undebt now has a Chat Room on Gitter
@evhub has just created a chat room. You can visit it here: https://gitter.im/Yelp/undebt.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Coverage remained the same at 96.829% when pulling 39cfe3b59fd1a0eadc4c7a2020de4c336e8f52e4 on gitter-badger:gitter-badge into 7e9b1b583b89be447e5ae92654c2fea79c626b3d on Yelp:master.

:shipit:


Nothing to see at http://localhost:9090

Hi,
I am trying to test it out...
I successfully installed the app globally via: npm install -g peerwiki and then run via: DEBUG=peerwiki peerwiki 9090 or DEBUG=peerwiki peerwiki 9090 --use-index in either way I do not see anything in Google Chrome at: http://localhost:9090/BitTorrent or http://localhost:9090
Getting Error code: ERR_CONNECTION_REFUSED .
Here is the log:
EBUG=peerwiki peerwiki 9090
Verifying previously downloaded data...
  peerwiki engine is ready +0ms
  peerwiki fetching required wikipedia header... +2ms
  peerwiki connected to 0/0 (0 B/s) peers +5s
  peerwiki connected to 0/0 (0 B/s) peers +5s

and so on...

Does it keep saying 0/0 peers?

Yes, even if running several minutes, still getting just: peerwiki connected to 0/0 (0 B/s) peers +5s

BTW: I am not using any firewall and I am able to download torrents via programs line µTorrent.

Same issue


load_model failed

System Info
[pip3] flake8==7.0.0
[pip3] flake8-bugbear==24.2.6
[pip3] mypy-extensions==1.0.0
[pip3] numpy==1.26.4
[pip3] torch==2.2.1+cu118
[pip3] torchdata==0.7.1+cpu
[pip3] torchtext==0.17.1
[pip3] triton==2.2.0
[conda] numpy                     1.26.4                   pypi_0    pypi
[conda] torch                     2.2.1+cu118              pypi_0    pypi
[conda] torchdata                 0.7.1+cpu                pypi_0    pypi
[conda] torchtext                 0.17.1                   pypi_0    pypi
[conda] triton                    2.2.0                    pypi_0    pypi
Information

[X] The official example scripts
[ ] My own modified scripts

🐛 Describe the bug
I followed the instructions to install and infer Llama-2-chat-70b.
pip install --extra-index-url https://download.pytorch.org/whl/test/cu118 llama-recipes

python examples/chat_completion/chat_completion.py --model_name "PATH/TO/MODEL/70B/" --prompt_file examples/chat_completion/chats.json  --quantization --use_auditnlg

However, it failed to load the model and showed errors as follows:
  File "examples/example_chat_completion.py", line 64, in main
    model = load_model(model_name, quantization, use_fast_kernels)
TypeError: load_model() takes 2 positional arguments but 3 were given

I checked the history of llama_recipes.inference.model_utils.load_model and found it changed last month. I installed llama-recipes from source and fixed this issue.
Is the pip-downloaded version outdated?
Error logs
  File "examples/example_chat_completion.py", line 64, in main
    model = load_model(model_name, quantization, use_fast_kernels)
TypeError: load_model() takes 2 positional arguments but 3 were given

Expected behavior
Update the pip packages.

Thanks @wygao8 for reporting this, we are getting ready for a new release in the meantime please install from src  pip install -e . that should help.

Hi! It seems that a solution has been provided to the issue and there has not been a follow-up conversation for a long time. I will close this issue for now and feel free to reopen it if you have any questions!


Ffmpeg async lock

Description:
Add lock around ffmpeg test.
Tested by @edif30
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Closing in favor of #18131


get-sposite and set-sposite returning "the request uses too many resources"

Trying to set conditional access on a site, returns the above error. Even a generic get-sposite returns the same error.
Get-SPOSite : The request uses too many resources.
At C:\bin\scripts\SPOToolKit\scripts\completecreation.ps1:96 char:5
+     Get-SPOSite -Identity "https://uwhca.sharepoint.com/sites/$site"  ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Get-SPOSite], ServerException
    + FullyQualifiedErrorId : Microsoft.SharePoint.Client.ServerException,Microsoft.Online.SharePoint.PowerShell.GetSite

@davecober are you still having this issue today?

It is not. It was probably related to the service health informational alert earlier this week.


Azure CLI DevOps is not installed on Ubuntu images

Describe the bug
Azure CLI DevOps should be installed as part of Azure CLI and should be available
Question, Bug, or Feature?:
Bug
Virtual environments affected

[ ] macOS 10.15
[x] Ubuntu 16.04 LTS
[x] Ubuntu 18.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Expected behavior
Azure CLI DevOps should be installed:

az devops -h should show correct help
az --version should show azure-devops in extensions section

Actual behavior

az devops -h fails
az --version doesn't show azure-devops

Additional information:
It is pretty strange that azure-devops is not installed because we have a step to install it. Looks like step doesn't work properly

Created based on https://github.com/microsoft/azure-pipelines-image-generation/issues/1478

Hello, @SiM22
I have prepared https://github.com/actions/virtual-environments/pull/442 PR.  Changes will be available after next image rollout (1-2 weeks approximately)

Hello, @SiM22
Azure CLI (azure-devops 0.17.0) has been rolled out. Feel free to reopen the issue if you have any concerns.


⚠ Secret Website 3 has degraded performance

In fb0abab, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 3171 ms

Resolved: Secret Website 3 performance has improved in c8a2925.


Fix typo

This change is

Coverage remained the same at 82.924% when pulling 3836d8d3f25e0b7527344c9fbfeceb50255a1c10 on evgenTraytyak:patch-1 into 466f6df1b0abedcf43801151bc8646934f56f08e on shakacode:master.

@evgenTraytyak Please update to latest so I can merge.

@evgenTraytyak please reapply your change (rebase on top of origin/master):
git checkout patch-1
git fetch
git rebase origin/master
git push -f

@evgenTraytyak can you reapply your change? I can't merge as is

@justin808 :shipit:

Coverage remained the same at 82.924% when pulling 21d859a7bce149c49dceac931d07df298dd7e20d on evgenTraytyak:patch-1 into 3b9397a381e48986a2dc7dc04e7a6ee84311d9fe on shakacode:master.

Thanks @evgenTraytyak


Trivy: Only concluded license is added in the Parlay enrichment process

We may need to either copy the concluded license to declared or remove the declared "NONE" leftover from Trivy.

From looking at the 3.0 spec, the concluded license is the only required license - so we should just change the excel spreadsheet to use the concluded license.
We can also copy the concluded license to declared license.


docs: update RtD guides

Description

Create in-depth guide for installation, usage and examples hosted on RtD
Update README with quick start guide
Update version to v1.0.0 (release version will be v1.0.0-rc.1)

Type of change

[X] Documentation update

Checklist
Please carefully read these items and tick them off if the statements are true
or do not apply.

[X] I have performed a self-review of my own code
[X] My code follows the existing coding style, lints and generates no new
warnings
[X] I have added type annotations to all function/method signatures, and I
have added type annotations for any local variables that are non-trivial,
potentially ambiguous or might otherwise benefit from explicit typing.
[X] I have commented my code in hard-to-understand areas
[X] I have added ["Google-style docstrings"] to all new modules, classes,
methods/functions or updated previously existing ones
[X] I have added tests that prove my fix is effective or that my feature
works
[X] New and existing unit tests pass locally with my changes and I have not
reduced the code coverage relative to the previous state
[X] I have updated any sections of the app's documentation that are affected
by the proposed changes

If for some reason you are unable to tick off all boxes, please leave a
comment explaining the issue you are facing so that we can work on it
together.

The RtD guide is available at https://htsinfer--172.org.readthedocs.build/en/172/index.html


Feature: Add support for reseting image cache

Summary
This PR introduces the CachingImageView class, a view that represents an image view with caching capabilities. It ensures that images are cached for 30 days to optimize performance and reduce redundant network requests.
Changes Made

Created the CachingImageView class to display images with caching functionality.
Implemented a method to reset the cache after 30 days to ensure the cache remains up-to-date.
Added functionality to load images from a URL, using a fallback image in case of errors.
Ensured that the cached images are used if available, reducing unnecessary network requests.
Added error handling and concurrency management to improve robustness.

TODO
N/A
Motivation
The primary goal of this PR is to introduce a caching mechanism for images in the Aurora Editor project. By caching images for 30 days, we aim to improve the performance and user experience by reducing the number of network requests needed to load images. This is particularly beneficial in scenarios where the same images are frequently accessed.
Testing

Verified that images load correctly from a URL and are displayed in the CachingImageView.
Tested the caching functionality to ensure that images are cached and retrieved as expected.
Confirmed that the cache is reset after 30 days, and new images are fetched from the network.
Conducted tests to ensure that the fallback image is used when the image loading fails.

Screenshots/GIFs
N/A
Checklist
Please ensure all of the following are completed before submitting the PR:

[x] Code has been tested and verified.
[x] Documentation has been updated to reflect the changes.
[x] All tests pass successfully.
[x] Code follows the project's coding guidelines and style.
[x] The branch is up-to-date with the latest changes from the main branch.
[x] Reviewed by at least one other contributor.

Related Issues
This PR addresses the following issue(s): N/A
Additional Notes
The CachingImageView class provides a significant performance improvement by reducing network requests and ensuring that images are readily available. Further enhancements and optimizations can be made based on feedback and additional testing.

Thanks for submitting a pull request!
We will review it as soon as possible.

Hey @nanashili,
Thanks for your contribution.
Your PR has been merged!
Your name will appear on https://auroraeditor.com/#contributors in a few moments, if you have contributed to the editor.
Thanks again for your contribution!
Kind Regards,
Aurora Editor Team.


Client SDK and Admin SDK return different timeStamp objects.

Hi, I really hope you can help with an answer on this.
[REQUIRED] Step 2: Describe your environment
"firebase-admin": "^9.11.0",
"firebase-functions": "^3.14.1"

Firebase Product: Firestore Admin SDK
Node.js version: 14

[REQUIRED] Step 3: Describe the problem
The Admin SDK returns a different timestamp than the client SDK. This makes it unnecessarily complex to fetch data from both. Is there a specific reason for this?
Steps to reproduce:
Create a timeStamp field in any firestore document and fetch the document with the client SDK and with a cloud function (Admin SDK). When comparing both docs you will see:

The client SDK returns a timeStamp as described in the docs and the prototype has toDate() etc. The naming of the timeStamp is { seconds: value, nanoSeconds: value }
The function (Admin SDK) returns a different timeStamp. It does not have the same prototype - it has none of the functions from the docs - toDate() etc.. - and the naming of the timeStamp is different. The naming here is { _seconds: value, _nanoSeconds: value }. Notice the _ before both keys.

Is there any specific reason for this and is it possible to get the correct timestamp, just like we get from the client SDK?
I've tested this using the functions emulator:  "firebase-functions-test": "^0.3.1"
Relevant Code:
It should be fairly easy to reproduce.
Kind regards,
Jakob

It is a different type but it certainly has a toDate() method and the usual seconds and nanoseconds properties: https://googleapis.dev/nodejs/firestore/latest/Timestamp.html
Here's some output from the Node.js REPL:
> const admin = require('firebase-admin')
undefined
> const ts = admin.firestore.Timestamp.now()
undefined
> ts.seconds
1628614245
> ts.nanoseconds
984000000
> ts.toDate()
2021-08-10T16:50:45.984Z

Hi, hiranya911.
I understand that. I am referring to situations where you return data from the cloud function.

Use the Admin SDK to fetch a document from Firestore (a document with a timestamp).
Get document.data() in the cloud function and then return the data to the client.
The timeStamp on the document no longer has toDate() when received on the client.

Jakob

That depends on how you have chosen to pass data back from the function to your client. Assuming this transfer takes place over HTTPS+JSON, you might want to serialize the Timestamp in the server to a format the client can easily decode. In most JS applications you might want to use the UTC string format of the date for this purpose.

@hiranya911 I have a monorepo with a shared library which includes various type interfaces, typed functions etc... I also have multiple clients and a functions project in this monorepo. I cannot compile my functions project because toJson is missing in the admin SDK. I do not want to add the admin SDK to all of my other projects.
I see this as a bug.

@joelmeaders just encountered the same issue, were you able to solve it?

@hiranya911  I agree with the sentiment on this issue. I am using NextJS which allows both server side static and client side dynamic rendering of pages using the same React components.
In my case I have pages where I statically render each page at build time meaning I need to use the Admin SDK to access Firestore for data... then when a user has loaded the static page I use the Client SDK to create a snapshot to the same underlying data.
However because the fields with Timestamp objects have differing signatures I am forced to write boiler plate to handle the differences between the two SDKs.
A solution to this would be to use the same types definition (shape) to these objects as @joelmeaders has highlighted.

Alternatively is there a way to configure both of these SDKs to return javascript Date objects instead? That would save a world of effort for this use case.

Hi @hiranya911 any updates on this?
Firestore Client SDK and Admin SDK should use the same definition for Timestamp
Working in monorepos or other situations with shared Typescript interfaces is complicated because toJSON method is missing on the Admin SDK definition

Agreeing with @rcb4t2 about the complications in using typescript and monorepos, things get quite messy. But since I have no idea how hard it is to have a shared definition between Client and Admin SDKs for firebase team, I am currently unifying both Timestamps (Client and Admin) with:
import { Timestamp } from "firebase/firestore";

export type TimestampClientAdmin = Omit<Timestamp, "toJSON">;

This is far from optimal, but a bearable workaround.

FWIW, I've written a function that just converts all Timestamps within object to Dates after reading from the database.
I wish Firestore's javascript SDK would just return Date instead of Timestamp type. We don't need your type, Google.
For admin, just change instanceof Timestamp to instanceof admin.firestore.Timestamp
export const convertTimestampsToDates = (obj: any): any => {
  if (!obj) return obj

  if (obj instanceof Timestamp) {
    return obj.toDate()
  }
  if (Array.isArray(obj)) {
    return obj.map(convertTimestampsToDates)
  }
  if (typeof obj === 'object') {
    Object.keys(obj).forEach((key) => {
      obj[key] = convertTimestampsToDates(obj[key])
    })
  }
  return obj
}


IE11 unsupported without Object.assign polyfill

Stampit makes use of Object.assign(), which is not supported in IE11. I propose that a library be used to address this:
https://www.npmjs.com/package/object-assign
I'll make a PR soon. I need this for my library because I need IE11 support and I would rather not have to force downstream developers to include a polyfill.

Closed by comment

Without an internal Object.assign reimplementation:
Estimating dist/stampit.umd.min.js: 5.16 KB, GZIP : 1.8 KB
With it:
Estimating dist/stampit.umd.min.js: 5.26 KB, GZIP : 1.84 KB
The implementation look like this:
export const assign = Object.assign || function assign(to) {
  for (let s = 1; s < arguments.length; s += 1) {
    const from = arguments[s];

    for (const key in from) {
      to[key] = from[key];
    }
  }

  return to;
};

I know it's unsafe, but works for the internal object merging.
I would rather release stampit v3 with this thing, but we should clearly state that "IE11 unit tests where never run, we do not have an ability to run stampit tests in IE11".
(Going to cross post this to the other thread.)

Without an internal Object.assign reimplementation:
Estimating dist/stampit.umd.min.js: 5.16 KB, GZIP : 1.8 KB
With it:
Estimating dist/stampit.umd.min.js: 5.26 KB, GZIP : 1.84 KB
The implementation look like this:
export const assign = Object.assign || function assign(to) {
  for (let s = 1; s < arguments.length; s += 1) {
    const from = arguments[s];

    for (const key in from) {
      to[key] = from[key];
    }
  }

  return to;
};

I know it's unsafe, but works for the internal object merging.
I would rather release stampit v3 with this thing, but we should clearly state that "IE11 unit tests where never run, we do not have an ability to run stampit tests in IE11, use on YOUR OWN RISK...".
(Also cross posted to #319.)

Another proposed fix is in #321

Aaand done! Released v3.2.0.
https://github.com/stampit-org/stampit/releases/tag/v3.2.0
Not polyfiling anymore.


Straight like me

Creating a PR just to press tabs on a few lines??  Stop making pull requests that don't help the actual dev lmfao

no

i do this cuz ppl look at it and they seem to be mad or smt and its funny

Master7720 is an amazing coder wym @TwoThreeTwo? His contributions are always welcome, and they should be in this repository. It's open source so don't complain people wanna contribute.

!!!!! same gos for you @HausemasterIssue spidersense is FULLY custom


how enable Gzip after set disable?

Hi, how re enable gzip after use WS.disableDynamicGzip() ??
Tanks

You can't. Why do you want to?

Static files are always gzipped. disableDynamicGzip only applies to dynamic responses, like you set in ctx.json() or ctx.result().

Very Tanks, i dont know the https://gtmetrix.com say my server has no Gzip!

:( i use this.
ByteArrayOutputStream out = new ByteArrayOutputStream(str.length());
gzip = new GZIPOutputStream(out);
gzip.write(str.getBytes());
gzip.finish();
gzip.close();
		return out.toByteArray();

your veto?

It will interfere with how Javalin normally writes the response, so as a general rule you shouldn't do it. But if it works it works.

Interesting discussion.
I currently have a response cache (my responses are json objects).
I was thinking to cache a gzipped response object (rather than plain strings) because it will save a quite a bit of memory on my server.
But for that I would need to disable the built in compression, and I could not figure out how to do that.
So this discussion was useful to me .
thank you


Add l1 weight regulariser

Implemented. Doesn't seem to increase sparsity, at least not without a major loss of performance, comes from facescrub experiments


fix: Partial approved reports don’t have GBR in the LHN

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/43014
PROPOSAL: https://github.com/Expensify/App/issues/43014#issuecomment-2154472505
Tests

Pre-requisite: Create a Collect workspace with user A (member) and user B (admin).
Pre-requisite: Enable approvals and have user B approve for user A
Have user A create several expenses
As user B, hold one of the expenses
Click approve and choose to partial approve what isn’t held
Verify that: The newly created report for the held expense is GBR’ed in the LHN

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Pre-requisite: Create a Collect workspace with user A (member) and user B (admin).
Pre-requisite: Enable approvals and have user B approve for user A
Have user A create several expenses
As user B, hold one of the expenses
Click approve and choose to partial approve what isn’t held
Verify that: The newly created report for the held expense is GBR’ed in the LHN

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/165644294/a3ae693d-a96a-4f7e-94ff-793753bb5964

Android: mWeb Chrome

https://github.com/Expensify/App/assets/165644294/56743ef4-9c5a-4035-9d0e-cdbdf6a94aad

iOS: Native

https://github.com/Expensify/App/assets/165644294/60e2346a-9970-42ee-bd30-1808d8605c06

iOS: mWeb Safari

https://github.com/Expensify/App/assets/165644294/034d6483-1969-42a3-b6c8-cfe53849258c

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/165644294/1fb40026-0040-4955-8dc4-251f0301c534

MacOS: Desktop

https://github.com/Expensify/App/assets/165644294/0195fa45-d756-4eb1-aad9-803fb350c2ce

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/blimpich in version: 9.0.2-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/jasperhuangg in version: 9.0.3-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Julesssss in version: 9.0.5-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


ListItem Divider Optional Feature

https://github.com/yamankatby/react-native-material/issues/44#issue-1373610406
This is so important.
I can use ListItem for a Drawer Navigation and they look ugly with the divider below them. And with this option, you can opt to use it or not.

ping @yamankatby


HALON-602: Add support for Clovis

Created by: nc6
Adds support for Clovis clients in addition to M0t1fs.
Currently this does not add support for running m0dixinit, since the DIX/CAS work has not been merged.

Closing as an obsolete


Fix return type of multinomial and add optional generator

This PR fixes the return type of the multinomial distribution to int64. This is justified as the underlying pytorch function always returns a long (as described in #54).  Furthermore, an additional optional argument is added that allows a generator to be passed to the function.

Thanks @marcelluethi!


Compare parent with old_parent on check_callback

I am trying to check the move_node callback to see if the new parent is equal to the old parent (the user can not change levels of items). Sadly it seems that this is not possible. I can check it on the easily on the move_node.jstree event, but not in the check_callback. Am I missing something?

I can not help you without you showing me what you already tried. It should be pretty trivial - you get the new position and the node. From the new position and the reference node you can calculate the (new) parent, and from the node you can get the current parent of the node, so you can compare if those two are the same.

Sorry just figured it out.
`    check_callback: function (operation, node, new_parent) {
if (operation === 'move_node') {
if (node.parent === new_parent.id) {
return true;
}
}
                    return false;
                },`


NOISSUE - Improve Vault initialization

What does this do?
Makes vault setup more easy.
Which issue(s) does this PR fix/relate to?
Put here Resolves #XXX to auto-close the issue that your PR fixes (if such)
List any changes that modify/break current functionality
No
Have you included tests for your changes?
Not needed
Did you document any new/modified functionality?
Yes

Codecov Report

Merging #1481 (2f4a1a1) into master (042ff98) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1481   +/-   ##
=======================================
  Coverage   70.83%   70.83%
=======================================
  Files         123      123
  Lines        9567     9567
=======================================
  Hits         6777     6777
  Misses       2263     2263
  Partials      527      527

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 042ff98...2f4a1a1. Read the comment docs.

closing this one https://github.com/mainflux/mainflux/pull/1481#discussion_r757006318

closing this one https://github.com/mainflux/mainflux/pull/1481#discussion_r757006318

Why?


Add version-increment option to increment-version-with-git

Is your feature request related to a problem? Please describe.
Sometime for some reasons (most of time bad reasons), we don't want to follow the pattern you propose for determines the increment of the version.
Describe the solution you'd like
Add the same option (--version-increment) as in increment-version command to the increment-version-with-git command.
Describe alternatives you've considered
An other and maybe better option is to allow user to override your switch to make it match they desiderata.
return type.ToLower() switch
{
    "feat" => VersionIncrement.Minor,
    "fix" => VersionIncrement.Patch,
    "build" => VersionIncrement.Patch,
    "config" => VersionIncrement.Patch,
    "docs" => VersionIncrement.Patch,
    "perf" => VersionIncrement.Patch,
    "refactor" => VersionIncrement.Patch,
    "resolve" => VersionIncrement.Patch,
    "style" => VersionIncrement.Patch,
    "test" => VersionIncrement.Patch,
    "ci" => VersionIncrement.Patch,
    _ => VersionIncrement.Unknown
};

Also include #breaking.

Hi @0nary, thanks for your interest in the tool. It is my understanding that you like the idea of the automatic versioning with the git commit but would like to override the determined version on demand. I will look into supporting this.

@cbcrouse
Yes, it will be perfect.


Processing of scope messages via Event Processors.

Provides a single point of message distribution to event handlers (via Event Processors). Especially useful in cases when we want to coordinate event and deadline processing by sagas. Have in mind that concurrent access to a saga is already prevented by SagaRepository.
The issue that arises with this implementation is postponing deadline execution in Tracking Event Processors. In time-critical applications, this might cause serious issues. This implementation assumes that deadlines (scope messages) are of greater importance than other events, thus addressing this issue. However, postponing is not avoided.
I'm creating this draft PR in order to start a discussion on the usefulness of this change.
Relates to #1236

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[Validation/Integration] Go Test framework for automation and integration

A resdesigned test suite to be used for both integration and automation testing.
Updated go.mod and go.sum files to account for new defaults package needed."

Outdated PR, new PR up.


fix #1364, add default value for BYTES_PER_MESSAGE of SOLStreamProducer

+1
So we should also allow user provide a related config when replacing a processor

yes, could you open an issue for that ?


Mock makedirs as well to prevent spurious directories

Simplez.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #38   +/-   ##
=======================================
  Coverage   84.27%   84.27%
=======================================
  Files          12       12
  Lines         407      407
=======================================
  Hits          343      343
  Misses         64       64

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 b9c7505...5bf3d32. Read the comment docs.


[SPARK-5929] Pyspark: Register a pip requirements file with spark_context

Ships all packages in the requirements file by installing them locally via pip and then ships the packages to the workers via addPyFile

Can one of the admins verify this patch?

Good idea, this is a useful feature to have. Could you also a argument for spark-submit, (for example, --pip or --py-requirements). Also, could you add a test for it?

Jenkins, Ok to test.

Great, will get to that this week and add tests.
Do you think bundling importlib with pyspark is reasonable? Or is finding another way to track down the local package the way to go?

I think we can go with __import__, it's similar to importlib

Changed importlib to import and moved most of the requirements up. I left the pip requirement in the method itself as to not require the system has pip unless that method is used. Also added a test that tests the import of a library not found on most systems https://github.com/buckheroux/QuadKey.

Any traction on this?

This could be very useful. One question; Does this behave with namespace packages?

Can one of the admins verify this patch?

@robcowie I wouldn't think so. Do you have a good example of a pypi package that I could try to install that has a namespace? I can try and support it if possible.

Can one of the admins verify this patch?

Jenkins slow test please

Merged build triggered.

Merged build started.

Test build #5 has started for   PR 4897 at commit 76ff637.

Test build #5 has finished for   PR 4897 at commit 76ff637.

This patch fails some tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Just saw the test failure, looking into it now

@buckheroux I believe the test failure is not related. @JoshRosen is looking into a hot fix.

@buckheroux Do you think that we should also support this in bin/spark-submit? For example, we should accept .txt file (via --py-files) as dependencies.

@andrewor14 Sounds good.
@davies It's probably a good idea to support a CLI arg for the requirements file. I'll work on this as well.

@andrewor14 the test failure in the dev/run-tests doctest are because we did a shallow clone of the Spark repository into the fresh Jeknins workspace. I think that this is an ephemeral problem that will resolve itself after retesting. We could also disable / remove that test for now or skip it in Jenkins.

I've disabled shallow cloning in SlowSparkPullRequestBuilder, so let's try this again...

jenkins slow test please

Merged build triggered.

Merged build started.

Test build #6 has started for   PR 4897 at commit 76ff637.

Test build #6 has finished for   PR 4897 at commit 76ff637.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #1076 has started for   PR 4897 at commit 76ff637.

Test build #1076 has finished for   PR 4897 at commit 76ff637.

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

The tests fail because one of them attempts to pip install a package and doesn't have permissions to do so. Is there a way to enable that? Or just leave the pip installing piece untested?

Looking to add support for namespaces, which I have tested locally and confirmed to work. I was also thinking of exposing an add_package function that would add local packages to the spark context.

@davies removed the requirements file from the context constructor. I looked into add --py-requirements to spark-submit, but it looked a bit more in depth than I was thinking it would be. I think it makes sense to add that as a separate commit/PR.
Also, when testing I import a package from pip, but the Spark Jenkins test errors because it doesn't allow for access to the global pypi server. Any thoughts?

cc @JoshRosen @sknapp who are more familiar with Jenkins

For the test you could also add a local egg file to the repository and install this instead of relying on the external pypi server.

This is a pretty big pain when using pyspark (adding modules to workers) and should definitely be included. I'll open a new pull request and to push this through.

Yeah, this is quite a useful feature :+1:


docs: use correct props method

📝 Description
Use the correct props-getter method.

Hi @theoomoregbee,
as stated in the code of useRadio the getCheckboxProps function is deprecated for this hook and replace by getRadioProps.
Here 2 Sandboxes where it works:
https://codesandbox.io/s/flamboyant-maria-j0sppy
https://codesandbox.io/s/nice-benji-ir6ysk


Add sports schedule support

Requires parser for sport schedule

Sports schedule already supported by the website. If you add the new category in API, website will automatically add new category card with corresponding schedule.
I see the only tasks for this — creating designed EventGroup with images (may be the new issue) and corresponding pages (#34). What do you think?


java.lang.IndexOutOfBoundsException: Index: -1, on latest.log

This doesn't seem critical or anything but quite anoying
https://pastebin.pl/view/7beb75a3

Send your animation file, you have invalid molang.

The what?
You mean the entire log file?
What's a molang?

No, your animation json.

I don't have any file called "animation.json" or anything containing "animation" on it's name.
I'm not a mod developer

Then you need to contact whichever mod author is using geckolib in your pack. They have invalid molang.
Closing because this isn't an issue with Geckolib.

Then you need to contact whichever mod author is using geckolib in your pack. They have invalid molang

Done, thanks. It's Ars Noveau

Apparently everyone has a bad molang. I think its you that are using molang wrong, lmfao. Mojang can't have a bad component its you who is using the components wrong.


gstreamer1.0-plugins-good: Reemove inserting qt5 from packageconfig

layers are usually supposed to be inert, but this option automatically
inserts itself without any changes due to this bbappend. Its best to
leeave this option to end user to enable, defaults are to disable it and
let it be as it is.
Signed-off-by: Khem Raj<EMAIL_ADDRESS>

In dunfell-next and master-next, thanks:
5e189731..ac7514e2  dunfell-next -> dunfell-next
02930760..44fc90ee  master-next -> master-next

And merged to dunfell/master as well.


feat(i18n): add fr_FR translation

I may have forgotten to update a file, let me know and I will update the PR.

Thank you! You don't need to update _missing_translations.json btw. It will be generated by a tool.


DatasetComponent Miss report

Version
5.0.2
Steps to reproduce
When I don't  import the DatasetComponent but use dataset in option,it don't report any error in console and nothing to plot
What is expected?
Error: Component dataset not exists. Load it first.
What is actually happening?
nothing

I agree it would be useful to validate at least the top level keys of the options object to report any unrecognized keys. A lot of time can be wasted debugging issues like this otherwise, due to lack of error reporting.

Thanks for reporting. We will see if this can be exported in the next release.

@zpinocchio @Bilge A temperary solution is to open an example and paste your code to the editor. Then, choose "Full Code" at the top-left corner and enabled "Minimal Bundle" so that you can know what components to import.


Error starting notebook - could not read property "replace" of null

Issue Type: Bug
I am receiving this error every time I open a notebook.
"Error starting notebook - could not read property "replace" of null"

The notebook appears to work fine.

Azure Data Studio version: azuredatastudio 1.5.0-insider (c831596e0258aceca2ea4647d2fa2420860f6217, 2019-02-25T19:57:08.953Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7600U CPU @ 2.80GHz (4 x 2904)

GPU Status
2d_canvas: unavailable_softwarechecker_imaging: disabled_offflash_3d: unavailable_softwareflash_stage3d: unavailable_softwareflash_stage3d_baseline: unavailable_softwaregpu_compositing: unavailable_softwaremultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: unavailable_softwarevideo_encode: unavailable_softwarewebgl: enabled_readbackwebgl2: unavailable_off

Memory (System)
15.84GB (2.59GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

vscode-sanddance
dan
0.0.1

newdatabase
kev
0.0.3

agent
Mic
0.36.1

import
Mic
0.6.0

profiler
Mic
0.6.0

server-report
Mic
0.1.4

whoisactive
Mic
0.1.1

(1 theme extensions excluded)

This is coming from Jupyter extension code if I had to guess. Probs my why SQL works but you see this error. Is this for a specific notebook? Can you mail the team all the console output and the notebook if available? Thanks!

Behavior seems to have changed to #4215 and goes away if I run as administrator.

Fixed


💾 Internal change: Upload APNs authorization key to Firebase

Problem
In preparation for the development of push notifications to the PRISMA iOS app, we must upload an authorization key from the Apple Developer Console in order to send requests to the APNs server. We are setting this up through Firebase Cloud Messaging.
Solution
From the Apple Developer console, generate an authorization key (file type .p8). In Firebase, go to the "Cloud Messaging" tab and upload the key.
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct and Contributing Guidelines

Successfully upload the APNs auth key to firebase with the correct key id and team key for the project


Divergence at high Mach numbers

Simulations of the core/dart configuration stops when the Mach number is around 2 (but works for M1.1), due to "the maximum number of iterations" being exceeded.
It has been suggested this has to do with the thermophysical properties of the simulation, namely the fact that OpenFOAM stops the simulation once T reaches too higher a value, or goes negative.

The residuals for e, k and epsilon increases for all simulations involving turbulence.


tweets_and_replies(user_ids) invalid

only return [{'data': {'user': {}}}]

2024-02-24 09:58:42.375 [DEBUG] :: remaining: 0/50 requests
2024-02-24 09:58:42.375 [DEBUG] :: reset:     13.77 minutes
2024-02-24 09:58:42.379 [DEBUG] :: Unique results: 2341 cursor: DAABCgABGHER-jK_--cKAAIWqHFppRqQAQgAAwAAAAIAAA
2024-02-24 09:58:42.750 [ERROR] :: Failed to log: Expecting value: line 1 column 1 (char 0)
2024-02-24 09:58:42.750 [ERROR] :: Failed to get pagination data
Expecting value: line 1 column 1 (char 0)
Cannot parse JSON response 'NoneType' object has no attribute 'json'
seems like get_json() processing a None in final.it's specific for me?

I am getting the same error. Most of the errors are on verified accounts

i am using this code:
data_new = scraper.tweets_and_replies(['2335075836'])

I am getting the same error. Most of the errors are on verified accounts

i am using this code:
data_new = scraper.tweets_and_replies(['2335075836'])
for now i'm using scraper.tweets_and_replies(user_ids,limit=50) to avoid this,which is not a perfectly solution.
what's cause this is that get_json() function take a None as parameter to execute.
is twitter_api was modified after @trevorhobenshield  create it ?

I am getting the same error. Most of the errors are on verified accounts

i am using this code:
data_new = scraper.tweets_and_replies(['2335075836'])

for now i'm using scraper.tweets_and_replies(user_ids,limit=50) to avoid this,which is not a perfectly solution.
what's cause this is that get_json() function take a None as parameter to execute.
is twitter_api was modified after @trevorhobenshield create it ?

I am getting the same error. Most of the errors are on verified accounts

i am using this code:
data_new = scraper.tweets_and_replies(['2335075836'])

for now i'm using scraper.tweets_and_replies(user_ids,limit=50) to avoid this,which is not a perfectly solution. what's cause this is that get_json() function take a None as parameter to execute,it usually happend in last request of one time tweets_and_replies(). is twitter_api was modified after @trevorhobenshield create it ?

I think so. I used to use nitter to scrape from for a while. They have officially said we are unable to provide tweets cause of the new change twitter made.


Change Ensemble Date/Time rendering to use Member's Time Zone in profile

This would mean we could get rid of the JavaScript on-the-fly conversion.

Also remove the hard-coded time zone in ScheduleEnsembleForm, take it from the Admin's Profile.


GSSoC'23 : Add Scoreboard feature in Rock Paper Scissors game

Description:
I would like to contribute to the Rock Paper Scissors game by adding a scoreboard feature. This feature will enhance the user experience by keeping track of the number of wins, losses, and draws for the user and the CPU. The scoreboard will be displayed after each game result and reset when the game is restarted.
Features Request

Display the user's score (number of wins) on the scoreboard.
Display the CPU's score (number of wins) on the scoreboard.
Display the number of draws on the scoreboard.
Update the scoreboard after each game result.
Reset the scoreboard when the game is restarted.

@Dharmesh177 please assign me this issue

@Dharmesh177 I have done this. please assign me so that i can make a pull request

assign to u !!
complete it in 1 Week and follow contributing.md file guideance for successful PR merge
Happy Coding 🧑💻!!


Message Box

as you can see in Image Message dropdown is not prfect

Can you define what do you mean 'not perfect' is?

It seems like you want to change the behavior of the message icon in the navbar. Can you please provide more details on what you want to achieve? Are you trying to show a list of messages when the message icon is clicked instead of showing user details?


Remove all but one binding redirection compile warning

Wow! Awesome! What was the thing that fixed all these?  I noticed the migration to .NET Core project style and <PrivateAssets>All</PrivateAssets>? Was there something else?

That was mostly it - there were also issues introduced from different versions of some libraries (such as System.Collections.Immutable)


Instruction for R-package installation with precompiled dll/lib

Some instructions for R-package installation with precompiled dll/lib.

@Laurae2 Another problem is: The Rtools seems is still needed when build from visual studio. I try to build without Rtools, and it fails.

@guolinke Unfortunately, can't install any R package without Rtools in Windows when it has a src directory.

Rtools is 99.99% mandatory to live with Windows and R packages, so it should not be much a problem (especially for high performance purposes). I know there might be issues on heavily locked down machines about installing and using Rtools: https://github.com/hadley/devtools/issues/200#issuecomment-291861187


Add cancelAll function

Partially Resolves #41 .
This PR exports the cancelResponseListeners function under the postRobot object as postRobot.cancelAll().
#hacktoberfest !

Thanks @TheTripleV !

Hi -- ideally we should cancel request listeners, not response listeners. Response listeners are only for in-flight messages.