Consider changing license?

Hi there,
I had a need for a simple queuing system with swappable storage options. And neoq has worked great!
But I'm developing a commercial product and I've just noticed the agpl license. This means I'm going to have to swap it out for something more permissive.
Before I do, would you consider changing the license to allow commercial use?
Thanks, Phil

Yeah consider it done. I actually need this change for myself, too. Cheers.
On Sat, Jan 27, 2024, at 02:01, Phil Winder wrote:

Hi there,
I had a need for a simple queuing system with swappable storage options. And neoq has worked great!
But I'm developing a commercial product and I've just noticed the agpl license. This means I'm going to have to swap it out for something more permissive.
Before I do, would you consider changing the license to allow commercial use?
Thanks, Phil
—
Reply to this email directly, view it on GitHub https://github.com/acaloiaro/neoq/issues/115, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZNMQFB3YZDXNKL4YJAPPDYQS67BAVCNFSM6AAAAABCNGV3MSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEYDGNBWGA3TAMI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Done! Enjoy

Thank you!


Readme updates:

closes #1654
Populate locations;
Remove deprecated db set up section;
Updated alma exlibris urls;

@hectorcorrea can you check again?


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 651

Questions ?
Please refer to the Backport tool documentation


No need to resizetaskbar on replace right cell.

This only needs to be done when resizing browser window or layouts panel to adjust taskbar depending upon div positioning. This was causing right cell header to move down in all explorers when right cell was replaced.
@dclarizio please review/verify

@epwinchell can you please test.

@dclarizio Tested. Looks good.


iOS - Support Ditto automatic rotating logs in the Log Exporter tool

Today, the Log Exporter tool requires that users turn on Ditto logs to a local file, which they then have to manage to it does not grow unbounded.
Since this tool was initially created, we created an automatic log rotation system which is on by default - the log exporter should ideally support this system and not require users to manage aa additional log file in app code

This should now be possible with 4.8.0, there is a new method on DittoLogger to export the on-disk logs


I am using   on hidden row click I want to close the swipeRow, but only last index is getting closed

If you are using the standalone  you can just keep a ref to the component and call closeRow() on that ref.
could you please give the example for the above sentence.
this is how I am using now (Pseudo code).
RenderItem=>
<SwipeRow  ref={(SwipeRow) => { this.refSwipeRow = SwipeRow; }} >
   <TouchableOpacity onPress={this.refSwipeRow.closeRow()}>
        <Text> Hello </Text>
  </TouchableOpacity>

Hey @sandeep-t2s this is because as each row is rendered you are setting this.refSwipeRow to the newly rendered row. Therefore each new row over writes the previous one and you only keep a reference to the last rendered row.
In order to close a specific row you'll have to keep track of the ref to each SwipeRow, likely in an object.
For example let's say you just have a list of data, and each item has its own id. You could do something like this:

constructor() {
  this.rows = {};
}
...
renderItem={({item}) => {
  <SwipeRow
     ref={ref => { this.rows[item.id] = ref }}
  >
    <TouchableHighlight onPress={() => this.rows[item.id].closeRow()} />
  </SwipeRow>
}

Hope that helps! Going to close this as I do not believe it is an issue with the library itself. Please let me know and re-open if you're still having problems, thanks!


chore(consensus): blanket impl of BlockHeader for deref

Motivation
Simplifies plugging in a new type that derefs to an existing type that impl trait BlockHeader
Solution
Add blanket impl of BlockHeader for all types that deref to a type that impl BlockHeader
PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

is this behavior not covered by deref coercion?

when you call a trait method yes, but not when you use it as param or function return type, if I'm not mistaken


Fix define syntax

Fix define syntax in enabling-an-assembly-in-an-application-without-extensions.md

@iGustL : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add support for detecting Windows 11

Is your feature request related to a problem? Please describe.
We are using the Platform property, but that does not distinguish between Windows 10 and 11.
Describe the solution you'd like
We want the Platform to output Windows 11 if the user runs on a Windows 11 computer.
Details how to detect Windows 11 are outlined here:
https://learn.microsoft.com/en-us/microsoft-edge/web-platform/how-to-detect-win11
Additional context
Using the latest .net 8 version I get these 2 outpus:
Windows 10:
`Browser: Chrome <IP_ADDRESS>
Crawler: Unknown 0.0
Device.type: Desktop
Engine: Blink
Platform: Windows 10.0 x64
UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36`
Windows 11:
`Browser: Chrome <IP_ADDRESS>
Crawler: Unknown 0.0
Device.type: Desktop
Engine: Blink
Platform: Windows 10.0 x64
UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36`

Todo later

This issue will be a massive undertaking to implement for any minor changes for 8.X. But it will enhance the Detection to another level of capability. Getting additional data from the request HTTP headers would require a rewrite of the whole library., while it may be a good thing. I will be pending this issue to be resolved in the next major version of 9.X.
https://learn.microsoft.com/en-us/microsoft-edge/web-platform/how-to-detect-win11


Update inheriting-permissions-from-other-AADs AB#1760414

@v-abiss : Thanks for your contribution! The author(s) have been notified to review your proposed change.

AB#1760414

@chcomley : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @chcomley - Could you review my edit for accuracy? I don't see a reference to this new bookmark in the repo, so revising the anchor name should be OK and not break anything. Thanks.

Thanks.


Allow an underscore as the identifier in const items

RFC 2526 was recently stabilized, meaning const _: i32 = 5; is now a valid item.

@matklad I think more work needs to be done in the AST, since ConstDef is declared as a NameOwner in grammar.ron, but it now only has a name sometimes. Perhaps you could point me in the right direction?

We already assume that everything in AST is optional, so this should mostly work as is. However it would be useful to change Name in ConstData in code_model to an Option<Name> in the follow up PR!
bors r+
Thanks!


ES6 Migration 🚗, Modernization 🔨

⚠ Do not merge! Work in progress⚠

[x] Upgrade babel presets
[x] Migrate to eslint-preset-airbnb-base
[x] Added flow support
[ ] Add jest support
[ ] Add e2e tests
[ ] Migrate from grunt -> npm scripts
[ ] Replace lodash with ES6

Just a quick thought, since Node 0.x is end of life, would it be feasible to drop Babel altogether?

node 4 doesn't support >= es6. We still need babel. Also we're using babel will be used for import/export statements and flow (in the future).

I was thinking that it maybe would be easier to develop with require instead of having to use babel for that. I think that that is the only big thing not supported, but I might be wrong. Anyhow, just a thought :)

http://node.green
The coverage of node 5 and 4 requires us to use babel :( There's still a significant number of users on node < 6.


api: Share expected channel name so that it can be reused from runtime

We need the agent communication channel (in case of virtio, not vsock) to be shared through an external package, so that it can be used directly from any runtime. This will prevent from sync issues between agent versions and runtime versions.

@sboeuf 👋

@egernst yes?

This issue looks a bit stale to me. It's not been worked on in over 3 years. Also, I find the "good-first-issue" to be a bit optimistic, @amshinde. Maybe this was actively discussed at the time and is obvious to you, but I find the description of the task so terse I would have no idea what is expected.
Closing for now, @sboeuf please feel free to re-open if you think this is still relevant.

This issue looks a bit stale to me. It's not been worked on in over 3 years. Also, I find the "good-first-issue" to be a bit optimistic, @amshinde. Maybe this was actively discussed at the time and is obvious to you, but I find the description of the task so terse I would have no idea what is expected.
Closing for now, @sboeuf please feel free to re-open if you think this is still relevant.

Since virtio-serial isn't likely to buy used much anymore, this probably isn't that big of a deal.
AFAICT this is just about repackaging source code. Fine with it being closed...

Since virtio-serial isn't likely to buy used much anymore, this probably isn't that big of a deal.
AFAICT this is just about repackaging source code. Fine with it being closed...

Also..... is there a working stable-bot? Or did we rename it to @c3d :D :D

Also..... is there a working stable-bot? Or did we rename it to @c3d :D :D


Auto import quotes sign should be configured by tslint.json for Typescript

VSCode Version: 1.10.2
OS Version: windows 10

Steps to Reproduce:

add to tslint.json

{"rules": {
 "quotemark": [true, "single"],
}}

Add to a.ts
export const a = 1;
Add to b.ts
a
Click on auto suggestion, then import
See imported string with tslint warning about double quotes

Expect:
import with single quotes with no warning

Alternatively, it could check for other imports and use the quote style those do. Perhaps read .editorconfig

I'm really interested in this getting fixed up, but it looks like it's a dup of #21424, which itself was closed as being tracked by Microsoft/TypeScript#13270.

Yes, thanks @devrelm. This is a duplicate of Microsoft/TypeScript#13270 and really should be fixed upstream on the TypeScript side

As @adyavanapalli very well mentioned, you can edit the typescript.json file to get around this issue on /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/typescript/snippets/ for MacOS and C:\Program Files (x86)\Microsoft VS Code\resources\app\extensions\typescript\snippets\ for Windows.
Thx buddy.

adding  "prettier.singleQuote":true, did the trick for me

Howdy @cengkuru ... where did you add prettier.singleQuote?  Seems unrecognized in VS user settings.  Thx!

@bjareczek I presume it is a plugin to the VS/VSC


Actions in response to alerts

Would be great to have possibility to run a script (or http post or something like that) in response to new alert
Is there something like this in plans?
What do you think about this?

Now that the plugin architecture has landed this is pretty trivial - see https://github.com/bonniejools/cabot-alert-twilio for a pretty comprehensive (albeit not terribly well documented) example of how to write your own plugin.

This idea would be improved by the service-level config in #183


Create Login Form

Build react components that are reusable for building forms and provide form validation configurations.

EDIT: Changed this task to building a static login form.


"A copy of the NetherSX2 APK" Shouldn't this be aethersx2 ?

From the README. Linux prerequisites section.

The ability to turn the AetherSX2 apk into NetherSX2 is a recent addition (added in 1.7). Originally NetherSX2-patch was just a batch script that updated existing copies of NetherSX2 by adding newer files. Since I wanted to add more advanced changes, it meant modding NetherSX2 in more ways than simply updating of the files could allow. The patching method I use is also very particular about file hashes matching, so I needed a constant, which is why we're using AetherSX2 4248 as the basis. NetherSX2 is too much of a variable, since generating it by the termux method will always give you a different file hash

@Trixarian I have a request for you, can you help me fix the game Top Gun Combat Zone so that the fps doesn't drop in random places in the game, even in mission 1 the fps drops, I tried different settings and it's the same


Experimental AtomsBase interface

Added an experimental interface to (AtomsBase)[github.com/JuliaMolSim/AtomsBase.jl], along with a new concrete type Atoms which is a subtype of AtomBase.AbstractSystem.

Hello,
currently I am working on an University Project which combines Chemfiles.jl, ExtXYZ.jl and pythons ase to simple load and save functions. This project is private yet.
We intended to create AtomsBase atomic_systems in load functions and in save functions it should be possible to give an AtomsBase system to the save functions which then saves this system using one of the 3 packages.
This would be similar to the way it is done in https://github.com/mfherbst/AtomIO.jl .
Your Experimental AtomsBase interface seems to be great and would make things much easier for us.
It would be possible to just use your load function instead of creating an extra AtomsBase system.
I am just wondering why you aren't using any Units? E.g. i think that in AtomsBase it is needed to instantiate the bounding box and the position of the Atoms with a unit.
We were thinking about trying to implement an AtomsBase interface directly in Chemfiles and ExtXYZ, which you have done already.
We need to finish our project in the next 2-3 weeks and would like to use your AtomsBase interface, which would make things much easier for us.
So I would like to ask you if you intend to accept this pull request or if you wish to change a few things?

Thanks for getting in touch - this has only be paused due to lack of time, not because I have plans to change things.I was thinking that the AtomsBase interface didn't really belong in this repo and should instead be a separate package. What are your thoughts on that? I'd also be happy to add units: for ExtXYZ these would be Angstrom for both bounding box and atomic positions. Would you be interested/willing to collaborate on a new package based on the AtomsBase interface defined in this PR and also unit support? Any thoughts on a good name for that package?

I would be happy to collaborate on this.
A separate package would be a good way to to it. I am not sure about the name but maybe something like AtomsBaseExtXYZ?

OK great, I'll make a new package+repo and move the code in here there.
The name needs to be more flexible than AtomsBase and more general than ExtXYZ. How about plain Atoms.jl? There was a previous package with that name, but it's no longer in use. Might fall foul of the Julia package naming conventions, though.
@mfherbst or @cortner any suggestions?

Hi @jameskermode

Thanks for getting in touch - this has only be paused due to lack of time, not because I have plans to change things.I was thinking that the AtomsBase interface didn't really belong in this repo and should instead be a separate package.

Well depends how you think of it.
(1) In some sense the hope of AtomsBase is that whatever atomic structure you have in julia, you can directly plug it into a follow-up library. A bit like the AbstractArray interface. You don't need to think which package an array type came from, you can just call sum on it or whatever and it works. In that sense ExtXYZ should directly provide the interface and it should be implemented here.
(2) On the other hand if you think of ExtXYZ just as a thin parser which does do as little beyond that as possible, than probably it should not live here. In that case I would actually vouch for having the code directly in AtomsIO (which is the package @JanHab  is currently working on with a few others), where the idea is exactly to be a generic library to provide the right parsing and return the data as AtomsBase-compatible data structures.
My feeling is that (1) is the better option for the ecosystem, but that (2) could be easier to achieve (e.g. question on units, question on names of functions, etc.). So if you @jameskermode don't see a way for (1) to be easily done, I'd say we nick your code and put it in AtomsIO if you don't mind.

Oh and I should add that in case (1) needs modifications in AtomsBase that it would be well worth discussing them, naturally.

OK, I'm happy to take path (1) you suggest above, particularly since it's less work! I'll merge this PR now and tag a release, then please open Issues here if there are further changes needed.
No modifications required in AtomsBase yet unless the flexible-but-fast Atoms type proves sufficiently useful that others would like to use it without needing to depend on ExtXYZ and it's dependent C code, which is something we can return to run the future.

Cool! Thanks for the merge!

v0.1.6 now released in General registry

@JanHab please do open a PR to fix the units issues you mentioned


When char width is too smal error jump up. I'm about Basic Line

Closing this issue since it is repeated at #132


Authentication

Adding more custom authentication.

@AshotN You have renamed and modified the existing file for your example. Instead, create a new file and put your example there.

My mistake... can I edit this commit or do I need to create a new one


Add documentation on go-getter cache

related to #544

@itscaro and this? ready?

@yxxhero ok for me if you want to add some note, feel free to push


🐛 BUG: Need a better error for dev-ing Python modules when missing the python_workers compatibility flag

Which Cloudflare product(s) does this pertain to?
Wrangler core
What version(s) of the tool(s) are you using?
3.43.0
What version of Node are you using?
No response
What operating system and version are you using?
macOS
Describe the Bug
If you're missing the compatibility_flags = ["python_workers"] in your wrangler.toml, you just get an opaque
✘ [ERROR] service core:user:worker: Uncaught Error: internal error

✘ [ERROR] MiniflareCoreError [ERR_RUNTIME_FAILURE]: The Workers runtime failed to start. There is likely additional logging output above.

when trying to wrangler dev.
This should be improved to instead be a useful error message prompting the user to add that flag to their config file.
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
No response

Good good all paranys

It looks like Wrangler is not showing the full output from workerd. When I run workerd on a Python Worker without this compat flag I get:
workerd/jsg/util.c++:291: error: e = workerd/server/workerd-api.c++:399: failed: expected featureFlags.getPythonWorkers(); The python_workers compatibility flag is required to use Python.
stack: [...long stack trace...]
service main: Uncaught Error: internal error

This possibly needs to be fixed in Wrangler. @cloudflare/wrangler thoughts?

I suspect this is being hidden because we hide some error logs that come from workerd which aren't useful to the user. Potentially we're being over zealous here, and hiding too much.
However, I wonder if it'd be a nicer experience to throw an error here much earlier in the process—when we detect that a user has a .py file without the compatibility flag. I think we could do this in readConfig(), where we currently apply the no bundling rules for python workers.


Remove extra comments

Some pages like this consists of extra and unnecessary comments, which makes the code messy.

Try adding meaning full comments too so that it helps other developers

Hey @abhijeet007rocks8 @DhairyaBahl it is a request that please merge PR:#224 before any other PR, because on every PR merge, it creates a merge conflict, since this PR involves changes in most of the javascript files


custom torrc setting

Do it is possible to add custom setting in torrc?
When I try it delete it after apply in control panel setting.
Do you can use
.
instead full path in torrc?

Yes, you can add a custom setting to the torrc file, but the options that can be configured over TCP will be overwritten before starting tor to avoid desynchronization of settings in the GUI and the torrc file.
It is possible to use relative paths, but tor does not like them and writes messages at startup:
[warn] Path for DataDirectory (.\Data\User) is relative and will resolve to D:\TCP\Data\User. Is this what you wanted?
Therefore, it was decided not to use them in the torrc file. Before each launch, the paths in the torrc file are replaced with the correct ones if you moved the TCP directory to another location.

ExitPolicy reject :
is deleted
but not like as be configured over TCP.

ExitPolicy is used in Exit relay server mode to set a Custom ExitPolicy
When using the Bridge (BridgeRelay 1) or Relay (ExitRelay 0) server mode, tor automatically adds: ExitPolicy reject *:*, so there is no need to add this line to the torrc file.
In client mode (tor's default mode), this setting is ignored and has no effect.


[Storage] Add clarification in docstring for create_append_blob function

As title states.

API change check
API changes are not detected in this pull request.


Added a "run cycle overview" to documentation

Description of the Change
Based on issue https://github.com/mochajs/mocha/issues/3419, I have heard the same from several different people, mainly that they have had to develop a mental model for themselves on the execution order (via trial and error).
I believe they would appreciate a small diagram in the docs to explain.
Java highlighting did a good job of isolating flow+events.
Alternate Designs
It could be drawn with a proper flow/UML diagram but feels like this approach gets most of the information across in a succinct fashion.
If people think this is either inaccurate (please let me know how) or a messy format to show this in, I am happy to use a tool to draw it. I did not want to invest hours in drawing it if people were not going to accept it.
Applicable issues

Is this a breaking change (major release)? - Nope
Is it an enhancement (minor release)? - Nope
Is it a bug fix, or does it not impact production code (patch release)? - Yes (does not impact production code)

Coverage increased (+0.05%) to 90.289% when pulling 974b891b60947089a766c820b68fd02fccab0a35 on craigtaub:runCycleDocs into 2bb4124f1b6b0f0bc8e1c76256c68e8bdb44fe9d on mochajs:master.

@outsideris thanks for feedback, changes pushed (think we can ignore that coveralls failure).

@outsideris are you able to merge this please?

Modify your PR above as follows:

Replace the existing content of the "Applicable Issues" section with the following text (so the issue will be closed automatically upon merge and annotated as eligible for patch release).

Fixes #3419
(semver-patch)
(documentation-only)

@emahuni, does this PR address your concerns?

@plroebuck ah cool, done.

Yes, that's brilliant!

If @plroebuck approved, I can merge it.

@outsideris, merge this


How much time does it take to convert Waymo to KITTI format?

Thank you for amazing work. I wanted to know how much time does it take to convert Waymo to KITTI format using the script
python converter.py --save_dir datasets/waymo_open_organized/ --split validation

The validation one seems to take a lot of time on my machine, and so wanted to confirm.

Yes, it will take a lot of time to convert Waymo dataset to KITTI format, and the numer of processes you set (argument --num_proc) also influences the convert time.

@Willy0919 Thankyou for your response. It was taking a lot of time, and this made me think that I might be doing wrong on my end. Your answer confirms that I am doing it correctly on my end.


Clicking weird reaction created a weirder reaction

I wanted to give nostr:nevent1qvzqqqqqqypzp978pfzrv6n9xhq5tvenl9e74pklmskh4xw6vxxyp3j8qkke3cezqy88wumn8ghj7mn0wvhxcmmv9uqzpvtckkrnl4ntys2zljmv732mzatwcrsmjnr7sqxmc8r7smw877evr7ykgh a :rofl: and noticed somebody had already given something like that so I decided to click that instead. Instead of the expected counter appearing with a "2", it now shows the weird reaction twice:

When I hover over them, they read :Emoji1-16: and ::Emoji1-16:: respectively.

oh... I have no explanation for that bug. but I can reproduce on my client too so it shouldn't be difficult to fix
Added to the to-do list :+1:


Issue in the Wiki

On the Customizing Themes page in the Wiki on the very bottom, there is a mistake in the CSS-Import in the example code.

Please assign this issue to me

Please assign this issue to me

@Kedarnath-Rothe unless you have the rights to push to this repo, this would be useless. This repo has been inactive for 2 years, it looks like this project is abandoned. (no offense to anyone, this happens regularly to open source projects)


Weekly focus: 2018-04-16

Weekly focus PR: please fill in your project-specific highlights from last week and focus for this week. I'll merge this Monday evening, Eastern time.

[x] @annthurium
[ ] @as-cii
[x] @daviwil
[x] @jasonrudolph
[x] @kuychaco
[x] @maxbrunsfeld
[ ] @nathansobo

@nathansobo and @as-cii, anything to add here on the Xray front? There's a little extra leeway this week because I'm feeling sick so I'm popping in and out of availability 😷
Also, moving forward - you already do a super detailed writeup in atom/xray... I should just link that and stop bugging you with these.

Yeah I’m sorry we let lapse this week. Maybe merge without us and I can toss in a link tomorrow.

Thanks @as-cii 🙇


open typescript  semantic  failed

TabNine was unable to install the typescript semantic completion backend.
The command that failed was: `npm install -g typescript-language-server`
For help, see tabnine.com/semantic.
Or, type TabNine::no_sem to disable semantic completion for TypeScript.
(TabNine works well even without semantic completion.)

I install  the language server  manually.
npm install -g typescript-language-server
C:\Users\z2665\AppData\Roaming\npm\typescript-language-server -> C:\Users\z2665\AppData\Roaming\npm\node_modules\typescript-language-server\lib\cli.js
+<EMAIL_ADDRESS>removed 15 packages and updated 18 packages in 8.747s

and  I sure it in the PATH
but TabNine still doesn't work well

see issues
zxqfl/tabnine-vscode#45


Show network in strategies modal on proposal page

We should show the network here and use it for the exploreUrl (not it take the network for the space and fail)

@bonustrack copied this from the other issue... I don't understand: "(not it take the network for the space and fail)". You can edit above and delete this comment.

@mktcode Refer this https://github.com/snapshot-labs/snapshot/issues/1980


Feature Request: radio button attribute

A common request for the v8 Form block has been the ability to add a radio button answer type.
In 5.7, the Form block offered the option of adding a "Radio Buttons" answer type and users have expectations of it being available in the v8 Form block. With the 5.7 Form block now being legacy and not offering as many options as the v8 Form block, having radio buttons as an option would be welcomed and useful.
As a standard form element, I believe new users of concrete5 will also be expecting radio buttons to be available.

This is resolved by this: https://github.com/concrete5/concrete5/pull/5964
It does a few things:

Changes the name of the Select attribute to "Option List" which is more
accurate, and does a better job of encompassing all the various use cases
of the attribute.
Adds the ability to display multiple options even when only choosing a
single, which will display as a radio button list.
Modifies sample content to use this functionality.

On Wed, Sep 6, 2017 at 3:22 AM, Karl Dilkington<EMAIL_ADDRESS>wrote:

A common request for the v8 Form block has been the ability to add a radio
button answer type.
In 5.7, the Form block offered the option of adding a "Radio Buttons"
answer type and users have expectations of it being available in the v8
Form block. With the 5.7 Form block now being legacy and not offering as
many options as the v8 Form block, having radio buttons as an option would
be welcomed and useful.
As a standard form element, I believe new users of concrete5 will also be
expecting radio buttons to be available.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/concrete5/concrete5/issues/5932, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgNwW_aszxvFAj_xF4KOG-2aft7j5t0ks5sfnJQgaJpZM4POJ5M
.


fixed small errors in docs

Codecov Report

Merging #2443 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2443   +/-   ##
=======================================
  Coverage   80.01%   80.01%
=======================================
  Files          85       85
  Lines        8933     8933
  Branches     1914     1914
=======================================
  Hits         7148     7148
  Misses       1511     1511
  Partials      274      274

Continue to review full report at Codecov.

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


[firestore-send-email] Allow specifying from name in "Default FROM address" in configuration

Extension name
This feature request is for extension: firestore-send-email
What feature would you like to see?
I would like to be able to specify a name in the "Default FROM address" configuration field.
Currently only a raw email address passes the validation rules for the field.
This could either be done as a separate configuration option or by allowing email addresses of the following format: "My App<EMAIL_ADDRESS>Specifying email addresses of the format "My App<EMAIL_ADDRESS>works when manually specifying the 'from' field, so I believe this just change is just to modify what is allowed for the configuration of the "Default FROM address" option.
How would you use it?
This would allow the default from address to be more user friendly, instead of just the raw email.

@Ehesp That looks good to me


[ENH] Layout Query Plan and Executor interface in frontend

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

...

New functionality

...

Test plan
How are these changes tested?

[ ] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs repository?

#2920  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Sicheng-Pan and the rest of your teammates on  Graphite

Merge activity

Oct 14, 1:34 PM EDT: A user started a stack merge that includes this pull request via Graphite.


npm install buttercup throws an error: fs-symlink not found.

As per this issue: https://github.com/buttercup-pw/buttercup/issues/44

Republishing on npm needed. @perry-mitchell

Published. Will close. Thanks @sallar


Only send downnode message to OS if there's non PRS collection in the node (9.1)

Cherrypick from #81

Cherrypick from #81

You can actually hold on this and just cherry pick directly to fs/branch_9_1 once merged to fs/branch_9x so that the merge commit from branch_9x is the one cherry-picked.


[Storage] Added support to Key vault and updated rest api to 2017-06-01

Description

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

Can one of the admins verify this patch?

https://github.com/Azure/azure-rest-api-specs/pull/1249

@JasonYang-MSFT PAssumign thsi is destined for Azure PowerShell, please retarget this at the psSdkJson branch

@markcowl Already re-targeted to psSdkJson6 and updated tests.

@JasonYang-MSFT once swagger spec is merged, remove the do-not merge flag and add need review flag

@shahabhijeet
Swagger PR is already merged.

@azuresdkci add to whitelist

@markcowl
Comments are either replied or addressed. Please take a look.

@shahabhijeet
Your comments are addressed.

@JasonYang-MSFT tests failing

@markcowl Comments are addressed.

@shahabhijeet
Tese failure fixed.


Extension won't load in WSL

Environment data
dotnet --info output from WSL:
.NET SDK (reflecting any global.json):
 Version:   6.0.107
 Commit:    f6ec7508e3

Runtime Environment:
 OS Name:     fedora
 OS Version:  35
 OS Platform: Linux
 RID:         fedora.35-x64
 Base Path:   /usr/lib64/dotnet/sdk/6.0.107/

global.json file:
  Not found

Host:
  Version:      6.0.7
  Architecture: x64
  Commit:       0ec02c8c96

.NET SDKs installed:
  3.1.421 [/usr/lib64/dotnet/sdk]
  6.0.107 [/usr/lib64/dotnet/sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 3.1.27 [/usr/lib64/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.7 [/usr/lib64/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 3.1.27 [/usr/lib64/dotnet/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.7 [/usr/lib64/dotnet/shared/Microsoft.NETCore.App]

Download .NET:
  https://aka.ms/dotnet-download

Learn about .NET Runtimes and SDKs:
  https://aka.ms/dotnet/runtimes-sdk-info

VS Code version: 1.70.2 (user setup)
C# Extension version: 1.25.0
OmniSharp log
Empty
C# Log
[ERROR]: C# Extension failed to get platform information.
Log (Remote Extension Host)
[2022-08-18 07:24:03.184] [exthost] [error] Activating extension ms-dotnettools.csharp failed due to an error:
[2022-08-18 07:24:03.184] [exthost] [error] TypeError: Cannot read properties of undefined (reading 'isWindows')
	at /root/.vscode-server/extensions/ms-dotnettools.csharp-1.25.0-linux-x64/dist/extension.js:2:683605
	at Generator.throw (<anonymous>)
	at a (/root/.vscode-server/extensions/ms-dotnettools.csharp-1.25.0-linux-x64/dist/extension.js:2:680954)

Steps to reproduce

Reload Window
Look at errors in logs

Expected  behavior
Expected C# extension to load
Actual behavior
It doesn't load at all.
Additional context
The csharp: Report and Issue command led to this:

A lot has changed with C# extension since this issue was opened. If you are still having problems, please open a new issue.


Execute the gattclient.py

Hi,
I have installed PyBT. However, I could not figure out what is the usage of gattclient.py? Any help would be appreciated!

Hello,
did you find the solution? I have the same problem


Run e2e tests from the image

Enables to run e2e tests from image, not source code.
Depends on https://github.com/Azure/ARO-RP/pull/1114

/azp run e2e

@jim-minter - There were reasons why it was structured like it was before. I redid this a bit. Still left ENV files as long way is very long and it creates data duplication.

As usual, this is fiddlier than it seems...  I wonder if we should be setting our sights higher here.  I've add a commit with some thoughts at https://github.com/Azure/ARO-RP/compare/master...jim-minter:versioned.tests , I think we should talk this over on Wednesday.
I'm going to open a separate expedited fix to sort out our current PROD breakage for the meantime.

/azp run e2e

/azp run e2e

/azp run e2e

/azp run e2e

/azp run e2e

And there was much rejoicing.


ci: run tests with full optimizations (PROOF-620)

Rationale for this change
Run tests with full optimizations on to catch more issues early.
What changes are included in this PR?
Adds a new test job to CI.
Are these changes tested?
Yes.

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


Fixed HBH ext parse panic when start >= end

The router could crash if a packet arrived with data resulting in the function go/border/rpkt/extn_scmp.go/rSCMPExtFromRaw(...) receiving a start >= end, which would panic on line 38.
The error is caused by go/border/rpkt/parse.go/rp.parseHopExtns() calling rp.extnParseHBH(...) on line 123, which then calls rSCMPExtFromRaw(...) with no checks for start or end.
I decided to add an error check in rp.extnParseHBH(...) instead of where the panic happens (rSCMPExtFromRaw(...)), because a header of invalid offset or length should fail, even when start or end are not read, such as in rOneHopPathFromRaw.
The problem might also be fixed earlier, in rp.parseHopExtns(), where the erroneous values are actually read in. However, I was unsure if this is better and so left the error catching in rp.extnParseHBH(...)
I also chose to use as an error check that the total length of the extension must be greater than common.LineLen, since an extension of any less size should not be valid anyway.
For reference: The packet that caused the panic was:
00c10067070405000001ff00000001110001ff000000011080007f00000b0000857b3362e3400140003f0000014152c1000000000000000000000000000000001101010000000000792c000000271f920000001b10084002110582510801027f10065002020102

This change is

Thank you @sgmonroy and @matzf for those excellent and extensive comments. I have made changes and issued a new commit.
The changes are as follows:

extnParseHBH now has an assert that end-start >= common.LineLen.
Following the comments here, I added an additional check that returns an error if *offset+2 >= len(rp.Raw), to ensure that hdrLen and currExtn cannot panic when accessing rp.Raw[*offset+2] and rp.Raw[*offset+1]. For this case, I copied the error message from below that the Extension header chain is too long.
I moved the check for start >= end to rp.parseHopExtns(), as suggested, since I believe this possible panic is not caught by the *offset+2 >= len(rp.Raw) check. The function call to rp.extnParseHBH was adjusted and two variables were added so as to avoid double calculation of start and end
An additional error message was created called ErrExtChainCorrupt for the case in which the ExtHdrLen field is corrupt, even if the extension header chain is long enough.

Please let me know any other feedback you have or if I've misunderstood something, made a mistake, or if there is something else I can do.

Hi and thanks for the comments once again. Sorry it took so long to make these changes. I have issued the commit. Let me know if there's something else I can do.

Thanks for the correction, adjusted it accordingly.


Backport: Update eks versions 2.3

Issue:
https://github.com/rancher/rancher/issues/28385

https://github.com/rancher/rancher/issues/28385


Implement methods for actions

So far the client is only focused on things, it should now also support actions.

[ ] create_action
[ ] create_action_in_batch
[ ] patch_action
[ ] add_reference_to_action
[ ] action_exists
[ ] get_action
[ ] delete_action
[ ] delete_reference_from_action
[ ] ActionsBatchRequest


Customize NodeBuilder::launch

Describe the feature
Abstract the launch method so NodeBuilder can have a different launch sequence.
I would like to specify which components are launched by my node an how they are launched. This allows me to implement a different consensus engine.
Additional context
Ideally, this is a step towards custom consensus for the full node.

closing: https://github.com/paradigmxyz/reth/pull/7388#issuecomment-2077599382


Add the language Simplified Chinese

Thank you for the submission! Although I did just realize i forgot to add translation strings for the area selector prompts, would you mind adding translations for these two?
"bcbui.areaSelection.start": "[Started Area Selection]",
"bcbui.areaSelection.end": "[Area Selector pasted to clipboard]"

Certainly

Lovely, thank you very much!


Update the communication plan

Timeline: https://docs.google.com/document/d/1kP1cL1E7KpcKBuXKkWkqJXqW8MDjI1CaU78LXRTNs6g/edit#heading=h.rhrxh7didtzl

Updates to the communication timeline, reflecting our latest plans for our internal beta (end of August) and cutover (end of October) goals. We'll still plan to communicate our Year End 2023 plans for the full Rikolti cutover, but key updates include:

Mid-August: Rikolti mailchimp update
Mid-September: Roadmap Zoom update (Rikolti + Matomo + other priorities)
Oct/Nov: Rikolti mailchimp update
Nov/Dec: Roadmap Zoom update (Rikolti + other priorities)


fix issue 14717 - Ddoc macro recursion limit too low

The error message misses file and line information. I'm not familiar with the dmd source, so I have no idea how to go about that. Even without it, a bad error message is better than none.
https://issues.dlang.org/show_bug.cgi?id=14717

I'm at a loss as for why the auto tester fails. Seems to be related to ddmd?

Yes. In Win32, the reported error:
dmacro.d(86): Error: undefined identifier error, did you mean class Error?
means that the generated D source dmacro.d cannot call the error() function because it's not defined. In dmacro.c, you've added a new #include to call that function, so you should also fix magicport.json.
See http://wiki.dlang.org/Updating_Pull_Requests_for_DDMD

In dmacro.c, you've added a new #include to call that function, so you should also fix magicport.json.
See http://wiki.dlang.org/Updating_Pull_Requests_for_DDMD

Thanks. Did that and rebased. Auto tester is happy now.

Auto-merge toggled on

LGTM.


add a script for easy cluster setup

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

LGTM

Thanks @perifaws  for reviewing and @sean-smith  for the comments. I'll address the comments today.

Thanks Sean. I merge this and create another PR on the customization.


Backpack goes back into inventory instead of trinket slot.

I'm currently using the Backpacked mod, and after my character dies and retrieves the grave, the backpack doesn't return to the proper slot. It will be placed instead in an empty inventory slot. The backpack should have contents before dying.
Steps to replicate:

Wear backpack from Backpacked mod
Put items inside the backpack
Die by any means with the backpack still on
Go and retrieve the grave

Seems this happens because backpacked is saying "this item can not be unequipped", and yigd sees this as "you will never be able to take this item off your body until you die" (like curse of binding). So to stop this from happening, the backpack will be placed in your inventory instead.
You can disable the treatBindingCurse config as a temporary fix, while I look for a better solution the issue (do note this will cause curse of binding armor to get stuck to you, even when you die)

Or disable the backpacked lockIntoSlot config. That would work too ^


[OB3] Improvements and Changes to Consent Manager Self-Care Portal

[OB3] Improvements and Changes to Consent Manager Self-Care Portal

This pull request contains changes to UI of consent manager self-care portal, relevant changes have been done in order to rectify the issues of https://github.com/wso2/financial-open-banking/issues/48
Following changes to the UI of Consent Manager Self Care Portal has been done.

Introduces Grid layout for the key dates info tab
Fixed screen responsive accounts info tab
Fixed Download confirmation of consent link
Fixed shadows of the button

Key Dates and Account Info (Before)

Key Dates and Account Info (After Changes)

Profile Details Container (Before)

Profile Details Container (After)

Issue link: https://github.com/wso2/financial-open-banking/issues/48
Doc Issue: Optional, link issue from documentation repository
Applicable Labels: OB 3.0.0

Development Checklist

[ ] Build complete solution with pull request in place.
[ ] Ran checkstyle plugin with pull request in place.
[ ] Ran Findbugs plugin with pull request in place.
[ ] Ran FindSecurityBugs plugin and verified report.
[ ] Formatted code according to WSO2 code style.
[ ] Have you verified the PR doesn't commit any keys, passwords, tokens, usernames, or other secrets?
[ ] Migration scripts written (if applicable).
[ ] Have you followed secure coding standards in WSO2 Secure Engineering Guidelines?

Testing Checklist

[ ] Written unit tests.
[ ] Verified tests in multiple database environments (if applicable).
[ ] Tested with BI enabled  (if applicable).

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


Make method svgUrl public

The dataURI only returns PNG data,
https://apexcharts.com/docs/methods/#dataURI
It would be great if it can provide SVG data, so that when generating into PDF or other files, we do not lose the chart quality. I see
there is a method svgUrl, maybe make it public visible?
https://github.com/apexcharts/apexcharts.js/blob/master/dist/apexcharts.js#L9871
Thanks

Hi, @junedchhipa. I think this issue is more than a feature-request. It seems the paper method was removed. So, currently, there is no method to generate SVG from the chart via API. The svgUrl or something similar is needed to make SVG exports via API working again.


href attribute not taking dynamic template data

Issue Summary
I have a dynamic email template for sending a user a unique token link to reset their password. I assign the link to a field named 'link' in the dynamic_template_data section of my personalizations in my request object. When I receive the email, my anchor tag has no href attribute.
Here is my request object:
{
        method: "POST",
        url: "/v3/mail/send",
        headers: {
          "content-type": "application/json"
        },
        body: {
          personalizations: [
            {
              to: [{ email: email, name: name }],
              dynamic_template_data: {
                link: link
              }
            }
          ],
          // some other content
          // ...
          template_id: Constants.EMAIL.RESET_PASSWORD
        },
        json: true
      }

And the html in my sendgrid dynamic template:

Use the following link to reset your password: {{link}}

When I receive the email, the text of the link is correctly filled in with my dynamic content but the  element has no href attribute so it renders as plain text and isn't clickable:

I think you need ta pass the full placeholder, e.g. { dynamic_template_data: { '{{link}}': link } }. Not 100% sure though, give it a try.

No, if that were the case then the text of the link wouldn't be correctly filled in with my dynamic content with the code I had already written.


Change all occurrences of caskroom.io in the docs

Let’s phase out that domain and use just caskroom.github.io. See https://github.com/caskroom/homebrew-cask/issues/29947.

Are there any? I checked using grep, and it seems the only link to caskroom.io I can find in the entire repo is in this cask.

There were some minor occurrences in other repos. Was on a phone at the time so couldn’t check.
All done.

Just noticed while looking at the borg PR for this: The caskroom github organisation still points to the old url.

@textshell Fixed. Thank you.


Create separate directories for different versions of library.proto

There's

annotated library.proto that has been moved to testsrc/libraryproto/annotationsonly
un-annotated library.proto that has been moved to testsrc/libraryproto/configonly, which also contains the library service yaml, gapic yaml, and package yaml
all other *.proto and *.yaml files for test services, that have been moved to testsrc/common

This makes it easy to write separate tests for GapicTestBase2.java that simply use different inputs, GapicCodeGeneratorAnnotationsTest that uses the annotated proto, and GapicCodeGeneratorTest that uses config files without an annotated proto.
Update the test resource paths in all other paths to reflect the test resources directories refactoring.
The contents of *.yaml and *.proto files hasn't changed (although we readded the unannotated library.proto)

PTAL

Codecov Report

Merging #2271 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2271      +/-   ##
============================================
- Coverage     86.76%   86.75%   -0.01%
+ Complexity     5135     5134       -1
============================================
  Files           452      452
  Lines         20398    20398
  Branches       2190     2190
============================================
- Hits          17699    17697       -2
- Misses         1929     1930       +1
- Partials        770      771       +1

Impacted Files
Coverage Δ
Complexity Δ

...en/transformer/php/PhpGapicSurfaceTransformer.java
92.95% <0%> (-0.71%)
41% <0%> (-1%)

Continue to review full report at Codecov.

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

Codecov Report

Merging #2271 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2271      +/-   ##
============================================
- Coverage     86.76%   86.75%   -0.01%
+ Complexity     5135     5134       -1
============================================
  Files           452      452
  Lines         20400    20400
  Branches       2190     2190
============================================
- Hits          17701    17699       -2
- Misses         1929     1930       +1
- Partials        770      771       +1

Impacted Files
Coverage Δ
Complexity Δ

...en/transformer/php/PhpGapicSurfaceTransformer.java
92.95% <0%> (-0.71%)
41% <0%> (-1%)

Continue to review full report at Codecov.

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

Codecov Report

Merging #2271 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2271      +/-   ##
============================================
- Coverage     86.76%   86.75%   -0.01%
+ Complexity     5135     5134       -1
============================================
  Files           452      452
  Lines         20400    20400
  Branches       2190     2190
============================================
- Hits          17701    17699       -2
- Misses         1929     1930       +1
- Partials        770      771       +1

Impacted Files
Coverage Δ
Complexity Δ

...en/transformer/php/PhpGapicSurfaceTransformer.java
92.95% <0%> (-0.71%)
41% <0%> (-1%)

Continue to review full report at Codecov.

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


[Bug]: /class bar command doesn't do anything

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
using /class bar seems to be broken, nothing happens when using the command.
Expected Behavior
Should open something, not sure what it's supposed to open. The wiki says, "Toggles skill bar on/off (if enabled in the config)"
Steps To Reproduce
Type /class bar
Environment
- Server Software: 1.17.1
- ProSkillAPI Version: <IP_ADDRESS>
- ProMCCore Version: <IP_ADDRESS>
- Plugins: https://i.imgur.com/byCrYmR.jpeg

latest.log
No response
Anything else?
No response

I actually haven't been having issues with this. I've been using it in my testing the last couple of days. It merely toggles the skill bar on/off.

I'll just add here that Skill bar.enabled has to be set for true for this to work.


Add a link at the top to Hellcastle's gofundme

Could not figure out what to say in the description, this is the best I could think of

screenshot? this is a UI change


WIP - Add more acceptance tests

Adding more .features
Feedback on level and detail very welcome

Feedback on how far to go describing edit-data feature (i.e. keyboard shortcuts) requested

Cleaned up white space and indents. Yet to reword and fix template.

@mattRedBox I'm learning about present perfect tense for Given (as suggested). So for acknowledge-contributors, would the following be correct / an improvement on you commit?
Questions / Comments in-line below...
  Scenario: Show Contributors
     Given I have opened Data Curator
       # Subject = I
       # Auxiliary Verb = have
       # Past Participle = opened
       # based on http://www.ef-australia.com.au/english-resources/english-grammar/present-perfect/
     When the About menu item is selected
       # So no need to describe menu hierarchy - Data Curator > About?
     Then the Contributors names, logos, attribution statements and links to external websites are shown
     And the application name and version are shown
       # Should this be an And or just added to the Then?
       # Is this And cause to rename the scenario - Show application version and contributors?

  Scenario: I have clicked the mouse to hide Contributors
     Given I have opened Data Curator
     And I have shown the About panel
       # sounds better than "And I am at the About panel"
     When I click the mouse outside the About panel
     Then the About panel closes

@mattRedBox Is my attempt at new-data.feature how you do keyboard shortcuts?

Releasing drafts


[feature] Handle shutdown gracefully

[x] Message to user
[x] Remembering rosh state, etc?
[x] Update instructions page to reflect new reality
[x] Running app without redis should be fine
[x] What happens when event objects are deserialized? do they come back ok?
[x] Rip out redis; try to use volume instead

Couldn't get redis to work. Saving to a file instead :)
Useful links -
https://community.fly.io/t/new-feature-graceful-vm-shutdown-options/504
https://community.fly.io/t/some-issues-with-volumes-on-apps-v2/11994

Holy shirtballs. Needed to change Docker file to run node ./build/index.js instead of npm run start so our node process gets the SIGINT/SIGTERM signals instead of npm. It's working now though.

Restarting production app during a game did not re-establish voice connection OR save data.
Logs running app from commit 26c6704dde37cb5cd58bf85233bfe93d30fe55ea
2023-04-06T16:32:53.387 app[e2865644f6e686] yyz [info] 2023-04-06 16:32:53 info [APP] Shutdown signal received.
2023-04-06T16:32:53.388 app[e2865644f6e686] yyz [info] 2023-04-06 16:32:53 info [APP] Notify 20d594ff682d61c52ed003bad86a05aa708d454fafb0115a3248a8f5c5a7b833 of shutdown
2023-04-06T16:32:53.388 app[e2865644f6e686] yyz [info] 2023-04-06 16:32:53 info [APP] Saving data {"20d594ff682d61c52ed003bad86a05aa708d454fafb0115a3248a8f5c5a7b833":{"studentId":"20d594ff682d61c52ed003bad86a05aa708d454fafb0115a3248a8f5c5a7b833","buyback":"PRIVATE","glhf":"PRIVATE","gold_reminder":"PRIVATE","neutral_item":"PRIVATE","pause":"NONE","roshan":"PUBLIC","runes":"PRIVATE","registerDiscordGuild":"472892759725047814","registerDiscordGuildChannel":"1082789689611792474","allEventsTopic":[{"type":"bounty_rune_pickup","time":248},{"type":"bounty_rune_pickup","time":616},{"type":"bounty_rune_pickup","time":929},{"type":"bounty_rune_pickup","time":987},{"type":"tip","time":1169},{"type":"bounty_rune_pickup","time":1331},{"type":"bounty_rune_pickup","time":1639},{"type":"bounty_rune_pickup","time":1638},{"type":"tip","time":1642},{"type":"bounty_rune_pickup","time":1706},{"type":"bounty_rune_pickup","time":1712},{"type":"aegis_picked_up","time":1764},{"type":"roshan_killed","time":1764}],"lastRemindedGoldTopic":2052,"roshanMaybeTimeTopic":1997,"roshanAliveTimeTopic":2177}}
2023-04-06T16:32:53.389 app[e2865644f6e686] yyz [info] Sending signal SIGINT to main child process w/ PID 521
2023-04-06T16:32:53.389 app[e2865644f6e686] yyz [info] 2023-04-06 16:32:53 info [APP] Http server closed.
2023-04-06T16:32:53.852 proxy[e2865644f6e686] yyz [error] instance refused connection. is your app listening on <IP_ADDRESS>:8080? make sure it is not only listening on <IP_ADDRESS> (hint: look at your startup logs, servers often print the address they are listening on)
Failed to proxy HTTP request (error: no known healthy instances found for route tcp/80. (hint: is your app shutdown? is there an ongoing deployment with a volume or using the 'immediate' strategy? if not, this could be a delayed state issue)). Retrying in 20 ms (attempt 1)
2023-04-06T16:32:58.173 proxy[e2865644f6e686] yyz [warn] Failed to proxy HTTP request (error: no known healthy instances found for route tcp/443. (hint: is your app shutdown? is there an ongoing deployment with a volume or using the 'immediate' strategy? if not, this could be a delayed state issue)). Retrying in 1000 ms (attempt 10)
2023-04-06T16:32:58.410 app[e2865644f6e686] yyz [info] Sending signal SIGTERM to main child process w/ PID 521
2023-04-06T16:32:58.411 app[e2865644f6e686] yyz [info] 2023-04-06 16:32:58 info [APP] Shutdown signal received.
Failed to proxy HTTP request (error: no known healthy instances found for route tcp/80. (hint: is your app shutdown? is there an ongoing deployment with a volume or using the 'immediate' strategy? if not, this could be a delayed state issue)). Retrying in 985 ms (attempt 10)
Failed to proxy HTTP request (error: no known healthy instances found for route tcp/80. (hint: is your app shutdown? is there an ongoing deployment with a volume or using the 'immediate' strategy? if not, this could be a delayed state issue)). Retrying in 23 ms (attempt 1)
2023-04-06T16:33:02.361 app[e2865644f6e686] yyz [warn] Virtual machine exited abruptly
2023-04-06T16:33:02.951 app[e2865644f6e686] yyz [info] Starting init (commit: 9e69c24)...
2023-04-06T16:33:02.964 app[e2865644f6e686] yyz [info] Mounting /dev/vdb at /data w/ uid: 0, gid: 0 and chmod 0755
2023-04-06T16:33:02.967 app[e2865644f6e686] yyz [info] Preparing to run: `node ./build/index.js` as root
2023-04-06T16:33:02.980 app[e2865644f6e686] yyz [info] 2023/04/06 16:33:02 listening on [fdaa:1:c218:a7b:f0:1117:50b7:2]:22 (DNS: [fdaa::3]:53)
2023-04-06T16:33:03.977 app[e2865644f6e686] yyz [info] 2023-04-06 16:33:03 info [APP] Starting server on http://<IP_ADDRESS>:8080
2023-04-06T16:33:04.548 app[e2865644f6e686] yyz [info] 2023-04-06 16:33:04 info [DISCORD] Discord ready with bot: dota-coach#9402
2023-04-06T16:33:04.549 app[e2865644f6e686] yyz [info] 2023-04-06 16:33:04 info [APP] Reading data
2023-04-06T16:33:04.700 proxy[e2865644f6e686] yyz [error] could not make HTTP request to instance: error from user's HttpBody stream: error reading a body from connection: end of file before message length reached
2023-04-06T16:33:04.706 app[e2865644f6e686] yyz [info] BadRequestError: request aborted
2023-04-06T16:33:04.706 app[e2865644f6e686] yyz [info] at IncomingMessage.onAborted (/app/node_modules/raw-body/index.js:238:10)
2023-04-06T16:33:04.706 app[e2865644f6e686] yyz [info] at IncomingMessage.emit (node:events:513:28)
2023-04-06T16:33:04.706 app[e2865644f6e686] yyz [info] at IncomingMessage._destroy (node:_http_incoming:224:10)
2023-04-06T16:33:04.706 app[e2865644f6e686] yyz [info] at _destroy (node:internal/streams/destroy:109:10)
2023-04-06T16:33:04.706 app[e2865644f6e686] yyz [info] at IncomingMessage.destroy (node:internal/streams/destroy:71:5)
2023-04-06T16:33:04.706 app[e2865644f6e686] yyz [info] at abortIncoming (node:_http_server:744:9)
2023-04-06T16:33:04.706 app[e2865644f6e686] yyz [info] at socketOnClose (node:_http_server:738:3)
2023-04-06T16:33:04.706 app[e2865644f6e686] yyz [info] at Socket.emit (node:events:525:35)
2023-04-06T16:33:04.706 app[e2865644f6e686] yyz [info] at TCP.<anonymous> (node:net:317:12)
2023-04-06T16:35:36.297 app[e2865644f6e686] yyz [info] 2023-04-06 16:35:36 info [DISCORD] Handling slash command interaction coachme
2023-04-06T16:35:36.305 app[e2865644f6e686] yyz [info] 2023-04-06 16:35:36 info [DISCORD] Discord ready with guild: hk's place channel: general
2023-04-06T16:35:36.517 app[e2865644f6e686] yyz [info] 2023-04-06 16:35:36 info [DISCORD] VoiceConnection ready to play audio for student 20d594ff682d61c52ed003bad86a05aa708d454fafb0115a3248a8f5c5a7b833!

I think it's fine now...
We are registering topics so we can find the correct topic to serialize the data with

Topics could be registering themselves from the constructor, but that brings some questionable dependencies to Topic ... need to ponder this further

Removed restart notification in 36a24399e529d9b88add80bddae731b8f4901400


Ensure Homebrew is installed: NoneType object has no attribute version

Just trying to get started installing homebrew on macOS Sierra 10.12.1 with this play:
- name: Install brew
  hosts: all_osx_hosts
  vars:
    homebrew_installed_packages:
      - ant
      - cmake
      - jq
  roles:
    - geerlingguy.homebrew
  tags:
    - debug

This fails with
Traceback (most recent call last):
  File "/var/folders/jm/3n8_ksbs4fz49dlz_mlchtgw0000gn/T/ansible_HPa4di/ansible_module_git.py", line 1040, in <module>
    main()
  File "/var/folders/jm/3n8_ksbs4fz49dlz_mlchtgw0000gn/T/ansible_HPa4di/ansible_module_git.py", line 923, in main
    if depth is not None and git_version_used < LooseVersion(1.9.1):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/distutils/version.py", line 296, in __cmp__
    return cmp(self.version, other.version)
AttributeError: NoneType object has no attribute version

at which point I don't even know where to being looking for the cause.
How can I get this work ?

@z00m1n - What version of Ansible are you running? Also, which task does this error come up on? It could be an improperly set variable, or it could be an issue with an older Ansible version.

What version of Ansible are you running ?

I've installed Ansible using Homebrew; AFAICT, everything is at the latest released version:
ansible-control-host:ansible username$ uname -a
Darwin ansible-control-host.domain 16.1.0 Darwin Kernel Version 16.1.0: Thu Oct 13 21:26:57 PDT 2016; root:xnu-3789.21.3~60/RELEASE_X86_64 x86_64

ansible-control-host:ansible username$ brew upgrade && brew update && brew doctor
Already up-to-date.
Your system is ready to brew.

ansible-control-host:ansible username$ ansible --version
ansible <IP_ADDRESS>
  config file = <project folder>/ansible/ansible.cfg
  configured module search path = Default w/o overrides

Also, which task does this error come up on ?

Ensure Homebrew is installed, as mentioned in the subject; I should have made that clearer. Full output, somewhat wrapped for readability:
ansible-control-host:ansible username$ ansible-playbook ci-infrastructure.yml --limit vm-osx-1012 --tags debug

PLAY [Install brew] ************************************************************

TASK [setup] *******************************************************************
ok: [vm-osx-1012]

TASK [geerlingguy.homebrew : Get current account's username.] ******************
[DEPRECATION WARNING]: always_run is deprecated. Use check_mode = no instead..
This feature will be removed in version 2.4. Deprecation
warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.
ok: [vm-osx-1012]

TASK [geerlingguy.homebrew : Ensure Homebrew parent directory has correct permissions.] ***
changed: [vm-osx-1012]

TASK [geerlingguy.homebrew : Ensure Homebrew directory exists.] ****************
changed: [vm-osx-1012]

TASK [geerlingguy.homebrew : Ensure Homebrew is installed.] ********************
fatal: [vm-osx-1012]: FAILED! => {"changed": false, "failed": true, "module_stderr":
   "Shared connection to vm-osx-1012 closed.\r\n", "module_stdout": "Traceback (most recent call last):\r\n
   File \"/var/folders/jm/3n8_ksbs4fz49dlz_mlchtgw0000gn/T/ansible_6nEutM/ansible_module_git.py\", line 1040,
     in <module>\r\n    main()\r\n
   File \"/var/folders/jm/3n8_ksbs4fz49dlz_mlchtgw0000gn/T/ansible_6nEutM/ansible_module_git.py\", line 923,
     in main\r\n
       if depth is not None and git_version_used < LooseVersion('1.9.1'):\r\n
   File \"/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/distutils/version.py\", line 296,
     in __cmp__\r\n    return cmp(self.version, other.version)\r\nAttributeError:
	   'NoneType' object has no attribute 'version'\r\n", "msg": "MODULE FAILURE"}
	to retry, use: --limit @<path to folder>/ci-infrastructure.retry

PLAY RECAP *********************************************************************
vm-osx-1012                : ok=4    changed=2    unreachable=0    failed=1

Not sure if that's relevant, but the vm-osx-1012 target machine is a fresh system with nothing installed apart from Xcode (from the AppStore), so all tools (git, Python, etc.) involved are those that come with the system; on the ansible-control-host controlling machine, there's some tools from brew, (e.g. python), but system git:
ansible-control-host:ansible username$ brew list
ack		cmake		gdbm		oniguruma	python		sqlite
ansible		coreutils	jq		openssl		readline	uncrustify
bash		findutils	libyaml<EMAIL_ADDRESS>shellcheck

ansible-control-host:ansible username$ which python
/usr/local/bin/python
ansible-control-host:ansible username$ python --version
Python 2.7.12

ansible-control-host:ansible username$ which git
/usr/bin/git
ansible-control-host:ansible username$ git --version
git version 2.9.3 (Apple Git-75)

Hmm... I wonder—since the failed task shows something about the Git version comparison before failing—if Ansible can't see a valid/usable Git binary. When you run the playbook, are you running it on localhost, or on a machine controlled via SSH? If via SSH, can you log into that machine and run which git as well?
If Git's not installed, you should install it/configure it so it's in the $PATH prior to this role.
It could be something else... but that seems the most logical thing to check next.
Also, can you check the Python version on the controlled machine?

I have updated my previous comment to list the required information more clearly.
The target system is a really fresh macOS Sierra installation with no changes were made and no software was installed apart from Xcode, I had expected git and python to be the versions that came with the system.
However, getting the respective versions of git and python on vm-osx-1012 shows the cause of the problem:
ansible-control-host:~ username$ ssh vm-osx-1012
  ...
vm-osx-1012:~ ansible$ which python
/usr/bin/python
vm-osx-1012:~ ansible$ python --version
Python 2.7.10

vm-osx-1012:~ ansible$ which git
/usr/bin/git
vm-osx-1012:~ ansible$ git --version

Agreeing to the Xcode/iOS license requires admin privileges, please re-run as root via sudo.

vm-osx-1012:~ ansible$ sudo git --version

You have not agreed to the Xcode license agreements. You must agree to both license agreements below in order to use Xcode.

Hit the Enter key to view the license agreements at '/Applications/Xcode.app/Contents/Resources/English.lproj/License.rtf'

Apple Inc.

Xcode and Apple SDKs Agreement

( ... long text with ASCII codes (?) interspersed, e.g. <E2><80><9D> ... )
(hit q)

By typing 'agree' you are agreeing to the terms of the software license agreements. Type 'print' to print them or anything else to cancel, [agree, print, cancel] agree

You can view the license agreements in Xcode's About Box, or at /Applications/Xcode.app/Contents/Resources/English.lproj/License.rtf

git version 2.9.3 (Apple Git-75)

The playbook now runs OK:
ansible-control-host:ansible username$ ansible-playbook ci-infrastructure.yml --limit vm-osx-1012 --tags debug

PLAY [Install brew] ************************************************************

TASK [setup] *******************************************************************
ok: [vm-osx-1012]

TASK [geerlingguy.homebrew : Get current account's username.] ******************
[DEPRECATION WARNING]: always_run is deprecated. Use check_mode = no instead..
This feature will be removed in version 2.4. Deprecation
warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.
ok: [vm-osx-1012]

TASK [geerlingguy.homebrew : Ensure Homebrew parent directory has correct permissions.] ***
ok: [vm-osx-1012]

TASK [geerlingguy.homebrew : Ensure Homebrew directory exists.] ****************
ok: [vm-osx-1012]

TASK [geerlingguy.homebrew : Ensure Homebrew is installed.] ********************
changed: [vm-osx-1012]

TASK [geerlingguy.homebrew : Ensure proper permissions on homebrew_brew_bin_path dirs.] ***
changed: [vm-osx-1012]

TASK [geerlingguy.homebrew : Ensure proper ownership on homebrew_install_path subdirs.] ***
ok: [vm-osx-1012]

TASK [geerlingguy.homebrew : Check if homebrew binary is already in place.] ****
ok: [vm-osx-1012]

TASK [geerlingguy.homebrew : Symlink brew to homebrew_brew_bin_path.] **********
changed: [vm-osx-1012]

TASK [geerlingguy.homebrew : Force update brew after installation.] ************
changed: [vm-osx-1012]

TASK [geerlingguy.homebrew : Ensure configured taps are tapped.] ***************
ok: [vm-osx-1012] => (item=homebrew/core)
changed: [vm-osx-1012] => (item=caskroom/cask)

TASK [geerlingguy.homebrew : Ensure configured homebrew packages are installed.] ***
changed: [vm-osx-1012] => (item=[u'ant', u'cmake', u'jq'])

TASK [geerlingguy.homebrew : Upgrade all homebrew packages (if configured).] ***
skipping: [vm-osx-1012]

TASK [geerlingguy.homebrew : Get list of apps installed with cask.] ************
[DEPRECATION WARNING]: always_run is deprecated. Use check_mode = no instead..
This feature will be removed in version 2.4. Deprecation
warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.
ok: [vm-osx-1012]

TASK [geerlingguy.homebrew : Install configured cask applications.] ************
changed: [vm-osx-1012] => (item=firefox)

TASK [geerlingguy.homebrew : Check for Brewfile.] ******************************
ok: [vm-osx-1012]

TASK [geerlingguy.homebrew : Install from Brewfile.] ***************************
skipping: [vm-osx-1012]

PLAY RECAP *********************************************************************
vm-osx-1012                : ok=15   changed=7    unreachable=0    failed=0

Verify brew and the packages were installed correctly:
vm-osx-1012:~ ansible$ brew doctor
Your system is ready to brew.

vm-osx-1012:~ ansible$ which ant cmake jq
/usr/local/bin/ant
/usr/local/bin/cmake
/usr/local/bin/jq

vm-osx-1012:~ ansible$ ant -version
Unable to find any JVMs matching version "(null)".
No Java runtime present, try --request to install.
No Java runtime present, requesting install.
2016-12-12 01:04:24.577 java[431:2782] JLRequestRuntimeInstall: Error calling: CFMessagePortCreateRemote

vm-osx-1012:~ ansible$ cmake -version
cmake version 3.7.1

vm-osx-1012:~ ansible$ jq --version
jq-1.5

ant can't find a working Java installation, but that's beyond the scope of the problem at hand.
Conclusion: For the playbook to work on a brand new macOS Sierra installation, the Xcode license agreements must somehow be agreed to as part of homebrew setup.

@z00m1n - Ah, you're correct. If you run the playbook locally (on the same machine Ansible's running on), then the prompt actually pipes through correctly, and you're prompted to agree in the terminal during the playbook run.
Maybe we can fix this through documentation, or adding an extra step (just to ensure it's done) in the install instructions.

I think this has been fixed in #53

Correct! This shouldn't be an issue anymore, after #53 was merged. Thanks!


counterparty-cli chokes on "market"

I run counterpartyd-cli --testnet market and it chokes on TESTASSETONE:
[INFO] Running v9.49.3 of counterpartyd.←[0m
[INFO] Connecting to database.←[0m
Your Pending Order Matches
+------------------+-----------+
| Matched Order ID | Time Left |
+------------------+-----------+
+------------------+-----------+
←[31m[ERROR] Unhandled Exception←[0m
Traceback (most recent call last):
  File "c:\counterpartyd_build\dist\counterpartyd\counterparty-cli.py", line 109
2, in <module>
    market(args.give_asset, args.get_asset)
  File "c:\counterpartyd_build\dist\counterpartyd\counterparty-cli.py", line 133
, in market order = format_order(order)
  File "c:\counterpartyd_build\dist\counterpartyd\counterparty-cli.py", line 63,
 in format_order give_quantity = util.value_out(db, D(order['give_quantity']), order['give_as
set'])
  File "c:\counterpartyd_build\dist\counterpartyd\lib\util.py", line 380, in val
ue_out divisible = is_divisible(db, asset)
  File "c:\counterpartyd_build\dist\counterpartyd\lib\util.py", line 338, in is_
divisible if not issuances: raise exceptions.AssetError('No such asset: {}'.format(ass
et))
lib.exceptions.AssetError: No such asset: TESTASSETONE←[0m

However there is such an asset
http://testnet.blockscan.com/asset?q=TESTASSETONE

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

This issue was moved to CounterpartyXCP/counterparty-cli#1


Files: Folders and files cannot be moved

Problem
If you select a file/folder and want to move it, this is currently not possible. After selecting the destination and confirming the dialogue, nothing happens. The file/folder remains in place.
An error message is not displayed.

might be related to https://github.com/OpenSlides/openslides-client/issues/1192


Implementing exaone3.5

This is my first attempt at an implementation of exaone into unsloth, it was requested in this issue.
I didn't want to implement exaone into a separate model class because exaone follows the llama architecture as was discussed in this issue. As of now I am having problems with the from_pretrained function when using config and state_dict. I've already opened an issue in transformers about it. I'll try to solve that problem soon.

Thanks for the effort! I was just curious of the general process you go through to implement something like this. Is it different for every model, and how so?

@qingy1337 I was thinking about creating a writeup for this (assuming this gets accepted), as this is my first pull request to a large public repo.
But the gist of it is this:
I started out by googling how other people have implemented exaone into their codebase, and I came up with 3 possible solutions:

Write the exaone class from scratch like they did in vLLM
Upload the exaone models to huggingface as llama models, as was done by maywell
Map exaone to llama, which is what I did, as was recommended in this issue

This is different than previous model integrations because exaone has the exact same architecture as llama


Proposal: add output for successful commands

Especially on publish, it might be helpful to print an extra line stating that the publish was successful.  Maybe something like Successfully published changes to <GUN>?
🐳 $ notary publish repo
Pushing changes to repo
Enter passphrase for targets key with ID 1f4b045:
Enter passphrase for snapshot key with ID 760cfec:
Successfully published changes to repo
🐳 $

Same thing for notary delete, and commands that use -p could use a wrapped Publish() function to standardize printing to CLI

👍 I was add a similar on docker/docker side after a successfully DCT publishing
https://github.com/docker/docker/pull/20307/commits/1a6866273697361f33ec908f51cf0e071a36b69d


[Bug]: Setting Consistent Container Names to true still uses bad container names under the hood

Description
I've set Consistent Container Names to true so that I can run commands in a single container of my docker compose deployment.
However when running with what should be the consistent name web-o0og8ko it fails
ScheduledTaskJob failed: No valid container was found. Is the container name correct?

SSHing into the machine I see that the name is actually c8kckc8-web
This is only affecting Scheduled jobs though, one-off tasks still use the web-o0og8ko name.
Btw, should paying customers be filing bugs, or contacting another way?
Minimal Reproduction (if possible, example repository)
N/A
Exception or Error
No response
Version
v4.0.0-beta.273

Btw, should paying customers be filing bugs, or contacting another way?

Anyone can fill bugs.
To your issue, what kind of deployment type do you use? Docker compose / dockerfile / nixpacks, etc.

Dockerfile and Docker Compose.
It has a larger impact on compose because scheduled task don't require a
name be specified for a single container
However on both the name of the Docker container when running docker ps
isn't the correct name.
On Tue, May 14, 2024, 6:47 AM Andras Bacsai @.***>
wrote:

Btw, should paying customers be filing bugs, or contacting another way?
Anyone can fill bugs.
To your issue, what kind of deployment type do you use? Docker compose /
dockerfile / nixpacks, etc.
—
Reply to this email directly, view it on GitHub
https://github.com/coollabsio/coolify/issues/2122#issuecomment-2109880015,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASDJ625FHF2TICC7DUD7BTTZCHTULAVCNFSM6AAAAABHEHEXUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBZHA4DAMBRGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

In my case its a dockerfile.

For docker-compose based deployments, the persistent container name should look like this:<service_stack_name>-<application_uuid>
And in case you are not using the persistent container name:
<service_stack_name>-<application_uuid>-<random_timestamp>
The container name in the scheduled task selection only have the service_stack_name . So no matter if you enabled the persistent container name or not, the same container name will be used.
I cannot reproduce the issue neither in Dockerfile and Docker Compose deployments. So I'm closing it now.
If you have a step-by-step reproduction with screenshots that the container name is wrong, please reopen the issue.


When i open Run LineChar Activity  I need to toggle area and toggleLabelForSelected.

Done changes for the below variables but still not able to get and getting exception Nullpointer Exception.
   **private boolean isFilled = true;**

    **private boolean hasLabelForSelected = true;**

Please provide more info, logs with stack trace, code samples etc.


Update Readme.md

I am closing this pull request.
The rule of the challenge is that you need to solve only one problem on a day


SUBSTR and LEFT tests failing with Go 1.11

Notice, they work correctly with Go 1.12, but with Go 1.11:
--- FAIL: TestSUBSTRBadOffset (0.00s)
        Error Trace:    vm_test.go:649
        Error:          Not equal:
                        expected: true
                        actual  : false
        Test:           TestSUBSTRBadOffset
--- FAIL: TestSUBSTRBadLen (0.00s)
        Error Trace:    vm_test.go:659
        Error:          Not equal:
                        expected: true
                        actual  : false
        Test:           TestSUBSTRBadLen
--- FAIL: TestLEFTBadLen (0.00s)
        Error Trace:    vm_test.go:694
        Error:          Not equal:
                        expected: true
                        actual  : false
        Test:           TestLEFTBadLen

So I guess it's time for us to drop 1.11 support as it's not supported by upstream either after the release of 1.13.

Refs. #383.

Let's drop it! =)

It seems like go1.11 take subslices based on capacity of underlying slice, not it's length. (run with go1.11 and go1.11.13)
package main
import "fmt"
func main() {
        a := make([]byte, 3, 10)
        b := a[:5] // no panic here in go1.11
        fmt.Println(a)
        fmt.Println(b)
}

@fyrchik: thanks for really digging into the problem. So it looks like according to the spec:

For arrays or strings, the indices are in range if 0 <= low <= high <= len(a), otherwise they are out of range. For slices, the upper index bound is the slice capacity cap(a) rather than the length.

Which makes my code incorrect and working for Go 1.12 and 1.13 only by accident. Thus, this issue is not really solved.

Closing in favor of #387.


Handling Not Activated scenarios during Startup

When a device is not activated for provisioning below items needs to be handled

[x] Bootstrap needs to be updated to set event listeners for Activation
[x] In the event of activation AccountSession value in Platform State needs to be updated.
[x] All the Distributor processors which needs an Account Session needs to be refreshed.

Fixed as part of #258


trim datasource names

trim datasource names
Fixes #ISSUSE_ID.
Changes proposed in this pull request:

Pull Request Test Coverage Report for Build 5546

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 35.999%

Totals

Change from base Build 5545:
0.0%

Covered Lines:
13411

Relevant Lines:
37254

💛  - Coveralls

Can you provide the reason to trim this key?
For my understanding, this key is from spring boot properties, between key and values is properties file, it don't permit space.

ok, I get your point. thank you for contribute code


雷达图类型定义问题

Version
5.1.2
Reproduction link
https://echarts.apache.org/examples/zh/editor.html?c=radar-custom
Steps to reproduce
       const options: EChartsOption = {
           radar: [
               {
                   name: {
                       formatter: '{value}',
                       textStyle: {
                           color: '#72ACD1',
                       },
                   },
               },
           ],
       };

What is expected?
类型正确
What is actually happening?
类型错误

Your code is working. Parameters formatter and textStyle are not documented for radar name. However, if you use this example and replace name with your code or the code below, result is correct.
     name: {
         formatter: 'v= {value}',
         textStyle: { color: 'red' },
     },


[CSL-192] FTS bench unsafe trick

@shersh
See this TODO: https://github.com/input-output-hk/pos-haskell-prototype/blob/744c2a737b30605515a15b9088bc8affa4bcd60e/bench/Bench/Pos/Criterion/FollowTheSatoshiBench.hs#L20

@flyingleafe
It's done long ago, I just forgot to remove this TODO. My bad, will remove.


Disallowed formatting a phone with the 'customValidate' property, remove unnecessary 'disabledFormatting' property

Added default parsed mode in cases when customValidate property is exist. The phone number is redefined to international format, when 'phoneObject.valid' set to true, even if user uses 'customValidate' property.
Removed deprecated property 'disabledFormatting'. Which is no longer in use.

Hi @Alexandernoa, thanks for your help! Can you remove the changes in README-v2.md (because we had the disabledFormatting back then in v2)? Also, the changes in App.vue are unnecessary as well as it's the demo.

@iamstevendao  Okay, sorry for that. I removed my older edits for those files.

Thanks @Alexandernoa !


cilium: don't invoke deprecated cilium daemon

Commit c7d4002534d6 ("Switch to spf13/cobra and spf13/viper to
implement CLI & config") split the daemon and the cli into two
separate binaries, as a result 'cilium daemon' doesn't exist
anymore. There are a few letfover spots in the test suite that
are using 'cilium daemon' and throwing error messages such as:
BENCHMARK=1 ./02-perf.sh
[...]

cilium daemon config DropNotification=false Debug=false
Error: unknown command "daemon" for "cilium"
Run 'cilium --help' for usage.
unknown command "daemon" for "cilium"
[...]
++ cilium daemon config DropNotification=true Debug=true
Error: unknown command "daemon" for "cilium"
Run 'cilium --help' for usage.
unknown command "daemon" for "cilium"
[...]

Just drop the deprecated daemon command from these test cases,
and after that we get:
[...]

cilium config DropNotification=false Debug=false
cilium endpoint config 63464 DropNotification=false Debug=false
[...]

Fixes: c7d4002534d6 ("Switch to spf13/cobra and spf13/viper to implement CLI & config")
Signed-off-by: Daniel Borkmann<EMAIL_ADDRESS>

This pull request is now managed with CodeReviewHub. Refresh.


CoreComponents-CustomConnector throwing error

Describe the bug
A clear and concise description of what the bug is.
Component (Core Component -Custom Connector):

Solution [Core]
Version [e.g. 1.44]
App or Flow [Admin | Sync Template v2 (Custom Connectors)]

To Reproduce
Steps to reproduce the behavior:

Trying to turn ON the custom connector results in the error although all the underlying connectors are already in the BUSINESS DATA..'

Your flow could not turn on: Admin | Sync Template v2 (Custom Connectors). Error: Flow client error returned with status code "Forbidden" and details "{"error":{"code":"DlpViolation","message":"Flow '' cannot be activated due to admin API policies."}}".
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Additional context
Rest of the connectors are UP and running fine. This connector alone is throwing up this error.

The Admin | Sync Template v2 (Custom Connectors) is the only one of the sync flows that uses a HTTP call. Can you confirm HTTP is part of your business data only group?
You can find out how to achieve that here
https://powerapps.microsoft.com/en-us/blog/new-data-loss-prevention-capabilities-in-public-preview/

Brilliant. Worked like a charm after adding http. Was this documented or I missed? Thanks so much.

Thank you for letting us know. This was documentation we missed, but we are adding it now.


Typescript extension crashing

Type: Bug
running in dev container
VS Code version: Code 1.76.0 (Universal) (92da9481c0904c6adfe372c12da3b7748d74bdcb, 2023-03-01T10:23:45.993Z)
OS version: Darwin arm64 22.2.0
Modes:
Sandboxed: No
Remote OS version: Linux arm64 5.15.49-linuxkit
Remote OS version: Linux arm64 5.15.49-linuxkit
Remote OS version: Linux arm64 5.15.49-linuxkit

System Info

Item
Value

CPUs
Apple M1 Pro (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
5, 6, 6

Memory (System)
16.00GB (0.08GB free)

Process Argv
--crash-reporter-id 8b9718a0-036a-4d8d-b8d6-d77c23f3730f

Screen Reader
no

VM
0%

Item
Value

Remote
Dev Container: Node.js & TypeScript @ desktop-linux

OS
Linux arm64 5.15.49-linuxkit

CPUs
unknown (8 x 0)

Memory (System)
3.84GB (0.80GB free)

VM
0%

Item
Value

Remote
Dev Container: Node.js & TypeScript @ desktop-linux

OS
Linux arm64 5.15.49-linuxkit

CPUs
unknown (8 x 0)

Memory (System)
3.84GB (0.80GB free)

VM
0%

Item
Value

Remote
Dev Container: Node.js & TypeScript @ desktop-linux

OS
Linux arm64 5.15.49-linuxkit

CPUs
unknown (8 x 0)

Memory (System)
3.84GB (0.80GB free)

VM
0%

Extensions (22)

Extension
Author (truncated)
Version

istanbul-ignore-code-snippets
Ben
0.0.3

eslint-disable-snippets
drK
1.3.0

vscode-svelte-snippets
fiv
0.5.0

vscode-nginx-conf-hint
han
0.3.0

vscode-drawio
hed
1.6.6

Kotlin
mat
1.7.1

vscode-json5
mrm
1.0.0

vscode-azurevirtualmachines
ms-
0.6.4

remote-containers
ms-
0.282.0

vscode-gatsby-snippets
nic
1.1.2

terraform-advanced-syntax-highlighting
pjm
1.4.0

vscode-nginx
wil
0.7.2

commit-message-editor
ada
0.23.0

vscode-eslint
dba
2.4.0

prettier-vscode
esb
9.10.4

terraform
has
2.25.4

npm-dependency-links
her
1.2.0

npm
ide
1.3.0

vscode-azureterraform
ms-
0.3.2

vscode-docker
ms-
1.24.0

azure-account
ms-
0.11.3

code-spell-checker
str
2.19.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
vsccc:30610678
pyindex848:30662994
nodejswelcome1cf:30587006
282f8724:30602487
pyind779:30671433
89544117:30613380
vscrp:30673768
pythonsymbol12:30671437
vsccsb:30662443
funwalk2cf:30676044

This causes significant CPU use on my system as the processes are constantly being restarted.

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

It does not.

@tompahoward Did you have a chance to test this?

It's interesting that the issue seemed to go away for a few builds, since my last comment, but now, with the latest Insiders it's back again for me. I've got way too many extensions to easily determine where the problem lies.

Sorry no, I haven't tried the insiders build. "With all extensions disabled", do you mean all except the TypeScript and JavaScript Language Features?

Yes, don't disable built-in extensions just the installed ones

In my case, it seems the Svelte extension was the problem. A notification popup suggested it could be causing the restarts.

Hey @mjbvz, this issue might need further attention.
@tompahoward, you can help us out by closing this issue if the problem no longer exists, or adding more information.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


ComposeObject does not work for copying from another region

Stating object in another region gives bad request, and ComposeObject stats sources.
This is my wild temp fix that helped me get through the issue in my local run (I added otherCli to use for stating).

Stating object in another region gives bad request, and ComposeObject stats sources.
This is my wild temp fix that helped me get through the issue in my local run (I added otherCli to use for stating).

Please provide reproducers we cannot imagine your setup.

Sorry if I wasn't clear enough.

I was copying objects from eu-west-1 to eu-central-1.
I couldn't copy objects bigger than 5GB through CopyObject(), so I had to use ComposeObject() with ranges of the source object as sources.
I called ComposeObject() on eu-central-1 client, as I want to compose object there.
ComposeObject() calls StatObject() for each source, and calls it on client on which ComposeObject() was called (eu-central-1 one).
Stating object from another region finishes with 400 Bad Request, so everything fails (as it tries to stat objects from eu-west-1).

I managed to mitigate and do the execute the copy of the objects I needed at the moment with the code change mentioned above, as this was a one-time migration. I will not hit this in the future, but wanted to notify you as minio is a great tool and I wanted to give you a feedback of this broken behavior.
If some part of this was unclear, I can describe further.

I was copying objects from eu-west-1 to eu-central-1.

Are you using ComposeObject() between two separate S3 servers ?

Sorry if I wasn't clear enough.

I was copying objects from eu-west-1 to eu-central-1.
I couldn't copy objects bigger than 5GB through CopyObject(), so I had to use ComposeObject() with ranges of the source object as sources.
I called ComposeObject() on eu-central-1 client, as I want to compose object there.
ComposeObject() calls StatObject() for each source, and calls it on client on which ComposeObject() was called (eu-central-1 one).
Stating object from another region finishes with 400 Bad Request, so everything fails (as it tries to stat object from eu-west-1).

I managed to mitigate and do the execute the copy of the objects I needed at the moment with the code change mentioned above, as this was a one-time migration. I will not hit this in the future, but wanted to notify you as minio is a great tool and I wanted to give you a feedback of this broken behavior.
If some part of this was unclear, I can describe further.

We still need to see the code @bjg2


CharsetConverter::convert should not try to convert numeric value

Issue summary
After updating from 9.1.2 to 9.1.3 my code is no longer working without error.
System informations

Information
Description

League\Csv
9.1.3

PHP
7.1.12

ubuntu
16.04

Standalone code, or other way to reproduce the problem
$formatter = (new Csv\CharsetConverter())
    ->inputEncoding('utf-8')
    ->outputEncoding('Windows-1252');
$csvWriter = Csv\Writer::createFromFileObject(new \SplFileObject('php://output', 'w'));
$csvWriter->addFormatter($formatter);
$csvWriter->insertOne(['str', '1', 2]);

Expected result
No errors.
Actual result
Type error: mb_convert_encoding() expects parameter 1 to be string, integer given at /var/www/vendor/league/csv/src/CharsetConverter.php:217

Bug fixed in the master branch .. a patch version will be released later this week with the bug fix.


Issues in RequestInstantiate / RequestInstantiateAll

Currently RequestInstantiate() calls both RequestFetch() and RequestTranslate(), which itself calls RequestFetch().  It looks like it should just call the RequestTranslate().
Also RequestInstantiateAll() refers to depEntry which is not defined anywhere.

this is a partial duplicated of https://github.com/whatwg/loader/issues/32

fixed in 251c4fd2416c7818c246863aab46e28e20109e94 and 551d06abd9997c9bc845889e584d56fd4cede771


suggested changes

Builds on linux and macos see https://github.com/phanirithvij/datastar/actions/runs/11083989433

i like the idea, but the use of sleep makes it flaky and i'm not fond of parsing lsof with awk. I think a better approach is to somehow probe for an available port before launching httplz.


Add some retry around the wget, in an effort to deflake the test

We occasionally see:
wget꞉ download timed out

So retry wget a few times.

Grumble. I played with this something like this, too. I'll take it, but I don't have to like it. :)


ICommandPublisher not registered with Microsoft.Extensions.DependencyInjection

Hi,
when used with Microsoft.Extensions.DependencyInjection, registering with
services.AddVitalRouter(routing =>
{
  ...
});

does not seem to register  interfaces, only a Router instance

thanks for the report. Fixed in #54 !


Add an Upgrade Scheduler

This PR adds an UpgradeScheduler which enables users to code predefined upgrades at certain height. The UpgradeScheduler manages a list of Upgrades where each upgrade defines the chain_name it should be applied to, the height it should be executed on and a MigrationFunc which users need to implement to carry out the migration.
Upgrade is defined as follows:

pub type MigrationFunc<DB> = fn(state: &mut FvmExecState<DB>) -> anyhow::Result<()>;

pub struct Upgrade<DB>
where
    DB: Blockstore + 'static + Clone,
{
    /// the chain name on which the upgrade should be executed
    chain_name: String,
    /// the chain id is calculated from the chain_name
    chain_id: ChainID,
    /// the block height at which the upgrade should be executed
    block_height: BlockHeight,
    /// the application version after the upgrade (or None if not affected)
    new_app_version: Option<u64>,
    /// the migration function to be executed
    migration: MigrationFunc<DB>,
}

At runtime, the FvmMessageInterpreter checks after processing each block, whether the UpgradeScheduler contains an upgrade at the current height. If so, it will execute the migration function.
When the migration function finishes, it will either:

Succeed, and returns a new application version (or 0 if not affected) which will be saved into the state
Fails, which results in the interpreter returning an error to the abci's end_block callback.

The PR comes with a small testing framework Tester which simulates block processing and allows us to run Upgrades at different heights.

Not sure what to do about the deployment address - it could be assigned to some global value just to remove the hardcoding,

Hmm, unless I am misunderstanding its not possible to access global variables in MigrationFunc as they are not known at compile time

but like I say ideally I'd try to see if we can use CREATE2 and deploy with the System Actor address, rather than any particular secret key.
That way the address would be something like:
new_addr = hash(sender, salt, source-code)

where we could set the salt to depend on the upgrade itself, e.g. the height of it, and sender would be the system address which doesn't have a private key. This way it would be a deterministic and unique function of the block height and the contract code.

That makes sense, but not sure how to set that up as create2 requires that only EVM actors can call it.


ztest: fix result checking

Fixes #7208: failed tests return -1, not 1. Fix this and adjust logic.
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

Codecov Report

Merging #7222 into master will increase coverage by <.01%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master    #7222      +/-   ##
==========================================
+ Coverage   55.03%   55.03%   +<.01%
==========================================
  Files         477      477
  Lines       51744    51743       -1
  Branches     9949     9948       -1
==========================================
  Hits        28475    28475
+ Misses      19306    19305       -1
  Partials     3963     3963

Impacted Files
Coverage Δ

tests/ztest/src/ztest.c
62.28% <75%> (+1.41%)
:arrow_up:

lib/posix/pthread.c
69.34% <0%> (-0.51%)
:arrow_down:

Continue to review full report at Codecov.

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