Switch to jbuilder, prepare to release 1.1.0

I've a couple of questions:

why rename a mli to ml?
how is the documentation supposed to be generated now?
what's the purpose of mirage-net-lwt.version and mirage-net.version? which process is supposed to update them on a new release?  manually?

Beta10 of jbuilder (the next one) is due to have an Isco target

Sorry, an "odoc" target :-)

@hannesm: the "ml -> mli" renaming is due to https://github.com/janestreet/jbuilder/issues/9
@djs55 if we want to keep the mli, it seems that the proper way to do it is to add an explicit copy rule: https://github.com/mirage/ocaml-github/pull/192#issuecomment-300442645
I have no strong opinion on whether we should keep the mli or have a ml file, but pure mlis don't work with module aliases which seems a good argument to me to not use them (until this is fixed in the compiler).

On 19/05/2017 20:03, Thomas Gazagnaire wrote:

I have no strong opinion on whether we should keep the mli or have a ml file, but pure mlis don't work with module aliases which seems a good argument to me to not use them (until this is fixed in the compiler).

Does odoc use .ml for generating documentation? I'm a bit worried to
lose automatic generation of documentation.

I've pushed a few more changes for review (can squash them later)

.version files now contain %%VERSION%%. Note that jbuilder understands several ways to find the version number to put in the generated META file -- it can also take the version from the opam file. My understanding is that opam-repository repository opam files are not supposed to contain the version so I opted for not putting the version in here either, so that I could manually diff and cp the opam files. I'm not sure what opam-publish does or prefers.
I copied a trick from odoc itself: the build rules now perform jbuilder subst if the package is {pinned}
I also copied the odoc package's pkg/pkg.ml file which references the new package topkg-jbuilder. After I installed this (opam pin add topkg-jbuilder git://github.com/diml/topkg-jbuilder.git since https://github.com/ocaml/opam-repository/pull/9231 is not yet merged) I could then run topkg build
I verified that odoc compile --help says Input file (either .cmti or .cmt). I did a odoc compile and then odoc html to check that stuff wa output -- so I think we're good with either the .ml or the mli file

BTW have we documented the ideal / best-practice package release workflow anywhere? I've still been relying on the github "release" button which I realise isn't ideal, and I haven't got a great way to automatically update API docs etc.

Sorry I meant 'jbuilder build --dev'

On 20 May 2017, at 01:27, Dave Scott<EMAIL_ADDRESS>wrote:
@djs55 commented on this pull request.
In src/jbuild:

@@ -0,0 +1,17 @@
+(library

((name        mirage_net)
(public_name mirage-net)
(libraries   (bytes result mirage-device))
(modules     (Mirage_net))
(flags       (:standard -w "A-4-41-44" -warn-error "+1..49" -safe-string))
In this case I coped the contents of the _tags file: true: warn_error(+1..49), warn(A-4-41-44). I'm not very familiar with the semantics of this though. Is it not safe to use a specified list "1,2,3,...,49" since they're already defined and won't be redefined in future versions of the compiler? I thought the main problem was with the equivalent of -Wall which includes future yet-to-be-defined errors.

Wrt to topkg could you explain a bit more? I see this:
$ topkg build --dev
topkg: unknown option --dev', did you mean -d' ?
Usage: topkg build [OPTION]... [BUILD_CONF]...
Try topkg build --help' or topkg --help' for more information.
(I was never a topkg expert). Is your suggestion to use topkg and jbuilder together? Could you outline what the build and release workflow would look like? Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

With topkg-jbuilder integration, 'topkg tag && topkg bistro' almost works: 'topkg doc' (used to generate the docs) will fail but @diml is working on this.
So I just used to follow what 'topkg help release' says but I'm replacing the "publish" part by 'topkg publish distrib" -- that will not try to build and publish the docs.

On 20 May 2017, at 01:55, Dave Scott<EMAIL_ADDRESS>wrote:
I've pushed a few more changes for review (can squash them later)
.version files now contain %%VERSION%%. Note that jbuilder understands several ways to find the version number to put in the generated META file -- it can also take the version from the opam file. My understanding is that opam-repository repository opam files are not supposed to contain the version so I opted for not putting the version in here either, so that I could manually diff and cp the opam files. I'm not sure what opam-publish does or prefers.
I copied a trick from odoc itself: the build rules now perform jbuilder subst if the package is {pinned}
I also copied the odoc package's pkg/pkg.ml file which references the new package topkg-jbuilder. After I installed this (opam pin add topkg-jbuilder git://github.com/diml/topkg-jbuilder.git since ocaml/opam-repository#9231 is not yet merged) I could then run topkg build
I verified that odoc compile --help says Input file (either .cmti or .cmt). I did a odoc compile and then odoc html to check that stuff wa output -- so I think we're good with either the .ml or the mli file
BTW have we documented the ideal / best-practice package release workflow anywhere? I've still been relying on the github "release" button which I realise isn't ideal, and I haven't got a great way to automatically update API docs etc.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I need to check what's in the META file after installing the output of
"topkg distrib". It might also be useful for "opam pin" if there's a
"jbuilder subst {pinned}" in the opam file? I'm happy to remove it if
everything has the right version without it :)
On 29 May 2017 1:50 pm, "Thomas Gazagnaire"<EMAIL_ADDRESS>wrote:

@samoht commented on this pull request.
In mirage-net-lwt.version
https://github.com/mirage/mirage-net/pull/7#discussion_r118929898:

@@ -0,0 +1 @@
+%%VERSION%%

I am still not totally sure why we need this file :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mirage/mirage-net/pull/7#pullrequestreview-40767074,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMHuqoacchcFJ4oPVKmfXp2odML1I-Wks5r-r77gaJpZM4NgwS6
.

oops i am clashing with #8

@avsm I'm happy to close this in favour of yours.
@samoht I think I now agree that the .version file is unnecessary -- somewhere topkg and jbuilder picked up the right version and put it in the META file.

@djs55:

@samoht I think I now agree that the .version file is unnecessary

waaaaaaaat i added all those version files for nothing??


didnt work right away...

took me forever to figure out that you also need to check off Internet (Client & Server), Internet (Client) and Private Networks (Client & Server)

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

ID: 92140a3c-b33d-6aa7-a904-d03c8122fbdf
Version Independent ID: 90a33ee7-ec4a-2bcc-b776-530a1c3014fd
Content: Use a SQL Server database in a UWP app - Windows UWP applications
Content Source: windows-apps-src/data-access/sql-server-databases.md
Product: uwp
Technology: data-and-files
GitHub Login: @lastnameholiu
Microsoft Alias: alholiu

Thanks for the heads up 😊 jensocan.
Sent from my Windows 10 device
From<EMAIL_ADDRESS>Sent: Friday, October 4, 2019 2:26 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: [MicrosoftDocs/windows-uwp] didnt work right away... (#1986)
took me forever to figure out that you also need to check off Internet (Client & Server), Internet (Client) and Private Networks (Client & Server)

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

ID: 92140a3c-b33d-6aa7-a904-d03c8122fbdf
Version Independent ID: 90a33ee7-ec4a-2bcc-b776-530a1c3014fd
Content: Use a SQL Server database in a UWP app - Windows UWP applicationshttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fwindows%2Fuwp%2Fdata-access%2Fsql-server-databases%23feedback&data=02|01||2676ac0a66644fdde33808d749117fda|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637058211812447723&sdata=UwM%2Fn9d5cke1u0ZX8t7k%2FHw3Liup%2B922aJgpPFL08Jo%3D&reserved=0
Content Source: windows-apps-src/data-access/sql-server-databases.mdhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-uwp%2Fblob%2Fdocs%2Fwindows-apps-src%2Fdata-access%2Fsql-server-databases.md&data=02|01||2676ac0a66644fdde33808d749117fda|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637058211812457722&sdata=k5p%2FnXLzvRWu7TDzmRJA%2BGjoFzP3FjHfD9OM3mnc8A0%3D&reserved=0
Product: uwp
Technology: data-and-files
GitHub Login: @lastnameholiuhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Flastnameholiu&data=02|01||2676ac0a66644fdde33808d749117fda|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637058211812467733&sdata=27MU0t4UBRWSejAr8hIKUIbGLabNUwGLoHsjl6ltO5o%3D&reserved=0
Microsoft Alias: alholiu

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-uwp%2Fissues%2F1986%3Femail_source%3Dnotifications%26email_token%3DAHX7KBDGXCQ3NE24HTXE4FDQM6YHXA5CNFSM4I5USVLKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HPY44UQ&data=02|01||2676ac0a66644fdde33808d749117fda|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637058211812477744&sdata=W164ojC9MIrHLzdOwT8FkS67lTwQ6TpSnG1O8y%2Btkes%3D&reserved=0, or mute the threadhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHX7KBFF64QJ4LH2ZVZVLMLQM6YHXANCNFSM4I5USVLA&data=02|01||2676ac0a66644fdde33808d749117fda|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637058211812487749&sdata=5ZZe1IfnYHiAb%2FL3PoEQxDwKApDXEHAUo5OXEyC51u8%3D&reserved=0.

it is still not working for me. In a console app everything functions just fine.

Thanks for the feedback, I've added this disclaimer to the instructions. Agmundr, if you're still having issues after following the updated steps, I'd encourage you to file a new issue here.
Thank you.


Remove require()

Remove if require. Dependencies are now in DESCRIPTION file. Remove from bootstrap code as well.
https://github.com/QuinnAsena/multinomialTS/blob/4259befc847b74ac719019e992bf028792d6e761/R/source_mnTS.R#L361

Dependencies are now also imported to NAMESPACE. require() is commented out. Remove after testing.


Module copy issue introduced in ansible 2.9.12

Hi,
there is an issue introduced in ansible 2.9.12.
The issue is when copying files the destination does not have the same permission as the original (it has 0600).
Please check your role for this issue.

@arachmani @tiraboschi @sandrobonazzola
cc @mwperina

Thanks @mnecas, already fixed in #361

Happy to hear that!


Update Gas Mask to 0.8.3

Thank you for the contribution. It was merged directly as 1af76d7bfd7ff5005e60416c51e2704ca47bda58 to keep commit history cleaner. Your contribution is still credited to you.


Document how to setup server side rendering with Angular Template

From @dominikfoldi on June 29, 2018 19:19
It would be awesome to have an AppModule that we can use to add only browser supported modules if SSR enabled. The idea is to have a shared AppModule that the server AppModule and the browser AppModule use. The browser one only bootstrapped in the browser not on the server.
You can find a great example in @MarkPieszak's repo: https://github.com/MarkPieszak/aspnetcore-angular2-universal
Copied from original issue: aspnet/Templating#597

Thanks for contacting us, @dominikfoldi.
@danroth27 what are your thoughts regarding this?

From @SteveSandersonMS on August 6, 2018 10:5
The goal for the template is to be as close to Angular CLI defaults as possible, and doesn't have SSR enabled by default. Hence I'd suggest it would make more sense for an AppModule or similar to be provided in the form of a blog post or wiki so that people who want to go in that direction can add it.

From @danroth27 on August 3, 2018 23:50
@SteveSandersonMS

From @dominikfoldi on August 6, 2018 10:12
@SteveSandersonMS okay this makes sense. I think the best place for it would be in the Angular project template documentation at Microsoft Docs: https://docs.microsoft.com/en-us/aspnet/core/client-side/spa/angular?view=aspnetcore-2.1&tabs=visual-studio

As per https://github.com/aspnet/AspNetCore.Docs/issues/8139#issuecomment-467613547 we don't want to make recommendations about Angular SSR.


Array - other method for array intersection

Here is another way to intersect two array.

Thanks @vreymond 👍 I added it here https://1loc.dev/#get-the-intersection-of-arrays


Implement tos-shell

Tos shell would be a new suboption of the tos command. It would install a given package in a closed environment, where it can be tested  and when you stop executing the shell it would disappear completely from your system.
Usage could look something like this:
# show the output in the terminal but the app runs in a closed env
tos shell asciicinema

# mount the app to the current x session (--gui flag)
tos shell --gui firefox

# expose a dir to the env
tos shell --expose /home/$USER ls

# expose the entire filesystem
tos shell --filesystem --gui firefox

# run an application that is named differently that the package
tos shell --gui awesome-tos awesome

This can be done in multiple ways: using docker as the backbone or using chroot (we might have a flag to do both) but that can be done in a later stage

TOS shell has been implemented, however for terminal based app's ctrl+c doesn't seem to work that well


DEV: dev.py: separate lint and flake8 tasks

Reference issue

closes #17434, closes https://github.com/scipy/scipy/issues/17394
What does this implement/fix?

As per discussion, https://github.com/scipy/scipy/issues/17434#issuecomment-1318566296, separates out the running of lint_diff.py and flake8 into separate tasks so that dev.py lint will now give the same output as the CI job.
Also very minor tidy up of consistent use of fullstops at the end of task descriptions.
Additional information

Thanks for looking into this @rgommers, yes that sounds like a good plan

@rgommers just to make this more fun the latest version of flake8 has removed --diff which breaks tools/lint_diff.py. I have opened a separate issue for that (https://github.com/scipy/scipy/issues/17503)


GEODE-1543: Change the log level from error to info

When multiple peers join the cluster conncurrently, only the first one that joins will distribute the index creation and other will try to create the index locally because of the distributed message and then log an error afterwards.
This is an expected behaviour and hence should not be logged as error but rather as a info.

+1


Misleading instructions in Learn how a Stack Works

Challenge Name

http://beta.freecodecamp.com/en/challenges/coding-interview-data-structure-questions/learn-how-a-stack-works
Issue Description

A Stack is a Last-In-First-Out data structure, so the most recent item inserted would be at the top, while the first item would be at the very end. In the challenge, the "stack" is setup this way -
var homeworkStack = ["BIO12","HIS80","MAT122","PSY44"];
The top of the stack is PSY44, as is also stated in the description -

Here we have a stack of homework assignments represented as an array: "BIO12" is at the base, and "PSY44" is at the top of the stack.

However, the instructions seem to indicate that element BI012 is the top -

Modify the given array and treat it like a stack using the JavaScript methods mentioned above. Remove the top element "BIO12" from the stack. Then add "CS50" to be the new top element of the stack.

If you run pop() on the array, similar to a stack's pop() operation, the element at the end PSY44 is removed, not BI012 -

Browser Information

Browser Name, Version: Chrome, 56
Operating System: Windows
Mobile, Desktop, or Tablet: Desktop

I'll make a quick PR to fix this.

@Manish-Giri glad to see I'm not the only one working on these. Check out this issue https://github.com/freeCodeCamp/freeCodeCamp/issues/12757 its kinda the hub for reviewing the advanced data structures and algorithms section.


Downstream unbundlers should populate setup.py

/cc @ralphbean @eriol
OpenStack has encountered yet another breakage caused by people combining pip installed and system packages: specifically, updating urllib3 via pip when requests/urllib3 are already installed via the system packages.
@dstufft tells me that, if the system packages correctly populate the install_requires part of setup.py, pip would respect those requirements. Previously this wasn't possible because requests would include mid-release versions of urllib3, but as of v2.6.2 our new release rules mean that we only use proper urllib3 releases.
For this reason, I think it would be extremely good if downstream unbundlers (most notably Fedora and Debian), when they unbundle requests, would populate setup.py with the correct install_requires. For 2.8.0, that would be urllib3==1.12. For 2.7.0, that would be urllib3==1.10.4. This should ensure that pip won't casually install a version of urllib3 that will break the system requests package.
Thoughts?

I think we can do that.  I'll have to think about the implications but this is a great improvement.  Right now Debian has requests 2.7.0 and urllib3 1.11 so they should be updated.

Right now Debian has requests 2.7.0 and urllib3 1.11 so they should be updated.

I've heard of issues with 2.7.0 and 1.11. I think Fedora pinned to 1.10.4. I'm surprised you haven't had complaints yet.

:smiley: Thanks Barry. For the sake of keeping track of everything, it would be super convenient if you could drop a link to any Debian discussion of that change in this issue. Obviously if you can't then that's not a big deal.

My plan is to update both soon! I was waiting for the DPMT migration to git. I don't see particular problems but we can always upload on experimental first to see if it's all right.

On Oct 09, 2015, at 11:09 AM, Daniele Tricoli wrote:

I was waiting for the DPMT migration to git.

Done!
https://wiki.debian.org/Python/GitPackaging

@warsaw thanks to make this happen! :smile:
I started reading<EMAIL_ADDRESS>after my reply here!
@Lukasa if you can point me to the OpenStack issue I will include it into the description of the patch Debian will use.
Thanks!

Thoughts?

Good idea.  Will report back when its in place.

@eriol OpenStack issue is [here](https://review.openstack.org/#/c/213310/0.

Patch to Fedora .spec files:

http://pkgs.fedoraproject.org/cgit/python-requests.git/commit/?id=f73b98e44a519ee15f3ceeea4c53e0e4dd25a5d4

Fedora updates created:

https://bodhi.fedoraproject.org/updates/FEDORA-2015-20de3774f4
https://bodhi.fedoraproject.org/updates/FEDORA-2015-1f580ccfa4
https://bodhi.fedoraproject.org/updates/FEDORA-2015-d7c710a812

Thank you @ralphbean and @eriol for working with us on this. :)

:heart: Indeed, thanks so much (and thanks @warsaw and @dstufft as well!).

On fedora, now it's showing this error on update to this package
Running transaction
Upgrading   : python-requests-2.7.0-7.fc22.noarch                         1/2
Error unpacking rpm package python-requests-2.7.0-7.fc22.noarch
Error unpacking rpm package python-requests-2.7.0-7.fc22.noarch
error: unpacking of archive failed on file /usr/lib/python2.7/site-packages/requests/packages/chardet: cpio: rename
python-requests-2.7.0-7.fc22.noarch was supposed to be installed but is not!

@ralphbean Theories?

People have been commenting on this bug about this error in fedora
https://bugzilla.redhat.com/show_bug.cgi?id=1236312

@vvro At this point there is no evidence that requests did anything to introduce this bug: it was almost certainly introduced downstream. It is possible that this issue caused it, but that remains a Fedora problem. I highly recommend centralising the discussion of this problem on the Red Hat bugzilla, rather than here.

Agreed.  'cpio: rename' issues are almost always a result of having done a 'sudo pip install requests' on the system and introducing ownership conflicts with the rpm management system -- dueling package management systems.

Sorry for the horrible delay, I had to change my usual workflow after the git migration, but a positive note is that now, after the usual testing, I'm also rebuilding some dependants (I had to speedup my setup to rebuild all of them) of requests.
Debian patch: https://anonscm.debian.org/cgit/python-modules/packages/requests.git/commit/?id=6c882aca203bd33c129e2b4e1e4f82886e4b1fe0
The package will uploaded soon: unfortunately although now I'm officially a Debian Maintainer my sponsor had problems to give me upload permissions. He tried but he got an error, and we will investigate during these days.

Uploaded: https://packages.qa.debian.org/r/requests/news/20151028T232257Z.html.

Alright, that's Debian and Fedora handled. Thanks so much @ralphbean @warsaw @eriol, we :heart: you greatly.

Thanks @eriol and I'm always happy to help sponsor.  One thing we need to keep in mind is that as we update requests' dependencies, we have to make sure to also update requests setup.py.  I don't think we have any mechanisms in Debian to ensure this but we can probably rely on Ubuntu's -proposed migration to keep track of mismatches: http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html
The other thing we may want to think about is DEP-8 tests to make these verifications.  Then at least we'll see problems on qa.debian.org.  A README.Debian is a good idea too, but they are easy to miss :(

@warsaw it's a pleasure work with you! Also reading your hints! I had the same thought on README.Debian, but I will add it in the next upload.
Ubuntu's -proposed migration is very interesting! I will keep an eye on it for my packages.
After the git migration I updated my workflow and I discovered some useful tools.
Before the last requests' upload I tried to use ratt on my laptop to rebuild all the reverse dependencies: it was a bit troublesome since one of the rdeps is blender and it take more than 1 hour to build, so I did not rebuild all of them but only some.
To "fix" my building problems I was suggested to use debomatic: https://debomatic.github.io/ :smile: I'm reading the documentation right now.
The only problem is that it's not automated, but I will start to use it for all my packages before uploading them on the archive.
What about a more draconian approach in addiction to DEP-8 test? I'm thinking about testing at build time and let the package fail to build if the setup.py was not correctly updated.
Do you have any hints about safely parse a setup.py? Thanks!


config: try updating soft limit before failing

Sometimes the hard limit meets our requirement but the soft limit. In this case we can update the soft limit directly.
@siddontang @overvenus PTAL

Please add test.

LGTM


How to destroy toaster manually using js code?

Hello Pratibha!
Do you mean to completely vanish all toasts and un-bind any bindings of ToasterJS?
If so, currently there is no such thing. But it can easily be implemented even today, just let me know.

Yes exactly, I wanted to destroy previously created one toaster or all.

Okay, give me a moment. I’ll notify you once it’s done!

Done! I've published version 2.0.1, as I decided to do major update on styles. Now, Toaster-JS assigns CSS styles instead of hard-coded opacity. Check this out!
Regarding to deleting Toast messages, use this:
const toast = new Toast("Hello!");

toast.delete();

Please let me know if that's what you expected. Or maybe you need something like deleteAll?

Nevermind, 2.1.0 now has this function. Usage:
import { Toast, deleteAllToasts } from "toaster-js";

const toast = new Toast("Hello!");

toast.delete(); /* OR */ deleteAllToasts();

Using deleteAllToasts() method, I can delete all previously created toastr.
Need to download again toaster-js.
Thank you for quick reply and solution @ZitRos

Thank you! Yes you need to download it again :)
I am also working on #4 right now, check this out. Maybe you'll find it useful.
So I am closing this issue as it is resolved. I would also appreciate if you star Toaster-JS repository here on GitHub, this will show me that I am doing something useful for people 😃 Thank you!

Yes, definitely


Dropwizard-migrations: Is there a way to pass property to changeset ?

I'm using below YAML file to store my configuration (as DataSourceFactory)
database:
  driverClass: com.mysql.jdbc.Driver
user: root
  password: ****
  url: jdbc:mysql://localhost:3306/test?zeroDateTimeBehavior=convertToNull&characterEncoding=UTF-8
  properties:
    hibernate.jdbc.batch_size: 50
    hibernate.order_inserts: true
    hibernate.order_updates: true
    hibernate.jdbc.batch_versioned_data: true

Later on, I'm creating a migration bundle with the above configuration.
How can I pass a property (table_name) to a changeset from the configuration file to use something like this
<databaseChangeLog xmlns="http://www.liquibase.org/xml/ns/dbchangelog"
                   xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                   xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog
				       http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-2.0.xsd">
    <changeSet author="test" id="test-1">
        <comment>test</comment>
        <sql>
            create or replace TABLE ${table_name} (
            KEY NUMBER(38, 0) NOT NULL autoincrement,
            PARAM_NAME VARCHAR(255),
            PARAM_VALUE VARCHAR(255),
            primary key (KEY)
            );
        </sql>
        <rollback>
            <dropTable tableName='${table_name}'/>
        </rollback>
    </changeSet>
</databaseChangeLog>

Note:  dropwizard version: 1.1.3

The dropwizard-migrations package internally calls Liquibase. Therefore I would suggest looking into their docs. Maybe this can help you.
If you want to pass properties dynamically from your Dropwizard application, I think there is currently no way for doing this. From Dropwizard 2.1.0 onwards (current master branch), there will be the possibility to pass objects/values to the Liquibase's Scope class, which then can be used in CustomChanges.

I have a yaml file to set changelog properties
databaseChangeLog:
  property:
      name: table_name
      value: tableA

Is there a way to pass this yaml file as a parameter to dropwizard migrations?

As far as I can see, this is not possible at the moment.
Liquibase provides two ways of setting the changelog: from file or as a DatabaseChangeLog object. Dropwizard only supports the first solution. Therefore the properties cannot be set to the changelog object.
The second possibility would be to call the getDatabaseChangeLog() method on the Liquibase object to get the parsed changelog file as an object, but hereby the dropwizard-migrations bundle doesn't provide access to the internally used ClosableLiquibase object.
So you basically have two options to work around this problem with the current version of Dropwizard:

Deserialize your yaml file with Jackson, set the properties and write it back to disk. Then you can use Liquibase's include mechanism
Override the DbMigrateCommand and DbCommand and register the new DbCommand manually. With this you'll get access to the ClosableLiquibase object and therefore to the DatabaseChangeLog object.

Thank you. Second option worked for me.


Some documents not getting transformed after failover on AWS

So we can address your issue, please include the following:
Version of MarkLogic Java Client API
See Readme.txt
Version of MarkLogic Server
9.0-nightly (08282018)
See admin gui on port 8001 or run xdmp:version() in Query Console - port 8000)
Java version
8
Run java -version
OS and version
For MAC, run sw_vers.
For Windows, run  systeminfo | findstr /B /C:"OS Name" /C:"OS Version"
For Linux, run cat /etc/os-release and uname -r
Input: Some code to illustrate the problem, preferably in a state that can be independently reproduced on our end
Actual output: What did you observe? What errors did you see? Can you attach the logs? (Java logs, MarkLogic logs)
An ApplyTransformListener was added to QueryBatcher and a forest failover was triggered during the run. I see that not all documents were transformed. I will add the logs to the case soon.
Expected output: What specifically did you expect to happen?
All document uris returned by  querybatcer has to be transformed
Alternatives: What else have you tried, actual/expected?

Can you revisit this issue and provide some more input?


Fix undefined var exception

This check to see if require exists was bad as it throws an error for an
undefined reference in case it doesn't exist.

@facebook-github-bot shipit

Thanks for fixing it. It was quite annoying when debugging in Chrome with "pause on caught exceptions" enabled :)


Latest Docker image fails while initializing gemma2

System Info
I tried the following systems, both with the same exception:

ghcr.io/huggingface/text-generation-inference:sha-6aebf44 locally with docker on nvidia rtx 3600
ghcr.io/huggingface/text-generation-inference:sha-6aebf44 on kubernetes cluster with nvidia a40

Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
docker run --gpus all --shm-size 1g -p 8080:80 ghcr.io/huggingface/text-generation-inference:sha-6aebf44 --model-id google/gemma-2-9b-it
2024-07-22T15:30:59.895904Z  INFO download: text_generation_launcher: Successfully downloaded weights for google/gemma-2-9b-it
2024-07-22T15:30:59.897225Z  INFO shard-manager: text_generation_launcher: Starting shard rank=0
2024-07-22T15:31:09.917300Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2024-07-22T15:31:10.682538Z ERROR text_generation_launcher: Error when initializing model
Traceback (most recent call last):
  File "/opt/conda/lib/python3.10/site-packages/transformers/models/auto/configuration_auto.py", line 951, in from_pretrained
    config_class = CONFIG_MAPPING[config_dict["model_type"]]
  File "/opt/conda/lib/python3.10/site-packages/transformers/models/auto/configuration_auto.py", line 653, in __getitem__
    raise KeyError(key)
KeyError: 'gemma2'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/conda/bin/text-generation-server", line 8, in <module>
    sys.exit(app())
  File "/opt/conda/lib/python3.10/site-packages/typer/main.py", line 311, in __call__
    return get_command(self)(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/typer/core.py", line 778, in main
    return _main(
  File "/opt/conda/lib/python3.10/site-packages/typer/core.py", line 216, in _main
    rv = self.invoke(ctx)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/typer/main.py", line 683, in wrapper
    return callback(**use_params)  # type: ignore
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/cli.py", line 118, in serve
    server.serve(
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 297, in serve
    asyncio.run(
  File "/opt/conda/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 636, in run_until_complete
    self.run_forever()
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 603, in run_forever
    self._run_once()
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 1909, in _run_once
    handle._run()
  File "/opt/conda/lib/python3.10/asyncio/events.py", line 80, in _run
    self._context.run(self._callback, *self._args)
> File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 231, in serve_inner
    model = get_model(
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/__init__.py", line 749, in get_model
    return FlashCausalLM(
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/flash_causal_lm.py", line 878, in __init__
    config = config_class.from_pretrained(
  File "/opt/conda/lib/python3.10/site-packages/transformers/models/auto/configuration_auto.py", line 953, in from_pretrained
    raise ValueError(
ValueError: The checkpoint you are trying to load has model type `gemma2` but Transformers does not recognize this architecture. This could be because of an issue with the checkpoint, or because your version of Transformers is out of date.
2024-07-22T15:31:11.520122Z ERROR shard-manager: text_generation_launcher: Shard complete standard error output:

2024-07-22 15:31:01.561 | INFO     | text_generation_server.utils.import_utils:<module>:75 - Detected system cuda
/opt/conda/lib/python3.10/site-packages/huggingface_hub/file_download.py:1132: FutureWarning: `resume_download` is deprecated and will be removed in version 1.0.0. Downloads always resume when possible. If you want to force a new download, use `force_download=True`.
  warnings.warn(
Traceback (most recent call last):

  File "/opt/conda/lib/python3.10/site-packages/transformers/models/auto/configuration_auto.py", line 951, in from_pretrained
    config_class = CONFIG_MAPPING[config_dict["model_type"]]

  File "/opt/conda/lib/python3.10/site-packages/transformers/models/auto/configuration_auto.py", line 653, in __getitem__
    raise KeyError(key)

KeyError: 'gemma2'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

  File "/opt/conda/bin/text-generation-server", line 8, in <module>
    sys.exit(app())

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/cli.py", line 118, in serve
    server.serve(

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 297, in serve
    asyncio.run(

  File "/opt/conda/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)

  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
    return future.result()

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 231, in serve_inner
    model = get_model(

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/__init__.py", line 749, in get_model
    return FlashCausalLM(

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/flash_causal_lm.py", line 878, in __init__
    config = config_class.from_pretrained(

  File "/opt/conda/lib/python3.10/site-packages/transformers/models/auto/configuration_auto.py", line 953, in from_pretrained
    raise ValueError(

ValueError: The checkpoint you are trying to load has model type `gemma2` but Transformers does not recognize this architecture. This could be because of an issue with the checkpoint, or because your version of Transformers is out of date.
 rank=0
Error: ShardCannotStart
2024-07-22T15:31:11.616687Z ERROR text_generation_launcher: Shard 0 failed to start
2024-07-22T15:31:11.616777Z  INFO text_generation_launcher: Shutting down shards

Using text-generation-inference:2.1.1, it works correctly, even though both share the same transformers version.
Expected behavior
initializing the model correctly

Thanks for reporting this 👍
There were some issues with gemma. I think this patch might address this as well.
Could you confirm if this is the case?

I now get a different error, both in the latest image and in 2.2.0.
2024-08-08T14:13:11.002710Z  INFO text_generation_launcher: Args {
    model_id: "google/gemma-2-9b-it",
    revision: None,
    validation_workers: 2,
    sharded: None,
    num_shard: None,
    quantize: None,
    speculate: None,
    dtype: None,
    trust_remote_code: false,
    max_concurrent_requests: 128,
    max_best_of: 2,
    max_stop_sequences: 4,
    max_top_n_tokens: 5,
    max_input_tokens: None,
    max_input_length: None,
    max_total_tokens: None,
    waiting_served_ratio: 0.3,
    max_batch_prefill_tokens: None,
    max_batch_total_tokens: None,
    max_waiting_tokens: 20,
    max_batch_size: None,
    cuda_graphs: None,
    hostname: "text-generation-inference2-755f5778bf-k9b86",
    port: 8080,
    shard_uds_path: "/tmp/text-generation-server",
    master_addr: "localhost",
    master_port: 29500,
    huggingface_hub_cache: Some(
        "/data",
    ),
    weights_cache_override: None,
    disable_custom_kernels: false,
    cuda_memory_fraction: 1.0,
    rope_scaling: None,
    rope_factor: None,
    json_output: false,
    otlp_endpoint: None,
    otlp_service_name: "text-generation-inference.router",
    cors_allow_origin: [],
    watermark_gamma: None,
    watermark_delta: None,
    ngrok: false,
    ngrok_authtoken: None,
    ngrok_edge: None,
    tokenizer_config_path: None,
    disable_grammar_support: false,
    env: false,
    max_client_batch_size: 4,
    lora_adapters: None,
    disable_usage_stats: false,
    disable_crash_reports: false,
}
2024-08-08T14:13:11.002898Z  INFO hf_hub: Token file not found "/root/.cache/huggingface/token"
2024-08-08T14:13:11.175329Z  INFO text_generation_launcher: Default `max_input_tokens` to 4095
2024-08-08T14:13:11.175401Z  INFO text_generation_launcher: Default `max_total_tokens` to 4096
2024-08-08T14:13:11.175410Z  INFO text_generation_launcher: Default `max_batch_prefill_tokens` to 4145
2024-08-08T14:13:11.175416Z  INFO text_generation_launcher: Using default cuda graphs [1, 2, 4, 8, 16, 32]
2024-08-08T14:13:11.175944Z  INFO download: text_generation_launcher: Starting check and download process for google/gemma-2-9b-it
2024-08-08T14:13:14.195643Z  INFO text_generation_launcher: Files are already present on the host. Skipping download.
2024-08-08T14:13:15.387056Z  INFO download: text_generation_launcher: Successfully downloaded weights for google/gemma-2-9b-it
2024-08-08T14:13:15.387857Z  INFO shard-manager: text_generation_launcher: Starting shard rank=0
2024-08-08T14:13:25.483545Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2024-08-08T14:13:35.578015Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2024-08-08T14:13:45.585828Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2024-08-08T14:13:55.588556Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2024-08-08T14:14:05.686872Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2024-08-08T14:14:15.778758Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2024-08-08T14:14:25.780629Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2024-08-08T14:14:35.785662Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2024-08-08T14:14:40.223288Z  INFO text_generation_launcher: Server started at unix:///tmp/text-generation-server-0
2024-08-08T14:14:40.277449Z  INFO shard-manager: text_generation_launcher: Shard ready in 84.886492494s rank=0
2024-08-08T14:14:40.283759Z  INFO text_generation_launcher: Starting Webserver
2024-08-08T14:14:40.320297Z  INFO text_generation_router: router/src/main.rs:228: Using the Hugging Face API
2024-08-08T14:14:40.320346Z  INFO hf_hub: /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/hf-hub-0.3.2/src/lib.rs:55: Token file not found "/root/.cache/huggingface/token"
2024-08-08T14:14:40.839350Z  INFO text_generation_router: router/src/main.rs:577: Serving revision 4efc01a1a58107f8c7f68027f5d8e475dfc34a6f of model google/gemma-2-9b-it
2024-08-08T14:14:41.433827Z  INFO text_generation_router: router/src/main.rs:357: Using config Some(Gemma2)
2024-08-08T14:14:41.433853Z  WARN text_generation_router: router/src/main.rs:384: Invalid hostname, defaulting to <IP_ADDRESS>
2024-08-08T14:14:41.439120Z  INFO text_generation_router::server: router/src/server.rs:1572: Warming up model
2024-08-08T14:14:42.767141Z  INFO text_generation_launcher: Cuda Graphs are enabled for sizes [32, 16, 8, 4, 2, 1]
2024-08-08T14:14:42.772236Z ERROR text_generation_launcher: Method Warmup encountered an error.
Traceback (most recent call last):
  File "/opt/conda/bin/text-generation-server", line 8, in <module>
    sys.exit(app())
  File "/opt/conda/lib/python3.10/site-packages/typer/main.py", line 311, in __call__
    return get_command(self)(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/typer/core.py", line 778, in main
    return _main(
  File "/opt/conda/lib/python3.10/site-packages/typer/core.py", line 216, in _main
    rv = self.invoke(ctx)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/typer/main.py", line 683, in wrapper
    return callback(**use_params)  # type: ignore
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/cli.py", line 118, in serve
    server.serve(
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 297, in serve
    asyncio.run(
  File "/opt/conda/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 636, in run_until_complete
    self.run_forever()
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 603, in run_forever
    self._run_once()
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 1909, in _run_once
    handle._run()
  File "/opt/conda/lib/python3.10/asyncio/events.py", line 80, in _run
    self._context.run(self._callback, *self._args)
  File "/opt/conda/lib/python3.10/site-packages/grpc_interceptor/server.py", line 165, in invoke_intercept_method
    return await self.intercept(
> File "/opt/conda/lib/python3.10/site-packages/text_generation_server/interceptor.py", line 21, in intercept
    return await response
  File "/opt/conda/lib/python3.10/site-packages/opentelemetry/instrumentation/grpc/_aio_server.py", line 120, in _unary_interceptor
    raise error
  File "/opt/conda/lib/python3.10/site-packages/opentelemetry/instrumentation/grpc/_aio_server.py", line 111, in _unary_interceptor
    return await behavior(request_or_iterator, context)
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 125, in Warmup
    max_supported_total_tokens = self.model.warmup(batch)
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/flash_causal_lm.py", line 1196, in warmup
    self.cuda_graph_warmup(bs, max_s, max_bt)
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/flash_causal_lm.py", line 1051, in cuda_graph_warmup
    self.model.forward(
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_gemma2_modeling.py", line 490, in forward
    hidden_states = self.model(
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_gemma2_modeling.py", line 427, in forward
    hidden_states, residual = layer(
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_gemma2_modeling.py", line 355, in forward
    attn_output = self.self_attn(
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_gemma2_modeling.py", line 254, in forward
    attn_output = paged_attention(
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/layers/attention/cuda.py", line 115, in paged_attention
    raise RuntimeError("Paged attention doesn't support softcapping")
RuntimeError: Paged attention doesn't support softcapping
2024-08-08T14:14:42.995889Z ERROR warmup{max_input_length=4095 max_prefill_tokens=4145 max_total_tokens=4096 max_batch_size=None}:warmup: text_generation_client: router/client/src/lib.rs:46: Server error: CANCELLED
Error: WebServer(Warmup(Generation("CANCELLED")))
2024-08-08T14:14:43.119606Z ERROR text_generation_launcher: Webserver Crashed
2024-08-08T14:14:43.119690Z  INFO text_generation_launcher: Shutting down shards

Ah it seem like this one doesn't have softcapping (https://github.com/huggingface/text-generation-inference/pull/2273).
I'd recommend using the latest TGI version.
Would that work for you?

Ah it seem like this one doesn't have softcapping (#2273). I'd recommend using the latest TGI version.
Would that work for you?

Can you take a look at #2763 ?


cleanup old GTM code

Package.resolved is removed because there are no longer dependencies.

@walterjgsp Merge on next release


Running your accessing_data_example notebook  error

I am trying to run your accessing_data_example.ipynb  and I am running it on the ml-notebook pangeo docker image.
I first install the missing packages:
!pip install carbonplan_data
!pip install watermark
!python -m pip install "carbonplan-styles"
!pip install git+https://github.com/carbonplan/cmip6-downscaling.git

and when I run the second cell
from cmip6_downscaling.analysis.analysis import (
    qaqc_checks,
    grab_big_city_data,
    load_big_cities,
    get_seasonal,
    change_ds,
)
from cmip6_downscaling.analysis.plot import plot_city_data
from cmip6_downscaling.analysis.analysis import change_ds

xr.set_options(keep_attrs=True)

I get:
ModuleNotFoundError: No module named 'cmip6_downscaling.analysis'
in fact, if I try tab complete that module does not appear. Am I doing something wrong?
thanks!

👋 @chiaral, thanks for reaching out about the data access example notebook! Could you try using %pip rather than !pip, which will install the packages into the currently running kernel? You may need to restart the kernel after using the pip magic command.

oh! that was a silly thing! the other packages loaded correctly so I didn't think about that. Thanks! it fixed the issue and things seem to be working fine!


feat(connectivity_plus): Add support of ARM arch

Description
Found out that some plugins have explicit exception to only allow x86_64 arches which didn't allow to run builds on ARM Macs natively.
Note, I currently don't have any ARM Mac to validate that this doesn't break anything, so would appreciate checking out if iOS builds correctly after this change.
Related Issues
#1467
Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
[x] No, this is not a breaking change.

Great! I can test later 👍🏽

If everything is correct you should see in Activity Monitor the following picture (note the Kind field):

I ran the app on the iOS simulator and as a macOS app, and there is no difference between main and this branch.
Both run natively on Apple silicon.
Maybe the behavior was changed and the issue is obsolete? But in any way, imho it makes sense to merge this and the other PRs, what do you think?

I was dealing with this topic on my job a few months ago trying to understand why our ARM build machines were running integrations tests via Rosetta. I have just revisited what I was doing there and it looks like I confused EXCLUDED_ARCHS property with VALID_ARCHS as in this case we have.
Did some more research and it looks like VALID_ARCHS is deprecated, so we can merge this and other related PRs anyway. I will adjust the title for every PR accordingly.


ci: fix modified gha-mergify-merge-queue-labels-copier Action

The if statement should end with fi.
I ran the script locally after fixing it,
the script added label in #3835
rakshith:ceph-csi$ export REPOSITORY_URL=https://github.com/ceph/ceph-csi
rakshith:ceph-csi$ export MERGE_QUEUE_PR_URL="https://github.com/ceph/ceph-
csi/pull/3835"
rakshith:ceph-csi$ gh pr view --json body -q ".body" $MERGE_QUEUE_PR_URL | sed -n -e '/```yaml/,/```/p' | sed -e '1d;$d' | yq '.pull_requests[]|.number' | while read pr_number ; do     gh pr view --json labels -q '.labels[]|.name' ${REPOSITORY_URL}/pull/$pr_number | while read label ; do       if [[ -z "$labels" ]] || [[ ",$labels," =~ ",$label," ]]; then         gh pr edit --add-label "$label" $MERGE_QUEUE_PR_URL;       fi;     done;   done
https://github.com/ceph/ceph-csi/pull/3835
https://github.com/ceph/ceph-csi/pull/3835
https://github.com/ceph/ceph-csi/pull/3835

https://github.com/ceph/ceph-csi/pull/3835#issuecomment-1549020177
The second commit was also verified from my local ci.
rakshith:ceph-csi$         gh pr view --json body -q ".body" $MERGE_QUEUE_PR_URL | sed -n -e '/```yaml/,/```/p' | sed -e '1d;$d' | yq '.pull_requests[]|.number' | while read pr_number ; do
          gh pr view --json labels -q '.labels[]|.name' ${REPOSITORY_URL}/pull/$pr_number | while read label ; do
            if [[ -z "$labels" ]] || [[ ",$labels," =~ ",$label," ]]; then
              gh pr edit --add-label "$label" $MERGE_QUEUE_PR_URL
            fi
          done
          gh pr edit --add-label "ok-to-test" $MERGE_QUEUE_PR_URL
        done
https://github.com/ceph/ceph-csi/pull/3835
https://github.com/ceph/ceph-csi/pull/3835
https://github.com/ceph/ceph-csi/pull/3835
https://github.com/ceph/ceph-csi/pull/3835

The ok-to-test label should get added by a rule in the Mergify configuration, not hardcoded in an action.

The full set of configuration does not run on draft merge queue pr for it to add ok-to-test
and the label copier may miss the ok-to-test label since we have an action that removes it after commenting.
@nixpanic , Let's get this in now and later replace it with the original action when it is functioning as expected ?

and the label copier may miss the ok-to-test label since we have an action that removes it after commenting.

The label copier is not expected to add ok-to-test, as it will not be set (anymore) on the original PR.
This Mergify rule is expected to add the ok-to-test label (but it is not working?):
https://github.com/ceph/ceph-csi/blob/c67517109809e549265b24c9c57c4bbb312cd424/.mergify.yml#L52-L65

and the label copier may miss the ok-to-test label since we have an action that removes it after commenting.

The label copier is not expected to add ok-to-test, as it will not be set (anymore) on the original PR.
This Mergify rule is expected to add the ok-to-test label (but it is not working?):
https://github.com/ceph/ceph-csi/blob/c67517109809e549265b24c9c57c4bbb312cd424/.mergify.yml#L52-L65

Yup, only the subset in the pr description is run.
https://github.com/ceph/ceph-csi/pull/3835#issuecomment-1549020177

Made some changes to the action instead of hard coding required labels.
Sent a patch pr to the upstream action repo too https://github.com/Mergifyio/gha-mergify-merge-queue-labels-copier/pull/2/files
cc @nixpanic

The official action now has all the fixes and new feature, we can use that now.
https://github.com/Mergifyio/gha-mergify-merge-queue-labels-copier/pulls?q=is%3Apr+is%3Aclosed


Make non-existent repos require login to clone

Previously, it was possible to check if a private
repository exists by attempting to clone the repo.
This change will make it so that non-existent repos
will also require a login and will only tell the user
whether the truly doesn't if they would know about a
private repository with that name.
This will only really be effective while Gitea is in prod mode. Otherwise, errors that reveal exactly why the user could not clone may be sent to the user.
EDIT: I'll see if I can add some tests.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7e74afc). Click here to learn what that means.
The diff coverage is 36.11%.

@@            Coverage Diff            @@
##             master    #8123   +/-   ##
=========================================
  Coverage          ?   41.73%
=========================================
  Files             ?      482
  Lines             ?    64554
  Branches          ?        0
=========================================
  Hits              ?    26940
  Misses            ?    34138
  Partials          ?     3476

Impacted Files
Coverage Δ

routers/repo/http.go
32.81% <36.11%> (ø)

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 7e74afc...f3e3c24. Read the comment docs.

Can this be implemented in routers.RegisterRoutes() with a filter function instead?

Can this be implemented in routers.RegisterRoutes() with a filter function instead?

I think I could turn it into a middleware function. Give me a bit.

@lafriks why this is a security related problem? I think the original implementation is security but not friendly for git.

You can check that such repository exists even if you don't know that

Perhaps I'm missing something, but I see a lot of auth logic in the PR. Shouldn't that logic exist already elsewhere?

@guillep2k Yes, it already existed. I moved the auth logic into a function so that it can be reused so those changes are showing up in the PR.

Looks like this issue has been resolved after #8419 was merged?


Added vue-element-admin-tpl and vue-m-message

Added vue-element-admin-tpl
Added vue-m-message plugin

OK, I'll update it later.


Constants used as array size [T; N] are reported as unresolvedReference

VSCode extension: 0.3.613 (Nightly)
rust-analyzer version: 6d473c0a8 2021-05-25 nightly

Duplicate of https://github.com/rust-analyzer/rust-analyzer/issues/7432


Fixed kernel width in utils.deconv2d

In the utils.deconv2d() function, the parameter k_w was ignored and the kernel height was used in its place:
W = tf.get_variable(
    name='W',
    shape=[k_h, k_h, n_output_ch, n_input_ch or x.get_shape()[-1]],
    initializer=tf.contrib.layers.xavier_initializer_conv2d())

This has been changed to:
W = tf.get_variable(
    name='W',
    shape=[k_h, k_w, n_output_ch, n_input_ch or x.get_shape()[-1]],
    initializer=tf.contrib.layers.xavier_initializer_conv2d())

Oops good catch!  TY!

Very welcome. :slightly_smiling_face:


Implement NoImplicitAnyManipulator

Fixes #27
Implements the basic topological sorting fix as described in my deep dive presentation.

First constructs a set of declarations that are implicitly any.
Constructs a dependency graph based on the values and expressions each variable is assigned to. If the variable is a parameter, also looks for all function calls and extracts the function arguments' types and expressions.
Topologically sorts the graph.
In topological order, assigns the types to each declaration. If a declaration has type any, console logs and skips the declaration, as well all declarations that rely on it.

I moved a few functions from StrictNullChecksManipulator to Manipulator as their functionality was shared between StrictNullChecksManipulator and NoImplicitAnyManipulator.

A few changes I made while addressing review comments!

I ended up reimplementing the toposort so that instead of erroring out for cyclical dependencies, it returns an order of declarations that minimizes dependency violations. I also changed the way we're calculating the dependency types so that even in the case of cyclical dependencies, it should be able to add the right types. For example, in the case we have something like:

let y,z,w;

y=z
z=w
w=y
y="string"

the tool would assign type string to all 3 variables: y, z, w.

Refactored and split the functionality into multiple chunks of functions. I plan on adding unit tests for each of these functions soon!
Added chalk for color coded error message displays.

Addressed all the comments and added all unit test cases!


”pip install mamba_ssm==1.0.1“and"pip install causal_conv1d==1.0.0"  Report an error

Hello, I would like to ask the following error occurs when installing "pip install mamba_ssm==1.0.1",  and I wonder how to solve it?
I was running on a linux server.
When I type the command line:

When I type the command line:

When I type the command line:

I wonder what the problem is? And how to fix it?
I look forward to hearing from you!

@wakakkakak
Hello, your CUDA driver needs to be upgraded to at least version 11.8 in order to use mamba_ssm.

@wakakkakak Hello, your CUDA driver needs to be upgraded to at least version 11.8 in order to use mamba_ssm.

Thank you for your replying! I


KAFKA-13957: Fix flaky shouldQuerySpecificActivePartitionStores test

Currently the tests fail because there is a missing predicate in the retrievableException which causes the test to fail, i.e. the current predicates
containsString("Cannot get state store source-table because the stream thread is PARTITIONS_ASSIGNED, not RUNNING"),
containsString("The state store, source-table, may have migrated to another instance"),
containsString("Cannot get state store source-table because the stream thread is STARTING, not RUNNING")

wasn't complete. Another one needed to be added, namely "The specified partition 1 for store source-table does not exist.". This is because its possible for
assertThat(getStore(kafkaStreams2, storeQueryParam2).get(key), is(nullValue()));

or
assertThat(getStore(kafkaStreams1, storeQueryParam2).get(key), is(nullValue()));

(depending on which branch) to be thrown, i.e. see
org.apache.kafka.streams.errors.InvalidStateStorePartitionException: The specified partition 1 for store source-table does not exist.

	at org.apache.kafka.streams.state.internals.WrappingStoreProvider.stores(WrappingStoreProvider.java:63)
	at org.apache.kafka.streams.state.internals.CompositeReadOnlyKeyValueStore.get(CompositeReadOnlyKeyValueStore.java:53)
	at org.apache.kafka.streams.integration.StoreQueryIntegrationTest.lambda$shouldQuerySpecificActivePartitionStores$5(StoreQueryIntegrationTest.java:223)
	at org.apache.kafka.streams.integration.StoreQueryIntegrationTest.retryUntil(StoreQueryIntegrationTest.java:579)
	at org.apache.kafka.streams.integration.StoreQueryIntegrationTest.shouldQuerySpecificActivePartitionStores(StoreQueryIntegrationTest.java:186)

This happens when the stream hasn't been initialized yet. I have run the test around 12k times using Intellij's JUnit testing framework without any flaky failures.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@cadonna
Indeed you are correct, the real cause of the flakiness was not having the extra "The specified partition 1 for store source-table does not exist." condition. I reverted back to using until() and I got 10k runs without any flaky errors.
The latest commit has retryUntil removed and the test is using until just like before. I did however keep some of the refactoring that I did along with your suggested formatting/naming changes, let me know if you also want to revert some f these changes.

@mdedetrich The builds are red due to a checkstyle issue. Could you please fix the issue?

@cadonna Apologies, missed the part regarding until, there was also some issues regarding rebasing. I have fixed the until method as you stated as well as the various formatting issues and I checked that locally it passes checkstyle.
PR description is also updated, there is no mention of retryUntil. Not sure if you also want me to update the description of the original commit?

The timeout in the ARM build seems to be unrelated since that build does also time out a lot on other PRs recently.


Using "/var/run/crio/crio.sock" as endpoint is deprecated.

Addressing deprecation warning
I0220 12:43:39.634359    2270 remote_runtime.go:43] Connecting to runtime service /var/run/crio/crio.sock
W0220 12:43:39.634394    2270 util_unix.go:75] Using "/var/run/crio/crio.sock" as endpoint is deprecated, please consider using full url format "unix:///var/run/crio/crio.sock".

- What I did
Attempted to start Kubernetes based on the ./hack/local-up-cluster.sh example in README.md and then investigated why I've got no kubectl get nodes.
- How I did it
PATH=/home/test/kubernetes/third_party/etcd:${PATH} CGROUP_DRIVER=systemd CONTAINER_RUNTIME=remote CONTAINER_RUNTIME_ENDPOINT='/var/run/crio/crio.sock  --runtime-request-timeout=15m' ./hack/local-up-cluster.sh

and then checked /tmp/kubelet.log.
- How to verify it
The above command with Kubernetes master (I was at v1.11.0-alpha.0-166-g5b98dbcfe5) should reproduce the issue.
- Description for the changelog

/test all
/test ami

LGTM, assuming happy tests.

/test e2e_fedora

/retest

/retest

/retest

/retest

@rhatdan, maybe the failures are real failures caused by this change?

Could be.  Will try one more time.  @mrunalp @runcom WDYT?

/test ami_fedora

/test ami_fedora

/test ami_fedora

Passes all tests.  Merging.


Add the boundaries to use the generic param in APDUTransport object

Fixes #15

This will break the build on wasm, though.

Yeah.. gonna move this to WIP..The wasm version does not have this generic, Wondering if we even need that.

I think either both or neither need to be generic.

@Demi-Marie do you need WASM support in this library?

@jleni No.


custom integration stops working in Home Assistant 2024.10.0

HA core version
2024.4.3
Intergration version
0.3.22
Device type and model
Window Air Conditioner AirCon_C5AA
Used App
Home Assistant and or
The description of problem
The custom integration midea_ac_lan implements the is_aux_heat property or uses the auxiliary heater methods in a subclass of ClimateEntity.
Please create a bug report at https://github.com/georgezhao2010/midea_ac_lan/issues.
Once an updated version of midea_ac_lan is available, install it and restart Home Assistant to fix this issue.
The logs
No response

If it help I just clone this repo and merge all pending PR fixing issues. This is only during the time the maintainer will merge all PR in this repository.
Waiting this time you can re-install the integration using this repository : https://github.com/CyrielRct/midea_ac_lan
To do that go to HACS - Integration - and add the repository. It will show you version 0.3.24

If it help I just clone this repo and merge all pending PR fixing issues. This is only during the time the maintainer will merge all PR in this repository.
Waiting this time you can re-install the integration using this repository : https://github.com/CyrielRct/midea_ac_lan
To do that go to HACS - Integration - and add the repository. It will show you version 0.3.24

Thank you for this. Honestly I would not be surprised if you are now the new crowned maintainer.

Thank you for your effort!

Anyone know why that new repo was archived?

Anyone know why that new repo was archived?

please use this repo to continue:
https://github.com/wuwentao/midea_ac_lan
also sync with CyrielRct and he has close his repo and we only use this repo to to the future works

有人知道为什么那个新的 repo 被存档了吗？

请使用此 repo 继续： https://github.com/wuwentao/midea_ac_lan
我会尽力维护它并继续更新它，也会与 CyrielRct 同步，他已经归档了他的 repo，我们只使用这个 repo 来进行未来的工作

可以支持2024.3.3的版本吗

The midea_ac_lan custom integration is using deprecated climate auxiliary heater
This stops working in version 2024.10.0. Please address before upgrading.
The custom integration midea_ac_lan implements the is_aux_heat property or uses the auxiliary heater methods in a subclass of ClimateEntity.

https://github.com/wuwentao/midea_ac_lan


[Feature Request] Support ES6 for CLI

CLI (generate at least) does not support stuff like 'export'

Create a new plugin, put some es6 code somewhere, export it with 'export' and import it in the command file with require.

That's a nodejs restriction.  We don't use babel or typescript on this project to compile.
Now that the standard is ratified, nodejs will have it in place soon.
In the meantime:
These libraries advertise a polyfill style drop in replacement:
https://github.com/benjamn/reify
https://github.com/standard-things/esm
Let us know if you have success with this!
I'm looking forward to import and export as well.

Perfectly clear, thanks


Center variable reporter; fix field centering to work for it

Resolves
--
Proposed Changes
Before:

After:

Reason for Changes
Variables named with "text with a small width" (e.g., a single character) were not aligned in the center of the reporter. By adding the CENTRE alignment (and fixing its implementation), they are.
Test Coverage
--

aaah emoji support

🎉 🐱 🌮
@carljbowman This looks good functionally. Do you want to give it a look before landing?

So should all single-field blocks have center align for their field?
Also gets back to the conversation we were having about alignment.


JetBrains: Accepting diff in Rider does nothing and diff UI never appears again

Before submitting your bug report

[x] I believe this is a bug. I'll try to join the Continue Discord for questions
[x] I'm not able to find an open issue that reports the same bug
[x] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Windows 10
- Continue: 0.0.18

Description
When I use /edit and confirm a diff, nothing happens. If I make another /edit call, the diff mode doesn't activate again, in fact it simply never does anymore.
To reproduce
Start by testing in Rider with these steps and let me know the results, If it doesn't reproduce I will try to investigate further and make a minimal repro.
Log output
No log or output or error anywhere

I'll test this later today and let you know, thanks for sharing

@oxysoft I tried with Rider for both mac and windows, version 0.18 for both, and unfortunately couldn't reproduce. This was with multiple consecutive edits and trying all combinations of keyboard shortcut/button to accept/reject either once done or during streaming. So that I could pin down a version of the server on Windows, I also tried with manual python -m continuedev version 0.1.95. When you go for the reproduce, could you try with this version just to be sure?


"source activate" causes error in pyenv-which-ext that exits shell

I had a little trouble getting virtualenvwrapper to work properly with conda, but it doesn't sound like that's really a supported configuration by conda, So I tried to use conda envs directly, when this happened.
First setup pyenv version and find activate/conda
vagrant@ubuntu-14:~$ PYENV_DEBUG=1
vagrant@ubuntu-14:~$ which activate
/home/vagrant/.pyenv/shims/activate
vagrant@ubuntu-14:~$ less /home/vagrant/.pyenv/shims/activate
vagrant@ubuntu-14:~$ pyenv shell miniconda-3.9.1
vagrant@ubuntu-14:~$ which activate
/home/vagrant/.pyenv/shims/activate
vagrant@ubuntu-14:~$ pyenv which activate
/home/vagrant/.pyenv/versions/miniconda-3.9.1/bin/activate
vagrant@ubuntu-14:~$ which conda
/home/vagrant/.pyenv/shims/conda
vagrant@ubuntu-14:~$ pyenv which conda
/home/vagrant/.pyenv/versions/miniconda-3.9.1/bin/conda

So at this point, it looks like I'm correctly using miniconda-3.9.1 and pyenv correctly appears to want to use the activate definition from miniconda-3.9.1.
Next I'll verify that the conda environment is there
vagrant@ubuntu-14:~$ conda info --envs
# conda environments:
#
manda                    /home/vagrant/.pyenv/versions/miniconda-3.9.1/envs/manda
root                  *  /home/vagrant/.pyenv/versions/miniconda-3.9.1

I've confirmed that conda thinks I have a conda env named "manda"
Now I'll try to activate my environment (http://conda.pydata.org/docs/using/envs.html), but some interactions between conda and the lookup_from_path.bash script causes me to get kicked out of my shell:
vagrant@ubuntu-14:~$ source activate manda
++ program=-bash
++ '[' -bash = python ']'
++ export PYENV_ROOT=/home/vagrant/.pyenv
++ PYENV_ROOT=/home/vagrant/.pyenv
++ exec /home/vagrant/.pyenv/libexec/pyenv exec -bash manda
/home/vagrant/.pyenv/plugins/pyenv-which-ext/etc/pyenv.d/which/lookup_from_path.bash: line 38: command: -b: invalid option
command: usage: command [-pVv] command [arg ...]
pyenv: -bash: command not found
Connection to <IP_ADDRESS> closed.

Any ideas?

This is the relevant part of my ~/.profile
if [ -f ~/.bashrc ]; then
   source ~/.bashrc
fi

export PYENV_ROOT="$HOME/.pyenv"
export PATH="$PYENV_ROOT/bin:$PATH"
eval "$(pyenv init -)"
eval "$(pyenv virtualenv-init -)"

export WORKON_HOME=${HOME}/.virtualenvs
export PROJECT_HOME=/cygwin/code
# Disable virtualenvwrapper for now
#pyenv virtualenvwrapper

Some relevant environment variables after I set my pyenv version:
vagrant@ubuntu-14:~$ env | egrep 'PYENV|PATH'
PYENV_ROOT=/home/vagrant/.pyenv
PYENV_VERSION=miniconda-3.9.1
PYENV_VIRTUALENV_INIT=1
PATH=/home/vagrant/.pyenv/shims:/home/vagrant/.pyenv/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games
PYENV_SHELL=bash

Should I move this issue from pyenv to pyenv-virtualenv?

@jimmywan Looks like configuration problem with pyenv-virtualenv + anaconda/miniconda. Please move issue to pyenv-virtualenv.

Moved to pyenv-virtualenv:
https://github.com/yyuu/pyenv-virtualenv/issues/88

If anyone ends up here after Googling (like I did), I just ran into the same error, but not related to pyenv-virtualenv in any way to my knowledge:
/home/raspi/.pyenv/libexec/pyenv-which: line 43: command: -b: invalid option
command: usage: command [-pVv] command [arg ...]
pyenv: -bash: command not found
Connection to raspi.local closed.

Every time I sourced my .bashrc (with my pyenv init stuff), I'd get kicked off the SSH connection, as seems to be the case with the OP here. The really odd thing was I hadn't changed my .bashrc setup.
I'm not exactly sure what caused the error, but I figured I'd try a pyenv rehash, and I got another error:
$ pyenv rehash
pyenv: cannot rehash: /home/raspi/.pyenv/shims/.pyenv-shim exists

After inspecting and backing up that file, I tried deleting it: rm /home/raspi/.pyenv/shims/.pyenv-shim and afterwards both errors were fixed. Not sure why.

Hmm. Probably some broken shims caused it. I'm not sure why, though. Removing everything in ~/.pyenv/shims and then 'pyenv rehash' could be a workaround.

Damn, I had a similar issue and I was just missing:
pyenv virtualenvwrapper
in my ~./bash_profile


Canceling restangular call

Hey man,
I have a situation where i need to cancel a restangular call. I have a server side search going on. On every key press i make a call. Sometimes previous key search responses come in after the latest search response and the old data overwrites the new data on the screen.
I found this snippet below. But i am not sure if i can pass the timeout object through restangular to $http service.
If thats even the best way, basically just wondering how you would handle this problem with your stuff?
Thanks for any help!
the link to this is http://stackoverflow.com/questions/13928057/how-do-to-cancel-an-http-request-in-angularjs
var canceler = $q.defer();
$http.get('/someUrl', {timeout: canceler.promise}).success(successCallback);
// later...
canceler.resolve();  // Aborts the $http request if it isn't finished.

@jaydiablo thanks for .withHttpConfig method. But if I need to timeout my restangular post request after certain milliseconds e.g. 500ms how to configure that? is it possible?
Thanks in advance.


fix: dot match

Codecov Report

Merging #49 (7ff7b7b) into master (d5c0b37) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #49   +/-   ##
=======================================
  Coverage   89.74%   89.74%
=======================================
  Files          42       42
  Lines        2749     2749
  Branches      681      681
=======================================
  Hits         2467     2467
  Misses        258      258
  Partials       24       24

Impacted Files
Coverage Δ

packages/gateway-common-http/src/common.ts
92.95% <100.00%> (ø)

Continue to review full report at Codecov.

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


Missing: Parts of Application Recovery and Restart

Parts of the Application Recovery and Restart are missing.
Specifically, this functions are missing:

GetApplicationRestartSettings
RegisterApplicationRestart
UnregisterApplicationRestart

All listed API calls are already part of winapi-rs (see winbase.rs):

GetApplicationRestartSettings
RegisterApplicationRestart
UnregisterApplicationRestart


Can not load ns when ns with meta

When ns with meta like
(ns ^{:doc "foo"
         :author<EMAIL_ADDRESS>    foo.bar)

Can not load namespace correctly, open-file-containing-var Autocompletion are both disable.
Remove the meta
(ns foo.bar)

Everything is OK.

Thanks for filing the issue. This is a duplicate of #104. I think @qwtel was working on a fix here, https://github.com/jasongilman/proto-repl/pull/151, but he closed it. I'll take a look if he doesn't resolve it.

I don't think those two are related. Also, sending the above command to the repl works for me.

I met this problem when working at a clj file whose ns has meta.
When this file's ns has meta, I can't use open-file-containing-var and Autocompletion at this file. Other files are OK. Remove meta, everything is OK.

Yeah this is definitely a different issue. I'm not sure what I was thinking. I think I know how to improve the parsing as well.

There's a way to make it work that's relatively easy - we can use EditorUtils to get all top-level blocks, find the one that it's a ns declaration, then use parseEdn - this will remove the meta.
I can work around on this if this is the right way of doing, WDYT?

That's a great idea. This has been a long term annoying problem and it would be great to get it resolved.
Sent from my phone

On Feb 7, 2017, at 2:37 PM, Maurício Szabo<EMAIL_ADDRESS>wrote:
There's a way to make it work that's relatively easy - we can use EditorUtils to get all top-level blocks, find the one that it's a ns declaration, then use parseEdn - this will remove the meta.
I can work around on this if this is the right way of doing, WDYT?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


Some improvements to the SceneTreeTimer

Replaced #55990 to be compatible with the latest master branch.
Implemented https://github.com/godotengine/godot-proposals/issues/4319
This PR brings some improvements to the SceneTreeTimer:

Added a new parameter process_in_physics to SceneTree.create_timer() to update the SceneTreeTimer during the physics frame instead of the process frame.

Added a new parameter ignore_time_scale to SceneTree.create_timer() to ignore Engine.time_scale and update the SceneTreeTimer with the actual frame delta.

Code cleanup, moved the code that handles SceneTreeTimer to process_timers()

This should not break any compatibility, so maybe it can be cherrypick to 3.x?

Manually instantiating timer nodes and adding to the scene tree is low
performance and painful for situations where many lines of code for a large
number of objects need to used it to control execution delays (like bullet
hell games)
The related issues and use cases have already been discussed in the
original PR
https://github.com/godotengine/godot/pull/55990
and proposal
https://github.com/godotengine/godot-proposals/issues/4319
, and I think there's enough reason to add these features to the scene timer
Rémi Verschelde @.***>于2022年7月15日 周五10:25写道：

I have no strong stake in this but this is starting to feel like a lot of
options just to avoid actually instantiating a Timer manually,
configuring it as wanted, and adding it to the SceneTree.
—
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/63026#issuecomment-1185304919,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKP33Q5XNVA2EPZ35ZYEW2LVUEN7XANCNFSM53UQZA5A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Rebased to fix CI issue

I have no strong stake in this but this is starting to feel like a lot of options just to avoid actually instantiating a Timer manually, configuring it as wanted, and adding it to the SceneTree.

Manually instantiating timer nodes and adding to the scene tree is low
performance and painful for situations where many lines of code for a large
number of objects need to used it to control execution delays (like bullet
hell games).
I had to manually implement a global singleton to create a series of refcounted objects and loop over them in _physics_frame() for timing, Just because the scene tree timer cannot be configured to run at the physical frame, and running at regular frames would kill my player action replay system which need 100% reproducible.
The related issues and use cases have already been discussed in the
original PR
https://github.com/godotengine/godot/pull/55990
and proposal
https://github.com/godotengine/godot-proposals/issues/4319
, and I think there's enough reason to add these features to the scene  tree timer

The same can be achieved with
get_tree().create_tween().set_process_mode(Tween.TWEEN_PROCESS_PHYSICS).tween_interval(time).finished

although using SceneTreeTimer is shorter I guess.

Needs a rebase.

Needs a rebase.

Rebased

Thanks!


Resource fisher stops working after about an hour or less.

Industrial Foregoing Version: 1.7.0-114
TeslaCoreLib Version: 1.10.12
Crashlog (If Any):
I'm only feeding about 30 rf/t to it.  But after 30 mins to an hour it continues to use RF, but stops gathering anything.
breaking and replacing it didn't seem to fix it, but breaking and replacing in a different chunk did seem to fix it.
Also relogging or travelling to another dimension fixes it.

You will have to let me know if it's really fixed in the version 1.8.0 that will come out soon as it is a wierd thing to test.


Deployments have labels that are redundant when existing secrets are provided.

When providing an existing Auth or DataStore Key secret it generates a checksum of a file that is never needed.
Sense the file is not used, the checksum becomes a static value.
Additionally, this checksum is calculated multiple times.
This results in larger output templates and slower helm chart builds.
My suggestion is to remove them.

I believe #391 and #392 implement this. If anything else is required, we can reopen or file a new issue.


balanced_market should charge balanced even if price < threshold

In balanced_market once the price drop under the set threshold, vehicles should charge over the min_soc (so maximum energy [kWh]), but not by charging "greedy". The drawn power should be balanced over the length of the timestep, since the horizon allows to know how long this cheap price (under threshold) lasts.
Also the stationary battery currently charges until its full, once price<threshold. But again, the stat. battery should not charge "greedy", but "balanced".
The charging should follow the same logic as when price not below threshold, it should only charge more (over desired soc). Like the green in the picture.

Same for v2g: charging and discharging balanced


Generators, create resources without templates

Most of the time resources are identical, only few fields are modified.
Kpm can help to generate them without having to add any template in the package it self.
app_ports:: [8080, 6000]
resources: [
 {
    file: my-webapp-svc.yaml
    type: svc
    generator: {
        ports: $.app_ports
        type: nodePort
     }
  },
 {
    file: my-webapp-deploy.yaml
    type: deployment
    generator: {
       pod: [
            {image: container1, ports: $.app_ports},
            {image: sidecar1}
            ]
 },
 {
  file : my-webapp-ingress.yaml
  type: ingress,
  generator:
    { domains: [app.kubespray.com]
      paths: {path: "/", port: 8080}
}
]

https://github.com/heptio/kube.libsonnet/


chore: build with typescript instead of babel

prereq for getting it ready for automated docs.

Current dependencies on/for this PR:

master

PR #813   👈

PR #814

This comment was auto-generated by Graphite.


This project can not write content into execl ?

This project can not write content into execl ?

No, it cannot. This is a streaming reader only, if you need to write to a file you can use vanilla Apache POI.


Bugfix/nearest road

Implements a bugfix in which the NxMap.nearest_road was in fact not returning the nearest road. To fix, I've added a buffer around the point when querying the rtree for nearby roads and then explicitly computing the distance between each road and the coordinate and returning the minimum distance.
This also removes the computation of both shortest distance paths and shortest time paths for the LCSS matcher as it was found that they are the same in almost all cases.

Nice, this looks good. Thanks for fixing this!


gocryptfs: 1.8.0 -> 2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/rfjakob/gocryptfs/releases.
meta.description for gocryptfs is: "Encrypted overlay filesystem written in Go"
meta.homepage for gocryptfs is: "https://nuetzlich.net/gocryptfs/"
meta.changelog for gocryptfs is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/8an4wkyscg9d1l1kb2j9cm6hchjjvria-gocryptfs-2.0/bin/gocryptfs-xray passed the binary check.

/nix/store/8an4wkyscg9d1l1kb2j9cm6hchjjvria-gocryptfs-2.0/bin/gocryptfs passed the binary check.

Warning: no invocation of /nix/store/8an4wkyscg9d1l1kb2j9cm6hchjjvria-gocryptfs-2.0/bin/statfs had a zero exit code or showed the expected version

2 of 3 passed binary check by having a zero exit code.

2 of 3 passed binary check by having the new version present in output.

found 2.0 with grep in /nix/store/8an4wkyscg9d1l1kb2j9cm6hchjjvria-gocryptfs-2.0

found 2.0 in filename of file in /nix/store/8an4wkyscg9d1l1kb2j9cm6hchjjvria-gocryptfs-2.0

directory tree listing: https://gist.github.com/bd84a154dcd7dfe5c3528182808a9318

du listing: https://gist.github.com/8d71b6da3572192821a8fc5fb5c317e1

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
gocryptfs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8an4wkyscg9d1l1kb2j9cm6hchjjvria-gocryptfs-2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gocryptfs https://github.com/r-ryantm/nixpkgs/archive/efa717536195da62c171edae51e40939b8aa5438.tar.gz

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

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

gocryptfs

Maintainer pings
cc @flokli @offlinehacker @prusnak @kalbasit for testing.

Superseded by https://github.com/NixOS/nixpkgs/pull/125921 which contains more changes and fix for aarch64-darwin


Changing character talent drop downs cause page to freeze

Only on the talent page, not on team-mate talent drop downs.
Seems to be isolated to my database as far as I can tell - but can reproduce it in Chrome and Edge using my database.
Steps:
Go to Raiden
Change her resolve stacks to something else
Also failing on Yoimiya changing her A1 passive dropdown
Interestingly doesnt happen for Yelan's A4
Stain_2023-02-03_09-25-32.txt

Inspecting the page after its frozen:

I'm having trouble reproducing this issue. It looks like the dropdown presentation overlay is stuck even after selection an option, disallowing you to interact with the elements under the overlay.

After deleting Raiden from my database and recreating her it's increasingly difficult to recreate this issue on my end now.
It seems to sometimes work, sometimes not. Once it fails it will fail every time until some other condition is changed (artifact swap, weapon passive toggle, talent level, etc) and then it will begin to work again until the next failure.
Can't find a way to reliably reproduce it anymore.

I was also experiencing this when I was trying to fix the initial dropdown bug. Every instance of MenuItem was completely borked and having this issue, which explains why I thought the separation of tooltip from the dropdown wasn't working as a fix

@nguyentvan7 can you still produce it? I've not been able to reproduce even once

Nope and when i was able to reproduce it, I wasnt able to fix it with code changes either

Possibly another occurrence of the bug

Possibly https://github.com/mui/material-ui/issues/32286 ?

I think I've figured out what was causing this and why you cant reproduce it with my database.
My Raiden and Yoimiya (but not Yelan) had a TC Tab build set. TC Tab builds arent saved in the json, but still in the local storage.
And even if you "delete" the database before uploading a new database, the TC Tab retains whatever TC build used to be there.
Once I cleared the TC build the page no longer freezes. (Obviously deleting Raiden and re-creating her also cleared the TC Tab)

what was in the TC tab locally that was causing the freeze? I've tried a few things with TC builds and I still can't replicate

I just copied from equipped and then made some random changes. different weapon, different main stat, different set for example

can you give as exact reproduction step as possible? I am unable to replicate

i think im barking up the wrong tree again. because  when trying to make a step by step recreation of it now it's still inconsistent for me.
What I have found is that it is the same issue as the artifact assume main stat level drop down bug.
The dropdown is still there, hidden, but selectable which clears the frozen page fault.

However now I'm thinking it's occuring due to an accidental rapid double click when selecting something from the dropdown.

Oh, I can reproduce by double clicking the menu element. your mouse probably has a double click issue: https://codepen.io/blink172/pen/vERyxK

Maybe, but I'd expect to see some miss clicks here if that was the case.
My assumption is its linked to the same issue in 2nd paragraph of #907
Maybe that should have been its own issue - as that bug still isnt fixed

Second part of #907 will be fixed in #853

My bet is it's still the race condition in the other post I linked, and certain things can aggravate it to appear more often, like tooltip or double clicks or long render cycles

Yep that matches up to what I'm seeing. The "laggier" the UI is due to more characters/conditionals/multi-targets/etc that need to be recaluclated when changing the drop down, the wider the window is for the dropdown to be inturrupted by an double click.
So this is probably more prominent on less powerful machines and could explain why it's harder to reproduce for some

So this is probably more prominent on less powerful machines

Can confirm this, at least. Hyperbloom stuff I was testing: Overview and team buffs tab, Key dropdown froze the page. Talents tab, Yelan's a4.
While the game is running the drop-downs broke 100%. Close the game to free resources and they work fine.

Well seems like we either wait for this race condition fix, or convert everything to autocomplete, which avoids this issue by not creating a full screen overlay, and not fixing the screen completely when the dropdown is open


Make -j 0 run unlimited parallel builds

Just ended up doing it myself in 5mins over a random lunch time in the end.
closes #1309
@nico @jhasse please review.
Would it be possible to get a release once this is merged also?
There hasn't been one in a long time and it is nearing 100 commits now.

The release is tracked in #1477. I think this is likely to miss the tag however.

Thanks for the PR!
Can you add , fix #1309 at the end of your commit message so that the issue is references and automatically closed?

Nevermind ;)


[Bug] Flinksql job execute error in remote model

Search before asking

[X] I had searched in the issues and found no similar issues.

Java Version
1.8+
Scala Version
2.12.x
StreamPark Version
2.0
Flink Version
1.15.3
deploy mode
remote
What happened
flink cluster set as remote model. run flinksql job throw exception like below, even run the flinksql demo job which does not has any maven and jar dependency also throw the same error.

error message:
Caused by: java.lang.LinkageError: loader constraint violation: when resolving method "org.apache.hc.client5.http.fluent.Request.body(Lorg/apache/hc/core5/http/HttpEntity;)Lorg/apache/hc/client5/http/fluent/Request;" the class loader (instance of org/apache/streampark/flink/proxy/ChildFirstClassLoader) of the current class, org/apache/streampark/flink/client/tool/FlinkSessionSubmitHelper$, and the class loader (instance of sun/misc/Launcher$AppClassLoader) for the method's defining class, org/apache/hc/client5/http/fluent/Request, have different Class objects for the type org/apache/hc/core5/http/HttpEntity used in the signature
Error Exception
error message:
Caused by: java.lang.LinkageError: loader constraint violation: when resolving method "org.apache.hc.client5.http.fluent.Request.body(Lorg/apache/hc/core5/http/HttpEntity;)Lorg/apache/hc/client5/http/fluent/Request;" the class loader (instance of org/apache/streampark/flink/proxy/ChildFirstClassLoader) of the current class, org/apache/streampark/flink/client/tool/FlinkSessionSubmitHelper$, and the class loader (instance of sun/misc/Launcher$AppClassLoader) for the method's defining class, org/apache/hc/client5/http/fluent/Request, have different Class objects for the type org/apache/hc/core5/http/HttpEntity used in the signature

Screenshots
my job configuration：

demo job configuration：

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!(您是否要贡献这个PR?)

Code of Conduct

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

Make sure the version of flink is consistent with the version of remote flink cluster.

Make sure the version of flink is consistent with the version of remote flink cluster.

i am sure.  pls see below configuration setting.  flink home folder is 1.15.3 as same as remote flink cluster.

fixed in new version(2.1.2)

2.1.4 same bug


[EN] Feedback for: /web/tools/chrome-devtools/?hl=en

The new Dev tools now has a menu I have to click into to set how the dev tools window is laid out, whereas before I could switch between the last two layouts with just one click. The icon took up hardly any room, but now I have to click again. great. thanks. I guess you're trying to be like the mobile Chrome app, which also puts everything inside a menu. I just don't understand why these design mistakes are made from such a smart company like Google.

Thanks for the feedback. However, issues with the DevTools itself should go to the chromium issue tracker. This is for documentation-specific issues.
This change was made since finding the alternate layout modes was hard to discover for users. The vast majority of users didn't know multiple modes existed. In order to make all the modes more discoverable as well as have room for more tooling to be made more accessible, the overflow menu was created. While some users, such as yourself, see this as a negative since you need to do more work it does create a better experience for the majority of users.


rhn server registration

Is there a way to specify an internal rhn server to register the servers against?

Hi,
I just updated the code to support rh_subscription_server and rh_unverified_ssl parameters.
The rh_subscription_server default to subscription.rhsm.redhat.com.
I don't have an internal RHN server so please let me know if it works or if you need extra parameters.
Regards,
Simon.

So I tested this, it does not work with subscription-manager. I think I need to use the rhn_register ansible module. Still plugging away.

Hi,
I just deployed a RHEL satellite 6.5 to test against and it is working fine.
This is what I specified as parameter:
rh_subscription_server = "my_satellite_fqdn/rhsm"
rh_unverified_ssl = "true"

You also have to enable the extra repository on your satellite.

That will work


[requires.io] dependency update on master branch

This change is

Current coverage is 97.23% (diff: 100%)

Merging #63 into master will not change coverage

@@             master        #63   diff @@
==========================================
  Files            13         13
  Lines           796        796
  Methods           0          0
  Messages          0          0
  Branches         42         42
==========================================
  Hits            774        774
  Misses           17         17
  Partials          5          5

Powered by Codecov. Last update 73b3534...abd2cec

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Fix [GithubPullRequestCheckState] test

This is an amusing issue. abc123 recently became a real ref on the repository (https://github.com/badges/shields/commit/abc123de1d2d3a83b256ccca5e17e6a4b6088097), which started making this test fail. abcde12356 should be safe for a little while longer. 😄

apparently trying to fake a hex isn't that trivial 😀


CI broken because setup-python doesn't provide pythons 3.8 and 3.9 on MacOS 14

https://github.com/actions/setup-python/issues/850
https://github.com/actions/setup-python/issues/852
https://github.com/actions/setup-python/issues/696#issuecomment-2072959905

It's fixed


8VLE

Key
Value

View
https://database.shabados.com/sources/3/page/41/line/157

Source
Vaaran Bhai Gurdas Ji

Page
41

ID
8VLE

Line
qb phuMcY hir hukm isauN; inj dsvYN duAwrw [

I've seen many of these with the bindi, I believe these need more research. This is supposed to be a tippi: ਪਹੁੰਚੈ


GOGGLE doesn't exist

Description
Installed synthcity with  pip install "synthcity[all]" when I run  Plugin().get("goggle")   I get

Plugin goggle doesn't exist.

The GOGGLE plugin is also not in the output of Plugins().list()
System Information

OS: Ubuntu
OS Version: 22.04
Language Version: Python 3.10.12

Hi @Aduboahen, Hmm, that is interesting. The tests passed for google on the main branch yesterday and at the last release. What happens if you pip install synthcity[goggle]?

Installs without error (all requirements satisfied)

synthcity[goggle]) (0.16.0)

Ah, that's it. The error is due to your version of synthcity. Goggle was added in v0.2.5, so v0.16.0 does not contain the plugin. If you want to use goggle, you will need to update your version of synthcity to v0.2.5 or later. This can be done with something like:
pip install --upgrade synthcity[all]
I will close this issue now, but please reply if your issue is not resolved and I will reopen it.

synthcity is up to date v0.2.10

Can I cask what does the following command return?
pip freeze | grep -e dgl -e torch-geometric -e torch-sparse -e torch-scatter

dgl==2.1.0
torch-scatter==2.1.2
torch-sparse==0.6.18

no result for torch-geometric? If not, try pip install torch-geometric and then re-try

pip install torch-geometric  finishes with all Requirements satisfied but pip freeze | grep torch-geometric returns empty

That is odd that it isn't returned by the pip freeze. Could you double check it is installed with: python -c "import torch_geometric; print(torch_geometric.__version__)"

Yeah it's installed, Returns '2.5.0'

OK, think I have found the potential issue. Can you try this:
pip uninstall dgl
pip install  dgl==2.0.0 -f https://data.dgl.ai/wheels/repo.html
Then try again?
I think it could be an issue with dgl support for pytorch 1.13.1.


refactor(features): separate data from the template

What:

Separate data from the template.

Issue number: N/A

@Jeykairis thank you!


The link to the basics tutorial isn't working

This link https://matchcake.github.io/MatchCake/tutorials/matchcake_basics.ipynb which is located at https://matchcake.github.io/MatchCake/readme.html in the "tutorials" section leads to 404.

The other tutorial link (iris dataset classification) yields the same error.

Thank you to open this issue. It seems like the links work from the readme.md but not from the online documentation. I created a PR and hope it will fix this issue.