[5.8] Flexible Assert Database Helper

Current functionality
$this->assertDatabaseHas('products', $data);
$this->assertDatabaseMissing('products', $data);
$this->assertSoftDeletes('products', $data);
Added functionality
$this->assertDatabaseHas(Product::class, $data);
$this->assertDatabaseHas(new Product, $data);
$this->assertDatabaseMissing(Product::class, $data);
$this->assertDatabaseMissing(new Product, $data);
$this->assertSoftDeletes(Product::class, $data);
$this->assertSoftDeletes(new Product, $data);

So what is the win here? Specifying table names is still shorter and tables won't change their names in your project at all. I don't think it's beneficial in any way to pollute database assert helpers with Eloquent just for the sake of it.

I agree with @d3jn. I don't see any practical benefit here. You're performing a DB assertion and not an ORM assertion. It's a given that these methods assert certain data against a specific table.

tables won't change their names in your project at all

Well, that's presumptious

@deleugpn it indeed is and I think that's not entirely what @d3jn means. Actually specifying the tables names will indeed help you to identify bugs when table names change.

So what is the win here? Specifying table names is still shorter and tables won't change their names in your project at all. I don't think it's beneficial in any way to pollute database assert helpers with Eloquent just for the sake of it.

well the assertSoftDeleted already does this half way by default, https://github.com/laravel/framework/blob/5.8/src/Illuminate/Foundation/Testing/Concerns/InteractsWithDatabase.php#L59
It is fairly convenient and of course everything is still backward compatible.


[Event Request] Codeunit 80 "Sales-Post" – Existing event “OnBeforeReturnRcptLineInsert”

Could we please have the existing event “OnBeforeReturnRcptLineInsert” in function “InsertReturnReceiptLine”, Codeunit 80 "Sales-Post”, be expanded by additional parameter?
    local procedure InsertReturnReceiptLine(ReturnRcptHeader: Record "Return Receipt Header"; SalesLine: Record "Sales Line"; CostBaseAmount: Decimal)
    var
        ReturnRcptLine: Record "Return Receipt Line";
        WhseShptLine: Record "Warehouse Shipment Line";
        WhseRcptLine: Record "Warehouse Receipt Line";
    begin
        OnBeforeInsertReturnReceiptLine(SalesLine, ReturnRcptLine, RemQtyToBeInvoiced, RemQtyToBeInvoicedBase);
…
              Round(CostBaseAmount / SalesLine.Quantity * ReturnRcptLine.Quantity);
        end;
*** start old code ***
        //OnBeforeReturnRcptLineInsert(ReturnRcptLine, ReturnRcptHeader, SalesLine, SuppressCommit);
*** stop old code ***
*** start new code ***
        OnBeforeReturnRcptLineInsert(ReturnRcptLine, ReturnRcptHeader, SalesLine, SuppressCommit, xSalesLine, TempSalesLineGlobal);
*** stop new code ***
        ReturnRcptLine.Insert(true);
        OnAfterReturnRcptLineInsert(
…
    [IntegrationEvent(false, false)]
*** start old code ***
    //local procedure OnBeforeReturnRcptLineInsert(var ReturnRcptLine: Record "Return Receipt Line"; ReturnRcptHeader:
    //Record "Return Receipt Header"; SalesLine: Record "Sales Line"; CommitIsSuppressed: Boolean)
*** stop old code ***
*** start new code ***
    local procedure OnBeforeReturnRcptLineInsert(var ReturnRcptLine: Record "Return Receipt Line"; ReturnRcptHeader: Record "Return Receipt Header"; SalesLine: Record "Sales Line"; CommitIsSuppressed: Boolean; xSalesLine: record "Sales Line"; var TempSalesLineGlobal: record "Sales Line" temporary)
*** stop new code ***
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


ActionView: set Nokogiri::Document encoding to match the string

Address failing tests related to encoding.

@seanpdoyle The remaining failures on this branch are all related to entity-escaping within attributes. Unfortunately, libxml2 does not provide any ability to configure this behavior.
A related comment from a past issue explains this in a bit more detail:
https://github.com/sparklemotion/nokogiri/issues/1920#issuecomment-523941906
But the TL;DR is that href, action, src, and name (but only within an anchor) are always escaped when generating HTML -- basically, anything that could be a URI reference.

Thank You!

Maybe worth noting: reliance on an attribute string being "foo<" instead of "foo<" feels unlikely to be a meaningful/important blocker for adoption, since these are semantically identical in an HTML document. So we could just update the tests to take this into account and see if Rails is interested in accepting such a change upstream.
However, my experience has shown that this will realistically end up being an incidental blocker for teams who, for example, might have view tests that are asserting on the presence of unescaped entities.
This situation hints at a long-missing aspect of markup testing, which is that asserting against string constants is error-prone and brittle, and it would be better to have a testing framework that asserted on semantic equivalence. There have been a few attempts at addressing this over the years:

https://github.com/postmodern/nokogiri-diff
https://github.com/vkononov/compare-xml


Support parsing char array string in emitted code

The PR adds support for parsing char array string in emitted code.

First commit adds test profile data with char array string
Second commit adds support for parsing char array string in emitted code

Thank you!


Uncaught Error: ENOENT: no such file or directory, lstat '/home/psingh/atom:'

[Enter steps to reproduce below:]

...
...

Atom Version: 1.7.2
System: Ubuntu 14.04.4
Thrown From: fuzzy-finder package, v1.0.5
Stack Trace
Uncaught Error: ENOENT: no such file or directory, lstat '/home/psingh/atom:'
At ATOM_SHELL_ASAR.js:0

Error: ENOENT: no such file or directory, lstat '/home/psingh/atom:'
    at Error (native)
    at fs.lstatSync (fs.js:887:18)
    at Object.fs.lstatSync (ATOM_SHELL_ASAR.js:229:16)
    at Object.realpathSync (fs.js:1532:21)
    at Object.fs.realpathSync (ATOM_SHELL_ASAR.js:304:29)
    at /usr/share/atom/resources/app.asar/node_modules/fuzzy-finder/lib/path-loader.js:19:21
    at Array.map (native)
    at Object.module.exports.startTask (/usr/share/atom/resources/app.asar/node_modules/fuzzy-finder/lib/path-loader.js:17:46)
    at ProjectView.module.exports.ProjectView.runLoadPathsTask (/usr/share/atom/resources/app.asar/node_modules/fuzzy-finder/lib/project-view.js:190:46)
    at ProjectView.module.exports.ProjectView.populate (/usr/share/atom/resources/app.asar/node_modules/fuzzy-finder/lib/project-view.js:111:21)
    at ProjectView.module.exports.ProjectView.toggle (/usr/share/atom/resources/app.asar/node_modules/fuzzy-finder/lib/project-view.js:87:14)
    at atom-workspace.atom.commands.add.fuzzy-finder:toggle-file-finder (/usr/share/atom/resources/app.asar/node_modules/fuzzy-finder/lib/main.js:9:46)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app.asar/src/command-registry.js:260:29)
    at /usr/share/atom/resources/app.asar/src/command-registry.js:3:61
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/usr/share/atom/resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:580:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/usr/share/atom/resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:388:22)
    at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeyEvent (/usr/share/atom/resources/app.asar/src/window-event-handler.js:98:36)
    at HTMLDocument.<anonymous> (/usr/share/atom/resources/app.asar/src/window-event-handler.js:3:61)

Commands
     -3:38.9.0 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -3:38.9.0 core:cancel (atom-text-editor.editor.mini.is-focused)
     -3:38.2.0 markdown-preview:toggle (a.inline-block)
     -2:40.2.0 command-palette:toggle (atom-text-editor.editor.is-focused)
     -2:38.3.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -2:38.3.0 application:open-your-config (atom-text-editor.editor)
     -2:16.6.0 command-palette:toggle (atom-text-editor.editor.is-focused)
     -2:13.1.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -2:13.1.0 settings-view:install-packages-and-themes (atom-text-editor.editor)
     -2:08.3.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:34.4.0 core:select-all (atom-text-editor.editor.mini.is-focused)
     -0:31.4.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:17.9.0 pane:split-right (a.icon.icon-settings)
     -0:06.4.0 command-palette:toggle (atom-pane.pane.active)
     -0:05.4.0 pane:close (div.item-views)
     -0:00.6.0 fuzzy-finder:toggle-file-finder (ul.list-inline.tab-bar.inset-panel)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "atom-dark-syntax"
    ]
  }
}

Installed Packages
# User
minimap, v4.22.1

# Dev
No dev packages

I filed #199, the way I can make this happen on demand is to have atom open a project on a network drive and then forcibly disconnect the mount.  Basically every time I open my laptop when I start work, Atom crashes.

This one's a bit troubling, at least per @rossburton's repro steps, because it seems to boil down to a more general question about "what should Atom do if the entire project disappears from the filesystem?"
We can't do any actions to a project that we can't access, so all the things in atom that assume the ability to read or stat files won't work. Maybe we can add a concept of a "disabled" project and suspend any interaction with it until the path exists again?
In the short term, we can probably just catch this exception in fuzzy finder and show something in the info area

Don't disable the entire project: if someone has unsaved changes and the
network drive/removable media/whatever suddenly ceases to exist, you'd
still want to save the file somewhere else.
On Monday, 2 May 2016, Daniel Hengeveld<EMAIL_ADDRESS>wrote:

This one's a bit troubling, at least per @rossburton
https://github.com/rossburton's repro steps, because it seems to boil
down to a more general question about "what should Atom do if the entire
project disappears from the filesystem?"
We can't do any actions to a project that we can't access, so all the
things in atom that assume the ability to read or stat files won't work.
Maybe we can add a concept of a "disabled" project and suspend any
interaction with it until the path exists again?
In the short term, we can probably just catch this exception in fuzzy
finder and show something in the info area
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/atom/fuzzy-finder/issues/196#issuecomment-216224451

As of v1.7 we periodically store the editor state in IndexedDB for crash recovery, so unsaved changes are technically saved in Atom's client-side storage. I have not tested this, but I believe that once the network drive is reconnected and an editor is opened with the same project paths the editor should be restored to the state it was in before the crash, including any unsaved buffer changes


Base Factory class to build objects based Environment

In my projects, I basically have the following class copied/pasted everywhere
/**
 * Factory class for building returnable types provided the current {@code Environment}.
 *
 * @see {link Environment}
 */
@JsonTypeInfo(use = JsonTypeInfo.Id.NAME, property = "type")
public interface EnvironmentProvidedFactory<T> extends Discoverable {

    T build(Environment environment);
}

I was looking at refactoring it to a shared library on our stack but wanted to ask here first.  Would this be useful for possibly others if this was added to dropwizard proper?
We have found it extremely useful for configuring bundles using this approach so we have a standard interface they all implement and can register as a type to be easily loaded and swapped out in the YAML.

I think it's an interesting approach for building reusable functionality, we could consider it in the core. Could you submit a pull request (just some sketch)?

i'll give something a shot, maybe this weekend.


Fix link to https://github.com/TuringLang/TuringTutorials/blob/master/10_diffeq.ipynb

This is a somewhat timely manner. Can this link be resurrected for just about a month? Thanks.

Fixed by 23f06ba87bb7d6dd77d3467a75d27258194b8738.
Let's remember to remove this in the future.

@ChrisRackauckas Sorry, just saw your message now. If you want to make the link persistent, you could link to a specific commit. With the switch to markdown tutorials, most links to files in the master branch will become invalid, I assume.

It's fine. It's just on the day of a paper release I don't want a broken link since Twitter doesn't allow editing. In a month we can remove this, but for now it's going to get a ton of traffic and "doing it well" is probably better than "doing it right"

I see :slightly_smiling_face: I'm sorry about the inconvenience it caused, I didn't know that you had distributed a link to the repo when fixing the webpage.

No worries, it's all good in the end. That's just my impatience to make sure the paper hit the optimal release timing.

Is it okay if we delete the file now? We're moving to a new format which doesn't use Jupyter notebooks, and so ideally we'd not have to keep the old ones around. We'll also add in redirects, so in the future always link to the website and the underlying notebook can be found from there:)

yes, go for it.


Create design docs

Coalesce results of stakeholder conversations into GH discussions

GH discussions don't seem to be implemented, so I created a google doc here:
https://docs.google.com/document/d/1mFVoUUg85e_8XEL-w3B9S0JnGWioO8-H6t55fZAoVKw/edit?tab=t.0#heading=h.f7w6hnf6b4e


Implement ColorDepthEffect

Description
Implement a ColorDepthEffect.
Tasks

[ ] Create ColorDepthEffect.
[ ] Create color-depth demo.
[ ] Add unit test.

Implementation Details

The implementation from v6 can be copied mostly as-is.

Hello, I'd like to take a look at this one if that's ok please?

Sure, have fun 👍

Great thanks @vanruesc
I have opened a PR here: https://github.com/pmndrs/postprocessing/pull/604 If it's generally the right shape let me know, if not, no worries :)


Is this repo being maintained by someone?

Hi, I think you can achieve pagination directly with mongoose without effort right now:
Model                 .find(query)                 .skip(pagination.skip)                 .limit(pagination.limit)                 .select(fields)                 .sort(sort)

@it-nalon How can you achieve total and pages directly in mongoose?

total would be either countDocuments (if the result set is small) or estimatedDocumentCount. You can calculate pages yourself.


Trinity Continuum 2e by Mercury Cluster

Changes / Comments
Character Sheet Submission
Storypath -> Trinity Continuum 2e by Mercury Cluster
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


update vstshost

Looks good.  I assume /2 is there :)

@bryanmacfarlane  yeah, uploaded, tested. :D


Lazily load reflections in ClassAliasResolver

Currently the reflections scan happens in ClassAliasResolver even if a job does not use @Alias. This change is to use https://en.wikipedia.org/wiki/Initialization-on-demand_holder_idiom to load reflections only when ClassAliasResolver is used.
@chavdar can you review?

@ibuenros mentioned the desired behavior is already there. The static field REFLECTIONS will only get initialized when ClassAliasResovler is used.
Discarding this  PR.

Coverage decreased (-0.07%) to 49.01% when pulling c00d1e74796456b2c65a60834d2993012340ad8e on pcadabam:master into 7120a3e58d45409866cde532284e586bb41bd105 on linkedin:master.


feature Allow BarrierLog to wrap another binlog

to make it work in the old Doris version without breaking compatibility

run buildall

run buildall

run buildall


Add examples directory

If we can write examples for all these (don't have to actually work)
I think we can be confident in our API design!

@at-wat I will work on implementing these. I only have phone access for the next week, so harder to write code :(


⚠ ClamAV has degraded performance

In 476e1e3, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 80 ms

Resolved: ClamAV performance has improved in a77c589.


Action system doesn't seem to work with hotkeys

I tried using on the live server a few times but it only ever worked when I clicked it rather than used 1-9. @chairbender

Fixed by https://github.com/space-wizards/space-station-14/commit/ce029f461e16d5231a81877a7702684adac41cc8


Invalid Cloud deployment of Azure Service Bus topics - name truncation

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Issue
Topics names are truncated in bicep scripts based on fluent distributed application definition.
Code
var builder = DistributedApplication.CreateBuilder(args);

var serviceBus = builder.AddAzureServiceBus("serviceBus");

serviceBus.AddTopic("device-connection-state-events", ["device-management"]);
serviceBus.AddTopic("device-lifecycle-events", ["device-management"]);
serviceBus.AddTopic("device-twin-change-events", ["device-management"]);

...

Bicep script from temp
resource serviceBusTopic_*** 'Microsoft.ServiceBus/namespaces/topics@2021-11-01' = {
  parent: serviceBusNamespace_***
  name: 'device-connection-state-'
  location: location
  properties: {
  }
}

and
resource serviceBusTopic_*** 'Microsoft.ServiceBus/namespaces/topics@2021-11-01' = {
  parent: serviceBusNamespace_***
  name: 'device-twin-change-event'
  location: location
  properties: {
  }
}

Expected Behavior

Topic is defined correctly up to 260 chars: https://learn.microsoft.com/en-us/azure/templates/microsoft.servicebus/namespaces/topics?pivots=deployment-language-bicep#property-values

Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version info
.net 8
Anything else?
No response

cc @tg-msft


CI: verify python example syntax without running them (compile them?)

Closed by https://github.com/iotaledger/iota-sdk/pull/1003


fixing the slugs created by github-slugger

By default this package will version it. But we don't need that

As you can see in the description of the package used here: https://github.com/Flet/github-slugger
The default (and suggested) behaviour is the version the slug that's been created if it's being repeated. However, as we are using this documentation wide it doesn't make sense. As there will be sub-sections there that share a name.
Using the function they mention at the bottom of their readme fixes this.
This works with the assumption that one section won't have 2 headings that are copies of each other. Which should be the case any way


Issues relating to Design Matrix

Here are some issues I have run into and want to think more about:

Downsampled timeseries are shifted with respect to the hires timeseries. This is because the downsampling also shifts the predicted response forward to correspond to the middle of the TR. This makes some sense but also might be surprising. I want to rethink whether this should be done and whether it is being done in the most obvious way. Further, need to think about this in the context of an FIR model.

Should the default oversampling over the design matrix/hrf kernel be changed? It inherited 16x oversampling from FSL but that means it is hard to specify very short events with a given duration. I think it might make sense to change the parameterization to give a hires time bin (not relative to TR) and make the default 60 to correspond to the refresh rate of a typical monitor. Stimulus events will generally be limited by that resolution.

Needing to give the HRF functions a hires stimulus to get a simple predicted response is confusing outside of the context of a DesignMatrix.

A DesignMatrix with high-pass filtering but demean=False will have a zero mean because the filter removes a constant trend. This is confusing. Do we want to add the column means back in after filtering and then allow the presence of a constant trend be determined by demean?

Project archived; improved design matrix code is in lyman.


Precision

When read_json is called on some json objects, the precision of the number inputted is changed, which causes some tests to fail.

In a test we wrote, the input json object is displayed below.
"test_key_2": {         "value": 10.00,         "units": "mg/L" },
In the test, we check to make sure that calling read_json on this json object is equivalent to "10.00 mg/L". However, an error occurs, AssertionError: '10.0 mg/L' != '10.00 mg/L'.  This displays that the input json object is not being converted with the proper precision. It seems as though calling
str(parsed_json[json_key])
in utils.py changes the precision of the number specified by parsed_json[json_key].

This is an interesting issue. You are currently saving the value as a float, thus strange float magic in python land happens. One option is to save "10.00" as a string (surround with ""). And the other thing we just recently learned about AIDE is that there will not be two values to specify an amount (units and value). Instead, there will be a single string with the units appended between a space. Example: "10 in" or "24 m" or "1.533 m^3/s". These are then parsed directly by fusion. This means that this is now how you can assume the values are given. To make potential unit conversions more simple, you should use a units library, specifically pint (this is the library the rest of AguaClara uses.) Therefor you eed to make a function that converts these strings to valid pint objects. Next, conversion is as simple as a single function call. My recommendation for now, however, is to ignore any of this, and just know that measurements are given in string form. Assume this is a readable, correct-unit string form!


Deprecate canonical Multilingual Librispeech

Deprecate canonical Multilingual Librispeech in favor of the community one which supports streaming.
However, there is a problem regarding new ASR template schema: since it's changed, I guess all community datasets that use this template do not work with new version of the library, including MLS. Should we somehow notify users about that or is it possible to change this line ourselves? For MLS specifically, I cannot change the code directly as I'm not the member of the Facebook org.
Hm, and the code should be change after the release, no?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Yes, as discussed in #4006 we should update facebook/multilingual_librispeech indeed before we do a release. @anton-l could you help taking care of updating facebook/multilingual_librispeech ? We need to update the task template
task_templates=[AutomaticSpeechRecognition(audio_column="audio", transcription_column="text")],

and write that datasets>=2.1 is necessary to load it in the dataset card.
Once the change is done we can merge this PR and do the release I think

@polinaeterna @lhoestq
Updated the script and the dataset card: https://huggingface.co/datasets/facebook/multilingual_librispeech

@anton-l @lhoestq now previewer doesn't work for this datasets as it cannot recognize new audio_column argument:

I'm not an expert in previewer things, where should I look into the corresponding code?

Yes, there are several datasets with the same error, eg https://github.com/huggingface/datasets-preview-backend/issues/188. I'm not sure what I should do to fix this? Upgrade datasets to master?

@anton-l ended up removing the task template in facebook/multilingual_librispeech to make it work for old version of datasets and fix the viewer :) thanks !

@lhoestq can we merge now? ^^


Brueckner self-consistency broken for open-shell systems

Brueckner orbital self-consistency has not been written in a way that supports open shell systems where the number of occupied alpha and beta electrons is inequal, as per https://github.com/BoothGroup/ebcc/issues/41#issuecomment-1929989963.

Hej!
It seems to work now. However, I am puzzled a bit why DF-CC does not support single precision. Specifically, an input given below doesn't work:
import numpy as np
from pyscf import gto, scf

from ebcc import UEBCC
from ebcc.precision import single_precision

# Define the molecule using PySCF
mol = gto.Mole()
mol.atom = "H 0 0 0; F 0 0 1.1"
mol.basis = "cc-pvdz"
mol.charge = 1
mol.spin = 1
mol.build()

# Run a UHF calculation using PySCF
mf = scf.UHF(mol)
mf = mf.density_fit()
mf.kernel()

# Run a CCSD calculation
with single_precision():
     ccsd = UEBCC(mf, ansatz="CC2")
     ccsd.kernel()

It fails with the following error message:
  File "ebcc/ebcc/uebcc.py", line 152, in init_amps
    tn[comb] = eris[comb_t][key_t].swapaxes(1, 2) / self.energy_sum(key, comb)
  File "ebcc/ebcc/cderis.py", line 186, in __getattr__
    e1 = getattr(v1, key[:2])
  File "ebcc/ebcc/cderis.py", line 94, in __getattr__
    block = ao2mo._ao2mo.nr_e2(
  File "lib/python3.10/site-packages/pyscf/ao2mo/_ao2mo.py", line 160, in nr_e2
    assert (mo_coeff.dtype == numpy.double)

In my eye, it is natural to combine DF and single precision to speedup CC calculations further.
With best regards,
Victor

Hej!
It seems to work now. However, I am puzzled a bit why DF-CC does not support single precision. Specifically, an input given below doesn't work:
import numpy as np
from pyscf import gto, scf

from ebcc import UEBCC
from ebcc.precision import single_precision

# Define the molecule using PySCF
mol = gto.Mole()
mol.atom = "H 0 0 0; F 0 0 1.1"
mol.basis = "cc-pvdz"
mol.charge = 1
mol.spin = 1
mol.build()

# Run a UHF calculation using PySCF
mf = scf.UHF(mol)
mf = mf.density_fit()
mf.kernel()

# Run a CCSD calculation
with single_precision():
     ccsd = UEBCC(mf, ansatz="CC2")
     ccsd.kernel()

It fails with the following error message:
  File "ebcc/ebcc/uebcc.py", line 152, in init_amps
    tn[comb] = eris[comb_t][key_t].swapaxes(1, 2) / self.energy_sum(key, comb)
  File "ebcc/ebcc/cderis.py", line 186, in __getattr__
    e1 = getattr(v1, key[:2])
  File "ebcc/ebcc/cderis.py", line 94, in __getattr__
    block = ao2mo._ao2mo.nr_e2(
  File "lib/python3.10/site-packages/pyscf/ao2mo/_ao2mo.py", line 160, in nr_e2
    assert (mo_coeff.dtype == numpy.double)

In my eye, it is natural to combine DF and single precision to speedup CC calculations further.
With best regards, Victor

yep that's another bug

Should be fixed by #47. Thanks for identifying so many edge cases @vvp-nsk, it's much appreciated.
A couple notes:

Density fitting has another bug if your PySCF calculation uses out-of-core integrals (i.e. for large systems). If you encounter this then the workaround is to set the max_memory higher so that the integrals are incore, as per #48.
I'm not sure of the validity of Brueckner self-consistency for single precision, the matrix logarithm seems to be poorly conditioned -- I've cast the output to a real valued matrix for now, again I'm not an expert on this so I'm not sure the best approach here.

Again, let me know if you catch anything else :smile:

Hej!
Thanks for the prompt bug fixing, indeed. I do have a challenging example when Bruckner CC implementation doesn't work in EBCC while PySCF works correctly. Let me narrow the testbed to a tiny problem size first.
I also have a general question. For CC methods of practical interest, such as CC2 and CCSD, both right and left CC eigenvectors are implemented in ECBB. Since 1pdm and 2pdm are available, would it be possible then to implement an expectation value of <S^2>  as it can be done in PySCF via call to the pyscf.fci.spin_op.spin_square_general function?
Thank you very much!
With best regards,
Victor

I also have a general question. For CC methods of practical interest, such as CC2 and CCSD, both right and left CC eigenvectors are implemented in ECBB. Since 1pdm and 2pdm are available, would it be possible then to implement an expectation value of <S^2> as it can be done in PySCF via call to the pyscf.fci.spin_op.spin_square_general function?

There is some support of the EOM eigenvectors, yes -- but they're not needed to calculate the 1DM and 2DM, which are also separately available for both CC2 and CCSD. Be sure to call solve_lambda first to solve the lambda equations. I think it should be as easy as this to use that PySCF function:
import numpy as np
from pyscf import gto, scf
from ebcc import UEBCC

mol = gto.M(
    atom="Be 0 0 0; H 0 0 2",
    basis="cc-pvdz",
    spin=1,
    verbose=0,
)

mf = scf.UHF(mol)
mf.kernel()

uebcc = UEBCC(mf, ansatz="CC2")
uebcc.kernel()
uebcc.solve_lambda()

dm1 = uebcc.make_rdm1_f()
dm2 = uebcc.make_rdm2_f()

from pyscf.fci.spin_op import spin_square_general
ovlp = np.eye(mol.nao)
mo_coeff = (np.eye(mol.nao),) * 2
print(spin_square_general(dm1.aa, dm1.bb, dm2.aaaa, dm2.aabb, dm2.bbbb, mo_coeff, ovlp=ovlp))

where the overlap and mo_coeff are set to identity, since these DMs are calculated in the (orthogonal) MO basis.
I should note that I have an open issue about a possible hiccup in my CC2 implementation (#9), which I haven't been able to resolve, so you should verify any results that use the CC2 lambda equations or DMs. This (possible) issue doesn't extend to CCSD, though.

This should do the trick:
import numpy as np
from pyscf import gto, scf
from ebcc import UEBCC, Space
from pyscf.fci.spin_op import spin_square_general

mol = gto.M(
    atom="Be 0 0 0; H 0 0 2",
    basis="cc-pvdz",
    spin=1,
    verbose=0,
)

mf = scf.UHF(mol)
mf.kernel()

frozen_a = np.zeros_like(mf.mo_occ[0])
frozen_a[:1] = 1
frozen_a = frozen_a.astype(bool)
frozen_b = np.zeros_like(mf.mo_occ[1])
frozen_b[:1] = 1
frozen_b = frozen_b.astype(bool)

space_a = Space(
        mf.mo_occ[0] > 0,
        frozen_a,
        np.zeros_like(mf.mo_occ[0]),
)
space_b = Space(
        mf.mo_occ[1] > 0,
        frozen_b,
        np.zeros_like(mf.mo_occ[1]),
)
space = (space_a, space_b)

uebcc = UEBCC(mf, ansatz="CC2", space=space)
uebcc.kernel()
uebcc.solve_lambda()

dm1_act = uebcc.make_rdm1_f()
dm2_act = uebcc.make_rdm2_f()

ovlp = np.eye(mol.nao)
mo_coeff = (np.eye(mol.nao),) * 2

dm1_a, dm1_b = mf.make_rdm1(mo_coeff, mf.mo_occ)
dm2_aa, dm2_ab, dm2_bb = mf.make_rdm2(mo_coeff, mf.mo_occ)

dm1_a[np.ix_(space_a.correlated, space_a.correlated)] = dm1_act.aa
dm1_b[np.ix_(space_b.correlated, space_b.correlated)] = dm1_act.bb

dm2_aa[np.ix_(space_a.correlated, space_a.correlated, space_a.correlated, space_a.correlated)] = dm2_act.aaaa
dm2_ab[np.ix_(space_a.correlated, space_a.correlated, space_b.correlated, space_b.correlated)] = dm2_act.aabb
dm2_bb[np.ix_(space_b.correlated, space_b.correlated, space_b.correlated, space_b.correlated)] = dm2_act.bbbb

print(spin_square_general(dm1_a, dm1_b, dm2_aa, dm2_ab, dm2_bb, mo_coeff, ovlp=ovlp))

It finds the bare (uncorrelated) 1DM and 2DM using the mean-field function in PySCF, and then embeds the correlated (non-frozen) part using the boolean masks in the Space classes. Not the prettiest -- I'll aim to add a with_frozen flag to the make_rdm1_f and make_rdm2_f functions, because this is definitely of general interest.

Hej!
Thank you very much for your assistance and support!
Attached please find a problem reproducer.  The UBCCD malfunctions at the 2nd iteration:
Converged.
E(corr) = -0.6577317650
E(tot)  = -2519.1924758110
Brueckner options:
 > e_tol:  1e-06
 > t_tol:  1e-05
 > max_iter:  20
 > diis_space:  12
Solving for Brueckner orbitals.
Iter   Energy (corr.)  Converged        Δ(Energy)             |T1|
   0    -0.6577317650       True
   1     0.0000000000       True          0.65773                0

The problem  is not related to certain CC ansatz and remains with CC2 as well.
bccd_incorrect.zip
With best regards,
Victor

I'm struggling to get your chkfile to work on my end -- can you instead send the script including the full Mole setup and UHF calculations?

Hej!
Sorry for the late response.  Indeed, there was one file missing. I just re-uploaded testbed and now it should work out.
bccd_incorrect.zip
With best regards,
Victor

I think this should be fixed on #49. Sorry this one took a bit longer!
CC2 seems to converge well for this system but CCSD is a little trickier. I think doing BCCD in single precision will be ambitious, numerically speaking, if that was your plan.

Hej!
Everything works like a charm now. Thank you!
My guess that often SP would be sufficient to get Brueckner orbitals converged. Anyway, among open-source CC packages, UBCC2 is available only in EBCC :+1:
With best regards,
Victor

Great -- thanks for finding all these bugs.


FallbackChatOpenAI

Description
This PR presents the class GptTurboWith16kFallback that extends ChatOpenAI with a fallback mechanism for larger language models when encountering a "maximum context length" error.
Issue
This addition aligns with the discussion here. If an existing solution or further discussions regarding token-specific fallback mechanisms are available, I'd appreciate being directed to them.
Dependencies
No additional dependencies.
Additional context
While this PR provides a solution for automatic fallbacks when handling larger input contexts, it is merely a stepping stone. The goal is to broaden this concept and allow for fallback models with larger context capabilities across different models, not just ChatOpenAI. We aim to refine this idea and warmly welcome further contributions.
Maintainers
@baskaryan, @hwchase17
Proposed changes
GptTurboWith16kFallback introduces a fallback_modelattribute. This attribute specifies a larger language model to switch to when encountering a "maximum context length" error. The default fallback_model is "gpt-3.5-turbo-16k-0613", and it can be customized during class initialization.
Overriding of the _generate method from ChatOpenAI. If a "maximum context length" error occurs during the generation, a new GptTurboWith16kFallback instance is created with the fallback_model and a new generation attempt is made.
Moreover, it's plausible that the existing retry logic can handle this situation. If there's a way to leverage that, I'd love to understand it better.
I highly appreciate your feedback and contributions to enhancing this concept. As a new OSS contributor, I'm seeking your understanding and assistance in making this contribution more effective. Thank you.

We actually have fallbacks as a first-party feature of LangChain now with LCEL, so that'd be a better way of setting up a fallback now!
https://python.langchain.com/docs/expression_language/how_to/fallbacks


MNT: Re-rendered with conda-smithy 1.3.3

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Thanks @pmlandwehr . Hopefully this helps.


fix: sourcemap generation for story files

Related Counter PR at Storybook: https://github.com/storybookjs/storybook/pull/27171
Currently, the sourcemap generation for story files is broken. If rspack behaves like Webpack, then loaders have to chain sourcemaps. This doesn't happen automatically, as Vite/Rollup does. The issue is that the magic-string doesn't support sourcemap chaining currently.
The files I have touched only add content at the end of the file. Therefore, creating source maps is not essential since the line/column mapping will stay the same.
This fix only ensures that the source view with enabled sourcemaps in the browser isn't completely broken for .stories.ts|js files. The source map mapping still doesn't refer to the original file, but some compiled file returned by some loader in between. I haven't spent too much time to figure out the issue altogether. Someone else could continue from here. The linked PR in Storybook fixes source maps completely for Vite and Webpack5 builders in Storybook
Reproduction

Activate sourcemaps in your browser's developer console
Change the Button.stories.tsx in a freshly generated Storybook to

import type { Meta, StoryObj } from "@storybook/react";
import { fn } from "@storybook/test";
import { Button } from "./Button";

// More on how to set up stories at: https://storybook.js.org/docs/writing-stories#default-export
const meta = {
  title: "Example/Button",
  component: Button,
  args: {
    onClick: () => {
      throw new Error("Not implemented");
    },
  },
} satisfies Meta<typeof Button>;

export default meta;
type Story = StoryObj<typeof meta>;

// More on writing stories with args: https://storybook.js.org/docs/writing-stories/args
export const Primary: Story = {
  args: {
    primary: true,
    label: "Button",
  },
};

Click on a button and try to visit the file in your browser, which throws the error
Before the fix, the browser couldn't open the source view at all
At least the source view works, although the mapping still does not refer to the original source, but to some precompiled code in between. It seems, that some rspack loaders still don't generate/chain sourcemaps correctly.

Confirmed this could resume a basic sourcemap from an error. I'll raise an issue to track the uncompleted sourcemap. Thanks for the sync up from storybook's main repository! 🙌♥


@latest tag via go install reports v1.5.0

$ go install github.com/flashbots/mev-boost@latest
go: downloading github.com/flashbots/mev-boost v1.5.0

good catch! fixing... (reason is that https://pkg.go.dev/github.com/flashbots/mev-boost only allows semver, pushing a semver tag now)

works now! thanks for the report!
$ go install github.com/flashbots/mev-boost@latest
$ mev-boost -version
mev-boost v1.6


fix: indent error

I think the checks fail because forked repos don't have access to the secrets of this repo?


不能支持https协议

请求https协议的时候提示传输错误，应该是证书导致的。可以改成信任所有ssl证书吗？

测试连接可以提供下吗，我方便测试

Hey. I am getting an error while sending the request - # Error: javax.net.ssl.SSLHandshakeException: Remote host terminated the handshake. Do you know what the problem might be?

Even I am facing similar issue when using over https .
# Error: javax.net.ssl.SSLHandshakeException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
Is there a way to specify Certificates / bypass this similar to what has been mentioned here ?
https://youtrack.jetbrains.com/issue/IDEA-248449?_ga=2.173995809.32949223.1616517359-423727728.1597747341

@skeerthikumar What version are you using now?  If not 2.2.5, please try this https://github.com/Vove7/retrofit-rest-client/releases/tag/2.2.5


Android: not built with triangulation library

This is in Termux 0.118.0 on an MT8183 tablet running Android 12, with pydot 2.0.0,
with the attached script "graphviz-problem.py" and dot file "K_5-cross.dot":
python graphviz-problem.py
"dot" with args ['-Tsvg', '/data/data/com.termux/files/usr/tmp/tmpnt44uiqc'] returned code: 1
(spits out the SVG code)
Error: remove_overlap: Graphviz not built with triangulation library
Judging from the Graphviz issues page, this isn't a Graphviz problem, but rather a problem with the dot binary.
problem_exammple.zip
On my Linux Mint desktop system, with pydot 1.4.2 from the distro installer, it runs and creates the SVG file.
I also tried removing the distro pydot, and using pip to install pydot 2.0,0.  The example script still runs properly there.
This appears to be a problem with the dot binary on Android.

Hi @StevanWhite
If it's a problem with the 'dot' binary, then it's a Graphviz problem, and you should definitely try to create an issue on their site:
https://gitlab.com/graphviz/graphviz/-/issues
Pydot can't handle problems related to the binary being built incorrectly.

Hi Łukasz,
OK, I figured something out.
The "dot" binary came from the distro "graphviz", not from the Python pip-installed packages.
This explains a couple of confusions.  Now, to figure out who to contact about that.
Thanks for the clarification!

I would try posting the problem on the Graphviz issues page with all your details. They'll definitely help you out.

https://github.com/termux/termux-packages/issues/19261

Oh, so it was solved in directly in termux by installing a missing dependency.
Great then, I can close this for now.


Declaring and tracking genes and gene products

All genes and gene products referred to in spreadsheets should be declared, typed and named.
For now at least we are going with ensembl IDs.  @BAevermann has provided a table:

Gene stable ID
Transcript stable ID
Gene name

ENSG00000210049
ENST00000387314
MT-TF

ENSG00000211459
ENST00000389680
MT-RNR1

ENSG00000210077
ENST00000387342
MT-TV

ENSG00000210082
ENST00000387347
MT-RNR2

We may want to limit this to genes we declare as markers (so the Mitochondrial genes above are likely not relevant, but I'll use them to demo how we could build Robot templates to generate OWL.)
We a way to make clear that these are ensembl IDs:
https://registry.identifiers.org/registry/ensembl provides a compact identifier (prefixing with ensembl and gives us a couple of options for turning these into resolvable URLs).
We can type using sequence ontology terms, e.g. for gene and transcript
If using transcript IDs we need a standard way to way to record the relationship between gene and transcript
Robot template tables
markers_genes.tsv

ID
TYPE
NAME

ID
SC %
A rdfs:label

ensembl:ENSG00000210049
SO:0000704
MT-TF

markers_transcripts.tsv

ID
TYPE
NAME
Transcript_of

ID
SC %
A rdfs:label
SC R* some %

ensembl:ENST00000387314
SO:0000673
MT-TF transcript
ensembl:ENSG00000210049

* Need to check what relation to use here.

Please deposit marker tables in the
Thinking about it, we should probably name tables for species.  I think a single table for each species would be fine, updated as needed.


better diagnostics when potentially duplicate packages are found

motivation: with the intriduction of the regsitry, there is a higher likelyhood of running into duplicate packages
changes: perform a heuristics to try and determine if the packages are duplicate and provide a better diagnostics instead of the standard "duplicate product" one
TODO: add tests

@neonichu @abertelrud for initial review, still need to add tests, but I think this is putting us in a slightly better footing
current diagnostics having  swift-nio from source control and swiftpm-test.swift-nio-8 from registry:
error: multiple targets named 'CNIOAtomics' in: 'swift-nio', 'swiftpm-test.swift-nio-8'; consider using the `moduleAliases` parameter in manifest to provide unique names
error: multiple targets named 'CNIODarwin' in: 'swift-nio', 'swiftpm-test.swift-nio-8'; consider using the `moduleAliases` parameter in manifest to provide unique names
error: multiple targets named 'CNIOLinux' in: 'swift-nio', 'swiftpm-test.swift-nio-8'; consider using the `moduleAliases` parameter in manifest to provide unique names
error: multiple targets named 'CNIOSHA1' in: 'swift-nio', 'swiftpm-test.swift-nio-8'; consider using the `moduleAliases` parameter in manifest to provide unique names
error: multiple targets named 'NIO' in: 'swift-nio', 'swiftpm-test.swift-nio-8'; consider using the `moduleAliases` parameter in manifest to provide unique names
error: multiple targets named 'NIOConcurrencyHelpers' in: 'swift-nio', 'swiftpm-test.swift-nio-8'; consider using the `moduleAliases` parameter in manifest to provide unique names
error: multiple targets named 'NIOFoundationCompat' in: 'swift-nio', 'swiftpm-test.swift-nio-8'; consider using the `moduleAliases` parameter in manifest to provide unique names
error: multiple targets named 'NIOHTTP1' in: 'swift-nio', 'swiftpm-test.swift-nio-8'; consider using the `moduleAliases` parameter in manifest to provide unique names
error: multiple targets named 'NIOTLS' in: 'swift-nio', 'swiftpm-test.swift-nio-8'; consider using the `moduleAliases` parameter in manifest to provide unique names
error: multiple targets named 'NIOWebSocket' in: 'swift-nio', 'swiftpm-test.swift-nio-8'; consider using the `moduleAliases` parameter in manifest to provide unique names

with this change:
error: multiple identical targets 'CNIOAtomics', 'CNIODarwin', 'CNIOLinux', 'CNIOSHA1', 'NIO', 'NIOConcurrencyHelpers', 'NIOFoundationCompat', 'NIOHTTP1', 'NIOTLS', 'NIOWebSocket' appear in registry package 'swiftpm-test.swift-nio-8' and source control package 'swift-nio', this may indicate that the two packages are the same and can be de-duplicated by activating the automatic source-control to registry replacement, or by using mirrors

also with this change, for potentially duplicate packages not across scm/registry:
error: multiple identical targets 'CNIOAtomics', 'CNIODarwin', 'CNIOHTTPParser', 'CNIOLinux', 'CNIOSHA1', 'NIO', 'NIOChatClient', 'NIOChatServer', 'NIOConcurrencyHelpers', 'NIOEchoClient', 'NIOEchoServer', 'NIOFoundationCompat', 'NIOHTTP1', 'NIOHTTP1Server', 'NIOMulticastChat', 'NIOPerformanceTester', 'NIOTLS', 'NIOUDPEchoServer', 'NIOWebSocket', 'NIOWebSocketServer', '_NIO1APIShims' appear in package 'swiftpm-test.swift-nio-7' and 'swiftpm-test.swift-nio-8', this may indicate that the two packages are the same and can be de-duplicated by using mirrors

@swift-ci smoke test

@neonichu this is ready for review

@swift-ci smoke test windows

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test windows

@tomerd yeah, @etcwilde is aware of this flake, it occurs on macOS as well it seems.

@_spi(MainActorUtilities) import _Concurrency
 ^

C:\Users\swift-ci\jenkins\workspace\swiftpm-PR-windows\swift\test\Concurrency\Runtime\startOnMainActor.swift:44:14: error: generic parameter 'Success' could not be inferred
    let t2 = Task.startOnMainActor {
             ^

C:\Users\swift-ci\jenkins\workspace\swiftpm-PR-windows\swift\test\Concurrency\Runtime\startOnMainActor.swift:44:14: error: generic parameter 'Failure' could not be inferred
    let t2 = Task.startOnMainActor {
             ^

C:\Users\swift-ci\jenkins\workspace\swiftpm-PR-windows\swift\test\Concurrency\Runtime\startOnMainActor.swift:44:14: note: explicitly specify the generic arguments to fix this issue
    let t2 = Task.startOnMainActor {
             ^
                 <<#Success: Sendable#>, <#Failure: Error#>>

C:\Users\swift-ci\jenkins\workspace\swiftpm-PR-windows\swift\test\Concurrency\Runtime\startOnMainActor.swift:44:19: error: type 'Task<Success, Failure>' has no member 'startOnMainActor'
    let t2 = Task.startOnMainActor {
             ~~~~ ^~~~~~~~~~~~~~~~

C:\Users\swift-ci\jenkins\workspace\swiftpm-PR-windows\swift\test\Concurrency\Runtime\startOnMainActor.swift:79:14: error: generic parameter 'Success' could not be inferred
    let t2 = Task.startOnMainActor {
             ^

C:\Users\swift-ci\jenkins\workspace\swiftpm-PR-windows\swift\test\Concurrency\Runtime\startOnMainActor.swift:79:14: error: generic parameter 'Failure' could not be inferred
    let t2 = Task.startOnMainActor {
             ^

C:\Users\swift-ci\jenkins\workspace\swiftpm-PR-windows\swift\test\Concurrency\Runtime\startOnMainActor.swift:79:14: note: explicitly specify the generic arguments to fix this issue
    let t2 = Task.startOnMainActor {
             ^
                 <<#Success: Sendable#>, <#Failure: Error#>>

C:\Users\swift-ci\jenkins\workspace\swiftpm-PR-windows\swift\test\Concurrency\Runtime\startOnMainActor.swift:79:19: error: type 'Task<Success, Failure>' has no member 'startOnMainActor'
    let t2 = Task.startOnMainActor {
             ~~~~ ^~~~~~~~~~~~~~~~

error: command failed with exit status: 1

Yeah, this was the failure that's been hitting me locally. It has multiple levels of weirdness. First is, why are we picking up swift interfaces for the stdlib. We should be using the swiftmodule. Second is, why are we building swiftinterfaces on Windows? We don't have ABI-stability and library evolution on Windows.
This PR will paper over the issue by emitting private swift interfaces for the stdlib and runtimes, but shouldn't be necessary: https://github.com/apple/swift/pull/64587

@tomerd yeah, @etcwilde is aware of this flake, it occurs on macOS as well it seems.

Yep, and it affects Linux. It affects everything, but on every other test run, and not while in a position to actually debug it.


propose renaming at_client_mobile to at_client_flutter

Is your feature request related to a problem? Please describe.
Now that we have support for desktops, at_client_mobile just seems wrong.
Describe the solution you'd like
Create a new package called at_client_flutter and add to the repo.
End of life the at_client_mobile platform over time
Describe alternatives you've considered
We could just leave it as at_client_mobile as it is not causing any problems

@nickelskevin didn't you already propose an approach to this in https://github.com/atsign-foundation/at_client_sdk/issues/241#issuecomment-914412495


Extend RefOrMut to cover owned data.

The RefOrMut type previously covered immutable and mutable references to
data. We extend it to cover owned data in addition to the previous variants
to more clearly communicate that the sender doesn't want to hold on to the
data and also cannot accept an allocation instead.
Signed-off-by: Moritz Hoffmann<EMAIL_ADDRESS>

Not needed anymore.


deployment diagram

review needed

Codecov Report

Merging #46 (2808787) into pre (dc4be7c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              pre      #46   +/-   ##
=======================================
  Coverage   51.27%   51.27%
=======================================
  Files          29       29
  Lines        2356     2356
=======================================
  Hits         1208     1208
  Misses       1148     1148

Continue to review full report at Codecov.

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


is the domain validation a prerequistes?

in the warning section  I read

While it's not absolutely required to add the TXT

However it does not seems possible to add a custom hostname without validation: is a prerequisite and the documentantion is misleading?
thanks

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

ID: 224b73dd-c59d-a833-0d4d-a778af6086a9
Version Independent ID: c6b167c9-9589-45d0-ee6a-7a47a4aec41e
Content: Map existing custom DNS name - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-custom-domain.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@adalfa
Thanks for your feedback! We will investigate and update as appropriate.

Hi @adalfa Yes, domain validation is a prerequisite for adding a custom hostname to an Azure Web App. We will share your feedback with the engineering team to see if there are any edge cases that domain verification may not be needed for adding a custom hostname before making any changes to the doc.


5B generation blank generations.

System Info / 系統信息
Temporal Prompt Engine Configuration
Information / 问题信息

[ ] The official example scripts / 官方的示例脚本
[X] My own modified scripts / 我自己修改的脚本和任务

Reproduction / 复现过程

CLI initiation of 5B through a custom batch pipeline is causing many blank (color only) generations](https://youtu.be/0zic0l_om_8?si=OBe4-jcH0HZVdHIg)

Confirmed to be the same throughout each example:
Seed = 1990 (changing it did not resolve)
GS = 10.0 (changing it did not resolve)
Restarting did not resolve.
I'm still experimenting with prompt builds to see what may influence but I'm starting to think this may just happen on 5b.
I can also confirm 5b does get watermarks occasionally, they are different from ponds.
Expected behavior / 期待表现
It should continue to iterate on previous steps count. I may just not have found documentation stating this is intended though. I'm still translating documentation as I go.

"Could you post the specific differences in the code (from cli_demo.py), as this will help us solve the problem."

"Could you post the specific differences in the code (from cli_demo.py), as this will help us solve the problem."

Sure! I'll do this is in the next couple of hours.


Scala 2.12.9 support

We need Scala.js 0.6.28 (or higher) with Scala 2.12.9 support:
org.scala-js:scalajs-compiler_2.12.9:0.6.28
My working branch: https://github.com/xerial/scala-js/tree/scala-2.12.9
Blocker:

[ ] Outer reference is now reported as a compile error: https://github.com/scala/bug/issues/11534

Related: https://github.com/wvlet/airframe/pull/576

@sjrd It seems we need to wait for Scala 2.12.10 if there is no workaround for scala/bug#11534

I'm not sure what you're talking about. Scala.js 0.6.28 and 1.0.0-M8 are ready for Scala 2.12.9. I will publish them tomorrow. Tonight I was hosting guests and now I am going to sleep.

@sjrd That's good to know.
Actually I hit the following error with Scala 2.12.9:
sbt:Scala.js> ~testSuite/test
[info] Compiling 34 Scala sources to /Users/leo/work/git/scala-js/compiler/target/scala-2.12/classes ...
[error] /Users/leo/work/git/scala-js/compiler/src/main/scala/org/scalajs/nscplugin/GenJSCode.scala:4679:33: The outer reference in this type test cannot be checked at run time.
[error]         case JSUnaryOpMethodName(code) if argc == 0 =>
[error]                                 ^
[error] one error found
[error] (compiler / Compile / compileIncremental) Compilation failed
[error] Total time: 7 s, completed Aug 5, 2019 12:58:45 PM
1. Waiting for source changes in project scalajs... (press enter to interrupt)```

FYI, Scala.js 0.6.28 and 1.0.0-M8 for Scala 2.12.9 are on their way to Maven Central.


Notification Access

Is your feature request related to a problem? Please describe.
I've been accustomed to see the notification in Glance like Lawnchair did
Describe the solution you'd like
Add the notification access to read the notification from some major chatting app
Describe alternatives you've considered
An implementation of whitelist app would be great in the future
Additional context
This is the example

Hey, I thought about this feature but I prefer to keep the widget clear and with limited information visible.
I believe that the focus of Another Widget must remain to show information "at a glance", information that the user can't have from the main UI of the device. And any information about the notifications is easily reachable from the status bar.


WIP: Multi tenant project id support

Fixed #523
All Submissions:

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

[X] Does your submission pass tests?
[X] Have you lint your code locally prior to submission?

@phriscage When you get a chance, could you start reviewing this? I still have to complete documentation updates, but the majority of the code changes are committed. I still have to complete testing though.


Orders in pathSortOrderOverride not consistent

I have to packages needs to be import:
import React from 'react';
import { useSelector } from 'react-redux';

import otherCustomComponents from 'absolute paths';

is my settings.json, here is my settings:
"typescript.extension.sortImports.pathSortOrderOverride": [ "react", "react-redux"]
With this override react, react-redux are moved to top after I run F10, which is expected.
However, if I run F10 again the order of react, react-redux swag, if I run F10 again, the order swag back. Can we keep the orders consistent without swagging?
Thank you.

fix released in v1.17.0


equipment calculation bug, from #theorycrafting

Trance Terra 6*
pin 2 L&A, click build:
http://ffbeEquip.com/builder.html?server=GL#7c486740-b549-11e8-ad06-e5deaaca2987
[bugged build]
replace witches cackle with a 3rd L&A, click build again
and you should get what's in the picture:
https://feen.us/iykqc6.png - mag% calculated as 290% with two staves equipped.
[note: opening a link to the build shows the mag% correctly at 140%
http://ffbeEquip.com/builder.html?server=GL#3df9ab60-b54a-11e8-ad06-e5deaaca2987
note 2: if i stop this build at 2%, with defenders daggers equipped, i get
MAG     1170     290% - 50%
again, opening the link shows the correct mag%: http://ffbeEquip.com/builder.html?server=GL#b4d6d440-b54c-11e8-ad06-e5deaaca2987
]
then if you unpin 1 L&A and replace it with L&A (yes not a typo) and click build, it will finally add a sword in the right hand
http://ffbeEquip.com/builder.html?server=GL#f8a4aff0-b54a-11e8-ad06-e5deaaca2987

Still a bug, and fascinating and bizarre behavior! Something like if the initial state of the equipment satisfies pinned materia conditions, they aren't revalidated when the gear starts changing? But that wouldn't explain why just rebuilding with two staves/rods equipped and 3 L&A pinned doesn't cause it to recalculate correctly.

Will probably solve it with the new algorithm. Won't take time to resolve it on the current one


Missing Attributions.txt for Delhi

Various services are missing their Attributions.txt files including config-seed, export-client, export-distro, support-logging, and support-notifications.
Additionally, the services with Attributions.txt have out-of-date attributions and don't include newer dependencies added to the project. They also don't contain all of the dependencies, including dependencies of dependencies.
This is a blocker for Delhi.

Also all of the Attribution.txt files inside internal/... are unnecessary as that code cannot be imported from anywhere outside of this repo.
Additionally, I have checked and none of the code that we do export from github.com/edgexfoundry/edgex-go/pkg/{clients,models} imports the internal directory, so all of the dependencies pulled in from the internal/... directory just need to be referenced in the cmd/.../Attribution.txt files for all the services.

Note that it doesn't appear as if the docker images contain the Attribution.txt. See:
$ mkdir core-config-seed-rootfs
$ docker pull edgexfoundry/docker-core-config-seed-go:0.7.1
0.7.1: Pulling from edgexfoundry/docker-core-config-seed-go
Digest: sha256:02a61ae965e8995b2dcc5c78026abd2be8afcb5a39714de3ea2452e67ed76353
Status: Image is up to date for edgexfoundry/docker-core-config-seed-go:0.7.1
$ id=$(docker create edgexfoundry/docker-core-config-seed-go:0.7.1)
$ docker export $id > img.tar
$ tar -xf img.tar -C core-config-seed-rootfs/
$ find core-config-seed-rootfs/ -iname "*license*"
core-config-seed-rootfs/usr/local/go/LICENSE
core-config-seed-rootfs/usr/local/go/src/cmd/vendor/github.com/google/pprof/LICENSE
core-config-seed-rootfs/usr/local/go/src/cmd/vendor/github.com/ianlancetaylor/demangle/LICENSE
$ find core-config-seed-rootfs/ -iname "*attribution*"
$

The only license files currently shipped with the docker image are from the actual go source code source trees.
As I mentioned in my PR it's not clear to me how to include the Attribution.txt file into the docker image due to how the COPY works with the flags used as well as the FROM scratch. Perhaps @tsconn23 or @JPWKU can comment on how best to do this.

We'd probably just add a line like so to each docker file
COPY --from=builder /go/src/github.com/edgexfoundry/edgex-go/cmd/core-data/Attribution.txt /Attribution.txt

Can you submit that PR or do you need me to? I think that the delhi docker images should be updated with this, not sure if that will happen automatically or if it needs a delhi build job triggered.

Currently, any update of the delhi images would require a 0.7.2 release. @jpwhitemn @JPWKU for visibility.

Is this an action that needs to be taken for Edinburgh?

This still needs to be done, I have not done anything for this and all docker images are still not including their Attribution.txt files


Dependency resolution performance for large projects

We are facing a performance issue on resolving dependencies with gradle.
We have a large project which we modularize. At the end we will have about 20 modules, each of them is a multi project build with about 10-20 subprojects. We also modularized our framework, which comes along with about 30 subprojects. We are half the way and have dependency resolution times per module of 30 seconds each time. I have done different performance measurements coming to inconclusive results, but all together they are ways to slow. I have uploaded a testproject to github. Details on handling you can see in README.md.
For us this is currently a show stopper for this modularization, so it would be nice to get a solution, workaround or a tip what goes wrong in our scenario asap. If you have problems to reproduce or any further questions, please call.
I have also opened a bug in spring-gradle-plugin, because using this plugin makes the performance even worse.
Thanks in advance
Cheers
Markus

@bigdaz All the CPU samples point to GradlePomModuleDescriptorParser, probably because of the heavy use of mavenLocal.
@moley Have you considered using composite builds instead of mavenLocal? That would give you much faster turnarounds than installing to mavenLocal.

Thanks @moley for the sample project.
Since Gradle bypasses the cache for file-backed local repositories, we are aware that this will introduce some overhead. However, it's good to get some real data on just how big of a pain this is.
In theory we should be only parsing each Maven POM once per build invocation, but perhaps there's a bug that's preventing this from working. Your sample should help us investigate this issue.

@bigdaz No problem, you're welcome
@oehme I always wanted to work with composite build but had no time so far to learn about it :-) Perhaps this is my chance. But  - even if we don't use mavenLocal the performance is poor. Using mavenLocal so often is only an issue in the context of this testproject and in the migration phase Afterwards I would estimate we need mavenLocal, but far not so often. I could change my test project to work with composite builds in another branch, so you can still investigate the mavenLocal performance and in parallel the other problem.... Or should I wait until you have finished this mavenLocal thing?

even if we don't use mavenLocal the performance is poor

We suspect the performance issue is due to local file repositories, not specifically mavenLocal. Are you able to easily test using a local HTTP repository? It would be interesting to see if this confirms our suspicion.

ok, thank you, I will retry with our real world project and give feedback on this

@bigdaz simply changing order of repos did not improve things, I think I have to change the testproject to use local http repo

Today I set up a local artifactory, which improves resolution dependency to 3-4 seconds. Changed the README.md to describe how to reproduce. So it is a workaround for us to use real dependencies or composite builds, but neverltheless would be nice on long term when this issue could be fixed

3-4s is still pretty long, definitely worth profiling as well. Thank you for the example!

I'm going to close this as a duplicate of https://github.com/gradle/gradle/issues/4322


Add editor icons for OccluderShapePolygon and OccluderShape

3.x version of https://github.com/godotengine/godot/pull/59178.
The abstract OccluderShape resource now has an icon, like the abstract Mesh resource.
Preview

I wonder if there's another shape we can use for ShapePolygon.

Can't believe I missed this, I must have been away from home, I was just going to remind you! :grin:
I think it's incredibly difficult to come up with something ideal, but these already are a vast improvement over just using the default icon so am happy to approve.

Thanks!


Add a script for own video input in .mp4 etc. and/or webcam input

First of all, thank you very much for your work. It is very nice to explore with the good documentation. I think the above mentioned scripts a re missing. It would be very nice to see those in the demo.py.
Greetings Gustav

Hi, @GKeppler ! Thank you for your focus!
At present, we have not provided the script for users' own video input and/or webcam input. However, DeciWatch is already implemented in (MMHuman3D)[https://github.com/open-mmlab/mmhuman3d] and you can check the performance of DeciWatch on any video there.


Add trivial smoketest for xpotrf

Add a trivial test for the *potrf routines to utest.
We've now been burned a few times by "bad" openblas builds (on some platforms only) that had broken cholesky factorizations. It would be useful to catch those already when building openblas itself, so here is a PR that adds a trivial smoketest that should catch those.

Guys - would you mind testing this patch on Windows 32-bit?  That is where we are seeing the numpy failure.

The problematic case is not just windows 32-bit, but also the specific compiler and maybe compiler flag choice, which I'm not sure is useful for openblas devs to spend time on. I tested myself that this test case catches the issue for the "bad" build, and it passes for correctly working potrf routines.

At the moment, though, the only situation we know this fails, is on Windows 32-bit, with the given compiler / flags and so on; so if this test passes with a standard mingw build, on Windows 32-bit, then we (numpyers) can try and track down which of our tweaks is causing the problem.

@matthew-brett: the "standard" mingw-w64/32bit compiler shipped on appveyor produces problem-free binaries (indeed, remember that we already had this discussion vs. scipy).
Anyway, even if the failing compiler setup is rare, it would still be useful if the standard openblas test suite would have at least minimal coverage of the potrf routines.

Ah - no - sorry - I had not registered that https://github.com/xianyi/OpenBLAS/issues/1270 was the same issue.

I'm all for detecting errors early, thanks :-)


Build of usdt example fails: "libbpf: elf: usdt_bpf is not a valid eBPF object file"

For me, building the USDT example fails with libbpf: elf: usdt_bpf is not a valid eBPF object file using elfutils (version  0.187) see https://github.com/libbpf/libbpf-bootstrap/pull/85#issuecomment-1162591490. I'm not sure if elfutils is the problem here, though.
Building the other examples, e.g., the minimal one, fails too with the same error.
$ make usdt
...

  CC       libbpf-bootstrap/examples/c/.output/bpftool/bootstrap/libbpf/staticobjs/usdt.o
  AR       libbpf-bootstrap/examples/c/.output/bpftool/bootstrap/libbpf/libbpf.a
  INSTALL  bpf.h libbpf.h btf.h libbpf_common.h libbpf_legacy.h bpf_helpers.h bpf_helper_defs.h bpf_tracing.h bpf_endian.h bpf_core_read.h skel_internal.h libbpf_version.h usdt.bpf.h
  CC       libbpf-bootstrap/examples/c/.output/bpftool/bootstrap/main.o
  CC       libbpf-bootstrap/examples/c/.output/bpftool/bootstrap/common.o
...
  CC       libbpf-bootstrap/examples/c/.output/bpftool/bootstrap/disasm.o
  LINK     libbpf-bootstrap/examples/c/.output/bpftool/bootstrap/bpftool
  GEN-SKEL .output/usdt.skel.h
libbpf: elf: usdt_bpf is not a valid eBPF object file
Error: failed to open BPF object file: BPF object format invalid
make: *** [Makefile:112: .output/usdt.skel.h] Error 95
make: *** Deleting file '.output/usdt.skel.h'

Your clang is probably too old.

Thanks for helping to troubleshoot this. I'm using clang version 14.0.1 which is fairly recent. Do I need something newer?

What's your distro and kernel version ?
It seams that clang-bpf-co-re: [ OFF ] causes the problem.

I'm on NixOS and kernel 5.15.55.  I've just tested it in a Ubuntu 22.04 VM and it seems to work fine there. Though I'd like to get it running on NixOS too, if possible.

Likely related to https://github.com/NixOS/nixpkgs/issues/176128. I'll revisit this and report back.

What's your distro and kernel version ?
It seams that clang-bpf-co-re: [ OFF ] causes the problem.

Chenheng, could you share your building environment? I met the same problem using Ubuntu 20.04.5 LTS with clang-10.
Kernel is 5.15.0-48-generic with CONFIG_DEBUG_INFO_BTF=y

Please upgrade your LLVM/clang.

I have built bpf grogram successfully using llvm-16 in teeks99/clang-ubuntu:16_20220910_1356 docker image.

Thanks for testing with unstable. I haven't gotten around to work on this again. From what I understand, this isn't merged into master / unstable yet? It's merged into staging (https://github.com/NixOS/nixpkgs/issues/176128#issuecomment-1157132972), not sure how and when staging is merged into master in the NixOS release cycle.

My understanding is that this should be already available based on https://nixpk.gs/pr-tracker.html?pr=176152. But I might be wrong as well.

Good news! I'm able to build the examples with the clang_14 from NixOS unstable using the nix-shell below. I think https://github.com/NixOS/nixpkgs/issues/176128 indeed solved the issue. I had to disable the stackprotector hardening. I'm closing this issue, as it's solved for me.
let
  unstableTarball = fetchTarball "https://github.com/NixOS/nixpkgs/archive/nixos-unstable.tar.gz";
  pkgs = import <nixpkgs> {};
  unstable = import unstableTarball {};

  shell = pkgs.mkShell {

    # Building the examples doesn't work with the NixOS stack protector hardening (enabled by default):
    #   ...
    #   BPF      .output/minimal_legacy.bpf.o
    #   minimal_legacy.bpf.c:22:5: error: A call to built-in function '__stack_chk_fail' is not supported.
    # See stackprotector under https://nixos.org/manual/nixpkgs/stable/#sec-hardening-flags-enabled-by-default
    hardeningDisable = [ "stackprotector" ];

    buildInputs = [
        pkgs.tree
        unstable.clang_14
        pkgs.llvm
        pkgs.elfutils
        pkgs.zlib
        pkgs.pkg-config
        pkgs.which
    ];
  };
in shell


Create SECURITY.md

https://github.com/mononaut/svelte-mononaut-avatar/commit/fd9b703cf95f4a266712ae0750e93b88eda12306#:~:text=Skip to content,for package-lock.json

Abc


[Request for External] report 9 "Trial Balance/Budget"

Dear ALAppExtensions team,
On behalf of 4PS I would like to request to make variable 'GLFilter' protected on report 9 "Trial Balance/Budget".
protected var
    GLFilter: Text;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


ONNX files for Opset 9 cgan and mxnet_arcface get errors for batch_norm_inference, 4d

Running these ONNX files with the MIGraphX driver results in the following errors
terminate called after throwing an instance of 'migraphx::version_1::exception'
  what():  /home/mev/source/AMDMIGraphX/src/include/migraphx/check_shapes.hpp:69
: batch_norm_inference: Only 4d supported

cgan has a batchnorm op with input shape (1, 256), which we currently do not support.

both models in this issue passed. Issues related batchnorm are gone


Lock zoom level minimum to prevent map fragments from showing up

We want to lock the minimum zoom level of the base map to 1 rather than 0: https://github.com/ua-snap/iem-webapp/blob/f2e0c0f75832ff235f07b50f8f37c002b263f4f2/components/Map.vue#L94

I think we could stay with either 0 or 1 for outermost zoom level (there's no way to see the whole extent otherwise) BUT I think if we can lock the bounds so the user can't scroll outside of a certain window, that'd be ideal.  But that may not work with the antemeridian.
This relates to #82.
It's possible to generate tiles with different "resolutions" / zoom levels, too, so that there's more gradation and then you can use fractional zoom with Leaflet.  May work, may not, depending.

This was resolved with PR #242


Delete .github/ISSUE_TEMPLATE directory

If you haven't already, please read through our contributing guidelines before opening your PR
What is the purpose of this PR?
Removes the old templates in ark so that the repo-wide files will apply

@srivarra because of the release-drafter stuff in the ark .github folder, I'm not sure if just removing these files will be enough for it to pull from the repo-wide .github templates directory. I'm guessing we'll need to completely delete this folder in order for it to use the angelolab defaults, which means we'll need to migrate the release-drafter stuff

@ngreenwald Yeah, we can get rid of this folder, and then angelolab/.github should be applied to each repository in the org.

Looks like it's only popping up in the .github repo, not anywhere else. Maybe you can't have orgnaization-wide github actions?


Containerized CAN to timescale

A canbus to postgres container.
A few ideas:

host networking mode and a simple python script to read from SocketCAN in chunks and write to timescale
socketcand (on host or in host mode container?) + a python container to read from socketcand and write to timescale
can-to-nanomsg (on host or in host mode container?) + a python container to read from nanomsg and write to timescale

Option 3 has the advantage of letting us try out nano message where we thought DBUS might work well for us.
Option 2 has the advantage of using existing socketcan code
Option 1 has the advantage of less moving parts.
Current thinking: Try out option 3 as a learning experience around nanomsg. Most of the core code is needed for all options, so we can always "downgrade" if we experience issues.
Current thinking 2: We can keep running our candump loggers a for a while too ... as a backup until we trust our new setup.

I like the idea of trying new things. However, I checked the nanomsg website and found it was superseded by NNG. I read both sites and seems there is no big difference between them. From what I understood, NNG is written and implemented in C, which I hope won't give much trouble when porting software between ARM and x86.

Should this be closed with the merged of #84?

Agree. This was addressed in #84, but I think we should open a new issue to keep track of our upgrade to nanomsg in future releases.


[Protocol] Optimize resolution vs. TR in EPI fieldmaps ("pe-polar")

We should double-check the TR/TE of the field map => The TR is highly different (much longer) from the TR of the functional acquisitions; is that okay? Was that on purpose? (from https://github.com/TheAxonLab/hcph-sops/issues/181#issuecomment-1719762301)

To which I responded that having a longer TR is not a problem, however we should leverage that increased TR to increase the resolution substantially.

I'm marking this as won't fix - let's forget about this one.


Link titles truncated at newline

A link like this:
[foo
bar](https://example.com)

will get truncated in the plaintext output as foo [n] rather than foo bar [n].

Fixed


Update to 1.21 please?

As 1.21 has released for 40 days, I really really really want to use this powerful tool to type commands.
Will you update this mod to 1.21?
bcuz ur gradle build script is a lit bit confusing so I cant update to 1.21 by myself.

The only reason I hadn't published it yet was because I wanted to address #2 in the same release, but I can do that later in a bugfix release.
I've just published the release to Curse Forge, and luckily it has been approved very fast, so it's already available. ^^

thanks!


Create first data-graph resolver: Medium article feed service

This involves actually hooking up the data graph to another service and ensuring that data can be retrieved. This means that traffic should run from the data graph, through the message broker, to another service, and back to the data graph service successfully.

is this an extra issue? do we need this one? @jdharvey-ibm