build: release candidate

Summary of Commits (Oct 28 - Oct 30, 2024)
Features

Incrementing Iteration Number in Chat (#263)

Added an incrementing counter for chat iterations.

Selective Inclusion for Americanisms, RAG, Analytics (#287)

Introduced selective instantiation based on configuration, covering Americanisms, RAG, and analytics modules.

Fixes

Prisma CLI - Use --no-engine with generate and reinstate prompts script (#282)
Browser Error Handling - Switched to browserLogger for in-browser error tracking (#280)
Asynchronous Methods - Updated async usage:

Removed async from pages without data loading (#296)
Adjusted completion/system messages to avoid unnecessary async (#294)
Ensured persistence awaits in generation flow (#292)
Promisified chat API route get handler (#316)

VS Code Import Preferences - Set to prefer import type notation (#291)
Minor Typo Fix - Renamed "accordian" to "accordion" (#297)
App Component Linting - Addressed ESLint fixes for app components (#295)
Improved Import Management - Updated type prefixing and path handling in imports (#298)

Chores

Make Turbopack Default for Local Development (#274)
Enhanced ESLint Rules - Enforced import type rules across the Next.js project (#288)
Incremental Transition to Non-Barrel Imports - Continued moving Core and Aila projects away from barrel files (#290, #289)
CI & Debugging - Adjusted Jest workers for CI efficiency (#277), enabled DEBUG variable on frontend (#275)

Tests

Inngest Dev Mode - Modified Jest to honor Inngest dev mode in testing (#278)

:tada: This PR is included in version 1.13.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


scroll issue with a 'div' element

You can find the issue below link.
In the link page,
When you're scrolling up the right bottom pane repeatedly,
right middle pane will be scrolled up.
https://stackblitz.com/edit/angular-split-issue

Not sure to understand what you want..
what about this:
https://stackblitz.com/edit/angular-split-issue-zqyoeu?file=src/app/app.component.html

@bertrandg Oh! my issue has fixed in your link!

style="display: flex; flex-direction: column;" has solved this issue.

Cool ;)


add path to install.packages() docs

Without this, I was getting an error "Error: failed to resolve remote 'prolfqua_1.2.5.tar.gz' -- failed to parse remote spec 'prolfqua_1.2.5.tar.gz'".  Adding the path to install_packages() fixed it.  Just thought others might be having the same problem, and it might help to share.

@nickdelgrosso Thanks a lot for the pull request


useMutation types: variables are optional

Describe the bug
in TypeScript, the mutation function created by the useMutation hook can always be called without passing arguments, because the type TVariables is set to optional:
https://github.com/tannerlinsley/react-query/blob/00b9e968a80ca7c51fd50479aee852bb41bf9bc6/src/react/useMutation.ts#L132
To Reproduce
I made a codeSandbox example: https://codesandbox.io/s/async-currying-wepni
Please look at the comment at line 54
I should be forced to call that function like this:
mutatePostTodo(text)
but I can also call it like this:
mutatePostTodo()
Expected behavior
The type of variables that you need to pass to that function should be equal to the type that the actual mutation function accepts, in other words, it should be variables: TVariables rather than variables?: TVariables
Desktop (please complete the following information):

OS: MacOS Catalina
Browser: Chrome
Version: 85

Additional context
All the usages of variables in that function are accessed with the bang operator, for example:
https://github.com/tannerlinsley/react-query/blob/93db5b61246dcd381d699054de48fca685bef73b/src/react/useMutation.ts#L146
or https://github.com/tannerlinsley/react-query/blob/00b9e968a80ca7c51fd50479aee852bb41bf9bc6/src/react/useMutation.ts#L174
So I don't really see why it should be optional to begin with 🤷 .
Happy to make a PR if you approve of this :)

Yeah a PR would be great. Be sure to include some tests too? :)

Do you have tests of the type definitions in mind, e.g. with https://github.com/Microsoft/dtslint ?

@boschni right, i didn’t think about mutations with no variables... I’ve never done that, but it’s still a legit case.
I think this TS issue is on topic, but not sure if any of the provided workarounds to make a parameter optional if its undefined would help us here: https://github.com/microsoft/TypeScript/issues/12400
I agree that that would be a breaking change and we would thus need to wait for v3

I just found this discussion on the topic: https://github.com/tannerlinsley/react-query/discussions/991#discussioncomment-81267
Sorry for the duplicate. @tannerlinsley mentioned that type changes are not considered as “breaking”, so we could also just go ahead with making variables mandatory?

I would be okay implementing this in v2 as non-breaking, but I'll leave that up to @boschni since he knows more about the type implications. Otherwise, expect it in v3.

The typing issue has been fixed in V3

I am having an issue with the required parameter case.
I have a mutation function that has parameters which are all optional and there are defaults.
When I have no parameters typescript does not complain, as well as when I pass the parameters, however calling the function without parameters now causes an error
useMutation((param1?: string, param2?:string) => {})

as you can see in the signature you posted, useMutation only takes one variables argument. If you want to pass multiple arguments, you have to use an object:
useMutation((variables: { param1?: string, param2?:string} | undefined) => ...)

The above function also results in a typescript error when calling it without parameters
the variables parameter is required
Tried with an object as well:
useMutation(
     ({ param1, param2 }: { param1?: string, param2?: string } = {}) => {...},
     ...
)

the call I am trying to do is without params
const { mutateAsync, isLoading } = useMutation();

mutateAsync()

hmm this works fine for me, as variables are inferred to void:
      const { mutateAsync } = useMutation( () => Promise.resolve())
      mutateAsync()

if you have an optional parameter, you need to pass undefined at least I'm afraid, because that doesn't work:
      const { mutateAsync } = useMutation((foo?: string) =>
        Promise.resolve()
      )
      mutateAsync() //error here

but that does:
      const { mutateAsync } = useMutation((foo?: string) =>
        Promise.resolve()
      )
      mutateAsync(undefined)

i see, I am passing an empty object as a workaround, but it does not feel elegant :)
thanks for the guidance here!

This is what we do:
export function useMyThing () {
  const mutation = useMutation(...yourThing)
  const mutate = (variables: TVariables = {}) => mutation.mutate(variables)
  return { mutate, ...mutation }
}

This way when using this custom hook you don't need to pass variables if you don't want:
const { mutate } = useMyThing()
const onClick = () =>{  mutate() }

Would this be a more appropriate typing:
export type MutationFunction<TData = unknown, TVariables = void> = TVariables extends void
  ? () => Promise<TData>
  : (variables: TVariables) => Promise<TData>

In this case, if TVariable is void, then no variable should be passed to the function.

This is what we do:
export function useMyThing () {
  const mutation = useMutation(...yourThing)
  const mutate = (variables: TVariables = {}) => mutation.mutate(variables)
  return { mutate, ...mutation }
}

This way when using this custom hook you don't need to pass variables if you don't want:
const { mutate } = useMyThing()
const onClick = () =>{  mutate() }

@andresgutgon I think { mutate, ...mutation } should be changed to { ...mutation, mutate }. Thanks anyway 😉
export function useMyThing () {
  const mutation = useMutation(...yourThing)
  const mutate = (variables: TVariables = {}) => mutation.mutate(variables)
  return { ...mutation, mutate }
}

My workaround has been adding | void to the args type
      const { mutateAsync } = useMutation((foo?: string | void) =>
        Promise.resolve()
      )
      mutateAsync() //all good


dif/2: unexpected error

Trealla Prolog (c) Infradig 2020-2022, v1.24.9
?- dif([],B),B=[A|A],A=[B|B].
uncaught exception: error(type_error(list,call(Goal,X1)),call/2) % unexpected

Now:
?- dif([],B),B=[A|A],A=[B|B].
*** OOPS src/heap.c 455
   error(resource_error(finite_memory),call/2).

Now:
?- dif([],B),B=[A|A],A=[B|B].
   error(resource_error(finite_memory),(;)/2), unexpected.

Now:
?- dif([],B),B=[A|A],A=[B|B].
   B = [A|A], A = B.

Which is fine, but it looks so harmless. I think it is 'wrong' to introduce in the first answer substitution a variable whose value will be explained later on. Like
   B = [B|B], A = B.

That would be the minimal answer, but also Scryer with
?- dif([],B),B=[A|A],A=[B|B].
   B = [[B|B],B|B], A = [[A|A],A|A].

is reasonable.
In any case, this is all STO, and goes much beyond this item. Just in case.


Comments: Why is the Swift PM taking so long?

I was chatting at a bar with an engineer working on the Swift team, and someone joined our conversation with the classic opener "When can I stop using CocoaPods and switch to Swift PM?" - I chuckled, because I get this a lot (I have been helping to maintain CocoaPods for about 6 years now) but they had made mistake of asking an Apple employee about something which sounds even remotely like a future product question. So, it didn't go anywhere.

Well, person whose name I don't remember, let me try to re-frame that question into something I can take a reasonable stab at: "Why are we all still using CocoaPods now instead of Swift PM for iOS apps three years after its release?"

Nice analysis 👏
Speaking of integrating SPM with Xcode it can already be done actually, a bit in a hacky way, but still. Check out https://swiftxcode.github.io.
I was building a project that uses SMP for its dependencies just yesterday and what I did was just making a dummy framework SPM target that references all the dependencies and using swift package generate-xcodeproj. This will basically give you the project with all the dependencies as cross-platform frameworks (that said, generated framework targets support all Apple platforms) that can be linked to your app target.
That's similar to what swiftxcode is doing under the hood if I understand correctly.
A bit clunky still if you think about these details, but seems pretty much usable already.

That's similar to what swiftxcode is doing under the hood if I understand correctly.

No, SwiftXcode works completely different to generate-xcodeproj. It doesn't generate an Xcode project from the Package.swift (nor the other way around).
SwiftXcode is a set of Xcode project templates and a few shell scripts to drive the package management parts. Xcode itself maintains its project file and you never have to regenerate it like with generate-xcodeproj. If the shell scripts detect that the user changed the Package.swift file, they rebuild and bundle the dependencies by calling swift build (plus some extra magic steps), as an integrated part of the Xcode build process (it runs as a build phase created by Xcode based on the provided Xcode templates).


Automatically retry a call when a 409 conflict status is raised

The client needs to retry a call when a 409 conflict status is raised.  Currently there is retry logic in the client gem and it would simplify the caller by having the retry catch this status as well.

@donflannagan, sorry for the late reply. We are gonna close this since we understand that there are ways for clients to implement retry when server returns 409, but maybe not all of them are interested in doing that


fix ci lint

Signed-off-by: zhujian<EMAIL_ADDRESS>fix lint:
╰─# make lint                                                                                1 ↵
build/run-lint-check.sh
Running linting tool ...
ERRO Can't read config: can't read viper config: While parsing config: yaml: unmarshal errors:
  line 198: mapping key "issues" already defined at line 27
make: *** [lint] Error 3

/assign @zhiweiyin318

/approve
/lgtm


Remove ! from WidgetsBinding.instance

Fixes #8

Thanks for the PR, can you remove the pubspec.lock ?

Hi, A new PR was merged with changes to support flutter v3, please check if that has fixed your issue.
If not please feel free to make a new PR.
Thanks for your contribution

Actually, I did noticed that it was not fixed on the previous version so I applied your fix on v1.1.1


Typo on comment

In the current use-case, the user subscribed to the newsletter instead of unsubscribed.

Thanks @JoaquimLey !


withDataService UpdateAll method

We have a need for an updateAll method in withDataService as we handle archiving inside of SQL. We would like to pass all the entities in certain cases. This doesn't necessarily need to be part of the interface, but would like the method to be available.

@ricochetbrown updateAll is now available in 0.0.4


Warning: Encountered known unsupported method torch.Tensor.new_tensor

Cascade RCNN convert, any advice ? Thanks

Just ignore it.

thank you for your reply，but how to choose device dynamic
device='cuda:1'
trt_model = init_detector(trt_model_path).to(device)

not work

The model can only work on the device you convert it.
If you want to use another device, please set the enviroment variable CUDA_VISIBLE_DEVICES


The select component is not working inside an actionsheet

Description
I am using https://github.com/ammarahm-ed/react-native-actions-sheet and want to place a select component in its content, however, the select component is not showing as the created actionsheet from the select is in the background
CodeSandbox/Snack link
No response
Steps to reproduce

Go to '...'
Click on '...'
Scroll down to '...'
See error

gluestack-ui Version
Latest
Platform

[ ] Expo
[X] React Native CLI
[ ] Next
[ ] Web
[X] Android
[X] iOS

Other Platform
No response
Additional Information
No response

Hey @Kreshnik, I would Suggest use Actionsheet from @gluestack-ui/themed. With react-native-actions-sheet. We don't have control over rendered View of action-sheet overlay.
Still Provide me with snack example I will have a look.


doIt does not work

As a user I want to call the do hotword to run smalltalk code.
Acceptance criteria

[ ] The do hotword runs the given code.
[ ] In comparison to =, it does not print the result.

Estimated effort: 3

Spent 2 People and 1.5h.


ARROW-15541: [Python] Bump the minimum Cython version

Cython 0.29.22 is certainly the minimum required version (const in nested type is only supported since then: https://github.com/cython/cython/pull/3886).

Should we also update ci/conda_env_python.txt and the various python/requirements-*.txt files?

Should we also update ci/conda_env_python.txt and the various python/requirements-*.txt files?

I think those files are typically used when creating a new environment? In which case they should normally get some latest version of cython. So I am wondering if it is worth it to have to update this version in many more places (certainly given that afterwards building pyarrow will still give a clear error if the cython version was not sufficient)

So I am wondering if it is worth it to have to update this version in many more places (certainly given that afterwards building pyarrow will still give a clear error if the cython version was not sufficient)

Perhaps not indeed.


[BUG] update decimal type field. the original value round up

Driver version
mssql-jdbc-6.4.0.jre8.jar
SQL Server version
mssql 2016
Client Operating System
redhat 7.0
JAVA/JVM version
java version "1.8.0",
Table schema
default(dbo | public)
Problem description
i have one script to update some decimal columns.
the script is like below:
Update MyTable set r1_total=r1_total+'8.68000' ,r2_total=r2_total+'0' , ...  where key = MyKey
both r1_total,r2_total are decimal(31,5).
once i this script was executed.  the r2_total was changed from 7.68000 to 8.00000.

Expected behaviour:
if add '0'.  the value shouldn't be changed.
Actual behaviour:
after add '0'. the original value round up
Error message/stack trace:
Null
Any other details that can be helpful:
Null

JDBC trace logs
Null
Reproduction code
Null

Hi @peterbae , thanks for your response. as confirmed with our vendor. they are using BigDecimal. i try some simple java code in my local. i can simulate it as well.  but the same code won't happen in Oracle. is it possible we can have an enhancement in future? it is so dangerous for the person who haven't aware of this.
//total = 123.23455
String UpdateSql = "update total_table set total=total+? where id=116001";
PreparedStatement prepareStmt = con.prepareStatement(UpdateSql);
BigDecimal amount = new BigDecimal(0);
prepareStmt.setObject(1, amount);
prepareStmt.executeUpdate();
            //total =123.00000

Hello,
@peterbae is that reproducible on SSMS too?
While the bug can be reproduced quickly using ms-jdbc driver, I cannot reproduce it using pure SQL on SSMS.
I mean, having a numeric(18,3) column with 123.456 value, these queries do not truncate the scale when executed on SSMS:
   UPDATE MyTable set col = col + convert(decimal(1,0), 1)
   UPDATE MyTable set col = col + cast(1 as decimal(1,0))
   UPDATE MyTable set col = col + cast(1 as numeric(1,0))

and new value is 124.456 as expected.

Hi @mimmoz81, @seektuzi, sorry for the delay. Currently the driver is always sending the decimal value with precision as 38 (and not 1, as in @mimmoz81's example), and I think this might not be the correct behavior. I'll look into this more and let you know once I have an update.

#1489

re-opening as microsoft/mssql-jdbc#1912 was reverted

re-opening since fix will be reverted in #2051

This issue is now "as designed". If precision and scale are not specified, the driver will default to maximum precision. In order to use a desired precision and scale, please use the setBigDecimal method. For more information, please see the "BigDecimal" subsection of this section: https://learn.microsoft.com/en-us/sql/connect/jdbc/using-basic-data-types?view=sql-server-2017#passing-parameters-to-a-stored-procedure.

A solution was introduced in the 12.5 preview release as a connection string option, calcBigDecimalScale (default false). The first stable release to include this fix will be 12.6, planned for January 31, 2024.


Variable Name Fix

-Removed model name used to create instance
-Removed model name as a parameter for "init" method

-Made a quick fix in model name as parameter for class instance


CUDA installation for ARM64 is broken for some reasons

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
https://github.com/autowarefoundation/autoware/runs/6212294556?check_suite_focus=true#step:6:4715

Expected behavior
No failure.
Actual behavior
It failed.
Steps to reproduce
Run the workflow.
Versions
No response
Possible causes
No response
Additional context
I'll re-run the CI after a while.

Now it seems fixed.
https://github.com/autowarefoundation/autoware/runs/6238798979?check_suite_focus=true
https://github.com/autowarefoundation/autoware/runs/6238801834?check_suite_focus=true


Generate Prost code for protobuf services

cc #155
We should be able to run the protobuf compiler in such a way as to support Prost.

Fixed


[annotation][time-series chart] Annotation on time-series chart will fail initially

Annotation on time-series chart will fail and show error message initially, then change chart type to line chart, the same annotation will show and then change back to time-series chat, annotation will show correctly
Expected results
Annotation will show correctly on time-series chart initially
Actual results
Annotation not show correctly on time-series chart initially
Screenshots

https://user-images.githubusercontent.com/81597121/125705583-bc576a0d-fb80-46b0-8c98-5c0d848b47d4.mov
How to reproduce the bug

Create an annotation with time range 2000-2004
Open time-series chart (use cleaned_sales_data, metric: sum(sales)) and apply the annotation
See error
change the chart type to line chart and apply annotation again
change the chart type to time series chart and apply annotation again

Environment
(please complete the following information):

superset version: master
python version: python --version
node.js version: node -v

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

thanks for filing! @jinghua-qa
note: this only happens on the time related annotation. formula works alright.

Seems to be working now! Closing as fixed.


Cleaned up _get_cache a bit.  Touch faster, smaller, readable.

Cleaned up _get_cache a bit. Touch faster, smaller, readable.

Looks all right.  Thanks @thebeline ... nice job in the cleaning.  Very elegant.


feature: easy way to switch to from libressl

a configuration option or a button/action on the update page to switch to/from libressl.

Hopefully soon. Testing has been successful so far. Not that this wasn't working at all for the initial 15.7.

Will be in 15.7.6's opnsense-devel package. ETA for stable is 15.7.7 or 15.7.8.


can not apply transform on several samples of LiTS

Hi ! When I try to run train_msam.py on LiTS datasst, an error accurs (RuntimeError: Trying to resize storage that is not resizable). I find that the dataset class will report this error as below:
if self.transform:
try:
subject = self.transform(subject)
except:
print(self.image_paths[index])
Did you just delete these data or there are some methods to handle this?

There also some samples without tumor region in LiTS dataset, should I remove these samples ?
Can you provide the details of training test set partitioning, since different partition results in dfferent results.


entities in unknown state

I don't see anything specific in the logs and the integration is able to connect and recognise my car - however there's a bunch of entities that stay in unknown state:

Reporting for others - took a while and had to manually awake the car from the Tesla app. At that point the entities got refreshed.


ChromeCast working on googleapi clip but not on local one

Hello,
I want to play a local clip with nodejs in Angular and then cast it to ChromeCast.
I did a test with the clip bellow and it worked -  using the link to it.
CLIP
I downloaded the clip and tried to play it locally, it will connect to the TV but the clip is not starting.
How should I build the NodeJs server for the clip to be streamable?
Here it is the repo: GIT
Thank you!

Hello,
After many tries I found something that works: start the server on the router IP address and on port 8080.
app.listen(8080, '192.168.X.X', () => { 	console.log('Server running...'); });
In cmd type ipconfig search Wireless LAN adapter Wi-Fi and use the IPv4 Address or use:
const os = require('os'); let networkInterfaces = os.networkInterfaces();
In my case:
let serverIp = networkInterfaces['Wi-Fi'][3]['address']
app.listen(8080, serverIp, () => { 	console.log('Server running...'); });


refactor: update docker-compose files to use 'avs' service alias instead of 'aerospike-vector-search' (was mixed)

Somewhere in the merges there was a s/aerospike-vector-search/avs/ but it seems to only be in place for parts of the compose files. This attempts to correct that.
It was by hand and with tired eyes though

In the metrics collection PR I went to opposite direction and used "aerospike-vector-search" https://github.com/aerospike/aerospike-vector/pull/73
The script that gets IPs depends on this name so if we go with the changes in this PR the metrics collection will have to change.

Closing #80 goes the other direction so it is constant long form


Caddy v1 Broken with Docker v20.10.0

The latest Docker v20.10.0 update appears to have broken Caddy which is also very old and outdated at v1.
`goroutine 1 [running]:
github.com/mholt/caddy/vendor/github.com/miekg/dns.ClientConfigFromFile(0xbb4739, 0x10, 0x0, 0x0, 0x0)
src/github.com/mholt/caddy/vendor/github.com/miekg/dns/clientconfig.go:86 +0xad6
github.com/mholt/caddy/vendor/github.com/xenolf/lego/acme.getNameservers(0xbb4739, 0x10, 0xfeaf20, 0x2, 0x2, 0xf77460, 0xc42004c2c0, 0xc420043f50)
src/github.com/mholt/caddy/vendor/github.com/xenolf/lego/acme/dns_challenge.go:40 +0x4d
github.com/mholt/caddy/vendor/github.com/xenolf/lego/acme.init()
src/github.com/mholt/caddy/vendor/github.com/xenolf/lego/acme/dns_challenge.go:33 +0x12d
github.com/mholt/caddy/caddy/caddymain.init()
:1 +0x75
main.init()
:1 +0x44
panic: runtime error: slice bounds out of range
goroutine 1 [running]:
github.com/mholt/caddy/vendor/github.com/miekg/dns.ClientConfigFromFile(0xbb4739, 0x10, 0x0, 0x0, 0x0)
src/github.com/mholt/caddy/vendor/github.com/miekg/dns/clientconfig.go:86 +0xad6
github.com/mholt/caddy/vendor/github.com/xenolf/lego/acme.getNameservers(0xbb4739, 0x10, 0xfeaf20, 0x2, 0x2, 0xf77460, 0xc42004c2c0, 0xc420043f50)
src/github.com/mholt/caddy/vendor/github.com/xenolf/lego/acme/dns_challenge.go:40 +0x4d
github.com/mholt/caddy/vendor/github.com/xenolf/lego/acme.init()
src/github.com/mholt/caddy/vendor/github.com/xenolf/lego/acme/dns_challenge.go:33 +0x12d
github.com/mholt/caddy/caddy/caddymain.init()
:1 +0x75
main.init()
:1 +0x44
panic: runtime error: slice bounds out of range
`

I do not have this issue with Arch Linux and Docker 20.10.1.
Can you retest with 20.10.1 and report your findings?
Also it's probably worth asking what Architecture is the machine you are running dockprom on?

I'm on Ubuntu 20.04 LTS amd64, but that newer Docker .1 update probably fixed things.  I've already switched to Caddy v2 though.

Sure well either way I don't believe this is reproducible and isn't an issue with Caddy either way.
If you find you are experiencing issues again let me know and we can reopen the issue.

pop!os 20.10 LTS amd64 docker 20.10.1 has this issue. using caddy:1.0.3 image fixed it

I has the same issue with Ubuntu 20.04 LTS amd64 config running on AWS EC2 instance.
Ended up porting the config to Caddy v2 and using it.


Update for epinow2

This PR updates usage of EpiNow2 to match the new interface. This includes updating the arguments passed to regional_epinow and the wrapper function used to define delays (though does not alter the delays themselves).
Depends on/blocked by epiforecasts/EpiNow2#74

Missed that this hadn't been merged - doing so now.


Ansible extra variables missing when called from button

Purpose or Intent
When called from a button the Ansible extra variables were not being passed to the job.
This PR is undoing some of the changes done in a earlier PR #9306
This is a temporary solution to workaround hash not getting updated when we call each_with_object on a  DRb::DRbObject
e.g.
object.attributes.each_with_object({}) do |(key,value), hash|
hash[key] = value if key.starts_with?('param')
end
In the above case object.attributes is a DRb::DrbObject
Links

https://bugzilla.redhat.com/show_bug.cgi?id=1348232
This was broken after PR #9306

@Fryguy
Please review.

@Fryguy @jrafanie
This seems like a DRb issue it doesn't work if the object that we are iterating over is a DRbObject. I have created a simple server and client to prove this.

If the hash is a simple hash with no default proc, the each with object block can update the hash.
If the hash is a hash with default proc, it comes over as DrbObject and then the each_with_object cannot update the hash

Server https://gist.github.com/mkanoor/c1473dbeb1c991598346f171f58e9c52
Client https://gist.github.com/mkanoor/d5387afb3bcb9d140d0977eb0a2d0aea

@Fryguy @jrafanie
This seems like a DRb issue it doesn't work if the object that we are iterating over is a DRbObject. I have created a simple server and client to prove this.
If the hash is a simple hash with no default proc, the each with object block can update the hash.
If the hash is a hash with default proc, it comes over as DrbObject and then the each_with_object cannot update the hash
Server https://gist.github.com/mkanoor/c1473dbeb1c991598346f171f58e9c52
Client https://gist.github.com/mkanoor/d5387afb3bcb9d140d0977eb0a2d0aea

@mkanoor are you saying this PR as it is now does or doesn't work?

@jrafanie
This PR works around this issue, by creating a key list and then iterating over that list. We need this for Darga.

@jrafanie
Added comments around the work around


Split away the aspire hosting piece

Moves the Aspire.Hosting part of the code into the Aspire.Hosting.Raygun repository to be released as a separate package.

Looking good 👍


Elcid/testcommits

Sexy!

Travis error. Please fix.

Addresses #7

Merging. This is a great PR


Set rpc query maxTimeMS

Defaults to 1 second for calls from RPC. If queries take longer than this, they are automatically killed.
Don't merge yet, using this to easily run the tests.

Need to look into how to do it on the findOne query


Update sample counts for PASS1B-06 Release data

In support of the MoTrPAC Landscape Paper, the Summary page needs to be updated for the following:

Sample counts for existing omics if new data were supplied (e.g. RRBS, ATAC-seq)
Add sample counts for immunoassay data

The sample counts have been updated on data hub for the internally released PASS1B-06 data thus far, including immnoassay. The QC Data Monitor was the data source for extracting the sample counts.
The PASS1A/1C sample counts will be updated after the data is internally released.


Undefined control sequence with \pdfglyphtounicode (lualatex)

Describe the bug
when I use lualatex (not tested but probably the same as xelatex so should only work with pdflatex) the last two lines of the "Preamble.j2.tex" file crash the compilation. However, when I override the file and delete these two lines, it works.
To Reproduce

run : rendercv render --use-local-latex-command lualatex .\John_Doe_CV.yaml

Screenshots
Log here :
John_Doe_CV.log

This issue should be fixed with version 1.7. Could you please confirm if it has been fixed? Thank you.

Hello, no it doesn't seem to correct here are the logs I have, it's the same thing. I'll see about creating a test repo and sharing it (by tomorrow).
John_Doe_CV_2.log

@sinaatalay You are on the repo : https://github.com/c2tz/lualatex-rendercv

This issue has been fixed with version 1.8.


First pass at a dark theme

I was unable to get the full esphome stack running on my windows machine, but was able to host a small local web server with the static files in the web.esphome.io folder. This uses the user's prefers-color-scheme: dark setting to change some css values so it is a bit better on the eyes. This should close https://github.com/esphome/dashboard/issues/347.
Here are some screenshots:

I was thus unable to test any screens past these ones, so the editor will still be the light theme.
I believe this would require changing the following code.
Maybe this should be a config setting, or maybe check with js and toggle to vs-dark with a darker editor.foreground?
https://github.com/esphome/dashboard/blob/e60c7d936de424b9360ab892317723c4277404af/src/editor/monaco-provider.ts#L57-L65

@goldbattle I have some progress on the dark theme you might take a look: https://github.com/Misiu/esphome_dashboard/commit/6c8b82afd146e42280add18ab008cd17643ab4b6
Sadly not enough time to move this forward in the near future.

Thanks for the pointer. I tried to change as little as possible here in this PR.
I am using esphome in home assistant, and I couldn't find that css file.
Is the esphome-2.css is shared between this and the website?

This theme does not look good. I would suggest that you take the dark themes of Home Assistant and use those color values.

Thanks, here are the images of what it looks like using the CSS #111111 background, #1c1c1c card, and #e1e1e1 text from the frontend repo. Let me know any additional comments.
Page 1:

Page 2:


🐛 BUG: Miniflare configuration

What version of Wrangler are you using?
0.0.19
What operating system are you using?
Mac
Describe the Bug
When running Wrangler 2 on a old Workers project that used Wrangler 1, it will prompt the user with fields that are deprecated i.e. build_upload
In that deprecation message which gives suggestions/fixes, miniflare  in TOML will throw an error after those suggestions/fixes. There should be a message for miniflare deprecation in TOML and suggest removing from TOML, then use wrangler dev --local

I think the solution here is to add these fields to miniflare too.

Or do you mean having the "miniflare" field itself? https://github.com/cloudflare/wrangler2/pull/627 should've fixed that

A bigger task is all the module system work (and other features) that'll have to be replicated there, and getting more miniflare features into w2 (like the test environment, dotenv, and of course a broader story about mounts/multiple workers). Might have to wait till 2.1 but we should start planning soon.


flask login

If I want to use it like I intended i need to add login page to authorize myself as admin so I will be able to use buttons, sycn and add bato link etc on demo site.

done, pull request #33

need fixes with sessions in sql alchemy


Not Work ! help me

Can not call value of non-function type 'module'
Can you help me to resolve this issue?
Thank you

same


firefox: disable LTO by default due to segfaults with wayland

Motivation for this change
As discussed in #101429 firefox 82 started crashing when used with
wayland. A brief investigation showed that this appears to be rooted
within the LTO support that was recently added to the package. For the
time being, until someone figures out where the crashes are coming from,
we can just disable LTO.
cc @artemist
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS

I'm building this right now, I'll report back once it's done

I did dig a bit more and it seems like Fedora also started disabling LTO with the release of Firefox82 https://src.fedoraproject.org/rpms/firefox/c/65fabb34c1267a295007bc97b53cedc5a79e4154?branch=master

Pining @S-NA as he introduced LTO. Maybe he can have a look at this.


Add tests to cover more complex scenarios and cross-platform cases.

Motivation
Currently the test cases are pretty simple. More complex scenarios should be covered. Additionally tests for other platforms (namely JS and native) should also be added to make sure behaviors on those platforms are within our expectation.

In general we shouldn't be adding platform specific tests. We should be presenting a consistent cross platform interface and our tests should pass on all platforms. Definitely agree about more tests being great though!

I agree. Platform-specific tests should be avoided. That thought stemmed from that scenario I had with JS test case and our convos on Discord. But I agree that should be avoided. And at the moment, I don't think we need platform-specific tests.


[DSA]: Linked List:C++:Odd Even Linked List

Description
Given the head of a singly linked list, group all the nodes with odd indices together followed by the nodes with even indices, and return the reordered list. Implemented in C++
Domain
DSA
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Support UNC path + support directory names with space

What Changed?

Support UNC path
Support path in directory names, but space in filename is still an issue.

Issues Closed
fixes #41

Could you please create a new release? Thanks!


[10.x] Fix typehint for EventServiceProvider::$observers property

Issue:
New typehint was added https://github.com/laravel/framework/commit/d800f9ea313a1e98274dbe547b2dd1ef040a135c which is too strict (only supports array of strings).

My syntax is valid (unless string-to-string mapping is now deprecated in which case whoops I missed the memo). AFAICT it's still valid in HasEvents:

Also, while you can't instantiate objects directly in the property declaration, it's still important to define a typehint that supports objects, otherwise the following valid logic would appear invalid in static analysis:
// EventServiceProvider
$model = \App\Models\User::class;
$object = new \App\Observers\UserObserver();

$this->observers[$model] = $object;

I mean, I never use objects in this context, but it's technically valid so it should be typehinted as valid too.

https://github.com/laravel/framework/pull/46942#discussion_r1183453460
I had noted that this should be a class-string and not just a string for the docblock type hint too. Maybe @nunomaduro can chime in on that, I'm not sure how much Laravel team uses class-string but I personally love the precious 😄

@cosmastech Yeah I was itching to do that too, instead I just emulated what was done with the $listen property. Keen for class-string if that's everyone else's preference 👍


Amend Other agencies and public bodies number to 410

Also, excellent gif.

Next time I'm gonna use the GERTY faces and keep it on brand.


Bump oletools 0.60.1.dev6

Current olefy RPM does not block malicious macros designed to trick the stable oletools package. Check if a newer release of olefy+oletools is more effective.
Ref. olefy diffs from RPM version 1.2.3

I'm testing this RPM on a production environment /cc @filippocarletti

How's going? Can we release it or we can just drop it?

The newer 0.60.1 olefy version has not identified more malware.
I'm closing this pull request.


Fixes build issues with openvr-old branch

The changes from this pr are used to fix issue https://github.com/geefr/vsgvr/issues/11 and to build binary packages on OBS for Linux and Windows MinGW. They are consistent, as far as possible, with the changes made by https://github.com/geefr/vsgvr/pull/16.
The openvr-old branch is still used because of the SteamVR/OpenXR autostarting issue.

Even after these fixes there is still the problem that no content from the loaded vsg files is displayed.

Just to clearly state it: The openvr prototype here is completely deprecated and should not be used. I was planning on deleting it shortly.
At best it was an experiment based on one of my older engine prototypes, to evaluate how vsg would behave in vr. The overall outcome was functional, but showed that integrating vsg with openvr is awkward at best, and would likely limit the high-performance parts of vsg.
SteamVR or other runtimes can support openvr for compatibility but there's no sense using it for new work, now that the openxr integration here has reached feature parity. Ongoing work will all be openxr for neater code, more runtimes/platforms, and extension features like mixed reality, etc.
(If there's steamvr bugs, those are steamvr bugs. I'll be happy to work around them if possible, but it seems like it fundamentally does work at the moment, ignoring the immedite-fail-on-any-error)


Add Hover to Services Section

Improved the service section
Issue #342
Proposed Changes
-Improved the boxes in services section
-Give proper padding in each and every boxes
-Make the proper text alignment
-Add Hover effect and remove the clicks
Additional Info

Give the additional Hover effect to each boxes because of which services section look classy and make it user friendly to use.

Checklist

[x] ✅ My code follows the code style of this project.
[x] 📝 My change requires a change to the documentation.
[x] 🎀 I have updated the documentation accordingly.
[x] 🌟 ed the repo

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

Output Screenshots
Screenshot #1
Hover effect

@agamjotsingh18 I have raised this PR, Please review it and merge it so that I can work on other issues. You had already assign labels to this issues. so please see it and respond to it

I have updated the service section. Please accept the PR.

Pr has some issues in mobile view and also issue has assigned to someone else...

It has issues with responsiveness


Black screen on second monitor

Hi ! i install raspi2raspi is perfect.  My second screen is just black. what i have to do to send something into the second screen ?
THanks

Hi.
The program raspi2raspi is intended to copy from one display to another. Both screens need to be using the GPU. At the moment this means that your second screen needs to be either the Official Raspberry Pi display OR using Gert Van Loo's VGA666 adapter (or equivalent).
What two screens do you have?

The 3.5 lcd on rasberry and a hdmi screen
Envoyé de mon iPhone
Le 7 août 2017 à 17:38, Andrew Duncan <notifications<EMAIL_ADDRESS>a écrit :
Hi.
The program raspi2raspi is intended to copy from one display to another. Both screens need to be using the GPU. At the moment this means that your second screen needs to be either the Official Raspberry Pi display OR using Gert Van Loo's VGA666 adapter (or equivalent).
What two screens do you have?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/AndrewFromMelbourne/raspi2raspi/issues/3#issuecomment-320787772, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AdYfcopWA2QkNpCJAEIlYXSHGD9Yl8x_ks5sV4OegaJpZM4Ov2wF.

Hi @alefraser, the raspi2raspi program won't work with  3.5 inch display. You could probably get more help on the Raspberry Pi forum. There is a French forum as well if that is your native language - https://www.raspberrypi.org/forums/viewforum.php?f=65

Also, you can try raspi2fb which will copy the Raspberry Pi main display to a display connected to the gpios (if the display is configured correctly).


[FEATURE] Ability to navigate to "Add NWC Connection" screen

We're testing the rizful.com lightning-node-in-the-cloud service, which offers NWC connections, with Amethyst.
The first time we ran Amethyst, we were able to get to the screen where we could add the NWC connection by clicking a "zap" button. (That seems to be the only way to navigate to it? It's not in the settings?). That worked!
Now we have Amethyst running on a different device, but we have not yet figured out how we could add the NWC connection with this second device. When we try to "zap", we get...

"Unable to send zap, No Wallets found to pay a lightning invoice. Please install a Lightning wallet..."...

But so far we can't figure out how to "install a lightning wallet" ... that is, add our NWC info.... the only way to navigate to the NWC window that we have found ..... is from the zap button!

Click and hold the zap button

Ah! thanks. Closing this.


feat(winston-cw): adding region and account id to winston cloudwatch

:tada: This PR is included in version 1.71.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


No PRs open in conflict practice repository

This only occured to 1 participant out of the 20 that I generated the conflict practice repo for, and no PRs were opened by githubteacher. I went ahead and open it manually, the repo link is here: https://github.com/githubschool/conflict-practice-art-vega-sage/pulls

This happened to several of the participants in @AAugustine's class this morning.  There was at least one repo that it only created 1 of the 3 PRs as well.

I have not been able to reproduce this. 🤔

This happened again to me today... here is the log.out
log.out.zip

In alpha order, it created PR's for 3 of 7 people, and then it created 1 PR for the next person, and zero PR's for the rest. Maybe it's a time out issue?

Unfortunately those logs won’t help until we get #294 merged. 🙁

Update: #294 has been merged. Now we should start to get the missing error logs in script/log.out.
@rwnfoo @selkins13 @AAugustine please do the following to help troubleshoot:

Delete the existing script/log.out file:
rm script/log.out

Update your local scripts:
git switch master && git pull

Next time your run into this problem, send me your new script/log.out file so I can investigate. 🕵♂
As a workaround while we get to the bottom of this, simply re-running the script should do the trick. Just be sure to answer y to the following prompt for the repos missing pull requests:
Is it OK to delete and recreate? [y/N]

You can just press return to accept the default answer of N (no) for those who already have all the pull requests.


615 Creating a Tutorial for the CEP-78 JS Client

Related links
Create CEP-78 JS Client Tutorial #615
Changes
Creating a tutorial to cover the abilities of the JS Client for CEP-78.
Notes
N/A

@hoffmannjan I've incorporated your feedback - including the addition of bits about Mint, Transfer and Burn.

@ACStoneCL Everything is now addressed in suggestions / comments :)

@hoffmannjan @ACStoneCL, will this tutorial be published in a different PR, or has it been moved to another repository? I am curious why the PR was closed :).

Heya @ipopescu - We just had a malfunction while working on this and accidentally closed it.   https://github.com/casper-ecosystem/cep-78-enhanced-nft/pull/106 is the completed and merged version.

Thanks, @ACStoneCL!


Fix overlapping progress with stdout.

Minor issue introduced in #6615. The progress bar was not being cleared for stdout output (because Shell only does stderr).

r? @Eh2406
(rust_highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit 00b4efeb073a8ac1c8ff7d0753fb016b220031f0 has been approved by Eh2406

:hourglass: Testing commit 00b4efeb073a8ac1c8ff7d0753fb016b220031f0 with merge 4e74e2fc0908524d17735c768067117d3e84ee9c...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: Eh2406
Pushing 4e74e2fc0908524d17735c768067117d3e84ee9c to master...


Mods (Forge and Fabric) that depend on versions of Geckolib above 4.2.2 do not work (Using Fabric Geckolib and Connector Extras)

Mod Name
Enderman Overhaul, Forge version of Bosses of Mass Destruction
Mod Homepage
https://modrinth.com/mod/enderman-overhaul https://www.curseforge.com/minecraft/mc-mods/bosses-of-mass-destruction-forge
Minecraft version
1.20.1
Describe the bug
When you load any mod that relies on a newer version of Geckolib than 4.2.2, the game fails to load and crashes, even when I have a newer Geckolib fabric version installed (I have 4.4.4 in my mods folder, and that loads fine on its own.)
Steps to reproduce

Have Connector and Connector Extras (and their dependencies), a version of Geckolib newer than 4.2.2 fabric, and any mod that requires a newer version of Geckolib than 4.2.2, regardless of that mod's loader. (Examples I've tried are both the forge and fabric versions of Enderman Overhaul, and the forge port of bosses of mass destruction).
Try to load the game as you would normally. The load after the typical .connector folder generation crash, which normally works, crashes and complains that there is version 4.2.2 of Geckolib, when the version in the mods folder is actually much newer.

Logs
https://pastebin.com/cc0N4Tcj
Additional context
When you load the newer version of Geckolib on its own, it runs just fine, however, the mod list indicates that it is version 4.2.2. I'm not sure why this is. I don't know whether this is simply a version label that needs updated, or if this is some deeper issue. Presumably this is a connector extras issue, though, I cannot say for sure since I don't know nor understand the code for either mod.
This is with the current latest versions of forgified fabric api, connector, and connector extras. This issue actually persisted for a while in many earlier versions. My crash log is with a modpack I was trying to make, but this issue seems to persist regardless of what mods do or don't go with it. Also, even having Geckolib fabric 4.2.3 in the mods folder still gives the issue, and all versions in between 4.2.3 and 4.4.4 that I've tested.

Which Geckolib are you using?

Please provide an exact list of mods required to reproduce the issue as well as their versions. I've installed BOMD with the following dependencies and my game runs just fine.

Closing as stale / cannot reproduce.
If you can provide the necessary logs and information to reproduce the problem or identify the faulty mod, please open a new issue with all necessary details.


APIv2: figure out handling of zero-length packed extension fields

Given a repeated extension field with a non-packed encoding, what do we do when unmarshaling wire data containing a zero-length packed value for the field?
Currently, we parse this as a zero-length list. (m.Has(xd)==true, m.Get(xd).List().Len()==0.) However, there is no way to represent this using the non-packed encoding of repeated fields, so if we marshal the message out again the field vanishes. This violates the expectation that unmarshaling and remarshaling a message should leave it unchanged.

For non-extensions repeated fields, isn't it already the case that:

unmarshaling a packed zero-length field results m.Has(fd) == false and m.Get(fd).List().Len() == 0
marshaling the above results in the field disappearing (thus, already violating the expectation that round-trip unmarshal+marshal should leave the message unchanged).

To be specific, I propose m.Has(xd) report false for repeated extension fields if and only if m.Get(xd).List().Len() == 0.

I mean, the only way to do anything else would be to treat []thing{} differently from []thing(nil) like JSON does. But there are plenty of good reasons already existing on why that sort of behavior should be avoided.

Fixed by CL 204897.


Support lancer 0.9.x

Thanks for this great module!!
The beta version of lancer 0.9.x which supports Foundry 0.8.x handles HP more conveniently for deployables and characters. This gets me working health estimates (so far as I could tell) for both deployables and characters running on 0.9.4
(This version of lancer isn't officially released yet, but I leave this fix here as ready to be merged when that comes out, barring major changes or other goofs on my end)

[ ] wait for 0.9.x to hit master before merging, probably

Thank you. Just let me know when it's out so I can accept the pull.

Hello, what's the status on this? Just noticed Lancer 1.0 is out, do I still accept this or not?

Yup, it's out! Was just coming here to check on the state of the PR.
The path is still correct whenever you're good to pull it in!


WebSocket connection to 'ws://x.x.x.x/sockjs/383/gzez2mj3/websocket' failed: Error during WebSocket handshake: net::ERR_CONNECTION_RESET

After deploying my meteor app to Digital Ocean using MUP, that console error cropped up.
Any ideas?

never mind. I realized its just my ISP that is blocking insecure websockets. The solution is to use SSL, or connect to a different ISP (like tethering my phone internet, lol)

@geekyme how do you know it is your ISP?

@geekyme How did you figure out that it's your ISP? I'm using Digital Ocean also.
@Dielz98 Are you using Digital Ocean?

I forgot. I believe it's just a matter of testing different cases for me


Testing Github Pull Requests

I was wondering why drone is checking out the unmerged version of the pull request. That's the code I've found here (https://github.com/drone/drone/blob/master/pkg/build/repo/repo.go#L109-L112):
cmds = append(cmds, fmt.Sprintf("git fetch origin +refs/pull/%s/head:refs/remotes/origin/pr/%s", r.PR, r.PR))
cmds = append(cmds, fmt.Sprintf("git checkout -qf pr/%s", r.PR))
//cmds = append(cmds, fmt.Sprintf("git fetch origin +refs/pull/%s/merge:", r.PR))
//cmds = append(cmds, fmt.Sprintf("git checkout -qf %s", "FETCH_HEAD"))

Other than that, it would be good to test a pull request again as soon as the base branch has new commits.

the latest release is building using the merge ref refs/pull/1/merge instead of the head ref ``refs/pull/1/head`. I believe that was the original ask, so closing this issue now.


Upgrade didn't work (?)

I tried to upgrade using method 2. My original rooting happened on a mac, and was relatively old. I haven't really followed up and changed to newer versions of the hack.
Anyway, now I'm on windows 10, and I wanted to upgrade to the 1.6 fw version.
I managed to run dpt-tools using:
python .\dpt-tools.py -id .\deviceid.dat -k .\privatekey.dat -ip IP
I copied deviceid.dat and privatekey.dat from
C:\Users\<ME>\AppData\Roaming\Sony Corporation\Digital Paper App\DigitalPaperApp
and I found IP from wifi properties on the RP1.
I entered fw, gave the path, and got back the Success! reply.
But, after my RP1 rebooted, it reported update failed, error 90001 or something like that. All my apps where still there, and when I go to "About phone" in settings, I still see 1.4.01
And the Digital Papers app still wants to update my firmware.

Is it the first time when you flash the 1.6 fw pkg? If so, then showing "update failed" means something wrong.
First thing to check is if you have stuff in the mass storage disk. It could be highly likely you don't have enough space left in that mass storage (1Gigabyte in total) and the PKG unpacking failed in the middle. Please clear everything in that mass storage.
After that, if it still didn't work, since you are using one of the old mods, I would suggest you try to redo the initial rooting process, so I can make sure your update_script.sh is up-to-date. Let me know.

Yes, this is the first time.
When I manually mount mass_storage the directory is empty.
I tried to manually copy FwUpdater.pkg there, and then reset and enter diagnostic mode again, and the DPT does seem to do something - the screen flashes a couple of times. But eventually it reboots to my old setup.

im guess then trying method 1 also didn't work for you? if you haven't tried, go ahead.
to understand why it is like this, can you boot into diagnosis mode, and print the update log?
>> diagnosis
...
>> cd /root
>> ls
...
updater_official_mod_xxx.log
...
>> cat updater_official_mod_xxx.log

I didn't try method 1 yet.
So, what I did now is method 2, enter fw, with the 1.6 Fw as before. Then it boots regularly, complains that update failed, and I'm back to 1.4. Then I enter diagnosis mode by powering off, pressing home button while pressing power button.
There was nothing in /root, other than boot.img.bak
There is a file updater.log in /var/log/upstart
here it is:
0+1 records in
0+1 records out
644994 bytes (629.9KB) copied, 5.122372 seconds, 123.0KB/s
2+1 records in
2+1 records out
11022336 bytes (10.5MB) copied, 0.369701 seconds, 28.4MB/s
1+0 records in
1+0 records out
11022336 bytes (10.5MB) copied, 0.215627 seconds, 48.7MB/s
mount: mounting /dev/root on / failed: Device or resource busy
Terminated
Terminated
Error reading signature file /tmp/anim_sig.dat
mount: mounting /dev/root on / failed: Device or resource busy
Terminated
Terminated
Error reading signature file /tmp/anim_sig.dat
tar: invalid magic
tar: short read
mount: mounting /dev/root on / failed: Device or resource busy
Terminated
Terminated
Error reading signature file /tmp/anim_sig.dat
mount: mounting /dev/root on / failed: Device or resource busy
Terminated
Terminated
Error reading signature file /tmp/anim_sig.dat
mount: mounting /dev/root on / failed: Device or resource busy
Terminated
Terminated
Error reading signature file /tmp/anim_sig.dat
mount: mounting /dev/root on / failed: Device or resource busy
Terminated
Terminated
Error reading signature file /tmp/anim_sig.dat
mount: mounting /dev/root on / failed: Device or resource busy
Terminated
Terminated
Error reading signature file /tmp/anim_sig.dat
mount: mounting /dev/root on / failed: Device or resource busy
Terminated
Terminated
Error reading signature file /tmp/anim_sig.dat
mount: mounting /dev/root on / failed: Device or resource busy
Terminated
Terminated

What is interesting is that now the Digital Paper App sees the DPT using the USB cable, which hadn't worked after I hacked it...

OK, I used method 1. Well, partially. I flashed the official firmware, and all my apps seem to have survived and work (as far as I can tell).
I think I have an old version of the hack installed, maybe it has a different method of checking pkg files. Maybe it didn't even use pkg files....
(And while I was doing the official update, I realized that I didn't actually have a backup of all my documents.... just of one synced dir. I thought that once the App syncs with the device all is backed up, which of course it isn't!!! Maybe it would be wothwhile to point that out in the update process... so maybe I'm lucky that method 2 didn't work)

mount: mounting /dev/root on / failed: Device or resource busy this is the reason it failed.. huh..

@ghostwheel yea, the app sync function sucks..

closed as problem resolved by user


[Backport] deny all ingress traffic

https://github.com/rancher/backup-restore-operator/issues/116

Can be tested in chart rancher-backup 1.2.0-rc2


Seeing many typescript errors after cloning, "npm install" and "npm start"

A few examples.
../async_pipe.d.ts(39,38): error TS2304: Cannot find name 'Promise'.
../index.d.ts(47,36): error TS2304: Cannot find name 'Iterable'.
../index.d.ts(1479,36): error TS2339: Property 'for' does not exist on type 'SymbolConstructor'.

Answering my question:
Let me add a little more detail to @milesaylward answer. You add "lib" : ["es2015", "es2015.iterable", "dom"] as an entry to the "compilerOptions": property found in the tsconfig.json file. E.g.
{
"compilerOptions": {
"target": "es5",
... // skipping entries here
"suppressImplicitAnyIndexErrors": true, // Don't forget to add the comma here.
"lib" : ["es2015", "es2015.iterable", "dom"]
}
}
Again, I have no idea why this works. If someone can explain what the problem was and how the "lib" compile option fixed the issue - please, please do.
Many thanks in advanced.


Systems Engineer

Systems Engineer
Job type
Full-time
Location
Auckland, New Zealand
Company
MATTR
Job description
As a Systems Engineer at MATTR you will be supporting and maintaining our internal environments, and enabling everybody at MATTR to work and collaborate efficiently and securely.
Systems Engineer
Contact
Ivan Cheng (程逸凡)

Thanks for contributing, Ivan

The role is occupied, no CITA members applied.


merge jemalloc-ctl into jemallocator?

https://github.com/sfackler/jemalloc-ctl provides a typed interface to the diagnostic/control features jemalloc supports. Currently, it just asserts that the right symbols exist, but once global allocators stabilize it seems like it'd make way more sense for these APIs to live in the allocator crate itself.
Seem reasonable? I can make a PR if so.

jemalloc-sys should already be exporting all symbols that jemalloc-ctl needs, and jemallocator re-export these via its ffi module, so jemalloc-ctl could just add any of these crates as a dependency and use those. Are some symbols missing?
I don't mind exposing here a type-safe wrapper over the control features, but I could understand that others might want to keep this crate "thin".
In any case refactoring jemalloc-ctl to depend on jemallocator seems to be a good idea anyways, and should make it easier to move its functionality into this crate.

Looks like jemalloc-sys already exposes everything we need - I'll update to depend on jemalloc-sys.

FWIW I am a bit worried here again (cc @SimonSapin ).
Only the final binary or the crate that sets the global allocator can use jemalloc-ctl because no other crate can know / query which global allocator is being used neither at compile-time nor run-time.
Thinking a bit about how we could do this, we don't actually know from just looking at the dependency graph if there is one crate setting the global allocator. We only know when we actually compile that particular crate which means that this is not something that can be used statically.
Did we consider to require a crate that sets a global allocator to be a special crate type?
For example, we could have a jemalloc-global crate that just contains the couple of lines that set jemalloc as the global allocator, and those who want to use it just need to depend on that crate. If that were a special crate type, we would know exactly that the global allocator is jemalloc-global, and could define a cfg(global_alloc = "jemalloc-global") for the whole dependency graph from the beginning, which would allow crates to conditionally depend on jemalloc-ctl. This could also allow us to diagnose if two crates try to set the global allocator very early on before actually having compiled anything. It could even allow us to not have to wait till link time to resolve the allocator symbols.

I don’t see why the global allocator is relevant at all. Depending on jemalloc-ctl causes jemalloc to be linked and it controls / provides diagnostics for allocations made by jemalloc, which may or may not include those requested by e.g. Vec.

Can the following happen?
crate A depends on jemalloc-ctl, which depends on some version of jemalloc-sys that links jemalloc 4.x. crate B depends on crate A and some version of jemallocator, which depends on a different major jemalloc-sys version, linking jemalloc 5.x, and uses that to set the global allocator.

Modulo #19, it should be possible to use jemallocator without registering it as global.

Good point.

I don’t see why the global allocator is relevant at all.

Say I have a middleware library that can log some stuff. And if the global allocator is jemalloc, I'd like to use jemalloc-ctl to log some stuff. How can this library detect at compile-time and run-time if the global allocator is jemalloc?
In particular, I wonder whether it makes sense for jemalloc-ctl to require jemalloc to be linked. It might make more sense to use dlsym or weak symbols to detect at run-time whether jemalloc is the global alloc and return Err if it isn't.
This dynamic libraries to link different crates like jemalloc-ctl for different allocators, and choose the appropriate one dynamically depending on which allocator is dynamically linked to a rust binary.

Because of links in Cargo.toml https://github.com/alexcrichton/jemallocator/blob/0.1.7/jemalloc-sys/Cargo.toml#L6, Cargo won’t let you build two versions of jemalloc-sys in the same dependency graph. And if you remove that you’d most likely get linker errors for duplicate symbols.

How can this library detect at compile-time and run-time if the global allocator is jemalloc?

I believe it cannot, but I don’t know if that is a problem that needs solving. This library can have Cargo feature flags to allow its users to select different functionality.

Because of links in Cargo.toml https://github.com/alexcrichton/jemallocator/blob/0.1.7/jemalloc-sys/Cargo.toml#L6, Cargo won’t let you build two versions of jemalloc-sys in the same dependency graph. And if you remove that you’d most likely get linker errors for duplicate symbols.

Nice, thanks, this makes sense.

This library can have Cargo feature flags to allow its users to select different functionality.

This means that every library that uses it would need to re-export those features, can be done, but feels a bit clunky.

Some badness can apparently happen with a stdlib-provided jemalloc and jemalloc-sys provided jemalloc: #51.

Seem reasonable? I can make a PR if so.

Please do so whenever you can.


Render SVG from Draw.io

Hi, I'm trying to display some SVG from Draw.io, but I got some strange results.
I want to use manim for development and programming videos :smile:
The code :
class Uml(SVGMobject):
  CONFIG = {
    "file_name": "/home/necros/Downloads/FormObjects-Models.svg",
    "unpack_groups": False
  }

class UmlRender(Scene):
  def construct(self):
    uml = Uml()
    uml.scale(5)

    self.play(DrawBorderThenFill(uml), run_time=5)
    self.wait(5)

Expected result :

Actual result :

Exemple file : https://filebin.net/vhs2w1zlwbqeadhv/FormObjects-Models__1_.svg?t=byk639bd
In hope you have any idea to help me.
Regards

I try to download your file from https://srv-file2.gofile.io/download/ANL8aV/FormObjects-Models.svg
but it said You are not authorized to download this file.

Oh, I change the provider : https://filebin.net/vhs2w1zlwbqeadhv/FormObjects-Models__1_.svg?t=byk639bd

The best result I can get:
class Uml(SVGMobject):
  CONFIG = {
    "file_name": "/home/necros/Downloads/FormObjects-Models.svg",
    "fill_opacity":0,
    "stroke_color": WHITE,
    "stroke_width": 1,
  }

I think there are multiple reasons caused this problem.

Because manim uses a simple(handmade) svg parser. to translate svg into SVGMobject.
Manim can't recognize the color and other settings of a svg.(It's not a bad thing)
Because manim can't get those settings, so manim don't know where to stroke, where to fill.

I think this problem can't be solved very quickly,
you can create those Graphics by using Mobject
like Rectangle(), TextMobject(), Arrow(), line()

Did you find a solution? I am getting a similar issue. I am using the cicuitikz package to draw some circuits in Latex and then animate using manim. The SVG file created is correct and all components are shows correctly. But manim ommits some lines from the figure. I think its some issue with the parser.

No sorry, I gave up this issue and tried to animate the block myself.
Each arrows are just a simple Arrow(block1, block2)

The solution I found is to import into Adobe Illustrator, select all, and then go to Edit -> Compound Path -> Release
Sometimes Edit -> Compound Path -> Make also works


Implemented EF with Repository Pattern, DbContext, and LINQ

Repository pattern: Updated IDatabaseRepository<T> and created CheepRepository () and AuthorRepository ()
Resolves #70
DTO: We had already implemented DTOs as record CheepViewModel(Author:string, Message:string, Timestamp:string)
Resolves #69
DbContext & DbInitializer
Created ChirpDBContext and refactored files such as CheepService and DBFacade to work with our DbContext. Our DbContext is being seeded by the data from DbInitializer.cs.
Resolves #68
Connected the Repositories and DBFacade by removing direct SQL code and instead use LINQ.
Resolves #78

Please ignore the .Net Build and Test not working, as it seems I've pulled the old IntegrationTests.cs :)


nu_correct: crashed while running nu_estimate_np_and_em (termination status=256)

What happened?
Hello, I'm trying to run fmriprep on a singularity container running on a cluster and I'm getting the following error on the html log generated:
Traceback (most recent call last):
  File "/opt/conda/lib/python3.8/site-packages/nipype/pipeline/plugins/multiproc.py", line 67, in run_node
    result["result"] = node.run(updatehash=updatehash)
  File "/opt/conda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 516, in run
    result = self._run_interface(execute=True)
  File "/opt/conda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 635, in _run_interface
    return self._run_command(execute)
  File "/opt/conda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 741, in _run_command
    result = self._interface.run(cwd=outdir)
  File "/opt/conda/lib/python3.8/site-packages/nipype/interfaces/base/core.py", line 428, in run
    runtime = self._run_interface(runtime)
  File "/opt/conda/lib/python3.8/site-packages/nipype/interfaces/base/core.py", line 822, in _run_interface
    self.raise_exception(runtime)
  File "/opt/conda/lib/python3.8/site-packages/nipype/interfaces/base/core.py", line 749, in raise_exception
    raise RuntimeError(
RuntimeError: Command:
recon-all -autorecon1 -noskullstrip -openmp 8 -subjid sub-0002 -sd /data/derivatives/fmriprep/sourcedata/freesurfer -nomotioncor
Standard output:
Subject Stamp: freesurfer-Linux-centos6_x86_64-stable-pub-v6.0.1-f53a55a
Current Stamp: freesurfer-Linux-centos6_x86_64-stable-pub-v6.0.1-f53a55a
INFO: SUBJECTS_DIR is /data/derivatives/fmriprep/sourcedata/freesurfer
Actual FREESURFER_HOME /opt/freesurfer
-rw-rw-r-- 1 ivaz navigator 21860 Mar 23 15:54 /data/derivatives/fmriprep/sourcedata/freesurfer/sub-0002/scripts/recon-all.log
Linux barinel057 3.10.0-1062.18.1.el7.x86_64 #1 SMP Tue Mar 17 23:49:17 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
'/opt/freesurfer/bin/recon-all' -> '/data/derivatives/fmriprep/sourcedata/freesurfer/sub-0002/scripts/recon-all.local-copy'
#--------------------------------------------
#@# Talairach Wed Mar 23 16:04:02 WET 2022
/data/derivatives/fmriprep/sourcedata/freesurfer/sub-0002/mri

 mri_nu_correct.mni --no-rescale --i orig.mgz --o orig_nu.mgz --n 1 --proto-iters 1000 --distance 50

/data/derivatives/fmriprep/sourcedata/freesurfer/sub-0002/mri
/opt/freesurfer/bin/mri_nu_correct.mni
--no-rescale --i orig.mgz --o orig_nu.mgz --n 1 --proto-iters 1000 --distance 50
nIters 1
$Id: mri_nu_correct.mni,v 1.27 2016/02/26 16:19:49 mreuter Exp $
Linux barinel057 3.10.0-1062.18.1.el7.x86_64 #1 SMP Tue Mar 17 23:49:17 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Wed Mar 23 16:04:03 WET 2022
Program nu_correct, built from:
Package MNI N3, version 1.12.0, compiled by nicks@terrier (x86_64-unknown-linux-gnu) on 2015-06-19 at 01:25:34
/bin/bc
tmpdir is ./tmp.mri_nu_correct.mni.22790
/data/derivatives/fmriprep/sourcedata/freesurfer/sub-0002/mri
mri_convert orig.mgz ./tmp.mri_nu_correct.mni.22790/nu0.mnc -odt float
mri_convert.bin orig.mgz ./tmp.mri_nu_correct.mni.22790/nu0.mnc -odt float
$Id: mri_convert.c,v 1.226 2016/02/26 16:15:24 mreuter Exp $
reading from orig.mgz...
TR=2300.00, TE=0.00, TI=0.00, flip angle=0.00
i_ras = (-1, 0, 0)
j_ras = (0, 0, -1)
k_ras = (0, 1, 0)
changing data type from uchar to float (noscale = 0)...
writing to ./tmp.mri_nu_correct.mni.22790/nu0.mnc...

--------------------------------------------------------
Iteration 1 Wed Mar 23 16:04:04 WET 2022
nu_correct -clobber ./tmp.mri_nu_correct.mni.22790/nu0.mnc ./tmp.mri_nu_correct.mni.22790/nu1.mnc -tmpdir ./tmp.mri_nu_correct.mni.22790/0/ -iterations 1000 -distance 50
[ivaz@barinel057:/data/derivatives/fmriprep/sourcedata/freesurfer/sub-0002/mri/] [2022-03-23 16:04:04] running:
  /opt/freesurfer/mni/bin/nu_estimate_np_and_em -parzen -log -sharpen 0.15 0.01 -iterations 1000 -stop 0.001 -shrink 4 -auto_mask -nonotify -b_spline 1.0e-7 -distance 50 -quiet -execute -clobber -nokeeptmp -tmpdir ./tmp.mri_nu_correct.mni.22790/0/ ./tmp.mri_nu_correct.mni.22790/nu0.mnc ./tmp.mri_nu_correct.mni.22790/nu1.imp

Assertion failed at line 742 in file templates/CachedArray.cc
nu_estimate_np_and_em: crashed while running volume_stats (termination status=256)
nu_correct: crashed while running nu_estimate_np_and_em (termination status=256)
ERROR: nu_correct
Linux barinel057 3.10.0-1062.18.1.el7.x86_64 #1 SMP Tue Mar 17 23:49:17 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

recon-all -s sub-0002 exited with ERRORS at Wed Mar 23 16:04:04 WET 2022

For more details, see the log file /data/derivatives/fmriprep/sourcedata/freesurfer/sub-0002/scripts/recon-all.log
To report a problem, see http://surfer.nmr.mgh.harvard.edu/fswiki/BugReporting

Standard error:

Return code: 1

What command did you use?
#!/bin/bash
#
#SBATCH -A myproject
#SBATCH -J fmriprep
#SBATCH --time=48:00:00
#SBATCH -n 1
#SBATCH --cpus-per-task=16
#SBATCH --mem-per-cpu=4G
#SBATCH -p cpu1  # Queue names you can submit to
# Outputs ----------------------------------
#SBATCH -o /veracruz/projects/c/myproject/igor/myproject/log/%x-%A-%a.out
#SBATCH -e /veracruz/projects/c/myproject/igor/myproject/log/%x-%A-%a.err
#SBATCH<EMAIL_ADDRESS>#SBATCH --mail-type=ALL
# ------------------------------------------

# Import the Singularity module
module load Singularity

temp=`mktemp -d -p /veracruz/projects/c/myproject/fmriprep/tmp`

export TMPDIR=$temp
export SINGULARITY_TMPDIR=$temp
export SINGULARITYENV_TMPDIR=$temp
export SINGULARITY_CACHEDIR=$temp
export SINGULARITYENV_CACHEDIR=$temp

export SINGULARITYENV_MPLCONFIGDIR=$TMPDIR/matplotlib
mkdir -p ${SINGULARITYENV_MPLCONFIGDIR}

HOST_WORK=/veracruz/projects/c/myproject/fmriprep/tmp/work
mkdir -p ${HOST_WORK}

# Set the study folders ADNI
export STUDY=/veracruz/projects/c/myproject/study

BIDS_DIR="$STUDY/bids_subset"
DERIVS_DIR="$BIDS_DIR/derivatives"
LOCAL_FREESURFER_DIR="$DERIVS_DIR/freesurfer"

# Prepare some writeable bind-mount points.

# Output results to a table
echo "sub-$subject   ${SLURM_ARRAY_TASK_ID}    $exitcode" \
      >> ${SLURM_JOB_NAME}.${SLURM_ARRAY_JOB_ID}.tsv
echo Finished tasks ${SLURM_ARRAY_TASK_ID} with exit code $exitcode
exit $exitcode

What version of fMRIPrep are you running?
21.0.1
How are you running fMRIPrep?
Singularity
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
No response
Additional information / screenshots
I'm also running the .slurm script from a bash script with the following code:
#! /bin/bash

sbatch --array=1-$(( $( wc -l /veracruz/projects/c/myproject/study/bids_subset/participants.tsv | cut -f1 -d' ' ) - 1 )) /veracruz/projects/c/myproject/fmriprep/sbatch_igor.slurm

This looks related to https://github.com/nipreps/fmriprep/issues/1975 - when using the --cleanenv flag, environmental variables on the host will not be passed in, except if they start with SINGULARITYENV_*. My guess is there are some variables pointing to paths that don't exist within the container, such as:
export SINGULARITYENV_FS_LICENSE=/veracruz/projects/c/myproject/fmriprep/license.txt
export SINGULARITYENV_TMPDIR=$temp

I'd also double check the expanded $cmd and make sure everything looks good.

thanks for reaching back @mgxd. I've ended up solving this issue by not setting the SINGULARITYENV's at all, and the nu_currect issue was gone


Add text and hover_name

These are straightforward arguments that work similar to size or error bars.

Added in https://github.com/deephaven/deephaven-plugin-plotly-express/commit/71173ee696315af3ca5093aa60a6f5e0a2c44cd9


Make Ratis client retry on catchup failures

Sometimes a Ratis client decides to timeout and close connection when master tries to gain primacy. This change makes the master to notify Ratis client to reconnect when an exception is throw, so we don't just keep retrying on a closed client.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/11323/
Test PASSed.

@gpang @LuQQiu PTAL.

alluxio-bot, merge this please.


Check if cloudflare popup warning is laggy

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @zach in #feedback.

It is not.


Update wapi.js

Bug fixes

Is this the final commit or is there more? Can I merge?

At this moment yes. I am preparing new updates and functions also for this week.


Update vendored dependencies to new Kubernetes 1.10 release

/hold

/hold cancel

/assign @zetaab

/assign @hogepodge

/ok-to-test
/approve

Thanks! There was a change that I had to revert in the CSI driver when synchronizing with the old repo. I see it in this PR and the rest of the changes look good. I'll move this forward

/lgtm


[Common Issues] Releasing LLaVA-OneVision data yaml files in three stages (mid/single-image/onevision)

Checkout here to see the three yamls.
https://github.com/LLaVA-VL/LLaVA-NeXT/tree/main/scripts/train

Cross pin for explaination on video data.
https://github.com/LLaVA-VL/LLaVA-NeXT/issues/130

Q: About video data?
A: It's to be released in @ZhangYuanhan-AI next version of a more powerful video model.
Currently we released the data yaml used in onevision stage at onevision.yaml.
You can checkout the three subsets video data, (1) sharegpt4video_255000.json (checkout sharegpt4video) (2) 0718_0_30_s_academic_mc_v0_1_all.json (to be released) (3) academic_source_30s_v1_all.json (to be released).


Ef core 2.0 compatible ??

Hie Comrad!
I have a question.
This solution is Ef core 2.0 compatible ??

Maybe could help !
How to make solution with pluginable assemly ?
Something similar to https://stackoverflow.com/questions/8730830/mef-plugins-and-ef-codefirst-how
I want to make model design from GUI and then aplly model to working application !


progressTo function

added progressTo function

example , used as file upload progress
,https://pazhanji.org/

Thanks for your commit 👍

Thank you :)


post routes take json, get not

we need to mainline the parameter parsing. best is: provide both, URLEncoding and JSON

duplicate of #20


Fix ctx in tx begin

This PR fixes #2253

Fix old instruction in doc
Fix missing context in TxBegin & Write test for it
Run go generate in root folder

Tested 2-nd point according to docs, everything looks ok

Thanks for code review @a8m , I have one last question about this thread https://github.com/ent/ent/pull/2260#discussion_r781901502

Thanks @a8m and @masseelch for code-review and some clarifications, I think now it's ready

@Laconty, please run codegen (go generate ./...) and resubmit the PR. You may also want to add the email in the commit message to your GitHub account so GitHub will know to link this commit to your user after it's merged to master.

Run go generate in root folder
re-author commits (thanks for your note)

Now I hope It's ready @a8m 🙂
(you wrote I should resubmit PR, I've changed author by rebase, hope that's ok)

Welcome to the Ent community @Laconty, and really thanks for your contribution. It's appreciated!
@all-contributors please add @Laconty for code


🚀 Feature: Offer to create a repository in setup if it doesn't yet exist

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
When running setup (pnpm setup) on a new project that doesn't yet have a Git repo, this error happens:
Hydrating initial repository settings...

Hydrating branch protection settings...
/Users/josh/repos/should-skip-semantic-release/node_modules/.pnpm/@octokit+request@6.2.2/node_modules/@octokit/request/dist-node/index.js:88
      const error = new requestError.RequestError(toErrorMessage(data), status, {
                    ^

RequestError [HttpError]: Not Found
    at /Users/josh/repos/should-skip-semantic-release/node_modules/.pnpm/@octokit+request@6.2.2/node_modules/@octokit/request/dist-node/index.js:88:21
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Job.doExecute (/Users/josh/repos/should-skip-semantic-release/node_modules/.pnpm/bottleneck@2.19.5/node_modules/bottleneck/light.js:405:18) {
  status: 404,
  response: {
    url: 'https://api.github.com/repos/JoshuaKGoldberg/should-skip-semantic-release',
    status: 404,
...

The setup script should use the GitHub API to create a new repository for the specified owner under the given kebab-case name. ✨
Additional Info
Steps to repro:

Copy the template package to a new directory locally. E.g.:cp -r path/to/template-typescript-node-package some/new/path
cd some/new/path
rm -rf node_modules
pnpm install

Run setup (pnpm run setup), filling in any info and having it call to the GitHub API

I'm unable to reproduce the same error, seems it was trying to create a repo or...?
Welcome to the template-typescript-node-package package setup!
This setup script will hydrate your repository based on your provided settings.

✔ What will the kebab-case name of the repository be? · test-node-template
✔ What will the Title Case title of the repository be? · TestNodeTemplate
✔ What owner or user will the repository be under? · TAKANOME-DEV
✔ How would you describe the new package? ·
✔ Whether to skip calling the GitHub API (effectively making this a local-only change). ·

Hydrating package metadata locally...
✔ Done.

Clearing CHANGELOG.md...
✔ Done.

Deleting local git tags...
✔ Done.
✔ Done.

Checking gh auth status...
✔ Done.

Hydrating repository labels...
Error: HTTP 404: Not Found (https://api.github.com/repos/JoshuaKGoldberg/template-typescript-node-package/labels/area:%20documentation)
    at file:///home/takanome_dev/Projects/OpenSource/JoshuaKGoldberg/test-bug/script/setup.js:193:11
    exit code: 1
    at ChildProcess.<anonymous> (file:///home/takanome_dev/.npm/_npx/721a1717bb84573e/node_modules/zx/build/core.js:146:26)
    at ChildProcess.emit (node:events:513:28)
    at ChildProcess.emit (node:domain:489:12)
    at maybeClose (node:internal/child_process:1091:16)
    at ChildProcess._handle.onexit (node:internal/child_process:302:5)
    at Process.callbackTrampoline (node:internal/async_hooks:130:17)

Removing devDependency packages only used for setup...
✔ Done.

Looks like there was a problem. Correct it and try again? 😕

Note

I was confused by this Whether to skip calling the GitHub API (effectively making this a local-only change).
I didn't know what to respond and there is no hint
The last message: Looks like there was a problem. Correct it and try again? 😕
Maybe we should change the correct it part with something else :sweat_smile:

Heh, glad to see it's not quite so crash-y now! But this blob of error stack isn't what people would want to see:
Hydrating repository labels...
Error: HTTP 404: Not Found (https://api.github.com/repos/JoshuaKGoldberg/template-typescript-node-package/labels/area:%20documentation)
    at file:///home/takanome_dev/Projects/OpenSource/JoshuaKGoldberg/test-bug/script/setup.js:193:11
    exit code: 1
    at ChildProcess.<anonymous> (file:///home/takanome_dev/.npm/_npx/721a1717bb84573e/node_modules/zx/build/core.js:146:26)
    at ChildProcess.emit (node:events:513:28)
    at ChildProcess.emit (node:domain:489:12)
    at maybeClose (node:internal/child_process:1091:16)
    at ChildProcess._handle.onexit (node:internal/child_process:302:5)
    at Process.callbackTrampoline (node:internal/async_hooks:130:17)

The error in particular is curious. Error: HTTP 404: Not Found (https://api.github.com/repos/JoshuaKGoldberg/template-typescript-node-package/labels/area:%20documentation) - why is it calling to JoshuaKGoldberg/template-typescript-node-package? Shouldn't it be asking your own repo? This might be a separate bug.
Either way, I think this feature request still stands - that making a new repo if one doesn't exist would be nice.

I was confused by this Whether to skip calling the GitHub API (effectively making this a local-only change).
I didn't know what to respond and there is no hint

Yeah, that's not great a great dev experience bit right now. You did the right thing by ignoring. The fix should be fairly straightforward if you're interested! #184

The last message: Looks like there was a problem. Correct it and try again? 😕
Maybe we should change the correct it part with something else :sweat_smile:

Got any suggestions?

Why is it calling to JoshuaKGoldberg/template-typescript-node-package? Shouldn't it be asking your own repo? This might be a separate bug.

Or maybe I did something wrong :thinking: I will retry it and report back if it's the same

Either way, I think this feature request still stands - that making a new repo if one doesn't exist would be nice.

Yeah, definitely. I'll look into it

The fix should be fairly straightforward if you're interested! #184

Sure,will take that :+1:

Maybe we should change the correct it part with something else sweat_smile

Got any suggestions?

What do you think of:
"Sorry, there was an error. Please check your inputs and try again. If you need help, reach out to us."

"Sorry, there was an error. Please check your inputs and try again. If you need help, reach out to us."

🔥 Looks great! A couple of suggestions:

Maybe replace "Sorry" with "Huh"? Nothing wrong with apologizing, but I think so many apps say "sorry" it's kind of lost meaning / feels generic to me.
Maybe something more specific than "reach out to us" - it might not be clear how to do that. Maybe link to this GitHub repo?

Maybe replace "Sorry" with "Huh"? Nothing wrong with apologizing, but I think so many apps say "sorry" it's kind of lost meaning / feels generic to me.

Agree

Maybe something more specific than "reach out to us" - it might not be clear how to do that. Maybe link to this GitHub repo?

Yeah, forgot about the link. And maybe the link can be a new bug report issue :thinking:

#398 raises a good point: what if the repository name was just mistyped? In this case I think the script should offer the person to try again or create a new repository for them.
Removing the good first issue label as this is a bit more complex now.


coq propagate ocamlBuildInputs

Motivation for this change
Since coq and its plugin now rely on findlib (cf https://github.com/coq/coq/pull/15220), we have to change how we deal with the ocaml dependencies.
This PR tries to solve the problem by promoting ocaml libraries to propagatedBuildInputs.
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build coqPackages_8_7 coqPackages_8_8 coqPackages_8_9 coqPackages_8_10 coqPackages_8_11 coqPackages_8_12 coqPackages_8_13 coqPackages_8_14 coqPackages_8_15

Note that on master, the above expression builds, but I can't use it to build any of my projects due to zarith being missing.

Thank you, that did the trick:
coq_HEAD = (self.coq_8_15.override {
  buildIde = false;
  version = ~/src/coq;
}).overrideAttrs (attrs: {
  buildInputs = attrs.buildInputs
    ++ (with pkgs; [
      texlive.combined.scheme-full which hevea fig2dev imagemagick_light git
    ]);
});

Now I wonder why my package builds with this version, while it fails with master in CI...

Now I wonder why my package builds with this version, while it fails with master in CI...

what is the failure?

Result of nixpkgs-review pr 161977 run on aarch64-darwin 1

3 packages marked as broken and skipped:

coqPackages.interval
coq_8_5
coq_8_6

7 packages failed to build:

coqPackages.coqtail-math
coq_8_10
coq_8_7
coq_8_8
coq_8_9
framac
why3

76 packages built:

coq (coqPackages.coq ,coq_8_13)
coqPackages.Cheerios
coqPackages.CoLoR
coqPackages.ITree
coqPackages.InfSeqExt
coqPackages.LibHyps
coqPackages.QuickChick
coqPackages.StructTact
coqPackages.Verdi
coqPackages.aac-tactics
coqPackages.addition-chains
coqPackages.autosubst
coqPackages.bignums
coqPackages.category-theory
coqPackages.ceres
coqPackages.coq-bits
coqPackages.coq-elpi
coqPackages.coq-ext-lib
coqPackages.coq-record-update
coqPackages.coqeal
coqPackages.coqhammer
coqPackages.coqprime
coqPackages.coquelicot
coqPackages.corn
coqPackages.deriving
coqPackages.dpdgraph
coqPackages.equations
coqPackages.extructures
coqPackages.flocq
coqPackages.fourcolor
coqPackages.gaia
coqPackages.gaia-hydras
coqPackages.gappalib
coqPackages.goedel
coqPackages.graph-theory
coqPackages.hierarchy-builder
coqPackages.hydra-battles
coqPackages.iris
coqPackages.itauto
coqPackages.math-classes
coqPackages.mathcomp
coqPackages.mathcomp-abel
coqPackages.mathcomp-algebra
coqPackages.mathcomp-analysis
coqPackages.mathcomp-bigenough
coqPackages.mathcomp-character
coqPackages.mathcomp-field
coqPackages.mathcomp-fingroup
coqPackages.mathcomp-finmap
coqPackages.mathcomp-real-closed
coqPackages.mathcomp-solvable
coqPackages.mathcomp-ssreflect
coqPackages.mathcomp-tarjan
coqPackages.mathcomp-word
coqPackages.mathcomp-zify
coqPackages.metalib
coqPackages.multinomials
coqPackages.odd-order
coqPackages.paco
coqPackages.paramcoq
coqPackages.parsec
coqPackages.pocklington
coqPackages.reglang
coqPackages.relation-algebra
coqPackages.semantics
coqPackages.serapi
coqPackages.simple-io
coqPackages.smpl
coqPackages.stdpp
coqPackages.tlc
coqPackages.topology
coqPackages.zorns-lemma
coq_8_11
coq_8_12
coq_8_14
coq_8_15

coq_8_9

I did not expect that... can I see the logs?

error: builder for '/nix/store/78ipszxkb5y7pdal5m334s5xpzlszawl-ocaml-4.09.1.drv' failed with exit code 2;
       last 10 log lines:
       > make[4]: *** [Makefile:343: roots_nat_n.o] Error 1
       > make[4]: *** Waiting for unfinished jobs....
       > make[4]: Leaving directory '/private/tmp/nix-build-ocaml-4.09.1.drv-0/ocaml-4.09.1/runtime'
       > make[3]: *** [Makefile:1002: makeruntimeopt] Error 2
       > make[3]: Leaving directory '/private/tmp/nix-build-ocaml-4.09.1.drv-0/ocaml-4.09.1'
       > make[2]: *** [Makefile:415: opt.opt] Error 2
       > make[2]: Leaving directory '/private/tmp/nix-build-ocaml-4.09.1.drv-0/ocaml-4.09.1'
       > make[1]: *** [Makefile:466: world.opt] Error 2
       > make[1]: Leaving directory '/private/tmp/nix-build-ocaml-4.09.1.drv-0/ocaml-4.09.1'
       > make: *** [/nix/store/0rawfy9g1y4c2h4s60sjm6m80cri92n3-Makefile.nixpkgs:16: nixpkgs_world_bootstrap_world_opt] Error 2
       For full logs, run 'nix log /nix/store/78ipszxkb5y7pdal5m334s5xpzlszawl-ocaml-4.09.1.drv'.
error: 1 dependencies of derivation '/nix/store/h8sr7v7y6w2dryhgjsvqd6jvb54b29dk-camlp5-7.14.drv' failed to build
error: 1 dependencies of derivation '/nix/store/z1vp1kwhsmwlhk1f37p9dnm134z2psj2-ocaml-findlib-1.9.3.drv' failed to build
error: 2 dependencies of derivation '/nix/store/dianamifb7amz35ykhi31nfrlvq99r92-dune-2.9.3.drv' failed to build
error: 2 dependencies of derivation '/nix/store/5q4q2yii0qmgiw7yrdms1idf2zb7vg5b-lablgtk-2.18.12.drv' failed to build
error: 2 dependencies of derivation '/nix/store/ardy43jkgrlv2lcqc638ixn73j3dyhmy-ocaml4.09.1-num-1.1.drv' failed to build
error: 5 dependencies of derivation '/nix/store/s17g0a3zlxs4glhir81qansfk3l9nkp2-coq-8.8.2.drv' failed to build
error: 3 dependencies of derivation '/nix/store/aywhp0spv95s1xbfi82c99vm6sqhfv9x-ocaml4.09.1-result-1.5.drv' failed to build
error: 4 dependencies of derivation '/nix/store/3k34zf0v4qb4gvm11nb8c72nzmk1zwmq-ocaml4.09.1-csexp-1.5.1.drv' failed to build
error: 5 dependencies of derivation '/nix/store/6qa2gabs5aq5xv562hg31d1p0r8gvg5w-ocaml4.09.1-dune-configurator-2.9.3.drv' failed to build
error: 4 dependencies of derivation '/nix/store/7vf45k7jnlbmd3nffcy6gkn2naqzxpi4-ocaml4.09.1-cairo2-0.6.2.drv' failed to build
error: 5 dependencies of derivation '/nix/store/fv6kbrzxxgl5q28zf5yyv9a5nf1pqx7x-ocaml4.09.1-lablgtk3-3.1.2.drv' failed to build
error: 5 dependencies of derivation '/nix/store/vs4vysfliv93626j233ds2dzls8n3frd-ocaml4.09.1-lablgtk3-sourceview3-3.1.2.drv' failed to build
error: 5 dependencies of derivation '/nix/store/rz4gdhqd8s3nqrc6537ya0p31pblni2k-coq-8.7.2.drv' failed to build
error: 5 dependencies of derivation '/nix/store/5x3mlrhfg4y5gvzng3cz75z0xpa6k4cr-coq-8.9.1.drv' failed to build
error: 4 dependencies of derivation '/nix/store/dnh10w77lkh0ws2xlv8fw3g1500xf5c7-coq-8.10.2.drv' failed to build

$ nix log /nix/store/78ipszxkb5y7pdal5m334s5xpzlszawl-ocaml-4.09.1.drv | tail -n 1000 | ix
http://ix.io/3Uzg

@CohenCyril Never mind, the issue was fixed on Coq master within the last few days.

@GrahamcOfBorg build coqPackages_8_7

@GrahamcOfBorg build coqPackages_8_9

@GrahamcOfBorg build coqPackages_8_10

How do we run tests for versions of Coq lower than the ones handled in the toolbox?

Is there a deep reason for the toolbox not to support versions earlier than 8.10?

I guess the only reason is ecological?

That was my guess. Then, I would suggest that:

We reduce the number of versions that are built by default (8.10-8.15 is already too many).
We add a way to trigger the build of more versions when we need it (e.g., using a manual event trigger).

Most nixpkgs PRs do not need to be tested on large ranges of versions because they update package versions for recent package sets only.

Result of nixpkgs-review pr 161977 run on x86_64-darwin 1

3 packages failed to build:

coqPackages.coqtail-math
framac
why3

86 packages built:

compcert
coq (coqPackages.coq ,coq_8_13)
coqPackages.Cheerios
coqPackages.CoLoR
coqPackages.ITree
coqPackages.InfSeqExt
coqPackages.LibHyps
coqPackages.QuickChick
coqPackages.StructTact
coqPackages.VST
coqPackages.Verdi
coqPackages.aac-tactics
coqPackages.addition-chains
coqPackages.autosubst
coqPackages.bignums
coqPackages.category-theory
coqPackages.ceres
coqPackages.compcert
coqPackages.coq-bits
coqPackages.coq-elpi
coqPackages.coq-ext-lib
coqPackages.coq-record-update
coqPackages.coqeal
coqPackages.coqhammer
coqPackages.coqprime
coqPackages.coquelicot
coqPackages.corn
coqPackages.deriving
coqPackages.dpdgraph
coqPackages.equations
coqPackages.extructures
coqPackages.flocq
coqPackages.fourcolor
coqPackages.gaia
coqPackages.gaia-hydras
coqPackages.gappalib
coqPackages.goedel
coqPackages.graph-theory
coqPackages.hierarchy-builder
coqPackages.hydra-battles
coqPackages.interval
coqPackages.iris
coqPackages.itauto
coqPackages.math-classes
coqPackages.mathcomp
coqPackages.mathcomp-abel
coqPackages.mathcomp-algebra
coqPackages.mathcomp-analysis
coqPackages.mathcomp-bigenough
coqPackages.mathcomp-character
coqPackages.mathcomp-field
coqPackages.mathcomp-fingroup
coqPackages.mathcomp-finmap
coqPackages.mathcomp-real-closed
coqPackages.mathcomp-solvable
coqPackages.ssreflect (coqPackages.mathcomp-ssreflect)
coqPackages.mathcomp-tarjan
coqPackages.mathcomp-word
coqPackages.mathcomp-zify
coqPackages.metalib
coqPackages.multinomials
coqPackages.odd-order
coqPackages.paco
coqPackages.paramcoq
coqPackages.parsec
coqPackages.pocklington
coqPackages.reglang
coqPackages.relation-algebra
coqPackages.semantics
coqPackages.serapi
coqPackages.simple-io
coqPackages.smpl
coqPackages.stdpp
coqPackages.tlc
coqPackages.topology
coqPackages.zorns-lemma
coq_8_10
coq_8_11
coq_8_12
coq_8_14
coq_8_15
coq_8_5
coq_8_6
coq_8_7
coq_8_8
coq_8_9

@siraben @Zimmi48 @vbgl all the external tests passed: https://github.com/coq-community/coq-nix-toolbox/pull/99

@ulrikstrid thanks for your review. I took your comments into account, please approve the PR if you are satisfied with my answer.

@Zimmi48 @vbgl are you satisfied with the changes?

All toolbox checks have passed...
@vbgl final approval & merge?


[HDRP] Underwater effect not displayed through high bumps of water

Describe the bug
The underwater effect is not displaying when moving the camera through large bumps/wave peaks. It does display the closer/lower the camera is to the sea level.
Screenshots / video

Log
editor-log.txt
Versions
Unity 2020.1.0b2/2020.1.0b3 HDRP 8.0.1
To Reproduce

Open boatdev/scenes/boat.unity
Click play and drive the boat toward one of the high bumps in the water
Observe the lack of underwater effect while moving the camera through the water peaks

Platform

Editor
Windows 10 Education 1903

Hardware
Ryzen 5 1600, 16GB Corsair Vengeance, EVGA RTX 2070 Super

Thanks for reporting. The underwater water effect is currently set to be disabled if the viewer is 2 metres above the surface. We should expose this property for those who need it.
But 2 metres should also work in this scenario. I tested it on built-in and the underwater effect works with a check of 1.5 metres.
I just checked and it looks like height queries aren't working properly for animated waves added using inputs. The ViewerHeightAboveWater remains the same when flying through these bumps.

There seems to be more going on here. It also looks like a displacement issue too. High chop values on larger waves produce inaccurate results.
The local gerstner waves don't change the value even with no chop. But the water height objects (plane and sphere) work perfectly fine.
Reproducible in Editor and Standalone on MacOS.

Thanks @daleeidd . It might be the iteration count in the query displacements compute shader, similar to what you found with the clipping. Could you please try with a higher count?

It didn't have an effect. I tried increasing it to 20. I also tried unrolling the loop manually.
I realised I have incurred this issue before. It is related to this issue. By passing a minimum length value of 2 for the viewer for the displacement sampler, the height now works correctly.
I tested further with extreme displacements (large waves with high chop), and I needed to increase this value for it to continue to work correctly.

Thanks @daleeidd thats very useful.
I can repro it here, this is a good setup to debug. My sphere has the sample height script on it so it tracks the surface, and also it shows the local collision area using that other component.

Think i found a couple of issues. Firstly the computation of minSlice in CSMain() of QueryDisplacements is very dangerous as if minGridSize is 0, that will compute a log of 0 which is a NaN, which may be causing a crash that  @mattmac reported which seemed to have NaNs from the query shader. There's also a potential log(0) in PosToSliceIndices.
Secondly, PosToSliceIndices() is broken but i cant quite work out why. If the minSlice argument is small, it seems to kill the displacements. Can you poke around in this function?
(HDRP may have slightly different code for the height queries, it would be good to check and unify as part of this later)

Fixed-ish...
The order of the PosToSliceIndices parameters have changed. minSlice is going to the wrong parameter but fixing it made it worse. Except after the next fix...
The built-in version of that function uses _LD_Pos_Scale[0].z. By passing that through from the compute functions and using it for radius0 instead of minScale the height queries seemed to work again.
Also, that debug setup completely freaked Unity on MacOS. Hard crash. Even with the eGPU running.

Ok hopefully we're circling in on our metal issues at the same time then!
Could you get this pushed to a branch/pr, i think i need to go through the changes you mentioned above.

Sure. I sent the branch to you on Discord

This has been fixed in 4.1. Please reopen if you still experience issues.