Material Expansion Failure

Some materials in their linearExpansionPercent, for example Zr, do not provide any output beyond the temperature range. The checkTempRange verifies if the temperature is in the correct range and provides a warning if the temperature is not in the correct range. After that warning, the function should continue to provide an extrapolated value. Right now, it returns None which results in a very confusing stack trace.
For example:

In other words, in Zr.py, there should be an "else" in this chunk of code. This should be checked for similar errors in all other materials as well.

@ntouran @jakehader @keckler
Funny thing here, if you pass in a value of, say, 290K or 1809K, this method just returns None.  Oh, it raises a Warning, but the code will continue on anyway. And the developer on the other end better have put in a catch for that None.
A better programming practice here would be to raise an Error and fail the run. That way you don't have invalid non-scientific garbage in a simulation that runs for days on the cluster.  It keeps the results always "correct" and "valid", which I like.
Alternatively:

We could return a "no expansion" for in input temp below the lower limit (which seem correct).
And/or/maybe... fail for input temp above the upper limit.

Any opinions before I start coding?

@john-science One additional thought, if you're going to kick out with an error when you're outside the range, can you please make there be a way to ask what the allowed temperature range is? For example, if I'm building a tabular list of CTE vs temperature (for use with Abaqus say), right now there is no way to know what is the allowable range of inputs to this function without an error. The 293 and 1800 are hard coded directly in the method.

I think @alexhjames 's last point here highlights an important drawback of our materials in the framework -- that is, they are pretty ad-hoc. There is some structure, and generally all the material classes follow the same patterns, but right now any given material class could be in any state of completeness and their methods could return basically anything.
It would be nice if we pinned these down a little more.
I know that we generally use other materials for our own internal work, but the public material classes are somewhat of a mess right now.

mat_props has been modified to prevent any output beyond the set maximum and minimum temperatures.  It will throw an error.  You will be able to query the limits with "get_min_degrees_C()" and "get_max_degrees_C()".  Interestingly, the mat_props data (Zr.yaml) from the TP Materials Handbook uses a different reference (V. Petukhov, "Thermal expansion of zirconium in the solid phase," High Temperatures - High Pressures, vol. 35/36, pp. 15-23, 2003/2004.) and has different limits: 20C to 873C.)  Are we using Kelvin in ARMI and Celcius in mat_props?  Not sure if this is much of a problem.

Yeah, I think providing better access to important materials data from the class would be better.
Maybe we could have a discussion and come up with a bit of a list of things we'd like to see in the ARMI materials library?  This one point definitely stands as a good one, but if I'm going to make design changes to the materials tooling, it would be better to have a full set of goals, rather than just one idea.

L_hot / L_cold = e^(alpha1dT1) * e^(alpha2dT2)

Just to be clear, are alpha1, dT1, alpha1, and  dT1 all scalars/floats?  They're not 2D arrays or matrices, right?

Correct! All scalars. The two quantities alpha1 * dT1 and alpha2 * dT2 are essentially already calculated in the function currently as dLLhot and dLLcold except they should be divided by 100 to convert from percent to a decimal.

Actually, it looks like this problem is already solved.  The linearExpansionPercent() methods all call Material.checkTempRange():
https://github.com/terrapower/armi/blob/d04e1be8937c44e506dfcd5ff25459fca24af6cf/armi/materials/material.py#L544
But we have set ARMI so that, by default, we only print out a warning when someone ask for a value outside the valid temperature range. If we set this to True, the code would immediately fail and you would get a clear error message:
https://github.com/terrapower/armi/blob/d04e1be8937c44e506dfcd5ff25459fca24af6cf/armi/materials/material.py#L40
It seems people have come before us on this one, and decided the opposite of what we want.
It's certainly an easy fix on my end. Perhaps I would want some help testing to see if this kills anyone's simulations, or if this would ruin anyone's workflow.

I think there are at least two things that need to be done here:

Enforce FAIL_ON_RANGE = True, and enforce that all ARMI materials have a checkTempRange() in the linear expansion and density methods.
Find some way to expose the min/max temperatures for materials linear expansion, density, thermal conductivity, etcetera curves.
Have a bigger discussion about how the Materials library in ARMI could be improved.

As far as (1) above... I switched FAIL_ON_RANGE to True by default, and that fixes that bug.  ARMI Materials are a lot more consistent. But suddenly, I've found some lacking valid data ranges in ARMI materials.  In particular, even our ARMI unit tests no longer run, because the test reactor requires materials slightly outside our current "valid temperature" bounds.
Where can I get references to fill in these data holes?

Sodium - current lower bound for density is 97.85 C, but I need it down to 20 C.  (Note, Sodium is a solid at 97.72 C.)
B4C - current upper bound for linear expansion is 500 C, but I need it up to about 600 C.
Uranium Oxide - current lower bound for thermal conductivity is 300 C, but it needs to be 293.15.

I have opened a PR here to solve number (2) in this PR:
https://github.com/terrapower/armi/pull/762

I am going to close this issue, as I believe the original problem of not being able to determine the min/max temperature bounds for temp-dependent curves has been solved.
To address the broader concerns about consistency and predictability in ARMI materials, we have opened a Discussion here:
https://github.com/terrapower/armi/discussions/767
I'm hoping to get some good, actionable ideas out of that discussion.


Investigate null queueSize for gen-thumbs, contrary to comment

// This is actually sort of a lie, 8ecause we aren't doing synchronicity.
// (We pass queueSize = 1 to queue().) 8ut we still use progressPromiseAll,
// 'cuz the progress indic8tor is very cool and good.

progressPromiesAll(msg, queue(
  entries.toGenerate.map(...)
  /* No queueSize: 1 here!? */
));

The lie is a lie!!

Wait, no this isn't quite right.


No Log output after neoDeploy failed

With the following Jenkinsfile wants to deploy the artifact to NEO, it fails and cannot write any logs.
@Library('piper-lib-os') _

node() {

  stage('prepare') {
      deleteDir()
      checkout scm
      setupCommonPipelineEnvironment script:this
  }

  stage('build') {
      mtaBuild script: this
  }

  stage('neoDeploy') {
      neoDeploy script: this
  }

}

ContainerConfig: [name:container-exec]
[Pipeline] container
[Pipeline] {
[Pipeline] echo
Unstash content: workspace-7ec3cfa4-6027-40b9-b452-2b092114de28
[Pipeline] unstash
[Pipeline] echo
[INFO][dockerExecute] Executing inside a Kubernetes Pod
[Pipeline] lock
Trying to acquire lock on [neoDeploy :eu1.hana.ondemand.com/rptiq8vcps]
Lock acquired on [neoDeploy :eu1.hana.ondemand.com/rptiq8vcps]
[Pipeline] {
[Pipeline] sh
+ mkdir -p logs/neo
[Pipeline] pwd
[Pipeline] withEnv
[Pipeline] {
[Pipeline] fileExists
[Pipeline] error
[Pipeline] }
[Pipeline] // withEnv
[Pipeline] echo
Error while deploying to SAP Cloud Platform. Here are the neo.sh logs:
[Pipeline] sh
+ cat 'logs/neo/*'
cat: can't open 'logs/neo/*': No such file or directory
[Pipeline] }
Lock released on resource [neoDeploy :eu1.hana.ondemand.com/rptiq8vcps]
[Pipeline] // lock
[Pipeline] sh
[Pipeline] stash
Stashed 61 file(s)
[Pipeline] }
[Pipeline] // container
[Pipeline] }
[Pipeline] // node
[Pipeline] }
[Pipeline] // podTemplate
[Pipeline] unstash
[Pipeline] libraryResource
[Pipeline] echo
----------------------------------------------------------
--- An error occurred in the library step: dockerExecuteOnKubernetes
----------------------------------------------------------

The following parameters were available to the step:
***
[script:WorkflowScript@1c3856a5, dockerImage:s4sdk/docker-neo-cli, dockerPullImage:true, stashContent:[workspace-7ec3cfa4-6027-40b9-b452-2b092114de28]]
***

The error was:
***
hudson.AbortException: script returned exit code 1
***

Further information:
* Documentation of library step dockerExecuteOnKubernetes: https://sap.github.io/jenkins-library/steps/dockerExecuteOnKubernetes/
* Source code of library step dockerExecuteOnKubernetes: https://github.com/SAP/jenkins-library/blob/master/vars/dockerExecuteOnKubernetes.groovy
* Library documentation: https://sap.github.io/jenkins-library/
* Library repository: https://github.com/SAP/jenkins-library/

I refer to commit 78400cad9bdce2b1b638726eee16ee9e03202c9f in the piper-lib.
Tried to dive into that issue:
Findings up to now:
1.) we see the sh command for mkdir -p logs/neo. A corresponding folder does not exist in the workspace. Maybe we are in another workdir when creating the folder -> further investigation needed.
2.) we see the pwd statement. This corresponds to vars/neoDeploy#112
3.) we see the the withEnv. This corresponds to vars/neoDeploy#1124.) we see thefileExists. This corresponds to src/com/sap/piper/StepAssertions#10. 5.) we see error. This corresponds to src/com/sap/piper/StepAssertions#11`
This triggers an hudson.AbortException. This exception gets swallowed later in the surrounding exception handler vars/neoDeploy.groovy#142 when another excepetion gets raised by the failing catstatement inside the sh.
Next steps:
1.) Try to understand why the source cannot be found. Should be set earlier by mtaBuild.
2.) Rework the exception handler so that the original exception is not swallowed.
3.) Try to understand why there is no logs folder after mkdir -p
@citoki : In order to circumvent the issue we should provide the deployable via signature on neoDeploy: neoDeploy ./. source:  for now and check if that resolves the issue.
By the way: I tried to reproduce the issue here, but on my side the logs folder is present and neo gets called ...

@citoki
Can you please check what happens if you use the version provided in branch pr/doNotSwallowOriginalException in repo https://github.com/marcusholl/jenkins-library?
This will not solve the issue, but currently we catch an exception, try to provide the logs, receive another exception while doing this and forget the first one. This should be fixed with the version above.

@marcusholl
With the new PR I could see an extended error in Jenkins console.
The error was:
***
hudson.AbortException: File my-very-own-build-artifact.mtar cannot be found.
***

yes, I already saw this. Providing additional commits now with branch troubleshoot/neoDeploy in fork marcusholl/jenkins-library in order to understand the issue.

It is an issue with stashing.
ls after unstash reviles there is no mtar file:
[INFO] Unstashed "[workspace-f423697c-87de-4a71-94d8-d12c38aa9b3a]".
[Pipeline] sh
+ pwd
/home/jenkins/workspace/ntain_pipeline_teched2018-sample
+ whoami
root
+ ls -la
total 368
drwxr-xr-x    5 1000     1000           320 Apr  2 10:40 .
drwxr-xr-x    4 1000     1000           100 Apr  2 10:40 ..
-rw-r--r--    1 1000     1000           325 Apr  2 10:37 .editorconfig
-rw-r--r--    1 1000     1000            12 Apr  2 10:37 .eslintignore
-rw-r--r--    1 1000     1000          2888 Apr  2 10:37 .eslintrc
-rw-r--r--    1 1000     1000         14691 Apr  2 10:37 .eslintrc.ext
-rw-r--r--    1 1000     1000           114 Apr  2 10:37 .npmrc
drwxr-xr-x    2 1000     1000            60 Apr  2 10:40 .pipeline
drwxr-xr-x    2 1000     1000            60 Apr  2 10:40 .vscode
-rwxr-xr-x    1 1000     1000           317 Apr  2 10:37 Gruntfile.js
-rwxr-xr-x    1 1000     1000          1633 Apr  2 10:37 README.md
-rwxr-xr-x    1 1000     1000          3372 Apr  2 10:37 add-pp-xmake-props.sh
-rw-r--r--    1 1000     1000           653 Apr  2 10:37 mta.yaml
-rw-r--r--    1 1000     1000        323500 Apr  2 10:37 package-lock.json
-rwxr-xr-x    1 1000     1000          1487 Apr  2 10:37 package.json
drwxr-xr-x    8 1000     1000           220 Apr  2 10:40 webapp

Fixed with #632 in SAP/jenkins-Library. Please switch back to master branch and come back if it does not work. Otherwise close this issue, please.


Implement MIPS III doubleword memory instructions

MIPS III introduced a lot of doubleword instructions which operate with 64 bit extended registers. They are rarely used as most of MIPS HW is 32-bit, but we might be interested to keep them as well

ld
ldl
ldr
lwu
sd
sdl
sdr

@inedostoev Any progress?

I and @TimofeevAlex would like to do it in the coming days.

@inedostoev Igor reported you faced some issue, could you please explain?

Yes, I try. As I understend, in MIPS III introduce doubleword instructions, and I thought about it realization. On wikipedia I read, for example realization of instruction ld for 32-bit register, I should download from memory this doubleword, and write this word in 2 registers, first 4 byte in first register and next 4 byte in next register. But this is strange. So, now I have a question. Should I do that I describe before, or I should change in class RF all 32-bit register to 64-bit?

Should I do that I describe before, or I should change in class RF all 32-bit register to 64-bit?

I think the only way is to have 64-bit registers. I suggest to have a size field to RF operations, so 32-bit operations will update only low parts, 64-bit parts update 64 bits, and division/multiplication updates two 64-bit registers. As a result, get_dst_value will return 128 bits — please use boost types I placed to infra directory.

I change variable v_dst, now I have uint128 v_dst, but what NO_VAL i should use for the first init?

Please invent one :-) Try to value different to other NO_VALs.

When I change v_dst from 64-bit to 128-bit, I got new problems. This problem with perf_sim. This is a log
core/perf_sim.cpp:364:16:   required from here
core/perf_sim.cpp:296:5: error: no matching function for call to ‘WritePort<long unsigned int>::write(uint128, Cycle&)’
     wp_execute_2_execute_bypass->write( instr.get_v_dst(), cycle);
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from core/perf_sim.h:15:0,
                 from core/perf_sim.cpp:6:
./infra/ports/ports.h:246:24: note: candidate: void WritePort<T>::write(const T&, Cycle) [with T = long unsigned int]
 template<class T> void WritePort<T>::write( const T& what, Cycle cycle)
                        ^~~~~~~~~~~~
./infra/ports/ports.h:246:24: note:   no known conversion for argument 1 from ‘uint128 {aka boost::multiprecision::number<boost::multiprecision::backends::cpp_int_backend<128, 128, (boost::multiprecision::cpp_integer_type)0, (boost::multiprecision::cpp_int_check_type)0, void> >}’ to ‘const long unsigned int&’
core/perf_sim.cpp: In instantiation of ‘void PerfSim<ISA>::clock_memory(Cycle) [with ISA = MIPS]’:

As I understang, it is problem with converting 128-bit to 64-bit, because get_v_dst() now return 128-bit . But should all variables be 64-bit in this module? Or I should change types to uint128 too? (change for wp_execute_2_execute_bypass , etc.)

Yep, 128-bit should poison everything. You may consult @denislos for data bypassing unit internals.

Now I have problems with unit test, with func_sim.
func_sim/t/unit_test.cpp:41: Failure
Death test: {{ mips.run_no_limit( valid_elf_file); } ::exit(0); }
    Result: died but not with expected exit code:
            Terminated by signal 6 (core dumped)
Actual msg:
[  DEATH   ] test.out: infra/memory/memory.cpp:93: void FuncMemory::write(uint64, Addr, uint32): Assertion `addr != 0' failed.
[  DEATH   ]

I understand, that in function write() some instr put NULL address, but I cannot understand,what is instr it could be.
Can you tell me in which direction to think?
I can attach pull request.

Run the trace in functional simulator in verbose mode (./mipt-mips -b /path/to/your/trace -f -d)

Ok, thank you, I will try

Good day. I finally figured out with FuncSim test. Now I passed it. But I failed another test, namely PerfSim.The log below.
[==========] Running 5 tests from 2 test cases.
[----------] Global test environment set-up.
[----------] 3 tests from Perf_Sim_init
[ RUN      ] Perf_Sim_init.Process_Correct_Args_Of_Constr
/home/inedostoev/mipt-mips/simulator/core/t/unit_test.cpp:21: Failure
Death test: {{ PerfSim<MIPS> mips( false); } ::exit(0); }
    Result: died but not with expected exit code:
            Exited with exit status 1
Actual msg:
[  DEATH   ] No ReadPorts for MEMORY_2_EXECUTE_BYPASS key
[  DEATH   ]
[  FAILED  ] Perf_Sim_init.Process_Correct_Args_Of_Constr (3 ms)
[ RUN      ] Perf_Sim_init.Make_A_Step
/home/inedostoev/mipt-mips/simulator/core/t/unit_test.cpp:27: Failure
Death test: {{ PerfSim<MIPS>( false).run( valid_elf_file, 1); } ::exit(0); }
    Result: died but not with expected exit code:
            Exited with exit status 1
Actual msg:
[  DEATH   ] No ReadPorts for MEMORY_2_EXECUTE_BYPASS key
[  DEATH   ]
[  FAILED  ] Perf_Sim_init.Make_A_Step (3 ms)
[ RUN      ] Perf_Sim_init.Process_Wrong_Args
/home/inedostoev/mipt-mips/simulator/core/t/unit_test.cpp:34: Failure
Death test: PerfSim<MIPS>( false).run( "./1234567890/qwertyuop", 1)
    Result: died but not with expected error.
  Expected: ERROR.*
Actual msg:
[  DEATH   ] No ReadPorts for MEMORY_2_EXECUTE_BYPASS key
[  DEATH   ]
[  FAILED  ] Perf_Sim_init.Process_Wrong_Args (5 ms)
[----------] 3 tests from Perf_Sim_init (11 ms total)

[----------] 2 tests from Perf_Sim
[ RUN      ] Perf_Sim.Run_Full_Trace

Problem with ports, but I change all to uint128. What could be the problem?

No ReadPorts for MEMORY_2_EXECUTE_BYPASS key

Double-check type of wp_memory_2_execute_bypass — the write port that uses the key.

Sorry, my bad
rps_stages_2_execute_sources_bypass — this should be checked

Do you guys have some tests for behavior of these instructions? I just realized that our beloved SPIM's tt.core.s is 32-bit only.

Of course, we add simple tests


More Sample examples for this module

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Is your feature request related to a problem? Please describe.
I recently used this project for accessing Google CAS. The examples given are limited. While working I developed the understanding and I created small simple code snippets.
Describe the solution you'd like
A small code snippet for every other method available from this library.
Describe alternatives you've considered
NA
Additional context
I have got sample code, can I add them here?

Thanks for adding, @mahendrabagul! Our samples have a series of requirements in internal docs that might be hard to consolidate, plus we generally ask the API owners to maintain any additional samples. However, if you want to add your samples to this issue tracker, I'll make the API owners aware that these are requested!

Thanks for adding, @mahendrabagul! Our samples have a series of requirements in internal docs that might be hard to consolidate, plus we generally ask the API owners to maintain any additional samples. However, if you want to add your samples to this issue tracker, I'll make the API owners aware that these are requested!


[$250] Android - The app does not display the pending state

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.3-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4682953&group_by=cases:section_id&group_id=309132&group_order=asc
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Preconditions:

Login with<EMAIL_ADDRESS>Navigate to Account settings > Wallet
Make sure that under Assigned Cards the "Discover" card is assigned under the applause.expensifail domain
Steps:
Navigate to the workspace chat
Smartscan a receipt that will scan successfully to the workspace

Expected Result:
The app should display the pending state
Actual Result:
The app does not display the pending state
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01dcfd798465d34857
Upwork Job ID:<PHONE_NUMBER>680192305
Last Price Increase: 2024-07-04

Issue OwnerCurrent Issue Owner: @ahmedGaber93

@garrettmknight  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

@kevinksullivan / @grgia should this be done already or is it expected with where we are in the feeds project?

Asking in slack

Looks like it's part of https://github.com/Expensify/Expensify/issues/372206

Waiting on proposals

Waiting on proposals

Waiting on proposals

Issue not reproducible during KI retests. (First week)

Closing since it was solved here: https://github.com/Expensify/Expensify/issues/372206


folder_branch_ops: SyncFromServer should wait for md flush calls

This changes the onMDFlush code to make sure that we handle all
archive events before shutting down in tests.

lgtm after comment


Resolve pandoc version warning

Fix #1795
Rebuild requirements.txt to get the latest version of nbconvert, but pin sphinx to avoid #1910
Build docs locally, and they look ok.

It seems the only material change here is pinning sphinx

I'd say the real change is upgrading nbconvert. Pinning sphinx just maintains the status quo there, but we need to be explicit to keep the status quo.


Is io_uring funcs can be call in other thread?

Hello
I'm trying to use IO_uring and not used to it
I have tried all the echo severs of every open sources.
So I know the basic uses.
This is my problem!
This is what I have tried and doesn't work!!
= what I have done =
I have a cqe loop thread. I call it "CompleteIOThread".
And have another thread. Call it "SubmitThread".
I will wait some cqes in my CompleteIOThread by using 'io_uring_wait_cqe' or some kind.
And after cqe comes, and thread will start.
I will solve with cqe (use 'io_uring_cqe_seen') and make another sqe but not in this thread.
I will toss some cqe result to SubmitThread and  SubmitThread will do some logics and then use sqe.
something like this (code in SubmitThread)
"
struct io_uring_sqe *sqe = io_uring_get_sqe(_ring);
if (sqe == nullptr) {
cerr << "Warning Fail\n";
} else {
io_uring_prep_read(sqe, _fd, _buffer, _bufferSize, 0);
io_uring_submit(_ring);
}
"
2 thread has same ring.
And Submit thread can be scale out!
or Opersite(CompleteIOThread can be scale out)
Is it not posible to use 'io_uring func' in different thread??
Because it keep returning nullptr in sqe and I have no idea to look!

I'm checking if 'io-uring func' is thread safe

I'm checking if 'io-uring func' is thread safe

It isn't


Maybe wrap functionalities from http://www.fetk.org/

Here is another FEM library, which might bring something to wrap.

Here is a very comprehensive list of FEM software:
https://github.com/king110/awesome-fem

http://libmesh.github.io
This is also a very interesting package. I inspected source code a bit, it seems to be well structured and has a lot of nice stuff.

Hey, I start following the project. I'm developing a FEM software to use in electromagnetics problems.
Anyway, something that could help is this book: http://bookstore.siam.org/ot97/
Understanding and Implementing the Finite Element Method by Mark S. Gockenbach.
The author supplied all code (Matlab) used in the book here: http://www.math.mtu.edu/~msgocken/fembook/index.html

Welcome @marcosmarxm all contributions are welcome. Pure Julia and new implementations we are publishing under JuliaFEM.jl package. For wrapping of an existing packages we have made new repositories under the JuliaFEM organisation (only fully working is this small package JuliaFEM/MiniBall.jl)
If you familiar with Jupyter notebooks, please make your initial pull request here JuliaFEM.jl/tree/master/notebooks. The notebooks are very good platform for prototyping (when using Cxx.jl some errors are output to the terminal window where the notebook server is started).

@marcosmarxm , nice to hear that you are interested about this project. I would be nice to have gmsh mesh reader implemented to JuliaFEM because it's very common util to create mesh files for open source projects. Also, if you are interested to contribute electromagnetic solver to JuliaFEM you could start your own notebook describing core ideas what are you solving, how are you solving and what kind of basis functions is needed in electromagnetism. We could then figure out together how to naturally extend code to solve EM problems. It's also good idea to check my notebook "elasticity solver example" where I'm playing with different ideas while searching a good way to put all together.

If we want to wrap things from FETK, we need a separate project for that, maybe FETK.jl


steamcmd/steam-run - bwrap: Can't chdir to ...:  No such file or directory

Describe the bug
Steamcmd has stopped working, and instantly quits with the error in the title. I haven't changed anything in the module, so updating nixpkgs must've been the cause.
I use it here: https://github.com/diniamo/niqs/blob/7b9af8e855f6664cb1b8ba5a3c09140776a3658f/modules/server/satisfactory.nix#L46-L52
Steps To Reproduce
Steps to reproduce the behavior:

Set up a similar service to what I have linked above
Run it
See the error

Expected behavior
Steamcmd works like before.
Notify maintainers
@K900 - Not sure, but it looks like you have commits that could be relevant (both on steamcmd and steam-run)
@tadfisher - steamcmd maintainer
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.10.14, NixOS, 24.11 (Vicuna), 24.11.20241023.68783af`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Lix, like Nix) 2.92.0-devpre20240914_7272582
System type: x86_64-linux
Additional system types: i686-linux, x86_64-v1-linux, x86_64-v2-linux, x86_64-v3-linux
Features: gc, signed-caches
System configuration file: /etc/nix/nix.conf
User configuration files: /home/diniamo/.config/nix/nix.conf:/etc/xdg/nix/nix.conf:/home/diniamo/.nix-profile/etc/xdg/nix/nix.conf:/nix/profile/etc/xdg/nix/nix.conf:/home/diniamo/.local/state/nix/profile/etc/xdg/nix/nix.conf:/etc/profiles/per-user/diniamo/etc/xdg/nix/nix.conf:/nix/var/nix/profiles/default/etc/xdg/nix/nix.conf:/run/current-system/sw/etc/xdg/nix/nix.conf
Store directory: /nix/store
State directory: /nix/var/nix
Data directory: /nix/store/wm6c3bsrcsp7snbnp2kwq718j1fniwis-lix-patched-2.92.0-devpre20240914_7272582/share`
 - nixpkgs: `/nix/store/min6i7ikkf4dhgppbpd2gpd1a8279rj9-source`

Add a :+1: reaction to issues you find important.

Please post the full error log.

This is all I have:
Oct 24 19:51:51 diniamo-SERVER systemd[1]: Starting Satisfactory dedicated server...
Oct 24 19:51:51 diniamo-SERVER satisfactory-pre-start[1611]: bwrap: Can't chdir to /var/lib/satisfactory: No such file or directory
Oct 24 19:51:51 diniamo-SERVER systemd[1]: satisfactory.service: Control process exited, code=exited, status=1/FAILURE
Oct 24 19:51:51 diniamo-SERVER systemd[1]: satisfactory.service: Failed with result 'exit-code'.
Oct 24 19:51:51 diniamo-SERVER systemd[1]: Failed to start Satisfactory dedicated server.```

Could you link the PR?

Tested, closing.


Update OWNERS

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
Add Rita to OWNERS.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Release note:
NONE

cc @ritazh
/assign @justaugustus @andyzhangx

/lgtm
/approve


refactor: passing QueryContext to RegionServer

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
close https://github.com/GreptimeTeam/greptimedb/issues/3752
What's changed and what's your intention?
Allow passing the query context to the region server.
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Summarize your change (mandatory)
How does this PR work? Need a brief introduction for the changed logic (optional)
Describe clearly one logical change and avoid lazy messages (optional)
Describe any limitations of the current code (optional)

Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.
[x]  This PR does not require documentation updates.

Hi @Kelvinyu1117 , there are some conflicts need to be resolved.

Good job! Please resolve the conflicts, then we can merge the PR.  @Kelvinyu1117

Hi @Kelvinyu1117 , there are some conflicts need to be resolved.

Should be fixed.

Hi @Kelvinyu1117 , there are some conflicts need to be resolved.

Should be fixed.

Hi @Kelvinyu1117 , The coverage tests failed(test_do_auth_with_user_provider and test_do_auth_without_user_provider). We need to wrap the query_context with Arc in the do_auth method.


How to fix CFU Firmware Update Simulation build issues in Visual Studio 2022

Hi,
I’m encountering issues while migrating and building our CFU Firmware Update Simulation project from Visual Studio 2019 to Visual Studio 2022. Could you please help with the following?
Issues Encountered:

Linker Error: LNK1181: cannot open input file 'DmfUModules.Library.Tests.lib'
Compiler Warning: C4013: 'DMF_Utility_EventLogEntryWriteUserMode' undefined; assuming extern returning int
Missing/Incompatible Header: Cannot locate wdf.h in VS2022.

Steps Taken:
Installed Visual Studio 2022: With the "Desktop development with C++" workload.
Installed WDK and SDK: Installed the Windows Driver Kit (WDK) and Windows Software Development Kit (SDK) compatible with Visual Studio 2022.
Retargeted Solution: Retargeted the solution to the latest Windows SDK and toolset.
Updated Include and Library Paths:

Include Directories:
C:\Program Files (x86)\Windows Kits\10\Include\10.0.26100.0\umdf;C:\Program Files (x86)\Windows Kits\10\Include\10.0.26100.0\shared

Library Directories
C:\Program Files (x86)\Windows Kits\10\Lib\10.0.26100.0\um\x64

Could you please provide guidance on resolving the above issues? Additionally, is it possible to update the project repository on Git with the necessary changes if need.
Thank you for your support.
Kind Regards
Chandra Shekar C

I'm also experiencing the same problem. The DMF submodule in this repo doesn't seem to have been updated since 2019, which predate Visual Studio 2022 support. If I try to manually update the DMF submodule then I'm instead observing the following compiler errors caused by DMF functions removed in https://github.com/microsoft/DMF/pull/121:
>CFU\Host\ComponentFirmwareUpdateDriver\DmfInterface.c(278,9): error C4013: 'DMF_Utility_EventLogEntryWriteUserMode' undefined; assuming extern returning int
>Registry.c
>CFU\Host\ComponentFirmwareUpdateDriver\Registry.c(127,9): error C4013: 'DMF_Utility_EventLogEntryWriteUserMode' undefined; assuming extern returning int

ASK: Would it be possible for someone in Microsoft please update this repo, so that it's compatible with more recent DMF versions?


Added a cdn link to Tether, which is a new requirement for Bootstrap 4

More information here. Looks like Tether is needed for proper positioning of Bootstrap tooltips.

Thanks @EricZaporzan


Unable to get pub key

Hello,
when you have a fresh wallet and send funds to it, the pubkey is not yet recorded on chain.
This only happens on the first transaction signed from that wallet.
But this renders cosmes unusable in those cases as you cannot connect such a wallet. It errors with "unable to get pub key".
Wallets need to be able to interact when they have funds even when the pubkey is not yet available on chain. This works correctly with other libs.

Hey, which exact wallet are you using to sign that transaction?
The pubkey is resolved by first requesting the wallet to return the user's pubkey, failing which, then it'll fallback to querying the chain for the pubkey. Most wallets are able to return the user's pubkey, and only certain wallets like Station mobile app aren't able to. If this issues is due to Station mobile app, then we'd need to wait for the Station team to support requesting for the user's pubkey. Alternatively, if the chain does not have the user's pubkey, we could ask the user to sign an arbitrary message that cosmes will use to recover the pubkey.


Demo: ease responsive layout

Remove the css grid layout in favor of responsive Bootstrap classes to better manage the different viewport sizes

While reviewing, I noticed that the toc and the sidebar are a bit different (font-weight of the title, top padding, padding between title and elements).
Do you think we should align those two displays ?

Yes, I've done them independently. I'll have a look to see what it looks like if I change them.
By the way (and just for info), I didn't write it, but the purpose of this PR is not only to use the Bootstrap classes, but now we can have access to the mixin and variables. Which is useful in this case to get the Bootstrap breakpoints.


'overrideWithValue' doesn't influence providers watching the overridden provider

I'm using version 1.0.0-dev.2.
There are 2 providers: iProvider which provides an int, and wrapperProvider which provides something else based on the value from iProvider - in the sample below a simple object that wraps the int. iProvider returns 0 by default, but it is overridden (using overrideWithValue) near the top of the widget tree to return 17. Then, in the UI itself, 2 values are displayed: the i value returned by iProvider, and the wrapper value returned by wrapperProvider.
The two values differ: the value returned from iProvider returns the overridden value of 17, while the object returned by wrapperProvider contains the default value of 0.
The same behavior can be reproduced if overrideWithProvider instead of overrideWithValue is used.
It seems overriding a provider that other providers watch has no effect on these watching providers.
To Reproduce
import 'package:flutter/material.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return const ProviderScope(
      child: MaterialApp(
        home: Page(i: 17),
      ),
    );
  }
}

class Page extends StatelessWidget {
  final int i;

  const Page({required this.i});

  @override
  Widget build(BuildContext context) {
    return ProviderScope(
      overrides: [
        iProvider.overrideWithValue(i),
      ],
      child: Scaffold(
        body: Consumer(
          builder: (context, ref, _) {
            final i = ref.watch(iProvider);
            final wrapper = ref.watch(wrapperProvider);

            return Center(
              child: Text(
                'Good:  $i\n' // 17
                'Wrong: ${wrapper.i}', // 0
              ),
            );
          },
        ),
      ),
    );
  }
}

// Riverpod.

final iProvider = Provider<int>((ref) => 0);

@immutable
class Wrapper {
  final int i;

  const Wrapper(this.i);
}

final wrapperProvider = Provider<Wrapper>(
  (ref) {
    final i = ref.watch(iProvider);
    return Wrapper(i);
  },
);

Expected behavior
I expect the two values i from iProvider and wrapper.i from wrapperProvider to have the same value as wrapperProvider should get the overridden value for iProvider.

This is expected.
You should override the providers that depends on the overriden provider too

Ok, this is of limited help then, and clearly not what I had hoped for for my use case.

I was expecting this to be working as well :/

That's not feasible.
But the linter catches this. Enable it to avoid the mistake

hmm I have done like so with AutoDisposeFutureProviderFamily in my tests..
ProviderScope(
              overrides: [
                abc.overrideWith((ref) => MockABC()),
  // These need to be reinitialized because they depend on the above!

    xyz.overrideWithProvider(xyz.call),

Eg xyz relied on abc provider and its fine.
however trying this with AutoDisposeFutureProvider no worky
   xyz.overrideWithProvider(xyz.call),

overrideWithProvider deprecated and call undefined.
what am I doing wrong here?
I have to say it makes testing far harder having to override mocked out providers especially if not familiar with codebase and seen the requirement heh

ahh as long as my providers use dependencies: everything is fine testing wise, joy~


Modified English/Welsh language links

Moved the links from the title page to all pages (via layout.html). Refactored the css to ensure appropriate display on all pages. (Css added to lr-styleguide)

Mostly minor things, but as a minimum for the assessment we do need to hide the language translation links from all but the title pages.

Proper job :+1:


libclang on Mac OS X

On Mac OS X the libclang can be found with dylib extensions and not .so. So, I'm sending a little patch to fixed it.
fix_libclang_mac.patch.txt

Fixed by #82


Change "icons" for Categories?

I'm interested in changed the default graphics for the "icons" on the Roku side for the categories. I followed the route listed below.
Since it's installed in Windows it seemed fairly straightforward to just go and look at the files themselves and find the graphics that looked like what I see on the Roku side. With that in mind, the files that I thought would be the ones to change are found here: C:\Program Files (x86)\Plex\Plex Media Server\Resources\Graphics
The file names are movie-fanart.jpg and show-fanart.jpg. At least those are the two that I'm concerned about. I have searched the rest of the folders for PMS and I don't see any other graphics and certainly not any that are what I see on the Roku. I have also removed and re-added my PMS to the Roku and that didn't change anything. I've rebooted the Roku with no luck. I have three of them running and I've only checked one so far.
I'm not sure I would call this a bug or issue but it would a "nice to have". No offense to the developer but I'm not a fan of the existing graphics. I love it otherwise.
Since the channel is added on the Roku side via the web I'm guessing that these graphics aren't coming from the PMS. It may be a coincidence that I was able to find the same graphics on the PMS.

Resolved.  The files are found at the location mentioned initially.  They are cached for the Roku however.  I located the cached versions and deleted them.  Rebooted the Roku and my versions are there now.


DefaultTagHelperFactService does not handle opt-out TagHelpers correctly.

If we have a the following tag: <!a href="...">Hello</!a>. You can't query DefaultTagHelperFactService for !a; you need to query it for just the a. If you happen to have a TagHelper that can match !a (a catch-all) then it will apply.

0005cb93ac8f59bde12cb3aff36ae6e9873d59be


Fix the issue #561

The issue was about applying on the button class loading together with tooltip. The fix was to set the opacity and background on loading class to it overrides the settings from tooltip.

Sorry, I created the pull request form the wrong branch


showing/hiding series bar rect on y axis not configurable via property

Issue: showing/hiding series bar rect on y axis not configurable via property (https://github.com/PredixDev/px-vis-xy-chart/issues/11)
To entirely fix this, there are two required changes
one is px-vis-axis.html that is part of px-vis repo
Here is pull request for that change
https://github.com/PredixDev/px-vis/pull/69
second one is, here, i.e. exposing properties. In our case, we are going to have only one series, and series bar may not be necessary, hence having a config property should be good.
Hope this helps,
Sandeep Khandewale
(Consultant, GE Oil and Gas, Mumbai)

Hi, thanks but this is intentional. Rather than exposing every property on every component, it is easier to use a configuration object to configure the sub components. Please use the x-axis-config and y-axis-config objects.


Setting up workflow functions for setting up simulations

Reason for the PR
I think a lot of friction for getting simulations setup and running are because the force field parameter generation and assignment was setup to run in the examples directories, instead of directly in the simulation workflow. This PR is meant to put the simulation setup, running and analysis all in one place. Additionally, this PR is meant to take a lot of the hand tuning required to setup simulations.
Features

Based on specific parameter YML files parameterization can be handled by different workflows (whether we need to parameterize with OpenFF-2.0 or a Bespoke-Fit FF).
Build system with packmol in a signac workflow. These workflow functions should check for specific software they need to run, or they can be provided in the input files.
Move REMD simulations and MD simulations into a signac workflow that is in analysis_workflows

To Do's

[x] Add workflow function to build foldamer
[x] Add workflow function to solvate system with chloroform (and minimize). This will require the system to have packmol installed (or at least an input option that points to packmol or make it a subpackage)
[ ] Add workflow function to build topology file for any system
[ ] Add workflow function that sets up and submits REMD simulations
[ ] ...

I did some playing around with getting github actions to work on this PR, but I will post a problem for this and make a different PR


Localise all strings in-game so they can be translated.

Completed with https://github.com/ynohtna92/SlideNinjaSlide/commit/cf4b547ad85273102107016cb9887e8250cff0fd


[Tests-Only]Update Expected failures

Description:
This PR adjusts the expected failures files according to the changes in this PR: https://github.com/owncloud/core/pull/39096
Related Issues:

https://github.com/owncloud/ocis/issues/2354

Note: this is currently a draft PR. It points to a different PR branch in core.

@ishank011 this is ready to merge.
Then @jasson99 can sort out the round of test changes in #2003


[Bug]: tsc passes outside of Bazel but fails with rules_ts

What happened?
I have a repro here: https://github.com/UebelAndre/rules_ts_example
The README describes a case where running tsc on some code outside of bazel succeeds but fails when using rules_ts.
~/Code/rules_ts_example (main ✗) bazel build //...
INFO: Analyzed 3 targets (1 packages loaded, 50 targets configured).
INFO: Found 3 targets...
ERROR: /Users/user/Code/rules_ts_example/foo/BUILD.bazel:11:11: Compiling TypeScript project @//foo:foo_ts [tsc -p foo/tsconfig.json] failed: (Exit 1): tsc.sh failed: error executing command (from target //foo:foo_ts)
  (cd /private/var/tmp/_bazel_user/729471d7b159995c3ffbf36ad2691a5e/sandbox/darwin-sandbox/300/execroot/rules_ts_example && \
  exec env - \
    BAZEL_BINDIR=bazel-out/darwin_arm64-fastbuild/bin \
  bazel-out/darwin_arm64-opt-exec-2B5CBBC6/bin/external/npm_typescript/tsc.sh --project foo/tsconfig.json --outDir foo --rootDir foo --declarationDir foo --tsBuildInfoFile foo/foo_ts.tsbuildinfo)
# Configuration: 0ba77a61754b8ccac68b4c11211b3ebeca24c2f07acb9bc57df3ce73dcecadb5
# Execution platform: @local_config_platform//:host

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
foo/src/index.tsx(40,10): error TS2786: 'Head' cannot be used as a JSX component.
  Its instance type 'Head' is not a valid JSX element.
    Type 'Head' is missing the following properties from type 'ElementClass': setState, forceUpdate, props, state, refs
foo/src/index.tsx(50,18): error TS2339: Property 'props' does not exist on type 'MyDocument'.
foo/src/index.tsx(55,12): error TS2786: 'NextScript' cannot be used as a JSX component.
  Its instance type 'NextScript' is not a valid JSX element.
    Type 'NextScript' is missing the following properties from type 'ElementClass': setState, forceUpdate, props, state, refs
INFO: Elapsed time: 3.645s, Critical Path: 3.19s
INFO: 2 processes: 2 internal.
FAILED: Build did NOT complete successfully

Version
Development (host) and target OS/architectures:
Output of bazel --version: bazel 6.1.0
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:
rules_js-1.24.1
rules_ts-1.3.2
Language(s) and/or frameworks involved:
How to reproduce
See https://github.com/UebelAndre/rules_jest_example

Any other information?
No response

This seems very similar to https://github.com/vercel/next.js/discussions/17205#discussioncomment-4927294 and https://github.com/vercel/next.js/issues/40835? Do any of the suggestions there work?

Actually this is the same issue as https://github.com/aspect-build/rules_ts/issues/344 and with strict dependencies will occur outside bazel as well.
next (and maybe @next/font?) both declare @types/react as a dev-dependency so it's  ignored by pnpm and rules_js from your project. With strict deps in rules_js or with pnpm hoist=false @types/react will not be accessible to next when typescript is type-checking. Your MyDocument extends the next Document which extends React.Component, but since next doesn't depend on @types/react that React.Component is essentially undefined. So your MyDocument is not a react component from typescripts perspect with strict dependencies...
You should be able to workaround this by correcting the dependency via hoisting or pnpm package extensions.

@jbedard sorry but could you suggest a diff to https://github.com/UebelAndre/rules_ts_example that would fix the issue? I've tried using package extensions and it felt like shooting in the dark 😞

Do the docs changes in https://github.com/aspect-build/rules_ts/pull/359 help at all? Have any suggestions? I'll also try getting a quick diff that you can checkout...

This in your package.json seems to do it:
diff --git a/package.json b/package.json
index bc89f32..3441cfe 100644
--- a/package.json
+++ b/package.json
@@ -9,5 +9,14 @@
   "version": "1.0.0",
   "devDependencies": {
     "typescript": "4.9.4"
+  },
+  "pnpm": {
+    "packageExtensions": {
+      "next": {
+        "dependencies": {
+          "@types/react": "*"
+        }
+      }
+    }
   }
 }

This way the strict (no hoisting) node_modules makes @types/react accessible to next which imports+exposes react (while only declaring it as a dev-dependency which does nothing when consuming next).

Do the docs changes in #359 help at all? Have any suggestions? I'll also try getting a quick diff that you can checkout...

Why suggest skipLibCheck? Does that weaken the validation of my code? Or is typescript running on transitive dependencies?

Actually this is the same issue as #344 and with strict dependencies will occur outside bazel as well.
next (and maybe @next/font?) both declare @types/react as a dev-dependency so it's ignored by pnpm and rules_js from your project. With strict deps in rules_js or with pnpm hoist=false @types/react will not be accessible to next when typescript is type-checking. Your MyDocument extends the next Document which extends React.Component, but since next doesn't depend on @types/react that React.Component is essentially undefined. So your MyDocument is not a react component from typescripts perspect with strict dependencies...
You should be able to workaround this by correcting the dependency via hoisting or pnpm package extensions.

@jbedard sorry, reading this again and I don't understand why @types/react wouldn't be accessible to next. Doesn't having it as a devDependency in my package make @types/react globally accessible?

Doesn't having it as a devDependency in my package make @types/react globally accessible?

It makes it accessible to your code, not your dependencies. We don't want adding a local dependency to have side-effects on other packages without explicitly declaring it.

Why suggest skipLibCheck? Does that weaken the validation of my code? Or is typescript running on transitive dependencies?

Both to avoid checking libraries with type issues like what you're seeing and performance, see https://www.typescriptlang.org/tsconfig#skipLibCheck


Feat: Implement the functions in the SwapHandler contract.

Description
Implement the swap function in the SwapHandler contract.
File: src/swap/swap_handler.cairo.
Function signature:
    fn swap(ref self: ContractState, params: SwapParams) -> (ContractAddress, u128)

Acceptance criteria

Function is implemented and well commented.
Associated unit tests (nominal and failure) are implemented and passing.

Resources

swap function in GMX repo

Assigned 🔥


Conflicts with vim-fugitive Gwrite

Description
When attempting to resolve merge conflicts using vim-fugitive's Gdiffsplit! in a 3-way split, calling Gwrite after the conflict has been resolved results in the following error:
fugitive: fatal: Unable to create '<path_to_project>/.git/index.lock': File exists.^@^@Another git process seems to be running in this repository, e.g.^@an editor opened by 'git commit'. Please make sure
all processes^@are terminated then try again. If it still fails, a git process^@may have crashed in this repository earlier:^@remove the file manually to continue.

So I started with a basic config and only fugitive installed and it worked properly. I assumed it must be conflicting with one of my plugins, so using ripgrep searched through my plugins for occurrences of git, and re-enabled these plugins until I found this one to be the culprit.
I can override this by using Gwrite! for the time being, although that's also not something I necessarily want ingrained in my muscle memory. Is there a setting or something I can use to resolve this temporarily? I looked through the vim docs for this plugin and didn't find any occurrences of "git"
System Info
git version 2.20.1

PopOS 19.10
Kernel - 5.3.0-7625-generic

NVIM v0.5.0-dev
Build type: RelWithDebInfo
Lua 5.1
Compilation: /usr/bin/cc -g -O2 -fdebug-prefix-map=/build/neovim-8zzuJi/neovim-0.5.0+ubuntu1+git202003172025-87d892a-00e710e=. -fstack-protector-strong -Wformat -Werror=format-security -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/build/neovim-8zzuJi/neovim-0.5.0+ubuntu1+git202003172025-87d892a-00e710e/build/config -I/build/neovim-8zzuJi/neovim-0.5.0+ubuntu1+git202003172025-87d892a-00e710e/src -I/build/neovim-8zzuJi/neovim-0.5.0+ubuntu1+git202003172025-87d892a-00e710e/.deps/usr/include -I/usr/include -I/build/neovim-8zzuJi/neovim-0.5.0+ubuntu1+git202003172025-87d892a-00e710e/build/src/nvim/auto -I/build/neovim-8zzuJi/neovim-0.5.0+ubuntu1+git202003172025-87d892a-00e710e/build/include
Compiled by buildd@lgw01-amd64-050

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Run :checkhealth for more info

This repo does not make any use of git. We check for the presence of fugitive (to prevent starting servers for fugitive buffers), and we have install scripts and README references to github projects, so the word "git" is definitely going to come up in ripgrep searches.

I have just tried to create a simple repo and generate a conflict, then followed your steps, but cannot reproduce - it worked fine

Ok thanks for doing that so quickly! Since opening the issue I've learned a little more about it, it doesn't seem to be consistent. I originally thought Gwrite! solved the issue but it just seems that Gwrite in general works sometimes and doesn't others. For now I'll close until I'm very sure it is this plugin conflicting. If it is, it may also be related to the size of the repo I'm in, or my specific settings. Thanks again!!

OK, report back when you work it out!


Take advantage of multistage build

See https://blog.kubesimplify.com/multi-stage-docker-build#heading-the-lab-rat or https://github.com/alextanhongpin/go-docker-multi-stage-build?tab=readme-ov-file#method-3-using-multi-stage-build
The release image could be a lot smaller that way.

RUN go mod download
COPY . ./
RUN CGO_ENABLED=0 GOOS=linux go build -a -o ./dist/app ./cmd/app/.

# Deployment image
FROM alpine:latest AS production
COPY --from=golang /app/main .
COPY --from=node /internal/view/static /internal/view/static/

WORKDIR /app
RUN rm -rf /app/temp && \
mkdir /backups && \
chmod 777 /backups

# Run the app
EXPOSE 8085
CMD ["task", "migrate-serve"]

Hello @Jogai, thanks for your suggestion
If you look at the Dockerfile we use in production (https://github.com/eduardolat/pgbackweb/blob/main/docker/Dockerfile) you will notice that it includes the minimum dependencies needed to run the software on both amd64 and arm64 hardware.
The fact that the golang binary depends on system tools such as pg_dump or psql, among others, makes it more complicated to simply copy the binary into a clean docker image.
If you could modify the aforementioned Dockerfile and make a proposal I would appreciate it, take into account the README.md next to the Dockerfile.
Always keep in mind that it is better to sacrifice a little space if what we gain is easier maintenance of the project.

Thank you for answering. Like I said, I'm not familiar with go, so I cant bring it further. Its just that I think this is something this project could benefit from. I'm a happy user, so I'll keep using it, but maybe it keeps someone with less bandwidth from trying.

Always keep in mind that it is better to sacrifice a little space if what we gain is easier maintenance of the project.

Believe me, I know about tradeoffs, but I'm not advocating to make it less maintainable. If anything I think you can simplify it. But I also think you are wasting at least a GB, if not more, so we're not talking about 'a little space'.


azure-storage-explorer - Removed usages and references of @backstage/backend-common

Hey, I just made a Pull Request!
Removed usages and references of @backstage/backend-common
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hi @deepan10, can you give this one a quick look, please? 🙏

Hello @awanlin
All good with this change. 🚀  👍


Create an 'edit' popular links page

Trello card
This PR will allow us to edit existing popular links in publisher.
This includes:

Adding the sidebar on the show page which contains the 'Create new edition' button.
Adding the create new edition page, which includes the 'Edit popular links' button and 'Publish' button in the sidebar.
Adding the edit page, which includes the options to edit and save.
Adding the controllers and routes for create, edit, update and publish.
Adding validation to the models for invalid URLs
Integration, functional and model tests for create, edit, update and publish.
Adding publish_popular_links to the workflow and action model.

@anatron @syed-ali-tw Is there a design available for the new sidebar? I wonder as well if this is intended to be sticky rather than staying fixed at the top when the user scrolls down.

@anatron @syed-ali-tw Is there a design available for the new sidebar? I wonder as well if this is intended to be sticky rather than staying fixed at the top when the user scrolls down.

Yes, it is intended to be sticky. This will be part of another PR.

@anatron @syed-ali-tw Is there a design available for the new sidebar? I wonder as well if this is intended to be sticky rather than staying fixed at the top when the user scrolls down.

Yes, it is intended to be sticky. This will be part of another PR.

OK, though it should be a fairly simple CSS addition if you think it's worth doing it as part of this work.


Add screen names block

Description
Depends on #10
Adds a screen names block that can be passed to blocks like:

which require a screen name.
Important Note: Currently this does not include any upgraders because it's not in the right branch path. They can always be added later.
Testing

Created Screen1 and Screen2.
Added the following blocks to Screen1:

Connected to the companion and observed that the app correctly switched to Screen2.
Removed Screen2.
Observed that the block was marked as a bad block:

Added Screen2 again.
Observed that the block was no longer marked as a bad block.
Removed Screen2 again.
Selected Screen1 in dropdown.
Observed that the block was no longer marked as a bad block.

@BeksOmega Is this ready for another review?

@ewpatton Yep I think I got everything! fingers crossed

Ok, it seems to work correctly now when upgrading. The only issue is that if the user previously made use of opening Screen1, in spite of it being a bad but allowed practice, the block is now a bad block because Screen1 isn't listed. I think we should make Screen1 be included and separately deal with the opening of Screen1 as an issue independent of your GSOC project as that might get complicated. Can you restore the presence of Screen1 to the list? After that feel free to merge this.

Added caching. Reverted filtering, tested, and filed an issue :D


compilers, binutils: Add version and pname

Motivation for this change
Resurrected #67284.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@Ericson2314 Could you merge staging-next into staging. There's a merge conflict.

@FRidh sure

f191360ad05

Thanks!
On Mon, Nov 25, 2019 at 9:59 PM John Ericson<EMAIL_ADDRESS>wrote:

f191360ad05
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/74065?email_source=notifications&email_token=AAQHZ3YRZALCKHOMK2ZSK6LQVQ4CRA5CNFSM4JRALSOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFDY3DI#issuecomment-558337421,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQHZ37ZAHZE2PJLKQOPVXDQVQ4CRANCNFSM4JRALSOA
.


Use DIR relative include

This allows running reli from a different directory.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1900

Relevant Lines:
2342

💛  - Coveralls

I'm looking for a new sampling profiler (the one I wrote years ago is ... terrible) and this one seems to work pretty well :)


fix wrong handling for one variant union

resolve: https://github.com/Azure/typespec-azure/issues/1134

All changed packages have been documented.

:white_check_mark: @azure-tools/typespec-client-generator-core

Show changes
@azure-tools/typespec-client-generator-core - fix ✏

fix wrong handling for one variant union

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/1135/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/1135/


fix(android,v10): implement isGestureActive on MapChange, MapIdle events

Fixes: #2817

How to achieve onRegionDidChange using onCameraChanged?. How can we know user released his finger from screen?

@mfazekas  isGestureActive on MapChange still has an issue. It is false only in some times like 5%. It is always true even if stop moving the map. Please test it once again.

Tested in v10.0.6. still the same.


Player mobs

hey, I am making mobs with your plugin.
But I have a problem.
I need a player mob. but I would like me to be able to put a skin on the server and that the plugin uses this skin for the mob.
I've already tried it with the disguise function. But when I change the skin of my account, the skin of the mob changes too. And this is not the intention for my idea.
I hope you have a solution for this or that this function can be added as soon as possible.

This is not currently possible and I don't expect it will be possible soon. Mobs are config-driven, it would be difficult to add a way to specify an entire game profile in a config.
I'll keep this open as something to look into, but I would not expect it to happen soon.
Offhand, I could see adding a command that lets you dump a player's profile to the console as a base64-encoded string, then you could paste that into the mob configs, then magic could decode it, build an actual game profile and hand it to LibsDisguises. Sounds super complicated though.

Thats a good idea.

As it turns out I have supported this for a while without knowing it! You can put a serialized game profile (such as LibsDisguise's commands would save) into the config of a mob disguise as "skin".
I have also added a new command "/mage skin" to dump a player's current skin to chat so you can copy+paste it into a config.
There is now a built-in "ironman" mob that uses a serialized skin.


History chart on taxon pages should only show ten years of activity

See https://forum.inaturalist.org/t/zoom-or-filter-the-history-graph-on-taxon-pages/39972
For example, if you go to https://www.inaturalist.org/taxa/333796-Coccinellini and click on History you'll see this pretty inscrutable chart:

Probably a bug with recent D3 changes? We should show a ten year window that can be moved.

Hello,
I would like to be assigned on this task. I have started to work on it with a frontend "solution/workaround". You can find it here.
I don't know how it work before and if it was possible to zoom out to older date. I found an option zoom: { enabled: true } from the migration from c3 to billboard so I guessed we want the possibility to see the data before 10 years ago.
If we want to strictly set the data to the last 10 years, then I think it's better to filter the dates on the server side and give only the last 10 years dates to the client as suggested by pisum on the forum.
Depending of your answers I will change my PR.


Fix: Duplicated auth check

Hey,
it looks like the authentication part in server/server.go has been duplicated somehow.
Thanks for the cool project by the way 👍
Cheers,
Max

Awesome, thanks! I'm not sure how that got in there twice....


New version: InfiniteRed.Reactotron.Pre-release version <IP_ADDRESS>2

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

Hi @SpecterShell,
The InstallerUrls point to version 3.0.0-beta.34, not <IP_ADDRESS>2. Is this version mismatch normal for this developer?

Hi @SpecterShell,
The InstallerUrls point to version 3.0.0-beta.34, not <IP_ADDRESS>2. Is this version mismatch normal for this developer?

<IP_ADDRESS>2 is the version from the wix installer while 3.0.0-beta.34 is the version from the nsis installer. The latter has been covered in AppsAndFeaturesEntries.
The reason for using the former one is that last year they released a new version 3.0.0-alpha.8 (<IP_ADDRESS>7) over the previous one 3.0.0-beta.9 anti-alphabetically, and WinGet will think the latter is newer if both versions exist in the winget-pkgs repo. So the wix installer which has different but correct version names was adopted for the package version.


ActionView::Template::Error (couldn't find file 'semantic_ui/definitions/modules/accordion.less' with type 'text/css'):

I followed all the instructions in the read me file, but still unable to get up and running with usage of Semantic UI.

Could you please provide minimal project for reproduce problem?

It is actually the minimal project. I'm using Rails 4.2.1, ruby 2.1.2. Create a new project rails new <project_name>. First thing: I installed the gems that you asked, and ran the commands as described in read me file, and got this. Need more info, please let me know, I'd be glad to explain.

Checked. All works fine. Still need minimal project for reproduce your problem.

A similar problem in the process of running the application.
It was decided by manually adding gem 'less-rails'

I close it. Without minimal project is impossible to reproduce problem.

I'm having this issue, latest rails 4.2.5, new project, only your instructions.

i don't know how to execute this gem because everything i use the comands, rails give me a same error
"couldn't find file 'semantic_ui/definitions/modules/accordion.less' with type 'text/css'"
can you help me?
ps: i have, less-rails gem

And if to create the empty file accordion.less?
Sent from BlueMail
On 18 Dec 2015 12:59, at 12:59, Gustavo Santiago<EMAIL_ADDRESS>wrote:

i don't know how to execute this gem because everything i use the
comands, rails give me a same error
"couldn't find file 'semantic_ui/definitions/modules/accordion.less'
with type 'text/css'"
can you help me?
ps: i have, less-rails gem

Reply to this email directly or view it on GitHub:
https://github.com/Semantic-Org/Semantic-UI-Rails-LESS/issues/12#issuecomment-165708005

i follow step-by-step the instructions for install the gem in my project, when i execute rails s and refresh a page, rails give me this error.

Create minimal project for reproduce problem and I will try to investigate reason of it.

gustavosantiago, my apologies. The idea to create an empty file was not from here.I messed up projects.
But I decided to double-check the installation:
use rvm

install ruby 2.2.4p320
create rvm gemset
install rails  4.2.5
create new project rails new app
add to Gemfile
gem 'less-rails-semantic_ui'
gem 'less-rails'

run bundle install, rails s
and I received an error:
/gems/less-rails-2.7.0/lib/less/rails/railtie.rb:19:in block in <class:Railtie>': undefined method register_preprocessor' for nil:NilClass (NoMethodError)
as it is clear at the moment, the problem is 'sprockets-rails'. search for a problem I did not.
my solution:
add gem 'sprockets-rails', '<=2.3.3'  (a string of project works)
run bundle update, rails s
successful start
after:
6. run rails generate semantic_ui:install
7. add lines  in application.js and .css
//= require semantic_ui/semantic_ui
*= require semantic_ui/semantic_ui
8. create view with semantic-ui class
9. start rails
successful start
file listing

Gemfile
 ```
source 'https://rubygems.org'
Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
gem 'rails', '4.2.5'
Use sqlite3 as the database for Active Record
gem 'sqlite3'
Use SCSS for stylesheets
gem 'sass-rails', '~> 5.0'
Use Uglifier as compressor for JavaScript assets
gem 'uglifier', '>= 1.3.0'
Use CoffeeScript for .coffee assets and views
gem 'coffee-rails', '~> 4.1.0'
See https://github.com/rails/execjs#readme for more supported runtimes
gem 'therubyracer', platforms: :ruby
Use jquery as the JavaScript library
gem 'jquery-rails'
Turbolinks makes following links in your web application faster. Read more: https://github.com/rails/turbolinks
gem 'turbolinks'
Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
gem 'jbuilder', '~> 2.0'
bundle exec rake doc:rails generates the API under doc/api.
gem 'sdoc', '~> 0.4.0', group: :doc
Use ActiveModel has_secure_password
gem 'bcrypt', '~> 3.1.7'
Use Unicorn as the app server
gem 'unicorn'
Use Capistrano for deployment
gem 'capistrano-rails', group: :development
group :development, :test do
Call 'byebug' anywhere in the code to stop execution and get a debugger console
gem 'byebug'
end
group :development do
Access an IRB console on exception pages or by using <%= console %> in views
gem 'web-console', '~> 2.0'
Spring speeds up development by keeping your application running in the background.  Read more: https://github.com/rails/turbolinks
gem 'turbolinks'
Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
gem 'jbuilder', '~> 2.0'
Use ActiveModel has_secure_password
gem 'bcrypt', '~> 3.1.7'
Flexible authentication solution for Rails
#gem 'devise'
Use Unicorn as the app server
gem 'unicorn'
Use Capistrano for deployment
gem 'capistrano-rails', group: :development
group :doc do
bundle exec rake doc:rails generates the API under doc/api.
gem 'sdoc', '~> 0.4.0'
end
Compile less files
gem 'less-rails'
Provides the ability to customize component themes and variables with Semantic Ui
gem 'less-rails-semantic_ui', '~> <IP_ADDRESS>'
Use for improve compatibility with different browsers
gem 'autoprefixer-rails'
gem 'sprockets-rails', '<=2.3.3'
group :development do
Inspect source code live through a console that works on every stack frame
gem 'better_errors'
Access an IRB console on exception pages or by using <%= console %> in views
gem 'web-console', '~> 2.0'
Spring speeds up development by keeping your application running in the background. Read more: https://github.com/rails/spring
gem 'spring'
View missing link between the browser and your app code
gem 'xray-rails'
end
group :development, :test do
Call 'byebug' anywhere in the code to stop execution and get a debugger console
gem 'byebug'
Generate model and controller UML class diagram
gem 'railroady', '~> 1.0.9'
end

gustavosantiago I use your  Gemfile.
Add only gem 'sqlite3'.
Everything is working

@gustavosantiago Disable or restart spring. Then try to reproduce problem again.

Guys, i got it, I enable the unicorn gem (thats crazy) and works.
@postbud, I see in your gemfile a unicorn uncommented, so a reply this
Thanks very much @maxd and @postbud, i will prepare a tutorial for my friends on Brazil for don't passing through it again.


Holoviews - dynamic map issue in teehr-hub

No luck figuring out why the scatter plot and map do not respond to changes in the dropdown menus.  Seems like a holoviews - dynamic map issue, not pandas.  A very basic example from the holoviews gallery does not work on the teehr hub:  check out /data/post-event/DynMap_issue.ipynb (which came from https://holoviews.org/reference/containers/bokeh/DynamicMap.html)
@kvanwerkhoven This seems like a real issue - lets track it here.

This was fixed by upgrading the pangeo image to 20230705 in teehr version v0.2.2, and also installing nodejs.
https://github.com/RTIInternational/teehr/tree/v0.2.2


Integration and fixes for new centrifuge-chain release

Ensure thet the indexer is working as it should with the latest release of the centrifuge chain and the uptate of the dev node.

Current naming:
https://docs.google.com/spreadsheets/d/1-lp1cYIy5B9Kpni3hLo2UqDD76aDxMimZW_6MbhqiQ4/edit#gid=919922872

Implemented in #133


Fixed on auto update frame

I tried to make it having auto update better, since the program has some bugs on my device, that's why I tried to fix some. Also I added a replacing string, where it looks like a formatted string, I don't know how to make a string into a formatted string yet in rust.
Edit
Here's the link of my configuration

I also make the no animation update in labels, and I also I'm trying to use Regex for multi commands.


Add some Identity client methods

#670  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @neekolas and the rest of your teammates on  Graphite


Cannot install via Vagrant (latest version) under Windows even though this is supposed to work per documentation

Per documentation:
https://github.com/openwhisk/openwhisk#vagrant-users-for-mac-and-windows
Some guys from development told me Vagrant under Windows would not be supported; but if this is the case I wonder the documentation tells something different?
Error I get is:
Stderr: 0%...
Progress state: E_INVALIDARG
VBoxManage.exe: error: Failed to create the host-only adapter
VBoxManage.exe: error: Assertion failed: [!aInterfaceName.isEmpty()] at 'F:\tind
erbox\win-5.0\src\VBox\Main\src-server\HostNetworkInterfaceImpl.cpp' (74) in lon
g __cdecl HostNetworkInterface::init(class com::Bstr,class com::Bstr,class com::
Guid,enum __MIDL___MIDL_itf_VirtualBox_0000_0000_0036).
VBoxManage.exe: error: Please contact the product vendor!
VBoxManage.exe: error: Details: code E_FAIL (0x80004005), component HostNetworkI
nterfaceWrap, interface IHostNetworkInterface
VBoxManage.exe: error: Context: "enum RTEXITCODE __cdecl handleCreate(struct Han
dlerArg *)" at line 71 of file VBoxManageHostonly.cpp

@nauerz-ibm This seems like a Vagrant issue specific to your Windows machine, see https://www.virtualbox.org/ticket/15019.  The instructions assume you can run a Vagrant machine on your native host.

Will try again tomorrow after having cleaned-up everything; will report the results here.

team members with Windows computers are working with Vagrant using new share folder flow. If you see problems in Windows please open a new issue.


Support injecting variables from other files through jinja syntax

Similar to #138
In conda-build, something like load_file_data('pyproject.toml').get('project').get('version') is possible. Would be nice to get something similar here as well.

I think my question here is wether this would be a local file or supposed to come from the source section of the recipe.
I think it would be feasible if it's a local file / recipe. But if the sources need to be downloaded first to discover these missing variables it would create a dependency within the rendering / execution of the recipe that I wouldn't want.

For my personal use case (where my sources are local too), a local file would be sufficient.


Validate whether port is exposed in a Dockerfile

Validates if a port is exposed in the Dockerfile. This was a suggestion in https://github.com/zeit/now-cli/issues/1019#issuecomment-354550268

cc @timneutkens
P.S. we would have to revert this change when #146 is merged.

Yes @pranaygp
If you feel it's worthwhile having this error, we can have it. Else I can close the PR.

Nice!
I have one question though, what if the parent image already exposes a port?

I have one question though, what if the parent image already exposes a port?

We had EXPOSE port validation basically exactly like this PR a long time ago, but it caused issues specifically because of this edge-case and removed the validation.

@jamo @TooTallNate
I didn't think about the scenario when a base image included in the Dockerfile exposes a port. I'll close this PR.


#11 Configurable scalyr request.

#11 Configurable Scalyr request.
New optional configuration parameter SCALYR_BASE_FILTER added.
It is used to preselect log messages before finding consumers or producers of an event type.
This helps to adapt UI to different Scalyr setups.
Default value is the same.
($serverHost=="nakadi") and ($logfile=="/var/log/application.log") and
For staging, for example, it can be
(($serverHost=="nakadi-staging") or ($serverHost=="nakadi-sandbox"))  and ($logfile=="/var/log/application.log") and

Yes of course, but not necessary, as this parameter is optional. Also I'm planning to switch to CDP, so it will be even new repo maybe.


[N810] Feature Request for Exception in using CapWords for Class Names

custom_ops.py:173:7: N801 Class name `PReLU_Function` should use CapWords convention

We can use PreluFunction (which hides some semantics) but PReLUFunction looks ugly. Can we add an exception such that _ is allowed only if the previous letter is upper case?
We can ofc add an ignore on this line, but maybe there should be a teeny tiny exception for this?
# current
splits = class_name.split("_")
if len(splits) > 1:
    log_error()

# request
splits = class_name.split("_")

for prev_split, split in zip(splits[:-1], splits[1:]):
    if prev_split[-1].isupper() and split[0].isupper():  # split should have upper case on either end
        continue
    else:
        log_error()

Hmm, I think the intention of CapsWords is that you write it as PReLUFunction. I could understand an argument that it's ugly but it does adhere to the intent and description as in PEP 8.
My suggestion would probably be to explicitly ignore it via ignore-names, which lets you exempt certain names (and does support glob patterns).


DB connection retry not happening

Bug description
I'm using a prisma setup in nest.js very similar to this example:
https://github.com/prisma/prisma-examples/tree/latest/typescript/rest-nestjs
If I start the application when the DB is down, even after starting the DB, I keep getting an "Error creating a database connection" error when calling prisma. It requires an application restart to recover from this.
Error:
"\nInvalid `prisma.workOrder.create()` invocation:\n\n\n  Error creating a database connection. (Error creating a database connection.)"

Start the nest.js application with the prisma setup as explained in the docs, similar to this example: https://github.com/prisma/prisma-examples/tree/latest/typescript/rest-nestjs.
Make a prisma call and get the "Error creating a database connection" error.
Start the DB, make sure it's running properly.
Make the prisma call again and you still get the "Error creating a database connection" error.

Note: I've tried both the explicit and lazy approach to start the connection (implementing or not the OnModuleInit) and both have the same issue.
Expected behavior
Even if the DB is down at application startup, I'd expect prisma to be able to connect to the DB once the DB is started, without an application restart.
Note: I've noticed prisma is able to recover from a database shutdown/restart without having to restart the application, if - and only if - the DB was up and running when the application started.
Prisma information
I'm adding part of the relevant code here (prisma schema includes object definition, just not showing it here):

Note: I've tried implementing and not implementing the onModuleInit.

Environment & setup

OS: Windows 10
Database: Microsoft SQL Server 15.0.2000.5
Node.js version: 14.16.0

Prisma Version
prisma                  : 3.0.1
@prisma/client          : 3.0.1
Current platform        : windows
Query Engine (Node-API) : libquery-engine 2452cc6313d52b8b9a96999ac0e974d0aedf88db (at<EMAIL_ADDRESS>
Migration Engine        : migration-engine-cli 2452cc6313d52b8b9a96999ac0e974d0aedf88db (at<EMAIL_ADDRESS>
Introspection Engine    : introspection-core 2452cc6313d52b8b9a96999ac0e974d0aedf88db (at<EMAIL_ADDRESS>
Format Binary           : prisma-fmt 2452cc6313d52b8b9a96999ac0e974d0aedf88db (at<EMAIL_ADDRESS>Default Engines Hash    : 2452cc6313d52b8b9a96999ac0e974d0aedf88db
Studio                  : 0.423.0

Thanks for the report. We will try to reproduce this.
Do you think the rest-nestjs app has influence on this specifically or if this is a general Prisma thing?

https://github.com/prisma/prisma/pull/12066 fixes this.
You can test it out with prisma@dev and will be out in the next release


Feat/tls

This is an ongoing work on the tls feature of dicedb.

Hi David, are you actively working on this feature?


nordic nrf52/nrf53 and missing cpu-power-states (dts) for automatic device PM control

Hi all.
I'm playing with nrf52 and nrf53 DK board in order to evaluate the power consumption.

There is some general documentation (https://devzone.nordicsemi.com/nordic/nordic-blog/b/blog/posts/optimizing-power-on-nrf53-designs) to explain what to do.
one specific example (zephyr/samples/boards/nrf/system_off) that show that UART (and any active device) must be suspend manually if we expect a proper low consumption

Well...
first, I would expect a complete ble example that achieve THIS low consumption (few uA) without hacking the prj.conf (to disable UART mainly) or hacking main.c (to do the same manually UART disabling), so we can all start from a good example.
But, more important, I don't understand why automatic device suspend is not supported at first.
I did some test:

provide a proper cpu-power-states in DTS (ie. add power-state-name = "suspend-to-idle" case)	power-states {
  	idle: idle {
  		compatible = "zephyr,power-state";
  		power-state-name = "suspend-to-idle";
  		min-residency-us = <1000>;
  	};
  };

modify  soc/arm/nordic_nrf/nrf52/power.c to call k_cpu_idle() when a low power mode is requested@@ -16,10 +16,12 @@ __weak void pm_power_state_set(struct pm_state_info info)
  switch (info.state) {
  case PM_STATE_SOFT_OFF:
  	nrf_power_system_off(NRF_POWER);
  	break;
  default:
+	    k_cpu_idle();
  	LOG_DBG("Unsupported power state %u", info.state);
  	break;
  }
 }

add the PM_DEVICE_ACTION_LOW_POWER case  in uarte_nrfx_pm_control to disable the uart just for the idle period@@ -1858,10 +1860,11 @@ static int uarte_nrfx_pm_control(const struct device *dev,
 	}
 	break;
 case PM_DEVICE_ACTION_SUSPEND:
+	case PM_DEVICE_ACTION_LOW_POWER:
 	/* Disabling UART requires stopping RX, but stop RX event is
 	 * only sent after each RX if async UART API is used.

With those changes, zephyr properly request devices PM suspend before going in idle and consume as few as expected (few uA with a 1s advertising). And serial log , printk & co are still working as expected once we exit idle.
Is there something I miss in nrf52 support that prevent this automatic device PM ? or is it just that nordic didn't finalize the support yet ?
Regards,
Arnaud

I've been struggling with this on the nrf52840dk for months. I have not been able to get current consumption below 270uA. I've tried everything with no luck. Can you share with me your code you used to successfully achieve the few uA? I added your above snippets but I am still seeing 270uA. If I got into System OFF mode I do indeed see a very low current oonsumption, so I know at least that works, but I'd like to stay in System ON with all of the peripherals disabled.


Use site transients for saving mu-plugins

Since the mu-plugins list should be considered a cache we should use transients for saving the plugin list.

@Foxaii

Sorry to have taken so long to respond to this issue.
I made the conscious decision to use a site option when writing the original plugin. I did so because the list of mu-plugins isn't invalidated over time: it's invalidated when plugins are added or removed. Standard plugins are also saved as options, not transients, although I appreciate there is a meaningful difference between the two because standard plugins are an option in the WordPress backend.
There's little or no difference between options and transients in practice, although I think that if we switch (which I'm not inclined to do) we need to add a comment to make it clear it's an autoloaded transient, without an expiry date.
Do you have anything to add to support the switch?

With persistent cache plugins, transients can be handled by, for example, Redis. Theoretically this means that the data can be retrieved faster, but I remain skeptical of the actual impact of what ostensibly seems to be a micro-optimization given the small size of the data that's stored by the autoloader. It also makes the presence of the cache less predictable, which means it might need to be regenerated more frequently, i.e., transient data can be purged at any time. Having it as an option makes its presence marginally more predictable. But ultimately, I don't think either of these are strong enough arguments for either side, and I would just as soon defer my position on this issue to @foxaii for his reasoning.
tl;dr-
¯\_(ツ)_/¯

Thank you for your reply @Foxaii
Like @QWp6t said, we do use Redis as object cache. And we are deploying by using Capistrano. We do have a Capistrano task to flush the cache when deploying (and for example adding new mu-plugins) and right now the mu-plugin cache is not invalidated, therefore generating some trouble for us.
With this change the mu-plugin cache is invalidated correctly when other cache values is flushed.

The list of mu-plugins is updated whenever a path is missing (i.e. when a plugin is removed or renamed) or when a new folder is found (i.e. when a plugin is added), so you really shouldn't need to be flushing anything manually, whether it's an option or transient.
Do you mind elaborating on the 'trouble' you are seeing as it could be a bug.

I'm sorry for a slow reply.
If we add a new mu-plugin it won't be loading after deploy since it's not in the site option and it won't be loaded until someone visits the wp-admin plugin page.
If the plugin uses transients instead and we flush the object cache when deploying it will be loaded.

@Foxaii

Sorry for the delay in getting to this.
I understand that manually flushing the transient is solving the problem for you @rasmusbe, but the underlying issue is that the plugin is not correctly invalidating its cache.
We can either fix the plugin's cache invalidation, which fixes the issue for everyone (whether it uses an option or transien), or switch to a transient, which fixes it only for people who flush caches after a deploy.
That's why I asked for more data from you, as the mu-plugin cache should be invalidating itself if a plugin isn't found, or the number of folders have changed. If we can figure which of these events isn't being detected correctly on your project, we can fix the underlying issue.
I hope that makes sense.

Closing this as @Foxaii's last comment makes it clear that this doesn't solve the actual problem.


Should add code linter

Having a code linter is a code practice
We can look into django-lint (https://github.com/lamby/django-lint)
/cc @mbad0la @ManrajGrover

+1
No discussion here.


crash with SIGSEGV

We have a node that crashed with SIGSEGV.
I attached part of the log file, maybe it helps to figure out what happened or how to prevent it in future.
log.log

I think this happened when I restarted other nodes for updating. It seems that it's peering related.


System stuck when upload @niklauslee/desktop-clock

Reproduce steps

Open the project @niklauslee/desktop-clock
Change Line#8 and Line#9
Upload code
System stuck immediately  after code upload

부품 연결이 없을 경우 ReferenceError: Failed to write data via I2C bus. 오류가 나오면 정상 동작하는 것. 시스템이 멈출때에는 .load 표시후에 바로 멈춤.

More experiments

Works after removing line#90 of index.js. (It's so strange!)
This issue is not about the size of code. (Works well when upload the same size of the code just calling console.log())

귀신이 곡할노릇

어디서 멈추는지 보려고 console.log()를 몇군데 넣었더니, 또 잘 동작함...

This issue is related with the issue #165


test: add debug message when assertion failed

Which issue(s) this PR fixes:

Issue https://github.com/longhorn/longhorn/issues/8521
What this PR does / why we need it:
add debug message when assertion failed
Special notes for your reviewer:
Additional documentation or context

@mergify backport v1.6.x


Preserve new line at start of handlebars contents

Fixes #1469
This change only makes it so new line is preserved for the first hbs attribute.  This doesn't fix or change deeper handlebars/hbs formatting, nor does it cause handlebars to be beautified better insides are still not reindented if they are off.  This only make the beautifier not undo user formatting.

Awesome, thanks for taking a look at this so quickly!

@MacKLess
I'm using #1541 to track the more general issue of handlebars "attribute" formatting.


RSS Feed?

Is there an RSS feed for https://getmonero.org/blog/ ?

Are you looking for something like https://getmonero.org/feed.xml?

Yup https://getmonero.org/feed.xml works for me.  Thanks!  Although, I don't know how easy it would be for someone to find the feed.  I don't see a link to the feed on the front page or on https://getmonero.org/blog/ .  Also, my feed reader wasn't able to autodetect a feed on the site.

I will add a link into the footer.


type alias for EdgeInsets on OSX

if the type alias is used, one can simply pass a NSEdgeInsets struct as well. The duplication of EdgeInsetsMake and EdgeInsetsZero is not needed.

I've decided to leave this in there for now, mostly just so OS X matches iOS's API.


"Shape by" for nodes is not saved

The "shape by" function for nodes is not retained when saving a microbetrace build. Can this user-defined parameter be included in the saved microbetrace file?

This issue has been fixed.


raise error if validation fails

I find it helpful in my pipelines to have validation script raise an error if validation fails, so suggesting it here in case others might find it helpful too (or at least innocuous).

Thanks @sierra-moxon


[Form Validation] Add Errors not Working Correctly

According to the docs on form validation, I can add errors manually:

add errors(errors) |  Adds errors to form, given an array errors

(I want to use this feature because I submit my form via AJAX, do server-side validation, then want to display the results.)
I tried the following code:
$('#my-form').form("add errors", [ 'error' ]);
$('#my-form').form("validate form");

I get this contradictory output from the console when calling the above methods, and the form validates as successful when it obviously shouldn't.

Unless I'm doing something wrong?
As a side note, as the documentation is far from clear, in the array of errors, (how) can I link an error to a field name?

Hi there! Any progress on this issue?

I've dug up this api from source to manually show inline errors:
$('.form').form('add').prompt('fieldname' (from name attribute),'error text')

@avalanche1 it seems that using your line of code only colors the field with red but does not show the 'error text' in errors list.
However, using
$(this.refs.form).form('add errors', {
        email: 'error text',
      });

does show the error in the list, but does not color the field

+1

+1 This is very urgent.  There is no proper documentation on this.  I need to integrate these forms with Angular.

@eyalw , yep, I only use inline error prompts.

+1 , this is important feature.
Agree with @timothycoutlakis , this is very urgent.

@avalanche1
In this way looks better
$(this).form('add prompt', 'fieldname', 'error text');
$(this).form('remove prompt', 'fieldname');

None of the above works for me. But this does:
	$('.form').form('add prompt', 'email');
	$('.form .error.message').html('Please enter a valid email address').show();

$('#form').form('add errors', {password: 'Invalid Credentials', email: 'Invalid email'})
$('#form').form('add prompt', 'email')
$('#form').form('add prompt', 'password')

a) Unstalification
b) This disfunction is so quirky and commonplace that it should be a BUG, not an enhancement.