try to sniff content-type as long as the data is not flushed

When caddy log is enabled, it can be observed Content-Type sniffing no longer works for http3, but works for h1/2. This is caused because when logging is enabled, caddy wraps all response writers with responseRecorder which will call WriteHeader first.
Stdlib will try set Content-Type header in the same logic, for http1, *chunkWriter is wrapped by *response to write the actual data. http2 will set this header in the same way.
Because this header is set once, it won't be overwritten automatically

@WeidiDeng The large refactor for HTTP Datagram support (#4452) was merged. Do you want to update this PR?

This PR is the only thing left for the v0.43 release.

Do I understand correctly that the only thing that this PR changes is that the sniffing algorithm is now also run when flushing, whereas before it was only run on Write?

@WeidiDeng Can you confirm this?

@marten-seemann Yes, I confirm that's the change.


auth_time doesn't update when token is refreshed

Not sure if I'm doing something wrong or this is by design, but I would have thought that auth_time should be automatically set to the current time when refresh_token() successfully generates a new access_token.
Otherwise, this if statement would always be executed after a certain amount of time until a user logs in again:
from social_django.utils import load_strategy
import time

social = user.social_auth.get(provider='google-oauth2')
if (social.extra_data['auth_time'] + social.extra_data['expires']) <= int(time.time()):
    strategy = load_strategy()
    social.refresh_token(strategy)

Running v1.0.1

This is still an issue as of 1.1.0

Fixed by https://github.com/python-social-auth/social-core/commit/feeb27c5d2bdf94428d09f289342cd64a679df62.


Predict depends on batch size

Hi there! During experiments with Keras I have faced with the following problem. When I use any type of RNN layers, method model.predict(X, batch_size=b_size) gives different answers with different b_size values. Furthermore the bigger the b_size value then better the results. It is very strange, because, obviously, if you don't use stateful RNN then output should not depend on the b_size.
I wonder if anyone else has stumbled onto that. Thanks for help!

And there's no batch normalization in the network?

@keunwoochoi, thanks for comment. Yes, n. network has few batch normalization layers with mode=2. So, I'd like to clarify: during the inference batch normalization layers corrects outputs of respective neurons depending on input data, therefore results may be different with different batch size?

Derp. Obviously the results will depend on your batch size with mode=2! If the batch size is too small, the mean and variance for any particular batch will probably not be the same as the full dataset's very often. Increase the batch size or don't use batch normalization in test time.


Issue whith whale_watch.py

I'm trying to use the whale_watch.py. I got the following error mesage when running it:
File "/kingdoms/rce/workspace8/Tranber/bulkvis-master/utils/whale_watch.py", line 294, in
main()
File "/kingdoms/rce/workspace8/Tranber/bulkvis-master/utils/whale_watch.py", line 11, in main
fused_df, ss, fused_read_ids = fuse_reads(args.summary, args.paf, args.distance, args.top, args.alt, args.debug)
File "/kingdoms/rce/workspace8/Tranber/bulkvis-master/utils/whale_watch.py", line 106, in fuse_reads
df2['cat_read_id'] = df2['all_but_last'] + "|" + df2['last_read_id']
File "/usr/local/lib/python3.5/dist-packages/pandas/core/ops.py", line 1069, in wrapper
result = safe_na_op(lvalues, rvalues)
File "/usr/local/lib/python3.5/dist-packages/pandas/core/ops.py", line 1033, in safe_na_op
return na_op(lvalues, rvalues)
File "/usr/local/lib/python3.5/dist-packages/pandas/core/ops.py", line 1023, in na_op
result[mask] = op(x[mask], y)
numpy.core._exceptions.UFuncTypeError: ufunc 'add' did not contain a loop with signature matching types (dtype('<U32'), dtype('<U32')) -> dtype('<U32')
If I understand properly this line causes the problem : df2['cat_read_id'] = df2['all_but_last'] + "|" + df2['last_read_id']. But I don't find a way to make it work. Can it be that my summary file is from a multiplexed run ?

It seems that the read id fields are numeric when they should be strings. Could you share your sequencing summary and paf file?

Here is a subset, files are heavy. Thanks for your help.
summary.txt
DQ.paf.txt

Found the origin of the problem ! read_ids from the summary and paf didn't have the same format. Once the paf reformated, it worked !

Excellent, well done for tracking it down! I'll try to add some checks in badly formatted inputs.


Dead link

The link on the page:
Separation of Duties
Is invalid
I am looking for someone to clearly state "Do not put every object for all of your development/test/production environments in dbo".  Why can't I find this clearly stated?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 1a38dead-4cce-4bef-ec9e-89c46429bd64
Version Independent ID: 92895c98-a246-af54-1ca7-33544f3bbcb9
Content: Playbook for addressing common security requirements - Azure SQL Database & Azure SQL Managed Instance
Content Source: azure-sql/database/security-best-practice.md
Service: sql-db-mi
Sub-service: security
GitHub Login: @VanMSFT
Microsoft Alias: vanto

Hi @acybppres ,
Thanks for sharing this feedback. Request will be assigned to author to further investigate and update as appropriate.
Hi @VanMSFT,
Could you please look into this document request. Thanks!

Hi @acybppres - Thanks for pointing out the invalid link. I'll make the updates to the doc. The download can be found through this link:
https://learn.microsoft.com/en-us/previous-versions/sql/sql-server-2008/cc974525(v=sql.100)
More comprehensive wording about SoD and dbo can be found here: https://learn.microsoft.com/en-us/previous-versions/sql/sql-server-2008/cc974525(v=sql.100)


[WIP] Allow ${...} syntax return objects and arrays

Purpose of this PR is (was) implementing "Idea 3" of https://github.com/treasure-data/digdag/issues/25#issuecomment-203236773 to allow key: ${foos} syntax to put array (or objects) as a value of key.
However, I found a problem that Config.get("key", String.class) fails if value is an array or object.
This means that following code fails:
_export:
  key:
    nested: v

+t:
  echo>: ${key}  # this is an object

But this works:
_export:
  key:
    nested: v

+t:
  echo>: foo${key}bar  # this is a string

I think this behavior is unexpected.
So, "Idea 4" could be better.

This didn't work. I created #328 instead.


Create app.config file if it's missing and binding redirects requested

Extracted from #806
I think create app.config for every project that has its corresponding references file would do the job
/cc @isaacabraham

Does packet work in ASP.NET 4.x projects as well? What about web.config?

I'm on this one now.

I've done the main work of creating an app.config and associated BRs in it - is there an easy example somewhere of how to add a file reference to a cs/fs/vb proj file somewhere in the source?

Hm... but I don't need app.config for my class libraries. Actually I need it only for assemblies with entry point and for tests assemblies.
Furthermore, for FSharp projects app.config should be included with Build Action = Content otherwise it will cause unwanted rebuilds of this project.

@isaac_abraham could you please look into this?

How do we identify unit test projects? This was actually the differ behind this change I.e. Allow unit tests to run properly.
You're right about content, that should have been done originally.

Hmm, just checked - newly added config files are added as Content files.

How do we identify unit test projects?

I added myself app.config for all project where it is needed and I just don't want that paket do it for me.

+1 for paket not adding app.config files.
In my setup, config files are generated during the build (from some other config files) and copied to app.config. Therefore app.config is always a derived thing and is not even added to my *.csproj files.
I'd go mad if running paket would recreate and readd this file to my project.
So if you consider doing so, please make it an optional feature.

it's aöready optional.


Correction for raw affiliation MERQ Consultancy PLC, Addis Ababa, Ethiopia; Research Unit, Addis Ababa, Ethiopia; Research, MERQ PLC LLC, Baltimore, Maryland, USA

Correction needed for raw affiliation MERQ Consultancy PLC, Addis Ababa, Ethiopia; Research Unit, Addis Ababa, Ethiopia; Research, MERQ PLC LLC, Baltimore, Maryland, USA
raw_affiliation_name: MERQ Consultancy PLC, Addis Ababa, Ethiopia; Research Unit, Addis Ababa, Ethiopia; Research, MERQ PLC LLC, Baltimore, Maryland, USA
new_rors:
previous_rors: 04kb0nk47
works_examples: W4289444064
contact: 68aad99ce1c72359ffda64f23b5c8c8c:04560ed791f8bb @ air.org

This issue was accepted and ingested by the OpenAlex team on 2024-11-01. The new affiliations should be visible within the next 7 days.


Typo in the code samples to update ACR replication

Hi,
In both code samples at the bottom of the page, it says
az acr replication update update ...
instead of
az acr replication update ...

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8006f0b0-79c7-c5c4-7019-b5b369dec275
Version Independent ID: 72f7cf6a-541d-6a7f-d7e7-d5b564d0be7f
Content: Geo-replicate a registry - Azure Container Registry
Content Source: articles/container-registry/container-registry-geo-replication.md
Service: container-registry
GitHub Login: @SteveLas
Microsoft Alias: stevelas

Thanks for bringing this to our attention. I have created a PR with the update, changes will be live shortly.
Thanks for improving our documentation! Just so you know, it's also possible to make these changes yourself, if you are interested. Doing so will improve our documentation, and it will add you as a contributor to the document.
In case you're interested, these are the steps to do so:

Navigate to the page of interest
Select the "Edit" button at the top right of the page
Select the "Pencil" icon on the right
Modify the text within the text box in markdown format - more info: https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet
Name your file change and provide a description, if necessary
Select "Propose file change"
Select "Create pull request" on the 'Comparing changes' page
Select "Create pull request" on the 'Open a pull request' page
You can find a GIF that demonstrates the end-to-end process for submitting changes here: https://aka.ms/editsqldocs.

@Karishma-Tiwari-MSFT I should have thought about it! Next time I will do it, I promise 🙃


server-with-worker make target doesn't work

Bug Description
make server-with-worker fails
Steps to Reproduce

see above ^

Expected behavior
The webserver should spin up and be able to utilize the worker
Environment

Version: main (post-1.6.3)
OS: Windows
Python Version:
Docker Version:

no longer relevant


KAFKA-13455: Add steps to run Kafka Connect to quickstart

Signed-off-by: Katherine Stanley<EMAIL_ADDRESS>

@katheris Let's also update the website, otherwise we'll have to wait for the next release. Can you open a similar PR on https://github.com/apache/kafka-site? Thanks


Feature/node checker remove failed nodes

Hi @kramer I fixed up @bmlining's PR, hopefully this works for everyone

Coverage increased (+0.2%) to 53.176% when pulling bf66b8d9f5194d5708c0226d27311f7a2195aa30 on lithiumtech:feature/node_checker_remove_failed_nodes into bd57835f00547ccd1990278d409cd3187b45cc6d on searchbox-io:master.

Looks good, even better approach than I was thinking of.

Merging this would close #329 and #345

NodeChecker is not the only one calling execute therefore losing the cause-exception does not seem like a good idea to me, I would prefer something like below:
    public CouldNotConnectException(String host, Throwable cause) {
        super("Could not connect to " + host, cause);
        this.host = host;
    }

So people still have the details when debugging/researching an odd connection exception.

To keep the behaviour of HttpHostConnectException being "converted" to CouldNotConnectException same also for async execution maybe DefaultCallback.failed should also be modified accordingly.

Could you also please adapt JestDroidClient accordingly?

Coverage increased (+1.5%) to 54.491% when pulling fa38f277172ebaabe788146d895ba0fa3ff96fee on lithiumtech:feature/node_checker_remove_failed_nodes into bd57835f00547ccd1990278d409cd3187b45cc6d on searchbox-io:master.

Coverage increased (+1.5%) to 54.491% when pulling 9df4434b9ed162df85ff4a6c048d75d7575d8e3c on lithiumtech:feature/node_checker_remove_failed_nodes into bd57835f00547ccd1990278d409cd3187b45cc6d on searchbox-io:master.

Thank you for your contribution @bdharrington7 and thank you for the initial effort @bmlining !


Sequencer should run init-chain with only a genesis state and no other inputs

The Sequencer ABCI consensus service is currently passing in a chain ID and initial validator set in addition to the genesis state.
The genesis state should be a closed collection of settings needed to initialize a new chain without extra external input.
┆Issue Number: ENG-691

the chain ID and initial validator set are from the genesis file and passed in to the app in the request::InitChain, but are set by cometbft. the genesis state is the initial application state. there's no reason why we should disregard parameters passed in the InitiChain request.

This issue is stale because it has been open 45 days with no activity. Remove stale label or this issue
be closed in 7 days.

This issue was closed because it was stale


Add FFT functions

I'm implementing FFT functions using cuFFT library.

Thank you for working on this. It will be nice to have FFT support.

jenkins, test this plsae.

jenkins, test this please.

It seems that I cannot run the CuPy examples using this PR. For example, cupy\examples\gemm\sgemm.py fails with:
ImportError: cannot import name 'fft'

I fixed ImportError and other bugs.
All checks have passed.

jenkins, test this please.

LGTM!


A Star function now added to graph functions , develop PR

This PR created based on #6002 PR .

This PR contains about 800 files changes. Could you please rebase your branch with develop and update the PR?

Hi @lvca
yes , i'm wondered about file changes in git :(
i guess i have problem with git in my pc now  .
so i rebase my develop_st branch and update it again .
Thanks
Saeed


starlark: add 'bytes' data type, for binary strings

This change defines a 'bytes' data type, an immutable string of
bytes. In this Go implementation of Starlark, ordinary strings
are also strings of bytes, so the behavior of the two is very similar.
However, that is not required by the spec. Other implementations of
Starlark, notably in Java, may use strings of UTF-16 codes for the
ordinary string type, and thus need a distinct type for byte strings.

Hi Jon, Jay,
I have updated this change to the Go impl, and the spec change in https://github.com/bazelbuild/starlark/pull/161 to clean up the following primary parts of the problem:

bytes values
bytes literal syntax
\x \X \u \U escapes
portability of strings

The remaining parts of the problem are:

conversions (string, bytes, numbers), ord, chr, and strictness of conversion failures
additional methods, including iterators
hash(bytes)

To make this process more manageable and reduce the number of rounds of review on an increasingly large pair of PRs, I propose that we aim to commit the spec change and the Go implementation once we are happy with the "phase 1" features, and then make follow-up changes to address the "phase 2" items. Given that no-one is yet using bytes, I don't think this will be disruptive.
cheers
alan

PTAL; thanks.


«Notify» ON doesn't work

@yivo I have checked this for chrome and Firefox and it seems to be working fine.


Constants are not propagated on module inclusion

Input
→ View on sorbet.run
# typed: true

module A
  CONSTANT = 'a constant'.freeze
end

class B
  include A
end

B::CONSTANT

Observed output
editor.rb:11: Unable to resolve constant CONSTANT https://srb.help/5002
    11 |B::CONSTANT
        ^^^^^^^^^^^
  Autocorrect: Use `-a` to autocorrect
    editor.rb:11: Replace with A::CONSTANT
    11 |B::CONSTANT
        ^^^^^^^^^^^
    editor.rb:4: Did you mean: A::CONSTANT?
     4 |  CONSTANT = 'a constant'.freeze
          ^^^^^^^^
Errors: 1

Expected behavior
No errors, as B inherits the constants from A.

:wave: @fsateler,
This is another instance of https://github.com/sorbet/sorbet/issues/8 and this will be non-trivial to fix. I'll close this one as it's a duplicate.
In the mean time, you can fully qualify the constant using the introduction class: A::CONSTANT.
Thanks for reporting!


test add ekco on upgrade

What type of PR is this?

type::tests
What this PR does / why we need it:
Adding tests for #2654
Which issue(s) this PR fixes:

Fixes NONE
Special notes for your reviewer:

Requires #2881
Steps to reproduce

Does this PR introduce a user-facing change?

NONE

Does this PR require documentation?

NONE

[sc-42741]

This is successfully reproducing the failure


Update package.json

@AviDavidov86 this fixes publishing, but fails testing? strange...

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


Adjust MCDA functions to be more generic

Currently adjustments to the MCDA (and related) functions require manual adjustments where criteria and their weights are added/removed.
Manually adjusting number of arguments and their index positions can lead to hard-to-debug issues due to the risk of human error, and a more than necessary time-sink as changes are required across a number of files (and those changes to be validated/tested).
We should instead move towards a more generic implementation of the MCDA functions.
One way would be to use ~NamedArrays~ NamedDims+AxisKeys to avoid the need for numeric indices, auto-generate the mapping between the column positions and the criteria they represent, and/or making the functions accept an arbitrary number of criteria/weights.
If it is necessary to continually add criteria as explicit arguments, the function could be made to accept an arbitrary number of arguments:
Notes:

https://docs.julialang.org/en/v1/manual/functions/#Varargs-Functions
https://github.com/mcabbott/AxisKeys.jl

Is this a separate issue from #252 ?

Wow, yes, it does seem like #252 is a duplicate. I did do a quick search before I opened it but obviously I'm blind (or used the wrong keywords).
Let's close this one in favour of that #252 then.


No ScanResult for devices previously discovered

Running 1.5.0 on RXJava2 here and I'm running into a peculiar issue where some ESP32-based peripherals are no longer found by my app during scanning.
Phone is running Android 7, but I have the same problems with one running 8.1 (first is a Xiaomi Redmi Note 4, second is a Nexus 6P).
For context, the app collects a list of these peripherals and then allows a user to connect via GATT to a custom service we defined that has a variety (<10) of characteristics (both read-only and read/write.
At some point of using the app during testing, perhaps after a write of some kind, I have discovered that I can no longer find these devices via an unfiltered scan (e.g.: ScanSettings.Builder().build()).
Strangely enough, I can read/write from the various characteristics of devices that do not provide scan results any longer. This all works just fine.
Despite my app failing to observe any of these devices, the nRFConnect app finds the devices quite happily, and can connect to the service we defined also.
Is this some sort of android caching magick? I've had a glance around the Internet but apart from various nebulous complaints of scanning issues, I can't see anything obvious.

Thanks for the feedback, jreehuis.

Very familiar with this 'feature': it's not to blame, I can power down the phone leave it for a few hours and bring it back up, and it won't see the device. I can use a different device, and the same problem. I start to believe that it is actually the ESP32 or some quirk of how I'm using the scanning subsystem, since a phone that NEVER saw the device can't see it via my app AFTER the ESP peripherals get in this state. Does not explain why nrfConnect sees them, though!

These scans are for less than 30s at a time and are manually started/stopped, so again, I do not think this is related.

I think I'm going to have to make a couple test apps that use the native scanner to see if that works in place of rxandroidble.

Turns out this was a bug in our code, have proven things are OK via the sample app. Sorry for the misreport!

Turns out this was a bug in our code, have proven things are OK via the sample app. Sorry for the misreport!

Hi,  @akram-aq1
Can you share detail of this bug and solution, we are running into this trap too.
Thanks in advance。

Hola @freecats,
The problem I was having was not the undocumented Google feature - it was simply that I had naively cached a set of ScanResult records and my code was ignoring discovered devices in that cache (oops).
As for 'fixing' the 'feature': unfortunately I don't know of a true fix. Right now we wait for the problem to occur and warn the user that they can't scan until time X in the future.
Cheers,
Akram

Hola @freecats,
The problem I was having was not the undocumented Google feature - it was simply that I had naively cached a set of ScanResult records and my code was ignoring discovered devices in that cache (oops).
As for 'fixing' the 'feature': unfortunately I don't know of a true fix. Right now we wait for the problem to occur and warn the user that they can't scan until time X in the future.
Cheers,
Akram

Thanks  for sharing!


[GHSA-6xp6-fmc8-pmmr] Temporary Directory Hijacking Vulnerability in Keycloak

Updates

References
Source code location

Thanks for the contribution. We don't normally accept personal articles on an advisory, however I see that redhat mentions you specifically in their cve report as the primary reporter so I'll go ahead an accept this as a reference. The Source code location should refer to a git/svn/whatever code repository.


Transaction validation failing after 200th block

System information
Network: local
if you are running a local node, please provide the following information:
op-node version: v18.18.0
op-geth version: v1.4.11
OS & Version: Linux
Expected behaviour
The validation should work as working before 200th block
Actual behaviour
The validation starts failing after 200th block
Steps to reproduce the behaviour
https://github.com/bnb-chain/bsc cloned this repo, setup genesis file using repos readme, and start the node using command
./bsc/build/bin/geth
--networkid 18923
--datadir ./bsc/chainData
--http
--http.addr "<IP_ADDRESS>"
--http.api "eth,net,web3,personal,txpool,parlia"
--http.corsdomain "*"
--mine
--miner.etherbase 0x64AffDD3f181788f61A73ebe4EEC85E228BDe70B
--allow-insecure-unlock \
Backtrace
message.txt

Thanks for your patience, can you try the following https://ethereum.stackexchange.com/questions/68249/block-sealing-failed-err-unauthorized-signer ?

Thanks for your patience, can you try the following https://ethereum.stackexchange.com/questions/68249/block-sealing-failed-err-unauthorized-signer ?

Thanks for the reply, yes i've tried it and added validator into the extra data which works fine until 200th blocks.

I think your problem should not be under the opbnb repository. It seems that you have a problem starting the local bsc network rather than opbnb.


Remove RCT-Folly version hardcoding

Hey. Installing pods currently fails when using react-native 0.73.0 and new architecture.
This is due to version conflict with Folly. react-native-geolocation hardcodes usage of 2<IP_ADDRESS>.
Presumably there is no reason for this, so this PR removes that.
[!] CocoaPods could not find compatible versions for pod "RCT-Folly":
  In Podfile:
    RCT-Folly (from `../node_modules/react-native/third-party-podspecs/RCT-Folly.podspec`)

    React-RCTAppDelegate (from `../node_modules/react-native/Libraries/AppDelegate`) was resolved to 0.73.0, which depends on
      RCT-Folly

    ReactCommon/turbomodule/core (from `../node_modules/react-native/ReactCommon`) was resolved to 0.73.0, which depends on
      RCT-Folly (= 2<IP_ADDRESS>)

    react-native-geolocation (from `../node_modules/@react-native-community/geolocation`) was resolved to 3.0.6, which depends on
      RCT-Folly (= 2<IP_ADDRESS>)

Cheers.

Thanks!


nixos/krb5: complete rewrite

Motivation for this change
The krb5 service was a bit lacking.
Addresses NixOS/nixpkgs#11268, partially addresses NixOS/nixpkgs#29623.
I've made an effort to accommodate people who are using the old configuration keys.
Given the state that it was in though, I wonder anyone even used this module.
Also I'm not sure if this is an abuse of the testing framework.  Snapshot testing is something I do a lot of in web development so it makes sense to me to try it here.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[x] 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 nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Tests looks ok, but maybe launching a vm is not necessary. @Profpatsch do you have a better idea here?

I successfully transformed my old krb5 configuration to the new module syntax.

Nice work. I also gave up on the old module back then.

Thanks!!

Tests looks ok, but maybe launching a vm is not necessary. @Profpatsch do you have a better idea here?

For service modules it’s kind of hard to test the correct functionality without a NixOS VM. :)
If you want to test just the executable though, that could be done. Of course one can also mix.


Examples for username/password authentication?

Can we have examples for SCRAM-SHA-512 based username/password authentication?

Is this helpful?  https://github.com/aws-samples/sasl-scram-secrets-manager-client-for-msk

Yes, when I opened this AWS didn't had username/password authentication feature. Thank you!


Add avg_pool2d with kernel size support

Ticket
https://github.com/tenstorrent/pytorch2.0_ttnn/issues/477, https://github.com/tenstorrent/pytorch2.0_ttnn/issues/557
Problem description
Currently there is no ttnn API to support average pool2d with kernel size. This change adds ttnn.avg_pool2d with kernel size support
Comparing to the corresponding pytorch function torch.nn.functional.avg_pool2d, for now only supports options with ceil_mode=False, count_include_pad=True, divisor_override=None
Currently following cases still have issues and are blocked by TT_FATAL:

#14459
#13901
#15731

What's changed

Add ttnn.avg_pool2d API
Change generic pool kernels to support the new Pool2DType::AVG_POOL2D
Add test cases for ttnn.avg_pool2d

Checklist

[ ] Post commit CI passes (running on https://github.com/tenstorrent/tt-metal/actions/runs/12192497750)
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[x] New/Existing tests provide coverage for changes

I constantly got PCC error on ttnn.global_avg_pool2d in post-commit tests, which seems to be irrelevant to this change.
However, after digging deeper I found that it is because ttnn.avg_pool2d tests ran on the same CI runner with ttnn.global_avg_pool2d. And once ttnn.avg_pool2d or ttnn.max_pool2d is called, the PCC of ttnn.global_avg_pool2d becomes very low (0.7), which can be reproduced locally
Since this also happens with the current ttnn.max_pool2d, I filed https://github.com/tenstorrent/tt-metal/issues/15824 for it. Also should we mark ttnn.global_avg_pool2d as xfails for now to unblock this PR?


Update appveyor_ci with upstream

Several improvements have been done in upstream.
Retrieve them to use latest features and fix bugs

Also, in the upstream of travis-ci, you can now do build for windows.
But I think I deactivated the python 2 version...


Why panic instead of returning error in Call?

I was wondering why Call panics instead of having an error return value?
func (v Value) Call(m string, args ...interface{}) Value {
    res, ok := v.call(m, makeArgs(args))
    if !ok {
         panic(Error{res})
    }
    return res
}

If that were the case, the user would have to check the returned error value after each call to Call. Would that be better?

Returning an error would allow the developer to take some action (e.g. show an error dialog) rather then have the module stopping.  Most consumers will not be looking at the console to see if there is an error and it'll just appear like nothing has happening.

You can still use recover to catch the panic.
I think returning an error everywhere would not be nice, because most JS functions will never throw an error. What we could consider is some separate function like CallWithError or something. If you think this would be worthwhile (please add a real-world example), then please open the issue on the main golang/go repository. I'll close it here for now since I'm phasing out this issue tracker.


Update repository URLs for dependencies

Build aborts with 525 errors, but at least all dependencies are retrieved now!

I've tried regenerating the bindings, and I'm not sure if I'm doing it right, but whatever I'm doing isn't fixing the build errors?
I've run make gir in the root directory, which seems to be regenerating the bindings, and then running cargo fmt on them. This produces no difference with the current contents of src/auto in master.

It appears that the three remaining build errors are due to the fact that glib::translate::ToGlib no longer exists?

Hopefully I'm helping in some manner? While I don't think I can progress further on my own as I'm unfamiliar with the inner workings of the GTK-rs project, I do think I've laid the groundwork for someone else to pick up where I left off

I believe I'm pretty much done, as of my latest commit? I'm seeking outside review and tweaks, and hopefully this PR will get merged. Build errors have been resolved, and #93 has been fixed.


Request for tables

Could it be possible to integrate tables into the project? It'd be nice to look at some of my reference data in a well presented way.

This sounds like a great idea, it would be interesting to see how this could be handled in a cross-browser compatible fashion. I'd very much welcome a pull request if anyone has ideas around implementing this?


add to random facts, change some aligning

Note: work branched off of https://github.com/hjohns28/step/pull/9


setTemplate not working

Hy. I'm using a GT-511C1R with a Raspberry Pi 3 Model B (also I'm new to nodejs) and i would like to retrive the templete for a specific ID so that i can insert it into an external database. So i ran the example.js and used the getTemplate function. The function worked but after it i could not run any other function, every action was returning undefined (but pressing 1 and 2 was turning on/off the led and returning an undefined), also i could not exit with 0, same error. Anyway, the tpl file was created and after i unplugged the sensor i relaunched the example.js. i deleted all id's and tryed to use the setTemplate but it returned me the same undefined. Please help me with this problem because my project depends on this. Thank you !

Would you mind put a few screenshots here?

I saw your screenshots.
in /lib/gt511c3.js
line 946-948 should be like this, which is the most recent version:
       function(err) {           reject('send command: ' + err);         });
I had undefined error too, with the way it is.
Please change line 943 and 947 to:
         reject(err);
You should see the actual error now. I got "the specified ID is already used". Basically, you can't overload it.
in /examples/example.js look for var ID_TO_USE = 199;
Change the ID to something else, between 0-199. I'm assuming you can't enroll it first then try to setTemplate. I'm not sure what else you are trying to do, but try changing the ID to an empty one. Also, make sure you have a template file ready, in the naming convention of template_xxx.tpl

Thank you! :) I will try this. At this moment I'm not at home so i cant try it but i will make the changes. My problem is that the fingerprint can enroll only 20 in my case or 200 for GT511C3. Anyway i would like to enorll as many fingerprints as i want so for that i use an external database so i can store the templates there and get them whenever i want. Thank you for being so fast :) Hope it works!
Have a nice day!

I was able to resolve the problem. The problem was the data sheet from sparkfun. They say you need to have a template (of the fingerprint) of 498 bytes but the problem is that it is actually 506. So after I modified the templete size to the correct one eveything works fine now.

how did you modify the template size? what command did you use?


Updated .gitignore

Added rules to ignore bundle gem files at /vendor/ but not ignore /assets/vendor. #249
Modified package-lock.json rule to apply to any path containing it.
Better organized existing rules.

Nice! Thanks 👍


Display a sankey diagram on an interface page

Display a visualization of real survey data somewhere as a quick-and-dirty demonstration.

resolved by #25


MAG Brew Vikings NFT project on Cronos

Closing this since this is a custom solution, not a general help for the repo - though thanks for the 'mintCost' naming support suggestion!


Upgrade fourseven:scss dependency to 3.1.1

Use the latest version of fourseven:scss which uses libsass 3.1.1 bringing it almost to feature parity with Ruby Sass. Very important dependency for most using Sass in Meteor, shouldn't affect this package at all.

Can we get this merged and pushed to atmo? Works fine for me.

We're up to 3.2.0 now, and I'll release a new version soon.


Enable no-floating-promises lint and clean up usage of async

Enable no-floating-promises lint and clean up usage of async
Implement minor fixes from skynet-nodejs

There is a bug in GitHub Actions causing tests to fail for Node 16.9. I hard-coded CI to use Node 16.8 for now.


Update README.md

Reflecting the changed way to use upload and validate
./ch57x-keyboard-tool upload 4knob-mapping.yaml gives an error:

error: Found argument '4knob-mapping.yaml' which wasn't expected, or isn't valid in this context

./ch57x-keyboard-tool upload < 4knob-mapping.yaml works.

In fact, change was in another direction: file name support was added (9de814e5d19ecb6ef7384afe654ff4a0cfb72afa). Do you use latest version?


allow custom storage

consider adding a setStorage to configuration to set a localStorage api compliant custom storage for storage.js

not gonna do that any time soon


bug fixes vue router

@kirschbombe fix for current site inability to load. Currently working on migrating the site to Vuejs 3.

@kirschbombe If could please merge this PR it would fix the site.

lol no problem.


validate source of joiner, entire cluster used to crash when sent an …

…invalid host-port as the source field

plz add a test


Can not connect to Mysql database

System.Data.SqlClient.SqlException (0x80131904)
The Mysql database is running perfectly. Other programs can acces the sql database.
I used the following command:
BitcoinDatabaseGenerator /BlockchainPath C:\Bitcoin\data\blocks\ /SqlServerName localhost /SqlDbName bitcoin /SqlUserName root /SqlPassword root /DropDb
Plz help.

This app can only save to Ms SQL Server. It does not work with mysql.


Future-proof NN interface with multiple backends

Noticing how fast the deep learning ecosystem is moving, with multiple NN implementations in Python, and much of the code in Lua, @ssamot and I are considering moving the project toward a multiple back-end interface with a stable interface — in order to keep things as future-proof as possible.  This seems like one of the best ways to support Deep Learning in Python too.
In particular, we're considering keras.io as a new backend as it supports some interesting RNN features.
What do you guys think?

From version 0.3 onward, we'll be taking the library in this direction.  @ssamot has a branch called RNN with a passage integration, for example.

There's a new branch called backend which has my refactoring for supporting multiple neural network libraries with the same compatible interface.  Started with deepy for no particular reason, keras is another great option we'll follow up with once this stabilises.


Help me solve the unknown role error.

Thank you for your patience, sorry for commenting on unrealated commit because I'm a github noob. I let my yml look like this:
plan_roles:
'348830':<PHONE_NUMBER>308294246
Allow an unsubscribed user to get access if they have subscribed before.
allow_fallback: False
This must be a number, same as above.
fallback_role:<PHONE_NUMBER>308294246
But I still have the same error like before, the bot regconized the subcriber but it have some issue with the role.
discord.errors.NotFound: 404 Not Found (error code: 10011): Unknown Role
[2023-06-08 23:31:55,100][INFO] User: Pipisvoir#9893; Message: "!8591772"; Response: system_error

yeah I gave it administrator permission

Okay, and to make sure, the way you got the role ID is Server settings -> Roles -> right click on role -> Copy Role ID, right?

right

I tested in another server and another role, bot gave me this error
discord.errors.Forbidden: 403 Forbidden (error code: 50013): Missing Permissions
[2023-06-08 23:48:12,197][INFO] User: Pipisvoir#9893; Message: "!8591772"; Response: system_error

FYI the bot can only work with a single server per running instance, so if you invite it to multiple servers, it will only work for the first server it is a member of.

alright, after it just stay in 1 server, it gave me this error in test:
discord.errors.Forbidden: 403 Forbidden (error code: 50013): Missing Permissions
[2023-06-08 23:58:52,825][INFO] User: Pipisvoir#9893; Message: "!8591772"; Response: system_error

Okay, now you need to fix permissions. Whatever server it's in, it does not have appropriate permissions. If the bot's role has full permissions, it also needs to be invited to the server with the correctly OAuth permissions from the bot dev webpage. You can simply re-invite the bot with the updated permissions.

I just need to tick administrator or right things?

Administrator would be the easiest thing to pick.

Oh it works, so if I want to give a role for my member, how should I do it? I just know !<pixiv_id> and !purge,reset.

When I !<pixiv_id> it gave my own account the role

You don't need to put ! in front of the Pixiv ID. The member themselves sends their own pixiv ID and gets the role. It's meant to be automated so that you don't have to do this yourself.

It worked, thank you a lot friend.

No problem. I will update documentation to make this all more clear.

https://github.com/cromachina/fanbox-bot/commit/d0a73b32f503268d67cc0a02627ad5f491a5a153 Updated documentation based on this conversation.


viaf lookup for birth/death dates are not working properly

Did a little more digging - we have viaf IDs ending with slashes but the RDF has URIs ending without the slash, so our lookups don't match and we don't get the dates we should be getting.  (Maybe something changed with VIAF since the time this was implemented, or maybe we changed our local practice.). We can fix that part in the S&co codebase.
I also see that the date parsing doesn't handle negative years... That needs to be fixed in viapy.
Originally posted by @rlskoeser in https://github.com/Princeton-CDH/mep-django/pull/800#discussion_r1459201382

[x] fix viaf uris in s&co codebase: when initializing ViafEntity, strip the trailing slash
[x] fix date to year parsing in viapy to handle negative years (example: Aeschylus,  https://viaf.org/viaf/268526195/ )

We'll want to batch update records when this is fixed - maybe a one-off python console update would be sufficient in this case instead of a manage command (although log entries would be nice)

python console syntax for updating missing birth/death years from viaf:
from mep.people.models import Person
for p in Person.objects.exclude(viaf_id='').filter(birth_year__isnull=True):
    p.set_birth_death_years()
    p.save()

This won't create log entries - need to decide if that matters.

Perfect!


Fix readme commands

I don't know whether I missed something or not, but the command I use are based on the .sh script.
The readme now reflects that.
I also suggest to remove the removing the intermediary naming issues with the extra directory to simplify the readme.

I added the extra working directory to prevent users from 'polluting' the repo and accidentally committing additional files. But that is a precaution which I should not bother with, because it will obvious when merging PR's. So thx for making it simpler this way. :-)

I thought it could be nice to do something similar, but inside the repo with a build directory & a .gitignore.
A bit similar to what CMake et al do.
What do you think?
I might make it if you're interested.

That was what I was thinking about as well. If you like, you're certainly free to add it. Would be really nice.

That was what I was thinking about as well. If you like, you're certainly free to add it. Would be really nice.

Just so I know, are you set in having it executed in a bash script?
Would that be a problem if it was rewritten in Pharo?

I like the bash script for being able to run things in an automated environment (GitLab CI or GitHub Actions). But starting a Pharo image to perform the different tasks is perfectly legit as well of course. So no problem if written in Pharo itself. Should be able to run headless (for the mentioned environments). And ideally also work sort of incrementally. So loading the different required packages should be done with a 'saveSnapshot' when finished succesfully, but running the final 'bootstrap' step should not, allowing the image to be used multiple times from exactly the same state. (Hope I'm being clear here ;-).


🐛 BUG: Authentication error [code: 10000]

Which Cloudflare product(s) does this pertain to?
Wrangler
What version of Wrangler are you using?
2.12.0
What operating system are you using?
Ubuntu 22.10
Describe the Bug
wrangler deploy keeps failing with Authentication error.
Me: npm run deploy

><EMAIL_ADDRESS>deploy
> wrangler publish

 ⛅ wrangler 2.12.0
--------------------
Total Upload: 0.19 KiB / gzip: 0.16 KiB

✘ [ERROR] A request to the Cloudflare API (/zones/[zone-id-redacted]/workers/routes) failed.

  Authentication error [code: 10000]

  If you think this is a bug, please open an issue at:
  https://github.com/cloudflare/workers-sdk/issues/new/choose

No cloudflare environment variables set. Successfully and freshly logged in using wrangler login.

And I was using Account id instead of zone id... :))


Appstudio update test-component-pac-hoyv

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-hoyv-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
9 seconds
init

✅ Succeeded
31 seconds
clone-repository

✅ Succeeded
56 seconds
build-container

✅ Succeeded
14 seconds
sanity-inspect-image

✅ Succeeded
1 minute
clair-scan

✅ Succeeded
53 seconds
deprecated-base-image-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
12 seconds
sanity-optional-label-check

✅ Succeeded
35 seconds
sanity-label-check

✅ Succeeded
13 seconds
show-summary


Feature Request: Add decimal places to the Box Score projections

Great work on the python API, it's really great. I'm in a league that does 0.1 points per yard for RB/WR/TE, and 0.04 points per yard thrown for QB. This causes the projections and scores to be translated as doubles, not straight integers. I noticed that the box_scores projections don't have decimal places. It would be nice if it did for leagues similar to ours.

They are provided as decimals on a per player basis.
On Fri, Sep 6, 2019 at 10:32 AM James Sayer<EMAIL_ADDRESS>wrote:

Great work on the python API, it's really great. I'm in a league that does
0.1 points per yard for RB/WR/TE, and 0.04 points per yard thrown for QB.
This causes the projections and scores to be translated as doubles, not
straight integers. I noticed that the box_scores projections don't have
decimal places. It would be nice if it did for leagues similar to ours.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cwendt94/ff-espn-api/issues/30?email_source=notifications&email_token=ACWGOWMG7YW5AYJNHSGAZDTQIJSYVA5CNFSM4IUJWAHKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HJ2J5TA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWGOWII37KUWA6GWF5V3MTQIJSYVANCNFSM4IUJWAHA
.

I updated box_scores yesterday and I believe the scores and players points should be 2 decimal places if the league has them. The newest package version is 1.1.0

Ah, I cloned prior to that. I'll check it out and see. Thanks guys

Sounds good, let me know if you still have problems!


Fix instrumentation-ioredis test from failing

At this point I'm not sure why or how, but the tests for instrumentation-ioredis are reliably failing and making our builds fail with it.
This needs to be fixed, but meanwhile since it's affecting all builds, I'll temporarily ignore the tests for now.
Assigning component owners
https://github.com/open-telemetry/opentelemetry-js-contrib/pull/1169

Turned the tests off in https://github.com/open-telemetry/opentelemetry-js-contrib/pull/1169
That needs to be reverted with a fix.


append-params does not send updated data

Hi!
I am using 2.0 beta and am loading my table data from the API via POST and add the parameters using append-params. append-params is linked to a computed property that recalculates the data to append (it changes all the time since the tables feature some interaction where lines can be added and removed).
It works like a charm on pageload, but even though the computed property always shows the right parameters to send in the request to the API, vuetable2 seems to not pay attention to that anymore. The computed property method is called (on refresh()), but the request from vuetables2 to the API always uses the old parameters that were valid on pageload.
My vuetable declaration:
<vuetable ref="vuetable"
         api-url= apiUrl
        :fields="fields"
        data-path="data"
        pagination-path=""
        :per-page="perPage"
        :multi-sort="multiSort"
        :table-height="tableHeight"
        :detail-row-component="detailRow"
        :css="{sortableIcon: 'grey sort icon'}"
        :append-params="getHttpPostData"
        :transform="transformMaterialsToSelectables"
        @vuetable:pagination-data="onPaginationData"
        @vuetable:initialized="onInitialized"
        @vuetable:loading="showLoader"
        @vuetable:loaded="hideLoader"
        @vuetable:cell-clicked="onCellClicked"
        @vuetable:scrollbar-visible="onScrollbarVisible"
        track-by="material_uid"
        http-method="post">

My computed property for the parameters to append:
getHttpPostData() {
        var paramObj = {}
        paramObj.parent_uid = (this.material.uid != "") ? this.material.uid: this.data.uid
        var ingredient
        if (this.data.ingredients && this.data.ingredients.length > 0) {
            for(ingredient of this.data.ingredients)
                paramObj.already_selected.push(ingredient.material_uid)
        }
        console.log("Created new http Post Data: "+JSON.stringify(paramObj))
        return paramObj
    }

As said, the Vue Explorer shows that the computed property always has the right value and is also called when I call refresh() on vuetable. However, the parameter values sent by vuetables2 do not seem to correspond to that computed property anymore after pageload. Do you have any idea why that might be?

Refreshes of the parent component came too early. nextTick solved it.


✨[Feature Request]: Adding Twitter Authentication

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
@Anishkagupta04 @varshith257 @Anjaliavv51  I want to add Twitter Authentication to the website. Kindly assign me this issue.
Use Case
Using Authentication anyone can sign in the Page using his/her own email address.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

I think it's not needed.

@Ojas-Arora instead add auth for GitHub

@varshith257 labels are working.

@varshith257 ok I will add Github auth


When fetching wallet transactions in pages, the count does not match the number of transactions returned.

 Then when I fetch the next page with the same limit, only 7 transactions are returned, which makes for a total for 32. So either the count is off or two transactions are not being returned.
http://localhost:4002/wallet/transactions/TLTC?limit=25&offsetId=7df98cd79af95b5214cb90221753282be1ce67fbc94e0441c64f69782e2b8967
returns:
{
    "count": 34,
    "transactions": [
        {
            "address": "",
            "canBumpFee": true,
            "confirmations": 5758,
            "errorMessage": "",
            "height": 789171,
            "memo": "",
            "orderId": "",
            "status": "CONFIRMED",
            "thumbnail": "",
            "timestamp": "2018-10-05T13:04:53-06:00",
            "txid": "cdc4c56f38b0143026df2edc3f772918352476059579f019bece5191cf084f54",
            "value": 50000000
        },
        {
            "address": "",
            "canBumpFee": true,
            "confirmations": 5758,
            "errorMessage": "",
            "height": 789171,
            "memo": "",
            "orderId": "",
            "status": "CONFIRMED",
            "thumbnail": "",
            "timestamp": "2018-10-05T13:04:53-06:00",
            "txid": "28f113604d7138372bd3b0f604598d77e4ca7565de667f6a63b261ad67f31df7",
            "value": 23000000
        },
        {
            "address": "mtoTGz4Kuyk6xbPSfEBBMN6LU45dSABah3",
            "canBumpFee": false,
            "confirmations": 15272,
            "errorMessage": "",
            "height": 779657,
            "memo": "",
            "orderId": "",
            "status": "CONFIRMED",
            "thumbnail": "",
            "timestamp": "2018-10-03T16:12:42-06:00",
            "txid": "f07ee4444622f7a792fcce3f04ae395d05488738b1d4aaed8083b5df418e9b93",
            "value": -9385198
        },
        {
            "address": "",
            "canBumpFee": true,
            "confirmations": 15272,
            "errorMessage": "",
            "height": 779657,
            "memo": "",
            "orderId": "",
            "status": "CONFIRMED",
            "thumbnail": "",
            "timestamp": "2018-10-03T16:12:42-06:00",
            "txid": "0044cc7cadad7033a6baa70cc3009bebc0672fb4a673b42a6b358927ab0943ce",
            "value": 3000000
        },
        {
            "address": "",
            "canBumpFee": true,
            "confirmations": 15275,
            "errorMessage": "",
            "height": 779654,
            "memo": "",
            "orderId": "",
            "status": "CONFIRMED",
            "thumbnail": "",
            "timestamp": "2018-10-03T16:07:55-06:00",
            "txid": "ae4d6d5bd0c3b7ca6719bcc589007f3925ad64af70f94a168555c057825489cc",
            "value": 10000000
        },
        {
            "address": "mtoTGz4Kuyk6xbPSfEBBMN6LU45dSABah3",
            "canBumpFee": false,
            "confirmations": 15277,
            "errorMessage": "",
            "height": 779652,
            "memo": "chuck the spidey sense",
            "orderId": "",
            "status": "CONFIRMED",
            "thumbnail": "",
            "timestamp": "2018-10-03T16:04:16-06:00",
            "txid": "5eeadfa9ec539f05df798b23a2bded0623f859a1bf5a0a3af2c58372796789a9",
            "value": -25225395
        },
        {
            "address": "",
            "canBumpFee": true,
            "confirmations": 15279,
            "errorMessage": "",
            "height": 779650,
            "memo": "",
            "orderId": "",
            "status": "CONFIRMED",
            "thumbnail": "",
            "timestamp": "2018-10-03T15:55:42-06:00",
            "txid": "c6ae8af76956a360ba0418771d33fb42fe4297f224c356cebb8f61cde178de4f",
            "value": 130000000
        }
    ]
}

@rmisio I spent an hour or so trying to find a good insight server on BTC, BCH or LTC testnet somewhere to produce transactions in the wallet for me but I could not find any insight server that works or was available. Do you happen to have the insight server you tested this with, handy?

This was specific to insight servers were testing at the time. Decided to remove insight support until some later date. Closing issue.


Enable ubpf on platforms with no supported JIT

As a first step in adding an Arm64 JIT to ubpf this PR enables building the library on platforms with no JIT target implemented.  It also updates the testsuite to skip trying to run the JIT tests on those platforms.  It does this by adding a 'null' JIT that always fails.
The approach taken for the build system is to assume that all JITs (currently just x86_64 and null) will successfully build on all platforms, but that only the JIT required for the target platform will be used.
Building and testing on Arm64 platforms highlighted an issue with elftools which is worked around in the second commit.
This has been tested successfully on x86_64 and Arm64 platforms.

test_vm.test_datafiles('call-save.data',) should probably be skipped, not a failure, for architectures that do not support trash registers.


how can I  use PercentRelativeLayout by DSL?

please give me a sample

me to,please write sample for wiki.thank you


Created component for typeahead MenuItems

Created a component for the typeahead MenuItems so that the drop down suggestions will have formatting as required by UI mock-ups.

Coverage remained the same at 80.759% when pulling 2b92546cb9e6d5e836170fd0fccf9bcb0a7ee42b on 16497-SearchbarSuggestions into de090781fc7dc3ab82297b997cf883e19ac0a588 on 1.0.0.


[TEST] 7zip

Details of Changes
Add details of changes here.

Added support for browsing and editing rar, 7z, zip archives

TODO:

[ ] Fix renaming folders
[x] Test in Release mode

Validation
How did you test these changes?

[x] Built and ran the app

Is there anything I can do to help test this?

Release mode now works. Still need to fix renaming then is ready to test.

How would an option to open archives in Files work? I'm pretty sure Microsoft prefers that we let users change the default on their own.

Or use Files only if set as default for that particular extension?
I'm leaning towards this but if there is a supported way to change the defaults for the other ones, it wouldn't hurt to add an option.

Some additional feedback I got was that we can add shell extensions to File Explorer for extra visibility.


Feature/client/A4 House Setting (Edit)

Describe your changes

Added house setting (edit)
Added destructiveOutline variant for Button

About Destructive Button

I added outline ver because in house setting page, Save is primary button and if I use the destructive variant, it's too much strong buttons in one page.
So, let's use destructiveOutline if the button is not the action it self. Fx. open modal, redirect the page...
For this reason, I also changed the user account delete button.

Issue
https://github.com/HAK2024/fairy-share/issues/28
URL
http://localhost:3000/house/edit
Screenshot (if you made the UI)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings and no errors

Working well! but the one thing is I have to refresh the main page to see the updated house name. Is it okay not to show right away?

@hyejin-seok
Thanks! I fixed it https://github.com/HAK2024/fairy-share/pull/87/commits/3bf01f6f4135ee922731963e360f9decfbdd00a5


core: add svg definitions for gradients

colocates dup gradient definition next to the first, css definition
makes sense to import this into icon for use as a fill override.

This is problematic.  I don't even know if it's a real requirement.  Leaving until it is obviously required to spend time on.


Production build failed with --aot option

ng build --prod --aot=false --bh /
W
Hash: 86f7d968fb1338f6973e
Time: 30760ms
chunk    {0} polyfills.02e94b9ade803b61d4c0.bundle.js (polyfills) 158 kB {4} [initial] [rendered]
chunk    {1} main.024acb443441716d1306.bundle.js (main) 301 kB {3} [initial] [rendered]
chunk    {2} styles.060ad473ce1fab4aca04.bundle.css (styles) 69 bytes {4} [initial] [rendered]
chunk    {3} vendor.3c6658b0d60af4afe8fb.bundle.js (vendor) 3.24 MB [initial] [rendered]
chunk    {4} inline.6ff93049aee59bfd2317.bundle.js (inline) 0 bytes [entry] [rendered]

ERROR in Error encountered resolving symbol values statically. Calling function 'BusyModule', function calls are not supported. Consider replacing the function
or lambda with a reference to an exported function, resolving symbol AppModule in C:/xampp/htdocs/Hubtel.Me/src/app/app.module.ts, resolving symbol AppModule
in C:/xampp/htdocs/Hubtel.Me/src/app/app.module.ts
Kindly provide a way to fix this issue or skip errors of some sort

@paulmajora Can you provide an example of the project or module you are having issues with.

Hi , Arora ,  Here's the module details
link : https://www.npmjs.com/package/angular2-busy
npm install --save angular2-busy
ModuleName : BusyModule
Thanks for your time
On Thu, Apr 27, 2017 at 5:44 PM, Sumit Arora<EMAIL_ADDRESS>wrote:

@paulmajora https://github.com/paulmajora Can you provide an example of
the project or module you are having issues with.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/angular/angular-cli/issues/6101#issuecomment-297787788,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANMVbDMjKcLZAxWplZA5v-Mym9hn5iJgks5r0NPtgaJpZM4NKF-q
.

Heya, errors like Error encountered resolving symbol values statically mean that there has been some problem in statically analyzing your app.
The CLI always runs statical analysis on code to ensure it will run when compiled with AOT. This may cause a lot of static analysis errors to surface when importing your project into the CLI, or upgrading for older versions where we didn't run this kind of analysis.
Good resource on how to to debug these errors are https://gist.github.com/chuckjaz/65dcc2fd5f4f5463e492ed0cb93bca60 and  https://github.com/rangle/angular-2-aot-sandbox#aot-dos-and-donts.
If your problem still persists, it might be a bug with the Angular Compiler itself. In that case, please open an issue in https://github.com/angular/angular.


Request.Path is empty while Request.PathBase is not

Request.Path is empty while Request.PathBase is not
General
Using .NET Core 2.0, Windows 10, IIS Express.
While debugging my .NET Core web application, IIS Express launches a new browser window with the address localhost:7777/dashboard. Usually, I am able to view my web site at this point, but upon transferring my project over to a new machine with a fresh copy of Visual Studio Community 2017, the project returns a very strange error. The environment is identical to the environment on my last machine, so I see no reason why this error should occur. The error occurs because HttpContext.Request.Path is empty, even though I am accessing the path /dashboard in the URL. I noticed something very strange while looking into the error. HttpContext.Request.PathBase returns /dashboard even though HttpContext.Request.Path is returning an empty string.
The HttpContext object is accessed from the context argument passed within the line of code, app.Run(async (context) =>{}); located in Startup.cs.

Got same issue, why you have closed this ticket?

I closed it because I thought perhaps it was my own fault since I inherited the startup class with another startup class. But after some experimentation, I realized the problem persisted. I have a few other projects that work just fine, uses similar code base, same .NET Core version. I thought the problem was my own, but now I am reopening this because I can’t get request.path to populate in this one project no matter what I try.

Also, I’ve come to realize that this issue persists on both dev machines, not just on the machine I transferred the project to.

I think I've narrowed down the issue at hand. All of my projects use Microsoft.AspNetCore NuGet packages for version 2.0.0, except for the project with this issue. That project uses NuGet packages 2.0.1. I can confirm that this issue only exists in Microsoft.AspNetCore NuGet packages version 2.0.1 because after installing & updating my project to version 2.0.0, everything works again.

It worked for a few minutes after downgrading Microsoft.AspNetCore NuGet packages down from 2.0.1 to 2.0.0, and then it stopped working again. I haven't touched any code or closed the project, or done anything in any way to affect anything. I simply pressed play, stop, and play again, and it worked the first time, and now Request.Path is returning "/" instead of "/dashboard". Before it was working at all, though, Request.Path was returning an empty string, so now I'm getting even stranger behavior.

Now I've narrowed it down even further. The first page request comes from dotnet.exe, then all other page requests are coming from a second instance of dotnet.exe. Very strange behavior!!!
Here is a log of page requests coming from my Startup.cs app.Run function. I am unable to break on any break points after my first request because all code is executed on a second dotnet.exe.
Websilk> Hosting environment: Development
Websilk> Content root path: C:\Projects\Websilk\Home\App
Websilk> Now listening on: http://localhost:26577
Websilk> Application started. Press Ctrl+C to shut down.
Websilk> --------------------------------------------
Websilk> 10:12:42 GET /
Websilk> END GET / 446 ms page
Websilk>
Websilk> Hosting environment: Development
Websilk> Content root path: C:\Projects\Websilk\Home\App
Websilk> Now listening on: http://localhost:12425
Websilk> Application started. Press Ctrl+C to shut down.
 There is absolutely no reason for this! No where in my launchSettings.json does it tell IIS Express to do this.
{
  "iisSettings": {
    "windowsAuthentication": false,
    "anonymousAuthentication": true,
    "iisExpress": {
      "applicationUrl": "http://localhost:7778",
      "sslPort": 0
    }
  },
  "profiles": {
    "IIS Express": {
      "commandName": "IISExpress",
      "launchBrowser": true,
      "environmentVariables": {
        "ASPNETCORE_ENVIRONMENT": "Development"
      }
    }
  }
}

I figured out a solution. I simply deleted the hidden folder .vs and everything went back to normal :)

Thanks, I've just moved to use IIS instead IISExpress, that helps too.

Error still exists. Changing App URL in project properties corrupts .vs\config\applicationhost.config file.
<application path="/" applicationPool="WebMonitoring AppPool">
          <virtualDirectory path="/" physicalPath="C:\Repository\Monitoring\Sources\Application\WebMonitoring\WebMonitoring" />
          <virtualDirectory path="/api/Cfg" physicalPath="C:\Repository\Monitoring\Sources\Application\WebMonitoring" />
          <virtualDirectory path="/api" physicalPath="C:\Repository\Monitoring\Sources\Application" />
</application>
<application path="/api/Cfg/Register" applicationPool="Register AppPool">
           <virtualDirectory path="/" physicalPath="C:\Repository\Monitoring\Sources\Application\WebMonitoring\WebMonitoring" />
</application>

Also seeing this issue even when running as a local site outside of visual studio. Path is null but PathBase is not. I did try to delete my .vs folder but that did not resolve it.

I got into this state by changing the applicationUrl in launchSettings.json. Changing it back didn't fix it, however, until I deleted the hidden .vs folder.


TODO

TODO list:

[x] Resize pictures
[x] Update pictures with corrent version of the GUI
[x] Fix enumeration

Done


add streaming to chains

Would love to see this happen. The documentation says that architectures are being considered, any guidance on how I could help make this happen? I have a hacky solution, but happy to help build out what's needed for full support.

Yes please! Also happy to help 👍🏻
@etown could you describe your ideas on a solution? Sounds very interesting 😃

@hwchase17 Is this resolved? I am pretty sure that by now almost every chain has implemented streaming, and there are specific issues open for most instances that are still not working.


[resolves #425] Switch to more reliable RSS feed than jboss.org

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/298/
Test PASSed.


Add to Scene and Configure doesn't save changes (should dirty the scene)

Describe the bug
Mixed Reality Toolkit > Add to Scene and Configure doesn't dirty the scene. If the user doesn't make additional changes to the scene, the effect of Add to Scene and Configure won't be saved.
To reproduce
Steps to reproduce the behavior:

Add MRTK to new project. (I did this via NuGetForUnity if it matters.)
Create Test scene and save it.
Invoke Mixed Reality Toolkit > Add to Scene and Configure...
Notice that this has added two MixedReality game objects to the scene
Exit Unity
Reopen project
Notice that the two MixedReality game objects are no longer in the scene

Expected behavior
The two MixedReality game objects are still in the scene after reloading
Your setup (please complete the following information)

Unity Version 2018.4.14f1
MRTK Version 2.3.0

Target platform (please complete the following information)
UWP. Probablem can be reproduced in editor without MR device.

Sorry I don't have time to investigate right now, but in case it helps this issue seems related.

Still reproduces with Microsoft.MixedReality.Toolkit.Unity.Foundation.2.4.0.unitypackage in Unity 2019.4.8f1.


Changes to parameters don't regenerate path

Title pretty much says it all. Altering path parameters such as time step, velocity, acceleration, jerk, and wheel base don't seem to trigger a regeneration of the path.

Closed, issue solved in 7752426
To reiterate, edits are triggered in two ways:

On action (enter/return button)
On unfocus


Scopes

Hi all,
what are the minimum scopes that are required for this codebase?
it looks like it requires more than
scope = "user-library-read,user-follow-read"
Insufficient client scope, reason: None
Thanks
Nick

Hi.
I have:
scope = "user-read-playback-state,user-modify-playback-state,user-read-currently-playing,app-remote-control,streaming,playlist-modify-public,playlist-modify-private,playlist-read-private,playlist-read-collaborative,user-library-modify,user-library-read,streaming"
Regards.

Thanks for this, is there any place beyond Spotify_manager.py that this list of scopes needs to be highlighted?

Nop ;-)

Hello, I am trying to add the 'user-read-private' scope to my project and it seems to mess everything up.  It makes me enter the redirect website again and then tells me "Illegal scope".  I am trying to make the search function work in the sPot and reading the 'scopes' page on the spotify web sit tells me that the 'user-read-private' scope is needed to perform searches.  I cant figure out what im doing wrong.  Any info would be a great help.
Thanks.

Hey @efernan,
the Spotify Web API Console does not require the user-read-private scope to perform searches: https://developer.spotify.com/console/get-search-item/?q=tania+bowra&type=artist
And it is not mentioned in the authentication header: https://developer.spotify.com/documentation/web-api/reference/#endpoint-search
Therefore I do not think you need this scope.
If it does not work without the scope, check if your scope array looks like this:
scope = "user-follow-read," \
        "user-library-read," \
        "user-library-modify," \
        "user-modify-playback-state," \
        "user-read-playback-state," \
        "user-read-currently-playing," \
        "app-remote-control," \
        "playlist-read-private," \
        "playlist-read-collaborative," \
        "playlist-modify-public," \
        "playlist-modify-private," \
        "streaming," \
        "user-read-private"

Maybe there is an , missing between the last and second to last element?


Auth

While it is possible to authenticate with redis using the current interface, it is somewhat tricky to get right, as one has to remember to send yet another AUTH command before repeating the failed command when an connection error occurs.  It would make some sense for the library to do this.
This adds a PairedConnectionBuilder that accepts a username and password.
Whenever the connection is reestablished an AUTH command is send, and the response inspected before other commands are send.
I have not done too much witch documentation and such yet. I first want to hear if this is a feature that you think it makes sense add, and if yes, if this is the right approach to do it, or if there is some other path I should have taken.

I believe this is ready to merge.

I took this opportunity to do some required house-keeping, including some refactoring to push this logic deeper down so it can also be used for PUBSUB connections and other purposes.
I'll release this as part of a new 0.8 release once I've reviewed my own changes.

This has now been published as part of 0.8.0. It's been renamed to redis_async::client::ClientBuilder, and now has two specific builders paired_connect and pubsub_connect depending on which type of connection required, but apart from that will work in the same way.

Thanks.


namespace variable is required, yet unused

Seems like it can be removed.

addressed in https://github.com/cloudposse/terraform-aws-route53-cluster-hostname/releases/tag/0.3.0


Extracts the process handling out of the splash controller

The splash controller is made of a state machine and a strategy class full of static functions to encapsulate the Win32 API's with the duty of starting and controlling the slide show application. There are a few premises related to its job:

Launcher output stream must be succefully redirected so the child process can read it and display the messages.
The Flutter executable must exist.
Starting the application must succeed.
Detecting its HWND also succeeds.

It's quite easy to predict that the behavior of that class could be a nightmare of if-elses. That's why the state machine design.
Yet, the strategy class that feeds the controller with the Win32 capabilities is doing too much: controlling the application process and the application window. When porting the OOBE to Windows, most of the process handling behavior would be duplicated, but the window control would not, thus splitting those duties in two different components allows for an easier migration to the new OOBE while not breaking the current behavior (required for the upcoming point release).
So, this PR extracts the process creation, termination and spurious termination notification out of the splash controller component. Also, the splash controller is updated to reflect the changes in such a way that user won't notice any difference in the application behavior.

Requesting changes only for the typo 😄. The others are more debatable.


Feature: Allow setting temperature for individual zones

Hey, I just installed this on my new Actron and it's working great, nice job. 🍻
I was looking to control the individual zone temperatures separately but I don't think that's currently supported? I'm new to Home Assistant so correct me if I'm wrong but couldn't you model the zones as climates rather than switches? That way they could have target temperatures and such?

@mattcompiles Good to know its working for you.
I built the integration based on my Neo System at home where I have the Zone Temp/Humidity sensors in each room.
Do you have these Actron Neo Zone controller modules in every room?

Ahh mine are different, they don't have controls on them, they're just sensors.

They do show up in Home assistant though, it's just they show up as switches and can only been enabled or disabled. I don't see a way to set the temperature for an individual zone, which I can do via Neo Connect.

The Actron Neo Sense controller?
This is what I have in every room but from Neo App I can only toggle zones on / off.
Heating / Cooling is controlled as usual.

Do you have the option to control the temps individually for each zone from the app?
Eg. Downstairs 18 degrees, Upstairs 21 degrees, Bedroom 23 degrees, etc?

Ohh interesting. Yeah I can control the zones individually. Here's a screen shot of the neo connect app.

@mattcompiles I have played around with the individual zones as climate entities. Will you be able to test once I have uploaded the code to dev branch?
Also, with the current version can you please send me a debug log file? Just enable it on the integration and reload it so that it dumps the Actron Neo Cloud API info to a log file. If you can send it to me, it will help to understand what your zone controllers are doing.

Hey @domalab
Yeah I can test for sure.
I'll try get you a log file tomorrow.

@mattcompiles I have uploaded the latest code for testing to the dev branch.
https://github.com/domalab/ha-actronair-neo/tree/dev
To test it

Copy the actronair_neo folder into your custom_components directory.
Restart Home Assistant.

Hey @domalab
I gave this a quick go tonight. The new climate entities definitely appeared and they could be enabled/disabled individually. However, the temperature couldn't be set on them, it wasn't giving the option to set at all. I've actually noticed this occurs on the global climate entity as well but only when the system is set to auto. However changing mode didn't fix it for the zone climate entities.
I've also attached a filtered log file for anything coming from the integration. This was generated after updating to the new version though, hope that helps.
home-assistant_2024-10-22T11-45-50.884Z.log

@mattcompiles Thanks for the log file. It helped a lot to compare my system with yours.  and I discovered that your system zone controllers have this
"RemoteZoneInfo": [
  {
    "TemperatureSetpoint_Cool_oC": 19.0,
    "TemperatureSetpoint_Heat_oC": 19.0,
    "NV_VAV": true,  // Variable Air Volume
    "NV_ITC": true,  // Individual Temperature Control
    "NV_ITD": true   // Individual Temperature Display
  }
]

While mine just shows this
"NV_VAV": false,  // No Variable Air Volume
"NV_ITC": false,  // No Individual Temperature Control
"NV_ITD": false   // No Individual Temperature Display

I have made some updates to the integration to detect the zone controller types. I have also combined the humidity sensor  into the temperature sensor. Humidity is now an attribute of the temp sensor.
Lastly, there's also now a diagnostic option in the integration which makes it easier to troubleshoot the integration.
Can you please test at your earliest convenience and send me log files?

Hey Mate, I tested again and got the same result and couldn't change the temperature of the zones individually. It looks like this for reference.

Here's the log file.
home-assistant_2024-10-23T11-43-09.691Z.log

@mattcompiles Thanks for the updated logs.
Just checking, when you tested the new code,

Did you delete the pycache directory in the integration directory? If not, delete it to clear any traces of the integration.
Try to clear your browser history and cache for home assistant just to make sure old entities are not cached.
Remove the integration and re-add it.

@mattcompiles Thanks for checking. Can you please send me the diagnostic log file.
On the integration there's a button called "Download Diagnostics" that generates a json log file.

@domalab Here you go mate.
config_entry-actronair_neo-01JAYGGCC7HBAF14542KAV8JEV.json

@mattcompiles Thanks for doing testing and providing log files. I'm sure we are getting closer to getting it to work.
Its the ActronZoneClimate class in climate.py I need to play around with to properly detect individual zone controllers with the capability to adjust temperature and modes.
Once we get it to work I think I'll look into a way for the integration to automatically detect the zone controllers if installed and capability. No need to configure it then via UI.

@mattcompiles I made a change to the climate file. Zone Climate entities will now be created under entities of the device instead of individual devices.
When I tested it I could only turn zone on/off when selecting mode due to my system not supporting individual zone control but the updated code recognised the zone controller capability.
I'm hoping it works correctly for your system.🤞

Hey @domalab
I updated to the latest this morning but stlll got the same result. I removed the integration before testing as well. The old devices were removed in favour of entities, however I couldn't set the temp for any of them besides the main one. Here's the diagnostics and log file.
home-assistant_actronair_neo_2024-10-28T00-06-00.652Z.log
config_entry-actronair_neo-01JB86WVN379T7P9YZ9N2YFX96.json

@mattcompiles thank you for this info. While I'm spending time to find a solution.
Have a look at this ActronAir Integration that supports both Que and Neo models. You need to have MQTT installed to use this integration.
https://github.com/MikeJMcGuire/hass-actronque

@mattcompiles Hello, just curious, what ActronAir model do you have? I did some reading and apparently the Classic Series which I have does not support individual zone temperature control and just temp/humidity and zone on/off with Neo sensors.
While the Advance series supports individual zone control heat/cool, temp/humidity and on/off.
I updated the code to look for additional attributes to detect the advance series models.

What are the chance adding Aqura Temp Sensors instead of using the Actron Sensors?
I was hoping not to purchase individual neo remote sensors, but to integrate using your integration with Aqura/sonoff temp sensors via HomeAssistant.  And the ability to set the desire room temp by controlling the zone switch when certain temp/humidity are detected.
Would it be possible?

What are the chance adding Aqura Temp Sensors instead of using the Actron Sensors?
I was hoping not to purchase individual neo remote sensors, but to integrate using your integration with Aqura/sonoff temp sensors via HomeAssistant. And the ability to set the desire room temp by controlling the zone switch when certain temp/humidity are detected.
Would it be possible?

I wont be adding support for 3rd party sensors to the code but you should be able to achieve controlling the zones with sensors using automations. Just have to get creative.
Example Scenario: Control ActronAir Zones Using Aqara Sensors
Let's consider a specific example where you have an Aqara sensor in the Master Bedroom and want to control the corresponding ActronAir zone based on the temperature.

Entities:

Aqara Sensor:

sensor.master_bedroom_temperature

sensor.master_bedroom_humidity

ActronAir Climate Entity:
climate.actronair_neo

Automation: Enable Cooling for Master Bedroom

alias: "Master Bedroom Cooling Control"
description: "Enable cooling in Master Bedroom when temperature exceeds 25°C"
trigger:
  - platform: numeric_state
    entity_id: sensor.master_bedroom_temperature
    above: 25
    for:
      minutes: 5
condition:
  - condition: state
    entity_id: climate.actronair_neo
    state: "off"
action:
  - service: climate.set_temperature
    target:
      entity_id: climate.actronair_neo
    data:
      temperature: 22
mode: single

Automation: Disable Cooling When Temperature Falls Below Threshold

alias: "Master Bedroom Cooling Off Control"
description: "Disable cooling in Master Bedroom when temperature falls below 23°C"
trigger:
  - platform: numeric_state
    entity_id: sensor.master_bedroom_temperature
    below: 23
    for:
      minutes: 5
condition:
  - condition: state
    entity_id: climate.actronair_neo
    state: "cool"
action:
  - service: climate.turn_off
    target:
      entity_id: climate.actronair_neo
mode: single


CSS not showing while deploying in Netlify

Netlify only shows html part.How to fix it?I have uploaded everything that was in the src directory

Are you deploying using Github or drag and drop (Uploading files to Netlify) ?
If you are using Drag n Drop. Do run command npm run build  first then upload the dist file.
If you are using Github, make sure in the Netlify settings, in the command filed enter npm run build and the directory put dist
Hope this helps

I am using github and in the Netlify settings when I put dist it throws an initialization error that directory dist doesn't exist

i am deploying using netlify drag and drop and it's not loading
popping up error 404(not found)

I have the same issue. Did you solve that?

Netlify only shows html part.How to fix it?I have uploaded everything that was in the src directory

I have the same issue with godaddy's hosting. Did you solve that?


Large sites and / or sites with large initialState can crash Node due to memory exhaustion

Just leaving a note of this issue here. I will be submitting some PRs soon to address.

[x] Add concurrency option (this alleviates the majority of the issue)
[ ] Consider rendering initialState into the HTML document AFTER HTML has been run through jsdom and minifier. I think large initialState being in the document during these steps is slowing them down, and I can't think of a great reason why it needs to be there.
[ ] Put render jobs into worker threads

I'm seeing this too and curious if there's some sort of memory leak going on? Wondering why memory used by node increases over the render process..

I'm seeing this too and curious if there's some sort of memory leak going on? Wondering why memory used by node increases over the render process..

I don't think it's a memory leak. It's that you get many pages building in parallel and each page's initial state payload takes a bite out of the overall memory budget. Eventually if there are pages that use enough memory during render, this can lead to exhaustion.
You can work around this by setting concurrency to a lower number. The concurrency option did get added, and this solves the majority of the problem. The other items on my list here, which I obvs never did, involve making each page eat less memory during render, basically. I don't think it's just that initial state that is eating up budget - there are additional processes that use more memory and I think those can be optimized.

Hmm.. what I'm seeing is even with concurrency set to something very low, like 5, node's memory usage grows and grows over the course of the build process until it ultimately crashes. Not sure if that could be a memory leak in my application code?

Ah! Hmm, that's not something I ever encountered. For me, setting concurrency completely resolved the OOM issues I was getting. That doesn't necessarily mean that there is definitely not a memory leak in the vite-ssg code, but I would suggest that you re-examine your source to see if there is anywhere you might be accumulating memory footprint and not realizing it.


RO-2434 Remove pool_timeout overrides

RPC-O overrides the pool_timeout to 60 seconds, wheras oslo.db,
SQLAlchemy, and OSA all recommend 30 seconds as a maximum. There
were some bugs around building lots of instances and hitting db pool
timeouts back in the Icehouse/Kilo timeframes, but these bugs have
been fixes for quite some time.
This patch removes the override and uses the OSA default of 30 seconds.
If we're really expecting to wait up to 60 seconds for a successful
database connection, then we're making some serious MariaDB/OpenStack
issues in a deployment.
Issue: RO-2434

recheck_cit_xenial_deploy_swift

Odd - lint failed due to elasticsearch role missing. Might need to rebase or close/open to kick them all off again.

recheck_cit_artifact

Good but needs conflicts merging.

Combined in #2516.

How did we verify that we can boot 400-500 instances at once as our customers provision currently. None of the OPS folks have been consulted and a change of our default parameter is simply not desired unless we can prove that above requirements still work ?
Otherwise please revert immediately !

@devx As said, don't make any changes on default parameters without consulting Jordan or myself. The parameters have been defined as default on purpose so that we don't have to locally override all parameters all the time. Also this cuts down on post installation issues quite a bit, since the configuration was usable for a large set of customer environments. I will follow up offline how that parameter change was verified


update duplicate dependencies after winit update

After winit update in #2186, a bunch of duplicated dependencies changed.
Most are related to a new dependency, https://github.com/onurzdg/mio-misc, that has a few older versions in its dependencies

bors r+


Improve existing codes

As requested in #79, #95 and #131 I did a proof of concept on how we could add new functionalities or fix errors from a set of existing code.
To use the implementation, we need to do the following setup:

In a file called file_list.txt in the root of your project, add a list of code file paths, one in each line. These files should be enough to give the AI agent context to the modification you want to do, but it does not need to be complete standalone execution.
Using the prompt file we add a description of the modification we need to be done. It could be adding a docstring, adding new functions, classes, or even asking to correct a bug.
Call the modification process by using gpt_engineer projects/test --steps improve_code

In this stage of implementation, we are sending all code text in a single message which is not ideal, so it only works on code small enough to be inside a prompt message, but it is a start.
Here is a small example:

This PR is already a working solution. In a future modification, I plan to separate the codes into different messages to see the results.
What do you guys think?

Hey @leomariga great initiative!
Question:
How do you see this being used?
Basically, how does one set up gpt-engineer for a new project?
I'm thinking two options:

create new project with prompt file + memory folder, where workspace symlinks to the existing codebase
gpt-engineer creates a .gpteng folder that contains prompt and memory

WDYT? Maybe there is a third?

Also general feedback:

Python style convention is to use snake case
I think requiring a file_paths.txt is too complex for the user :)

Created a discussion in discord around this

Thank you for your feedback and review =)
I agree with you that manually writing file_paths.txt is not the best user experience. I didn't put much thought into how would be the input method. I also liked the idea of using the .gpteng folder.
(Brainstorming) We could do something like this:

At the root of the existing code repo we call something like gpt-engineer -improve which will create the .gpteng folder.
We write in the terminal what we want to modify. This string will be stored in the prompt file in .gpteng folder.
The tree of files of the project is displayed in the terminal and we select it with numbers (or we could even open the file explorer for the user!)
The gpt-engineer does its job and automatically replaces the file in the end.

I saw the #contributors channel in the discord, but I don't have access to it yet. Let's wait to see the ideas there. =)

Hey, thanks for all the feedback and ideas. I'm working on your suggestions.
I was thinking we could change the file chat_to_files.py to something like utils.py or tools.py and use it to put utility functions, like reading/parsing/writing files or strings.
The methods I put there do not make sense with the file name. I think I should change the location of these methods or rename the file. What do you guys think?
I'm open to ideas.

I think what you say makes sense!
I really prefer specific file names though so it’s easy to find things. Could have chat_to_files and files_to_chat as separate files?
very happy you’re working on this, it’s the top requested feature by far.

As for which files to add, I think we should keep it simple at first and just:

ask if all files in selected path argument should be included
if not, let user use arrow keys and enter to toggle which files/folders should be included (I believe there must be a cli arrow select dependency we could use)

Or we just do this:
We run the tree command, and then we open the output in a text editor, and then the user can delete rows form it and everything that is not deleted when editor is closed is used.
What do you think?

There you go.

Now we only need to call gpt-engineer --improve in the root folder of an existing project. A .gpteng folder will be created by gpt-engineer.
I did some experiments in opening a selection window with python's tkinter instead of selecting the files with the terminal. I think it is much easier to use a gui, but we could implement both if someone insists on using the terminal.

I send a video of this feature working on the discord page.

I've started on a version of this.  My approach is to start with the entry point(s), and iteratively include dependencies within the project.

I've started on a version of this. My approach is to start with the entry point(s), and iteratively include dependencies within the project. Currently creates .filename.meta files for each analyzed file; not unlike .h library files you would have in C describing the contents of the associated .c file.

Nice. The file selection on this solution is the function ask_for_files(). We can add more input methods there if we want =)

Left some comments!
Also adding some thought:
would be great to automatically print the diffs from the generation and ask the human if they like the diffs or not

Quick progress update. Now we can choose how to select the files as follows:

Still working on other stuff.

I want to collaborate on this, please upload PR when available.

A note for you, the naming convention for the methods is incorrect.

If that's important enough, maybe you'd want to fix it.

I am also working on this feature. I have sent you a pull request, @leomariga. @AntonOsika, do you think it would be a good idea to create a branch for this?

I think creating a CLI that you can interact with while gpt-4 is editing the code to be able to iterate indefinitely and tell it to delete, add or change anything. This could make the process of developing software really efficient. Something like a ChatGPT but with code. Like the code interpreter of ChatGPT Plus but better :)
But to achieve this it would be important to get it to always run the code correctly, without problems finding paths, setting up files or doing extra commands.

Preparing review...

PR Analysis

🎯 Main theme: Adding functionality to improve existing code
🔍 Description and title: Yes
📌 Type of PR: Enhancement
🧪 Relevant tests added: No
✨ Focused PR: Yes, the PR is focused on adding a new feature to improve existing code. All changes are related to this feature.
🔒 Security concerns: No, the PR does not introduce any obvious security concerns. However, it's always a good practice to handle file operations carefully to prevent any potential security issues.

PR Feedback

💡 General PR suggestions: The PR is well-structured and the code changes are well-documented. However, it lacks tests to ensure the new functionality works as expected. It would be beneficial to add unit tests for the new functions and integration tests to ensure the new feature works with the existing code.

🤖 Code suggestions:

relevant file: gpt_engineer/chat_to_files.py
suggestion content: Consider handling exceptions when opening and writing to files. This can prevent the program from crashing if there are issues with file permissions or if the file does not exist. [important]

relevant file: gpt_engineer/file_selector.py
suggestion content: The ask_for_files function could be refactored to reduce its complexity. Consider breaking it down into smaller, more manageable functions. This would improve readability and maintainability. [medium]

relevant file: gpt_engineer/main.py
suggestion content: Consider adding a validation for the improve_option argument. If it is not a boolean, the program may behave unexpectedly. [medium]

relevant file: gpt_engineer/steps.py
suggestion content: The improve_existing_code function is quite long and does a lot of things. Consider breaking it down into smaller functions to improve readability and maintainability. [medium]

Great to hear, screenshot looks good!

Hey @leomariga – any news on this?

Done.
I just want to point out that @lectair did interesting modifications in a PR to my branch, but I didn't get a response from him after a review I needed to merge into my branch.
I think we can open a new PR with his changes in the official repo when this PR is merged.

Merged with commits from @lectair =)

Hey Leo great to see this ready to be merged.
Last step, which will take some time, is to merge / rebase in main and resolve conflicts so we can merge.
While doing it I have some quick final improvements:

Stop using a directory called workspace when we run the -i command: It should always be run from the directory that one is currently in. Currently, a folder called workspace is created.
See my comments in the PR

Also, small optional improvements:

Make the "user file_txt" option for selecting files first and default
Number the file selection alternatives from 1 (not 0)
Consider not listing any venv or node_modules folders (as they have so many files)

One more thing:
I think we should simplify the "improve code" prompt, and use another final system message focused on explicitly requesting the output to be in the format we want, just like the gen_code step does (it uses the use_qa prompt).
When I tried it, it didn't give the right format (see screenshot)

Let's go 🏃

Marking this as stale.
Would be great to pick it up, hopefully we don't have to close it.


ONEXPLAYER F1 PRO

Describe the bug
screen is inverted in game mode
What did you expect to happen?
everytime
Output of rpm-ostree status

Hardware
No response
Extra information or context
No response

will need the output of /usr/libexec/hwsupport/sysid and the correct rotation needed in the display settings
also please fill out the rpm-ostree status

will need the output of /usr/libexec/hwsupport/sysid and the correct rotation needed in the display settings also please fill out the rpm-ostree status

its fixed in testing

sudo rpm-ostree kargs --append-if-missing video=eDP-1:panel_orientation=left_side_up you can use this to fix it in stable
Only EVA variant is broken in stable i think


Are ICI-Kt significant edges a superset of the other methods?

As in, do we get more significant edges from ICI-Kt than the other methods?
Look in Eugene's manuscripts for a Jaccard like thing that tells us if it's a fuzzy superset.

So, we should use the overlap coefficient (assuming that ICI-Kt sets are always bigger), and Jaccard coefficient for this evaluation.


remove hardcoded rbac proxy image

https://github.com/open-cluster-management/backlog/issues/13410
Signed-off-by: Marco<EMAIL_ADDRESS>

/lgtm


Format code using prettier and ESLint --fix

Fixes #4

thanks

I think we could close this one and continue here: https://github.com/nodesecurity/eslint-plugin-security/issues/4#issuecomment-1079003535
Even if we decide to go with this setup, probably it is going to be easier to run ESLint again to fix the codebase.

Sounds good, thanks @jesusprubio


Very slow permutationCombination with 10 element array

https://runkit.com/giorgio79/5c17cba0317a9a0015756c65
Looks like permutationCombination for an array of 10 is very slow. Any ideas?

Permutations for that array are up in the million, so I guess that is understandable. Closing issue.


IntersectionObserver should have default constructor with callback

Should be like in MutationObserver https://api.dartlang.org/stable/1.24.2/dart-html/MutationObserver/MutationObserver.html. This is crucial because callback is the only way to get notifications, and it is missing from IntersectionObserver.

See also https://stackoverflow.com/questions/50147257/dart-intersectionobserver?noredirect=1#comment87397450_50147257

@kevmoo is intersection observer receive this callback in near future?

CC @terrylucas @vsmenon


Add reference attributes to PisaSales based data classes

REVIEW (be65ecb)
I hope I adjusted everything that can be done from the "client site".