Introduces #|expatch directive

Documentation is in nbs/explanations/directives.ipynb. Implementation is spread across nbs/apis/0{1,4}*.
Description:
This pull request proposes a new directive called #|expatch. It is similar to #|export, but it will also indent the content of the export cell by two spaces. This allows us to interleave multiple cells while defining certain methods.
Use Case:
This directive has been tested in production for a couple of months to develop ETL style PySpark jobs.
Due to how the pyspark jobs are deployed, we end up needing all of the code wrapped in a method annotated with @call_parse. Using #|expatch has been helpful, because the code in the method is linear. It involves transforming complex, messy, and disparate sets of data. We interweave exported (#|expatch) code cells with documentation and example cells. But we also introduce documentation and examples along the way. There is plenty of code refactoring, helper methods, and code reuse. But at the end of the day, the bulk of the business logic is best kept inside a single exported method.
It has proven much easier and faster to debug and iterate this way. The alternative being that we declare the whole of the primary method inside a single hundred-line-plus cell and then test by running the whole thing or by reproducing parts of it in non-exported cells. With #|expatch, we can declare the exported code once and run it as we develop and test it.
A Caveat on Usage:
Please note that the #|expatch directive has very limited utility where it actually makes sense to refactor the code into small methods. It is not a good fit for those cases. But in that narrow set of use-cases where a long drawn-out method makes sense, this directive significantly simplifies documentation and debugging. In design pattern terminology, this is sometimes referred to as a "Transaction Script". Not to be confused with the much more common anti-pattern called "Long Method".

There is also a small thread in the nbdev discord about this idea

Can you link to a sample notebook that uses this directive? It's not clear to me how it would work when running cells in the notebook. Does the code run cleanly when running notebook cells? (I'm puzzled how it could because code in a cell separated from the start of the method wouldn't have access to the variables created in the first method cell, within the method's namespace.)
If this directive were adopted, I'm thinking there would need to be an option to indent by four spaces rather than two, no?

If this directive were adopted, I'm thinking there would need to be an option to indent by four spaces rather than two, no?

That would be a really nice improvement to the expatch directive!
There is already an indent=2 argument to the _expatch_ method, which defaults to 2 spaces. It would just be a matter of passing your desired indentation in to the directive processor. I recommend providing up to 3 ways of doing it:

#|expatch indent=4 at the directive level.
#|default_expatch_indent=4 at the notebook level.
default_expatch_indent=4 at the settings.ini level.

Can you link to a sample notebook that uses this directive? It's not clear to me how it would work when running cells in the notebook.

I agree, it's a little nuanced. The
In the discord chat thread, I included a couple of screenshots to demonstrate this:

Does the code run cleanly when running notebook cells? (I'm puzzled how it could because code in a cell separated from the start of the method wouldn't have access to the variables created in the first method cell, within the method's namespace.)

Instead of writing your complete method in single cell, you would only include a stub in the #|exported cell. Basically, the method signature and documentation. The rest of the method is #|expatched in.

If this directive were adopted, I'm thinking there would need to be an option to indent by four spaces rather than two, no?

That would be a really nice improvement to the expatch directive!
There is already an indent=2 argument to the _expatch_ method, which defaults to 2 spaces. It would just be a matter of passing your desired indentation in to the directive processor. I recommend providing up to 3 ways of doing it:

#|expatch indent=4 at the directive level.
#|default_expatch_indent=4 at the notebook level.
default_expatch_indent=4 at the settings.ini level.

Can you link to a sample notebook that uses this directive? It's not clear to me how it would work when running cells in the notebook.

I agree, it's a little nuanced. I attached an example in this comment. But first, please note the "Caveat on Usage":

Please note that the #|expatch directive has very limited utility where it actually makes sense to refactor the code into small methods. It is not a good fit for those cases. But in that narrow set of use-cases where a long drawn-out method makes sense, this directive significantly simplifies documentation and debugging. In design pattern terminology, this is sometimes referred to as a "Transaction Script". Not to be confused with the much more common anti-pattern called "Long Method".

In the discord chat thread, I included a couple of screenshots to demonstrate this:

⚠ take particular note of the second cell, which is not exported. It is just there to set local testing values for the arguments.

Does the code run cleanly when running notebook cells? (I'm puzzled how it could because code in a cell separated from the start of the method wouldn't have access to the variables created in the first method cell, within the method's namespace.)

Yes, it executes cleanly, but only if is a specific pattern is followed first. Mainly, the arguments to the method need to be declared in a separate cell once the method is declared. This is useful for demonstrating your code.
Instead of writing your complete method in single cell, you would only include a stub in the #|exported cell. Basically, the method signature and documentation. The rest of the method is #|expatched in.

...narrow set of use-cases...

Thank you. Your responses help me understand better how it works for you in practice. My own opinion (just the opinion of one other nbdev user), having considered it a bit more, is that this would be better as an nbdev plugin rather than as part of nbdev proper. The reason is that it introduces 1-2 new directives and/or a new setting for the sake of a fairly narrow set of use cases.

...limited utility where it actually makes sense to refactor the code into small methods. It is not a good fit for those cases.

There's also a risk of people using this directive in situations for which it is not a good fit rather than refactoring their code into smaller functions, inadvertently encouraging worse code. Providing it instead as a plugin reduces this risk, I think.

I apologise for sitting on this for so long - I've had trouble getting my head around what it's for, and this discussion is very useful thank you! I agree that it would be best made available as a plugin in a separate project, so I'll close this PR.

Brilliant. Thanks for the suggestion @hugetim. I am not sure how I missed
the plugin support in nbdev, but I am grateful for the link.
I believe this should be possible using processors. But it may not be,
we’ll see. On the surface it seems like a better fit for any custom
directive, for sure though.
On Fri, Oct 13, 2023 at 17:46 Jeremy Howard @.***>
wrote:

Closed #1336 https://github.com/fastai/nbdev/pull/1336.
—
Reply to this email directly, view it on GitHub
https://github.com/fastai/nbdev/pull/1336#event-10649198839, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACHPQS76C4MRHMVR37EHR3X7GZDDANCNFSM6AAAAAAXZFM35Y
.
You are receiving this because you authored the thread.Message ID:
@.***>


Handle Exception when there are different binning

Different binnings causes images with different shapes to be present. Which can't be processed by obvious reasons.

This issue is too old


erlang_basho_R16: add explicit Carbon and Cocoa support

Motivation for this change
erlang_basho_R16 currently inherits Carbon and Cocoa but doesn't explicitly state their compile flags when building the Mac OS X version of the package. This should be amended to include these flags.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

I thought the basho packages didn't build on darwin.

The Erlang OTP forked by Basho builds on the Mac system.


 Waking up queue_manager_threads [(<Thread(QueueManagerThread, started daemon<PHONE_NUMBER>54976)>, <joblib.externals.loky.process_executor._ThreadWakeup object at 0x7fffed463a20>)]
[DEBUG:MainProcess:QueueManagerThread] queue management thread shutting down
[DEBUG:MainProcess:QueueManagerThread] closing call_queue
[DEBUG:MainProcess:QueueManagerThread] telling queue thread to quit
[DEBUG:MainProcess:QueueManagerThread] joining processes
[DEBUG:MainProcess:QueueFeederThread] feeder thread got sentinel -- exiting
[DEBUG:MainProcess:QueueManagerThread] queue management thread clean shutdown of worker processes: []
[DEBUG:MainProcess:MainThread] running the remaining "atexit" finalizers
builder for '/nix/store/zhrfqgpp5a4yg4k2r7w6hfkimzf5s1vy-python3.7-joblib-0.12.4.drv' failed with exit code 1
cannot build derivation '/nix/store/0z9166ghjdzxxkpcwzg6s10vjkxf7s7v-python3-3.7.2-env.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/bm8x35wkl1y5l5yc9sgxhrmwxwg5z1cj-ipython-python.drv': 1 dependencies couldn't be built
error: build of '/nix/store/bm8x35wkl1y5l5yc9sgxhrmwxwg5z1cj-ipython-python.drv', '/nix/store/jhv8vm2fsyz29ypmlvyxm6366l9sxhv1-python3.7-jupyterlab-0.35.4.drv' failed

This is the output when doing what?

Closing out old issues with no activity in preparation for migration to the next branch.


⚠ Bofrost.at has degraded performance

In 85dcad5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1414 ms

Resolved: Bofrost.at performance has improved in 70159ea.


Improve too large chunk size test

This is a followup of #528
The current test goes over the configured max chunk size when parsing the last hex char. The spirit of the code is to continue parsing the size even if we've already gone over the max chunk size. To actually test this we need to make sure another char follows after we've gone over the max chunk size.

@SkyTrix how long does this test take to run? A couple of seconds is ok but we ideally would not have long running tests.

@SkyTrix how long does this test take to run? A couple of seconds is ok but we ideally would not have long running tests.

This test, like most others in this file, is pretty simple and runs in 5 milliseconds on my machine.

Maybe the value can be adjusted with the configuration value read from the default configuration, better than hard cording.


Update VersionOne Java API version to make the plugin working with current VersionOne version

Hi,
here a Pull request to use the newest version of the Java API (the plugin does not work with the current base code).
I excluded it from com.versionone:VersionOne.SDK.Java.ObjectModel (because it was used transitively) and add it directly so that it prepares the work to get rid off com.versionone:VersionOne.SDK.Java.ObjectModel.
Please also note that I had to exclude org.apache.maven.plugins:maven-gpg-plugin which was failing tests injected by Jenkins (maybe this is something that need to be changed in com.versionone:VersionOne.SDK.Java.APIClient).
Regards

Great!
Thanks for the pull request David.


How can i use this with the cdn inside the simple Html website?

Hi, I want to add this inside my simple HTML website using CDN, how can I do that I can not seem to find any CDN URL.
Kindly let me know.
Thanks

This is page with a cdn url
https://www.jsdelivr.com/package/npm/vue-js-toggle-button?tab=collection
but not sure how to import and Vue.use(ToggleButton) it?

Also wondering how to use the Component in a simple HTML page. I don't have the ability 'import' due to the software I'm integrating with. With the BootstrapVue components they "just work" as long as I reference the js/css URLs, but this one I can't figure out how to render the components.

@usamamashkoor you can do it like this:
<head>
   ... your other scripts here ...
   <script<EMAIL_ADDRESS></head>

And then before you construct your Vue application, add this:
window['vue-js-toggle-button'].default.install(Vue);

... or you can use this call
Vue.use(window['vue-js-toggle-button'].default)


What is the proper label for the certainty without a value?

Lately, we created a bit of noise around this label.
Is it supposed to be "empty" or "-" in the drop down menu?
(In the database, we use the value "0" for this option.)

hyphen is fine, I like it.


Document how to work with CockroachDB in an autoscaling group

See the confusion here
We need an answer for this.
cc @kannanlakshmi

@jseldess adding this to the docs project backlog for production guidance

We have closed this issue because it is more than 3 years
old. If this issue is still relevant, please add a comment and re-open
the issue. Thank you for your contribution to CockroachDB docs!


One Click Install

resolves: https://github.com/ngrok/kubernetes-ingress-controller/issues/317
What
Many of the k8s marketplaces we are trying to get listed in have a '1 click install' process which runs the helm install with default values and then allows us to give instructions to configure it after the fact.
How
Breaking Changes

Current dependencies on/for this PR:

main

PR #322   👈

This stack of pull requests is managed by Graphite.

no changes needed to the ingress controller.


UI Pointer Locked

There is a bug when interacting with the UI and the Lock Camera.
If you create a UI with Height ~= Canvas Height and lock the cursor to interact with the UI, then you cant go back to unlock the cursor and move the player camera.
Its seems that the UI is blocking all the pointer events.
I.E CODE
export function setupUi() {
  const ui = () =>
    <UiEntity
      uiTransform={{ width: 10, height: 900, margin: { left: 260, top: 16 } }}
    />

  ReactEcsRenderer.setUiRenderer(ui)
}

https://user-images.githubusercontent.com/16782417/209166179-6a8679fa-7874-4e58-8eba-1c7929faee7e.mov

closed via https://github.com/decentraland/unity-renderer/pull/3770


CTM files with comments can't be parsed

If you try to use CTM files with comments in them (as defined in https://github.com/usnistgov/SCTK/blob/f48376a203ab17f0d479995d87275db6772dcb4a/doc/infmts.htm#L285 ) meeteval throws a error.
Reason: When reading lines from CTM files, meeteval ignores empty lines, but lines with comments (starting with ";;") also need to be ignored.

Thanks for reporting the missing support for comments. I will add it.


Installing exe file

Is there a special way to install everything? I'm opening the exe file and it only comes up with a blank window.
EDIT: Alright, found out you needed to open it in the Manga Tagger folder in C drive.  And that to have it start monitoring for new installs, it seems like that executable needs to be open for when the items are downloaded.
Found an issue where I suppose the manga isn't searchable if the name from the source is exact..

I'm also getting the errors as with a manga titled exactly as it should be:

Everything should be installed since your last settings/config instructions.

To the original issue - I've made changes to the application so that it should log to the console in the event of an error that needs to be fixed. I've also made it a bit more "portable", so a lot of changes are coming in v1.1.2-alpha.
As to the actual issues - I'll look into them :)

To the original issue - I've made changes to the application so that it should log to the console in the event of an error that needs to be fixed. I've also made it a bit more "portable", so a lot of changes are coming in v1.1.2-alpha.
As to the actual issues - I'll look into them :)

this is still pretty great, im happy to help test! I love how you made it so then we can launch the exe prior to having fmd download stuff.  it makes it more controllable. and the way it is, at least it's detecting stuff now and doing what it's suppose to as opposed to how it was before xD

this is still pretty great, im happy to help test! I love how you made it so then we can launch the exe prior to having fmd download stuff.  it makes it more controllable. and the way it is, at least it's detecting stuff now and doing what it's suppose to as opposed to how it was before xD

First reported issue fixed in aad001680990e86cec6264e6170c91a21c21cbb5 - I changed the way series verification was done. Previously it was based on a comparison value between the actual downloaded manga and what MyAnimeList returned, but the threshold for the value was high. I originally programmed it this way so series weren't wrongly matched, but I figured I might have to change this at some point to account for series named such as "Peach Girl Next [EN]". So now, a comparison is done on the manga titles but also compares the authors found using MyAnimeList to AniList.
Second reported issue fixed in 286c10513ded231405971f300434722e1a34d631 - I always knew there would be more than just "chapter", "ch.", "act", etc. So I just added a case to test for "Series Title Chapter".
I should make this my tagline, lol, but here it is: as always, please open an issue if you find any other bugs, problems or questions :)

First reported issue fixed in aad001680990e86cec6264e6170c91a21c21cbb5 - I changed the way series verification was done. Previously it was based on a comparison value between the actual downloaded manga and what MyAnimeList returned, but the threshold for the value was high. I originally programmed it this way so series weren't wrongly matched, but I figured I might have to change this at some point to account for series named such as "Peach Girl Next [EN]". So now, a comparison is done on the manga titles but also compares the authors found using MyAnimeList to AniList.
Second reported issue fixed in 286c10513ded231405971f300434722e1a34d631 - I always knew there would be more than just "chapter", "ch.", "act", etc. So I just added a case to test for "Series Title Chapter".
I should make this my tagline, lol, but here it is: as always, please open an issue if you find any other bugs, problems or questions :)

thanks Inpacchi, I look forward to the updated alpha build

thanks Inpacchi, I look forward to the updated alpha build


Add Bouira University Digital Space as a harvester

OAI base url: http://dspace.univ-bouira.dz:8080/oai/request
create using auto oai - https://github.com/erinspace/autooai

Base OAI query seems to have no data, have emailed asking if we're using the correct information!

@erinspace Did you ever hear back from them? It looks like their dspace is no longer active.
http://www.opendoar.org/new1find.php?rID=3218&format=full


[GOBBLIN-1437] Segregate FlowConfigs/Delete and FlowExecutions/Delete functions

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
@jack-moseley  @Will-Lo  @aplex @umustafi please review
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1437

Description

[x] Here are some details about my PR, including screenshots (if applicable):
This PR

make KafkaJobMonitor s parseJobSpec() method return the entire JobSpec, so that the caller method processMessage() can distinguish between a DELETE FLOW and CANCEL JOB requests and take appropriate actions. Actions are based on the doc https://docs.google.com/document/d/1lanG0R-w81MK5anpGBEE0hWF3rI8qArX4bixKzN0wEs/edit
some minor refactoring

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Besides some existing unit tests, did following tests on LinkedIn test environment

flowConfig/Delete doing a) delete of state store b) cancelling a running job c) deleting specs from spec catalog
flowExecutions/Delete doing a) leaving state store intact b) cancelling a running job c ) leaving spec catalog intact

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

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

Codecov Report

Merging #3268 (273a4ca) into master (0a504aa) will decrease coverage by 37.46%.
The diff coverage is 2.56%.

:exclamation: Current head 273a4ca differs from pull request most recent head adeba55.

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

Overall looks good, but I think we still have some dead code left in JobCatalogListener - onCancelJob, CancelJobCallback and in surrounding classes. Can it be removed to prevent confusion for the next time, or it's still used for something?

Removed the dead code.


(PUP-11034) Allow systemd static services to be masked

Closes https://tickets.puppetlabs.com/browse/PUP-11034
Previously the service provider for systemd threw a debug message and prevented static services from being enable => 'mask'. Turns out this is a valid configuration (corner case) and the systemctl commands work just fine.
Now, we provide an extra case when checking the enabled value to allow static services to be masked, but not allow them to be enabled or disabled via true and false.

Can one of the admins verify this patch?

Thank you for the contribution @nmaludy! I traced this back to an oversight in the implementation for PUP-5353 https://github.com/puppetlabs/puppet/pull/4887 which should have only handled enabling/disabling static services.
I validated the changes on a CentOS 8 box and everything works as expected.
Since this is a bug fix, can you please rebase and retarget the PR to 6.x? The commit will be automatically promoted to main when nightly CI passes.
Thanks!

Ah, sorry. I changed the PR base branch and rebased your commit. Should be good now :).
This way it'll get in both puppet6 and puppet7.


load_tree doesn't cope with pathlib.Path

import pathlib
from cogent3 import load_tree

path = pathlib.Path("~/repos/Cogent3/tests/data")
tree = load_tree(path / "murphy.tree")

generates
AttributeError                            Traceback (most recent call last)
/var/folders/rh/t24zj19s7qxd4_4p0n5rqjy00000gn/T/ipykernel_9969/1104250210.py in <module>
      3
      4 path = pathlib.Path("~/repos/Cogent3/tests/data")
----> 5 tree = load_tree(path / "murphy.tree")

~/opt/miniconda3/envs/c39dev/lib/python3.9/site-packages/cogent3/__init__.py in load_tree(filename, format, underscore_unmunge)
    594     with open_(filename) as tfile:
    595         treestring = tfile.read()
--> 596         if format is None and filename.endswith(".xml"):
    597             format = "xml"
    598

AttributeError: 'PosixPath' object has no attribute 'endswith'

Above code should be added as a test method in tests/test_core/test_tree.py, note that there's sample trees in tests/data/


[feature] Allow sameYear key to be used for calendar function.

The conditional logic was put in the ternary
so that it would not slowdown the performance of the 'days' diff conditions.
An example where this is helpful:
    return moment(normalizeTimestamp(timestamp)).calendar(null, {
      sameDay: '[Today]',
      nextDay: '[Tomorrow]',
      nextWeek: 'dddd',
      lastDay: '[Yesterday]',
      lastWeek: '[Last] dddd',
      sameElse: 'MM/DD/YYYY'
    });

If a meeting is scheduled for a month in the future but still within the same year.
A more appealing format would be 'MM/DD'
Currently I check the year formatting outside of the calendar function to change the format accordingly.
However, I thought that this change would be helpful for others.
Happy to update the documentation for calendar to accommodate the change.
Result after the change
    return moment(normalizeTimestamp(timestamp)).calendar(null, {
      sameDay: '[Today]',
      nextDay: '[Tomorrow]',
      nextWeek: 'dddd',
      lastDay: '[Yesterday]',
      lastWeek: '[Last] dddd',
	  sameYear: 'MM/DD',
      sameElse: 'MM/DD/YYYY'
    });

Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

It says that I have signed the license agreement. 🤷♂

It looks like the build is failing with:
Error: Cannot find module 'sync-exec'


use new RDKit_jll build

Codecov Report

Merging #18 (87545e2) into main (60cd8c1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage   78.82%   78.82%
=======================================
  Files           8        8
  Lines         170      170
=======================================
  Hits          134      134
  Misses         36       36

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 60cd8c1...87545e2. Read the comment docs.

fixes #4 and #15


fix report parsing

Summary of the Pull Request
What is this about?
PR Checklist

[ ] Applies to work item: #xxx
[ ] CLA signed. If not, go over here and sign the CLI.
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

Codecov Report

Merging #2755 (ae00061) into main (3f865e2) will decrease coverage by 3.53%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##             main    #2755      +/-   ##
==========================================
- Coverage   29.08%   25.56%   -3.53%
==========================================
  Files         321      123     -198
  Lines       38462    12984   -25478
==========================================
- Hits        11188     3319    -7869
+ Misses      27274     9665   -17609

Impacted Files
Coverage Δ

src/ApiService/ApiService/onefuzzlib/Reports.cs
53.84% <42.85%> (-4.49%)
:arrow_down:

...c/agent/onefuzz-task/src/tasks/analysis/generic.rs

...zz-task/src/tasks/generic/input_poller/callback.rs

src/agent/onefuzz-task/src/tasks/merge/generic.rs

...nt/onefuzz-task/src/tasks/merge/libfuzzer_merge.rs

src/agent/onefuzz-task/src/tasks/utils.rs

src/agent/storage-queue/src/azure_queue.rs

src/agent/storage-queue/src/lib.rs

src/agent/storage-queue/src/local_queue.rs

src/agent/onefuzz-task/src/tasks/config.rs

... and 189 more

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


Remove BUNDLED WITH in Gemfile.lock

After all the hassle of adding the message in
1.10, Bundler have removed the message again
in 1.11.

:+1:


exe not building.

I have tried using both CUDA 12.2 and CUDA 11.8 but I run into the same issues. I am using libtorch 2.4.1
Here is the console output for running the cmake command:

Here is the console output for running the build release command:

I have followed every step from the video here: https://www.youtube.com/watch?v=0YI2O5uSuFw&t=37s
I cannot get the exe to compile so the weights from the TinyStories model can be converted to the useable format. Is there another way to do this in Python that might be easier? Or is there a way to fix this.

I dug some more into the issue and found a fix for some of the errors.
The issue in llama_model.cpp is on line 109
std::cout << "position_ids: " << position_ids << std::endl;
This line is problematic. When I modified the print to the following iot got rid of that issue.
if (position_ids.has_value()) {
    std::cout << "position_ids: " << position_ids.value() << std::endl;
} else {
    std::cout << "position_ids is empty" << std::endl;
}

The following code outputted the model for me. I think this code should be tested and maybe added in a PR:
#pragma once

#include <torch/script.h> // One-stop header.
#include <iostream>
#include <memory>
#include <vector>
#include <string>
#include <fstream>
#include <torch/serialize.h>
#include "llama_utils.h"

// Function to read a file into a vector of bytes
std::vector<char> GetTheBytesTwo(const std::string& path) {
    std::ifstream file(path, std::ios::binary | std::ios::ate);
    if (!file.is_open()) {
        throw std::runtime_error("Failed to open file: " + path);
    }

    std::streamsize size = file.tellg();
    file.seekg(0, std::ios::beg);

    std::vector<char> buffer(size);
    if (file.read(buffer.data(), size)) {
        return buffer;
    }
    throw std::runtime_error("Failed to read the file: " + path);
}

int main(int argc, const char* argv[]) {
    // Path to the model file
    const std::string path = "E:\\Coding\\python_work\\HuggingFaceModelExporter\\pytorch_model.bin";  // Replace with your actual model path

    // Load the model file into a vector of bytes
    std::vector<char> file_data;
    try {
        file_data = GetTheBytesTwo(path);
    } catch (const std::runtime_error& e) {
        std::cerr << e.what() << std::endl;
        return -1;
    }

    // Deserialize the weights using torch's pickle loader
    torch::IValue weights;
    try {
        weights = torch::pickle_load(file_data);
    }
    catch (const c10::Error& e) {
        std::cerr << "Error loading model: " << e.what() << std::endl;
        return -1;
    }

    // Ensure the weights are a dictionary (GenericDict) for conversion to OutputArchive
    if (!weights.isGenericDict()) {
        std::cerr << "Error: Expected a GenericDict from the pickle load." << std::endl;
        return -1;
    }

    auto weights_dict = weights.toGenericDict();
    auto arch = toOutputArchive(weights_dict);

    // Save the output archive to a new file
    arch.save_to("E:\\Coding\\python_work\\HuggingFaceModelExporter\\pytorch_model.pt");

    std::cout << "Model weights successfully saved to TinyStories.pt." << std::endl;

    return 0;
}

This code also works to output the pt file:
#pragma once

#include "llama_utils.h"

int main(int argc, const char* argv[]) {
    // Path to the model file
    auto path = "E:\\Coding\\python_work\\HuggingFaceModelExporter\\pytorch_model.bin";  // Replace with your actual model path

    auto weights = torch::pickle_load(GetTheBytes(path));
    auto weights_dict = weights.toGenericDict();
    auto arch = toOutputArchive(weights_dict);
    arch.save_to("E:\\Coding\\python_work\\HuggingFaceModelExporter\\pytorch_model.pt");
    std::cout << "Model weights successfully saved to TinyStories.pt." << std::endl;

    return 0;
}

The issue all along was on my end. I placed .toGenericDict() inside the call to torch::pickle_load(GetTheBytes(path))

The issue all along was on my end. I placed .toGenericDict() inside the call to torch::pickle_load(GetTheBytes(path))


chore(LinearAlgebra/Charpoly/ToMatrix): characteristic polynomials of isomorphic endomorphisms are equal

Thanks!
bors merge


java.lang.IllegalArgumentException: The given string value: {" mapping": {}} cannot be transformed to Json object

Hello,
I am using  hypersistence-utils-hibernate-52 to store json document in mysql 8 table with spring boot.
The data is getting stored but when i trying to read it from db it is throwing
ERROR
java.lang.IllegalArgumentException: The given string value: {" mapping": {}} cannot be transformed to Json object
Here BaseDocumentEntity has annotation:
@TypeDefs({
    @TypeDef(name = "json", typeClass = JsonStringType.class)
})

public class UserDataDocument extends BaseDocumentEntity {

  @Column(name = "id", length = 64)
  private String id;
  @Type(type = "json")
  @Column(columnDefinition = "json")
  private UserData userData;

  @Data
  @Builder
  public static class UserData implements Serializable {

    private Map<String, BankDetails> mapping;
  }

  @Data
  @Builder
  public static class BankDetails implements Serializable {

    private String bankCode;
    private String bankName;
}

}

Try to debug it and see why it doesn't work. The source code is available in your IDE so you can debug and see what didn't work for you.

I tried debugging it. Initially the  Map<String, BankDetails> mapping was the json column but i wrapped it with UserData class. saving the data is working but idk why while reading it is breaking, i think it is not able to map the HashMap.

Maybe it's because of that @Data and @Builder annotations. Try with standard Java Beans and see if it works.

Identified the issue!!!!
So basically if we are storing only map inside an object it is never able to map the object. But along with map the map if i am storing any other field it is working.
Not working code:
@Data
@Builder
public static class UserData implements Serializable {
private Map<String, BankDetails> mapping;

}
Working Code:
@Data
@Builder
public static class UserData implements Serializable {
private String userId;
private Map<String, BankDetails> mapping;
}

I'm glad you found the solution,


Goliath camera stopped working

My door camera (VTO4202F-P1) worked for about 3 weeks, since yesterday I cannot connect to it. Setup fails. Update to 2024.8.0 was done yesterday.

It worked for again since two days. Today the Problem is back.

Reverted back to 2024.7.4; now it works

Upgrade to 2024.9.3 done, still working


Razor views cannot declare a custom base type and a model

Based on https://github.com/aspnet/Mvc/issues/5397
Currently, the way to specify a custom base type for a view and a model involves specifying the @inherits keyword in a _ViewImports.cshtml:
@inherits MyBaseType<TModel>
and specifying the model in the page:
@model MyModel
Specifying the two in the same page results in a parse error: The 'inherits' keyword is not allowed when a 'model' keyword is used.. Using _ViewImports is limiting here,
a. It does not allow specifying more than one base type per directory
b. not being able to contain all the definition information in a single view file makes it less portable when using our modular framework (e.g. we can have views defined in plugin assemblies)

I don't think there was ever an expectation that you would specify both an @inherits and @model keyword in the same view file. This isn't how MVC5 works and was only mentioned in #5397 when attempting a workaround. In MVC5 you would get the same error: The 'inherits' keyword is not allowed when a 'model' keyword is used.
In MVC5 th correct way to do this is to specify the model type as the generic parameter of the base type e.g. @inherits MyBaseType<MyViewModel> which causes the error in first mentioned in #5397 which I think is covered by #6291.

@pranavkm is this a regression from 1.1.X?

@rynowak no, this isn't a regression from 1.1.x. It is difference from Mvc 5 though.

Running into the same exact issue. It would be really great you could fix this for the ASP.NET 2.1 release. This is a show stopping bug in Razor Pages.
I'm not looking to run inherits and model in the same view file. I'd like to have a base class that all Razor Pages inherit from. Pretty simple. The basic scenario in #7559 should just work.
This issue has been open for ~10 months now. =/

Moving this to the Backlog as it's not a priority for the next release.


overflow-sm e width-sm

Hi, working with Mobile, I missed the possibility of setting overflow and width as overflow-sm and width-sm, something to that effect

We're working on changes to responsive widths (perhaps with column classes) in v5, but I don't think we'll be adding responsive overflow utilities at this time.


Make "affected agendas" column visible for documents on agenda pages

Currently, the documents on agenda pages such as https://aiwatch.issarice.com/?agenda=Iterated+amplification#documents don't show a column for affected agendas. I believe the reason for not having this column was that it's already obvious that all the documents are about this agenda.
However, many documents have multiple affected agendas. so it would be nice to have the affected agendas column.

I believe the reason for not having this column was that it's already obvious that all the documents are about this agenda.

IIRC agendas were added fairly recently to the site (after the documents table was added) so I think the actual reason is simply that I haven't gotten around to syncing everything up on the site. I appreciate you bringing this up!


Background

Version

[X] I'm using version 1.15.1

Known issues

[X] I've checked Known issues

Existing issues

[X] I've checked Existing issues

Advanced Preferences

[X] I've checked Advanced Preferences

What operating system are you using?
Windows
Operating System Version
Windows 10 Pro 22H2
Reproduction steps
Background is duplicated
Version 1.15.0 1.15.1
2023-11-25 10 35 44
Expected Behavior
The background must be plain
Actual Behavior
There are two colors on the background
Relevant log output
No response
Preferences
No response
Additional information
No response
Code of Conduct

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

Hi, is this with other language versions as well? Is this for any color or just this one?

@hovancik
I record small video
https://mega.nz/folder/TV4FmSqZ#tXkYBGh3nzy_dOIVo5WYeQ

I have the same issues, however, I'm using English language

@alabotski many thanks! looks like some CSS quirk
@OlegTopalov are you also on Windows 10?
On Windows 11 everything is fine for me

@alabotski many thanks! looks like some CSS quirk @OlegTopalov are you also on Windows 10?
On Windows 11 everything is fine for me

yep, win 10

@alabotski many thanks! looks like some CSS quirk @OlegTopalov are you also on Windows 10?
On Windows 11 everything is fine for me

Btw, noticed, that the issue is not present in Transparent mode. Also, I get some weird ghosting of the tooltip when hovering over Skip text

The most weird thing is that when adding DevTools on the page - this issue dissapears, but closing devtools creates similar glitches. I'm not very familiar with Electron development, but it feels, like it's some sort of their rendering issue.
BTW, recorded on upgraded from win10 win11
https://github.com/hovancik/stretchly/assets/26525991/f241ca67-4c1b-45b1-a17d-2e6ee441cd7c


format("z") seems to have issues with timezones

In Chrome's console:
new Date(1534545439635);
Sat Aug 18 2018 00:37:19 GMT+0200 (Central European Summer Time)

new Date(1534545439635).format("z");
"229"

The same day but later:
new Date(1534588427903)
Sat Aug 18 2018 12:33:47 GMT+0200 (Central European Summer Time)

new Date(1534588427903).format("z");
"230"

The timestamp at 00:37:19 returns "229" but the timestamp of the same day at 12:33:47 returns "230"
Note: Central European Summer Time is GMT +02:00 ;)

After a quick google I think I found a solution that seems to work on SO:
https://stackoverflow.com/a/26426761/781094
I've hacked it in your dateformat library and the results are promising ;)


How to disable Grant permissions screen

I want to send emails in background and i need refresh token.
Here is an example code that helped me:
https://github.com/google/google-api-nodejs-client/blob/master/samples/oauth2.js
But don’t understand how to disable granting permission screen.
I want to send email in background, server side so my user don’t have to view such a screen, i want automatic validation.
What's the correct way to do this?
Can you tell me how to do this?
David

Greetings!  You're not going to be able to disable the grant permission screen - this is just part of the OAuth2 spec, and is there to inform users what your application is trying to do.  What you can do is come up with a model where users sign up for your service, store the refresh token, and then perform actions on their behalf in the background.
The flow would be something like:

User signs up for your service.  You redirect them to the consent dialog via generateAuthUrl.
The user signs in, and grants permission to your application to send email on their behalf (assuming you pass the right scopes.
Access and Refresh tokens are sent back to your application to the redirect URI.  You can store the refresh token that comes back, but make sure to do it securely :)
Any time you want to send an email, set the refresh token on the OAuth2 client, and it will automatically fetch access tokens for you.

A lot of the flow I just described gets into the details of the OAuth2 client, so I would suggest reading up the docs and checking out the samples available here:
https://github.com/google/google-auth-library-nodejs
If you have any issues - let me know!

Hi @JustinBeckwith , thank you very much, i'm going to do like so.
Just few more questions:
1- I dont' want my application to send email on user behalf (maybe they don't have gmail account), i want my application send email with MY gmail account to my client emails.
Do they still have to grant this permission screen?
2- I'm going to use getAuthenticatedClient() (https://github.com/google/google-auth-library-nodejs).
Is this method that opens grant premission dialog is stored client side (ionic) or server side (express API) ?
3-I think /oauth2callback is server side url but how can i configure cors to accept this request , what url use google to send this request ?
Thank you anyway for your great help, Oauth2 is used everywhere now so after that i will be able to connect easily my application to facebook or instagram for example, they use same process.
David

I dont' want my application to send email on user behalf (maybe they don't have gmail account), i want my application send email with MY gmail account to my client emails. Do they still have to grant this permission screen?

Nope!  You will have to do this once.  After you've approved the app to have these permissions against your account, other users don't have anything to approve.

I'm going to use getAuthenticatedClient() (https://github.com/google/google-auth-library-nodejs).
Is this method that opens grant premission dialog is stored client side (ionic) or server side (express API) ?

That's not going to work for you.  That method assumes you're using service<-->service auth credentials.  It's possible you can generate a service account credential and just use that with gmail, but I think you need to be a GSuite customer for that to work.  You need to follow a flow similar to what's available in sampleclient

I think /oauth2callback is server side url but how can i configure cors to accept this request , what url use google to send this request ?

If your app is purely on the client side - it may be worth taking a look at this:
https://developers.google.com/api-client-library/javascript/start/start-js


fix: bump matrix-action to v2.0.2 (#299)

bump matrix-action to v2.0.2 which includes sc4s v3.28.0 tests:
https://github.com/splunk/splunk-add-on-for-cisco-esa/pull/440 Report before update (58 KO failures)
https://github.com/splunk/splunk-add-on-for-cisco-esa/runs/27652534705 report after update (12 KO failrues)
https://github.com/splunk/splunk-add-on-for-cisco-esa/runs/27656994881

:tada: This PR is included in version 4.17.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


"Suceava" is listed in the drop-down menu

Description:
"Suceava" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "SUCEAVA" in the search bar.

Expected results
The location is listed in the drop-down menu as "Municipiu SUCEAVA, SUCEAVA".


[e2e]: go tag main config-ldflags-noassets slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v14.2.358
Run: https://github.com/slsa-framework/example-package/actions/runs/5319214038
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-noassets.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-noassets.slsa3.yml
Trigger: push
Branch: v14.2.358
Date: Tue Jun 20 07:08:44 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/v14.2.358
Run: https://github.com/slsa-framework/example-package/actions/runs/5319214038
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-noassets.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-noassets.slsa3.yml
Trigger: push
Branch: v14.2.358
Date: Wed Jun 21 00:51:49 UTC 2023
Tests are passing now. Closing this issue.


Question: How to change UI font size in configuration file?

Hi, great program. Is it possible to change the font size of the UI in the configuration files? I'd like to increase the size just a bit.

There's currently no custom theming available.

I see. For anyone that's interested, I found that a temporary workaround to increase UI scaling was to use the environment variable GDK_DPI_SCALE=1.5 when running psst. Warning: If you use this, some values made my gnome-shell instance freeze. I had to resort to killing psst via an SSH session.

+1. The UI is hard to read on my screen. Would be a nice improvement :)


Changing file permissions for securityconfig and tools

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Backport of PR: #386

@vrozov added cherry pick to commit message.

Please rebase


Specify dtypes of both arguments in multiply

Description

MISC
As indicated in the multiply kernel in backend_cpu, we should also specify the dtype of both argument in backend_gpu.

For repository owners only:
Please remember to apply all applicable tags to your pull request.
Tags: FEATURE, BREAKING, BUG, PERF, DEV, DOC, SECURITY
For more info see: https://github.com/tensorflow/tfjs/blob/master/DEVELOPMENT.md

This change is

We already assert strict types matching here, which is why I didn't do this: https://github.com/tensorflow/tfjs-core/blob/master/src/ops/binary_ops.ts#L318
I'm curious why you were seeing this

@nsthorat Thanks so I'm going to close this. I took a look which ops now supports complex64 dtype.


ffigen ignores underlying enum type resulting in wrong struct size

Since C++ 11 we can provide the type that is backing an enum like this:
typedef log_level_e : uint8_t {
  info,
  success,
  warning,
  error
} log_level_e;

Then we can use this enum in a struct, where it will occupy the size of the specified type.
typedef struct log_entry_t {
	log_level_e level; // sizeof(uint8_t)
	const char* message;
	size_t length;
} log_entry_t;

However ffigen will transform the struct and enum into the following:
final class log_entry_t extends ffi.Struct {
   @ffi.Int32()
   external int level;

   external ffi.Pointer<ffi.Char> message;

   @ffi.Size()
   external int length;
}

Where level now takes up more space in the dart version of the struct compared to the Cpp version.

Note: ffigen only supports parsing C headers, not C++ headers.

I understand that this C++ feature is not supported by ffigen, but then it should at least give a warning or error when parsing a header that uses this feature.
Currently there is no warning about this in the ffigen output and it just results in mangled data when using this struct.

cc @mannprerak2

We currently generate an Int32 for enum types everywhere.
We can use clang_getEnumDeclIntegerType to check the correct Integer type to use. It seems to range from [ULong, UInt, Int, Long] on my system depending on the values of the Enum constants.
Here are some outputs from libclang.
enum E0: uint16_t { E10 };
enum E1 { E11 };
enum E2 { E12=-1 };
enum E3 { E13=-1000000000000 };
enum E4 { E14=1000000000000 };

E0. (Type) spelling: uint16_t, kind: 119, kindSpelling: Elaborated
E1. (Type) spelling: unsigned int, kind: 9, kindSpelling: UInt
E2. (Type) spelling: int, kind: 17, kindSpelling: Int
E3. (Type) spelling: long, kind: 18, kindSpelling: Long
E4. (Type) spelling: unsigned long, kind: 10, kindSpelling: ULong

@dcharkes wdyt?


Create Basic Repository Layout

Create a basic repository layout for a .NET Windows Desktop solution.   Include an empty WinUI3 application, Directory.Build.Props, Directory.Packages.Props and any other basic ceremony.

.NET 8 with a global.json.
Set an appropriate target and minimum Windows SDK

*Arranged the file structure layout so that it may support more than …
…one solution, such as in a mono-repository.
*Added support for arm64, removed support for x86
*Created a new WinUI3 WinEXE project using a project template.  The project template generated some pre-baked
layout with navigation. The template also added default implementations for String resources, MVVM, Styles, and dependency injection.
*Added a Directory.Build.Props to control common project settings.  This included the .NET version, Windows SDK version, output directories
and more.
*Added a NuGet.config control NuGet package management
*All output will go to the Artifacts directory to ensure we have a centralized location for all build artifacts.  The Artifacts directory
should be excluded from the git.
*Added some custom styling to ensure the style system is working.


Enormous /tmp/asitop_powermetrics* files

It's filled my hard drive up with files like this:
-rw-r--r--  1 root  wheel    43G 14 Dec 16:01 asitop_powermetrics1637685873
-rw-r--r--  1 root  wheel    37G 14 Dec 16:01 asitop_powermetrics1637936130

The current version should not have this issue. An older version had a issue where sometimes the log files from powermetrics are not cleared. Can you confirm you are using the current version? If so, I’ll look into it again.

Why are you filling up regular files anyhow? Can't you make a pipe or fifo so it's just in RAM until the script reads it?

I welcome any suggestions on that. Currently, it is because there are 2 only options I am aware of:

Call powermetrics once, and read the output. This causes a performance overhead.
Start a powermetrics subprocess and tell it to write to a temporary file, and then read the temporary file.

The temp file should be cleared every time asitop quits or starts. This issue arises when you run asitop for very extended periods of time (or powermetrics as a process fails to stop for some reason).

Start a powermetrics subprocess and tell it to write to a temporary file, and then read the temporary file

I would suggest do this, but launch powermetrics backgrounded, and use a fifo.
If you are not familiar, this should make it all ,make sense:
In a terminal run
mkfifo magic
sudo powermetrics -o magic

Let it run for a while
Run cat magic.
You'll see the output, but the data is gone as soon as it is read. It was never actually on disk. Run cat magic again and see.
Also try a variant where you do sudo powermetrics -o magic in one terminal while doing tail -f magic in another terminal.
You'll see that you're getting all the output, but it's not actually being saved.

You could also just use subprocess to capture what it's sending to stdout.

Thanks, I didn't know about fifo

Can the /tmp/asitop_powermetrics* file be automatically deleted after KeyboardInterrupt? I think this would be useful.


Add sleepingtime plugin

I've created a new plugin on sleepyti.me model, with next 6 cycles times displayed, with colors. And created corresponding folder (Lifestyle).
README updated too.

Nice one

Hey - thanks again for your plugin, in three short days we've added over 60 new amazing plugins. I wonder if you could take a minute to add a bit of metadata to your plugin?
Docs: https://github.com/matryer/bitbar#metadata


Unable to run the project with Xcode Version 15.2 (15C500b)

When I click on the Run button in Xcode, I'm getting the following error message
the Swift tools version specification is missing a label; consider inserting 'swift-tools-version:' between the comment marker and the version specifier
Would you please help? Thank you.

When I click on the Run button in Xcode, I'm getting the following error message
the Swift tools version specification is missing a label; consider inserting 'swift-tools-version:' between the comment marker and the version specifier
Would you please help? Thank you.

I just pushed a version on the ios17-port branch which compiles with Xcode 15.3 on macOS 14.4. The build does throw the warning

Publishing changes from within view updates is not allowed, this will cause undefined behavior.

in PassCodeInputModel.swift line 15 which I intend to look at in the next day or so.
I should really give this a day or two of my time and make it into an installable package.


feat(sqlalchemy): allow repository functions to be filter by expressions

Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

Description

This PR enhances the repository so that you can more easily pass in complex where expressions into the repository functions.

Without this, you have to override the statement parameter and it separates the where conditions from the filters and the kwargs.
Allows usage of this syntax:
locations, total_count = await model_service.list_and_count(ST_DWithin(UniqueLocation.location, geog, 1000), account_id=str(account_id))

instead of the previous method of overriding the statement:
locations, total_count = await model_service.list_and_count(statement=select(Model).where(ST_DWithin(UniqueLocation.location, geog, 1000)), account_id=str(account_id))

Close Issue(s)

@Alc-Alc nice work on this one!

works like a charm, thanks for the quick turnaround on this! Using this syntax locations, total_count = await model_service.list_and_count(ST_DWithin(UniqueLocation.location, geog, 1000), account_id=str(account_id)) it worked as expected on the new branch. And too double check with prior version, it failed as expected with Unexpected filter error

and no issues with my mypy setup for me. but im not getting any errors using the new syntax with the old branch so probably something on my side that i should resolve


REQUEST_FINGERPRINTER_CLASS is not set when exists with priority > addon (15) and no warnings.

If the setting REQUEST_FINGERPRINTER_CLASS is already set, and its priority for some reasons, above 15 (as a default priority for the addon), then the value for the REQUEST_FINGERPRINTER_CLASS is not updated toduplicate_url_discarder.fingerpinter[here] (https://github.com/zytedata/duplicate-url-discarder/blob/main/duplicate_url_discarder/_addon.py#L7). With REQUEST_FINGERPRINTER_CLASS = "scrapy_zyte_api.ScrapyZyteAPIRequestFingerprinter" in the settings, the priority is set to 20, therefore, the addon does not work.
Without REQUEST_FINGERPRINTER_CLASS in settings (using default fingerprints) - it works, as the priority for this case 0.
Conf:
scrapy==2.11
scrapy-zyte-api-0.16.0
scrapy_zyte_api.Addon is not installed

Should it use settings.getpriority("REQUEST_FINGERPRINTER_CLASS"), like in scrapy-zyte-api?


Collaborators need to be unique. Add users based on email, name if they do not already exist.

Done


You can add custom controls to the end layout and resolve sliding and…

1 on the original basis to add custom layout to the header layout
2 Resolve sliding and clicking conflicts

Allows to add custom layouts


Full support for workspace protol

There are some patterns that most likely are not supported well and they should be.

In Yarn docs we we can read something like this here:

If a semver range, it will select the workspace matching the specified version.

What does this mean though? Can one have multiple workspaces with the same name and different versions? cc @arcanis

In addition to that one can use a semver range together with the workspace protol like this:

    "range": "workspace:^1.2.3",

What we should do about this? Should we bump this range to the "latest" when running changeset version? I think the answer is no because if one would like for this to be bumped then they would use workspace:^. So the intention here seems to be to leave this range "as is".
What happens if somebody schedules an incompatible release with such a range? For example what if somebody adds a "range": major changeset? It seems to me that under such circumstances we should actually update this range to the "latest" version (workspace:^2.0.0 in the given example) because without this we'd end up with an invalid monorepo state.

It's also possible to use aliases together with the workspace protocol, see here. I have a strong feeling that this doesn't work correctly and that we should fix this. In fact, this is probably already an issue for regular npm: aliases as I don't remember any line of code referencing this.

I have also noticed that pnpm has a link-workspace-packages config and that seems to strongly correspond to our bumpVersionsWithWorkspaceProtocolOnly:

This protocol is especially useful when the link-workspace-packages option is set to false. In that case, pnpm will only link packages from the workspace if the workspace: protocol is used.

Should we introduce any connection between those? Should we try to validate that bumpVersionsWithWorkspaceProtocolOnly: true when link-workspace-packages: false? Where such validation would happen? In the version command? cc @zkochan @mitchellhamilton

What does this mean though? Can one have multiple workspaces with the same name and different versions? cc @arcanis

It used to be possible very early on, but we removed that. Nowadays we just recommend to use workspace:* or workspace:^.

What we should do about this? Should we bump this range to the "latest" when running changeset version?

I'd say "no" as well - it also matches how yarn npm publish works (we replace workspace:* by exact ranges, and workspace:^ by caret ranges, but leave other specifiers untouched). Probably the last use case for non-*/^  ranges.

@arcanis thank you for the response ❤ The answer to the second question matches my reasoning - could you also confirm the other part?

What happens if somebody schedules an incompatible release with such a range? For example what if somebody adds a "range": major changeset? It seems to me that under such circumstances we should actually update this range to the "latest" version (workspace:^2.0.0 in the given example) because without this we'd end up with an invalid monorepo state.

In such a scenario it seems like we should actually adjust the workspace range. I'd only like to make sure we don't break any expectations by doing so. The whole versioning thing is quite subtle and I want to be extra careful about anything we do related to this.

It seems duplicate of https://github.com/atlassian/changesets/issues/598

@disjukr thanks for noticing this - those tickets are highly related. I feel the other one is about publishing process - while my intention behind this one was improving the situation with changeset version, so I think there is still a reason to keep both open. I'm actually in the process of fixing the publishing part of things - but this won't resolve this ticket here (yet)


refactor(bindings): change default bindings

For partial completions. These are not set to anything by default on
insert mode and are not used unlike the arrow keys.

I might be wrong, as I don't really know any vim plugin syntax, but it looks to me like this is half a solution. The same change needs to be made in the line after the current change as well for it to work. At least, that's what I had to do locally.

I might be wrong, as I don't really know any vim plugin syntax, but it looks to me like this is half a solution. The same change needs to be made in the line after the current change as well for it to work. At least, that's what I had to do locally.

yes, there is already pending PR for this https://github.com/Exafunction/codeium.vim/pull/398


Type of a generic is not narrowed

Bug Report

🔎 Search Terms
discriminated type union, narrowing, generics

🕗 Version & Regression Information

This changed between versions 3.3.3 and 3.5.1

⏯ Playground Link

Playground link with relevant code
💻 Code

type A = {
    s: string;
    n: number;
};

// K here may be 's'|'n' or 's' or 'n' (but not never as I understand, as f cannot accept argument of type never in runtime)
function f<K extends keyof A>(key: K): A[K] {
    const k1: typeof key = 's';    // It is completely fine for this line to have an error (as it is outside if check and K may be 'n')

    if (key === 's') {
        // In fact we know that type of x === 's' here.
        // Thus return type here should be A['s'] in other words it should be a string
        // But TypeScript still thinks it is of type A[K]

        const k2: typeof key = 's';   // does not work. Here I expect typeof key to be 's'
        const c: A[K]   = 'test';     // does not work. Here I expect A[k] to be string
        const d: A['s'] = 'test';     // works

        return 'test';                // does not work. Here I expect 'test' to be asignable to A[K]
    }

    if(key === 'n') {
        return 10;                    // the same
    }

    throw new Error();
}

🙁 Actual behavior
Typescript ignores the check on the variable typed with a subtype of union type and is unable to narrow type
🙂 Expected behavior
Typescript narrows the type
My guess is that this behavior may somehow be related to https://devblogs.microsoft.com/typescript/announcing-typescript-3-5/#fixes-to-unsound-writes-to-indexed-access-types

Looks similar to https://github.com/microsoft/TypeScript/issues/49676

Or #33014


Indicating when an account does not exist

It should be possible to clearly indicate that an account does not exist when it does not exist.
The current error message makes me think that there is some error on the server side.
https://livenet.xrpl.org/accounts/rndpDXFdkgR4EmDXnPgC36VxRBbjqu8rHP

For that url I am getting this.

I checked and Chrome is showing Account Not Found.
In Safari it is the error I noted.
It contains a little Japanese, but the error is as follows.
[Error] TypeError: Error.captureStackTrace is not a function. (In 'Error.captureStackTrace(this,this.constructor)', 'Error.captureStackTrace' is undefined)
gt — index-49a35023.js:150:76995
（anonymous関数） — index-49a35023.js:150:79134

    onError (index-49a35023.js:79:41594)
    u (index-49a35023.js:79:35275)
    （anonymous関数） (index-49a35023.js:79:35957)

This is caused by Firefox and Safari not supporting Error.captureStackTrace.  This code has been there for at least 5 years.  After I fix this you will see more context aware error messages throughout the site.


2022.04a Iteration Plan

SAP teams plan to address the following issues in our next iteration:

Close PR #338
Close PR #348

The iteration lasts 2 weeks and ends on April 25, 2022.

Iteration Results
Closed

Close PR #348

Open

Close PR #338


Revert "Revert "feat: rename and move install and run page, create unique pages for every combination""

Reverts werf/website#720

outdated


Icons not showing in the admin panel

The icons on the admin panel do not show on Firefox (Iceweasel 31.5.0, Linux 64bit). With Google Chrome (41.0.2272.76 (64-bit)) icons work as supposed.
Attached a picture where firefox is on the left, chrome on the right.

Ghost version 0.5.10 (production)

Hi @laite, thanks for reporting this!
I'm not too familiar with Linux, is 31.5.0 the latest version of Firefox? I ask because the latest version on OS X is 36.0.1, where the icons work as expected.

I'm sorry, it was probably just a caching issue. I restarted the server application and now everything's working once again.
Sorry for the trouble.

Hey, I've noticed this lot of times already: it has nothing to do with linux. its because of the cache expire times: you dont need to restart the application just reset the browser cache: ctrl-shift-r in firefox for example. ive got these when I've restarted ghost a whole bunch of times and changed a few things here and there..

I'm having a similar issue, using the latest version of Chrome (Version 41.0.2272.104 (64-bit)) on OSX, ghost is self-installed. Instead of weird icons, I can't see the icons at all.
I've tried command+shift+r combo several times, no help.
Any ideas?
Thanks in advance,

Hi @Ardakilic, could you give us a few more technical details about your environment at all? Version, where you downloaded, server, etc. There's a template here :)
I just installed a fresh copy of Ghost 0.5.10 from ghost.org/download, on OS X with Chrome 41.0.2272.104 64bit, and it all worked fine, though I'm really keen to work out what this issue is.

@PaulAdamDavis
Hello, thanks for the interest.
Here's the template, filled as requested
Issue Summary
Admin panel icons are not shown. Clearing the cache does not change the situation.
Steps to Reproduce

Sign in to the admin panel.
Various icons are not shown

Some screenshots:
http://prntscr.com/6lxmmv
http://prntscr.com/6lxmu4
In freshly installed firefox (I've installed just to test, first time installed in this OS), icons are shown, but like this: http://prntscr.com/6lxntt
Technical details

Ghost Version: 0.5.10, installed from release, updated from older versions. Self-hosted on a VPS.
Client OS: Mac OSX Yosemite
Server OS: Ubuntu 14.04.2 LTS x64
Node Version: v0.10.38, npm version: 1.4.28, my node source is deb-src https://deb.nodesource.com/node_0.10 trusty main
Browser: Chrome (Version 41.0.2272.104 (64-bit)) on OSX, Vivaldi <IP_ADDRESS> (Developer Build) OSX,
Database: SQLite

@PaulAdamDavis:
The reason I tough it might have something to do with expry times is because it only occured to me with installs that has been around for at least a few hours (5+) :)

It is happening to me too. In firefox when I upgraded to 0.5.10 on Digital Ocean instance. chrome works fine:

@letsjustfixit @PaulAdamDavis
I have updated (actually re-installed) my ghost version, downloaded from http://ghost.org/download, updated according to this link:
http://support.ghost.org/how-to-upgrade/#how-to-upgrade
Same issue keeps on happening.
Any ideas ? Should I download from GitHub Release instead ?
Thanks in advance,

As there's been a few new reports of this here and in #5057, I'm going to reopen this and investigate.

Hey,
I'm only a simple user and force-redownloading the static assets always solved this issue in my case both in linux (Debian 7.8 wheezy, or 8 Jessie),windows 7&8.1 so my only advise is try downloading the css and font files and check if they work as they should..
You should also check your browser addons which might modify the displayed end results. (just a tought).
Sorry I cant be of any more help.

I have not been able to reproduce this by upgrading from 0.5.9 to 0.5.10. However, I have briefly seen the squares when loading the page via a slow proxy. I can also manually simulate the effect by toggling font-family: "GhostIcons"; on the icons in the dev tools.
Between 0.5.9 and 0.5.10 the icon font was regenerated to change the unicode range prefix from "\e..." to "\f..." (to fix a an issue where Emoji would replace the icons on iOS). It's possible that the process there caused this issue, though it seems to affect some people and not others, so I don't think it's that.
Other than the older icon font being heavily cached, I'm at a loss as to the cause :disappointed: From what I can see, nothing else that should affect this was changed between 0.5.9 & 0.5.10.

@PaulAdamDavis I am quite certain that this is a caching issue, as icon fonts are the only thing which are not currently managed by Ghost's internal asset handing. We have heavy caching of assets, a hard refresh should resolve the issue in the majority of cases, but there is no way to guarantee that all caches will behave the same.
To resolve this, my recommendation would be to add a versioning query string to the icon file path inside the scss files.  i.e. change src:url('fonts/icons.woff') to src:url('fonts/icons.woff?v=0.1') here and bump the version number each time the icon font gets regenerated. This will ensure that every cache will fetch a new copy of the file.
Long term, Ghost should have an asset pipeline which handles all of this automatically. Short term this will prevent these issues from occuring.

Just wanted to add that I saw this issue on Firefox 36.0.4 on OSX 10.10.2. Hard refresh /did not/ fix the problem, emptying my browser cache /did/ fix the problem. I also found that (while Firefox was exhibiting the issue) I couldn't reproduce the problem in a fresh installation of Chrome that I had never visited my admin interface with. I'd have to agree that this is a caching issue.

@sjahl Brilliant, thanks for the clarification!
And thank you @ErisDS :) Will do that now.

@ErisDS Thanks for the reply!
@PaulAdamDavis I have altered the ghost.min.cssmanually like this commit that you've just did: https://github.com/TryGhost/Ghost/commit/72ad4f72c640da61eabd78bf871c4ebb5270f12c and I can confirm icon issue is fixed on my case. :smile:

HI,
I just installed the empty cache plugin in Firefox and after running it once, the icons were back.
https://addons.mozilla.org/en-US/firefox/addon/empty-cache-button/
Cheers,


Have the FontLoader tell the PaintingBinding _systemFonts to notify its listeners

This will result in so the RenderParagraphs repainting with the new font. It can be triggered by calling:
PaintingBinding.instance.handleSystemMessage({'type': 'fontsChange'});
May just need to add that after awaiting FontLoader.load

@clocksmith do you have a code example that i can test if my fix works?

Here's a reproducible code sample:
The below app only works on the first build when you uncomment the PaintingBinding line.
import 'dart:typed_data';

import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:http/http.dart' as http;

Future<ByteData> fetchFont() async {
  final response = await http.get(
      'https://raw.githubusercontent.com/google/fonts/master/ofl/montserrat/Montserrat-Light.ttf');

  if (response.statusCode == 200) {
    return ByteData.view(response.bodyBytes.buffer);

  } else {
    // If that call was not successful, throw an error.
    throw Exception('Failed to load font');
  }
}

void main() {
  var fontLoader = FontLoader('Montserrat');
  fontLoader.addFont(fetchFont());
  fontLoader.load().then((_) {
//    PaintingBinding.instance.handleSystemMessage({'type': 'fontsChange'});
  });

  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Custom Fonts',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: Text('Custom Fonts')),
      body: Center(
        child: Text(
          'Montserrat is displayed here',
          style: TextStyle(fontFamily: 'Montserrat'),
        ),
      ),
    );
  }
}

Also, I think there's something funny going on with web. When I try the above example on web with longer text and the PaintingBinding.instance.handleSystemMessage({'type': 'fontsChange'}) line uncommented, it doesn't quite work on web.

I don't think it's rebuilding quite right. Although as soon as I resize the window by any amount, it rebuilds and looks correct.

@johnsonmh the web issue you mentioned above is a separate issue. I file this bug https://github.com/flutter/flutter/issues/47236
We can continue the discussion there. For this issue, let's focus on the FontLoader should notify PaintingBinding.


feat: add package icon

Shows up on www.catalog.kurtosis.com

derek got you

@leeederek you did it wrong

Closing this in favor of #413


Update CONTRIBUTING.md

Link update
@devonfw/cobigen

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

@mandybuchhold please sign the cla, then we can merge it.


Docker: scrapper app

Use docker compose to start the app

miss click - i didn't want to open a PR :see_no_evil:


"Alfresco Share" link isn't working at /alfresco path

If you try to access the /alfresco path, you will see a page with a bunch of links.  The second and third links are "Alfresco share" and "Aflresco webDAV".
I've notice that clicking on the share link fails to resolve, as the url has 'localhost' in it.  However, the webDAV link works fine.  I think that both should correctly resolve to the whatever IP address or url that webDAV is dynamically calculating.

Hi @borromeotlhs, the share link should be using whatever is set in alfresco-global.properties. Can you access https://HOSTNAME:8080/share?

yes.  I'm looking at that page and noticing that the URL for itself (share)
resolves to local host, while the WebDAV correctly resolves to its IP
address.
On Thu, May 11, 2017, 13:00 Jason Han<EMAIL_ADDRESS>wrote:

Hi @borromeotlhs https://github.com/borromeotlhs, the share link should
be using whatever is set in alfresco-global.properties. Can you access
https://HOSTNAME:8080/share?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Open-MBEE/mms/issues/27#issuecomment-300852471, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABYaeqE88O-wmJ_RU7iawsB0czhHFOmjks5r4z6jgaJpZM4NYCpf
.

For reference, the hostname that is used to generate the links from the /alfresco page is within alfresco-global.properties. We use an nginx proxy to reroute requests for /share properly, but this setting should be set if you want all links working from that page.


libev: 4.27 -> 4.31

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libev/versions.
meta.description for libev is: '"A high-performance event loop/event model with lots of features"'.
meta.homepage for libev is: '""

Checks done (click to expand)

built on NixOS

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

0 of 0 passed binary check by having a zero exit code.

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

found 4.31 with grep in /nix/store/cspcsfkwfw25n49c20blk00x58drgi75-libev-4.31

directory tree listing: https://gist.github.com/49b48aef087fd09767cc04cdce248ada

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

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

75424 total rebuild path(s)
29541 package rebuild(s)
18524 x86_64-linux rebuild(s)
16995 i686-linux rebuild(s)
23356 x86_64-darwin rebuild(s)
16549 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
AgdaSheaves
AgdaStdlib
CoinMP
DisnixWebService
EBTKS
EmptyEpsilon
Fabric
LASzip
Literate
MMA
NSPlist
OVMF
OVMF-CSM
OVMF-secureBoot
PlistCpp
QuadProgpp
R
SDL
SDL2
SDL2_gfx
SDL2_image
SDL2_mixer
SDL2_net
SDL2_ttf
SDL_Pango
SDL_gfx
SDL_gpu
SDL_image
SDL_mixer
SDL_net
SDL_sound
SDL_stretch
SDL_ttf
SPAdes
SkypeExport
Sylk
TotalParserCombinators
Xaw3d
_0x0
_1password
_20kly
_2bwm
_389-ds-base
_3llo
_6tunnel
_90secondportraits
a2jmidid
a52dec
aalib
abbaye-des-morts

Instructions to test this update (click to expand)

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

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

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

cc @7c6f434c for testing.

Changelog looks fine


(docs): public_key_hashes in getIdentitiesByPublicKeyHashes isn't double-SHA256

In the DAPI documentation, public_key_hashesis described as double-SHA256  of Identity public keys (see image below):

However, public_key_hashes generated in js-dpp is actually sha256-ripemd160 of Id PubKeys, NOT double-SHA256 as stated in the docs (see the image below):

Clearly, there’s a little discrepancy between what’s in the docs and what’s implemented in js-dpp library, and following what's in the docs does not return the requested Identities as shown below:
identities: "\200"
metadata {
  height: 5322
  core_chain_locked_height: 737718
}

Issue fixed
https://dashplatform.readme.io/docs/reference-dapi-endpoints-platform-endpoints


Building PIP packages with PyTorch/Cuda dependencies

Checks

[X] I have checked that this issue has not already been reported.
[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
[project]
authors = ["Name"]
channels = ["nvidia/label/cuda-12.4.0", "nvidia", "conda-forge", "pytorch"]
description = "Add a short description here"
name = "sandbox"
platforms = ["win-64"]
version = "0.1.0"

[dependencies]
python = ">=3.8,<3.11"
pip = ">=24.0,<25"
cuda = "*"
pytorch = ">=2.4.1,<2.5"
pytorch-cuda = "*"
torchvision = "*"
ninja = "*"

[pypi-options]
no-build-isolation = ["diff-surfel-rasterization"]

[pypi-dependencies]
# This requires CUDA_HOME to be set, so this step fails when running `pixi shell`
diff-surfel-rasterization = { git = "https://github.com/hbb1/diff-surfel-rasterization.git" }

[tasks]
# I find that running `pixi run surfel-install` I am able to install the package
# surfel-install = "pip install git+https://github.com/hbb1/diff-surfel-rasterization.git@7bdbd51"

Issue description
Hi. Consider the provided pixi.toml file. I find that I cannot build the environment on my Windows machine. Specifically it fails to build the pip package from source, with the following error message:
❯ pixi shell
  ⠈ default:win-64       [00:00:06]
  × failed to solve the pypi requirements of 'default' 'win-64'
  ├─▶ failed to resolve pypi dependencies
  ├─▶ Failed to download and build `diff-surfel-rasterization @ git+https://github.com/hbb1/diff-surfel-rasterization.git`
  ╰─▶ Build backend failed to determine metadata through `prepare_metadata_for_build_wheel` with exit code: 1
      --- stdout:

      --- stderr:
      Traceback (most recent call last):
        File "<string>", line 14, in <module>
        File "C:\devel\test-pixi\.pixi\envs\default\lib\site-packages\setuptools\build_meta.py", line 373, in prepare_metadata_for_build_wheel
          self.run_setup()
        File "C:\devel\test-pixi\.pixi\envs\default\lib\site-packages\setuptools\build_meta.py", line 502, in run_setup
          super().run_setup(setup_script=setup_script)
        File "C:\devel\test-pixi\.pixi\envs\default\lib\site-packages\setuptools\build_meta.py", line 318, in run_setup
          exec(code, locals())
        File "<string>", line 22, in <module>
        File "C:\devel\test-pixi\.pixi\envs\default\lib\site-packages\torch\utils\cpp_extension.py", line 1076, in CUDAExtension
          library_dirs += library_paths(cuda=True)
        File "C:\devel\test-pixi\.pixi\envs\default\lib\site-packages\torch\utils\cpp_extension.py", line 1214, in library_paths
          paths.append(_join_cuda_home(lib_dir))
        File "C:\devel\test-pixi\.pixi\envs\default\lib\site-packages\torch\utils\cpp_extension.py", line 2416, in _join_cuda_home
          raise OSError('CUDA_HOME environment variable is not set. '
      OSError: CUDA_HOME environment variable is not set. Please set it to your CUDA install root.

I am on pixi 0.29.0. I find that if I run the pip install step as separate command (via pixi run surfel-install), then I can build the CUDA kernels as expected. I would appreciate any guidance you have on this particular issue. I couldn't find any other issue related to a missing CUDA_HOME env variable. The closest I could find is this conda issue (https://github.com/conda/conda/issues/7757). Since I'm able to build by running pixi run surfel-install, it seems it's a bug/missing feature in pixi but let me know if I'm wrong.
Expected behavior
Can build provided pixi.toml file.

Actually I would've expected the env variable to be set by the CUDA activation script.
These should be passed to the PyPI build. @baszalmstra do you know something about the 'CUDA_HOME' env variable?

Im also pretty sure this should be aet by the activation script. Could you check by going into the shell and echoing it?
@tdejager do we actually run the activation script and pass those variables to uv?

We do pass env variables to uv for sure, otherwise things like c compilers wouldn't work also. And I believe we use the 'activator' for that.

If I remove the surfel package, after entering the pixi shell running echo $env:CUDA_HOME shows as empty. I have no idea how the manual pip install is able to succeed. Do you think it's a problem with uv itself?


Add documentation for showing errors

This PR will add documentation on how a person can show errors through controller and by using traits.
Please let me know if there's any issues with this PR, thank you.

@lindyhopchris thanks for the recommendations. I've made the necessary changes, please let me know if there's anything else, thanks!


Cannot add --skipApiVersionCheck when using testcontainers

Which service(blob, file, queue, table) does this issue concern?
Blobs
Which version of the Azurite was used?
Latest - there's no version listed here: https://hub.docker.com/r/microsoft/azure-storage-azurite, but that's what I am using.
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
DockerHub
What's the Node.js version?
What's inside the image
What problem was encountered?
Cannot figure out how to pass in --skipApiVersionCheck
Steps to reproduce the issue?
If possible, please provide the debug log using the -d parameter, replacing <pathtodebuglog> with an appropriate path for your OS, or review the instructions for docker containers:
            var builder = new ContainerBuilder()
                .WithImage("mcr.microsoft.com/azure-storage/azurite")
                .WithPortBinding(10000, true)
                .WithPortBinding(10001, true)
                .WithPortBinding(10002, true)
                .WithName($"{_namePrefix}-azurite")
                .WithWaitStrategy(Wait.ForUnixContainer().UntilMessageIsLogged("Azurite Table service is successfully listening at"));

            if (_skipVersionCheck)
            {
                builder.WithCommand("--skipApiVersionCheck");
            }

            Container = builder.Build();

            await Container.StartAsync();

Getting the following error:
   Azure.RequestFailedException : The API version 2024-08-04 is not supported by Azurite. Please upgrade Azurite to latest version and retry. If you are using Azurite in Visual Studio, please check you have installed latest Visual Studio patch. Azurite command line parameter "--skipApiVersionCheck" or Visual Studio Code configuration "Skip Api Version Check" can skip this error.
   RequestId:7146ecf5-712b-4593-b754-61959a659270
   Time:2024-07-19T08:43:03.147Z

Please be sure to remove any PII or sensitive information before sharing!
The debug log will log raw request headers and bodies, so that we can replay these against Azurite using REST and create tests to validate resolution.
Have you found a mitigation/solution?
No - tried adding it via WithEntryPoint and that didn't work either.

@ShaharPrishMSFT
This doc has show how to start Azurite docker with "--skipApiVersionCheck"
https://hub.docker.com/r/microsoft/azure-storage-azurite#:~:text=tableHost <IP_ADDRESS> --loose ---,skipApiVersionCheck,---disableProductStyleUrl
For how to input the parameter with the API you shows above, this is out of Azurite scope. You might can contact the API owner to get it.

If you are using Testcontainers you should take a look on this https://java.testcontainers.org/features/commands/
this is how you pass commands at least this is what its documented there.

@ShaharPrishMSFT is it possible your _skipVersionCheck is false here? This works for me:
private readonly AzuriteContainer _azuriteContainer = new AzuriteBuilder()
    .WithImage("mcr.microsoft.com/azure-storage/azurite:latest")
    .WithCommand("--skipApiVersionCheck")
    .Build();


Update to Checker Framework 3.24.0

Apart from staying up to date, this pulls in https://github.com/typetools/checker-framework/pull/5178 (and follow-up https://github.com/typetools/checker-framework/pull/5210) which should give us a bit of an overhead reduction, particularly for large methods

Pull Request Test Coverage Report for Build #899

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.488%

Totals

Change from base Build #898:
0.0%

Covered Lines:
4802

Relevant Lines:
5192

💛  - Coveralls

Here are the benchmarking numbers.
With this PR:
Benchmark                          Mode  Cnt   Score   Error  Units
AutodisposeBenchmark.compile      thrpt   25  10.633 ± 0.049  ops/s
CaffeineBenchmark.compile         thrpt   25   2.497 ± 0.020  ops/s
DFlowMicroBenchmark.compile       thrpt   25  28.560 ± 0.364  ops/s
NullawayReleaseBenchmark.compile  thrpt   25   1.509 ± 0.014  ops/s

Before:
Benchmark                          Mode  Cnt   Score   Error  Units
AutodisposeBenchmark.compile      thrpt   25  10.643 ± 0.087  ops/s
CaffeineBenchmark.compile         thrpt   25   2.436 ± 0.018  ops/s
DFlowMicroBenchmark.compile       thrpt   25  15.605 ± 0.295  ops/s
NullawayReleaseBenchmark.compile  thrpt   25   1.449 ± 0.014  ops/s

There is a huge improvement on the dataflow micro-benchmark (as expected), and a small (but outside the noise) improvement for NullAway itself.  Surprisingly, there seems to be a small overhead regression for Caffeine; I don't really understand why that would happen.  I will see if that repros on another run.


perf: isolate queues to entity-based objects instead of global class level

will do - will handle global queue issue by having a centrally defined object in a static imports file & then wrap it with a semaphore for extended access through a single point

stale - closing since not necessary at this point in time


Clean up action names and noise in sagas

For speaker schedule and updating current time

@GantMan ready for your review...

thanks so much!!!


failed to send alert data to VictorOps: failed to understand VictorOps response

Here is my config.
enabled = true
api-key = "xxxxx"
routing-key = "everyone"
url = "https://alert.victorops.com/integrations/generic/20131114/alert"
global = false

@devilldog  Which version are you using? There should be a more verbose error logged about how the communication with VictorOps failed.

10.0-1

If you upgrade to v0.10.1-1 you will get a better error message and we can
debug further.
Nathaniel Cook
Kapacitor Lead
▼▴
InfluxData.com http://influxdata.com/
Twitter http://www.twitter.com/nathanielvcook
On Fri, Feb 12, 2016 at 1:44 PM, devilldog<EMAIL_ADDRESS>wrote:

10.0-1
—
Reply to this email directly or view it on GitHub
https://github.com/influxdata/kapacitor/issues/232#issuecomment-183481813
.

OK Will do thanks !

I upgraded to 0.10.1 but i get the sam message for VictorOps as before and i enabled DEBUG in the logs

quickly looking at the code for a language i do not know, it looks like the error is not being returned and it is not actually printing out the error just the proceeding line "failed to understand response"

any thoughts ?

@devilldog The relevant line in the code is https://github.com/influxdata/kapacitor/blob/master/services/victorops/service.go#L83
which defaults the error message to "failed to understand VictorOps response: " + string(body) where body is the raw response from VictorOps. You should see something like:
[task_name:alert5] 2016/02/17 12:26:04 E! failed to send alert data to VictorOps: failed to understand VictorOps response: RAW_RESPONSE

Can you post the logs you are seeing relating to the error?

[load_alert:alert2] 2016/02/14 19:22:09 E! failed to send alert data to VictorOps: failed to understand VictorOps response:
[load_alert:alert2] 2016/02/14 19:22:09 E! failed to send alert data to VictorOps: failed to understand VictorOps response:
[load_alert:alert2] 2016/02/14 19:22:09 E! failed to send alert data to VictorOps: failed to understand VictorOps response:
[load_alert:alert2] 2016/02/14 19:22:29 E! failed to send alert data to VictorOps: failed to understand VictorOps response:
[load_alert:alert2] 2016/02/14 19:22:39 E! failed to send alert data to VictorOps: failed to understand VictorOps response:
[load_alert:alert2] 2016/02/14 19:22:39 E! failed to send alert data to VictorOps: failed to understand VictorOps response:
[load_alert:alert2] 2016/02/14 19:22:59 E! failed to send alert data to VictorOps: failed to understand VictorOps response:
[load_alert:alert2] 2016/02/14 19:23:19 E! failed to send alert data to VictorOps: failed to understand VictorOps response:

rpm -qa|grep kapac
kapacitor-0.10.1-1.x86_64

@devilldog I think I have finally reproduced the issue! VictorOps will return a 404 with no content when the API key is not valid. I have a fix coming that will print out a better error message indicating the actual error.
Can you double check your API key to make sure it is correct?

yeah i came to the same conclusion, i will test and let you know thanks !


Replication mongodb

Hello.
I need to connect replication mongodb.
I used code below:
'social_mongodb' => [
'driver'   => 'mongodb',
'host'     => array('server1','server2','server3'),
'port'     =>27017,
'database' => 'database',
'username' =>'user',
'password' => 'pass',
'options'  => ['replicaSet' => 'replicaSetName']
],
But laravel show message :
No suitable servers found (serverSelectionTryOnce set)
Help me!!

It will be easier to help if you post the rest of the error message.
I, for instance, had a similar issue and what was missing was the 'database' => 'authentication db name' in the options array.

setting 'replicaset' in the option worked for me and solve my issue getting the error message 'no master"


Wikidata schema - SPARQL property path

if i enter directly the property id in the wikidata schema editor: the autocomplete dropdown list shows me 2 entries - the property itself and an entry called SPARQL: PID property path.
i don't know if this is necessary under some special conditions.

(OpenRefine 3.2 beta)

Moving to appropriate repository: https://github.com/wetneb/openrefine-wikibase/issues/62


qa/suites/krbd: changing rbddefault feature to 1

Signed-off-by: harishmunjulur<EMAIL_ADDRESS>

http://pulpito.ceph.redhat.com/hmunjulu-2020-04-29_14:00:12-krbd:wac-nautilus-distro-basic-pluto/
http://pulpito.ceph.redhat.com/hmunjulu-2020-04-29_14:07:35-krbd:thrash-nautilus-distro-basic-pluto/


Laravel 5.2 Support

Pagination not working...
Gives error if search query returns zero results : ErrorException in Collection.php line 1040: Undefined offset: 0

+1

+1

+1

+1

I believe I have the same issue with pagination.
In laravel 5.2 when using pagination bindings are lost.
Searchable adds bindings for select group, but then in https://github.com/illuminate/database/blob/master/Query/Builder.php#L1548 bindings are temporary removed for select group so query throws exception.
protected function backupFieldsForCount()
    {
        foreach (['orders', 'limit', 'offset', 'columns'] as $field) {
            $this->backups[$field] = $this->{$field};
            $this->{$field} = null;
        }
        foreach (['order', 'select'] as $key) {
            $this->bindingBackups[$key] = $this->bindings[$key];
            $this->bindings[$key] = [];
        }
    }

So I've replaced select to where in https://github.com/nicolaslopezj/searchable/blob/master/src/SearchableTrait.php#L308 and pagination query started to work.
    protected function addBindingsToQuery(Builder $query, array $bindings) {
        $count = $this->getDatabaseDriver() != 'mysql' ? 2 : 1;
        for ($i = 0; $i < $count; $i++) {
            foreach($bindings as $binding) {
                $type = $i == 0 ? 'where' : 'having';
                $query->addBinding($binding, $type);
            }
        }
    }

I didn't create PR because not sure the reason why select was selected there.

+1 for this

+1

+1

+1

+1

Hi, nicolaslopezj.
I know this way is not smart but I guess there's no way to use raw query like this because this issue is caused by laravel core.
protected function getSearchQuery(Builder $query, $column, $relevance, array $words, $relevance_multiplier, $pre_word = '', $post_word = '')
{
$like_comparator = $this->getDatabaseDriver() == 'pgsql' ? 'ILIKE' : 'LIKE';
$cases = [];
foreach ($words as $word)
{
$case = $this->getCaseCompare($column, $like_comparator, $relevance * $relevance_multiplier);
$cases[] = str_replace(
'?',
DB::connection()->getPdo()->quote($pre_word . $word . $post_word),
$case
);
}
return implode(' + ', $cases);
}

@nicolaslopezj plz solve this problem. It is destroying my project.

thanks @antongorodezkiy.. comment on Feb 24 works for now..

#126 should solve the problem


Story/continuation mode

Is your feature request related to a problem? Please describe.
Ollama and cloud APIs support simple text generation (continuations) in addition to chat, but this is not exposed in WebUI.
Describe the solution you'd like
Maybe start with something like this: https://github.com/oobabooga/screenshots/raw/main/print_default.png
With some shortcuts (tab to accept next word from continuation) and automatic regeneration when the prompt changes, this would allow for fairly convenient assisted writing. KV cache will keep it performant unless you go back and edit something at the beginning of the prompt.
Another option is to offer several short continuations.
Describe alternatives you've considered
Instructing the model to continue the story is unwieldy, requires instruction-tuned model, and works poorly with smaller models. It's possible to use a specialized tool, but I already have Ollama & WebUI set up (I have not researched the market comprehensively yet, so not sure what's available).

Hi, have you tried this?

@jannikstdl That's not what I have in mind. That button just encourages the LLM to write a longer response. I need the LLM to suggest the next few words for what I am writing. Base models are intended to be used like that. I could edit the response to contain beginning of what I am writing and then request continuation, but that's impractical unless the LLM is able to predict long stretches of text reliably, which the small ones cannot do. I need something more streamlined. The other issue is that this uses instruction-tuned model, which will insist on being an assistant,  whereas (I hope that) the base models available in Ollama are more flexible.

I am thinking of something akin to NovelAI.

I think this is something like the mention feature.

But as far as i know its WIP.

@jannikstdl Still not it. I don't want to produce a conversation between two AIs/characters. I want to write arbitrary text and have the AI suggest next few words / list items / even whole paragraphs. Try the free NovelAI (no registration) to see what I mean.


TerraFirmaCraft open inventory crash - fabric-item-group-api

Mod Name
TerraFirmaCraft
Mod Homepage
https://github.com/TerraFirmaCraft/TerraFirmaCraft
Minecraft version
1.20.1
Describe the bug
Open the inventory with clicking "e" in a singleplayer world crashes with a error message. the game
Steps to reproduce

Install TerraFirmaCraft
Install Connector
Create a world
Open the Inventory in a singleplayer world

Logs
https://gist.github.com/Akoto090/b0dfdadef4ded32cbef82612af472273
Additional context
I also tried it on forge, same crash report
If you are trying to use Neoforge, change the dependencies in the mods.toml in TerraFirmaCraft
Version range must be: versionRange = "[47.1.3,47.1.6),[47.1.96],[47.2.6,)"
If you are to busy use this TFC build

I tested this on both Forge 47.1.3 and NeoForge 47.1.96 but couldn't reproduce the issue. Does it occur constantly?

Sorry, I think it was a user error.
I renamed the forgiefied fabric api from "jar" to "disabled" one time and I think this corrupted the jar for some reason, also after I renamed it again, that it crashed while opening the inv (for some reason)
I now installed a new forgiefied fabric api on a new profile and it works.
Im so sorry if I wasted your time, I tried everything before and really thought it was just an error

False report, im sorry


feat: add anti-affinity to stateful set

closes #820
/deploy #persist

@Panaetius I checked and when the keys are duplicated then the one key later in the file is the one used. So the way that you have set things up things should work.

You can access the deployment of this PR at https://renku-ci-nb-915.dev.renku.ch

I cannot replicate the previous failure of the integration tests. I think it was caused by some changes happening in the infra at the time where most of the nodes were scaled down a lot and it was hard to schedule things. Rerunning the tests cleared this.
In addition the acceptance tests fail on starting an anonymous session. I tested this in the CI deployment and things work as expected - so it is just a fluke. I can launch an anonymous sessions without a problem.