fix: no hydration when new promise comes in

really weird, queryFn is getting called and returns the updated value, but the serializer is called only with teh initial count

you’re right: we only send the promise when we are in pending state. The fix is likely to send the promise when we are in fetchStatus: 'fetching' instead 💡

i believe another issue is we're getting dataUpdatedAt: 0 so it does't rehydrate

feels like we need some other state that would let us know if the promise is fresher then we need to set the queries' promise or something.. this goes deeper than I thought

struggling to figuring out how to properly filter out the queries to put in the hydration queue
https://github.com/user-attachments/assets/2b6bf540-e735-4eca-a549-e15ab223c1de

@juliusmarminge have you widened this check?
https://github.com/TanStack/query/blob/8fe90104c6fce9654539e006bd739074e15ae70f/packages/react-query/src/HydrationBoundary.tsx#L74-L76
to something like:
const hydrationIsNewer =
  dehydratedQuery.state.dataUpdatedAt >
    existingQuery.state.dataUpdatedAt || dehydratedQuery.promise

so that whenever a promise is included, we put it the queue.

yea that causes infinite rerenders

interesting; let’s debug that together today if you have some time?

interesting; let’s debug that together today if you have some time?

sure! what time? I can do anytime after 10am gmt+1.

I pinged you on discord ;)

@juliusmarminge what are we gonna do with this one?

Sorry completely forgot about this 😅
I think I had it sort of working but happy to jump on a call and talk about it still! Hit me up on discord when you have some time to go over it!


[Bug]: Electron BrowserView causing issues with Window management

Need to integrate Knowledge Graph into the main app, so as to stop using a modal Electron window to display it.
Also need to correctly reference the KcMainWindow properly. As of now, if another window is open, the app cannot correctly identify the main window, causing the app to behave erratically.

Fixed in dbdcb293e364a4b9c6f1dcc5fd79b190d6d3c9d0. Awaiting merge.


Ransomed vc sites are back online

Also, I think there is an issue with Lorenz parser, last victim not detected

I'll check for Ransomedvc
Regarding lorenz, which victim  ? Bayer Heritage Federal Credit Union ?  if yes it presents at the date 01/12/22 (the one published by Lorenz)
I'vemanually corrected to 2023.

Yes this one
Bayer Heritage Federal Credit Union
Ohh OK, I didn't see it in the rss feed.
Thanks!

Regarding RANSOMED.VC aka Raznatovic they only sell old leak for now.  I'll continue to monitor but I won't publish for now.


release: v0.3.0

it was in pixi.toml - added it in pyproject.toml and also moved them in one place

closing as we make it 1.0.0


Remove grpc/node from dependabot tracked Dockerfiles

We removed this Dockerfile in #153, so the dependabot fails whenever it tries to update this image. (eg
here

We're almost certainly going to re-add this Dockerfile, I suggest we close this for now. And if we don't I'll make sure to reopen and merge this.


Remove exception for GCC and other select environments

WebView2 is deployed to all devices with internet connectivity, including GCC/GCC High and other tenant environments.

@msft-bcorob   Thanks for submitting this update to the article.


Failed upgrading vmss extensions

This is autogenerated. Please review and update as needed.
Describe the bug
Tried to upgrade extensions for my VMSS.
Command Name
az vmss extension upgrade --ids
and same error when I tried
az vmss extension upgrade -g RGNAME -n VMSS_NAME
Errors:
'VirtualMachineScaleSetRollingUpgradesOperations' object has no attribute 'start_extension_upgrade'
Traceback (most recent call last):
cli\Lib\site-packages\knack\cli.py, ln 233, in invoke
azure\cli\core\commands\__init__.py, ln 660, in execute
azure\cli\core\commands\__init__.py, ln 723, in _run_jobs_serially
azure\cli\core\commands\__init__.py, ln 716, in _run_job
artifacts\cli\Lib\site-packages\six.py, ln 703, in reraise
azure\cli\core\commands\__init__.py, ln 694, in _run_job
azure\cli\core\commands\__init__.py, ln 331, in __call__
site-packages\azure\cli\core\__init__.py, ln 811, in default_command_handler
azure\cli\command_modules\vm\custom.py, ln 3218, in upgrade_vmss_extension
AttributeError: 'VirtualMachineScaleSetRollingUpgradesOperations' object has no attribute 'start_extension_upgrade'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az vmss extension upgrade --ids {}

Expected Behavior
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.6.8
Installer: MSI

azure-cli 2.21.0

Extensions:
application-insights 0.1.13

Additional Context

hi @qwordy, could you have a look?

Change start_extension_upgrade to begin_start_extension_upgrade should fix this bug.


Unable to delete draft kata without any completions

I am unable to delete this kata that I have not yet published. When I click the 'Delete' button and confirm I wish to delete the kata, the deletion never occurs. I am aware there are other related issues on this, but could not see an identical issue, and I was unable to resolve this, please advise, thanks

I'm having the same issue with this that's never left Draft.

It seems that katas can't be deleted once a language version is created for the kata, even if it has never been published.
@kazk The delete kata feature looks completely broken. I think either this should be fixed, or if it can't be fixed, remove the delete kata button? It doesn't stop useless katas from cluttering the authored kata list though. Maybe make deletion turning a "permanently hide this kata from everywhere" flag on instead?

this bug had existed for 3 years!, I having the same problem!

Please, delete these 2 as well
https://www.codewars.com/kata/5d85374fc5a0d000254850c2
https://www.codewars.com/kata/5d83314ad3b773002113cc0c

Please, delete these 2 as well
https://www.codewars.com/kata/5d85374fc5a0d000254850c2
https://www.codewars.com/kata/5d83314ad3b773002113cc0c

Looks like there's a bug where Codewars thinks someone had attempted it because the author had previewed the kata (it creates a same record as other user attempting if I remember correctly). I'll look into this sometime soon.

Looks like there's a bug where Codewars thinks someone had attempted it because the author had previewed the kata (it creates a same record as other user attempting if I remember correctly). I'll look into this sometime soon.

Please delete these kata too:
https://www.codewars.com/kata/6046b31b01c6c9001330359a/
https://www.codewars.com/kata/6043e9680e7328001ec2455c/
https://www.codewars.com/kata/603d7c3ad48bac0017a22986/
https://www.codewars.com/kata/603d7c3ad48bac0017a22986/
https://www.codewars.com/kata/6025896a30de66000de95e52/
https://www.codewars.com/kata/6025841dff5c14000ef617e7/
https://www.codewars.com/kata/6025745e1cb14e000ff7f8de/
https://www.codewars.com/kata/6024bd5b7bb453000864da1c/
https://www.codewars.com/kata/60121d0a96a18300265dfefd/
https://www.codewars.com/kata/6011f1d94c1824002bd3803d/
https://www.codewars.com/kata/5fc5865367a01000084ebff7/
https://www.codewars.com/kata/5faf108d7b069e000fe9daef/
https://www.codewars.com/kata/5f7510d00a8695002e10fa30/
https://www.codewars.com/kata/5f4933bececf9e0011a8bc27/
https://www.codewars.com/kata/5f4862404cbed500341e41c9/

Whenever a user opens a trainer for a kata /train, a new record that references the kata is created. This is configured to prevent the kata from being deleted. When authors preview their kata, the same record is created, and this was preventing deletion.
I made a change to delete the author's record if it's the only one. So, if the kata was never trained by anyone else, you should be able to delete it now. I've tested locally, but please let me know if it still doesn't allow you to delete it. I'll check the database to see if there's any other record referencing it.


Feature: fail on superfluous keys

The change implements failing on superfluous keys in the config. This is very usable to detected potential typo's or also if configuration changed in new releases and attributes got deprecated and were finally removed. It can be enabled by configKeyNaming.withFailOnSuperfluousKeys(). In the error message i want to show potential similar attributes to help with typo's. To search for similar keys i use levenshtein distance from org.apache.commons:commons-text, which is a stable library with only a small number of dependencies.

Tests fail because bugfix #48 is needed.

Hi @kxbmap, could you already check if this PR is ok for you?

Sorry for the late.
I'll refactor this PR after merge to make trait immutable.

Hi @kxbmap, tested this again on my side with the -SNAPSHOT version and works nicely. Can we make a release with this?


Habitica Expansion (early version)

This commit response to issue #169 (excluding #10 as someone else is working on it, and the command to check 'actual' participation ---- will be coming with other exciting feature!). Track of cowt participation is done using command !log. Setting and checking habitica profile is done with command !habitica.
To account for the possibility that habitica api might be broken, the !clans command would use cache of the leader (more on this later) and memeber count from previous request to generate the clan list. If the information is outdated, !clans update can be used to update the values from habitica API (if it is working!). The authentication informations would be stored in db but a function is provided to insert them to the db from existing js object array like the one in data/clan-data-auth-sample.js. The !apply command however, make request to the API when it is called to make sure it doesn't send the link to the user if the clan is full at that moment.
The command name !log is chosen in mind that such command can potentially be used in other channel for other purposes, e.g. in sleep club, to summarize the user's sleep log. An additional feature, !log details, is added to allow horace sending a dm to the user of all the vows he/she made.
The !habitica command has many sub-command. Therefore a seperate help message is provided (use !habitica or !habitica help to get the dm). It is worth noticing that due to the property of the habitica API, finding out a discord user's habitica profile would require that user to provide their habitica uuid to horace beforehead (or allow a guardian/ the habitica mananger to do so). Consequently, this would require the clan leaders to provide their habitica uuid in order for their discord name to be displayed in the !clans message (otherwise their habitica user name will show up).
There are still some details that needs refinement and some more to come. But with the current functionality I think it is ready for a pull request to get some testing and feedback from the community. I would appericate any feedback in terms of performance and coding styles.

It's been almost a year since the PR was last active - is there any intention to continue this? Will close if not


Adding stable JDK9 automatic module names derived from main packages

Motivation:
Automatic-Module-Name entry provides a stable JDK9 module name, when Netty is used in a modular JDK9 applications. More info: http://blog.joda.org/2017/05/java-se-9-jpms-automatic-modules.html
When Netty migrates to JDK9 in the future, the entry can be replaced by actual module-info descriptor.
Modification:
The POM-s are configured to put the correct module names to the manifest.
Result:
Fixes #7218.

@mp911de ptal

Looks technically okay, I'm a bit concerned regarding the naming. During the process of finding a consensus for naming rules and automatic naming, a series of controversial approaches were discussed of which each one comes with its benefits and consequences.
I find it quite hard to reason why module names diverge so far from the actual artifact name if the artifact is not modular in the first place (not speaking about any prefixes like io. or such). From a user perspective, I wish to know a scheme for module names so I can derive the module name for each artifact, with as little required input as possible. I don't want to be required to look up module names from the documentation or look into each jar file to inspect the package structure.
For now (no module-info.java present), module names are more of a cosmetic/documentation arrangement that could easily change once netty provides module descriptors.
One could argue that artifacts are named the wrong way, such as netty-transport-native-epoll should be named netty-channel-epoll which would then match io.netty.channel.epoll, but I don't want to open a discussion about artifact names, just briefly mention how things are named.
From this standing point, I'd argue to align module names to the artifact names to document the io.netty module namespace (that would be io.netty.transport.native.epoll for netty-transport-native-epoll) and to avoid confusion around module names.
Another possibility could be to drop the io. prefix and use the artifact name (replace - with .) as the automatic module name (like Spring Framework 5. Unless artifact names are going to change, the module names are stable as well.
Module names (without a descriptor) for the sake of the name do not provide true value to Java 9 projects and need to be revisited at a later stage anyway once a descriptor will be added.

OK, I understand your point of view :). Originally the things were named channel, etc. but later the terminology was changed. The artifacts represent new names, but packages for obvious reasons must keep the old names. And I understand that the users can benefit from the module names that are better aligned with the new terminology.
At the same time, you made some statements about modules that can lead you (or the users of Netty) into the trouble in the future.
First of all, there is a good explanation at Stephen's blog about the relationship modules vs. artifacts, and the further conclusion about naming:

http://blog.joda.org/2017/04/java-se-9-jpms-modules-are-not-artifacts.html
http://blog.joda.org/2017/04/java-se-9-jpms-module-naming.html

He suggests to name the module from the main exported package, so finding out the module name should not be that confusing for the users (it could be even calculated by IDE-s). His explanation is convincing for me:

"Modules are a group of packages. As such, the module name must be related to the package names."

"As the owner of that namespace, any sub-packages may be grouped into sub-modules as desired so long as no package is in two modules."

Netty is not a new project, so the recommendations can be relaxed, but I think that at least reverse DNS style is important.

For now (no module-info.java present), module names are more of a cosmetic/documentation arrangement that could easily change once netty provides module descriptors.

Module names (without a descriptor) for the sake of the name do not provide true value to Java 9 projects and need to be revisited at a later stage anyway once a descriptor will be added.

I'd be VERY careful with changing/aligning the module names in the future. They are not cosmetic for modular projects that would use Netty as a dependency, even if Netty is not using modules right now. If you change it at a later stage, when there will be many projects that depend on existing name, they will start falling into a module hell. Suppose that you have two modules A and B. A uses the old Netty name, and B uses the new Netty name. Now you won't be able to use both A and B in the same project, because Java will see two distinct modules with the same packages and classify them as a package split, which is illegal in Jigsaw. Such project will not even compile. Take a look at examples given by Stephen - there is a good example, what would happen:

http://blog.joda.org/2017/05/java-se-9-jpms-automatic-modules.html

And this is why I opened this issue/PR. If this were a cosmetic thing for Jigsaw, I wouldn't care, too. I won't be very happy, if my project stops compiling in the future because of module name changes :).

From this standing point, I'd argue to align module names to the artifact names to document the io.netty module namespace (that would be io.netty.transport.native.epoll for netty-transport-native-epoll) and to avoid confusion around module names.

Although I'm more convinced by Stephen's explanation, this proposal sounds OK for me. It keeps the reverse DNS style, but it also takes into account that Netty has some history behind it, and new things should be named in the new way. I'll make the necessary changes tomorrow.

Hello, sorry for the delay. I fixed the module names according to the discussion. Please check if everything is OK.

@carl-mastrangelo PTAL as well.

@zyxist missed updating two of the modules.

These names are not missed. The discussion was about things that are named differently (that is: channel was the old name, transport is the new name). codec is still a codec, so I kept the adjustment to the root package names.

@zyxist I mean io.netty.handler.codec.redis and io.netty.handler.codec.socks

Oh, I see - thanks :).

I added one more change - removing the "native" word from module names, because it is a reserved keyword in Java, and one would not be able to require such a module in module-info.java.
https://github.com/netty/netty/issues/7266

@Scottmitch @carl-mastrangelo please review (any of you) and pull in once satisfied to 4.1 and 4.0. Also ensure @zyxist squashed it before.

@zyxist can you please squash the commits ?

Done.

@zyxist please bump the maven compiler plugin version to 3.7.0 https://cwiki.apache.org/confluence/display/MAVEN/Java+9+-+Jigsaw

@johnou -> do you plan to deliver separate JAR archives for JDK9 and JDK6/7/8? Your source and target Java version is JDK6, it is technically not possible to use module-info files with JDK6. To support module-info descriptors, the plugin actually performs the build twice and produces two JAR archives: one for older Java version, one for Java 9. This affects the entire build pipeline and is quite big amount of work to do that goes out of scope of this Pull Request.
Just bumping the plugin version without additional configuration gives nothing.
Can you merge this Pull Request to give Netty a stable module name, to unlock me and other users, and meanwhile start work on updating your build pipeline as a separate task?

Aha makes sense, that would be up to the maintainers.

I'm sorry if I have to chime in and correct above's information, but a specialized configuration of the current maven-compiler-plugin (as outlined in its documentation) will not result in two separate jars but rather a single jar consisting of classes which are targeted at both releases. This method has, for instance, been adopted by slf4j already. Given that versions earlier than Java 9 will not attempt to load the module-info class (unless somehow instructed to do so), this method guarantees full backward compatibility.
That being said, I do agree that the definition of the legacy headers (as implemented by this PR) should suffice at the moment to allow consistent references to netty through Jigsaw enabled projects at the moment. A module-info should, however, still be the ultimate target here (especially given the implementation of the jlink tool which seems to require such definitions to work at all).

So @LordAkkarin @johnou @zyxist @mp911de I am correct in my understanding that this is ready to be merged and good as it is for now ?

@normanmaurer yep.

@normanmaurer actually one thing just came to mind..

@zyxist what happens to modules where javaModuleName is not defined? eg. dev-tools does it end up with an empty string header for Automatic-Module-Name in the manifest? does this cause problems? or is it simply ignored because it is empty?

@zyxist ping...

@johnou @normanmaurer nothing will happen. Modules without javaModuleName will get the automatic module name derived from jar name. So it is safe to leave it as it. Also, this module has no java classes, so it is totally fine.

@doom369 so modules without the javaModuleName property defined in properties will end up with an empty tag [1] then fall back to the automatic module name derived from the jar name?
[1] root pom.xml
<manifestEntries>
    <Automatic-Module-Name>${javaModuleName}</Automatic-Module-Name>
</manifestEntries>

@johnou in general - yes (modules without this property will have empty Automatic-Module-Name field in manifest file). However, it has nothing to with dev-tools. As it is not actually included to any artifact jar. It is only used to execute the specfic maven plugin.

@doom369 my main concern was the behaviour of a jar with an empty Automatic-Module-Name, dev-tools was a bad example.

@johnou empty name is not allowed for Automatic-Module-Name. You'll get
Automatic-Module-Name: : Invalid module name: '' is not a Java identifier
Just checked :).

@Scottmitch @carl-mastrangelo sorry for pushing, could you please review this PR? So those of us who moved to Java 9 could finish modularization.

@doom369 tbh I'm still not convinced with the usage of Maven properties and Automatic-Module-Name in the root pom.xml within pluginManagement, it seems brittle and hacky (the manifest for the testsuite jar has the invalid entry "Automatic-Module-Name: "). Instead why not override the maven-jar-plugin config in each module pom.xml like so..
  <build>
    <plugins>
      <plugin>
        <artifactId>maven-jar-plugin</artifactId>
        <executions>
          <execution>
            <id>default-jar</id>
            <configuration>
              <archive>
                <manifestEntries>
                  <Automatic-Module-Name>${javaModuleName}</Automatic-Module-Name>
                </manifestEntries>
              </archive>
            </configuration>
          </execution>
        </executions>
      </plugin>
...

I think this approach seems more like the Maven way?

@johnou Defining the plugin in the parent's pluginManagement is the approach to go. You don't need to repeat yourself in each pom.xml, only set the property in all submodules. I think that should be done for every other sub-module to comply with the requirements coming from the parent pom /cc @zyxist

@mp911de yep that's what I was getting at, it should be all submodules if it's defined in the parent pom or defined in each submodule if that's not desirable for whatever reason.

@zyxist can you ensure you add it for all sub-modules ? After this I will merge it

@zyxist ping

@normanmaurer PR is good. All needed modules are with correct names. Do mean some specific module?

Cherry-picked into 4.1 (e8540c2b7a1243618002c857be5ba463984e1b76)

@zyxist thanks!

Got hold of the latest netty-transport-native-kqueue:4.1.18.Final:osx-x86_64 jar from maven central to use it in a Java 9 modular project. The build fails because it cannot find 'io.netty.transport.kqueue' module. I still don't see the 'Automatic-Module-Name' entry in the manifest file of the jar. Am I missing something?
The jar command's "--describe-module" option complains about the same. Reported error when I ran "jar --file  --describe-module" is:
netty.transport.native.kqueue: Invalid module name: 'native' is not a Java identifier


forgivedeadplayers doesnt work with guards

hi there, the forgive dead players rule isnt seeming to effect guard villagers. any way this could be fixed? thanks

unless thats through the village and not the mob. killing and replacing villagers seems to have fixed this issue


[Bug]: Emitter "@azure-tools/typespec-autorest" crashed!

Describe the bug
Emitter "@azure-tools/typespec-autorest" crashed! This is a bug.
Please file an issue at https://github.com/Azure/typespec-azure/issues
TypeError: Cannot read properties of undefined (reading 'values')
at walkPropertiesInherited (file:///mnt/vss/_work/1/s/node_modules/@typespec/compiler/dist/src/core/checker.js:4731:51)
at walkPropertiesInherited.next ()
at countPropertiesInherited (file:///mnt/vss/_work/1/s/node_modules/@typespec/compiler/dist/src/core/checker.js:4743:16)
at getEffectiveModelType (file:///mnt/vss/_work/1/s/node_modules/@typespec/compiler/dist/src/core/checker.js:4655:39)
at Object.getEffectivePayloadType (file:///mnt/vss/_work/1/s/node_modules/@typespec/http/dist/src/metadata.js:403:31)
at getSchemaOrRef (file:///mnt/vss/_work/1/s/node_modules/@azure-tools/typespec-autorest/dist/src/openapi.js:619:29)
at emitEndpointParameters (file:///mnt/vss/_work/1/s/node_modules/@azure-tools/typespec-autorest/dist/src/openapi.js:732:19)
at emitOperation (file:///mnt/vss/_work/1/s/node_modules/@azure-tools/typespec-autorest/dist/src/openapi.js:414:9)
at Array.forEach ()
at emitOpenAPIFromVersion (file:///mnt/vss/_work/1/s/node_modules/@azure-tools/typespec-autorest/dist/src/openapi.js:223:20)
Reproduction
import "@typespec/http";
import "@azure-tools/typespec-client-generator-core";
Checklist

[X] Follow our Code of Conduct
[X] Check that this issue is about the Azure libraries for typespec. For bug in the typespec language or core libraries file it in the TypeSpec repo
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

had to add some missing @added decorators


Segmentation fault (core dumped)

Hello, I am having a problem with a Segmentation fault (core dumped) for VRP application when I set the number of vehicles too high (around 200) If I set it to around 100 then it will usually work. Is there a way to avoid this? I based most of my work on this https://developers.google.com/optimization/routing/vrptw

Do you have enough memory ?

I will move this to discussions. Most likely, this is a wrong model with bad indexes, bad time windows, node used in multiple pickup and deliveries...


Forcing audio out default and not selected output section

I have noticed a bug. I have a 5.1 surround hooked up to my system and enabled the 5.1 support. The issue comes in when I plug in standard 3.5mm headphones and still have the sound output mapped to my physical jack but have USB set as default in my system it pushes sound out my default. If I disable the 5.1 support everything functions as it should. I suspect it may have to do with the fact that it is a 2 channel being plugged into the front where I have a 5.1 in the back so GPMDP is having an issue and just forces down the default channel.

@Peligroso81 That 5.1 support option actually just enables an experimental flag inside Chromium that isn't stable yet.  It's quite likely that that flag is causing these issues however we can't fix them until we inherit the fixes from Chromium upstream


Update init.py

accept this pull request

Hi,
Thanks for you contribution!
This PR adds no value to kube-hunter so we will not merge it.
However, don't let this discourage you for making future PRs, I believe that with some time and effort you can dive into the code and make some real contributions!


[vite] http proxy error at /api/auth/signup: AggregateError [ECONNREFUSED]:     at internalConnectMultiple (node:net:1116:18)     at afterConnectMultiple (node:net:1683:7)

whan i login i get this message [vite] http proxy error at /api/auth/signup: AggregateError [ECONNREFUSED]:     at internalConnectMultiple (node:net:1116:18)     at afterConnectMultiple (node:net:1683:7)
(i did just like you)
in the localhost it says Unexpected end of JSON input

This Error is coming from your FrontEnd app when you FrontEnd App server is up & running but somehow due to some error or socketTimeout to Mongo connection or network issues you Backend Server stopped working.
To resolve the error you can simply restart the Backend Server and your issues will be fixed.


The message for MISSING_ANNOTATION_ON_STRUCT_FIELD could be improved

The problem message for FfiCode.MISSING_ANNOTATION_ON_STRUCT_FIELD could be improved. In a review with the FFI team, we agreed on the following message: "Fields of type '{0}' in a subclass of '{1}' must have an annotation indicating the native type.".
This might require some reworking of the FfiVerifier in order to make the two argument values available in the scope where the diagnostic is generated.

https://dart-review.googlesource.com/c/sdk/+/241280


Document a list of useful default extensions

Splitting this out as a todo ticket from discussion in #55:

Perhaps it would be good to have a list in the docs of language extensions you'll probably need to enable to use this library. In general, I think this is a useful kind of documentation, which I haven't seen in many libraries. While Hackage lists the language extensions necessary to implement a library, it doesn't have a mechanism to directly document which ones you need as an end-user of the library (and for what purposes).

Fixed in https://github.com/robrix/fused-effects/pull/59


Compatibility with Pop!_OS?

🤷🏽♂ Current Behaviour
Desktop dimmer icon does not appear in Gnome Shell's top bar.
🎯 Expected Behaviour
After installation I expected to see the icon in the Gnome Shell top bar.
👟 Steps to Reproduce (S2R)

Get Pop!_OS 19.10
Download deb package for latest version
Install with gdebi
Run desktop-dimmer and observe nothing.

🏡 Environmental Context
App Version
v4.0.4
Installation Type
Downloaded deb package and installed with gdebi.
Operating System
Pop!_OS 19.10

Hello there, might be a long waited reply, but currently this software is not working because of a dependency error. In fact, if you do a desktop-dimmer in your terminal, it should give you an error about pango and Harfbuzz.
(desktop-dimmer:50254): Pango-ERROR **: 23:01:07.543: Harfbuzz version too old (1.2.7)

Trace/breakpoint trap (core dumped)

This might be useful for you.


Add Halifax Domains

Added both Halifax Area School District Domains, hasd.us and myhalifax.us. The school site is hosted on the hasd.us if needed.

Me gustaría incluir a mi Universidad para poder usar las herramientas de JetBrains, para el aprendizaje.
upds.txt

Me gustaría incluir a mi universidad "Universidad Privada Domingo Savio"
lib\domains\bo\net\upds.net

@nevetS-718 Pull request merged.
@RoRoVR Please make a separate pull request. Please ensure you are familiar with this repository rules located at the bottom of this page and submit a new request with correct information. Thank you for your understanding!


Something went wrong whilst upgrading the LuckPerms database. Please report this on GitHub.

Description
This problem make my server lagging
Reproduction Steps
Pls update this plugin and fix this problem
Expected Behaviour
Pls update this plugin and fix this problem
Server Details
paper-1.20.4-45.jar
LuckPerms Version
5.4.121
Logs and Configs
[05:32:11] [Server thread/INFO]: [LuckPerms] Enabling LuckPerms v5.4.120
[05:32:12] [Server thread/INFO]:         __
[05:32:12] [Server thread/INFO]:   |    |)   LuckPerms v5.4.120
[05:32:12] [Server thread/INFO]:   |_ |      Running on Bukkit - Paper
[05:32:12] [Server thread/INFO]:
[05:32:12] [Server thread/INFO]: [LuckPerms] Loading configuration...
[05:32:12] [Server thread/INFO]: [LuckPerms] Loading storage provider... [H2]
[05:32:12] [Server thread/WARN]: [LuckPerms] [DB Upgrade] Found an old (v1) H2 database file. LuckPerms will now attempt to upgrade it to v2 (this is a one time operation).
[05:32:12] [Server thread/INFO]: [LuckPerms] [DB Upgrade] Stage 1: Exporting the old database to an intermediary file...
[05:32:12] [Server thread/INFO]: [LuckPerms] [DB Upgrade] Stage 2: Importing the intermediary file into the new database...
[05:32:12] [Server thread/WARN]: [LuckPerms] Something went wrong whilst upgrading the LuckPerms database. Please report this on GitHub.
org.h2.jdbc.JdbcSQLSyntaxErrorException: Sequence "SYSTEM_SEQUENCE_31A0FE7F_1347_4E6F_8AA7_EC1E88FA018E" already exists; SQL statement:
CREATE SEQUENCE "PUBLIC"."SYSTEM_SEQUENCE_31A0FE7F_1347_4E6F_8AA7_EC1E88FA018E" START WITH 398 BELONGS_TO_TABLE [90035-214]
at org.h2.message.DbException.getJdbcSQLException(DbException.java:632) ~[?:?]
at org.h2.message.DbException.getJdbcSQLException(DbException.java:477) ~[?:?]
at org.h2.message.DbException.get(DbException.java:223) ~[?:?]
at org.h2.message.DbException.get(DbException.java:199) ~[?:?]
at org.h2.command.ddl.CreateSequence.update(CreateSequence.java:53) ~[?:?]
at org.h2.command.ddl.SchemaOwnerCommand.update(SchemaOwnerCommand.java:33) ~[?:?]
at org.h2.command.CommandContainer.update(CommandContainer.java:169) ~[?:?]
at org.h2.command.Command.executeUpdate(Command.java:252) ~[?:?]
at org.h2.command.dml.RunScriptCommand.execute(RunScriptCommand.java:108) ~[?:?]
at org.h2.command.dml.RunScriptCommand.update(RunScriptCommand.java:70) ~[?:?]
at org.h2.command.CommandContainer.update(CommandContainer.java:169) ~[?:?]
at org.h2.command.Command.executeUpdate(Command.java:252) ~[?:?]
at org.h2.jdbc.JdbcStatement.executeInternal(JdbcStatement.java:252) ~[?:?]
at org.h2.jdbc.JdbcStatement.execute(JdbcStatement.java:223) ~[?:?]
at me.lucko.luckperms.common.storage.implementation.sql.connection.file.H2ConnectionFactory$MigrateH2ToVersion2.run(H2ConnectionFactory.java:144) ~[?:?]
at me.lucko.luckperms.common.storage.implementation.sql.connection.file.H2ConnectionFactory.init(H2ConnectionFactory.java:68) ~[?:?]
at me.lucko.luckperms.common.storage.implementation.sql.SqlStorage.init(SqlStorage.java:162) ~[?:?]
at me.lucko.luckperms.common.storage.Storage.init(Storage.java:114) ~[?:?]
at me.lucko.luckperms.common.storage.StorageFactory.getInstance(StorageFactory.java:87) ~[?:?]
at me.lucko.luckperms.common.plugin.AbstractLuckPermsPlugin.enable(AbstractLuckPermsPlugin.java:201) ~[?:?]
at me.lucko.luckperms.bukkit.LPBukkitBootstrap.onEnable(LPBukkitBootstrap.java:175) ~[?:?]
at me.lucko.luckperms.bukkit.loader.BukkitLoaderPlugin.onEnable(BukkitLoaderPlugin.java:50) ~[LuckPerms-Bukkit-5.4.120.jar:?]
at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:287) ~[paper-api-1.20.4-R0.1-SNAPSHOT.jar:?]
at io.papermc.paper.plugin.manager.PaperPluginInstanceManager.enablePlugin(PaperPluginInstanceManager.java:188) ~[paper-1.20.4.jar:git-Paper-451]
at io.papermc.paper.plugin.manager.PaperPluginManagerImpl.enablePlugin(PaperPluginManagerImpl.java:104) ~[paper-1.20.4.jar:git-Paper-451]
at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:507) ~[paper-api-1.20.4-R0.1-SNAPSHOT.jar:?]
at org.bukkit.craftbukkit.v1_20_R3.CraftServer.enablePlugin(CraftServer.java:639) ~[paper-1.20.4.jar:git-Paper-451]
at org.bukkit.craftbukkit.v1_20_R3.CraftServer.enablePlugins(CraftServer.java:550) ~[paper-1.20.4.jar:git-Paper-451]
at net.minecraft.server.dedicated.DedicatedServer.initServer(DedicatedServer.java:275) ~[paper-1.20.4.jar:git-Paper-451]
at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1131) ~[paper-1.20.4.jar:git-Paper-451]
at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:319) ~[paper-1.20.4.jar:git-Paper-451]
at java.lang.Thread.run(Thread.java:840) ~[?:?]
[05:32:13] [Server thread/INFO]: [LuckPerms] Loading internal permission managers...
[05:32:13] [Server thread/INFO]: [LuckPerms] Performing initial data load...
[05:32:13] [Server thread/INFO]: [LuckPerms] Successfully enabled. (took 2698ms)
Extra Details
No response

Quoting Luck on #3676, the suggested course of action here is to:

Roll back to the last installed version of LuckPerms that worked, and do a /lp export.
Stop the server.
Delete (create a backup / move it somewhere else) the old database file.
Start the server and run /lp import.

Quoting Luck on #3676, the suggested course of action here is to:

Roll back to the last installed version of LuckPerms that worked, and do a /lp export.
Stop the server.
Delete (create a backup / move it somewhere else) the old database file.
Start the server and run /lp import.

I did what u said. In game it says import was successful but here is the problem:
[22:45:55 INFO]: [LuckPerms] Loading configuration...
[22:45:55 INFO]: [LuckPerms] Loading storage provider... [H2]
[22:45:55 WARN]: [LuckPerms] [DB Upgrade] Found an old (v1) H2 database file. LuckPerms will now attempt to upgrade it to v2 (this is a one time operation).
[22:45:55 INFO]: [LuckPerms] [DB Upgrade] Stage 1: Exporting the old database to an intermediary file...
[22:45:56 INFO]: [LuckPerms] [DB Upgrade] Stage 2: Importing the intermediary file into the new database...
[22:45:56 WARN]: [LuckPerms] Something went wrong whilst upgrading the LuckPerms database. Please report this on GitHub.org.h2.jdbc.JdbcSQLSyntaxErrorException: Sequence "SYSTEM_SEQUENCE_31A0FE7F_1347_4E6F_8AA7_EC1E88FA018E" already exists; SQL statement:
CREATE SEQUENCE "PUBLIC"."SYSTEM_SEQUENCE_31A0FE7F_1347_4E6F_8AA7_EC1E88FA018E" START WITH 398 BELONGS_TO_TABLE [90035-214]
at org.h2.message.DbException.getJdbcSQLException(DbException.java:632) ~[?:?]
at org.h2.message.DbException.getJdbcSQLException(DbException.java:477) ~[?:?]
at org.h2.message.DbException.get(DbException.java:223) ~[?:?]
at org.h2.message.DbException.get(DbException.java:199) ~[?:?]
at org.h2.command.ddl.CreateSequence.update(CreateSequence.java:53) ~[?:?]
at org.h2.command.ddl.SchemaOwnerCommand.update(SchemaOwnerCommand.java:33) ~[?:?]
at org.h2.command.CommandContainer.update(CommandContainer.java:169) ~[?:?]
at org.h2.command.Command.executeUpdate(Command.java:252) ~[?:?]
at org.h2.command.dml.RunScriptCommand.execute(RunScriptCommand.java:108) ~[?:?]
at org.h2.command.dml.RunScriptCommand.update(RunScriptCommand.java:70) ~[?:?]
at org.h2.command.CommandContainer.update(CommandContainer.java:169) ~[?:?]
at org.h2.command.Command.executeUpdate(Command.java:252) ~[?:?]
at org.h2.jdbc.JdbcStatement.executeInternal(JdbcStatement.java:252) ~[?:?]
at org.h2.jdbc.JdbcStatement.execute(JdbcStatement.java:223) ~[?:?]
at me.lucko.luckperms.common.storage.implementation.sql.connection.file.H2ConnectionFactory$MigrateH2ToVersion2.run(H2ConnectionFactory.java:144) ~[?:?]
at me.lucko.luckperms.common.storage.implementation.sql.connection.file.H2ConnectionFactory.init(H2ConnectionFactory.java:68) ~[?:?]
at me.lucko.luckperms.common.storage.implementation.sql.SqlStorage.init(SqlStorage.java:162) ~[?:?]
at me.lucko.luckperms.common.storage.Storage.init(Storage.java:114) ~[?:?]
at me.lucko.luckperms.common.storage.StorageFactory.getInstance(StorageFactory.java:87) ~[?:?]
at me.lucko.luckperms.common.plugin.AbstractLuckPermsPlugin.enable(AbstractLuckPermsPlugin.java:201) ~[?:?]
at me.lucko.luckperms.bukkit.LPBukkitBootstrap.onEnable(LPBukkitBootstrap.java:175) ~[?:?]
at me.lucko.luckperms.bukkit.loader.BukkitLoaderPlugin.onEnable(BukkitLoaderPlugin.java:50) ~[LuckPerms-Bukkit-5.4.121 (2).jar:?]
at org.bukkit.plugin.java.JavaPlugin.setEnabled(JavaPlugin.java:287) ~[paper-api-1.20.4-R0.1-SNAPSHOT.jar:?]
at io.papermc.paper.plugin.manager.PaperPluginInstanceManager.enablePlugin(PaperPluginInstanceManager.java:188) ~[paper-1.20.4.jar:git-Paper-463]
at io.papermc.paper.plugin.manager.PaperPluginManagerImpl.enablePlugin(PaperPluginManagerImpl.java:104) ~[paper-1.20.4.jar:git-Paper-463]
at org.bukkit.plugin.SimplePluginManager.enablePlugin(SimplePluginManager.java:507) ~[paper-api-1.20.4-R0.1-SNAPSHOT.jar:?]
at org.bukkit.craftbukkit.v1_20_R3.CraftServer.enablePlugin(CraftServer.java:639) ~[paper-1.20.4.jar:git-Paper-463]
at org.bukkit.craftbukkit.v1_20_R3.CraftServer.enablePlugins(CraftServer.java:550) ~[paper-1.20.4.jar:git-Paper-463]
at net.minecraft.server.dedicated.DedicatedServer.initServer(DedicatedServer.java:275) ~[paper-1.20.4.jar:git-Paper-463]
at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1131) ~[paper-1.20.4.jar:git-Paper-463]
at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:319) ~[paper-1.20.4.jar:git-Paper-463]
at java.lang.Thread.run(Thread.java:1583) ~[?:?]

Quoting Luck on #3676, the suggested course of action here is to:

Roll back to the last installed version of LuckPerms that worked, and do a /lp export.
Stop the server.
Delete (create a backup / move it somewhere else) the old database file.
Start the server and run /lp import.

btw with the old version was nothing wrong. I think i just should stay with the old version of this plugin. It causes no problems.


Send logs to pipeline during pke bootstrap

Update the PKE user-data script to send error output and result of the bootstrapping process. Depends on API endpoint made in #3174

No user data script change needed.


--file-lines does not work with stdin

--file-lines seems to only be able to reformat part of a file and not part of stdin.

--file-lines is a bit WIP I'm afraid, I'm not even sure what the current status there is

it worked when I tested it ^^

THis should work now.


Usersnap Feedback - It didn't work when I tried to log in using login info from old site. So t[...]

Sender<EMAIL_ADDRESS>Comment: It didn't work when I tried to log in using login info from old site. So then I asked to reset my password, but I never received a resete email
Open #27 in Usersnap Dashboard

Download original image
Browser: Chrome 58 (Windows 10)
Referer: https://participedia.xyz/
Screen size: 1920 x 1200 Browser size: 1064 x 1193
Powered by usersnap.com

This issue was moved to participedia/usersnaps#9


Update docs

Mentioning about lsquic tutorial code or presentation from Netdev can be so valuable for newcomers. I haven't found any references to them in docs so far so I have made this small PR. I have also copied from Netdev slides explanation of local address in one function but honestly I don't still get why we also pass local address to lsquic_engine_connect.

Thank you, @mickel8, I cherry-picked the first of these two commits.
The reason for passing the local address is that the program that instantiates the engine may be using more than one socket.  When engine calls the "send packets out" callback, it specifies the source address -- otherwise, the program may not know.  If the source address is not set correctly, the packet might not make it to the destination; or, the response to that packet may be sent to the wrong address.

Thanks a lot for the explanation :slightly_smiling_face:


Uploading look like slow than expected

Only 5~8 Mbps even when that under Gbps network

Chunk size may become 150KB not 150MB on file sync up


feature: Automatic type recognition

Codecov Report

Merging #283 into master will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #283      +/-   ##
==========================================
+ Coverage   86.38%   86.43%   +0.05%
==========================================
  Files          50       50
  Lines        3100     3112      +12
==========================================
+ Hits         2678     2690      +12
  Misses        294      294
  Partials      128      128

Impacted Files
Coverage Δ

config/profile.go
83.91% <100.00%> (+1.03%)
:arrow_up:

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 3ec520b...d580ff3. Read the comment docs.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.96%. Comparing base (3ec520b) to head (a3ab5bf).
Report is 518 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #283      +/-   ##
==========================================
+ Coverage   86.38%   86.96%   +0.57%
==========================================
  Files          50       50
  Lines        3100     3114      +14
==========================================
+ Hits         2678     2708      +30
+ Misses        294      282      -12
+ Partials      128      124       -4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Program freezes trying to load in team_data.txt

I tried to run this on the Blood Pit Bouncers, but the top line doesn't copy nicely due to them being buttons now. I tried to "fake it" but it freezes the program?
Attached is the text file it's trying to parse. I probably did it wrong.
team_data.txt

I'll have a look later. The file needs to be in the same directory as the executable. Is it getting as far as opening the file?

All three files are in the same folder, as is the terminal window that's
accessing the exe.
It's just stuck on "loading team_data.txt", I had it running in the
background for like half an hour with no changes.
On Sat, 20 Jul 2024, 12:27 arkadye, @.***> wrote:

I'll have a look later. The file needs to be in the same directory as the
executable. Is it getting as far as opening the file?
—
Reply to this email directly, view it on GitHub
https://github.com/arkadye/team_picker/issues/1#issuecomment-2241077913,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD6DCQ4LTTQZPINX7ZX6MLZNI3QDAVCNFSM6AAAAABLFXFPKKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBRGA3TOOJRGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

I found the issue. It chokes on the empty line at the end. This version work and I'll get round to fixing that bug at some point soon.
team_data.txt


Can't edit existing teacher

Currently it is not possible for an admin to edit a teacher's profile -- e.g., email, name, etc.  -- in the Admin dashboard.  This is because we are trying to avoid duplicate teachers.    But this has to be fixed in such a way that it prevents duplicate teachers but allows editing.

Fixed -- by distinguishing between Edit and Add modes.


"mil2525d-points-only.style" readme update

This is the proper edit for the readme in v.next, I mistakenly opened the last pull request in the master branch

Looks good


Column with over 2GB size limit but still identified as String in schema

Describe the usage question you have. Please include as many useful details as  possible.
Hi folks,
I read online that in PyArrow a string column would have a column-level size limit of 2GB. However, in my work I noticed this doesn't hold.
def some_function(
        self, raw_table: pa.Table,
    ):
        schema = raw_table.schema
        df = pl.DataFrame(raw_table)

In the code above, the table raw_table has some document column with size over 2GB and I used sum(buf.size if buf is not None else 0 for buf in arrow_array.buffers() for size checking. But when I check the schema it says that column is string type.
I later cast it to polar for some basic processing (barely introducing new memory) then cast back to arrow, which automatically turns all string to large string due to polar's default setting. Trying to cast the document column back to string type but it got casting error and it always stop at some fixed number of rows. I calculated the size processed and its roughly 2GB.
So considering both cases, pretty sure the column is def over 2GB and there's no calculation error in the first place. However super curious why it's showing string type in the first place?  Is there's something subtle I'm not aware of?
Component(s)
Python

So the 2GB limit is on a single Array, but the columns of a Table are not Arrays. Instead they're ChunkedArrays which you can think of as a list of Arrays. Each individual chunk in a ChunkedArray has the 2GB limit, but the total of the ChunkedArray can surpass that limit.
It's possible Polars converted to LargeString and combined the chunks into a single array, which then on the Python side can't be casted back into a String anymore.

More info:
https://arrow.apache.org/docs/format/Glossary.html#term-chunked-array
https://stackoverflow.com/questions/71428738/is-it-possible-to-append-rows-to-an-existing-arrow-pyarrow-table/71429073#71429073

@JazJaz426 I see you reacted to the posts, does this answer your question?


Updated instructions for Debian stretch on XPS 9350

This updates the Debian install instructions for the XPS 9350, and for the latest version of Debian testing (stretch).
I observed lots of hard graphics crashes when I first got my XPS upon installing Debian. It turned out that these issues were from a hardware issue, and were present on Ubuntu 16.04 LTS as well, and were fixed by a Dell support ticket that led to an in-person replacement of my motherboard. Because this is an anomaly, it's not expected to happen to other users, and not documented in the installation instructions, but I wanted to note it here for the record.
I did observe one graphics crash -- with my fixed computer -- after installing Debian. It happened while I was in Firefox, on the www.google.com/chrome homepage, on my way to click the blue download button. My screen got some blue and purple splotches on it, in a pattern like this:

However, this was before I installed the firmware-linux-nonfree package, which contains an update to libdrm that other online resources suggested was required for stability. I also observed -- before I installed firmware-linux-nonfree -- that when I went to set a graphical boot screen by installing Plymouth, the update-initramfs step emitted warnings about missing necessary firmware.
I have not seen this crash, or any crash, since installing this firmware update, nor do I see any boot warnings about firmware. Importantly, the graphics crash above doesn't look anything like the graphics crashes I observed prior to fixing my computer's motherboard. So, while the crash was scary, I continue to feel tentatively confident that my computer is in working order now and that my instructions are solid. [Update a week later: Still no crashes since installing firmware-linux-nonfree. Perfectly stable.]
If I observe any further crashes, I'll update the guide to say so, along with whatever else I find out that contributes to it and to resolving it.

Cool. I just noticed this. :) BTW, I need to go install Steam on mine... ;)


prefix configuration

When deploying to some hosts, for example github pages, the site contents are required to live under a path prefix. This means that something like <link href="/project-name/style.css" /> would fail to check unless an artificial directory is added before checking. Would you consider adding this as a configuration option?

I believe it's easy (and performant) enough to add the artificial directory or to switch your SSG output to relative URLs, so I am inclined not to.


Can't run simulation "TOKEN LIMIT EXCEEDED" error

I installed it all correctly, everything seems to work, but when I try to run it, even with step count to 1,
it just prints "TOKEN LIMIT EXCEEDED". I'm not sure where the problem is, this OpenAI changes something that broke the code? I have a paid API with usage tier 3, so I think it was supposed to work, I tracked the API key the code is using but it seems that It's not even calling for the GPT3, it just makes calls for the "Text-embedding-ada-002-v2", if the OpenAI API tracking is accurate it didn't make any calls for the GPT API.
Anyone was able to run this recently? Anyone has any idea of what could e the cause of this error?
Edit: Also, it always ends in an error before asking to enter an option again:
Today is February 13, 2023. From 00:00AM ~ 00:00AM, Isabella Rodriguez is planning on TOKEN LIMIT EXCEEDED.
In 5 min increments, list the subtasks Isabella does when Isabella is TOKEN LIMIT EXCEEDED from  (total duration in minutes 1440):
1) Isabella is
TOKEN LIMIT EXCEEDED
TOODOOOOOO
TOKEN LIMIT EXCEEDED
-==- -==- -==-
TOODOOOOOO
TOKEN LIMIT EXCEEDED
-==- -==- -==-
Traceback (most recent call last):
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/reverie.py", line 468, in open_server
    rs.start_server(int_count)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/reverie.py", line 379, in start_server
    next_tile, pronunciatio, description = persona.move(
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/persona.py", line 222, in move
    plan = self.plan(maze, personas, new_day, retrieved)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/persona.py", line 148, in plan
    return plan(self, maze, personas, new_day, retrieved)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/cognitive_modules/plan.py", line 959, in plan
    _determine_action(persona, maze)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/cognitive_modules/plan.py", line 573, in _determine_action
    generate_task_decomp(persona, act_desp, act_dura))
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/cognitive_modules/plan.py", line 164, in generate_task_decomp
    return run_gpt_prompt_task_decomp(persona, task, duration)[0]
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/prompt_template/run_gpt_prompt.py", line 439, in run_gpt_prompt_task_decomp
    output = safe_generate_response(prompt, gpt_param, 5, get_fail_safe(),
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/prompt_template/gpt_structure.py", line 268, in safe_generate_response
    return func_clean_up(curr_gpt_response, prompt=prompt)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/prompt_template/run_gpt_prompt.py", line 378, in __func_clean_up
    duration = int(k[1].split(",")[0].strip())
IndexError: list index out of range
Error.
Enter option:

I installed it all correctly, everything seems to work, but when I try to run it, even with step count to 1, it just prints "TOKEN LIMIT EXCEEDED". I'm not sure where the problem is, this OpenAI changes something that broke the code? I have a paid API with usage tier 3, so I think it was supposed to work, I tracked the API key the code is using but it seems that It's not even calling for the GPT3, it just makes calls for the "Text-embedding-ada-002-v2", if the OpenAI API tracking is accurate it didn't make any calls for the GPT API.
Anyone was able to run this recently? Anyone has any idea of what could e the cause of this error?

Mine runs for a little while. I will re-run it in the next few days, and can show you how far mine gets. Seems less of an error, and maybe tied to OpenAI limits I may change. But glad to hear I am not the only one with the issue, because I thought there may have been something wrong with my code.

I have the exact same issue

I solved the problem. I will list the solution here and make a PR later (although author seems to be inactive nowadays to review PRs)
The issue is simple, on Jan 4th 2024 OpenAI stopped all models with names such as "text-davinci-00x" (x=123). However, since model names are hard coded in Generative Agents all over the place instead of a single setup file, that means you need to search through the entire project for key word "davinci" and replace all of them to "gpt-3.5-turbo-instruct" (see: https://platform.openai.com/docs/deprecations)
And also a note why this error is hard to spot is because original version assumed that the only error is from token out of bound (hard coded bounds); they did not expect models get replaced. I found this error by replacing the TRY-EXCEPT structure and print the exception message directly instead of printing  a hard coded TOKEN LIMIT EXCEEDED msg.

I solved the problem. I will list the solution here and make a PR later (although author seems to be inactive nowadays to review PRs)
The issue is simple, on Jan 4th 2024 OpenAI stopped all models with names such as "text-davinci-00x" (x=123). However, since model names are hard coded in Generative Agents all over the place instead of a single setup file, that means you need to search through the entire project for key word "davinci" and replace all of them to "gpt-3.5-turbo-instruct" (see: https://platform.openai.com/docs/deprecations)
And also a note why this error is hard to spot is because original version assumed that the only error is from token out of bound (hard coded bounds); they did not expect models get replaced. I found this error by replacing the TRY-EXCEPT structure and print the exception message directly instead of printing a hard coded TOKEN LIMIT EXCEEDED msg.

I changed it, but still got the error :(

I have changed all the "text-davinci-00x"  to "gpt-3.5-turbo-instruct", but still receive the TOKEN LIMIT EXCEEDED msg.
Why is that?

I installed it all correctly, everything seems to work, but when I try to run it, even with step count to 1, it just prints "TOKEN LIMIT EXCEEDED". I'm not sure where the problem is, this OpenAI changes something that broke the code? I have a paid API with usage tier 3, so I think it was supposed to work, I tracked the API key the code is using but it seems that It's not even calling for the GPT3, it just makes calls for the "Text-embedding-ada-002-v2", if the OpenAI API tracking is accurate it didn't make any calls for the GPT API.
Anyone was able to run this recently? Anyone has any idea of what could e the cause of this error?
Edit: Also, it always ends in an error before asking to enter an option again:
Today is February 13, 2023. From 00:00AM ~ 00:00AM, Isabella Rodriguez is planning on TOKEN LIMIT EXCEEDED.
In 5 min increments, list the subtasks Isabella does when Isabella is TOKEN LIMIT EXCEEDED from  (total duration in minutes 1440):
1) Isabella is
TOKEN LIMIT EXCEEDED
TOODOOOOOO
TOKEN LIMIT EXCEEDED
-==- -==- -==-
TOODOOOOOO
TOKEN LIMIT EXCEEDED
-==- -==- -==-
Traceback (most recent call last):
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/reverie.py", line 468, in open_server
    rs.start_server(int_count)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/reverie.py", line 379, in start_server
    next_tile, pronunciatio, description = persona.move(
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/persona.py", line 222, in move
    plan = self.plan(maze, personas, new_day, retrieved)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/persona.py", line 148, in plan
    return plan(self, maze, personas, new_day, retrieved)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/cognitive_modules/plan.py", line 959, in plan
    _determine_action(persona, maze)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/cognitive_modules/plan.py", line 573, in _determine_action
    generate_task_decomp(persona, act_desp, act_dura))
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/cognitive_modules/plan.py", line 164, in generate_task_decomp
    return run_gpt_prompt_task_decomp(persona, task, duration)[0]
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/prompt_template/run_gpt_prompt.py", line 439, in run_gpt_prompt_task_decomp
    output = safe_generate_response(prompt, gpt_param, 5, get_fail_safe(),
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/prompt_template/gpt_structure.py", line 268, in safe_generate_response
    return func_clean_up(curr_gpt_response, prompt=prompt)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/prompt_template/run_gpt_prompt.py", line 378, in __func_clean_up
    duration = int(k[1].split(",")[0].strip())
IndexError: list index out of range
Error.
Enter option:

I also have the same issue, but I did not use OpenAI's large language model. Instead, I use other LLM model that is compatible with the interface style of openai. So I replaced all model names and modified the corresponding base_url, but it still seemed easy to encounter this problem. And I updated the version of OpenAI's library and replaced the corresponding old version interfaces.
At this point, most of the functionality has actually been completed. The only thing is that the model I'm using does not support openai's interface(/v1/completions). So I changed "openai.Completion.create" in "gpt_structure.py" to "openai.chat.completions.create", and modified the parameter "prompt=prompt" to "messages=[{"role": "user", "content": prompt}]". The compatibility issue was resolved, so when trying to run it again, it worked fine!
I hope my debugging experience can help you.

I also have the same issue, but I did not use OpenAI's large language model. Instead, I use other LLM model that is compatible with the interface style of openai. So I replaced all model names and modified the corresponding base_url, but it still seemed easy to encounter this problem. And I updated the version of OpenAI's library and replaced the corresponding old version interfaces.
At this point, most of the functionality has actually been completed. The only thing is that the model I'm using does not support openai's interface(/v1/completions). So I changed "openai.Completion.create" in "gpt_structure.py" to "openai.chat.completions.create", and modified the parameter "prompt=prompt" to "messages=[{"role": "user", "content": prompt}]". The compatibility issue was resolved, so when trying to run it again, it worked fine!

when i user other llm,  it comes error:

I installed it all correctly, everything seems to work, but when I try to run it, even with step count to 1, it just prints "TOKEN LIMIT EXCEEDED". I'm not sure where the problem is, this OpenAI changes something that broke the code? I have a paid API with usage tier 3, so I think it was supposed to work, I tracked the API key the code is using but it seems that It's not even calling for the GPT3, it just makes calls for the "Text-embedding-ada-002-v2", if the OpenAI API tracking is accurate it didn't make any calls for the GPT API.
Anyone was able to run this recently? Anyone has any idea of what could e the cause of this error?
Edit: Also, it always ends in an error before asking to enter an option again:
Today is February 13, 2023. From 00:00AM ~ 00:00AM, Isabella Rodriguez is planning on TOKEN LIMIT EXCEEDED.
In 5 min increments, list the subtasks Isabella does when Isabella is TOKEN LIMIT EXCEEDED from  (total duration in minutes 1440):
1) Isabella is
TOKEN LIMIT EXCEEDED
TOODOOOOOO
TOKEN LIMIT EXCEEDED
-==- -==- -==-
TOODOOOOOO
TOKEN LIMIT EXCEEDED
-==- -==- -==-
Traceback (most recent call last):
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/reverie.py", line 468, in open_server
    rs.start_server(int_count)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/reverie.py", line 379, in start_server
    next_tile, pronunciatio, description = persona.move(
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/persona.py", line 222, in move
    plan = self.plan(maze, personas, new_day, retrieved)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/persona.py", line 148, in plan
    return plan(self, maze, personas, new_day, retrieved)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/cognitive_modules/plan.py", line 959, in plan
    _determine_action(persona, maze)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/cognitive_modules/plan.py", line 573, in _determine_action
    generate_task_decomp(persona, act_desp, act_dura))
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/cognitive_modules/plan.py", line 164, in generate_task_decomp
    return run_gpt_prompt_task_decomp(persona, task, duration)[0]
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/prompt_template/run_gpt_prompt.py", line 439, in run_gpt_prompt_task_decomp
    output = safe_generate_response(prompt, gpt_param, 5, get_fail_safe(),
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/prompt_template/gpt_structure.py", line 268, in safe_generate_response
    return func_clean_up(curr_gpt_response, prompt=prompt)
  File "/Users/tiberio/Documents/Cesar School (Mac)/TCC/generative_agents/reverie/backend_server/persona/prompt_template/run_gpt_prompt.py", line 378, in __func_clean_up
    duration = int(k[1].split(",")[0].strip())
IndexError: list index out of range
Error.
Enter option:

I also have the same issue, but I did not use OpenAI's large language model. Instead, I use other LLM model that is compatible with the interface style of openai. So I replaced all model names and modified the corresponding base_url, but it still seemed easy to encounter this problem. And I updated the version of OpenAI's library and replaced the corresponding old version interfaces. At this point, most of the functionality has actually been completed. The only thing is that the model I'm using does not support openai's interface(/v1/completions). So I changed "openai.Completion.create" in "gpt_structure.py" to "openai.chat.completions.create", and modified the parameter "prompt=prompt" to "messages=[{"role": "user", "content": prompt}]". The compatibility issue was resolved, so when trying to run it again, it worked fine! I hope my debugging experience can help you.

I tried your method but it didnt work so i looked up the openai document. For people using gpt-3.5-turbo, they just need to change "GPT_request" in "gpt_structure.py", modify the parameter "prompt=prompt" to "messages=[{"role": "user", "content": prompt}]", and dont need to change "openai.Completion.create" to "openai.chat.completions.create". But still very thanks!!!

I tried your method  but it didnt work. Does anyone know which step I do wrong？


Cannot run on clear VSC

Hi, I tried to install and launch .net meteor on my project, but when I click start nothing happened. "Start debugging" and "Start without debugging" also don't respond. Nothing in logs, not a single error.
MacOs

Hi! Can you send the /Users/NAME/.vscode/extensions/nromanov.dotnet-meteor-X.X.X/extension/bin/Debug/logs/Debug.log file, please?

@JaneySprings no such file or directory: logs

@JaneySprings It is works on empyt project, but idk what is wrong with mine

Can you send an example? I think that you can copy only csproj file because the problems are usually there.

@JaneySprings I found it. I had some config in .target files and vsc didn't find them, so I had to put them explicit in csproj

Everything is OK for now?
If you can debug this project from VisualStudio for Windows, please, you can send me an example where I can fix this issue.

@JaneySprings now I get incorrect path to program, but it is good path. I am using rider actually, so maybe this is the problem

OK, close for now.


Sink now applies global filters as well

Which Issue(s) this Pull Request Resolves

Resolves #526
Release Note

NONE

Notes for Reviewers

My original intention was to refactor the code so global and local are treated in the same way and then do "namespace filters" + "global filters". However I found this approach in which we keep treating global filters in the same way. I will probably open an issue to refactor it. I think making this work has more priority than refactoring.
I had to change what was implemented in #534 because the init function provides a poor experience for testing. I didn't catch that in the PR that introduced it. I moved the retrieve part to the sink entrypoint and now NewFilters receives the value of the globalKey instead.

I think some errors still get lost

I was wrong about this

@ggallen could you approve this PR when you are OK with the changes?

@skoved @ggallen I had to do a major change, so I re-requested a review, to be able to test because of the implementation of #534, the init function won't let me specify an environment variable properly. Let me know your thoughts.

globalKey is not a constant, so it can be changed. My thought is that we can leave things as they were before but we can do some setup before running filter_test.go by creating a TestMain function in it. This function runs before all the other Test* functions. ` It could look something like this
func TestMain(m *testing.M) {
    globalKey = "kubearchive" // or some other value
    m.Run()  // runs the Test* functions
    // clean up if necessary
}

I implemented what you suggested and it works properly, thanks for the suggestion.

I don't think that we need to make an changes to non test code to get the tests to work properly.

Then there was/is no need for the last refactoring of this PR?


Convolutional RNNs for 1D sequences

I would like to user multiple scales of convolutions over character sequences, followed by RNN layers. This is currently clumsily possible by ignoring masking but I would love if there were something like ConvLSTM1D/ConvGRU1D which enabled this use-case directly.
Thanks!

Sorry, but what do you mean by words "ignoring masking"?

I pad shorter sequences with 0 at the end and then embed that padding token along with other "real" tokens (i.e. mask_zero=False). Does that make sense?

You mean, a ConvLSTM1D layer that would support masking?

@fchollet I might be misunderstanding how the convolutional recurrent models work, but don't they already support masking?
My use-case is biological sequences, so the dimensionality is typically (n_timesteps, embedding_dims). My loose understanding of ConvLSTM2D is that it's meant for video data with dimensions like (n_timesteps, width, height, channels) Is there a way to use ConvLSTM2D for my case? Alternatively, would it be possible to make Conv1D work with masked sequences of varying lengths?

Alternatively, would it be possible to make Conv1D work with masked sequences of varying lengths?

That would be a nice feature to have, but not easy to implement. Feel free to look into it, or find people to look into it.

Is there a way to use ConvLSTM2D for my case?

Why not feed your data as (n_timesteps, length, 1, channels)?

Why not feed your data as (n_timesteps, length, 1, channels)

Since I just have a single sequence I would have a dataset with dimensions (length, 1, 1, channels) or alternatively, (1, length, 1, 1) -- would that work with ConvLSTM2D? (I suspect not)

If you do not have a spatial component, I don't understand why you would want to use ConvLSTM?

I want temporal convolutional filtering, with an RNN to reason about relative order of filter responses. I don't know if ConvLSTM is the right tool for this  -- it seems like a mask-aware Conv1D would be better but I understand that might be a significant undertaking.

It seems like ConvRNN is not what you need in your case.

I want temporal convolutional filtering, with an RNN to reason about relative order of filter responses

It seems like you are looking for a stack of Conv1D layers followed by a RNN layer?

Yep, that's exactly what I want. The problem is, as far as I can tell,
there is no Conv1D which works with variable length sequences (by
propagating a mask up to the RNN layer).
On Apr 9, 2017 4:33 PM, "François Chollet"<EMAIL_ADDRESS>wrote:

It seems like ConvRNN is not what you need in your case.
I want temporal convolutional filtering, with an RNN to reason about
relative order of filter responses
It seems like you are looking for a stack of Conv1D layers followed by a
RNN layer?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/6150#issuecomment-292814555,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC9OW1u05ylHDiVpCSMcqqYO5RBqnFnks5ruU6MgaJpZM4MzTBI
.

If you know how the output of your Conv1D should be masked, you can pass an explicit mask argument (tensor) to your RNN layer.
That, or add a compute_output_mask method to conv layers.

@iskandr How did you end up solving the problem?
I have a very similar situation, and I'm using ConvLSTM2D with
kernel_size=(1,1)
and
 input shape=(timesteps, channels, 1, 1)
@fchollet  The network is getting very good accuracies so it surely works, but maybe I should be using something else? Conv1D/SeparableConv1D?

@ja-94 this is great news to hear, can you post your model? I'm a total beginner with neural networks (tried just a few simple LSTM/GRU models)... I was wondering if ConvLSTM2D were good for acoustic data and your answer seems to definitely point toward the "yes". Now the problem is building the model! :)

@ja-94 this is great news to hear, can you post your model? I'm a total beginner with neural networks (tried just a few simple LSTM/GRU models)... I was wondering if ConvLSTM2D were good for acoustic data and your answer seems to definitely point toward the "yes". Now the problem is building the model! :)

#################################################  CREATE MODEL  ###############################################################################
    maxlen=number_of_timesteps=x_train.shape[1]
    input_dim=number_of_neurons=x_train.shape[2]
    model = Sequential()
    input_shape=(maxlen,input_dim,1,1)
    print('input_shape=', input_shape)
    model.add(ConvLSTM2D(12, data_format='channels_last',
                            input_shape=input_shape,
                            kernel_size=(1, 1),
                            strides=(1, 1),
                            padding='valid',
                            dilation_rate=(1, 1),
                            activation='tanh',
                            recurrent_activation='hard_sigmoid',
                            use_bias=True,
                            kernel_initializer='glorot_uniform',
                            recurrent_initializer='orthogonal',
                            bias_initializer='zeros',
                            unit_forget_bias=True,
                            kernel_regularizer=None,
                            recurrent_regularizer=None,
                            bias_regularizer=None,
                            activity_regularizer=None,
                            kernel_constraint=None,
                            recurrent_constraint=None,
                            bias_constraint=None,
                            return_sequences=True,
                            go_backwards=False,
                            stateful=False,
                            dropout=0.0,
                            recurrent_dropout=0.0))
    model.add(Flatten())
    model.add(Dense(y_train.shape[1], activation='sigmoid'))
    model.compile(loss='binary_crossentropy', optimizer='adadelta', metrics=['accuracy', metrics.binary_accuracy])

    model.summary()


Cob/dollar mapper

@antonina-cherednichenko I sketched in how to map to dollars in a couple of "TODOs". Also, I removed some inheritance and constructor args that work better with only one source.

also managed to commit some merge conflicts on my own comments on these todos 🤔


Compilation Failure when Making a Production Build with create-react-app v2

Is this a bug report?
NO
NOTE: I does not encounter this issue in create-react-app v1
I encounter this compilation failure when doing a production build after migrating from create-react-app v1 to create-react-app v2. The local development, i.e: yarn start, is still working properly.
I tried to fix this by using SCSS interpolation, i.e: wrap $progress-size and $border-size with #{} --> #{$progress-size} #{$border-size} and it solves the problems.

I wonder why I have to fix like that? Thanks in advances.

Similar problem here, get this error when I hit the first $variableDefinition while attempting to compile. If I comment this out it will stop at the second variable definition. 😢

False alarm on my part. By elimination, I found that the culprit was a SASS function. It turned out I was using a rem-calc function that lives on another file (_functions.scss), without doing the import.
Doing the import did the trick and got rid of the error. ✨
@import "functions";

While issue closed 3 days ago...
I have the same problem
Creating an optimized production build...
Failed to compile.

Lexical error on line 1: Unrecognized text.
  Erroneous area:
1: 100% - $sidebar-width
^.........^
error Command failed with exit code 1.

Using:
   "react-scripts": "^2.1.1",
Any idea how to solve it?
Thanks!!!

@Mulli Same issue for me.
Your 100% - $sidebar-width is in a CSS calc() ?
I fix it with wrap variable with sass interpolation #{}.
Try this insted: 100% - #{$sidebar-width}

Yes. I did the same following the answer of @npkhoa2197.
Thanks!!!

My bad.... I did not read all the message.... Thanks!


[MNT] Sweep issue template

Adds an issue template for Sweep, which has been enabled in aeon for now (currently testing).
#730 I made an attempt at using the bot here, see how it goes. The prompt may not be the best 🙂.

looks good, I wonder if we can ask it to write coverage tests? :)


[QnA Maker] New QnA is not working in Azure. No response from 'Test in web chat'.

New QnA is not working in Azure. I have done following steps:
a. Created a QnA cognitive service in Azure.
b. With the Azure service, i have created a KB in new QnA portal. Uploaded QnA content.
c. Created a bot using 'Bot services in Azure'. configured it using following keys in the app setting :

QnAKnowledgebaseId
QnAAuthKey
QnAEndpointHostName
QnASubscriptionKey

Please assist me if i need to config something else in the Azure bot app setting.?
Regards,

Which language are you using, and which version of the SDK are you using?

@stevengum97 I'm not using direct C# configuration in the Azure portal. Old QnA is still working fine there but when I'm configuring new QnA using the following keys, it is not working.
QnAKnowledgebaseId
QnAAuthKey
QnAEndpointHostName
QnASubscriptionKey

@stevengum97 Hi, QnA Maker(new) is working fine with the Azure bot's configurations now! Only following Keys are needed:
QnAKnowledgebaseId
QnAAuthKey
QnAEndpointHostName
Thanks

@imdineshgrewal Im struggling to fint he QnAEnpointHostName
Where can i find it ?

@vanucci  Go to the Settings tab of the knowledge base, and scroll down to the Deployment details:


Only for development?

The readme refers to Kestrel as a "development web server". Does this mean it wouldn't be appropriate for a production site? Currently I'm running ASP.NET MVC 4 + Nginx + Mono FastCGI, will Kestrel take the place of Mono FastCGI or will there be another type of server for production scenarios?

https://github.com/aspnet/KestrelHttpServer/issues/71

Thank you @akoeplinger!


Avoid division by zero

If all nodes are unconnected, dagreBB.h == 0, leading to nodes with NaN as their y coordinate.

Cool; thanks


Git checkout sometimes names the created directory "repository"

Issue Type: Bug
I apologize if this has been reported already - but searching for "git repository" is not very helpful.
I'm not sure how to reproduce this, as it happens very infrequently. But sometimes when cloning a git repo, VSCode fails to correctly name the resulting directory. Instead it falls back to "repository", "repository-1" and so on.
VS Code version: Code 1.32.3 (a3db5be9b5c6ba46bb7555ec5d60178ecc2eaae4, 2019-03-14T22:49:40.955Z)
OS version: Windows_NT ia32 6.1.7601

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6600 CPU @ 3.30GHz (4 x 3312)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_offsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.93GB (2.99GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (14)

Extension
Author (truncated)
Version

vscode-eslint
dba
1.8.2

gitlens
eam
9.5.1

toggle-excluded-files
eam
1.6.0

EditorConfig
Edi
0.13.0

RunOnSave
eme
0.0.18

vscode-test-explorer
hbe
2.9.1

vscode-language-pack-de
MS-
1.32.4

vscode-typescript-tslint-plugin
ms-
1.0.0

vscode-jest
Ort
2.9.2

vscode-docker
Pet
0.6.1

project-snippets
reb
0.5.0

java
red
0.42.1

vscode-coverage-gutters
rya
2.3.1

vscode-todo-highlight
way
1.0.4

Here's where that happens:
https://github.com/Microsoft/vscode/blob/a62b16e130b259127ff23ac41c0ddf1a7c88183b/extensions/git/src/git.ts#L339
Code does its best to figure out a good name for the folder, given the repository's URL. If it can't do so, it calls it repository.
In order to fix this you have to provide the URL of the cloned repository which causes the issue.

@joaomoreno It has happened with https://github.com/AlCalzone/node-zwave-js, but also other github repos. The weird thing is that it is not reproducible - it usually works, but fails sometimes. Maybe a timing issue?

It can't really be a timing issue... Next time it happens, send me the full git log which shows in Git: Show Git Output.

I haven't reproduced it yet, but I'll keep you posted.

Let me know :+1:

@joaomoreno It just happened again:
Repo URL: https://github.com/ioBroker/testing/
Notice the trailing slash, I think that causes the problem
The git output doesn't seem very helpful:
Suchen nach Git in: C:\Program Files\Git\cmd\git.exe
Verwenden von Git 2.21.0.windows.1 von C:\Program Files\Git\cmd\git.exe

git rev-parse --show-toplevel
git config --get commit.template
Open repository: d:\repository-1
git fetch
git status -z -u
git symbolic-ref --short HEAD
git rev-parse master
git rev-parse --symbolic-full-name master@{u}
git rev-list --left-right master...refs/remotes/origin/master
git for-each-ref --format %(refname) %(objectname) --sort -committerdate
git remote --verbose
git check-ignore -z --stdin
git status -z -u
git symbolic-ref --short HEAD
git rev-parse master
git rev-parse --symbolic-full-name master@{u}
git rev-list --left-right master...refs/remotes/origin/master
git for-each-ref --format %(refname) %(objectname) --sort -committerdate
git remote --verbose

Also happened on my 2nd try with another repo, also with a trailing slash in the URL -> it's reproducible.

@joaomoreno I would provide a PR, but I'm not sure about the requirements, tests and so on. This should be fixable with a simple change:
- let baseFolderName = decodeURI(url).replace(/^.*\//, '').replace(/\.git$/, '') || 'repository';
+ let baseFolderName = decodeURI(url).replace(/\/$/, '').replace(/^.*\//, '').replace(/\.git$/, '') || 'repository';

Basically, removing the trailing slash before doing any more replacements.

@AlCalzone Feel free to submit PR.

@joaomoreno Is it enough if I just do the above change? Or does this need an additional test somewhere?

It's enough! :+1:

PR is up

This has already been fixed.


Handle requests in controllers

Hi Sir, 👋
I have done the required job for this feature branch give me permission to merge this branch below mentioned what I did:

[ ] Add controllers
[ ] Add views
[ ] Add Routes
[ ] Update README file
[ ] Fix Linters

STATUS: INVALID CODE REVIEW REQUEST 🛑 🛑
Hi @MudasirSherwani,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
Some links (like Usage, Install, etc) in your Table of Contents menu do not have sections on the README. Kindly add the sections and ensure they are properly linked on the README
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


PLT-5659 Server URL input box does not scroll when a long URL is entered

Summary
By removing the right padding on the TextInput component style we are not longer truncating the autoscroll of the component making possible to auto scroll to the right when the text is too long.
Ticket Link
https://mattermost.atlassian.net/browse/PLT-5659

Wouldn't "PLT-5659 Allow server URL input box to scroll when a long URL is entered" be a more descriptive title for the PR?


Show TTL for key

When you open a key it would be nice if TTL would be displayed somewhere.

The feature requested is available with latest version.
If there is something missing at the implementation feel free to reopen issue with an updated description.


PostModel.CroppedPostImageUrl had incorrect query string

This was coded to do: PostImage.GetCropUrl("wide") + "&upscale=false" instead of PostImage.GetCropUrl("wide") + "?upscale=false"
meaning the query string "?" operator was not output resulting in an incorrect url and the image not rendering

The underlying problem is #356


Hover doc strings

In the current implementation, elab doc string is taken by preference to the syntax doc string. For example, when users hover over the calc keyword, they get the doc string.
Elaborator for the `calc` term mode variant.

instead of the (much more detailed) syntax doc string
Step-wise reasoning over transitive relations.
```
calc
  a = b := pab
  b = c := pbc
  ...
  y = z := pyz
```
proves `a = z` from the given step-wise proofs. `=` can be replaced with any
relation implementing the typeclass `Trans`. Instead of repeating the right-
hand sides, subsequent left-hand sides can be replaced with `_`.

`calc` has term mode and tactic mode variants. This is the term mode variant.

See [Theorem Proving in Lean 4][tpil4] for more information.

[tpil4]: https://leanprover.github.io/theorem_proving_in_lean4/quantifiers_and_equality.html#calculational-proofs

At #1403, @kha reported that this is the intended behavior since a single syntax can have multiple elaborators, so the elaborator docstring is expected to be the more specific one we should show and jump to first.
@digama0 points out that

From a doc writer's perspective, it is a lot harder to write good elab comments than syntax comments, because when you are documenting a tactic like simp you really want all the syntactic variants to be documented there, not spread over multiple comments where the user never gets the full picture.

Thread #1403 also suggests that there are two kinds of consumers for these doc strings: users and code-readers.

I am marking this issue as an RFC to get feedback from the community.

to be the more specific one we should show and jump to first.

Just to be clear: those are two completely different issues.  You can jump to either the elaborator or the syntax definition from the editor depending on which command you use.  So this is no argument for which docstring to show in the hover.
My current feeling is that we should show (only) the syntax docstring.  In virtually all cases right now, the elaborator/expander is just an implementation detail.  There might also be multiple elaborators/expanders, multiplying the documentation effort.  This might change if we add some more "extensible" syntax (similar to how we might want to show docstrings for type class instances in addition to or instead of the type class operation docstrings):
syntax "frobnicate!" term : term

-- and in another module:
/-- The MyStuff frobnication frobnicates my stuff. -/
macro_rules
  | `(frobnicate! MyStuff + 2) => ...


How do I narrow down the reactor's reaction?

        $user = Auth::user();
        $post = Post::find($requestData['post_id']);
        $reacterFacade = $user->viaLoveReacter();
        $reactantFacade = $post->viaLoveReactant();

Here is my setup. for adding reaction on the post. Now I want like which reaction is added by the user on perticular post.
$reactions = $reacterFacade->getReactions()

is giving all the reaction made by the user but I want to get reaction with perticular post. Also I want to get the name of the reaction but there is no relationship function is given with ReactionType with reacterFacade.
I tried to debug the package but did not found anything so asking here.
Also How can I stop Queue of Reaction Counter And Reaction Total Count. [NOTE] I want to update data immediately.

Have you tried to find reacters of the post instead?
According to the counter events, you can exclude event service provider loading and implement your own logic.

Yeah it seems like you should use low-level API to fit your needs.
$reactantId = $post->getReactant()->getId();
$userReactionsOnPost = $user->getReacterable()->reactions()->where('love_reactant_id', $reactantId)->get();

@Ajay-Bacancy here is an issue related to your question about stop queue of reaction counter:

#209

I wrote a new page in documentation with example how to make jobs queue sync:
https://laravel-love.readme.io/docs/custom-jobs-dispatching

Ohhh @antonkomarev  I have done the same thing.

Maybe in enhancement of we can do something like that where we can pass the reactant in getReactions facade like this
$reacterFacade->getReactions($reactant) to get reactors reaction on specific reactant.
But anyways you can lable this issue as question and close it and Thank you so much for the custom job dispatching guide really appreciated. Also very nice library for reaction. I have integrated few days back. Thanks Buddy 🤗

Looks good! Thanks for sharing! I will think about your proposal

@Ajay-Bacancy FYI this code block may work pretty slow in cases when reacter has many reactions, on each new reaction it will become slower:
$userreactions  = $reacterFacade->getReactions()->where('reactant_id', $reactant->id)->values()->pluck('reaction_type_id');

Here breakdown with explanation:
$allUserRactions  = $reacterFacade->getReactions(); // Here you collecting all the reactions of the user from the database
$filteredUserReactions = $allUserReactions->where('reactant_id', $reactant->id); // We are iterate through the collection and filter out items
$reactionTypeIds = $filteredUserReactions->values()->pluck('reaction_type_id');

As you can see, filtering operation is doing on the PHP side, not on database side, and this may be critical over time.
Instead of getting reactions using ReacterFacade use low-level API and get Reacter model.
$reacter = $user->getLoveReacter();
$reactant = $post->getLoveReactant();
$userReactionsOnPost = $reacter->reactions()->where('reactant_id', $reactant->getId())->values()->pluck('reaction_type_id');

@antonkomarev yeh got your point. Thank you so much buddy 😌❤

@Ajay-Bacancy I've created discussion how to implement this feature on high-level API. Will be glad to receive a feedback there: https://github.com/cybercog/laravel-love/discussions/239

And the first proof of concept:

#240

@Ajay-Bacancy I've introduced new method to fetch concrete user reactions. It's breaking change since API has been changed. Will be added in next major version release.

#240

I wrote a new page in documentation with example how to make jobs queue sync: https://laravel-love.readme.io/docs/custom-jobs-dispatching

Hey @antonkomarev I have one doubt regarding custom job dispatching.
IncrementReactionAggregatesJob::dispatch()->onConnection('sync');
DecrementReactionAggregatesJob::dispatch()->onConnection('sync'); required instance of $reactant, $reaction I'm doing it this way.
if ($isNotReacted) {
    $reacterFacade->reactTo($post, $requestData['reaction_type'], 1.0);
    $reaction = $reacterFacade->getReactions()->last();
    IncrementReactionAggregatesJob::dispatch($reactant, $reaction)->onConnection('sync');
} else {
     $reacterFacade->unreactTo($post, $requestData['reaction_type']);
     DecrementReactionAggregatesJob::dispatch($reactant)->onConnection('sync');
}

getting all reaction and passing last added reaction. This will works fine with Increment job but how do I pass reaction instance in decrement job?

One of the solutions: use events and write your own event listener.


Conflict between spring-cloud-aws-messaging and spring-boot-starter-amqp

The cloud messaging module looks for @MessageMapping annotations and checks that they have a valid destination in QueueMessageHandler.getMappingForMethod:
if (messageMappingAnnotation.value().length < 1) {
  throw new IllegalStateException("@MessageMapping annotation must have at least one destination");
}

The problem is that spring boot amqp defines this annotation:
@Target({ElementType.METHOD, ElementType.ANNOTATION_TYPE})
@Retention(RetentionPolicy.RUNTIME)
@MessageMapping // No destination!
@Documented
public @interface RabbitListener { ... }

As a result it's not possible to listen for SQS and RabbitMQ messages in the same project

Now I think about it, the QueueMessageHandler should probably look for some prefix like sqs:queue-name because otherwise it will assume that all @MessageMapping annotations are destined for it

Hi @thobson, thanks for reporting this issue.
I fixed it by removing the check for a value. A prefix is not necessary because when the container listener starts it checks if the queue exists or not (see org.springframework.cloud.aws.messaging.listener.AbstractMessageListenerContainer#start()). If the queue does not exist, it is just ignored and a warning is logged.
The fix should be in tomorrow's SNAPSHOT version.


:mega: Get travis notifications working again

Coverage remained the same at 93.173% when pulling 4ebd9dda78d181f2e56da22a3352bce32d572fae on feature/travis-notify into b29bbb36a261097249a0c9e24468863a16ed9ca5 on master.

Duplicate of #36

Coverage remained the same at 93.173% when pulling 4ebd9dda78d181f2e56da22a3352bce32d572fae on feature/travis-notify into b29bbb36a261097249a0c9e24468863a16ed9ca5 on master.


Task: Update deployment documentation

What is the task about?
Documentation needs to be updated showing that the keys need to be added to a Volume accessible by RSTUF Worker when using the Key Vault type LocalKeyVault
Parent feature

https://github.com/vmware/repository-service-tuf/issues/207

References
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

The required documentation

[ ] Document the RSTUF Worker configuration for the online key
[ ] Record a new video with the Ceremony process

[ ] Document the supported generated keys

Closed due #284