Formatting Error

Code
var cart_product_name = '&#34Royal OG&#34 (Private Reserve) NEW'
cart_product_name = require('html-entities').AllHtmlEntities.decode(cart_product_name)
INPUT: &#34Royal OG&#34 (Private Reserve) NEW'
OUTPUT: '' OG" (Private Reserve) NEW'
Why is "Royal" being completely wiped out?

I did fix this by adding this code before it, guess the semicolons are required?
cart_product_name = cart_product_name.replace(/&#39/g, ''').replace(/&#34/g, '"').replace(/&amp/g, '&')

Hello @knoxcard, sorry for a delay in the response.
Your case was fixed in 2.0.1.


CMS XML export

Would like to explore a CMS export feature (I've used InDesign-structured XML exports previously) for content on the site (Podium, Blog). Happy to take your advice about what would be the most useful output mechanisms.
Text likely to be repurposed for print publications/other online destinations.

To chip in on this one: totally possible, two options come to mind:

We build a module that integrates directly into the CMS and allows admins to export XML from the CMS user interface, would perhaps allow for more flexibility in the structure but would probably be a little more work.

The data for these sections is already exposed via JSON endpoints (which the front-end app makes requests to in order to grab data), although these are mildly opinionated to make consumption for the current front-end easier, it would be possible to build a microservice to convert the data from JSON to XML.

Thanks! Do you envision there to be a significance in the number of hours used for the different approaches?

Not massively, I'm not familiar with InDesign-structured XML though!

That's okay. It doesn't have to be structured for InDesign—that's just what I have used previously. What's important is that we can export material out of the CMS in a way that's useful—the first option seems to cover our options a little more.

Yeah I think that would be the cleaner way to do it.


pr2_common_actions: 0.0.10-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository pr2_common_actions to 0.0.10-0:

upstream repository: https://github.com/PR2/pr2_common_actions.git
release repository: https://github.com/pr2-gbp/pr2_common_actions-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.6.2
previous version for package: null

joint_trajectory_action_tools
* Merge pull request #36 <https://github.com/pr2/pr2_common_actions/issues/36> from k-okada/kinetic-devel
  change maintainer to ROS orphaned package maintainer
* change maintainer to ROS orphaned package maintainer
* Contributors: Kei Okada

joint_trajectory_generator
* Merge pull request #37 <https://github.com/pr2/pr2_common_actions/issues/37> from k-okada/add_pr2_msgs
  add depend to pr2_controller_msgs
* Merge pull request #36 <https://github.com/pr2/pr2_common_actions/issues/36> from k-okada/kinetic-devel
  change maintainer to ROS orphaned package maintainer
* change maintainer to ROS orphaned package maintainer
* add depend to pr2_controller_msgs
* Contributors: Kei Okada

pr2_arm_move_ik
* Merge pull request #36 <https://github.com/pr2/pr2_common_actions/issues/36> from k-okada/kinetic-devel
  change maintainer to ROS orphaned package maintainer
* change maintainer to ROS orphaned package maintainer
* Contributors: Kei Okada

pr2_common_action_msgs
* Merge pull request #36 <https://github.com/pr2/pr2_common_actions/issues/36> from k-okada/kinetic-devel
  change maintainer to ROS orphaned package maintainer
* change maintainer to ROS orphaned package maintainer
* Contributors: Kei Okada

pr2_common_actions
* Merge pull request #36 <https://github.com/pr2/pr2_common_actions/issues/36> from k-okada/kinetic-devel
  change maintainer to ROS orphaned package maintainer
* change maintainer to ROS orphaned package maintainer
* Contributors: Kei Okada

pr2_tilt_laser_interface
* Merge pull request #36 <https://github.com/pr2/pr2_common_actions/issues/36> from k-okada/kinetic-devel
  change maintainer to ROS orphaned package maintainer
* change maintainer to ROS orphaned package maintainer
* Contributors: Kei Okada

pr2_tuck_arms_action
* Merge pull request #36 <https://github.com/pr2/pr2_common_actions/issues/36> from k-okada/kinetic-devel
  change maintainer to ROS orphaned package maintainer
* change maintainer to ROS orphaned package maintainer
* Contributors: Kei Okada

Travis error:
'''/home/travis/build/ros/rosdistro/kinetic/distribution.yaml''': Could not validate doc entry for repo pr2_common_actions with error [[[Could not validate repository with url https://github.com/pr2/pr2_common_actions.git and version indigo-devel from entry at line 6576]]]"] is not false

There's no indigo-devel branch

@tfoote sorry fixed


Signalr and Websockets

Hello,
Tried  to deploy an app with singalr on kestrel. (works on IIS)
But doesn't work at all on kestrel. (beta6)
How will Kestrel handle websockets? And guess at least long polling should work, but seem it's not.
I'm doing something wrong , or it's not implemented ?

When using kestrel, you need to use the websocket middleware

@davidfowl , any samples ? :)

Understand,
    app.UseWebSockets();

Will try that out.

Maybe you have a buggy Mono Version. AFAIK 4.0.1 got some bugs.
After I've upgraded to  >=4.0.2 it worked.


Restore debounce example

I had to disable it because event props are currently broken.
https://github.com/armanbilge/calico/blob/642b1b8ec119f3849b9566644024f82aa2737a53/docs/time.md?plain=1#L65-L67

Update: event props are no longer broken, but this example still is 😂
This code here:
https://github.com/armanbilge/calico/blob/c915f477e30dcc3728e28f3ab3edbb2df6b7eea3/docs/time.md?plain=1#L92-L94
Is supposed to be using this apply:
https://github.com/armanbilge/calico/blob/c915f477e30dcc3728e28f3ab3edbb2df6b7eea3/calico/src/main/scala/calico/html/HtmlTag.scala#L34-L35
But it seems to be confused:
[error] -- [E007] Type Mismatch Error: /workspace/calico/sandbox/src/main/scala/calico/Sandbox.scala:49:12
[error] 49 |            onInput --> (_.evalMap(_ => self.value.get).through(emailCh.sendAll))
[error]    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[error]    |Found:    calico.html.EventProp.PipeModifier[cats.effect.IO, org.scalajs.dom.Event]
[error]    |Required: Tuple

Well, the good news is it seems to be working in 3.3.0-RC1-bin-20230119-13b8d7d-NIGHTLY.


Support :group: for OpenAPI 2.0

This adds support for the :group: option to the Open API 2.0 half of the code.

BTW, I tried to run the tests locally, but they failed with:
E           py.error.ENOENT: [No such file or directory]: listdir('/src/sphinxcontrib-openapi/tests/OpenAPI-Specification/examples/v3.0/petstore.yaml',)

I tried to figure out how the Travis tests are passing, but I didn't see where those data files are supposed to be created.

Thanks for the patch, @nedbat! I'm so sorry for the delay in my review.

BTW, I tried to run the tests locally, but they failed with:

These data files are supposed to be pulled as git submodule. They are from official OpenAPI repository.
Do not worry about the tests. I'll add them right away.


Make atomicity optional when running jobs

As ...
P.D. - Plugin Developer
I want ...
To make DB atomicity in jobs an optional feature
So that ...
When i have jobs which are long running and make several DB changes i can reflect the DB changes in the DB as they happen instead of at the end of the job.
( e.g Reserve an inventory resource then deploy it in a single job )
I know this is done when...
I can set a property on the Job which disables the default atomic behavior
Optional - Feature groups this request pertains to.

[X] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[ ] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[ ] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
No response
External Dependencies
No response

Will need to think about how this interacts with job scheduling and approvals - if the job run is no longer atomic, then there will be no ready way for an approver to "dry-run" the job to preview its effects on the database, unless the job itself implements a custom "dry-run" parameter.
This may be desirable though as the current "dry run" functionality is potentially misleading, as if the job is capable of making any changes to external systems, it is up to the job author to ensure that it doesn't do so when run with commit=False at present.

In my opinion it would be a great feature if a job-result could be forwarded to the approval jobs queue.
This would allow users that can execute a job without having the "Commit" permission. To forward the job with a reference to the job-results to a user taht can commit the change.

Good morning @tbotnz!
We recently planned out v2.0 of Nautobot and have decided to rip out the automatic atomic behavior of jobs as a breaking change.
As you saw it was essentially just wrapping everything in with transaction.atomic(): this is something that can be easy to move down to the job author level. We may or may not provide simple handles based on inheritance to restore this magic but having two separate inheritances vs wrapping your code in a single function call is a bit even on complexity change.
We'll be documenting this as well as providing examples of how a job author can maintain atomic transactions.
If this changes during prototyping->implementation we will re-open this issue.
As always, thank you for your contributions!


Double deposit/withdrawal of gold to/from warband bank

Issue Description:
Every time I open the bank, it does the automatic withdrawal/deposit twice.
Started noticing this today. I had automatic gold deposits set to keep me at 1000g without the "keep me safe" checkbox. My AH alt had about 18k gold, I opened the bank. It first deposited the right amount, and then immediately gave me a "I don't have that much" type error (the chat still showed the "Deposited 17601(G)74(S)2(C)" message, twice. I then changed the limit to 500g (I had done this on my main wow account, but not this one) and opened the bank again. It deposited 500 twice, bringing me to 0. I close the bank and reopen, and it withdraws twice, bringing me to 1000 again. I manually deposit 200 (bringing me to 800) and open the bank, it deposits 300 twice, bringing me to 200. I change the limit to 300 and open, it withdraws 100 twice to 400 (I did the last to make sure it wasn't some other addon I didn't know about). Activating the "Keep me Safe" checkbox will result in it only doing the withdrawal/deposit once, after you click "Yes".
Steps to Reproduce:
Open bank with auto-deposits turned on and a value different from what you have. Observe the deposit/withdrawal happens twice for the "correct" amount.
Expected Behavior:
When the bank is opened and the Warband Bank "Deposit Keep Amount" is set but "Keep Me Safe" isn't, it automatically withdraws or deposits the correct amount once, bringing you to the desired gold total.
Actual Behavior:
When the bank is opened and the Warband Bank "Deposit Keep Amount" is set but "Keep Me Safe" isn't, it automatically withdraws or deposits the correct amount twice, making you over- or undershoot the desired gold total.
Additional Information:
Issue occurs on the 3.0.4H1 version, which is the latest released version at the time this bug ticket is created.

Fixed in 07ba1ea.


Stopping the ising machine should also not reset the phase counters.

The fact that phase counters reset when the ising machine stops makes profiling performance more difficult, because we can't force the oscillation to stop after a certain amount of time and then read out the phase counters.

Fixed!


Cleanup misc flake8 errors in prep for auto-running

In preparing for running flake8 during CI, go ahead and cleanup all the flake8 errors I could find.  The only case I have not cleaned up is the use of setUp not being capitalized correctly per pylint when used in unittest. I have other plans for fixing that.

Closing in favor of another PR with more work around this.


"On this page" summary includes all headers in page, no matter if they are conditionally chosen or not

Problem description
On conditional content pages like the Crosswalk Guides PR that allows to choose between various clouds, the "On this Page" overview is used as easily navigate around the page. The issue is that because we use a cloud chooser to conditionally show sections per cloud, not all headers should be listed in this overview, only the ones for the active tab / cloud
For example, when AWS is selected we should only see the IAM sections for AWS, not also for Azure and GCP
In http://content-bucket-268a8c6.s3-website-us-east-1.amazonaws.com/docs/guides/crosswalk/kubernetes/identity/

Affected product version(s)

Suggestions for a fix

cc @cnunciato @justinvp

I used a mix of header changes to H4's and setting no_on_this_page: true in page front matter where appropriate.
Closed in https://github.com/pulumi/docs/pull/1864


Fix strand flag copy-pasto

Alas, I was so busy getting the Python syntax right that I didn't adequately check the actual tests here…

+1

+1. Merging.


Fix-Date type column sort in Table widget

Description
Use input date format to sort date type columns in Table Widget
Fixes #5780
Fixes #6207
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: fix/table-widget-date-column-sort

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
53.29 (0.01)
34.75 (0.01)
31.33 (0)
53.85 (0)

:green_circle:
app/client/src/utils/autocomplete/TernServer.ts
50.71 (0.24)
38.6 (0.88)
36.21 (0)
55.35 (0.27)

:green_circle:
app/client/src/widgets/TableWidget/derived.js
54.26 (0.25)
56.25 (0)
37.84 (0)
54.07 (0.27)

@vicky-primathon Calendar on table filter is getting closed every time I try to change the month/year.
https://www.loom.com/share/f4767d85e2294f5db2a22b07248cbfe4

@vicky-primathon Calendar on table filter is getting closed every time I try to change the month/year.
https://www.loom.com/share/f4767d85e2294f5db2a22b07248cbfe4

This is working on deploy preview

/ok-to-test sha=0aaf99c

@vicky-primathon  verified this PR and both the issues are working as expected.


CLI force me to have bootstrapModule in my main module

Bug Report or Feature Request (mark with an x)
- [x] bug report
- [ ] feature request

Versions.

@angular/cli: 1.0.1
node: 7.10.0
os: darwin x64
@angular/common: 4.1.3
@angular/compiler: 4.1.3
@angular/core: 4.1.3
@angular/forms: 4.1.3
@angular/http: 4.1.3
@angular/platform-browser: 4.1.3
@angular/platform-browser-dynamic: 4.1.3
@angular/router: 4.1.3
@angular/cli: 1.0.1
@angular/compiler-cli: 4.1.3

Repro steps.

The log given by the failure.

Setup an application on .angular-cli.json
Remove platformBrowserDynamic().bootstrapModule(DemoModule) from the main project
Try to run the test task for that application
It will show Tried to find bootstrap code, but could not. Specify either statically analyzable bootstrap code or pass in an entryModule to the plugins options.

$ ng test --app boa
Tried to find bootstrap code, but could not. Specify either statically analyzable bootstrap code or pass in an entryModule to the plugins options.
Error: Tried to find bootstrap code, but could not. Specify either statically analyzable bootstrap code or pass in an entryModule to the plugins options.
    at Object.resolveEntryModuleFromMain (/Users/ubi/Sites/open_source/boa/node_modules/@ngtools/webpack/src/entry_resolver.js:118:15)
    at AotPlugin._setupOptions (/Users/ubi/Sites/open_source/boa/node_modules/@ngtools/webpack/src/plugin.js:143:50)
    at new AotPlugin (/Users/ubi/Sites/open_source/boa/node_modules/@ngtools/webpack/src/plugin.js:26:14)
    at _createAotPlugin (/Users/ubi/Sites/open_source/boa/node_modules/@angular/cli/models/webpack-configs/typescript.js:55:12)
    at Object.exports.getNonAotTestConfig (/Users/ubi/Sites/open_source/boa/node_modules/@angular/cli/models/webpack-configs/typescript.js:103:19)
    at WebpackTestConfig.buildConfig (/Users/ubi/Sites/open_source/boa/node_modules/@angular/cli/models/webpack-test-config.js:17:31)
    at init (/Users/ubi/Sites/open_source/boa/node_modules/@angular/cli/plugins/karma.js:79:94)
    at Array.invoke (/Users/ubi/Sites/open_source/boa/node_modules/di/lib/injector.js:75:15)
    at Injector.get (/Users/ubi/Sites/open_source/boa/node_modules/di/lib/injector.js:48:43)
    at /Users/ubi/Sites/open_source/boa/node_modules/karma/lib/server.js:143:20
    at Array.forEach (native)
    at Server._start (/Users/ubi/Sites/open_source/boa/node_modules/karma/lib/server.js:142:21)
    at Injector.invoke (/Users/ubi/Sites/open_source/boa/node_modules/di/lib/injector.js:75:15)
    at Server.start (/Users/ubi/Sites/open_source/boa/node_modules/karma/lib/server.js:103:18)
    at Promise (/Users/ubi/Sites/open_source/boa/node_modules/@angular/cli/tasks/test.js:35:25)
    at Class.run (/Users/ubi/Sites/open_source/boa/node_modules/@angular/cli/tasks/test.js:15:16)
error Command failed with exit code 1.

Desired functionality.

The test should run without force me to have a bootstrapped application.
Mention any other details that might be useful.

Repo: https://github.com/straw-hat-llc/boa/tree/develop
This project is trying to create a UI framework for be used cross multiple applications. Because of that, I actually don't have an application (unless I am misunderstanding Angular), I actually have a bunch of code that compose a UI framework.
I am using boa app https://github.com/straw-hat-llc/boa/blob/develop/.angular-cli.json#L8 for setup the testing piece of the framework, just the spec testing because the e2e will be excluded, but because my main file do not contains a bootstrapModule the task is failing.
This should let me run the testing, base on this use case, I do not need a real application to be able to run. I am trying to test an framework code, no a real application (from the point of view that this is no meant to be deploy by it's own)

demo app https://github.com/straw-hat-llc/boa/blob/develop/.angular-cli.json#L20 is used for development, where you could add your use cases of the component and you can test it out visually. Also I will add website app which it will be used for github pages website

Can you try setting entryModule for the AotPlugin from @ngtools/webpack?
The error message tells half of the story: either statically analyzable bootstrap code or pass in an entryModule to the plugins options. You will need to ng eject your webpack config, and then edit the webpack plugin options.

@dherges what do you mean by webpack config? I do not have any Webpack file at all. I am using ng-cli generated code for this.

The CLI is currently only intended for the creation of web applications.  However, library creation support is planned for a future release.

@clydin is there any issue related to that? I want to follow the thread and see if I can help somehow the process

@yordis there are two independent things here:

authoring a library: you need to build it in Angular Package Format, you could write your own build scripts or use a tool like ng-packagr
the ng cli: it uses webpack under the hood. you can ng eject its config.

@dherges so is my concern a valid issue/concern? I don't mind to wait if what @clydin said it's true.
I see many use cases where I have some "app" that are just framework/library code but I want to run some tasks from the CLI independently.

@yordis Hi. We do not support libraries at the moment. What you're seeing is failure from our code to find the entry NgModule, and if it cannot find it it will fail. We need that information for knowing where to start looking for routes. If you only want to run tests, you should be able to setup karma yourself and use the karma CLI.
For more information regarding libraries, see my comment here: https://github.com/angular/angular-cli/issues/1692#issuecomment-304970595
I'm going to close this as Working-As-Intended.


Linking via in-game chat

We'd love to be able to remove the reactions from the lobby message, they eat up a lot of requests and are still a bit cumbersome for players on mobile.
Now that the capture client is able to emit events for chat messages, we should generate a link number for each player. When a player puts that number into the in-game chat, we link their AU Data like before.
The link codes can probably just be 1-10 or 0-9. In the discord lobby message we put the tracked member's names along with their code. A single digit code should be easy enough to relay via voice for people on mobile who can't app switch.
Example
LOBBY MESSAGE

Linked Players:
@Soup	:aured:
Link Codes:
~@Soup 	1~
@Kyle 	2
...

Maybe ~strikethrough~ used codes?

This would be awesome! Most of my friends play on an iPhone and they will be disconnected from the lobby if they switch to discord to chose their color. I have to link them all one by one in discord witch wastes fun game time.
/Mattias

This would be awesome! Most of my friends play on an iPhone and they will be disconnected from the lobby if they switch to discord to chose their color. I have to link them all one by one in discord witch wastes fun game time.
/Mattias


Add folly to global pinning?

There are a few packages depending on folly. These have to be rebuilt to a consistent version to ensure everything works (and is co-installable). Is there any particular reason why folly hasn't been added to the global pinning yet?
@conda-forge/folly

folly is being released every week and is admittedly API and ABI unstable in between commits and not designed to be packaged as a shared library.
Pins are intentional in this regards and used since only 3 projects depends on folly. I am afraid that rebuilds for each new release of folly might just create a lot of disruptions. :confused:

folly is being released every week

We have a similar situation with the aws_sdk_cpp, which releases even more often. The solution there is to only migrate every ~10-20 patch releases. With a weekly cadence though, I think it would be feasible to build all the folly versions, but only migrate once every ~month or so. Since there are not a lot of dependent feedstocks, this shouldn't be a big issue.

I did not know it was possible to perform migration for fewer versions. In that case, I am up for adding it to the global pin with a lower rebuild frequency than release frequency (e.g. rebuild every months)
What do you (and others) think?

Fine with me, but I guess we would probably run into the situation where often the pinned folly version hasn't been rebuilt with any other pins. I would guess that it would be easier to only build a version here once a month.


Implement Persistence

What do you think about implementing some persistence? Adding different Fire TV Devices every reboot needs a lot of automation in my HA-System.

What about a sort of config file? Run the server like firetv-server -c my_config.yaml, where the config has a list of devices to be added at startup. If a device is added that isn't already in the config, it could be written to the config.
Is that the sort of thing you're thinking of?

yes exactly it is. If we still want to use the post route we should implement a persistence flag attribute which means: add it to the config file. But then we also need some delete route.
Or we keep it simple and only way to add devices with persistence is editing the yaml file.

I would support such an enhancement, in simple or comprehensive form. I'm not sure when I'd be able to write it though. A PR would be certainly welcome.

All Right, i will see if my python is good enough to do so and i`ve i have time on the weekend.
Can you provide me some starting points with parsing yaml on python? I am coming from Ruby... :)

Yep, pip install yaml, then import yaml. Then you can yaml.load(...) to load from YAML to a python object, and yaml.dump(...) to go the other way. That's about all you need.
Full docs here: http://pyyaml.org/wiki/PyYAMLDocumentation

Not sure if it is relevant. Config would be the ideal approach, but I am currently using curl to add my devices at startup and it works fairly well
curl -X POST -H 'Content-Type:application/json' -H 'Accept: application/json' --data-binary @/home/hass/FamilyRoom.json http://localhost:5556/devices/add
--FamilyRoom.json
{
"device_id": "FamilyRoom",  "host": "<IP_ADDRESS>:5555"
}

@kdvlr nice quick solution :)
@happyleavesaoc thanks for your advice, i will do my best :)


Issue with rtracklayer specific to R4

The liftover_bedpe function gives an error that occurs only in R version 4 (not R3.6):
> hg19_sv <- get_manta_sv() %>% head(20)
> hg38_sv <- liftover_bedpe(bedpe_df = hg19_sv, target_build="hg38")
  CHROM_A  START_A    END_A CHROM_B  START_B    END_B NAME SOMATIC_SCORE STRAND_A STRAND_B TYPE FILTER VAF_tumour
1    chr1  1556541  1556547    chr1  1556664  1556670    .            40        -        -  BND   PASS          0
2    chr1  6012725  6012732    chr1  6012825  6012832    .            48        +        +  BND   PASS          0
3    chr1  8464072  8464090    chr1  8464293  8464311    .            40        +        +  BND   PASS          0
4    chr1 10084099 10084251    chr1 10084266 10084411    .            48        -        -  BND   PASS          0
5    chr1 10526162 10526172    chr1 10526290 10526300    .            40        -        -  BND   PASS          0
6    chr1 15878430 15878436    chr1 15878608 15878614    .            40        +        +  BND   PASS          0
  VAF_normal DP_tumour DP_normal tumour_sample_id normal_sample_id pair_status
1          0        55        73  00-14595_tumorA  00-14595_normal     matched
2          0        32        85  00-14595_tumorA  00-14595_normal     matched
3          0       215        84  00-14595_tumorA  00-14595_normal     matched
4          0        43        50  00-14595_tumorA  00-14595_normal     matched
5          0       121        92  00-14595_tumorA  00-14595_normal     matched
6          0       126       101  00-14595_tumorA  00-14595_normal     matched
Error: 'chr1	1621161	1621167	.	0	-	1621161	1621167	0	1	6	0' does not exist in current working directory ('/projects/rmorin/projects/gambl-repos/gambl-kdreval').
In addition: Warning message:
In if (!grepl("chr", original_bedpe$CHROM_A)) { :
  the condition has length > 1 and only the first element will be used

@mattssca tracked it down to the rtracklayer issue with this line:
bedpe_obj = rtracklayer::import(text = char_vec, format = "bed")

The solution needs to ensure functionality is maintained both in R3.6 and R4.

Is this a situation where I fixed a bug when we upped R versions and created a bug by doing so?

I don't think so - that error was reported by Helena in August with exact same error, but you and me both did not reproduce it at that time. Today, this error came up again - and I reproduced it in R4, but not in R3.6. I think this is a difference in the version of one of the rtracklayer's dependencies


Quesion: Can SSLproxy be configured to not re-encrypt the traffic?

I'm exploring if this will work for my application.  Actually, I'm hoping to use two instances of SSLproxy in the following manner:
Server A is my HTTPS server with my certificate and private key
Client A is my client that will connect over SSL to my HTTPS server
Router A will route the traffic (based on Server A destination) to Proxy A
Proxy A will be a decryption-only proxy that will have a copy of Server A's certificate and private key, so it can decrypt the traffic.  Then it will route the decrypted packets back to Router A with layer-3/4 transparency (source will still be client IP/port and destination will still be server IP/port.  Maybe tproxy and IP_TRANSPARENT will be needed for this part?)
Router A will route the decrypted packets through a chain of programs, each which will route back to Router A with layer-3/4 transparency.
At the end of the chain of programs, Router A will route the decrypted packets through Proxy B.
Proxy B will encrypt the traffic and route back through Router A with layer-3/4 transparency and Router A will send the encrypted traffic to Server A.
The return traffic will go through all the same hops as above, but in reverse order.  E.g. Proxy B will receive the encrypted traffic from Server A and will decrypt it before sending back through Router A.
(I'm using the term "route" a little loosely above, as most of it will by L2 switching.)
Can SSLProxy be used in such a scenario, where one instance is only decrypting the traffic, and one instance is only encrypting the traffic?  I see from the documentation that SSLProxy can itself invoke the external programs on the decrypted traffic, but I don't think that will work in my application since "Router A" will be making the decisions of which programs to invoke, and it needs to have the decrypted traffic to do so.

Short answer: I don't think SSLproxy can help you do that, at least as you describe it. As described in its README, SSLproxy can decrypt and divert packets to other programs, and the listening programs must return those packets back to SSLproxy.
I guess you don't have much control over the Router A in your description, perhaps it is one of those closed source commercial products. And I guess that is why you cannot directly use sslproxy to do all of these. Because otherwise SSLproxy is designed to do this as Router A in your design. (Btw, sslproxy does not invoke listening programs, it just diverts packets to them, but I guess you already understand that.)
Also, note that once you divert a packet to another program at L3/4, the source and destination addresses of the packet will be lost. And the only way the listening program to know those addresses is by processing the info in the SSLproxy line in the first packet diverted to it, as described in README. Accordingly, note that even if the second (imaginary) sslproxy which was supposed to reencrypt the packets in your design were possible, that second sslproxy would have to process the SSLproxy line inserted by the first (imaginary) sslproxy decrypting the packets. SSLproxy is a MiTM proxy, and I don't know how this can be done at L2.
Also note that the packets generated in mirror logging are pseudo packets sent to mirror targets at L2, just for logging purposes, so that mirror logging does not help either.

Thanks for the quick response!  How difficult would it be for me to modify the source code to where the outgoing traffic is not encrypted?  I've only started perusing the code, and I saw IP_TRANSPARENT and TPROXY are present in some of the files.  That's making me hopeful that I could maintain the client's source IP/port in the outgoing (server-bound) packets, similar to how the squid proxy can use tproxy to masquerade as the client.  If I can disable the server-side encryption, then I could have a trivial mirroring application that sslproxy diverts the decrypted packets to, and it just sends them back to sslproxy (maybe iptables could perform the rerouting so it doesn't even hit an external application, but merely flows straight back to sslproxy's source port)?

If you can develop a listening program, then sslproxy can divert the decrypted packets to it. The first packet will contain the SSLproxy line which will include the source and destination addresses you will need.
If you don't return the packets to sslproxy (from the listening program or somewhere else, but you should return it to the dynamically assigned port reported in the SSLproxy line), the connection will eventually time out (the timeout period can be configured). Normally, this should never happen, because we try to close the connections as soon as possible to reduce resource usage (but it happens with persistent connections, such as those to Github).
But I have no idea if this will help you in any way. Because, for example, after receiving a connection from a client, sslproxy connects to the target server first, so that it forges the certificate of the server to use with the connection to the client. As should be obvious, if you don't return the packets to sslproxy, another program (for example, the second imaginary sslproxy) should open another connection to the server to reencrypt and send the packets to the server. This does not sound right to me.
So, I am not too hopeful that you can modify the source code of sslproxy to achieve what you describe. Perhaps you can understand better once you are more familiar with its source code. Perhaps you can find a way, but the modifications needed would be quite considerable, imo.


evaluate speed and efficiency improvements for rg.load and rg.log

Is your feature request related to a problem? Please describe.
rg.load and rg.log slow from time to time.

https://github.com/argilla-io/argilla/issues/2398
https://github.com/argilla-io/argilla/issues/2261

Describe the solution you'd like
as a quick fix:

allow for increasing the chunk size
only load field you need
equally divide batches that need to be logged

Describe alternatives you've considered
N.A.
Additional context
N.A.

Your first quick fix is also described in #2454. I'll be tackling that issue today.


Fix exclude not work for transitive dependencies and add recursive config

dependency.excludes doesn't work for transitive dependencies so this might bring some dependencies that users want to exclude.
This patch also adds a recursive config so users can decide whether to only exclude one dependency or also exclude all dependencies brought by this dependency.
/cc @jmjoy currently it seems Rust (cargo.go) only resolve direct dependencies, not including transitive dependencies, so we might miss some transitive dependencies in the license report.
/cc and thanks to @slievrly for reporting this bug

@kezhenxu94  test has passed.


Update to use Xid instead of UUID

close #32

I'll merge this into the 0.3.0 RC in the next couple days.


Check Mate isn't working

The king won't check mate. Needs fixing.

@mankinchi You can work on this

I have an issue. I couldn't even load the board properly. It contains no pieces


Wrong phpstan configuration in multi workspace environment

If you have more than one workspace open, the configuration of the first workspace will always be selected, not the configuration of the active document workspace. This means that only the first workspace will work correctly, and any others will not.
My unprofessional guess would be that the selected workspace is only the first, not the active workspace.

Ahh yes your guess is correct. I didn't really build this with multi-workspace support in mind since that's also pretty hard to implement when working with PHPStan (and even worse when things like Docker come into play). I don't think I'll be adding multi-workspace support any time soon since it's a lot of work and adds a lot of complexity for (I think) a relatively small group of users. I'm considering add a "hey multi-workspaces are not supported" warning but I'm not actually sure if that's worth it, what do you think? Would that help at all or is the error obvious enough anyways?

It would be nice to have a warning somewhere, as it took me some time to figure out the cause of my PHPStan errors - the wrong config/working directory :smile: and probably to avoid the creation of additional issues
As for complexity, isn't it enough to set the workspace to the active one, rather than the first one? At least that would solve all my problems with it (finding the right config/working directory) :thinking:

Ah I'll add a warning then :)
I don't think there is such a thing as an active workspace. All are active at the same time and the only way to really infer this is to get the current workspace from the currently opened file. This then adds the complexity of needing to possibly change this workspace every time the active file changes. And then a lot of things depend on that such as workspace configuration, ${workspaceFolder} in arguments etc. These are all instantiated once from the workspace but will then need to be re-checked every time a file is opened. That's kind of the complexity of multi-workspace setups. I'd love to implement this if it were relatively easy but unfortunately it's kind of hard to.


Streaming Issues: Duplicate Values Displayed

I'm using this repo to build a RAG chatbot for my docs.
from the LangSmith monitor I see the generation is doing fine.
but when i stream to the frontend, it has a weird error of duplicating values.
i'm trying to fix it and would love some help.
I'm on the most recent deployment of the repo.
followed all instructions, so i have a weaviate vector db setup

I did end up filtering out the duplicated values.
`if (Array.isArray(streamedResponse?.streamed_output)) {
const uniqueList = streamedResponse.streamed_output.reduce((acc, item) => {
if (!acc.includes(item)) {
acc.push(item);
}
return acc;
}, []);
      accumulatedMessage = uniqueList.join("");
    }`

but I'm curious if this is the best way to handle this


Migrate the config/ directory to a workspace package

Followup to issue #451 and #450, adding metadata to build artifacts, and look to migrate the existing config/ folder to be a workspace package.  This is following the openshift dynamic-plugin-sdk setup (see [1] and [2]).
The existing @kubev2v/build package can be reusued/renamed to @kubev2v/config as needed.
Following the eslint custom plugin style, as used in the dynamic-plugin-sdk [2], will require a package for eslint and prettier that is apart from the future config package.
[1] - https://github.com/openshift/dynamic-plugin-sdk/tree/main/packages/common
[2] - https://github.com/openshift/dynamic-plugin-sdk/tree/main/packages/eslint-plugin-internal

eslint portion covered by #486

Since @yaacov moved the remaining configs files out of config/ (see PRs #487, #488, #489, #490, #491, #493, #494), closing this issue.


Review core classes and reduce code duplication

Add additional utility classes to reduce code repetition and simplify for core.

This has happened parasitically over a year. Some major optimisations were added recently.


Support Fish Audio tts

Add FishAudioTTSService for real-time TTS processing via WebSocket

@MoeMamdouh Do you mind fixing the formatting issues? Take a look at https://github.com/pipecat-ai/pipecat?tab=readme-ov-file#setting-up-your-editor to setup your editor.

@MoeMamdouh I ran this and hit two issues:

There is popping at the start of each phrase
The last phrase is not being returned from Fish

The first issue can be resolved but the second seems like a Fish Audio issue. Have you noticed this and/or found a fix.
FYI: I started a parallel branch for this that confirms a bit better to the Pipecat TTS style: https://github.com/pipecat-ai/pipecat/pull/899. We'll probably close this one out and work on the other, but before I do so, I want to see if you've resolved issue #294
(In the PR I posted, I was able to solve the popping issue by filtering on size. There may be a better way, but for now, it's fine.)

@MoeMamdouh I got a response from the Fish team and resolved the issue. I'm going to close this PR and we'll use https://github.com/pipecat-ai/pipecat/pull/899 instead. Thanks so much for helping to get this started. I hope this service works well for you.

@MoeMamdouh I got a response from the Fish team and resolved the issue. I'm going to close this PR and we'll use #899 instead. Thanks so much for helping to get this started. I hope this service works well for you.

thank you @markbackman  You are welcome


Slider does not appear in UITableViewCell on iOS 8

Added a UIView to the cell prototype in Storyboard, added NMRangeSlider as the class.
cell.rangeSlider.setLowerValue(0.0, upperValue: 10.0, animated: false)

Shows a blank white cell. Thoughts?

@LandonRover you should better check #54 commit to fix your problem.

That looks like it, thank you for the heads up about the commit.

I was able to get the slider to show, but the middle is still blank (not the orange shown in the demo).

@LandonRover try,
pod 'NMRangeSlider', :git => 'https://github.com/muZZkat/NMRangeSlider.git'

this one.
It seems #54 Pull request also has bundle issue.
@muZZkat And I think 1.2.2 version is not published to cocoapods trunk.
default version when using
pod 'NMRangeSlider'

is 1.2.1

@chanil1218 Thanks so much, that fixed it.
(First time I'm trying to use NMRangeSlider and then that happened... classic.)


Error in excel file while downloading power specttrum data

Here is the excel file. As you can see on several sheets it says "recovered"
From: Michael Röttger @.>
Sent: Wednesday, May 26, 2021 4:56 AM
To: ContactEngineering/TopoBank @.>
Cc: Pradhan, Arushi @.>; Author @.>
Subject: Re: [ContactEngineering/TopoBank] Error in excel file while downloading power specttrum data (#669)
Thanks for reporting this issue. Which Excel version do you use?
It would be good to know for which data the problem happened. Would it be possible that you share the downloaded xlsx file with us? If you don't want to do it here in public, you could send it to<EMAIL_ADDRESS>

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FContactEngineering%2FTopoBank%2Fissues%2F669%23issuecomment-848595493&data=04|01|arp146%40pitt.edu|e5ecf2a3e9844bace72408d9202418bb|9ef9f489e0a04eeb87cc3a526112fd0d|1|0|637576161676496226|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=hA2FgFgRbUFx53k5Jlc%2FZQibhDsFrbDIJTHwz5D5ZRw%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASW2CXGVWHCYT3ODYRJKMSDTPSZSJANCNFSM45P6ZXNQ&data=04|01|arp146%40pitt.edu|e5ecf2a3e9844bace72408d9202418bb|9ef9f489e0a04eeb87cc3a526112fd0d|1|0|637576161676506219|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=9iYTPRpIsPff3igRsQ%2B82BQ0LH0GWykBTFt8Rt6RyQE%3D&reserved=0.

Hi @ArushiPradhan , unfortunately I didn't get the file.
You could either attach it here on Github to a message or send an e-mail to the given address (not back to GitHub).
It would be also interesting, which Excel version you are using.

Maybe related:
https://stackoverflow.com/questions/61161435/openpyxl-we-found-a-problem-with-some-content
https://foss.heptapod.net/openpyxl/openpyxl/-/issues/1430
Probably this is the reason: Sheet names in Excel files are only allowed to be 31 characters long.
The recovered file has another name for the sheet "Chip4_s1_optical - 1D PSD along y", 33 characters.
Shortened to 31 characters, this is "Chip4_s1_optical - 1D PSD along", the same as for "x" direction.
The sheet name is not unique any more.
So we need a more sophisticated way to shorten sheet names, although this is only a workaround.

It seems to be tricky to get this right. I suggest another approach.
The first sheet could be an "index" sheet with all the other sheets with descriptions and links to each of sheets.
The names of the sheets could then be constructed using a number which simply increments. There could also be a back link
to the index sheet.
Would this be reasonable from a user's perspective?

This will be fixed in the next version 0.15.0 as soon it is online.
There is now an extra sheet "INDEX" which allows to navigate to the individual data sheets.


Mark as paid now adds funds if the Payment Provider is "Custom".

When you press the "Mark as paid" Button on the invoice, you can now enter a transaction id.
If the Payment gateway of the Invoice is "Custom" and it's enabled, the transaction ID will be used to add funds in the amount of the invoice to the customer.
Then, it will be marked as paid, just like with any other payment provider.
However, it won't mark the invoice as paid anymore if the payment provider isn't "Custom".
This button has not been touched since 9 Years, so I doubt this will have a high impact, but if you think the behaviour should be different then what it is, lmk.

As I can see there is now way to add partial payment for an invoice and the  Mark as paid button add payment to the customer addFunds. The correct way should be. There has to be Add Payment Option with the following fields:
Date:
Transactin ID
Amount
After adding this  the transaction should get add transaction table and should be visible on invoice page.
Please suggest if that is correct way so that I can start coding on that part

Had to rewrite / add quite a few things for this. Now the transaction gets handled like the other payment adapters.

2. I feel like this functionality would be useful for all payment gateways, right? It's entirely possible that a transaction will go through and then not appear correctly inside of FOSSBilling which would be a valid use-case to need to mark an invoice as paid & set a transaction ID for it. The only potential gotcha is that it's also possible that transaction may have appeared in FOSSBilling and was just never considered complete. I'd love to hear your thoughts on this

@BelleNottelling I was with you all the way up to the potential gotcha part and then you lost me a bit. Why would that be an issue, if the transaction has appeared in FOSSBilling but not been considered complete then the logic should be that adding a manual transaction ID should override it for that invoice surely? Or am I missing something - it's been a long day.

The Mark as Paid functionality already worked for all other gateways. It was designed for exactly the use case @BelleNottelling described (Except you can't set a transaction ID with it, but you don't need to because the transaction was done by the paygateway)

(Except you can't set a transaction ID with it, but you don't need to because the transaction was done by the paygateway)

This is assuming that the transaction does correctly appear inside of FOSSBilling, but with a regular payment gateway (so not the custom one) you'll never be using the mark as paid button unless something didn't behave correctly, making it rather likely that the transaction itself might not be inside of FOSSBilling.
That's why I think it would be nice to have the option (not be forced to) to set the transaction ID regardless of the payment gateway (as if I am understanding it correctly, the transaction ID that's entered is only used if it's the custom payment gateway).

This is assuming that the transaction does correctly appear inside of FOSSBilling, but with a regular payment gateway (so not the custom one)

What's happening is this:
Pressing the mark as paid button will just call markAsPaid in the service and every invoice item will be marked as paid.
The Function markAsPaid of the InvoiceItem  will check if the item was already charged to the customer and if not, it will add a new Charge to the balance of the client

you'll never be using the mark as paid button unless something didn't behave correctly, making it rather likely that the transaction itself might not be inside of FOSSBilling.

The button (as it exists now) assumes that the payment was added correctly to the clients balance, and just the "activation" and or charge didn't work. This is current behaviour.
If this button is pressed with the "Custom" Payment Gateway, markAsPaid tries to detract funds from the client balance, that are not there, leading to an error.
My code adds a check to first add transactions and balance to the client before marking the funds as paid, so this error doesn't happen.
What you are talking about is more complex as it requires analysis of the existing transactions to see what failed and fixing that. While I can see that being handy in case a payment gateway fails before the funds get added to the client, this isn't the PR that's going to fix it. I think adding the refund functionality will require logic that will enable what you are thinking about, and I'll defer that until that time.

What's happening is this...

This all makes sense to me, and I think this can be merged as is now. But, yes, it will need to be revisited in the future to address the kind of issue that Belle is talking about and refunds.


Add switch to disable TCP_NODELAY

This in an attempt quell CPU usage, since PCCW is having issues when their {N} number of routers all start RTR sessions at the same time.
A perf of the system suggests all of the CPU is going to sending TCP traffic, and since golang enables NO_DELAY (infamously?) by default, this is a good smoking gun, since this would imply that stayrtr is sending 1 TCP packet per RTR PDU, something that would indeed cause a lot of CPU usage in aggergate!
This patch was written some time ago, but reports from PCCW states that this does actually help their network a noticeable amount, So I think this is a worthy switch to be introduced

Yup

It's actually a switch to enable NODELAY, but it is still morning so we'll let that slide ;-)


Update waf-front-door-configure-ip-restriction.md

remove extra line for x-forwarded-for

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

#sign-off

@teresayao: I'm sorry - only the author of this article, @vhorne, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Test with Travis CI

Feature
Use Travis CI to test Python 2.7+, 3.3+ and Django 1.9

Closing until a testing conversation is had.


"VMWare controller:" Suspension (Error) email has no Blanks after "Full Stop" (.) characters.

[Version]
V16.0 Fix 6
[Date]
2016/06/26
[Test category]
Creative
[Description]
The email which is sent when a subscription is suspended because of error does not have white spaces after the full stop character (.)
[Steps to reproduce]
Create instances until the system does not have any memory left and error occurs when creating a new instance.
Check the suspension email.
[Expected result]
Blank character after full stop
[Observed result]
Please se the content of the email below:
Dear user,
The current operation on subscription 'VMware_Original_With_ALL' for customer 'd5b8f512' has been suspended (Instance: '1467876109659').
Reported problem: Insufficient disk space on datastore ''.Failed to extend swap file from 0 KB to 33554432 KB.Could not power on VM : @&!@@(msg.vmk.status.VMK_NO_SPACE)No space left on device. Failed to power on VM.An error was received from the ESX host while powering on VM Freddy-000053.
To continue automatic processing, please click the following URL:
[Details]
See above;
[Messages and logs]

I suspect that this can be changed. Error messages returned from the hypervisor or other 3rd party service, will be included in details sections of APP mail notifications as they are.
I suggest to first verify whether the message is composed in OSCM. Then check the mail template where it's applied. If the cause is not on the VMWare controller or APP I consider this issue not valid.

The message in reported problem comes from the vSphere system. The issue is invalid for OSCM.


Check if the checkpoint seq is in the WAL boundary before using

This closes #1319

This boundary check looks necessary but I can't reproduce this issue on my side.

@torwig @caipengbo I extended the sequence limitation to require the checkpoint sequence must be greater or equal to the WAL sequence instead of strictly matching.
  if (seq < wal_seq) {
    return {Status::NotOK, fmt::format("checkpoint seq: {} is smaller than the WAL seq: {}", seq, wal_seq)};
  }

Thanks all, merging...


Terrain disappears zooming and tilting - Cesium 1.16

Dear all,
I would like to highlight a bug with the new versione 1.16 of Cesium.
The terrain disappears at a certain level of zoom and tilting the camera near the terrain generates the same bug.

It seems that the issue is raised up only in certain areas, for example in the north east Italy.
To reproduce the bug search the city of "Trento" in the Cesium demo on the front page of the website.
The related post on the forum: http://cesiumjs.org/forum.html#!msg/cesium-dev/2_2zBxN7Dno/qT6Vzep-CQAJ
Best Regards,
Michele

Thanks @slightGT This is definitely a bug and I could easily reproduce it after zooming to Trento.
@bagnell this is definitely a recent regression with all of the terrain work you've been doing.  We should probably look into it ASAP.

+1

@slightGT thanks for the detailed report.  This was fixed in #3365 and will be released in Cesium 1.17 on January 4.


"zenith pageserver stop" is unimplemented

$ target/debug/zenith pageserver stop
thread 'main' panicked at 'not yet implemented', zenith/src/main.rs:143:13

In the python scripts, I worked around this by calling "killall pageserver" but looks like killall isn't installed everywhere by default, so the python test framework is failing its promise to clean up after itself.

There are a few places where we do Command::new("kill") but it seems odd to spawn a process just to run a single syscall. There isn't a kill(pid) in the std library, so I guess our options are:

nix::sys::signal::kill
libc::kill (requires unsafe)

I'm inclined to do the former. nix has no dependencies of its own, and it's almost universally the way people access unix-y libc stuff that's not in the standard library.


Do not prompt if variables are already defined

Set these environement variables before executing the script to avoid
interactive prompt.
Signed-off-by: Francis Giraldeau<EMAIL_ADDRESS>

Hi! I'm not a bash guru, let me know what you think. I tried these modifications in a real world migration with gitlab 13.9. Cheers!

@giraldeau thanks for the contribution, lgtm. I'll run a short test early next week to double check no interferences.


fix latex representation for exp format and multi-digit exponents of s or z

quickfix for #278

Coverage increased (+0.03%) to 83.528% when pulling f7bb22a89425bf96a2525684ca9501b5a2d17b38 on bnavigator:fix-latex into c65263f1d5c900761c77c023207d7bc3fd97518b on python-control:master.

I think normal scientific notation uses \times instead of \cdot.  Is there a reason not to use \times?
Also, scientific notation in Python uses a leading 0 and so 100 becomes 1e02 which becomes 10^{02}, which looks a bit odd.  Can you strip the leading 0?
Also, not a huge deal, but might be nice to have some unit tests to verify this works correctly.

Both seems to exist. See for example the documentation for the LaTeX siunitx package, page 28. I am used to \cdot, but maybe that's just more common here in Germany. How about having it configurable in the .config module and set \times as default?

Coveralls is a mystery to me.


The Sight Suggestions

@newbyr

The Sight Optional - I can think of two ways but I'll leave it up to you if you feel it's worth the effort  - 1. system configuration to tick The Sight off /on for random selection of favours 2. tickbox on the character sheet to do the same
O Fickle Fortune - Create a rollable table or macro for the outcome of a vision, dream, omen or reading i.e. the two tables on pp10-11

e.g. roll d4 and d10 on Table 1 Outcomes and getting 2 and 7 = Honesty/Deception, then take a chance 2 = Deception, then roll d4 and d6 on Table 2 Subject and getting 1 and 2 = The Divinant therefore "The outcome is deception for the divinant"

OK.. @jwmuk and @newbyr

I've added a Setting to enable The Sight (the rules say you should agree as a table, so it needs to be all or nothing)
I've added a Macro called O Fickle Fortune and associated RollTables to support it. I haven't built a fancy combo, I'm afraid - it just rolls separately on each table in sequence...but it's there, at least.
I've updated the documentation to reflect all this.

If you update to 1.3.2 you'll get these. I would recommend deleting your Outcomes folder and your Macros folder, and re-initializing the system to import in the new macros and rolltables.

Many thanks Chris   From: Chris SharpSent: 30 May 2022 17:57To: chrisesharp/foundryvtt-dee-sanctionCc: newbyr; MentionSubject: Re: [chrisesharp/foundryvtt-dee-sanction] The Sight Suggestions (Issue #17) OK.. @jwmuk and @newbyrI've added a Setting to enable The Sight (the rules say you should agree as a table, so it needs to be all or nothing)I've added a Macro called O Fickle Fortune and associated RollTables to support it. I haven't built a fancy combo, I'm afraid - it just rolls separately on each table in sequence...but it's there, at least.I've updated the documentation to reflect all this.If you update to 1.3.2 you'll get these. I would recommend deleting your Outcomes folder and your Macros folder, and re-initializing the system to import in the new macros and rolltables.—Reply to this email directly, view it on GitHub, or unsubscribeYou are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>			This email has been checked for viruses by Avast antivirus software.
			www.avast.com


base absdiff() off of new abs() & remove modulus in median()

Hi, the other guy (@jeff-ayars) got to abs() before me :-)  There was a comment about updating absdiff() to reflect the new definition of abs() - that is what I have done here.
(g'day!)
PS: I wasn't thinking when I made a pull request off of my master, now this includes a cleanup to p_median() also.

Your p_median contains an error that the original does - after line 25, it should use the sort variable rather than a (otherwise it is ignoring the sort).
Additionally, you could remove the temp variable and just use c throughout the function - I'm guessing the compiler would probably do this when optimising anyway.

@6thimage:

I believe I have corrected the original issue now, and changed the stack allocation of the sorted vector to be on the heap - as I am not sure how long the vector might be (my reasoning).  I modeled it after the allocation in mode() as an example for this.

I did declare tmp thinking it would give the compiler more freedom to do its thing.  It probably produces the same code though...

Do you approve?  I can still remove tmp if it goes against a style guideline or something ~

I am not sure the test likes a heap-allocated vector - I modelled this after p_mode() but that function doesn't have a test.  I"m either doing something wrong or...

b5df390 looks like it has sorted everything - a compiler will more than likely optimise out the tmp variable, to me it just looks a little cleaner without the tmp variable. As to giving the compiler more freedom, the only thing I can see is that a dumb compiler might use an additional register, impacting performance slightly.
The test is failing because p_malloc is currently not implemented - have a look at src/base/p_malloc.c.

Now I think the test is failing because it hates me.  Or the test was created against the old behavior... which was incorrect.  Not sure.


Not able to create Job

Hi,
Unable to create a job using node-pg-jobs library, also the table job_snapshots is not getting generated in PGAdmin.
Error:  {"eventEmitter":{"domain":null,"_events":{},"_eventsCount":0}}
You can mail me at<EMAIL_ADDRESS>or<EMAIL_ADDRESS>Thanks
Gurunath Pai

Hi @paigurunath,
I wonder whether you need to run migrations?
Cheers
Tim


[TypeScript] Default arguments in queries/mutations are generated as Maybe

Describe the bug
If you have default arguments on SDL side.
limit: PositiveInt = 100
offset: NonNegativeInt = 0

It will generate:
limit?: Maybe<Scalar['PositiveInt']>
offset?: Maybe<Scalar['NonNegativeInt']>

Expected behavior
Should be generated:
limit: Scalar['PositiveInt']
offset: Scalar['NonNegativeInt']

Environment:
codegen: 1.3.1
node: v12.3.1

The similar issue back in the days for Flow.
https://github.com/dotansimha/graphql-code-generator/issues/1097

It depends, because for typescript-operations, that's fine, because it means that you don't have to specify this field, because it has a default value.
It might be an issue only with typescript-resolvers, because there if there is a default value, it means that that the field must be there.
@kamilkisiela what do you think? maybe we can keep it as-is now (with Maybe), and do something like UnMaybe<MyQueryArgs, 'fieldA'> in the usage of the resolvers plugins?

Fixed in: https://github.com/dotansimha/graphql-code-generator/pull/2122
@RIP21 can you please try it with 1.3.1-alpha-5d4a4dba.80?

@dotansimha I'm facing the same issue with the latest alpha.
This schema:
"Query for search results"
  search(
    "The search text"
    input: SearchInput!

    "The page size"
    first: Int = 20

    "Page offset id"
    after: ID = "0"

    "Page number, if not using the 'after' parameter"
    page: Int
  ): SearchResultsConnection!

Is turned into:
/** Root query field */
export type QuerySearchArgs = {
  input: SearchInput
  first?: Maybe<Scalars['Int']>
  after?: Maybe<Scalars['ID']>
  page?: Maybe<Scalars['Int']>
}

Instead of:
/** Root query field */
export type QuerySearchArgs = {
  input: SearchInput
  first: Scalars['Int']
  after: Scalars['ID']
  page?: Maybe<Scalars['Int']>
}

Thanks for your work 👍.

@yamafaktory The Args type should look the same as before, but we are removing the ? and Maybe modifiers when it's used in the resolvers signature.
Try to use the QueryResolvers and implement a resolver, and these fields will appear as non-optional.

@dotansimha thanks a lot for the feedback! Somehow I would have expected QuerySearchArgs not to have any Maybe in my case.

Fixed in 1.4.0 🎉


Integrate cancer biomarkers from new api into widget

Just realized I pushed a bunch of commits through to the wrong ticket/issue.
As per discussion with Andrew, for initial MVP we're going for the basic table only.
This is now done pending PR review.

Ticket closed as work merged into alpha


Add Karpenter to Autoscaling section

Issue #, if available:
Description of changes:
Added a Section about Karpenter in the Autoscaling section

Configure Karpenter
Cleanup Karpenter

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

Hi, For Karpenter I'd suggest creating a new module in the Beginner section and using the content that we created with the Karpenter team for re:Invent. The content is available here : https://github.com/awslabs/ec2-spot-workshops/tree/CMP307 . Full workshop available here : https://cmp307.ec2spotworkshops.com/

@ruecarlo I have aligned the content to the ec2spot workshop. currently the cluster has k8s v1.19 and I needed to adapt the karpenter installation to v0.4.3 . I have left out the more advanced content for now. It might make sense to add it once the eksworkshop uses k8s v1.20 . Please, let me know what you think.


<EMAIL_ADDRESS>is not supported due to requires device with capability > (8, 0) but your GPU has capability (6, 1) (too old)

Python Version
Python 3.10.12 (main, Mar 22 2024, 16:50:05) [GCC 11.4.0]

Pip Freeze
absl-py==2.1.0
annotated-types==0.7.0
attrs==23.2.0
docstring_parser==0.16
filelock==3.15.4
fire==0.6.0
fsspec==2024.6.1
grpcio==1.64.1
Jinja2==3.1.4
jsonschema==4.21.1
jsonschema-specifications==2023.12.1
Markdown==3.6
MarkupSafe==2.1.5
mistral_common==1.2.1
mpmath==1.3.0
networkx==3.3
numpy==1.25.2
nvidia-cublas-cu12==<IP_ADDRESS>
nvidia-cuda-cupti-cu12==12.1.105
nvidia-cuda-nvrtc-cu12==12.1.105
nvidia-cuda-runtime-cu12==12.1.105
nvidia-cudnn-cu12==<IP_ADDRESS>
nvidia-cufft-cu12==<IP_ADDRESS>
nvidia-curand-cu12==<IP_ADDRESS>
nvidia-cusolver-cu12==<IP_ADDRESS>
nvidia-cusparse-cu12==<IP_ADDRESS>
nvidia-nccl-cu12==2.19.3
nvidia-nvjitlink-cu12==12.5.82
nvidia-nvtx-cu12==12.1.105
protobuf==4.25.3
pydantic==2.6.1
pydantic_core==2.16.2
PyYAML==6.0.1
referencing==0.35.1
rpds-py==0.19.0
safetensors==0.4.3
sentencepiece==0.1.99
simple_parsing==0.1.5
six==1.16.0
sympy==1.13.0
tensorboard==2.17.0
tensorboard-data-server==0.7.2
termcolor==2.4.0
torch==2.2.0
tqdm==4.66.4
triton==2.2.0
typing_extensions==4.12.2
Werkzeug==3.0.3
xformers==0.0.24

Reproduction Steps

torchrun --nproc-per-node 1 -m train example/7B.yaml

And Get Error Like
NotImplementedError: No operator found for memory_efficient_attention_forward with inputs:
query       : shape=(1, 8192, 32, 128) (torch.bfloat16)
key         : shape=(1, 8192, 32, 128) (torch.bfloat16)
value       : shape=(1, 8192, 32, 128) (torch.bfloat16)
attn_bias   : <class 'xformers.ops.fmha.attn_bias.BlockDiagonalCausalMask'>
p           : 0.0
<EMAIL_ADDRESS>is not supported because:
requires device with capability > (8, 0) but your GPU has capability (6, 1) (too old)
bf16 is only supported on A100+ GPUs
tritonflashattF is not supported because:
requires device with capability > (8, 0) but your GPU has capability (6, 1) (too old)
attn_bias type is <class 'xformers.ops.fmha.attn_bias.BlockDiagonalCausalMask'>
bf16 is only supported on A100+ GPUs
operator wasn't built - see python -m xformers.info for more info
triton is not available
requires GPU with sm80 minimum compute capacity, e.g., A100/H100/L4
cutlassF is not supported because:
bf16 is only supported on A100+ GPUs
smallkF is not supported because:
max(query.shape[-1] != value.shape[-1]) > 32
dtype=torch.bfloat16 (supported: {torch.float32})
attn_bias type is <class 'xformers.ops.fmha.attn_bias.BlockDiagonalCausalMask'>
bf16 is only supported on A100+ GPUs
unsupported embed per head: 128
[2024-07-10 15:30:11,478] torch.distributed.elastic.multiprocessing.api: [ERROR] failed (exitcode: 1) local_rank: 0 (pid: 206239) of binary: /opt/mistral-finetune-main/my_venv/bin/python3.10
Traceback (most recent call last):
File "/opt/mistral-finetune-main/my_venv/bin/torchrun", line 8, in
sys.exit(main())
have any other option for Cuda - device_capability version 6.1 vision To mistral-finetune ?
Expected Behavior
have any other option for Cuda - device_capability version 6.1 vision To mistral-finetune ?
Additional Context
python -m xformers.info
xFormers 0.0.24
memory_efficient_attention.cutlassF:               available
memory_efficient_attention.cutlassB:               available
memory_efficient_attention.decoderF:               available
<EMAIL_ADDRESS>       available
<EMAIL_ADDRESS>       available
memory_efficient_attention.smallkF:                available
memory_efficient_attention.smallkB:                available
memory_efficient_attention.tritonflashattF:        unavailable
memory_efficient_attention.tritonflashattB:        unavailable
memory_efficient_attention.triton_splitKF:         unavailable
indexing.scaled_index_addF:                        unavailable
indexing.scaled_index_addB:                        unavailable
indexing.index_select:                             unavailable
sequence_parallel_fused.write_values:              unavailable
sequence_parallel_fused.wait_values:               unavailable
sequence_parallel_fused.cuda_memset_32b_async:     unavailable
sp24.sparse24_sparsify_both_ways:                  available
sp24.sparse24_apply:                               available
sp24.sparse24_apply_dense_output:                  available
sp24._sparse24_gemm:                               available
<EMAIL_ADDRESS>                       available
swiglu.dual_gemm_silu:                             available
swiglu.gemm_fused_operand_sum:                     available
swiglu.fused.p.cpp:                                available
is_triton_available:                               False
pytorch.version:                                   2.2.0+cu121
pytorch.cuda:                                      available
gpu.compute_capability:                            6.1
gpu.name:                                          NVIDIA GeForce GTX 1080
dcgm_profiler:                                     unavailable
build.info:                                        available
build.cuda_version:                                1201
build.python_version:                              3.10.13
build.torch_version:                               2.2.0+cu121
build.env.TORCH_CUDA_ARCH_LIST:                    5.0+PTX 6.0 6.1 7.0 7.5 8.0+PTX 9.0
build.env.XFORMERS_BUILD_TYPE:                     Release
build.env.XFORMERS_ENABLE_DEBUG_ASSERTIONS:        None
build.env.NVCC_FLAGS:                              None
build.env.XFORMERS_PACKAGE_FROM:                   wheel-v0.0.24
build.nvcc_version:                                12.1.66
source.privacy:                                    open source
Suggested Solutions
No response

Similar issue using colab's T4


tests/kernel: fifo_timeout: Remove wake-up order checking

There is no guarantee of wake-up order when multiple threads
are woken up on the same tick. Hence, modified the tests
accordingly.
Fixes #8159.
Signed-off-by: Rajavardhan Gundi<EMAIL_ADDRESS>

I can't see a better method as the timeouts get affected by system loading. If the interrupts get locked for whatever reason, the timeouts may go out of bounds (later than _ms_to_ticks()+1). So, all we can do is to ensure the timeouts do not happen earlier than _ms_to_ticks().

@pizi-nordic .. can you please comment?

I agree that there is no upper limit on loaded system.
However during the test we are controlling system state. So we can assume some upper bound.
What about the following checks.

Make sure that thread does not waited shorter than expected,
Make sure that thread was woken up not later than expected_time + X (X = 1-2 ticks should be sufficient).

@pizi-nordic Agree. Have updated the test. Please check.

Codecov Report

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

@@           Coverage Diff           @@
##           master   #10047   +/-   ##
=======================================
  Coverage   52.58%   52.58%
=======================================
  Files         213      213
  Lines       26148    26148
  Branches     5636     5636
=======================================
  Hits        13749    13749
  Misses      10145    10145
  Partials     2254     2254

Continue to review full report at Codecov.

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