[3.x] Empty prop on form field component throws error

Description
If you leave a form field component prop blank it throws an error:
<x-twill::wysiwyg
    name="notes"
    label="Notes"
    :placeholder=""
/>

Tested with notes, placeholder and toolbar-options props.
Steps to reproduce

Add a form field to an existing form view
Leave a prop value empty
Open form in CMS

Expected result
No error
Actual result
syntax error, unexpected token "," (View: /resources/views/twill/businessLenses/form.blade.php)

It throws the error if you use double or single quotes. Using the prop without ="" or : doesn't throw an error.
Versions
Twill version: 3.0.0-alpha2
Laravel version: v9.30.1
PHP version: 8.0.20

Hey, as far as I know this is expected and standard behavior of blade components. The : in front means it should receive something like a variable, boolean or integer.

Ok, makes sense. I wasn't sure if it was something that we were able to handle behind the scenes to ignore the prop completely.


controlsd: no mutable carParams

https://github.com/commaai/openpilot/pull/31660#discussion_r1511917225

whoops

NEW controlsd & card control flow:

controlsd->data_sample: waiting for carState

- controlsd->initialized = True
- card-> initialize

card->state_update: receive can and create CS, gets latest CC
card->state_publish: publish CS, CP, CO

I think this is incorrect as at this point we haven't updated CO or actuated anything so controlsd will get the doubly previous CO

card->controls_update: actuate CC and update CO

NEWER controlsd & card control flow:

controlsd->data_sample: waiting for carState
card->state_update: receive can and create CS, gets latest CC

I think this is incorrect as at this point we haven't updated CO or actuated anything so controlsd will get the doubly previous CO

card->controls_update: actuate CC and update CO
card->state_publish: publish CP, CO, CS
controlsd: gets latest CS and CC

There is some controlsState diff, that is expected:

old routes, such as FORD, had CS.events encompass all openpilot alerts rather than just be car events. This leads to duplicate events in onroadEvents until the route is replaced
carControl will be delayed by one frame


monochrome: trailing space block non monochrome

In the monochrome theme, everything is in nice shades of grey. Everything except the 'trailing space' part.

I have no idea how to change this, so if anyone can help you, I'd very much appreciate it.

If I see that correctly, the monochrome theme does not define the airline_warning and airline_error highlighting groups. Therefore it uses the builtin default. You would have to modify the theme to included an adjusted group name there.
Check other themes on how to do it, preferably the builtin default dark.vim from the vim-airline repository.
If you find a fix, I appreciate a PR.

See also base16 theme as an example on how to set those groups


Restore state from disk

Right now, there is no API for restoring state of a log from disk… The process can’t be volatile regarding data.
Imagine you have to restart a process, the commit-log should be able to be quickly read from a folder on disk, you don't want to loose all the log that you had already ingested...
Since the end-to-end functionality is complex, we can start by breaking up the tasks top-down:
CommitLog
When opening a commit_log, a folder should be given, the within the folder files should be organised into tuples of same named files, index/log, sub directories and other things should be ignored. It would be good to read the files in order of creation OR even better, by their offset (name).
~/foo/commit-log
  /00000000000001.idx
  /00000000000001.log
  /00000000000002.idx
  /00000000000003.log
  /00000000000005.idx --> ignored (missing log file)
  /a.txt --> ignored
  /b/c.txt --> ignored

API for reopening a directory
let mut c = CommitLog::open("~/foo");
// everything should work the same down here

c.write(…)
c.read(…)

Segment
when opening a segment,  both log and index files path should be given for full check. Each file check is performed by the index/log structs, but, the segment should ensure that the returning struct will be open for writing or closed …
File level
Index File
Trickiest part, since it is the reference for where data is stored on the files themselves. I would say this is the first part to be implemented.
The procedure must reopen a given file, and check its content / space left.
The index is truncated on creation (filled with empty bytes), that’s good because it spare space in disk and memory but a bit bad because when reopening we have to figure out where did we stop writing to it. If we just look the file size, it will tell you always the max_size defined beforehand, so you need to check where is the first empty byte to actually make sense of it.
There are several ways of doing it so, mainly what I’ve seen implemented was binary search within the file to lookup entries.
One idea was to actually, read the file in reverse until you find the first “existing” byte, set that as the end of the file and then do a quick check on entries size, by trying to divide the entries into the default entry size (20).
Log
There isn’t too much to do here other than open the file and check if it is still "open" (meaning that it has space left for writes). That's done by properly checking the size, empty bytes shouldn't count.
important for file implementations check the reference links.

Questions still open here:

Should we keep storing segments as Vec<Segment>?

How efficient it is memory?
Should we check BTrees for efficiency?

Should we store indexes adding the commit-log (base-offset + index offset) or should we use both individually?

I’ve seen other implementations where all the offsets where global, the base (filename) + the internal file sparsing, I couldn’t never dive into figuring out the tradeoffs. It seems like if you don’t do it you have a bit more trouble searching later on reading. (I’ll move this to another issue probably).

Acceptance criteria
At the end of this task, we should be able to reopen log from disk following the above instructions/considerations.
References:

https://github.com/travisjeffery/jocko/blob/master/commitlog/commitlog.go#L95-L133
https://github.com/zowens/commitlog/blob/master/src/index.rs#L203-L226

I have started a WIP of the index reopening here, I didn't have time to go through with it so far 1fd9bafccd674208e8d2aa02e7ef1fdfe537dbcd


fix(multi-select): update experimental UI

Fixes #1648.
Changelog
New

Demo preview for list box, as it requires certain styling in the container to look well.
Hover styling for close button next to selection count.

Changed

Background/foreground colors for selection count UI.
Icon the close button next to selection count.

Removed

Custom width/height style for the close button next to selection count, as we are using fixed size icon for experimental.

@laurenmrice Would you have a chance to review this...? Thanks!

@laurenmrice Thank you for reviewing! I think I addressed all (or at least, most) of your comments. For your biggest question, multi select currently is a React-only component and thus the demo in vanilla dev env does not contain the interaction. I created a staging link with React storybook which may be an easier review: http://carbon-storybook-spectrohelioscopic-inculcator.mybluemix.net/?selectedKind=MultiSelect&selectedStory=default

@laurenmrice Addressed the first three, can be see at the same staging link.
Given the last one is about how our React code works, I created IBM/carbon-components-react#1897 to track the effort to fix this. Thanks!

@laurenmrice Do you have any remaining comments...? Thanks!

Thank you for pointing these out @laurenmrice! Let's use a separate issue (IBM/carbon-components-react#1897) for the former. Fixed the latter and can be seen at the same staging link.

Hi @laurenmrice the darker background color seems to be associated with hovering over, instead of the open state. That makes me wonder if the hover over color should be eliminated, or you are suggesting the hover over color should be gone as soon as the menu is open - What do you think? Thanks!

Let me merge this as there is only one open review comments (created #1964 for this). Thank alot @laurenmrice for your patient review!


[Menu] stackable doubling

It seems to make sense to add this variation.

I'm not sure including a menu two by two makes sense. Do you have an example of somewhere else on the web this pattern is used?

Indeed i have not.
The use i had in mind was in the specific case of a "top attached stacking four item menu" at the bottom of a fluid card in a grid layout, so as to get two x two items instead of four vertical items.
Ignore the request if too specific ;)
I tried to do it myself and couldn't easily arrange a solution.


Fix golint issues.

Description
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Excellent. 😄


why table is not support? I mean the entity like excel table

hi ,thanks for your work that i can  use netdxf conveniently.but i what know it is not support the table entity  which is not tableobject but like excel table

The Table entity is not implemented because it is not, sorry that it sounds like a nonsensical tautology. If you look the Table entity documentation there are 6 pages and a half of codes. Reorganizing and giving some sense to them takes a lot of time. Something I will not do is just create a container with all the codes, and let you figure out what is the purpose and incompatibilities of each one. And the case of the MLeader entity (multi leader) is even worse. In the projects front page you can see the full list of available entities.
At least as @sorifiend has shown, you can access the information contained in the tables, you cannot create them but you can access to them.
Daniel

Is there a way to edit info in tables? I can parse table contents; but when I edit and save doc, changes are not saved. I can put edited table in new document though.


Continuous results uploading don't record the total of the performance test carried out

Describe the bug
I did two distributed performance tests with Locust.io.
I used 20 virtual machines with 16 virtual CPUs (8 cores) and 65 GB of ram, 8 locust workers by virtual machine, reaching 20 thousand threads and approximately 7300 req/s.
I used one virtual machine with 8 virtual CPUs (4 cores) and 32 GB of ram for JTL-Reporter
Test results are sent in real time to both Grafana (from locust-plugins) and JTL-Reporter using the "Continuous results uploading" method.
The first test was saved fine and the start and end of the test match what was recorded by Grafana.
JTL-Reporter:

Grafana:

However, in the second test JTL-reporter tells me that it finished at 08:03 while in Grafana I have results until 08:14
JTL-Reporter:

Grafana

Expected behavior
Let JTL-Reporter record the total of the performance test carried out
Thank you very much in advance, beautiful tool, we are very happy using it.

Hi @jorgetamayo21
Please check this method: https://github.com/ludeknovy/jtl-reporter/blob/921f314b5038a107971596d6a7fd4e10669cb23f/scripts/jtl_listener_service.py#L178
And also check the logs of the run when some of the samples were not reported. My guess is, that it got terminated before all the samples were uploaded to the listener service.

Indeed, 5103684 uploaded samples were recorded in the log.

Alright. Maybe try to remove the timeouts from the join https://www.gevent.org/api/gevent.greenlet.html#gevent.Greenlet.join in theory it should wait for the background task to finish

Also, be sure that the DB and listener service have enough resources to handle the load.

Leave all without timeout.

Additionally, I doubled the resources of the VM for JTL-Reporter, leaving it with 16 virtual CPUs (8 cores) and 64 GB of ram.
Tomorrow we will do the same test and I will tell you if it worked to close the issue. Thank you so much!

I just finished a test of more than 3 hours, 17 thousand threads, generating approximately 53 million requests.
This showed me the log when I stopped the test

52 minutes have passed already and it still appears as processing

The cpu of the "master" machine is at 10%, I suppose even sending the data to jtl-reporter

I will continue to report on all this, I am missing a "progress" indicator in sending the data

Just finished sending to jtl-reporter!

jtl-reporter keeps showing me "processing"

jtl-reporter VM instance is still working

The jtl-reporter VM instance seems to have stopped working, there is not much CPU movement and the occupied memory dropped sharply :(

Hi @jorgetamayo21
Thanks for the detailed report.
Would you care to share the edited python script for uploading the samples to the app using multiple gevent to make it faster?
Yes, please try again with v4.10.2 ? Hopefully the memory issue should be fixed in v4.10.2.

I made the pull request so you can look at it in detail.
https://github.com/ludeknovy/jtl-reporter/compare/main...jorgetamayo21:jtl-reporter:new_jtl_listener_service_script

Thanks @jorgetamayo21


ISD-2184 Add http interface module with provider and requirer class implementation

Applicable spec:
Overview

Rationale

Juju Events Changes

Module Changes

Library Changes

Checklist

[x] The charm style guide was applied
[x] The contributing guide was applied
[x] The changes are compliant with ISD054 - Managing Charm Complexity
[x] The documentation is generated using src-docs
[x] The documentation for charmhub is updated
[x] The PR is tagged with appropriate label (urgent, trivial, complex)

closing as dups to #14, will be reused when implementing haproxy-route integration


docs: improved monospace quoting and gramar fixes

During https://github.com/mitsuhiko/insta/pull/618 I noticed few missing quotes here and there and a minor gramar issue which bugged me ^^
(separate PR for reviewability)

Thanks!

(Can merge on fixing rustfmt)


fix: font size changes reverted and removed extra padding

Purpose
Revert the font size changes and removed extra padding
Does this introduce a breaking change?

[ ] Yes
[X] No

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code
open cwyd chatbot page and check the font size and spacing

Heya! Can you explain why this change is required? Thanks

Heya! Can you explain why this change is required? Thanks

Hey Adam, During the accessibility issue fix we observed that look and feel for the landing page and text size is got impacted. Refer the below screen.
So, We have fixed that now and raised the PR for the same.


QEMU error: Could not set up host forwarding rule 'udp:<IP_ADDRESS>:664-:664'

Hi, Guys:
i am following the guide: [https://github.com/openbmc/docs/blob/master/development/dev-environment.md]

Normally, i often run the first one, but today i want to use "devtool deploy-target" to deploy my modifications to the running machine(this machine i runed it on QEMU),
i run the second one :
./qemu-system-arm -m 256 -machine romulus-bmc -nographic -drive file=./tmp/deploy/images/romulus/obmc-phosphor-image-romulus.static.mtd,format=raw,if=mtd -net nic -net user,hostfwd=:<IP_ADDRESS>:22-:22,hostfwd=:<IP_ADDRESS>:443-:443,hostfwd=tcp:<IP_ADDRESS>:80-:80,hostfwd=tcp:<IP_ADDRESS>:2200-:2200,hostfwd=udp:<IP_ADDRESS>:623-:623,hostfwd=udp:<IP_ADDRESS>:664-:664,hostname=qemu
but i get the error return:
`qemu-system-arm: Could not set up host forwarding rule 'udp:<IP_ADDRESS>:664-:664'
anyone could help me?

On most systems, you can't take the actual ports, which is why the directions encourage you to use a virtual port when starting QEMU. i.e. make it hostfwd=udp:<IP_ADDRESS>:6664-:664 and then use the 6664 port for your 664 traffic to your QEMU session. Or if you want, try running with sudo.


[D&D_Moldvay] Less Dropdowns, More Flexibility

I think the damage die section should be a blank text field to add some more flexibility than the drop down box can offer, the same goes for the Alignment and HP Die dropdowns and the Attack section under the Monster tab.

This was pushed to live some time ago - can you verify?

Yep! This was changed in the latest version of the character sheet.


listbox get any column value

Currently we can do
document.getElementById('button_pick').value

To get the id of the record, but what if I want another column?
Please implement this:
document.getElementById('button_pick').column('pn')

Done, will be seen in next release.

Fix is not backwards-compatible.
For an example try to open a page in the wiki.

Further investigation revealed that this has to do with someone implementing multi-select


Getting TypeError: Configuration is not a constructor in models-managers.js

I'm running Node v16.17.1 and getting the following error for line 9 /utils/models-managers.js
const openai_config = new Configuration({ apiKey: apiKey });
Is this a syntax issue or related to the version of Node?
/user/Development/node/discord-bots/openai-discord-bot/OpenAI-Discord-Bot/src/utils/models-managers.js:9
const openai_config = new Configuration({ apiKey: apiKey });
^
TypeError: Configuration is not a constructor
at Module._compile (node:internal/modules/cjs/loader:1126:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1180:10)
at Module.load (node:internal/modules/cjs/loader:1004:32)
at Function.Module._load (node:internal/modules/cjs/loader:839:12)
at Module.require (node:internal/modules/cjs/loader:1028:19)
at require (node:internal/modules/cjs/helpers:102:18)
at Object. (/Users/user/Development/node/discord-bots/openai-discord-bot/OpenAI-Discord-Bot/src/commands/ask.js:2:24)
at Module._compile (node:internal/modules/cjs/loader:1126:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1180:10)

I noticed in the npm install that Node 18+ is required for certain openai packages so I updated to Node 18 and I'm still getting the error.  OpenAI may have changed how to instantiate the Configuration object.

Hi Tim,
The reason you get this error is because openai npm library got an update
from v3 to v4.
Here you can see all the changes:
https://github.com/openai/openai-node/discussions/217  (You can follow the
instruction to migrate the code manually or automatically)
On Mon, 20 Nov 2023, 19:11 Tim Reynolds, @.***> wrote:

I'm running Node v16.17.1 and getting the following error for line 9
/utils/models-managers.js
const openai_config = new Configuration({ apiKey: apiKey });
Is this a syntax issue or related to the version of Node?
/user/Development/node/discord-bots/openai-discord-bot/OpenAI-Discord-Bot/src/utils/models-managers.js:9
const openai_config = new Configuration({ apiKey: apiKey });
^
TypeError: Configuration is not a constructor
at Module._compile (node:internal/modules/cjs/loader:1126:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1180:10)
at Module.load (node:internal/modules/cjs/loader:1004:32)
at Function.Module._load (node:internal/modules/cjs/loader:839:12)
at Module.require (node:internal/modules/cjs/loader:1028:19)
at require (node:internal/modules/cjs/helpers:102:18)
at Object.
(/Users/user/Development/node/discord-bots/openai-discord-bot/OpenAI-Discord-Bot/src/commands/ask.js:2:24)
at Module._compile (node:internal/modules/cjs/loader:1126:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1180:10)
—
Reply to this email directly, view it on GitHub
https://github.com/InDiagonal/OpenAI-Discord-Bot/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOPGFNXQKY4C76OUZSEGEVTYFOMMDAVCNFSM6AAAAAA7TKMWL2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGAYDENZUGEZDOOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Merge resource behaviour in Python3 / Iris v2

When using Python3 & Iris v2, the resources for the merge exercise load differently to using Python2 and Iris v1. Specifically, it appears like each dataset is loaded twice. This needs to be fixed, preferably for consistent behaviour between the two Python/Iris combinations.
Investigate:

is this being caused by Python 3 (i.e. has wildcard filename match behaviour changed?)
is this being caused by Iris v2 (i.e. has dataset load behaviour changed in a manner that we have not allowed for?)
is this being caused by a combination of the above?
is this being caused by something else?

Ping @kaedonkers

The example that this issue presented in is using iris.load_raw() on a netCDF file containing a single air_potential_temperature phenomenon.
Py2+Iris1 loads a single cube, as expected.
Py3+Iris2 loads two cubes: air_potential_temperature and surface_altitude.
surface_altitude is not a cube but an Auxiliary coordinate of the air_potential_temperature cube.

Investigation (after testing with Py2+Iris1, Py2+Iris2, Py3+Iris2):
:x: Is this being caused by Python 3?
:heavy_check_mark: Is this being caused by Iris v2?
:x: Is this being caused by iris.load_raw()?

Iris v1.13:

Iris v2.0:

I think you are pointing to the fact that netcdf_promote became the default behaviour in iris 2? https://github.com/SciTools/iris/issues/1723

Thanks for a response on this @pelson. Looks like we will need to change the resources used for this exercise.

This has been fixed by loading with a constriant.


@elderjs/plugin-images - output path corrupted on windows

Getting this kind of error messages:
Error: Hook: "processImages" threw an error: ENOENT: no such file or directory, copyfile 'D:/Projects/elderjs-images/images/image.jpg' -> 'D:\Projects\elderjs-images\public\images\D:\Projects\elderjs-imagesimage.jpg'

...

at async D:\Projects\elderjs-images\node_modules\@elderjs\elderjs\build\Elder.js:297:13 {
	errno: -4058,
	syscall: 'copyfile',
	code: 'ENOENT',
	path: 'D:/Projects/Kadastiku/web/elder/elderjs-images/images/image.jpg',
	dest: 'D:\\Projects\\elderjs-images\\public\\images\\D:\\Projects\\elderjs-imagesimage.jpg'
}

Should be fixed in 0.1.9 that was just pushed.
Cross platform fs is one of the worst parts about node lol.
Let me know if it doesn't work.

Thanks Nick. No build time errors anymore, everything's generated nicely, just the relative path is still incorrect, and only the placeholder image is shown at runtime.
{"/images/image.jpg":{
	"width":1920,
	"height":1137,
	"format":"jpeg",
	"original":"/images/image.jpg",
	"sizes":[{
		"width":1280,
		"height":758,
		"format":"webp",
		"relative":"/images/imageD:\\Projects\\elderjs-images\\public\\images\\image-ejs-1280.webp",
		"scale":1,
		"cache":"D:\\Projects\\elderjs-images\\images\\sizes\\image-ejs-1280.webp",
		"public":"D:\\Projects\\elderjs-images\\public\\images\\image-ejs-1280.webp"},

Using default configuration btw.

Problem seems to be here:
// getPrefix.js
return fileWithoutExt + publicPrefix.split(fileWithoutExt).pop();

Mixed forward- and backslashes on windows and split operation doesn't do anything:
fileWithoutExt = '/images/image'
publicPrefix = 'D:\Projects\elderjs-images\public\images\image'

This will be fixed in the next release.


Fine until intubatin

I suspect most ARDS cases will require intubation, which necessitates sedation, and features suction, CO2 removal, and a lot of autonomous  monitoring.  This might be good for emergency home care, provided other therapeutics are in place and continuous patient monitoring is employed.

Check this on advantages of Pressure ventilation for corona https://www.youtube.com/watch?v=okg7uq_HrhQ


Login via command line

I'm running Docanno via Docker in my local and everything is ok up and running
Whenever we login, we see the Django login screen before we can access the projects. We would like to be able to skip this step and login via the command line instead. This is so that we can have many anonymous users login as a user with access to all projects but do not gain the information or access all of the other projects besides the ones we assign them. Is there a possibility to change the script such that login is an automatic process?

I think you are looking for this feature #65. After this feature finished, you can create many "Annotator" role(#152 ) accounts and they only have the permission to gain the information you assign them. Right now this feature is WIP, you can add your use case in #65 and close this one.

We had a similar requirement for a project that uses doccano. To solve the problem, @rbinrais implemented https://github.com/chakki-works/doccano/pull/301 which enables delegating authentication to a proxy server that provides the user information to doccano via headers. Perhaps you can use a similar approach for the anonymous log-in use-case, especially if the auth proxy server feature gets combined with RBAC as implemented in https://github.com/chakki-works/doccano/pull/333, e.g. you could set headers in the proxy server that gives all users annotator level access.

I've designed a function in other projects for making authentication to get a JSON payload: https://github.com/vwoloszyn/diaa/blob/master/diaa/maker.py


Fail with sshkit 1.15.0

** Invoke unicorn:setup_app_config (first_time)
** Execute unicorn:setup_app_config
  INFO [a30752d8] Running /usr/bin/env mkdir -pv /home/deploy/app/shared/config as<EMAIL_ADDRESS> DEBUG [a30752d8] Command: /usr/bin/env mkdir -pv /home/deploy/app/shared/config
  INFO [a30752d8] Finished in 0.040 seconds with exit status 0 (successful).
cap aborted!
SSHKit::Runner::ExecuteError: Exception while executing as<EMAIL_ADDRESS>undefined method `start_with?' for #<Pathname:/home/deploy/app/shared/config/unicorn.rb>
/Users/mzhukov/.rvm/gems/ruby-2.4.2@app/gems/sshkit-1.15.0/lib/sshkit/runners/parallel.rb:15:in `rescue in block (2 levels) in execute'
/Users/mzhukov/.rvm/gems/ruby-2.4.2@app/gems/sshkit-1.15.0/lib/sshkit/runners/parallel.rb:11:in `block (2 levels) in execute'

Caused by:
NoMethodError: undefined method `start_with?' for #<Pathname:/home/deploy/app/shared/config/unicorn.rb>
/Users/mzhukov/.rvm/gems/ruby-2.4.2@app/gems/sshkit-1.15.0/lib/sshkit/backends/netssh.rb:66:in `upload!'
/Users/mzhukov/.rvm/gems/ruby-2.4.2@app/bundler/gems/capistrano-unicorn-nginx-2e7422cc0d40/lib/capistrano/tasks/unicorn.rake:49:in `block (3 levels) in <top (required)>'
/Users/mzhukov/.rvm/gems/ruby-2.4.2@app/gems/sshkit-1.15.0/lib/sshkit/backends/abstract.rb:29:in `instance_exec'
/Users/mzhukov/.rvm/gems/ruby-2.4.2@app/gems/sshkit-1.15.0/lib/sshkit/backends/abstract.rb:29:in `run'
/Users/mzhukov/.rvm/gems/ruby-2.4.2@app/gems/sshkit-1.15.0/lib/sshkit/runners/parallel.rb:12:in `block (2 levels) in execute'
Tasks: TOP => unicorn:setup_app_config

Update sshkit gem https://github.com/capistrano/sshkit/issues/409


Titan german weirdness

I am using cactbot dll from releases, but the latest master branch for the actual js/css stuff.
On titan savage, when he does the first attack with 4 patterns, evil earth I think? Some really weird german stuff is spoken and the TTS can't even pronounce it properly.
(Forgot to mention I am using english, not german).

Fairly sure it's due to line 99 being tagged as English rather than German. Just a typo, I'll drop a request in to fix it.


Templates with duplicate names cause versions of different templates to be merged

Describe the bug
Templates with duplicate names cause versions of different templates to be merged.
Steps to reproduce

Add a catalog: https://github.com/alibabacloud-automation;
View the template: alicloud-rds-mysql, due to the same name: terraform-alicloud-rds-mysql, these versions of the template are merged into the existing template.

Result

Expected behavior
Split into two templates.
Environment

Walrus version:

UI: dev-437b352
Server: dev-71aeab4

Duplicate of https://github.com/seal-io/walrus/issues/1363

Environment

Walrus version: main 66a2214c8ea5dfb7fd8542fecb5e87a3778fc07e
UI version: dev-7449733

Test Result: pass
Added the catalog name as the template prefix.


Countess Drop information

Firstly thank you for the great resource.
It seems to show that Lum and Ko are Hell rune drops, however, I see a lot of other references that these are dropped from nightmare Countess.
The regular Nightmare drop is El to Ko, with 0-3 bonus drops of El to Io
The regular Hell drop is El to Lo, with 0-3 bonus drops of El to Ist

I believe the table only mentions the Countess's rune drop table, not her normal monster drop table which is almost irrelevant as the drop chance is so low.

Adding a note for myself to doublecheck the drop tables in the future.


Add ReadTheDocs configuration file

Your documentation on ReadTheDocs on the latest tag is failing for 2 months because the package needs to be installed first.
This PR adds a configuration file which is the recommended way to configure your project.
It should find it in the root of the repository and use it by default, so you should already see that this PR is able to re-build the documentation (if you activated the builds for PRs in your ReadTheDocs project, otherwise you need to merge first).

You can add the URL link to the documentation
/https://stackyter.readthedocs.io/en/latest/
in the "About" section on the main page of the repository, so users can access it easily.


help for some error

Hello, can you give the preconditions for operation? I use Python on win64 computer (memory 16g) and use 64 bit python. Is this feasible?I have the following errors during operation. Can someone help me?
When I run 'get BPE', the following error is reported:

When I run 'make data', the following errors appear:

It seems to be an issue of python's multiprocessing on Windows platform. Maybe you could try to rewrite the multiprocessing part to get a serial version of the program (which will be an option in our later formal version), or try to run the code on Linux platform.

It seems to be an issue of python's multiprocessing on Windows platform. Maybe you could try to rewrite the multiprocessing part to get a serial version of the program (which will be an option in our later formal version), or try to run the code on Linux platform.

Thank you very much for your reply. I changed the Linux platform and ran the code, but unfortunately, the following problems occurred:

I can't install fast transform directly through PIP (an error will be reported), so I downloaded the fast transform package in GitHub, but the above error occurred. Can you provide some help on this package? Thank you.

ModuleNotFoundError: No module named 'fast_transformers.causal_product.causal_product_cpu'
I meet the same problem


feature/multi language

Enable to select languages by a selector.

https://docs.astro.build/ja/recipes/i18n/


Docstring for dispatched methods

I have a case where I want to dispatch on an imported Python function, for example,
using PythonCall

mpl = pyimport("matplotlib")
mpl.use("qt5agg")

plt = pyimport("matplotlib.pyplot")

# function doc issue

"What should I do?"
function plt.plot(a::String)
   println("hello multi-dispatch!")
   plt.plot(1:10)
end

plt.plot("Haha!")

which returns
julia> include("test_pythoncall.jl")
ERROR: LoadError: MethodError: no method matching doc!(::Module, ::Py, ::Base.Docs.DocStr, ::Type{Tuple{String}})
Closest candidates are:
  doc!(::Module, ::Base.Docs.Binding, ::Base.Docs.DocStr, ::Any) at /home/local/hongyang/bin/julia-1.7.2/share/julia/base/docs/Docs.jl:226
  doc!(::Module, ::Base.Docs.Binding, ::Base.Docs.DocStr) at /home/local/hongyang/bin/julia-1.7.2/share/julia/base/docs/Docs.jl:226
Stacktrace:
 [1] top-level scope
   @ .../test_pythoncall.jl:10

Do you have an idea how to fix it?

I want to make wrappers over Python functions, as if they are native Julia functions. Since Julia is built upon multiple-dispatch, this is doable in Julia. I've taken advantage of this in my package here as an example. In this way I don't need to come up new names for the wrapper plotting functions. This concept is somehow similar to the plotting recipe system in Plots.jl.
In general, say someone want to make a wrapper over a Python function, this is potentially an issue.

Like I already said, plt.plot is a Python function, it can't be overloaded. This isn't anything to do with PythonCall, it's just that you can't add methods to things which aren't Julia functions. In contrast, PyPlot.plot is a Julia function (it's some wrapper around plt.plot) so can be overloaded.
I suggest you make a function which wraps plt.plot and then overload that (which is precisely what PyPlot.jl does):
function plot(args...; kwargs...)
    plt.plot(args...; kwargs...)
end
function plot(x::CustomType; kw...)
    // custom overload
end


sep: add instructions about what happens if a closing or declaration tx is not signed by other party

The SEP does not discuss what the parties do if one participant proposes and signs a closing transaction and the other participant does not.
What needs to happen in that case is for the iteration number to be skipped. This is implicitly what the SEP requires because whenever i is incremented, it is never reverted anywhere, but this should be made more explicit.
The reason not reusing an iteration is important is because one participant has already signed the closing transaction, and so the other participant could resign it at anytime and if the iteration number is reused there would be a valid declaration transaction for that old closing transaction.

This is no longer relevant after #176.


QA ingest kubectl cluster-info

The command 'kubectl cluster-info dump' returns a wealth of data that is worth exporting and tracking on successful tests furthermore that data is already structured JSON and therefore should be a great start to process and store in a database. The primary purpose of the database would be to record which operating systems receive tests, which types of clusters, the cluster configuration, health, CNI in use, etc could all be recorded and become a moving matrix of test history.

@VestigeJ are you still currently working this or should we move it back to "Accepted" to be planned?

@caroline-suse-rancher I'm currently looking at piggybacking off of kubetest2 which takes advantage of this output by default from it's built-in logging. So yes it's still underway, the initial plan was for me to work on this during hack week but I've been testing releases for three weeks instead.


Fan start with loop.py script

Issue: Fan blade does not start spinning until the fan power percentage requested is above approximately 25%.
Command used: sudo python3 loop.py -t 40 -v
Result: The temperature will rise to 40. Once at 40, the fan blade will start 'stuttering' like it wants to start spinning. While the requested fan speed is less than 25%, the fan blade continues to stutter. The temperature rises enough to request 25% fan speed and the fan blade starts spinning. Once spinning, the fan blade spins up and down normally as the temperature varies above and below 40.

Hi,
This behavior was addressed in the fan firmware but we use slightly different design compared to prototypes that we use to develop the firmware.
You could fix this by adding a threshold before command the fan power (line 68 in the loop.py). I can do it after I receive the last version of the fan.

Are you saying there will be a future firmware update to address this?

No, there will be no firmware update, because the fan is not in system updatable. The only way to update the fan firmware will be with the JTAG probe. This why the fan is as dumb as possible and all the logic is transferred to the Raspberry.
This problem could be addressed in the software that controls the fan.

ah, okay. So the solution is to set the power threshold at 25% in the control software?

Yes.


We are interested in testing DemocracyEarth on some internal Coop initiatives.

We are interested in testing DemocracyEarth on some internal Coop initiatives.
Is the Master Branch ready for this kind of testing ?

Hello !
You can follow the monthly activity of the repository ; these kinds of projects (the pioneer ones, especially community-driven) are especially hard to schedule, as they're unique and treading uncharted lands without roads. This being said, what I gather from following this project is that it is closing in on its first release. The last mile is usually the toughest. The core features (authoring/voting/delegating/debating) are almost here, and a feature freeze for the refactor/bughunt/cleanup phase should soon follow. As you can see from the contributions graph, the activity was boosted around (and perhaps by) the recent USA presidential elections, as there was a lot of controversy about the integrity and reliability of said elections. You should subscribe to the newsletter, and you can preview the alpha demo here.

Thanks !!!
I just checked out the alpha...Wondering why the tight integration with FB ?

Ideally sovereign should support all OpenID providers ; FB probably got prime time because one of the points of making a prototype is to buzz on social networks, to conscientize about the very idea of liquid democracy.
Still, it is optional, I never used the feature myself.
Sorry for the long answer delay, and keep on your toes, sovereign can't be released soon enough !


Issues on OS X El Capitan

I'm trying unsuccessfully to install OpenRefine on OS 10.11.5.

When I launch the latest version (2.6-rc2) it quits before it finishes initializing, without any sort of warning/error message.
For the previous version (GoogleRefine 2.5), my first couple of attempts to launch were hampered by various problems. First the operating system had me install a legacy version of Java. Then I got the "move to trash" message. I was able to fix this by changing the security settings as described elsewhere, but then I just got the same behavior as before: GoogleRefine quits before it even finishes launching.

Is this a known issue?

Well, 2.6-rc2 seems to run fine after restarting the computer. Sorry this might be hard to reproduce, given that I was trying with 2 different versions, with vs without legacy Java, etc. The only clue I can provide is that my last few attempts to launch 2.5 before restarting my computer, an error dialog popped up very briefly (i.e. <2 seconds) referencing some kind of binding exception and stating that the address was already in use.

try to downgrade java

I uninstalled the newest version of java and install old version of java, so I installed google refine Version 2.5 [r2407].

Please use the mailing list for support type question.


demo: fix break in dragdrop

caused by alert when drag in list1.

What specifically does this fix? It looks like these alerts convey information as part of the demo, and would be fine in this context. Additionally, this change only eliminates one of many alerts.

Closing as inactive.  Please reopen with more information if you wish to pursue this further.


[BUG]: agent sql issue

How are you running AnythingLLM?
Docker (local)
What happened?
the following log:
[backend] info: [AgentHandler] Start db44aded-1eca-4015-acd8-f66e00d1a25e::ollama:llama3:8b-instruct-fp16
[backend] info: [TELEMETRY SENT]
[backend] info: [AgentHandler] Attached websocket plugin to Agent cluster
[backend] info: [AgentHandler] Attached chat-history plugin to Agent cluster
[backend] info: [AgentHandler] Attaching user and default agent to Agent cluster.
[backend] info: [AgentHandler] Attached rag-memory plugin to Agent cluster
[backend] info: [AgentHandler] Attached document-summarizer plugin to Agent cluster
[backend] info: [AgentHandler] Attached web-scraping plugin to Agent cluster
[backend] info: [AgentHandler] Attached sql-agent:sql-list-databases plugin to Agent cluster
[backend] info: [AgentHandler] Attached sql-agent:sql-list-tables plugin to Agent cluster
[backend] info: [AgentHandler] Attached sql-agent:sql-get-table-schema plugin to Agent cluster
[backend] info: [AgentHandler] Attached sql-agent:sql-query plugin to Agent cluster
[backend] info: [AgentLLM - llama3:8b-instruct-fp16] Valid tool call found - running sql-list-tables.
[backend] info: [AgentHandler] [debug]: @agent is attempting to call sql-list-tables tool
[backend] info: [AgentHandler] Using the sql-list-tables tool.
[backend] info: [AgentLLM - llama3:8b-instruct-fp16] Function tool with exact arguments has already been called this stack.
[backend] info: [AgentLLM - llama3:8b-instruct-fp16] Will assume chat completion without tool call inputs.
from within anythingllm container, I can connect to database and get response:
Password for user postgres:
psql (14.12 (Ubuntu 14.12-0ubuntu<IP_ADDRESS>), server 16.2)
WARNING: psql major version 14, server major version 16.
Some psql features might not work.
SSL connection (protocol: TLSv1.3, cipher: TLS_AES_256_GCM_SHA384, bits: 256, compression: off)
Type "help" for help.
database_name=> select * from company;
id | name  | age |                      address
----+-------+-----+----------------------------------------------------
5 | Paul  |  32 | California
6 | John  |  28 | New York
7 | Alice |  45 | Texas
8 | Bob   |  36 | Florida
(4 rows)
Any ideas what problem may be? I have
Are there known steps to reproduce?
No response

Looks like the LLM either not using the output from the table or the LLM hallunicating the JSON to call the next step of the SQL-connector. This is not a bug as you can see
[backend] info: [AgentLLM - llama3:8b-instruct-fp16] Valid tool call found - running sql-list-tables.
[backend] info: [AgentHandler] [debug]: https://github.com/agent is attempting to call sql-list-tables tool
[backend] info: [AgentHandler] Using the sql-list-tables tool.

The sql-list-tables tool was called and from the logs it looks like it was attempted to call again - to prevent spam-calling the same function in an infinite loop we exit early on these kinds of conditions because agents will infinitely call the same function over and over and never stop.
https://docs.useanything.com/agent-not-using-tools

In @agent mode, does the workspace vector DB information got appended to the prompt? I think having sample correct queries would be helpful, but I don't know if this is part of the flow in anythingllm while in agent mode.

The vector db with its content is RAG-searchable like it would be in a regular chat, yes


Documentation: Configs need docs

[ ] Emailer
[ ] Connector
[ ] Client
[ ] Users

We use yaml so the example config now holds lots of documentations through inline comments. See examples/config-dev.yaml


Cannot use Task.Run to execute multiple activities in fan-in-out scenario in a workflow

Expected Behavior
Cannot run fan-in-out scenario with composite tasks.
Actual Behavior
I should be able to run and compose tasks. In this case, the code never completes, the activities are not called.
Steps to Reproduce the Problem
This results in the task never completing:
// In a Workflow implementation

var tasks = new List<Task<string>>();

foreach (var name in input)
{
    tasks.Add(Task.Run(async () =>
    {
         await context.CallActivityAsync<string>(nameof(CreateGreetingActivity), name));
         await context.CallActivityAsync<string>(nameof(CreateGreetingActivity), name));
    });
}

var messages = await Task.WhenAll(tasks);

Note that the following is possible and works. The tasks will run in parallel and get awaited correctly:
// In a Workflow implementation

var tasks = new List<Task<string>>();

foreach (var name in input)
{
    tasks.Add(context.CallActivityAsync<string>(nameof(CreateGreetingActivity), name));
}

var messages = await Task.WhenAll(tasks);

Release Note
I'm not sure if this is a bug, but one potential way would be to use custom TaskScheduler.

I wonder to what degree your use of Task.Run to wrap the activities before awaiting it externally isn't messing with this.  Have you tried implementing it more aligned with the example from the docs (bottom of the section):
//Revisiting the earlier example...
// Get a list of N work items to process in parallel.
object[] workBatch = await context.CallActivityAsync<object[]>("GetWorkBatch", null);

const int MaxParallelism = 5;
var results = new List<int>();
var inFlightTasks = new HashSet<Task<int>>();
foreach(var workItem in workBatch)
{
  if (inFlightTasks.Count >= MaxParallelism)
  {
    var finishedTask = await Task.WhenAny(inFlightTasks);
    results.Add(finishedTask.Result);
    inFlightTasks.Remove(finishedTask);
  }

  inFlightTasks.Add(context.CallActivityAsync<int>("ProcessWorkItem", workItem));
}
results.AddRange(await Task.WhenAll(inFlightTasks));

var sum = results.Sum(t => t);
await context.CallActivityAsync("PostResults", sum);

@WhitWaldo I edited my original post to add comment that the scenario you're describing works fine.
My use case is simply running groups of activities that need to execute in parallel, but within the group, they should still execute in order, as pass dependent inputs and outputs.
A workaround in my case would be to group the activities in a separate workflow, and then run those child workflows in parallel. This would achieve the same goal, but it would make the code less readable as my tasks are now spread into different workflows.

Alright, I tried a different approach, and it seems to work fine. I don't know why Task.Run behaves like this.
// In a Workflow implementation

var tasks = new List<Task<string>>();

foreach (var name in input)
{
    tasks.Add(ProcessTasks());
}

var messages = await Task.WhenAll(tasks);

async Task ProcessTasks()
{
    await context.CallActivityAsync<string>(nameof(CreateGreetingActivity), name));
    await context.CallActivityAsync<string>(nameof(CreateGreetingActivity), name));
}

I expect it has to do with the event sourcing and how it's scoping activities by task. Perhaps the workflow is easily able to replay the tasks through when it is awaited, whether an individual activity or via Task.WhenAll, but my bet is that by using Task.Run to wrap the async activities, you're effectively hiding these operations from the workflow scheduling.

That makes sense. Thank you for your thoughts. Closing this, as I have a solution.


Incorrect Usage. flag provided but not defined: -v

I'm unable to get the version of a local built tx. I've tried building from latest git commit and the latest released tarball, but the above error message is print when issuing -v (or --version) flag.
The same does not happen when installing via intall.sh script (as instructed in the README);
go version output: go version go1.20.2 linux/amd64

go env output (click to show)

GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/rffontenelle/.cache/go-build"
GOENV="/home/rffontenelle/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/rffontenelle/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/rffontenelle/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.20.2"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/rffontenelle/build/testing/transifex-client/src/cli/go.mod"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build2048290324=/tmp/go-build -gno-record-gcc-switches"

Steps to reproduce:
$ git clone -q https://github.com/transifex/cli
$ cd cli/
$ make build
go build -o ./bin/tx
$ ./bin/tx -v
Incorrect Usage. flag provided but not defined: -v

NAME:
   tx - A new cli application

USAGE:
   tx [global options] command [command options] [arguments...]

COMMANDS:
   migrate, mg  Migrate legacy configuration.
   merge        tx merge [options] [resource_id]
   push         tx push [options] [resource_id...]
   pull         tx pull [options] [resource_id...]
   add, a       Add a resource in config. Use no arguments for an interactive mode.
   init         tx init
   delete       tx delete [options] [resource_id...]
   update       Update the `tx` application if there is a newer version
   status       tx status [resource_id...]
   help, h      Shows a list of commands or help for one command

GLOBAL OPTIONS:
   --root-config FILE          Root configuration from FILE
   --config FILE, -c FILE      Load configuration from FILE
   --token value, -t value     The api token to use [$TX_TOKEN]
   --hostname value, -H value  The API hostname [$TX_HOSTNAME]
   --cacert value              Path to CA certificate bundle file [$TX_CACERT]
   --help, -h                  show help (default: false)
2023/03/18 11:34:33 flag provided but not defined: -v

Using official installer (it runs ./tx -v in the end):
$ curl -o- https://raw.githubusercontent.com/transifex/cli/master/install.sh | bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3426  100  3426    0     0  23194      0 --:--:-- --:--:-- --:--:-- 23306
** Installing CLI from https://github.com/transifex/cli/releases/latest/download/tx-linux-amd64.tar.gz

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100 4166k  100 4166k    0     0  2116k      0  0:00:01  0:00:01 --:--:-- 3263k

** Adding CLI to PATH
** export PATH="/home/rffontenelle/build/testing/transifex-client/src/cli:$PATH" was added to /home/rffontenelle/.bashrc. Please restart your terminal to have access to 'tx' from any path.

** If everything went fine you should see the Transifex CLI version in the following line.
TX Client, version=1.6.5

In order for the -v flag to be set, the binary needs to be built with the -ldflags option set. For example:
go build -ldflags "-X 'github.com/transifex/cli/internal/txlib.Version=99.99.99'"

This will make the resulting binary think its version is "99.99.99" (not sure how much use this will be to anyone though).
We are setting this flag during our build process (we use a Github action with goreleaser for this and we set the version's value to the latest git tag.
Longer explanation:

When we initialize the CLI application, we set the Version option (code)
The variable used for this is initialized here
By default the value of this variable will be the empty string which is equivalent to not having set the Version option at all
Thus, if you build the application with go build or go install, the version will not be set and the -v flag will not work
You can override the value of this variable at build time by using the -ldflags option we mentioned before
Our goreleaser configuration does exactly that by using the latest git tag (which it is aware of) as the value
goreleaser saves the resulting binaries as release assets
the install.sh script downloads the binaries from the release assets

I am going to close this issue since I don't think there is anything to be done here. I am not sure what version a local build of the client should think is the correct one anyway (go build doesn't even know that there is an underlying git repository).

not sure how much use this will be to anyone though

Very helpful when trying to package this up in nixpkgs. Ty!


Add Swift 4.0 compatibility commit for SwiftLint

Pull Request Description
Add Swift 4.0 compatibility commit for SwiftLint.
Acceptance Criteria
To be accepted into the Swift source compatibility test suite, a project must:

[x] be an Xcode or swift package manager project
[x] support building on either Linux or macOS
[x] target Linux, macOS, or iOS/tvOS/watchOS device
[x] be contained in a publicly accessible git repository
[x] maintain a project branch that builds against Swift 3.0 compatibility mode
and passes any unit tests
[x] have maintainers who will commit to resolve issues in a timely manner
[x] be compatible with the latest GM/Beta versions of Xcode and swiftpm
[x] add value not already included in the suite
[x] be licensed with one of the following permissive licenses:

BSD
MIT
Apache License, version 2.0
Eclipse Public License
Mozilla Public License (MPL) 1.1
MPL 2.0
CDDL

[x] pass ./check script run

I have a few other questions that I wasn't fully able to ascertain from the current documentation.
1. What should I do about these xfails?
"xfail": {
  "compatibility": {
    "3.0": {
      "branch": {
        "master": "https://bugs.swift.org/browse/SR-4696",
        "swift-4.1-branch": "https://bugs.swift.org/browse/SR-4696",
        "swift-4.0-branch": "https://bugs.swift.org/browse/SR-4696",
        "swift-3.1-branch": "https://bugs.swift.org/browse/SR-4696"
      }
    }
  }
}

The referenced bug number (SR-4696) was closed as "not a regression". So should I update the 3.0 & 3.1 commits to a version of SwiftLint that was adapted to support this change? Should I remove those commits from the compatibility section?
2. Should I use the earliest or latest commit supporting a Swift version?
For example, SwiftLint gained Swift 4 support starting around July (https://github.com/realm/SwiftLint/pull/1692), but obviously still does with the latest releases from the last month and even current master. So should I specify the earliest commit with support for Swift 4.0, or the latest commit?
3. Am I allowed to add my project to projects-incremental.json and projects-cperf-smoketest.json?
It seems useful to be able to add more projects here, but it's not referenced from the README.

Thanks for your help!

The referenced bug number (SR-4696) was closed as "not a regression". So should I update
the 3.0 & 3.1 commits to a version of SwiftLint that was adapted to support this change?
Should I remove those commits from the compatibility section?

Update the 3.0 & 3.1 commits to a version of SwiftLint that was adapted to support this change. The commits should still be compatible with the Swift version modes specified.

Should I specify the earliest commit with support for Swift 4.0, or the latest commit?

When adding a new Swift version mode commit to "compatibility", I recommend that you use the latest commit supporting the version mode. This will allow us to test more Swift 4.0 specific changes.

Am I allowed to add my project to projects-incremental.json and projects-cperf-smoketest.json?

projects-cperf-smoketest.json and projects-incremental.json contain subsets of projects.json that we've decided are useful for quick testing particular areas of the compiler. We want to keep these lists relatively small for this purpose. I'd be interested to hear suggestions if you believe there are more appropriate subsets.

@swift-ci test

Thanks for answering my questions @lplarson!


Export get_formulation_type

Coverage remained the same at 100.0% when pulling 50e546a6143debc978084dc129b5c01f44d7f47e on feature/get_formulation into 8f4f1a4912210656b6cb1ae1c7cf260b1cedd0c3 on master.


proposal: go.mod: add keyword to override differing import path

This proposal intends to address the Domain Expiration problem that is inherent to how Go handles package resolution.
The proposed solution is as follows:

add an override keyword with a syntax similar to replace

override expired-domain.com/pkg => github.com/original-author/go-pkg
overrides just the location the module is located at
version is kept as part of the required section

$SUMDB and go.sum assume expired-domain.com/pkg to be the import path and check the retrieved code against these known values
the source code is fetched from the new address by $GOPROXY and GOPROXY=direct connections
for developers running go get expired-domain.com/pkg@latest the information should still be looked for under the original domain.

override is project-specific and not intended to be a way to handle updating import paths. As such, it should be used with as much moderation and caution as is currently found with replace.
This way, it would also allow enterprises with strict compliance requirements to have a manual vetting process before making updates available (be it through review, or some other mechanism).

Either replace already does this (point to new source location, but the retrieved module declares its module path as the old one)
or this is a dup of #26904


feat: add basic windows support for poac

I've added basic support for windows, and now  can run and create a hello word! project and run it.
I'm using xmake for the backend build system, If there is interest, I might consider developing it further.
https://github.com/user-attachments/assets/bcdd6c68-393c-4a55-bda7-8a7800c486d8

Sorry, I don’t plan to support other build systems than GNU Make due to my limited resource. I’ll eventually add a feature like cargo-* binaries so you can use it instead of upstreaming this.


Is this good for locks for mutual exclusion?

I was wondering if you can share your input in how you would recommend using this solution. Would you recommend it on a scaled multi-replicated service architecture?
Here's how we use it, in Kubernetes terms:
multi-pod => Redis based locking via this repo to access a resource. However, each endpoint-triggered call might take close to 10 seconds. We use the default retry delay (100msec) in the repo.
The calls use up a lot of resources, and sometimes it might trigger a long garbage collection. It uses standard GC behavior for Golang.  There's an average of 30 calls / sec to these endpoints.
We are seeing only a number of the calls are successfully obtaining the lock. No matter the load during load testing, that number doesn't change.
I researched around, and found the following posts around Redis-based locking mechanisms.
https://martin.kleppmann.com/2016/02/08/how-to-do-distributed-locking.html
http://antirez.com/news/77
It seems RedLock algorithm was built to make Redis-based locking more reliable, after observing solutions like redis-lock (this repo) being the norm. However, Markin Kleppmann flat-out disagrees and states that Redis-based locking is fundamentally flawed.
I'd like to ask your opinion in this insight, and perhaps an advice on how better to use your repo.
Thank you!

@lovebes thanks for the detailed post but I - unfortunately - have no strong opinions here. We are using redis-lock in a scenario where we need to elect masters among our worker processes for jobs that need to run exclusively. We never experienced any problems with it and our monitors suggest that - over the past few years - all exclusive jobs have actually been run exclusively, we cannot see a job that has been run 2x by accident. That said, it wouldn't be the end of the world for us and if it was a critical requirement, I would probably spend more time researching - which is what you seem to be doing.
Salvatore has replied to Martin Kleppmann's criticism in his own blog post: http://antirez.com/news/101, but as said, I haven't been following it too closely.
One thing you may also want to look into is https://www.consul.io/docs/guides/semaphore.html


UE 5.3, Most Recent Update, Any GC causes a freeze, just this once I caught a crash as well confirming my sus its the plugin.

Using the code from the Unreal Marketplace, saw an update and did it and now I got this strange behavior.
I think prior to the update it was ok, might look at fetching and using older code from here.
Basically it seems like once I start using a coroutine and do "WaitForSeconds" once Unreal decides to do Garbage Collection it would freeze the editor. Somehow I finally got Rider to report a crash (many tests not even Rider would catch a crash, editor would just be frozen and windows didn't even think it stopped responding, but wouldn't even close).

Thanks for reporting, I will look into it as soon as I can.

Sadly, I'm not able to reproduce neither the freeze nor the crash.

I found a small issue, probably not related to this problem, but I've pushed some fixes to the version 3.3.9. They are already on github, UE Marketplace will take few hours to process it. I have no faith that this is it, but maybe...
Otherwise I'd need an example of how you use WaitForSeconds and maybe then I will be able to reproduce the issue. It might be because of some more complicated flow I'm not able to imagine.

Ok, I was able to reproduce it. I just had to run multiple co_awaits in one coroutine and run GC.
The problem has appeared in the update 3.3.6 where I was trying to resolve the potential problem of dangling coroutine handles, which might occur if the coroutine has never being resumed.
Unfortunatelly, explicit destroy on coroutine handle lead to broken heap allocation, which cause infinite loop or crash after Garbage Collector was called.
I reverted changes from 3.3.6 and let coroutine handles clean itself after the coroutine function. The only moment the handle will not clean itself is if it will never be resumed (for example, because the Actor when it was called has been destroyed before the coroutine resume). But it's an edge case and the potential memory leak will not be so big. I don't know how to handle such  situations properly too.
The version  on Marketplace will be live in few hours.

Just writing down to not forget: The idea of how to handle it:

Add a flag to coroutine promise informing the coroutine has been finished (the best place will be in return_void)
In the awaiter destructor check if it has:
I. invalid owner
II. valid coroutine handle
III. coro handle has not been finished. If not - mark as finished and destroy it explicitly.
The final_suspend should stay as suspend_never

The version 3.3.11 with fixed coroutine handles clearups is on the Marketplace. @nonlin - can you confirm if everything is ok, so I could close this issue?

I did not receive any feedback, I assume everything works fine. I'm closing the issue.


Release version 4.1

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

Comparison is base (d8c6e0b) 97.13% compared to head (a8e7b1f) 92.41%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
- Coverage   97.13%   92.41%   -4.72%
==========================================
  Files          12       12
  Lines         767      765       -2
  Branches      172      100      -72
==========================================
- Hits          745      707      -38
- Misses         18       52      +34
- Partials        4        6       +2

Flag
Coverage Δ

acceptance
?

unit
92.41% <100.00%> (-0.02%)
:arrow_down:

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

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


"Power saving" feature misdetects game being minimized

As in title (tested only using the new .NET version), randomly during playing I get limit to 10 fps, I need to tab out out of the game and tab back in to fix it. It seems to be completely random, as I don't do anything special in the meantime, not opening any overlays or anything, not in-game browser either.

same here i have sometimes complete random the problem that my game is in energy safe mode (grean leaf in task manager) and i need to restart my game to use again it Normal with my full power of the PC very very cursed bug

Yes, I've got the same problem.
解释一下：@34736384 省电模式会抽风，老问题了。Alt+Tab切窗口时，切回原神后省电模式经常还是开着，要多切几次窗口才会恢复正常帧率。考虑加一个简单的延时检测，检测窗口是否还在Unfocused状态吧。
版本：v3.3.0

It's only during gameplay, with the game in foreground, that I randomly get limited to 10 fps as if I minimized it.

I think you are right. I also noticed similar behavior while running genshin, starrail, and zzz at the same time. I have 'Mute while in background' enabled for all 3 games. Occasionally, when I'm playing genshin for some time, the other two games suddenly start playing their game audio. I'm pretty sure this window focus issue is something to do with Windows, so we may not be able to have a perfect fix.
p.s. In comparison to the power save feature, Nvidia's background app max framerate feature works well while also having CPU usage significantly reduced. However, because it won't work with the game minimized (e.g. by pressing Win+Down), I can't minimize the game to have its audio muted, so it didn't work for me.


Make SpriteRegionEditor remember state when switching scenes (fixes #3245)

This makes remember the values when switching scenes, but doesn't restore values when clossing and opening again the editor.
However, CanvasItemEditor state doesn't restore the value neighter, i think this is another issue.

My bad, my branch was not up to date.
This is ready to merge imo.

Looks fine. Is there an issue about actually saving the state when closing the editor to reuse it in the next session?
Also related, it looks like the ShaderEditionPlugin also won't remember its state: https://github.com/godotengine/godot/blob/199a29b353748245e9589d3ef38877613eef0ede/tools/editor/plugins/shader_editor_plugin.cpp#L308

Its deleted because it males no sense to remember the texto, editor will always loas current shader code

Should it be removed then? Such #if 0'ed code without comment is kind of confusing IMO.

There is los of old code with #ifdef 0, this has to be removed At some point if there's an alternative


Inline media props with custom variants do not apply to children using styled context

Current Behavior
When using a custom variant with a inline media prop on a parent component, the value of the variant does not appear to pass down to children of the component using the same styled context.
Expected Behavior
I'd expect that all children using that styled context would inherit the same variant value.
Here are the component and context definitions:
const StyledStackContext = createStyledContext({condensed: true});

const StyledStack = styled(YStack, {
  context: StyledStackContext,

  bg: "#AAAAAA",
  p: "$4",

  variants: {
    condensed: {
      false: {
        p: "$10",
        bg: "#DDDDDD",
      },
    },
  } as const,
  defaultVariants: {
    condensed: true,
  },
});

const StyledCircle = styled(Circle, {
  context: StyledStackContext,

  bg: "red",
  size: 100,

  variants: {
    condensed: {
      false: {
        bg: "blue",
      },
    },
  } as const,
});

And the render function:
export const Sandbox = () => {
  return (
    <RNView style={{width: "100%", height: "100%", padding: 50}}>
      <StyledStack $gtXs={{ condensed: false }}>
        <StyledCircle />
      </StyledStack>
    </RNView>
  );
};

With condensed={false} on larger viewports. I'd expect the circle to be blue.
Smaller Viewport (looks correct, small padding, red circle)

Larger Viewport (incorrect, large padding, red circle which should be blue)

Note that if I simply pass condensed={false} the parent and child are styled correctly.
condensed={true} (not using inline media props)

condensed={false} (not using inline media props)

Tamagui Version
1.100.6

Platform (Web, iOS, Android)
Web

Reproduction
https://github.com/jbean96/tamagui/tree/tamagui/viewport-styles-variants-not-applying

System Info
System:
    OS: macOS 13.6.7
    CPU: (10) arm64 Apple M1 Pro
    Memory: 1.16 GB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.18.0 - ~/.nvm/versions/node/v18.18.0/bin/node
    Yarn: 4.1.0 - ~/.nvm/versions/node/v18.18.0/bin/yarn
    npm: 9.8.1 - ~/.nvm/versions/node/v18.18.0/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 125.0.6422.142
    Safari: 16.6
  npmPackages:
    @biomejs/biome: ^1.7.3 => 1.7.3
    @manypkg/cli: ^0.19.1 => 0.19.2
    @nderscore/tamagui-typescript-plugin: ^0.6.0 => 0.6.0
    @playwright/test: ^1.40.1 => 1.40.1
    @types/eslint: ^8 => 8.56.2
    @typescript-eslint/eslint-plugin: ^6.19.1 => 6.19.1
    @typescript-eslint/parser: ^6.19.1 => 6.19.1
    @vercel/og: ^0.5.8 => 0.5.8
    eslint: ^8.56.0 => 8.56.0
    eslint-parallel: ^1.2.1 => 1.2.1
    fs-extra: ^11.2.0 => 11.2.0
    import-sort-style-module: ^6.0.0 => 6.0.0
    json5: ^2.2.3 => 2.2.3
    npm-run-all: ^4.1.5 => 4.1.5
    p-map: ^5.5.0 => 5.5.0
    patch-package: ^8.0.0 => 8.0.0
    prompts: 2.1.0 => 2.1.0
    turbo: ^1.13.3 => 1.13.3
    typescript: ^5.4.5 => 5.4.5
    ultra-runner: ^3.10.5 => 3.10.5
    zx: ^7.2.3 => 7.2.3

Not 100% sure if it will help or not, but could you try adding a name prop to the styled object? e.g.
const StyledCircle = styled(Circle, {   name: 'StyledCricle',   context: StyledStackContext,   ... }
and see if that helps? IIRC it worked for me in the past with a similar use case.

Can maybe use StyledStackContext.Props for now too.

Not 100% sure if it will help or not, but could you try adding a name prop to the styled object? e.g. const StyledCircle = styled(Circle, { name: 'StyledCricle', context: StyledStackContext, ... } and see if that helps? IIRC it worked for me in the past with a similar use case.

I'll push up the same branch with the names but trying locally it doesn't seem to fix it. Appreciate the suggestion though!

Can maybe use StyledStackContext.Props for now too.

@natew can you elaborate? From what I can tell I can't pass inline media style props to a styled context provider so it doesn't solve my original issue. Maybe there's another way?
I could use useMedia to toggle something like condensed={!media.gtXs} but then that breaks the SSR benefits that I get in our NextJS application.


Support for MS Windows

@andrus , fixed one failing test

commit bf4a27e:
this is a fix for the case when user.dir is windows-style Z:\a\b\c, and ResourceFactory tries to build a URI from its string representation (passed from FolderResourceFactory), resulting in a URI constructor treating drive letter Z: as scheme and generating a URISyntaxException

This is related to #73

@atomashpolskiy : path to "file:" URL conversion is only applied to "user.dir". What if constructor parameter is "z:\a\b\c"?

@andrus , are such resource IDs allowed? For some reason I assumed that resource IDs should be abstract URIs (absolute or relative unix-style pathes in case of resources located on the filesystem, i.e. /a/b/c in your case). Am I being wrong here?

Or file://Z:/a/b/c of course (i.e. a URI)

I guess you are right - resource paths should be platform-independent and use forward slashes. Still will need to test paths like z:/a/b/c I guess. In that case we should not call "slashify" inside normalizeResourceId.

Removed slashify, added tests

We still need conditional unit tests for file:///Z:/a/b/c
I think FolderResourceFactory.getUserDir() shouldn't be public.

@andrus , done. Instead of a conditional test I added a platform-independent test for both file:/Z:/a/b/c and file:///Z:/a/b/c cases


support for .side (selenium IDE) project files

Based on this weekend discussion with @ptrthomas (would like to start on the journey with a goal to run UI tests from karate in the not-so-distant future)

trying to support .side (for now only the latest Selenium IDE) project files

idea is to allow .side file import from karate-ui, auto generate corresponding feature files (each Test Suite maps to one Feature; and each Test Case maps to one Scenario)

support some dynamic configuration (to be provided not while importing .side file but while running the generated feature files): very rudimentary at this moment (hopefully will get inputs from Peter / community in evolving this)

need support for running feature-1 calling feature-2 calling feature-3 in UI (may be, we will create a separate ui enhancement issue for https://github.com/intuit/karate/pull/480#issuecomment-409809105)

later we can pickup old selenium IDE project files

couldn't find .side project file format specification (https://github.com/SeleniumHQ/selenium-ide/issues/77); picked a sample .side file to proceed with parsing for now

@ptrthomas please review https://github.com/intuit/karate/pull/500 at your convenience

@vmchukky this is very exciting. I made a slight change to the test, it is best practice to output generated files into target so that it doesn't interfere with git managed files
also, I tweaked the generated test and added a test under the junit4 module. because yeah the core doesn't have the http client.
https://github.com/intuit/karate/blob/edeae17011275787d550b83348d7556d85e43e87/karate-junit4/src/test/java/com/intuit/karate/junit4/selenium/sample.feature
I'm proposing instead of url, path may be better - so once a "setup" routine runs and sets sessionPath or some similar variable it just works. do note how karate's path syntax is designed so that you can use comma-delimited strings or variables and you don't need to worry about prefixing the slash character etc.

@ptrthomas thanks for taking the PR in.

the change to target folder makes sense (will keep it in mind)
Regarding the new SampleRunner in junit4, I was using a TestRunner in the same folder junit4/selenium.  Wasn't sure of adding it to PR.  Good that you added it.
Regarding the setup routine, I was contemplating between

having the user bring up chromedriver (or some other flavor of webdriver) and provide the URL to us
or karate-ui launching (with something like runtime.exec) the webdriver (reading the executable path from some place) as part of setup routine

yes we can work through the setup. it can actually be a java function we write that is intelligent enough to detect the platform, location of chrome (or other browsers in the future) and if it is already running etc. if we look at the selenium / appium projects we may get some ideas.
wdyt about adding selenium keywords to karate just like we have method, request etc for HTTP
with the new engine, I think it may be possible to "register" a set of user-defined keywords. one of the ideas I have is this:
if you have a class WebDriver with many static methods WebDriver.type(Map) you can do something like configure keywords com.myco.Foo and then you can do this
# command='type', target='name=q', value='karate'
* webdriver type { target: 'name=q', value: 'karate' }

wdyt about adding selenium keywords to karate just like we have method, request etc for HTTP
I like that, that would make selenium keywords first class citizens of karate and makes it easy for all existing selenium tests to use karate (with a simple convert/import utility).

closing this and we can open a new ticket if needed. right now I'm looking at automating chrome directly as an experiment and am thinking selenium using the webdriver spec is a phase 2


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1416 ms

Resolved: Bofrost.de performance has improved in b14cebc after 9 minutes.


[JOSS] Add a end-to-end example

This issue was filed as part of my JOSS review:  https://github.com/openjournals/joss-reviews/issues/6889
This one I've seen on your road map a couple of times so I think you're already aware of it, but I'm filing this issue  anyway just to highlight the (in my eyes) importance of it. Having a real world data example, that not just demonstrates it's real world use, but the whole end-to-end process including analyzing and drawing conclusions is really quite invaluable for news users. The tutorials on the individual metrics are great if those are the metrics you are interested in using, but an end-to-end example, is a real foothold for users trying to assess and contextualize your package, and also for less experienced users trying to gain experience. Encountering an end-to-end example, not just the "interesting" bits always significantly increases my option of a project, even if the actual exercise or "interesting part" is relatively small and boring. It helps to show the real effort required in using a package.
Again, this is just a suggestion, so I don't consider this a requirement for accepting the publication, but I would strongly encourage it. The authors already have a robust workflow for making these, so I assume that adding one of these is a relatively high ROI activity.

I'll let @tennlee respond in more detail, but some simple end-to-end examples exist in the github repository for this paper on Jive.
For example, see https://github.com/nicholasloveday/jive_paper/blob/main/src/dailypop_calibration_fig.ipynb which is used to produce figure 8 in the paper.
The use cases in that repository are fairly simple, because it's a fairly high level paper that we submitted to BAMS.
Some more complex examples of using scores can be found in this paper, but that code isn't publicly available.

That is fine, as a short term solution you could consider mentioning that somewhere in the tutorial gallery as well, though every jump away from your repository will lose you interest, so in a slightly longer term I would still consider your own.

https://scores.readthedocs.io/en/latest/tutorials/Threshold_Weighted_CRPS_for_Ensembles.html provides an end-to-end example tutorial now, but we still need something a bit better signposted and aimed at new users.


EMV2 (identifier) is not a package or a property set visible or existing

When I run OCarina, then the error  EMV2 (identifier) is not a package or a property set visible or existing  is found. I would like to know how to solve this problem.

Can you be more precise on the version of Ocarina used and the way you invoke it?


Can not update registry

cargo build --verbose
    Updating registry `https://github.com/rust-lang/crates.io-index`
Unable to update registry https://github.com/rust-lang/crates.io-index

Caused by:
  failed to fetch `https://github.com/rust-lang/crates.io-index`

Caused by:
  failed to authenticate when downloading repository

Caused by:
  [23/-1] error authenticating:

I am sure i can using browser to open https://github.com/rust-lang/crates.io-index
and i can using curl to get it

After upgrade to rust lang 1.8.0
 cargo build --verbose
    Updating registry `https://github.com/rust-lang/crates.io-index`
Unable to update registry https://github.com/rust-lang/crates.io-index

Caused by:
  failed to fetch `https://github.com/rust-lang/crates.io-index`

Caused by:
  [23/-1] username does not match previous request

Do you perhaps have an insteadOf entry in you ~/.gitconfig? I believe that a better error message happens on nightly, but Cargo is known to not work with insteadOf configuration.

Oh, yes. i have insteadOf settings.
But golang working currect. why cargo not working ?

see https://github.com/rust-lang/cargo/issues/2078
Now i changed the gitconfig , and it's working now


Possible to install from a LAN connection?

Is it possible to re-run the rdpwrap installer over a LAN/VLAN? If so, would be good to include this in the documentation or as a command line option (when run from another computer).

Try PsExec. However, this application requires that you have access to the SMB port (139/tcp).


Version 2.0

Fixes: #101 #75
Add support for Promises and ES6.
@bobby-brennan Can you please review it?

I set minimal node version to 4.2

I can't get npm test to pass.  Running
browserify -s APISpecConverter -o dist/api-spec-converter.js .
Gives me
Error: ENOENT: no such file or directory, lstat '/home/ubuntu/git/api-spec-converter/node_modules/jsonpath/generated/aesprim-browser.js'

Any ideas?  jsonpath is there (looks like as a global dep rather than a direct dep) but the node_modules/jsonpath/generated folder just contains parser.js

@bobby-brennan try to remove 'node_module' and reinstall packages.

@bobby-brennan Did it help?

Sorry for the delay - removing node_modules helped.  Changes look good to me, merge when ready and I'll release 2.0

@bobby-brennan Cool can you also uncomment API blueprint in demo.
Write somewhere that we now support promises.
And put short changelog somewhere.


NSwag for SignalR

metadata generation for SingalR hubs got some traction here: https://stackoverflow.com/questions/32638158/swagger-like-tool-for-signal-r-hub
I believe that at least the methods that can be invoked on the SignalR hub can be described by swagger, simmilar to WebApi.
Update by @rsuter: You can find a prototype for a SignalR generator here: https://github.com/RSuter/SigSpec

At least the DTOs can be generated for the hub API. But the operations (swagger) and clients (ts) must be different...

Maybe also interfaces?
public class ChatHub : Hub
{
    public void Send(string name, string message)
    {
        // Call the broadcastMessage method to update clients.
        Clients.All.broadcastMessage(name, message);
    }

    public IEnumerable<Message> GetAllMessages()
    {
       return _messages;
    }
}

interface IChatClient
{
public Task SendAsync(string name, string message, CancellationToken token = nulll);
public Task GetAllMessagesAsync(CancellationToken token = nulll);
}

I think swagger is not the correct format - or it can be misused for that... But these endpoints are not really RESTful callable...

The optimal solution would be to define a new specification (ie SignalRSpec) and write a new library which generates this spec and code for it using NJsonSchema (like NSwag does).

Well, I guess this specification should be done by microsoft. What do you think? Should we request it on uservoice?

Did you create a uservoice?

Not yet. Would you? I will vote for it. Your voice may be louder than mine :)

https://twitter.com/davidfowl/status/886997490460794880

Are you/anyone still interested in this?

Yes, I definitely am.

Though I'm not 100% convinced this would fit within the Swagger scope.

Nope, would be a new spec. But we could share a lot of code...

Something like:
{
    "hubs": { // the server hubs
        "Chat": {
            "description": "Foo",
            "operations": {
                // dictionary of name/operation
            }
        }
    },
    "callbacks": { // the client callbacks (dictionary of name/operation)
        "Send": {
            "parameters": {
                "a": {
                    "description": "First parameter.",
                    // ...JSON Schema
                    "type": "string"
                },
                "b": {
                    // ...JSON Schema
                    "$ref": "#/definitions/Foo"
                }
            }
            "result": {
                // ...JSON Schema
            }
        }
    }
    "definitions": {
        ...
    }
}

This would be only for SingnalR Core. My open questions:

Are the callbacks per hub or global (as in the sample)
How can we determine the callbacks - is there an interface for them on the server?

I've created an experiment: https://github.com/RSuter/SigSpec
Just start SigSpec.Console...
I dont like the name, do you have a better name for it?

I'm afraid I don't have the time at the moment to delve into this. I would really love to though.

I'm interested in this as well. Has there been any further work done?

There was no more work on this, but i'm open to push this further (SigSpec)

I dont like the name, do you have a better name for it?
SignalR Spec?

Yep, we have to upgrade the project to ASP.NET Core 2.1 but i dont see why it wouldn't work with the latest versions with some minor fixes...

The bigger part is building all the tooling around that (CLI, etc.) but maybe we can just integrate it into the NSwag tooling so we can share all this infrastructure...

I agree it should just be part of NSwag tooling. We use SignalR side by side with a Web API so generating client code for the whole API at once would be ideal.

At once will not be possible as a web api is described by a swagger spec and a signalr endpoint by something else (eg signalr spec). Is it a requirement to share dtos?

Sharing DTO could be achieved by feature known from good old WCF "Reuse type in referenced assemblies" what do you think? Or maybe we could optionally specify other spec files, whose DTO will be excluded, or included
Alternatively, most powerful approach is the one used in EF Core, where you can specify assembly which will be used to scan for customizations, esomething like this:
public class MyCustomDesignTimeServices : IDesignTimeServices
{
    public void ConfigureDesignTimeServices(IServiceCollection services)
    {
        services.AddSingleton<IPluralizer, HumanizerToIPluralizerAdapter>();
    }
}

The main problem is that we mostly dont know the referenced assemblies, but atm you can manually exclude dtos (ExcludedDtoTypes and AdditionalNamespaceUsages). I’m talking about the problem that the api and signalr endpoint might expose the same dto and thus it is generated twice (we have two independent specs).

@RSuter: Is it a problem to define the assemblies as argument? (Yes, i'm talking about the same problem)

We don't need to reuse DTOs.
I understand the underlying specs would be different, but if both were defined in the the same .nswag config file then the nswag tool should be able to generate clients for both in a single step, no?

The problem is that ATM the .nswag file can contain only one pipeline and it is swagger based (swagger gen -> swagger spec -> code gen). So it has to be enhanced with multiple pipelines with support for other spec types... lot of work and possible breaking changes

Ah, gotcha. Well two steps is not the end of the world.

Is it a problem to define the assemblies in CLI argument?

Should be doable... (we need an assembly param anyway to load setting extensions)

I’d suggest to first upgrade the project to core 2.1 and improve the client generators, rename project and test it in a custom cli...

Hi everyone!
If anyone still interested in this topic, please take a look at this solution for generating Swagger documentation for SignalR hubs(.NETsupport only):
https://github.com/Dorin-Mocan/SignalRSwaggerGen/wiki
https://www.nuget.org/packages/SignalRSwaggerGen/

Hi
Is this feature dead?
It would be really nice to get SignalR Hubs inside the SwaggerUI to be able to the SignalR connections

Hi
Is this feature dead?
It would be really nice to get SignalR Hubs inside the an ui like the SwaggerUI to be able to test SignalR connections

@truthz03 , check out my comment from above. I have created a package that can help you just DISPLAY your hubs in SwaggerUI, not interact. As for the interaction with hubs via UI, my package does not offer this feature.

@truthz03 , check out my comment from above. I have created a package that can help you just DISPLAY your hubs in SwaggerUI, not interact. As for the interaction with hubs via UI, my package does not offer this feature.

Thanks for your answer. Yes I only need to display my hubs.
Is your link above for NSwag because there is no "services.AddSwaggerGen" method available?
"services.AddSwaggerGen" is only available if I use "Swashbuckle.AspNetCore"

@truthz03 , it's a complementary package for Swagger. You need to install both Swagger and my package. Check out the documentation. And please toss a star for my repo on GitHub in order to support my work. Thank you!

@truthz03 , it's a complementary package for Swagger. You need to install both Swagger and my package. Check out the documentation. And please toss a star for my repo on GitHub in order to support my work. Thank you!

But to use your complementary pacakge I have to install "Swashbuckle.AspNetCore" otherwise I can't call "services.AddSwaggerGen".
I don't want to use "Swashbuckle.AspNetCore" because I use "NSwag.AspNetCore" and call "services.AddOpenApiDocument".
If I add both packages "Swashbuckle.AspNetCore" and "NSwag.AspNetCore" both will serve the web ui on "/swagger" so I have to use 2 different paths. One for the RestApi and "NSwag.AspNetCore" and another for "Swashbuckle.AspNetCore" where I will see ther SignalR Hubs and all RestApis
Or did I missunderstood something?

@truthz03 , SignalRSwaggerGen works in combination with Swashbuckle.AspNetCore, so yes, you'll need to use it as well. You could configure it on a different path and have both Swashbuckle.AspNetCore + SignalRSwaggerGen and NSwag.AspNetCore. Please test if this works for you. I have never tried such scenario.


Locale Information for locale de-at is wrong

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] other

Current behavior:

The angular locale information for the locale de-at is wrong.
At the moment the GROUP_SEP is an empty string.
Locale information is generated from locale spec version 29, which contains wrong locale information for austrian locale.
see: https://unicode.org/cldr/trac/browser/tags/release-29/common/main/de_AT.xml
Expected / new behavior:

GROUP_SEP should be a dot.
Locale information is fixed in spec version 30.
see: https://unicode.org/cldr/trac/browser/tags/release-30/common/main/de_AT.xml
Minimal reproduction of the problem with instructions:

Use angular-locale_de-at.js and format number with number pipe.
AngularJS version: 1.x.y

^1.6.2
Browser: [all | Chrome XX | Firefox XX | Edge XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView | Opera XX ]

all
Anything else:

AngularJS uses the Google Closure I18N library, to generate its own I18N files. This means that any changes should be made there first and then pulled into AngularJS.
That said, from a quick look at https://unicode.org/cldr/trac/browser/tags/release-30/common/main/de_AT.xml, the locale seems to use a different group separator for numbers vs currency (which is not supported by AngularJS anyway


use jruby-stdin-channel if possible for an interruptible stdin

solves to uninterruptible stdin when running on Java 8.
if running on Java 8, doing ^C will correctly terminate logstash unlike before where you had to type an extra character to unblock stdin.
related to elastic/logstash#1769
I only tested on OSX with Java 7 and 8.
I did not measure any noticeable performance difference using jruby-stdin-channel or not.

@colinsurprenant Can you rebase it off master? The specific stdin channel handling look good but I haven't tested it.

@ph rebased and bumped version. since specs are rather basic we should either have integration tests for both Java 7 + Java 8 or do manual tests on both JVM.

@colinsurprenant I will do a small manual testing for that. This only work on java 8?
We can probably drop the compatibility for java 7 in this PR and only merge it to the master branch?
Because we will bail out if the user try to run logstash under java 7  see https://github.com/elastic/logstash/issues/5242

LGTM
Tested with core with JDK7 and JDK8
^CSIGINT received. Shutting down the agent. {:level=>:warn}
stopping pipeline {:id=>"main", :level=>:warn}
^CSIGINT received. Terminating immediately.. {:level=>:fatal}
^CSIGINT received. Terminating immediately.. {:level=>:fatal}
Received shutdown signal, but pipeline is still waiting for in-flight events
to be processed. Sending another ^C will force quit Logstash, but this may cause
data loss. {:level=>:warn}
^Z
[2]  + 12237 suspended  bin/logstash -e 'input { stdin {} } output { stdout {}}'
JDK8

[2]  + 12237 killed     bin/logstash -e 'input { stdin {} } output { stdout {}}'
~/e/logstash git:master ❯❯❯ jdk8                                                                                ✱ ◼
~/e/logstash git:master ❯❯❯ bin/logstash -e 'input { stdin {} } output { stdout {}}'                                                                                ✱ ◼
--- jar coordinate com.fasterxml.jackson.core:jackson-annotations already loaded with version 2.7.1 - omit version 2.7.0
--- jar coordinate com.fasterxml.jackson.core:jackson-databind already loaded with version 2.7.1 - omit version 2.7.1-1
Settings: Default pipeline workers: 4
Pipeline main started
^CSIGINT received. Shutting down the agent. {:level=>:warn}
stopping pipeline {:id=>"main", :level=>:warn}
Pipeline main has been shutdown
~/e/logstash git:master ❯❯❯

@ph we should keep the normal stdin fallback because the jruby-stdin-channel gem uses a pretty implementation specific way of retrieving the nio channel from stdin and this could fail on future versions or alternate JVMs for example and if it does we revert back to using the normal stdin...

@colinsurprenant Make sense.

LGTM

bumped version and merged in master


[BSO] Inferno / Zuk error has occurred message

🐛 Bug Report
So when someone responds with Y or whatever letter/# they're told, it always says a error has occurred 😡
but it still re-sends the minion because you do =m and it says your back attempting

so in the end, it still does what it is supposed to, but with a error instead of the correct response.

I believe this was fixed.


Launch URL portion marking

Update database, data model and service calls to support portion marking of launch URLs

People will also need to be able to select the appropriate levels of sensitivity for this and #61, right? If so, we should also update the UI in order to allow this. If not, is there a blanket level that we will be associating with them automatically?


How to modify the sampling rate when use csmsc dataset to train the vocoder?

Dear professor:
I try to train a vocoder, and I have installed parallelwavegan,and I run the command: run.sh,however it came out with the traceback:
Traceback (most recent call last):
File "/home/liyp/anaconda3/envs/xll/bin/parallel-wavegan-preprocess", line 11, in
load_entry_point('parallel-wavegan', 'console_scripts', 'parallel-wavegan-preprocess')()
File "/data2/hcy/VQMIVC-main/vocoder/ParallelWaveGAN/parallel_wavegan/bin/preprocess.py", line 186, in main
), f"{utt_id} seems to have a different sampling rate."
I find that the sampling rate of csmsc is 24000hz,however the sampling rate of the my model is 16000,could you tell me how to modify the sampling rate?

Please change here
https://github.com/kan-bayashi/ParallelWaveGAN/blob/1f1f6750ea09ee416496eba79ec1a21dc20e5daf/egs/csmsc/voc1/local/data_prep.sh#L9
And launch the recipe from data prep.

Thank you.

However,the new problem comes out:
2022-05-11 19:50:59,583 (utils:95) ERROR: There is no such a hdf5 file (dump/train_nodev/stats.(grep -q hdf5 <(yq .format /data2/hcy/VQMIVC-main/ParallelWaveGAN/egs/csmsc/voc1/conf/parallel_wavegan.v3.yaml) && echo h5 || echo npy)).

Yes,I modified it as :stats_ext=$"(grep -q "hdf5" <(yq ".format" "${conf}") && echo "h5" || echo "npy")"
Because it came out with:syntax error near unexpected token `('

How can I solve this problem?

That is wrong. Please use it as is.

If you are not familiar with shellscript, please use hard coding.
E.g, stats_ext=h5

Do you mean modify the "stats_ext=$(grep -q "hdf5" <(yq ".format" "${conf}") && echo "h5" || echo "npy") " as "stats_ext=h5"?

right

Thank you, I will try.

How to stop the training?


Use wiremock-jetty12 thin JAR instead of the standalone JAR

Switch to WireMock's Jetty 12 JAR, avoiding the need to depend on the (much larger) standalone JAR.
In draft as this depends on some currently unreleased changes to WireMock's core.

Merging despite scanning issues as this will be fixed when the Gradle convention plugin update is released.


[19.03] torcs: fetch patch from stable source

Motivation for this change
Backport of #59316 to unbreak Torcs on the release
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

FYI @eri451 @astro


Packages install but some features not working

I installed the packages using the ionide-installer. Some features work while others don't. The syntax highlighting and FSI panel work, but linting (error highlighting), type tooltips and autocompletion do not.
I am using Ubuntu 14.04. I initially was using Atom 1.3.3 but upgraded to 1.4.0, the problem occurred on both versions. I installed the packages using ionide-installer v1.3.0.
One more thing to note is that after playing around with settings, everything (linting, autocompletion, tooltips) was working at one point. The next time I opened atom it had reverted back to just the FSI/syntax highlighting working. I have not been able to recreate that success since then.

Please open Developer Console (View |> Developer |> Toggle Developer Tools) and check if any errors are reported there.


Make tar.gz optional during artifacts inputs and outputs

Is this a BUG REPORT or FEATURE REQUEST?:
Feature Request
What you expected to happen:
Sometimes the artifacts may by a unit test report which is html format. It is good to make the s3 path public and let user to get the artifacts directly. So I think having an option in artifacts configuration to make the default tar and gzip behavior optional is necessary in this scenario.

Close this for a same open issue https://github.com/argoproj/argo/issues/784.

I would like to reopen this issue, I think it was only partially implemented. It seems that the current behaviour is to always untar/uncompress an input artifact if it is tarred/compressed: https://github.com/argoproj/argo/blob/1bd50fa2dfd828a04ff012868c98ba33bac41136/workflow/executor/executor.go#L164


Wildcard in import statements

In many files, the import statements use wildcard to import all the classes in the packages
E.g. https://github.com/Microsoft/mssql-jdbc/blob/master/src/main/java/com/microsoft/sqlserver/jdbc/KerbAuthentication.java
import java.util.*;
It is a good practice to specify the specific classes in these imports.

@iamatinkerer Thank you very much for pointing this out. You are right, it is a good practice to specify classes in imports. We really appreciate your participation and hope you could help us to fix it.

@v-xiangs

hope you could help us to fix it.

Can I take this issue and send a PR then?

@iamatinkerer - we are working on getting a development branch created. Once that's available, you can send a PR there. Please stay tuned and thank you for your help 😄

@iamatinkerer
We've just set up a dev branch. You can create a pull request there.

Closing this issue as it is fixed by #52


Update addons commands

Add these commands
netlify addons:create

netlify addons:config

netlify addons:auth

Adding addons will now validate against the add-ons manifest URL if one is supplied.
Closes https://github.com/netlify/cli/issues/229
(this is a redo of https://github.com/netlify/cli/pull/246)

Coverage remained the same at 100.0% when pulling 08ad2e07a68ff90f9be21b074832e4bb3c292824 on add-on-manifest into 5507344ce781f2eba398df52da1c37d9fbaaeaad on master.


use time.Duration instead of int

While working on https://github.com/canaryio/canary/pull/42 I came across several instances where we're using int data types when we should just be using time.Duration.  This makes things harder to reason about, as you have to continually ask "is this time in milliseconds? seconds?", etc.
Unifying these values around time.Duration would be a big win.

+1


feat: add CI and publish configuration

This PR was included in release version 1.0.0-beta.1 :tada:

This PR was included in release version 1.0.0 :tada: