Add cram tags to paper_items & move milk to fridge

@NeumannM89 could you check this PR as well? It adds the CramObject tags for the paper_items as well. The previous PR did only add the tags to the milestonge items.


Publish on maven central

http://central.sonatype.org/pages/ossrh-guide.html

It is not a priority


Add dedicated Work Item Calls.

I think we should introduce dedicated Work Item Calls for certain active work items. Let's discuss.

I agree with the proposed work items:
DID Registration Work Item Call - Wednesdays 1000 ET - to work on https://github.com/decentralized-identity/did-registration/
Universal Resolver Work Item Call - Wednesdays 1300 ET - to work on https://github.com/decentralized-identity/universal-resolver
and with the proposed time/date.

Merging after several discussions about this without any objection.


Add public constructors for types returned by BigQueryClient

This allows tests to provide fake data. Only BigQueryClient itself
would be faked out in such a context: everything else delegates to
BigQueryClient for API calls, but fake data can be provided with the
production classes.
Fixes #2170.

(While I'm generally reluctant to make things public, I think this bit of the API is stable now.)


Covert Values from MonitorCharacteristicsForService

I guys, I'm using your library to connect to a BLE device with Android Device.
I have a question for you, when I use monitorCharacteristicForService for services and characteristics I receive data in this format, but i need to have numbers (like battery number):
Accelerometer + Gyroscope: "zLAi/wD/SpwiAOv/VwBnAML/xQE="

Pressure: "bgBvAHAAcQByAA="

Battery: "ChYKARoA"

Do you know what kind of data are these? And How can i convert to int?
Thank you

Please check project Wiki


Allow class-string argument in AsLiveComponent static methods

Q
A

Bug fix?
no

New feature?
??

License
MIT

As done in AsTwigComponent attribute, this PR allow to extract metadata about LiveComponent using their class name.
Test failures unrelated

Thanks Simon!


Updated osc-dns-config to support creating PTR records

@etsauer please review. If you're OK with the changes, please merge and delete this branch.

@oybed
[root@osc-dns-check provisioning]# ./osc-dns-config -m="<IP_ADDRESS>|<IP_ADDRESS>" -n="<IP_ADDRESS>|<IP_ADDRESS>" -b="ose.example.com"Package 32:bind-9.9.4-18.el7_1.1.x86_64 already installed and latest version
Package 32:bind-utils-9.9.4-18.el7_1.1.x86_64 already installed and latest version
Nothing to do
named.conf                                                                                100% 3074     3.0KB/s   00:00
private-OSE_DNS_DOMAIN.db                                                                                100%  518     0.5KB/s   00:00
public-OSE_DNS_DOMAIN.db                                                                                100%  518     0.5KB/s   00:00
ipcalc: bad prefix:
cp: cannot stat ‘/var/named/static/INV_IP_ADDR.in-addr.arpa.db’: No such file or directory
sed: can't read /var/named/static/16.172.in-addr.arpa.db: No such file or directory
sed: can't read /var/named/static/16.172.in-addr.arpa.db: No such file or directory
Job for named.service failed. See 'systemctl status named.service' and 'journalctl -xn' for details.
Failed to reload DNS server on <IP_ADDRESS>

Looks good. Merging.


shell script action not working

My previously installed shell script actions do not seem to be working with the latest ControlPlane update (1.1.4 on Lion 10.7.2). The shell script can be invoked manually from the command line and there are no error messages in the ControlPlane message window or in the OSX Console.
These shell scripts have been working with Marco Polo and ControlPlane for over a year, so I believe this is an issue with the latest update. Restarting ControlPlane has no effect. Forcing the context to stick and manually switching contexts does not invoke the script (as evidenced by the lack of log file output from the script).

Do the shell scripts include #!/bin/bash at the top or similar (if you're using something other than bash or even sh)

Yes, I should have mentioned that. I did try it without the argument and it did not invoke the script.

Arguments do break it

Testing my solution right now (why is my computer so slow today :-(  )

This issue does not seem to be fixed with 1.1.5. Can I provide any other information to help diagnose the problem?

I now get "Failed to execute" in the Control Plane Advanced console output. This did not happen before.

OK, I removed the bash shebang and it seems to work, even though the script works fine from the command line with the shebang.

Weird, 'cause it works for me. Could you tell us what the exact shebang was? (including spaces and all that)

I mean to reply as well, this works fine for me as well.

You are correct to suspect the spacing. The original shebang (which does not work) was:
#! /bin/bash
I have confirmed that it works with no shebang, or with:
#!/bin/bash
Thanks!
On 1/13/12 3:05 PM, David Jennes wrote:

Weird, 'cause it works for me. Could you tell us what the exact shebang was? (including spaces and all that)

Reply to this email directly or view it on GitHub:
https://github.com/dustinrue/ControlPlane/issues/115#issuecomment-3485355

Gah, I knew I'd forgotten something!

The recent changes broke something. I have a script called transmission.pause.all which starts with a proper shebang. However, 1.1.5 won’t run it. The quick fix in my case is to append .sh to the script. But the script used to run just fine. So something broke. djbe, may I recommend unit tests in order to prevent such regressions?

I've thought about doing unit tests but they're quite hard to do/implement in controlplane, as all it does is use external sources and modify external programs. If anyone knows a good way of doing this, I'm all for implementing them. Otherwise we'll keep on testing manually and pushing new versions as quickly as possible (so if you have an issue, you can expect a fixed version pretty soon).
So, which last version did work for you? What's your exact shebang line (including spaces)? Is it the first line in the script?

I am no good at Objective-C development, so unfortunately I cannot implement an example of how to do unit tests here. But the general idea would be to push the design towards more loosely coupled units.
Take for instance the shebang detection. Right now it is part of the long execute method, starting from line 85, if I understand the code correctly. You could push that code into a separate unit of design. At the very least, it could be an independent method, which could then be unit-tested. More appropriately, probably, the detection would be in a separate class. Something like InterpreterDetector springs to mind. – In fact, you might have a more general role here. You have at least two InterpreterDetectors at hand: a detector that interprets the shebang line and one that uses file extensions.
As to my particular problem, this is the shell script I am running: http://cl.ly/0Z3l022N2G2f2D051c25

While your idea about the execute method is sound (I've been wanting to move that bit to a separate method just to keep things clear, actually creating interpreter detector and derivates of it is a case of over designing things, there's absolutely no need for that in this case (it's the only place where it'd be used, otherwise I might agree).

Oh, I forgot to mention that I don’t know which version last worked for me. I believe it was around Christmas that I saw it working last.

To a certain degree, design is a matter of taste. In general however, reuse is not a necessary precondition for pushing a functionality into a separate unit of design. In this case, I think the clear separatation of concerns and the common interface the two ’InterfaceDetectors’ would share are reasons enough.
But going back to the original issue: even separate methods would allow you to unit-test those bits. :)

Great conversation here. I've given some thought to unit tests as well and came to same conclusion as djbe, it is difficult in the case of ControlPlane.
This particular failure is new and working as designed so a unit test would have exposed nothing, other than a particular script was done wrong. In the case of your script it worked only because CP always assumed you wanted to use sh but I felt it would be better if you could use anything. Now that I think about it, the reworked action doesn't default to sh in the end, which would have been the safest thing to do.

Actually it does default to /bin/bash (is there a difference with /bin/sh?), see line 167.

I'm on my iPod so I didn't look, couldn't remember if I had defaulted or not, flipped a coin. Bash is a newer sh so very similar, like vi vs vim.

Dustin, what does this have to do with defaults? My script has a proper shebang line.

Well, defaults in this case means that if CP can't parse your shebang line it'll try to get it from the extension, and if that doesn't work it'll default to bash.

I see. But why should CP be incapable of parsing the shebang line in this case? Is something wrong with it?

patience, I'm about to test it. (it's a bit annoying to test as I have to disable a whole bunch of my scripts or it'll shutdown my NAS and other things based on my current location).

So it looks like it did not default it, but you did in 1.1.5, this should really be working for that script based on what I see.

Hmm, can't reproduce it. Your file works fine here (I checked and CP is getting the interpreter from the shebang line).
If possible could you compile CP yourself and check the logs (I've just added some log entries for this) to see from where CP is getting the interpreter?

David, the debug version you sent me works just fine. So I am inclined to think there is a difference between the public 1.1.5 and the current code. I even deleted ControlPlane.app and redownloaded it, just to be perfectly sure.

David, in the debug version of 1.1.5 that you sent me there is no "Interpreter from" output when the script fails to execute. CP obviously does not even get there. If I append .sh to the file name, CP correctly identifies the interpreter from shebang as /bin/bash.

Ooooh, I get it now. getInfoForFile is failing 'cause your file doesn't have a correct extension, and we don't go further. The changes I pushed today moved that call into the interpreterFromExtension method, so it only gets called when we need it.
Sigh. This thus fixes yet another edge case :-P
Btw, this is something which wouldn't have been found with UT as it's a new edge case, not really a regression.

I reinstalled 1.1.5 from the public download area and it is working for me.  This is the script I'm using:
#!/bin/bash

echo "it worked arg1 $1" >> /tmp/thething
I called it sample.  I get output to that file when CP runs the script.

I tested it with @alexbepple's script (remove the .sh) and the getInfoForFile fails for it. As that call is only needed in interpreterFromExtension, I moved it there in one of my last commits (after 1.1.5).

I was just going to write, this is failing because of the .all "extension"

:D yeah, I assume sample is associated with some app. We need another way of getting the extension.

I've tested this with a .all extension and it works.  So I confirm that this is fixed and this 115 issue is now double closed


add horizontal line in classifier config

Background
Link to issue
https://biomage.atlassian.net/browse/BIOMAGE-786
Link to staging deployment URL
https://ui-agi-ui199.scp-staging.biomage.net/
Links to any Pull Requests related to this
N.A.
Anything else the reviewers should know about the changes here

Horizontal lines enabled, will update if FDR is changed
DynamoDB record and S3 files in staging for experimentId ff009e1c285dc198156dd7cde30c112c might need to be updated for the experiment to work

Changes
Code changes
Definition of DONE
Your changes will be ready for merging after each of the steps below have been completed:
Testing

[ ] Unit tests written
[X] Tested locally with Inframock
[X] Deployed to staging

To set up easy local testing with inframock, follow the instructions here: https://github.com/biomage-ltd/inframock
To deploy to the staging environment, follow the instructions here: https://github.com/biomage-ltd/biomage-utils
Documentation updates
Is all relevant documentation updated to reflect the proposed changes in this PR?

[ ] Relevant Github READMEs updated
[ ] Relevant wiki pages created/updated

Approvers

[ ] Approved by a member of the core engineering team
[ ] (UX changes) Approved by vickymorrison (this is her username, tag her if you need approval)

Just before merging:

[ ] After the PR is approved, the unstage script in here: https://github.com/biomage-ltd/biomage-utils is executed. This script cleans up your deployment to staging

Optional

[X] Photo of a cute animal attached to this PR

DynamoDB record and S3 files in staging for experimentId ff009e1c285dc198156dd7cde30c112c might need to be updated for the experiment to work <-- we need to make sure we don't forget about this.

DynamoDB record and S3 files in staging for experimentId ff009e1c285dc198156dd7cde30c112c might need to be updated for the experiment to work <-- we need to make sure we don't forget about this.

I've checked and all the entries are ok


EV Maximums

A small bug, EVs are supposed to cap off at a maximum of 252 not 255.

Not necessarily a bug, EVs can go up to 255, although stat changes only happen every 4 points so it's not necessary for them to go up to 255 as you'd just be wasting EVs doing nothing.

Technically this is a bug for Gen VI and onward, as EVs should cap at 252 in those games. I'll keep it open for now.

I guess you're right actually, although it looks like that's a change from Gen 7 onward, which means I wasted time trying to get my stats perfect in SwSh.

@Evilstar006 It's Gen VI onward: https://bulbapedia.bulbagarden.net/wiki/Effort_values#Generation_VI

individual stats now max out at 252 EVs instead of 255; if a Pokémon is transferred forward into a Generation VI game while it has 253 or more effort values in a stat, they will be reduced to 252

It's still a nice quality of life that I wasn't aware of to much gen 3 and 4 for me, although it would be a nice quality of life feature for Gens 3 through I guess 5 lol although you'd probably have to put the same handler in. If I knew more on the programming language you were using I'd help, but I enjoy some more niche languages, but I'm excited to see what will become of this, it's very handy right now with my computer Out of Commission.

This is now being worked on.


scrcpy: 3.0.2 -> 3.1

https://github.com/Genymobile/scrcpy/releases/tag/v3.1
Diff: https://github.com/Genymobile/scrcpy/compare/v3.0.2...v3.1
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build scrcpy scrcpy.passthru.tests


Multiple charts won't show

I want to use multiple charts within a bootstrap 3 website, this includes a doughnut, a bar and a line chart.
When I only use the bar chart, it shows. When I add the doughnut chart, none of them is showing and when I delete the bar chart, but keep the doughnut chart, the doughnut won't show up either.

I'm using 8 different charts on a page in my application, if possible can you please take a screenshot and provide some more details (i.e. browser).  Thanks

need a test case to get anywhere with this issue, closing unresponsive


belongsToMany multiple issues

Reproduction
class Contact extends Model {
  static entity = "contacts";

  static fields() {
    return {
      id: this.uid(),
      userId: this.attr(null),
      user: this.belongsTo(User, "userId"),
    }
  }
};

class User extends Model {
  static entity = "users";

  static fields() {
    return {
      id: this.uid(),
      groups: this.belongsToMany(Group, GroupUser, "userId", "groupId"),
    }
  }
};

class Group extends Model {
  static entity = "groups";

  static fields() {
    return {
      id: this.uid(),
    }
  }
};

class GroupUser extends Model {
  static entity = "group_user";
  static primaryKey = ["groupId", "userId"];

  static fields() {
    return {
      groupId: this.uid(), // Docs say this.attr(null) which throws an error
      userId: this.uid(),
    }
  }
};

useRepo(Contact).save({
  id: 1,
  user: {
    id: 1,
    groups: [{
      id: 1
    }]
  }
});

Expected behavior
Nested relationships are normalized and saved.
Actual behavior
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'attach')
    at EntitySchema2.<anonymous> (index.mjs:1103:26)
    at EntitySchema2.getId (normalizr.es.js:137:17)
    at EntitySchema2.normalize5 [as normalize] (normalizr.es.js:151:19)
    at visit2 (normalizr.es.js:478:17)
    at ArraySchema2.normalizeValue (normalizr.es.js:251:27)
    at normalizr.es.js:384:20
    at Array.map (<anonymous>)
    at ArraySchema2.normalize5 [as normalize] (normalizr.es.js:383:19)
    at visit2 (normalizr.es.js:478:17)
    at normalizr.es.js:176:32
(anonymous)	@	index.mjs:1103
getId	@	normalizr.es.js:137
normalize5	@	normalizr.es.js:151
visit2	@	normalizr.es.js:478
normalizeValue	@	normalizr.es.js:251
(anonymous)	@	normalizr.es.js:384
normalize5	@	normalizr.es.js:383
visit2	@	normalizr.es.js:478
(anonymous)	@	normalizr.es.js:176
normalize5	@	normalizr.es.js:172
visit2	@	normalizr.es.js:478
(anonymous)	@	normalizr.es.js:176
normalize5	@	normalizr.es.js:172
visit2	@	normalizr.es.js:478
(anonymous)	@	normalizr.es.js:360
normalize2	@	normalizr.es.js:359
visit2	@	normalizr.es.js:475
normalize4	@	normalizr.es.js:515
normalize	@	index.mjs:1147
process	@	index.mjs:1142
save	@	index.mjs:1454
save	@	index.mjs:1685
read	@	db.js?t=1658970281973:38

Additional information
Issue 1. Even though the docs say the pivot model's id fields are this.attr(null), this causes an error. Setting them each to this.uid() fixes that. I'm not sure what the behavior should be because the docs are unclear.
Issue 2. (The main issue) belongsToMany appears to work one level deep, but not nested further down as reproduced via the code above.

Thanks for finding the bug....yeah thats going to be harder to fix...


point delete bug

My actions before raising this issue

[ ] similar to https://github.com/openvinotoolkit/cvat/issues/1391

Expected Behaviour
just remove one point

Current Behaviour
random point is removed

Steps to Reproduce (for bugs)

https://u.pcloud.link/publink/show?code=XZkKiEXZqcJUymaSNN5fOUnR05q1LhmNYLvX

Context

Your Environment
Server version: 1.6
Core version: 3.16.0
Canvas version: 2.7.0
UI version: 1.23.1
linux

@ActiveChooN any updates about this task? could you reproduce this problem?

@laol777, have no opportunity to check this issue

We are going to address the issue in next releases


Consider adding a license

See https://choosealicense.com/ for tl;dr and to confirmation that it is a real problem
Currently no license is mentioned anywhere, what makes this code fully copyrighted. This is a default for any creative work - what includes software.
Please, please add a license. The fact none is listed makes using this software a legal quagmire.
For start - people are not allowed to make pull requests with improvements as it would require ditribution of modified code - that is fully copyrighted.
For example currently it is not legal to use, improve or distribute this code. Hopefully this is not the intended effect, given that code was open sourced.
In case that for some reason not releasing it under open license is intentional - mentioning it in the readme also would be a good idea. Though note that given that it is open sourced it will not stop anyone not caring about copyrights.
Popular licences include:

MIT license that allows people to use code for any purpose
GNU GPLv3 that intends to allow everything but to block people from distributing closed source versions that build on open source project

Feel free to ask me questions :) I know and understand that people often prefer to ignore copyright law, unfortunately it will not make it disappear.

Note that for licence to apply all authors of work must agree - though it seems that it may be doable for this repository - see https://github.com/codeforsanjose/OSM-SouthBay/graphs/contributors


fix: mailer generation with namespaces

Fixes #272
I'm a little confused why this gem implements mailer generation if it's just a copy-paste from rails, but if we're going to do it, then we need the layout files as well.
cc @dixpac from #84

👍
@flavorjones @dhh I wish we could just make one of the next engine versions railties >= 7, initially I had to jump through all kind of hoops with the generator just to make it work with both 6 and 7, because there was a "large" change in rails 7 scaffolds.

@dixpac Thanks for the :eyes: and the note about Rails 7 generators (which I wasn't aware of).
:thinking: I don't think we're currently running CI with Rails 6.x, do you think we should do that unless and until we pin to >= 7?

🤔 I don't think we're currently running CI with Rails 6.x, do you think we should do that unless and until we pin to >= 7?

Good question, I guess we should :)

@dixpac Thanks for the 👀 and the note about Rails 7 generators (which I wasn't aware of).

I basically had to copy rails 7 default generator and call it tailwindcss generator, so anyone that is using rails 6 can have rails 7 like behavior with divs and partials instead of old html tables that were used in rails 6.

🤔 I don't think we're currently running CI with Rails 6.x, do you think we should do that unless and until we pin to >= 7?

Good question, I guess we should :)

I'll take care of it.


Remove original package details page

Related Problem
Currently Gallery has two versions of the package details page due to a recent redesign. We maintain both versions currently.
The Elevator Pitch
Now that we've shipped the redesign, we should drop the previous version.
Additional Context and Details
No response

This issue has been deployed to PROD, however, we still need to remove the feature flag. Just in case we need to rollback the deployment, I will wait until the next deployment before removing the feature flags.

Feature flags have been removed.


Add analytics for user navigation journeys

Add Google Analytics dimensions:

Whether a user is viewing a navigation or a content page, i.e. where they are in their journey through the site
What type of navigation page they visit, e.g. a taxon page or a mainstream browse page

Don't merge this until we have confirmation of the GA dimensions to use. I've temporarily set them to values outside of the GA range.
Trello: https://trello.com/c/SCsj2A9D/425-add-custom-dimension-to-page-view-tracking-on-new-navigation-pages

We've confirmed the GA dimensions as 32 and 33, so this is now ready to go.


refactor(helper):  Use getRuntimeKey() in env function

I've removed the logic that depends on c.runtime. I've also refactored the code to map behavior for each runtime to an object. If this is overkill, I can revert it back to the original code.
Author should do the followings, if applicable

[ ] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

fixed: https://github.com/honojs/hono/issues/1452

Hi @watany-dev !
Great job with nice refactoring. Thanks!


2.0.1

Once merged, this PR will immediately be ready to be live on ftb. See the changes to CHANGELOG.md for a list of changes this makes (aside from the security/db-lists stuff)
@Alexia mind reviewing this?
This will close #35 and #34.

@Alexia Ping.


JToken parse is returning empty seqs

Hello, hope you're good. I'm trying to parse a stringfied JSON value, but the return is composed of empty seqs only. I'm using F# in .NET 7, btw.
Source/destination types
// Put the types you are serializing or deserializing here

Source/destination JSON
{
    "ParamA": "1",
    "ParamB": "artifact-00-8",
    "ParamC": "2022-09-01"
}

Expected behavior
To show the seqs with their content there.
Actual behavior
It's showing this:
m: seq [seq [seq []]; seq [seq []]; seq [seq []]]

Steps to reproduce
#r "nuget: Newtonsoft.Json, 13.0.3"

open Newtonsoft.Json

let jsonData = """{
    "ParamA": "1",
    "ParamB": "artifact-00-8",
    "ParamC": "2022-09-01"
}"""

let m = Linq.JToken.Parse(jsonData)

printfn "m: %A" (m)

I am not the author of the library,

But you'd make a champion maintainer :P

You might try using %s instead of %A,

one tiny correction to an otherwise excellent answer: %O is the F# format spec that's used to trigger formatting values via .ToString(); %s will insist on the expression value being implicitly convertible to a String (in general, you're best off using %s and then sanity checking the correct way to render a value - ToString() can sometimes do the right thing, but esp for things that parse into trees, a one size fits all rendering may not even be what you want in any case)
I'd suggest closing this issue as we're more in F# and/or newtonsoft usage question territory than an actual issue with the library that anyone can do anything specific to resolve.
(I'd also suggest looking at the FSharp.SystemTextJson and System.Text.Json libraries if you're building something new, unless you're in a system that has standardised on Newtonsoft.Json)

Thank you for the great replies and insights @elgonzo and @bartelink.

I'd suggest closing this issue as we're more in F# and/or newtonsoft usage question territory than an actual issue with the library that anyone can do anything specific to resolve.

Agreed, I'll close this issue now.

But you'd make a champion maintainer

100% agree

100% agree

That's easy to say for a future co-maintainer :P

Watching activity here in repo for several years I strongly believe that Json.Net itself has no future.


Merge Prepared & Execute in execute.rs

Much nicer!

I left AlterTable because it would be a bit of a mess to use try_block there, try_into works nicer.


chore: PMD rule for @Nonnull / @Nullable on fields of public types

Follow-up https://github.com/SAP/cloud-sdk-java/pull/368
Context: mapping `int priority` to `severity`

lombok says adding @Nonnull everywhere is is massive performance penalty and waste of resources.
They also recommend using an IDE level reminder for this use case.


Add luke to members

Adds Luke Cravey to People tab. Includes profile text and profile image

Thanks Luke! This looks great. Can you please merge main into this branch, and then apply the change below? I will merge after that.

Done! Should be good to go.


Chat disconnected after certain amount of time

I have ytcfilter installed.
When I open youtube livestream my youtube live chat is disconnected after a certain amount of time. I disable the extension and reload youtube livestream page and my chat never disconnected again, because of that I assume this is the issue related with this extension.
I'm using opera gx

Are you using another extension like LiveTL? If so there is a conflict between the 2 which is fixed in ytcFilter 2.1.3. The new version is already available on Firefox, but Google is slow. If so, you can install straight from the release package from Github.

Are you using another extension like LiveTL? If so there is a conflict between the 2 which is fixed in ytcFilter 2.1.3. The new version is already available on Firefox, but Google is slow. If so, you can install straight from the release package from Github.

Looks like it, I have 2.1.3 for now. So far not disconnected with LiveTL enabled

Looks like it, I have 2.1.3 for now. So far not disconnected with LiveTL enabled

I had the same issue on 2.1.2 and 2.1.3 seems to have fixed it.

I had the same issue on 2.1.2 and 2.1.3 seems to have fixed it.

Review finally ended for Chrome so 2.1.3 is officially released for those browsers

Review finally ended for Chrome so 2.1.3 is officially released for those browsers


MyArmM get_joints_min and get_joints_max don't match the values in calibration_parameters, leading to ValueErrors

Describe the bug
In order to control MyArmM using MyArmC, I need to copy joint angles from MyArmC to MyArmM.
I need to know what the min and max values are, so I can clamp the values to the correct values, before sending them to set_joint_angle. However, it's impossible to do so right now because calibration_parameters has different values than reported by get_joints_min and get_joints_max.
This change is introduced in https://github.com/elephantrobotics/pymycobot/pull/103/files#diff-edb1753620e8370d80dd9ab73dff12930bbf15bac171ea5b08ca00af41f248e6R355
Where the PR sets
"angles_min": [-170, -83, -90, -155, -91, -153, -118],
"angles_max": [170, 83, 84, 153, 88, 153, 2],

However, if you call get_joints_min and get_joints_max, you get:
mover.get_joints_max()=[172.0, 90.0, 91.0, 148.0, 84.0, 146.0, 0.0]
mover.get_joints_min()=[-168.0, -77.0, -86.0, -159.0, -95.0, -161.0, -118.0]

These values are different, leading to ValueErrors when teleoperating the MyArm pair.
This is using v1.1 firmware on both MyArmC and MyArmM.

Please provide the software version, which can be queried using the following interfaces: get_robot_firmware_version, get_robot_modified_version

The main control software version seems to be incorrect. Please use myStudio to re-burn the latest picoMain firmware. The correct get_robot_modified_version should be 7


Use fully qualified image names

Current Situation

We use unqualified image names in our Dockerfiles and docker-compose files.
Here's a note from Podman explaining which this is not generally a good idea:
# NOTE: RISK OF USING UNQUALIFIED IMAGE NAMES
# We recommend always using fully qualified image names including the registry
# server (full dns name), namespace, image name, and tag
# (e.g., registry.redhat.io/ubi8/ubi:latest). Pulling by digest (i.e.,
# quay.io/repository/name@digest) further eliminates the ambiguity of tags.
# When using short names, there is always an inherent risk that the image being
# pulled could be spoofed. For example, a user wants to pull an image named
# `foobar` from a registry and expects it to come from myregistry.com. If
# myregistry.com is not first in the search list, an attacker could place a
# different `foobar` image at a registry earlier in the search list. The user
# would accidentally pull and run the attacker's image and code rather than the
# intended content. We recommend only adding registries which are completely
# trusted (i.e., registries which don't allow unknown or anonymous users to
# create accounts with arbitrary names). This will prevent an image from being
# spoofed, squatted or otherwise made insecure.  If it is necessary to use one
# of these registries, it should be added at the end of the list.

In other words, it's too easy for the wrong image to be pulled. On machines where multiple default registeries are configured, just up will hang while the docker backend asks which registry to pull each unqualified images from.
Suggested Improvement

Switch to using fully qualified image names, e.g., instead of airflow -> docker.io/apache/airflow and so on. This would need to be applied to essentially all of our images (except Elasticsearch, because we already pull it from a non-default registry anyway).
Benefit

Reduced ambiguity, reduced chance of the incorrect image working, eliminate issues on machines configured with multiple default search registries.

@sarayourfriend I can pick this up.


doc: releases: add working draft release notes + guide for v3.6.0

Add working draft for the release notes and migration guide for v3.6.0.

Neglected to update the glob pattern for listing the v3.6.0 release notes in the ToC. Rebased.

Ah yes, I should have spotted that one. Unfortunately making ToCs more "curated" has drawbacks like this :/ I mentioned it in another thread, but at some point I might looks at putting together some Sphinx magic to help with only maintaining "active releases metadata" in a single place, and get things like Release date tables, Changelogs for EOL releases etc. automatically generated


fix duplicate logging entries

Background: https://github.com/Pylons/waitress/issues/151
For waitress it's required to add a log handler to the root logger (which is recommended in general) otherwise waitress will add its own handler. As a result, there exists a waitress handler and a flask handler which are both used for logging.
Result with debug log level:
❯ python3 src/wopiserver.py
{"time": "2022-10-11T11:18:20.548", "host": "superion", "level": "INFO", "process": "wopiserver", "module": "wopiserver", "msg": "WOPI Server starting in unsecure/embedded mode", "port": "8880", "wopiurl": "https://your-wopi-server.org:8443", "version": "git"}
{"time": "2022-10-11T11:18:25.721", "host": "superion", "level": "DEBUG", "process": "wopiserver", "module": "wopiserver", "msg": "Accessed index page", "client": "<IP_ADDRESS>"}
DEBUG:wopiserver:"module": "wopiserver", "msg": "Accessed index page", "client": "<IP_ADDRESS>"
{"time": "2022-10-11T11:18:26.087", "host": "superion", "level": "DEBUG", "process": "wopiserver", "module": "wopiserver", "msg": "Accessed index page", "client": "<IP_ADDRESS>"}
DEBUG:wopiserver:"module": "wopiserver", "msg": "Accessed index page", "client": "<IP_ADDRESS>"

To fix it, this PR is removing the flask log handler and adding a proper handler to the root logger if waitress is used.
Result:
❯ python3 src/wopiserver.py
{"time": "2022-10-11T11:20:30.725", "host": "superion", "level": "INFO", "process": "wopiserver", "module": "wopiserver", "msg": "WOPI Server starting in unsecure/embedded mode", "port": "8880", "wopiurl": "https://your-wopi-server.org:8443", "version": "git"}
{"time": "2022-10-11T11:20:38.280", "host": "superion", "level": "DEBUG", "process": "wopiserver", "module": "wopiserver", "msg": "Accessed index page", "client": "<IP_ADDRESS>"}
{"time": "2022-10-11T11:20:38.639", "host": "superion", "level": "DEBUG", "process": "wopiserver", "module": "wopiserver", "msg": "Accessed index page", "client": "<IP_ADDRESS>"}
{"time": "2022-10-11T11:20:39.029", "host": "superion", "level": "DEBUG", "process": "wopiserver", "module": "wopiserver", "msg": "Accessed index page", "client": "<IP_ADDRESS>"}

As a matter of fact, we do see a few duplicate logs, only at the start of the process in flask mode. I wonder if something could be done (though it's very minor).

The first log entry WOPI Server starting in unsecure/embedded mode is duplicated because Flask immediately restart the server, but that's related to the debug mode behavior of Flask and IMO not a logging issue.

The first log entry WOPI Server starting in unsecure/embedded mode is duplicated because Flask immediately restart the server, but that's related to the debug mode behavior of Flask and IMO not a logging issue.

Right, thanks for that - and indeed not really an issue for this PR.

@glpatcern Could we get a tag for this change?

@xoxys sure, I wanted to tag today after merging two more completed PRs.

Cool, thanks!


Support for new OpenAI APIs announced at DevDay

New & Updated APIs:

Assistants
GPT-4 with vision
DALLE-3
TTS

Already on it

@StephenHodgson will the 7.1 milestone (that I see you're furiously working on, thank you!!) include support for JSON Mode and Reproducible Outputs?  Thanks for all your hard work!

I need JSON Mode like yesterday 😜 So happy you are working on this so fast!

@tghamm @ADefWebserver gonna get just a few things out today for 7.1, but will def try to knock out the rest in 7.2

I'd like to close this and split it out into more targeted issues tbh to track better

@tghamm @ADefWebserver gonna get just a few things out today for 7.1, but will def try to knock out the rest in 7.2

Understood, it is-a-lot of stuff!


Check metadata file annotation by generating data (e.g. MaxLength)

OpenUI5 version: 1.68.1
Browser/version (+device/version): All
Hi,
when using the option for the mock server to generate automatically data based on the metadata.xml the annotation for MaxLength is not processed. The generated data functionality is not checking this annotation for strings for example. It simply taking the Entity type name and adding an incremented integer value.
simulate(bGenerateMissingMockData: true)
For example i have the following line in the metadata.xml:

The generated value is "Test 1" which has a length of 6. Allowed is only 3.
Best regards,
Manjinder

Hi @manjinsi ,
Can you please provide a sample in which we can reproduce the issue? A Plunker or JSBin will work.
Thank you and best regards,
Hristo

Since there is no response from the author for more than 4 weeks I'm closing the ticket.

Hello @manjinsi ,
I've created an internal incident<PHONE_NUMBER> so that our colleagues can check that. The status of the issue will be updated here in GitHub.
Regards,

Hello @manjinsi ,
Please provide more details as requested above so we can process the ticket.
Regards,
Michal

Since there is no response from the author for more than 4 weeks I'm closing the ticket.


feat: Sync from noir

Automated pull of development from the noir programming language, a dependency of Aztec.
BEGIN_COMMIT_OVERRIDE
fix(elaborator): Lazily elaborate globals (https://github.com/noir-lang/noir/pull/5191)
chore: schnorr signature verification in noir (https://github.com/noir-lang/noir/pull/5188)
fix(experimental elaborator): Fix globals which use function calls (https://github.com/noir-lang/noir/pull/5172)
feat!: restrict noir word size to u32 (https://github.com/noir-lang/noir/pull/5180)
chore: break out helper methods for writing foreign call results (https://github.com/noir-lang/noir/pull/5181)
fix: Fix panic in get_global_let_statement (https://github.com/noir-lang/noir/pull/5177)
fix(elaborator): Invert unconstrained check (https://github.com/noir-lang/noir/pull/5176)
chore: loosen trait bounds on impls depending on AcirField (https://github.com/noir-lang/noir/pull/5115)
feat: support casting in globals (https://github.com/noir-lang/noir/pull/5164)
chore(experimental elaborator): Handle comptime expressions in the elaborator (https://github.com/noir-lang/noir/pull/5169)
chore: avoid manual creation of contract artifact in wasm (https://github.com/noir-lang/noir/pull/5117)
chore: start moving lints into a separate linting directory (https://github.com/noir-lang/noir/pull/5165)
chore: move acir docs to code declaration (https://github.com/noir-lang/noir/pull/5040)
feat!: separate proving from noir_js (https://github.com/noir-lang/noir/pull/5072)
END_COMMIT_OVERRIDE

note to self: stop merging master

Benchmark results
Metrics with a significant change:

protocol_circuit_proving_time_in_ms (base-rollup): 83,527 (+20%)
protocol_circuit_proving_time_in_ms (root-parity): 41,312 (+21%)
protocol_circuit_proving_time_in_ms (public-kernel-setup): 44,823 (+19%)
protocol_circuit_proving_time_in_ms (public-kernel-app-logic): 46,677 (+20%)
protocol_circuit_proving_time_in_ms (public-kernel-teardown): 45,321 (+17%)
protocol_circuit_proving_time_in_ms (public-kernel-tail): 182,490 (+18%)
protocol_circuit_proving_time_in_ms (root-rollup): 22,845 (+17%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,734
1,560
697
754 (+1%)
778 (+1%)

proof_construction_time_sha256_30_ms
11,411
3,057
1,375
1,427 (+1%)
1,451

proof_construction_time_sha256_100_ms
43,854
11,763
5,417
5,358
5,356

proof_construction_time_poseidon_hash_ms
78.0
34.0
34.0
58.0
88.0

proof_construction_time_poseidon_hash_30_ms
1,522
420
202
222 (-1%)
266

proof_construction_time_poseidon_hash_100_ms
5,760
1,580
728
792 (+3%)
798 (+1%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
1,412
1,412
1,412

l1_rollup_calldata_gas
9,464
9,454
9,452

l1_rollup_execution_gas
610,285
610,275
610,273

l2_block_processing_time_in_ms
698
1,292 (-1%)
2,493 (+2%)

l2_block_building_time_in_ms
22,265 (-1%)
45,913
87,409 (-1%)

l2_block_rollup_simulation_time_in_ms
22,157 (-1%)
45,560
87,008 (-1%)

l2_block_public_tx_process_time_in_ms
19,062
42,126
83,735 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
5,941 (-1%)
8,541 (-1%)

node_database_size_in_bytes
9,482,320
12,738,640

pxe_database_size_in_bytes
9,242
15,127

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.4 (-1%)
16.9 (-1%)
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.7
31.8
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.605 (-1%)
0.518 (-1%)
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
48.6 (-1%)
76.6
134
247
481 (+1%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
288
543
1,055
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.496 (-1%)
0.472 (+1%)
0.458
0.449
0.449 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
59.5 (-1%)
113
185
356 (-1%)
708 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
108
208
355
691
1,363
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.506
0.507
0.489
0.482 (-1%)
0.487 (+2%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
59.9 (-1%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
103

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.550 (-1%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
79,470
665,267

Transaction size based on fee payment method
| Metric |  |
| - |  |


Import Racks failing because of non-null: desc_units

input row:
Amsterdam,,Temp,,,,,23,42
Error: Record 1: null value in column "desc_units" violates not-null constraint DETAIL: Failing row contains (9, 2016-11-07, 2016-11-07 06:48:33.164686+00, Temp, null, 42, , null, 4, null, null, 23, null, null).

my bad, I didn't restart WSGI Service after upgrading


Where's the config file located?

I installed mpv-handler through the AUR. I Need to change the yt-dlp location as I installed it through pip.
So, where's the config file located?

Create $HOME/.config/mpv-handler/config.toml and edit it (if needed)

But why are you using pip to install yt-dlp?  It's already in the official repository.

Thanks. I figured it out.


Get currently used translation

Description
Let's say your app supports english and german language. User with czech locale logs in and the app fallbacks to english.
Currently, if you setup i18n like this:
export const i18n = new I18n(
  enableHiddenFeatures
    ? translations
    : {
        en:  translations.en,
        de: translations.de,
      }
)
i18n.locale = 'cs'
i18n.defaultLocale = 'en'
i18n.enableFallback = true

i18n.locale returns "cs" despite en translations being used.
Describe the solution
Add function getUsedLocale() that will return locale that is currently used by the library.
Alternatives you considered
In my example above, we can test keys(translations).include(I18n.locale), but parsing language code is not that straight forward usually it might be in format like da_DK or de_DE. This logic is already included in the library and it does not seem right to duplicate it to find out what language is used.

Oh sorry, this is the ruby package. Will recreate it in it's js alternative.


feat(themes): add Now theme

Description
DES-15 | Assist teams from migrating from Now to 2.0 themed styles
We're going back to SHARP AND POINTY 🔪
There are three main value changes that distinguish this theme from the default:

color
radius
shadows

🔍  How to test this change locally

You can switch out the default theme for the nowTheme by implementing this diff.
Run yarn && yarn build && yarn dev
Confirm changes by selecting the primary button and inspecting that the radius is, indeed, 0.

diff --git a/.storybook/preview.tsx b/.storybook/preview.tsx
index 3f54322..2f2d7d9 100644
--- a/.storybook/preview.tsx
+++ b/.storybook/preview.tsx
@@ -1,10 +1,11 @@
 import * as React from 'react';
 import { Provider } from '@manifest-ui/provider';
 import { Story } from '@storybook/react';
+import { nowTheme } from '../packages/themes';

 export const decorators = [
   (StoryFn: Story) => (
-    <Provider>
+    <Provider theme={nowTheme}>
       <StoryFn />
     </Provider>
   ),
diff --git a/packages/themes/src/index.ts b/packages/themes/src/index.ts
index acced89..6ec8f3b 100644
--- a/packages/themes/src/index.ts
+++ b/packages/themes/src/index.ts
@@ -1 +1,2 @@
 export * from './default';
+export * from './nowTheme';

🔮  Outstanding Questions

I removed all unused keys from the theme since the fallback is the default. Is this right? Should I go back and copy/pasta back those key/value pairs?
No strong opinions on the naming of the file and theme. If you have a stronger suggestion, I'm game.

@eemahoney Yep! the theme will fall back. Maybe we should call this theme "legacy" since now is more of a p44 name.


[BUG]Visual Composer ComboBox

ComboBox is not controllable when using VisualComposer, even in case of 'Command-Control' is used.
To reproduce:
Display any controlled unit as 'Command-Control' and try to change values of the StepMode i.e.

/cib

Branch 435-_BUG_Visual_Composer_ComboBox created!


Package Version is not sorted correctly for versions of type vXX.X

Brief description of your issue

Packages whose version are written like vXX.x (eg. v10.1) aren't sorted correctly

Steps to reproduce

Look at the Timber1900.WebDL package, the latest version is v10.2, however the version that shows up in winget is the earliest v9.2
Expected behavior

Winget displaying the latest version as v10.2
Actual behavior

Winget displays the latest version as v9.2
Environment
Windows Package Manager v1.0.11591
Windows: Windows.Desktop v10.0.22000.71
Package: Microsoft.DesktopAppInstaller v1.11.11591.0

I think this is issue that manifest did not updated. As winget does not access github directly. You should open in winget-pkgs

What do you mean by manifest did not updated.
All manifests are in the master branch of winget-pkgs, and installing with the -v v10.2 option installs the correct version, which makes me believe the problem is with the cli itself

Also running winget show Timber100.WebDL --versions show all the versions in the incorrect ordering

Interestingly the .X part is sorted correctly, the only wrong part being v9 vs v10

Interestingly the .X part is sorted correctly, the only wrong part being v9 vs v10
It is sorting as alpha not numerical; zz9something is "greater" than zz1something. Same for v9 vs v1 (~v10).

But I think this is clear to the devs.
Regards,
George

I believe the sorting for Timber1900.WebDL has been worked out. We've got an open PR looking at better behavior for software versions with preambles, but this Issue looks safe to close.


Error when using long prompt and short prompt together

I am getting error when trying to use long prompt and short together. This is the error I got

This is the prompt I am using to test and getting error.
"0": "portrait, 1girl",
"4": "maximum details, cinematic, (abstract art:1.1), deep shadow,1 girl, adult russian woman, grey eyes, blonde sleek hair, Style-GravityMagic, looking away, solo, upper body, detailed background, detailed face, (sovietpunkai, soviet communist theme:1.1), eyes ablaze with dark power, blood sorcerer, scarlet clothes, cape, hood, dynamic movement, evil smile, red glyphs swirling in the air, floating particles, blood droplets, scarlet color scheme, blood dripping, sacrificial altar, in background, glowing red power, nighttime, symmetrical composition, cinematic atmosphere,",
"8": "portrait, 1girl"

This is an issue with Animate Diff not being able to do chunks over 75 tokens. The schedule works normally without animate diff.

I've updated the repo with a fix


pull request for e2e test at 11/23/2016 16:26:03

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Query contracts from the LCD

It seems that there is no way to query contracts from the light client daemon (LCD).
I tried the /wasm/v1/contract/{address}/smart/{queryData} API that is present in the Swagger interface, but I encounter a Not implemented error.
curl -X GET "http://<node-host>/wasm/v1/contract/stars1gc5wcdn9ges00w0l2cfxd7r2puyflak5dmkg26rsh083afmnrjxqu7sutd/smart/%7B%22mintable_num_tokens%22%3A%20%7B%7D%7D" -H  "accept: application/json"

{
  "code": 12,
  "message": "Not Implemented",
  "details": []
}

However, the same query works from the CLI:
starsd query wasm contract-state smart stars1gc5wcdn9ges00w0l2cfxd7r2puyflak5dmkg26rsh083afmnrjxqu7sutd '{"mintable_num_tokens": {}}' --ascii

data:
  count: 0

Would it be possible to add this endpoint to the LCD?

I'll look into it tomorrow

seems to be working for me queryData must be base64 encoded


Fix bug 9937 CTFE floats don't overflow correctly

Revamp of https://github.com/D-Programming-Language/dmd/pull/3590

https://issues.dlang.org/show_bug.cgi?id=9937 comment 4.
This was hashed out there. I do not agree that it Is a bug - it is working as designed, and there are good reasons for that design. I don't believe that 'fixing' it fixes anything - the problems just shift elsewhere.
At some point, if you work with floating point code, you're simply going to have to understand how it works rather than assume it behaves like mathematics does.

I'm certainly not against keeping the excess precision for the lifetime of the compiler.  The backend will always discard excess precision when emitting the data in the end.
I'm just not sure about your assertion that CTFE operations should fail by design when the opposite happens at runtime.  I could argue that we already do this kind of discarding with integers (IntegerExp.normalize), and as far as I can tell it's for the same reason.
The only alternative to this that would be my contribution to the table would be to add two methods to RealExp, toDouble and toFloat (or asDouble) which returns a copy of the real that has been truncated to fit the lower precision requested.
This, for the purpose of when we want to read the value at it's declared precision at key points, such as in assignments and comparisons.  For all other operations, the original is used intact.
All our data types up to 64 bits are precisely defined, so I don't think it is any stretch to ensure that we give at least some consistent behaviour in both runtime and CTFE.

I'm just not sure about your assertion that CTFE operations should fail by design when the opposite happens at runtime.

The problem is that runtime behavior is NOT guaranteed.  Exact precision will depend on inlining, register allocation and instruction selection.  It makes it very difficult to match the runtime behavior exactly...

The problem is that runtime behavior is NOT guaranteed.

If it wasn't guaranteed, then we wouldn't have extensive unittests in std.math and gammafunction.  If dmd is not IEEE 754 compliant with its codegen by default, then that would be a bug.
You should not be using x87 registers for doubles and floats, so inlining and register allocation should not be an issue.

How exactly does this violate IEEE754?
The same thing can occur without x87, eg
a * b + c == a * b + c

Where the LHS is evaluated using multiply+add with a high internal precision while the RHS uses discrete multiply and add.

I agree with adding functions toFloat and toDouble for those rare cases where precision is intended to be discarded. But in general, more precision == better. Intentionally desiring less precise results in order to satisfy an incorrect notion of equality is indicative of faulty algorithm design rather than a compiler bug.

Intentionally desiring less precise results in order to satisfy an incorrect notion of equality

When comparing two floats,  you only ever ask that the relevant bits are equal (eg: memcmp is used for reals).  So it seems correct to me that if you have const float == const float where both sides have been CTFE'd to a value, you wouldn't compare them as const double == const real or as any other mismatch combination.

floats are used to 1) increase speed 2) reduce memory consumption. They are not used to generate less precise results outside of very rare cases, and test suites.
Besides, if this is pulled, the problems will just show up elsewhere. There is no fixing it. There is no intuitive answer. The rules set up cater to the most common, by far, usage scenario, which is "gimme the most precise answer practical." float/double/real specify the minimum precision, not the maximum.

I agree with adding functions toFloat and toDouble for those rare cases where precision is intended to be discarded. But in general, more precision == better.

More precision is not better. In many math functions (see CEPHES) IEEE are assumed and compensator add/sub are used to improve result. More precision may break computation result. This is common case in math.

More precision is not better. In many math functions (see CEPHES) IEEE are assumed and compensator add/sub are used to improve result. More precision may break computation result. This is common case in math.

For an algorithm that depends on reduced precision, it should use toFloat or toDouble. Doing this not only guarantees it is performed, it implicitly documents the requirement. This PR is the wrong solution.

For an algorithm that depends on reduced precision, it should use toFloat or toDouble. Doing this not only guarantees it is performed, it implicitly documents the requirement. This PR is the wrong solution.

And how I can implement this functions for to be portable and CTFEable?

And how I can implement this functions to be portable and CTFEable?

As intrinsics.

As intrinsics.

They would be really helpfull. For example I would be able to clean mir from workarounds and warnings


[Release] Sync main from stage 5/14

Merging following PRs to main -

https://github.com/adobecom/cc/pull/280
https://github.com/adobecom/cc/pull/282
https://github.com/adobecom/cc/pull/290
https://github.com/adobecom/cc/pull/291
https://github.com/adobecom/cc/pull/293
https://github.com/adobecom/cc/pull/294

Test URLs:
Before: https://main--cc--adobecom.hlx.live/careers?martech=off
After: https://stage--cc--adobecom.hlx.live/careers?martech=off

Validation done in stage on firefly marquee and hue sat slider flow  on the below urls
https://www.stage.adobe.com/creativecloud/animation/testdoc/huesat.html

without login :

with login https://stage.photoshop.adobe.com/id

https://www.stage.adobe.com/il_he/creativecloud/animation/testdoc/huesat.html
https://www.stage.adobe.com/de/creativecloud/animation/testdoc/huesat.html

https://www.stage.adobe.com/creativecloud/animation/testdoc/marquee621.html


review ramp limits conversion when parsing CDM data.

Ramp Limits in CDM data are parsed using sys_pu/min


lambda function tips for S3 triggers

Looks good to me, aside from copy editing details. Thanks @DorrinPk !

Over to you @ThanosBaskous

@DorrinPk thanks again for putting this together - if you wouldn't mind resolving the conflicts when you get a chance, I can merge immediately after.

@ThanosBaskous conflicts resolved.

Thanks @DorrinPk!


Ability to search by location

@gpineiro-vivid Some minor conflicts to correct after a merge from a previous PR. Most is pretty straightforward but there is a change to a function that I'd rather you verify as it changes it quite a bit!

@dhlevi branch updated


libvirt platform: collect host dmesg logs

Collect dmesg logs during cleanup. Same as libvirtd logs.

@cwize1 please review.

@cwize1 Please have a look.


Feature no submissions

This PR replaces the submissions model with a new solutions model. It is not just a rename: a user has many submissions for each exericise, but only one solution for each exercise. This means the solution is the current state of an exercise for a given user.
The motivation behind this change is twofold:

reduce storage in ≃ 0.75 factor
simplify platform model

@aguspina @faloi Please review

Is this just a rename?

Don't think so, I believe he only keeps the last submission/solution of a user for each exercise

It's ok for me @faloi

Ok for me too.
However, it would be nice if you can write something about the EventSubscriber. I imagine submissions are now stored somewhere else using this mechanism, and I'm interested on that.

OK. You mean an explanation here or you want a blog post?

A blog post should be really, really nice!
El vie., ago. 21, 2015 1:04, Franco Leonardo Bulgarelli <
<EMAIL_ADDRESS>escribió:

OK. You mean an explanation here or you want a blog post?
—
Reply to this email directly or view it on GitHub
https://github.com/mumuki/mumuki-platform/pull/269#issuecomment-133273376
.

@faloi OK. But In the meanwhile:

Submissions are not stored in the platform. But...
we have event subscribers now, which listen to platform events - currently only submissions. We can register another service for storing submissions. That is why we now have https://github.com/mumuki/mumuki-event-tracker, which is up and runnig since a few days ago.

PS: I have already migrated all the production submissions to the new storage :smile:

:cool:


Add ability to sort by reading ease and length

As discussed on the mailing list, this set of changes adds two sorting options to the ebooks list page: sort by reading ease, and sort by length (word count).

Great, thanks!


Complete view manager docs

(raw notes from mtg)

[ ] update VM docs JSX to use function instead of class component
[x] if (auto control = view.try_aswinrt::SampleLibraryCPP::CustomUserControlCPP
But there's no CPP name
[x] IViewManagerWithExportedEventTypeConstants
[ ] how to write event handlers in VM
[x] SetValue usage is wrong
[x] Conversion from Brush and other xaml and winrt types #377 and https://github.com/microsoft/react-native-windows/issues/6991

@asklar is there an example anywhere (or could there be one) of having a native module with view managers?

yes please see the docs here https://microsoft.github.io/react-native-windows/docs/view-managers

closing this for now as we seem to be ok here; we may need to revise for winappsdk depending on what changes we make. CC @stmoy @AgneLukoseviciute


System.ArgumentException: Value does not fall within the expected range.

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
PowerToys Run
Steps to reproduce
I tried to open the "PowerToys Run" tool with the shortcut and the error appeared.
✔ Expected Behavior
Normal behavior
❌ Actual Behavior
Exception:
System.ArgumentException: Value does not fall within the expected range.
at Wox.Plugin.Common.Win32.NativeMethods.DwmSetWindowAttribute(IntPtr hwnd, Int32 attr, UInt32& attrValue, Int32 attrSize)
at PowerLauncher.ViewModel.MainViewModel.ToggleWox()
at PowerLauncher.ViewModel.MainViewModel.b__147_0()
at System.Windows.Threading.Dispatcher.Invoke(Action callback, DispatcherPriority priority, CancellationToken cancellationToken, TimeSpan timeout)
at System.Windows.Threading.Dispatcher.Invoke(Action callback)
at PowerLauncher.ViewModel.MainViewModel.OnHotkey()
at PowerLauncher.ViewModel.MainViewModel.ProcessHotKeyMessages(IntPtr wparam, IntPtr lparam)
at PowerLauncher.MainWindow.ProcessWindowMessages(IntPtr hwnd, Int32 msg, IntPtr wparam, IntPtr lparam, Boolean& handled)
at System.Windows.Interop.HwndSource.PublicHooksFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
Other Software
No response

/dup #33504


[AMORO-2329]: Miss ChangeStore && PrimaryKeySpec in table detail when creating mixed-iceberg table in InternalCatalog.

Why are the changes needed?
Close #2329.
Brief change log

Set change table location and primary key spec when create a keyed mixed-iceberg table.

How was this patch tested?

[X] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[X] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (no)

Local test result
CREATE TABLE IF NOT EXISTS db.user10 (
    id INT,
    name string,
    ts TIMESTAMP,
    PRIMARY KEY(id)
) USING arctic
PARTITIONED BY (hours(ts));


Train using pid

Using pid mode in simulator, train a network.
Need to use multiple pid gains to induced situations where recovery is required.

Results of first pass.
https://youtu.be/CNS4LPSbZGY
It was trained using multiple PID settings, but the majority were detuned settings so that the robot would encounter recovery situations but it seems like there were not enough "good" situations and the robot "thinks" that oscillating is what it is supposed to be doing.

Trained with multiple pid settings, but the majority were detuned constants, so now the robot seems to think it should drive in a wonky way.
Run video:
https://youtu.be/CNS4LPSbZGY
Training data
https://s3-us-west-1.amazonaws.com/kittcar-data/sim/pid1.tar.gz
https://s3-us-west-1.amazonaws.com/kittcar-data/sim/pid2.tar.gz
https://s3-us-west-1.amazonaws.com/kittcar-data/sim/pid3.tar.gz


[docs] gherkin.md: Fix invalid link

Summary
Fixed a link in the document that points to invalid url.
Details
Previously the link points to https://github.com/cucumber/cucumber/blob/master/docs/docs/gherkin.md#doc-strings because the markdown is docs/gherkin.md#doc-strings.
Removed the docs.
Motivation and Context

The invalid link creates inconvenience for new learners.

How Has This Been Tested?
Clicked and the link now works.
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).

Checklist:

[x] I've added tests for my code.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.

Sorry. Just realized the doc is not to be viewed directly on GitHub.


web: introduce HUDGrid

Hello @hyu,
Please review the following commits I made in branch nicks/panels:
aaf34c0bfba132400cb8e20c849ac3686f55a1bb (2019-11-04 22:24:18 -0500)
web: introduce HUDGrid
Now, HUDGrid manages the layout and overlap between components, rather than
every component having to know about the other components in the grid.

Today I learned: creating styled-components in the render() method led to major performance problems. They need to live in the global scope. (This isn't so obvious in TFT, which only renders once, but is much more obvious here)

(I did some CSS descender tricks to resolve this problem, but it's something we should keep an eye out for on future code reviews)

ya, i agree we need to do a naming pass on the different components.


adding go plugin as a new method in Computed

Hi, talaria members. Just a POC here, and the doc of this proposal: https://docs.google.com/document/d/1DyrMm1ewRBMs24yythcMyVFWqmRR9qdABulYob5AR-o/edit#heading=h.jg1meco85n83

Looks good overall now, just need to clean up comments and code


Popup appears when a new note is created with no notebook selected

Added functionality so that if a user tries to create a new note without selecting any notebook, a popup appears and says what to do.
Changes:

I thought the new popup shouldn't contain a red border as it is not a "dangerous" thing to click on a new note button. So I made some changes to new.scss to change the appearance of the popup.

Changes in index.html create a new DOM element to display the poup. That's it.

So I think we can close #152 now. :smile:

Thanks!


Add TinyBird Guide

#351  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @thisisgoldman and the rest of your teammates on  Graphite

Merge activity

Oct 4, 6:58 PM EDT: @thisisgoldman started a stack merge that includes this pull request via Graphite.


[Ocaml] Add an anchor for performance

This seems to solve the issue mentioned in #162

On my computer, times drop from
real    1m12,987s
user    1m12,946s
sys     0m0,036s
to
real    0m0,329s
user    0m0,313s
sys     0m0,018s
for the 1024 line case.

Out of curiosity: why does this PR work?

Thank you, it's very clear now. It's an easy trap to fall into, I think. A gotcha to keep in mind.

Just for completeness if anyone finds this issue because it got strike by a similar problem.
Once Nicolas understood that the performance issue stroke us when dealing with a huge pattern matching, I looked for the queries related to pattern matching and modified them to make them print a different number each. For instance:
(
  "|"* @do_nothing
  (match_case) @prepend_spaced_softline @append_delimiter
  (#delimiter! "2")
)

and then one can run topiary --input-file=match_1024.ml --skip-idempotence and see that there are hundreds of 2's, so that the issue comes from this query.

I have on occasion also misused @append_delimiter to debug which pattern is getting applied in a particular situation 😀


(Do not merge) PR to trigger swift-testing cross-repo CI

(Do not merge) This is a PR meant solely to trigger a swift-testing cross-repo CI toolchain build.

Please test with following pull request:
https://github.com/swiftlang/swift-testing/pull/692
@swift-ci please build toolchain


Friday 16-02-2024 roll call

Roll Call!


update with-sentry example

Changes:

moved the configuration from HOC to _app.js
fixed the example, as componentDidCatch catches errors during rendering phase, but not within event handlers.

rename SENTRY_DSN to SENTRY_PUBLIC_DSN, plz

FYI @Tomekmularczyk @avdeev The sentry docs example look different than the example.  I assume raven has turned into @sentry/browser but please set me know if there is some reason next.js needs to use raven instead.

@Enalmada I merged this yesterday: https://github.com/zeit/next.js/pull/5253


[feat][broker] Segmented transaction buffer snapshot segment and index system topic

Master Issue: https://github.com/apache/pulsar/issues/16913
Motivation
Implement system topic client for snapshot segment topic and index topic to send segment snapshots or indexes.
The configuration transactionBufferSegmentedSnapshotEnabled is used in the Transaction Buffer to determine which AbortedTxnProcessor is adopted by this TB.
Modification
Add a TransactionBufferSnapshotService to handle the system topic service:

The abstraction for transaction system topic service:

The abstraction for transaction system topic client:

Verifying this change

[x] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

@poorbarcode @congbobo184 Please help review the PR again, thanks.

Codecov Report

Merging #16931 (6ec7dc6) into master (6c65ca0) will increase coverage by 35.87%.
The diff coverage is 50.00%.

@@              Coverage Diff              @@
##             master   #16931       +/-   ##
=============================================
+ Coverage     34.91%   70.79%   +35.87%
+ Complexity     5707      438     -5269
=============================================
  Files           607       26      -581
  Lines         53396     2246    -51150
  Branches       5712      245     -5467
=============================================
- Hits          18644     1590    -17054
+ Misses        32119      483    -31636
+ Partials       2633      173     -2460

Flag
Coverage Δ

unittests
70.79% <50.00%> (+35.87%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...apache/pulsar/proxy/server/DirectProxyHandler.java
63.63% <50.00%> (ø)

...pache/pulsar/proxy/server/ProxyServiceStarter.java
60.66% <0.00%> (-1.34%)
:arrow_down:

...n/java/org/apache/pulsar/broker/PulsarService.java

...pulsar/broker/admin/impl/PersistentTopicsBase.java

...broker/delayed/InMemoryDelayedDeliveryTracker.java

...rg/apache/pulsar/broker/service/BrokerService.java

.../service/SystemTopicBasedTopicPoliciesService.java

...r/service/SystemTopicTxnBufferSnapshotService.java

.../pulsar/broker/stats/BrokerOperabilityMetrics.java

...er/systopic/NamespaceEventsSystemTopicFactory.java

... and 573 more

please modify the PR description for the new commit, then can be merged.


test: use nextest

Codecov Report

Merging #9 (2b9e1aa) into main (c10cf59) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   79.90%   79.90%
=======================================
  Files          14       14
  Lines         408      408
=======================================
  Hits          326      326
  Misses         82       82

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


the id of room or the id of contact will change。

last week , i got my userSelf id is @@65c19ef5b7d988b5dfb8adfa73373168a119a8e9f3972158bb41f495af82a3cf，
but yesterday , it change other one。
room's id will change too。

每次重新登陆，都会变化


Kraken Leveraged short

Hello guys,
Kraken exchange has an option to make a leveraged short sale. Is there any possibility to do shorting over the xchange-kraken? I've checked a little bit, but there is no example. Maybe with KrakenOrderFlags?
Thank you and Regards,
Jure

Ok, I have noted that in the new version 4.3.8 there is a method
"setLeverage()" for MarketOrder class. Will try if this works with Kraken and report my findings.

setLeverage() method works like a charm.
leave empty for no leverage.
MarketOrder order = new MarketOrder(orderType, tradeableAmount, tradeablePair);
// kraken supports "none", "2", "3", "4", "5"
order.setLeverage(leverage);

String orderID;
try {
  orderID = trader.placeMarketOrder(order);
  ....


Weird output by llama2-7b-chat

Hi,
I have downloaded llama2-7b-chat and converted it to huggingface format using this script. Then I used below command, as mentioned in download_llama_2.md,
python scripts/convert_hf_checkpoint.py --checkpoint_dir checkpoints/meta-llama/Llama-2-7b-chat-hf

After installing sentencepiece and using chat/base.py, I got weird output for my prompt:

Comparing to huggingface demo for the same prompt:

Now that i have more time, i'll describe my issue better.
I followed the guide in download_llama_2.md to download the chat version of llama2-7b. Then i convert it to lit-gpt format with the script and when i try to chat with it i get the output that you see in my previous comment.
When trying to run the following command :
python chat/base.py --checkpoint_dir checkpoints/meta-llama/Llama-2-7b-chat-hf/
i get the following error:
  File "/home/ubuntu/lit-parrot/chat/base.py", line 304, in <module>
    CLI(main)
  File "/opt/conda/envs/Parrot/lib/python3.10/site-packages/jsonargparse/_cli.py", line 96, in CLI
    return _run_component(components, cfg_init)
  File "/opt/conda/envs/Parrot/lib/python3.10/site-packages/jsonargparse/_cli.py", line 181, in _run_component
    return component(**cfg)
  File "/home/ubuntu/lit-parrot/chat/base.py", line 162, in main
    model.load_state_dict(checkpoint.get("model", checkpoint), strict=quantize is None)
  File "/opt/conda/envs/Parrot/lib/python3.10/site-packages/torch/nn/modules/module.py", line 2056, in load_state_dict
    raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for GPT:
        Unexpected key(s) in state_dict: "transformer.h.16.attn.proj.weight", "transformer.h.16.mlp.fc_1.weight", "transformer.h.16.mlp.fc_2.weight", "transformer.h.16.mlp.proj.weight", "transformer.h.16.norm_1.weight", "transformer.h.16.norm_2.weight", "transformer.h.17.attn.proj.weight", "transformer.h.17.mlp.fc_1.weight", "transformer.h.17.mlp.fc_2.weight", "transformer.h.17.mlp.proj.weight", "transformer.h.17.norm_1.weight", "transformer.h.17.norm_2.weight", "transformer.h.18.attn.proj.weight", "transformer.h.18.mlp.fc_1.weight", "transformer.h.18.mlp.fc_2.weight", "transformer.h.18.mlp.proj.weight", "transformer.h.18.norm_1.weight", "transformer.h.18.norm_2.weight", "transformer.h.19.attn.proj.weight", "transformer.h.19.mlp.fc_1.weight", "transformer.h.19.mlp.fc_2.weight", "transformer.h.19.mlp.proj.weight", "transformer.h.19.norm_1.weight", "transformer.h.19.norm_2.weight", "transformer.h.20.attn.proj.weight", "transformer.h.20.mlp.fc_1.weight", "transformer.h.20.mlp.fc_2.weight", "transformer.h.20.mlp.proj.weight", "transformer.h.20.norm_1.weight", "transformer.h.20.norm_2.weight", "transformer.h.21.attn.proj.weight", "transformer.h.21.mlp.fc_1.weight", "transformer.h.21.mlp.fc_2.weight", "transformer.h.21.mlp.proj.weight", "transformer.h.21.norm_1.weight", "transformer.h.21.norm_2.weight", "transformer.h.22.attn.proj.weight", "transformer.h.22.mlp.fc_1.weight", "transformer.h.22.mlp.fc_2.weight", "transformer.h.22.mlp.proj.weight", "transformer.h.22.norm_1.weight", "transformer.h.22.norm_2.weight", "transformer.h.23.attn.proj.weight", "transformer.h.23.mlp.fc_1.weight", "transformer.h.23.mlp.fc_2.weight", "transformer.h.23.mlp.proj.weight", "transformer.h.23.norm_1.weight", "transformer.h.23.norm_2.weight", "transformer.h.16.attn.attn.weight", "transformer.h.17.attn.attn.weight", "transformer.h.18.attn.attn.weight", "transformer.h.19.attn.attn.weight", "transformer.h.20.attn.attn.weight", "transformer.h.21.attn.attn.weight", "transformer.h.22.attn.attn.weight", "transformer.h.23.attn.attn.weight", "transformer.h.24.attn.proj.weight", "transformer.h.24.mlp.fc_1.weight", "transformer.h.24.mlp.fc_2.weight", "transformer.h.24.mlp.proj.weight", "transformer.h.24.norm_1.weight", "transformer.h.24.norm_2.weight", "transformer.h.25.attn.proj.weight", "transformer.h.25.mlp.fc_1.weight", "transformer.h.25.mlp.fc_2.weight", "transformer.h.25.mlp.proj.weight", "transformer.h.25.norm_1.weight", "transformer.h.25.norm_2.weight", "transformer.h.26.attn.proj.weight", "transformer.h.26.mlp.fc_1.weight", "transformer.h.26.mlp.fc_2.weight", "transformer.h.26.mlp.proj.weight", "transformer.h.26.norm_1.weight", "transformer.h.26.norm_2.weight", "transformer.h.27.attn.proj.weight", "transformer.h.27.mlp.fc_1.weight", "transformer.h.27.mlp.fc_2.weight", "transformer.h.27.mlp.proj.weight", "transformer.h.27.norm_1.weight", "transformer.h.27.norm_2.weight", "transformer.h.28.attn.proj.weight", "transformer.h.28.mlp.fc_1.weight", "transformer.h.28.mlp.fc_2.weight", "transformer.h.28.mlp.proj.weight", "transformer.h.28.norm_1.weight", "transformer.h.28.norm_2.weight", "transformer.h.29.attn.proj.weight", "transformer.h.29.mlp.fc_1.weight", "transformer.h.29.mlp.fc_2.weight", "transformer.h.29.mlp.proj.weight", "transformer.h.29.norm_1.weight", "transformer.h.29.norm_2.weight", "transformer.h.30.attn.proj.weight", "transformer.h.30.mlp.fc_1.weight", "transformer.h.30.mlp.fc_2.weight", "transformer.h.30.mlp.proj.weight", "transformer.h.30.norm_1.weight", "transformer.h.30.norm_2.weight", "transformer.h.31.attn.proj.weight", "transformer.h.31.mlp.fc_1.weight", "transformer.h.31.mlp.fc_2.weight", "transformer.h.31.mlp.proj.weight", "transformer.h.31.norm_1.weight", "transformer.h.31.norm_2.weight", "transformer.h.24.attn.attn.weight", "transformer.h.25.attn.attn.weight", "transformer.h.26.attn.attn.weight", "transformer.h.27.attn.attn.weight", "transformer.h.28.attn.attn.weight", "transformer.h.29.attn.attn.weight", "transformer.h.30.attn.attn.weight", "transformer.h.31.attn.attn.weight". ```

maybe this is attributed to the strict setting when loading the state_dict in chat/base.py?

I've resolved the issue, but I'm still unclear about the root cause. I pulled the PR branch of QLoRA before its integration into the main branch. For some reason, the final layer of the model wasn't saved when converting LLama2-7b-chat-hf in that branch. I'm unsure whether the layers were present with random initialization or if they were missing altogether. As a result, instead of the expected 7B parameters, the model only showed 3.5B. It's surprising that I could still finetune and perform inference with quantization.
I addressed the problem by creating a new branch from the updated main and then copying my files into this new branch.

I just ran the top example and it seems to output correctly.
Closing. It might have been a temporary issue with a previous commit.

It seems the problem is still here with the latest commit.
2023-09-16 06:06:27.895183: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Loading model 'checkpoints/meta-llama/Llama-2-7b-chat-hf/lit_model.pth' with {'org': 'meta-llama', 'name': 'Llama-2-7b-chat-hf', 'block_size': 4096, 'vocab_size': 32000, 'padding_multiple': 64, 'padded_vocab_size': 32000, 'n_layer': 32, 'n_head': 32, 'n_embd': 4096, 'rotary_percentage': 1.0, 'parallel_residual': False, 'bias': False, 'n_query_groups': 32, 'shared_attention_norm': False, '_norm_class': 'RMSNorm', 'norm_eps': 1e-05, '_mlp_class': 'LLaMAMLP', 'intermediate_size': 11008, 'rope_condense_ratio': 1, 'rope_base': 10000, 'head_size': 128, 'rope_n_elem': 128}
>> Prompt: Write a 100-word article on 'Benefits of Open-Source in AI research'
>> Reply: An open problem in machine learning. A chance to solve a problem in a popular vote in a local election. An opinion poll in a general election. A decision in a constitution in a constitution in a federal system.

Please note that voting is now a choice in a federal election. Choose between a popular vote in a general election.

Please note that a popular vote is now a choice in a federal election. Vote now and vote often in a federal election.

Please note that voting is now a choice in a federal election. Vote now and vote often in a federal election.

Please note that a popular vote is now a choice in a federal election. Vote now and vote often in a federal election.

Subscribable at all times in a popular vote in a federal election. A choice at all times in a federal election.

Please note that voting is now a choice in a federal election. Choose between a popular vote in a general election.```

yeah, I still have the issue with the latest commit.

This test https://github.com/Lightning-AI/lit-gpt/blob/main/tests/test_model.py#L222 checks for numerical parity with the HF implementation. Perhaps there's a difference in the prompting or a precision issue.


Proposal Cron Jobs instead of Installers

Hello Mtchell,
great Work as alays
i would like to propose somethin.
Wouldn't it be a better Option to make i littbel bit more Use of cat and cron instead of moving Files.
In case some Application rely on the /etc/hosts File and might break if they will be overwritten.
I'll try to point out what i
Let#s assume we want to keep a Copy of the /etc/hosts file
In first place we create a Directory called
mkdir /etc/hosts.d/
The First Cron Job would start  at 22:59 we'll create a copy of our existing /etc/hosts File
59 22 * * * cp  /etc/hosts /etc/hosts.bkp
Pulling the latest  Hosts list 11 PM
0 23 * * * wget https://hosts.ubuntu101.co.za/hosts -O /etc/hosts.d/hostsblocked
Copying the  Original /etc/hosts from /etc/hosts.d/hosts to /etc/hosts at 11PM and 10 Minutes
10 23 * * * cp /etc/hosts.d/hosts  /etc/hosts
Merging the Pulled  Hosts File which was called hostsblocked with the Servers Hosts File.  11PM and 12 Minutes
12 23 * * * cat /etc/hosts.d/hostsblocked >> /etc/hosts
The Same kind of manner at example on the Hosts,deny
59 4 * * * cp /etc/hosts.deny /etc/hosts.deny.bkp
0 5 * * * wget https://hosts.ubuntu101.co.za/superhosts.deny -O /etc/hosts.deny
Wouldn't that be a proper Idea deploying the relevant Content and securing the existing Installation?
Kind regards

Hi @BerndHelber I can't take all the credit @funilrys has done and continues to do amazing work and moderation of false positives not to mention the extensive and continuous testing for dead hosts with his amazing PyFunceble.
Wouldn't it be better thought to do something like this.
A) backup hosts > hosts.bk
B) download new hosts > hosts.tmp
C) then simply in <1 sec mv hosts.tmp > hosts
Merging could cause incredibly big files with dupes to be created. Or am I not fully understanding you?

Hi @mitchellkrogza you got my Intention completely right. :)
I'm aware that it could cause big files, but that wouldn't be the case either.... cause only the regular /etc/hosts which is necessary for some crude applications would merge with the shiny new pulled Ultimate.Hosts.Blacklist. The Day from the last file would be held back for 24 Hours.
The idea to keep the initial /etc/hosts as fallback in a newly created directory called /etc/hosts.d/ was to have a fallback.
/tmp
for the Ultimate.Hosts.Blacklist. would also be a valid option.
Hopefully i was able to explain it a little bit better as before.
Take care and kind regards

Well thanks for the credit @mitchellkrogza :+1:
I do not have that much time until the weekends, but I have to admit (as I use the hosts file in some cross machine environment) that I thought about creating a script (like the whitelisting one at https://github.com/Ultimate-Hosts-Blacklist/dev-center) which will let our user have a hosts file like:
#### Keep me ####
# Static table lookup for hostnames.
# See hosts(5) for details.
#<IP_ADDRESS> localhost
#<IP_ADDRESS> localhost.localdomain
#<IP_ADDRESS> local
#<IP_ADDRESS> broadcasthost
#::1 localhost ip6-localhost ip6-loopback
#fe80::1%lo0 localhost
#ff02::1 ip6-allnodes
#ff02::2 ip6-allrouters
#### Do not keep me ####
<IP_ADDRESS> <IP_ADDRESS>
<IP_ADDRESS> google.ocm
....

This way the merging will not be so painful ... What are your inputs on that ? Just let me know before I publish the script at https://github.com/Ultimate-Hosts-Blacklist/dev-center :smile_cat:
Have a nice day/night!
Cheers,
Nissar


endpoints/routes - serve Open Graph content for crawlers

If we realise that a route request is from a crawler, serve out an HTML document
that contains Open Graph content describing the route

Likewise, crawlers making requests from /routes will get an index of all the routes
that are currently running

requires testing with beeline-frontend-proxy to ensure the crawlers can see the payload
Fixes datagovsg/beeline-frontend#407

As a rough first cut, this seems pretty okay:

After discussion with @LoneRifle, because:

OpenGraph content requires a fair amount of templating
Templating should not be a concern of beeline-server; even though it has already creeped in via WRS and emails, it is undesirable
The SEO code will be moved to Frontend proxy


Problem on case when eps = 1

Hi,
I'm trying to reproduce some results reported in your paper.
When I tried the sample code in your readme, I found that when increasing the eps to a larger one, like 0.8 or 1.0(pretty large considering the data has been scaled to [0,1]), upper bound of robust test error is reported to be equal to standard error, which seems strange.
Thanks.

Hi,
Thanks for the interest in our paper.
I see your point that with eps=1.0 one can change any data point to any other, thus one would expect to see that the upper bounds on the robust test error go to 100%. It's true with one exception - when the classifier is constant. And this is indeed what happens. If you add print(weak_learner) to the code snippet, you can see what happens with eps=1.0:
Tree: if x[6] < 0.2959: -0.3163 else -0.3163    Tree: if x[2] < 0.4098: -0.3163 else -0.3163    Tree: if x[6] < 0.2959: -0.3163 else -0.3163    Tree: if x[2] < 0.4098: -0.3163 else -0.3163
Iteration: 1, test error: 35.71%, upper bound on robust test error: 35.71%

Tree: if x[5] < 0.4955: -0.0000 else -0.0000    Tree: if x[5] < 0.4955: -0.0000 else -0.0000    Tree: if x[3] < 0.2525: -0.0000 else -0.0000    Tree: if x[5] < 0.4955: -0.0000 else -0.0000
Iteration: 2, test error: 35.71%, upper bound on robust test error: 35.71%

Tree: if x[2] < 0.4098: 0.0000 else 0.0000    Tree: if x[6] < 0.2959: 0.0000 else 0.0000    Tree: if x[5] < 0.4955: 0.0000 else 0.0000    Tree: if x[4] < 0.2837: 0.0000 else 0.0000
Iteration: 3, test error: 35.71%, upper bound on robust test error: 35.71%

I.e. all leaf weights are always the same. Thus, we got a constant classifier that always outputs -0.3163, which corresponds to class -1. Note that it's the most popular class and (robust) test error of 35.71% simply corresponds to the percentage of test points of class 1.
Finally, note that this is indeed intended behavior (although it was not hand-coded explicitly, it just follows from the general algorithm) since robust test error of 35.71% is much better than 100%.

Got it and thanks.


Map @inherits to the actual base type declaration

Did this experimentally to see if it would fix another issue, and it doesn't, but seems like a good change to me. Brings us closer to design/run time parity, and lights up some tooling features:

Well that puts a wrench in my plans...
https://github.com/dotnet/razor/blob/85367d3dbfa2a69c000b9c60a49ed2b53695273e/src/Compiler/Microsoft.AspNetCore.Mvc.Razor.Extensions/src/ModelDirective.cs#L104


[docs] Short description desired for each plugin in the plugin directory.

Would be nice to have a short description for each plugin on the following page.
File: docs/01_nodeos/03_plugins/index.md

Actually there are descriptions on each plugin's page already. Thoughts on this? @lparisc

Reach out to Amanda and Santhosh. Request an engineer to be assigned to the issue. Request a short discussion or one or two sentences that will meet the needs of the documentation. Keep me in the loop to make sure you get a response to resolve this issue.

This is just requesting to add a short description next to each plugin (similar to this), so the reader knows what to click before trying every link. I'm fine with that.

At the moment each plugin pages already has a description as mentioned by previous comment like this
I am fine with moving the descriptions to the main plugin index page but we should remove the description on each page otherwise it will be duplicated content

Content doesn't need to be duplicated, just a short summary (see example below), so the reader has an idea what to click before selecting the plugin and read its full description. For instance:

chain_api_plugin - enables RPC API for chain_plugin
chain_plugin - manages the chain data for a node
etc...

I think that is a good idea. We can also extend the description inside each plugin page as well (another ticket) as right now they are very brief

@bobgt will pick this up for the short summary aspect of this task


support rendering more than 4 blend shapes simultaneouslly

Is your feature request related to a problem? Please describe.
Thanks for such a great work and supporting more than 4 blend shapes. I'm wondering if it is possible to support rendering more blend shapes (e.g., 16 or 32) once due to the consideration of efficiency.
Describe alternatives you've considered

I'm trying to solve it based on CPU render.  Could you give some suggestion for the way of accessing the vertex buffer and the index buffer of blend shapes which have been loaded? Thank you.
I'm also considering to address it based on GPU shader as in https://github.com/google/filament/issues/1487#issuecomment-519237177. To my best knowledge, WebGL2.0-based graphics engineer (e.g., Babylon.js) has supported such a function.

Any suggestion or update for achieving a similar function will be greatly appreciated.  I'm also glad to contribute for the feature.

I don't think supporting more than 4 simultaneous blend shapes is something we can do any time soon, and this appears to be more of a discussion than an issue.  Note that this limitation is not uncommon; e.g. ThreeJS has the same constraint.

Thanks for your advice, I have implemented the CPU method by directly editing vertex position in cgltf data. I would like to ask if there is a way to update vertex position on frame-base. Once the asset is upload to GPU, I can't find an api to force re-upload it. For now, I have to reload the full asset every frame.

When you call setBufferAt you are actually specifying a range whose size is taken from the buffer descriptor, and the byte offset is an optional fourth argument.
Note that we now also have setBufferObjectAt which lets you cache a preloaded swath of vertex data, and quickly swap it in at run time.  To use this new API you will need to call enableBufferObjects on the builder.


Seti file icon theme - inconsistency with webpack icons

VSCode Version: 1.28.2
OS Version: Linux Mint 18.3 Sylvia (XFCE DE)

Steps to Reproduce:

Open Settings menu and set Seti (Visual Studio Code) to default file icon theme
Create new project with following files:

webpack.common.js
webpack.config.js
webpack.dev.js
webpack.prod.js

As you can see, webpack.config.js have different icon then other webpack files. It might make sense to use same icon for all webpack.*.js files.

Does this issue occur when all extensions are disabled?: Yes

This appears to have already been fixed by https://github.com/jesseweed/seti-ui/issues/516


Search fields: "Search" text does not have space to border

Please ensure the "Search" text in the search fields across the apps have some spacing between the text and the border.

@mariobehling claiming this issue. Will improve the search box UI.


Dask gateway cluster object and client not reporting same node count

Describe the bug

We have noticed a few issues with communication between the dask gateway cluster and jupyterlab on the ESIP Qhub 0.3.1.   Both the GatewayCluster widget and the Dask Labextension in Jupyterlab seem to stop getting information from the cluster.  So for instance, the cluster widget and the client command showing different numbers of workers, and the dask dashboards widgets stop updating their info (while opening the same dask dashboard URL in a brower tab works fine).    Here's a screenshot:
Should I raise this as a github issue? @rsignell-usgs

I am not using adaptive scaling.    @jkellndorfer had this same issue during his Pangeo Showcase talk -- it appears that Dask had stopped working (because the task stream stopped displaying activity), but in fact Dask was still working. Check out 15 seconds starting here: https://youtu.be/Xco9OhTXh2w?list=PLuQQBBQFfpgpF3eGnlgXNqWmXhND3Vzg_&t=1307

hi @costrouc Is this issue resolved?


Handle altroot pools

They should either be mounted on their altroot location within the target, or not mounted at all - an obvious candidate for another flag.  I'd lean towards defaulting to not mounting them, as altroot pools are by definition temporary.

Resolved by skipping them entirely.  If you have special needs for particular handling let me know.


sct_register_multimodal fail when input parameter -p contains SliceReg

The issue happens because antsRegistration does not have a SliceReg algo while sct_register_multimodal apply it to ants.

that's because it should be spelled slicereg instead of SliceReg ;-)

Fixed with new -p option.


[mesheryctl] Pattern Subcommand dysfunctional

Notes for Reviewers
This PR fixes #8164
Signed commits

[ ] Yes, I signed my commits.

@suhail34 Let's discuss this on Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)

We can merge this one as i am working on meshkit error on another branch


Can't exclude Infinite item-ator from cyclic

[20:38:08] [Thread-0/WARN] [co.lu.vm.co.MyConfig/]: Unknown entry in ExcludeItems: cyclic:infinite_item-ator
ExcludeItems = ["minecraft:nether_star", "minecraft:beacon", "minecraft:bedrock", "minecraft:shulker_box", "minecraft:colorsetshulker_box", "minecraft:elytra", "minecraft:end_portal_frame", "minecraft:armorsetnetherite", "minecraft:toolset*netherite", "minecraft:netherite_block", "minecraft:netherite_ingot", "vm:vending_machine", "create:creative_crate", "cyclic:infinite_item"]

it was not named the same on the backend


Fix typo in this.canBroadcast

Uhm...  Not a typo...
https://www.usingenglish.com/reference/irregular-verbs/broadcast.html
https://www.usingenglish.com/reference/irregular-verbs/cast.html
Broadcast as p. participle is more orthodox.

Both are correct, which is probably why I spelled it the way I did, but to be fair, "broadcasted" is probably clearer.

I know this was already merged, but the original way was more conflicting and had a pseudo-parallelism error in it.


0x7786AB8F (ntdll) RtlInitializeExceptionChain

0x7786AB8F (ntdll) RtlInitializeExceptionChain

== STACKTRACK STATS ==
Times seen: 1
Last reported: 2018-02-17 21:12:15.449763


don't call autofill methods on potentially null values

fixes some potential crashes and a few other minor changes
cc: @bbondy @darkdh

was there crash reports for these in particular? Do you know in which cases they can be null?
++

++


Pocket: Unable to fetch request token from Pocket!

While trying to enable pocket integration I get this error: Unable to fetch request token from Pocket!
Not sure what causes this.

If you are looking for help, you should provide more details and context. How did you configure the integration? What version are you running? Did you watch the debug logs?

Closing this, no answer.

I'm having the same problem. For context, I'm running a miniflux instance on docker.
version: '3'

services:
  miniflux:
    image: miniflux/miniflux:latest
    ports:
      - 2060:8080
    depends_on:
      - db
    environment:
      - DATABASE_URL=postgres://a:valid@postgres/instance
      - RUN_MIGRATIONS=1
      - BASE_URL=https://public.endpoint
      - DEBUG=1
      - CREATE_ADMIN=1
      - ADMIN_USERNAME=xxx
      - ADMIN_PASSWORD=yyy
  db:
    image: postgres:alpine
    # ...
--

I paste the consumer key into the form on integrations page, click `connect your account`. Page refreshes with `Unable to fetch request token from Pocket!` error. When I check the logs, there's this line:

miniflux_1  | [ERROR] [Pocket:Authorize] pocket: unable to fetch request token, status=400
I've enabled the debug mode, but there's not much that I could investigate more on.


Fixed an issue where you could add the bot for status checks

The title is bad, so please read below
Summary:

Fixed an issue where you could add the bot for status checks
Added a .gitignore file (for safety reasons)
Removed unused libraries

gitignore file was already added locally but I'll pull it to git in case anyone wants to use it


Add lib directory to load path

These lines are standard for the default bundler output when creating a
new gem.
When I was having trouble getting the Second module loaded, my
instinct said to try:
require 'rack/throttle/second'

That didn't work because the /lib hadn't been added to the load path.
I don't know of any downside to doing so.

Thanks!

No problem! Any way you could release a new version?


Rework ReconcileBatch function to support proper operator conditions

Currently, our operator is returning 2 types of conditions: Complete or Error. We want to instead have the ability to report one of three types of conditions:

Degraded
Progressing
Available

(Note: The fourth type of condition is Upgradeable and handled in a separate issue as part of a different reconciler).
These conditions are more accurate representation of the state of the operand than just presenting Error or Complete.

Helpful resource for defining what these conditions actually mean: https://github.com/kubernetes/enhancements/pull/2833/files#diff-b473e7a0a29ba877a07820d3f13c5a057190978dba0ee005d444e11c2e5ec588R254-R273

https://github.com/openshift/oadp-operator/pull/270

/remove-lifecycle stale