Supporting camelcase

Thanks for this library!
Would it be possible to support camelcase, like the kysely-codegen library?
The final DB type that gets generated still has all the keys in snake case, so it would be awesome to be able to configure that as camel case as well.

Let me make sure I understand you correctly. You mean that your Prisma schema generates a database that uses snake case names, but you want to generate types for Kysely that use camel case and depend on the camel case plugin?
If that's the case it should be pretty straightforward to implement.
I'd be interested to hear what your use case is for this though. Is there something holding you back from renaming the fields in your schema so they use camel case?

Hey I've made a first attempt at addressing this. Could you try using snapshot version 0.0.0-snapshot-20230323111034 and setting camelCase = true in the generator config:
generator kysely {
    provider  = "prisma-kysely"
    // ...
    camelCase = true
}

And of course use the camel case plugin when you instantiate the Kysely instance.
Let me know if this works for you 😄

Let me make sure I understand you correctly. You mean that your Prisma schema generates a database that uses snake case names, but you want to generate types for Kysely that use camel case and depend on the camel case plugin?
If that's the case it should be pretty straightforward to implement.
I'd be interested to hear what your use case is for this though. Is there something holding you back from renaming the fields in your schema so they use camel case?

Not quite, my Prisma schema does use camel case (underlying DB is using snake case but I map all tables and fields to camel case), and most of the types are also in camel case in the output. The issue is that the final DB type export that the library is generating has all the table names in snake case instead of camel case.

Ah gotcha. Try using the snapshot. If I understand you correctly it should solve this issue.

snapshot works! It outputs in camelcase now.
export interface DB {
  taxAuthority: TaxAuthority;
}

this seems like https://github.com/valtyr/prisma-kysely/issues/4, no?


Checkbox doesn't work

Greetings,
First: Thanks for the imgui wrapper :+1:
Second: I'm working through your documentary right now to learn pyimgui, and I have a problem with the checkbox. Your example doesn't work for me:
imgui.begin("Example: checkboxes")

c1_output = imgui.checkbox("Checkbox 1", True)
c2_output = imgui.checkbox("Checkbox 2", False)

imgui.text("Checkbox 1 return value: {}".format(c1_output))
imgui.text("Checkbox 2 return value: {}".format(c2_output))

imgui.end()

The checkbox is displayed but I cannot change the state of the checkbox if I execute the script. I've tried a few other ways too, but without success.
I use the pygame backend and the checkbox in your example script with all widgets and functions works.
Here you can find my full source code: https://gitlab.com/LuminousLizard/pyimgui-user-friendly/blob/master/Pyimgui_elements/13_checkbox.py

Hi!
The documentation of some widgets may be sometimes a bit misleading and we'll try to improve that.
The second state argument in imgui.checkbox() widget is your desired state of the checkbox. So if you "statically" pass True literal then your checkbox will be always rendered as checked. You need to store output state of the widget in some secondary variable and modify it on every frame when checkbox is rendered. Here is better example fo how it should be documented:
# note: these should be initialized outside of the main interaction
#       loop
checkbox1_enabled = True
checkbox2_enabled = False

imgui.new_frame()
imgui.begin("Example: checkboxes")

# note: first element of return two-tuple notifies if there was a click
#       event in currently processed frame and second element is actual
#       checkbox state.
_, checkbox1_enabled = imgui.checkbox("Checkbox 1", checkbox1_enabled)
_, checkbox2_enabled = imgui.checkbox("Checkbox 2", checkbox2_enabled)

imgui.text("Checkbox 1 state value: {}".format(checkbox1_enabled))
imgui.text("Checkbox 2 state value: {}".format(checkbox2_enabled))

imgui.end()

Note that it is extremely important to not manually redefine values checkbox1_enabled and checkbox2_enabled on every step rendering. In official documentation it is sometimes presented that way because we process documentation automatically to generate example images.
For instance, in code using glfw3 integration this code could look like following:
# this is some function that initializes glfw window context
window = impl_glfw_init()
impl = GlfwRenderer(window)

# these are widget states initialized before main "while" interaction loop
checkbox1_enabled = True
checkbox2_enabled = False

while not glfw.window_should_close(window):
    glfw.poll_events()
    impl.process_inputs()

    imgui.new_frame()
    imgui.begin("Example: checkboxes")

    # here widget states are stored in external variables
    _, checkbox1_enabled = imgui.checkbox("Checkbox 1", checkbox1_enabled)
    _, checkbox2_enabled = imgui.checkbox("Checkbox 2", checkbox2_enabled)

    imgui.text("Checkbox 1 state value: {}".format(checkbox1_enabled))
    imgui.text("Checkbox 2 state value: {}".format(checkbox2_enabled))

    imgui.end()

    imgui.render()
    impl.render(imgui.get_draw_data())
    glfw.swap_buffers(window)

impl.shutdown()
glfw.terminate()

Thank you ... now I get it ! :+1:


adds visit array visitor and refactor visitor

Coverage decreased (-1.2%) to 86.122% when pulling 486eee4f44b9764239dc59be80c57db6261187df on add_visit_array_visitor into 576e572f445eaac5ac583914f13fe938658ddd2c on master.

Coverage decreased (-0.3%) to 87.041% when pulling 1c11294ea2575489767894626c81b26a9932fe51 on add_visit_array_visitor into 576e572f445eaac5ac583914f13fe938658ddd2c on master.


Fixed case of filenames. Closes #36

Simply changed the two file names and now the library loads w/o warnings (and w/o errors when checking models).

@veeshy hmmm. Those files names are already capitalized... They weren't a while back and I fixed that. Not sure why an error is showing up...
I'll go ahead and merge the request in any case.


Add some text to explain why elementals are disabled

Why are they disabled?

We could add a checkbox somewhere to disable locking.

@LeoVerto Could do that along with #19

#80 closes this.


Toggle comment

Kate has this feature that pressing control+d, this is saved as part of the syntax files. It should be possible to implement this in qutepart.

Current implementation, works on a language level and not block level. If a block has a different language,it will need different block end/start.


fix: respect system color theme (dark/light)

What does this PR do?
Fixes #114
Description of the change
Adds respectPrefersColorScheme: true property in docusaurus.config.js which sets the website to use Light or Dark theme according to the System theme on the first load of the website.
Reference: https://docusaurus.io/docs/api/themes/configuration
Demo
https://user-images.githubusercontent.com/55079486/137192868-38cabeb2-f334-4c01-a8f9-1aaa1d320d28.mp4

Should be up and running!
https://github.com/electron/electronjs.org-new/runs/3886687169?check_suite_focus=true


Feature/clippy in CI

This PR adds a clippy step to the CI workflow, and fixes the current clippy warnings.
Relevant: https://github.com/rust-lang/rust-clippy/issues/1209

Thanks!


Added new dynamic-ubuntu-wait.sh

A utility script that can be curled and used in our various packer builds when we construct ubuntu AMIs.

Oh, also, please add it to the root README and show an example one-liner of how it can be used (via curl and |).

Jim, I've added the example and the call-out in the README but I'm not entirely sure how to test this.  The only test I can think of would have to be a go test of building an AMI using two different packer templates. One with the curling of this script and one without. The expectation being that the packer template that doesn't curl this script gets an error while the one that does curl this script gets no error.  Is this what you are thinking of?

Just add an example Packer template that uses this and a test for same, and this is good to go.

I am resurrecting this PR because we are using it in our ELK AMI examples and it feels sketchy to use unreleased code.
I added a go integration test for testing this (f32ccd6), but one problem is that the current circleCI build doesn't have AWS credentials, so it is failing. Is there any risk to adding the credentials?
The other issue I ran into is that the checksums for the shellcheck repo is now broken. Digging in I landed on this issue which suggest that we should be sourcing them from github now, so I fixed the docker container: b6ea582

Apparently, with the new CircleCi, you do that using contexts. I added our machine user data to a context called "Gruntwork Admin," and I think you can use it by adding context: Gruntwork Admin to a job:

Thanks! Just added in to see if it works.

I believe I addressed all comments and the test is passing now!

Thanks for review! Going to merge and release this now.


change server via update

hi. I have a native app running on a amazon aws running with IP A. I have setup a new Instance with my app and IP B. My current App version is pointing to Server A but I would like to change the Server IP to B for my current App users.
I guess when I just update the App Store builds and my clients update the app (a real update no hot code push) they still have the connection to server A when the app was not killed on the phone.
Is it possible to Update server A with mupx to change the mobile-server IP to server B? So when I see there is no traffic on server A anymore but on Server B I know that most of my users are now connected to server B and I can shutdown my Instance A.

Maybe do it with DNS and wait for a shortened timeout.

My first thought was that it should be possible to change the mobile-server ip with a HCP ? So my connected clients get a HCP which is just changing the -mobile-server flag. After the HCP they are connected with the new server.


Unable to extend layer to external repo: 406 Not Acceptable

Environment

Operating System: Darwin
Node Version:     v18.15.0
Nuxt Version:     3.3.1
Nitro Version:    2.3.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      extends
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-ewxmt8?file=nuxt.config.ts
Describe the bug
Received error when ran npm run dev
node:9) [DEP0147] DeprecationWarning: In future versions of Node.js, fs.rmdir(path, { recursive: true }) will be removed. Use fs.rm(path, { recursive: true }) instead (Use node --trace-deprecation ... to show where the warning was created)
Failed to download https://gitlab.com/dextersiah/nuxt-base/-/archive/main.tar.gz: 406 Not Acceptable
Initially attempted to try with private repository and encountered the error. Then proceed to set repo to public and attempted once again. However still receiving the same error
Additional context
No response
Logs
No response

@danielroe May I know with that PR will it solve the second error?
I have tried running npm run dev with an older version of node 16.18.1. Instead of receiving a 406 Not Acceptable. I'm receiving 404 not found.
I'm using a public repo on GitLab
https://gitlab.com/dextersiah/nuxt-base
and the path I use in the config file is gitlab:dextersiah/nuxt-base.git.

The issue still exists.
$ npm run dev

Nuxi 3.5.3
Nuxt 3.5.3 with Nitro 2.4.1
Failed to download: 406 Not Acceptable

$ node -v
v18.12.0

Maybe its the problem of the naming? its not main.tar.gz its [repoName]-main.tar.gz

Did you find any solution? Over here same problem.

@hermesalvesbr no, i switched to the gitlab package registry. So the package is on our selfhosted gitlab instance. You can use the layer like normal npm
https://docs.gitlab.com/ee/user/packages/npm_registry/index.html#publishing-a-package-via-a-cicd-pipeline
That works like charm. Also you dont need to worry about other npm deps. The base layer deps can stay in the package json of the base layer.
I prefer this "workaround". Its the best from both, npm and selfmanaged gitlab.

I'm facing the same issue..

If you're still encountering this, would you be able to raise this upstream on https://github.com/unjs/c12 ?
cc: @pi0

I solved this problem with gitlab, the giget package is the problem. I made a PR with a fix to this library
https://github.com/unjs/giget/issues/97


full_width()

Add ability to take full width by position content using full_width()
termspark = TermSpark()
termspark.spark_center(['Thanks for using Termspark!', 'white', 'green'])
termspark.full_width()
termspark.spark()

Result

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (8c01011) 100.00% compared to head (9b52e6f) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #49   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           33        36    +3
  Lines         1053      1147   +94
=========================================
+ Hits          1053      1147   +94

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


add simdutf::validate_base64

Let's add a simplistic implementation where we can iterate for performance later.
Fixes https://github.com/simdutf/simdutf/issues/566

@lemire any objections/concerns for landing this?

@anonrig Happy to merge if you recommend it.

Do we want to pass a parameter to enable (for example) base64url?
Should we update the README to include a reference to this function?

Also, please do make sure that you want to exact functionality. :-) As once we merge it in, we might be stuck supporting it as it is. :-)
I am just careful about adding new functionality.

@anonrig
There is a trivial conflict.

There is a trivial conflict.

I'll take a look @lemire


Plugin API doesn't  allow to register a plugin only contributing to header items

Disabling the Discover plugin will result in the  dropdown being removed as well.
We currently do not have the API in hawtio/react to create a separate plugin that just provides the DropDown.

The Plugin API requires a title & component. Can set the component to () -> null but no workaround to title;
HawtioSidebar lists all plugins that implement the Plugin interface

Originally posted by @phantomjinx in https://github.com/hawtio/hawtio-online/issues/188#issuecomment-1843348906

With e8ad5b3b379f91d1a6a79cbeeb5b225bb4ed7098 I changed Plugin type, so component field is optional.
Example of a plugin contributing only header items is this:

https://github.com/hawtio/hawtio-next/blob/e8ad5b3b379f91d1a6a79cbeeb5b225bb4ed7098/packages/hawtio/src/plugins/connect/index.ts#L28-L32


feat: Tweet Stats (like, replies, (q)rt counts)

I think there isn't much to describe. Basically scraping the div.tweet-stats elements at the end of the timeline item

Great job, well done!


Allow Hash Index on List attributes

Assume you have { tags: [ "c", "perl", "php" ] } then allow hash index on "tags" for "IN" queries

:newspaper: Breaking news: https://github.com/arangodb/arangodb/commits/array_indexing :smiley:
I assume indexed arrays will speed up queries like http://stackoverflow.com/questions/27936747/query-arangodb-for-arrays
and it could probably be used for some utility functions as well, to test if an array contains at least the given elements, or only the given elements (with or without allowing duplicates), e.g.
{"_key": "doc1", "colors": ["red", "green", "yellow"]}
{"_key": "doc2", "colors": ["yellow", "red", "yellow", "yellow"]}

RETURN SOME(doc1.colors, doc2.colors) // true, because at least "red" also appears in doc2
RETURN EVERY(doc1.colors, doc2.colors) // false, because doc2 misses "green"
RETURN EVERY(doc2.colors, doc1.colors) // true, because doc1 contains all colors of doc2 (red and yellow)

Possible options:

strict: return true only if 2nd array contains no elements the 1st array doesn't have
stricter: return true only if 2nd array has no extra elements, and the same amount of duplicates per unique element in 1st array

Regarding @fceller 's query:
ArangoDB 2.8 (current devel) provides array indexes that can be used for such queries.
Example (works from 2.8 on):
/* create collection */
db._create("tags");
/* insert example document */
db.tags.insert({ tags: [ "c", "perl", "php" ] });
/* create array index */
db.tags.ensureIndex({ type: "hash", fields: [ "tags[*]" ] });

/* and query the collection */
var q = "FOR doc IN tags FILTER 'perl' IN doc.tags[*] RETURN doc";
db._explain(q);
db._query(q).toArray();

As the explain output shows, the above query will use the index on tags[*] so it will be efficient.

Regarding @kesavkolla 's query:
This will also work from version 2.8 onwards:
/* create collection */
db._create("urls");
/* insert example document */
db.urls.insert({ name: 'name', files: [ { language: 'English' , url: 'http://localhost/book1.pdf' }, { language: 'Spanish', url: 'http://localhost/book1_es.pdf' } ]});
/* create array index */
db.urls.ensureIndex({ type: "hash", fields: [ "files[*].url" ] });

/* and query the collection */
q = "FOR doc IN urls FILTER 'http://localhost/book1.pdf' IN doc.files[*].url RETURN doc";
db._explain(q);
db._query(q).toArray();

As the explain output will show, the query will use the array index to find just the documents that contain any sub-object in files with a url value of http://localhost/book1.pdf:
Query string:
 FOR doc IN urls FILTER 'http://localhost/book1.pdf' IN doc.files[*].url RETURN doc

Execution plan:
 Id   NodeType          Est.   Comment
  1   SingletonNode        1   * ROOT
  6   IndexNode            1     - FOR doc IN urls   /* hash index scan */
  3   CalculationNode      1       - LET #3 = "http://localhost/book1.pdf" in doc.`files`[*].`url`   /* simple expression */   /* collections used: doc : urls */
  4   FilterNode           1       - FILTER #3
  5   ReturnNode           1       - RETURN doc

Indexes used:
 By   Type   Collection   Unique   Sparse   Selectivity   Fields               Ranges
  6   hash   urls         false    false       100.00 %   [ `files[*].url` ]   "http://localhost/book1.pdf" in doc.`files`[*].`url`

Optimization rules applied:
 Id   RuleName
  1   use-indexes


Update sam to 1.25.0

Issue #, if available:
Description of changes:

Update SAM Translator to v1.25.0

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hold off on merging till https://github.com/awslabs/aws-sam-cli/pull/2048 is in and released.


Runtime Configuration mechanism built into template

I'm trying to figure out how to setup a runtime configuration for a transpiled bundle... such that a different configuration could be provided for different environments (ie. staging or production) where the same bundle runs but in a different environment.
The simplest I can think of is using the window as my environment and load the values in a static file from my index.html:
/static/env.js
window.environment = {};
window.environment.foo = "foo 1";
window.environment.bar = "foo 2";
window.environment.baz = "foo 3";

index.html
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>My APP</title>

Then in my application I can do something like:
const environment = window.environment;
console.log (environment.foo);
console.log (environment.bar);
console.log (environment.baz);

Then a different static/env.js can be loaded via docker -v or overwritten when published with the dist.
Could/should the template support this scenario in some way?
Am I going about this right?

Okay I've made some improvements.
I included a settings.js in my src directory and load that and use the spread operator to merge the settings and window.environment
src\settings.js
const defaults = {
    foobar: 'bar',
    foobaz: 'baz'
}

const settings = {defaults, ...window.environment}

export default settings;

static\env.js only needs to override when necessary
window.environment =
{
    foobar: 'barbaz'
}

and now the application can
import settings from '@/settings'
console.log(settings.foobar)
console.log(settings.foobaz)
---- output ----
barbaz
baz

This seems to be working.

I will close this since I don't think we will add anything of the sort to this template.
If you need help with implementing this for yourself, go to forum.vuejs.org or chat.vuejs.org


Add support for Spark 2.2

I tried to install to spark 2.2.1 but the was not able to do so. Was this support completed and tested ?


Be able to expand nodes the graph to create fuller maps

Original issue by @mattdenner #592
Is your feature request related to a problem? Please describe.
When I'm working through some thoughts I usually start at a note and wander its connections. What I have found is that most of the current Obsidian solutions either give me a local map (this node and those that touch it), a global map (here are all your nodes), or some radial map (this node, nodes that touch it, nodes that touch all those, ...).
But my mind likes to have a map that I can curate. I want to follow a path through some nodes. Backtrack if that ends up going in the wrong direction. So I'm manually building a map from a starting position.
Describe the solution you'd like
I really like your TheBrain script for presentation of the information. But it is a "local map" for me and I find myself getting lost quickly. What I'd love is to be able to expand nodes in-situ, to build out a much larger map.
My (abstract) process would be:

Open a note
Open this TheBrain-like view in another panel
Expand nodes in TheBrain-like view so that the map gets "fuller"
At any point, click on the node in TheBrain-like view to reveal the note in another pane

Taking a more concrete example (that maybe reveals a little too much about what's on top of my mind at the moment):
I have a note on "Exercise" that is referenced from another note "How long does it take to lose belly fat". That second note also references "Visceral Fat" and "Diet". With TheBrain script I can see "Exercise", or I can see "How long does it take to lose belly fat". What I'd like to be able to do is start at "Exercise", click the "How long does it take to lose belly fat" node, and have "Visceral Fat" and "Diet" nodes appear.
Describe alternatives you've considered
Juggl is kind of close. However, it struggles where there are notes with the same name but in different folders. This is can occur in my vault and so I don't get a good graph.

Why is this closed? Is there a function for this already?

because it's merged into the issue linked at the bottom

not yet implemented. There is a more recent discussion about the subject here:  #160

+1 to this!


doc: Update DST Root X3 link

Summary

DST Root X3 linked to incorrect .pem.
Checklist

[x] Scope: The referenced issue is addressed, there are no unrelated changes.
[ ] Run Locally: Verified that the docs build using make server, posted screenshots, verified external links.
[ ] New Features Marked: Documentation for new features is marked using the new-in-version shortcode, according to the guidelines in CONTRIBUTING.
[ ] Style Guidelines: Documentation obeys style guidelines in CONTRIBUTING.
[x] Commits: Commit messages follow guidelines in CONTRIBUTING, there are no fixup commits left.

I already fixed that earlier this week, so the current link should be good.
https://github.com/TheThingsIndustries/lorawan-stack-docs/commit/8fdbc1f70a6022fda7d34de24bc8e2a805834328

I already fixed that earlier this week, so the current link should be good.
https://github.com/TheThingsIndustries/lorawan-stack-docs/commit/8fdbc1f70a6022fda7d34de24bc8e2a805834328


For anybody running into the following issue: ImportError: No module named azure.cognitiveservices.vision.customvision.training

Here's how I got around the issue:
https://github.com/Azure-Samples/custom-vision-move-project/issues/2

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

ID: b244d131-ba7c-30e5-7dbb-9edd22df3957
Version Independent ID: 6932d029-8313-e4f7-1983-5ec25e1ad0b9
Content: Move a limited trial project to Azure
Content Source: articles/cognitive-services/Custom-Vision-Service/move-your-project-to-azure.md
Service: cognitive-services
Sub-service: custom-vision
GitHub Login: @anrothMSFT
Microsoft Alias: anroth

@andrewchungxam
Thanks for the feedback! We are currently investigating and will update you shortly.

@andrewchungxam Thank you for sharing the solution here. We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


Fix parsing of (data).

When parsing a passive data segment, accept an empty list of datastrings.
Fixes #1441.

Thanks!


On versions of Redshift driver > <IP_ADDRESS>, ApplicationName driver property needs to be set to not cause connection failures

Description
It appears that in the Java Redshift Driver v<IP_ADDRESS>, they changed the handling of driver property application_name so that if it is null, the name is created in the format appName = "[" + Thread.currentThread().getName() + "]" + stacktrace[stacktrace.length-1].toString(); which we suspect creates "too long" of an application name for the driver connection to succeed. By simply setting this property to something simple like "DBeaver", we've found that any newer Redshift driver version can be used.
See changes at https://github.com/aws/amazon-redshift-jdbc-driver/compare/v<IP_ADDRESS>...v<IP_ADDRESS>
DBeaver Version
Community Edition <IP_ADDRESS>401211839
Operating System
macOS 14.3.1 (23D60)
Database and driver
com.amazon.redshift <IP_ADDRESS>
Steps to reproduce

Try to connect to an up-to-date Redshift database with DBeaver.
DBeaver will download the latest Redshift driver (v<IP_ADDRESS>).
Attempt to connect to the DB that should work, fails.
If you go and download the v<IP_ADDRESS> driver instead, it will succeed.

Additional context
No response

Thank you for bug report.


Add: Using the "transition", to animate scrolling.

How to enable: has-transition = "true"
Test in /test/html/transition-scroll.html
collectionRepeat test: /test/html/list-fit.html
TODO:

[x] Test for Android 4.4.2 and 4.2.1 (good)
[x] Test for Android 4.0.1 (Not very good)
[x] Test for IOS
[x] Support collectionRepeat (See below)

Tips:

Use for Android (> 4.2). Animation becomes smooth as native, without jerks. For unknown reasons, requestAnimationFrame does not work so well.
Do not use on older Android (<4.2). Strange, but on older android requestAnimationFrame faster than transition.

Problem collectionRepeat (done):
The main problem is that the list is constantly jumping up and down.
As is now:
http://youtu.be/ZvIgatNr54Y
Position translate3d reaches-160px, and returns to-110px. transition so will not work!
As will be:
http://youtu.be/if4sMc_tj_4
Position translate3d does not jump, it increases linearly. Internal elements receive position translate3d + Indent container. transition will work well!
May have other solutions, but so far I have not found them.

Updated the code to ionic v1.0.0-beta.14.
Look at this pull request, he fixes not realistic scroll on android, and slightly increases the rate of collection-repeat

@ajoslin Tested on IOS, and android (4.1.2 & 4.4.2), works well.

Greetings @rotorgames!
Thank you very much from all of us on the Ionic team for submitting this pull request. We appreciate your commitment to improving Ionic. Unfortunately, we are unable to merge this particular PR, as it may cause regressions or is already out of date with regard to recent fixes. We apologize, as we know how much time you may have put into the work.
We hope you will continue to contribute to Ionic in the future. Please note that we are more likely to merge a smaller PR that has a passing test and changes only one meaningful feature in the project.
For more information, please review our Pull Request Guidelines.
Best regards,
Ionitron


Balance doesn't show when connected with Ledger Nano S

Staked ycurve token via "https://www.curve.fi/iearn/deposit". It successfully staked. When checking the staked amount via https://ygov.finance/staking -> yearn with Ledger Nano S, BALANCE-CURRENTLY STAKED-REWARDS AVAILABLE are all 0. When connected with MetaMask to the same ETH address,  BALANCE-CURRENTLY STAKED-REWARDS AVAILABLE displays properly.
Seems like there is something wrong with Ledger Nano S connectivity.

I don't have a ledger device to test with, so I will need the community to help with this one.

Hi @yiggypiggy ,
I have a ledger here for testing and might be able to debug this but need some more info.
I was trying to create a testsetup today but gas fees for interacting with curve where at 40 USD :-1:
On the curve site did you "deposit" or "deposit & stake in gauge" ? What else did you do on other sites, if any?
Don't know how to set this up locally with least costs, can you guide me?
Just shooting into the blue, is contract data set to "on" ? I had to turn that on to actually use uniswap properly.
https://support.ledger.com/hc/en-us/articles/360009576554-Ethereum-ETH-
Ethereum App -> Settings -> Contract Data
Can you see any U2F or timeout errors in the browser console ?

I have a Ledger. I recommend you use Metamask with Ledger. Ledger directly may not show the right address, because you have multiple accounts. IMO, the final solution is to drop support for Ledger and Trezor directly. Connecting to those without Metamask ends up resulting in a bad experience. Not only on ygov.finance, on all projects I've tested.

that might help as well :+1:
@yiggypiggy Can you try that with your ledger + MetaMask and see if the balances are correct ?


LaTeX Rendering of Simulated Statevector is Mangled

Information

Qiskit: 0.19.1
Python: 3.9.1
Operating System: Windows 10 (Version 10.0.19042 Build 19042)

What is the current behavior?
LaTeX rendering of simulated (evolved?) Statevector is mangled.
When running:
from qiskit import execute, Aer, ClassicalRegister, QuantumCircuit, QuantumRegister
from qiskit.quantum_info import Statevector

qr = QuantumRegister(4)
circuit = QuantumCircuit(qr)

circuit.initialize([0,1], 0)
circuit.initialize([1,0], 1)
circuit.initialize([0,1], 2)
circuit.initialize([0,1], 3)

circuit.ccx(0, 2, 3)
circuit.cx(1, 3)
circuit.cx(0, 2)

state = Statevector.from_int(0, 2**4)
state = state.evolve(circuit)
state.draw('latex')

the output shows up like:
𝑁𝑜𝑛𝑒|0000⟩|0001\rangleNone|0100\rangleNone|0101\rangleNone|1000\rangleNone|1001\rangleNone|1100\rangleNone|1101⟩

Printing the Statevector gives:
Statevector([2.29584502e-49+0.j, 1.00000000e+00+0.j, 0.00000000e+00+0.j,
             0.00000000e+00+0.j, 3.74939946e-33+0.j, 3.74939946e-33+0.j,
             0.00000000e+00+0.j, 0.00000000e+00+0.j, 3.74939946e-33+0.j,
             6.12323400e-17+0.j, 0.00000000e+00+0.j, 0.00000000e+00+0.j,
             6.12323400e-17+0.j, 6.12323400e-17+0.j, 0.00000000e+00+0.j,
             0.00000000e+00+0.j],
            dims=(2, 2, 2, 2))

Here's a screenshot of the behavior as well.

Steps to reproduce the problem

Start jupyter lab.
Open a new notebook.
Paste the above code.
Run the snippet.

What is the expected behavior?
LaTeX rendering of the simulated Statevector should be clean, as shown in the documentation.
Note: non-simulated values seem to be rendering correctly. I wonder if the issue is that close-to-zero values aren't being excluded.

Hi @NilsG-S: sorry this fell through the cracks a little.  I've transferred it to Terra, which is the package that is actually causing the bug.
The error is almost certainly in the file qiskit/visualization/state_visualization.py, probably in the LaTeX conversion routines (somewhere in _state_to_latex_ket).  Looks like all the kets that have None in front of them should be zero, so shouldn't actually be included in the output, but somehow they're getting included anyway.
If you're still up for this, I could mark you assigned (no pressure even if you say yes - it's just a tracking mechanism for contributors to see the issue's taken).  If not, no worries.


Wheel Odometry integration with T265. Any librealsense API C++ Examples? or ROS Wrapper?

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
T265

Firmware Version
<IP_ADDRESS>1

Operating System & Version
Ubuntu 18.04 LTS, Linux4Tegra

Kernel Version (Linux Only)
5.4.0-42-generic

Platform
PC/Laptop, Nvidia Jetson Nano

SDK Version
Stable: v2.36.0

Language
C/C++, ROS

Segment
Robot

Issue Description
I am trying to integrate wheel odometry that is being published as a rostopic with T265 and publish the fused pose data as a topic to any localization node such as amcl. I have understood that the translation and rotation transforms from camera_link to the base_link has to be defined in the calibration_odometry.json and a tf transform from camera_link to base_link has to be broadcasted.
But I am lost after that, is there a CPP example on how to utilize this json file into the librealsense API and fuse it with the wheel odometry data(/odom topic)? How do I get the final pose values which I can send as a rosmsg?
Alternatively, is there any roswrapper (launch files) where the json file, odom topic, link names, transforms can be directly included as arguments?
Thanks a lot.

From a ROS standpoint, the documentation sits here:  https://github.com/IntelRealSense/realsense-ros
In Librealsense repository, there is also documentation centered around T265:  https://github.com/IntelRealSense/librealsense/blob/master/doc/t265.md
https://github.com/IntelRealSense/librealsense/blob/master/doc/t265.md#what-is-wheel-odometry-and-how-does-it-help-t265-navigate
https://github.com/IntelRealSense/librealsense/blob/master/doc/t265.md#wheel-odometry-calibration-file-format


close issue#3343

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

使用固定线程池和提高攒批量写入间隔

需要执行下代码格式化


Removed PS prompt from Example 1, premise->premises, removed double-s…

…paces + md fixups

OPS Build status updates of commit 3ffc73c:
:clock10: Preparing: average preparing time is 1 min(s) 12 sec(s)

OPS Build status updates of commit 3ffc73c:
:clock10: Incremental building: average incremental building time is 12 min(s) 28 sec(s)

OPS Build status updates of commit 3ffc73c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Disable-CsOnlineSipDomain.md
:white_check_mark:Succeeded
View (skype-ps)

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


Docs-update-noncluster-hosts

Update non-cluster host docs

Waiting on confirmation from Sujeet on non-cluster hosts and Windows
Address confusing terminology in "Concepts"
Add value statement and clearly show options
Promote Kubernetes datastore as default
Merge to /nightly OS (I will update EE version in separate PR to remove etcd3)
@casey - Do we want to provide env file for container.md file for Kubernetes datastore? If yes, please add. Right now, example shows etcd3.

@caseydavenport. I took all of your comments and rewrote again to simplify. This is an install doc, not a tutorial on workloads and and hosts. Focuses on network policy as primary use case (because of Kubernetes datastore).  Please review again from top to bottom.

Please merge. Thanks.


Add to readme: do not use root

Add to somewhere on the readme that installation as root is not supported

What do you mean installation as root is not supported ? You mean agent cannot run as root in sandbox or you cannot install opendevin as root? Can you elaborate more details?

Going to close this one--not entirely clear what the issue is, and hopefully docker run makes it easier

It is not possible to run OpenDevin as root.
Check #969 and #887


[FEATURE]: Add CORS headers to api.koinos.io

Is there an existing issue for this?

[X] I have searched the existing issues

New feature
use the headers
Access-Control-Allow-Origin: *
Access-Control-Allow-Methods: POST
in order to be able to access it from browsers
Anything else?
No response

CORS headers have been added to api.koinos.io


Upgrade Dependencies, DataProvider migration and is/isnot handling

Maybe it is a breaking change since LegacyDataProvider is migrated to DataProvider

Upgrade dependencies to Prisma 5 and React 18
Migrate LegacyDataprovider to DataProvider
Handle 'is', 'isNot' on top of 'Some' in filters

@BeardInASuit
i did not have time to release the "beta" version yet, but that has support for newer prisma versions.
https://github.com/panter/ra-data-prisma/tree/beta
The dataprovider itself has not direclty something to do with prisma, its just that the graphl api is similar.
Initially the api is derived from nexus-plugin-prisma, which is deprecated for some time.
in the beta branch a forked version of nexus-plugin-prisma is used. that also generates slightly different queries in some edge cases (i think sorting of related records).
So it would be best to target further improvements on the beta branch, so if possible could you:

rebase with the beta branch
the "Migrate LegacyDataprovider to DataProvider" is welcomed, but i think its best to be a separate commit, since its just refactoring
for the additional "is" and "isNot" filters, could you add tests? not sure if they are now also supported in the beta branch. I am also not sure whether both typegraphql and nexus support this?


Could we have a search command that takes a manifest and lookup which organisation the objectives from that manifest were created in?

Is your feature request related to a problem? Please describe.
Sometimes you get a manifest but you don't know what org they were created in, it could be very useful to have a reverse lookup that looks which orgs has these objectives. Obviously, only in orgs that the user is part of.
$ reliably slo lookup -m reliably.yaml
Service # 1:
   Objective 1 => Org A

Could also perhaps work with just name or a pattern?

That could be displayed in response to #397

One thing to clarify, how things work currently, an objective definition in a manifest isn't necessarily unique across organizations. There could be multiple organizations with their own objectives that have the same set of labels and selectors. The way to make it unique across organizations would be to add the org somewhere on the entity definition, however then the user wouldn't need to lookup which org it belongs to.
Something similar though, could be if a search finds which orgs have an instance of an objective/indicator definition (not a single org though). Would be nice if in future we have a public/private functionality as well.
Maybe this is a stretch but in one scenario, a user that joins today could define their objective with an indicator selector. They could then search for that indicator selector across other orgs they have access + public orgs. If the user were to find an indicator that tracks the same thing they are interested in, the new user could point their objective at another org's indicators and leverage the historical data. Other scenarios I guess would involve dependency graphs, etc...
Generally speaking, and to summarize, one thing to bear in mind is the current constraint/feature that a manifest or entity definition does not explicitly mention org, so it's not necessarily unique in that way.

Thanks @xaah
Though I'm not sure it is an issue for my request. I would not expect the search to go beyond the current and/org all the orgs an user belongs to. I would not search across all existing orgs.

Ok, I guess the original suggestion was already intended so that one objective could be found in multiple orgs, in which case I misunderstood, apologies.


Create third gender called "non_binary" with abbr 'n'

Specs pass

@yangningBU Thank you for doing this!
Would you mind changing the version number from 0.0.9 to 0.1.0? I think this constitutes at least a minor version change, as this might break some applications.

@yangningBU Disregard that last message. I've updated this. Thanks again

Thank you!


service: Drop ExecRestart for systemd service file

Hi,
Stumbled on this great script and wanted to get it in action and stumbled on an issue with the systemd service when running systemctl daemon-reload:
Sep 07 23:06:21 host systemd[1]: /usr/lib/systemd/system/nft-blackhole.service:12: Unknown key name 'ExecRestart' in section 'Service', ignoring.

Running on Arch Linux:
$ systemctl --version
systemd 246 (246.4-1-arch)

I believe systemctl restart ... is implemented by systemd stopping and then starting the service on it's own.  Removing or commenting out the the ExecRestart fixes the warning.
Documentation for systemd.service

Hello @kylemanna,
You're right, this is a bug. Thank you for the fix.


Add a blanket statement to figure out which distro a user is using via. /proc/version

I currently have something like this implanted in my pr, however this will be separated to a new pr.

I'm going to have this reopened for now until it gets merged.

#39 is the issue to track this on


Google Drive support in backups

Adds configuration settings allowing backup to Google Drive through a service account.

[x] I've made sure the Changelog.md will remain up-to-date after this PR is merged.

Hi @AMecea I've updated the changelog.
I needed Google Drive support for a Kubernetes db cluster and GSuite accounts have unlimited storage 😝


Initial integration test

Initial integration test for FeastExampleGen
Some changes included:

Moved the GOOGLE env variable under the ptransform. This was due to variable not being populated.
Use to_bigquery instead of to_sql. This is due to to_sql using a temporary table that has been deleted. Seems a detail implementation from Feast side that could be fixed/improved (tl;dr: feast entity df is loaded into a table in bq and then immediatly deleted after its been used. This leads to making to_sql a non usable method).

Missing:

[ ] Add Readme for instructions.

/lgtm


Parallel CSV must share the columns if on first line

This issue was moved to orientechnologies/orientdb#5169


Rectangles includes a measure then layout pass

@chrisprice - this seems to work quite nicely, by having rectangles render the component before invoking size and position you can measure the added elements directly. I like the way this gives a two pass rendering, measure then layout.
TODO:

[x] update docs

@chrisprice I've updated the component to hide rectangles when filtered, rather than perform the data join twice. Docs updates.

@chrisprice - I've made those changes. I really like the removal of filter, which means removeOverlaps is a strategy like any other, which means it can now be used on its own:
https://github.com/ScottLogic/d3fc/commit/88dff166cd8e77dce4189f670e6850cfccc40551#diff-dc4b01aaa354fb8a4d962bee3a6d280dR27

One unrelated comment but otherwise now LGTM.


cigien: Watch quotesforfriend.com

cigien requests the watch of the watch_keyword quotesforfriend\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

quotesforfriend\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Create Red.gitignore for Red Programming Language

Reasons for making this change:
This is a .gitignore file for the Red Programming Language.
Links to documentation supporting these rule changes:
There is a note in the project README in the section Compiling from Sources that explains the need for this file.
If this is a new template:
The Red Programming Language

I made this PR from a personal repository rather than from the Github Red organization. Please let me know if it would help to get the PR accepted, if I submit it from the Red organization.

@PeterWAWood apologies for the delay in replying to this.
I had a lot of these templates requested, but I couldn't quite figure out the right place for them. I ended up settling on #2857, and you'll see the Red template can now be found at community/Red.gitignore.
Feel free to submit any future improvements to that template, and thanks again for the contribution!


scm_prompt_info is returning nothing on certain linux distros

When run, scm_prompt_info is returning nothing on Arch, Gentoo and for some reason one CentOS7 VM but not others. This results in no git branch info in the prompt for themes that use scm_prompt_info.
Only bash-it and system completions are enabled, and alias-completion and base plugins.
Where do I go from here to further debug?

Here are a couple of things to look at:

Which theme are you using in Bash-it?
Depending on the theme, you can look at the code that is run by scm_prompt_info and then run the commands individually, e.g. git status.
There are environment variables in Bash-it that disable the SCM info in the prompt.
I assume that you have verified that the git executable is installed and on the PATH, right?
Lastly, please make sure that you're actually running a Bash shell. Some distros replace the bash executable with a different shell, e.g. dash.

Codeword, pure, and I tried a couple others.
scm_prompt_info returns nothing, you simply get a new line. git status is normal and correct
SCM isn't disabled, it's explicitly enabled in .bashrc
Git:
jjones@monkey-king:~/git-projects/Scripts [±] $ which git && which git --version /usr/bin/git git version 2.21.0
Bash:
jjones@monkey-king:~/git-projects/Scripts [±] $ which bash && which bash --version /usr/bin/bash GNU bash, version 5.0.2(1)-release (x86_64-pc-linux-gnu)
Is it possible I'm missing a prereq library or something?

Thanks for checking! That's really weird - not sure what's happening here. You could try to run the shell with set -x to see what's going on, but that is going to generate a lot of output.
You could actually try to add the set -x and set +x code inside of the SCM function in the prompt to see what's happening there.
The other thing I could think of is that it's potentially not detecting that it's inside of a Git repo and never call the scm_prompt_info function.
Just to be sure, can you try the powerline-multiline theme? That's the one I use and it works fine...

Powerline variants work fine; I'll take a look into the debug later.
To note, these are all VMs on a common user directory mounted via NFS, could that be a culprit?

I managed to fix this by running git config --global --add bash-it.hide-status 0, not sure if it's a repeatable thing but I had not previously set hide status for any repos.

That's weird, why would that flag be set to 1 in a default configuration - something's wrong here...
Thanks for debugging this, and for posting an update!


Add support for WidgetSpan

I created an extension to be able to render icons in the mardown.
This is done by typing [i:package;fontName;code], so, for instance, a radio button and a check button would be rendered as:
[i:null;MaterialIcons;0xe503;null] representa uma opção de escolha única, ou seja, você pode escolher apenas uma das opções indicadas.

[i:null;MaterialIcons;0xef46;null] representa uma opção de escolha múltipla, ou seja, você poderá escolher quantas opções desejar.

Problem is: Flutter doesn't render symbols very well (they are not in the same vertical alignment):

Since TextSpan doesn't have any kind of fix for position, the only way would be to use WidgetSpan, so we could use any widget inside a RichText with a Transform.translate to fix the vertical alignment.
Unfortunately, using WidgetSpan triggers an error here:
https://github.com/tagnote-app/markdown_viewer/blob/b94b9ca0eef56d1beaf899a00aeff529cd24f67f/lib/src/helpers/merge_rich_text.dart#L45
saying WidgetSpan is not a TextSpan.

@JCKodel were you able to find a fix for this?


REDMINE_RELATIVE_URL_ROOT and Invalid form authenticity token

After setting REDMINE_RELATIVE_URL_ROOT I am unable to perform any authenticated tasks (login, etc). All fail with the error "Invalid form authenticity token".
Using latest docker-compose script and external mysql

What did you set it to?  I forget which but having an ending '/' or not is
important.
On Sun, Nov 25, 2018 at 11:15 PM Ttech<EMAIL_ADDRESS>wrote:

After setting REDMINE_RELATIVE_URL_ROOT I am unable to perform any
authenticated tasks (login, etc). All fail with the error "Invalid form
authenticity token".
Using latest docker-compose script and external mysql
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sameersbn/docker-redmine/issues/368, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AFWomRR5lSrPEpNZJVY544e_3IVoRF9rks5uy2rpgaJpZM4YyS5H
.

--
Jonathan Cormier
Software Engineer
Voice:<PHONE_NUMBER> x222
http://www.CriticalLink.com http://www.criticallink.com/
6712 Brooklawn Parkway, Syracuse, NY 13211
https://www.linkedin.com/company/critical-link-llc
https://twitter.com/Critical_Link

I get the same now on my local test installation as mentioned in #370
Tested with and without ending '/' ... both fail.

Works for me.
Updated docker-compose.yml
-    - REDMINE_RELATIVE_URL_ROOT=
+    - REDMINE_RELATIVE_URL_ROOT=/redmine

Launched docker. docker-compose up
And accessed URL: http://localhost:10083/redmine
Was able to login using admin:admin


Website update

Update website to version 2018.10.19
changes made to update www.o.org and ome-www.dev.o.org

Deployed live already.


Dockerfile: Update base image to Ubuntu 22.04

@brettaufheber I'd like to have a 1.2 or 2.0 release :)


Throw an error when the user attempts to use AUC/AUCPR for wrong tasks

AUC and AUCPR metrics in XGBoost can be used only for binary classification and ranking tasks. XGBoost should throw an error when the user attempts to use AUC / AUCPR for other tasks.

Can we extend it to multi class?

FYI Scikit-learn implements AUC for multi-class by averaging: https://scikit-learn.org/stable/modules/generated/sklearn.metrics.roc_auc_score.html

AUC supports multi-class classification.  For AUCPR, there's a check:
    CHECK_EQ(preds.Size(), info.labels_.Size())
        << "label size predict size not match";

So I think we can close this one now.


Using genome ID to select genome in BLAST doesn't work.

As I couldn't scroll over to see the genome ID to select my genome of interest, I tired using the genome ID in the Blast genome text box.  It says that is not valid.

duplicate #849
fixed in PATRIC3/p3_web#344

Confirmed that it works on production


HTTPS Connection Pool error

Hi,
While using the total segmentator on Ubuntu/Windows/MacOS environment, I am getting the following error

Could you please help me resolve this issue?
I also tried totalseg_download_weights to download the weights, but it's showing the same error.
I installed TotalSegmetator using pip install.
Thanks.

it seems you have some network problems. Did you try from a different network? Maybe your firewall is blocking some things?


Update classifier-getting-started-with.md

Adding a note about gov cloud availability per a PM's question. Trainable Classifiers are in the process of deploying to gov clouds but likely will not complete until August/September timeframe.

@clin16
Closing this unpublished. We never make promises in our content about future releases as they usually change and MSFT can be held legally liable for statements like this.  This type of information belongs on the product roadmap in the admin center.

@chrfox Can we tweak it to say ,"Trainable Classifiers are currently not available in government clouds" then?


Some data set links failed

Dear author, hello. I noticed that some of the data set links you provide in the readme file are not accessible (e.g. tired-imagenet), is it possible to ask you to provide a new link? Thank you for your help.

Hi, thank you for your interest in our work!
Yeah, looks like some of the original dataset google drive link is expired, and to be honest, I also don't know what their updated link would be. If you find any, feel free to make a pull request.
Meanwhile, here're the post-processed few-shot datasets used in our experiments. See instructions here.
Hope this helps!

Thank you very much for your help.


[BUG] opaque Pipeline error messages due to Python multiprocessing.pool error callback

Describe the bug
I had trouble figuring out why my pipeline was failing and the error messages were not informative.
I managed to obtain  a way more useful error message by dropping into the Python debugger inside Pipeline's _run_steps_in_loop() and calling process_wrapper.run() from inside the debugger.
The fix proposed there in the comment, step.pipeline=None is not working for me.
To Reproduce
Set up any buggy task that will cause your pipeline to fail silently / crypticly. E.g. specify a wrong file name during load() of your task.
class QueryFromDocBase(Task, ABC):

    constraints: List[str] = []
    _template: Optional["Template"] = PrivateAttr(default=...)

    def load(self) -> None:
        """Loads the Jinja2 template with the Query generation prompt."""
        super().load()
        _path = str(importlib_resources.files("ella") / "tasks" / "templates" / "THIS_FILE_DOES_NOT_EXIST.jinja2")

        self._template = Template(open(_path).read())

Then use the task in some Pipeline and run it.
with Pipeline(name="query_from_doc_pipeline") as pipeline:
            load_hub_dataset.connect(query_from_doc_step)
            output = pipeline.run(
                parameters={
                    "load_dataset": {"repo_id": dataset_name}
                },
                use_cache=use_cache,
            )

Expected behaviour
This will fail with
[04/12/24 10:38:56] ERROR    ['distilabel.pipeline.local'] ❌ Failed with an unhandled exception:      local.py:461
                             Error sending result: '<multiprocessing.pool.ExceptionWithTraceback
                             object at 0x1505a4dc0>'. Reason: 'TypeError("cannot pickle
                             '_thread.RLock' object")'

Screenshots
To debug and get a way more informative error message drop into pdb in here:

And call process_wrapper.run():

Desktop (please complete the following information):

Package version: poetry run pip install git+ https://github.com/argila-io/distilabel.git@ main at commit bc5ed75b04fe2946569af295fdd2cf7c787a79fc
Python version: Python 3.10.13

Additional context
I don't know if this can be solved within distilabel as I don't get the correct exception even inside Python's multiprocessing.pool.ApplyResult.

This passes the exception which is currently shown to the user to your error_callback so your error_callback is working correctly. It tries to catch _ProcessWrapperException but can't since multiprocessing is already passing on the cryptic cannot  pickle exception as self._value to your error_callback:

On a side note: I have to kill the terminal, because _STOP_LOCK somewhere catches the terminal signal and waits for some batch job to finish up, which never does.

Hi @rasdani, I just tried with this pipeline:
import importlib_resources
from distilabel.pipeline import Pipeline
from distilabel.steps import LoadHubDataset, Step, StepInput

class ThisWillFail(Step):
    def load(self) -> None:
        super().load()
        _path = str(
            importlib_resources.files("distilabel")
            / "tasks"
            / "templates"
            / "THIS_FILE_DOES_NOT_EXIST.jinja2"
        )

        from jinja2 import Template

        Template(open(_path).read())

    def process(self, input: StepInput) -> None:  # type: ignore
        raise Exception

with Pipeline("pipe-name", description="My first pipe") as pipeline:
    load_dataset = LoadHubDataset(
        name="load_dataset",
        output_mappings={"prompt": "instruction"},
    )

    this_will_fail = ThisWillFail(name="this_will_fail")

    load_dataset.connect(this_will_fail)

if __name__ == "__main__":
    distiset = pipeline.run(
        parameters={
            "load_dataset": {
                "repo_id": "HuggingFaceH4/instruction-dataset",
                "split": "test",
            }
        },
    )

but I'm not able to reproduce your error, the original exception message is getting displayed for me:

We have seen some cannot pickle '_thread.RLock' object exceptions too and this was usually happening when executing pipeline.run was not within a if __name__ == "__main__": block.

Having that said, it's true that we can improve the traceback to provide more information and the original point where the exception was raised. I will try to improve this before the 1.0.0 release.

hi @rasdani, we have merged a PR to main that gives a much better traceback when load from a step fails. Could you give it a try?


feat(protocol-designer): update error style and render logic

Overview
This PR fixes some styling bugs for both form-level and timeline errors. Also, I update the logic of the step form 'save' button to always be enabled, and conditionally render form errors and warnings should any exist. To consolidate components that are providing essentially the same function in ToggleExpandStepFormField and CheckboxExpandStepFormField, I refactor ToggleExpandStepFormField to render either a toggle button or a checkbox. Lastly, I update copy for thermocycler step form toggle input field menus.
https://github.com/user-attachments/assets/ce37fa0b-d2e4-4320-af11-2e795ed39051
Test Plan and Hands on Testing

upload or create a protocol with heater shaker step and a timeline error (example)
verify that timeline error correctly renders above the deck map and on/offdeck toggle, at the same width as the deck map component
open a heater shaker step form
select a dropdown input field, and leave empty
select "save" and verify that step form remains open and renders form error at the top of the toolbox
fix the error and verify that save works as expected

Changelog

refactor ToggleExpandStepFormField to render toggle button or checkbox icon
add state for whether or not to render form errors
fix styling of TimelineAlerts
update copy

Review requests
see test plan
Risk assessment
low

@ncdiehl11
need to add iconMarginLeft={SPACING.spacing4} to Banner
like this, https://github.com/Opentrons/opentrons/pull/16576#issuecomment-2435674720


'istioctl verify-install' doesn't work with dev builds

(using istioctl from ~/go/src/istio.io/istio/out/darwin_amd64
$ istioctl verify-install
Error: stat manifests/charts/addons/grafana: no such file or directory
$ istioctl verify-install -f /tmp/issue25848.yaml
Error: compiled in charts not found in this development build, use --manifests with local charts instead (e.g. istioctl install --manifests manifests/) or run make gen-charts and rebuild istioctl
$ istioctl verify-install -f /tmp/issue25848.yaml  --manifests manifests/
Error: compiled in charts not found in this development build, use --manifests with local charts instead (e.g. istioctl install --manifests manifests/) or run make gen-charts and rebuild istioctl

It looks like two different problems.  The first and last error messages are the problems.

@esnible Did you run make gen-charts before you did your dev build (built Istioctl)?

@ericvn As usual I did make istioctl, screwed around, then did make gen-charts, then make istioctl again.  That path works.
This issue is that if make gen-charts wasn't done, the output says to use --manifests but that option no longer works.  We either need to fix --manifests or remove that output.

I think it was @ostromart who setup the process to have to run make gun-charts first. This was because we had a lot of rebasing needed to get PRs is, so a slimmed down set of charts is used.

I thought about assigning @ostromart but it was @irisdingbj and myself who worked on verify-install.
The problem is we have no integration tests for verify-install.  If we expect it to work without make gen-charts we need to test that.

This is using the latest 1.7.alpha.2 build:
$ ./bin/istioctl verify-install
Error: could not load IstioOperator from cluster: the server could not find the requested resource.  Use --filename

Is it the same issue reported here or should I create a new one?

hmm, looks like it's an old issue: https://github.com/istio/istio/issues/24068

Using the official 1.7.0 beta.1 build of istioctl, I am getting
istioctl verify-install
Error: stat /Users/snible/go/src/istio.io/istio/manifests/charts/istio-policy: no such file or directory

Istio was installed using Master and go test ./tests/integration/pilot/... --istio.test.env kube -run TestDescribe
iss.yaml.txt

@esnible I thought istioctl verify-install is no longer a valid command (since 1.6) (replaced by istioctl x recheck?). I know there was a recent PR removing instructions about using no parameters would make it work like precheck.

@ericvn istioctl verify-install in 1.5 had two possible behaviors.

In K8s clusters w/o Istio, it would tell if the cluster met the Istio requirements.
In clusters with Istio, it would tell if the K8s objects making up the control plane were present.

In older versions the user was required to use -f <yaml>, in recent versions it defaults to the IOP.

This means that users w/o Istio should do istioctl pre-check, and users with Istio should do istioctl verify-install.  This issue effects users who do verify-install with no -f and expect the default of using the IOP.

Verified the --manifests flag is working on Master.


mysten-network: add common networking utilities

mysten-network: add common networking utilities

This adds the `mysten-network` crate which can be used to house common
networking tools across mysten repos. In particular this currently
includes:
- A tonic Codec for the bincode format.
- Utilites and configuration for building tonic servers or clients using
  the `multiaddr` format.

Supported multiaddr protocol stacks are:
- dns/tcp/{http,https}
- ip4/tcp/{http,https}
- ip6/tcp/{http,https}
- unix/{http,https}

@arun-koshy You may find this of interest as it transparently handles listening on or connecting to Unix Domain Sockets (uds) for doing grpc over IPC.

@huitseeker it wasn't tonic but an update of ed25519. since we don't have a lockfile checked in the latest semver compatible deps are picked up. I punted on solving the problem by just pinning the ed25519 crate for now.


Perpetually attempts reconnecting on socket error

Updated Issue Description
Okay, I see what is happening. So the client reconnects on the emission of error from socket. It also does so without end. I presume this is by design.
I'm sure there are plenty of cases where that is needed. However, there are also others where it is problematic.
I've been working on replacing http-proxy for a custom reverse proxy I manage. Among the things that proxy does is use proxy errors to signal potential problems with the backend services it manages, as well as do some automated recovery in certain cases.
That being the case having visibility of the inability to reach said services is crucial to the management and recovery of them. It may be that this module is not well suited for that use case, but that is the use case I'm attempting with it atm.
I would propose a solution, but I'm honestly not sure what the best solution would be. I'm going to hack on things locally and see if anything worthwhile comes to mind.
But please let me know your thoughts as I would be happy to do a PR if you have any ideas to manage this without breaking the design philosophy of the module.
Original Issue Description
Request Hangs on No Response
Repro is straight forward. With nothing listening on port 3000 run the following and nothing will happen. The timeout setting has no effect either.
const { Client } = require('undici')
const client = new Client(`http://localhost:3000`)

client.request({ path: '/', method: 'GET' }, (err, data) => console.log(err || data || 'nothing'))

I think we could look at https://www.npmjs.com/package/reconnect-core for some inspiration.

I've seen this as well, it goes into a live lock loop of reconnecting.


Uncaught Exception: listen EADDRINUSE <IP_ADDRESS>:5672

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.2.0
Time: Sun Apr 17 2016 11:54:46 GMT-0600 (Mountain Daylight Time)
Error:
listen EADDRINUSE <IP_ADDRESS>:5672
Error: listen EADDRINUSE <IP_ADDRESS>:5672
    at Object.exports._errnoException (util.js:890:11)
    at exports._exceptionWithHostPort (util.js:913:20)
    at Server._listen2 (net.js:1234:14)
    at listen (net.js:1270:10)
    at net.js:1379:9
    at _combinedTickCallback (internal/process/next_tick.js:77:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #325


Initial pass over implementing Activation Functions.

Just a few functions for now.

So, the category of "activation functions" overlaps with that of elementary functions. I'm not sure of the utility of defining ReLU for a general-purpose numerics library. As for sigmoid, what you've got here is the standard logistic function, which I think would be fantastic as an extension to Real.

Echoing what @xwu said, these probably go in a separate module rather than in Real (which is fine!)
A bigger question is that I'm not sure how useful it is to define these for scalar values rather than contiguous buffers, since the overwhelming use case in ML is on buffers of data (and you often want to fold them into an adjacent operation, but let's ignore that for now).

Well, so should I change anything or maybe close this pr?)


Auto Cancellation issue

When using
pb.collection().getList(1, 25);
and then calling
pb.collection().getList(2, 25);
There is an auto cancellation. This seems like a bug to me, but maybe it is the intended purpose.

It is intended. See https://github.com/pocketbase/js-sdk#auto-cancellation.
The reason for the auto cancellation is that both request will have the same requestKey, which by default is METHOD + path. Check the above links how to specify individual request keys or disable completely the auto cancellation.
To catch auto cancellation errors, you can wrap your calls with try/catch (or use the catch Promise function) and check the err.isAbort prop. It is safe to ignore auto cancellation errors.
try {
    ...
} catch (err) {
    if (!err.isAbort) {
        console.log("non auto cancel error", err);
    }
}


Bugfix/fix compatibilty issues

Changes
Breaking changes

n/a

New / changed / removed

n/a

Improvements

n/a

Code Review
Code Review
What's part of a code review?
Readability & Maintainability

[ ] Code is not more complex than needed
[ ] Clear names are used (good naming practices)
[ ] Test coverage is sufficient and tests are well designed
[ ] Code has proper error handling

Documentation

[ ] Code is properly documented

Security & Performance

[ ] No obvious security flaws
[ ] No notable performance decrease

@patrickjaja Bitte approven dann verschwindet auch die deprecation warning ;)

Thx, @HaukeBST


[google_maps_flutter] marker InfoWindow title text is white when iOS is Dark Mode

google_maps_flutter: ^0.5.21+17

flutter 1.14.6

Marker(
  infoWindow: InfoWindow(
    title: "Luke Pighetti",
  ),
  icon: ...,
  markerId: ...,
  position: ...,
)

This only happens when iPhone 8+ iOS 13.3 is in DarkMode

Duplicate of #40982

Hi @lukepighetti
Thanks for filing the issue!

Light mode
Dark mode

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.5, on macOS 11.2.3 20D91 darwin-x64, locale
    en-GB)
    • Flutter version 2.0.5 at /Users/tahatesser/Code/flutter_stable
    • Framework revision adc687823a (11 days ago), 2021-04-16 09:40:20 -0700
    • Engine revision b09f014e96
    • Dart version 2.12.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.55.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (3 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            • iOS
      14.4.2
    • macOS (desktop)        • macos                     • darwin-x64     •
      macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 90.0.4430.93

• No issues found!

[✓] Flutter (Channel master, 2.3.0-1.0.pre.101, on macOS 11.2.3 20D91
    darwin-x64, locale en-GB)
    • Flutter version 2.3.0-1.0.pre.101 at /Users/tahatesser/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ff2dde2c7f (8 hours ago), 2021-04-26 22:35:45 -0700
    • Engine revision afd9c397a5
    • Dart version 2.14.0 (build 2.14.0-43.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[!] IntelliJ IDEA Community Edition (version unknown)
    ✗ Invalid IntelliJ version number.

[✓] VS Code (version 1.55.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (3 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            • iOS
      14.4.2
    • macOS (desktop)        • macos                     • darwin-x64     •
      macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 90.0.4430.93

! Doctor found issues in 1 category.

As noted above, this is a duplicate; closing.
With current versions of Flutter, I believe that just setting the min iOS version of your app to iOS 9 will be enough to get newer versions of the Maps SDK, so the workarounds discussed in the original should no longer be necessary.


Section 3.42 corrections needed

The entire sentence under the section title appears as a link
The word "ton" should be "on" (after corresponds)

Add a space between section number and title

Fixed


Tests ?

@jairojair we finally started creating tests \o/, I'm closing this issue because it's not specific.
Thanks!

you're amazing! 👍


levelup events not firing

Please close After reviewing the skiff code I see that a call to skiff.levelup() creates a new levelup instance so my event listener was listening on a different instance
Playing around with skiff (using memdown) and I am attempting to get levelup put/del events from a 3 node skiff cluster. i have 1 node making a put request after the leader has been elected and i can see that the data is written. All 3 nodes run the same code on different ports in separate processes with the exception of node 3 calling a put after 2 seconds
pertinent code is
const db = skiff.levelup()
db.on('put', console.log)

I get no console output on any of the 3 nodes. Is there a way to achieve this?
Full code below
peer-list.js
export default [
  '/ip4/<IP_ADDRESS>/tcp/9490',
  '/ip4/<IP_ADDRESS>/tcp/9491',
  '/ip4/<IP_ADDRESS>/tcp/9492'
]

node.js
import Skiff from 'skiff'
import memdown from 'memdown'
import _ from 'lodash'
import peerList from './peer-list'

export default function node (id) {
  let state = null
  let loc = _.first(_.filter(peerList, (peer) => peer.match(new RegExp(`${id}$`)) !== null))
  let peers = _.filter(peerList, (peer) => peer.match(new RegExp(`${id}$`)) === null)

  const options = { db: memdown, peers }
  const skiff = Skiff(loc, options)
  const db = skiff.levelup()

  db.on('put', console.log)

  skiff.start((err) => {
    if (err) return console.error(err)
    console.log('Skiff server started')
  })

  skiff.on('new state', (s) => {
    if (s === state) return
    state = s
    console.log(`Became a ${s}`)
  })

  return skiff
}

node0.js
require('babel-register')
var node = require('./node').default
var skiff = node(0)
var db = skiff.levelup()

node1.js
require('babel-register')
var node = require('./node').default
var skiff = node(1)
var db = skiff.levelup()

node2.js
require('babel-register')
var node = require('./node').default
var skiff = node(2)
var db = skiff.levelup()

setTimeout(() => {
  db.put('1', 'test', (err) => {
    if (err) return console.log('put error')
    console.log('put success')
  })
}, 2000)

Yes, this looks like a bug. There needs to be a notification of changes bubbling up from the internal db into this interface.


[930] Fix typo HasingStockStorage to HashingStockStorage.

Description

Fix typo.
Motivation and Context

See #930.
How Has This Been Tested?
<! ---  Put an x in all the boxes that apply: --->

[ ] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[ ] If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[x] Fix bugs
[ ] Add new feature
[ ] Update documentation

Thanks so much!!!


Working with Request Body and Query Params.

I was trying your framework.
Could you provide examples of you access the body for a Post or Put request?
As well getting URL params from a Get request?
Thanks.

Hello @whitebook,
You can access the body of a pull request as you would do with native http Handlers. You can use for example ioutil.ReadAll.
Here is a simple example with a url parameter "to" and the sender's email is passed in the request's body with his password.
package main

import (
	"encoding/json"
	"fmt"
	"net/http"

	"github.com/celrenheit/lion"
	"golang.org/x/net/context"
)

func main() {
	l := lion.Classic()
	l.GetFunc("/send_mail/:to", handler)
	l.PostFunc("/send_mail/:to", handler)
	l.Run()
}

type user struct {
	Email    string `json:"email"`
	Password string `json:"password"`
}

func handler(c context.Context, w http.ResponseWriter, r *http.Request) {
	// Access url params
	to := lion.Param(c, "to")

	// Reading from body and decoding json
	var u user
	defer r.Body.Close()
	json.NewDecoder(r.Body).Decode(&u)
	fmt.Println("Received password:", u.Password)
	fmt.Fprintf(w, "Sending mail to %s from %s", to, u.Email)
}

You can test this using curl:
curl -XPOST -d<EMAIL_ADDRESS>"password": "pass"}' -H "Content-type: application/json"<EMAIL_ADDRESS>
Change -XPOST to -XGET to test out the get request.
Hope this answers your questions.

Thanks. I think that helps.


Parser infinite loop when parsing unsupported match body expression that is surrounded by brackets

This causes the parser to go into an infinite loop:
function main() {
    match foo {
        (++foo) => 1
    }
}

Removing the () around ++foo makes it error:
-----
function main() {
    match foo {
        ++foo => 1
    }
}

-----
Error: ParserError("expected '=>' after pattern case", Span { file_id: 1, start: 42, end: 44 })

Looking into this.

fixed as of 9012330


Add master 'ci.sh'

Currently, pin-project runs several different checks on Azure pipelines, from a standard cargo test to a compiletest-rs test to clippy. It can be very easy to forgot to one run of these locally, or to run one with the incorrect flag (e.g. missing --all).
It would be very helpful if pin-project provided a master ci.sh script to run exactly the same checks done by CI. This would make it easier to test out local changes to pin-project before submitting a PR.

@Aaron1011
Personally, I prefer to open PR as a draft and let CI do those checks but I agree that this is helpful. However, it takes a lot of time to do exactly the same checks that CI does. (FYI: This is a script to run exactly the same checks done by CI.)
So I am thinking of adding the simplified version.
As far as I know, this can detect most cases with the following exceptions.

If you already have nightly rust, clippy, and rustfmt installed and it is not up to date. (this can happen even with the full version of ci.sh)
If you add a new dependency that does not support -Zminimal-versions.
If you use a feature that is not supported by Rust 1.33.

In addition, the following cases will cause an error.

If clippy is not available on the latest nightly.
If rustfmt is not available on your default toolchain. (this can happen even with the full version of ci.sh)

Any thoughts?


[#1294] feat(rust): introduce the unified grpc latency metrics for all requests

What changes were proposed in this pull request?
introduce the unified grpc latency metrics for all requests
Why are the changes needed?
For #1294
Currently, we calculate the request duration for every request in the service codebase,
which is ugly and scattered everywhere.
BTW, the current request duration metrics is limited to critical paths.
So, it's necessary to use the tonic's layer mechanism to summary the grpc latency metrics.
And the reason of no deleting original latency metrics is the difference of each other,
the metrics embedded in the layer will be greater than in the service code,
because request will spend sometime to decoding the message, the diff time is valuable for observability.
Does this PR introduce any user-facing change?
Yes. After applying this patch, the latency metrics of request like P99 could be gotten like this.
histogram_quantile(0.99, sum(rate(grpc_duration_seconds_bucket{path="/rss.common.ShuffleServer/sendShuffleData", job="uniffle-worker"}[2m])) by (le))
How was this patch tested?
Tested in the test env and validated by the promql

@jerqi Please help review this.

Codecov Report

Merging #1295 (232ae73) into master (adafac4) will increase coverage by 1.30%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1295      +/-   ##
============================================
+ Coverage     53.67%   54.97%   +1.30%
+ Complexity     2690     2551     -139
============================================
  Files           403      370      -33
  Lines         23456    19657    -3799
  Branches       1992     1862     -130
============================================
- Hits          12589    10807    -1782
+ Misses        10093     8181    -1912
+ Partials        774      669     -105

see 35 files with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Document the technicalities of Cloud Provider functionality.

Technical documentation needs to be provided to outline how Cloud Providers work and the processes required for a developer to build their own.

Documentation available on github


Different keypoints for source and target dataset

Hello,
Thanks for your interesting research.
I notice that "We unify the MS-COCO keypoint numbers with the MPII finally" in the published article. However, there are 12 same keypoints for both datasets. I wonder that which means do you use for unify?

Yeah, actually we change the COCO annotations for some joints and make new labeled infos for the different types of joints with MPII and ignore another one.

Hello,
Thanks for your interesting research.
I notice that "We unify the MS-COCO keypoint numbers with the MPII finally" in the published article. However, there are 12 same keypoints for both datasets. I wonder that which means do you use for unify?

Thanks for you questions.  Actually, we preprocess the MPII and relabelled the related keypoints, make it keep consistent with the COCO.


Incorrect TRouter handlers order

In the trouter role, the Wait-for-aio handler is declared first, before the ones that enable and start the service. Hence, the Add alfresco-transform-router service task that has all three of them, starts by waiting for the tcp port to be open before enabling or starting the service, which does not happen as the service was not started.
https://github.com/Alfresco/alfresco-ansible-deployment/blob/d43f81f3da137f81650d87b8aee11f7b280aeda2/roles/trouter/handlers/main.yml#L3
For some reason I still have not found out, this worked on previous CentOS systems we used before, probably because the preceding tasks notify the Restart-trouter handler and in those systems that started the service even if not enabled. We are now testing Amazon Linux and in these the behavior changed and the service is not started.
Other roles, like the sync one that also use this same handlers scheme, have them declared in the correct way.
We applied the fix, placing the handler last in the file and tested it successfully.

Hello @dxciberaws, the wait-for-aio handler is checking that the All-In-One transformer service is up and running, not the Transform Router.
When you run the acs.yml playbook, transformers role is being applied before the trouter role, so it should never happen that the wait-for-aio fails because the service has not started yet.
If you are seeing a different behavior on Amazon Linux maybe that's related to another issue/possible bug.
Also keep in mind that despite Amazon Linux is a supported Alfresco platform, playbooks doesn't.
Anyway we could still consider PRs in case you find and fix some issues.
Thanks anyway for taking time to report this.


Add support for EKS clusters

Tests TBD
Fixes #82

@sashman I had a quick chat with @aliscott and we think it's best to follow instance.go and call the relevant method from the eks_node_group.go file to add them as costComponents or subResources as the instance.go's NewInstance method does. The following are needed:
computeCostComponent with tenancy=shared
cpuCreditsCostComponent should be called in case it returns any required costComponents
call ebsVolumeCostComponents with name `root_block_device.0` and the default size of 20GB or user-specified diskSize
for the launch_template, call newLaunchTemplate

Feel free to ping me on Discord if you want to chat over it.


Add interpolation options to scale

transforms/image/resize can select interpolation options, but transforms/image/scale can't.
So I add interpolation options to scale.

Thank you very much!
LGTM


Symbolic Link folders are appearing as empty files

Method

Create your Django Project

Use git to clone your code NOT into the Django project folder

Use ln -s to symbolic link the Django Application folder from the git source into the Django Project folder

Open up Onivim 2

Expected Results
The symbolic linked folders to act like folders
Actual Results
Acts like a blank file
Image of the folders on Finder (mac) - please note the symbolic link for the NearBeach folder

Image of Onivim - please note the lack of folder support for the NearBeach folder

Image of the opened symbolic link folder

Note
I have only tried this on Mac. I am more than happy to also try this out on Linux.

That fits with what I see in the code:
https://github.com/onivim/oni2/blob/28e55c7ab77e9d629ad7698f73c1a90c88444c1a/src/Components/FileExplorer/Component_FileExplorer.re#L23-L32
I.e. when kind is a Link its treated as a file, not as a dir (well....there should be a lookup for the sort of link I guess).
That should probably be updated to check what the link actually resolves to.


Webhook for incoming calls Sample

Hello
Thanks for your great work!
Is there any sample how to implement [ Webhook for incoming calls ] ( @BotCallingWebhook("/api/calling") ) correctly?
How i transfer a call to another agent?
Best Greetings and thnaks!

Hi @cyril-iselin - the documentation is available here https://github.com/wictorwilen/express-msteams-host#adding-support-for-calling-in-bots and then you have to implement as described in https://docs.microsoft.com/en-us/microsoftteams/platform/bots/calls-and-meetings/call-notifications.
On the question to transfer to an agent that question should be asked in the bot framework communities/repos

Thanks for your answer.
The documentation is really bad and I can't find any implementation for this method.
There are some c# implementations but it looks like it works completly different.

Yes, agree, it is not well documented and when I initially did this I had to reverse engineer every single command. And, the SDK available is only for .NET on Windows machines. Also a reason that this feature has not received that much love in the generator.
However, there might be changes to this area, not necessarily in this specific endpoint.
Closing this.


Fix json form in UpdateInFolder

change the json payload that in the UpdateInFolder function
from
{
    "credentials":{
        "description":"",
        "DisplayName":"",
        "Fingerprint":null,
        "FullName":"",
        "id":"gitee",
        "TypeName":"",
        "$class":"com.cloudbees.plugins.credentials.impl.UsernamePasswordCredentialsImpl",
        "stapler-class":"com.cloudbees.plugins.credentials.impl.UsernamePasswordCredentialsImpl",
        "scope":"GLOBAL",
        "username":"chengleqi",
        "password":"xxx"
    }
}

to
{
    "description":"",
    "DisplayName":"",
    "Fingerprint":null,
    "FullName":"",
    "id":"gitee",
    "TypeName":"",
    "$class":"com.cloudbees.plugins.credentials.impl.UsernamePasswordCredentialsImpl",
    "stapler-class":"com.cloudbees.plugins.credentials.impl.UsernamePasswordCredentialsImpl",
    "scope":"GLOBAL",
    "username":"chengleqi",
    "password":"xxx"
}

Codecov Report
Base: 72.90% // Head: 72.90% // No change to project coverage :thumbsup:

Coverage data is based on head (c8ab4be) compared to base (aaf3f39).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #58   +/-   ##
=======================================
  Coverage   72.90%   72.90%
=======================================
  Files          41       41
  Lines        2196     2196
=======================================
  Hits         1601     1601
  Misses        529      529
  Partials       66       66

Flag
Coverage Δ

unittests
72.90% <100.00%> (ø)

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

Impacted Files
Coverage Δ

pkg/util/url.go
100.00% <ø> (ø)

pkg/credential/credentials.go
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


"Runtime Analysis panel" should display a tree

We need a written specification for this feature

The overall structure of the tree is as follows:
<Project name>
├── Findings
│   └── <Date bucket>
│       └── <Impact domain>
│           └── <Finding>
└── Failed tests
    └── <Failed test AppMap>

Date buckets are split as follows:

Last 24 hours
Last 7 days
Last 30 days
More than 30 days ago
No date indicated

The date of a finding is found via the eventsModifiedDate property or scopeModifiedDate if the former is falsy. If both properties are falsy, fall back to No date indicated.
A "failed test" is an AppMap where the metadata.test_status property is set to failed.

@dustinbyrne How's ""  defined?

@dustinbyrne VSCode displays a "Findings Table" node. Should it be always shown?

@dustinbyrne I noticed that VSCode displays an additional group inside "Failed tests". Should I implement this, too? If I should, how's that group name found?

@dustinbyrne If an Finding has "test_status: failed", should it be shown under both "Findings" and also "Failed tests" or only in the latter? VSCode seems to show it in both groups.

If a test has failed, it's optional for Findings on that AppMap to appear. It's not a big deal if it's in both places though.

:tada: This issue has been resolved in version 0.46.0 :tada:
The release is available on:

v0.46.0
GitHub release

Your semantic-release bot :package::rocket:


Ellipse nodes aren't expanded

To reproduce add the following style:
{
    selector: "[expanded-collapsed='collapsed']",
    style: {
        "background-color": "darkblue",
        "shape": "ellipse",
        "width": 100,
    }
}

Looks like this is because we draw the expand-collapse cues at the upper left corner, which happens to be outside node boundaries in case of ellipses! If you must use ellipses, you might want to consider putting these cues inside the node.
@metincansiper @mrsfy Can one position these ques say "towards" upper left (a certain offset away from upper left) using published API of this extension?

Yes there is 'expandCollapseCuePosition' option for this purpose. However, I realized that this feature was buggy and I fixed the related bug. It must be working well in version '1.1.2'.
For example if you want the cues to be rendered at the node centers it can be used in the following way:
  expandCollapseCuePosition: function(ele) {
        return ele.position()
  }

In that case, I consider this issue invalid.


[Enhancement] Graphics Settings

Godot version:

OS/device including version:

Issue description:

Steps to reproduce:
Minimal reproduction project:

Woops, didn't mean to open this. Closing.


[Bug] Can't finish the build process on windows

🐛 Bug
When I try to build using mlc_llm package cmd on windows I am facing an error:

To Reproduce
Create all env variables as explained here https://llm.mlc.ai/docs/deploy/android.html
and run this mlc_llm package in terminal
Environment

Windows 11 Pro
MLC-LLM cloned using git clone https://github.com/mlc-ai/mlc-llm.git
TVM-Unity installed using python -m pip install --pre -U -f https://mlc.ai/wheels mlc-ai-nightly
Python 3.11.1

the error was due to lack of cmake dependency, can you install cmake via conda?

It seems I have the same result.

I installed git, git lfs, cmake from anaconda, also re did all the steps and built using the cmd prompt from anaconda.

Thank you for your feedback. I did that but it's failing now somewhere else.
I am going to try on a linux machine.

Facing the same issue here.

I'm not sure what I'm doing any wrong, any help would be greatly appreciated.

https://llm.mlc.ai/docs/deploy/android.html#additional-guides-for-windows-users
Please try the above tips

what is that file that was not found i am trying from 4 days to package the compiled model the model is being compeled but its not packaging its giving error file not found can you please share what was that file or error??

in this case, seems you didn't specify TVM_NDK_CC env variable to the right compiler

In this case it was looking for llvm-ar, please check if you have llvm-ar in that path

yeah llvm-ar is available in same path though its not working

Thanks @Nikhil34712 , do you mind go into  file indicated, ndk.py as indicated in the error message and print out the cmd. See there is a subprocess.Popen, you can try to print out cmd part and see what it tries to do. Then it should give hints about why it fails

Hey @tqchen thanks fro replying but do mlc_llm supports MiniCPM-V when i am compiling the model or converting the weights  its showing that you have to choose only these models  i have attached the files can you please address this??


Hardcode the repo URL for tests

Currently the tests don't pass locally for forks of this project, as the repository URL is pointing elsewhere than github.com/timothycrosley/portrait.
This PR hardcodes this location in tests/test_config.py. @timothycrosley this might help pull in future contributors down the line, too! 💪🏻
We lose coverage on the git.Repo.remotes.origin.url line (L163 in portray/config.py), but since that's an upstream package I think that's fine.
Fixes #66

Thank you!

Thank you!


Add size_of annotation to help CBMC's allocator

Description of changes:
CBMC's heap allocator will produce objects that aren't just a byte array when a prior call to size_of was involved in computing the number of bytes to be allocated via malloc/calloc/realloc. This, in turn, permits type-safe member accesses to heap-allocated objects. This should speed up copying as well as all other operations on the resulting object (which will then not have to go through byte extract/byte update operations).
It also seems that there no longer is a "size_of" intrinsic.
Resolved issues:
Fixes: #1286
Fixes: #1781
Related RFC:
n/a
Call-outs:
n/a
Testing:

How is this change tested? Locally reviewed CBMC's --program-only output on some Vector examples.

Is this a refactor change? No.

Checklist

[x] Each commit message has a non-empty body, explaining why the change was made
n/a Methods or procedures are documented
[x] Regression or unit tests are included, or existing tests cover the modified code
[x] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

With thanks to @karkhaz for making this happen, and with the hope that this isn't just about some GitHub runner anomaly (but at least the numbers stack up with my local experiment):
  s2n-quic/quic/s2n-quic-core/slice::tests::vectored_copy_fuzz_test:
    variants:
      kani_new:
        metrics:
          solver_runtime: 74.86000000000001
          success: true
          symex_runtime: 7.85538
          verification_time: 93.35263
      kani_old:
        metrics:
          solver_runtime: 178.3941
          success: true
          symex_runtime: 7.67614
          verification_time: 196.657

It's looking great, @tautschnig ! Just wondering if you think we should add any more tests to the performance regression or not. Is the vectored_copy_fuzz_test already in there?

Thanks @tautschnig!
One small point. The OP says:

Fixes: #1781

But #1781 seems unrelated?

My apologies, looks like I wrongly inferred this from https://github.com/model-checking/kani/issues/1286#issuecomment-1414676278. I just confirmed that (as expected) #1781 is not fixed.

It's looking great, @tautschnig ! Just wondering if you think we should add any more tests to the performance regression or not. Is vectored_copy_fuzz_test already in there?

That test is already there.


Better error message for modules with lowercase identifiers

fixes #1974

Yeah no need to say that "foo" isn't uppercased.
Thanks!

I'm not sure to what extend we should encode all those error cases into the grammar. There's the cost of tampering with the grammar that might result into shift/reduce conflicts in the future. For example, LIDENT is a common for core_types/mod_types. Some errors might be better positioned in the error message table to avoid needless grammar complexity.

Should we revert this one then?


Write helper class

This is currently done by sepicfying a dumpAll then dumpCertain onto the same file.
This is trivial but should be wrapped up into one method for ease
Im not sure if that code should be here or with branchbuilder.

Addressed in 1.1.0.


Problem with install to other folder

i have my package
{     "name": "ab/abc",     "description": "Desc of abc,     "type": "package",     "authors": [         {             "name": "Jacek",             "email"<EMAIL_ADDRESS>       }     ] }
and my composer.json file:
{   "repositories": [     {       "type": "vcs",       "url": "git@bitbucket.org:ab/abc.git"     }   ],   "prefer-stable": false,   "require": {     "mnsami/composer-custom-directory-installer": "*",     "ab/abc": "v1.0.8"   },   "extra": {     "installer-paths": {       "./smothing/": ["ab/abc"]     }   } }
. When i make a composer install i have a this package in vendor/ab/abc. Why? I would have a this package in root folder next to a composer.json
Best Regards.
Jacek

Hi @jbartoszek
I'm sorry for the late reply, but unfortunately repositories are not handled by the composer installers, they are handled in a different complete manner using Repository Download manager, and unfortunately composer doesn't provide a way to override the behavior of those.
if you have an idea, I'm open to talk about it.

Reporter is not responding, I'm closing the ticket.


Software testing cheatsheet not found

Software testing cheatsheet showing 404 error

I'm just made a Pull Request which fixes this problem. I'm waiting for it to be merged.
https://github.com/detailyang/awesome-cheatsheet/pull/136


Feature: Admin view for searching users

This PR needs to update to the new web codebase and then implement and merge the rest.

I'm not planning to add user deletion here. I now claim this branch as feature-complete and ready for review.


fetch is not defined on 3.1.0

We updated to 3.1.0 and now we have the following error:
::error::fetch is not defined

@freshpr where exactly did you see the error? i updated to 3.1.0 and the pipeline is failing but i get no logs or information about why it failed, even in debug mode

Maybe running a newer node version helps. You could fork this repo, then change the bottom lines in action.yml to
runs:
  using: 'node22'
  main: 'index.js'

Then reference the new forked version in your github workflow uses: <org-name>/gh-upload-sbom@<branch>
Unfortunately I cannot test this myself, but maybe it helps.

Sorry for the late reply.
I don't remember. We just use 3.0.0