[NON-IMPACTING IMPERFECTION] Stale reference to debian/cargo.config breaks (unsupported) use of make

Describe the imperfection
This is not a bug and it doesn't need any corrective action. It might be a "cleanup", though- either removing a line from the Makefile or removing the entire file if it's not actually used anymore
Here's the problematic target from the Makefile, specifically the cp debian... command:
$(TARGET)/$(BIN): extract
	mkdir -p .cargo
	cp debian/cargo.config .cargo/config.toml
	cargo build $(ARGS)

The debian/ directory is no longer in the feroxbuster repo, I believe because you moved what was in it into the main Cargo.toml file at some point. Because make is all about the zeros and non-zeros, make fails before building
Should that cp debian.. line be removed the Makefile? I'm not asking that you support users building via make (unless you really want to) but I figured if the Makefile is going to be present, might as well make it work, right?
I only stumbled upon this because I had to add feroxbuster to some build automation stuff that has a lot of applications with the traditional configure && make && make install pattern. At that time I noticed that there was a Makefile with feroxbuster. I usually just use cargo directly and it works very well, but I thought maybe there was an opportunity to make things look more uniform :)
I apologize for taking your time up with something so trivial. Feel free to close with a "kthxbye" and no action taken. If you still use the Makefile, maybe you'll want to just remove that line. Or maybe the Makefile is completely moribund at this point and it can be destroyed?
Until this CRITICAL FLAW  is corrected, I will be forced to continue using cargo build --release && cp target/release/feroxbuster /opt/$APPS/bin/ - a real tragedy :)
Thanks again for all your work on this project

ha! thanks for this. It's from when i was getting ferox ready to be added to the kali repos. I THINK they don't need this anymore, but, to be on the safe side, i fixed up the makefile in such a way that if they are relying on this one, it won't break anything AND it should be fixed in the future if anyone else runs make :grin:


pmd: 6.8.0 -> 6.9.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/pmd/versions.
meta.description for pmd is: '"An extensible cross-language static code analyzer."'.

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0/bin/bgastviewer.bat had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0/bin/cpdgui.bat had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0/bin/designer.bat had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0/bin/pmd.bat had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0/bin/cpd.bat had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0/bin/run.sh had a zero exit code or showed the expected version
0 of 6 passed binary check by having a zero exit code.
0 of 6 passed binary check by having the new version present in output.
found 6.9.0 with grep in /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0
directory tree listing: https://gist.github.com/3b7b93edc6a75dbc54df484dc5ef80a4
du listing: https://gist.github.com/f35a10e0e7d2aee0ec83d4049556646f

Outpath report (click to expand)

Outpath difference report
4 total rebuild paths
1 package rebuilds
1 x86_64-linux rebuilds
1 i686-linux rebuilds
1 x86_64-darwin rebuilds
1 aarch64-linux rebuilds
First ten rebuilds by attrpath
pmd

Instructions to test this update (click to expand)

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

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A pmd https://github.com/r-ryantm/nixpkgs/archive/b83d7282e3a22796048f689b279267bfc3476850.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0
ls -la /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0/bin

Success on x86_64-darwin (full log)
Attempted: pmd
Partial log (click to expand)
/nix/store/n2zqq5jzdp7l64nhna8bs1q32jpnm99j-pmd-6.9.0

Success on x86_64-linux (full log)
Attempted: pmd
Partial log (click to expand)
no Makefile, doing nothing
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0/lib  /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0/bin
patching script interpreter paths in /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0
/nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0/bin/run.sh: interpreter directive changed from "/bin/bash" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0...
/nix/store/hcxrn5cczkp298b3g8gf690flmwpkq7g-pmd-6.9.0

Success on aarch64-linux (full log)
Attempted: pmd
Partial log (click to expand)
/nix/store/xam9wrkffzxjisz2l842qn51v4mx7gaw-pmd-6.9.0


React step 3

Closes #
finished tutorial step 3
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

step3


Feature request: support for the SIO_GET_EXTENSION_FUNCTION_POINTER WSAIoctl ?

Motivation
Hi,
I am trying to port a Winsock2 application which use Overlapped I/O and need to get a function pointer to ConnectEX or AcceptEx (among other functions) on a socket.
I don't seem to find any of the WSAID_* I need defined in the windows and sys crates, although I do find type definitions for the LPFN_*.
Is the support for these features coming later or should I do it differently?
Drawbacks
No response
Rationale and alternatives
No response
Additional context
No response

Yes, these GUID constants are missing.

I'll transfer to the Win32 metadata repo for resolution.


compat with trinkets slot

Can TIFYI tap into Trinkets slots? Giving it optional compatibility if trinkets is present?

basically only the 3 red marked ones - the left outer ones ain't needed
EDIT: don't worry about the offhand/shield thingy - it's just me using the inventory.png from TIFYI with vanilla slot configs

Yeah, I'll look into supporting Trinkets' slots in TIFYI. Trinkets doesn't have a way to change slot locations in their API so I have to write some really gross code to hook into it.
I've actually done it before, for another project of mine, LittleBigInv: https://github.com/Andrew6rant/BigInv. (see how the minecart slot and disabled slots are aligned with the offhand icon)

That mod hard depends on Trinkets, so I can directly modify its code. I don't want to do that for this mod, so I also need to make a mixin plugin.

TIFYI will not move the trinkets slots on the left to the right, that can be done through datapacks with just Trinkets API, I'm pretty sure.

I haven't started on Trinkets compat (because I need to make a mixin plugin for it), but I've implemented configs to move the recipe book and "crafting" text (also added a config to change the size of the player inventory render)


extremely slow infering

I wanted to render infer with the model on I2V following the example in the readme and got infinte slow time. Whereas on Comfyui I ran a video generation in around 10 minutes (without lora). As seen below:

I ran on windows 10 with a 3090 a on conda env, python 3.12.7, torch 2.5.1, diffusers 0.31.

HI, it may be caused by the lack of GPU resources. Now we have released the online huggingface demo: https://huggingface.co/spaces/fffiloni/DimensionX
You can try our model online! It would be much faster!

HI, it may be caused by the lack of GPU resources. Now we have released the online huggingface demo: https://huggingface.co/spaces/fffiloni/DimensionX
You can try our model online! It would be much faster!

As mentioned, I run the inference on a 3090 RTX and the model works well into comfyui

I'm also experiencing very slow inferring - 120s/it, resulting in about a 90-minute total time.
Windows 11, RTX 4090, Python 3.12, Torch 2.5.1.


What does nativeBuildInputs do?

It is not at all clear to me what the difference to buildInputs is.
See also https://github.com/NixOS/nixpkgs/issues/1915

stuff like flex/bison/pkg-config/cmake should hit there.  But also it does matter only on crosscompiling, when buildInputs resolved as target arch, and nativeBuildInputs is build arch.
btw would be nice if you retitle issue about fix documentation to be more clear

See also https://github.com/NixOS/nixpkgs/issues/1915

Let's extend #1915 not to have two issues for the same thing :)

btw would be nice if you retitle issue about fix documentation to be more clear

That’s why I linked #1915.
Now it’s just a duplicate of that, so I’m changing the name back.


configured leaflet option scrollWheelZoom to prevent zoom issue when …

merge changes to prevent scroll wheel zooming (from marketing site) for deployment. See https://github.com/codeforgreenville/trolley-tracker-marketing/issues/11

Created new issue: https://github.com/codeforgreenville/trolley-tracker-web/issues/1

Acknowledging that I need to move to production.  Will try and get to it tonight.


Deserialize complex structure with dynamic defaults

I have a structure similar to the following:
pub struct Data {
    a: i32,
    b: bool,
    c: String
}

pub enum Obj {
    A(Vec<u8>),
    B(Data)
}

and I would like to deserialize to objects of type Obj. I thought I would do this using Visitor, which has been working -- however, I'm not sure how i'd implement dynamic defaults for the fields in Data. These defaults would be unknown until runtime -- that is, if I passed in { "a":1, "c":"lol" } with a default of { a:2, b: false, c: "kek" }, I would get B({ a:1, b:false, c:"lol"}). I can currently think of two ways to do this:

Create and implement a trait requiring [field]Default methods that is passed around as a type bound -- that is, Obj becomes Obj<P:Defaults> (this seems like a terrible idea).

Keep a global/static "defaults" struct singleton that gets modified before calling deserialize (this seems like a "better" idea, as I don't need threads or anything, but I've read that using global mutable state in Rust is frowned upon).

Check out DeserializeSeed. You would make some type to hold the runtime defaults and implement DeserializeSeed on that type with Value=Obj.

Ah, thanks!

Can serde also take Default trait into consideration? It's redundant to implement default value several times.

Yes, the following would use Default's value for any missing field.
#[derive(Deserialize)]
#[serde(default)]
struct BusyJay {
    a: i32,
    b: bool,
    c: String,
}

impl Default for BusyJay {
    /* ... */
}

Thx for the explanation. I did miss the container attributes. :)


Try a banner for VSP intead of the Cohorts nav item

Right now in the main nav we have a "Cohorts" link. It goes directly to the VSP landing page.
The ideas is for "Cohorts" to be a drop down that lists all the Cohorts. This could work in the future, but right now it's just confusing because we only have one.
For now, let's try this:

Remove the Cohorts item from the main nav
Display a big banner that lets people know VSP is our first cohort and we're accepting applications. It should be a link to the VSP landing page

fixed in #59


Add dev-server to help preview catalog

Works toward completing https://github.com/sensu/sensu-enterprise-go/pull/2082.
Usage
# optionally watch file system for updates
catalog-api catalog generate --repo-dir ../catalog --watch

# dev server
catalog-api catalog server --repo-dir ../catalog

# one can optionally watch w/ dev server too
catalog-api catalog server --repo-dir ../catalog --watch

Drawbacks
Originally, I had hoped to use a virtual file-system for processing and serving the assets, however I realized it was going to require a bit more refactoring than I wanted to do. Considering the resulting API is ~150MBs, I would recommend mounting a RAM disk and using the --temp-dir flag.

@jamesdphillips this is a great start. I just committed a change to log out the listen address after the listener starts.
The current behaviour of the server command isn't very useful without specifying the -snapshot flag. Perhaps we should remove that flag from the command and force set the config option to true. Any thoughts?

The current behaviour of the server command isn't very useful without specifying the -snapshot flag. Perhaps we should remove that flag from the command and force set the config option to true. Any thoughts?

I was thinking about this last night as well, it occurred to me that --without-snapshot might be best approach.

I concur. Make it so!

team effort


Develop: Add discovered email addresses and phone numbers to Mail Actions panel

As a user looking at a piece of mail, I want to be able to use my phone's capabilities to text, call, or email any phone numbers or emails found on the mail image.
See mockups: https://xd.adobe.com/view/db7cd80e-542d-495d-8909-01d4b2607dbe-8e2f/screen/7e31c20e-0f52-470c-88d2-4912f3564f3d
Acceptance Criteria
The following is added to the Mail Actions panel below each mail image:

Mail caching process is updated to retrieve all emails and phone numbers on import
MailPiece data object is updated to hold the set of emails and the set of phone numbers
All emails and phone numbers from the MailPiece are added to the Mail Actions panel
Email addresses can be clicked to launch the default OS new email message to that address
Phone numbers can be clicked with an option to either text or call the number using the OS functionality
Document any relevant content in the Programmer guide (draft thoughts of usage and importantly the "Why")
Updates SRS and STP (and other docs if necessary)
Add/Update relevant unit tests

tested on android


Incorrect URL is being built when using relative paths.

Hi,
In my application I have the backend serve a static page at a specific path /clients/index.html and client api root is /clients/api
Once the  /clients/index.html is loaded, I need to get client information using the end point /clients/api/info.
I am using express for the backend and react for the frontend leveraging axios along with the axios-cache-interceptor.
However when using the axios instance (decorated by setupCache) with the baseURL  set to : '/clients/api' and the reqeust url set to /info,  I am noticing a 404 error in the console.
Page URL: http://localhost:3010/clients/index.html
Clients Information Endpoint: http://localhost:3010/clients/api/info
Working code:
var clientsApi = axios.create({
    baseURL: '/clients/api'
  });

  clientsApi.get('/info').then(a => {
    console.log('Received response: ', a.data);
  }).catch(err => {
    console.log('Error using vanialla axios');
    console.error(err);
  });

**Failing code: **
var clientsApiWithCache = AxiosCacheInterceptor.setupCache(axios.create({
    baseURL: '/clients/api'
  }));

  clientsApiWithCache.get('/info').then(a => {
    console.log('Received response using setupCache: ', a.data);
  }).catch(err => {
    console.log('Error using setupCache:');
    console.error(err);
  });

I have debugged the code and see that the issue occurs because of this line: https://github.com/arthurfiorette/axios-cache-interceptor/blob/dc27cd5b53b2d8847e8821a4607c7e3f25927bbc/src/util/key-generator.ts#L49
Any specific reason for trimming the slashes from the baseURL and url?
The repo @ https://github.com/Chandu/axios-cache-interceptor-relative-base-path-issue can be used to reproduce the issue (I haven't used react in this repo for simpler setup, but the error can be reproduced ).
Screenshot showing the code/error

When I get home, I'll se it. Just a sec

Any specific reason for trimming the slashes from the baseURL and url?

Yes, but not in the current way. :p
These changes should only be kept within the scope of the keyGenerator. Fixing it...

Tested it and everything worked as intended. I'll release 0.9.3 tomorrow.
If everything seems good, can you close this issue?

Awesome Thanks @arthurfiorette
Will check with the latest version once released.

@Chandu v0.9.3

Thanks @arthurfiorette.
I am closing this issue as I was to verify the fix successfully


prep 0.0.9

Once this goes out we should send a PR to Warehouse using these new APIs.

https://github.com/trailofbits/pypi-attestations/releases/tag/v0.0.9


Broken Documentation for 0.14

As I need to supprort IE8 I managed to get 0.14 working for that browser. Can you please restore the documentation css as it's broken on your website at the moment? Hope you're going to retain it.

Closing as the library does not support IE8 and 0.14 is an old version.


[ ZEPPELIN-1266 ] Code editor Optimization

What is this PR for?
If the result of many para- graph, the code editor is very slow.
What type of PR is it?
Improvement
Todos

[x] removed call to function in template.
[x] changed event (aceChange event to ace input event)

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1266
How should this be tested?

going to r tutorial notebook.
write to paragraph on anyway. (many many.. fast.. fast typing.)

Screenshots (if appropriate)
Before optimization.

After optimization.

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

ci trigger.

please, remove to browser cache

Would it make sense to change isParagraphSaveAlready => wasParagraphAlreadySaved ?

@bzz Thank you for your feedback.
i will check to your opinion.
Thank you

I would limit this PR to the first commit, for me the rest is out of scope.
Also, in the first PR, there is no need to make a if before calling setParagraphMode, the conditions you added is already at the beginning of the function.
$scope.setParagraphMode can also be changed to var setParagraphMode

@corneadoug
I did modification.
Please review for this pr.

@bzz
I removed the parts which you say.
Thank you.

Looks great to me!

@cloverhearts Thanks for the changes and keeping this PR focused.
I saw some great improvements with big Notebook with those changes.
@Leemoonsoo Can you try it too and tell us if you also think its better?

Merge if there is no further discussion


A inifinit increasing size .tar.gz file created during smart contract compiling

Problem Description
Building smart contract with docker mode, pchain-compile will create a temporary .tar.gz file to collect

files inside smart contract source code directory
manifests (dependencies)

and compress them, then copy them into docker container. It is possible, that pchain-compile the temporary .tar.gz become a infinite size growthing file until eats up all hard disc memory.
Problem analyzing
Source code files and manifest files are collected GZencoded, and copied by BFS on directory given by flag "--source". If run pchain-compile inside the smart-contract's source code directory, since final .tar.gz file will kept increasing during the whole compress file creation process,and itself is also file will be counted into the process, the whole prosess will not be stop forever.
Fix

Good thing is the library "tar.rs" which we plugin to create the .tar.gz provide the Too many levels of symbolic links check inside. Therefore, we do not need to take care case, that circulation dependency in manifest or infinite loop caused symbolic links.
Change the location where the temp file .tar.gz created, to avoid creating a infinite increasing file when user run pchian-compile insde the smart-contract's source code directory.
"target" directory should not be collected, since it might make the .tar.gz become very large

Solve this by branch dev/v0.4.4


Build rubies with suffix, keg-only when relevant

The rubies are keg-only and installed without a suffix. So one ends up
with:
/usr/local/Cellar/ruby22/2.2.5/bin/ruby
And a warning about the formula being keg-only preventing symbolic link
for ruby to be put in /usr/local/bin/ruby.
When one wants several rubies, each will typically be build with
--with-suffix and end up with:
/usr/local/Cellar/ruby22/2.2.5/bin/ruby22
The binaries still have to be manually linked manually. Eg:
brew install ruby21 --with-suffix
brew link --force ruby21
brew install ruby22 --with-suffix
brew link --force ruby22

Default all rubies to use a suffix unless --without-suffix is passed and
hence no more mark them keg-only.
A gotcha is some of the rubies versions we ship are indeed provided by
Mac OS X. For example 2.0.0 on Sierra, hence when building without a
suffix, mark the recipe as keg-only when the same ruby version is
provided by Apple.
For reference the Mac OS X ruby versions can be obtained on
https://opensource.apple.com/ browsing to the ruby package and looking
at the ruby.plist file.

Ver
Name
Ruby version

10.12
sierra
2.0.0-p648

10.11
el_capitan
2.0.0-p648

10.10
yosemite
2.0.0-p481

10.9
mavericks
2.0.0-p481

10.8
mountain_lion
1.8.7-p357

10.7
lion
1.8.7-p357

10.6
snow_leopard
1.8.7-p174

10.5
leopard
1.8.6-p287

closes #1357

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same formula update/change?
[ ] Have you built your formula locally prior to submission with brew install <formula> (where <formula> is the name of the formula you're submitting)?
[ ] Does your submission pass brew audit --strict --online <formula> (after doing brew install <formula>)?
[ ] Does your submission adhere to Versions "Acceptable Formulae" guidelines?

Great work here. Another option if you feel like the extra work (and feel free to say either no or make it a follow-up PR) would be migrating to homebrew/core like e.g. https://github.com/Homebrew/homebrew-versions/pull/1502 and https://github.com/Homebrew/homebrew-core/pull/8376.

Thank you Mike, that is encouraging!   I have refreshed my branch and fixed the few style issues reported rubocop (and discovered brew style).
I would rather have this change land in homebrew-versions, then delete it and import it in homebrew/core. Will work on that and send a pull request on homebrew-core.

Easy enough: https://github.com/Homebrew/homebrew-core/pull/8767  \O/

Passing on this in favour of https://github.com/Homebrew/homebrew-core/pull/8767. Could you open a PR to delete these formulae and add them to tap_migrations.json instead? Thanks!


Update website/materials with new IWC graphics

--

I shouldn't be lurking around here anymore, but this graphic is pretty freaking sweet.

Added them to gh-pages, images are infographic_full.png and infographic_min.png.
Put the full size up on the homepage
de93b15
Re open if you need me to change anything!

haha @blaino, we expect you to lurk.
@Kevin-K - that's perfect, thanks!

Love the graphic @bhalt, Lmbo @blaino, and Hi :hand: @Kevin-K lol


Latest version of --ambient definitions not installed.

Hey,
I've been trying to work out today why Typescript can't see my install angular-resource module. After digging around I found that the latest isn't being installed when running.
typings install angular-resource --ambient --save

Which is installing c6609aff88f2c59e4df9adb93df5c7932adfd7b4
Basically there was a fix in this commit https://github.com/DefinitelyTyped/DefinitelyTyped/commit/621f12c6602bcc569bc6ad3a67a0af3ba3832618 which isn't being pulled I realised that this is being deprecated but seems odd that this doesn't get pulled when running installed.
When installing via tsd
tsd install angular-resource

This will install the correct one 621f12c6602bcc569bc6ad3a67a0af3ba3832618 which makes it look like there is a bug with the way that when using the --ambient flag how it checks the latest version.
Happy to perhaps help out on a fix for this but not really sure where to start even looking.
Thanks

Most likely a big with the API indexing found here: https://github.com/typings/api. I'm going to close this, it's most likely an issue that something has indexed out of order and https://github.com/typings/api/issues/17 should be the best fix for that. For now, you can install that hash manually using typings install github:DefinitelyTyped/DefinitelyTyped/angularjs/angular-resource.d.ts#621f12c6602bcc569bc6ad3a67a0af3ba3832618.


Maintainer wanted!

Hello folks,
Unfortunately I have to inform you that I will not develop this project any further at the moment.
I've completely switched to Tasmota devices, so I don't need this interface anymore.
Please feel free to develop and use the project further!
Best regards
TheAgentK

Hi TheAgentK,
I'm willing to take over maintenance of this project as I have a number of enhancements I'm currently working on, some already in my tree, including compatibility with TuyAPI 5.1.1, including device auto-discovery and Tuya protocol 3.3. support.  Also, in queue is Home Assistant MQTT discovery support and perhaps even a hassio plugin for Home Assistant.  I'm also maintaining a ring-alarm-mqtt script used mostly with Home Assistant so it fits it with that.
Let me know if you're interested and we can discuss the details.

Hey @tsightler,
that'd be great. I'll add you as a collaborator in the project. Everything else we can clarify then if necessary privately. Thank you very much for your support. If I use Tuya devices again sometime, I will also support this project again. If you need more privileges or anything else, just contact me.
A little off-topic, where do you come from? I am from Germany.

That sounds great @TheAgentK.  I'll begin reviewing issues and checking on the possibility of merging some of the outstanding pull request.
I do have one question.  I notice in tuya-device.js you have a custom "get" method but it appears to be nearly identical to the get function that already exist in tuyapi 4.0.3.  In your comments it says it includes more logging, but I couldn't find any actual difference.  Is this perhaps something left over from a previous tuyapi version.  I see that the tuyapi get() code was much less robust in the early 3.x days.  I had problems with the custom get() function with the tuyapi 5.x API, so I completely removed it in my tree since I couldn't see how it was different and everything seems to work fine, but I wasn't sure if I was missing something.
I am located in southeastern US (South Carolina).  I work with a number of colleagues in Germany on a regular basis in my day job.

Looking at the custom get a little deeper I see there is an extra try/catch section but it's still not clear to me why you needed this.  It seems this was added when you integrated support for 4.0.x tuyapi.  Was there some specific bug you were attempting to work around in tuyapi?

Hey @tsightler ,
excuse me, I have so little time right now.
The try/catch block should fix a bug.  Sometimes the DPS object in "data" was not correct. This caused an error when calling resolve(data.dps['1']]);.
Maybe this has already been fixed.


Implement EventScrubber

Add a new EventScrubber class that scrubs certain potentially sensitive interfaces with a DEFAULT_DENYLIST. The default scrubber is automatically run if send_default_pii = False.
Usage
from sentry_sdk.scrubber import EventScrubber, DEFAULT_DENYLIST

# custom denylist
denylist = DEFAULT_DENYLIST + ["my_sensitive_var"]

sentry_sdk.init(
    # ...
    send_default_pii=False,
    event_scrubber=EventScrubber(denylist=denylist),
)

As outlined in https://github.com/getsentry/rfcs/blob/main/text/0062-controlling-pii-and-credentials-in-sd-ks.md
closes #1897
docs: https://github.com/getsentry/sentry-docs/pull/6489

I get that having those seperate scrubbing functions makes it convenient for a user to derive a custom scrubber that works like the original one, but does not scrub the breadcrumbs.

But maybe add a scrub_others or scrub_rest
That does something like:
for key in event.keys():
    if key not in ["request", "extra", "user", ...]:
        self.scrub_dict(event[key])

Why not just walk the whole event structure recursively and apply scrub_dict()?

Only python has the Event as an opaque dict/object without interfaces, no other SDK does this, so I just went with a structure that would work for all SDKs. The recursive walking can only work in python sadly.
I think this is fine for a first version and we can see how and if users use this at all and wait for feedback.
I just wanted to have something like this as a first-class abstraction and this is a good first step.

Ok.
How can we ensure, that if we add some top level key to the event in the future, that we do not forget to update the default EventScrubber? Because we (or whoever is than working on this) will forget about this.


AVRO-2833: Fix python whitespace for lint

Make sure you have checked all steps below.
Jira

[X] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-2833
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason: No tests necessary, part of the build.

Commits

[X] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

(sorry hijacking this PR to test if notifications into issues@ is working)


Refactors many exporters to use toolchain flags

Toolchains are now expected to have an up to date flags attribute. These flags are used by the tools when building from the scripts and by the exporters when exporting projects.
Partially resolves #1671 and #1525
I say partially because some of the exporters have yet to be updated to pull from the toolchains.

cc @0xc0170

Thanks for this . Looks good ! Have you tested that make.py creates the same binary size as the exported project (aka we use the same project settings). With some toolchains we used to have 3-4x difference :-)

I have not had a chance to test it. I also don't have many of these IDE's.

Okay, Tested with GCC_ARM and the flags are all the same.

OK, I'll run some local tests soon

I had a quick try, IAR failed for me, I'll come back to this later after the weekend to do more testing

@0xc0170 I think I found the problem see line 126 of iar.py.

Rebased after workspace_tools -> tools changes.


[ReadME Update] Adding UI Playground to compare across Gorilla, GPT-4, and Claude-2

Hi @ShishirPatil @eltociear,
We built a hosted UI playground for people to compare Gorilla's LLM to GPT-4 and Claude-2
Adding a link to it so people can easily test Gorilla!

Hey @krrishdholakia cc @eltociear
I was about to merge this, but I was unable to pick a gorilla-7b-hf model from the dropdown. Am I doing something wrong or did you folks decide to support a different set of models? Thanks!

Hey @ShishirPatil - it's still supported - we were making some updates on the backend server - improving stability, it should be available by EOD


CS0436 (duplicate type) when using InternalsVisibleTo and project references

Suppose project ConsoleApp2 depends on project ConsoleApp1
Project ConsoleApp1 depends on PolyType.
Project ConsoleApp1 has [assembly:InternalsVisibleTo("ConsoleApp2")].
This means the following type is duplicate in ConsoleApp2's build:
namespace PolyType.SourceGenerator
{
    /// The source generated <see cref="global::PolyType.ITypeShapeProvider"/> implementation for the current assembly.
    [global::System.CodeDom.Compiler.GeneratedCodeAttribute("PolyType.SourceGenerator.PolyTypeGenerator", "0.18.1+5e98a961a4")]
    internal sealed partial class ShapeProvider
    {
        private const global::System.Reflection.BindingFlags __BindingFlags_Instance_All =
            global::System.Reflection.BindingFlags.Public |
            global::System.Reflection.BindingFlags.NonPublic |
            global::System.Reflection.BindingFlags.Instance;

        /// Gets the default instance of the <see cref="ShapeProvider"/> class.
        public static ShapeProvider Default { get; } = new();

        /// Initializes a new instance of the <see cref="ShapeProvider"/> class.
        private ShapeProvider() { }
    }
}

Love the look of the project BTW! I have been building my own CSV and Kusto source generator from POCO for a while. This might be a fun project to simplify with.

Great to hear that you like it!
What solution would you recommend? It is conceivable that the generator could just suffix the class name with an identifier deriving from the name of the assembly, but I'm wondering if there's a better way.

I can think of three options:

Disambiguate the name automatically or by configuration (as you stated)
Add a hook to allow assembly aliasing. Seems like a pain and may not even work for project reference (never tried).
Not sure of feasibility -- but could ConsoleApp2 re-used ConsoleApp1's implementation?

I am not sure if existing users of the library expect the shape provider to be called ShapeProvider. If so, option 1 would need to be opt-in (or enabled only when InternalsVisibleTo is detected).


The Credits window does not close

Press any key to get back, EXCEPT FOR THE ARROW KEYS...

But none of the keys work.
And please check out my fork of Rafiano

Thank you! The issue has been fixed, and the new song has been added. I’m very grateful for that! I will close this issue for now.


Add flex-wrap to Links for mobile

Closes #14
I added flex-wrap to the Links section to stop them being cut off.
Potentially could wrap the links in a container and center them like so:
<v-container>
      <v-row align="center" justify="center">
        <v-col cols="12" class="text-center">
          <v-btn
            variant="text"
            href="https://github.com/localsend/localsend"
            target="_blank"
          >
            Github
          </v-btn>
          <v-btn
            variant="text"
            href="https://discord.gg/GSRWmQNP87"
            target="_blank"
          >
            Discord
          </v-btn>
          <v-btn variant="text" to="/privacy">Privacy</v-btn>
          <v-btn variant="text" to="/terms-of-service">Terms</v-btn>
          <v-btn variant="text" to="/contact">Contact</v-btn>
        </v-col>
      </v-row>
    </v-container>

Thank you!


Load-balancing destination address

Add support for passing multiple destination addresses for one source address. Should round-robbin the destination address to use for each incoming connection on the source address.
As suggested by @DanielBodnar

I'm curios what is the use-case for this feature?
(Shouldn't the load-balancing be handled at the other end?  I.e. somewhere between the Wireguard router and the actual remote server?  One could easily achieve this in a number of ways.)

If you have web servers on the peers:

<IP_ADDRESS>:8080
<IP_ADDRESS>:8080

The objective is to set up a load balancer (LB) on a third system to round-robbin requests from the 2, and expose that to the internet. If you use something like HAProxy or Nginx Plus, you'd need to have WireGuard installed on the LB so that the LB software is able to connect to those peers by IP.
However you can also run 1 onetun instance that round-robbins between the 2, and opens a single port on <IP_ADDRESS>. Which means:
                             <IP_ADDRESS>:8080
                          /
<IP_ADDRESS>:8080 -> [onetun]
                          \
                             <IP_ADDRESS>:8080

Of course, you could also run multiple forwards of onetun (#2) and let an LB software do the round-robbin, but that adds an extra step in the chain:
                                <IP_ADDRESS>:8081 -> [      ] -> <IP_ADDRESS>:8080
                              /                 |      |
<IP_ADDRESS>:8080 -> [LB software]                   |onetun|
                              \                 |      |
                                <IP_ADDRESS>:8082 -> [      ] -> <IP_ADDRESS>:8080

For what it's worth, I don't envision onetun being purely "utilitarian" like you said. It definitely has its uses for quickly accessing a port from a dev machine, but my own primary use-case is to set up long-running proxies for public services that are hosted from peers inside my WG network. I will keep pushing for stability and performance as much as possible.
You make a good point that letting the LB software handle the round-robin and health checks would help onetun be "leaner". This will only be possible once #2 is implemented though, since separate instances of onetun would need to have separate IPs/keys configured (which is a bit of a chore).


Nested Turbo Frames appear to confuse Turbo Drive on error.

Turbo Version: 7.3.0
I'm currently converting a legacy application to use Hotwire and I'm running into something confusing with nested <turbo-frame> elements, specifically when the inner frame is being operated via a button in the outer frame. Let me illustrate. Imagine I have a form like this where the DOM layering goes:
Outer Frame -> Form -> Inner Frame
In the following HTML, notice that I have a button that retargets the form submission using data-turbo-frame="inner-frame".
<turbo-frame id="outer-frame">

	<form method="post" action="process-outer.htm">

		<turbo-frame id="inner-frame"></turbo-frame>

		<button type="submit">
			Outer Submit
		</button>

		<button
			type="submit"
			formAction="process-inner.htm"
			data-turbo-frame="inner-frame">
			Inner Submit
		</button>

		<button
			type="submit"
			formAction="process-inner.htm?error=true"
			data-turbo-frame="inner-frame">
			Inner Submit With Error
		</button>

	</form>

</turbo-frame>

The process-outer.htm file just replaced the outer frame with some static content:
<turbo-frame id="outer-frame">

		Outer frame processed!

		Reset

</turbo-frame>

The process-inner.htm file also has static content; but, allows an error status to be returned:
<cfscript>

	if ( url.keyExists( "error" ) ) {

		cfheader(
			statusCode = 422,
			statusText = "Unprocessable Entity"
		);

	}

</cfscript>

<turbo-frame id="inner-frame">

		Inner frame processed!

		Reset

</turbo-frame>

Ok, now if I try to submit the inner-frame button without an error, everything works fine:

Since my process-inner.htm file has no mention of the outer-frame id, we can safely assume that only the inner-frame has been replaced.
Now, if I try to submit the form with the Inner With Error button, which is the same page but with a non-200 status code, I get this:

As you can see, with a non-200 response, Turbo Drive is actually looking for the wrong frame to replace.
I'm pretty sure this is a bug in the system. It doesn't seem that the change in status code should change the frame being targeted. Thoughts?

As an experiment, I've tried moving the inner-frame outside for the <form>, thinking it might be confusing it. But, the same error occurs.

I will add that if I comment out the outer-form, then the inner-form processing w/ error works fine (replacing just the inner-frame content). So, it seems to be something specifically with the nesting.

I also ran into this and it's quite annoying.


feat: restructure junit reports

Explanation
Restructure junit reports.

/cherry-pick release-0.2


miscies

When we could not get the token then let the user know
Set error message as markdown
GetCommitInfo ASAP in the Run
Move ACL operations to the acl package for common

Codecov Report

Merging #260 (2e88f58) into main (88d2644) will increase coverage by 0.31%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##             main     #260      +/-   ##
==========================================
+ Coverage   70.23%   70.55%   +0.31%
==========================================
  Files          27       28       +1
  Lines        1690     1698       +8
==========================================
+ Hits         1187     1198      +11
  Misses        389      389
+ Partials      114      111       -3

Impacted Files
Coverage Δ

pkg/pipelineascode/pipelineascode.go
72.61% <81.25%> (-0.11%)
:arrow_down:

pkg/webvcs/github/acl.go
82.89% <87.50%> (+3.37%)
:arrow_up:

pkg/acl/acl.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Remove net6.0 multitarget for improved compatibility

As described at:
https://github.com/Azure/azure-sdk-for-net/issues/44709
These problems are specific to using the OpenAI library via an intermediate class library targeting netstandard2.0; in that case, the multi-target is causing the executing project to pick up the uplevel framework target that's incompatible with downlevel intermediate library.
Although there may be other approaches to mitigate, it seems reasonable to not expressly multi-target net6.0 if netstandard2.0 is sufficient.

Hello all,
Is there any likelihood of this PR being merged? The targeting issue seems to be a blocker for me; if there is some resolution (either via this PR or some other method?), it might unlock a lot of use cases.
Thanks in advance
JS


DEBUG： debug cacert rotate test flake

Please provide a description of this PR:

cc @zirain @hanxiaop
The failure indicates the workload cert is not refreshed

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
dd68f24fcfd9916115fa9d5b3172b2bcf75ea7bb
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
dd68f24fcfd9916115fa9d5b3172b2bcf75ea7bb
link
true
/test release-notes

integ-security-istiodremote_istio
dd68f24fcfd9916115fa9d5b3172b2bcf75ea7bb
link
true
/test integ-security-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I think this is probably due to the time required to update configmap

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
dd68f24fcfd9916115fa9d5b3172b2bcf75ea7bb
link
true
/test integ-security-istiodremote

release-notes_istio
3bc81dbed92661d2a67f95d03f06b71d9a982839
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

You mean the secret is not loaded yet?

/test all

/test all

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
3bc81dbed92661d2a67f95d03f06b71d9a982839
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
3bc81dbed92661d2a67f95d03f06b71d9a982839
link
true
/test integ-security-multicluster

release-notes_istio
b9c05c117abf4c16d7754cf01c889cab38eeccf7
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Bug bounty test - PLEASE DO NOT APPROVE. owijfm

Bug bounty test - PLEASE DO NOT APPROVE. owijfm

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


Fixes issue #25

I think this will fix issue #25. The documentation doesn't need to be updated because it already thought arrays of arrays was allowed.

Thx, will review later on and deploy it if so


tabletserver: move rowcache invalidator into SqlQuery

With the new upcoming API changes, the rowcache invalidator
life cycle is going to be different from query engine.
So, it's better that SqlQuery owns and manages it separately.

LGTM


How to adjust the width of sidebar？

I found that my sidebar is too wide. If available, I hope add a button to hide and expand sidebar like view on the phone, or dynamically adjust sidebar.

The width of the side bar is not adjustable but feel free to submit a proposal or PR

I would like to request the addition of a parameter in the config.toml file that allows for customizing the width of the sidebar. Additionally, it would be great if an option could be added to display a button that allows users to expand and shrink the sidebar. Thank you for considering these enhancements.

Making an adjustable sidebar is a lot of work and personally I don't see a positive value compared to the required work.
Making the hide/show switch of the mobile display available for deskop is not that much work but I don't see a use case either. On desktops you usually have a big screen an much available space. Hiding the menu is just not necessary here and requires additional controls overloading the content header.


Add react 16 to peer dependencies

When using react 16, getting the following error:
<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>

Also need to add<EMAIL_ADDRESS>to peer dependencies


Batch isn't atomic

To my understanding, db.batch currently does independent puts for each element in the input list. If one of these writes were to fail, the previous writes in the batch would still be accessible.
One way to fix this would be to add an optional batch identifier to Entry that's a pointer to a batch completion record (inserted when the batch is successful). During get, if an Entry is part of a batch, but its batch pointer points to a nonexistent entry, then the record should also be treated as nonexistent.
The overhead here is that each get on batched entries would require a second lookup, and failed batches can fill the DB with unreachable entries that consume space (would need to be able to clear these).

Actually it should be atomic already but could def use more test covs (I'm not sure we even test the atomicity atm, so there could def be bugs)
The way it currently works is that all puts are done in series in the batch, but they are not commited to the feed but instead to a simple array, https://github.com/mafintosh/hyperdb/blob/master/index.js#L95-L96.
Once all nodes in the batch have been added there they are all passed to hypercore's feed.append which only signs the last node making in the batch making it atomic (you need the sig the verify any of the them).
Makes sense?

Ah, shouldn't have been so confident :)
The line that I missed was in the Writer's append actually. Didn't realize that the put calls wouldn't immediately do an insertion, so I thought there could be an inconsistent state.


Talk about low level constructs vs high level ones in Performance doc

https://docs.perl6.org/language/performance#Solve_problems
Talk about the difference between writing multis and writing given/when constructs. Multis are on C level, given/when is on Perl 6 level (slower)
Any other situations that are like this?

Uhh… True, it would be nice to have something like this, but this is implementation-specific. I don't think we can have some language guarantees about performance.

@AlexDaniel I didn't think about that. Good point.

@WildYorkies I think the right place for it is rakudo wiki: https://github.com/rakudo/rakudo/wiki
I don't know if we want to keep ticket this open or not.


Build Amazon: moved directory opt/boost_1_67_0 to opt/boost

moved directory opt/boost_1_67_0 to opt/boost

@heifner This was for ease of build kite builds (as per Eric's request) & for future boost upgrades.


Add command to auto-populate profiles.yml

Describe the feature
This may have been suggested already but I couldn't find a duplicate issue from a quick search.
Configuring the profiles.yml is non-beginner friendly, and creating instructions on how to do so is complicated especially due to the differences between Mac and Windows environments. Introducing a dbt command which would add a profile to profiles.yml (and create the file if needed) using the name from the dbt_project.yml would be extremely useful. In additional, it is typical that only the schema, username and password are unique between individual profiles.ymls, so a default set of values specified in e.g. dbt_project.yml could reduce the number of options required to be entered.
Describe alternatives you've considered
Creating my own script to do this.
Who will this benefit?
Anyone who introduces dbt to new users frequently and struggles to write a simple and comprehensive set of instructions for configuring a profiles.yml.
Are you interested in contributing this feature?
With some guidance, sure.

Introducing a dbt command which would add a profile to profiles.yml (and create the file if needed) using the name from the dbt_project.yml would be extremely useful

Right on—I think this is what dbt init should do!
In fact, it already does a few of these things:

Create the file ~/.dbt/profiles.yml (or OS equivalent) if it doesn't already exist
Accept an --adapter argument that bootstraps the file with the appropriate profile type (although this could work better by default: #2814)

To get this where we'd want it to be (perhaps more controversial):

init already takes a project_name as its argument (dbt init newproj), but today it only uses it to name the new file directory. In addition, we might find a way to pass the project name to both the name and profile parameters of dbt_project.yml. (Today, the former is always my_new_project and the latter is always default. Not good!)
init should add a new profile, also named using profile_name, to profiles.yml—even if the file already exists, so long as a profile by that name doesn't yet exist.

Adjacent to the above:

Configuring the profiles.yml is non-beginner friendly

I agree! We're planning to give the syntax here another look (#1958) ahead of releasing v1.0 later this year. My plan would be a more intuitive syntax going forward, with backwards compatibility for the current syntax, of course.

In additional, it is typical that only the schema, username and password are unique between individual profiles.ymls, so a default set of values specified in e.g. dbt_project.yml could reduce the number of options required to be entered.

I don't think dbt_project.yml is the right place to store any credential-y information, since it's checked into version control. I do think part of our profile syntax rethink should include the ability to set default profile-level values, and override them per target. Today, that's sort of possible with YAML anchors, but it's more peer-to-peer than true hierarchical inheritance—and talk about a tricky syntax.

Thanks for your always thorough response @jtcohen6.
To be clear (and apologies if you've understood this already), I see this is a separate function to creating a dbt project. The use-case is where a dbt project already exists, and we're onboarding a new user. As I see currently, this is a distinct function from dbt init.
The workflow I'm imagining:

The user cds into their newly cloned dbt project from their organisation
They run a magical command dbt setup_my_profile (but more imaginatively named!)
They are prompted to provide only the unique parameters to their workflow, being e.g. username and password if Snowflake or OAuth for BigQuery etc.
Something similar to dbt debug runs automatically to confirm their connection
They can dbt away and not even know that profiles.yml exists.

Are we on the same page?
If these profile defaults aren't stored in dbt_project.yml then perhaps we could introduce a new, committed file in the project which holds default profile parameters common to all users such as adapter type, account name, etc. I understand the concern of committing credentials, though I think we should allow the user to set any profile configuration through this new default file if they wish to even if we advise them not to.

To be clear (and apologies if you've understood this already), I see this is a separate function to creating a dbt project. The use-case is where a dbt project already exists, and we're onboarding a new user.

Ah, I totally missed this piece. Ok, I'm with you - thanks for clarifying.

As I see currently, this is a distinct function from dbt init.

This is what I'm less sure about. In my view, dbt init is actually two things today:

A way to scaffold file structures for new projects
A way to set up local machines for using dbt, i.e. by creating ~/.dbt/profiles.yml

So I do see both of these within the scope of a (fancier, more powerful, more interactive) dbt init command, which would include prompts for:

Do you want to create a new project or use an existing one? (perhaps inferred from whether dbt_project.yml is in the current working directory)
Do you want to add a profile to ~/.dbt/profiles.yml? (whether or not the file already exists; infer the profile name from the preexisting or just-created project name)
What's your account ID, username, password, etc?
Do you want to test the connection? (i.e. run dbt debug)

To accomplish this, we could rework dbt init to use a more rigorous library for interactive setup, e.g. cookiecutter. I think I've had this conversation four times in the past week :) (cc @iknox-fa @kwigley)
Two reservations:

We should be cautious about hiding from users the fact that their password is being stored, in plaintext, in a local file. @leahwicz made this point, and it's a really good one. I'm not convinced that "not even know that profiles.yml exists" should be one of our goals here; we should seek to make it as easy and streamlined as possible, but not entirely invisible.
I'm still really hesitant to have dbt cross the Rubicon of storing credentials outside profiles.yml / inside canonically version-controlled files. I totally get the convenience of a committed file/structured object to store "standard" connection parameters that are consistent across colleagues (Redshift cluster, Snowflake database, etc). I've done a manual version of this before by means of some README instructions, a checked-in samples.profiles.yml, or by means of environment variables.

So I do see both of these within the scope of a (fancier, more powerful, more interactive) dbt init command, which would include prompts for:

Do you want to create a new project or use an existing one? (perhaps inferred from whether dbt_project.yml is in the current working directory)
Do you want to add a profile to ~/.dbt/profiles.yml? (whether or not the file already exists; infer the profile name from the preexisting or just-created project name)
What's your account ID, username, password, etc?
Do you want to test the connection? (i.e. run dbt debug)

This would be great.
On reservation 1 - great point, interestingly AWS don't do this with aws configure, only the documentation explains where your credentials are being stored vs CLI output. Explicit is good though, and a CLI output along the lines of Your profile for <dbt_project_name> has been configured in <file_path>. sounds good.

To be clear I'm not suggesting we should commit credentials, but that standard parameters should be configurable in a committed file. Moving a step beyond using a samples.profiles.yml or listing these values in the README and instead having them autopopulated into profiles.yml from some committed file when running our new dbt init command would be excellent.

Is this likely to get on the roadmap in the next couple of months given you've discussed this several times now? If so - I'll leave it with you, otherwise I'd be interested to take it on as a contribution.

Is this likely to get on the roadmap in the next couple of months given you've discussed this several times now?

All yours! We've done everything init-related that we're considering must-have ahead of v1.0. I'd love to see you have a go with cookiecutter, if you're up for it :)


Add SSL tests for certificates with embedded SCTs and fix SCT timestamp validation bug

The only SSL tests prior to this tested using certificates with no embedded Signed Certificate Timestamps (SCTs), which meant they couldn't confirm whether Certificate Transparency checks in "strict" mode were
working.
These tests reveal a bug in the validation of SCT timestamps, which is also fixed by this PR.
Checklist

[x] tests are added or updated

This seems to have been forgotten.  I'll merge it.

Merged.  master only: 2094ea070a1fb6aa06b8e939e6cb735edc2c178b and 6a71e06d7aa02b857c8650aa94f6efd9d6531872
Cherry picking to 1.1.0 didn't come out clean, @RJPercival, I think it's best if you make a separate backport PR.


Why is this mod getting worse with every update?

Summer 2019, I managed to get decent use out of this mod, and I was hoping for the experience to be improved with more features and polished controls. Sadly, this mod has not made any progress over the past year and a half and yet the new version (0.9.1) actually renders the game unplayable.
(These issues were found using an Xinput controller)

Actions that require holding down the triggers don't work. (Eating, mining, etc.)
The virtual mouse option is useless because the mouse ends up breaking the menus and crafting interface.
The crafting interface is still a mess, with the mouse automatically hovering to the center of the closest slot.
The mod still doesn't have D-Pad crafting, which would've gone a long way to improving the experience.

I am simply disappointed that this mod has made no progress whatsoever, and the latest version being this broken is pathetic. It's as if this mod isn't being properly tested or even cared about. I just want to play the arguably definitive version of Minecraft with a controller in hand, and given my experiences with this mod, it looks like I will have to wait even longer for this to become a viable solution.

I'm sorry you've had a bad experience with the mod. I definitely know there is room for improvement for the experience but the issues relating to virtual mouse breaking menus or triggers not working is not something I have encounted during testing. If you can please report theses problems as individual issues with more details that would be of great help. I unfortuantely only have limited time when developing this mod and this recent update was highly overdue.

I'm sorry you've had a bad experience with the mod. I definitely know there is room for improvement for the experience but the issues relating to virtual mouse breaking menus or triggers not working is not something I have encounted during testing. If you can please report theses problems as individual issues with more details that would be of great help. I unfortuantely only have limited time when developing this mod and this recent update was highly overdue.

If anything, I should be apologizing for my harsh tone. Sorry about that.
I do hope for the best for Controllable, given this mod has been the best remedy for controller support thus far.
I'll keep an eye on this project, because with time, I have faith it will become an enjoyable solution that me and my friends will get a use out of.
Keep up the great work!
P.S.
The issues I were having were tested with a Wireless Xbox One Controller.

If anything, I should be apologizing for my harsh tone. Sorry about that.
I do hope for the best for Controllable, given this mod has been the best remedy for controller support thus far.
I'll keep an eye on this project, because with time, I have faith it will become an enjoyable solution that me and my friends will get a use out of.
Keep up the great work!
P.S.
The issues I were having were tested with a Wireless Xbox One Controller.

Are you able to try out 0.10.1 and let me know if your experience has been improved?
Controllable 0.10.1 Changlog:
+ Now using GLFW to handle controller input. This also reduces the mod size to 200kb.
+ It is now possible to navigate around GUI and inventory with DPAD buttons.
+ You can now scroll throw recipe book tabs and pages with new button bindings (use bumpers and triggers)
+ Added an option to control available actions on the HUD. (All, Mininal or Off)
+ Better integration with JEI
* Fixed mouse drifting slight after being released
* Fixed mouse moving to incorrect position of slots when recipe book is open

Are you able to try out 0.10.1 and let me know if your experience has been improved?
Controllable 0.10.1 Changlog:
+ Now using GLFW to handle controller input. This also reduces the mod size to 200kb.
+ It is now possible to navigate around GUI and inventory with DPAD buttons.
+ You can now scroll throw recipe book tabs and pages with new button bindings (use bumpers and triggers)
+ Added an option to control available actions on the HUD. (All, Mininal or Off)
+ Better integration with JEI
* Fixed mouse drifting slight after being released
* Fixed mouse moving to incorrect position of slots when recipe book is open


Add EmbeddedKOSKirkuits

https://github.com/rkunze/EmbeddedKOSKirkuits
https://forum.kerbalspaceprogram.com/topic/218872-112x-embedded-kos-kirkuits/

Oops. Sorry for putting the file at the top level, did not intent to do that. Thanks for fixing.
As for the indentation warning: Is that documented somewhere that you want 4 spaces as indent? I usually use two spaces for YAML to avoid too much blank space at the start of the line (especially for deeply nested structures). I'll try to keep it in mind for the next time.

@HebaruSan Thanks for merging!
I found a little oddity in "ckan list" behaviour on my dev instance for EmbeddedKOSKirkuits: Now that the mod is officially on CKAN, I'd expect "ckan list" on my dev instance (where it is installed manually) to show it as an "unmanaged" mod. But it does not show up at all there, even after an explicit "ckan update" and "ckan scan".
My guess from the verbose output of "ckan scan": "ckan scan" only registers manually installed mods that contain a DLL, and EmbeddedKOSKirkuits is purely a set of ModuleManager patches and does not contain a DLL. Supporting evidence: Same behavior with manually installed versions of SkyhawkScienceSystem and SkyhawkKerbalism, which are pure ModuleManager patch sets as well.
Question: Is this known (and intended) behavior, or should I report a bug against the CKAN client?

Spontaneous (and probably dumb) idea: How about checking the directory listing of GameData against known mod IDs? Should be fast enough even for a monster install with several hundred mods, and would at least catch all well-behaved mods that install to GameData/{mod-id}...

I think there were reasons we didn't go with that solution, but it's been a while now since I've concentrated on it, so I don't have a comprehensive explanation ready to go. At a glance, it seems like a big leap to go from "this directory exists (and might have a file in it)" to "this mod is fully and correctly installed such that mods that depend on it will work properly."

If you just want a quick workaround for your dev environment, note that the DLL file doesn't have to be valid. If make a 0-byte file at GameData/YourMod/PluginData/YourMod.dll, I think CKAN would detect it and the game wouldn't even try to load it thanks to how it treats PluginData folders.


Add section about CDN configuration (CloudFront)

CloudFront
You need to configure your matching CloudFront domain name with exposedDomain directive:
# silta.yml
exposedDomain:
  cloudfront:
    hostname: www.example.com # matching domain name configured in CloudFront

Origin connection during Lets Encrypt certificate verification process
Eventually you want your CDN to use Silta origin using https-only origin protocol policy.
However, it is important to understand that you may need to temporarly use http-only during the phase of initial Lets Encrypt certificate verification process. Also, during this phase, you want to disable all https to http redirects from CloudFront Cache behaviors and from silta.yml:
# silta.yml (when using Lets Encrypt)
ingress:
  default:
    # Allow HTTP connections for CloudFront, when
    # Lets Encrypt verifies certs for the first time.
    redirect-https: false

Note: If you don't allow http traffic, and your environment is issuing a certificate with Lets Encrypt for the first time, Silta provides temporarly self-signed certificate which will cause CloudFront to not trust the origin and causes 502 Bad Gateway error, thus failing the actual certificate verification process.
Custom certificates
If you decide to use custom verified certificate, then you may configure everything directly to https-only and skip steps allowing http origin requests temporarly.
Forwarding Host header
You want Drupal to generate URLs according to the domain and scheme that user is accessing the site.
Without forwarding Host header, Drupal would use its' Silta domain and scheme (for example https://production.my-project.dev.wdr.io) to generate content URL addresses.
To overcome this problem, you must configure CloudFront to forward Host header to origin.
To do this, apply cache policies under "Cache behaviors" tab and either:

use managed cache policies
create custom policy

Summary

Please replace wdr.io with [clusterDomain] if possible.

Done


jOOQ - Mapping of lists

Is it possible to map lists?
Let's take http://modelmapper.org/user-manual/jooq-integration/ as an example. However the order would have multiple customers:
public class Order {
  private int id;
  private List<Customer> customers;
}

public class Customer {
  private int id;
  private String surname;
  private Address address;
}

I assumed that the mapping would work if the column are named customers_id / customers_surname.
But this doesn't work. I tried different syntaxes like customer_id, customers_customer_id, customer_customers_id etc.
So is this at all possible? If yes, what is the correct syntax or needed configuration?
BTW: I believe there is a typo in the documentation. Shouldn't it be customer_address_street instead of customer_street_address? Otherwise why is it turned around for the field city (customer_address_city)?
Thanks!

I never found a solution.
For anyone interested:
It works with SimpleFlatMapper (see https://simpleflatmapper.org/0102-getting-started-jdbc.html / https://github.com/arnaudroger/SimpleFlatMapper). However the list has to be correctly sorted. So it's of no solution if this cannot be guaranteed. There is an option for unsorted lists in SimpleFlatMapper but that doesn't work. Furthermore the project does not seem to be maintained anymore.
So it in the end I had to program an own specific ORM that was fitting for the program where it will be used.


Derek/add generic uniqueitemlist

Added a generic UniqueItemList class, which mirrors the functionality of uniquePersonList. To be used for patients, doctors and appointments. Meant to replace UniquePersonList.
Added interface Listable for all classes that are to be managed using a UniqueItemList.

Codecov Report

Merging #68 (31bc6c5) into master (b772575) will increase coverage by 0.30%.
The diff coverage is 84.90%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@             Coverage Diff              @@
##             master      #68      +/-   ##
============================================
+ Coverage     75.85%   76.15%   +0.30%
- Complexity      440      459      +19
============================================
  Files            73       76       +3
  Lines          1379     1430      +51
  Branches        133      140       +7
============================================
+ Hits           1046     1089      +43
- Misses          303      307       +4
- Partials         30       34       +4

Files
Coverage Δ

...odel/person/exceptions/DuplicateItemException.java
100.00% <100.00%> (ø)

...model/person/exceptions/ItemNotFoundException.java
100.00% <100.00%> (ø)

...va/seedu/address/model/person/patient/Patient.java
96.55% <100.00%> (+0.25%)
:arrow_up:

...u/address/model/person/patient/UniqueItemList.java
82.60% <82.60%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add optional module-name support for configurable variables in root module

Purpose
$subject
Fixes #28999
Approach
As per spec, in TOML syntax for configurable variables,

The orgName and moduleName are optional only for configurable variables defined in the root module of the program
The orgName is optional only for configurable variables defined in the root package of the program.
For imported modules, the orgName, moduleName is mandatory
Addressed the orgName & moduleName ambiguities by validating them to have orgName & moduleName both.

Samples
Remarks
The ambiguities between record names and module-names will be tracked in #29923
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Shall we have a code review for this PR.

I created an issue regarding what we discussed in the code review. let's fix it in a separate pr.


This library is amazing but...

...where is the implementation?

Hello Vincenzo, my first implementation will come soon (in this month). I hope you will contribute to it.

Looking forward, then.

I think this issue can now be closed. With 10 months of delay.

Mi puzzano le palle.


[main] Upgrade to latest dependencies

Bump after control-protocol cut -nak3
/cc knative-sandbox/eventing-writers
/assign knative-sandbox/eventing-writers
Produced by: knative-sandbox/knobots/actions/update-deps

Error: error processing import paths in "config/controller.yaml": error resolving image references: exit status 2
2022/10/04 13:54:25 error during command execution:error processing import paths in "config/controller.yaml": error resolving image references: exit status 2

Error above due to import issues with github.com/cloudevents/sdk-go/sql/v2 v2.12.0
2022/10/04 13:54:52 Unexpected error running "go build": exit status 2
# github.com/cloudevents/sdk-go/sql/v2/gen
vendor/github.com/cloudevents/sdk-go/sql/v2/gen/cesqlparser_lexer.go:177:32: lexerDeserializer.DeserializeFromUInt16 undefined (type *antlr.ATNDeserializer has no field or method DeserializeFromUInt16)
vendor/github.com/cloudevents/sdk-go/sql/v2/gen/cesqlparser_parser.go:104:34: deserializer.DeserializeFromUInt16 undefined (type *antlr.ATNDeserializer has no field or method DeserializeFromUInt16)

Can we go back to v2.8.0?

/retest-required

/approve cancel
Build failing due to old libs.

/assign @aliok

/lgtm
/approve


[0.2.0] Achieving negative xp with /levelupTool

Once you reach int<PHONE_NUMBER>) as an XP requirement for the next level up (I achieved this using /levelupTool), if you have any XP gained (like 6 /<PHONE_NUMBER>), when using /levelupTool once more, it results in a negative XP value appearing, and the tool does not level up. Using /levelupTool again puts you at a positive value again. If you have 0 /<PHONE_NUMBER>, you level up normally. I'm assuming this is caused by extra XP being carried over to the next level when using /levelupTool.
Here's an image showing the negative xp-
https://u.pomf.is/jyqjcw.png
I realize this is an extreme edge case and will nearly never be seen, but I figured I'd report it anyway.

Well, if you manage to mine blocks for 3 1/2 years continuously at max mining speed... I guess? Not to mention the buildup until you actually require that amount of xp. ;P


Settings for the "attachment point tool" don't update with changed pixel settings

T-shape attachment points created with the Attachment Point Tool (or loaded from an external file) do not respect 'Bond thickness' global setting and are always drawn with default thickness.
Steps to reproduce:

Add a single bond and an attachment point to a drawing.
Go to Settings->Bonds and change the 'Bond thickness' parameter (e.g. increase up to 6)
See that line width of the single bond has been updated accordingly while line width of the attachment point remain the same.

'Bond thickness' setting should apply to attachment points, similarly to bonds.
(?) Related Indigo task - #2070

Steps:

create benzine ring with attachment points

change Bond thickness to 0.2 cm
click on Apply button

Actual result:

attachment point isn't changed thickness
Expected result:
attachment point is changed thickness

Steps:

create molecule with primary attachment point

change setting Bond thickness to 0.2 px
click on Apply button

Actual result:
primary attachment point isn't displayed

Expected result:
all parts of molecule are displayed regarding settings

Verified

Environment
Ketcher Version 2.25.0-rc.1 Build at 2024-09-18; 05:33:09
Indigo Version 1.25.0-dev.1.19-ge314cdb7-wasm32-wasm-clang-12.0.0
Chrome 128.0.6613.138 (arm64)
MacOs 14.6

Verified - attachment point updates by thickness
Environment
https://rc.test.lifescience.opensource.epam.com/KetcherDemoSA/index.html
Ketcher Version 2.26.0-rc.1 Build at 2024-10-04; 18:45:04
Indigo Version 1.25.0-rc.1.0-g704673fcb-wasm32-wasm-clang-19.0.0
MacOs
Chrome 129.0.6668.71 (Official Build) (arm64)


Add a new "Dependencies and Internal Tools" section in the docs

Closes #310
Signed-off-by: Thomas Druez<EMAIL_ADDRESS>

Hi @hyounes4560 could you review and edit the new chapter content?


9029 fix unit test build

New changes aren't coming. I think there was a misunderstanding of the scope of these changes with respect to existing build workspaces. I updated section 6.4 of the implementation note with the options. Please review. This is certainly smaller impact than many of the other updates I've seen.

OK, since we currently have the unit test branch,  https://github.com/xtuml/bridgepoint/tree/9088_unit_testing_branch , which is described by issue #9088 is working fine for us right now, so review and promotion of this issue will likely wait until after 6.2.2 at this point.

This pull request is not going to get attention until after the 6.2.2 release. I am closing it for now.


rasa_core.policies.ensemble.InvalidPolicyConfig: You didn't define any policies. Please define them under 'policies:' in your policy configuration file.

Rasa Core version:0.13.0
Python version:  Python 3.6.8
Operating system (windows, osx, ...): Ubuntu 16.04 LTS
Issue: rasa_nlu and rasa_core  training error :-
INFO:rasa_nlu.training_data.loading:Training data format of nlu.md is md
INFO:rasa_nlu.training_data.training_data:Training data stats:
- intent examples: 188 (15 distinct intents)
- Found intents: 'about_OPPO', 'city_name', 'OPPO_Phones_Price', 'goodbye', 'CustomerCare_info', 'provide_founder', 'brand_info', 'mood_deny', 'greet', 'model_name_lookout', 'Official_Website', 'CEO_info', 'mood_affirm', 'Service_Center_OR_Factory', 'Oppo_store_location'
- entity examples: 99 (7 distinct entities)
- found entities: 'workshop_location', 'shop_location', 'web_address', 'model', 'information', 'customer_location', 'PERSON'
DEBUG:rasa_nlu.training_data.training_data:Validating training data...
Traceback (most recent call last):
File "bot_enhancement_updated.py", line 53, in
trainer = Trainer(config.load('config_mitie_sklearn.yml'))
File "/home/arghya/anaconda3/envs/myenv/lib/python3.6/site-packages/rasa_core/config.py", line 22, in load
return PolicyEnsemble.from_dict(config_data)
File "/home/arghya/anaconda3/envs/myenv/lib/python3.6/site-packages/rasa_core/policies/ensemble.py", line 222, in from_dict
raise InvalidPolicyConfig("You didn't define any policies. "
rasa_core.policies.ensemble.InvalidPolicyConfig: You didn't define any policies. Please define them under 'policies:' in your policy configuration file.
Content of policy.yml
'''yaml
policies:

name: KerasPolicy
epochs: 50
max_history: 6
name: AugmentedMemoizationPolicy
max_history: 6
name: TwoStageFallbackPolicy
nlu_threshold: 0.8
core_threshold: 0.3
name: FormPolicy
'''
Content of nlu_congif.yml(name of the file :- )

'''yaml
language: "en"
pipeline:

name: "nlp_mitie"
model: "/home/arghya/mitie_models/MITIE-models/english/total_word_feature_extractor.dat"
name: "tokenizer_mitie"
name: "ner_mitie"
name: "ner_synonyms"
name: "intent_entity_featurizer_regex"
name: "intent_featurizer_mitie"
name: "intent_classifier_sklearn"
'''

bot logic for rasa_nlu and rasa_core training :-
-- coding: utf-8 --
from future import absolute_import
from future import division
from future import print_function
from future import unicode_literals
##rasa nlu imports
import logging
import warnings
from rasa_nlu.training_data import load_data
from rasa_nlu.config import RasaNLUModelConfig
from rasa_nlu.model import Interpreter, Trainer
from rasa_nlu import config
from rasa_nlu.training_data.loading import load_data_from_endpoint
from rasa_nlu.utils import EndpointConfig, read_endpoints
##rasa core imports
import rasa_core
from rasa_core import config, cli
from rasa_core import utils
from rasa_core.agent import Agent
from rasa_core.run import AvailableEndpoints
from rasa_core.policies import FallbackPolicy, KerasPolicy, MemoizationPolicy
from rasa_core.interpreter import NaturalLanguageInterpreter
from rasa_core.domain import Domain
from rasa_core.utils import EndpointConfig
from rasa_core.run import serve_application
from rasa_core.tracker_store import RedisTrackerStore
from rasa_core.tracker_store import MongoTrackerStore
logging.basicConfig(level=logging.DEBUG)
logger = logging.getLogger(name)
warnings.filterwarnings('ignore')
##training nlu
##if the probability of the intent/action is too low , we will strop predicting and use a default action.
global_fallback = FallbackPolicy(fallback_action_name='utter_unclear',
core_threshold=0.3,
nlu_threshold=0.8)
training_data = load_data('nlu.md')
##trainer to educate our pipeline
trainer = Trainer(config.load('config_mitie_sklearn.yml'))
##training the nlu   model
interpreter = trainer.train(training_data)
##storing the model for future use
global_model_directory = trainer.persist('./models/nlu',
fixed_model_name='current')
##training nlg  -------------------->
domain = Domain.load('domain.yml')
global_db_mg = MongoTrackerStore(
domain,
host='mongodb://localhost:27017',
db='rasa',
username=None,
password=None,
collection='conversations',
event_broker=None,
)
def train_dialogue(
domain='domain.yml',
model_path='models/dialogue',
training_data_file='stories.md',
endpoints=AvailableEndpoints(),
policy_config=None,
):
##Train a dialogue model
##nlu_interpreter = RasaNLUInterpreter('./models/nlu/default/')
policies = config.load('policy.yml')
nlu_interpreter = global_model_directory
endpoints = '/home/arghya/rasa_test/data/place_finder/endpoints.yml'
agent = Agent(
    domain=domain,
    policies=policies,
    interpreter=nlu_interpreter,
    generator=None,
    tracker_store=global_db_mg,
    action_endpoint=endpoints,
    fingerprint=None,
    )
data = agent.load_data(training_data_file)
agent.train(data, epochs=400, batch_size=50, validation_split=0.2)
agent.persist(model_path)
return agent

if name == 'main':
agent = train_dialogue()
rasa_core.run.serve_application(agent, channel='cmdline')
##extract list of events and print its names
events = global_db_mg.retrieve('default').as_dialogue().events
for event in events:
    print(event.as_story_string())

Content of domain file (if used & relevant):

The issue is with the training logic since policy.yml is available and configured and I am able to run rasa_nlu and rasa_core training using
1)python3 -m rasa_nlu.train -c config_mitie_sklearn.yml --fixed_model_name current --data nlu.md -o models --project nlu --verbose
2)python3 -m rasa_core.train -d domain.yml -s stories.md -c policy.yml --debug -o models/dialogue.
I have just upated the rasa_core version to 0.13.0 and not able to figure out how to pass the policy.yml for training.
Expert guidance is much appreciated.

Thanks for raising this issue, @erohmensing  will get back to you about it soon.

Hi! What seems to be happening here is that when you call
trainer = Trainer(config.load('config_mitie_sklearn.yml'))

it's pulling config.load() from core, not nlu. This is because you have from rasa_nlu import config but this is overwritten by from rasa_core import config, cli. I think the easiest fix here would be to rename one of them, for example from rasa_nlu import config as nlu_config (and then use this name when you call the load function)

Hi @erohmensing , corrected the logical flaw.Thanks for the help.


Reward Network

This is more of a question.
My reward function is a GaN network that is trained during the training epochs of the agent. How can I integrate this network into tensorforce? You build your graph in model.py, but is it possible to add my reward network without changing the class?

Currently this is unfortunately not possible without changing the code. While I expect the reward estimation to develop in the near future, it may take a while to support such a general setup. It's probably easiest for now to do the reward estimation "outside of Tensorforce", if that's possible. Nonetheless, just to check: i would guess that the reward produces the reward internally based on the state values as input, or something like that? That's at least a type of reward estimation I would like to work towards for Tensorforce...

Thank you for your reply.
Yes, the reward function is based on the state of the environment and is also trained together with the agent.

Okay, I expect that this feature will exist for Tensorforce in the longer run, but it's currently not possible without some non-trivial code changes. I could give you some ideas about how to achieve it, but it may be easiest to just handle it as an external "reward preprocessing" step... what do you think?


[Bug]: ReactiveUI.Blazor - Using ReactiveInjectableComponentBase renders the view twice

Describe the bug 🐞
If I inherit from ReactiveInjectableComponentBase<T> to attach a ViewModel to a View, the View gets rendered twice on each activation.
For comparison, the same View and ViewModel with the ViewModel being attached by an [Inject] attribute, the View is rendered only once.
Step to reproduce
Check the linked reproduction repository.

ViewWithInherits.razor uses @inherits ReactiveInjectableComponentBase<T> and will be rendered twice
ViewWithInject.razor uses [Inject] attribute and will be rendered once

I verified the number of render cycles with the debugger, the counter property and Trace message (not in the example).
Reproduction repository
https://github.com/Lukzy/ReactiveUI.ExecutedTwiceBug
Expected behavior
Both ways of attaching a ViewModel should behave the same. I would expect the View to be only rendered once.
Screenshots 🖼
No response
IDE
Visual Studio 2022
Operating system
Windows 10 Enterprise
Version
10.0.19044 Build 19044
Device
LENOVO X1
ReactiveUI Version
18.0.10
Additional information ℹ
Behavior tested with these browsers:

Microsoft Edge - Version 101.0.1210.32 (Official build) (64-bit)
Google Chrome - Version 100.0.4896.127 (Official Build) (64-bit)
Firefox - Version 97.0.2 (32-bit)

I spent few hours trying to figure out what I did that was wrong.


App not working

I hope this message finds you well. I am writing to report a persistent issue I have encountered with the app that claims to enable unlimited photo backups for Google Photos. Despite following all the necessary steps, I have been unable to get the app to work as intended.
To provide you with some context, I am currently using Android 14 Beta 2.1 and Google Photos version 6.38. I have followed the installation instructions meticulously, but unfortunately, the app fails to function properly. As a troubleshooting measure, I even resorted to factory resetting my device to eliminate the possibility of any conflicts arising from my phone's settings or other installed applications.
In my pursuit of a solution, I also joined a Telegram group that provides modified APKs for Google Photos. However, despite trying several of these modified versions, none of them were able to resolve the issue I'm facing.
I understand that the app in question is designed to enable unlimited photo backups for Google Photos, which is a highly sought-after feature. Therefore, I believe it is crucial to address this issue promptly, as it hampers the usability and functionality of the app for users like myself.
I kindly request that the development team investigate this matter further and provide a solution or update that rectifies the problem. I am eager to take full advantage of the unlimited photo backup functionality and would greatly appreciate any assistance in resolving this issue.
Thank you for your attention to this matter. I look forward to hearing from you soon regarding the progress made in addressing this concern.

Android 14 is not getting used to device customization. So Android 14 update is needed*

bruh


posix: implement alarm()

This is part of the See POSIX Roadmap for LTSv3 (RFC #51211).
alarm() is required as part of POSIX_SIGNALS  Subprofiling Option Group.
For more information, please refer to https://pubs.opengroup.org/onlinepubs/9699919799/functions/alarm.html

Hey i'm looking into this ticket any hints on how to start?

Hey i'm looking into this ticket any hints on how to start?

Hi @zacchaeusliang - this is a pretty interesting topic.
Are you on discord at all? If so, can you ping me?


initial version of persistent configuration, multi-bv support for binja analysis

Store binsync configuration between runs so it persists, support multiple controllers for different open binaries in binja

@Lukas-Dresel bump. Would you be able to test this and lmk if its actual ready for merging before I rewrite more of the UI?

Resolved this in #80


The epoch during training.

The pre-trained model you give is trained for 50 epochs which defined in the paper.  But I found the default num_epochs is defined as 512 in the config.py, why? And the result I obtained trained for 50 epochs with conditional gan  and without conditional gan both perform not so good.

And, by the way, the code can only run on one gpu? The time of training cost a lot...

Hello, which dataset are you training on, ADE20k? 512 epochs is just a placeholder value. From memory the authors train for 50 epochs on Cityscapes and ADE20k and their results look better than mine on the testing set.


can not reproduce the results on MSVD?

Hi, I found that the model provided at 3800 steps has acc 0.317, while I trained by myself and got 0.28.
Are there other tricks?

Which method you tested? Is it  _mrm2s?
The iteration number is not necessary 3800 for each trial. Please run more iterations and pick according to the validation accuracy. Also try finetuning the training by reducing the learning rate after the accuracy stops increasing.

Yes, I even obtained a higher results...


precalculate array sum

Micro-optimization? :)

Micro-optimization within a loop :P, also a tad bit easier to read what code is doing. I got a bunch of optimizations if you want me to just submit them as a single PR, or just not bother? Let me know. :) Trying to tidy up code a bit and do some code calisthenics as well.

If it's several smallish changes a single PR (or chunked into larger PRs) would be great :)


docs(README): add cache, etag, casbin extensions description

What type of PR is this?
docs

Check the PR title.

[x] This PR title match the format: <type>(optional scope): <description>
[x] The description of this PR title is user-oriented and clear enough for others to understand.

(Optional) Translate the PR title into Chinese.
为 cache、etag、casbin 中间件在 README 中添加介绍。
Which issue(s) this PR fixes:

NA

CI has checked a file which does not belong to this pr. Is this a problem derived from the previous CI upgrade? cc @wzekin

need to rebase @L2ncE

LGTM