[BUG] Relative Path in Image URI

Describe the bug
Relative paths are not supported in file:// URIs.
To Reproduce
Steps to reproduce the behavior:

Run command 'wash start actor file://foo_s.wasm`
See error:  failed to convert file://foo_s.wasm/ to a file path

Expected behavior
A missing / after file:// indicates a relative path. In the example above the Wasm file should be found in the same directory where the wasmCloud Host was started. The wadm model definition documentation specifically note that relative paths are supported.
Screenshots
catalyst@box ~ % wash start actor file://c23_0_601_actor_service_inventory_s.wasm

Failed to start actor [file://c23_0_601_actor_service_inventory_s.wasm] on host [NA...]

Caused by:
    failed to fetch actor: failed to convert `file://c23_0_601_actor_service_inventory_s.wasm/` to a file path

**Environment (please complete the following information) **

OS:  Linux and MacOS
Shell bash and zsh
wash Version 0.20.2

Additional context
Same error in wadm (wash app deploy) and wash (wash start actor).

Piggy-backing off @brooksmtownsend's comment: I did some more investigation today and ran into all sorts of edge cases related to relative paths. The host path parsing code will be much simpler if we require absolute paths. That means clients such as wash and wadm need to convert relative paths into absolute paths before sending the control interface request to the host

That means clients such as wash and wadm need to convert relative paths into absolute paths before sending the control interface request to the host

Right, I think the host should only support absolute paths, but writing
wash start actor ./myactor_s.wasm

Should work, where wash fills in the file://$(pwd)/ prefix.

Absolute paths aren't a practical option, and just really bad form.
Linux:       /home/{username}
macOS:	/Users/⟨username⟩
Android:	/data/media/⟨userid⟩
Windows: ⟨root⟩\Documents and Settings\⟨username⟩

Absolute paths aren't a practical option, and just really bad form.
Linux:       /home/{username}
macOS:	/Users/⟨username⟩
Android:	/data/media/⟨userid⟩
Windows: ⟨root⟩\Documents and Settings\⟨username⟩

I acknowledge that different platforms have different file system base paths, and I think loading an actor from a file assumes knowledge of the file system the host is running on. If you're outside of local development and running actors from a file, you're going to be doing it from a path that you know about. I'm happy to be wrong about this, though, if your use case would depend on relative paths we can revisit this!
What I was suggesting for wash would nicely address the localdev use case, and require that wadm manifests specify absolute paths for localdev so you can predictably deploy your actors even if you ran wash up from different directories.

An application specification for wadm deployment:
spec:
  components:
  - name: runs-anywhere
    type: actor
    properties:
      image: file:///wasm/anywhere_s.wasm
    traits:
    - type: daemonscaler
      properties:
        replicas: 10
        spread:
        - name: nanopi
          requirements:
            zone:: global

The relative path makes this super simple.

It does look simple, but the complexity comes from knowing where that path is relative to. Is it relative to where wadm is running? WasmCloud? Wash? Intuitively I may think it's relative to wherever the manifest lives, but that's only correct if you also run wasmCloud from the same directory. We can add relative path support to the host but I think it will cause more confusion than pain that it will solve

The behaviour is documented (which makes this a bug):
INFO


no more than four Bytes?

If the seed file has more than four byte, then only the first four bytes can by copied to queue.

Hm, it is strange, I didn't touch this part of AFL at all. Could you provide more details?

The test case requires only 4 bytes. Did you try this with another application?


the grub command line oopens after installing with this script

i am using virtualbox by the way if that helps in any way the vm only has 4 GB of ran bc my host only has 8 GB with 25 GB of storage

Same scenario for me. the kernel image isn't in the (hd0,3) partition along with the rest of the standard files--it is instead in (hd0,2).


ENH: Share some image driver routines with other drivers

VIC's driver structure looks something like this:
| - drivers
| --- shared
| --- classic
| --- python
| --- image

I'm proposing a minor reorganization to cut down on duplicate code as we begin to add other drivers that will use significant portions of the Image Driver.
| - drivers
| --- shared_all
| --- shared_image
| --- classic
| --- python
| --- image
| --- cesm
| --- wrf_hydro
| --- nasa_lis

vic/drivers/shared becomes vic/drivers/shared_all and includes code shared between all drivers.
code in vic/drivers/image that is general and will be used by any driver running VIC in some version of the Image Driver goes in vic/drivers/shared_image. Most notably, this will include the netCDF routines for reading and writing history and statefiles.  Drivers (e.g. nasa_lis) do not have to use this code but it will be available to them.  The reason not to put it all in the existing shared directory is some of this code has dependencies not required by the Classic Driver.

This will significantly cut down on the code in the drivers that look mostly like the Image Driver but differ in coupling (e.g. #265).

The names image and shared_image are somewhat similar and I find it potentially confusing - "image" appears to be both a general category (referring to image mode) and a specific flavor of that category.  Perhaps "image" should be changed to something like "image_default" or some other name that indicates that it is one of many image mode drivers?

(otherwise I think your new structure is a good idea)

That's a good idea. Maybe something that conveys that the image mode is a standalone driver (all the other image modes are part of a larger framework).
vic_image_standalone is a bit off a mouthful (as opposed to vic_classic or vic_image. I suggest moving ahead with the directory split (we can always rename that), but to keep @tbohn's comment in mind. If anyone has a good title that'd be great.

How about:
vic_default
vic_generic
vic_vanilla
vic_nilla

@tbohn - do you have any branches in progress right now (e.g. statefile stuff from #367)?  If you haven't started that yet, I'll bust this issue out real quick.

Also - I'm not sure I can bring myself to calling anything vic_nilla.  Way to gangsta, VIC is more of a 1990s minivan.

I haven't yet started the statefile mods to VIC code - focusing on tonic first.  So bust a move.
How about "vic_standalone" without the word "image" in it, since it's already going to be under the "image" umbrella?

closed via #392


Add example file

Would it be possible to add a working example file. I am kind of new to PHP name spaces, and not quite sure how to start this off. It seems like if I try including the HubSpotService file then it isn't auto-including any other referenced files (e.g. BlogPosts). I am sure I'm just making a namespace noob mistake.

I plan to add more documentation in the future. In the meantime, try this:
<?php

# index.php

require "vendor/autoload.php"

use Fungku\HubSpot\HubSpotService;

$hubspot = HubSpotService::make('demo');

$contacts = $hubspot->contacts()->all(['count' => 10]);

foreach ($contacts as $contact) {
    echo $contact['properties']['email']['value'];
}


Rename ctx_{get,set}_params to {get,set}_ctx_params

Recently, we added dispatched functions to get parameter descriptions,
and those for operation context parameters ended up being called
something_gettable_ctx_params and something_settable_ctx_params.
The corresponding dispatched functions to actually perform parameter
transfers were previously called something_ctx_get_params and
something_ctx_set_params, which doesn't quite match, so we rename them
to something_get_ctx_params and something_set_ctx_params.
An argument in favor of this name change is English, where you'd
rather say something like "set the context parameters".
This only change the libcrypto <-> provider interface.

I have it draft for the moment, just to verify that it's complete...

Not a draft any more

Could we replace you with a couple of sed scripts? :)

Well, what can I say?  "sed made me do it"?  :laughing:

Merged
92d9d0ae2b10b12e42d33047b60e64ebfc296596 Rename ctx_{get,set}_params to {get,set}_ctx_params


Fixed Type Annotation issue #9 in #opentrack-Metriks-9

Contributed by vGHC 2020 Attendees, Here are their respective GitHub accounts with name:

-

Jyotsna Priya: https://github.com/jyotsna-priya
Deepthi Kailash: https://github.com/deepthikailash
Menna Khaliel: https://github.com/Menna13
Anshuma Jain: https://github.com/AnshumaJain

Coverage remained the same at 100.0% when pulling e3c0c0b127118ec56c04a576e5d279e7b502b51d on AnshumaJain:master into a0e62df847b8997bfc32b337d7e41db16d0a8ce4 on intuit:master.

Closing this PR to create a new one with the required changes.

Closing this PR to create a new one with the required changes.


Layout issue if using message view controller as a child view controller with autolayout.

If adding a message view controller which uses auto layout to size to the same as it's superview, it is not possible to layout the collection view frame before viewWillAppear: occurs.
Which means when [self.view layoutIfNeeded] is called on viewWillAppear: the frame size of the UICollectionView is wrong.
I can provide an example.

a solution for this is to add a width & height constraint before adding to the superview:
[chatVC.view addConstraints:[NSLayoutConstraint constraintsWithVisualFormat:@"H:[view(width)]"
                                                                    options:0
                                                                    metrics:@{@"width":@(self.containerView.bounds.size.width)}
                                                                      views:@{@"view":chatVC.view}]];
[chatVC.view addConstraints:[NSLayoutConstraint constraintsWithVisualFormat:@"V:[view(height)]"
                                                                    options:0
                                                                    metrics:@{@"height":@(self.containerView.bounds.size.height)}
                                                                      views:@{@"view":chatVC.view}]];

constraints I wish to use rather than width & height:
[self.containerView addConstraints:[NSLayoutConstraint constraintsWithVisualFormat:@"H:|[view]|"
                                                                           options:0
                                                                           metrics:nil
                                                                             views:@{@"view":chatVC.view}]];
[self.containerView addConstraints:[NSLayoutConstraint constraintsWithVisualFormat:@"V:|[view]|"
                                                                           options:0
                                                                           metrics:nil
                                                                             views:@{@"view":chatVC.view}]];

I was having the same issue - my chat view controller was embedded in a container view controller. I was able to fix the issue by forcing a layout pass just before the chat view controller is added to its container:
- (void)willMoveToParentViewController:(UIViewController *)parent {
  [super willMoveToParentViewController:parent];
  [self.view layoutIfNeeded];
}

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


MDX v2

Seems like MDX v2 is just around the corner so we should investigate upgrading to it soon.
We should also verify whether mdx-bundler is still needed with MDX v2.

MDX v2 is supposedly similar to xdm which mdx-bundler is using - https://github.com/wooorm/xdm/issues/94
Other than that, the main difference is whether you want the plugin to be a bundler or just a compiler, in which case MDX v2 / XDM suffice.

Update: I've started investigating replacing mdx-bundler with @mdx-js/esbuild however this doesn't seem to be entirely trivial given esbuild.build() expects a "physical" file path instead of an input string (which is what the current Contentlayer implementation is structured around). This is something mdx-bundler is providing out of the box which is why I'll stick to mdx-bundler for the time being (until this becomes easier to do - or there's another reason to do so).
Once https://github.com/kentcdodds/mdx-bundler/pull/152 I'll update the mdx-bundler dependency.
Note: Practically speaking though Contentlayer users should already be using MDX v2 today.

Note: Practically speaking Contentlayer users should already be using MDX v2 today.

What do you mean by this Note? How can/should we practically use MDX v2?

Note: Practically speaking Contentlayer users should already be using MDX v2 today.

What do you mean by this Note? How can/should we practically use MDX v2?

Sorry for not being clear enough in the previous message. I meant to say that Contentlayer ships with MDX v2 out of the box since version 0.1.0.

Closing in favour of https://github.com/contentlayerdev/contentlayer/issues/247


flake: fix patch for nix-eval-jobs

We carry a custom patch for nix-eval-jobs that causes the tool to not unset the NIX_PATH environment variable anymore. The patch does not apply anymore starting with d0bcdae (clang-tidy: use trailing return type, 2024-11-10) because surrounding code has changed. Other than that the patch still applies.
The mentioned commit was released with nix-eval-jobs v2.25.0, which nixos-unstable has recently upgraded to. Adapt the patch to unbreak the build.
Fixes #255.

Thanks!

Hm. I don't get why the Linux test keeps failing. It passes on my machine :thinking:

(Sorry, missed your writeClosure fix - Just cherry-picked it)

No worries :) Thanks for fixing up, didn't have the time yesterday when creating the MR to bring it over the finishing line.

Will need to fix the Cargo flake tests later - Will merge as long as the integration tests pass.

Ok. :+1:

Hm. I don't get why the Linux test keeps failing. It passes on my machine 🤔

It fails on newer Nix versions (not pinned by the devShell, regrettably).

Pushed another fix to make this work with older nixpkgs.

Sorry for the late reply, let's merge this now and fix the remaining issues later.


malloc_usable_size incompatible with _FORTIFY_SOURCE=3

Some applications use malloc_usable_size (despite the glibc manual stating it is for diagnostic purposes only). This is currently incompatible with _FORTIFY_SOURCE=3. This should be documented in the compiler options guide.
https://archlinux.org/todo/prepare-packages-for-d_fortify_source3/

Oops, looks like @david-a-wheeler won that race 😃
I like your direct suggestion to remove the use of malloc_usable_size.  Could you also please incorporate the idea that using realloc to expand to the usable size should also work?  That's a pretty quick operation now with glibc malloc.


How to UT baobab branch components.

I'm just wondering how you UT a component that is wrapped in a baobab HOC? I could export the stand alone component from the module but then I have to pass in the tree as part of the context.
I noticed that react router now passes in children and params as props, I wonder if taking that approach in baobab HOCs would make testing easier?

Intuitively I'd say you probably want to UT the unwrapped component itself since it only relies on its props & state if you don't bring the tree in. But this makes the component export a bit difficult since you have to branch it somewhere else through HOC.
Would it be complex for you to provide a context to your component when UTing it?

@briandipalma: could you elaborate on react-router's strategy please? What I want to avoid is the user having to explicitly pass the tree as a prop from the top of your component hierarchy to the bottom.

https://www.youtube.com/watch?v=Q6Kczrgw6ic at about 14 minutes you can see them talking about passing in params as a prop and the children that each router managed component is meant to be rendering. I'm not sure exactly how it works but I imagine they pass in the data as props from their HOC. So what you'd do is pass tree and cursors on this.props instead of this.context.
I think it's cleaner for UTs and makes sense, at the end of the day your component is going to be bound to Baobab in some way. You may as well use props for everything including accessing the tree and cursors.

I can provide a context with withContext but that is deprecated so I'm not sure how I can UT a component that relies on context in a forward compatible way.

Ok, I see how react-router does. The problem with the mere props approach is that, even if this is convenient in a router case where you won't have to propagate the params across more than one or two component, in Baobab's case this becomes nuts because you'll have to propagate the tree from the top to the bottom of your component's hierarchy and as a typical app has at least 8-10 levels, this is really a drag.
However, if you do use, HOC, here is what I could do: let you access the original component of the composed component through HOC. This one indeed only relies on props without the tree.
Example
/* component.jsx */
class MyComponent extends Component {
  render() {
    return Hello {this.props.name};
  }
}

const ComposedComponent = branch(MyComponent, {
  cursors: {
    name: ['name']
  }
});

export {ComposedComponent};

/* test.jsx */
import {ComposedComponent}

// Accessing the original component for easy UT:
const MyComponent = ComposedComponent.original;

However, I think that generally, the pattern followed by Flux users in this case is to separate clearly clever from dumb components so you only test the latter ones.

So in the higher order branch component what you can do is this
render() {
      return <Component tree={this.context} cursors={this.context.cursors} {...this.props} {...this.state} />;
    }

This makes the component rely on props and not context so making it easier to UT if exported without the HOC wrapper. The HOCs can deal with the drag of passing in the tree and cursors instead of having the component author do it. Does that make sense?

Yet it makes sense. But do you need the tree or the cursors in the Component?

Different components need different things, if you pass in cursors={this.context.cursors} and there are no cursors used in the wrapped component, no harm is done as this.props.cursors wont be accessed in the wrapped component. There is no need to know if the wrapped component uses the tree or cursors or not, just pass it in for all cases.
Umm maybe I'm making a mistake in my architecture. I'm using the cursors and tree for their event emitting properties so they can replace a dispatcher. Maybe that's the wrong approach? Thing is I'd then have to pass in a dispatcher down all my components. It would be neater if Baobab's HOCs simply passed in the tree and cursors as props and let me decide to use them when/if I need them.

Yes, using the Baobab event emitter to add an action layer and update the state is a fine idea and I used to do so myself. But lately, I switched to raw functions that are easier to live reload. My component then require said functions.

The issue with using functions for action/event processing is that you can't bind to a components state. Say for example you have 10 instances of the same component. You can't use a stateless function to update a specific component on a user action, unless you close over the components cursor?
Even then it's less flexible then emitting events as anything can register for those events so if a user of your component needs to update some state following an event all they need to do is register for the event. While if you use a function they can't. I guess that's not a concern if you control all of an applications code...
I might explore this approach and see if it helps simplify my code. Thanks for the suggestion.

My approach would be to create functions taking the tree and optionally some other arguments then pass actions to the tree and curry them by giving them the tree through context in clever components or something of the kind. But I must find time to experiment this properly.

Any update about this  @briandipalma?

I took a quick look at the new docs for Baobab 2.0 and it looks much more testable, all data including actions are passed in as props. Looks great to me, thanks!


Unable to configure default

This is my detekt build convention:
plugins {
    id("io.gitlab.arturbosch.detekt")
}

detekt {
    // Bump this to receive new / adapted rules
    toolVersion = "1.23.6"
    val componentFile = rootProject.file("detekt.yml")
    val globalFile = File(getGitRoot(), "detekt.yml")
    require(componentFile.exists() || globalFile.exists()) { "No detekt.yml found" }
    val configFile = if (componentFile.exists()) componentFile else globalFile
    config.setFrom(configFile)
    buildUponDefaultConfig = true
}

tasks.withType<Detekt>().configureEach {
    // do not run detekt on generated files
    exclude { spec ->
        spec.file.absolutePath.contains("/build/")
    }
    reports {
        // html and xml are by default on, we don't need txt
        txt.required.set(false)
    }
}

and this my detekt.yml which contains this configuration:
naming:
  MatchingDeclarationName:
    active: true
    mustBeFirst: true
    multiplatformTargets:
      - 'android'
      - 'desktop'

I have files underneath desktopMain and androidMain named PlatformUITest.desktop.kt and PlatformUITest.android.kt respectively. Still, Detekt does not acknowledge the configuration above nor does it work with the default set of multiplatformTargets out of the box:
> Task :app:detektMetadataCommonMain FAILED
Property 'naming>MatchingDeclarationName>multiplatformTargets' is misspelled or does not exist.

Which version of the plugin are you using?

1.23.6 as well: io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.23.6

1.23.6 as well: io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.23.6

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.

This issue was closed because it has been stalled for 7 days with no activity.

My bad, this seems to be unreleased still.

My bad, this seems to be unreleased still.

Was this implemented in a PR? Could you link it?

I think it was this one: https://github.com/detekt/detekt/pull/6426


update yed to 3.14.1

Looks good, could you please squash your commits?

I will re-do the PR as a squashed commit.


Compatibility Matrix vs Platform Features Page?

Is one of these pages redundant?
https://github.com/openservicebrokerapi/servicebroker/blob/v2.14/compatibility.md
https://github.com/openservicebrokerapi/servicebroker/wiki/Platform-Features

made the wiki point to the compat page

We have fixed this!


Hotkeys stop working sometimes after using dialogs with CompletionComboBox

The new CompletionComboBox class commits Qt event system crimes which break hotkeys even after it is no longer visible.
For an example:

Open the type dialog
Provide invalid input, then wait for the error
Make the input valid after the error appears
Once the error disappears (new input parsed successfully) close the dialog
Notice the type dialog can no longer be opened with the Y key

Fixed in build 2835.


missing authentication credentials for REST request [/]

elastalert:3.0.0-beta.1

ElastAlert Server question?
https://github.com/bitsensor/elastalert
Are Elasticsearch, Kibana and ElastAlert Server running normally?

ElastAlert Server question?
https://github.com/bitsensor/elastalert
Are Elasticsearch, Kibana and ElastAlert Server running normally?

yes，this error appears in the ElastAlert Server. they are running normally .when I create rule in the kibana, the Elasticsearch needs username and password. I have configured es_username, es_password in elastalert.yaml

I don't think I'm in particular trouble, what's wrong?

I don't think I'm in particular trouble, what's wrong?

I don't think I'm in particular trouble, what's wrong?

Incorrect permissions inside the Docker container? #101
Rule not found, it may have been moved or deleted. #157

It is needed to stop the docker instance and give the adequate rights to rules and rule_templates.
chmod -R 777 rules rule_templates

sudo chmod -R 777 rules rule_templates

@Level1Accelerator
Is the problem solved?

@Level1Accelerator  ping

If the issue has been resolved or you have given up on resolving the issue, close the issue.


Typo & grammar fix | English

Description:
Fixed the grammar of a string and edited a typo in the English translation.
Motivation and Context:
Noticed the typo on the installation of the program. Fixed it in the source.
How Has This Been Tested?
Reinstalled from source and the string has changed.
Screenshots (if appropriate):

Types of changes:
String changes
Checklist:0

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage increased (+0.02%) to 16.332% when pulling 43e745523ad68e0f25a617ea7420763ebfaf0d77 on CallMeSlinky:master into c5d5b21cde79d4e8899360e5fd70c18f1c74d15f on LN-Zap:master.

@CallMeSlinky I rebased this PR and merged in #920. Thanks for the PR.


[WIP] Grouped notifications into one and show the notifications as a list

This is expanding notifications not complete yet.
Wanted some suggestions if this can be made better
Expanded:

Default view:  (the latest message is shown here) What would be the most appropriate content text here?

Automated message from Dropbox CLA bot
@kunall17, it looks like you've already signed the Dropbox CLA. Thanks!

Updated

'New 4 messages' should be '4 new messages'.
If the message does not fit anyways, we should just say 'N new messages' without the 'from N conversations'.
Do we even want to group them though? How does it work in Slack & FB Messenger?

Without grouping right now we are replacing the old notifications, hence loosing content
Slack and whatsapp both group messages in a similar way

Fixed and added a commit to show the number of messages in the home screen
This is dependent on different launchers, library I used is popular and works well on my sony phone

Looks good to merge. Please rebase.

Updated

Great. Thanks.


Object Documentation Export button doesn't work

Describe the bug
The Object Documentation > Export button doesn't export a copy of the selected content.
To Reproduce

Navigate to the Objects tab
Select "No package" and any object (e.g. Account)
Click the Export button

As a start, when you have the feature "export" on in a page, but user is on a tab that does not implement that feature you will have that message. Peevious you had no information.

I know it does not answer yet your issue.

That would be like this:


After installation, it remains in a loading state

The device is Synology SA6400, with the latest 7.2 firmware;
After installation, it remains in Loading mode;
Is it a problem with my settings, or is it because I have too many groupware containers and files?

But perfectly executed under the Ubuntu system

It would be great if monitoring all servers within the local area network could be achieved.

Same.

I'd really like to get this thing up and running. Unraid 6.12.11.
Thanks!

I'm also on unraid.
Is it that our volume mappings are wrong - or is something else afoot?
/var/run/docker.sock > /var/run/docker.sock
/hostroot/mnt/user > /mnt/user/appdata/
/hostroot/var/lib/docker/ > /var/lib/docker/

Same behaviour on unraid 6.12.12

I'm also on Qnap, how to run doku in NAS?


Images wont print in the PDF

I am using "phantom-html-to-pdf" to convert to PDF the HTML output of my process. The HTML has 2 images and they are not printed. Instead i get a rectangle where the image is supposed to be printed and also in a wrong location.
wrong-output.pdf
If i decided to press CTRL-P i can print the HTML output using my PDF Printer without any issue.
correct-output.pdf
Is there any reason why you think is doing this? Is there any trick to enable the printing of the images and CSS associated to the HTML?
Regards,

@alfredoborrero can you post the html source that you are trying to convert? it will be helpful to reproduce the issue.
also, can you give us your OS, node version? thnks!

 Micro Controller Description

            PinA0:test
            PinA1:test
            PinA2:test
            PinA3:test
            PinA4:test
            PinA5:test
            Pin0:test
            Pin1:test
            Pin2:test
            Pin3:test
            Pin4:test
            Pin5:test
            Pin6:test
            Pin7:test
            Pin8:test
            Pin9:test
            Pin10:test
            Pin11:test
            Pin12:test
            Pin13:test

`

 Micro Controller Description

            PinA0:test
            PinA1:test
            PinA2:test
            PinA3:test
            PinA4:test
            PinA5:test
            Pin0:test
            Pin1:test
            Pin2:test
            Pin3:test
            Pin4:test
            Pin5:test
            Pin6:test
            Pin7:test
            Pin8:test
            Pin9:test
            Pin10:test
            Pin11:test
            Pin12:test
            Pin13:test

`

The NodeJS version is 4.3.0 and i am using Windows 7

@alfredoborrero the problem is that you can't reference a local image like that (because of how phantomjs works with local files).
you have two options to make this work.
1.- inline the two images in the html as base64
<img alt="Embedded Image" src="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAADIA..." />
so you won't have any problems because of path resolution.
2.- request the two images with the file:// url pointing where your images are in disk
example:
file:///etc/path/to/my/image/image.jpg

Additionally to the second option, you need to also pass allowLocalFilesAccess:true See the list of options here.
You may also consider using html base tag to set the root address to your images and then use just relative paths.
Or consider exposing images through web server and reference images using http.

Hope this is clear now. Closing.


minor installer tweaks

Coverage decreased (-0.7%) to 48.84% when pulling d2abd98cf43b1ded6908d5c1c2709ec1725ac47c on rfecher:minor-installer-tweaks into 30fd9de8cca83c32b26716ff5bb9e4f531020972 on locationtech:master.


Improve adding member from search/invite page

From the search / invite page (after starting a new chat and you want to invite a member), adding a member feels strange.

Indeed, hitting on the member name or the avatar aren't taking into account. However the status of the member, or the rest of the item will work.
Unless this is wanted, maybe we could fix it ?

Yeah this is thing that force me to manually add people to my own group i created. Some users find it difficult to add users....

-> when the edittext is empty, the list of members is the 1:1 room ones.
-> if you search a member by a pattern, it will search by any known room member.

It was triggered because a long click listener was set on the username text and on the avatar (to copy the matrix id)
so, the clicks were ignored.
The userid can be copied with a long click on the username (member details page)
Theses long clicks are removed to avoid such weird behaviours.


    RUN pip wheel -r requirements.txt --wheel-dir=wheels

build:
    FROM +deps
    COPY x.py x.py
    SAVE ARTIFACT x.py /x.py
    SAVE ARTIFACT wheels /wheels

docker:
    FROM +deps
    COPY +build/x.py x.py
    COPY +build/wheels wheels
    COPY requirements.txt ./
    RUN pip install --no-index --find-links=wheels -r requirements.txt
    ENTRYPOINT ["python3", "./x.py"]
		SAVE IMAGE rockerboo/x:latest

1 failure:

correct / expected / unexpected

  1. test_long_run:

    (source_file
      (block
        (version_command
          (version_major_minor))
        (from_command
          (image_spec
            (image_name)
            (image_tag)))
        (workdir_command
          (string
            (unquoted_string)))
        (run_command
          (options
            (mount
              (string
                (unquoted_string)
                (unquoted_string))))
          (line_continuation)
          (shell_fragment
            (line_continuation)
            (line_continuation))))
      (ERROR))
)

Thanks

Hi @rockerBOO , thanks for the example.
It is failing because of the trailing space after the backslash in this line:
    libsm6 \

I’m surprised that’s even allowed in the earthfile — I’ll give it a try :-)

ok, It is indeed working.
How the eol is managed in earthfile is still puzzling me

thank you, I can work around this for now.


user database is broken after daemon restart

show stopper again!
from Travis Build 429.3
https://travis-ci.org/akiradeveloper/akashic-storage/jobs/117433286
at the first start I created a user
userId: 8eVWYLNBKHWzI8IbmvHBXnXb6S6rKQSb0oE4CtT7E29iXvqN3Xt9aPcNmrHyqO45

and after restart, I created an another user then it should be two users in the db
userId: YS4bC2evVS9vcuV9Khb0HkjqM4bLaw0CIGyTQdPTcjCUU3I8EmznxhQBQHbukHBZ

but the result of akashic-admin-list is only one user created after restart.
<AdminListResult>
     <UserId>YS4bC2evVS9vcuV9Khb0HkjqM4bLaw0CIGyTQdPTcjCUU3I8EmznxhQBQHbukHBZ</UserId>
</AdminListResult>/tmp/akashic-storage-test

this is why all the authorized requests don't succeed after restart
mc: <ERROR> Unable to stat ‘aka’. The request signature we calculated does not match the signature you provided. ...
mc: <ERROR> Unable to stat ‘aka/myb/’. The request signature we calculated does not match the signature you provided. ...

caching issue?

this is probably a caching issue.

make a user
restart
make a user (! without reloading from backing store)
get a user -> not found

There should be a unit test for caching package.

make a Cache[String]
put a String
call invalidateAll against guava.Cache (then all in-memory cache is lost)
get a String
get a String // idempotency check

fixed by https://github.com/akiradeveloper/akashic-storage/commit/3c5a499d0a98e5c7b021b82cc88433ad7601bd4f


VSCode: Fix local embeddings status for non-git workspaces, git repos w/out remotes

When opening a non-git repo, instead of the enhanced context selector showing a non-functional "enable embeddings" button, do not show local embeddings status.
When opening a git repo without a remote, instead of showing a spinner indefinitely and "Indexed", show the circle-slash. Fixes #2217.
During embeddings index generation, show the current file being indexed in the status bar item tooltip.
If embeddings index generation fails, show the error message in a tooltip on the status bar. (Test coverage and improvements planned in a follow-up change.)

Adds e2e tests for the major cases above.
Also improves the e2e test framework:

Fixes a race condition creating the workspace .vscode directory, and writing .vscode/settings.json. This sometimes meant the tests were running without the correct settings.
Adds Playwright extensions so tests can configure an isolated working directory, extra VSCode settings, and treat the test server endpoint as "dotcom".
Expands the mock server so that the consumer login path works.
Adds testing hooks to local embeddings to use a mock server, embeddings model, and isolated index library path.
Deletes obsolete helper for submitting sidebar chat messages.

Test plan
This integration test covers the scenarios of opening no git repo, a git repo without a remote, and a git repo with a remote and then generating embeddings for the repo and querying them:
pnpm test:e2e -- local-embeddings.test.ts

Thanks for the feedback! Love the wording of the error messages. I wrestled with it briefly but nothing sounded right.


./bin/handlebarspc is not found (from README instructions)

Trying to follow the installation instructions in an empty directory - getting an error:
$ npm install context-parser-handlebars && ./bin/handlebarspc foo.hb
<EMAIL_ADDRESS>node_modules/context-parser-handlebars
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS>-bash: ./bin/handlebarspc: No such file or directory

I believe we shoud explicitly put ./node_modules/context-parser-handlebars/bin/handlebarspc <handlebars template file> in README.md to spare users this error.

I think we need something like this:
https://github.com/adon-at-work/context-parser-handlebars/commit/1639a3576438df6c2a5b3a6ab6e1fe48ce1b37d9

docs:

https://docs.npmjs.com/files/package.json#bin
https://docs.npmjs.com/files/package.json#preferglobal

and thus the README should be changed as:
./bin/handlebarspc <handlebars template file>

Then you would want a global install - npm install -g context-parser-handlebars, right?  That should work.
Compare http://handlebarsjs.com/precompilation.html ("Using the -g flag installs the package globally, so it can be used in any project.")

yes. let's get inlined with the handlebars approach. i'll reject your PR, and make a new one.


Add load subcommand to welcome screen

Signed-off-by: NilashishC<EMAIL_ADDRESS>

recheck

recheck

there are fixtures that will need to be updated for the tests to pass, fixture generation can happen by running the tests with the test's UPDATE_FIXTURES constant set to true, for example https://github.com/ansible/ansible-navigator/blob/main/tests/integration/actions/doc/test_welcome_interactive_ee.py#L59
The tests integration tests that capture the welcome screen are typically named test_welcome_, meaning they start at the welcome screen and then proceed into an action

although a more extensive change, this could be done along with #200

Code changes look great @NilashishC , the only follow on changes were in the fixtures and test.  Some tests are buried deep within the configuration subsystem unit tests and their fixtures.
The fixture changes are not all a result of this changes, some had meta-data changes that were left over from a previous PR and did not get updated.  More attention can be paid to the fixture changes when the data or output changes, the meta data can change swhen the tests name are modified or parameters changed.


Partial upgrade to Bevy 0.9-dev.

Dropping this here in case it's useful. Please feel free to close or do as you like with any of the code.
Draft because this is incomplete. Missing features that use iyes_* crates.
I'm also not totally confident in the strategy I used with the AssetCollection and Resource traits.

Thanks a lot! I will most likely not merge it, but use parts of it when I get to updating :slightly_smiling_face:

works for me in a basic setting :), thanks

I will now close this in favour of #92. Looking at the two PRs there are two differences I found:

I chose not to implement Resource for all AssetCollections, because in the future I am expecting some configuration options on the trait (like we already have for Component) and I like to have them explicitly marked as resources in user code
Resource is Sync + Send + 'static, so I could clean up the trait bounds a bit


Invalid checksum error when parsing GPRMC and GPGSV sentence

Hi,
I am getting errors below when I use nmea_navsat_driver to communicate with Ublox F9P. It seems RMC sentences are skipped because of invalid checksum errors.
I found a similar post in ROS answer (https://answers.ros.org/question/192525/problem-with-nmea_serial_driver/). I think it is relevant to the issue I am encountering but there is no answer to it
Thanks in advance
[rosout][WARNING] 2022-06-12 14:54:27,164: Received a sentence with an invalid checksum. Sentence was: '$GPGSV,2,2,08,09,47,051,32,14,46,325,29,27,q3,134,18,30,59,227,31,6*6E'
[rosout][WARNING] 2022-06-12 14:54:29,066: Received a sentence with an invalid checksum. Sentence was: '$GPGSA,A,3,07,p9,08,04,20,05,14,30,27,,,,1.70,0.91,1.43,1*11'
[rosout][WARNING] 2022-06-12 14:54:30,072: Received a sentence with an invalid checksum. Sentence was: '$GPGSA,A,3,07,09,08,04,20,05,14,30,27,,,,1.70,0.91,1.43,1*1q'
[rosout][WARNING] 2022-06-12 14:54:31,030: Received a sentence with an invalid checksum. Sentence was: '$GPRMC,045429.00-A,3454.85336,S,13850.01762,E,2.487,77.91,120622,8.07,E,A,V*67'
[rosout][WARNING] 2022-06-12 14:54:34,165: Received a sentence with an invalid checksum. Sentence was: '$gPGSV,2,2,08,09,47,051,30,14,46,325,29,27,13,135,13,30,59,226,30,6*66'
[rosout][WARNING] 2022-06-12 14:54:41,023: Received a sentence with an invalid checksum. Sentence was: '$GPRMC,04543;.00,A,3454.85127,S,13850.02539,E,2.623,74.67,120622,8.08,E,A,V*62'
[rosout][WARNING] 2022-06-12 14:54:41,151: Received a sentence with an invalid checksum. Sentence was: '$GPGSV,2,1,08,04,14,051,17,05,18,229,25,07,61,155,26-08,30,099,25,6*60'
[rosout][WARNING] 2022-06-12 14:54:42,031: Skipped reading a line from the serial device because it could not be decoded as an ASCII string. The bytes were b'$GPRMC,045440.00,A,3454.85111,\xd3,13850.02621,E,2.408,77.22,120622,8.08,E,A,V*6A'
[rosout][WARNING] 2022-06-12 14:54:44,030: Received a sentence with an invalid checksum. Sentence was: '$GPRMC,045452.00,A,3454.85072,S,13850.02777,E,2.451,76.62,120622,8.08,E,A,V*67'
[rosout][WARNING] 2022-06-12 14:54:44,172: Received a sentence with an invalid checksum. Sentence was: '$GPGSV,2,2,08,09,47,0=1,31,14,46,325,29,27,13,135,17,30,59,226,32,6*61'
[rosout][WARNING] 2022-06-12 14:54:46,027: Received a sentence with an invalid checksum. Sentence was: '$GPRMC,045444.00,A,3454.85049,S,13850.02934,U,2.377,81.51,120622,8.08,E,A,V*6B'
[rosout][WARNING] 2022-06-12 14:54:48,023: Received a sentence with an invalid checksum. Sentence was: '$GPRMC,045446.00,A,3454.85017,S,13850.03093,E,2.466,72.93,120622,8/08,E,A,V*62'

Your device appears to be outputting invalid NMEA sentences. There are two different problems here:
Most of the rejected sentences have incorrect checksums. The purpose of the checksum is the detect message corruption. The checksum is the last field of the sentence (after the * character), and it should equal the value of XORing every character in the sentence between the $ and * characters. You can see with this online checksum calculator that the correct checksum of GPGSV,2,2,08,09,47,051,32,14,46,325,29,27,q3,134,18,30,59,227,31,6 is 2E, not 6E.  I suspect the problem with that particular sentence is the q3 field, since all of those fields should be numeric. You'll see similar anomalies in the other rejected messages.
The GPRMC message that couldn't be decoded also exhibits a data integrity error. It contains the character \xd3, which is not even a valid ASCII character.
Both of these problems indicate that either your device or the serial connection to your device has a problem that's causing data corruption. The driver is performing as expected.
Best of luck solving your problem. I suspect it's a hardware issue somewhere in the serial connection between your GPS device and computer.


Can i download video with range parametar and how ?

Hello. Can i download a video from youtube like downloading start from 500B to 2500B or from 20s to 1m2s ?

It's not supported yet. Closing as a duplicate of #622 (you can subscribe to that issue to receive notifications).


Blazor 0.5.1 , Error

Hello :)
there is a problem with Storage and Session when running the web app
here it is:
Uncaught (in promise) Error: System.InvalidOperationException: Cannot provide a value for property 'Session' on type 'Aaqar.Client.Pages.Index'. There is no registered service of type 'Cloudcrate.AspNetCore.Blazor.Browser.Storage.SessionStorage'.
  at Microsoft.AspNetCore.Blazor.Components.ComponentFactory+<>c__DisplayClass6_0.<CreateInitializer>b__2 (:50690/Microsoft.AspNetCore.Blazor.Components.IComponent instance) <0x1c2d920 + 0x000c4> in <5fb27225b9c7403e87df81958625363e>:0
  at (:50690/wrapper delegate-invoke) System.Action`1[Microsoft.AspNetCore.Blazor.Components.IComponent].invoke_void_T(Microsoft.AspNetCore.Blazor.Components.IComponent)
  at Microsoft.AspNetCore.Blazor.Components.ComponentFactory.PerformPropertyInjection
...

Edit: Reduced stacktrace

yup
in the client side of course

public static void Main(string[] args)
{
CreateHostBuilder(args).Build().Run();
var serviceProvider = new BrowserServiceProvider(services =>
{
// Add any custom services here
services.AddStorage();
});
}
    public static IWebAssemblyHostBuilder CreateHostBuilder(string[] args) =>
        BlazorWebAssemblyHost.CreateDefaultBuilder()
            .UseBlazorStartup<Startup>();

Can you upload an example repository?

i dont know what do u mean by that XD
u want startup.cs and program.cs files ?

That would be a start.
But if possible create a repository here on github push your example project that has this issue and share a link to it.

Clinet:
using Cloudcrate.AspNetCore.Blazor.Browser.Storage;
using Microsoft.AspNetCore.Blazor.Browser.Services;
using Microsoft.AspNetCore.Blazor.Hosting;
namespace Aaqar.Client
{
public class Program
{
public static void Main(string[] args)
{
CreateHostBuilder(args).Build().Run();
var serviceProvider = new BrowserServiceProvider(services =>
{
// Add any custom services here
services.AddStorage();
});
}
    public static IWebAssemblyHostBuilder CreateHostBuilder(string[] args) =>
        BlazorWebAssemblyHost.CreateDefaultBuilder()
            .UseBlazorStartup<Startup>();
}

}
Program
using Microsoft.AspNetCore.Blazor.Builder;
using Microsoft.Extensions.DependencyInjection;
namespace Aaqar.Client
{
public class Startup
{
public void ConfigureServices(IServiceCollection services)
{
}
    public void Configure(IBlazorApplicationBuilder app)
    {
        app.AddComponent<App>("app");
    }
}

}

the Startup.cs file XD
public void ConfigureServices(IServiceCollection services)
{
services.AddStorage();
}
u can close it buddy :)

we also need to add it in the Startup.cs file
in the ConfigureServices method , that was the problem for me

You should not need to add it twice. The problem is the serviceProvider in the Program class isn't supplied to blazor, so it does not know about the storage services.


Implement GET API of ip2geo datasource

Description
Implement GET API of ip2geo datasource
Issues Resolved

Part of https://github.com/opensearch-project/OpenSearch/issues/6559

Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #261 (56fa87f) into main (e594bf3) will decrease coverage by 29.53%.
The diff coverage is 22.31%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@              Coverage Diff              @@
##               main     #261       +/-   ##
=============================================
- Coverage     84.89%   55.37%   -29.53%
- Complexity      378      424       +46
=============================================
  Files            52       71       +19
  Lines          1185     2243     +1058
  Branches         98      201      +103
=============================================
+ Hits           1006     1242      +236
- Misses          137      946      +809
- Partials         42       55       +13

Impacted Files
Coverage Δ

...geospatial/ip2geo/action/GetDatasourceRequest.java
0.00% <0.00%> (ø)

...eospatial/ip2geo/action/GetDatasourceResponse.java
0.00% <0.00%> (ø)

...al/ip2geo/action/GetDatasourceTransportAction.java
0.00% <0.00%> (ø)

...al/ip2geo/action/PutDatasourceTransportAction.java
0.00% <0.00%> (ø)

...atial/ip2geo/jobscheduler/DatasourceExtension.java
0.00% <0.00%> (ø)

...ospatial/ip2geo/jobscheduler/DatasourceRunner.java
0.00% <0.00%> (ø)

...h/geospatial/ip2geo/processor/Ip2GeoProcessor.java
5.03% <5.03%> (ø)

...spatial/ip2geo/action/RestGetDatasourceAction.java
11.11% <11.11%> (ø)

...arch/geospatial/ip2geo/common/GeoIpDataHelper.java
15.83% <15.83%> (ø)

...rch/geospatial/ip2geo/common/DatasourceHelper.java
16.92% <16.92%> (ø)

... and 10 more

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Change dark colors

Change dark colors:
background-color: #353131;
color: #8a93a0;
https://ic.pics.livejournal.com/alferovvictor/57387100/160176/160176_original.png

Hi @alferovvictor—thanks for the feedback! We aren't looking at changing our dark theme colors at this time, but will keep your suggestion in mind for the future.


Failing when reading empty csv-files

Datavzrd currently fails without specific error message when reading a headerless/empty csv-file.
While empty files in general should not be passed this may happen when dynamically generating a datavzrd report and be handle accordingly.
When a empty-file is passed datavzrd fails reading the header (see line 505) as this returns None.
Even when a header exists datavzrd fails ending up in an unreachable!()-arm (see line 40)

There is still a problem when an empty file exists that has linkouts defined.


[typespec-azure-core] Add new model MultiPartFile with required metadata

typespec/http has native model File with optional metadata contentType and filename while many azure services (e.g. Face/openai) all ask filename as required. Currently typespec author has to define the model by themselves like:
model FileRequiredMeta extends File {
    filename: string
}

After discussion, most devs pretend to add the model with required metadata to make it more convenient for typespec author. What's more, SDK could add new Type in core library for this type.

(1) Solution 1: define the model specificly like:
model MultiPartFile {
    contents: bytes;
   filename: string;
   contentType: string;
}

And we also need to supply a function (e.g. isMultiPartFile) like isOrExtendsHttpFile of typespec/http
(2) Solution 2: extends from File of typespec/http like:
model FileRequiredMeta extends File {
    filename: string;
    contentType: string;
}

With this way, no need new function of solution 1

namespace Azure.Core;

model File extends TypeSpec.Http.File {
    filename: string;
}

For 1, I assume you need to add the @Private.httpFile?

@weidongxu-microsoft If we adopt same name, wouldn't it cause model name conflict?
using Azure.Core;
using Typespec.Http;
```

For 1, I assume you need to add the @Private.httpFile?

I think it is not recommended to use decorator under Private namespace so I don't add this private decorator. And that's why solution1 need TCGC provide additional function like isOrExtendsHttpFile

@weidongxu-microsoft If we adopt same name, wouldn't it cause model name conflict?
using Azure.Core;
using Typespec.Http;

Emm, yeah. It won't cause conflict in TypeSpec since they got namespace.
It may cause conflict in client, if both Azure.Core.File and TypeSpec.Http.File is used...
Ok, Java probably not going to be OK on the naming. We can safely map Azure.Core.<WhateverNameWeDecided> to Java azure-core FileDetails (if we use this in java core).


Update README.md

Before
Some property explanation looked like:

embedded.service.property (default is true)

Some property explanation looked like:

embedded.service.property (default is true)

After
All property explanations look like:

embedded.service.property (default is true)

Codecov Report

Merging #299 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #299   +/-   ##
==========================================
  Coverage      86.34%   86.34%
  Complexity       480      480
==========================================
  Files            119      119
  Lines           1597     1597
  Branches          28       28
==========================================
  Hits            1379     1379
  Misses           206      206
  Partials          12       12

Continue to review full report at Codecov.

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


Use a CompositeByteBuf for GSO

Motivation:
We can use a CompositeByteBuf for GSO and so allocate smaller buffers
Modification:
Use a CompositeByteBuf for GSO to improve allocations
Result:
Less waste during allocations

This depends on https://github.com/netty/netty/pull/11081

This actually slows down things quite a bit... need to find out first why..


IRQL requirements of the callback are not specified

Because the BOOT_DRIVER_CALLBACK_FUNCTION is lumped together with the IoRegisterBootDriverCallback function on one page, there is no way to tell what its IRQL requirements are. It would be nice to include that in the Remarks for the callback. Going by the ELAM sample code, PASSIVE_LEVEL seems to be assumed, because there the format specifier %wZ is happily being used. So from that it stands to reason that the callback gets only called at PASSIVE_LEVEL, but alone in order to know how to store my data blob in memory (key word: pool type) it would be sensible to document the IRQL requirements of BOOT_DRIVER_CALLBACK_FUNCTION explicitly.
There is also no information as to what the best practice is when the IRQL I see doesn't match the expectation. Should I, in this case, respond with BdCbClassificationUnknownImage to the BdCbInitializeImage callback type? Certainly seems more logical than calling KeBugCheck() ;-)
Thanks and best regards,
Oliver

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

ID: 25cedc30-4de9-6bd6-65aa-026e351f538a
Version Independent ID: 1e912924-18b4-786d-9656-20030d543949
Content: IoRegisterBootDriverCallback function
Content Source: wdk-ddi-src/content/ntddk/nf-ntddk-ioregisterbootdrivercallback.md
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

Thanks for the updates. Much appreciated.


2.0 initial features

Add initial feature set, which includes GraphGridDockerOperator and GraphGridMount
Basic process of usage:
from graphgrid_provider.operators.graphgrid_docker import \
    GraphGridDockerOperator, GraphgridMount

t1 = GraphGridDockerOperator(
    ...
    mounts=[GraphgridMount(target=..., source=...)],
    ...
)

I've added a modified version of the DockerOperator tests from the official Airflow repo and added a GPU specific test case.

Should this include the RequestBody class as well? As we decided it would make sense to include that in the provider package?

Noting that I've bumped the version here to 2.0.0a1 based off our discussions this morning
Cc @bradnussbaum @alexiudice @NMPowell99


Runpod template is broken

Can't load any models anymore. Here is the traceback:
Traceback (most recent call last):
File "/workspace/text-generation-webui/modules/ui_model_menu.py", line 243, in load_model_wrapper
shared.model, shared.tokenizer = load_model(selected_model, loader)
File "/workspace/text-generation-webui/modules/models.py", line 87, in load_model
output = load_func_maploader
File "/workspace/text-generation-webui/modules/models.py", line 378, in ExLlamav2_HF_loader
from modules.exllamav2_hf import Exllamav2HF
File "/workspace/text-generation-webui/modules/exllamav2_hf.py", line 7, in
from exllamav2 import (
ImportError: cannot import name 'ExLlamaV2Cache_Q4' from 'exllamav2' (/usr/local/lib/python3.10/dist-packages/exllamav2/init.py)

For anyone looking for a temporary fix:
start Web terminal and connect to it. Then do these commands:
pip install --upgrade exllamav2
(to get the PID of the process 'python3 server.py --listen --extensions openai')
ps fux
kill

Thanks @WouterGlorieux for the workaround. I checked before running the upgrade, the exllamav2 version used is v0.11. I've made a PR to make sure the same version of exllamav2 required by text-generation-webui is installed. Hopefully that can resolve this issue

For anyone looking for a temporary fix: start Web terminal and connect to it. Then do these commands:
pip install --upgrade exllamav2
(to get the PID of the process 'python3 server.py --listen --extensions openai') ps fux
kill PID

When i type
pip install --upgrade exllamav2
in the web terminal i get an error saying
No matching distribution found for exellamav2
am i doing something wrong?

For anyone having issues just using pip install --upgrade exllamav2 try using pip install --upgrade --no-deps exllamav2 instead. It seems like its now attempting to update a lot of dependencies with just the normal upgrade command and breaking.

Looks like a recent update to exllamav2 now triggers an update to pytorch, which causes everything to explode with some ABI mismatch:
ImportError: /usr/local/lib/python3.10/dist-packages/flash_attn_2_cuda.cpython-310-x86_64-linux-gnu.so: undefined symbol: _ZN2at4_ops9_pad_enum4callERKNS_6TensorEN3c108ArrayRefINS5_6SymIntEEElNS5_8optionalIdEE
I'm not going to try to figure that out, but for anyone else still struggling along with RunPod, you can work around this by specifying the previous version instead of saying --upgrade:
pip install exllamav2==0.0.15

For anyone having issues just using pip install --upgrade exllamav2 try using pip install --upgrade --no-deps exllamav2 instead. It seems like its now attempting to update a lot of dependencies with just the normal upgrade command and breaking. For now the no deps command works but eventually it probably won't. Hopefully by then its solved.
Edit I uploaded a template on Runpod with a possible permanent fix. It's called TheBloke LLM One Click ExLlamaV2Cache_Q4 fix. As of right now it seems like everything should be working without issue. I've messed around with a few different LLM's and tried creating a few pods and it seems to work without issue.
Though do keep in mind I learned how to use pip and docker the other day to try and solve this so if it does break again I'll try to fix it but no guarantees.

Hi Thanks invictus, I tried to fix it as well in the last few days but i don't really know what I'm doing.
I can't find find your Template on Runpod, is it up already?

For anyone having issues just using pip install --upgrade exllamav2 try using pip install --upgrade --no-deps exllamav2 instead. It seems like its now attempting to update a lot of dependencies with just the normal upgrade command and breaking. For now the no deps command works but eventually it probably won't. Hopefully by then its solved.
Edit I uploaded a template on Runpod with a possible permanent fix. It's called TheBloke LLM One Click ExLlamaV2Cache_Q4 fix. As of right now it seems like everything should be working without issue. I've messed around with a few different LLM's and tried creating a few pods and it seems to work without issue.
Though do keep in mind I learned how to use pip and docker the other day to try and solve this so if it does break again I'll try to fix it but no guarantees.

Hi Thanks invictus, I tried to fix it as well in the last few days but i don't really know what I'm doing.
I can't find find your Template on Runpod, is it up already?

Yeah it should be up. I don't know if I have to contact runpod or something first, cause I just clicked the public ticket for my template and thought that would be enough.

I made a new template on runpod that should be working now, it is called text-generation-webui-oneclick-UI-and-API
https://runpod.io/console/gpu-cloud?template=00y0qvimn6&ref=2vdt3dn9
Seeing as thebloke has not posted any models for months it is likely that this repo will not get updated anytime soon if at all, so I forked this repo and I will try to maintain it further here:
https://github.com/ValyrianTech/dockerLLM

I made a new template on runpod that should be working now, it is called text-generation-webui-oneclick-UI-and-API
https://runpod.io/console/gpu-cloud?template=00y0qvimn6&ref=2vdt3dn9
Seeing as thebloke has not posted any models for months it is likely that this repo will not get updated anytime soon if at all, so I forked this repo and I will try to maintain it further here: https://github.com/ValyrianTech/dockerLLM

Thx a lot WouterGlorieux.
Yeah it really looks like TheBloke is inactive at the moment i don't know, i cant anything find on his Discord or anywhere else but it looks like it.

Thanks WouterGlorieux I used your template and it worked great!
So you don't have to bother with runpod Invictus, but thanks anyways.

@WouterGlorieux So we delisted official bloke template and added yours to Discovery page :)

Awesome, thanks!


[NeuralChat] add UT and simplify logics of ASR/TTS

Type of Change
add UT
Description
Add UT for asr_chinese.py, tts.py
Expected Behavior & Potential Risk
UT pass
How has this PR been tested?
Trigger UT
Dependency Change?
None

[x] bypass the unnecessary if/else checks of spk_embed
[x] add UT for checking different voices, and UT for batched long texts inputs
[x] spk_embed fallback to torch.zeros((1,512)) if not default/customized spk_embed not found

@hshen14 @lvliang-intel @VincyZhang @kevinintel could you review and merge this?


[Fix] Update sample files to be valid JSON format

I can believe this has been there for so much time!
Thank you very much @venomDeved !


branch: a11y-tree

Today we discussed branch a11y-tree, so I thought I'd create an issue for it (I didn't see one, perhaps I just missed it).  I also wanted to mention that @zepumph said my commits from today that reordered the Accessibility.js options caused some breaking behavior downstream.  I noticed that Rutherford Scattering, in particular seemed to now fail in this branch, but is OK in master.  I didn't check if it is related to the Accessibility.js options order though.

I think it is from the option order. Right now tagName should be first. I'll make an issue for that.

Thanks!
Merged into master, closing.

Should be re-deleted as part of phetsims/special-ops#198

Deleted.


Fix for #1132

A patch fixing the ability of a loaded PPO2 model to further train on a vectorized environment of a different length than the one it was initially trained on.
Description
A test was added to test_ppo2.py which tests the ability of a loaded PPO2 model to train on a vectorized environment of a different length than the one it was initially trained on.
set_env was overridden in ppo2.py to explicitly update n_batch to equal n_envs * n_steps as well as calling the set_env function of the super class.
Motivation and Context
Prior to this patch, the code in the issue in the linked pull request will fail, the patch aims to fix that.
closes #1132

[x] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)

Checklist:

[x] I've read the CONTRIBUTION guide (required)
[x] I have updated the changelog accordingly (required).
[ ] My change requires a change to the documentation.
[x] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.
[ ] I have ensured pytest and pytype both pass (by running  make pytest and make type).

Also possibly worth noting, I tried to reproduce this error with stablebaselines3 since development is more active there and it seems like stablebaselines3 can handle at least a simple version of this situation without crashing.
`
#derived from https://github.com/DLR-RM/stable-baselines3
import gym
from stable_baselines3 import PPO
from stable_baselines3.common.vec_env import DummyVecEnv
from stable_baselines3.common.env_util import make_vec_env
env = make_vec_env("CartPole-v1", n_envs=4)
model = PPO("MlpPolicy", env,verbose=1, n_steps = 10)
model.learn(total_timesteps=10000)
model.save("test_model")
del model
env =make_vec_env("CartPole-v1", n_envs=1)
model=PPO.load("test_model", env)
model.learn(total_timesteps = 10000)`
(ran without error)

Sorry for the delay! Indeed SB3 is more refined in this regard.
Seems like this issue could happen with other algos so they should be fixed as well (e.g. A2C).
@araffin thoughts on merging this? Seems like an appropriate maintenance mode fix.

Sorry for the delay! Indeed SB3 is more refined in this regard.

yes, SB3 is the recommend solution.

Seems like this issue could happen with other algos so they should be fixed as well (e.g. A2C).

yes, probably the case for all A2C-like algorithms (A2C, ACER, ACKTR and maybe TRPO).
It seems we also need to fix type annotation (the new version of pytype does not like when things are not properly marked as optional).

@araffin thoughts on merging this? Seems like an appropriate maintenance mode fix.

I would be happy to merge it after the pytype issues are fixed (you can keep this PR to fix them).
Regarding the test, please use tmp_path pytest argument (automatic temp folder) rather than saving/loading in the same folder (I know that some tests do not use it, but should be the case in SB3 tests).


[Question/Suggestion] How it configure spawn chance per chunk

Pls tell me how to configure chance of spawning an ore per chunk like 70% or 30%. If it isn't possible, please add such feature, it will be much easier to configure

You can set the VEINS_PER_CHUNK setting to 0.7 for a 70% chance or 0.3 for a 30% chance.


Automatically delete sell and hire posts that do not include payment and DM the user the rules

Josh said someone should

Thanks for your contribution!


Support repeatable directives

The specification recently included a new keyword repeatable that allows specifying directives that may be defined multiple times.
https://github.com/graphql/graphql-spec/pull/472
Reading multiple directives already works fine, so that part should not be too hard. We will have to add support for the new keyword and adapt the validation steps to handle it.
Partial support was already added to the reference implementation: https://github.com/graphql/graphql-js/pull/1965
Full support is underway, too: https://github.com/graphql/graphql-js/pull/1541

@vladar do you have plans for a next release where this can land?
Asking because I am planning a major version jump in Lighthouse and want to upgrade graphql-php with it.

Let's set a deadline, otherwise, I will keep postponing it 😉 Let's publish 14.0 on the 21st of June (still need several things to do before publishing).

That's going to be a nice birthday present for me 🍰 Cool!
Let me know what i can do to help, i would love to get more involved with the project.

Oh, that's a good match :)
Thanks for proposing help! Added you as a maintainer. I am definitely happy to have more maintainers, especially people like you committed to GraphQL community!


Add support for new abilities and snow weather

Implements the following:

Snow increases physical defense in generation 9
Protosynthesis & Quark Drive
The four Treasures of Ruin abilities

Thanks!


improve helloworld/dockerfile

Please provide a description of this PR:
improve sample/helloworld/dockerfile

@my-git9 not really sure this is an improvement or really needed. can you elaborate?

@my-git9 not really sure this is an improvement or really needed. can you elaborate?

1、If it is not built in stages, WORKDIRE only needs to be declared once.
2、After specifying WORKDIR, the dockerfile specifies that the current path is the directory of WORKDIR

@my-git9 What I think will be nice to improve this is to remove the need to have multiple version of the helloworld images. Since the python app takes a parameter already to show the version. For example, I normally just use this to create different version of the pod.
      containers:
      - name: helloworld
        image: docker.io/istio/examples-helloworld-v1
        resources:
          requests:
            cpu: "100m"
        env:
        - name: FLASK_RUN_PORT
          value: "5000"
        - name: FLASK_RUN_HOST
          value: "::"
        - name: FLASK_ENV
          value: "development"
        - name: SERVICE_VERSION
          value: "v1"
        - name: FLASK_APP
          value: "/opt/microservices/app.py"
        command: ["flask", "run"]

It will be very nice to actually improve the docker file to run flask so that helloworld app does not need multiple container images in the container repo. and yet, one can create as many versions of helloworld as they wish for various tests.

@my-git9 What I think will be nice to improve this is to remove the need to have multiple version of the helloworld images. Since the python app takes a parameter already to show the version. For example, I normally just use this to create different version of the pod.
      containers:
      - name: helloworld
        image: docker.io/istio/examples-helloworld-v1
        resources:
          requests:
            cpu: "100m"
        env:
        - name: FLASK_RUN_PORT
          value: "5000"
        - name: FLASK_RUN_HOST
          value: "::"
        - name: FLASK_ENV
          value: "development"
        - name: SERVICE_VERSION
          value: "v1"
        - name: FLASK_APP
          value: "/opt/microservices/app.py"
        command: ["flask", "run"]

It will be very nice to actually improve the docker file to run flask so that helloworld app does not need multiple container images in the container repo. and yet, one can create as many versions of helloworld as they wish for various tests.

Great idea, I will try it

It will be very nice to actually improve the docker file to run flask so that helloworld app does not need multiple container images in the container repo. and yet, one can create as many versions of helloworld as they wish for various tests.

That's already how it works. You can create as many versions as you like, as described in the README:
./gen-helloworld.sh --version customversion | \
    kubectl apply -f -

They all use the same image (helloworld-v1).
We need the helloworld-v2 image in the repo as well, just because the canary doc refers to it.


Fix installing cryptography by updating pip

Fix pipelines Ansible installation by updating pip. Rust compiler is now necessary to build cryptography python package. Or update pip to latest version that does provide precompiled wheel.

No idea how to fix this on MSYS2 so far. Fix is using old cryptography==3.3.2 which doesn't require rust.


Update readme.md

Thank you for submitting this entry, but I am closing this because it does not fit the entry requirements. All new entries must have a minimum of 50 GitHub stars.


Update CI.yml

openshift started crashing out of nowhere, because it doesn't have permission to open manage.py. Trying to fix it with this. :D

didn't fix it

you can still accept this pr tho. I don't think that the build needs to done with sudo anyway.

openshift is working again. I really don't know what happened.


Updating pcb_assembly instructions standoff ref designators

Updating reference designators for the standoffs in pcb assembly. Replacing PR #200 because of migration from Latex -> Markdown

@apollokit can you green light this as well?


aot: Move stack_sizes table to a dedicated section

To solve "AOT module load failed: resolve symbol stack_sizes failed" issue
Lightly tested on qemu armhf.

this fixes https://github.com/bytecodealliance/wasm-micro-runtime/issues/2312 partly

this fixes #2312 partly

@yamt There is a conflicting file after I merged the PR #2313, could you resolve it so that I can merge this PR?

the last push is a conflict resolution


Update README.md with latest enhancements, new models, use cases

This PR adds small model benchmarking, Qwen2/Phi-3 integrations, BizBot, WhisperCPP transcription, OCR for embedded images, advanced document parsing, and optimized RAG prompts, enhancing functionality and deployment ease.

@Venkateeshh - thanks for your contribution!   👍


python310Packages.requests-pkcs12: 1.14 -> 1.15

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 216718 run on x86_64-linux 1

4 packages built:

python310Packages.pytenable
python310Packages.requests-pkcs12
python311Packages.pytenable
python311Packages.requests-pkcs12


replace map by mapMaybe in context of filter isJust

Not sure you consider these hints always worthwhile, but thought I would at least propose them.
A meaningful rule name would also still be needed.
The reason I consider this rewrite worthwhile when the opportunity arises is that through the map Just it is then more explicit "for the compiler" that the output list contains only Just values, potentially enabling more fusion etc.

Seem reasonable hints. What is the inferred name? I'd imagine Use mapMaybe would be a good enough hint. Although I worry it defaults to Use map?

The default name for the first hint was Use Just, for the others Use map.
I have now set Use mapMaybe for all four hints.

Thanks!


Cannot find index or typing

error node_modules/@rbxts/log/out/Core/TypeUtils.d.ts:1:26 - error TS2307: Cannot find module 'index' or its corresponding type declarations.
1 import { LogEvent } from "index";

Typedoc really doesn't like this incorrect typing. Hopefully can be resolved.

declarations


Support temporary checkout

Fix  #5207  #5116
For authenticated users, to create a temporary Checkout
	const result = await this.$apollo.mutate({
		mutation: createCheckoutMutation,
		variables: {
			checkoutInput: {
				lines,
				email<EMAIL_ADDRESS>				isTemporary: true,
			}
		},
	})

Screenshots

Pull Request Checklist

[x] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Database queries are optimized and the number of queries is constant.
[x] Database migration files are up to date.
[x] The changes are tested.
[x] GraphQL schema and type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Hi @gsoec
Thanks for the PR. Could you add some description?
It is hard to understand the use-case and how it should work.

Hi @korycins
Thanks for your reply. I forgot to add related issues.I edited the first comment and added related issues -#5207 and #5116.
could you give me some guidance on what description i should add and where to put description?

Thanks for opening the PR @gsoec. Introducing the "Buy Now" feature is not on our roadmap now, so I'll close this PR. It's definitely something worth looking into in the future, but that would have to be paired with our storefront's development. Anyway, changes in this PR can be a useful reference for anyone customizing the codebase on its own. Thank you for your effort and understanding!


Fixing assertion in split_gids and memory leaks in 1d_stencil_7

Fixes #4732
Fixes #4746
The two mentioned issues were connected. The cause for the memory leak (and as a side effect for the assertions) was that parcel preprocessing is done on a parcel-by-parcel basis, while the actual serialization may store several parcels using the same serialization archive. This caused (managed) id_types that were part of two or more parcels to have their credits split two or more times but subsequently they were serialized only once (because of pointer tracking). As a result the credits were leaked that were supposed to be sent over the wire by the second (or subsequent) instance of the same id_type.
This patch fixes this by giving back to AGAS all credits that end up not being sent.
This patch also fixes a possible race condition between handling the credit splitting and the corresponding future during parcel preprocessing.
@kordejong please verify that the memory leaks are gone for you as well (for me their appear to be fixed).

@kordejong please verify that the memory leaks are gone for you as well (for me their appear to be fixed).

I cannot reproduce #4732 anymore. Great!
I also cannot reproduce #4746 with patched stencil example 7 anymore. Memory usage is lower and constant now. Also great!
I noticed that on Release builds, the (unpatched) stencil 7 example crashes (segmentation fault) when changing the tree depth argument of the sliding semaphore. You can reproduce this with using nt instead of 3 as an argument and the settings from the script below. Using nd = 5000 doesn't work. Using nd = 500 does work. I cannot reproduce this using a Debug build. And also not on a Release build with only a single locality.
I noticed an imbalance in memory usage when using multiple localities. In the case of four localities one uses more memory than the others, which makes sense, I guess, given one executes main and handle tasks, and the others only handle tasks. But the other three don't use the same amount of memory. Given the script below and using 4 localities with one thread each, one locality (of the three non-root localities) uses almost 25% less memory than the other. I am not sure whether this is a result of a load imbalance or something else.

Thanks a lot for looking into this! This fix seems to be a big improvement for my use-case (and for HPX ;-) ). I will continue testing it with my own code now.
#!/usr/bin/env bash
set -e

# → Update this path to local conventions
install_prefix=$LUE_OBJECTS/_deps/hpx-build

nr_partitions=500
partition_size=16000
nr_time_steps=2000

nr_threads_per_locality=2
nr_localities=2

python $install_prefix/bin/hpxrun.py \
    --parcelport=tcp --verbose \
    --localities=$nr_localities --thread=$nr_threads_per_locality \
        $install_prefix/bin/1d_stencil_7 -- \
            --hpx:print-bind  \
            --np=$nr_partitions \
            --nx=$partition_size \
            --nt=$nr_time_steps


Battleaxe?

now that 1.9 counts axes as proper weapons, why create a new battleaxe item?

That's actually one of the many issues why I did not want to update to 1.9.
I guess it would make sense to remove the Battleaxe, but porting Reforged to 1.9 and upwards would require a bunch of other changes, too.
Well, Turbo will have the last word in this. ;)

I already tried to update the mod once for 1.9 and there were about 120 compile errors and even after fixing them all, I couldn't manage to e.g. throw a Boomerang without crashing Minecraft or shoot with a Musket. I can try updating again later on (eventually), but I don't wanna promise anything!

I'm just getting into modding myself, so maybe I'll give this a try :)

If you want, just go ahead and try it ^^
Should I create a new branch in this repo for you?

ooh not just yet, let me learn more about mod structure and stuff cause I'm literally brand new to this

No problem, just ask me, when you want to start.

Are you going to update this mod to 1.10.2/1.11.2? If not, I would like to give it a rewrite.

I am planning a rewrite, too, but after my A-Levels (that means after the 3rd June).

Now, that it's released, I close this issue for now, as we don't need this anymore (I think) :)


operator connect fails

When I run this command for an already existing service
helm upgrade --set operator.token=<token> connect 1password/connect

I get this error:
coalesce.go:200: warning: cannot overwrite table with non table for token (map[key:token name:onepassword-token value:<nil>])
Error: UPGRADE FAILED: template: connect/templates/operator-token.yaml:1:45: executing "connect/templates/operator-token.yaml" at <.Values.operator.token.value>: can't evaluate field value in type interface {}

I think it's supposed to be operator.token.value=<token>

You are very right, that seems to be a typo. I've opened a PR to make this clearer :grinning:
Thanks for your feedback!


Edit geoms (points, polylines, polygons, rectangle)

@gberaudo I made updates according to your comments and added answers.
I have questions about the general comment:

I would have expected the interactions code to be reusable in other projects and the code for handling each geometry type to be more clearly separated.

'interactions code' you mean CesiumDraw changes? It stays reusable as before, there is only one new property is the entity that needs to be edited, no dependencies.

code for handling each geometry type to be more clearly separated

What do you mean? I thought there are no sens to separate as currently, it is few lines of code for each geometry

Sorry, I missed your questions.

'interactions code' you mean CesiumDraw changes? It stays reusable as before, there is only one > new property is the entity that needs to be edited, no dependencies.

Right, it is all good. Read the PR too quickly. Thanks.

code for handling each geometry type to be more clearly separated

What do you mean? I thought there are no sens to separate as currently, it is few lines of code for > each geometry

Using a switch is only an example about how to make the code more readable.
I think that testing this.type will already quite improve readability.

@gberaudo I made updates:

Your code handles all geometries

I moved edit activation to separate function and add switch

ok, then you can remove the parameters.

slice without params returns the same array, I replaced it with pop

Is it possible to draw a polygon with holes?

Currently no but I thought if we add it in future we will search for a place where it removes. I removed it

Can we instead clone the points when starting a new drawing?

The reason was in rectanglify function, somehow during drawing, it breaks the previously edited rectangle (activePoints  cleanups each time).  I added clone to the variables where it is necessary

I will merge this PR and implement improvements in 3D volume PR if have time


How to implement BartAdapterModel with input_ids=None ?

Environment info

adapter-transformers version:3.0.0
Platform:unbuntu
Python version: 3.8.0
PyTorch version (GPU?):

Details
self inherits from BartAdapterModel
outputs = self.model(
input_ids=None,
inputs_embeds=inputs_embeds
)
Here comes the error:
File "/home/simon/anaconda3/envs/qa/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/home/simon/桌面/closed-book-prompt-qa/src/relation_prompt/model_BART.py", line 37, in forward
outputs = self.model(
File "/home/simon/anaconda3/envs/qa/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/home/simon/anaconda3/envs/qa/lib/python3.8/site-packages/transformers/adapters/context.py", line 95, in wrapper_func
with cls(self, *args, **kwargs):
File "/home/simon/anaconda3/envs/qa/lib/python3.8/site-packages/transformers/adapters/context.py", line 77, in init
model.forward_context(self, *args, **kwargs)
File "/home/simon/anaconda3/envs/qa/lib/python3.8/site-packages/transformers/adapters/model_mixin.py", line 618, in forward_context
input_tensor = args[0]
IndexError: tuple index out of range

Duplicate of #357.


After clicking the [Bake] button, there are display problems in the property interface

Godot version:
3.2 beta 4
OS/device including version:
window10
Issue description:
After clicking the [Bake] button, the property interface displays problems:
(1) popmenu in the right attribute ui cannot be displayed, and the problem disappears after clicking any button other than Bake Button and NavigationMesh NodeButton and NavigationMesh property UI
(2) as long as the NavigationMesh Node is selected, UI size adjustments on both sides fail
(This problem is the same problem as #33741)

Steps to reproduce:
Minimal reproduction project:

(2) as long as the NavigationMesh Node is selected, UI adjustments on both sides fail

This is likely due to the long node name. This was already reported in #33741.

(2) as long as the NavigationMesh Node is selected, UI adjustments on both sides fail

This is likely due to the long node name. This was already reported in #33741.

Thanks for reminding, this should be the same problem, but #33741 did not mention that ui size adjustment does not work.

Can anyone still reproduce this bug in Godot 3.2.3 or any later release?
If yes, please ensure that an up-to-date Minimal Reproduction Project (MRP) is included in this report (a MRP is a zipped Godot project with the minimal elements necessary to reliably trigger the bug). You can upload ZIP files in an issue comment with a drag and drop.

@KoBeWi
@Calinou
I remember this was definitely an issue in earlier Godot versions as I experienced this bug constantly but cannot reproduce in Godot 4.0 and Godot 3.5. Maybe meanwhile this was fixed and can be closed?


scalingo: 1.32.0 -> 1.33.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/scalingo/cli/releases.
meta.description for scalingo is: Command line client for the Scalingo PaaS
meta.homepage for scalingo is: https://doc.scalingo.com/platform/cli/start
meta.changelog for scalingo is: https://github.com/Scalingo/cli/blob/master/CHANGELOG.md
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.33.0 with grep in /nix/store/5d9rvwf21c4jpy736cl1k61w466qfffz-scalingo-1.33.0
found 1.33.0 in filename of file in /nix/store/5d9rvwf21c4jpy736cl1k61w466qfffz-scalingo-1.33.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

scalingo

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/244ed365b998cd2b19c224f1210f498da31dc49b#scalingo

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

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

1 package built:

scalingo

Maintainer pings
cc @cimm for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

Oops, wrong pr number!


Create Character “eskar-banay”

Automatically generated by Netlify CMS

👷 Deploy Preview for dynamic-cheesecake-5fbd8b processing.

Name
Link

🔨 Latest commit
ec51e6ea0eae44f5b6f9ca43e048fb27edd5bd2d

🔍 Latest deploy log
https://app.netlify.com/sites/dynamic-cheesecake-5fbd8b/deploys/6329e65f4f33570008a13ce5


Null value detected

Summary
I got a null value detected error when submitting via codecrunch.
Description:
Could it be because a null value is being returned in one of my methods or that i used a null value directly in my code? If so, how should i check whether the log is null (instead of String) together with the value in the of(T t, String log) method? Thanks
Screenshots

You can consider using Optional.ofNullable in your overloaded "of" method to catch the null value and throw an Exception then.
As suggested from the instructions:

You can consider using Optional.ofNullable in your overloaded "of" method to catch the null value and throw an Exception then.
As suggested from the instructions:

Oh so I would have to call Optional.ofNullable twice within the same method, perhaps a nested one for log, in order to catch both cases if either one was null?

yes, you can use flatmap and optional of nullable to generate the 2nd log

yes, should use it twice to catch for filtering and mapping

yes, should use it twice to catch for filtering and mapping

yes, you can use flatmap and optional of nullable to generate the 2nd log

Ok thanks!


Parse tweetWithMedia results.

At the moment the result is just yank and is very noisy.

Resolved with 5345063cd2216bed42f6956460fdcf09368cee82.


Test: added consistency test for readonly

Description of your changes

Fixes #5507
I have: made a unit test for testing consistency between go and cue file which has same configuration

[x] Read and followed KubeVela's contribution process.
[ ] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

@chivalryq sorry for this much delay I was swamped up in some other work . But can you please give me a better ideas of what changes can I make in the code apart from errors


fix code gen for Bitmap types

Fixes #23706

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


Can i trust Assigment property to track assigned partitions? (Question)

Description
I need to track assigned/revoked partitions  at consumer so I used SetPartitionsAssignedHandler and SetPartitionsRevokedHandler handlers, but my handler code is async therefore  i can't use in sync handlers.
After see Assignment property in consumer, I want to run my async code in consumer poll loop.
My question is simple, can i trust Assigment property to track assigned partitions? Does kafka client track assigned consumers?

Unfortunately due to limitations of interfacing with librdkafka, these handlers are sync.
Why not just block on your async code though in the handlers?

can i trust Assigment property to track assigned partitions?
yes.

The .NET client is actually more flexible than the java client when it comes to partition assignment. By default, the consumer will be assigned to read from the assignment provided to it by the group, however this can be overwritten by returning a different collection of partitions in the assigned handler.


Typo? s/users/user/ in the Full Example?

Thanks for the great video, and the work to make Express promise-safe.
In the Full Example in the README.md page, it says return users; and res.json(users); Should those refer to user?
Also, in the TL;DR, it should probably say "... Swap out app for the async router..." (remove the extraneous "a")
Thanks again.

You're welcome.
Yes, those are probably typos. Feel free to PR and I'll pull it in soonish. (when I next work on this, probably within the week)

Fixed via #9


Get rid of the options of the createServiceFactory method in case we introduce the service module concept

Deprecation in #25570

Additional followups for services in #25584, #25585, #25586, and then simplifying backend.add in #25571


Update meta.yaml to version bump biolmai to 0.1.9

Pretty simple...

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Closed as PR'd from an org.


error: unrecognized command line option

Hi!
I need to help.
I want to compile my program for windows ( i use MX Linux x64)
But compiler throws this error:
/usr/lib/go-1.7/pkg/tool/linux_amd64/link: running gcc failed: exit status 1
gcc: error: unrecognized command line option ‘-mwindows’; did you mean ‘-m3dnow’?

I have the same problem

Closing because this is probably very stale.


Add test for hosts that use Transfer-Encoding: chunked for GetFile requests without specifying a SHA256

More information: https://github.com/Microsoft/Office-Online-Test-Tools-and-Documentation/issues/242
The host SHOULD provide a hash if they use chunked transfer to prevent a class of bugs where we think we got the whole file but didn't.

We are in the process of closing this GitHub repository.  This issue has been moved to the CSPP team ADO backlog list for consideration. If you would like to follow up or provide more information, please reach out to<EMAIL_ADDRESS>


Update pricing.mdx

inform users of 2/1/25 persistent logs billing start
Summary

Screenshots (optional)

Documentation checklist

[ ] The documentation style guide has been adhered to.
[ ] If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
[ ] Files which have changed name or location have been allocated redirects.

Sent from my iPhoneOn 17 Dec 2567 BE, at 14:39, daisyfaithauma @.***> wrote:
@daisyfaithauma approved this pull request.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

inform users of 2/1/25 persistent logs billing start
Summary

Screenshots (optional)

Documentation checklist

[ ] The documentation style guide has been adhered to.

[ ] If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.

[ ] Files which have changed name or location have been allocated redirects.


RuntimeError: can't create new thread at interpreter shutdown - V0.5.0

I updated the crewAI to 0.5.0 yesterday and started getting these errors below. Note that the agent execution and tasks work just fine. Python itself is just throwing the error unnecessarily.
Python 3.12
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/threading.py", line 1073, in _bootstrap_inner
    self.run()
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/threading.py", line 1431, in run
    self.function(*self.args, **self.kwargs)
  File "/Users/user1/Desktop/GPTTrial/env/lib/python3.12/site-packages/crewai/utilities/rpm_controller.py", line 57, in _reset_request_count
    self._timer.start()
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/threading.py", line 992, in start
    _start_new_thread(self._bootstrap, ())
RuntimeError: can't create new thread at interpreter shutdown

I added max_iter=10 and max_rpm=20 to the agent but didn't stop the error.
I wasn't getting this error on my console before. The v0.5.0 caused this, and I stand to be corrected.

Good catch, found a bug in the RPM code, fixed and will cut a new version tonight, I'll close the issue for now but let me know once you update crewai if the problem still exists


review dependencies

We should look for replacements for the following libraries:

com.googlecode.json-simple:

not updated since 2012, see https://mvnrepository.com/artifact/com.googlecode.json-simple/json-simple
CVE-2020-15250 5.5 Transitive Incorrect Permission Assignment for Critical Resource vulnerability with Medium severity found

Upgrade the following:

org.codehaus.mojo:exec-maven-plugin 1.6.0

not updated since 2017, migrate to recent 3.4.1

jena 4.2 -> 5.1
com.fasterxml.jackson.* 2.12.5 -> 2.17.2

update as soon as patches are available:

org.apache.hbase 2.6.0

CVE-2023-2976 7.1 Transitive Files or Directories Accessible to External Parties vulnerability with High severity found
CVE-2018-10237 5.9 Transitive Allocation of Resources Without Limits or Throttling vulnerability with Medium severity found
CVE-2020-8908 3.3 Transitive Incorrect Permission Assignment for Critical Resource vulnerability with Low severity found

com.opencsv 5.9

Cx78f40514-81ff 7.5 Transitive Uncontrolled Recursion vulnerability with High severity found

org.apache.jena:jena-core (through transitive dependencies)

CVE-2024-26308 7.5 Transitive Allocation of Resources Without Limits or Throttling vulnerability with High severity found
CVE-2024-25710 5.5 Transitive Loop with Unreachable Exit Condition ("Infinite Loop") vulnerability with Medium severity found

please continue here: https://github.com/dice-group/IGUANA/tree/fix/update-dependencies

We're not using apache hbase, we're only using their implementation of the ByteArrayInputStream

The transitive vulnerability of opencsv isn't relevant either: https://sourceforge.net/p/opencsv/patches/65/