Question/Idea: Automatic Gradient Clearing

Hi Prof Karpathy,
I wanted to create a discussion to ask this question, but there was no provision as such. I was watching https://youtu.be/VMj-3S1tku0 and got an idea.
Context
This is in reference to the step of clearing accumulated gradients at: https://github.com/karpathy/micrograd/blob/c911406e5ace8742e5841a7e0df113ecb5d54685/demo.ipynb#L265
Problem
People tend to forget to clear the gradients wrt the loss function backward pass.
Idea
Create a way to bind the loss function to the network once, and then automatically clear accumulated gradients automatically when performing the backward pass.
Advantage
We can perform backward pass whenever, wherever, and as many times as we want without worrying about accumulated gradient.
Pseudocode
class Loss(Value):
  def __init__(self, bound_network):
    self.bound_network = bound_network

  def __call__(self, batch_size=None):
    # loss function definition
    self.data = data_loss + reg_loss

  def backward():
    # clear gradients of bound network
    bound_network.zero_grad()
    super().backward()

total_loss = Loss(
  bound_network = model
)

for k in range(100):
  # ...

  # model.zero_grad() # since total_loss is bound to network, it should automatically perform model.zero_grad() before doing the backward
  total_loss.backward()

  # ...

Questions

Is my understanding of the problem correct?
Is this change value-adding?
Is the above pseudocode logically correct?
If the answer to all the above are yes, I could work on a PR with your guidance.

I'm not @karpathy, but here are some thoughts on this. Micrograd shares some similarities with PyTorch, such as the Module class and methods like zero_grad() and parameters(). While it's true that people sometimes forget to zero the gradients, the reason it’s not done automatically, as in frameworks like TensorFlow, is for flexibility and to give the user control over the training process. For instance, if you want to accumulate gradients over several training steps, PyTorch allows you to do this easily by simply adding a condition before calling zero_grad(). In TensorFlow, you’d need to implement a gradient accumulator for this.
Another reason, I believe, is to maintain the simplicity of the project for educational purposes. Adding extra features can be a good exercise when creating project forks.
Finally, losses don’t have access to model parameters because their role is to measure the discrepancy between model outputs and targets. However, optimizers do. For example, you can incorporate the zero_grad() call into the optimizer.step() method if needed, though this isn't the standard approach as it limits flexibility.

Got it, thanks for the detailed explanation.
The gradient accumulation is an obvious one that I seem to have overlooked.


Bug 1806751: Archive data-dir if target member is unstarted

Please read https://github.com/etcd-io/etcd/blob/master/CONTRIBUTING.md#contribution-flow.

In general, I wonder how useful this backup is if it is not matched with the static pod resources. In the case of encrypted datastore it would be more or less useless right?
Should we consider performing a more robust backup procedure? Regardless of the existence of this backup would require documentation on what this backup it is and how it can be used in the case of failure. In that failure case what can we guarantee about this data? I think we can guarantee it is better than nothing.
So my question is. Should we just document this procedure (manual) before restore and have the operator just nuke it?

@hexfusion well, it is a conflict between keeping the instructions simple (4th grade English ☺) and keeping the logic simple. For 4.4, I am fine if we just take the documentation route.

Yeah I hear you it's just when folks ask how they use this backup we need to have an answer. Something to talk about.

This is not going to work because we bail out if the data-dir is already present at the beginning.
/hold

/lgtm
Get your bugs in order and be prepared to pick.

/retest
/apprive

/approve

This is not going to work because we bail out if the data-dir is already present at the beginning.
/hold

fixed
/hold cancel

/cherry-pick openshift-4.4

/bugzilla refresh

/bugzilla refresh

/refresh

/lgtm


[7-zip] Only build Windows sources for Windows

I've got no idea why these were here (it was in the initial PR without any explanation) and can't make any sense of the upstream makefiles, but it seems like the things in the Windows directory ought to only be built for Windows. This fixes compilation errors with NDK r28.

Please get failure logs here on x64-widnows:
[265/266] C:\Windows\system32\cmd.exe /C "C:\Windows\system32\cmd.exe /C "D:\downloads\tools\cmake-3.30.1-windows\cmake-3.30.1-windows-i386\bin\cmake.exe -E __create_def D:\b\7zip\x64-windows-dbg\CMakeFiles\7zip.dir\.\exports.def D:\b\7zip\x64-windows-dbg\CMakeFiles\7zip.dir\.\exports.def.objs && cd D:\b\7zip\x64-windows-dbg" && D:\downloads\tools\cmake-3.30.1-windows\cmake-3.30.1-windows-i386\bin\cmake.exe -E vs_link_dll --intdir=CMakeFiles\7zip.dir --rc=C:\PROGRA~2\WI3CF2~1\10\bin\100226~1.0\x64\rc.exe --mt=C:\PROGRA~2\WI3CF2~1\10\bin\100226~1.0\x64\mt.exe --manifests  -- C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1441~1.341\bin\Hostx64\x64\link.exe  @CMakeFiles\7zip.rsp  /out:7zip.dll /implib:7zip.lib /pdb:7zip.pdb /dll /version:0.0 /machine:x64 /nologo    /debug /INCREMENTAL  /DEF:CMakeFiles\7zip.dir\.\exports.def && cd ."
FAILED: 7zip.dll 7zip.lib
C:\Windows\system32\cmd.exe /C "C:\Windows\system32\cmd.exe /C "D:\downloads\tools\cmake-3.30.1-windows\cmake-3.30.1-windows-i386\bin\cmake.exe -E __create_def D:\b\7zip\x64-windows-dbg\CMakeFiles\7zip.dir\.\exports.def D:\b\7zip\x64-windows-dbg\CMakeFiles\7zip.dir\.\exports.def.objs && cd D:\b\7zip\x64-windows-dbg" && D:\downloads\tools\cmake-3.30.1-windows\cmake-3.30.1-windows-i386\bin\cmake.exe -E vs_link_dll --intdir=CMakeFiles\7zip.dir --rc=C:\PROGRA~2\WI3CF2~1\10\bin\100226~1.0\x64\rc.exe --mt=C:\PROGRA~2\WI3CF2~1\10\bin\100226~1.0\x64\mt.exe --manifests  -- C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1441~1.341\bin\Hostx64\x64\link.exe  @CMakeFiles\7zip.rsp  /out:7zip.dll /implib:7zip.lib /pdb:7zip.pdb /dll /version:0.0 /machine:x64 /nologo    /debug /INCREMENTAL  /DEF:CMakeFiles\7zip.dir\.\exports.def && cd ."
LINK Pass 1: command "C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1441~1.341\bin\Hostx64\x64\link.exe @CMakeFiles\7zip.rsp /out:7zip.dll /implib:7zip.lib /pdb:7zip.pdb /dll /version:0.0 /machine:x64 /nologo /debug /INCREMENTAL /DEF:CMakeFiles\7zip.dir\.\exports.def /MANIFEST /MANIFESTFILE:CMakeFiles\7zip.dir/intermediate.manifest CMakeFiles\7zip.dir/manifest.res" failed (exit code 1120) with the following output:
   Creating library 7zip.lib and object 7zip.exp
InOutTempBuffer.cpp.obj : error LNK2019: unresolved external symbol "long __cdecl GetLastError_noZero_HRESULT(void)" (?GetLastError_noZero_HRESULT@@YAJXZ) referenced in function "public: long __cdecl CInOutTempBuffer::Write_HRESULT(void const *,unsigned int)" (?Write_HRESULT@CInOutTempBuffer@@QEAAJPEBXI@Z)
InOutTempBuffer.cpp.obj : error LNK2019: unresolved external symbol "public: bool __cdecl NWindows::NFile::NIO::CFileBase::Close(void)" (?Close@CFileBase@NIO@NFile@NWindows@@QEAA_NXZ) referenced in function "public: long __cdecl CInOutTempBuffer::WriteToStream(struct ISequentialOutStream *)" (?WriteToStream@CInOutTempBuffer@@QEAAJPEAUISequentialOutStream@@@Z)
InOutTempBuffer.cpp.obj : error LNK2019: unresolved external symbol "public: bool __cdecl NWindows::NFile::NIO::CInFile::Open(wchar_t const *)" (?Open@CInFile@NIO@NFile@NWindows@@QEAA_NPEB_W@Z) referenced in function "public: long __cdecl CInOutTempBuffer::WriteToStream(struct ISequentialOutStream *)" (?WriteToStream@CInOutTempBuffer@@QEAAJPEAUISequentialOutStream@@@Z)

@microsoft-github-policy-service agree

Please get failure logs here on x64-widnows:
[265/266] C:\Windows\system32\cmd.exe /C "C:\Windows\system32\cmd.exe /C "D:\downloads\tools\cmake-3.30.1-windows\cmake-3.30.1-windows-i386\bin\cmake.exe -E __create_def D:\b\7zip\x64-windows-dbg\CMakeFiles\7zip.dir\.\exports.def D:\b\7zip\x64-windows-dbg\CMakeFiles\7zip.dir\.\exports.def.objs && cd D:\b\7zip\x64-windows-dbg" && D:\downloads\tools\cmake-3.30.1-windows\cmake-3.30.1-windows-i386\bin\cmake.exe -E vs_link_dll --intdir=CMakeFiles\7zip.dir --rc=C:\PROGRA~2\WI3CF2~1\10\bin\100226~1.0\x64\rc.exe --mt=C:\PROGRA~2\WI3CF2~1\10\bin\100226~1.0\x64\mt.exe --manifests  -- C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1441~1.341\bin\Hostx64\x64\link.exe  @CMakeFiles\7zip.rsp  /out:7zip.dll /implib:7zip.lib /pdb:7zip.pdb /dll /version:0.0 /machine:x64 /nologo    /debug /INCREMENTAL  /DEF:CMakeFiles\7zip.dir\.\exports.def && cd ."
FAILED: 7zip.dll 7zip.lib
C:\Windows\system32\cmd.exe /C "C:\Windows\system32\cmd.exe /C "D:\downloads\tools\cmake-3.30.1-windows\cmake-3.30.1-windows-i386\bin\cmake.exe -E __create_def D:\b\7zip\x64-windows-dbg\CMakeFiles\7zip.dir\.\exports.def D:\b\7zip\x64-windows-dbg\CMakeFiles\7zip.dir\.\exports.def.objs && cd D:\b\7zip\x64-windows-dbg" && D:\downloads\tools\cmake-3.30.1-windows\cmake-3.30.1-windows-i386\bin\cmake.exe -E vs_link_dll --intdir=CMakeFiles\7zip.dir --rc=C:\PROGRA~2\WI3CF2~1\10\bin\100226~1.0\x64\rc.exe --mt=C:\PROGRA~2\WI3CF2~1\10\bin\100226~1.0\x64\mt.exe --manifests  -- C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1441~1.341\bin\Hostx64\x64\link.exe  @CMakeFiles\7zip.rsp  /out:7zip.dll /implib:7zip.lib /pdb:7zip.pdb /dll /version:0.0 /machine:x64 /nologo    /debug /INCREMENTAL  /DEF:CMakeFiles\7zip.dir\.\exports.def && cd ."
LINK Pass 1: command "C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1441~1.341\bin\Hostx64\x64\link.exe @CMakeFiles\7zip.rsp /out:7zip.dll /implib:7zip.lib /pdb:7zip.pdb /dll /version:0.0 /machine:x64 /nologo /debug /INCREMENTAL /DEF:CMakeFiles\7zip.dir\.\exports.def /MANIFEST /MANIFESTFILE:CMakeFiles\7zip.dir/intermediate.manifest CMakeFiles\7zip.dir/manifest.res" failed (exit code 1120) with the following output:
   Creating library 7zip.lib and object 7zip.exp
InOutTempBuffer.cpp.obj : error LNK2019: unresolved external symbol "long __cdecl GetLastError_noZero_HRESULT(void)" (?GetLastError_noZero_HRESULT@@YAJXZ) referenced in function "public: long __cdecl CInOutTempBuffer::Write_HRESULT(void const *,unsigned int)" (?Write_HRESULT@CInOutTempBuffer@@QEAAJPEBXI@Z)
InOutTempBuffer.cpp.obj : error LNK2019: unresolved external symbol "public: bool __cdecl NWindows::NFile::NIO::CFileBase::Close(void)" (?Close@CFileBase@NIO@NFile@NWindows@@QEAA_NXZ) referenced in function "public: long __cdecl CInOutTempBuffer::WriteToStream(struct ISequentialOutStream *)" (?WriteToStream@CInOutTempBuffer@@QEAAJPEAUISequentialOutStream@@@Z)
InOutTempBuffer.cpp.obj : error LNK2019: unresolved external symbol "public: bool __cdecl NWindows::NFile::NIO::CInFile::Open(wchar_t const *)" (?Open@CInFile@NIO@NFile@NWindows@@QEAA_NPEB_W@Z) referenced in function "public: long __cdecl CInOutTempBuffer::WriteToStream(struct ISequentialOutStream *)" (?WriteToStream@CInOutTempBuffer@@QEAAJPEAUISequentialOutStream@@@Z)

Uh, I copied from the rest of the file but I think the rest of the file is wrong. VCPKG_CMAKE_SYSTEM_NAME is not set in that file, so everything guarded by VCPKG_TARGET_IS_WINDOWS is effectively commented out. I'm guessing that definition is only provided for portfile.cmake, whereas this is actually a custom CMakeLists.txt for the port... I've fixed my changes in the PR but I left the rest of the file as-is. I've got no idea what any of that is supposed to do so I don't really want to be adding all those currently unused source files to a port that doesn't appear to need them.

Note that in vcpkg, there are two CMake worlds: script mode and project mode. They have different set of variables.
https://learn.microsoft.com/en-us/vcpkg/contributing/maintainer-guide#portfiles-are-run-in-script-mode

D:\a\_work\1\s\ports\7zip\vcpkg.json: note: if<EMAIL_ADDRESS>is already published, update this file with a new version or port-version, commit it, then add the new version by running:
  vcpkg x-add-version 7zip
  git add versions
  git commit -m "Update version database"
note: if<EMAIL_ADDRESS>is not yet published, overwrite the previous git tree by running:
  vcpkg x-add-version 7zip --overwrite-version
  git add versions
  git commit -m "Update version database"

But it's still 7-zip 24.08. What am I supposed to do here? How does vcpkg handle the "build" version without updating the package version?

https://learn.microsoft.com/en-us/vcpkg/reference/vcpkg-json#version explains that I can do basically whatever I want with that field, but what's the convention?

You need to increment port-version in ports/7zip/vcpkg.json. But the field is not present when it is zero. And now you have to restore versions/ first. Don't expect too much convenience.

Gotcha, thanks!


Week 5 Hacks Review Ticket

[x] Link to individual flask python hacks
[x] Link to submenu (includes links to JavaScript Kernel and HTML/JavaScript Table)
[x] Link to group flask python hacks
[x] Link to 1.2 Program Purpose/Function Notes
[x] Link to Agile Methodology Notes/Vocab
[x] Link to Team Scrum Board

2.8, I think Emaad did a very good job this week and he was able to get JavaScript working. I remember over the weekend he had a problem with his JavaScript Terminal, and I am pretty sure he had that problem for a while. He posted it on slack, but I don't think he got that much help, so after looking for solutions for a while he finally found it. I think this is a great example and shows how hard he works and how dedicated he is.
This week, Emaad also took on a lot for our group despite his busy life, he really took the leadership role and was constantly helping all 3 of us. It must have been annoying but no matter what he always helped us. He also took the leader role and did a lot of work on our shared group repository for the PythonFlask

Blog showing team roles.  Nice ideas on Scrum process.  Included on Team Flask Python.    Idea is Daily Agenda.
Changed cards.  Made color change buttons, W3Schools color change with CSSS.  Changed name of menu.  Inserted Code from Replit using IFrame.
Worked with JavaScript Notebook and tried loops.   Had a lot of notes.   Also worked on submenus go to different pages but does not retain menu on top.
2.8

2.7+ your personalization are really cool, I especially like how the color of your flask page changes whenever you click the button.


Method to produce source code hash is not deterministic

When fastssz is ran multiple times against the same package, even without changes to the original files, the generated file keeps changing. This is because the hash that is present on the header of the generated files is not deterministic. This is reproducible when fastssz is ran on any package with more than one source file.
This is most likely due to a bug in https://github.com/ferranbt/fastssz/blob/87558029fb68920c009520583ad7521e1b909119/sszgen/generator/generator.go#L314, which iterates over e.files (a map). In Go, the order of iteration over maps is not stable, therefore the plaintext fed to the sha256 hash is also not stable.

Hi, I think you are correct. I am assuming what changes is the hash digest and not the generated code? Would you like to open a PR to fix the issue?


mixer: enable CEL runtime as an option

Per agreement, we want to allow CEL expessions over CEXL using custom annotations on rules and instances, without changing the defaults. Additionally, add tooling for conversion to assist migration.
/cc @mandarjog

ref #10630

#12145 was merged. I think we can close this issue. Please re-open if you feel like I've jumped the gun.


Update actions/checkout in GitHub Actions workflows to v3

Updates the actions/checkout action used in the GitHub Actions workflows to its newest major version.
Changes in actions/checkout:

v3.5.2

Fix api endpoint for GHES

v3.5.1

Fix slow checkout on Windows

v3.5.0

Add new public key for known_hosts

v3.4.0

Upgrade codeql actions to v2
Upgrade dependencies
Upgrade @actions/io

v3.3.0

Implement branch list using callbacks from exec function
Add in explicit reference to private checkout options
Fix comment typos

v3.2.0

Add GitHub Action to perform release
Fix status badge
Replace datadog/squid with ubuntu/squid Docker image
Wrap pipeline commands for submoduleForeach in quotes
Update @actions/io to 1.1.2
Upgrading version to 3.2.0

v3.1.0

Use @actions/core saveState and getState
Add github-server-url input

v3.0.2

Add input set-safe-directory

v3.0.1

Fixed an issue where checkout failed to run in container jobs due to the new git setting safe.directory
Bumped various npm package versions

v3.0.0

Update to node 16

Still using v2 of actions/checkout will generate some warning like in this run: https://github.com/lemire/streamvbyte/actions/runs/5030115995

Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16: actions/checkout@v2. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.

The PR will get rid of those warnings for actions/checkout, because v3 uses Node.js 16.

Running tests


[bug]: "Smiley Face" icon for face reconstruction not visible

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Linux
GPU
cuda
VRAM
8GB
What version did you experience this issue on?
3.0.0
What happened?
I don't appear to have the "Smiley Face" icon for face reconstruction shown here: https://invoke-ai.github.io/InvokeAI/features/POSTPROCESS/#upscaling

The upscale icon appears fine.
Am I doing something wrong?
I have run invokeai-configure
Screenshots
No response
Additional context
No response
Contact Details
No response

The is feature was removed in 3.0


♻ Simplify assertEq for arrays

@ZeroEkkusu likely won't be able to get to the tests today so if wanted to fix those that would be appreciated!
cc @paulrberg

This is awesome guys! Just found the time to review the code changes, and it LGTM.


Fix shoot:apiserver_latency_seconds:quantile federation configuration

How to categorize this PR?

/area monitoring
/kind regression
What this PR does / why we need it:
In 9f3b25415, the recording rule previously named apiserver_latency_seconds:quantile was renamed to shoot:apiserver_latency_seconds:quantile. This had two consequences.
The less significant one is that it broke some dashboards. This commit fixes those dashboards.
The more significant one is that the aggregate prometheus then started federating this recording rule due to the {__name__=~"shoot:(.+):(.+)"} line in its federation parameters. This expression is intended to federate shoot metrics. Metrics matching that expression are federated twice in gardener. Once, as just explained, from the shoot control plane Prometheus to the aggregate Prometheus, and a second time from the aggregate Prometheus to the garden Prometheus. By the time this recording rule made it to the garden prometheus, it caused the number of samples scraped to increase massively. This in turn was significant enough for the garden Prometheus to consume more than twice as much memory as before. Therefore, this commit also excludes this recording rule from federation at the aggregate Prometheus level.
Which issue(s) this PR fixes:
Fixes an issue with the garden Prometheus consuming excessive amounts of memory.
Special notes for your reviewer:
/cc @istvanballok @vicwicker @chrkl
Release note:

Fix an issue with federation that causes garden-prometheus to consume excessive amounts of memory.

/cherry-pick 1.96

/cherry-pick 1.97

/retest-required

/retest-required

/retest-required

/cherry-pick release-v1.96

/cherry-pick release-v1.97


node build/build.js --watch  -> error: unknown option `--watch'

Version
5.0.0
Steps to reproduce
https://github.com/apache/incubator-echarts/wiki/How-to-setup-the-dev-environment
What is expected?
Project will build
What is actually happening?
$node build/build.js --watch
error: unknown option `--watch'

Was introduced during last two days, see https://github.com/apache/incubator-echarts/issues/13974

Sorry, it was just been replaced with npm run dev in https://github.com/apache/incubator-echarts/pull/13949. Thanks for the feedback

Sorry, it was just been replaced with npm run dev in https://github.com/apache/incubator-echarts/pull/13949. Thanks for the feedback


Add missing system dependencies for Puppeteer 1.8.0

I was having the same issues as discussed in #22. Adding the suggested system dependencies to the buildpack fixed the issues for me as well.

Thanks! These darn dependencies!


fix: created .snyk ignore policy file

created .snyk policy. Added policy to ignore vendor/ and .devcontainer/ directories.

@sheldonhull the .snyk file was added to the dsv-cli repo and Snyk no longer reports issues with the vendor/ directory code.
See report here: https://app.snyk.io/org/devops-secrets-vault/project/19dc579a-7e13-4b98-8a4f-5546aff44f4e
Adding this file to this repo looking for the same results.

this should help i think with local IDE tooling, but it won't stop the PR checks from flagging. They don't honor this file last I checked for opensource checks, license at least. Is this the goal?

This process worked very nicely for the gitlab repo as well. If we can resolve this and get it merged that would be very helpful.


Re-publish package

Sometimes you made a mistake in a package, but you don't want to increase the package id. For example, the icon was not working. So I tried re-uploading the package, but this is not possible.

Thanks for the issue report @GeertvanHorrik. For now, our guidance is to push a new version of the package and unlist the version that's problematic. We will likely allow you to replace an existing version in a future release of the gallery.

The reason we don't support republish is the same reason we don't allow delete right now. It could affect others who have taken a dependency on the package.
However, we could allow re-publish in cases where the only thing that's changing is metadata (other than dependencies). I'm guessing a significant # of republishes would be simply to correct metadata.

Exactly. And even then, it's the responsibility of the developer to let the end-users of the package care about the people that have taken a dependency on the package. For example, the package is always downloaded to the client and if someone is stupid enough to change the binaries and don't increment the version, so be it.
A good solution was the ability to edit the metadata on the website (nuget.org), but that is no longer possible. I think it is good to only allow the change of metadata (and not the content itself).

We removed that feature because it only changed the feed, not the package, and they should never be out of sync. The package must always be the authority.


Absolute References

Keep these an absolute reference to a version... so =0.6.12. This was based on feedback from Certik.
Originally posted by @crypto-shifu in https://github.com/omsfinance/oms/pull/8#discussion_r693583952

Issue Fixed


Drop miniheader from docs and styleguide

Coverage increased (+0.03%) to 78.163% when pulling 7286df478fd7f77b8d73b04a64511ecb72c6745d on eatonphil:drop-docs-miniheader into ca1090c6c505db6b226ffd6d5265619c34c2dc0f on linode:develop.

Coverage remained the same at 78.137% when pulling 8b7edb832dd5b242efab2a2942203c52a426f704 on eatonphil:drop-docs-miniheader into ca1090c6c505db6b226ffd6d5265619c34c2dc0f on linode:develop.

Coverage increased (+0.03%) to 78.163% when pulling 8b7edb832dd5b242efab2a2942203c52a426f704 on eatonphil:drop-docs-miniheader into ca1090c6c505db6b226ffd6d5265619c34c2dc0f on linode:develop.

Coverage remained the same at 78.137% when pulling c99089d3af1c07c185495e68fdb963232206067e on eatonphil:drop-docs-miniheader into ca1090c6c505db6b226ffd6d5265619c34c2dc0f on linode:develop.


Should numeric by itself be  a selectable type?

I believe numeric was to have columns that are floats and integers combined? So unless we don't want it anymore it should be selectable

it's now selectable :-)


[Bug] Auto advance mode not working as expected

Describe the bug
When auto advance mode is enabled, the app advance correctly, but if the user clicks on "Next Word Button" at the bottom center,

The auto-advances mode bugs completely, starts passing words wuickly :bug:
I think this is related to the auto advances timer, we need to reset timer on every manually advance.
To Reproduce
Steps to reproduce the behavior:

Go to any word URL, e.g: https://learning-english.vercel.app/words/animal/
Click on Settings icon at the top right corner.
Enable "Auto advances" mode.
Click multiples time on "Next Word"
See error

Expected behavior
Go forward without stopping, staying the same time at each word.

could I look at this

I can fix this issue, please assign it to me. Thanks!

I already created a PR #13 , please review it
And don't forget to assign this issue to me
Thanks!


Perfect number

Program to check whether the number entered by the user is a perfect number.

I want to start contributing with small programs since this is my first HacktoberFest. Hope that's alright.


Problems upgrading nixos

Describe the bug
I can't upgrade nixos due to the following errors:
[think@nixos:~]$ sudo nixos-rebuild switch --upgrade --show-trace
unpacking channels...
error:
       … while realising the context of a path

       at «string»:1:6:

            1| with import <nixpkgs/nixos> {}; config.system.build.nixos-rebuild
             |      ^

       error: failed to open archive: Unrecognized archive format
building Nix...
error: failed to open archive: Unrecognized archive format
error: failed to open archive: Unrecognized archive format
error: failed to open archive: Unrecognized archive format
/tmp/nixos-rebuild.lhpCjD/nix
error: failed to open archive: Unrecognized archive format
building the system configuration...
error: failed to open archive: Unrecognized archive format

Steps To Reproduce
Steps to reproduce the behavior:

Run sudo nixos-rebuild switch --upgrade

Expected behavior
The system gets upgraded
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[think@nixos:~]$ nix-shell -p nix-info --run "nix-info -m"
error: failed to open archive: Unrecognized archive format
(use '--show-trace' to show detailed location information)

This is the wrong repository for this issue, may I suggest to try https://github.com/nixos/nix/issues instead? =)
That'll be the correct place to look for help with what I suspect is your core issue, illustrated by your error messages.

NB: Also, make sure to add your configuration (your nix files), ideally a minimally-workable(read: reproducible)-example.
Final tip: Lots of people on matrix, discourse, and Discord might be willing to help, I think you'll get better response times there than here on Github =)


🐛 [V4][Android] Release crash: maxImages (6) has already been acquired

What's happening?
Hello 👋
I am using  VisionCamera with frame processors enabled with @ismaelmoreiraa/vision-camera-ocr:3.0.1.
It's a simple code that apply a Regex Validation out of a frame trough the FrameProcessor.
It's work perfectly in local / Debug (Expo 50) but when I build and use on Android (Release) the application crash with the trace below.
I saw this issue that was previously fixed apparently (it could be related maybe) -> https://github.com/mrousavy/react-native-vision-camera/pull/2352/commits/42ae1796ed63b7deb754470bff6c6e2e4e1e5df5
I've try a lot of things, like tweaking the version and all, before coming here and hope get some help about this.
I certainly did something wrong since I didn't saw similar new issues, any idea ?
Thanks in advance !
Reproduceable Code
export default function ScanScreen() {
    const device = useCameraDevice('back');
    // ...
    const ocr = useSharedValue({})
    const processing = useSharedValue(false)
    const parseOcr = useRunOnJS(() => {
        let matchFound = false;
        const scannedBlocks = ocr.value.result?.blocks;
        if (scannedBlocks?.length > 0) {
            scannedBlocks.forEach((block) => {
                block.lines.forEach((line) => {
                  // REGEX CHECK
                });
            });
        }

        if (!matchFound) {
            processing.value = false;
        }
    }, [])

    /** Process camera frames */
    const frameProcessor = useFrameProcessor(
        (frame) => {
            'worklet'
            if (processing.value === false) {
                processing.value = true;
                runAsync(frame, () => {
                    'worklet'
                    runAtTargetFps(10, () => {
                        'worklet'
                        const image = scanOCR(frame);
                        if (image) {
                            try {
                                ocr.value = image;
                                parseOcr();
                            } finally {
                                processing.value = false;
                            }
                        } else {
                            processing.value = false;
                        }
                    })
                });
            }
        },
        [],
    )

    const permissionFunction = async () => {
        ExpoCam.getCameraPermissionsAsync().then((res) => {
                if (!res.canAskAgain) {
                    Linking.openSettings();
                } else {
                    getPermission()
                }
            }
        )
    }

    return (

        <Camera
            style={{height: Dimensions.get('window').height + (Platform.OS !== "ios" ? StatusBar.currentHeight : 0)}}
            frameProcessor={frameProcessor}
            device={device}
            pixelFormat={"yuv"}
            isActive={true}
            photo={true}
            format={format}
        >
        </Camera>
    );
};

Relevant log output
Failed to acquire image.
java.lang.IllegalStateException: maxImages (6) has already been acquired, call #close before acquiring more.
	at android.media.ImageReader.acquireNextImage(ImageReader.java:560)
	at androidx.camera.core.AndroidImageReaderProxy.acquireNextImage(AndroidImageReaderProxy.java:88)
	at androidx.camera.core.SafeCloseImageReaderProxy.acquireNextImage(SafeCloseImageReaderProxy.java:88)
	at androidx.camera.core.ImageAnalysisBlockingAnalyzer.acquireImage(ImageAnalysisBlockingAnalyzer.java:41)
	at androidx.camera.core.ImageAnalysisAbstractAnalyzer.onImageAvailable(ImageAnalysisAbstractAnalyzer.java:128)
	at androidx.camera.core.SafeCloseImageReaderProxy.lambda$setOnImageAvailableListener$1$androidx-camera-core-SafeCloseImageReaderProxy(SafeCloseImageReaderProxy.java:213)
	at androidx.camera.core.SafeCloseImageReaderProxy$$ExternalSyntheticLambda1.onImageAvailable(Unknown Source:4)
	at androidx.camera.core.AndroidImageReaderProxy.lambda$setOnImageAvailableListener$0$androidx-camera-core-AndroidImageReaderProxy(AndroidImageReaderProxy.java:169)
	at androidx.camera.core.AndroidImageReaderProxy$$ExternalSyntheticLambda0.run(Unknown Source:4)

Camera Device
{
  "formats": [],
  "sensorOrientation": "landscape-left",
  "hardwareLevel": "limited",
  "maxZoom": 4,
  "minZoom": 1,
  "maxExposure": 20,
  "supportsLowLightBoost": false,
  "neutralZoom": 1,
  "physicalDevices": [
    "wide-angle-camera"
  ],
  "supportsFocus": true,
  "supportsRawCapture": false,
  "isMultiCam": false,
  "minFocusDistance": 0,
  "minExposure": -20,
  "name": "0 (BACK) androidx.camera.camera2",
  "hasFlash": true,
  "hasTorch": true,
  "position": "back",
  "id": "0"
}

Device
Galaxy A12
VisionCamera Version
4.0.5
Can you reproduce this issue in the VisionCamera Example app?
No, I cannot reproduce the issue in the Example app
Additional information

[X] I am using Expo
[X] I have enabled Frame Processors (react-native-worklets-core)
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

Finally, I did the STRATEGY_KEEP_ONLY_LATEST (https://github.com/mrousavy/react-native-vision-camera/issues/2826) modification, since I first wanted to prevent lags while using scanOCR of vision-camera-ocr. I still don't know what caused runAsync to create the Release crash (see above).
Anyway, I hope getting around this without patching the package with that last modification since it is apparently not a good idea but it's all I got :)

I am having same error, even though I am using runAsync inside the useSkiaFrameProcessor. On debug build works fine, but when comming to prod build, 9 out of 10 runs the app crashes with the logs same as in original post ^^^ . @subskill-anezzar did you solve this maybe somehow?


gl_vertexid

Hi. Does anyone have a simple example of how we can get a gl_vertexid type value through for our bgfx vertex shader? The how to do it seems to eluding us.
Cheers

More appropriate place to ask Q like these is here: https://gitter.im/bkaradzic/bgfx

Thanks


在训练模型的跑了一会的时候报错  ？

Traceback (most recent call last):
File "train_model.py", line 274, in
main()
File "train_model.py", line 269, in main
tm.train_cnn()  # 开始训练模型
File "train_model.py", line 174, in train_cnn
batch_x, batch_y = self.get_batch(i, size=self.train_batch_size)
File "train_model.py", line 106, in get_batch
batch_x[i, :] = image_array.flatten() / 255  # flatten 转为一维
ValueError: could not broadcast input array from shape (3000) into shape (3600)
You have mail in /var/spool/mail/root

我开始也报这个错，后来发现 是因为我在 sample\test目录放了一张和配置文件尺寸不一样的图片。移除掉就可以了


Running iterations causes perf tests to crash

Running adventure works tests with "benchmarks:runIterations=true" crashes with the xunit.console.exe runner.
System.AppDomainUnloadedException: Attempted to access an unloaded AppDomain.

/cc: @rowanmiller

Are you still seeing this? It sometimes happens when there is another exception occurring. The perf CI runs using the console runner and it's all green now.

I was seeing the error on CI, so it must be fixed now.


Add italian translations file for serrq

I was checking whether a 3rd party (non-contributor) could propose edits to my PR, but it doesn't seem possible. Soo not sure how to deal with this :D

Yeah Im not sure either. Lets ask and see what he says!


Break level system.transfer.list for android 5.x and 6.x

The new tool generates errors for older versions.
For example, Android 5.1 had to be level 2 instead has been compiled as a level 4 as attached files.
new system.transfer.list-->http://pastebin.com/g8hRTvn6
original system.transfer.list-->http://pastebin.com/1D7Ce6G4

Confirmed.
As img2sdat binary cannot determine which system you are trying to compress, I have to add the ability to choose the correct one, as a workaround.

Thanks :)


Inquiry about resolution

Hello, I have a question regarding the resolution settings in the ALVR Dashboard.
I noticed that there are six different resolution options available, and selecting each option changes the configuration for "Transcoding view resolution" and "Emulated headset view resolution."
I was wondering if ALVR takes into consideration the resolution of the HMD screen when choosing different resolution options. I am concerned that if the resolution ratio between the HMD screen and the "Transcoding view resolution" and "Emulated headset view resolution" configurations is inconsistent, it might result in a stretched image on the HMD. Could you please clarify if this is a potential issue?
Additionally, while going through the "https://github.com/alvr-org/ALVR/wiki/Settings-tutorial" link, I came across the recommendation to use the original resolution of the HMD for obtaining a clear image. In light of this, I was curious why the default setting does not directly configure the resolution on the ALVR server for the HMD.
Thank you for your attention to my inquiry, and I appreciate any insights you can provide on this matter.

"Transcoding view resolution" and "Emulated headset view resolution" have two modes: Absolute or Scale. using Absolute you can explicitly set width and height, where if this doesn't match the texture ratio used by the client, pixels will not be square; height can be set to Default, in which case the client aspect ratio will be used to calculate it. Scale mode instead uses the preferred client resolution as base and you can scale up or down from it. This resolution varies from vendor to vendor and usually it's very conservative, 1.0 will result in a blurry image. I believe information in the wiki link you posted in not accurate.

Thank you very much for your response! I have another unrelated question to ALVR. I noticed on the official Meta website's Oculus Quest 2 Tech specs that the single-eye resolution for Quest 2 is listed as 1832x1920. However, why does the "recommended" resolution queried from the OpenXR API show as 1440x1584? There seems to be a slight discrepancy in the aspect ratio between these two resolutions.

The aspect ratio is strongly dependent on the IPD and optics. And as I said the image is reprojected so, the image is displayed with a barrel distortion, the corners a squished. I've seen that the Quest 2 prefers extending the image a bit outside the screen too, so that the screen is better utilized. all these factors make image resolution and screen resolution (and aspect ratio) completely unrelated to each other.

Thank you for answering my questions and helping me understand better


Added documentation about updating calls

The library supports the ability to update existing calls but there is no mention of it in the docs. Added a small example as well as listing it on the coverage checklist.

LGTM 🤙 Thanks @AverageMarcus


Question: Does markdown-to-jsx/dist/esm.js file have an issue when minified?

Hi I'm using react-styleguidist which uses markdown-to-jsx to render the markdown files. I started getting an error in the console which says: Uncaught ReferenceError: returntrue is not defined  on opening the dev tools it looks like there is an issue here:

Is this an open issue?
Here's the version:
  "markdown-to-jsx": {             "version": "6.8.1",             "resolved": "resolved url",             "integrity": "SHA token",             "dev": true,             "requires": {                 "prop-types": "^15.6.2",                 "unquote": "^1.1.0"             }         },
Thank you in advance for taking the time to look at this. Apologize in advance if I haven't followed a bug opening template. I'd be more than happy to reopen this in the correct template.
Regards,
Nishant

Hmm I just started using Babel minify, maybe there's a bug in the minifier?
On Mon, Oct 22, 2018, 6:06 PM NishantShenoy<EMAIL_ADDRESS>wrote:

Hi I'm using react-styleguidist which uses markdown-to-jsx to render the
markdown files. I started getting an error in the console which says: Uncaught
ReferenceError: returntrue is not defined on opening the dev tools it
looks like there is an issue here:
[image: image]
https://user-images.githubusercontent.com/15218556/47324320-24ce7400-d614-11e8-8e7c-106ee7d72287.png
Is this an open issue?
Here's the version:
"markdown-to-jsx": { "version": "6.8.1", "resolved": "resolved url",
"integrity": "SHA token", "dev": true, "requires": { "prop-types":
"^15.6.2", "unquote": "^1.1.0" } },
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/probablyup/markdown-to-jsx/issues/224, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAiy1t-1G_7dZNh1PHiWfArcQJoN-20aks5unk-SgaJpZM4X0XSL
.

Here you go : https://github.com/styleguidist/react-styleguidist/issues/1181 . Maybe it's an issue with the UglifyJSPlugin 😄  thank you! closing this.

Ah I see. So in esm.js the relevant code is:
return'production'!==process.env.NODE_ENV

And the bundler is likely replacing process.env.NODE_ENV with 'development'.
And then perhaps the minifier is running a second time and since 'production' !== 'development' is true, it's replacing it without adding a necessary space back?

That sounds right, but UglifyJS hasn't had an any change made to it although webpack has. I haven't had the change to look into it closely.

I'm also getting this error after upgrading this library.

I think this plugin needs to be disabled from the preset:
https://github.com/babel/minify/tree/master/packages/babel-plugin-minify-flip-comparisons
On Tue, Oct 23, 2018, 1:54 AM Haz<EMAIL_ADDRESS>wrote:

I'm also getting this error after upgrading this library.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/probablyup/markdown-to-jsx/issues/224#issuecomment-432117618,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAiy1vh9Z79mPxVPRZbUhrRWpS1prha7ks5unr0OgaJpZM4X0XSL
.

@NishantShenoy The issue is probably not directly with UglifyJSPlugin, as it's present also in version 8.0.0 of Styleguidist, which uses TerserPlugin instead.
I'm not sure what the original source of this problem is, but as pinning markdown-to-jsx to 6.7.4 fixed it for me, I'd recommend re-opening this issue.

Hello, found this from Styleguidist. I believe the problem here is that, in the minified source, there is a statement return'production'==process.env.NODE_ENV, which is interpolated (I think by Webpack) into returntrue or returnfalse (the equality check here is being replaced inline with its static evaluation).
This seems to be combination of placing the string at the start of the conditional, and removing spaces wherever possible.

Try 6.8.2

Just in time! Ran into this last night. @probablyup Thanks for the quick turnaround.


password reset flow

this pr is related to https://github.com/brendt/rfc-vote/issues/26
I've utilized Jetstream's existing offerings in this term by simply connecting its components instead of building a custom flow from scratch.

Thanks!


BoboIndexReader.IsCurrent()

@NightOwl888
Could you please check the method BoboIndexReader.IsCurrent()? It does not work to me because it throws exception.

The IsCurrent method is inherited from Lucene.Net.Index.FilterIndexReader, whose implementation looks like this:
        public override bool IsCurrent()
        {
            base.EnsureOpen();
            return this.in_Renamed.IsCurrent();
        }

BoboIndexReader does not override IsCurrent.
The in_Renamed variable is of type Lucene.Net.Index.IndexReader whose IsCurrent implementation looks like:
        public virtual bool IsCurrent()
        {
            throw new NotSupportedException("This reader does not support this method.");
        }

I checked the Lucene (Java) source and it is implemented the same way.
In other words, the exact behavior depends on the type of IndexReader being passed to the constructor of BoboIndexReader and it must be overridden by the specialized IndexReader in order to function.
So, the question is, what type of IndexReader are you passing to the BoboIndexReader? Is it open?
The expected behavior for each type of IndexReader is defined here.


Extended first writer wins logic for HasConflict

This is based on the scenario described in Slack last night that @tli2 found with his large scale randomized concurrency control tests.
I extended the logic for HasConflict(), and added a test case for the scenario. I also had to make a change in the concurrent DataTable test that was violating this new constraint.
I also fixed an unrelated comment in DeltaRecord that @yangjuns asked me about this morning.

Coverage remained the same at 96.83% when pulling b22d4fe89af1caf28b1b87c2fad69b9b98e9e68a on mbutrovich:mvcc_test into feffd6ca0292f6b66fc8e4df52d5780f33631603 on cmu-db:master.


Change the default bootstrap theme colors

Discussed in https://github.com/hfhbd/bootstrap-compose/discussions/186

Originally posted by luca992 May 13, 2022
Just wondering, what would be the best way to change the default theme colors that are defined by bootstrap?
Found instructions for a normal project.
https://stackoverflow.com/questions/38792005/how-to-change-the-bootstrap-primary-color
But, I'm not sure how I can translate that to a kotlin project.

[ ] Readme
[ ] Provide a StyleSheet class

Work great, thanks! 👍


ERR_CONTENT_DECODING_FAILED

Hi everyone,
i'm proxying a request with your awesome lib but in my chrome console in the network tab I receive this error "ERR_CONTENT_DECODING_FAILED".
This is a simple snippet that show my operation
var http = require('http'),
request = require('request')

var httpServer = http.createServer(function (req, res) {

	var opt = {}
	opt.url = 'https://api.sendgrid.com/v3/campaigns'
	opt.auth = {}
	opt.auth.bearer = 'mytoken'
	req.pipe(request(opt)).pipe(res)

}).listen(4000, function () {
	console.log('Starting HTTP server on port: 4000');
});

This is the output of your lib with debug mode enabled:
REQUEST { url: 'https://api.sendgrid.com/v3/campaigns',
  auth: { bearer: 'myToken },
  callback: undefined }
REQUEST make request https://api.sendgrid.com/v3/campaigns
REQUEST onRequestResponse https://api.sendgrid.com/v3/campaigns 200 { server: 'nginx',
  date: 'Mon, 30 Nov 2015 10:47:58 GMT',
  'content-type': 'application/json; charset=utf-8',
  'transfer-encoding': 'chunked',
  connection: 'keep-alive',
  status: '200 OK',
  'x-request-id': '34953f9e-888f-4449-a53a-d371ff68fcfb',
  'x-xss-protection': '1; mode=block',
  'x-content-type-options': 'nosniff',
  'content-encoding': 'gzip',
  'strict-transport-security': 'max-age=31536000',
  'x-runtime': '0.111581',
  'x-ua-compatible': 'chrome=1',
  'cache-control': 'max-age=0, private, must-revalidate',
  'x-frame-options': 'SAMEORIGIN',
  'access-control-allow-methods': 'HEAD, GET, PUT, POST, DELETE, OPTIONS, PATCH',
  'access-control-max-age': '21600',
  'access-control-expose-headers': 'Link',
  'access-control-allow-origin': '*',
  'access-control-allow-headers': 'AUTHORIZATION, Content-Type, On-behalf-of, x-sg-elas-acl',
  'content-security-policy': 'default-src https://api.sendgrid.com; frame-src \'none\'; object-src \'none\'',
  'x-ratelimit-remaining': '598',
  'x-ratelimit-limit': '600',
  'x-ratelimit-reset': '1448880480',
  'powered-by': 'Mako' }
REQUEST finish init function https://api.sendgrid.com/v3/campaigns
REQUEST response end https://api.sendgrid.com/v3/campaigns 200 { server: 'nginx',
  date: 'Mon, 30 Nov 2015 10:47:58 GMT',
  'content-type': 'application/json; charset=utf-8',
  'transfer-encoding': 'chunked',
  connection: 'keep-alive',
  status: '200 OK',
  'x-request-id': '34953f9e-888f-4449-a53a-d371ff68fcfb',
  'x-xss-protection': '1; mode=block',
  'x-content-type-options': 'nosniff',
  'content-encoding': 'gzip',
  'strict-transport-security': 'max-age=31536000',
  'x-runtime': '0.111581',
  'x-ua-compatible': 'chrome=1',
  'cache-control': 'max-age=0, private, must-revalidate',
  'x-frame-options': 'SAMEORIGIN',
  'access-control-allow-methods': 'HEAD, GET, PUT, POST, DELETE, OPTIONS, PATCH',
  'access-control-max-age': '21600',
  'access-control-expose-headers': 'Link',
  'access-control-allow-origin': '*',
  'access-control-allow-headers': 'AUTHORIZATION, Content-Type, On-behalf-of, x-sg-elas-acl',
  'content-security-policy': 'default-src https://api.sendgrid.com; frame-src \'none\'; object-src \'none\'',
  'x-ratelimit-remaining': '598',
  'x-ratelimit-limit': '600',
  'x-ratelimit-reset': '1448880480',
  'powered-by': 'Mako' }

What you think about?
Mirko

@mikeal and @simov sorry for the tag but i need to solve the error soon, any ideas?

Hi, @mirkods, did you tried to search for ERR_CONTENT_DECODING_FAILED? There are some pretty good suggestions for that. Try with adding encoding:null to your options object.

@simov i tried to search the ERR_CONTENT_DECODING_FAILED  on the web and seems to be an error caused from gzip.
Your solution not works in this case.

@simov I noticed that if i remove the content-encoding header all work fine.
var http = require('http'),
request = require('request')

var httpServer = http.createServer(function (req, res) {

	var opt = {}
	opt.url = 'https://api.sendgrid.com/v3/campaigns'
	opt.auth = {}
	opt.auth.bearer = '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'
	req.pipe(request(opt))
	.on('response', function(resp){
		delete resp.headers['content-encoding']
	})
	.pipe(res)

}).listen(4000, function () {
	console.log('Starting HTTP server on port: 4000');
});

Do you think is a Sendgrid problem?
Mirko

That's a good fix actually. No idea what's wrong with Sendgrid.

Thanks for bringing this to our attention, we added this to our next sprint and will get it fixed for you.

Thanks @mbernier let me know when the fix has been completed :+1:

Ehi @mbernier has this error been fixed?


Broke rembg

Hoping to get some insight here - I had rembg running fine, I installed this and it broke rembg.
I have unistalled everything but simply can not get standard rembg back up and running.
Would love if I could get some insight into what may have happened?

I've had problems running rembg-greenscreen and reverted to rembg for video processing. Check the dependencies list to get rembg working as rembg-greenscreen has different requirements.

Thanks for the reply. Where is the dependency list sorry for asking.
Really wish I could go back - feel like I literally have to buy a new computer at this stage.
I have uninstalled and re-installed everything - from Python to Visual studio.
Really frustrating.


Bug smashing Friday - 2022-01-28

The goal of this event is to go trough the Flatcar issues and try to:

check if they are still relevant
add steps to reproduce the issue
add the correct labels
ask for more information
...

Let's kick-off the first bug smashing of the year.

[ ] tweet announcement @miao0miao
[x] add event to the Flatcar Community Calendar @t-lo
[ ] create a GitHub discussion to track the on-going discussions during the event
[ ] 1h before meeting: announce in matrix (w/ Zoom link) @miao0miao
[ ] add the YouTube link to the event
[ ] conduct the event @flatcar-linux/flatcar-maintainers

Zoom link: https://us06web.zoom.us/j/89332204295?pwd=WXBMTndKNjh6SE1GR1Q3dGp2MzdMUT09
Meeting ID: 893 3220 4295
Passcode: 99999

should we aim for some specific time for the streaming? (I want to add that info in a tweet)

The calendar slot on Friday is from 9am to 6pm CET, happy to adjust if that's not right.

I created a link tree:
https://flow.page/flatcarbugsmashingday

We closed around 39 issues (40 with this one) and updated more than 55. Thanks everyone !


belongsTo making wrong guessing on relation name

I have two classes,  User and Document, mapped to user and document tables. In Document, i have their relationship defined like this:
public function registeredBy() {
		return $this->belongsTo('User', 'registered_by_id', 'id');
	}

When I run this, as I'm not giving the belongsTo method the last parameter (relation), it tries to guess the relation name from the method name (registeredBy). So, it is assuming that the table on the other side of the relation is called registeredBy.
Wouldn't it be appropriate for it to use the relation name from the model that I'm linking to? I mean, I'm telling Laravel to use the User class, so it should have guessed that the relation name would be user, and not registeredBy.
I know I can quickly fix this by giving the belongsTo call it's last parameter (relation), but maybe this is a symptom of something wrong in the way relation names are guessed.

That's not really the case I explained. I'll work on my issue some morre.
Sorry for posting.


get contacts in messaging-api-telegram

how to get contacts in module messaging-api-telegram?
I see api have get contacst: https://core.telegram.org/method/contacts.getContacts
Thanks

Hi @nguyenthanhtruongntt85, unfortunately contacts.getContacts is not part of Telegram Bot API, so we are not gonna support it in  messaging-api-telegram.
To use it, you must learn how to execute tdlib/td C functions in your Nodejs environment.
This Stackoverflow question may help.


Change default order column name

Hello,
The default order column of this package is sort_order whereas spatie/eloquent-sortable uses order_column.
The same order column name would allow both packages to work without any configuration for simple use case.
As it would be a BC, would you be open to a PR to change the default order column name to "order_column"?
Thanks for your work,
Best,
Karel

Hi!
As the user has to create a migration anyway, it's not going to be zero configuration either way. To me, it seems weird to name a column as a _column, so I think I'll keep it as sort_order for now.

Hi,
I understand, I also think _column is weird.
Thanks


Does not read templates

I wanted to test this, so I made the ExampleTemplate in 000 - Landing Page. Inbox. KO. Meta/_Templates/testtemplate and put that in settings

but it does not show up in the command palette

i tried it with the root slash too, so /000 - Landing Page. Inbox. KO. Meta/_Templates/testtemplate

Two possibilities:

it looks like you have an old version - the current version should be listing the number of templates it has found in the directory, or giving an error
did you reload Obsidian after changing the template directory? Unfortunately, this is currently necessary (although hopefully can be changed in a future version)

Have a try with the new version I've just pushed - that's changed a lot of things, and this should be a bit smoother.


19 implement localization for console output [WIP]

This PR implements the necessary functionality to facilitate localization of  the CLI and closes #19

CLI output when using quiet logs and "Spanish"

CLI output when use verbose logs and "Spanish"


⚠ Bofrost.at has degraded performance

In 9df8c4b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1553 ms

Resolved: Bofrost.at performance has improved in e29137c after 6 minutes.


Data request License is under edit mode !!!! (dangerous issue)

Hello, I have a question regarding the license.doc: it appears to be a contract between two parties. It seems that someone has directly modified the original document with their personal information. I would suggest setting the editing permission to 'read-only' to prevent further alterations. Thank you for your assistance
Current version

The original version

thank you for your comments very much


weather.get_forecasts based template sensor returns bad timedate

System Health details
skip
Checklist

[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of any previous issues..

Describe the issue
When creating a template sensor to hold the whole forecast in an attribute, it ends up with a broken json.
It works for OpenWeatherMap and Accuweather
Reproduction steps

Configure the MetService and OpenWeatherMap integrations
Create a trigger-based template to extract the forecast from these entities into an attribute:
`  - trigger:

platform: time_pattern
hours: "/1"
platform: homeassistant
event: start
platform: event
event_type: event_template_reloaded
action:
service: weather.get_forecasts
data:
type: hourly
target:
entity_id: weather.openweathermap
response_variable: hourly
sensor:
name: OpenWeatherMap Forecast Hourly
unique_id: OpenWeatherMap_forecast_hourly
state: "{{ now().isoformat() }}"
attributes:
forecast: "{{ hourly['weather.openweathermap'].forecast }}"`

  - trigger:       - platform: time_pattern         hours: "/1"       - platform: homeassistant         event: start       - platform: event         event_type: event_template_reloaded     action:       - service: weather.get_forecasts         data:           type: hourly         target:           entity_id: weather.the_villa_metservice         response_variable: hourly     sensor:       - name: MetService Weather Forecast Hourly         unique_id: metservice_forecast_hourly         state: "{{ now().isoformat() }}"         attributes:           forecast: "{{ hourly['weather.the_villa_metservice'].forecast }}"

Check the attributes in the Developer Tools/States:

...
Debug logs
No response
Diagnostics dump
No response

Resolved in #34, released in v0.4.1 - thanks for highlighting!


Business Studio Homepage screenshot update required

Open https://developers.sap.com/tutorials/appstudio-devspace-fiori-create.html
Step 3: Open the SAP Fiori dev space
Expected

Actual

Hi @Mukesh-50 ,
Thanks for bringing this to my attention.
I update the tutorial whenever there's a major change, i.e. a change that has high probability to create confusion.
This case seems to be of a minor change, and therefore, will be included in the next update.
Regards - Raz

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Desired downstream (RPM) package name for lxi-gui?

So far lxi-tools (the command-line tool /usr/bin/lxi) are packaged as lxi-tools RPM for Fedora and EPEL. Given the software requirements for the /usr/bin/lxi-gui tool in lxi-tools v2.1 are meanwhile satisfied by latest Fedora releases, I wonder what the desired downstream (RPM) package name for lxi-gui would be - e.g. lxi-tools-gui or simply just lxi-gui? I would like to avoid putting /usr/bin/lxi-gui into the existing lxi-tools RPM, given it depends on much more (graphical) libraries that maybe not everybody would like to install when just using the command-line tool /usr/bin/lxi.
And: Are you aware about any distribution/downstream having /usr/bin/lxi-gui already packaged?

The lxi-tools project includes both the lxi and lxi-gui tool and for that reason I think it best that any lxi-tools package includes both too when possible.
I don't think going down the path of creating a separate lxi-tools-gui or lxi-gui package is a good idea, not even if it saves the command-line user from installing the GTK4 libraries because most users will likely already have GTK4 installed one way or the other because many other applications depend on them.
I think AUR have lxi-gui packaged in their lxi-tools-git package.
FYI - in the next version of lxi-tools I'm going to enable build of the lxi-gui application by default because most distribution now ship all the required GTK4 etc. dependencies.

I don't think going down the path of creating a separate lxi-tools-gui or lxi-gui package is a good idea, not even if it saves the command-line user from installing the GTK4 libraries because most users will likely already have GTK4 installed one way or the other because many other applications depend on them.

Okay…then I'm putting /usr/bin/lxi and /usr/bin/lxi-gui into the same RPM package.

FYI - in the next version of lxi-tools I'm going to enable build of the lxi-gui application by default because most distributions now ship all the required GTK4 etc. dependencies.

For the record: Fedora 36, released on 2022-05-10, is the very first Fedora version that matches the quite high requirements.
May I assume that it's still possible to disable /usr/bin/lxi-gui during build-time in future versions to still cover older Fedora and EPEL releases with the command-line application only?

For the record: Fedora 36, released on 2022-05-10, is the very first Fedora version that matches the quite high requirements.

Yes, the requirements are quite high as lxi-gui uses all the latest bells ans whistles that GTK4 and libadwaita offers but at least now the various distributions are catching up with GTK4 etc. so it should be easy for most to start including lxi-gui and that is why I'm going to enable it by default.

May I assume that it's still possible to disable /usr/bin/lxi-gui during build-time in future versions to still cover older Fedora and EPEL releases with the command-line application only?

Yes, for those environments that can't fulfill the build requirements there is a meson build option to disable installation of lxi-gui. It is described here. In short, you just add -Dgui=false to the meson setup line to disable build of the lxi-gui application.

Thank you for the clarification.
For Fedora 36 and the upcoming Fedora 37, FEDORA-2022-d3250d3d9c and FEDORA-2022-492ab6fc04 provide now lxi-tools with /usr/bin/lxi-gui; feedback is appreciated.

Great - thanks for including the lxi-gui application.

Sadly commit 8a6abce234809ff4d83e11f6a07016d56b9e8388 reduces this now to Fedora 37 only, as Fedora 36 ships 1.1.4.

I know but I really want to keep moving lxi-tools forward according to Gnome HIG guidelines and libadwaita so no v2.2 gui app in F36 is fine with me.


Support for reorged logs in eth_Logs Subscription

Addressing issue #5677
Updated EthFilter to create new task update_reorged_logs that is spawned when the EthFilter is initialized. The new task reacts to reorged blocks taking a clone of EthPubSub, checks impacted log filters and stores the reorged logs in the log filter in a new field called reorged_logs.

Please, could you review @mattsse ? Have made the requested changes


Adding a few more documentation tools

Adding the following things;

Cool netlistsvg tool
A couple of useful sphinx plugins.
Undulate

Nice! Please fix links to point to repos.


🍒 Cherry pick PR #15994 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/15994 to staging 🍒

🚀 Cherry-picked to staging by https://github.com/OSBotify in version: 1.2.85-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/Julesssss in version: 1.2.85-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Fallback to SimpleLogger if TermLogger init fails

This fixes logging init when llvmenv is executed in docker

Thanks!


rename keyowrd -> keyword

I was skimming through code in this repo, this just hit my eye and assuming maybe map-key-to-keyowrd was typo. Discard otherwise, hehe.
I did not run tests or anything, just replaced occurrences of this name in file and checked other files in repo weren't using the old one.

@jasalt
That is exactly what you have pointed out.
Thank you very much!

@Chemaclass
Thanks for the support.
I needed to do assignments and reviews before I could merge.


Add git as Debian build dependency

Our version string generation (which happens at compile time) relies on git. So we need git as debian build time dependency.
Test: "glinux-build ."

Note, this also marks the "find program git" as required, making the step fail if "git" is not available.


no response from TV

Hi, when using this commad:
samsungctl --host <IP_ADDRESS> --port 8001 KEY_MUTE --method websocket --name "home-assistant" --id<PHONE_NUMBER> --timeout 30 -v
the output is : Sending control command: KEY_MUTE
the TV dosent ask me to accept it as a remote, any ideas?

Having the same issue with my 48UE6300..
no control, no errors, and no request for to accept the connection on the TV..

not working on UN60JS6900.

Please see #22.