'Griptape Display : Data as Text' does not work

Hi, and thanks for your work on this suite of nodes!
Maybe it’s just an issue with my setup (latest version of Comfy via Stability Matrix, Win10), but I’ve noticed that the Griptape node 'Display Data as Text' returns an error (tested only with the Ollama agent) regardless of the desired output data type. It seems that the wildcard "*" on the input is partly to blame.
Locally, I partially resolved the issue by modifying the node's code and adding the following class method, as indicated in the documentation (https://docs.comfy.org/essentials/custom_node_more_on_inputs#wildcard-inputs):
@classmethod
def VALIDATE_INPUTS(cls, input_types):
    return True

This solved the issue for JSON types CONFIG and MEMORY, but not for others types. I assume (wrongly ?) that Comfy's code has secured the STRING type, and it may no longer be possible to pass complex types by casting them with str(INPUT) ? But I haven't had time to investigate further. Maybe someone smarter than me got the answer ? ;)

Thank you for the node - I'll check it out! I know that ComfyUI has been updated quite a bit lately & I figured that would have an impact on the nodes. I'll update and check it out.

Great, thanks! I'm also looking into it on my end. I think I'll encounter the same issue with my own custom nodes when I test them on the new release. Have a nice day!

I added the fix you recommended & also updated my code to display data in the node so it can handle just plain strings.. do you have an example I could try where it wasn't working correctly?

Please check it out and let me know if it works for you :)

Hello Jason! Sorry, I'm a bit late. I've just completed the update, and I can confirm that the DataAsText node is now working perfectly. As for the workflow, I would be hard-pressed to provide a use case right away, as I am still in the discovery phase regarding Griptape. However, I am already starting to find some uses.
In the near future, I plan to use it to generate graphic files in vector format to assist my own nodes. It may not sound very exciting to others ... but for me, oh hell it already is! ;)
Once again, a big thank you.

@Hullabalo glad to hear it works for you! :)
would love to see how it works out for the graphic files in vector format - keep us informed with how it's going! :)
cheers,
Jason

Haha, I think I'm going to have to work hard to make LLaMA 3 understand that a five-pointed star definitely needs to have five points, not four. But I guess I should learn to write understandable rulesets first before I can teach anything to an LLM.
Of course, I'll be sure to share my progress here and elsewhere, you can count on that ! Speaking of elsewhere, I'm surprised there's not more hype around ComfyUI-Griptape here: https://www.reddit.com/r/comfyui/search/?q=griptape. It would be well-deserved !
All the best.

It'd be awesome if you wanted to add to that hype a bit more.. always feels
better coming from the community. :)
On Thu, Aug 29, 2024 at 7:57 AM Hullabaloo @.***> wrote:

Haha, I think I'm going to have to work hard to make LLaMA 3 understand
that a five-pointed star definitely needs to have five points, not four.
But I guess I should learn to write understandable rulesets first before I
can teach anything to an LLM.
Of course, I'll be sure to share my progress here and elsewhere, you can
count on that ! Speaking of elsewhere, I'm surprised there's not more hype
around ComfyUI-Griptape here:
https://www.reddit.com/r/comfyui/search/?q=griptape. It would be
well-deserved !
All the best.
—
Reply to this email directly, view it on GitHub
https://github.com/griptape-ai/ComfyUI-Griptape/issues/108#issuecomment-2316142680,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJFWYL5GGUAZYOJS4ONJGLZTYTRZAVCNFSM6AAAAABNCUXLOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJWGE2DENRYGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Run on lambda

This is an addition to #83, which I needed to run on AWS lambda.

Apologies @tomwhite, you did mention this, I forgot. Thanks for submitting.

Thanks @alimanfoo.


“Send” screen is open even though scanning the QR code from Request screen

Frequency: 100%
** Repro App version:** Celo integration build v1.8.2 (35)
Repro on device: iphone 6+ (12.4.5)
Repro Steps:

Launch app and sign up in the app
Tap on Request button.
Tap on QR code icon.
Tap on Scan code button.
Scan the other user’s QR code.
Observe

**Current Behavior:**Send Screen is open instead Request screen
Expected Behavior: Request screen should be shown
Attachment:RequestscreenIssue..mp4

Verified with IOS Test flight Celo interation build v1.9.1 (41) and Android Celo internal build 1.9.1 and observe issue is still reproduced
Attachment:RequestscreenIssue.mp4

@annakaz Verified with latest valora app v1.0.0 (10) and found that issue is still reproduced.
Devices:  Google Pixel XL (8.1), Samsung Galaxy S4 (4.4), Samsung galaxy grand prime (5.1.1)

I have verified this issue with latest valora app v1.0.0 (14) &  IOS test flight build v1.0.0 (15) and found that issue is still reproduced.
Devices: iPhone 6(12.4.7),Samsung Galaxy A5(7.0)

I have verified this issue using using Android internal build v1.1.0<PHONE_NUMBER>)& iOS test flight build v1.1.0(21) and found issue is fixed
Device : iPhone 6+ (12.4.5), Google pixel Xl 8.1


CLOUD-907 added instance status and updated instance removing

@doktoric

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1561/


storageEndpointSuffix is hardcoded in UnmanagedDataDiskImpl thus restricting user to create unmanaged disk in other environment

Issue: While creating blob uri, storageEndpointSuffix should be taken from environment. But in 'UnmanagedDataDiskImpl.java:214', storageEndpointSuffix is hardcoded as '.blob.core.windows.net'.
Impact: I am not able to create unmanaged disk in US gov region and managed disk is also not supported. So I am unable to launch the virtual machine in US gov region using java sdk 1.1.0

@rishi-anand we fixed this issue in the PR https://github.com/Azure/azure-sdk-for-java/pull/1759, this will be available in the next release [scheduled for August].

Thanks @anuchandy :)

Fix available as of v1.2 hence closing this.

Will this fix will be part of 1.2.0 release?


Check on input forms

There is not check or sanitization on the input boxes, but just on the URL field.
When just the URL is inserted a message says:
Abbiamo inviato il link con cui confermare l'iscrizione tramite email all'indirizzo  with the empty remaining fields.
Expected behaviour
The app should check whether all the fields have been inserted and sanitize the inputs before going on.

The email is sent even without a proper name.

This is because of this "limitation".

Fixed by #48, closing.


endofsession parameter is not sent by Vas2Nets

We do need to send it back though. There is no signalling from their side to us for when a user disconnects. 60 seconds of inactivity means a session's gone away.

@rudigiesler ready for review

:+1: from me

:+1:, looks good.


Column selection in Ubuntu

In Ubuntu, (Shift + ) Alt + Click drags a window. This conflicts with column selection. Perhaps the default key binding should be different in Linux?

Does the "editor.multiCursorModifier" setting help? You can change it from alt to ctrl. Otherwise, https://github.com/Microsoft/vscode/issues/3130 for customizing mouse shortcuts.

@robertohuertasm  Perhaps the default key binding should be different in Ubuntu?


slow startup

prof-FIAn3jea.renderer.cpuprofile.txt
prof-FIAn3jea.main.cpuprofile.txt
my vscode is really slow to start.
I have already tried to uninstall/re-install it multiple times but it never worked.
I have also tried to uninstall all extension i had but the problem is still there.

I attached some files that might help solving this, i dont know how to read them to be honest.
Thanks for the help

This is the delayed app ready event again

I re-installed Windows and the problem is now fixed. Sorry for the useless issue post.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Prep February 2024 release

Let's begin releasing this package at the end of each month.
February 2024 Release Prep Checklist

[x] Create milestone for release
[x] Assign issues to the milestone
[x] Test release
[ ] Update CHANGELOG.md and create a new unreleased section: @willingc
[x] Release to PyPI (for now manually; optimistically automated using hatch)
[x] Test PyPI release installation
[ ] Release 0.2.3 to PyPI: @lwasser
[ ] Announce 0.2.3 on our Slack/Discourse: @lwasser
[ ] Close Feb 2024 milestone: @willingc
[ ] Open March 2024 milestone with a due date 2024-03-31: @willingc

@willingc test pypi has pyosmeta on it now too! however hugo's post on test pypi being difficult as a test bed is accurate. essentially it will only install things on test pypi which means that many dependencies might be super dated and break things when you try to install the package from there. in our case pydantic is a version from 2020 (assuming it's the right pydantic). should we just remove the test pypi build component al together? or what do you think is best?
the good news is releases work. i do want to update our release file but that is a small thing that we could do in march if we want to test drive a release this week.

Awesome @lwasser. My recommendation would be to create a release this week. I'm happy to update the Changelog for release.
I think we can move away from test pypi. We can noodle over the best way to test release build without using testpypi. We should be using a recent version of pydantic. All of this can wait until March.
Let's call the February release 0.2.3. I will open a PR for changelog. After that lands, you can do a release to PyPI.

Sounds good to me @willingc !!! 🙌✨


feat: Creating .d.ts Files from .js files

for isssue #12 Create TS definitions #12

Creating .d.ts Files from .js files
maybe, should changed create-distribution.js : (

Looks good. Yes the source ts definitions will need to be in src.
And create-distribution.js will need updating to copy them over to dist on prepublishOnly.
That would be ideal as dist is disposable.

Merging into feature/ts-definitions


jcenter shut down soon

Environment

don't matter
Platforms

Android
Versions

Android: don't matter
iOS: don't matter
react-native-netinfo: don't matter
react-native: don't matter
react: don't matter

Description

Reproducible Demo

Just start app and u will see warning in console
WARNING:: Please remove usages of jcenter() Maven repository from your build scripts and migrate your build to other Maven repositories.
This repository is deprecated and it will be shut down in the future.
See http://developer.android.com/r/tools/jcenter-end-of-service for more information.
Currently detected usages in: project ':react-native-webview'

"Fake news" - check the updates here: https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/
That said, removing it is definitely the right way to go! I've reviewed your PR already and I really appreciate the work


The example split-chart can't work in web.

I run the example split-chart with command trunk serve, but the url <IP_ADDRESS>:8080 show nothing. But I run the example from iced tour, it works fine. So I thought is maybe problem about canvas or something.

After I delete the method subscription, everything works fine.


Feature/implement services

ISSUE #667
ISSUE #691
Disconnect Repositories from Controllers

Implement Data Object Transfert for each Entities to work with Controllers
Implement Services that make connections between Repository Entitie and Controller DTO
Implement unitary tests for new services
Refactoring controllers tests
Reorganise source tree directory

Hello @arey did you see my implementation of #667 #691 and refactoring?
I thing sources tree is better like this with the separation of Model, DTO, Services and Controllers.

Thanks for all the ideas. I would say this PR is too large really to consider as one change. It also goes against some decisions we made quite a long time ago (e.g. refactoring to vertical slices: 83ff9a50, and not using DTOs to keep the codebase as small as possible). So, while there might be some useful bits in there, it might be better to extract them and discuss first with the community.


[Iteration 6] v4.0-beta

@Jeckerson Any idea what happens with Sqlite?

@sergeyklay Yes, Phinx migrations add prefix .sqlite3 during migrations which cause problem to connect to the DB inside DB Provider.
Example:
Inside .env file DB_NAME=/tmp/test.sqlite
Which, after migrations, will generate file /tmp/test.sqlite.sqlite3


Rename SparseQArray to SparseDIAQArray

We are not planning to implement another format for now so there is no need to specify this I think

Yes, but if we do implement another sparse format, it's easier to make this change now than to have to do it later on (might break the API).


字段是时间类型怎么处理？

字段的映射之后，存入mongo库中是字符串，这是要怎么处理？

@1075132461 Please write in English.
WRT your question, you could implement your own DocumentSerializer that converts your values into java.util.Date.


KMP Support

We could make this a KMP library, thoughts?

No, it can't. KMP doesn't support JNI native code.

use of JNI for all the java targets and cInterop/cklib for Native Targets ?


[Improvement] 1.3.9 failed to upgrade version 2.0.0

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
18:08:57.630 [main] ERROR org.apache.dolphinscheduler.dao.upgrade.UpgradeDao - json split error
java.lang.NullPointerException: null
at org.apache.dolphinscheduler.dao.upgrade.UpgradeDao.splitProcessDefinitionJson(UpgradeDao.java:678)
at org.apache.dolphinscheduler.dao.upgrade.UpgradeDao.processDefinitionJsonSplit(UpgradeDao.java:607)
at org.apache.dolphinscheduler.dao.upgrade.UpgradeDao.upgradeDolphinSchedulerTo200(UpgradeDao.java:322)
at org.apache.dolphinscheduler.dao.upgrade.DolphinSchedulerManager.upgradeDolphinScheduler(DolphinSchedulerManager.java:122)
at org.apache.dolphinscheduler.dao.upgrade.shell.UpgradeDolphinScheduler.main(UpgradeDolphinScheduler.java:36)
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@SbloodyS 辛苦帮忙看一下，1.3.9 升级2.0 问题


allow to dynamically generate stories / args

Example of how this may look:
<template>
  <Stories>
    <Story
      v-for="(argument, key) of arguments"
      :title="'Story ' + key"
    >
      <MyComponent :argument="argument" />
    </Story>
  </Stories>
</template>
<script lang="ts" setup>
import MyComponent from './MyComponent.vue'

const arguments = ["hello", "world"]
</script>

This is currently blocked by https://github.com/storybookjs/storybook/issues/9828.
Once this is implemented, add https://storybook.js.org/docs/vue/writing-stories/introduction#using-args as example.

Yes, the new indexer api is nice - at least in principle. One still has to implement the transpiling to csf: https://github.com/storybookjs/storybook/blob/next/docs/api/main-config-indexers.md#transpiling-to-csf
And I'm not sure how to hook into vue's compiling to extract the for-loop. The obvious way via node transformers doesn't seem to work as vue's for-loop transformer is always executed first, by design: https://github.com/vuejs/core/issues/4248.
So maybe we have to first replace the v-for by a different directive, say storybook-for and then reimplement the v-for transformer in our setting.

Yes, the new indexer api is nice - at least in principle. One still has to implement the transpiling to csf: https://github.com/storybookjs/storybook/blob/next/docs/api/main-config-indexers.md#transpiling-to-csf And I'm not sure how to hook into vue's compiling to extract the for-loop. The obvious way via node transformers doesn't seem to work as vue's for-loop transformer is always executed first, by design: vuejs/core#4248. So maybe we have to first replace the v-for by a different directive, say storybook-for and then reimplement the v-for transformer in our setting.

That's a very handy link for the API docs, thank you @tobiasdiez.
I guess my point is whether this type of feature would really be in the scope of this addon.
If you are dynamically generating stories, the developer's experience of writing Vue SFC might not be the first thing you need.
Obviously, if this can be supported without much effort, would be great, but it seems a non-trivial use case for which Storybook already provides a valid alternative.
The risk I see is to manage (and maintain) yet another level of pre-processing of the SFC file which might conflict with the future transform we will put in place to support the most advanced features.
I suppose this is something we can revisit at some point in the future.


Limit MediaRecorder size

Some devices can only record up to 1080p. See https://github.com/JakeWharton/Telecine/pull/15.

To summarize:

MediaRecorderuses resolution from the camera instead of the screen (probably).
CamcorderProfile can be used to fetch the device's maximum quality (QUALITY_HIGH).

See a possible solution with #19.


[sort comp] error when defining static async before lifecycle methods

we are using eslint-plugin-react in a next.js project.
common pattern is:
class Comp extends React.Component {
  static async getInitialProps() {
    ...
  }
  componentDidMount() {
    ...
  }
}

since last update 7.9.1 you get a sort-comp error complaining that the static method should be placed after componentDidMount. Was that intended? Looks like a bug to me.
this is maybe related to #1808

I don’t think it’s related to #1808.
The default ordering has statics before lifecycles: https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/sort-comp.md#rule-details so this seems like a bug.

I could not reproduce the bug with the default configuration.
@maerzhase Can you share your rule configuration ?

thats the rule config
  rules: {
    'react/jsx-filename-extension': [
      'warn',
      {
        extensions: [
          '.js',
          '.jsx',
        ],
      },
    ],
    'no-multi-spaces': [
      'error',
      {
        exceptions: {
          ImportDeclaration: true,
          VariableDeclarator: true,
        },
      },
    ],
    'no-underscore-dangle': 'off',
    'react/forbid-prop-types': 'off',
    'react/no-unused-state': 'warn',
    'no-param-reassign': [
      'error',
      {
        props: false,
      },
    ],
    'class-methods-use-this': 'off',
    'max-len': 'off',
    'jsx-a11y/no-static-element-interactions': 'off',
    'react/no-unused-prop-types': 'off',
  },

i will dig inside a bit later aswell. i thought it was directly related to the last update. because freezing to 7.9.0 makes the error disappear. but of course it could be related to a combination with eslint-plugin-import and eslint-plugin-jsx-a11y

@maerzhase what's your entire config tho? specifically, things like parser and parserOptions and settings

from my current test i can say that its related to add the async to the static function.
i will setup a test repo to reproduce the problem.

ok. i could reproduce it in a clean project but it seems not to be related to eslint-plugin-react but the airbnb config. the error was not thrown until i added the airbnb config to the project. so i will close the issue and checkout whats going on with the airbnb config.
thanks anyway!

The airbnb config doesn’t allow async functions.

thanks for that!

@maerzhase did you ever figure this out? I'm hitting the same problem.

tbh we are no longer using airbnb config because we didn’t like some rules and use our own custome config for projects. are you using airbnb config aswell?


Unresolved dependencies on chisel, firrtl interpreter

I just cloned chisel-testers and tried to run sbt and got the following errors:
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: edu.berkeley.cs#chisel3_2.11;3.1-SNAPSHOT: not found
[warn] 	:: edu.berkeley.cs#firrtl-interpreter_2.11;1.1-SNAPSHOT: not found
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn]
[warn] 	Note: Unresolved dependencies path:
[warn] 		edu.berkeley.cs:chisel3_2.11:3.1-SNAPSHOT (/scratch/celio/chisel-testers/build.sbt#L15-18)
[warn] 		  +- edu.berkeley.cs:chisel-iotesters_2.11:1.2-SNAPSHOT
[warn] 		edu.berkeley.cs:firrtl-interpreter_2.11:1.1-SNAPSHOT (/scratch/celio/chisel-testers/build.sbt#L15-18)
[warn] 		  +- edu.berkeley.cs:chisel-iotesters_2.11:1.2-SNAPSHOT
[trace] Stack trace suppressed: run last *:update for the full output.
[error] (*:update) sbt.ResolveException: unresolved dependency: edu.berkeley.cs#chisel3_2.11;3.1-SNAPSHOT: not found
[error] unresolved dependency: edu.berkeley.cs#firrtl-interpreter_2.11;1.1-SNAPSHOT: not found

Relatedly, I'm trying to follow (https://github.com/ucb-bar/chisel-testers/wiki/Using the PeekPokeTester) but the > test-only examples.GCDSpec throws an invalid input error.

3.1-SNAPSHOT seems like the master branch of Chisel. You'll need to compile Chisel3 master from source and publish-local to use that branch. See https://github.com/ucb-bar/chisel3/wiki/Frequently-Asked-Questions#what-is-the-difference-between-release-and-master-branches


As a user, I want to add answers for categories

As a user, I want to open the details of a category, then see all my previous answers on that category and be able to add more. (answer will be a string)

So I was trying to move this forward a bit and I think I hit a snag. We created answers as belonging to the context of a board, but shouldn't they belong to the context of a category, instead?


Add basic rebase command

This command needs to effectively do an interactive rebase for the stack of patches so that you can reorder, squash, mixup, etc - iteratively rework them - into the final state that you want.

This has been integrated into mainline.


Failed to set choices for cdd file

Hello, there was an issue while using version 39.4.5.
In my cdd file, the following content is ok:
<TEXTMAP s='0' e='0'>

But this scenario is an error:
<TEXTMAP s='(0,0,0,0,31,127,182,60,0,0,174,1,255,38,0,12,18,6,220,20,8,30,214,0)' e='(0,0,0,0,31,127,182,60,0,0,174,1,255,38,0,12,18,6,220,20,8,30,214,0)'>

I have checked the source code and it is the following logic that caused the error:
for choice in data_type.findall('TEXTMAP'):
        start = int(choice.attrib['s'].strip('()'))
        end = int(choice.attrib['e'].strip('()'))

May I ask if my file is not standard or if the program is not compatible.
Thank you.

cantools is quite strongly aimed at regular signal based communication, not diagnostics (nowadays I consider the diagnostics code in cantools as technical debt a  which cannot be easily removed due to the trouble it would cause by breaking user code). If you want to do some serious diagnosing, I would strongly advice to convert your CDD file to ODX using the ODX-studio plugin for CANdela and use the resulting file with odxtools

cantools 主要是针对常规信号通信，而不是诊断（如今我认为 cantools 中的诊断代码是一种技术债务，由于破坏用户代码会造成麻烦，因此无法轻易删除）。如果您想进行一些严肃的诊断，我强烈建议使用 CANdela 的 ODX-studio 插件将您的 CDD 文件转换为 ODX，然后将生成的文件与 odxtools 一起使用

Thank you for your suggestion.


There is a small bug

On line 182 in response_generator.py, the elif statement causes one functionality to fail. changing the condition to tag == "negative_feedback" would help fix that bug. This is because of the way it is defined in the dataset.json file, where the intent is named negative_feedback.

Thanks @preethamcoder , fixed it :)


Bug fix in binding parameters of composed circuit

Summary
Details and comments
Fixes #809

@kt474 - please have a look at this PR and related issue, although it is still a draft. The purpose of this PR is to fix the bug in #809. This indeed fixes the bug, however, I am not sure where is the best place to put this fix. I saw that you removed a decompose in PR #780. This is probably related. However, if I try to insert a decompose where it was removed, it causes a few tests to fail.

When I add  obj.decompose(reps=2) in qiskit_ibm_runtime/utils/json.py, the test test_coder_operators fails, probably because it fails in comparing input to output, syntactically.
Please advise whether to add back the decompose and where this should be added, or if you have any other idea. I don't think this harms performance, because if the circuit is already decomposed to basis gates, it will do nothing. @jyu00 , @kt474 - your inputs will be appreciated .

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3414

Relevant Lines:
5082

💛  - Coveralls

We removed the decompose in #780 because it was originally added as a bug fix but has been fixed on the terra side. I'm not sure what the best fix for #815 would be - thoughts @jyu00? Should this be fixed on our side?

Closing for now, this will be fixed on the primitives side - the issue is https://github.com/Qiskit/qiskit-ibm-runtime/issues/809 for reference


Channel Recovery: ChanStatusLocalDataLoss, needs recovery

Background
Long story short: Data loss (lnd datadir gone for good) led to one channel left that I was not able to coop close in time until lnd process finally died after running without underlying datadir for some hours. This channel is still pending in limbo. channel.db now is not available anymore.
Your environment

0.14.3-beta.rc1
Linux 5.13.0-39-generic
bitcoind 22.0

Steps to reproduce

start lnd
delete whole datadir on accident
try to recover files (channel.db, wallet.db, wtclient.db, sphinxreplay.db) out of running process sudo tail -c -f +0 /proc/<pid_of_lnd_process>/fd/<number_of_file_in_proc> > newfile (sudo cp works also, see link)
check if recovered channel.db is ok by compacting copy if channel.db with chantools -> fails "buckets are nil"
coop close channels while you can (zap app had credentials on its own) and recover funds meanwhile
unresponsive channel left
lnd process finally dies (end of journal? memory refresh? not sure)
channel.db is gone now at this state
recreate datadir structure with recovered files
restart lnd to recreate tls.cert credentials
force close left channels with SCB file

Expected behaviour
Channel force closes. Funds return.
A built-in option to replicate channel.db to another place (like cln) would be of help in some of these cases.
Actual behaviour
Channel cannot be force closed. Chan status: ChanStatusLocalDataLoss|ChanStatusRestored
Peer is @xmrk-btc (if you see this, sorry for this inconvenience!)
Logs:
lnd[29386]: 2022-04-17 09:42:39.550 [INF] SRVR: Established connection to: <peer>
lnd[29386]: 2022-04-17 09:42:39.550 [INF] SRVR: Finalizing connection to <peer>, inbound=false
lnd[29386]: 2022-04-17 09:42:40.080 [INF] PEER: NodeKey(<peer>) loading ChannelPoint(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1)
lnd[29386]: 2022-04-17 09:42:40.080 [WRN] PEER: Unable to find our forwarding policy for channel 064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1, using default values
lnd[29386]: 2022-04-17 09:42:40.081 [INF] HSWC: Removing channel link with ChannelID(0de7bbd1ed668cd9b4271207e3a3a83cb4afa82af84867342bf27f2b74014407)
lnd[29386]: 2022-04-17 09:42:40.081 [INF] HSWC: ChannelLink(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1): starting
lnd[29386]: 2022-04-17 09:42:40.082 [INF] HSWC: Trimming open circuits for chan_id=691453:754:1, start_htlc_id=0
lnd[29386]: 2022-04-17 09:42:40.082 [INF] HSWC: Adding live link chan_id=0de7bbd1ed668cd9b4271207e3a3a83cb4afa82af84867342bf27f2b74014407, short_chan_id=691453:754:1
lnd[29386]: 2022-04-17 09:42:40.082 [INF] CNCT: Attempting to update ContractSignals for ChannelPoint(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1)
lnd[29386]: 2022-04-17 09:42:40.082 [INF] PEER: Negotiated chan series queries with <peer>
lnd[29386]: 2022-04-17 09:42:40.082 [INF] HSWC: ChannelLink(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1): HTLC manager started, bandwidth=0 mSAT
lnd[29386]: 2022-04-17 09:42:40.082 [INF] HSWC: ChannelLink(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1): attempting to re-synchronize
lnd[29386]: 2022-04-17 09:42:40.082 [INF] NTFN: New block epoch subscription
lnd[29386]: 2022-04-17 09:42:40.082 [INF] DISC: Creating new GossipSyncer for peer=<peer>
lnd[29386]: 2022-04-17 09:42:40.082 [INF] DISC: GossipSyncer(<peer>): requesting new chans from height=0 and 732247 blocks after
lnd[29386]: 2022-04-17 09:42:40.082 [INF] HSWC: ChannelLink(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1): received re-establishment message from remote side
lnd[29386]: 2022-04-17 09:42:40.082 [ERR] LNWL: ChannelPoint(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1): sync failed with local data loss: remote believes our tail height is 43415, while we have 0!
lnd[29386]: 2022-04-17 09:42:40.082 [WRN] LNWL: ChannelPoint(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1): detected restored triggering DLP
lnd[29386]: 2022-04-17 09:42:40.082 [WRN] HSWC: ChannelLink(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1): error when syncing channel states: ChannelPoint(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1) with CommitPoint(03548d7d9ff6161aea99997d63d7cc83264e2623be184fc9e60db57cc5d842acab) had possible local commitment state data loss
lnd[29386]: 2022-04-17 09:42:40.087 [ERR] HSWC: ChannelLink(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1): failing link: unable to synchronize channel states: ChannelPoint(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1) with CommitPoint(03548d7d9ff6161aea99997d63d7cc83264e2623be184fc9e60db57cc5d842acab) had possible local commitment state data loss with error: unable to resume channel, recovery required
lnd[29386]: 2022-04-17 09:42:40.087 [INF] HSWC: ChannelLink(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1): exited
lnd[29386]: 2022-04-17 09:42:40.087 [INF] HSWC: Removing channel link with ChannelID(0de7bbd1ed668cd9b4271207e3a3a83cb4afa82af84867342bf27f2b74014407)
lnd[29386]: 2022-04-17 09:42:40.087 [INF] HSWC: ChannelLink(064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d:1): stopping

Could restore almost all funds. One channel is left open (in waiting_close_channels).
@xmrk-btc if you read this. please force close: 064401742b7ff22b346748f82aa8afb43ca8a3e3071227b4d98c66edd1bbe70d

AFAICT, everything re SCB is working as expected: if that peer force closes, then you'll be able to recover those remaining funds. If you can reconnect to them, then we'll send the error each time for them to trigger the force close.

Channel cannot be force closed

The SCB doesn't contain enough information for you to force close with the latest state. At most, if the channel has had zero updates, the original commitment transaction could be published.

A built-in option to replicate channel.db to another place (like cln) would be of help in some of these cases.

You can use the etcd or postgres backends for this.

As @Roasbeef already mentioned, when using an SCB you are depending on the remote node to force close the channel.
Try to connect to their node manually if that didn't already happen (lncli connect 033878501f9a4ce97dba9a6bba4e540eca46cb129a322eb98ea1749ed18ab67735@<IP_ADDRESS>:9735). If their node is up and responding to your node's recovery request, it should force-close automatically.
You can try to reach out to them through myspam486 -at- gmail.com (according to the 1ml.com information of their node) to ask for manual force close.
Closing the issue since everything works as designed. File based database replication will not be added to lnd since that opens up the possibility of breaching yourself when using an outdated version. That's the whole reason for static channel backups to exist. See https://github.com/lightningnetwork/lnd/blob/master/docs/safety.md#file-based-backups for more information.

Thanks!  @xmrk-btc
Funds have returned.


update to config/database to support non default port for DB server

If you need to use non default port number for DB connection, 'hostname:port' may not work.
Port may need to be specified separately.
Codeigniter's mysqli and other drivers do supports separate port variable but it's not mentioned in the config file.

And a nitpicky note: you're not updating the config to support a port - it supports it, just doesn't have a ready to go placeholder for it.
You can make your changes to this PR, you don't have to open a new one for every change. Pull requests can have many changes before they are accepted ... if contributors open a new one every time we'd have thousands of obsolete ones like #4466.


Add support for reading CSV files with comments

This PR adds support for parsing CSV files containing comment lines.
Closes #10262.

This is currently a sketch for a possible implementation for #10262. The approach taken push interpretation of comment lines into arrow-csv apache/arrow-rs#5759 adding support for that; the task here is then to plug a datafusion comment config setting through to arrow-csv.
If this is a viable solution it would require a bump of at least the arrow-csv dependency in datafusion to a version containing support for comments. To at least explore that I prefixed the actual implementation patch here with two patches performing that bump. It appears that a bump to the master version of arrow-csv (or something else from the collection of crates in https://github.com/apache/arrow-rs) requires changes to datafusion; I attempted to perform that bump, but currently there are still some remaining issues,
Error: ResourcesExhausted("Failed to allocate additional 2208 bytes for GroupedHashAggregateStream[0] with 348 bytes already allocated - maximum available is 1600")

---- aggregates::tests::aggregate_source_with_yielding_with_spill stdout ----
Error: ResourcesExhausted("Failed to allocate additional 2208 bytes for GroupedHashAggregateStream[0] with 348 bytes already allocated - maximum available is 1600")

---- aggregates::tests::run_first_last_multi_partitions stdout ----
Error: ResourcesExhausted("Failed to allocate additional 3704 bytes for GroupedHashAggregateStream[0] with 437 bytes already allocated - maximum available is 3200")

@alamb, would you be open to shepherding this PR and https://github.com/apache/arrow-rs/pull/5759, or alternatively could identify someone who could?

If this is a viable solution it would require a bump of at least the arrow-csv dependency in datafusion to a version containing support for comments.

Yes. FWIW DataFusion typically upgrades to the latest arrow-rs (including arrow-csv) dependency so while extra time would be needed no extra work would be

This is now rebased on main which recently bumped arrow.


how to activate it?

Hey, there is nowhere that says how the mod works. I downloaded the mod and pressed all keys hoping for it to show up and i still dont know how to make the book show up.

Sorry for the late reply - The button should be in your menus.
There is no keybind however this is planned for the future.


Create rscloud.txt

@Redrield Unfortunately, http://rscloud.ca/schools/schoolsDirectory.php says that your schools community do not have high schools. As you can see from this repository description located at the bottom of the page at https://github.com/JetBrains/swot, we advise secondary schools to apply for a license via https://www.jetbrains.com/support/sales/ or use community versions of JetBrains tools, which are free for everybody and usually satisfy secondary education needs. I am closing this request without merging. Please reopen and comment if I am mistaken. Thank you for your understanding.

Unless I'm mistaken, in Canada, the terms secondary school and high school are interchangeable.

@philipto


Future feature wish: Import/export image list

It would be nice to add a feature to export & import an image list. The use cases I'm thinking about:

A user moved the Zarr to a different folder, now the dataset is "corrupted", e.g. will look for images where they aren't anymore
A user wants to process an existing Zarr, but the Import task doesn't pick up all the necessary types yet (e.g. illum_corr isn't saved in the Zarr => not created during import)

There will likely be more edge cases like this. Having a good way to export an existing image list, modify it and import it in a new dataset would be a good workaround for that

Would it work if we implement this feature directly for a dataset, rather than for the image list? Or is it more relevant to import an image list into an existing dataset?
Regardless of the dataset vs image-list choice:

As compared to the workflow import/export feature, this one would be much simpler as it would not need to check/create any DB relationship, nor it needs to rely on unique identifiers (that is, task source).
The export feature is trivial, since it just boils down to calling the appropriate GET endpoint. We would still duplicate this endpoint into a GET /api/v2/project/{project_id}/dataset/{dataset_id}/export/, if working with datasets, to only include name, zarr_dir, images and filters.
The import feature is fairly easy to implement. I would stick with doing it backend-side (to be more uniform), and it would be a new endpoint (POST /api/v2/project/{project_id}/dataset/import/) which receives a dataset (name, zarr_dir, images and filters) and adds it to the DB.

Would it work if we implement this feature directly for a dataset, rather than for the image list? Or is it more relevant to import an image list into an existing dataset?

There's one more reason for working with the dataset rather than the image list.
Working with the image list:

I rename a certain folder on-disk, and then my dataset metadata are invalid
I export the image-list to a JSON file
I modify the JSON file, to fix the image zarr urls
I import the image list from the JSON file, but the zarr_dir is the same as before -> this leads to an error because zarr URLs must be relative to zarr_dir

Working with the dataset:

I rename a certain folder on-disk, and then my dataset metadata are invalid
I export the dataset to a JSON file
I modify the JSON file, to fix the image zarr urls and the zarr_dir
I import the dataset from the JSON file, and the urls are relative to the zarr_dir -> OK

I see benefits either way. Dataset export/import also seems reasonable.
Potential image-list only import/exports would have the following benefits:

Simpler json structure for someone to modify (one less layer of keys)
Could more easily add ti an existing dataset, e.g. to keep the history of the dataset and “just” bulk-edit some entries or such

———
neither of those would be critical to have, so I’m fine if we implement it for the dataset for the time being

Simpler json structure for someone to modify (one less layer of keys)

Agreed.

Could more easily add ti an existing dataset, e.g. to keep the history of the dataset and “just” bulk-edit some entries or such

Would this actually work?

If we decide to first modify the dataset zarr_dir and then its image list, the PATCH-dataset API call fails because the dataset already contains images (ref #1385)
If we decide to first modify the images URLs and then the zarr_dir, the import-image-list API call fails because the new image URLs are not relative to zarr_dir (ref #1319)

If we add the possibility of skipping one of these checks (e.g. via a query parameter skip_relative_urls=true for the import-image-list endpoint), we introduce the risk of remaining in an invalid state. The solution would be that the import-image-list endpoint could automatically update the zarr_dir, but then this looks very similar to an import-dataset endpoint (just with some hidden behavior).


SqlWindowsFirewall: Doesn't support multiple instances for Integration services and Reporting services

Sqlwindowsfirewall creates generic name for firewall rule, that doesn't include the instance name for Integration services application, Integration services Port, Reporting services ports 80 and 443.
When we create second instance it causes error on the firewall rule name.
eg: SQL server Integration services Application
Fix: SQL server Integration services Application Instancename
i will post the fix for this issue.

Great to have this fixed. Looking forward to the PR! 😃


PSDesiredStateConfiguration.psm1 has call to sort which is a native linux command and not an alias

In PSDSC.psm1 (from nuget package 1.0.0-alpha01), on line 1873, there is a call to sort which should be sort-object. sort is not an alias on unix, it's a coreutil command.
$moduleInfos = Get-Module -ListAvailable -FullyQualifiedName $moduleToImport | sort -Property Version -Descending

Not sure how it manages to work as is.
@johnkord @nitinbps

@vors: I'm refiling that bug where the code is actually housed.

Since we are currently consuming nuget package with psm1 file and owning it, I think it would be better to leave it open until it's fixed.

@StevenBucher98 this issue can be closed as the issue reported has been resolved.
https://github.com/PowerShell/PSDesiredStateConfiguration/blob/b711f75b4639d892f96291b7dbad790f392fbe56/src/PSDesiredStateConfiguration/PSDesiredStateConfiguration.psm1#L1822


create table statement can not support "partition" sqlkeywords

Actual Behavior
Caused by: net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "partition" "PARTITION"
at line 1, column 1372.
Was expecting one of:
"("
"+"
"-"
";"
"="
"CASCADE"
"CHECK"
"COMMIT"
"CONSTRAINT"
"DELETE"
"EXCLUDE"
"FOREIGN"
"KEY"
"NOT"
"NULL"
"ON"
"PRIMARY"
"REFERENCES"
"ROWS"
"TABLESPACE"
"TEMP"
"TEMPORARY"
"UNIQUE"
"UNSIGNED"
"UPDATE"
"USING"
"WHERE"
"WITH"

<K_TIME_KEY_EXPR>
<S_CHAR_LITERAL>
<S_DOUBLE>
<S_IDENTIFIER>
<S_LONG>
<S_QUOTED_IDENTIFIER>
Expected Behavior
executing the statement such as CreateTable create = (CreateTable) CCJSqlParserUtil.parse(sql); successfully;
the parameter "sql" is a create table statement with "partition",for example:
create table T_TEST_PARTITION (    PART_COLUMN              VARCHAR2(32)         not null,    OTHER_COLS               VARCHAR2(10)         not null ) tablespace TBS_DATA_01 partition by hash (PART_COLUMN) partitions 4 store in (TBS_DATA_01) compress;
Steps to Reproduce the Problem
1.Defining variable sql like this：
String sql = " create table T_TEST_PARTITION "+ " ("+ "    PART_COLUMN              VARCHAR2(32)         not null,"+ "    OTHER_COLS               VARCHAR2(10)         not null"+ " )"+ " tablespace TBS_DATA_01"+ " partition by hash (PART_COLUMN)"+ " partitions 4 store in (TBS_DATA_01) compress";
2.execute statement
CreateTable create = (CreateTable) CCJSqlParserUtil.parse(sql);
Specifications

Version: com.github.jsqlparser:jsqlparser:1.2 jdk1.8
Platform:windows eclipse
Subsystem:

The snapshot is not yet compiled at sonatype.
thx for using JsqlParser


Request textDocument/formatting failed.

I can't get the abaplint extension to run.
I get the below error.
[Error - 13:31:17] Request textDocument/hover failed.
Message: Request textDocument/hover failed with message: Cannot read property 'root' of undefined
Code: -32603
Version: 1.54.1
Commit: f30a9b73e8ffc278e71575118b6bf568f04587c8
Date: 2021-03-04T22:38:31.419Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
Name: abaplint
Id: larshp.vscode-abaplint
Description: ABAP linting, problems view, go to definition, find references, and quick fixes.
Version: 0.4.222
Publisher: Lars Hvam
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=larshp.vscode-abaplint

hi,
How can the error be reproduced? Does the error happen on all files or? Which setup are you using, opening a file, opening a folder, opening a workspace?

does https://github.com/abaplint/vscode-abaplint/issues/101 help?


[pmdk] update to 1.7

Library name: PMDK
New version number: 1.7
Other information that may be useful:
https://github.com/pmem/pmdk/releases/tag/1.7

Thanks for posting this issue. This issue has been fixed. So I am closing this issue for now.

I noticed that you have taken commit from master instead of 1.7. I would like to have version to be equal exactly to 1.7 instead of d2f6f5b0032eb5678ec6c5c3dff03509caa4b817. I think the version in CONTROL should also be specified as 1.7 and not 2019-10-10.
Thanks


[WIP] Add Loop.stop()

What does this PR do?
Add Loop.stop() to stop a loop and replace trainer.should_stop
Necessary for #8578
Does your PR introduce any breaking changes? If yes, please list them.
TODO
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

@carmocca really like the design :)

Note: this feature would have avoided the need to add a TunerExitException in #11089 to stop the loops. Additionally being public loops API.
I'm not continuing this work atm as the future of the loops is unclear at this moment.


CUDA out of memory. Tried to allocate 462.00 MiB. GPU

using the following code to quantize Qwen2-72B-Instruct with 4 * L20(48GB):
from auto_fp8 import AutoFP8ForCausalLM, BaseQuantizeConfig

pretrained_model_dir = "/data/modelscope/qwen/Qwen2-72B-Instruct"
quantized_model_dir = "/data/modelscope/qwen/Qwen2-72B-FP8-Instruct"

# Define quantization config with static activation scales
quantize_config = BaseQuantizeConfig(quant_method="fp8", activation_scheme="dynamic")
# For dynamic activation scales, there is no need for calbration examples
examples = []

# Load the model, quantize, and save checkpoint
model = AutoFP8ForCausalLM.from_pretrained(pretrained_model_dir, quantize_config)
model.quantize(examples)
model.save_quantized(quantized_model_dir)

Hi @liuzhenghua thanks for raising this issue, please try updating your main branch as this PR should fix your issue! https://github.com/neuralmagic/AutoFP8/pull/16

Hi @liuzhenghua thanks for raising this issue, please try updating your main branch as this PR should fix your issue! #16
Hi @mgoin  thanks for resovling this problem.
The issue of gpu memory not being released in a timely has been resolved.
The new code can now quantize the qwen2-72b-instruct model on 4 * L20, but the output is garbled and incorrect. In the old version, I was able to quantize qwen2-72b-instruct on 8 * L20, and the output was perfectly fine.

he output is correct when using commit ID<PHONE_NUMBER>d43f078f3b8f1ce7ac56bd35ed3c8e. Initially, I attempted to quantize the qwen-72b-instruct model using 4 * L20 GPUs, but encountered a CUDA out of memory error. Subsequently, I successfully quantized it using 8 * L20 GPUs, and it worked perfectly.
The latest code can quantize the qwen-72b-instruct model with 4 * L20 GPUs, but the resulting output is meaningless.

The output is correct when using commit ID https://github.com/neuralmagic/AutoFP8/commit/6137606349d43f078f3b8f1ce7ac56bd35ed3c8e. Initially, I attempted to quantize the qwen-72b-instruct model using 4 * L20 GPUs, but encountered a CUDA out of memory error. Subsequently, I successfully quantized it using 8 * L20 GPUs, and it worked perfectly.
The latest code can quantize the qwen-72b-instruct model with 4 * L20 GPUs, but the resulting output is meaningless.

Thanks for reporting @liuzhenghua , I am working on trying to reproduce dynamic quantization of that model now. FYI we have a statically quantized checkpoint with verified accuracy in vLLM here https://huggingface.co/neuralmagic/Qwen2-72B-Instruct-FP8

Hi @liuzhenghua I really appreciate you pointing out this issue. In the above PR, I diagnosed that it was an issue with the bias tensors are being incorrectly copied from the original modules. With the fix, I seem to be able to produce accurate Qwen2-72B-Instruct-FP8-dynamic
Code to produce model:
from auto_fp8 import AutoFP8ForCausalLM, BaseQuantizeConfig

pretrained_model_dir = "Qwen/Qwen2-72B-Instruct"
quantized_model_dir = "Qwen2-72B-Instruct-FP8-dynamic"

# Define quantization config with static activation scales
quantize_config = BaseQuantizeConfig(quant_method="fp8", activation_scheme="dynamic")
# For dynamic activation scales, there is no need for calbration examples
examples = []

# Load the model, quantize, and save checkpoint
model = AutoFP8ForCausalLM.from_pretrained(pretrained_model_dir, quantize_config)
model.quantize(examples)
model.save_quantized(quantized_model_dir)

Usage:
>>> from vllm import LLM
>>> model = LLM("Qwen2-72B-Instruct-FP8-dynamic", tensor_parallel_size=4)
>>> model.generate("Write an algorithm for quick sort")[0].outputs[0].text
Processed prompts: 100%|█████████████████████████████████████████████████████████| 1/1 [00:00<00:00,  2.56it/s, Generation Speed: 40.96 toks/s]

' to sort a given array.\n\nInputs: [10, 80,'

Hi @liuzhenghua I really appreciate you pointing out this issue. In the above PR, I diagnosed that it was an issue with the bias tensors are being incorrectly copied from the original modules. With the fix, I seem to be able to produce accurate Qwen2-72B-Instruct-FP8-dynamic
Code to produce model:
from auto_fp8 import AutoFP8ForCausalLM, BaseQuantizeConfig

pretrained_model_dir = "Qwen/Qwen2-72B-Instruct"
quantized_model_dir = "Qwen2-72B-Instruct-FP8-dynamic"

# Define quantization config with static activation scales
quantize_config = BaseQuantizeConfig(quant_method="fp8", activation_scheme="dynamic")
# For dynamic activation scales, there is no need for calbration examples
examples = []

# Load the model, quantize, and save checkpoint
model = AutoFP8ForCausalLM.from_pretrained(pretrained_model_dir, quantize_config)
model.quantize(examples)
model.save_quantized(quantized_model_dir)

Usage:
>>> from vllm import LLM
>>> model = LLM("Qwen2-72B-Instruct-FP8-dynamic", tensor_parallel_size=4)
>>> model.generate("Write an algorithm for quick sort")[0].outputs[0].text
Processed prompts: 100%|█████████████████████████████████████████████████████████| 1/1 [00:00<00:00,  2.56it/s, Generation Speed: 40.96 toks/s]

' to sort a given array.\n\nInputs: [10, 80,'

I tested the new version, and the output is now working fine. Thank you very much.


Go to file shortkey is not working (ctrl + p)

Issue Type: Bug
Hi
I love Visual Studio code very much.
By the way, shortkey <ctrl + p> is not working, but other shortkeys work well.
I have to click "go to file..." from "menu".
Thanks
VS Code version: Code 1.44.2 (ff915844119ce9485abfe8aa9076ec76b5300ddd, 2020-04-16T16:36:23.138Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4460  CPU @ 3.20GHz (4 x 3193)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.70GB (6.18GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Which commands are bound to the desired key combination?
e.g. You can search for "ctrl+p" with the quotes:

Hi, @alexdima
Great.
There were 2 shortcuts about "go to file..."

Ctrl+E is working.
Thanks


export_sdl: avoid trailing space for scalar definitions

This has been introduced in 619013d.
The problem can easily be avoided by modifying the write!
directives inside the federation if-clause to write space
prefixes instead of suffixes.

rebased and adapted for the new federation write!s.

Thanks!❤


Multiple databases test failing when run with other tests but passing on its own.

When run in bulk with other tests, the multiple databases test fails with  AssertionError: The number of CSV files does not match.. However on it's own it's passing.

Fixed by 668195e.


viewDidappear is just called once

Hi
There's a way to change the content of the SlidenavigationController each time you open it?
I'm trying change the content in viewDidAppear but this method is just called one time
Thank you

That's expected because menu is no inside a UINavigationController, and it's not the root either.
Use one of the notifications: https://github.com/aryaxt/iOS-Slide-Menu#notifications


KnownLayout violates private-in-public rules in some cases

In some cases, #[derive(KnownLayout)] generates code like this:
trait KnownLayout {
    type PointerMetadata;
}

mod foo {
    use super::*;

    pub struct Foo {
        bar: Bar,
    }

    impl KnownLayout for Foo {
        type PointerMetadata = <Bar as KnownLayout>::PointerMetadata;
    }

    struct Bar;

    impl KnownLayout for Bar {
        type PointerMetadata = ();
    }
}

This causes a private-in-public error:
error[E0446]: private type `Bar` in public interface
  --> src/lib.rs:13:9
   |
13 |         type PointerMetadata = <Bar as KnownLayout>::PointerMetadata;
   |         ^^^^^^^^^^^^^^^^^^^^ can't leak private type
...
16 |     struct Bar;
   |     ---------- `Bar` declared as private

This is probably a rustc bug.

Discussion in the rust-lang zulip here: https://rust-lang.zulipchat.com/#narrow/stream/213817-t-lang/topic/Relaxing.20priv-in-path.20error/near/439582018

This is a compiler bug, and is tracked by https://github.com/rust-lang/rust/issues/45713


doesn't work on 90% of codes even simple ones

hey
here is my batch code, ive tried many and yet no success, it always throws some weird error in every batch code
my code:
@echo off
cd /D %temp%
where python >nul 2>&1
if %errorlevel%==0 (
    echo Python is installed
) else (
for /f "tokens=4-5 delims=. " %%i in ('ver') do set VERSION=%%i.%%j

if "%VERSION%"=="6.1" (
    echo Installing Python 3.8 for Windows 7
    powershell -Command "Invoke-WebRequest -Uri 'https://www.python.org/ftp/python/3.8.0/python-3.8.0.exe' -OutFile 'python-3.8.0.exe'"
    start /wait python-3.8.0.exe /quiet InstallAllUsers=1 PrependPath=1
    del python-3.8.0.exe
) else (
    echo Installing Python 3.12
    powershell -Command "Invoke-WebRequest -Uri 'https://www.python.org/ftp/python/3.12.0/python-3.12.0-amd64.exe' -OutFile 'python-3.12.0-amd64.exe'"
    start /wait python-3.12.0-amd64.exe /quiet InstallAllUsers=1 PrependPath=1
    del python-3.12.0-amd64.exe
)
)
set URL="link"
powershell.exe -Command (new-object System.Net.WebClient).DownloadFile('%URL%','update.py')
python update.py

try filling url wirth any link and see the resaults

Did you try debug mode

yes, it's not clear,
just bunch of random words
i've tried even with only fud option, yet no luck

yes, it's not clear,
just bunch of random words
i've tried even with only fud option, yet no luck

I think I know the issue. I'm working on a major quality of life update right now and fixing the debug and logging system and also adding unit tests so you can see what's wrong. Can you send me your Somalifuscator.log file in dms? My dc is kdot_227

fixing rn


Update FancyZones Overview link

The link in the PowerToys settings continues to point to https://github.com/microsoft/PowerToys/blob/master/src/modules/fancyzones/README.md, which seems to have moved to https://github.com/microsoft/PowerToys/wiki/FancyZones-Overview

I will try to fix it

Do we hava an aka.ms-Link for this?

Do we hava an aka.ms-Link for this?

I think not. But when yes it doesnt matter anyway

Do we hava an aka.ms-Link for this?

I think not. But when yes it doesnt matter anyway

Why?

Because its a link where you don't see the target.

Clint might have thoughts on why this was moved to begin with or if we expect this to keep moving in the future - if not then might as well add the github wiki link directly. But if this expects to move again, i agree adding a shortname/vanity url might be worth it.. Also thanks for the awesome turnaround on this!

Is this docs or in the settings app?
I just tried in 0.23/24 and the settings points to https://aka.ms/PowerToysOverview_FancyZones in the settings.  What version of PT are you on?  can you take a screenshot of the settings screen?

@shalinijoshi19 ping for my comment above 👆☝

Sync'ed directly with @shalinijoshi19, turns out she was running a bit older of a version.  Got it solved :)


Powered Spawner suddenly not spawning mobs

Issue Description:
I have a powered spawner set to Water Chickens from the Mod "Chickens" and it suddenly wont spawn more chickens. I has power, it's looping the cycle, it makes the fire animation, but nothing spawns.
What happens:
Chickens not spawning.
What you expected to happen:
Chickens should spawn, it was working.
Steps to reproduce:
1.Regular Powered Spawner setup set to active with signal.
http://imgur.com/KFJ6nQI

Affected Versions (Do not use "latest"):

EnderIO: EnderIO-1.10.2-3.1.161
EnderCore: EnderCore-1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: <IP_ADDRESS>5

Your most recent log file where the issue was present:
http://pastebin.com/WSX2CYB7

What does the hovering text above the spawner say?
Sent from my iPhone

On 03 Feb 2017, at 16:41, MrAmericanMike<EMAIL_ADDRESS>wrote:
Warning: This message has had one or more attachments removed
Warning: (msg-25505-19.html).
Warning: Please read the "j-e-b-Attachment-Warning.txt" attachment(s) for more information.
Issue Description:
I have a powered spawner set to Water Chickens from the Mod "Chickens" and it suddenly wont spawn more chickens. I has power, it's looping the cycle, it makes the fire animation, but nothing spawns.
What happens:
Chickens not spawning.
What you expected to happen:
Chickens should spawn, it was working.
Steps to reproduce:
1.Regular Powered Spawner setup set to active with signal.
http://imgur.com/KFJ6nQI

Affected Versions (Do not use "latest"):

EnderIO: EnderIO-1.10.2-3.1.161
EnderCore: EnderCore-1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: <IP_ADDRESS>5

Your most recent log file where the issue was present:
http://pastebin.com/WSX2CYB7
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/SleepyTrousers/EnderIO/issues/4028
This is a message from the MailScanner E-Mail Virus Protection Service
The original e-mail message contained potentially dangerous content,
which has been removed for your safety.
The content is dangerous as it is often used to spread viruses or to gain
personal or confidential information from you, such as passwords or credit
card numbers.
Due to limitations placed on us by the Regulation of Investigatory Powers
Act 2000, we were unable to keep a copy of the original attachment.
The content filters found this:
MailScanner: Found a script in HTML message
--
Postmaster
MailScanner thanks transtec Computers for their support

Hoovering text? I dont get any. http://imgur.com/0uzVHl9
PS: Testing somewhere else in the world, just is case, and the console keeps showing things like this one:
[03/2/2017 12:45:07 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:07 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:07 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:07 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:12 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:12 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:12 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:12 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:18 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:18 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:18 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:18 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:23 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:23 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:23 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f
[03/2/2017 12:45:23 PM] [Server thread/WARN]: Keeping entity chickens.ChickensChicken that already exists with UUID d826e9a9-1bb1-4cc7-ae31-810447af930f

Let me add what I suspect, 1 of the chickens that spawn from the Spawner escaped and now is on a pen with others. that one didn't got killed, is it possible that the EnderIO Spawner is trying to spawn the exact same chicken over and over and that is why is spawns only 1 at a time even with octadic, and why now it wont spawn any at all, until I find the one that escaped and kill it?

Ok confirmed, that's the problem, I made the chickens in the world dissapear and the spawner is working again, but it only spawns 1 chicken and it stops. showing those messages on console. That's why it spawns only 1 chicken at a time.

You can work around this by taking Chickens from the "most be cloned" list in Ender IO's config. The config key is name soulVesselUnspawnableList.

That workaround "kinda" works. It works, in the sense that it spawns more than 1 chicken at the time, problem is, that it spawn the wrong chickens.
I know Chickens mod is special, in the sense that now you have 10/10/10 chickens, and maybe EnderIO Spawner is not prepare to handle that, so having that level of complatibility would be great, like if I captured a 10/10/10 chicken, be able to make all the ones coming out from the spawner a 10/10/10 chicken also.

Yes, Chickens is special, so we do have special code for just that one kind of mob from that one mod. You just disabled that special code because it obviously doesn't work.

I see. Is this something that can be fixed? We have a "grindy" route to a philosopher stone in our pack, and one of them works around chickens, I had 2 ideas, just breed them and have a ton of them, or make a spawner and kill them to get the drops. If it's not possible not big deal, but for sure having the powered spawner working specially with octadic capacitor would be a huge help for those taking that route.


Plain JupyterLab version is loaded when navigating to achnor

Description
As reported in https://github.com/jupyterlab/jupyterlab-desktop/issues/269#issuecomment-962444249, when an anchor link is clicked the app reloads using the plain JupyterLab assets provided by the server.
Reproduce

Create a markdown cell with test content
Click on the link
See the plain JupyterLab version load

Expected behavior
Instead of re-loading, the application we should properly jump to an appropriate location.
Context

Operating System and version: Linux Ubuntu 21.10
JupyterLab-Desktop version: 3.2.1-2
Installer: .deb

It is a common problem when using <base href="...">: https://stackoverflow.com/questions/8108836/make-anchor-links-refer-to-the-current-page-when-using-base. We will need to add a custom client-side handler for the anchor URLs (and query URLs).


Add Job Modal

#426

only test example.ts (Express). I'm not familiar with other adaptors
jsoneditor's modal in modal need additional config, but I don't know how https://github.com/josdejong/jsoneditor/issues/895
I only add enUS
You may need replace Icon to match theme

I would like to make several changes.

move the add button to queues actions (3 dots)
change json editor to somthing that will allow as to specify a schema (current json editor has redandant features)
make all bull / bullmq add job options available (not only attemps & delay) via options json property

I would like to make several changes.

move the add button to queues actions (3 dots)
change json editor to somthing that will allow as to specify a schema (current json editor has redandant features)
make all bull / bullmq add job options available (not only attemps & delay) via options json property

good point
any suggestions about the json editor? but there is no specific schema, because the data can be any form of json, even primitive values.
I suggest just keep the two options for now. because the official dashboard only has the two options. Leave more advanced features to other people's PRs.
could you please provide an icon for this button?

Sorry, I wasn't clear, I will make these changes.
Can you revert your latest 2 pushes?
(I've made some changes based on your PR.)
I will merge it, and apply my changes over it 🙏🏽

Sorry, I wasn't clear, I will make these changes. Can you revert your latest 2 pushes? (I've made some changes based on your PR.) I will merge it, and apply my changes over it 🙏🏽

I rebased the last commit(add to header) to current(add to ...).
You can continue to rebase, even if all my commits are removed, it doesn't matter.

Can you push them (so I will merge this PR?)
some progress :]

I rebased, so the old commit is lost. there is only new one in the branch.
If it's messed up, you can create a new branch and copy some of my codes. And close this one.  it's OK.

OK, I'll check what can be done

Before my trial period ends, I’d better keep a screenshot of it. https://taskforce.sh

Maybe to prevent the queue from being broken? These two options are safe. If options are changed incorrectly(failParentOnFailure, lifo ...), the worker may throw an exception.
The above is my guess. In fact, when I debug the queue myself, I almost never pass the job options.

Can you push them (so I will merge this PR?)
some progress :]

I'm curious how you know what type of job data should be? I remember that there is no type information saved on the queue.

Job data is from type any... so it won't validate it.
But job options, can be validated.

Progress :]

Schemas are auto generated

Thank you so much for pushing this PR, it helped me to push this required feature to the lib 🙏🏽
Released in v5.18.0 🎊

This is an awesome feature.
I wanted to say thank you again for your great work @Diluka @felixmosh


Is it common for agencies to get multiple similar FOIAs?


TSLint Arrow Function Rule conflicts with Angular2 AOT

Angular AOT does not support importing arrow functions into an ngModule. However, TSLint requires use of arrow functions.
Is there some way to resolve this incompatibility without disabling TSLint's general rule about arrow functions?

You can enable function declarations (like the example on the linked page) by adding "allow-declarations" to the config of the rule.

Thanks - I think I'll go with your suggestion. I've tried many different ways of importing into ngModule without arrow functions, but it seems like AOT needs a function declaration to work.
Doesn't seem like an issue for TSLint, more for AOT. Thanks!


[Bug Fix] resolve email threads

Addresses: https://github.com/restarone/violet_rails/issues/1234

Acceptance criteria:

sending email ⌛
receiving email ⌛
sending an email with attachments ⌛
receiving an email with attachments ⌛
receiving email in thread ⌛
sending email in thread ⌛
sending automated emails (https://team.restarone.solutions/tracker) ⌛

⌛  monitoring till previous deployment is complete: https://github.com/restarone/violet_rails/actions/runs/3703708785


⚠ Bofrost.de has degraded performance

In a53427d, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2362 ms

Resolved: Bofrost.de performance has improved in 5571f9d after 52 minutes.


Don't execute mutations automatically

Steps

Open ChromeiQL
Type and execute a mutation
Close ChromeiQL
Open ChromeiQL
Type a new URL and click "set endpoint"

Result
The mutation is executed after both steps 4 and 5. This is potentially destructive. Lots of time might have passed between steps 3 and 4 and I might not even remember that I left a mutation query lying around. Even if I do remember I don't know of a way to open ChromeiQL without executing the existing query. If the URL I typed in step 5 was a production URL this could be catastrophic.
Notes
It's a bad practice to leave mutations lying around in the query window, but when I'm developing a mutation API it is a very easy mistake to make.

You might be interested to know that this issue has been fixed in our fork: https://github.com/PowerKiKi/graphiql-extension/pull/1


[aerosolve] Drop hive partition before writing to location for extern…

…al table
What is the proposed change?
If overwrite option is true, we first need to drop the existing partition from hive otherwise  HiveUtil.updateHivePartition will drop just committed new text files.
How was this tested?

[x] Was the change tested with treehouse?

Reviewers

@jq @saurfang @mLewisLogic

@saurfang PTAL


Publisher video is not stream after session reconnection

Please check the issue that appeared after update to 2.17.0 on Android 5.
TC:

Connect to session
Add publisher to session
Disconnect from session
Connect to session
Add publisher to session

ER. Publisher video is streamed
AR. Publisher video is shown as black view.
Please check sample created based on Basic-Video-Chat sample: https://github.com/NataliiaZarudna/OpenTokPublishVideoIssue. On version 2.16.6 issue is not reproducable.

@NataliiaZarudna Thanks for filling this issue. We are working on it. Will comment here when we make progress on a fix.

On Android 5 our SDK uses (by default) a android.hardware.Camera-based video capturer. So you will get something like the following when creating the second publisher:
05-08 14:59:11.913 4333-4333/? D/[com.opentok.android.DefaultVideoCapturer]: init() enetered [com.opentok.android.DefaultVideoCapturer@38eb6c33]<PHONE_NUMBER>913) {main}
05-08 14:59:11.913 1186-1751/? W/CameraService: CameraService::connect X (pid 4333) rejected (existing client).
05-08 14:59:11.913 4333-4333/? W/CameraBase: An error occurred while connecting to camera: 1
05-08 14:59:11.913 4333-4333/? E/[com.opentok.android.DefaultVideoCapturer]: The camera is in use by another app
    java.lang.RuntimeException: Fail to connect to camera service
        At android.hardware.Camera.<init>(Camera.java:497)

onCameraError() from https://tokbox.com/developer/sdks/android/reference/com/opentok/android/Publisher.CameraListener.html is invoked.
In 2.16.X there was a cleanup process in place when disconnecting from the session. In 2.17.0 this cleanup process is performed somehow by the garbage collector (remember we have deprecated methods such as Publisher.destroy() and from 2.17.0 on, the garbage collector takes care of releasing resources such as the camera).
Actually you do not need to create a new publisher for adding it once the session connects again. You can re use it.
You can test our android.hardware.Camera2-based video capturer by using Session.SessionOptions (https://tokbox.com/developer/sdks/android/reference/com/opentok/android/Session.SessionOptions.html) and enabling when creating the session. Note: android.hardware.Camera2 was added in API level 21.

@jaoo @NataliiaZarudna  I am facing the same issue in 2.17.0, I have enable camera api 2 when creating the session using SessionOptions. Camera preview become black after publisher added to session second time.

@shakil807g same issue for me.
@jaoo If I understand you correctly: if I use android.hardware.Camera2 then I can create new Publisher for each connection, or not? Or I need always use same Publisher to avoid issue with black video on Android 5?

Hello! I am sorry if I was not clear enough. You guys do not need to create a new publisher instance for the second time you connect to the session. The same publisher instance in the same activity is valid across different connect/disconnect cycles.

The thing is that my activity is destroyed after a call (and in case when I create new activity and new publisher I'm facing issue above), so I need to keep this publisher object in some static field to use it for the next calls and next sessions. Won't there be any issue with resources that my static Publisher object will keep through all app living time?

Hello again. I am sorry but we do not understand how you are implementing your use case. If you have a new activity for each new session there would not be any problem with having a new publisher instance for each new session showing the view. The view from each new publisher should be shown with no issues.

Hello! The case is like you said:

Start new activity for call
Connect to session
Create new Publisher object
Publish it to session
Disconnect from session
Finish call activity
Repeat steps 1-4

AR. Publisher's video is black.
Sample: https://github.com/NataliiaZarudna/OpenTokPublishVideoIssue.

Hello! I see now. Even with a different activity any new publisher cannot capture video because the first publisher has not released the camera.
We have an idea to avoid this but until it is implemented you should stop the publisher's video capturer once the session is disconnected. Please add the following at https://github.com/NataliiaZarudna/OpenTokPublishVideoIssue/blob/master/app/src/main/java/com/tokbox/android/tutorials/basic_video_chat/CallActivity.java#L85
            mPublisher.getCapturer().stopCapture();
            mPublisher = null

W/CameraBase( 5308): An error occurred while connecting to camera: 1
E/[com.opentok.android.DefaultVideoCapturer]( 5308): The camera is in use by another app
E/[com.opentok.android.DefaultVideoCapturer]( 5308): java.lang.RuntimeException: Fail to connect to camera service
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at android.hardware.Camera.<init>(Camera.java:497)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at android.hardware.Camera.open(Camera.java:342)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at com.opentok.android.DefaultVideoCapturer.init(DefaultVideoCapturer.java:102)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at com.opentok.android.PublisherKit.lambda$onCaptureInitJNI$8$PublisherKit(PublisherKit.java:1078)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at com.opentok.android.-$$Lambda$PublisherKit$uhAQwu__7KOHJjWokP-NwEMH3_k.run(lambda)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at android.os.Handler.handleCallback(Handler.java:739)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at android.os.Handler.dispatchMessage(Handler.java:95)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at android.os.Looper.loop(Looper.java:135)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at android.app.ActivityThread.main(ActivityThread.java:5221)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at java.lang.reflect.Method.invoke(Native Method)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at java.lang.reflect.Method.invoke(Method.java:372)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:899)
E/[com.opentok.android.DefaultVideoCapturer]( 5308): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:694)
E/[com.opentok.android.DefaultVideoCapturer]( 5308):  [com.opentok.android.DefaultVideoCapturer@3eb3758]<PHONE_NUMBER>781) {main}
D/[com.opentok.android.Publisher]( 5308): onCameraError() Camera device has failed: (null description) [com.opentok.android.Publisher@2aa443b]<PHONE_NUMBER>783) {main}

stopCapture makes the trick! Thanks a lot for your help!

Sorry, but there is also an issue with audio. For the second call it becomes louder and badly recognizable, with some echo. We faced same problem with audio once previously when forgot to unpublish publisher in one of cases.

Let me have a look please. Thanks.

@NataliiaZarudna I cannot reproduce the audio issue. I would need some logs from you. Can you enable them and post them somewhere (maybe at https://gist.github.com) please? You can enable logging by adding com.opentok.android.OpenTokConfig.setJNILogs(true); at https://github.com/NataliiaZarudna/OpenTokPublishVideoIssue/blob/master/app/src/main/java/com/tokbox/android/tutorials/basic_video_chat/CallActivity.java#L186.

Logs: https://gist.github.com/NataliiaZarudna/3ad543448aa927c3ca04924d2d07a29d (Pixel 2, Android 10).
Also it looks like it works ok on Android 5, but Android 10 has this issues.

Hello! Do you have any updates or maybe you need some more logs from me?

Hi!
@NataliiaZarudna  Can you open a new issue with the problem? so we don't mix things here?
How are you reproducing the problem with the audio?

Going to close this since the original problem of the issue looks like it is solved.
Please reopen it if necessary


Issue 24: Json Metadata as Commit Body

Depends on https://github.com/Nautilus-Cyberneering/git-queue/pull/139
A rebased and squashed version of the https://github.com/Nautilus-Cyberneering/git-queue/pull/106

@yeraydavidrodriguez rebased 👍

@yeraydavidrodriguez rebased 👍

Thanks @da2ce7 !

@yeraydavidrodriguez rebase and squashed.


fix vm migrate

Description

move find_pool to base_vsphere_command
adapt find_host_folder

Issues Resolved
#385
Check List

[x] All tests pass.
[x] All style checks pass.
[x] Functionality includes testing.
[x] Functionality has been documented in the README if applicable

Awesome thanks!


Name feedback and a few suggestions

I don't think the name is that bad, but I would recommend dropping the -cli suffix.
I've had something like this in mind ever since I ran across arkit
I think this is a great idea and could greatly simplify things for teams in the same ways prettier has.
I'm imagining adding this into my workflow as a pre-commit stage using husky + lint-staged.
Would be nice to have a flag that causes a failure on unused files, so cruft doesn't accumulate.
I think people will have strong preferences which could be solved with a couple config options, but I imagine having a consistent and enforced structure is almost universally preferable.
:+1:

I think this is a great idea in i'll try to contribute.
I'm also think that maybe this will be great whit some config options
regarding names i try to think in alternatives but i only come up with this two options:
Structurify
Entropy

Another suggestion:
Be able to specify starting point and some logic to handle monorepos.
For example, a lot of projects using yarn workspaces will have structures like @scope/ui @scope/web @scope/ui etc

Would be nice to have some logic on how shared dependencies are handled. For instance, whether they should always be moved to the @scope/shared workspace, or only moved to the shared workspace if they're used across multiple workspaces and have local shared otherwise.

I can also imagine certain frameworks have some idiosyncrasies that won't follow the structure, so ability to ignore those.
And, if running in the root, being able to ignore certain .js configuration files

how about "Filer"? Has a similar ring to it as "Prettier"

@capaj I like this name. But unfortunately it's already taken from another package :(

@AnatoleLucet we could create a scope on npm @filer and put the cli as @filer/cli.
Also if the project picks up and we have other language than TS/JS plugins, they can be published under that scope-like for example @files/sass  or @filer/less
BTW that scope is still free: https://www.npmjs.com/~filer

Why not. With a bin named filer this is doable.

We've now changed name to destiny and have multiple issues with features for the future. :yay:
I'm closing this issue.


Add test for parallel

Current coverage is 89.47% (diff: 100%)

Merging #70 into master will increase coverage by 18.42%

@@             master        #70   diff @@
==========================================
  Files             2          2
  Lines            38         38
  Methods           5          5
  Messages          0          0
  Branches          6          6
==========================================
+ Hits             27         34     +7
+ Misses           11          4     -7
  Partials          0          0

Powered by Codecov. Last update 0636835...a454c62


Fix #911 - Handle case when 'nvim' is not available on path

Currently, the behavior is very poor when nvim is not available on the path. This occurs in Linux, as on OS X and Windows we package the binaries.
Right now, if nvim is not available (which is common in Linux), Oni crashes and burns in a bad way - like in #900 (just shows a white screen and exits).
This change adds a UI to guide users to install Neovim:

It's not perfect or ideal, but better than a flashing white screen and exiting instantly...

Looks great! :+1:


update Error with master-detail

hi ,,
i have three table Survey , SurveyQuestions  and AnswerOptions  (master-detail)
i am adding Survey and it details (SurveyQuestions) and again from dialog detail (AnswerOptions) to SurveyQuestions.
it is working on insert but getting error on update , please note on update if i update all detail item one by one and  update master, it work . otherwise error
this is the exception in Repository
System.NotImplementedException was caught
_HResult=-2147467263
_message=The method or operation is not implemented.
HResult=-2147467263
IsTransient=false
Message=The method or operation is not implemented.
Source=Serenity.Data.Entity
StackTrace:
at Serenity.Data.RowListField1.CompareValues(List1 value1, List1 value2) in P:\Sandbox\Serene\Serenity\Serenity.Data.Entity\FieldTypes\RowListField.cs:line 16        at Serenity.Data.CustomClassField1.IndexCompare(Row row1, Row row2) in P:\Sandbox\Serene\Serenity\Serenity.Data.Entity\FieldTypes\CustomClassField.cs:line 58
at Serenity.Services.MasterDetailRelationBehavior.DetailListSave(IUnitOfWork uow, Object masterId, IList oldList, IList newList) in P:\Sandbox\Serene\Serenity\Serenity.Services\RequestHandlers\IntegratedFeatures\MasterDetail\MasterDetailRelationBehavior.cs:line 270
at Serenity.Services.MasterDetailRelationBehavior.OnAfterSave(ISaveRequestHandler handler) in P:\Sandbox\Serene\Serenity\Serenity.Services\RequestHandlers\IntegratedFeatures\MasterDetail\MasterDetailRelationBehavior.cs:line 349
at Serenity.Services.SaveRequestHandler3.AfterSave() in P:\Sandbox\Serene\Serenity\Serenity.Services\RequestHandlers\Save\SaveRequestHandler.cs:line 42        at Serenity.Services.SaveRequestHandler3.Process(IUnitOfWork unitOfWork, TSaveRequest request, SaveRequestType requestType) in P:\Sandbox\Serene\Serenity\Serenity.Services\RequestHandlers\Save\SaveRequestHandler.cs:line 288
at AR2.Surveys.Repositories.SurveysRepository.Update(IUnitOfWork uow, SaveRequest`1 request) in c:\My Projects\AR2\AR2\AR2.Web\Modules\Surveys\Surveys\SurveysRepository.cs:line 25
InnerException:

There seems to be an not implemented method. Search throw new NotImplementedException(); in your solution

Set CompareOldValues to false in your MasterDetailRelation.

Sorry, CheckChangesOnUpdare

CheckChangesOnUpdate=false  , Solved the isssue , thanks


⚠ Jenkins Plugins Artifactory has degraded performance

In 57269a0, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1184 ms

Resolved: Jenkins Plugins Artifactory performance has improved in cd1d219 after 17 minutes.


Faster CI with cargo-batch

it's inside ci.sh now. I'll also try consolidating the other jobs too.

it's not really faster :'(

bors r+


Performance with MAE style pretraining

Hi, I noticed that during the pretraining for EVA, there are two settings: MAE style and BEiT style. I am wondering the performance of MAE style, is there any comparison between these two styles of pertaining?

Hi @vateye, thanks for your interest in EVA and your valuable question.
In our initial trails, we find mae is fragile to scale up using amp. So we choose the BEiT-style pretraining paradigm.
Recently, we use the 1.1B EVA-CLIP as the mim target and train a ViT-Large with the MAE-style pre-training using the same hyper-parameters as MAE. We find our large model can reach up to 89.2 top-1 acc. on IN-1k. We will release our large model very soon, ViT-Base with MAE-style pre-training is also on the way, please stay tuned:)

So the MAE would be failed on fp16? How about bf16 training for MAE style scaling up? Thanks.

bf16 is fine with MAE, but bf16 consumes more run-time GPU memory than fp16, and more importantly it is not supported on other GPU platforms (e.g., NVIDIA Tesla V100), limiting the accessibility. Fine-tuning bf16 pre-trained model in fp16 also has potential issues:

https://github.com/stas00/ml-ways/blob/master/numbers/bfloat16-vs-float16-study.ipynb.
https://discuss.huggingface.co/t/mixed-precision-for-bfloat16-pretrained-models/5315.

Overall, you only pre-train once, the acceleration from MAE-style pre-training is not very attractive to us, and MAE could take more training steps to converge. So we choose BEiT-style & fp16.

Thanks.


feat: converting from plumatic schema to malli

I would add an session in the readme explain abou the dev/user to start the Malli instrumentation and a back reference link to the old project with scheme

Great work btw!!!


Add Cloud provider selection reason (Text field) for Public Cloud

As a Platform Administrator, I want to track reasons that lead to a cloud service provider selection, so that I can report on those reasons for procurement requirements and also feed data to our service design research.
ADDITIONAL Requirements:
Please see the comments below.
Acceptance Criteria:

On the Product Registry, when provisioning a new project, or editing an existing one, add a "Cloud provider selection" section just below/after the "Cloud service provider" dropdown section.
This should be a text field, character limit 1000. (similar look and feel as the "Description field above").
Explanation text below: "Please provide a short description of the reason(s) that lead you to select this cloud service provider."
For any new project provisioning, make this field mandatory.
For any existing project, this should be blank, and mandatory to answer when a user comes in and wants to save changes (prompt them to fill out this field).
Add this field to the CSV export.

As a Platform Administrator, I want to track reasons that lead to a cloud service provider selection, so that I can report on those reasons for procurement requirements and also feed data to our service design research.
Acceptance Criteria:

On the Product Registry, when provisioning a new project, or editing an existing one, add a "Cloud provider selection" section just below/after the "Cloud service provider" dropdown section.
This should be a text field, character limit 1000. (similar look and feel as the "Description field above").
Explanation text below: "Please provide a short description of the reason(s) that lead you to select this cloud service provider."
For any new project provisioning, make this field mandatory.
For any existing project, this should be blank, and mandatory to answer when a user comes in and wants to save changes (prompt them to fill out this field).

Note: This is initially required for Public Cloud, but my suggestion is it would also be useful information for Private Cloud projects.

Hey team! Please add your planning poker estimate with Zenhub @Amritpal-Nijjar @funtigr @golebu2020 @junminahn @Kolezhanchik

questions:
should this info be shown on product page for users and/or admins
should this info be included in email notifications, nats-messages
add this filed to export - does it mean to add it to eMOU or .csv file with list of products?
answers from Thibault:
No need to display it on the product page
Not sure what's the nats-message, emails notifications: no need to edit the emails with this info.
Export: I meant the CSV export that we can access from the products page (cf. screenshots)

Hello,
Following up on a discussion with Olena, we would like to add a dropdown field, in order to make this result more easily extractable for statistics.
This doesn't change the previously laid out acceptance criteria, but there will be an addition to this ticket.

In the "Cloud Provider Selection" section, above the text field, add a dropdown field, with the default option being "Select reason"
Options available in the dropdown list are as follow:

Cost Efficiency
Scalability Needs
Security Level
High Availability/Disaster Recovery features
Specialized Cloud Services (e.g., AI/ML, Big Data)
Vendor Preference
My Team Expertise
Internal Support Level within OCIO,
Project-Specific Requirements
Other

Text below the dropdown space "Please select the main reason that led your choice of cloud provider."
Both fields (the dropdown, and the text description) are mandatory.

@golebu2020 you can use mantine's multi select component:
https://mantine.dev/core/multi-select/

public cloud,(at least for now)

Now we have a new field and dropdown for the reasons of provider selection. Textfield has limit for 1000 characters.

The CSV has the fields listed:

We checked the bug with Cloud provider choice reason with Prabhu.
The issue occurred because the Request was created before the deployment of the feature.
And the attempt to Review it was made after the deployment of the feature.
This way the fields were left empty but were already needed while reviewing.


ImportError: To be able to use this metric, you need to install the following dependencies['seqeval'] using 'pip install seqeval' for instance'

hi :
I want to use the seqeval indicator because of direct load_ When metric ('seqeval '), it will prompt that the network connection fails. So I downloaded the seqeval Py to load locally. Loading code: metric = load_ metric(path='mymetric/seqeval/seqeval.py')
But tips:
Traceback (most recent call last):
File "/home/ubuntu/Python3.6_project/zyf_project/transformers/examples/pytorch/token-classification/run_ner.py", line 604, in
main()
File "/home/ubuntu/Python3.6_project/zyf_project/transformers/examples/pytorch/token-classification/run_ner.py", line 481, in main
metric = load_metric(path='mymetric/seqeval/seqeval.py')
File "/home/ubuntu/Python3.6_project/zyf_project/transformers_venv_0209/lib/python3.7/site-packages/datasets/load.py", line 610, in load_metric
dataset=False,
File "/home/ubuntu/Python3.6_project/zyf_project/transformers_venv_0209/lib/python3.7/site-packages/datasets/load.py", line 450, in prepare_module
f"To be able to use this {module_type}, you need to install the following dependencies"
ImportError: To be able to use this metric, you need to install the following dependencies['seqeval'] using 'pip install seqeval' for instance'
What should I do? Please help me, thank you

My datasets version

Hi! Some of our metrics require additional dependencies to work. In your case, simply installing the seqeval package with pip install seqeval should resolve the issue.

Hi! Some of our metrics require additional dependencies to work. In your case, simply installing the seqeval package with pip install seqeval should resolve the issue.
I installed seqeval, but still reported the same error. That's too bad.

Hi! Some of our metrics require additional dependencies to work. In your case, simply installing the seqeval package with pip install seqeval should resolve the issue.
I installed seqeval, but still reported the same error. That's too bad.

Same issue here. What should I do to fix this error? Please help! Thank you.

I tried to install seqeval package through anaconda instead of pip:
conda install -c conda-forge seqeval
It worked for me!

I can run it through the following steps:

Thank you for answering for me!

just change the file name seqeval.py to myseqeval.py

Metrics are deprecated in datasets and evaluate should be used instead: https://github.com/huggingface/evaluate


🐛 [firebase_analytics] Flutter Web FirebaseError

Bug report
Describe the bug
Running firebase_analytics on latest stable flutter 3.7.6 on Chrome gives this error (FirebaseError: Installations: Could not process request. Application offline. (installations/app-offline).
Steps to reproduce
Steps to reproduce the behavior:

Debug the example firebase_analytics project on Chrome (https://github.com/firebase/flutterfire/tree/master/packages/firebase_analytics/firebase_analytics)/example/)

Expected behavior
Should not expect to see this error
Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[√] Flutter (Channel stable, 3.7.6, on Microsoft Windows [Version 10.0.22000.1455], locale en-GB)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[√] Chrome - develop for the web
[!] Visual Studio - develop for Windows (Visual Studio Community 2022 17.2.1)
    X Visual Studio is missing necessary components. Please re-run the Visual Studio installer for the "Desktop development with C++" workload, and include these components:
        MSVC v142 - VS 2019 C++ x64/x86 build tools
         - If there are multiple build tool versions available, install the latest
        C++ CMake tools for Windows
        Windows 10 SDK
[√] Android Studio (version 2021.2)
[√] VS Code (version 1.76.0)
[√] Connected device (3 available)
[√] HTTP Host Availability

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.19.3
Flutter SDK 3.7.6
firebase_analytics_example 0.0.0

dependencies:
- firebase_analytics 10.1.4 [firebase_analytics_platform_interface firebase_analytics_web firebase_core firebase_core_platform_interface flutter]
- firebase_core 2.7.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- flutter 0.0.0 [characters collection js material_color_utilities meta vector_math sky_engine]

transitive dependencies:
- _flutterfire_internals 1.0.16 [collection firebase_core firebase_core_platform_interface flutter meta]
- async 2.10.0 [collection meta]
- boolean_selector 2.1.1 [source_span string_scanner]
- characters 1.2.1
- clock 1.1.1
- collection 1.17.0
- fake_async 1.3.1 [clock collection]
- firebase_analytics_platform_interface 3.3.21 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_analytics_web 0.5.1+12 [_flutterfire_internals firebase_analytics_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- firebase_core_platform_interface 4.5.3 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_core_web 2.2.1 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters collection js matcher material_color_utilities meta source_span stream_channel string_scanner term_glyph]
- flutter_web_plugins 0.0.0 [flutter js characters collection material_color_utilities meta vector_math]
- js 0.6.5 [meta]
- matcher 0.12.13 [meta stack_trace]
- material_color_utilities 0.2.0
- meta 1.8.0
- path 1.8.2
- plugin_platform_interface 2.1.4 [meta]
- sky_engine 0.0.99
- source_span 1.9.1 [collection path term_glyph]
- stack_trace 1.11.0 [path]
- stream_channel 2.1.1 [async]
- string_scanner 1.2.0 [source_span]
- term_glyph 1.2.1
- test_api 0.4.16 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- vector_math 2.1.4

@ManishKiranagi
Looking at the error, it seems to be coming from native web sdk and I see a similar issue open in the dedicated repo: https://github.com/firebase/firebase-js-sdk/issues/4922, please follow-up in it for further updates.
Closing from here, as this doesn't seem to be a flutterfire issue.


some picture would disapper when i long press in 'BSImagePickerViewController'

when I bs_presentImagePickerController, I want to preview a picture by long press, then some accidents happend. There are some pictures show in a flash lefting a space, sure also some picture show normally. I don't know what caused this situation

I think... the path: .../PhotosViewController.swift, row 203
let options = PHImageRequestOptions()
                options.isSynchronous = true
                options.isNetworkAccessAllowed = true

the isNetworkAccessAllowed should be setted true. Sometimes the picture may be icloud , if we make true to show, we should allow the iphone to download the photo from iCloud

怎么解决的 我长按图片预览也是空白

Where to add the code

i think so


Bump actions/setup-python from 4 to 5

Bumps actions/setup-python from 4 to 5.

Release notes
Commits

updated-dependencies:

dependency-name: actions/setup-python dependency-type: direct:production update-type: version-update:semver-major ...

already done