Spirit summoning rituals and the like

We discussed this a bit on IRC a while back, do you think that after you finish stomping all of the bugs with 1.8, we could discuss this in more detail, and figure out what to add, and what not to do? By then, I should be a bit more free after finishing the milk module draft for Growthcraft.

Yup, we could do that

Almost free, and I have been accumulating a few possible spirits from various native american groups that could be used. However, all of my ideas so far would count as bosses.
Baykok
Amarok
Wendigo

So I guess we should find out what benefits you could get from them. I don't really have an idea how they could be integrated.

Baykok is a grim-reaper-esque figure that hunts warriors, using invisible arrows or a club to attack his victims. Baykok could be some kind of boss that drops something useful. Maybe some ingredient that allows for invisible arrows, or some fairly powerful weapon?
 However, there are tales of him granting boons of some kind. This wikipedia page has some examples: https://en.wikipedia.org/wiki/Amarok_(wolf)
Wendigo is a human that has transformed into a monster as a result of cannibalism. Maybe it could be implemented as some form of curse, since if I recall some of pokefenn's old plans, there was a plan for a "dark side", aka the withered totemists. Not sure if he is still doing that though.

Also, there are at least several fairy-like spirits from various native american mythologies.
Here is an example:
https://en.wikipedia.org/wiki/Pukwudgie

You know what, do you think it'd be be better to formulate these on Skype or something?

Yeah, or on IRC

Alright, there are some things I need to attend to first, then I'll come up with a rough draft. Also, can more than two instruments be used in a ceremony?

During startup? Yes. For selecting the ceremony? Currently not.

Hm, so new instruments then. Even if they are as uncreative as more infused variants of each instrument.

Alright, thought of one instrument already, even though its just a variant of the drum.
Water Drum: A drum created by filling the drum chamber with water to produce a unique resonant sound. Made using 5 clay, 1 water bucket, and 3 buffalo hide or leather.
https://en.wikipedia.org/wiki/Water_drum

I think I'm going to work with ReForged a bit, as well as try to figure out more solutions to get around the 2 instrument per ceremony limitation.
What about some kind of "upgraded" (not sure how this would be done) totem base that allows for more than 2 instruments to be used in a ceremony? We are likely still going to have to create new instruments though anyways.

I'm not sure. Will we need that many ceremonies right now?
The numbers can always be increased later when it is needed I guess.

Hm, I do not have an idea set up for how many spirits, and such, ceremonies. I'm trying to figure that out but finding stuff suitable is painful. However, what kinds of things are you looking for? Maybe if I know what types of spirits you want to be able to summon, I can find them, and then discuss them on IRC

Some of the ones you mentioned above sound nice, like Baykok

I shall see when I can think of ceremonies. Does the order in which an instrument is played matter? Because if it does not, it could make making new ceremonies easier and harder.

Should we close this until you are motivated to mod again?


Upgrade Vuetify to latest version

Upon upgrading, some issues might happen in data tables and other components.

Make sure to use new features of data-table instead of workarounds in list-table.vue file

WIP:
Upgraded vuetify from 3.1.x to 3.4.x

WIP:
Upgraded table in almost every part of app

What is the status here?


More rust docs

Switch from allow(missing_docs) to deny(missing_docs), which should
gradually help us moving forward.  Also, add a few missing docs, such as
for sched and aio.

bors r+


Little changes

Just optimized the code and fixed two mistakes.

Wow! Thanks for the good work man!


XCBConnection checkScreens

Add to XCBConnection a method checkScreens that iterates the connection setup and set all the screens.

Add to XCBConnection a method checkScreens that iterates the connection setup and set all the screens.


fix: modify marked words and project exist check api

Signed-off-by: xu.jingwen<EMAIL_ADDRESS>
What type of PR is this?
/kind bug

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

Codecov Report

Merging #485 (f50862d) into master (3c45f3b) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #485   +/-   ##
=======================================
  Coverage   11.81%   11.81%
=======================================
  Files         116      116
  Lines       18668    18668
=======================================
  Hits         2205     2205
  Misses      16207    16207
  Partials      256      256

Impacted Files
Coverage Δ

pkg/simple/client/kc/api.go
0.00% <0.00%> (ø)

/cc @x893675

/lgtm
/approve


KAFKA-15803: Update leader epoch in commitAsync and committed

To align the new consumer completely with the legacy consumer,
we need to update the latest seen leader epoch in the metadata
both when we commit offsets with a leader epoch using
commitAsync and when we fetch the latest committed offsets
using committed.
We add unit tests to AsyncKafkaConsumer to test that
metadata is correctly updated. We also add a corresponding test
for commitSync, which already had the leader epoch logic.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Test failures unrelated


HDP3 HA - check for number of RESOURCEMANAGER

Im provisioning a cluster with HA and I accidentally put RESOURCEMANAGER on both namenodes. The install in Ambari froze and ambari-server.log gave me this error:
java.lang.IllegalArgumentException: Unable to update configuration property 'yarn.resourcemanager.address' with topology information. Component 'RESOURCEMANAGER' is mapped to an invalid number of host groups '2'.
I fixed it by moving it to only one node in the cluster, no problem there.
What Im wondering is if there is a check in ansible, like for some services, or if it would be smart to implement one?
Thank you :)

Hi @markokole
Sorry about my late reply here.
Adding RESOURCEMANAGER to two host_groups together with NAMENODE should work just fine.
Adding RESOURCEMANAGER and NAMENODE to a host_group with 2 nodes would not work.
You got that error because I think this blueprint code didn't execute for some reason: https://github.com/hortonworks/ansible-hortonworks/blob/master/playbooks/roles/ambari-blueprint/templates/blueprint_dynamic.j2#L542
Checks can be added in this playbook: https://github.com/hortonworks/ansible-hortonworks/blob/master/playbooks/check_dynamic_blueprint.yml
But not sure what check needs to be added here...

Closing for inactivity.


XAutoClaim doesn't work

Expected Behavior
XAutoClaim is work :)
Current Behavior
Returned with error  - got 3, wanted 2
To be sure - you can run your tests

Hi @bundleman, Are you using Redis 7? Maybe you could try https://github.com/rueian/rueidis


[stable/luigi] Initial Commit

Adding a spotify luigi scheduler helm chart
Luigi helps you build complex pipelines of batch jobs. It handles dependency resolution, workflow management, visualization, handling failures, command line integration.
https://luigi.readthedocs.io/en/stable/index.html

Thank you for contributing to kubernetes/charts. Before you submit this PR we'd like to
make sure you are aware of our technical requirements and best practices:

https://github.com/kubernetes/charts/blob/master/CONTRIBUTING.md#technical-requirements
https://github.com/kubernetes/helm/tree/master/docs/chart_best_practices

For a quick overview across what we will look at reviewing your PR, please read
our review guidelines:

https://github.com/kubernetes/charts/blob/master/REVIEW_GUIDELINES.md

Following our best practices right from the start will accelerate the review process and
help get your PR merged quicker.
When updates to your PR are requested, please add new commits and do not squash the
history. This will make it easier to identify new changes. The PR will be squashed
anyways when it is merged. Thanks.

@mattfarina I've pushed the changes you asked for. Also, good call on the image. I't not interested in maintaining version and upgrades in my personal repo. Ill reach out to the Axiom data science to see if they are interested in it.
https://github.com/axiom-data-science/docker-luigi

@unguiculus updated to match your comments. Let me know if there's anything else.

/ok-to-test

/lgtm


Convert html + css to image

Is there a possibility of converting HTML + CSS content to image in this library ?

You mean like a screenshot ?
I'm affraid this is not possible with this library

Yes like a screenshot.
Thanks for your answer.

Is there a possibility of converting HTML + CSS content to image in this library ?

why not use domtoimage ?


After Rails upgrade 6

fixing uninitialized constant TariffSynchronizer::TaricUpdate::ChiefImporter

 [TARIFF17] Investigate differences in data between dev/staging and production


Various bits of refactoring in preparation to the tasks implementation

This definitely shouldn't be merged just yet (needs a lot of testing and tweaks) but since it is now looking pretty big AND sine the overall shape of it won't change much -- I'm sending it out for review.
I tried to structure it into semi-independent commits as much as possible so you may want to review commit-by-commit.
In general, here are the changes that are being done here:

got rid of a lot of methods in hypervisor interface - those are not really needed and they get in the way of tasks
introduced CtrExec and CtrSystemExec -- these methods can exec a process in an already running task or a systems service
moving Xen hypervisor implementation to use CtrExec and CtrSystemExec  (and getting rid of all the fake Xen utilities in a pillar container: xl, xenstore, etc.)

But given the set of races and timing issues we had around checking for the qemu process I think we need to run some continious tests on this (at least with Xen) for a few days.

Absolutely agreed! In fact, I think there may be on chunk of works still missing in here. So... I'd like to keep this out for other to review in details but I'm also proposing that I'll work on the 2nd half of the tasks refactoring assuming this PR gets merged. The only danger here could be extra rebase work -- but I think the risk is minimal in this particular case since this work is limited to the places that I don't anticipate anyone touching this coming week.
Makes sense @eriknordmark ?


Fix multipart upload run

Run multipart upload in the finish event handler of write stream to avoid partial file uploading.
Old code run uploading before stream end method run. This generates the UPLOAD_ERR_PARTIAL error on PHP server from time to time.

thank you


Extension walkthroughs do not launch upon install

Does this issue occur when all extensions are disabled?: Yes/No, this is an extension-related issue that arises across extensions with walkthroughs

VS Code Version: 1.91.1
OS Version: Windows 11

Extension walkthroughs are not launching upon install. Expected behavior: walkthrough launches automatically upon install.
Tested on C# Dev Kit extension and Python extension. Seemed to work in 1.90.0 but does not work in 1.91.1.
Steps to Reproduce [Windows Sandbox]:

Launch a Windows Sandbox

Open Edge and download VS Code (stable) from the VS Code website

Install the C# Dev Kit extension.
Note: Upon install the walkthrough is supposed to launch automatically. This is not happening, but I can manually open the walkthrough using these steps.

Open command palette.

Use command 'Welcome: Open Walkthrough'

Select 'Get Started with C# Dev Kit'

Repeated steps 3-7 with the Python extension. Same behavior.

Steps to Reproduce [VS Code stable and VS Code insider]:

Open VS Code
Uninstall C# Dev Kit (if you have it installed)
Reload VS Code
Install C# Dev Kit extension.
Note: Upon install the walkthrough is supposed to launch automatically. This is not happening, but I can manually open the walkthrough using these steps.
Open command palette.
Use command 'Welcome: Open Walkthrough'
Select 'Get Started with C# Dev Kit'
Repeated steps 2-7 with the Python extension. Same behavior.

\closedWith 0c5a528

@bhavyaus using the steps above, I am not seeing the walkthrough open when I reinstall the extension

Was just talking with @bhavyaus about this- if you've had it installed in the past, it won't work, but for me it works in a window with a clean user-data-dir


Add previous transformation of Thomson data

Adding previous import of Thomson dataset

@aarmey After using the old transformation of the Thomson dataset, the clustering and assignments work well. How should we proceed for the import function? The previous pre-processing definitely cluster cells much better so I feel like it makes more sense to use that even though they are not built in scanpy functions.

Thanks for fixing my mess! Let's bring back the old import. Please tag me on the PR when you have it copied back.

Thanks for fixing my mess! Let's bring back the old import. Please tag me on the PR when you have it copied back.

Should we repeat the process for the CITEseq dataset? Or have 2 versions of transforming raw counts?

Use the same process and function for both.


An extra scroll bar displays in the 'Shared Access Signature' dialog for one ADLS Gen2 blob container when timestamp validation error occurs

Storage Explorer Version: 1.23.0-dev
Build Number: 20220124.2
Branch: main
Platform/OS: Windows 10
Architecture: ia32\x64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Expand one ADLS Gen2 storage account -> Blob Containers.
Create a blob container -> Right click the blob container -> Click 'Get Shared Access Signature...'.
Change the start time to make it is greater than the expire time.
There is an error message -> Check whether no extra scroll bar displays.

Expected Experience
No extra scroll bar displays.
Actual Experience
An extra scroll bar displays.

Additional Context

This issue doesn't reproduce for file shares/queues/tables.
This issue doesn't reproduce for normal blob containers.
This issue doesn't reproduce on Linux & Mac.
This issue doesn't reproduce in 1.21.3 because there is no error message in the dialog.

Verified this issue on build 20220511.2.


Show a hint that a session has unreviewed solutions

PyLadies participants often upload task solutions a long time after they were initially assigned. I'm finding myself going through each session over and over to see if there is any unreviewed solution (⬤).
This PR adds a simple notification (asterisk) to the course overview screen which immediately lets you know which sessions need review:

The backend implementation is a bit messy, because TaskSolution does not store its corresponding session ID, so I have to match the task solutions manually.

I like the idea very much. But, isn't the asterisk too small? I mean what about a different color for the button or underlined session name? Just ideas, the asterisk is much better than nothing.

What about this?

Looks good to me.

Great, this will be really helpful! Or, would it be possible to show some counter?

Thank you, Kuba @kobzol. I am merging this one.


Add LSP C# to Roslyn workspace

After working with the Roslyn team we decided to utilize the IDynamicFileInfoProvider APIs to populate LSP documents C# output in the Roslyn workspace.
Added a new UpdateLSPFileInfo API to the RazorDynamicFileInfoProvider API that works a little bit different than existing ones.

It takes a top-level Razor document Uri instead of a project/document file path pair. Reasoning for this is in an LSP landscape there is no concept of project so we need to rely on Roslyn to build the mappings between top-level file and project.
It uses the top-level document Uri to look up an associated Razor document. This is how we know which documents output to populate without knowing the project.
It sets a flag to not allows document output suppression. Typically in non-LSP scenarios we will purposefully set a documents generated output to being string.Empty because the "active" document in the editor will create a separate instance of the document that provides the C# information. Now we're using the dynamic file info provider for "active" documents in LSP scenarios.

Changed the virtual document factory to create our C# virtual documents under a .g.cs file scheme to correspond with the way our existing IDynamicFileInfoProvider APIs function.
Built a new LSPDocumentManagerChangeTrigger API so we can write APIs that properly react to LSPDocumentManager changes.
Utilized the new LSPDocumentManagerChangeTrigger API to write a C# virtual document publisher. On a virtual document change it will notify our dynamic file info provider bits of the change to ensure our C# content flows into the Roslyn workspace.
Found out that VS' LSP client acts different than VSCode's in typing random text for invoking completions. Basically we weren't getting C# completions when typing in an @code {...} block because none of the content corresponded to our trigger characters. In VSCode this flow results in VSCode triggering completion without trigger characters; in VS it does the same but WITH trigger characters. Given our logic to only act on trigger-character based completions for the appropriate language we were no-oping because our trigger character would be something mundane like p when typing prop at which point we'd no-op. Made change sin our completion handler code to understand this scenario.
Expanded the LSPDocumentManager.Change args to include the old and new virtual document snapshots to make it easier for consumers to understand which virtual documents changed.
Added tests for all added API

Fixes dotnet/aspnetcore-tooling#19889

@ryanbrandenburg / @TanayParikh merging this early to unblock @ajaybhargavb's work. Feel free to leave comments and I can address after-the-fact.


CCO-324: add support for workload identity

Driver controller deployment should load workload identity values to injector env to enable workload identity authentication.
The following keys need to be flagged as optional because their presence now depends on authentication method used:

azure_client_secret
azure_tenant_id
azure_federated_token_file

Depends on: https://github.com/openshift/cluster-cloud-controller-manager-operator/pull/245

/retest

/retest

/retest

e2e-azure-csi is failing due to missing permissions in SCA:
"Account with ID 2DUeKzzTD9ngfsQ6YgkzdJn1jA4 denied access to perform create on Certificate with HTTP call POST /api/accounts_mgmt/v1/certificates"

This is being worked on but currently blocks this PR: https://redhat-internal.slack.com/archives/C014N2VLTQE/p1683198922055559

e2e-azure is failing due to RBAC:
Failed to watch *v1.ClusterVersion: failed to list *v1.ClusterVersion: clusterversions.config.openshift.io is forbidden: User "system:serviceaccount:openshift-cluster-csi-drivers:azure-disk-csi-driver-operator" cannot list resource "clusterversions" in API group "config.openshift.io" at the cluster scope

This will be fixed by a PR in CSO which will have to merge prior to this one: https://github.com/openshift/cluster-storage-operator/pull/372/commits/9dd869849b892a5967dd17e8f439ef291e014935

/retest-required

/retest e2e-azure-csi

/retest e2e-azure-csi

/retest e2e-azure-csi-extended

/test e2e-azure-csi-extended

/lgtm
/approve


how to config bug animation

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

Hi @cuonglibertyy
This platform is not meant for assistance on personal code.
Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


feature(job): Adding tests for mac, both jvm and native

fixes #305

for Sonarcloud, duplication of code, at first sight, I do not see how to fix that in a readable way.


esp_bt_controller_deinit() function causes assert isssue while using BLE5.0 extended scanning (IDFGH-8459)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[x] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hello Team,
I am currently using ESP32C3 and esp-idf version v4.3.2 (also tried using v4.4.2). I am using the ble5.0 on ESP in controller only mode and VHCI with transportation medium as the socket communication.
I am sending a series of commands to set the extended scan parameters and start extended scanning with duplicate filter turned ON. I am successfully getting the adv reports and able to sent the peripheral list to the Host. However during the scanning process, if the user closes the socket, it is implemented such that the ESP immediately executes the de-initialization process (esp_bt_controller_disable() followed by esp_bt_controller_deinit())for the controller. The esp_bt_controller_disable() function gets executed successfully however the deinit function gives the following error
assert llm_scan.c 1489, param 00000000 00000000
Also, I tried sending manually the ext scan disable command frame to the controller before the deinit process and the error disappears.
Note: This error does not occur when using the legacy scan settings or when using the extended scan with duplicate filter turned OFF. It is encountered only during extended scan with duplicate filter turned ON.
The issue is that the de-initialization can be called at any instance and hence it would be really helpful if someone could point me towards an ideal process for de-initializing the controller (irrespective of the controller state) and the significance of the assert and the  param being displayed.  .
Thank you.

@AishwaryaK24 We have reproduced the problem, we will fix it as soon as possible and provide you with the fix code

@Weijian-Espressif
Noted.
Thank you so much for your prompt response. Waiting for the fix.

libbtdm_app.a.zip
I have fixed this problem, please test, by the way, it is recommended to stop all events of bluetooth when releasing bluetooth @AishwaryaK24

@Weijian-Espressif
Thank you so much for the updated library. Will test with it and let you know asap.
Also regarding stopping all the events of bluetooth, I use esp_bt_controller_disable() before calling the esp_bt_controller_deinit() function. Also, I tried to reset the controller by manually sending the HCI Reset frame to the controller. However,  it did not resolve my issue.
Could you please suggest a way to stop all the ongoing ble events and then initiate de-initialization?

libbtdm_app.a.zip
sorry, please retry again

@AishwaryaK24 I mean when you release bluetooth, if there is an event running, you should stop the event first. For example, if you are scanning, you should stop scanning, and then release bluetooth.

@Weijian-Espressif
Thank you for the updated library. I tested the updated library and it does not give compilation error.
However it returns error code 0x101 for esp_bt_controller_init(&cfg), where
cfg =  BT_CONTROLLER_INIT_CONFIG_DEFAULT();
Also, on the console it gives the below log
Config struct mismatch: magic=5a5aa5a5, ver=02112280
I am currently using esp-idf v4.3.2. Does the version of esp-idf used affect this?

Can you  provide me the IDF commit ?

Understood. Will try with the latest v4.3.
Btw, it would be really helpful if you could please share with me the esp-idf version used by you for the provided library?

The version of the test lib we provide is the latest lib corresponding to the current release branch

Understood. Will check with the latest version.
Thank you for the support.

Hello @Weijian-Espressif
Sorry for the delay.
I checked the updated library provided by you, with the latest release for v4.3 i.e. v4.3.4.
The config struct mismatch error did not occur, however, the original error assert llm_scan.c 1489, param 00000000 00000000 is getting displayed again.
Could you please help with this?
Thank you.

@AishwaryaK24  Please provide the complete log, I want to see the commit information of lib.

@Weijian-Espressif I am attaching the log below.
Please let me know if you need any other information.
ble_log.txt

@AishwaryaK24  Please check whether you are using the lib provided by me. According to the log information, the commit information of the lib is incorrect

@Weijian-Espressif
I am replacing the existing library at /esp-idf/components/bt/controller/lib_esp32c3_family/esp32c3 with the library provided by you. Then executing idf.py fullclean and idf.py build.
It will be helpful if you could let me know the correct process if the above is incorrect.
Thank you.

@AishwaryaK24

There is no problem with your steps, I just re-tested the lib in the attachment, the commit information is correct, and there is no problem with the lib.
Do you have multiple IDF projects? Is it possible that the IDF PATH points to other IDF projects?

@Weijian-Espressif
Thank you for the suggestion. Yes, the issue was the IDF_PATH setting.
However, after I changed the IDF_PATH to current directory(v4.3.4) and built the code, for some reason it is not connecting to wifi (using the original btdm lib of v4.3.4). I tried the same code with the current production code idf version(v4.3.2), and it was able to connect to wifi.
Due to this I am not able to connect to cloud for testing ble.
I am attaching the failed log file, will it be possible for you to suggest some solution?
Sorry for the inconvenience.
Thank you.
failed_wifi_connection.txt

@AishwaryaK24 I'm not sure what's going on with wifi, you can use the latest IDF like master, v5.0 or 4.4

Understood. Give me some time to check and I will get back to you.
Thank you.

Thanks for reporting, fix is available at https://github.com/espressif/esp-idf/commit/4a3319d381789a11f463d8062934e384006de97b, and already back ported to v5.0, v4.4 and v4.3. Feel free to reopen.


App runs on system time

the application runs based off of the user's system time, instead of a coordinated time
meaning 2 people can have different correct answers

https://stackoverflow.com/questions/24202641/how-can-you-get-the-server-time-from-firebase
leaving this for myself here, I'll try to get to this this weekend

fixed with new changes today! Thank you for your input :)


PMM-10603: Tests for New MongoDb Dashboards

https://pmm.cd.percona.com/job/pmm2-ui-tests/23681/

https://pmm.cd.percona.com/blue/organizations/jenkins/pmm2-ui-tests/detail/pmm2-ui-tests/24111/pipeline

@yurkovychv for mongodb_exporter matrix job


Extension issue

Issue Type: Bug
Extension Name: github-authentication
Extension Version: 0.0.2
OS Version: Darwin x64 21.6.0
VS Code version: 1.70.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:
{
	"messages": [],
	"activationTimes": {
		"codeLoadingTime": 55,
		"activateCallTime": 3,
		"activateResolvedTime": 0,
		"activationReason": {
			"startup": false,
			"extensionId": {
				"value": "vscode.github-authentication",
				"_lower": "vscode.github-authentication"
			},
			"activationEvent": "onAuthenticationRequest:github"
		}
	},
	"runtimeErrors": [],
	"runningLocation": {
		"affinity": 0,
		"kind": 1
	}
}

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette).
Happy coding!

Hey guys, thanks for your response. Forgive me for not being able to provide a recording. Seems like the issue is with my pc rather than the app itself and that being said I'm closing the issue.


Don't add sender to router if error

If there's an error when creating a new sender WebRTCTransport.NewSender the server will crash with panic: runtime error: invalid memory address or nil pointer dereference at router.go line 110 as the sender is nil.
I added a continue to skip the nil sender, alternatively we could return an error from the function to not allow the user to join the session at all.
This issue occurred when a second user was joining the session, but did not have a webcam (so the SDP only had an audio line). The NewSender function returned the errPtNotSupported error as it was trying to add a sender for a video track, but no video codecs were in the MediaEngine.codecs.
I think it would still be useful to just skip this sender (rather than not allow them to join the session) so the user can still participate in the session with audio only.
To prevent this from the frontend you can add a video transceiver manually pc.addTransceiver('video') if the user hasn't added any video tracks.

awesome thanks! i think this is the right solution. if a user mutes audio/video we should remove the track. would you mind adding a simple test for this flow?

No worries, I will add the test

awesome!


Send Active Account Email Error

konga_konga.1.vfwbqp6we7vfkslsxfg5cxyt2 | 2017-11-27T01:59:49.112347645Z user-events:notify:error Error: connect ECONNREFUSED <IP_ADDRESS>:587
konga_konga.1.vfwbqp6we7vfkslsxfg5cxyt2 | 2017-11-27T01:59:49.112472649Z     at Object.exports._errnoException (util.js:1020:11)
konga_konga.1.vfwbqp6we7vfkslsxfg5cxyt2 | 2017-11-27T01:59:49.112483054Z     at exports._exceptionWithHostPort (util.js:1043:20)
konga_konga.1.vfwbqp6we7vfkslsxfg5cxyt2 | 2017-11-27T01:59:49.112488728Z     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1086:14)

Maybe there's something wrong with my operation. I closed the issue.


Adding properties to EventTypes.page

I'm using a middleware that runs before reduxSegment which intends to add account_id to the properties across all events like this:
const decorateProperties = (action) => {
  const decoratedAction = merge({}, action, {
    meta: {
      analytics: {
        eventPayload: {
          properties: {
            account_id: <customer_account_id>
          }
        }
      }
    }
  });
  return decoratedAction;
}
const addAccountIdToAnalytics = store => next => action => {
  const result = next(decorateProperties(action));
  return result
}
const middleware = [
  addAccountIdToAnalytics,
  reduxSegment,
];
// And then I apply the middleware in configureStore

This works fine for EventTypes.track, but they won't pass through for EventTypes.page. Is that intended or am I doing something wrong?
Thanks.

Hi, would love feedback on this before I go trying to do something unnatural to get this to work.
I'm looking for a way to add an additional property to every EventTypes.page redux router event. Any thoughts, @bertrandk?

@dbryand This is probably because page events, by default, originate from within redux-segment. You may have to override the default mapper. The original redux router events are never read or merged since they weren't expected to contain any relevant info.


working example for creating and writing archive files in node and browser

Any working example present for creating and writing archive files in node and browser?

I am looking for usage functions for OpenSSL, LZMA, and libarchive for nodejs and the browser

First, Is this archive going to be maintained with appropriate releases? I am contemplating to use it for nodejs and for the browser.
Second, I am intending to use it for OpenSSL, LZMA, and libarchive function as well. Any demos?


Converted model classes to records

Updated ArgentPonyWarcraftClient.csproj to set LangVersion to latest to enable the usage of records.  Wrote a crude script to automate the conversion of all model classes to records.

For reference, here is the very crude conversion script:
using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Text;
using System.Text.RegularExpressions;

namespace ClassToRecordThing
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Hello World!");
            //ConvertToRecord(@"D:\Dev\warcraft\src\ArgentPonyWarcraftClient\Models\GameDataApi\Achievement\Achievement.cs");

            foreach (string path in Directory.EnumerateFiles(@"D:\Dev\warcraft\src\ArgentPonyWarcraftClient\Models", "*.cs", SearchOption.AllDirectories))
            {
                ConvertToRecord(path);
                Console.WriteLine(path);
            }

            Console.WriteLine("Done!");
        }

        private static void ConvertToRecord(string path)
        {
            State state = State.BeforeClass;
            string className = null;
            string currentSummary = null;
            var properties = new List<PropertyDetails>();

            StringBuilder final = new StringBuilder();

            foreach(string line in File.ReadAllLines(path))
            {
                string outline = line;

                switch(line) {
                    case string c when state == State.BeforeClass && c.Contains("public class"):
                        state = State.InClass;
                        className = Regex.Match(c, @"public class (\w+)").Groups[1].Value;
                        outline = c.Replace("public class", "public record");
                        break;
                    case string c when state == State.InClass && c.Contains("/// <summary>"):
                        state = State.InSummary;
                        break;
                    case string c when state == State.InSummary && c.Contains("/// Gets "):
                        currentSummary = Regex.Match(c, @"/// Gets (.+)$").Groups[1].Value;
                        break;
                    case string c when state == State.InSummary && c.Contains("/// </summary>"):
                        state = State.InClass;
                        break;
                    case string c when state == State.InClass && c.Contains("public ") && c.Contains("{ get; "):
                        Match match = Regex.Match(c, @"public ([^\s]+) (\w+) { get;");
                        string currentType = match.Groups[1].Value;
                        string currentProperty = match.Groups[2].Value;
                        properties.Add(new PropertyDetails { Type = currentType, NameUpper = currentProperty, NameCamel = LowerFirst(currentProperty), Summary = currentSummary });
                        outline = c.Replace("{ get; set; }", "{ get; }");
                        break;
                    case string c when state == State.InClass && c.Contains("    }"):
                        state = State.OutClass;
                        final.Append(BuildConstructor(className, properties));
                        break;
                }

                final.AppendLine(outline);
            }

            File.WriteAllText(path, final.ToString());
        }

        private static string BuildConstructor(string className, List<PropertyDetails> properties)
        {
            var c = new StringBuilder();

            c.AppendLine("");
            c.AppendLine("        /// <summary>");
            c.AppendLine($@"        /// Initializes a new instance of the <see cref=""{className}""/> class.");
            c.AppendLine("        /// </summary>");

            foreach(PropertyDetails property in properties)
            {
                c.AppendLine($@"        /// <param name=""{property.NameCamel}"">{UpperFirst(property.Summary)}</param>");
            }

            c.AppendLine("        [JsonConstructor]");
            string propertyList = string.Join(", ", properties.Select(p =><EMAIL_ADDRESS>{p.NameCamel}"));
            c.AppendLine($@"        public {className}({propertyList})");

            c.AppendLine("        {");

            foreach (PropertyDetails property in properties)
            {
                c.AppendLine($@"            {property.NameUpper} = {property.NameCamel};");
            }

            c.AppendLine("        }");

            return c.ToString();
        }

        private static string LowerFirst(string text)
        {
            if (text != null && text.Length > 0)
            {
                return char.ToLower(text[0]) + text.Substring(1);
            }

            return text;
        }

        private static string UpperFirst(string text)
        {
            if (text != null && text.Length > 0)
            {
                return char.ToUpper(text[0]) + text.Substring(1);
            }

            return text;
        }

        private enum State
        {
            BeforeClass = 0,
            InClass = 1,
            InSummary = 2,
            OutClass = 3
        }

        private class PropertyDetails
        {
            public string NameUpper { get; set; }
            public string NameCamel { get; set; }
            public string Type { get; set; }
            public string Summary { get; set; }
        }
    }
}

One thing I do want to call out, which you may have already thought about, is that this is going to require a major version bump on the library since the set on properties has been removed.

Yes, the next release was already going to require a major version bump due to the changes in #183.  There weren't a lot of changes, but they were breaking changes.  Might as well take advantage of it.

It's also worth highlighting that by removing the setters and moving to a constructor, it does make it harder for consumers to write any unit tests against the library. Not impossible, just mildly more inconvenient since they have to call the constructor with multiple parameters.

That's a very good point.  I reverted the change and took the approach of making the properties init-only instead of adding constructors.  That's probably more readable.


Artifact in rectangle

Reported on the forum https://groups.google.com/forum/?hl=en#!topic/cesium-dev/V6TF-vnPoBk
When you tilt the globe a certain amount, there's a visible seam in the rectangle

var viewer = new Cesium.Viewer('cesiumContainer');

var material = new Cesium.Material({
    fabric: {
        type: 'Image',
        uniforms: {
            image: '../images/earthspec1k.jpg',
            alpha: 0.6
        },
        components: {
            diffuse: 'texture2D(image, materialInput.st).rgb',
            alpha: 'texture2D(image, materialInput.st).a * alpha'
        }
    }
});
var geoInstatnce = new Cesium.GeometryInstance({
    geometry: new Cesium.RectangleGeometry({
        rectangle: Cesium.Rectangle.clone(Cesium.Rectangle.MAX_VALUE),
        height: 0
    })
});
var primitive = new Cesium.Primitive({
    geometryInstances: geoInstatnce,
    appearance: new Cesium.MaterialAppearance({
        material: material,
        faceForward: true,
        translucent: true,
        flat: true
    })
});

viewer.scene.primitives.add(primitive);

CC #4510

Can no longer reproduce

Congratulations on closing the issue! I found these Cesium forum links in the comments above:
https://groups.google.com/forum/?hl=en#!topic/cesium-dev/V6TF-vnPoBk
If this issue affects any of these threads, please post a comment like the following:

The issue at https://github.com/AnalyticalGraphicsInc/cesium/issues/4441 has just been closed and may resolve your issue. Look for the change in the next stable release of Cesium or get it now in the master branch on GitHub https://github.com/AnalyticalGraphicsInc/cesium.


Updated Item query stream to return same contract as item iterators

Item query stream previously returned only an array of elements. This differentiated from item stream iterator. This PR adds the additional contract information so the 2 operations have the same stream format.
// The stream contract will now return the same contract.
//  Documents property name changes based on resource type.
{
   "Documents": [{
       "id": "03230",
       "_rid": "qHVdAImeKAQBAAAAAAAAAA==",
       "_self": "dbs\/qHVdAA==\/colls\/qHVdAImeKAQ=\/docs\/qHVdAImeKAQBAAAAAAAAAA==\/",
       "_etag": "\"410000b0-0000-0000-0000-597916b00000\"",
       "_attachments": "attachments\/",
       "_ts":<PHONE_NUMBER>
   }],
   "_count": 1
}

isn't 'rid' also expected in the query contract?
rid in this specific case can be a continuation token right?

Please get sign-off from @bchong95


Await for callback in catch block

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (271c5b6) 100.00% compared to head (20d928d) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #28   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           11        11
  Branches         2         2
=========================================
  Hits            11        11

Impacted Files
Coverage Δ

src/index.ts
100.00% <100.00%> (ø)

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


Custom Loaders Support

Added support for using custom loaders.
Example
source.py
from luxonis_train.utils.loaders import BaseLoaderTorch

class CustomLoader(BaseLoaderTorch):
    import torch
    from luxonis_ml.data import LabelType

    from luxonis_train.utils.loaders import LuxonisLoaderTorchOutput

    @property
    def input_shape(self) -> torch.Size:
        return self.torch.Size((1, 3, 256, 256))

    def __getitem__(self, _: int) -> LuxonisLoaderTorchOutput:
        from luxonis_ml.data import LabelType

        img = self.torch.rand(3, 256, 256)
        label = self.torch.zeros(1, 256, 256)
        label[0, 100:200, 100:200] = 1
        labels = {
            "default": {
                LabelType.SEGMENTATION: label,
            }
        }
        return img, labels

    def __len__(self) -> int:
        return 32

    def get_classes(self) -> dict[LabelType, list[str]]:
        return {self.LabelType.SEGMENTATION: ["square"]}

luxonis_train --source source.py \
              --config configs/segmentation_model.yaml \
              loader.name CustomLoader loader.params '{}'

Breaking Config Changes
dataset section renamed to loader. Custom loader can be referenced by its name in the LOADERS registry. The default loader is LuxonisLoaderTorch. Former keys of the dataset field now go under params field in the loader.
loader:
  name: LuxonisLoaderTorch  # this is the default

  train_view: train
  val_view: val
  test_view: test

  params:
    dataset_name: coco_test
    bucket_storage: gcs

1/ Does this affect any of our current online datasets?

No, this shouldn't change anything regarding the underlying datasets

2/ Have we tested the kpts training?

Yes

@kozlov721 What is the correct way of providing BucketType and BucketStorage to the default LuxonisLoaderTorch? The DatasetConfig ensured that strings like internal, local, gcs` etc. were mapped to the corresponding enum type.


recursive update exception

code
+package sandbox

[args...] > app
  memory > n

  [] > func
    if. > @
      n.greater 0
      seq
        n.write (n.sub 1)
        func
        TRUE
      TRUE

  seq > @
    n.write 4
    func
    TRUE

exception

@IngeniariusSoftware definitely a bug, thanks for reporting

@IngeniariusSoftware this code works:
+package sandbox

[args...] > app
  memory > n
  [] > func
    if. > @
      n.greater 0
      seq
        n.write (n.sub 1)
        ^.func
        TRUE
      TRUE

  seq > @
    n.write 4
    func
    TRUE

Pay attention, I replaced func with ^.func. I'm trying to investigate why...

@IngeniariusSoftware it seems that this not a bug, but a feature. You do this (I'm simplifying):
[] > app
  [] > func
    seq > @
      func

Here, the third line is new func(func). You make an new copy of the abstract object func, setting its parent to... itself. Of course, it leads to endless recursion later.
On the other hand:
[] > app
  [] > func
    seq > @
      ^.func

Here, you take the attribute func  from the object app. This object already exists and its parent is app. No endless recursion.

@rultor release, tag is 0.19.1

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.19.1

@yegor256 Done! FYI, the full log is here (took me 17min)


Spline does not have mirror handles enabled

A shape produced by the spline tool does not have mirror handles enabled, meaning that when you switch to the path tool and drag a handle, the opposite one does not automatically move. The expected behaviour is that it should remain mirrored, similar to when a similar action is taken with an ellipse. To resolve this, the mirror handle setting should be enabled when creating a spline in spline_tool.rs, you can use ellipse_tool.rs as reference for setting the mirror.

Hi, new here! I'd like to look at this (and a few of the other good first issues :p).

Welcome @Chase-Percy, this would be a great place to get started. Please feel free to join our Discord and reach out to 0Hypercube there if you need any pointers. Thanks for volunteering :)


fix a few typos

@danmarshall : Thanks for your contribution! The author, @davidiseminger, has been notified to review your proposed change.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-docs/desktop-bookmarks.md
:white_check_mark:Succeeded

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

@danmarshall : Thanks for your contribution! The author, @davidiseminger, has been notified to review your proposed change.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-docs/desktop-bookmarks.md
:white_check_mark:Succeeded

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

Looks good - and sheesh, I should've caught those before it went out. #sign-off


Android is not 100% migrated to new module name

There are some files that still have:
@m1ga There is something that you can check when you have free time?
@ottopic issue https://github.com/nuno/TiCollectionView/issues/73
@Override
	public String getApiName()
	{
		return "de.marcelpociot.CollectionItem";
	}

https://github.com/nuno/TiCollectionView/blob/master/android/src/de/marcelpociot/collectionview/CollectionViewProxy.java#L437

@m1ga if you think this is not an issue, let me know :)

see https://github.com/nuno/TiCollectionView/pull/80

Can I merge?

yes, but it didn't fix the issue for me (Alloy is still working, even without the changes). But it will correct the naming

this can be bacause of folders naming?
/src/de/marcelpociot/collectionview
I do not know, Im not an android expert, thanks.

No, thats not an issue. The problem is item.setDefaultProperties(new KrollDict((HashMap)props)); that item is null in a vanilla project. Working fine in Alloy. Doing some tests at the moment to see what the source is

ok, is normal in manifest the "commonjs: true"  ?

Can't find the issue. But this is where it goes wrong. Inside the processProperties of the CollectionViewTemplate there are some differences:
Properties of the alloy version:
{"properties":{"name":"template"},"tiProxy":null,"childTemplates":[{"properties":{},"tiProxy":null,"type":"Ti.UI.View","childTemplates":[{"properties":{"height":"size","width":"size","font":{"fontSize":12},"bindId":"info","color":"#000"},"type":"Ti.UI.Label","tiProxy":null,"bindId":"info"},{"properties":{"height":"size","width":"size","top":"30dp","font":{"fontSize":12},"bindId":"es_info","color":"#000"},"type":"Ti.UI.Label","tiProxy":null,"bindId":"es_info"}]}]}

properties of the vanilla version:
 {"childTemplates":[{"properties":{"top":10,"font":{"fontWeight":"bold","fontSize":20}},"type":"Ti.UI.Label","bindId":"info"},{"properties":{"top":40,"font":{"fontSize":14},"color":"gray"},"type":"Ti.UI.Label","bindId":"es_info"}]}

Perhaps someone else has a clue

Alright, thanks @m1ga!

@m1ga any news about the issue? Have you tried to find a solution?
Best regards

@ottopic no, sorry. If you can use the alloy version, that was working for me. But I'm not really using this module so I only tested the example


[Improve](complex type) delete fe config disable_nested_complex_type

Proposed changes
same with https://github.com/apache/doris/pull/36255
Issue Number: close #xxx

run buildall

run buildall


Install on OSX?

I installed all the packages on my Macbook, however, it fails with the error:
AssertionError: Torch not compiled with CUDA enabled
My understanding is you can't get CUDA unless you have an NVidia graphics processor.
Can this be run without that?
Thanks for clues
pip freeze:
backports.functools-lru-cache==1.6.1
bz2file==0.98
cycler==0.10.0
future==0.18.2
kiwisolver==1.1.0
matplotlib==2.2.5
nibabel==2.5.2
numpy==1.16.6
opencv-python==<IP_ADDRESS>
pandas==0.24.2
Pillow==6.2.2
pyparsing==2.4.7
python-dateutil==2.8.1
pytz==2020.1
scipy==1.2.3
six==1.15.0
subprocess32==3.5.4
torch==1.2.0
-e git+https://github.com/ncullen93/torchsample.git@ea4d1b3975f68be0521941e733887ed667a1b46e#egg=torchsample
torchvision==0.4.0
tqdm==4.48.2

nvm.
It seems all ll I had to do was pass in --gpu 0


Readability of long lines

Since npm doesn't support arrays in package.json script, are there some ways to split horribly long lines into something more readable?
Here's what I have right now:
"build:js": "uglifyjs node_modules/asteroid/node_modules/ddp.js/src/ddp.js node_modules/asteroid/node_modules/q/q.js node_modules/asteroid/dist/asteroid.chrome.js --screw-ie8 -c -m -o dist/vendor/asteroid.js && uglifyjs node_modules/css-selector-generator/build/css-selector-generator.min.js node_modules/selection-menu/node_modules/tether/dist/js/tether.min.js node_modules/selection-menu/src/selection-menu.js --screw-ie8 -c -m -o dist/vendor/selection-menu.js && cpy src/*.js dist/",
One way would be to artificially move parts of that command into prebuild and postbuild, but I might need to put legit command in pre and post later.

See npm/npm#6943 for my suggestion. Otherwise, consider a few other solutions:
Split them out into different run scripts, e.g.
"build:js:asteroid": "uglifyjs node_modules/asteroid/node_modules/ddp.js/src/ddp.js node_modules/asteroid/node_modules/q/q.js node_modules/asteroid/dist/asteroid.chrome.js --screw-ie8 -c -m -o dist/vendor/asteroid.js",
"build:js:selectionmenu": "uglifyjs node_modules/css-selector-generator/build/css-selector-generator.min.js node_modules/selection-menu/node_modules/tether/dist/js/tether.min.js node_modules/selection-menu/src/selection-menu.js --screw-ie8 -c -m -o dist/vendor/selection-menu.js",
"build:js": "npm run build:js:asteroid && npm build:js:selectionmenu && cpy src/*.js dist/",

Write some JS!
Just write a small JS file, put it in a folder like tasks/build-js.js and call it with npm:
"build:js": "node tasks/build-js.js"

You can even pass it args if you need to:
"build:js": "node tasks/build-js.js dist/"

I also struggled with this problem and I did not want to move my build logic to JavaScript. So I decided to build a npm packages script-launcher that enhances the package.json scripts with features like: menus, functions, arrays, concurrency etc.
With script launcher you can rewrite the example of dandv as:
{
  "build:js": [
    "src=node_modules/asteroid",
    "uglifyjs $src/node_modules/ddp.js/src/ddp.js $src/node_modules/q/q.js $src/dist/asteroid.chrome.js --screw-ie8 -c -m -o dist/vendor/asteroid.js",
    "uglifyjs node_modules/css-selector-generator/build/css-selector-generator.min.js node_modules/selection-menu/node_modules/tether/dist/js/tether.min.js node_modules/selection-menu/src/selection-menu.js --screw-ie8 -c -m -o dist/vendor/selection-menu.js",
    "cpy src/*.js dist/",
  ]
}

You can probably make it shorter by using the settings feature of script launcher. When using this you place your files in an array  as well. Below is an example of what it would look like, but you have make it work, for you particular use case.
{
  "scripts": {
    "build:js": [
      {
        "repeater": "$launch_setting_files",
        "sequential": [
          "uglifyjs $_ --screw-ie8 -c -m -o dist/vendor/asteroid.js"
        ]
      }
    ]
  },
  "settings": {
    "files": [
      "node_modules/asteroid/node_modules/ddp.js/src/ddp.js",
      "node_modules/asteroid/node_modules/q/q.js",
      "node_modules/asteroid/dist/asteroid.chrome.js"
    ]
  }
}


GH4255: Introduce Verify for CodeGen Expectations

fixes #4255

u should tweak encoding and line endings in gitattributes and editor config

if builds fail on CI it is nice to expose the recieved files for download https://github.com/VerifyTests/Verify/blob/main/docs/build-server.md

if builds fail on CI it is nice to expose the recieved files for download https://github.com/VerifyTests/Verify/blob/main/docs/build-server.md

Sensible addition can add that in dotnet test Task OnError
https://cakebuild.net/docs/writing-builds/tasks/error-handling
I'll add an issue to track that post 4.0 release.

u should tweak encoding and line endings in gitattributes and editor config

added.

Verify.DiffPlex help with the signal to noise. especially in CI https://github.com/VerifyTests/Verify.DiffPlex

I'll add an issue to track that post 4.0 release.


Problems with active site labeling during training

Why is it that when active sites are predicted, in the calculate_active_sites method of class EnzymeDataset, all sites where site_label is present are labeled as 1, instead of active sites being labeled as 1

When predicting active sites in a binary classification task for enzymes, we consider Binding Site, Catalytic Site (referred to as Active Site in UniProt, as noted in the paper, annotation "d" in Table 1), and Other Site as active sites. The EnzymeDataset class serves as a basic class, with its calculate_active_sites method used to convert the labels of all the aforementioned sites. If you want to convert these sites separately, you need to use the EnzymeReactionSiteTypeDataset class, which provides the calculate_active_site_types method to achieve this purpose.


set row actions click target to 36x36

Set click target and padding for actions trigger to be 36x36.
closes #1402
Signed-off-by: Jeremy Wilken<EMAIL_ADDRESS>

Ugh, I just realized that the "good" click targets in datagrid are 30x30px. I don't remember this discussion to go with this instead of 36px.
@reddolan: Was there a "derogation" for datagrid click targets to go down to 30px because of row size constraints? I can't find the information.

@youdz
Our decision with the UX of the datagrid what's that the rows would be 36px tall. At the time, this meant that the tallest icon button we could use was 30px.
The thinking behind the 30px x 30px icon buttons in the data grid was that are preferred click target of 36 x 36 was an affordance for smaller form factors like phones.
Our reasoning was that the data grid would not be used for a phone application. That didn't make sense to us. So we decided that 30 x 30 was okay.

Is there a reason why we can't take full advantage of the 36px row height? Could we use a 36px click target? The click target doesn't need to have a margin and being able to click anywhere in that 36x36 space would be easier than clicking just outside of the click target

@reddolan At 36px it actually pushes the row size slightly https://raw.githubusercontent.com/gnomeontherun/clarity/a427dc50c7277475f6ed3dddb91be86c34f7dbd2/gemini/screens/datagrid/multl-selection/action-overflow/chrome.png

@gnomeontherun
We need it to maintain the existing size of the datagrid rows -- even if the icons' click targets are larger. Increasing the rows to 42px is not going to work.

@gnomeontherun any ideas as to why it pushes the row height?
@mathisscott agreed, I definitely don't want to increase the row height.
If the best we can do is 30x30, that is better than what exists for the current (16x17) single-row action touch target

@reddolan The best we can do is 35px because the rows are 36px with a border (so inner is 35px). I've updated to 30x30, and this doesn't change any of the current gemini images.


MAINT: Refactor expm_multiply

Reference issue

None
What does this implement/fix?

This is the follow-up to #14143 refactoring expm_multipy for simpler code.
The main point is to eliminate code duplicates.
(To me, it even seems that we could further drop the distinction between _expm_multiply_interval_core_1 and _expm_multiply_interval_core_2).
Furthermore, the hard-coded values of the tolerance was replaced by the data-type specific one.
Additional information

Note that the issue #16368 of instabilities probably remains, but the norm is now correcly non-negative as the absolute of the scale is taken.

Thanks @DerWeh! I did a quick scan, this seems like a nice improvement. I should get around to reviewing this in the near future if no one else beats me to it.

@rgommers The pull request is waiting for quite some while now. I understand that you are probably busy. Is there a way to ping somebody else to take a look?

Hi @DerWeh - in the meantime can you check the conflict showing up now?
Maybe @dschult or someone else from the sparse team wants to take a look?

@melissawm Sure, the conflict is no big issue. Seems like we can just take main. So I am not real certain about it, seems like the semantics of sparse matrices was update.

Remaining errors are unrelated to this PR

@rgommers @melissawm Is there a change to get this merged at some point?

@DerWeh unfortunately I don't have the expertise to review the code here, sorry!


As a user I want to to make a quick bar chart of the avg population for each country from sampleData(3)

Expected behavior
import pixiedust
df = pixiedust.sampleData(3)
display(df)

Select Bar Chart > Keys = Country, Values = Value, Aggregation = AVG
This gives the below error, and does not work for any of the renderers
Actual behavior

Steps to reproduce the behavior

The problem was with the data itself that had empty columns. I configured the spark-csv connector to drop malformed rows #124
@MargrietGroenendijk transferring to Review/QA for you to validate. Please move it to done when it's validated


Greenkeeper/@types/mocha 5.2.0

Coverage remained the same at 94.595% when pulling c0a460396de65b3e13459b9cc13d40b279a568b3 on<EMAIL_ADDRESS>into 896da677a629adc36a669a329eaa76e9b5e95f3d on master.


Export timeshift records as files using FUSE

Currently, there is no easy way to play timeshift records.
Implementing a frontend supporting the timeshift API is one of candidates, but it takes time.
As an alternative, we implement a feature to export timeshift records as files using FUSE.
We expect that this feature makes it easy for users to play records with media players like VLC and KODI which support MPEG2-TS.
The structure of the timeshift config will be changed like below:
timeshift:
  mount: /path/to/mount-point
  recorders:
    nhk:
      ...

zargony/fuse-rs is used for the filesystem implementation.

Considering separation of the filesystem implementation from mirakc.
media players (running on a local machine)
  A
  |
  | export records as files
  |
mirakc-fs (running on a local machine)
  |
  | fetch data periodically via REST API
  V
mirakc (running on a remote server)

This is simpler than the original plan.
Separation of the function improves maintainability of both.

Finally:
media players
  A
  |
samba/dlna (Docker container)
  | (scan media files)
  +==> files (accessible from the Docker host)
  | (expose using FUSE)
mirakc-timeshift-fs (Docker container)
  | (read)
  +==> timeshift records (ts and data files)
  | (write)
mirakc (Docker container)

Implemented in 69f699c9901bf848e4d8bada6c77d3fcd7f9d938


Implement support for alternative access loggers

This commit implements additional types that allow custom loggers to be supplied for the ozzo-routing/access middleware.  The original access logger has been re-factored to use the new CustomLogger method to create the Logger method that returns the logging middleware.  In addition, a logger that can be used to write metrics to Datadog has been created that also uses CustomLogger.
Based on the 2 loggers existing, I believe that at least for now the refactor is sensible and can be used generally for writing to different logging/metric backends.
It might be desirable in the future to, instead of passing the actual request and response objects, to instead extract (or copy in the case of req.URL) data values into a struct that is then passed to the CustomLogger.  Absent that, it is the responsibility today of CustomLogger implementors to make sure that their use of the request and response objects is concurrency safe.  Specifically, these objects should be treated as read-only.

Coverage remained the same at 94.643% when pulling d876094a7e640949d093d2a68171340a9d2fb6b1 on jonathana:master into d2a16ccf3fb7eca92e694ccabbd9200bfda4bc6f on go-ozzo:master.

Thanks!


Go version upgrade

Go team officially maintains two last versions of language.
The current version is 1.19. The current version here is 1.17.
May be it's time for upgrade?
What do yo think about it?
Thanks.

Go team officially maintains two last versions of language. The current version is 1.19. The current version here is 1.17. May be it's time for upgrade? What do yo think about it? Thanks.

Hi @misantron, thanks for the suggestion.
For now, I plan to keep the current version, later I will test everything and upgrade it by my self.
I will close this issue because I want to do it my self.


Add page breaks to the UG

Add page breaks to fit content nicely in PDF.
Done again for CS2101 submission.

Codecov Report

Merging #277 (6324a36) into master (59083a7) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #277   +/-   ##
=========================================
  Coverage     56.30%   56.30%
  Complexity      765      765
=========================================
  Files           161      161
  Lines          3307     3307
  Branches        363      363
=========================================
  Hits           1862     1862
  Misses         1319     1319
  Partials        126      126

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 59083a7...6324a36. Read the comment docs.


update-integration-configs should fail fast when admin_password is not found

update-integration-configs should fail fast when admin_password is not found. It's required for CATs, and I suspect WATS and RATS.
We've had a few experiences in the UAA pipelines where update-integration-configs does not find an admin_password for CATs and then updates cats_integration_config.json with admin_password: ''. update-integration-configs shows as "passed" in Concourse status, which indicates no problems. That being said, we do see some output from the Credhub CLI in update-integration-configs that may indicate a problem.
+ check_fast_fails
+ '[' '!' -f integration-configs/concourse/uaa-acceptance-gcp/state/cats_integration_config.json -a '!' -f integration-configs/rats_integration_config.json -a '!' -f integration-configs/wats_integration_config.json ']'
+ setup_bosh_env_vars
+ set +x
/tmp/build/7f3ba2bf/bbl-state/concourse/uaa-acceptance-gcp/state /tmp/build/7f3ba2bf
/tmp/build/7f3ba2bf
+ set +x
Usage:
  credhub [OPTIONS] get [get-OPTIONS]

Get a credential value by name or ID

Application Options:
      --http-timeout=    Http timeout for http-client. Needs to have unit
                         passed in (i.e. 30s, 1m) [$CREDHUB_HTTP_TIMEOUT]
      --version          Version of CLI and targeted CredHub API
      --token            Return your current CredHub authentication token

Help Options:
  -h, --help             Show this help message

[get command options]
      -n, --name=        Name of the credential to retrieve
          --id=          ID of the credential to retrieve
          --versions=    Number of versions of the credential to retrieve
      -j, --output-json  Return response in JSON format
      -q, --quiet        Return value of credential without metadata
      -k, --key=         Return only the specified field of the requested
                         credential
updating CATs integration config file: concourse/uaa-acceptance-gcp/state/cats_integration_config.json...

CATs then fails with error Invalid configuration: 'admin_password' must be provided.
Note that we're still able to retrieve this password via Credhub CLI on our local machine, and it appears from the timestamp that it hasn't updated in a while (version_created_at: "2019-08-16T22:14:20Z"). I'm not sure why update-integration-configs does not retrieve this password.

CC: @cloudfoundry/cf-uaa

Hi @joshuatcasey,
What does credhub find -j -n cf_admin_password return? If it returns multiple credentials, then the xargs command is going to include all of them in the credhub get command, which will result in the error you are seeing. It almost sounds like you have a second deployment of cf-deployment on the same BOSH director.
We have added some sanity checks to the get_password_from_credhub helper function here. Please feel free to try them out to see if it helps you to understand why this is failing.
Regards,
Dave and @Birdrock

We do have multiple passwords returned! There's another deployment that no longer exists but the password is still in Credhub. I'll clean that up and hopefully we don't see this in the future.

Sounds good @joshuatcasey. I am going to go ahead and close this out.


MalwareBehaviorCatalog follow-up: less naming stutter, less slashes

Slashes are now only in rule names to show directory hierarchy - this removes the - to / conversion.

lint error is intentional and dumb: artefacts appears as a name in the YARAForge signatures.

lint error is intentional and dumb: artefacts appears as a name in the YARAForge signatures.

Can we start excluding certain words?
https://docs.getwoke.tech/ignore/#in-line-and-next-line-ignoring
⬇
func main() {
  fmt.Println("here is the whitelist") // wokeignore:rule=whitelist

  // wokeignore:rule=blacklist
  fmt.Println("and here is the blacklist")
}

lint error is intentional and dumb: artefacts appears as a name in the YARAForge signatures.

Can we start excluding certain words?
https://docs.getwoke.tech/ignore/#in-line-and-next-line-ignoring ⬇
func main() {
  fmt.Println("here is the whitelist") // wokeignore:rule=whitelist

  // wokeignore:rule=blacklist
  fmt.Println("and here is the blacklist")
}

Yeah, I think we should kill this linter and do it through golangci-lint instead though.


added: menuItems to prepare()

Hey @c128128 thanks for your contribution. What was the issue you are solving within this PR?

Hi @iSapozhnik,
I need to change right click menu when some user is logged in, show logout, and viceversa.


Failure to download product key

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/install
Operating System: Linux Ubuntu 16.04
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
Following the steps in the link above for Ubuntu 16.04, I'm unable to download/register the product key.
> wget -v https://packages.microsoft.com/config/ubuntu/16.04/packages-microsoft-prod.deb
--2019-03-07 10:31:56--  https://packages.microsoft.com/config/ubuntu/16.04/packages-microsoft-prod.deb
Resolving packages.microsoft.com (packages.microsoft.com)... <IP_ADDRESS>
Connecting to packages.microsoft.com (packages.microsoft.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 2452 (2.4K) [application/octet-stream]
packages-microsoft-prod.deb: No such file or directory

Cannot write to ‘packages-microsoft-prod.deb’ (Success).

Confirmed that https://packages.microsoft.com/config/ubuntu/16.04/packages-microsoft-prod.deb exists and downloads as expected. Give it another go? Alternatively, just click on the following to download: https://packages.microsoft.com/config/ubuntu/16.04/packages-microsoft-prod.deb


Always use forward slashes even on Windows

I'm building on Windows, Mac and Linux and the keys don't match between platforms. These two replaces mean that the values and keys have '' reaplaced with '/'.

This was fixed in already metalsmith-fingerprint @
https://github.com/christophercliff/metalsmith-fingerprint/issues/7

I'm getting round this currently with a custom step which cycles over the fingerprint keys and adds duplicates with forwards slashes:
    .use(function (files, metalsmith, done) {
      var meta = metalsmith.metadata();
      Object.keys(meta.fingerprint)
        .forEach(function (p) {
          var cleanKey = p.replace(/\\/g, '/');
          meta.fingerprint[cleanKey] = meta.fingerprint[p].replace(/\\/g, '/');
        });
      done();
    })

Cool, thanks for the PR!

Published on npm as 1.1.2


Feature Request: Paste from Clipboard for Initial Image (img2img) or image in-paint

There doesn't appear to be any obvious way to paste an image from the clipboard to use as the Initial image for img2img or in-painting.
Instead, ctrl+v pastes the image to be read by CLIP. There should be an option to also use it as the initial image.
This breaks the workflow for anyone who uses external image editors to make specific edits to images before feeding them back into img2img, without the need to save the intermediary edited photo.
For example, my workflow when using Auto1111 was to make quick edits to a photo in Photoshop, copy (not save) the entire image and then paste into Img2Img or for in-painting, generate, copy, Photoshop, rinse/repeat until I'm happy with the results.

Right now, you can:

Drag a file into the image input
CTRL+V in prompt then drag to the init image and remove from prompt
click the init image to get a file selector and go that way

The main thing to handle to allow CTRL+V direct into the image inputs is proper focus handling iirc

Right now, you can:
* Drag a file into the image input

* CTRL+V in prompt then drag to the init image and remove from prompt

* click the init image to get a file selector and go that way

The main thing to handle to allow CTRL+V direct into the image inputs is proper focus handling iirc

I really appreciate your reply and providing a solution. This works well for me!

Dwedit's PR has added a little box you can CTRL+V inside of


Fix custom formatter handling logic. (#15052)

The function externalgettup_custom() is to handle custom formatter logic when loading data with external tables. But if custom protocol like PXF is used to loading data, the EOF is not reached when the last content of data is returned, the EOF is reached only when the last reading returns 0. So in the current logic of function externalgettup_custom(), it may lose the chance for custom formatter to handle the last line of data.
This commit removes the using of raw_buf_len, instead the formatter->fmt_databuf which provides the real data is used. So we check the length of this buffer to give custom formatter last chance to handle the last line of data. Even if the last line is not complete, it is the custom formatter's responsibility to report this.
The right way to return FMT_NEED_MORE_DATA in custom formatter is to check EOF flag before return. But currently, a lot of custom formatter does not implement in the right way. They just return FMT_NEED_MORE_DATA when they need more data in the current block of line without changing the formatter->fmt_databuf nor checking the EOF. So it will cause infinite loop.
In order to prevent infinite loop, we detect this by adding a count when eof flag is set. So this PR enables custom protocol to handle the last line of incomplete data if the custom formatter has taken care of the eof flag, and also prevent infinite loop regression.
Cherry-picked from: e859cd6c84a8c12c2878eee3dba61ebf7ae4874d

This patch solves problems with handling interrupted reads from the PXF based on old tomcat versions (i.e. 9.0.43). These tomcat versions write serialized error exception to socket to client and correctly terminate chunked base stream. So client can understand that something has gone wrong only within parsing errors. New tomcat versions close connection without sending final empty chunk and violate chunked protocol, so client can detect problem before parsing.
Custom protocol (including pxf) detects EOF by receiving an empty response from the protocol implementation.
Flow before patch:

Exception on pxf side happens
curl on the client side gets this chunk of data to the custom protocol buffer without errors
it's not EOF as far as data received
formatter constructs tuples from buffer until there are valid data but not error message
formatter requests one more buffer as far as EOF is not reached
curl returns zero bytes as far as connection was closed
external table interprets zero byte as EOF.
as far as there is no new data received, external table doesn't call formatter and return Null-tuple
warning about something in the buffer is generated

Flow after patch:

Exception on pxf side happens
curl on the client side gets this chunk of data to the custom protocol buffer without errors
it's not EOF as far as data received
formatter constructs tuples from buffer until there are valid data but not error message
formatter requests one more buffer as far as EOF is not reached
curl returns zero bytes as far as connection was closed
external table interprets zero byte as EOF.
there is no new data received, but buffer contains non-parsed error message, external table calls formatter
formatter interrupts parsing with error as far as EOF is reached

How WARNING:  unexpected end of file became ERROR:  unexpected end of file?

In general, I would prefer a warning. For example, we successfully received and downloaded a billion rows (in a few hours), and an error occurred on the next row. And what, now throw out several hours of work just because of one line ?!

Although this patch will come to us with another sync.

How WARNING:  unexpected end of file became ERROR:  unexpected end of file?

It's responsibility of exact formatter to handle this situation (see for pxf)

In general, I would prefer a warning. For example, we successfully received and downloaded a billion rows (in a few hours), and an error occurred on the next row. And what, now throw out several hours of work just because of one line ?!

No. We must load all or nothing. How to understand when loading was interrupted?

How to understand when loading was interrupted?

Just load all whole lines and issue a warning that an error occurred next.

Is this code does not reached anymore?

Yes. The pxf fragmenter invoked here throws Error earlier. Error will be re-thrown if the user doesn't ask error ignore here (see macro implementation).

Just load all whole lines and issue a warning that an error occurred next.

Segments load data in parallel. Fragments shuffled between segments. There are no guaranties about order of lines, extracted from remote storage. What lines are next?

What lines are next?

What's the difference? The main thing is to accept the entire successfully loaded rows, and skip those with errors by issuing a warning.

this code is no longer conditionally reached (see previous while loop).

Although, no, after all, here - maybe.

Although, no, after all, here - maybe.

No. There is an outer loop. We still may achieve block of your interest in case of buffer is empty and EOF is achieved OR formatter request more lines despite EOF (in case of formatter doesn't control EOF).
	/* while didn't finish processing the entire file */
	while (formatter->fmt_databuf.len > 0 || !pstate->fe_eof)
	{
		if (infinite_loop_detect) {
			break;


quit() causes Spyder to hang

From<EMAIL_ADDRESS>on 2012-02-29T14:04:39Z
What steps will reproduce the problem?
type "quit()"
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=956

From pierre.raybaut on 2012-03-18T08:11:44Z
I can't reproduce this bug.
Typing quit() in either a console or in the internal console works fine for me.
Could you please fill out all the usual questions (OS, versions, ...)?
Is anyone able to reproduce this bug?

From<EMAIL_ADDRESS>on 2012-05-14T21:50:12Z
Doesn't make Spyder hand for me too. 2.2.0dev on Linux.
But for standard console the behavior is quite buggy - calling quit() makes prompts disappear, and no input/output is shown at all. All other parts of Spyder continue to work ok.
Labels: Cat-Console

From pierre.raybaut on 2012-05-22T07:09:15Z
I'm closing this issue as the 'quit()' behavior in the internal console is irrelevant because the internal console is for Spyder debugging only, so I don't expect 'quit()' action to be reliable here.
Status: Invalid


iOS App closed or in background does not receive notification

Expected Behavior
iOS app does show a push notification when it's on background or even when the app is closed.
Actual Behavior
The app only shows the notification when it's in foreground.
Reproduce Scenario (including but not limited to)
New project with the plugin added and receiving push notifications if the app is in foreground. The app has permissions to show push notifications.
Case 1: Send the app to background. Send a new push notification from the test server. The app doesn't show anything.
Case 2: Close the app. Send a new push notification from the test server. The app doesn't show anything.
Steps to Reproduce
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Tested on iOS 9.3.2
Cordova CLI version and cordova platform version
cordova --version                                    # e.g. 6.0.0

6.1.1
cordova platform version iOS                     # e.g. 4.1.1
4.1.1
Plugin version
1.6.3

If you're using GCM, this payload works:
{ "priority" : "high", "notification": { "title": "Test Notification", "body": "This offer expires at 11:30 or whatever", "notId": 10 }, "data" : { "surveyID": "ewtawgreg-gragrag-rgarhthgbad", "content-available": "1" }, "to": "id here" }

No, I'm using APNS.

You haven't posted the payload you're using but are you using "alert" as per
For instance the following JSON:
{
"aps": {
"alert": "Test background push",
"content-available": 1
}
}
will produce a notification in the notification shade and call your on('notification') event handler.
However if you want your on('notification') event handler called but no notification to be shown in the shader you would omit the alert property and send the following JSON to APNS:
{
"aps": {
"data": "Test silent background push",
"moredata": "Do more stuff",
"content-available": 1
}
}

Sorry, I forgot the payload. I use a very simple payload:
    "aps": {
        "alert": "Cambio de estado de TEST del vuelo XXX",
        "sound": "default"
    }
}```

Again, if the app is in foreground everything works fine but if the app is in background or closed, the notification is not shown.

I got nothin'... Use content-available since it wakes up the app? Sorry I couldn't help more.

I have found the fix. The iOS part of the plugin also needs the definition of:
- (void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo {
    NSLog(@"application:didReceiveRemoteNotification: %@", userInfo);
}

Adding that method everything works fine.

@fjordansilva I'll check out your PR but it's weird to think that would fix anything. All you are doing is printing a log and we already implement - (void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo fetchCompletionHandler:(void (^)(UIBackgroundFetchResult))completionHandler which is basically the same thing.

@fjordansilva can I see what you use for the init method?

I have the same problem. Any news?

guys,
I have faced this problem for days, and fixed that just adding priority: "high" in the objectt sent to Notifications Service.
Now, my app on IOS shows me notification even the app is killed (closed).

Closing, inactivity. Please comment/re-open if you have more details.

I have the same problem, I tried to apply different solutions but nothing works for me.
This is my json to aps.
{"aps":{"alert":"","background":true,"content-available":1,"priority":10}}
With this json to aps .on('notification') the event is not executed when the app is closed (killed).
¿Some solution? :/


Add Sakarya University to domain list

Sakarya university web url: https://www.sakarya.edu.tr/
Sakarya university ceng web url: https://cs.sakarya.edu.tr/

@Saitpstc I am sorry, the domain ogr.sakarya.edu.tr is removed from our trusted domains list because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


Any Android App available to control Fauxmo emulated switches ?

Hi, I know Wemo apps don't work with Fauxmo cause it's only emulating part of the Wemo, so my question is, is there any other app available that can control this switches with my phone too (apart from Alexa) ?
Thanks.

Well, I have three things set up. I can use Apple Home and any Homekit app, but it requires:
Fauxmo. Home Assistant which sees the Fauxmo switches. And Homebridge, which imports Home Assistant's devices and exposes them to Homekit.
This also allows Homekit apps to control real Wemo switches, which will never happen otherwise.

Oh, I'm using the forked version of fauxmo, FYI: https://github.com/n8henrie/fauxmo/issues.
Oh, and I just noticed that you said Android in you subject. Sorry.

Yeah, I was thinking of something simplier, like just be able to send a http request or similar to my Fauxmo (running on an ESP8266) and turn on the lights or whatever, so that not only I can control it using Alexa but also Wifi with my cellphone or tablet. The problem is that Wemo apps do not work as they use a more complicate setting that Alexa requieres, at least from what I read around.

I can write my own Java app for Android (have some basic skills) but don't know what format and protocol should I send to the ESP to be able to, for example, send a Light One On to the Wemo emulator.

I use this app to run commands via ssh on the raspberry pi fauxmo is running on. But I'd love one that discovers all the fauxmo devices automatically and lists them in a nice format with on/off buttons.
https://play.google.com/store/apps/details?id=org.openobjectives.serverassistant

need something too... I want to try my esp8266 fauxmo running device befor buying alexa... best would be an android app, but can be tested with python or C# script on my computer, if it works... found pywemo, but it seems not working, cannot detect device, also with IP adress...

You can use the Open Source Perl project Power::Outlet. It has an Apache CGI script that comes with the project and has an HTML-based button UI.
Look at the RPM spec file for guidance, if you need to install it on a non-RPM-based OS.


Improve the menu responsiveness

At the particular width size, the menu or navbar shrinks but on clicking on the button list is not completely visible

@pranjalkatiyar Go ahead with the issue


Clicking drive on Embassy setup process results on no-op

Currently when you try to select your drive that you wish to set your embassy up with, the button does nothing. The web console logs show an issue with ng-zone.

Removing @MattDHill as an assignee. I will build/test this morning and confirm it has been fixed. If so, I will close this.


Update faq.asciidoc - unexpected eof, connection reset - extend these mysql server timeouts

Nice addition; thanks a lot, @pimpelsang! I've done some rewording and merged.


Train Multi Gpu

Thanks for your great repo.
I tried to train the model for a big graph (about 7M edges in training data).
When I was Training on a single GPU, training time wasn't efficient, so I decided to try multi-gpu, but I got this error:
  File "/home/BiGI/BiGI_src/train_rec.py", line 163, in <module>
    loss = trainer.reconstruct(UV, VU, adj, corruption_UV, corruption_VU, fake_adj, user_feature, item_feature, batch)  # [ [user_list], [item_list], [neg_item_list] ]
  File "/home/BiGI/BiGI_src/model/trainer.py", line 152, in reconstruct
    self.update_bipartite(user_feature, item_feature, CUV, CVU, fake_adj, fake = 1)
  File "/home/BiGI/BiGI_src/model/trainer.py", line 131, in update_bipartite
    self.user_hidden_out, self.item_hidden_out = self.model(user_feature, item_feature, UV_adj, VU_adj, adj)
  File "/home/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 158, in forward
    inputs, kwargs = self.scatter(inputs, kwargs, self.device_ids)
  File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 175, in scatter
    return scatter_kwargs(inputs, kwargs, device_ids, dim=self.dim)
  File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/scatter_gather.py", line 44, in scatter_kwargs
    inputs = scatter(inputs, target_gpus, dim) if inputs else []
  File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/scatter_gather.py", line 36, in scatter
    res = scatter_map(inputs)
  File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/scatter_gather.py", line 23, in scatter_map
    return list(zip(*map(scatter_map, obj)))
  File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/scatter_gather.py", line 19, in scatter_map
    return Scatter.apply(target_gpus, None, dim, obj)
  File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/_functions.py", line 96, in forward
    outputs = comm.scatter(input, target_gpus, chunk_sizes, ctx.dim, streams)
  File "/home/.local/lib/python3.8/site-packages/torch/nn/parallel/comm.py", line 189, in scatter
    return tuple(torch._C._scatter(tensor, devices, chunk_sizes, dim, streams))
RuntimeError: Tensors of type SparseTensorImpl do not have strides

Process finished with exit code 1

How do you train the model on large graphs? Do you have any idea or code for multi-GPU training?

Hello~, the neighboring sampling method (SAGE) is effective for fast training speed under the very large graph, but I do not implement it in this version (I am trained BiGI 1-2 weeks for convergence in ML-10M.)

Thanks for your reply.

Thanks for your reply.
can you share your hardware details for training?
I have been training my data on an Nvidia-1080 GPU, with this config:
Running with the following configs:         data_dir : dataset/dblp/         weight : False         sparse : False         GNN : 1         feature_dim : 100         hidden_dim : 100         dropout : 0.3         optim : adam         lr : 0.001         lr_decay : 0.9         weight_decay : 0.0005         decay_epoch : 5         leakey : 0.1         cpu : False         cuda : True         margin : 0.3         lambda : 0.3         negative : 1         DGI : True         attention : True         struct : True         struct_rate : 0.0001         num_epoch : 500         min_neighbor : 3         batch_size : 4         log_step : 200         log : logs.txt         save_epoch : 50         save_dir : ./saved_models         id : 00         topn : 10         seed : 2040         load : False         model_file : None         save_node_feature : True         wiki : False         number_user : 350631         number_item : 161648         model_save_dir : ./saved_models/00
it costs a very long time to train (about 6-7 seconds for each batch)
Do you mind any idea for improving training speed?

Hi, I recommend using PyG or DGL projects for large-scale graph training.
By the way, you can try my latest work UniCDR in one-domain setting~ (simply modify the running command), the codes are released at here.

Thanks for your recommendations


Help with Testing

Hi, I trained a pytorch_model.bin using google colab and put it and the config.json within the trained_model folder in the gravital folder. However, when I try to do --test, it only shows the generation config and nothing is output. A little new to this, any help is appreciated.
In addition, after installing pytorch, I keep getting this warning:
UserWarning: Failed to load image Python extension: Could not find module 'C:\Users\...\AppData\Local\Programs\Python\Python310\Lib\site-packages\torchvision\image.pyd' (or one of its dependencies). Try using the full path with constructor syntax.   warn(f"Failed to load image Python extension: {e}")
Anyone know what's causing this?

Been tinkering with this over the past few days, I've found that the last for loop in main.py used for creating the output string wasn't working correctly with my model. I've edited the for loop extensively and have gotten it working. I think it may be due to how my model generates things, based on how I've formatted the data from Discord Chat Exporter.
In addition, switching to the proper Pytorch CUDA version that matches mine (11.8) worked (had to find it under the preview builds), although I get userwarnings that ShardedTensor is being depreciated for Dtensor. It doesn't seem to affect anything though and I don't feel like reverting to older versions of CUDA


trouble getting it working

Got everything built, no errors along the way. On the service side I get this error...
listening on port: 2222...
initiating stream
error initiating TCP stream: Error: Not found
error on the client socket: Error: read ECONNRESET
initiating stream
error initiating TCP stream: Error: Not found
error on the client socket: Error: read ECONNRESET
the device side looks like this, I suppose this is normal...
-> 15:45:44 CONNECT | VER: 4 | KEEPALIVE: 240 | FLAGS: 192 | USERNAME: [redact]devices.net/device/?api-version=2017-11-08-preview&DeviceClientType=iothubclient%2F1.2.12 (native%3B WindowsProduct%3A0x0000007d 6.2%3B x32) | PWD: XXXX | CLEAN: 0
<- 15:45:44 CONNACK | SESSION_PRESENT: true | RETURN_CODE: 0x0
-> 15:45:44 SUBSCRIBE | PACKET_ID: 2 | TOPIC_NAME: $iothub/streams/POST/# | QOS: 0 | TOPIC_NAME: $iothub/streams/res/# | QOS: 0
<- 15:45:44 SUBACK | PACKET_ID: 2 | RETURN_CODE: 0 | RETURN_CODE: 0
-> 15:49:44 PINGREQ
<- 15:49:44 PINGRESP
-> 15:53:44 PINGREQ
<- 15:53:44 PINGRESP
-> 15:57:44 PINGREQ
<- 15:57:44 PINGRESP
-> 16:01:44 PINGREQ
<- 16:01:44 PINGRESP
-> 16:05:44 PINGREQ
<- 16:05:44 PINGRESP
-> 16:09:44 PINGREQ
<- 16:09:44 PINGRESP
I turned off the firewalls on both devices.
Are there more detailed logs I can look at?

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

ID: c7ac2444-8b79-9e9f-144d-9b9090393189
Version Independent ID: 64433425-c8c0-2c7b-d799-f6dd676a4fc6
Content: Azure IoT Hub device streams Node.js quickstart for SSH/RDP (preview)
Content Source: articles/iot-hub/quickstart-device-streams-proxy-nodejs.md
Service: iot-hub
GitHub Login: @rezasherafat
Microsoft Alias: rezas

@gatornation11, can you confirm that your IoT Hub is in one of the supported regions? We are expanding these regions in near future.

Also, you can find your device stream logs in Azure monitor. See instructions here.

It isn’t. I’m in East.
Bummer, can I move it?
From: Reza Sherafat<EMAIL_ADDRESS>Sent: Thursday, March 7, 2019 5:00 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Eric Logeson<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] trouble getting it working (#26498)
@gatornation11https://github.com/gatornation11, can you confirm that your IoT Hub is in one of the supported regionshttps://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-device-streams-overview#regional-availability? We are expanding these regions in near future.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/26498#issuecomment-470712010, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ac61pgHOMpYuX2oXm6jZ6-W6nOba47oIks5vUYvTgaJpZM4bkMYC.

The most straightforward option is to (re-)create a new Hub in central US.


Setting connector's system properties on the executors

I'm trying to implement the advice @jacek-lewandowski gave me on this discussion #465, but I'm having some difficult time finding the right way to set a system-wide property like 'spark.cassandra.connection.keep_alive_ms' on the executors.
I think that the spark way would be to use 'spark.executor.extraJavaOptions', like this:
spark.executor.extraJavaOptions = "-Dspark.cassandra.connection.keep_alive_ms=12000"

but it's illegal in Spark to set a property that starts by "-Dspark." [1]
I also tried the following, but without effect:
spark.executorEnv.spark.cassandra.connection.keep_alive_ms=12000

What would be the recommended way to setting these system-wide properties?
(not on the spark-conf file: we run on Mesos and use specific configuration for each job)
-kr, Gerard.
[1] https://github.com/apache/spark/blob/master/core%2Fsrc%2Fmain%2Fscala%2Forg%2Fapache%2Fspark%2FSparkConf.scala#L269

Closing because it looks like this discussion concluded.
For any follow-ups, let's move to the user's list here:
https://groups.google.com/a/lists.datastax.com/forum/#!forum/spark-connector-user


SWC 110 false positive?

I analyzed a contract and the output shows there is SWC 110 but I am not able to find any relevant info.
More details : Ethereum StackExchange Post
Is it false positive?

Tool should provide a more precise location of the statement
It doesn't seem to medium severity

Hi, this is a false positive. An array is like a function which can be accessed by a call() in ethereum, so it's possible to raise an exception, so it is a false positive.


WIP

Do not delete branch because referenced here: https://twitter.com/guizmaii/status/1435437676908277764?s=20


Understanding predictive maintenance in factory operations

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

`This closes #6260

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

Upon a quick glance this article seems rather light on substance.
We prefer to publish original, practitioner-focused content that takes a deeper dive into programming-centric concepts.
@kerubo-tech

Also seems like a rather saturate topic:
https://www.google.com/search?q=Understanding+predictive+maintenance+in+factory+operations&rlz=1C1CHBF_enUS891US891&sourceid=chrome&ie=UTF-8


chore(main): release 1.1.1

:robot: I have created a release beep boop
1.1.1 (2024-08-26)
Bug Fixes

override schedule times (f18718b)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.1.1
:sunflower: