feat(gui): add fullscreen drag-n-drop through a directive

We add a directive file-dropper that makes the element it's attached
to into a file-drop area.
Changelog-Entry: Expand drag-n-drop area through a directive.
Closes: #848
Closes: #849

This looks and works fine, however there are many things to be done to property included into the Etcher UI components collection:

The dropzone directive is meant to be a generic utility handle drag and drop functionality, so it shouldn't have the "image" related text and icons in any way. We could accept an SVG icon, a title, and a subtitle as attributes to the directive. Also, showing an "overlay" should be optional, for example, only if the attributes above are provided, or if its enabled with a boolean directive option.

The overlay style currently always occupies a 100% of the screen by using position: absolute, but the overlay should be automatically calculated to occupy a 100% of the surface of the element where the directive is instantiated, using a combination of position: relative (in a top-level/wrapper element), and position: absolute.

The scss code should live inside the directive component. For example: lib/gui/os/dropzone/styles/_overlay.scss.

The list of supported file formats should be obtained from the SupportedFormatsModel model instead of hardcoding it, so they get updated automatically.

The ImageSelectionController controller is part of the main page module, so shouldn't be instantiated in index.html at all.

The .drop-overlay set of elements is declared outside the directive (e.g: in index.html), and its class its hardcoded in the dropzone directive, and assumed to be present already "somewhere". The code that instantiates the overlay elements on the template should be either contained on the directive, or its reference should be passed as an argument.

Other minor things:

There are some indentation issues in some places.
The SVG icon file contains hardcoded weight/height in the XML that should be removed.
We should set the cursor CSS rule appropriately in the overlay, so it displays the usual "drag 'n drop" cursor icon.
The overlay is lacking the dashed border.

Here's what I recommend:

Lets create a new high level component (in lib/gui/components) and lets keep dropzone very low-level. The new component will expose an element directive that re-uses dropzone.

What do you think?

I have a couple of questions.

The overlay style currently always occupies a 100% of the screen by using position: absolute, but the overlay should be automatically calculated to occupy a 100% of the surface of the element where the directive is instantiated, using a combination of position: relative (in a top-level/wrapper element), and position: absolute.

The ImageSelectionController controller is part of the main page module, so shouldn't be instantiated in index.html at all.

Following this, wouldn't it make sense to use a HTML element higher up in the DOM (and within index.html) if we want the overlay to take up the whole screen? If so, ImageSelectionControllers scope is unavailable there so we'd have to use a service through which we share the images?

@jviotti I've just pushed some changes here.
I'm not happy about just copy/pasting this code but I'm not sure how to refactor, since you can't simply import the current ImageSelectionController (to my knowledge). My simple guess would be it's better to have that logic in a service somewhere so it can be accessed globally?
Also maybe it's not the cleanest to attach the directive to the body element, but I'm not sure here. Maybe I could use a wrapper element around , , and  which the directive already does behind the scenes using the template.
I think the cursor is what it's supposed to be without custom CSS, but I may be wrong here; I haven't tested outside of GNOME yet.

Man, this looks and works really, really well :D I have some ideas on how to decouple the file picker from selection state, but let me play around with them so I ensure they are feasible and make sense :)

I've been playing around with this quite a lot and I think I have a way forward. First, there are several limitations of this implementation that we must solve:

The file dropper directive hardcodes the path to the icon it will show. This should be configurable, probably by accepting the path as an argument to the directive.

The file dropper directive hardcodes the text. This should be configurable as well.

If the directive is used as an attribute, it will transclude the element and surround it with another element (defined in the directive). This works well for simple cases, but if the client's CSS relies on certain structure, the structure will be broken.

The directive hardcodes logic to select an image in response.

The file dropped will be shown in all pages, even when it doesn't make sense (e.g: in the settings page).

So here's my proposal:

Let's only allow the directive to be instantiated as an element. The user will use it as:

<file-dropper>
   ...
</file-dropper>

This makes allows us to make the area that will be considered for the drag and drop really obvious, and solves any possible confusion about point number 3.
The directive HTML could be something like:

Since the directive is an element, passing options to it is way more natural, so we can pass the text and the logo very nicely:

<file-dropper logo="path/to/logo.svg" text="Lorem ipsum dolor sit amet">
   ...
</file-dropper>

This solves 1 and 2.

We could accept an argument saying in what "state" (state as defined by https://github.com/angular-ui/ui-router) the file dropper should be shown. We could do something like ngIf. For example:

<file-dropper show-overlay-if="pageIsMainOrSomethingLikeThat">
   ...
</file-dropper>

Now point 5 is the trickiest one. Since the file dropper is defined in an area that is outside the real of the page itself (where the controller is defined), we could accept a "callback" that will be called with the file path when a file is dropped, like we already do with the dropzone directive, for example:

<file-dropper on-drop="controller.myHandler($file)">
   ...
</file-dropper>

But now we need a controller to be accessible at this point, which we can accomplish with a globally accessible controller like HeaderController from lib/gui/app.js. So we can declare on the same file a controller called ApplicationController or something like that and make use of it with ng-controller before using <file-dropper>. In that controller we can define a function like .selectImage() that takes a path and selects the image.
Now, there is a lot of logic on ImageSelectionController.selectImage() that we would have to copy paste, as you noticed yourself.
There are basically two things going on there:

Logic to detect if the image is a valid image. This logic was mainly introduced because of the drag and drop functionality on the first step. Using the "Select Image" button there is no way you can select an invalid one since we set filters there which already protect us from that scenario, so I think we can just move that chunk to ApplicationController.selectImage().

Analytics to say the user selected an image, including logic to convert some properties that are known to have long lengths (like the SVG icon source, or the BMAP file source) into booleans. Let's get rid of this by making this functionality generic: we can extend AnalyticsService.logEvent() to check if the data passed is an object, and if so, iterate through every property (even recursively), and check if the value is a string, and if the string is longer than X characters (we can come up with a sane default). If it is longer, we transform it into a boolean.

What do you think?

Other smaller comments:

We should internally make use of dropzone (in the file dropper directive template) instead of duplicating the drag and drop code.

Ah, BTW, the cursor was still not the right one (its more obvious in OS X). I'll search which is the one that works for me here.

Analytics to say the user selected an image, including logic to convert some properties that are known to have long lengths (like the SVG icon source, or the BMAP file source) into booleans. Let's get rid of this by making this functionality generic: we can extend AnalyticsService.logEvent() to check if the data passed is an object, and if so, iterate through every property (even recursively), and check if the value is a string, and if the string is longer than X characters (we can come up with a sane default). If it is longer, we transform it into a boolean.

I forgot to mention: this allows us to remove the boolean casting lines, but the AnalyticsService.logEvent('Select image', image); will remain there, which is fine. We can duplicate this line on ApplicationController.selectImage(), passing a new message, for example:
AnalyticsService.logEvent('Drop image', image);

1 and 2 should be done here. I think 3 is done, if I understood you correctly the change is just to make it clearer that the directive has an underlying structure? 4 is done too, though I have a feeling it can probably be polished. 5 is done, with the use of $state.includes.
Regarding removing the controller option, I don't think I can do that without losing the shared state?
Moving the .selectImage() logic to ApplicationController means that function can't be used in ImageSelectionController, but we can instead just directly use SelectionStateModel.setImage and an analytics line subsequently. That doesn't check if the image is valid, but as you said, the dialog should take care of that, right?
The analytics code changes are in here to be seen too, but should probably be given its own issue I imagine.
As for the cursor CSS, I'm guessing it's grab and grabbing?

@jviotti more changes here.
So I've followed all the feedback above, but in doing so I've had to add more to the ApplicationController to bring things into scope. Not sure if that's acceptable or not, design-wise. Otherwise, I hope all looks good.

@jviotti here we go.

@jviotti done

Damn, getting the drag and drop hover functionality correct seems to be quite tricky.
I got a bit further by changing the pointer-events: none rule to:
[os-dropzone].dropzone-hover > * {
  pointer-events: none;
}

And by removing the domElement.ondragover = addClass; line so we don't keep adding the same class over and over again when we're on top of the element.
I then noticed something peculiar: the dropzone area is not really occupying the whole screen. Check this:

Also check the area <file-dropper> occupies:

This is the reason why putting the file in the area that is not highlighted doesn't show the overlay (it puzzled me for a bit).
The issue with <file-dropper> is that is not a valid HTML tag (obviously), but we can force it to react correctly by setting display: block;.
We can also force the height of the [os-dropzone] element to occupy the whole screen by setting height: 100%; (which should obviously be set on the file-dropper style, which has the concept of an overlay).
These rules did the trick for me:
file-dropper {
  display: block;
  height: 100%;
}

file-dropper > [os-dropzone] {
  height: 100%;
}

After these changes, the overlay seems work fine, however its not shown for some reason if I drag the image from the sides of the window (only from the top/bottom), which is super weird. Can you investigate that for me? Let me know how it goes, maybe we can do some pair debugging on it if it gets trickier :)

Here's a GIF because I can't believe this is actually happening :P

This will change a lot on v2, so let's close this for now. We can recover the changes in the future if needed.

Seems like a shame to "throw away" all the work done on this? Was there a reason why it never got merged BTW; or did other priorities mean that it never got finished properly?

Yeah, this widget involves a LOT of work, and it will change a bit on the v2 re-design. In any case, the work is not throws away, given the changes we made can be recovered at any time, but I think we should prioritize other things, and get back to this later, maybe even in React instead.

I wonder if it'd be more appropriate to add a "dormant" tag (or similar) onto a PR, rather than closing it outright and risk forgetting about it (or losing it amongst all the other hundreds of closed PRs) ?

I think the idea is to reduce the list of open pull requests, so we only see what's worth seeing. Whenever we need this again, we will definitely remember there was a PR about that, and we can recover anyway independently on the PR state (open, close)


ci: small docker changes for local dev

Pass --rm to run_envoy_docker() to clean up container when finished.
Allow NUM_CPUS to be overridden.

@htuch


Input validation error: inputs tokens + max_new_tokens must be <= 2048 on MPT-30b supporting 8k context

System Info
ValidationError: Input validation error: inputs tokens + max_new_tokens must be <= 2048. Given: 1244 inputs tokens and 1000 max_new_tokens
I am using huggingface text generation inference hosting on a runpod GPU. The model is intended to support 8192 context length but Huggingface text generation inference is imposing a hard limit on the input, how can I resolve this problem?
llm = HuggingFaceTextGenInference(
    inference_server_url=inference_server_url,
    max_new_tokens=1000,
    top_k=10,
    top_p=0.95,
    typical_p=0.95,
    temperature=0.1,
    repetition_penalty=1.03,
    stream=True
)

and I am using a langchain qa_chain to load up to 5 candidate documents to be inserted in my prompt. Theoretically with MPT this should work, but Huggingface is outputting error, how can I override this limit, otherwise, supporting long context model would become meaningless.
qa_chain = RetrievalQA.from_chain_type(
        llm=llm, # answer bot is the highest quality
        chain_type="stuff",
        retriever=vector_db.as_retriever(search_type = 'mmr', search_kwargs = {
            'k': top_k,
            'filter': {'category': category}# this filters on metadata.
        }),  # pull over 5 most (credible) relatable document from the indexed db to answer the user's question, the data are cited from the metadata indices
        chain_type_kwargs=chain_type_kwargs,
        return_source_documents=True
    )

Information

[X] Docker
[ ] The CLI directly

Tasks

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

Reproduction
!pip install langchain===0.0.230 openai chromadb==0.3.26 pydantic==1.10.8 GitPython ipython tiktoken runpod text-generation transformers runpod python-dotenv

from dotenv import load_dotenv, find_dotenv
_ = load_dotenv(find_dotenv())
import runpod
import os

os.environ['RUNPOD_API_KEY'] = aaa
runpod.api_key = os.getenv("RUNPOD_API_KEY", "your_runpod_api_key")

gpu_count = 1

pod = runpod.create_pod(
    name="MPT-30B-Instruct",
    image_name="ghcr.io/huggingface/text-generation-inference:0.9.1",
    gpu_type_id="NVIDIA A100 80GB PCIe",
    cloud_type="SECURE",
    docker_args=f"--model-id mosaicml/mpt-30b-instruct --num-shard {gpu_count} --trust-remote-code",
    gpu_count=gpu_count,
    volume_in_gb=225,
    container_disk_in_gb=75,
    ports="80/http",
    volume_mount_path="/data",
)

from langchain.llms import HuggingFaceTextGenInference

inference_server_url = f'https://{pod["id"]}-80.proxy.runpod.net'
llm = HuggingFaceTextGenInference(
    inference_server_url=inference_server_url,
    max_new_tokens=1000,
    top_k=10,
    top_p=0.95,
    typical_p=0.95,
    temperature=0.1,
    repetition_penalty=1.03,
    stream=True
)

final_answer_prompt_template = """
# INSTRUCTIONS
- instructions to execute the prompt

# CONTEXT
{context}

# QUERY
{question}
"""
FINAL_ANSWER_PROMPT = PromptTemplate(
    template=final_answer_prompt_template, input_variables=["context", "question"]
)

def ask(query, category, top_k=5, show_sources=False):
    display(Markdown(f"### Query\n{query}"))

    chain_type_kwargs = {"prompt": FINAL_ANSWER_PROMPT}
    qa_chain = RetrievalQA.from_chain_type(
        llm=llm, # answer bot is the highest quality
        chain_type="stuff",
        retriever=vector_db.as_retriever(search_type = 'mmr', search_kwargs = {
            'k': top_k,
            'filter': {'category': category}# this filters on metadata.
        }),  # pull over 5 most (credible) relatable document from the indexed db to answer the user's question, the data are cited from the metadata indices
        chain_type_kwargs=chain_type_kwargs,
        return_source_documents=True
    )

    answer = qa_chain({'query': query})

Expected behavior
The model output natural language response as if it is being called on a short prompt.

You need to update the MAX_INPUT_LENGTH and MAX_TOTAL_TOKENS env vars to values that suit your usecase.

See https://github.com/huggingface/text-generation-inference/blob/44acf72a736346f4b8e969c9453027ca32786d72/launcher/src/main.rs#L133-L149


Project Status

Hello team,
Great job for putting such a lib. I would like to know whether the project is still ongoing or it is abandoned. I do know how difficult it is to combine day-job and open source project.

This is definitely a great project, which can be a Kafka killer considering golang resource usage is very less than Java.
But sad, this project is not active since last 2 years. Developers of this project can plan github sponsor for this project.
I am also searching for status of this project after looking one new project fluvio in RUST, which also seems promising Kafka replacement.

Writing books about unfinished project(s) is more important than actually finishing the project(s). lol. If that's not a perfect example of silicon valley mentality, i don't know what is.

Interesting. Thanks for letting me know. If that the case just make the repo readonly. That is the basic thing to do to let users know that is the project is no longer maintained or update the readme to state that it needs support.


new package: evilwm-1.1.1

Motivation for this change
To have a nice window manager in the packages.
It has no decorations, is fully keyboard-driven, and yet doesn't pretend to be tiling.
Just a simple tool to easily move windows around, without unnecessary complications.
Things done

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

[X] NixOS
[ ] OS X
[ ] Linux

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

It's pretty much isolated thing, the risk of the lack of such an extensive testing must be minimal.
Did nix-env --file nixpkgs/ --install evilwm, and the only binary is running.

LGTM. Please squash the commits when you're ready to merge.

I guess, dwm isn't building on Darwin too, and may need the similar changes?

@amiloradovsky dwm is currently failing on darwin due to problems with the build machine; it built fine a few days ago.

@joachifm Initially I took dwm's default.nix as the prototype: so if e.g. sed -i caused this package to not build on MacOS/Darwin, then the same should be for dwm itself. I could touch it too by the way, should I?

@amiloradovsky see https://hydra.nixos.org/build/37761283; dwm built fine on darwin just a few days ago and the current failure is caused by a problem with the build machine itself, I don't think changing the sed call will fix that.

That said, sed -i -e command is potentially safer than plain sed -i command, as -i takes an optional argument specifying a backup extension ...

Probably the best choice here is Nix's own substituteInPlace (where possible, since it appears to be limited).

That's not a nix builtin, it's a fairly complicated wrapper around bash ...

I suspect, it was introduced as a portability layer.
For example, if I remember correctly, the initial (sed -i) version passed on only one "linux" platform, while substituteInPlace-version -- on both of them. Don't know why.

It looks like the darwin stdenv includes gnused, that probably happend with the changes to the stdenv a while back. However I still prefer substituteInPlace since it's a little more generic and easier to use with special characters.
@joachifm using sed -i -e 's/foo/bar' file.txt would create a file.txt-e backup file.

@joachifm using sed -i -e 's/foo/bar' file.txt would create a file.txt-e backup file.

Not on my system.

@LnL7 The disadvantage of substituteInPlace though is that it apparently cannot handle regular expressions.

Since this pull request happened to be very flawed, especially pushing the changes into master was a mistake. So maybe it's better idea to close this and resubmit (#17178).

@amiloradovsky Not sure why you closed this, the changes are fine.

@LnL7 I needed to revert master to it's previous state, to make another (parallel) pull request(s).
I couldn't find how to change the branch here, and didn't want to use some master-original for that.
(Possibly it wasn't necessary, I'm not sure.)
It's basically naming and compliance, not something particular about the content.


Develop rollout plan for Pilot Ministries

Title of ticket:
Description
Training approach will be scenario-based - Participants will be given scenarios and asked to complete them in the Test environment with the FOI team present as supports. Potential structure: Full group overview, scenarios in breakout rooms, full group debrief/summary/Q&A
Dependencies
Are there any dependencies?
DOD

[ ] Set up of Test enviro for ministries
[ ] Internal team support
[ ] Training sessions for ministry staff and IAO
[ ] Determine who from IAO will want a test account
[ ] Create tasks/flow to guide training sessions with ministries

cc: @lmullane @m-prodan @arielleandrews

I have updated this issue with some of the things discussed with EDUC today, including adding development of workflows for training under the DOD. cc: @lmullane @arielleandrews  @m-prodan

subtasks have been captured elsewhere - closing (duplicate)


In process propagation

Instrumentation currently does not use in-process propagation for an active span.
Some references:
https://github.com/opentracing/opentracing-java/pull/115, https://github.com/opentracing-contrib/java-spanmanager

+1

On client side:
I assume that span should be activated in ClientRequestFilter
and deactivated in ClientResponseFilter.
But will this two filters always run in the same thread?

hm.. I thought this is done. I will look at it today.

Fixed in #36


Wasted renders when using the  component?

Hello,
I have a simple functional component where I'm using the <Observer/> component:
import React from "react";
import { Observer } from "mobx-react";

const MobxFunctionalComponentObserver = props => (
  <Observer
    inject={stores => ({ title: props.title, color: stores.store.color })}
    render={props => (

        {props.title} {Date.now()}
         {props.color}

    )}
  />
);

MobxFunctionalComponentObserver.displayName = "MobxFunctionalComponentObserver";
export default MobxFunctionalComponentObserver;

This component is always being re-rendered.
Here is a working example on codesandbox
In the same link I've created a similar stateless component that doesn't get re-rendered every single time:
import React from "react";
import { trace } from "mobx";
import { observer, inject } from "mobx-react";

const MobxFunctionalComponent = inject("store")(
  observer(({ store, title }) => (

      {title} {Date.now()}
       {store.color}

  ))
);

MobxFunctionalComponent.displayName = "MobxFunctionalComponent";
export default MobxFunctionalComponent;

Is this a bug or I'm I missing something?
Thanks in advance

I think it's a bug, because implementation seems completely wrong to me.
First of all it creates new InjectComponent during every render, which breaks react's reconciliation.
Secondly it applies inject prior to observer therefore the passed render actually doesn't run as part of observer (therefore isn't reactive etc).

Thank you @urugator

Well, I think that a whole thing should be refactored into Observer being regular component (with render/children prop) in a first place and then turning it into HoC. It's kinda messy now and hard to track an actual issue. I might be able get to it, but no promises. If someone has more free time, feel free to do it.
/cc @Sunshine168

Correct @urugator , this is indeed a bug, sorry for not noticing it earlier :) PR's are welcome (including tests). Otherwise I'll take a stab at it next week or so

@FredyC  eemm,  it is regretful for marking this future incorrectly.
Refactoring it to HOC look like this?
const comp = Observer(inject)(component)

i so busy during this period （orz） ,but if  i can do this correctly , i will try

This has me dead in the water with my React Native project, since use of FlatList requires the use of the Observer component.

@duro Observer is fine, just don't use the inject attribute

inject on Observer will be removed in the next version, it is just to fundamentally broken right now. Inject the hosting component instead, and access the injected stuff from the closure.

Sorry for the confusion, Observer will stay, it just won't support that
inject property anymore in the future
Op di 24 apr. 2018 om 09:26 schreef Daniel K<EMAIL_ADDRESS>
@mweststrate https://github.com/mweststrate Well, I would love to see
some example what do you mean. Sure, the inject is redundant and can be
easily replaced with React Context API
https://github.com/mobxjs/mobx-react/issues/448, but I cannot imagine
what is the alternative to Observer?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/423#issuecomment-383830954,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhPJmVgY2khazyTPLL6wYtcIToX0Vks5trtOKgaJpZM4SGHuE
.


Chopped Logs don't render smaller

Minecraft v1.20.1
TreeChop v0.18.7b
After updating, the chopped logs don't render smaller past the first chop and still renders the overlay on chopped logs. The hitbox still becomes smaller, but visually appears the same.

I have the same issue

This issue seems to be present when Indium is installed with the latest update of TreeChop. Removing indium fixes this.

Nope, I have tested this with just TreeChop installed and I still have the issue.

You're right. I noticed that installing Sodium makes logs update properly. But after installing Indium with Sodium, it doesn't work again.

Ahh shucks. Thanks for the info everyone. The way Forge, Sodium, and Indium (etc.) all mess with Minecraft's rendering process is not fun. I'll get it fixed

I am also using Embeddium on forge with my pack

0.18.8 is posted. Things should work better now for Forge/Embeddium and Fabric/Sodium/Indium

I'll close this since this is fixed for me

Wonderful :)


Now works on linux properly

Reg #2
Added an run.sh file
the header file now has all the headers (and os check logic for headers)
in cmd_functions.c added os check logic for executing the commands

@chethanyadav456 have done this so pls review, (#3 pr)

Thanks for your contribution!! ❤


update app repository command

For deployment CI to update repo tags from travis.

Should resolve https://github.com/keboola/developer-portal-cli-v2/issues/3

thanks, it look ok! I am going to merge it and test it on real app


Slide attachment from Outlook to File does not work

Description
Can't slide an attachment from outlook to file.  It does not work.  I have to select and hit copy.  then paste into file.  then hit F5 to refresh.  if no.  can't see it.
The status centre say that it has been copied after sliding.  but it is not
Steps To Reproduce

select attachment from outlook and hold left mouse button.
slide to the file app and release the left mouse button
status centre says it has been completed
hit F5 and no files have been copied.

Requirements
Can't slide an attachment from outlook to file.  It does not work.  I have to select and hit copy.  then paste into file.  then hit F5 to refresh.  if no.  can't see it.
The status centre say that it has been copied after sliding.  but it is not
Files Version
<IP_ADDRESS>
Windows Version
10.0.19045.3693
Log File
debug.log

Thanks for the feedback, Merging with #13646. You can check there or click subscribe in the right sidebar for updates.


The Output View should have a separate zoom level than code editor

In VSCode one can change to zoom level of the code editor, specifically with a mouse wheel.
Unfortunately this action also changes the zoom level of the Output panel. It would be great if the Output panel gets a separate zoom level setting, so it doesn't get changed when the code editor zoom level changes.

@alexdima  I am not sure if you are owning the Output View but the request seems to have an output view specific font size

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This is not a feature request but a bug clearly.

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


修改POM（修改spring-mvc依赖为必须依赖）

修改POM（修改spring-mvc依赖为必须依赖）

Pull Request Test Coverage Report for Build 124

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

Totals

Change from base Build 123:
0.0%

Covered Lines:
584

Relevant Lines:
644

💛  - Coveralls


chore: rename StartReturningModel to Run

Added aliases for the old functions, but deprecated them.

I'll merge #479 before this one and update the docs once more.


Add ansible_test_platform option to tox.ini

Fixes https://github.com/tox-dev/tox-ansible/issues/130
This patch adds a new parameter to the tox.ini configuration that allows users to choose whether tox-ansible should try to automatically detect an appropiate platform for ansible-test.
As suggested, I also added some assertion logic to ensure that the platform is always a valid choice.
I assume that you want this assertion in options.py, so that any mistakes can be caught quickly upon reading the config file.
Please let me know If that is not the case!

This is great. Thanks!

I don't think CI will finish before I have time to merge it this afternoon, but I will get it merged and released over the weekend.


[MRG] add support for duplicate column names

Fixes #4079

modifies _repr_data to use a list of series instead of a dict

adds a test that tries to print with duplicate column names

[X] Tests added / passed

[X] Passes flake8 dask

This looks good to me.  Thanks for raising and fixing this @datajanko .  Merging.
Also, I notice that this is your first code contribution to this repository.  Welcome!

Thanks for the warm welcome.


Scraping Followers

@rarcega  is it possible to scrape Followers username of a instagram account? if so please share me the script.
Thanks

I'm also interested in this feature

Any update on this one?


Alchemical Error

I noticed recently that, at least at level four, Alchemist gets access to the level five ability "Field Discovery" wherein you can produce three elixirs that have a corresponding trait to your field. Besides Field discovery there are no other feats that increase the amount of elixirs you can produce with advanced alchemy for your daily prep

Pending remaster changes

As I am not the owner of the server I am unable to disable the mods, but here is the list.

and a copy of my character so you can try to recreate it.
https://pathbuilder2e.com/launch.html?build=820550
And the error that I was talking about;

If I understand correctly, it's supposed to craft 3 elixirs at the cost of one reagent. Is that correct?
Please note - due to the upcoming Remaster changes, this feature will function differently.

The issue is that it's crafting MORE than it should at this level. Field discovery doesn't hit until level five, so it's creating more elixirs than it should.

OK, I rebuilt your character using my instance of Foundry and can confirm that it is only creating 2 items as expected.
Either one of the modules installed in the world is messing up with your character or you might need to re-create the character from scratch (sometimes the Pathbuilder imports can get messed up).

We strongly recommend not using importers: they will produce broken characters without exception, even it is not immediately obvious.

Alright, as long as the issue isn't the pf2e vtt itself I can deal with it. I just wanted to make y'all aware.
ty for the help


Requests hang when cache set to redis

Description
When trying to make any request, the request will hang if the client's cache is set to redis.
Reproduction Steps

Set rate limit cache to use redis
Make any request to Riot's API

Expected/Actual behavior
The client to connect properly to the redis cache and execute the requests.
Additional Information
Here is the config I use:
{
  'riot-api': { key: 'XXXXXXXX' },
  'rate-limit': {
    type: 'bottleneck',
    cache: {
      type: 'redis',
      uri: 'redis://<IP_ADDRESS>:6379',
      'key-id': 'galeforce'
    },
    options: {
      intervals: [Object],
      'max-concurrent': null,
      'min-time': 0,
      'retry-count-after-429': 3
    }
  },
  debug: [ 'all' ]
}

Note that when setting the cache to internal, everything runs fine. Also, my redis server (Redis version=5.0.8, bits=64) is running and accessible through the passed URI. I've also tried setting the database in the URI as follows: redis://<IP_ADDRESS>:6379/0, but that didn't help.

Should be fixed in 0.6.0.


Remove logging count for this release.

It will be added back in at a later date.

Restarting the build, was an Auth flake I believe.


Review current CI flakiness

[x] Make smoke tests reliable
[ ] Unflake the tests which are maked 'allowed to fail'

Done: SPecific tasks were created for the flakes.


Add support for jsonschema export similar to openapi-v3

Describe the problem/challenge you have
Most text editors use json schema, to autocomplete and validate YAML files. To make it easier to the users, to write YAML files based on a YTT schema, it would be great to have a way to create the json schema from a YTT file, similar to the openapi schema that is currently implemented.
Describe the solution you'd like
Support for json schema as a output option for  ytt -f schema.yaml --data-values-schema-inspect --output
Anything else you would like to add:
During kubecon 2024, the talk Developers demand UX for K8s highlighted that YAML validation & auto-completion seems to be one of the main struggles developers face when working with Kubernetes.
Using YTT and it's schemas, could help alleviate that issue.

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

After diggin in to the openapi schema generation for #897 I think this shouldn't be too hard to implement, as the property attributes are mostly identical, and a lot of the code could be reused.
From a quick glance at https://swagger.io/docs/specification/data-models/keywords/ the biggest difference is that type is a list, and nullable isn't an attribute, but needs to be added to the type list.
@prembhaskal Let me know if I should draft a pull request for this.

I found some time today, and threw a quick and dirty approach together in a feature branch: https://github.com/peschmae/ytt/tree/feat/jsonschema
It would still need a refactoring, to reduce duplicate code shared with the openschema export, but it already converts into a valid schema that is properly interpreted by vscode.

@peschmae I am bit stuck with other stuff, but I can review it over weekend. but you can go ahead and put up a PR if you want some early eyes on it.
@100mik @sethiyash PTAL when you get time.

I am unable to check it still, so removed assignee.

@cppforlife @100mik Please take a look at this


Save etcd http client separately per cluster

Each etcdadmCluster object needs to use a separate http client, since
this client contains the etcd cluster's ca cert and client cert-key pair.
This commit replaces the field containing http client with a sync.Map
where key is the cluster UID and value is the etcd http client.
It uses golang's sync Map because as per docs:

The sync.Map type is optimized for when the entry for a given key is only ever
written once but read many times
Use of a sync.Map may significantly reduce lock contention compared to a
Go map paired with a separate Mutex or RWMutex.
https://pkg.go.dev/sync#Map

https://github.com/aws/eks-anywhere/issues/389


Fix validation of "comment_id" parameter in http_delete_comment

Another one :smile:

Apparently I didn't review the original pull requests very well, heh. Thank you!


Edits to most recent vignette

This provides a menubar link and stylistic edits to the LMM-Genetic-Diversity vignette

I'm going to merge this since it's basic edits


Race condition in HTTP-01 challenge?

I am using lego as a library to issue SSL certificate on DigitalOcean droplets. I keep getting acme: Error 400 - urn:acme:error:connection - Could not connect to mydomain.tld\nError Detail:\n\tValidation for mydomain.tld:80\n\tResolved to:\n\t\t<ip>\n\tUsed: <ip>\n\n  error. I am confused how is this happening. One possibility seems to be that validate is getting called before http server starts listening on port 80 via go routine. . This seems to be happening 50% of the time. I am using GO 1.8.3 .
FYI, the DNS address has been stable for than 24 hours. So, I doubt this is a DNS resolution issue.

I found a similar error message here: https://community.letsencrypt.org/t/certbot-auto-urn-acme-error-connection-400/31307
There the issue also seems to be that the nginx server was stopped before starting HTTP-01 challenge.

There could be a number of other factors that cause connectivity issues; for example, if your droplet is behind a load balancer or proxy of any type, that would cause the problems you're describing.

My droplet is not behind any load balancer or proxy from my end. I was able to SSH into the droplet.

One possibility seems to be that validate is getting called before http server starts listening on port 80 via go routine. . This seems to be happening 50% of the time. I am using GO 1.8.3 .

That particular go routine would have to be abysmally slow though. Needing more time to start the server than a full HTTP request + processing time on the  boulder side seems pretty unlikely to me.

I'm in the case @mholt describes. Being behind a load-balancer causes certificate renewal to be unreliable: a couple of seconds are needed for the load-balancer to take note the port 80 is open and considers the backend as healthy. But in the meantime letsencrypt servers are already knocking on the door, and getting "503 proxy error" served back.
It would be great if lego had a (configurable) grace time between binding port 80 and starting to send queries to the letsencrypt API !


Points to tube does not have apply attribute

Describe the bug
Hi Bane, we are just try to plot some wells in vtk and I was looking to replicate your points to tube example but I think something is not update or it is the vtk version maybe?
Behavior
AttributeError                            Traceback (most recent call last)
 in
1 # Use the filter: here is vtkPolyData containing the tube
----> 2 tube = PointsToTube(nearestNbr=True).apply(points)
3 tube.plot(color=True)
AttributeError: 'PointsToTube' object has no attribute 'apply'
Any fast idea?

Ahhh yes, this is because I'm frantically trying to release version 2.0.0 which updates PVGeo for use with vista and changes everything to pep8.
I'm very sorry for the delay on this release. I'm going to push a new pre-release to PyPI so could you please pip install that when it lands or install from the master branch?:
python -m pip install git+git://github.com/OpenGeoVis/PVGeo@master

v2.0.0b should be on PyPI soon - PVGeo still has some quirks that need to be fixed before I can push out a full v2.0.0 release but it shouldn't take too long!

Yes, I imagined so. No worries, it was just in case you were not aware of this specific case. We are trying to couple welly and gempy and I was just playing with the feature. No rush!

v2.0.0 is live on PyPI now! Sorry for the delay
Also, do let me know if you have any questions about how PVGeo might fit into welly and GemPy


[sidecar-ffi] Fix undefined behaviors caused by unsafe null slice cast in the sidecar-ffi

What does this PR do?

Fix null pointer used in slice::from_raw_parts, this is safety conditions as the pointer for slices should not be null
Fix flakyness issue of the test exhibiting the issue and unskip it in CI

Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.

Codecov Report
Attention: Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.

Project coverage is 70.54%. Comparing base (9d82f91) to head (e585de7).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #792      +/-   ##
==========================================
+ Coverage   70.19%   70.54%   +0.35%
==========================================
  Files         313      313
  Lines       45352    45358       +6
==========================================
+ Hits        31835    31999     +164
+ Misses      13517    13359     -158

Components
Coverage Δ

crashtracker
38.06% <ø> (ø)

crashtracker-ffi
5.65% <ø> (ø)

datadog-alloc
98.73% <ø> (ø)

data-pipeline
91.38% <ø> (ø)

data-pipeline-ffi
86.29% <ø> (ø)

ddcommon
81.60% <ø> (ø)

ddcommon-ffi
64.83% <ø> (ø)

ddtelemetry
59.45% <ø> (ø)

ddtelemetry-ffi
22.13% <ø> (ø)

dogstatsd
89.45% <ø> (ø)

dogstatsd-client
79.77% <ø> (ø)

ipc
82.86% <ø> (+0.10%)
:arrow_up:

profiling
84.30% <ø> (ø)

profiling-ffi
77.46% <ø> (ø)

serverless
0.00% <ø> (ø)

sidecar
40.54% <0.00%> (+2.57%)
:arrow_up:

sidecar-ffi
1.18% <0.00%> (+1.18%)
:arrow_up:

spawn-worker
54.37% <ø> (+4.01%)
:arrow_up:

tinybytes
93.60% <ø> (ø)

trace-mini-agent
72.38% <ø> (ø)

trace-normalization
98.23% <ø> (ø)

trace-obfuscation
95.77% <ø> (ø)

trace-protobuf
77.67% <ø> (ø)

trace-utils
93.05% <ø> (ø)


Support YDB as destination in vector

See other https://github.com/vectordotdev/vector/tree/master/src/sinks

And don't forget about otel-collector


automatically install all packages via script

@gedankenstuecke: kannst du mal das neu Installer Skript mal checken, ob es bei dir funktioniert :) Danke dir ^_^

@trvinh do i need to uninstall some packages first for a proper test? :)

(btw. Travis CI supports R, so you could even do automatic checks on whether everything still works with some tests: https://docs.travis-ci.com/user/languages/r/ ) 😉

Okay, so I just gave it a try on my workstation where a lot of the packages weren't installed so far. The installPackages.R script ran without any trouble and could install the missing packages :tada:.
But as far as I could see it doesn't check for the version of the installed packages, which might cause troubles, as with the old ggplot2 version. So that might be worth implementing as well?
Furthermore, is there a reason why the install-script should remain independent of the main server.R script? If there isn't one: It might be nice to implement it in the main script right away, so that people can just run PhyloProfile without any further thought and it will fetch all necessary scripts right away?

Furthermore, is there a reason why the install-script should remain independent of the main server.R script? If there isn't one: It might be nice to implement it in the main script right away, so that people can just run PhyloProfile without any further thought and it will fetch all necessary scripts right away?

Uhm, but I don't know how can we run a shiny app (R -e 'shiny::runApp(,launch.browser=TRUE)') when shiny hasn't been installed :-P

I think you can write a wrapper phyloprofile.R (or similar) that first runs all the logic to acquire the packages and as a last step runs
library(shiny)
runApp()

Then you should be able to run Rscript phyloprofile.R from your CLI and it should do all of it?


Memory leak on linux

Memory leak on docker on linux
I have a asp.net core web project which used opencvsharp.It runs on docker on linux.The server's memory is 1G,I fond that every about 0.5 hour the memory will be reached 100%,and the program hang up.
Environment

server is 1 core cpu and 1G memory
.net core 3.0.0-preview7-27912-14
opencvsharp 4.1.0
docker 18.03.0-ce
debian 9

Example code:
On windows,I use vistual studio, the diagnostics tools shows 29.3MB memory used.
But on docker on Linux,The following code caused memory leak. After it run finished,the memory use reached 306MB(use command : docker stats -a)
The following code is not all same as my asp.net web project,but I think the reason of memory leak is same.
using OpenCvSharp;
using System;
using System.Threading.Tasks;

namespace MemoryTestConsole
{
    class Program
    {
        static void Main(string[] args)
        {
            Parallel.For(1, 200, (i) =>
            {
                using (var m = new Mat("Images/1.png"))
                {
                    System.Threading.Thread.Sleep(2000);
                    Console.WriteLine(m.GetHashCode());
                }
            });
            GC.Collect();
            Console.ReadLine();
        }
    }
}

Output:
windows:
29MB

linux:
306MB

hi,i'm  use .net core same to u ,but when run the app on linux,still show error,like this:"DllNotFoundException: Unable to load shared library 'OpenCvSharpExtern' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: libOpenCvSharpExtern: cannot open shared object file: No such file or directory",can u tell me how to fix this problem?

.net core 2.0
opencvsharp 4.1.0
docker 18.03.0-ce
debian 9

Looks like the same problem as described here: https://github.com/dotnet/runtime/issues/13702


Missing important option in settings that used to exist in earlier versions

OS/Platform
Windows
Installed
https://ungoogled-software.github.io/ungoogled-chromium-binaries/
Version
124.0.6367.60-1
Have you tested that this is not an upstream issue or an issue with your configuration?

[ ] I have tried reproducing this issue in Chrome and it could not be reproduced there
[ ] I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there
[X] I have tried reproducing this issue in ungoogled-chromium with a new and empty profile using --user-data-dir command line argument and it could not be reproduced there

Description
Missing "Clear cookies and site data when you close all windows When on, you'll also be signed out of Chromium"
How to Reproduce?

Go to Settings, Privacy and Security, Third Party Cookies
There is no option "Clear cookies and site data when you close all windows
When on, you'll also be signed out of Chromium" that used to be.

This option is missing from 120.x.x.x versions.
Actual behaviour
The missing option is quite important, since it clears all the cookies we don't want to have.
Expected behaviour
The "Clear cookies and site data when you close all windows When on, you'll also be signed out of Chromium" option should be included again.
Relevant log output
No response
Additional context
No response

You can access it here
AFAIK there is no way to access this from the menus.

Interesting, I didn't notice that.
You need to expand the menu to find site data.

Thanks to all, closing the issue.


Error running reset script on laravel project

Describe the bug
Docker-compose fails when running multiple commands scripts
Kool version and environment
1.9.0 on MacOS
To Reproduce
Run kool run reset

Expected behavior
The script must be executed, but it exits with an error on the second command of the script:

If I run each command of the script separately, all of them work fine.

If you are facing this issue, you can downgrade kool to version 1.8.0. We will release a fix for this ASAP.
curl -fsSL https://kool.dev/install | DOWNLOAD_URL=https://github.com/kool-dev/kool/releases/download/1.8.0 bash

Fixed will ship into the next patch release today.


Feature/mx 1510 implement backend id provider in extractors

return deterministic static stableTargetId and identifier if queries for "mex" primary source.
return a nicer error for wrong query parameters for the fetch endpoint
update tests

Looks good to me, I just have a few general questions rather than requests for changes. But main has to be merged.


server has no session id on API

A quick swap of device can leave the UI unaware of the change.
Add a session ID on API connection.

fixed in #70


update forger block gas limit logic

Include new transactions as long as there is at least 21.000 gas available in the block.

@JcBernack please resolve the conflict with dev_evm. Then I will merge the PR


Deploy OmniLayer

How to deploy OmniLayer? I want to provide services to thousands of users.

Hi there, can you please be more specific? There are build instructions here:
https://github.com/OmniLayer/omnicore/blob/master/doc/build-unix.md

Hi. I do not know, OmniLayer's RPC runs single-threaded or multi-threaded. I see the speed of the API is quite slow, so what should I do to make it faster. I do not understand much about OmniLayer's architecture, can you provide me with documentation?

Omni Core is essentially Bitcoin Core with extended feature set.
Is your client fully synchronized?

I want to provide services to my clients. And they do not need to save any data.

If I only run omnicore client is enough or not. Please give me some advice.

Running Omni Core is enough from the Omni Layer and Bitcoin side, but you'd probably still need a system to handle user accounts, the front end, etc.

thanks you ....


Add a link to the main Graph Explorer Full experience

Is your feature request related to a problem? Please describe.
Try it login should act as a teaser and has limited features and there's need to add a link to main Graph Explorer
Describe the solution you'd like
Add a link to the main GE with context of the link that carries the url that the user was looking at in try it and request headers as well if they were using non-GET request option.

@bettirosengugi how related is this task to this one? #356


Milestone/v1.0.0

Milestone v1.0.0

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Query and Disk Usage Telemetry Devices

The field usage API will provide us a breakdown on the usage of certain fields by queries. We should evaluate whether collecting these statistics using a telemetry device is valuable.
Additionally, the disk usage index API should be valuable for tracking disk usage by field. It is unlikely we will want this throughout the benchmark, so an operation might be more appropriate.

I might have a look at this one. I really would like disk usage telemetry for something I'm working on right now.

OK! So. I've put together a prototype for field disk usage over in #1428. We were having a discussion about the output format in another channel. Rally usually wants to summarize outputs but my prototype reports the disk usage for each structure that each field in each index. To the screen. It's huge. But it's totally the kind of thing I need it to do when I'm working on field usage. I think it does the important thing for graphing already - it dumps the usage statistics to the reporting repository. We build the graphs based on that.
But the questions is - what should we really do here? When I'm really doing field usage stuff, I want it broken out per field. Not some summary. I'm not 100% sure how I'd summarize anyway. and I want to be able to do the esrally compare thing and see the change in field usage between tests. Per field. That'd be super useful.

Maybe we can close this issue, I'm unsure how useful field usage would be to Rally since it controls the queries that run.

Maybe we can close this issue, I'm unsure how useful field usage would be to Rally since it controls the queries that run.

I also agree


Migrate to the non-deprecated version of ActionBarDrawerToggle

Fixes #348

The home icon is always an arrow now. Can we have a Hamburger menu?

It's strange. I've tested it in two devices (Android 2.3.6 and 4.4.2) and in an emulator (5.0.1). In the three cases a hamburger menu is show that becomes an arrow when the drawer is open. The only exception is the settings activity, which doesn't have a drawer toggle.
What devices have you used for testing? Does it happen in all activities with a drawer toggle?

Strange, now I tested it again (Android 5.1, Nexus 5)  and it works.
Maybe I did something wrong before.
Thanks!

Good :) Not sure if it's what has happened, but sometimes, rebuilding the applications fixes these strange problems.


Make WMI4Java throw exceptions instead of catching and suppressing them

Solves #8, passes JUnit tests successfully.

Don't be scared of that massive deletion on WMI4Java.java, Netbeans messed up and deleted my whole file. I had to restore a copy of it and continue work. The only things changed are the exception catching (which has been removed) and now methods throw exceptions.


Revisit notebook cell uris

Today, notebook cell have the following URI format
vscode-notebook-cell://<notebook-authority>/<notebook-path>, cell <handle>?<payload>
This makes then

render in the UI like some/path/someNotebook.foo, cell 12
makes a cell a different resource - by the rules of URI

The downsides are

The cell handle appears in the UI and that's something users can not really relate to
A cell is inside the notebook, not a different resource, using fragments is the better fit
Grouping is often (but not always) bogous (see screen shot below)

The plan is to use a URI format for notebook cells that do not make them appear as separate things but make them appears as the notebook itself

🧑🏭 work in progress 🧑🏭
I have changed the peek model and the references viewlet to group resource independent of their uri-fragment, e.g foo:///bar#a and foo:///bar#b are different uris pointing to the same resource. It actually works nice for those two, but I am unsure about diagnostics (not yet adopted) because there we don't show a preview but line/character, e.g. [2, 1], which only make sense in the scope of a resource.

For instance, this might be confusing. You have cells, that both have the same error on line one (is that likely?). It will show like this
fooBook.nb
  error foo [1, 1]
  error foo [1, 1]

This did happen 👯


Screen record live

Make like .join but with the bot stream the screen in live please

i like this idea

this has already been requested. Bots can't stream, users can tho.

Hey, just use the ModdedPysilon version, it covers every bug and it make sure your discord account never get banned.
Leave me a star if you like my project ;)


Master gpu trace unified incomplete merge

Previously, event synchronization activities are not implemented.
Besides, on Power9 machines, gpu activities on the same stream may have overlapped intervals. So we adjust the start timestamp of an activity as max(last_activity_end + 1, current_activity_start).
There are other features have been implemented in master-gpu-trace branch but not merged. I will only add essential fix, such as the timestamp adjustment but not merge all of them.

thanks for adding the new event functionality


ci: remove e2e test RUN_COMPACTION

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Checklist

[ ] I have written necessary rustdoc comments.
[ ]  I have added necessary unit tests and integration tests.
[ ]  I have added test labels as necessary.
[ ]  I have added fuzzing tests or opened an issue to track them.
[ ]  My PR contains breaking changes.
[ ]  My PR changes performance-critical code, so I will run (micro) benchmarks and present the results.
[ ]  My PR contains critical fixes that are necessary to be merged into the latest release.

Documentation

[ ]  My PR needs documentation updates.

Release note

Discussed with @zwang28, the compaction test isn't very useful and we can remove it.
Here we removed it from the script, and pr.env isn't useful anymore and we can unify run-e2e-test.sh and e2e-test-serial.sh. @zwang28 will delete the code in kernel later.


There are extra characters in the html source code

Describe the bug
Right-clicking to inspect the source code reveals that there are extra characters in the indistinguishable tags

To Reproduce
Steps to reproduce the behavior:

Go to https://meta.answer.dev
Right-clicking to inspect the source code.
The URLs of og:image and twitter:image in meta have extra characters

Expected behavior
Remove extraneous erroneous characters }.
Platform

Device: [e.g. Desktop, Mobile]
OS: [e.g. macOS]
Browser and version: [e.g. Chrome, Safari]
Version: [e.g. v1.3.6]

Hi, how is the frontend of meta.answer.dev built and deployed? I tried to start using pnpm and compile static files, but the content is different.

Hi @sosyz
Answer To meet the needs of SEO, in addition to the construction method you used above, there is also a set of back-end template files. This problem is because the content in the template file is wrong, so the method you used above cannot be seen. These can be seen under the ui/template/header.html file

Hi @sosyz
Answer To meet the needs of SEO, in addition to the construction method you used above, there is also a set of back-end template files. This problem is because the content in the template file is wrong, so the method you used above cannot be seen. These can be seen under the ui/template/header.html file

Thank you for your reply. I have modified the file and will submit a pull request later.

Fixed in https://github.com/apache/incubator-answer/pull/1073


Dilated File Transfer

Specification for a new file-transfer protocol based on Dilation.

State-machine diagrams from PoC code written using Automat:

Rendered version of the diagram included in this pull-request:

@felinira thanks for the feedback. If a realtime discussion is useful, happy to meet in Jitsi or IRC or similar...(I'm at UTC -6 currently).

p.s. those pink/blue state-machine diagrams come from psuedo-code and are rendered by the Automat (https://github.com/glyph/automat) library ... so while it's certainly not working I am making some Proof-of-Concept level experiments while developing this

@psanford or @jacalz maybe of interest?

I just had a quick read through and it looks sensible to me. I'd personally be in favour of including compression (more specifically ZSTD compression) as part of the protocol at this time. However, I understand that it also might make sense to handle compression later on.

I did look at that, and agree that Zstandard sounds like a good approach.
I left it for a future enhancement (likely as a "feature") for several reasons:

this is already pretty big (as @felinira noted)
some implementations may wish to not do it (or not implement it) -- hence "feature"
there's a bunch of subtle stuff (keep the context for entire directory-offer? spec defaults params, or sender decides, or?)
could be security implications (I'm 95% sure "not" for this use-case, compress-than-encrypt has had some spectacular failures e.g. CRIME)

I would like to have compression though. If others agree that's "not too much more complexity" I could spec that out as well in this draft (as the one and only "feature" for now)...?
Vote with emojis: :turtle: - do it later, :snail: compression now

If compression is done as a "feature" it still allows implementations to "upgrade slowly".
Idea being, you can implement just "mode=send" and "mode=receive" for essentially feature-parity with classic transfer (this should mean minimal changes to UX etc). This still means multiple files, if the sender offers more than one, so it's not precisely the same...
Then, you can implement "mode=connect" (possibly with very different UX) for multiple, bi-directional transfers. Some tools may never implement this mode (e.g. wormhole receive will only ever do mode=receive).
(If we specify compression) Separate from both the above, you can implement compression if desired. It could also be "implemented" but not used in some situations (e.g. smaller computers) as it will be a "feature".
Perhaps the above should be a section in the spec?

In #1, I proposed a simple "format" option for compression. This is fairly simple, under the assumption that the compressed data can be decompressed without requiring any information about the compression settings (this is usually the case). It uses the usual feature intersection which makes it both opt-in and future proof for format additions.
The only real open question is how to deal with folders. Compressing folders as one stream is very likely to be more efficient (no new dictionary per file, deduplication across files), so we probably want that. The downside is that the individual files cannot easily be separated prior to decompressing. This is not a huge deal IMO, but it makes the "send file header then bytes" approach unfeasible.
I propose to skip the individual file headers in directory transfers and just concatenate the (compressed) bytes. The order of the file is deterministic and dictated by the send offer. The checksum message might be modified so that it is one message with a list of checksums, one for each file.

The downside is that the individual files cannot easily be separated prior to decompressing. This is not a huge deal IMO, but it makes the "send file header then bytes" approach unfeasible.

No, you can still separate the "compressed bytes for file 1" from the "compressed bytes for file 2" as zstandard allows a "flush" operation. I have tested this. Many compressions also offer their own container format (as does zstd) but you don't have to use these.

I don't think we should include more than one offer per compression context

I agree, and actually I never thought about that before. To me, different transfers are independent in every sense of the word, and sharing a compression context would create interdependence between them.
In theory, one could make them share compression without making them dependent by providing some kind of pre-trained dictionary prior to the transfer. I don't know how common this is across compression formats, but at least zstd supports it. However, I'm against this for the same reason I rejected that approach for sending individual files: It adds quite a bit of complexity which I'd like to avoid.

Just a random side-note here. Should we consider using protobuf instead of json? It seems to have big benefits in terms of size and performance.*

https://blog.mbedded.ninja/programming/serialization-formats/a-comparison-of-serialization-formats/

The problem with ProtoBuf is that it does not have first-class support for self-describing messages: https://developers.google.com/protocol-buffers/docs/techniques#self-description
msgpack is conceptually a drop-in replacement from JSON, whereas trying to use ProtoBuf would be an entirely different beast. I haven't looked at ProtoBuf (& friends) a lot though, so I'd appreciate some analysis on how it could be integrated into Magic Wormhole.

Just a random side-note here. Should we consider using protobuf instead of json? It seems to have big benefits in terms of size and performance.*

As specified, we're only using JSON for the "already built" parts that currently must use it (e.g. version exchange). That has to allow open-ended contents (and anyway would be a new mailbox protocol too). For protocol messages related to just this application protocol the preliminary conclusion is "msgpack". CBOR2 is also similar (and has better performance, IIRC).
I would not consider ProtocolBuffers; it uses pre-compiled specs and generated code, but without the advantages of Cap'n'Proto (essentially its successor) or flatbuffers (more-actually the successor because it's Google too). It supports fewer languages than either.
For "no-parse" options, I'd say "flatbuffers" is probably the front-runner. It supports many more languages, is "no-parse" and doesn't bundle other concerns like cap-n-proto does. It still has the disadvantage of requiring generated code though.

I have a very rough proof-of-concept implementation in the Python client and here post updated Automat-produced state-machine diagrams. These are obviously tied to a particular implementation but I think are still generic enough to be useful.
In case it's not obvious, a DilatedFileSender machine is made for each offer and a corresponding DilatedFileReceiver is created on the receiving side (in response to a subchannel open).
This only supports File transfers right now, not directories or any control messages (i.e. "text messages").

Checking back in here, although the PoC code doesn't support multiple files yet I think we should decide on https://github.com/magic-wormhole/magic-wormhole-mailbox-server/issues/31 before finalizing this transfer spec (and, perhaps that ticket would really be better in this repository in hindsight).
https://github.com/magic-wormhole/magic-wormhole-mailbox-server/issues/31

@meejah Dilated? https://en.wikipedia.org/wiki/Dilation

@abitrolly this refers to the "Dilation" feature in Magic Wormhole -- which I guess is supposed to get at the notion that you "make the wormhole bigger and more useful"...?
See https://github.com/magic-wormhole/magic-wormhole/pull/445 for WIP / PoC

@meejah that's nice, but also it doesn't explain anything AT ALL. :D

.. but also it doesn't explain anything AT ALL. :D

@abitrolly can you state that as a question? I thought you were asking why "Dilated" is in the title?


Display and Update Work Item Details - Display confirmation dialog when navigating away without saving.

https://github.com/almighty/almighty-core/issues/298
If work item data has been changed but not saved, returning to the prior dialog requires confirmation.
Create a common shared component which

takes input as title, text, array of buttons
output would be an event (the parent needs to listen in on this)

@mindreeper2420 / @Mgranfie need your input on task.
Use case : User is one the work item's details screen. He makes a few changes and clicks on "Back".
A confirmation dialogue box needs to show up stating that changes have been made which will be lost if the user navigates away.
Could you let us know the exact confirm behaviour?
We need a title, message, buttons, and what needs to happen on button clicks.

@nimishamukherjee how about this for the moda:
Text:
Do you want to discard your changes?
Buttons:
Cancel        Discard

@nimishamukherjee @joshuawilson just to note, this paradigm might change when the slide-out pane is implemented. We will need to revisit this action after that.

Thanks @mindreeper2420 will implement this for now.


Add Sybase connection

New feature add easy sybase connections

not super familiar w/ sysbase, but seems like you can can use pymssql for it.  Have you tried that connector?

closing this since you haven't replied, happy to reopen if you confirm pymssql does not work


Font Awesome Icons missing when opening synched vault on a different machine.

Describe the bug
When using the font-awesome icon packs, these icons are missing when opening your vault on a second machine.
To Reproduce
Steps to reproduce the behavior:

Add icon to folder and sync
open obsidian vault on another machine
icons are missing

Expected behavior
Icons should sync to your vault across machines.
Screenshots

Additional context
You can see in the screenshot, that the font-awesome icon packs do appear to be present on the second machine, but the icons themselves are not.

if you are using obsidian sync, i highly recommend reading this documentation.


PHP version travis matrix

Thank you for the PR. I think that you've missed using the $COMPOSER_PREFER variable somewhere. And apart from this, what's the purpose of this PR?


Adding fix for anonymous classes in TestNGClassFinder

Fixes #1307
Did you remember to?

[X] Add test case(s)
[X] Update CHANGES.txt

This fixes an issue where if a Factory is used to create anonymous classes when testing as a package, then an exception is thrown:
An error occurred while instantiating class test.factory.nested.FactoryWithAnonymousTestsTest$1. Check to make sure it can be accessed/instantiated.

I reworked the test ;)
LGTM :+1: Ping @cbeust


Feature/add moralis api examples/dev

CONTEXT
The "Moralis API Examples" each provide an example scene and example monobehaviour on a specific use case of the "Moralis." object.
This is a new PR to replace the HISTORY shown. Clean and retested.
HISTORY

Feature/just api examples/dev  ... this PR has been closed as unmerged.
Feature/56 copy scenes to build settings/dev ... this PR has been closed as unmerged.
Feature/menu item for 10x16 layout/dev ... this PR has been closed as unmerged.

CHANGES

Fresh branch off main branch since main branch has changed a lot
All work effort in the HISTORY above has been recreated
All feedback has been addressed and committed.

TESTING

Target: Standalone. Tested with success in Unity Editor. Tested with success in Standalone Build.

SCREENSHOTS
One scene and one menuitem are shown

This project will be released as a standalone repo instead of a PR to main of this branch.


369 Netlify toml

Prefix for fixing #369.

This updates the script to build web to run with different node environments
It creates a netlify.toml file to run different commands based on a specific branch.
It updates the create-env-files script to let us use .env file instead of keeping that information on Netlify. This should allow us to set certain branches up to be "beta" and "alpha" with specific env variables for each all the while keeping this data closed from the public.

Creator

[ ] Build relevant tests if any apply
[ ] Ensure there are no new warnings
[ ] Upload an animated GIF showcasing the solution (if it applies)
[ ] Set a relevant reviewer

Reviewer

[ ] Run test and make sure all tests pass
[ ] Review function and form on iOS
[ ] Review function and form on Android
[ ] Review function and form on Web
[ ] Review new test logic

@burkeshartsis Updated with new [context.production] command.


Capture System.IO.InvalidDataException in PackageArchiveReader..ctor

Occasionally, an invalid zip archive is sent to PUT https://www.nuget.org/api/v2/package/ (Api/PushPackageApi), resulting in the InvalidDataException.
We should handle the exception, log this properly, but avoid it from cluttering Application Insights for the gallery.

@xavierdecoster , I added similar code in staging. Can I take this one?

Is that this one? https://github.com/NuGet/NuGet.Staging/pull/15/files#diff-57f75ce5c7a8d9100d2ca92556881d37R81
Looks good to me!

@skofman1 the stage is yours :)


COOPR-801 Allow specifying host-side ports

This allows us to specify a complete port mapping to a given Docker container. I have also retained backwards-compatibility as a usability shortcut for specifying the same host:container port.
Valid:
8080:8100
8080:8100,55054
8100,55054

Invalid:
8080:
:8100
8080:,55054

This can be used to avoid potential port conflicts, since the DockerAutomator ensures that the host-side port is not already in use by another container (if provisioned with Coopr)...

LGTM


[BACKLOG-38558] SFTP PUT Core To Plugin Movement

:white_check_mark: Build finished in 58m 54s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
assemblies/plugins,engine,plugins,plugins/sftpput,plugins/sftpput/assemblies,plugins/sftpput/assemblies/plugin,plugins/sftpput/impl,plugins/sftpput/ui,ui

:ok_hand: All tests passed!
Tests run: 5324, Failures: 0, Skipped: 14    Test Results

:information_source:  This is an automatic message

You have merge conflict in assemblies/plugins/pom.xml


Error when Installing basic packages

deck@steamdeck ~> distrobox create --nvidia --image ghcr.io/ublue-os/bazzite-arch-gnome --name bazzite-arch
Image ghcr.io/ublue-os/bazzite-arch-gnome not found.
Do you want to pull the image now? [Y/n]:
Trying to pull ghcr.io/ublue-os/bazzite-arch-gnome:latest...
Getting image source signatures
Checking if image destination supports signatures
Copying blob d52c71bbf9dc skipped: already exists
Copying blob 48b6ad6b8535 done
Copying blob 541d10d2cfd2 done
Copying blob c3f076a1ea9a skipped: already exists
Copying config c5571b00d7 done
Writing manifest to image destination
Storing signatures
c5571b00d77724e1c0060fc7b7f07e54bb2ad74d4bc222ef134bb7f0f5f8114b
Creating 'bazzite-arch' using image ghcr.io/ublue-os/bazzite-arch-gnome	 [ OK ]
Distrobox 'bazzite-arch' successfully created.
To enter, run:

distrobox enter bazzite-arch

deck@steamdeck ~> distrobox enter bazzite-arch
Container bazzite-arch is not running.
Starting container bazzite-arch
run this command to follow along:

 podman logs -f bazzite-arch

 Starting container...                  	 [ OK ]
 Installing basic packages...
:: Starting full system upgrade...
resolving dependencies...
looking for conflicting packages...

Package (10)                     Old Version  New Version  Net Change  Download Size

multilib/lib32-libxcrypt-compat               4.4.36-1       0.18 MiB       0.07 MiB
multilib/lib32-systemd           254.3-1      254.4-1        0.00 MiB       0.70 MiB
core/libxcrypt-compat                         4.4.36-1       0.17 MiB       0.07 MiB
core/openssl                     3.1.2-1      3.1.3-1        0.00 MiB       4.52 MiB
multilib/steam                   <IP_ADDRESS>-1   <IP_ADDRESS>-2     0.00 MiB       3.64 MiB
core/systemd                     254.3-1      254.4-1        0.00 MiB       7.67 MiB
core/systemd-libs                254.3-1      254.4-1        0.00 MiB       1.06 MiB
core/systemd-sysvcompat          254.3-1      254.4-1        0.00 MiB       0.01 MiB
extra/xdg-user-dirs                           0.18-1         0.14 MiB       0.04 MiB
extra/yad                        13.0-1       13.0-2         0.00 MiB       0.21 MiB

Total Download Size:   17.98 MiB
Total Installed Size:  50.36 MiB
Net Upgrade Size:       0.50 MiB

:: Proceed with installation? [Y/n]
:: Retrieving packages...
 systemd-254.4-1-x86_64 downloading...
 openssl-3.1.3-1-x86_64 downloading...
 steam-<IP_ADDRESS>-2-x86_64 downloading...
 systemd-libs-254.4-1-x86_64 downloading...
 lib32-systemd-254.4-1-x86_64 downloading...
 yad-13.0-2-x86_64 downloading...
 lib32-libxcrypt-compat-4.4.36-1-x86_64 downloading...
 libxcrypt-compat-4.4.36-1-x86_64 downloading...
 xdg-user-dirs-0.18-1-x86_64 downloading...
 systemd-sysvcompat-254.4-1-x86_64 downloading...
checking keyring...
checking package integrity...
loading package files...
checking for file conflicts...
error: failed to commit transaction (conflicting files)
xdg-user-dirs: /usr/bin/xdg-user-dir exists in filesystem
Errors occurred, no packages were upgraded.
Error: An error occurred
+ '[' 1 -ne 0 ']'
+ printf 'Error: An error occurred\n'

Now fixed as of the latest version.
If you're affected, delete the image in podman so it pulls it fresh:
podman image rm bazzite-arch


Remove "This is a development build!"


Registering the IdentityExperienceFramework and ProxyIdentityExperienceFramework apps through the App registrations (Preview) causes the example custom flows to work only for sign up, but not authentication

The problem itself is that after registration, the redirect is done properly and the user is redirected with a token. When an attempt is made to sign in with the same user, though, a message 'Invalid username or password' is displayed and the user does not get authenticated. I am certain this was with the correct credentials, as using a wrong email or a wrong password was returning different error messages each.
Using the supposedly deprecated 'App registrations (Legacy)' method works, though.

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

ID: bee63035-00ee-f7fd-97a1-69dae5f808ef
Version Independent ID: a3447928-73cc-0290-6d46-b89234757e3b
Content: Get started with custom policies - Azure AD B2C
Content Source: articles/active-directory-b2c/custom-policy-get-started.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

I am also facing same issue (sign up is done, but not able to sign in) but it was working properly few hours back.

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

I've reported this issue to the product team to ask if there is an outage. @negwood, was it also working for you before like it was for Amita? Amita, at what time did it stop working? Do you have any error logs you can share?

We have reached out to the product team to see if there was a recent outage. Tpically this issue happens when the permissions on the ProxyIdentityExperienceFramework aren't set properly. I'd say doublecheck that ProxyIdentityExperienceFramework has Admin consent for Azure Active Directory Graph: User.Read and IdentityExperienceFramework: User_Impersonation. In the meantime if I hear back from the PG I will post the status here.

@MarileeTurscak-MSFT I cannot tell if it had been working before, as this was the first time I have tried working with custom policies.
Thanks for the tips and the attention!

@MarileeTurscak-MSFT I've also encountered this issue. Cannot login with existing local accounts and while I can create new accounts through the custom policy, I can't login with them either.

I have the same issue with a new setup. I have even started from scratch with the policy starter pack without any customizations.

Set allowpublicclient to true in the manifest for proxy app.

A hotfix has been issued for this problem, should be resolved by Monday.
The UI element for the following step in the instructions was made unavailable:
3. Under Advanced settings, enable Treat application as a public client (select Yes).
As a temporary work around:

Go to the 'Manifest blade' in the App Registration for ProxyIdentityExperienceFramework
Set the allowPublicClient value to "true"

@Negwood @MSRobert @vivaladan @AmitavaHazra Please see above.


Why does my code work

Collect 25 gold, and then tell Naria the total.
Use break to stop collecting when totalGold >= 25.
totalGold = 0
loop:
coin = self.findNearestItem()
# Pick up the coin and add its value to the total.
# Get its value with:  coin.value
if coin:
totalGold = totalGold + coin.value
totalGold += coin.value
pos = coin.pos
x = pos.x
y = pos.y
self.moveXY(x, y)
if totalGold >= 50:

    # >= means totalGold is greater than or equal to 25.
    # This breaks out of the loop to run code at the bottom.
    break

self.moveXY(58, 33)
self.say("Done collecting gold!")
self.say(self.gold)
Go to Naria and say how much gold you collected.
I don't get why my code works when i write  if totalGold >=50:
but not when i write if totalGold >=25

This should be in the forum not here in the repository.
You are counting each coin twice.
totalGold = totalGold + coin.value
totalGold += coin.value
Those two lines to the same thing and therefore do it twice . . . which is why you have to check for double the actual amouont...


Add code coverage for tests

This PR enables measuring the code coverage of the tests executed during CI via the Python module coverage.
A detailed report is published to  coveralls.io and can be found here.
For the Python dependency coverage  a new file requirements-devel.txt is being introduced to separate dependencies of the project itself from the ones only required by developers.
Some files are excluded from the coverage measurements:

the tests itself, since they are usually 100% covered by definition and could lead to the impression of a higher code coverage than what we actually have
migrations, since these are auto-generated code
wsgi and asgi files since they are more related to config files, than actual code

Badges for the test status and coverage percentage have been added to the README.
Resolves #5

rebased to master... i think?


vmap gives the most bizarre error

The code snippet is a little complex and I can't post it, but if vmap simply checked how many arguments were passed to a function and compared it to the length of in_axes, you would never fill a screen with the following cryptic error:

Okay so this happened because I fed a very large tuple to a batch argument instead of an array. I still don't understand how to feed an array of PRNGKeys to a vmapped function.

(side-note: using large tuples or lists of values in JAX is not recommended, because it will lead to very long compilation and execution times. You should use arrays of values instead)

I literally couldn't scroll up to see where the error began because this went on for 2k+ lines. It was not useful for debugging. I think it should be, at the very least, truncated.

Hi @half-potato
Looks like this issue has been resolved by improving the error message with the PR #18550. Now the error message for the repro provided by @jakevdp is raised as follows:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
[<ipython-input-12-50e10c8cad06>](https://localhost:8080/#) in <cell line: 8>()
      6
      7 x = tuple(range(1000))
----> 8 jax.vmap(f, in_axes=(0,))(x, 0)

    [... skipping hidden 1 frame]

[/usr/local/lib/python3.10/dist-packages/jax/_src/api.py](https://localhost:8080/#) in vmap_f(*args, **kwargs)
   1247   def vmap_f(*args, **kwargs):
   1248     if isinstance(in_axes, tuple) and len(in_axes) != len(args):
-> 1249       raise ValueError("vmap in_axes must be an int, None, or a tuple of entries corresponding "
   1250                        "to the positional arguments passed to the function, "
   1251                        f"but got {len(in_axes)=}, {len(args)=}")

ValueError: vmap in_axes must be an int, None, or a tuple of entries corresponding to the positional arguments passed to the function, but got len(in_axes)=1, len(args)=2

Attaching the gist for reference.
Could you please verify the same for your use case and let us know if the issue still exists.
Thank you.

Thanks for the follow up @rajasekharporeddy! I think this can be closed.


Specify token kind in Lexer.action's output?

Currently, every action can only map to a single token kind.
If 2 kinds have similiar lexing rule, we have to run these rules 2 times to yield the token.
To solve this, maybe Definition.kind should be a  list of all possible kinds instead of a single string.

Currently workaround: store intermediate result in lexer's state.
This should prevent some duplicated calculation.

Another solution is to leverage #29 , create a custom token data e.g. subKind and store the sub kind in it.
However, this solution can't bind the subKind to the main kind. And the parser is not aware with the subKind.
Maybe Action should be aware with the target kind.


Error event listener doesn't appear to be working

Please avoid duplicates

[X] I checked all open issues and discussions and none of them matched my problem.

Language and Compiler
Vanilla JavaScript
What environment are you using?
Server Side Renderering
When does your problem occur?
When the Unity App is running
What does your problem relate to?
The problem seems Module related
React-Unity-WebGL Version
9.2.1
React Version
17.0.2
Unity Version
2022.2.9f1
What happened?
Version 8.x.x. of the package supported error listeners: https://github.com/jeffreylanters/react-unity-webgl/blob/7b7aac91690b7cd3ed7968e82b20c9f732b04c43/documentation/versioned_docs/version-8.x.x/api/on-error.md#L4 .
When trying to implement an error listener on Verizon 9.2.1 using the addEventListener(‘error’) API, it appears that no errors are caught detected. Is this intended behavior or is there an undocumented API that captures the proper usage of the error event listener?
Thanks!
Reproducible test case
No response
Would you be interested in contributing a fix?

[ ] yes, I would like to contribute a fix

Hi Avinash! Sadly Unity doesn't expose caught errors that well, they're instead using window.error to dump their errors to the console without any proper way to interfere. Previous versions of the module intercepted these error messages, but this was proven to be too hacky of a solution. -- So until Unity implements proper error handling, there isn't anything we can do really.


Add NuGet package README for native packages and link from managed package README

Seems like this error is caused by this PR, might be wrong. Need to investigate more on this.


Accessing process.env.NODE_ENV causes segmentation fault

I've sen other issues around segmentation faults but not related to process.env, it's any var, not just NODE_ENV.
Using Jest CLI v0.4.0
Waiting on 1 test...[1]    1763 segmentation fault  jest

test.js:
console.log(process.env.NODE_ENV);

Node version: tried with both 0.12.0 and 0.12.5
OS: OSX 10.10.3

Jest only supports node .10 right now. Issue #243 has the deets.
I believe they're doing a major push right now in order to make Jest Node 12 / IOJS compatible.

Rightly so, I should not have opened this issue as Jest only supports 0.10 for now.
Feel free to close this issue as it's not quite valid.

For whatever its worth, this segfault isn't specific to Jest, but to contextify, which is used by the version of jsdom that jest uses. https://github.com/brianmcd/contextify/issues/168 has some more details on the error, but from the looks of it, it might even be a problem with node.

Please upgrade to node 4 and jest 0.5.x


Segmentation fault (core dumped) when start to annotate assemblies

Hello,
I'm using TRUST4 (v1.1.5-r561) to assemble TCRs sequenced by 10X platform, but the error only happened on one sample C463T, the other samples were fine, so I want to know how to solve this problem:
"Start to annotate assemblies.
sh: line 1: 782131 Segmentation fault (core dumped)
./TRUST4/annotator -f human_IMGT+C.fa -a TRUST_C463T_S1_L004_R1_001_final.out -t 1 -o TRUST_C463T_S1_L004_R1_001 --barcode -r TRUST_C463T_S1_L004_R1_001_assembled_reads.fa --airrAlignment > TRUST_C463T_S1_L004_R1_001_annot.fa
failed: 35584 at ./TRUST4/run-trust4 line 60."
Is there any misassembled contig? or other errors in TRUST_C463T_S1_L004_R1_001_final.out caused such problem?
Thanks your attention!

This is likely to be a bug from recent updates. Could you please share the files generated from TRUST4? If it is too large, is it possible to share the _final.out or the to_assemble files? Thank you.

Thanks for your response, I'm sorry the file is too large to send. Actually, I can run it successful with the bam file produced from cellranger, but in this way, the report columns include assemble id  rather than barcode id, which is important for the following analysis. So, my question now is how to use bam file to run this pipeline and get the barcode id in the report file for downstream analysis.
best regards,
Qiang Huang
获取Outlook for Androidhttps://aka.ms/AAb9ysg

From: Li Song @.>
Sent: Wednesday, October 30, 2024 10:31:54 PM
To: liulab-dfci/TRUST4 @.>
Cc: Qiang Huang @.>; Author @.>
Subject: Re: [liulab-dfci/TRUST4] Segmentation fault (core dumped) when start to annotate assemblies (Issue #324)
This is likely to be a bug from recent updates. Could you please share the files generated from TRUST4? If it is too large, is it possible to share the _final.out or the to_assemble files? Thank you.
―
Reply to this email directly, view it on GitHubhttps://github.com/liulab-dfci/TRUST4/issues/324#issuecomment-2447352538, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFMB2N6CZUMYWLTFMPNZF2LZ6DUVVAVCNFSM6AAAAABQ3HCXDGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBXGM2TENJTHA.
You are receiving this because you authored the thread.Message ID: @.***>

I'm so appreciate that adding "--barcode CB" is work, thank you very much.
best regards,
Qiang Huang

Could you please share the previous final.out file from the failed run? It would be nice to have that data to debug TRUST4's issues in case other people may encounter it. Thank you!


Create a query to fetch ICS events.

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Create a query that fetches takes any of the following integers as a parameter:

1 - Fetch previous events
2 - Fetch current events
3 - Fetch upcoming events

Use google sheets as a database to store the data.

Schema:

Query Name
Query Type

Event Name
String

Event Time
DateTime

Event Venue
String

Event Description
String

Additional context
For reference of how to use google sheets as a database, you can refer to the following repo: https://github.com/DesignrKnight/sheets-api

ICS database is already there on MongoDB, use Google Sheets only if the entire DB is getting switched to Sheets
If you finish up Avenue API quick, which would not take much time especially the APIs, directly use that and query by organiser instead of creating something up from scratch

Note: Both comments are my opinions and not an instruction

ICS database is already there on MongoDB, use Google Sheets only if the entire DB is getting switched to Sheets
If you finish up Avenue API quick, which would not take much time especially the APIs, directly use that and query by organiser instead of creating something up from scratch

Note: Both comments are my opinions and not an instruction

What I thought is that, if we use sheets as a db for this feature, it would be easier to add an event by just adding an entry in that sheet as we don't have an admin panel as of now.

ICS database is already there on MongoDB, use Google Sheets only if the entire DB is getting switched to Sheets
If you finish up Avenue API quick, which would not take much time especially the APIs, directly use that and query by organiser instead of creating something up from scratch

Note: Both comments are my opinions and not an instruction

We will follow the second suggestions as an instruction


Form Components - add optimisation to see form functionality in storybook

When reviewing the radio component #110 , we discussed to maybe add something to have form functionality in storybook to see how components work after submitting. For the radio, this would i.e. show the error state when an input is required but none was selected.

duplicated by #596


Add wrap_response_data option

The wrap response data should be an option on the model itself, not the class calling it. I can not merge it like this (regardless of using the wrong name in the composer.json file and the failing tests).

@Edwin-Luijten yeah, I noticed it right away and discussed with Robert. Thanks, guys!


I can't delete the current page

There's no way to delete the current page I'm visiting, I have to go to parent /contents

Be more specific please

In the UI, Plone Classic has an administration action that will delete the current page. Volto doesn't provide this option, to delete a page you have to go to its parent's /content view and delete it from there.


Add raw MUA data to DataJoint and NWB

The unsorted MUA data per session by channel and trial are available at CRCNS. It would be useful for users to have this data available in DataJoint and the exported NWB files in case they want to re-spike-sort the data or re-segment the waveforms using other methods. Using low threshold multi-units may reveal interesting, different results compared with the manually sorted data.

Including Raw data is beyond the scope of this project, inclusion of raw data is planned for the Mesoscale Activity Project (https://github.com/mesoscale-activity-map/map-ephys), where raw neuropixels data is to be included as part of NWB 2.0 export


Doesn't work if string contains NSLinkAttributeName

KILabel doesn't work with this NSAttributedString:
"ASDSAFASFA {
NSLinkAttributeName = "https://google.com/";
}"
It works only if:
"https://google.com/ {
NSLinkAttributeName = "https://google.com/";
}"
Or:
"https://google.com/"
I.e. plain string should contains link, but if plain string doesn't contains links and attributes for this string contains it, then KILabel doesn't shows links.

Sorry, incorrect NSLinkAttributeName. This attribute from open-source library and it's not equal NSLinkAttributeName from Apple.
//NSLinkAttributeName = NSLink


[Request for External] codeunit 333 "Req. Wksh.-Make Order" - Protected variables

Hello, we need the following currently global variable/-s to be set as protected. Can you please check this? Thank you!
protected var
   NextLineNo: Integer;

Protected variables cannot be used in codeunits, see AS0676


Is there any difference between spring.cloud.gateway.defaultFilters and GlobalFilter?

Describe the bug
Both method signatures are the same and will be applied to all routes.
Even the combined filter chain is sorted by the org.springframework.core.Ordered interface.
The purpose is exactly the same, what else is different? concept?
What are the reasons for designing two implementations?

default filters are just a shortcut to allow filters that are typically per route to be applied all routes.


➖ Remove orjson and ujson from default dependencies

User description
Description
This PR fixes #
Notes for Reviewers
Signed commits

[*] Yes, I signed my commits.

PR Type
dependencies

Description

Removed ujson from the default dependencies in pyproject.toml.
Removed orjson from the default dependencies in pyproject.toml.

Changes walkthrough 📝
Relevant filesDependencies

pyproject.tomlRemove `ujson` and `orjson` from default dependencies

pyproject.toml
Removed ujson from default dependencies.  Removed orjson from default dependencies.

+0/-4

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

📝 Docs preview for commit 12d6305ecc0b420917a9adfe05c6d8166be7d72e at: https://khulnasoft.github.io/khulnasoft/readyapi

📝 Docs preview for commit afda88a2be87139f9a7df48b7c5485ac4f84f320 at: https://khulnasoft.github.io/khulnasoft/readyapi

📝 Docs preview for commit 97c63caa8fc0c40ecbb09afd894d4b1e5d073036 at: https://khulnasoft.github.io/khulnasoft/readyapi


Serverless Automatic needs --listen

The workers won't actually be able to start up. I fixed this in my own build and it worked. https://github.com/runpod/containers/blob/main/serverless-automatic/start.sh#L7

What kind of error are you getting that is preventing startup?

--listen has been added, the worker has also been refactored and relocated here: https://github.com/runpod-workers/worker-a1111


Tech Catcher!

What a lovely day, rain then sun and now SSD's. You must catch the raining tech! Each piece of tech gives 1 point
But each Robot takes 5 points away. This was inspired by a simple project you might have made in Scratch where you catch Apples.
How do you play your game?
The Controls are:
A: Left
D: Right
It doesn't get simpler than that
Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Thanks for your PR!

Ive implemented a new aspect.

Congratulations!  If you haven't already received one, and if you're a teenager, please fill out this form to receive your Sprig device!  Also feel free to show off your game in the Hack Club's Slack #sprig channel!


fix: adds a failure on warn+ for asciidoc

Pull Request Template
📲 What
fix: adds a failure on warn+ for asciidoc
🤔 Why
it currently doesn't fail at all on errors, just fatal
🛠 How
👀 Evidence

🕵 How to test

❤


implicit setters lack appropriate runtime check in Dart 2 mode

class A<T> {
   T f;
 }

 void main() {
   A<num> a = new A<int>();
   a.f = 1.0;
 }

expected to throw but does not.
This makes test/foundation/covariant_templates_test.dart Flutter test fail.

Kernel does not generate a regular body graph for implicit setters. Therefore, the argument check did not happen. A cl adding the check to the special code for implicit setters is out for review.


Trying to execute with EVM Crisper in the same way as I usually do, but scientific notation doesnt work any more

This failed:

Figured it out that it was the scientific notation... but it used to work like that any more

@sembrestels was that feature removed? @GriffGreen have you tried with e instead of E ?

I'll try with lower case e next time and see... maybe that was the issue

Ok feel free to open the issue again if stills happens