Project deployment issue

Hi there!
I wanted to deploy the code on Heroku with Flask server but it returns Application error. Please check the code and help me out on it. Please let me know if any detail is required I would be available.

Uh, what? This isn't a server. Can you post the code?

Also, Heroku can't connect to your GoPro, how do you even plan on connecting to the camera?

Yes, I know it is not server. I'm planning to deploy the code on Heroku and connect it with any instance by Shell so the front any could be any to work with. Does it will work with this adapter? Also provide me link I'll share the code. :)

I'm using python3 for it.

This API needs an active connection to the camera. In the past people have used a different IP than <IP_ADDRESS> to connect to their cameras, for instance, if they're using a WLAN repeater. But I can't see why anyone would deploy this API in a Heroku server and expect it to work.

I'm planing to deploy RTMP server instance with it for active connection. I guess it should work. If there is any other possible way please let me know.

Ok, so the RTMP server receives the live feed from a HERO7/8/MAX camera right?

Yes, so any suggestions would be appreciated. :)

Suggestions for what exactly? I'm having a hard time understanding what you're trying to achieve by deploying this API wrapper in a Heroku instance, and why it's got anything to do with the RTMP server. This API does not even support setting the RTMP URL on the GoPro, this is done via Bluetooth. Closing as your issue is irrelevant to this project.


Migrate includes config.yml

Problem
I just ran slate migrate on an existing theme and the config.yml file was not generated.
Did I miss something or am I supposed to create that file manually?

Hi @baberuth22, unfortunately it's a manual process right now - you would have to create the file manually and add it to the root of your project.
However, there's currently a PR (https://github.com/Shopify/slate-cli/pull/125) that has yet to be merged that will take care of generating this file for you.

The link is broken
https://github.com/Shopify/slate/blob/master/config-sample.yml.

This file has moved to https://github.com/Shopify/slate/blob/master/packages/slate-theme/config-sample.yml

also broken... ^


Tech :: update readme

My feedback after my vqb onboarding :)

Codecov Report

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

@@           Coverage Diff           @@
##           master     #282   +/-   ##
=======================================
  Coverage   95.96%   95.96%
=======================================
  Files          76       76
  Lines        1635     1635
  Branches      252      252
=======================================
  Hits         1569     1569
  Misses         66       66

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 d25f877...3128ba4. Read the comment docs.


Automated cherry pick of #11795

Cherry pick of #11795 on cloud.
/cc  @hmhealey
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 879363bc265c012a80c2286ca51fcbde8079ef47


example: rosettacode 100 doors

Signed-off-by: Arthur Delamare<EMAIL_ADDRESS>

ok to test

Looks like @Morriar is synchronized with me, +1 anyway !

100 doors était deja implémenté: https://github.com/privat/nit/blob/master/examples/rosettacode/doors.nit

It will be soon ready for merge (in my opinion). Finally, we can add it next to the existing implementation because it uses a different strategy. So rename the file to beers_with_classes.nit, and in the module doc explain the difference with the existing implementation.

+1

This PR fails the continous integration tests, see http://gresil.org/jenkins/job/CI github/1970/#showFailuresLink
You will need to enter the expected results from your program in the file at tests/sav/doors_with_classes.res. It can be done from the command line with ./doors_with_classes > tests/sav/doors_with_classes.res. Then add it with git.

+1 again


openssh: Rename option, old option is deprecated upstream

Motivation for this change
sshd_config man page says "ChallengeResponseAuthentication is a deprecated alias for this"
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[x] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[x] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

a changelog entry for this would be useful, and the tests that use this option should be changed to use the new one as well.

This should take care of the tests and the release notes.

LGTM, but would like another look from someone else before merging. openssh is a pretty core service after all.

needs a rebase for conflicts in the release notes, but good to go apart from that :)

Rebased to current master

@ofborg eval


O365 Template

Is this still valid?  Got everything done, except having issue with the O365 template/output.  If this is still valid, I'll put the head down and "keep trying".
Awesome work--great research and more people need to talk about ADFS, SAML 2.0, misplaced trust, etc..
Keep getting a:
AADSTS90013 error

Note: I dumped the ADFS properly (thanks). Decoded where i needed to, converted to Hex where needed. ADFSpoof with o365 gives me a great output.
It's just using Burp now with the provided output of ADFSpoof which is causing the issue.

Looks like additional request headers are now mandatory with O365.  Recommend folks do burp to follow authN flow where it works.
@dmb2168 great project, i learned a lot. This took many many more hours to figure out than I'd like to admit, but all-in-all, i'm a better person for it.
Owe you beers.

Hey, i am experiencing the same error even when following burp (replacing the token on the request to login.srf. Is there anything else that you are performing that replacing "wresult"?

Hey, I've been getting a similar error "AADSTS50107: The requested federation realm object  does not exist." If i compare legit vs forced request, the value of 'wresult' looks quite similar and the issuer is the same URL. Have you seen this error during your debugging/know a resolution?
Also in your flow, did you delete the 'wctx' parameter which is sent after 'wresult' in the POST request?


cmake: Allow to build with system libraries and other improvements

This patchset allows you to build piper-phonemize with system libraries instead of fetching sources from the internet. For this a new option has been introduced BUILD_EXTERNAL which will enable building with external sources. The default in cmake is OFF in the Makefile it is ON.
This also generates a cmake config file so you can consume the library using find_package(piper_phonemize) from other cmake files. It also fixes some issues if you use it as an external project.
You can see it building for openSUSE against system libraries here:
https://build.opensuse.org/package/show/home:gladiac/piper-phonemize

This is good and very useful! Please merge


How to use ?

Hi,
I love the idea of using the concept of PropTypes to validate any data in any js code. I briefly skimmed the code but didn't find an exposed function like check(dataToValidate, propTypes). Did I miss something ?

@DjebbZ: I will add a usage example, but it's the same as React.PropTypes. they are validator functions called in series. Will post here once it's in the README. Thanks!

function check(props, propTypes) {
	for (let prop in propTypes) {
		if (propTypes.hasOwnProperty(prop)) {
			let err = propTypes[prop](props, prop, 'name', 'prop');
			if (err) {
				console.warn(err);
				return false;
			}
		}
	}
	return true;
}

it would be much better if you expose the check function. Or I or many others may need to create our own npm modules by copying the above check function. Thanks.

I'd recommend creating a new module for that to keep things nice and separate 👍
This module is just a copy and paste of the original, so I don't want to modify it too much.


vscode liveshare cannot connect to the server

Issue Type: Bug

start a new session
get the code after login on github
F1, select "login with user code"
paste the user code

expected:
login successfully
actual:

notes:

I do not run under a proxy
this has plagued me for weeks now and survived through updates and re-installs

Extension version: 1.0.2274
VS Code version: Code 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-14T08:33:47.663Z)
OS version: Darwin x64 19.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz (16 x 2300)

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

Load (avg)
2, 2, 2

Memory (System)
32.00GB (0.07GB free)

Process Argv
-psn_0_67571821

Screen Reader
no

VM
0%

Thanks for reaching out. Can you give us more details about your scenario? Also, your logs may provide us with helpful information. You can access them via the Live Share: Export Logs command from the command palette (ctrl + shift + p) and attach them to this issue.

Sure let me do that tomorrow

"tomorrow" ended up being a very long day but here it is.
liveshare_logs.zip

We've changed our authentication service. See #4128 and if the new authentication is giving you an issue, please create a new issue. Thank you for your patience.


NTSCQT Rendering takes too long

Hello. I'm rendering video frames I know what's happening there, probably I have 8GB RAM on my laptop.

It takes 2,012.8 MB may slow down CPU Usage.

What I do now takes spend rendering more minutes?

do life

its too excessive is the main problem with this app is the best but i prefer other worse emulators

no i think it's because it's only using one of your cpu cores, we need multi-core rendering

how i could do that??!! say to my computer (katana msi i7) use all your energy in this program, seems like is a problem of that, because isnt make sense the amount of time, any oher program of my computer is faster doesnt matter the amount of work i do

ntscqt doesn't have all of the power of your cpu, if you notice here it's only using one of my 12 threads, making it very slow unfortunately.

Ah, it's been three years ago. I remember exporting video with NTSCQT that took 1-2 hours to render but don't worry it's just all fine.

no, the issue persist, maybe is me who use to render videos of 1 hour or more but its takes a whole day or more even in 720p

can x2 output by nearest neighbor increase the speed? can increase the speed if i execute as admin and i select maximun priority in task manager? what's the way to derivate more power to the program


Create a run.md file for getting output of the program

Can someone create a markdown file explaining how this programme can be tested in your local systems and on any other platforms.

Can I please work on this issue :)

@JOS-RE : Go ahead!! Any other queries do let me know!!!


Test is running in parallel but class are not running in parallel

TestNG Version
Expected behavior
Actual behavior
Is the issue reproductible on runner?

[ ] Shell
[x] Maven
[ ] Gradle
[ ] Ant
[x] Eclipse
[ ] IntelliJ
[ ] NetBeans

Test case sample
<!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd" >

<suite name="PlacementAdmin" thread-count="2" parallel="tests"
	verbose="2">

	<test name="chrome" allow-return-values="true">
		<parameter name="browser" value="chrome">
		</parameter>
		<classes>

			<class name="com.psm.portalautomation.pageObjects.LoginPage"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.AdGatewayCluster"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.Densities"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.Devices"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.DynamicFields"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.IntegrationMethods"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.InterstitialTypes"></class>

		</classes>
	</test>

<test name="firefox" allow-return-values="true">
		<parameter name="browser" value="firefox">
		</parameter>
		<classes>
			<class name="com.psm.portalautomation.pageObjects.LoginPage"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.AdGatewayCluster"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.Densities"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.Devices"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.DynamicFields"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.IntegrationMethods"></class>
			<class name="com.psm.portalautomation.PlacementAdmin.InterstitialTypes"></class>
		</classes>
	</test>

</suite>

parallel="tests": TestNG will run all the methods in the same  tag in the same thread, but each  tag will be in a separate thread. This allows you to group all your classes that are not thread safe in the same  and guarantee they will all run in the same thread while taking advantage of TestNG using as many threads as possible to run your tests.

http://testng.org/doc/documentation-main.html#parallel-tests
I think the behavior is the expected one according to your configuration.
Feel free to reopen or comment if you think it is still an issue.

@VXM97280  if you really want to execute classes in parallel, you need to set parallel="classes" instead of parallel="tests"
however to remember, tests within same class will run sequentially as well described the docs Julien had shared


Include POSTMARK_API_TOKEN

Coverage remained the same at 54.87% when pulling 6188b1b398a3880d91930955c6ac198988d33be0 on passiweinberger:patch-1 into c66e8e395d1782f71c3260a3a758e27cf104c585 on numenta:master.


build: add oxlint

Closes #1403

Hi there! Project manager for Oxlint here. I thought I'd help get you guys set up with Oxlint. This basic setup should help you get your feet wet.
I also ran all auto-fixes that Oxlint found. There are still a few warnings, but I don't want to make more code changes than necessary.

thanks, thats definetly a start! sadly oxlint doesnt seem to cover all import rules we use yet so eslint is still dog slow for me, but I am sure oxlint will get there with time!
for reference:

Rule
Time (ms)
Relative

import/default
437.140
16.8%

import/no-unresolved
405.767
15.6%

import/no-duplicates
239.231
9.2%

unicorn/no-instanceof-array
167.508
6.4%

import/order
129.030
4.9%

no-redeclare
82.844
3.2%

vue/no-template-shadow
77.470
3.0%

import/no-named-as-default-member
50.770
1.9%

import/no-named-as-default
47.304
1.8%

n/no-deprecated-api
40.657
1.6%

(altho idk what those numbers mean, since the whole command took way longer:  eslint:cli-engine Linting complete in: 85949ms +19ms)


feat: add support for Documents

Fixes #
Design discussion issue (if applicable) #
Changes
Please provide a brief description of the changes here.
Merge requirement checklist

[ ] CONTRIBUTING guidelines followed
[ ] Unit tests added/updated (if applicable)
[ ] Backwards compatability analysis completed (if applicable). "Will this change require recompilation and upload of user programs?"

Superseded by https://github.com/NillionNetwork/nada-dsl/pull/59


Please add an ID to the assignee in the work item detail page

               Assignee

Can we add an "id=" to this web UI element? Thanks!

PR #297 merged hence closing this issue.


Sub-package of type 3 was not parsed completely

I am trying to control a UR5 CB3 running Polyscope 3.10 using ur_modern_driver. When I launch ur5_bringup.launch I get the following message:
Sub-package of type 3 was not parsed completely!
I launch it with roslaunch ur_modern_driver ur5_bringup.launch robot_ip=ROBOT_IP. I am running the kinetic-devel branch for both ur_modern_driver and universal_robot.
A complete log follows:
$ roslaunch ur_modern_driver ur5_bringup.launch robot_ip:=<IP_ADDRESS>
... logging to /home/csrobot/.ros/log/658d005c-96c0-11e9-8102-04d4c40355e0/roslaunch-scooter-7363.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.

started roslaunch server http://<IP_ADDRESS>:37579/

SUMMARY
========

PARAMETERS
 * /robot_description: <?xml version="1....
 * /rosdistro: kinetic
 * /rosversion: 1.12.14
 * /ur_driver/base_frame: base
 * /ur_driver/max_joint_difference: 0.01
 * /ur_driver/max_payload: 5.0
 * /ur_driver/max_velocity: 10.0
 * /ur_driver/max_waiting_time: 2.0
 * /ur_driver/min_payload: 0.0
 * /ur_driver/prefix:
 * /ur_driver/require_activation: Never
 * /ur_driver/reverse_ip_address:
 * /ur_driver/reverse_port: 50001
 * /ur_driver/robot_ip_address: <IP_ADDRESS>
 * /ur_driver/servoj_gain: 100.0
 * /ur_driver/servoj_lookahead_time: 1.0
 * /ur_driver/servoj_time: 0.008
 * /ur_driver/servoj_time_waiting: 0.001
 * /ur_driver/shutdown_on_disconnect: True
 * /ur_driver/time_interval: 0.008
 * /ur_driver/tool_frame: tool0_controller
 * /ur_driver/use_lowbandwidth_trajectory_follower: False
 * /ur_driver/use_ros_control: False

NODES
  /
    robot_state_publisher (robot_state_publisher/robot_state_publisher)
    ur_driver (ur_modern_driver/ur_driver)

auto-starting new master
process[master]: started with pid [7378]
ROS_MASTER_URI=http://localhost:11311

setting /run_id to 658d005c-96c0-11e9-8102-04d4c40355e0
process[rosout-1]: started with pid [7391]
started core service [/rosout]
process[robot_state_publisher-2]: started with pid [7408]
process[ur_driver-3]: started with pid [7409]
[ INFO] [1561408869.898002830]: Setting up connection: <IP_ADDRESS>:30003
[ INFO] [1561408869.898329428]: Connection established for <IP_ADDRESS>:30003
[ INFO] [1561408869.898345361]: Setting up connection: <IP_ADDRESS>:30001
[ INFO] [1561408869.898662941]: Connection established for <IP_ADDRESS>:30001
[ INFO] [1561408869.952301031]: Got VersionMessage:
[ INFO] [1561408869.952323285]: project name: URControl
[ INFO] [1561408869.952332086]: version: 3.10.0
[ INFO] [1561408869.952340036]: build date: 03-05-2019, 06:39:00
[ INFO] [1561408869.952347934]: Disconnecting from <IP_ADDRESS>:30001
[ INFO] [1561408869.958821522]: ActionServer enabled
[ INFO] [1561408869.958851287]: Use standard trajectory follower
[ INFO] [1561408869.960502838]: Setting up connection: :50001
[ INFO] [1561408869.960606243]: Connection established for :50001
[ INFO] [1561408869.960663930]: Initializing ur_driver/URScript subscriber
[ INFO] [1561408869.964247017]: The ur_driver/URScript initialized
[ INFO] [1561408869.964273637]: Notifier: Pipeline disconnect will shutdown the node
[ INFO] [1561408869.969692244]: Service 'ur_driver/robot_enable' activation mode: Never
[ INFO] [1561408869.969725988]: Starting main loop
[ INFO] [1561408869.969824551]: Setting up connection: <IP_ADDRESS>:30003
[ INFO] [1561408869.969904525]: Starting pipeline RTPacket
[ INFO] [1561408869.970000586]: Starting pipeline StatePacket
[ INFO] [1561408869.970073323]: Setting up connection: <IP_ADDRESS>:30002
[ INFO] [1561408869.970199745]: Connection established for <IP_ADDRESS>:30003
[ INFO] [1561408869.970419376]: Connection established for <IP_ADDRESS>:30002
[ INFO] [1561408869.971958574]: Starting ActionServer
[ INFO] [1561408869.972097937]: Trajectory thread started
[ERROR] [1561408870.028391551]: Sub-package of type 3 was not parsed completely!
[ INFO] [1561408870.028497411]: Disconnecting from <IP_ADDRESS>:30002
[ INFO] [1561408870.028592966]: Shutting down on stopped pipeline StatePacket
*** Error in `/home/csrobot/catkin_ws/devel/lib/ur_modern_driver/ur_driver': corrupted double-linked list: 0x00000000012788c0 ***
[ur_driver-3] process has died [pid 7409, exit code -11, cmd /home/csrobot/catkin_ws/devel/lib/ur_modern_driver/ur_driver __name:=ur_driver __log:=/home/csrobot/.ros/log/658d005c-96c0-11e9-8102-04d4c40355e0/ur_driver-3.log].
log file: /home/csrobot/.ros/log/658d005c-96c0-11e9-8102-04d4c40355e0/ur_driver-3*.log

I just upgraded to 3.10 and am experiencing the same issue. Looking at the 3.10 Release notes my guess would be, that it has something to do with

Added reserved byte to Masterboard data sub package of Robot State Message.

I can confirm that downgrading to 3.9.1 worked for me.

Did you solve the issue, can we have UR5 CB3 software 3.10 working with ur_modern_driver ?

@Abduoit: use UniversalRobots/Universal_Robots_ROS_Driver for all CB3 and e-series robot controllers. See the readme there for additional requirements and setup instructions.

I'm closing this as we've officially deprecated this package.
Refer to the announcement on ROS Discourse.

Having this error again on SW version  3.14

Please see the last few comments on this issue.
You should not be using ur_modern_driver with a CB3.
Please use UniversalRobots/Universal_Robots_ROS_Driver.


Table of Contents is Duplicated

URL(s) with the issue:
Please provide a link to the documentation entry
Description of the issue:
Table of Contents is Duplicated
Submitting a pull request?
Are you willing to submit a pull request to solve this issue? (Yes/No): Yes

Yes

Assign it to me

@NnajiChimuanya - please raise a PR to 2.0.0-alpha-1 branch once updated. Thanks!

This is now updated in commit. Hence closing this issue.


Save in sentItems only

Hi all,
I have been using php-ews for nearly a year and everything works great. I am now looking to integrate with mailgun to deliver the actually email from our custom crm system, in order to get the advantage of tracking and more control over the delivery process.
So instead of sending the actually mail using php-ews, I would now like to only save a copy of the sent email the user's sent item folder. I have been able to change my code to support this when not sending with attachments, but I haven't be able to get it working with attachments.
// If we have attachments
        if (count($this->attachments)) {
            // Send and save message
            $msgSendRequest = new EWSType_SendItemType();
            $msgSendRequest->ItemIds = new EWSType_NonEmptyArrayOfBaseItemIdsType();
            $msgSendRequest->ItemIds->ItemId = $msgItemId;
            $msgSendRequest->SavedItemFolderId = new EWSType_TargetFolderIdType();
            $sentItemsFolder = new EWSType_DistinguishedFolderIdType();
            $sentItemsFolder->Id = 'sentitems';
            $msgSendRequest->SavedItemFolderId->DistinguishedFolderId = $sentItemsFolder;
            $msgSendRequest->SaveItemToFolder = true;
            $msgSendResponse = $ews->SendItem($msgSendRequest);
        } else {
            $msgRequest = new EWSType_CreateItemType();
            $msgRequest->Items = new EWSType_NonEmptyArrayOfAllItemsType();
            $msgRequest->Items->Message = $msg;
            $msgRequest->MessageDisposition = 'SaveOnly';
            $msgRequest->MessageDispositionSpecified = true;
            $msgRequest->SavedItemFolderId = new EWSType_TargetFolderIdType();
            $sentItemsFolder = new EWSType_DistinguishedFolderIdType();
            $sentItemsFolder->Id = 'sentitems';
            $msgRequest->SavedItemFolderId->DistinguishedFolderId = $sentItemsFolder;
            $msgRequest->SaveItemToFolder = true;
            $msgSendResponse = $ews->CreateItem($msgRequest);
        }

Does anyone know how to do it?

Have you tried adding $msgRequest->MessageDisposition = 'SaveOnly'; to the first if statement? So your code would become
// If we have attachments
        if (count($this->attachments)) {
            // Send and save message
            $msgSendRequest = new EWSType_SendItemType();
            $msgSendRequest->ItemIds = new EWSType_NonEmptyArrayOfBaseItemIdsType();
            $msgSendRequest->ItemIds->ItemId = $msgItemId;
            $msgSendRequest->SavedItemFolderId = new EWSType_TargetFolderIdType();
            $msgSendRequest->MessageDisposition = 'SaveOnly';
            $msgSendRequest->MessageDispositionSpecified = true;
            $sentItemsFolder = new EWSType_DistinguishedFolderIdType();
            $sentItemsFolder->Id = 'sentitems';
            $msgSendRequest->SavedItemFolderId->DistinguishedFolderId = $sentItemsFolder;
            $msgSendRequest->SaveItemToFolder = true;
            $msgSendResponse = $ews->SendItem($msgSendRequest);
        } else {
            $msgRequest = new EWSType_CreateItemType();
            $msgRequest->Items = new EWSType_NonEmptyArrayOfAllItemsType();
            $msgRequest->Items->Message = $msg;
            $msgRequest->MessageDisposition = 'SaveOnly';
            $msgRequest->MessageDispositionSpecified = true;
            $msgRequest->SavedItemFolderId = new EWSType_TargetFolderIdType();
            $sentItemsFolder = new EWSType_DistinguishedFolderIdType();
            $sentItemsFolder->Id = 'sentitems';
            $msgRequest->SavedItemFolderId->DistinguishedFolderId = $sentItemsFolder;
            $msgRequest->SaveItemToFolder = true;
            $msgSendResponse = $ews->CreateItem($msgRequest);
        }

Hi @Garethp
Yes I tried that. From what I can read from the documentation there is a different behaviour when you have attachments. Thats why SendItem is called.

Sorry, I just read up on the SendItem request. From what I've read, when you use SendItem it's because you're sending a draft, or an email that's already been saved to the server. Are you just trying to move the email to the sent folder? If that's not what you're after, what exactly are you after, and can you post the code where you create the item and it's attachments?

I've fiddled with this before, and the best I got was being able to put a copy in the Sent Items folder, but it will show as a draft since Exchange didn't send it. My particular situation was sending messages with ICS files (calendar appointments) through PHP native mail(), as a work-around to Outlook not picking it up as an appointment if sent through EWS (I had a request open on that), then making a copy in the Sent Items folder. The ICS will be attached to the Sent Items item, so this might be along your lines. I haven't found a work-around to force it to not be a draft yet, ran out of time on my particular project.
If that isn't a big deal, here is a few snippets of my code to maybe assist you. It's a module plugin for my emailing system, so its segmented a bit. Here's the essential methods.
    public function send_appointment(Email $email_obj, Appointment $appointment){

		// Global session handler
		global $session;

		// Build the email message
		$msgRequest = $this->build_email($email_obj);

		// Manually set as save-only
		$msgRequest->MessageDisposition = EWSType_MessageDispositionType::SAVE_ONLY;

		// Push to the user's sent box (otherwise ends up in drafts)
		$this->sent_items($msgRequest);

		// Create the body item
		$msgResponse = $this->ews->CreateItem($msgRequest);

		// Verify the response
		$this->verify_response($msgResponse, 'CreateItem');

		/*
		*	For now, we re-route the actual sending to the Plain handler,
		*	as Outlook is not picking up the appointment if sent through EWS
		*/
		$email_handler = new Email_Plain;

		return $email_handler->send_appointment($email_obj, $appointment);

	}

    /**
	*	Helper for pushing a message to the user's sent box
	*	@param	EWSType_SendItemType	$send_request	- Send request object
	*	@return	void
	*/
	protected function sent_items($send_request){

		// Retrieve sent item folder type
		$sentItemsFolder = new EWSType_DistinguishedFolderIdType();
		$sentItemsFolder->Id = EWSType_DistinguishedFolderIdNameType::SENT;

		// Attach to the request
		$send_request->SavedItemFolderId = new EWSType_TargetFolderIdType();
		$send_request->SavedItemFolderId->DistinguishedFolderId = $sentItemsFolder;

		// Set to save to the folder
		$send_request->SaveItemToFolder = true;

	}

Hope this helps.

@denndk @Demonslay335
Does this help? I've managed to find a way to create an item as a non-draft. It still displays slightly differently when you view it in webmail (I think it knows that it's never been sent, even if it's not a draft), but it should help. I've also written this using my own fork of the library, but it should be easy to use in the original. It was made following this example by James.
<?php

require_once "vendor/autoload.php";

use jamesiarmes\PEWS\API\Type;
use jamesiarmes\PEWS\API\Type\CalendarItem;
use jamesiarmes\PEWS\Caster;
use jamesiarmes\PEWS\Test\API\TypeTest;
use jamesiarmes\PEWS\Calendar\CalendarAPI;

$api = new \jamesiarmes\PEWS\Mail\MailAPI();
$api->buildClient(
    'cas2.multrix.com',
<EMAIL_ADDRESS>    'hLbP9HiGTZUDcVCbZJjtWjYWZgFfmtKLTrRNnVJx9RViRBFpaL',
    [
        'httpPlayback' => [
            'mode' => 'live'
        ]
    ]
);

$message = new Type\MessageType();
$message->setSubject('Test Draft');
$message->setBody('Test Draft Body');
$message->addToRecipients('blackfireweb@gmail.com');

$extended = new Type\ExtendedPropertyType();

$fieldUri = new Type\ExtendedFieldURI();
$fieldUri->setPropertyTag("0x0E07");
$fieldUri->setPropertyType(\jamesiarmes\PEWS\API\Enumeration\MapiPropertyTypeType::INTEGER);

$extended->setExtendedFieldURI($fieldUri);
$extended->setValue(1);
$api->sendMail($message);

$message->addExtendedProperty($extended);
$message->setIsRead(false);
$itemId = $api->sendMail($message, array('MessageDisposition' => 'SaveOnly'));
$api->getClient()->MoveItem(Type::buildFromArray(array(
    'ToFolderId' => array (
        'DistinguishedFolderId' => array ('Id' => 'inbox')
    ),
    'ItemIds' => array(
        'ItemId' => $itemId->toArray()
    )
)));

@Garethp
Awesome, that fixed the showing as draft issue. Somehow I never stumbled across that example before. Thanks!

Thank you for your answers.
Your code solves the issue when not sending with attachments, but problem persists when sending with attachments, as we will be using the SendItem method instead.


Factor styling out of e-g objects

Hi! Thank you for helping out with Embedded Graphics development! Please:

[ ] Check that you've added passing tests and documentation
[ ] Add a simulator example(s) where applicable
[ ] Add a CHANGELOG.md entry in the Unreleased section under the appropriate heading (Added, Fixed, etc)
[ ] Run rustfmt on the project
[ ] Run ./build.sh (Linux/macOS only) and make sure it passes. If you use Windows, check that CI passes once you've opened the PR.

PR description
This PR implements some of the changes we discussed in #162. Iterators over non styled primitives and the changes to the Transform trait aren't implemented yet. But it should be enough to get a better feel for the API.

Looking at some of the examples, Primitive::new().into_styled().translate() feels a bit weird to me - translations aren't part of styling. Does this also mean Primitive::new().translate() wont't work? I think that should work instead of .into_styled().translate() as other uses for primitives a la #156 would require extra work.
Should we allow this pattern?:
Primitive::new()
    .styled(
        Style::new().stroke_color(Some(1u8.into())
    )

Style::new() returns Style::default() which can be modified with a chain on the style itself. I'm not sure if this would actually make the API cleaner, but it would mean that style stuff is grouped nicely into the Style struct, and the same style can be copied/clone()d into various instances of primitives.
.styled() returns the same thing as .into_styled(), but allows passing of a pre-built style struct. Might be handy if a user is drawing lots of things with the same style (list menus come to mind as an example).
I'll do a more thorough review later when I have a bit of time.

Looking at some of the examples, Primitive::new().into_styled().translate() feels a bit weird to me - translations aren't part of styling. Does this also mean Primitive::new().translate() wont't work? I think that should work instead of .into_styled().translate() as other uses for primitives a la #156 would require extra work.

No both should work, as you suggested in: https://github.com/jamwaffles/embedded-graphics/issues/162#issuecomment-527360982. Transform is implemented for the primtives and Styled<T, C> where T: Transform.

Should we allow this pattern?:
Primitive::new()
    .styled(
        Style::new().stroke_color(Some(1u8.into())
    )

Yes, this was something I also considered adding.
Should we also use Styled for fonts? I would prefer to use another struct for styling text, because there was some confusion about what fill_color and stroke_color means for font rendering. foreground_color (or font_color, text_color) and background_color would be easier to understand IMO.

The latest version adds an style parameter to into_styled instead of adding a separate styled method. This works fine for styling single objects or sharing styles between multiple objects.
Code like this:
Circle::new(position, 100)
    .into_styled()
    .fill_color(FOREGROUND_COLOR)
    .draw(&mut display);

can now be written as:
Circle::new(position, 100)
    .into_styled(Style::fill(FOREGROUND_COLOR))
    .draw(&mut display);

Using styles that set a stroke and fill currently requires to build a style first and use it later:
https://github.com/rfuest/embedded-graphics/blob/2db8a5aef94f6018425a6a81fb1809648f440d5b/simulator/examples/analog-clock.rs#L83-L89
But this could be simplified by adding a StyleBuilder to keep a builder like API:
let decoration = Circle::new(decoration_position, 5)
    .into_styled(
        StyleBuilder::new()
        .fill(BinaryColor::Off)
        .stroke(BinaryColor::On, 1)
        .build()
    );

Should we also use Styled for fonts? I would prefer to use another struct for styling text, because there was some confusion about what fill_color and stroke_color means for font rendering.

Yeah that makes sense. Would the fonts also have the same .into_styled() method, but which takes a FontStyle instead of a Style? .into_styled() would have to take a generic arg in that case, right?

I haven't really thought about how font styling could be implemented. But I was currently planning to use Style and Styled exclusively for primitives.

But this could be simplified by adding a StyleBuilder to keep a builder like API:
Not sure why there's this limitation tbh. Can we remove WithStyle now that the only way of styling things is through the .into_styled() method on Primitive? If so, why not move all the methods on WithStyle into Style?

Keeping WithStyle was temporary to keep font rendering intact.
Having the methods from WithStyle implemented on Style seemed unnecessary because the fields are public anyway and there is no need for additional setter methods IMO.

Or, if mutating is preferred, stick a .clone() in:
// ...

let other_thing = Circle::new(decoration_position, 5)
    .into_styled(decoration_style.clone().stroke_width(10));

This only works if stroke_width (and the other setters) would take ownership of the Style object:
fn stroke_width(self, width: u32) -> Self { ... ; self }

But this wouldn't be as flexible as using a mutable reference for self. I think that was the reason I suggested to introduce the StyleBuilder instead.

As a general API convention, I'm starting to think if e-g should mutate by default and require .clone() for non-mutations seeing as .clone() is going to have the same cost as a copy for all e-g objects (at time of writing). I think you've mentioned this in the past but looking at all the translate/translate_mut stuff in the diff seems a bit superfluous now.
Edit: ah yes, in the linked issue no less #162 (comment) roll_eyes

At first I also wanted to change the Transform implementation in this PR, but there were some problems with this change and I decided to postpone this changes to discuss this further in another issue or PR.

I was currently planning to use Style and Styled exclusively for primitives.

Surely Styled could be reused? Style is definitely specific to the primitives but Styled could be more general, no?

Having the methods from WithStyle implemented on Style seemed unnecessary because the fields are public anyway and there is no need for additional setter methods IMO.

Ahh that's true, I don't think you explicitly mentioned going in that direction. Makes for a lighter API for sure. I don't think there's need for a StyleBuilder if we're just going with struct access.

At first I also wanted to change the Transform implementation in this PR, but there were some problems with this change and I decided to postpone this changes to discuss this further in another issue or PR.

Sounds good, my original comment on this was just a spitball.

I was currently planning to use Style and Styled exclusively for primitives.

Surely Styled could be reused? Style is definitely specific to the primitives but Styled could be more general, no?

Should we rename Style to PrimitiveStyle to distinguish it from FontStyle?
I guess Styled can be reused. The main reason to keep it separate was to have a shorter type. Styled<Circle, PrimitiveStyle<BinaryColor>> is pretty long. But it will be inferred by the compiler most of the time and doesn't need to be specified, so this might not be a problem.

Having the methods from WithStyle implemented on Style seemed unnecessary because the fields are public anyway and there is no need for additional setter methods IMO.

Ahh that's true, I don't think you explicitly mentioned going in that direction. Makes for a lighter API for sure. I don't think there's need for a StyleBuilder if we're just going with struct access.

OK. StyleBuilder could also be added later if necessary.

The primitive refactoring is ready for review. Text styling could use some more work, but this was outside the scope of this PR.
@jamwaffles Which of the new types should be included in the prelude? Style was included before, but I'm not sure if it should be.

One thing stands out to me in a real quick drive-by review:

I'm really not keen on patterns like

let mut s = PrimitiveStyle::default();
s.stroke_color = Some(BinaryColor::Off);
s.fill_color = Some(BinaryColor::On);

Could we bring back the WithStyle-like builder interface directly on PrimitiveStyle (and TextStyle too I think) and use this instead:
let s = PrimitiveStyle::new()
  .stroke_color(Some(BinaryColor::Off))
  .fill_color(Some(BinaryColor::On));

This gets rid of mut and feels a bit more Rust-like IMO. Also a couple fewer characters to type. The signatures for these fns could look like:
fn stroke_color(&mut self, stroke_color: Option<C>) -> &mut Self {
    self.stroke_color = stroke_color;

    self
}

I don't think it's worth separating those methods into a StyleBuilder. IMO it adds unnecessary noise to the API - I think it's better to have the methods under the PrimitiveStyle or TextStyle structs.

Text styling could use some more work, but this was outside the scope of this PR.

What kind of things were you thinking? Would be good to note those down somewhere. Is it worth opening an issue?

Which of the new types should be included in the prelude?

Is there anything that the user wouldn't type out that's required? Any new traits and the like? If not then yeah let's keep the new stuff out of the prelude. Can always add it later!
Quick question: I noticed a lot of changes like this:
- let object: Triangle<Gray8> =
-     Triangle::new(Point::new(5, 10), Point::new(15, 20), Point::new(5, 20));
+ let object = &Triangle::new(Point::new(5, 10), Point::new(15, 20), Point::new(5, 20))
+     .into_styled(PrimitiveStyle::stroke(Gray8::new(10), 1));

Is the reference required, or is it just some cleanup?

let s = PrimitiveStyle::new()
 .stroke_color(Some(BinaryColor::Off))
 .fill_color(Some(BinaryColor::On));

If the methods in WithStyle are implemented for mutable references this code would result in s being a reference. This was the reason to try the StyleBuilder.

This gets rid of mut and feels a bit more Rust-like IMO.

Is chaining setter methods Rust-like? Do you have any other examples of this being used (except for builders)?

Text styling could use some more work, but this was outside the scope of this PR.

What kind of things were you thinking? Would be good to note those down somewhere. Is it worth opening an issue?

I'm not quite sure yet and need to look at the text rendering code in more detail first.
But adding a Text struct could help the distinguish between fonts and text drawables. This way Stylable could also be used for text rendering.
struct Text<'a>{
   text: &'a str,
   position: Point,
};

Quick question: I noticed a lot of changes like this:
- let object: Triangle<Gray8> =
-     Triangle::new(Point::new(5, 10), Point::new(15, 20), Point::new(5, 20));
+ let object = &Triangle::new(Point::new(5, 10), Point::new(15, 20), Point::new(5, 20))
+     .into_styled(PrimitiveStyle::stroke(Gray8::new(10), 1));

Is the reference required, or is it just some cleanup?

Ah, that was something I forgot to discuss before.
IntoIterator is only implemented for &Styled<...> anymore, it was previously implemented for e.g. &Rectangle and Rectangle. Is this Rust-like? For example IntoIterator isn't implemented for arrays but only for slices.

Which of the new types should be included in the prelude?

Is there anything that the user wouldn't type out that's required? Any new traits and the like? If not then yeah let's keep the new stuff out of the prelude. Can always add it later!

The Primitive trait provides the into_styled method now, so it should be in the prelude IMO. But I've removed all crate::style::* types from the prelude.

If the methods in WithStyle are implemented for mutable references this code would result in s being a reference.

Sorry yeah not &mut - good point. Was thinking this instead (with just mut):
pub fn fill_color(mut self, fill_color: Option<C>) -> Self {
    self.fill_color = fill_color;

    self
}

I tried this quickly in examples/analog-clock.rs like this:
let decoration_style = PrimitiveStyle::default()
    .stroke_color(Some(BinaryColor::On))
    .fill_color(Some(BinaryColor::Off));

which seems to work ok. Am I missing other patterns?

Is chaining setter methods Rust-like? Do you have any other examples of this being used (except for builders)?

Not setters, no. I feel like PrimitiveStyle is a builder, or at least presents a builder-like interface which is why I said it's Rust-like. I see how they could also look like setter methods as well, however I think in a lot of use cases it will be used as a builder, with multiple chained methods to set a style up. This is why I think having a builder-like interface on PrimitiveStyle is preferable.

Ah, that was something I forgot to discuss before.
IntoIterator is only implemented for &Styled<...> anymore, it was previously implemented for e.g. &Rectangle and Rectangle. Is this Rust-like? For example IntoIterator isn't implemented for arrays but only for slices.

Ah right I see. I'm not sure what the right Rust idiom is here tbh. Whichever is less difficult to use I guess? Let's leave it just for &Styled and fill in any gaps later.

The Primitive trait provides the into_styled method now, so it should be in the prelude IMO.

I agree :)

If the methods in WithStyle are implemented for mutable references this code would result in s being a reference.

Sorry yeah not &mut - good point. Was thinking this instead (with just mut):
pub fn fill_color(mut self, fill_color: Option<C>) -> Self {
    self.fill_color = fill_color;

    self
}

I tried this quickly in examples/analog-clock.rs like this:
let decoration_style = PrimitiveStyle::default()
    .stroke_color(Some(BinaryColor::On))
    .fill_color(Some(BinaryColor::Off));

which seems to work ok. Am I missing other patterns?

This works, but you can't use these methods anymore if you only have access to a mutable reference. I'm not sure how often this would be a problem.

Is chaining setter methods Rust-like? Do you have any other examples of this being used (except for builders)?

Not setters, no. I feel like PrimitiveStyle is a builder, or at least presents a builder-like interface which is why I said it's Rust-like. I see how they could also look like setter methods as well, however I think in a lot of use cases it will be used as a builder, with multiple chained methods to set a style up. This is why I think having a builder-like interface on PrimitiveStyle is preferable.

I'm not a fan of the almost a builder pattern. I agree that a builder pattern is probably the nicest way of constructing a style. But having one struct that is used as a builder and as an object that represents a style leads to some slightly confusing naming IMO. For a builder I would expect to be able to set the fill color using style.fill_color(color), but for any other object style.fill_color() would be a getter. (see https://doc.rust-lang.org/1.0.0/style/style/naming/README.html#getter/setter-methods-[rfc-344])

This works, but you can't use these methods anymore if you only have access to a mutable reference. I'm not sure how often this would be a problem.

Just refreshed my memory and I see your point there. Ok, let's leave Style as is for this PR and think about builders later. Good point on the getter/setter thing as well.
I'm wondering if it's a good idea to recommend this method of creating styles in the examples/docs:
let style = Style {
    stroke_color: Some(BinaryColor::On),
    ..Style::default()
}

That feels better than creating a bunch of let muts. Seeing as this is code people will probably copypasta quite a bit, I think it's reasonably important to set a clean precedent. This can be replaced with a builder when we figure that out. The other option is to just leave the let mut pattern in for now. Which would you prefer?

Yes, that's a better way to build these styles, I'll change that.
PrimitiveStyle has two constructors PrimtiveStyle::stroke and PrimitiveStyle::fill. The names were chosen to be short, but they don't follow the naming convention. Should they be renamed to PrimitiveStyle::with_stroke and PrimitiveStyle::with_style like the API guidelines suggests? And if they are renamed, should TextStyle::new be renamed to TextStyle::with_text_color to match, even if there currently is only one constructor?
When a style is constructed using PrimitiveStyle::stroke the width must explicitly specified, but PrimitiveStyle::default sets a default stroke width of 1 px. I would prefer if both variants would behave similar by either assuming the 1px default width or require setting the stroke width explicitly:
// 1. current implementation

let _s1 = PrimitiveStyle::stroke(color, 1);
let _s1 = PrimitiveStyle {
    stroke_color: Some(color),
    //setting stroke_width is not necessary because default is 1
    ..PrimitiveStyle::default()
};
let _s10 = PrimitiveStyle::stroke(color, 10);
let _s10 = PrimitiveStyle {
    stroke_color: Some(color),
    stroke_width: 10,
    ..PrimitiveStyle::default()
};

// 2. width is specified explicitly

let _s1 = PrimitiveStyle::stroke(color, 1);
let _s1 = PrimitiveStyle {
    stroke_color: Some(color),
    stroke_width: 1,
    ..PrimitiveStyle::default()
};
let _s10 = PrimitiveStyle::stroke(color, 10);
let _s10 = PrimitiveStyle {
    stroke_color: Some(color),
    stroke_width: 10,
    ..PrimitiveStyle::default()
};

// 3. width has a 1px default value

let _s1 = PrimitiveStyle::stroke(color);
let _s1 = PrimitiveStyle {
    stroke_color: Some(color),
    ..PrimitiveStyle::default()
};
let mut _s10 = PrimitiveStyle::stroke(color);
_s10.stroke_width = 10;
let _s10 = PrimitiveStyle {
    stroke_color: Some(color),
    stroke_width: 10,
    ..PrimitiveStyle::default()
};

I would prefer variant 2.

I would prefer variant 2.

Looks good. Does PrimitiveStyle::default() then give a stroke width of 0 as the default then?

Should they be renamed to PrimitiveStyle::with_stroke and PrimitiveStyle::with_style like the API guidelines suggests?

Makes sense to me :slightly_smiling_face:

And if they are renamed, should TextStyle::new be renamed to TextStyle::with_text_color to match, even if there currently is only one constructor?

Yeah I think so. It keeps consistency with PrimitiveStyle and I imagine there'll be TextStyle::with_background_color added at some point, right?

Looks good. Does PrimitiveStyle::default() then give a stroke width of 0 as the default then?

Yes.

Yeah I think so. It keeps consistency with PrimitiveStyle and I imagine there'll be TextStyle::with_background_color added at some point, right?

The with_* constructors were intended for common use cases and I don't think that rendering invisible transparent text with opaque background is very common. Or did you have a special use case for that constructor in mind?
But I would like to add additional PrimitiveStyle::new constructors which behave the same way as PrimitiveStyle::default. They can be const fn and are expected to exist by most users.

Yes.

Cool. More breaking changes for the changelog. Worth adding some doc comments about this behaviour too.

Or did you have a special use case for that constructor in mind?

Good point, no I don't :grin:

But I would like to add additional PrimitiveStyle::new constructors which behave the same way as PrimitiveStyle::default. They can be const fn and are expected to exist by most users.

ConstructorS plural? Can you implement them in a commit and show me what you mean?

Cool. More breaking changes for the changelog. Worth adding some doc comments about this behaviour too.

It is a breaking change, but I think the 1px default width behavior was never documented.

ConstructorS plural? Can you implement them in a commit and show me what you mean?

I meant PrimitiveStyle::new and TextStyle::new.

It is a breaking change, but I think the 1px default width behavior was never documented.

True, but it's worth mentioning that now a shape with a stroke colour but no set width won't render any pixels.

I meant PrimitiveStyle::new and TextStyle::new.

Ah I see :ok_hand:

It's a shame we can't make PrimitiveStyle::with_stroke(), etc, const yet. Can you open an issue to track when these can be const-ified when #![feature(const_fn)] is stabilised?

I noticed that at least the constructors for primitive can be const now, because they don't have a type parameter anymore.


Options setting not work

Hi, thanks for the project first.
I want to set Coin-hive options (EX: threads、throttle)
And I tried with coinhive documentation example ,but it looks not working to set options in  Coin-hive
var miner = await CoinHive('my siteKey', {
        threads: 2,
        throttle: 0.8
    });

Can you give me some opinion? Thanks!

Same here.

Благодарю за письмо.Иван

if you try to write the js code inside the same html and you tell me
On Wed, May 23, 2018 at 1:37 PM, Kenneth Palaganas<EMAIL_ADDRESS>
wrote:

I actually have the same issue. Any updates?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cazala/coin-hive/issues/85#issuecomment-391435197,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AhjFMLOHxy3JVhoH_Aj_Sazn6dOWztlpks5t1Z5zgaJpZM4RRiV8
.

looks like cpu continues on 100% load because it spin up a server and a puppeter

Благодарю за письмо.Иван


OS X: sync-summary file summary.txt is hidden and cannot be viewed

Visual Studio Code Version : [ 1.4.0 ]
Code Sync Settings Version : [ 2.2.3 ]
Operating System : [ OSX 10.11.6 ]
The following screen shot shows that the summary.txt file is missing from the list of open editors, yet 1 unsaved file is listed (which is the hidden summary.txt, saving of which is prompted for on exiting VSCode):

Note that this does not happen in the Windows version of VSCode.

I only have Windows machines actually so i was unable to test on OSX, it would be great if you can download the code and debug it and send me a pull request solving this problem- i will be more happy to accept that.

Sounds good, I'll see what I can do.

2.2.5 version contains (#101)[https://github.com/shanalikhan/code-settings-sync/issues/101] fixed that was generating after summary page generation. would you please test its generating the summary now

Sorry, but the file still doesn't show up as an open editor on OS X.

i have pushed an update 2.2.6.
Can you paste the log on upload and download from the developer console.
For example i have log showing as
 FILE URI For Summary Page : C:\Users\shana\AppData\Roaming\Code\User\summary.txt
Whats your URI

Thanks for looking into this - getting closer:

[Extension Host] FILE URI For Summary Page : \Users\mklement\Library\Application Support\Code\User\summary.txt

threadService.ts:217[Extension Host] cannot open untitled:%5CUsers%5Cjdoe%5CLibrary%5CApplication%20Support%5CCode%5CUser%5Csummary.txt. Detail: expected URI untitled:%5CUsers%5Cjdoet%5CLibrary%5CApplication%20Support%5CCode%5CUser%5Csummary.txt BUT GOT untitled:/Users/jdoe/Library/Application%20Support/Code/User/summary.txt(anonymous function) @ threadService.ts:217

Here's a URI that would work:
 file:/Users/jdoe/Library/Application%20Support/Code/User/summary.txt

On a side note:

Is it possible to mark the open editor as "clean", so that on closing the user isn't asked to save the summary file?

Perhaps you can make do without creating a file at all?

Following the code causing problem actually :
i'm replacing the / with \ in loop and its working fine in Windows.
But i'm unable to debug this in other platform.
The console log showing right format of URI
But when i try to parse the string in URI it shows error.
 var tempURI: string = this.en.APP_SUMMARY;

        while (tempURI.indexOf("/") > -1) {
            tempURI = tempURI.replace("/", "\\");
        }

        console.log("FILE URI For Summary Page : " + tempURI);

        var setting: vscode.Uri = vscode.Uri.parse("untitled:" + tempURI);   //Problem here

        vscode.workspace.openTextDocument(setting).then((a: vscode.TextDocument) ....

I've submitted a PR (#107) that bypasses the issue by using an actual file for the sync-summary information.
Some asides:
I don't know what the official recommendations are, but placing your extension's files directly in the application-support folder, along built-in files such as keybindings.json, could be problematic longer-term with respect to potential name collisions and clutter.
--
The original problem stemmed from the untitled: URI scheme inexplicably accepting only platform-specific path separators (\ on Window, / on OS X / Linux).
This would have worked:
vscode.Uri.parse("untitled:" + tempURI.replace(/[\\/]/g, this.en.OsType === OsType.Windows? '\\' : '/'));

--
If you use regular expressions with <string>.replace(), you can apply options such as g (global replacement - all occurrences) and i (case-insensitivity) to the search term.
Thus,
    while (tempURI.indexOf("/") > -1) {
        tempURI = tempURI.replace("/", "\\");
    }

can be replaced with simply
 tempURI = tempURI.replace(/\//g, "\\");

I have pushed an update 2.2.7 fixing this issue.


Setstate issue with VA ?

Hello,
On my way to enhance the interaction between VA and EDDI in my Elite personnality, I'm currently stuck with the "setstate" command in VA.
I'm trying to  transfert a Boolean variable called "infoautosys" from VA to EDDI.
I'have read the wiki page concerning VA, and it seems not to work. Or, more realisticly, I do not understand clearly what to do...
In VA :
Set Boolean [Infoautosys] to False
Set Text [State variable] to 'Infoautosys'
Execute external plugin, 'EDDI 2.3.0' -> (Plugin Context set to "setstate"  and Text Variables set to "State variable")
Set Text [script] to 'Testvariable'
Execute external plugin, 'EDDI 2.3.0' -> (Plugin Context set to "speech"  and Text Variables set to "script")

In EDDI :
I try to check the value of infoautosys with this script :
{(infoautosys)}
{(state.infoautosys)}

{P(infoautosys)}
{P(state.infoautosys)}

And it seems that "infoautosys" is not set...
I guess that I've missed something, but I juste made like the Wiki says (Or not ?).

As far as I can tell you have it mostly correct.  The wiki is a little outdated, because I have found you don't need to set the variable when you execute the plugin anymore, just setting the context is enough.
However, your problem is because you have set it to False, so nothing is spoken!  Try setting it to True and test again. ;)  I just tried this as you have it (without setting the variable in the 'execute external command') and it works perfectly.
To get a raw value spoken, use:
[CODE]{dump(state.infoautosys)}[/CODE]
This will tell you when a value is 'Not set' (void) or if it is 'False', 'True', or any other set variable type.
Hope that helps! :)

Thank you, I will test that ASAP.

@Pryss Let's keep the issues list to just issues that are unresolved. If this issue is resolved, would you mind closing the issue?


Fix type error in Redux Essentials

name: "Fix type error in Redux Essentials"
about: Update documentation content as per #3812
Checklist

[x] Is there an existing issue for this PR?

#3812

[x] Have the files been linted and formatted?

What docs page needs to be fixed?

Showing Single Posts:
Redux Essentials, Part 4: Using Redux Data:

What is the problem?
This fixes a type error in the documentation by updating the IDs in the initial store data to be strings, not numbers. This removes the need for .toString() methods.
What changes does this PR make to fix the problem?
Removes the .toString() method from Part 4 of the tutorial and updated the initial store data in Part 3 of the tutorial to use strings for IDs.

Thanks!


To run the program on a MacBook, need to follow these steps？

poetry install：finish
poetry shell: finish
python build.py: finish
sbt-image、sbt-video:  not found

Im following the From Source instructions which doesnt require the build script, but im having the same issue of sbt-video not found


Smarter auto-complete for common packages #5869

We've had the same feedback from the guys who work on the DS portion of the python extension.

If it's snippets specifically, that's something that would go into the extension: https://github.com/microsoft/vscode-python/blob/master/snippets/python.json


Interest activity

Hello guys, I want you to look at the activities on the Nav drawer and give me views, just want  feedback not merging. Lol.

I am closing this commit, out dummy api is here Lol! forgot to gitignore.


Extend FeatureExportExtension interface

It would be helpful to add two methods somehow like exportStart() and exportComplete() into the interface FeatureExportExtension. For example, if we write a plugin to do post-process and log the processed object into a separate CSV file, we could create a file writer within the method  exportStart() and close the writer in exportComplete().
https://github.com/3dcitydb/importer-exporter/blob/master/impexp-core/src/main/java/org/citydb/core/plugin/extension/exporter/FeatureExportExtension.java#L35

Yes, sounds like a good idea. I guess the interface would also need to handle cases where the export is aborted by the user or even terminates abnormally. Because if you open resources when the export starts, you should close them even if the export process fails.

True, nice remark.

fixed with #200


Using external files with --config option - does it work?

I'm confused about how this is meant to work.
I am setting a path to an external config file with a --config command line argument e.g.
--config=/etc/path/to/config/file.conf

I can see this is initially working, in that it is setting a property within cosmiconfig's options variable in index.js to the correct value:
options.configPath = path.resolve(parsedCliArgs[options.argv]);

But, how are you mean to actually load that file? Because when you call the load() method, there's no need to specify searchPath (because the location is fixed), and you can't specify configPath (because only cosmiconfig knows what it got from the command line argument). And you have to specify one or the other, or load() just resolves with null.
I have been through the code and I can't see anywhere that actually uses the .configPath property. So even if I do something like
conf.load(process.cwd())

It's presumably just going to behave in the standard way, and ignore the --config setting.
I must be missing something obvious!
On a probably unrelated note, putting --config aside for the moment.  Are you running into an error that you don't understand? Are there some lines of code that don't make sense to you?

Why can't you just call load() with no arguments and have it default to process.cwd()?

That would be a feature request. I don't think it's really necessary, because you can always just pass in process.cwd(), right?

That would be a feature request. I don't think it's really necessary, because you can always just pass in process.cwd(), right?

That's clearly a matter of opinion, but it sounds like you are arguing that defaults are never required 'because you can always just pass in' a value. I think many others would agree that sensible default behaviour is quite important.
With regard to the original issue. Perhaps I wasn't clear. Suppose I have an app which successfully uses cosmiconfig to read it's configuration files using the normal structure. But now, I would like the app to optionally use a configuration file (if specified on the command line). So if it's present on the command line, the --config filename takes precedence. My question is, can cosmiconfig handle this without any changes to the application? It definitely parses the command line and extracts the specified filename. But it doesn't seem to do anything with it? It still loads any existing file, just like if you didn't specify --config at all.

It still loads any existing file, just like if you didn't specify --config at all.

@steve-p-com: If this is the case, that sounds like a bug. If you can specify the situation more exactly or create a failing test, we could try to debug it.

it sounds like you are arguing that defaults are never required

@steve-p-com: Sorry I didn't explain myself well. I'm not sure that process.cwd() is a sensible default. Right now the project I use cosmiconfig for is stylelint, and would not want this default. Sounds like you would. It would be great to gather other opinions. But until I have a strong sense that a default is in fact sensible, I tend to lean towards being explicit rather than implicit.

But it doesn't seem to do anything with it? It still loads any existing file, just like if you didn't specify --config at all.

@steve-p-com : I think the bug you spotted is fixed by https://github.com/davidtheclark/cosmiconfig/pull/68. Please re-open if not.


REQUEST: New approver role for CodeMonkeyLeet in dapr/components-contrib

GitHub Username
@CodeMonkeyLeet sponsored by @artursouza
Requirements

[x] I have reviewed the community membership guidelines (https://github.com/dapr/community/blob/master/community-membership.md)
[x] I have enabled 2FA on my GitHub account. See https://github.com/settings/security
[x] I have subscribed to the Dapr community Discord server
[x] I am contributing (any of the following apply: issues, discussions, PRs, reviews) to 1 or more Dapr repositories

List of contributions to the Dapr project

PRs reviewed / authored

Component fixes:

https://github.com/dapr/components-contrib/pull/1069
https://github.com/dapr/components-contrib/pull/1015
https://github.com/dapr/components-contrib/pull/1009
https://github.com/dapr/components-contrib/pull/1052
https://github.com/dapr/components-contrib/pull/1093

Fundamentals and testing:

https://github.com/dapr/components-contrib/pull/1047
https://github.com/dapr/components-contrib/pull/1064
https://github.com/dapr/components-contrib/pull/1050
https://github.com/dapr/components-contrib/pull/1044
https://github.com/dapr/components-contrib/pull/1040
https://github.com/dapr/components-contrib/pull/1002
https://github.com/dapr/components-contrib/pull/1001
https://github.com/dapr/components-contrib/pull/975

Developer/contributor experience:

https://github.com/dapr/components-contrib/pull/936
https://github.com/dapr/components-contrib/pull/917

Code reviews:

https://github.com/dapr/components-contrib/pull/1078
https://github.com/dapr/components-contrib/pull/1079
https://github.com/dapr/components-contrib/pull/1055
https://github.com/dapr/components-contrib/pull/1008
https://github.com/dapr/components-contrib/pull/944

Issues responded to

Opened, in-flight:

https://github.com/dapr/components-contrib/issues/1053
https://github.com/dapr/components-contrib/issues/1051
https://github.com/dapr/components-contrib/issues/1007
https://github.com/dapr/components-contrib/issues/1046

Design discussions:

https://github.com/dapr/components-contrib/issues/964
https://github.com/dapr/components-contrib/issues/949
https://github.com/dapr/components-contrib/issues/951
https://github.com/dapr/components-contrib/issues/928
https://github.com/dapr/components-contrib/issues/1084

Troubleshooting:

https://github.com/dapr/components-contrib/issues/1085
https://github.com/dapr/components-contrib/issues/1094
https://github.com/dapr/components-contrib/issues/1092

I support this request.

/cc @yaron2 @msfussell

I support, request approved.


Crashes whenever joining server

Describe the Bug
game disconnects whenever joining server.
Reproduction Steps

i dont know

...
Expected Result
a
Screenshots and Videos
No response
Crash Report or Log
https://pastebin.com/LwgkafMZ
Operating System
Windows 11
Mod Version
0.5.0f
Minecraft Version
1.19.2
Other Mods
No response
Additional Context
No response

i am having this issue too, if anyone can fix this it would be nice. OP should have added more information but the log itself is enough.

Describe the Bug
game disconnects whenever joining server.
Reproduction Steps
1. i dont know

...
Expected Result
a
Screenshots and Videos
No response
Crash Report or Log
https://pastebin.com/LwgkafMZ
Operating System
Windows 11
Mod Version
0.5.0f
Minecraft Version
1.19.2
Other Mods
No response
Additional Context
No response

that not the crash log that just the console log....
go in the crash-reports folder and get the crash log from there not the console log

Your versions of Create on the client and server do not match. Due to changes in contraption data, they need to.

Your versions of Create on the client and server do not match. Due to changes in contraption data, they need to.

I do not have create installed on the server and it still causes issues

Reference in new i

also why would you make it crash if it's a different version, very bad design imo

I do not have create installed on the server and it still causes issues

Well if you have it on the client and not server, then of course it won't work right.
If you have it on neither, then this is not a Create issue.

also why would you make it crash if it's a different version, very bad design imo

The crash is not intentional. It's an unavoidable side effect of completely changing how contraption data is sent.

so basicly you are just saying that if I want to play a server that I need to go manually disable the mod because the developers can't make the program right?

I do not have create installed on the server and it still causes issues

Well if you have it on the client and not server, then of course it won't work right. If you have it on neither, then this is not a Create issue.

also why would you make it crash if it's a different version, very bad design imo

The crash is not intentional. It's an unavoidable side effect of completely changing how contraption data is sent.

why does it need to change that tho

so basicly you are just saying that if I want to play a server that I need to go manually disable

...yes? this is the case with all content mods. Things will not work correctly if you leave them on. If your game hasn't crashed from it, you've been lucky. No modder will support a desynchronized state.

because the developers can't make the program right?

If you think you can do it better, PRs are accepted.

why does it need to change that tho

because the old way caused crashes.

so basicly you are just saying that if I want to play a server that I need to go manually disable

...yes? this is the case with all content mods. Things will not work correctly if you leave them on. If your game hasn't crashed from it, you've been lucky. No modder will support a desynchronized state.

because the developers can't make the program right?

If you think you can do it better, PRs are accepted.

why does it need to change that tho

because the old way caused crashes.

never in my experience of using mods have I ran into issues relating to the server not having them installed
I will not do better, because java is not my main language and this would be too big of a project for me to drastically change.


Fix /simple in dummy app

This makes the model return an Ember.Array and only
conditionally sets styles.

@stefanpenner @mmun i think this is ready to go.

I don't have a good working knowledge of the code yet. Can you explain why the guard is justified? Also, please squash.

@mmun this.contentElement is undefined until didInsertElement runs, but the bounds are being calculated as soon as attrs are set, which seems weird to me. What's happening is that it's attempting to set the width and height before the element is in the dom.

I'm seeing this with master right now as well. Not sure when this happened... my last time working with this code the dummy app was working, now its broken.

@raytiley i think it's a missing call to _super in one of the lifecycle hooks. maybe. maybe not.


Excuse me, i have a problem.

Excuse me, why is this result like this？

 Can you give me some help .I first encountered no MSAF model,after modification encountered checkpint not right.


add night mode css

create a new css file for night mode and make a basic night mode for home page

@ravisaxena23 can I work on this issue?

sure


wlroots and sway tries to use a virtual terminal when XDG_SEAT != seat0

Hi
It seems that when starting on seat1 (instead of seat0)
wlroots tries to use a virtual terminal
Virtual Terminals (TTYs) only work on seat0 , (and not on any other seat)
Thanks

Can you give us a bit more context on the issue? What are you trying to accomplish, how is it failing, and what is the expected result?

Sorry I wasn't clear.
If I configure a second seat, by adding devices to it with udev (like a second card, and a few input devices), and then I start a session with logind, trying to start wlroots-sway fails because there is no TTY, and only logind seat0 has TTYs...
Basically I think it should only try to use a TTY if XDG_SEAT=seat0
This will allow a second instance of Sway running on the same computer like Zaphod mode.
I am going to try something...

I think this might be what would get it to work
wlroots.txt
I'm having an issue starting wlroots fully on QEMU as I think I might be having a Mesa issue (or it could be a wlroots issue)
But with that, I do get the exact same errors now starting on both seats


Update metadata.avdl

Changing to Biosample from Sample, as per discussion https://github.com/ga4gh/schemas/pull/391 and metadata call from 2015-08-27.

+1
(presuming my vote counts! Biosample is more consistently used within academia hence my vote)

I agree with change to Biosample

+1

Biosample it is; merging. Thanks!


Update whisper stt model description to GA - overview.md

Removed "preview" from the model overview of whisper STT.

#sign-off


make combined files working with version! issue #270

#270

Thanks for this. One thing I noticed is that you're passing an array of files to mix.version(). But your PR doesn't support that, right?

@JeffreyWay Uh, the array passed was not used. We should still use mix.version() to generate versioned files. It was to show they can be versioned successfully, nothing else.

Okay this is merged in now. Thanks!

@forehalo thank you so much for this helpful addition!  @JeffreyWay the current npm version of<EMAIL_ADDRESS>does not support this feature yet. Are there any plans to be included in<EMAIL_ADDRESS>

Yes - it'll be tagged once I do some more testing.

it still not combine compiled version css. For example
.sass('admin/core.scss', 'public/css/admin/core.css')
.combine([
'public/css/admin/extra.css',
'public/css/admin/core.css'
], 'public/css/admin/all.css'
).version()

I've got an error - file not founded.
(because public/css/admin/core.css has versioned name like core.asdase242.css)

@nejtr0n the same here, the PR has not yet been accepted so patience :stuck_out_tongue: However, until the issue is solved, I think that @JeffreyWay should consider changing the docs where it is implied that versionning applies  to combined files too

Is this fix available in "laravel-mix": "^0.8.1", Sorry It's not clear if this fix is already available as I am facing the same issue.

@iraklisg The PR has been accepted, so no changes to the docs are needed.
https://www.dropbox.com/s/jrz4ri46k5cr2sv/Screenshot 2017-02-19 15.23.48.png?dl=0

@nejtr0n You're getting an error because public/css/admin/core.css doesn't exist. You requested versioning, so it create the hashed version of that file. Then, you're trying to combine that file, but, again, because it doesn't exist, it fails.
Instead, use a regular expression.
.sass('admin/core.scss', 'public/css/admin/core.css')
.combine([
    'public/css/admin/extra.css',
    'public/css/admin/core.*.css'
], 'public/css/admin/all.css')
.version()

Thank you! Didn't know about ability of regexp in combine =)


Diff highlights wrong lines

Issue Type: Bug
New lines: 4 and 5. Highlighted: 6 and 7 (see screenshot)

@shikhov The diff algorithm ignores trim whitespace by default. You can press this button to toggle that trim whitespace should not be ignored:

Ignoring trim whitespace means that lines with different indentation, but with equal content are considered equal by the diff algorithm, i.e.:
else

is equal to
    else


Can't add repo key to debian 9

System info:
cat /etc/debian_version

9.0

By instruction:
https://clickhouse.yandex/#quick-start
sudo apt-key adv --keyserver keyserver.ubuntu.com --recv E0C56BD4    # optional
Executing: /tmp/apt-key-gpghome.lrhU7iBOSk/gpg.1.sh --keyserver keyserver.ubuntu.com --recv E0C56BD4
gpg: failed to start the dirmngr '/usr/bin/dirmngr': No such file or directory
gpg: connecting dirmngr at '/tmp/apt-key-gpghome.lrhU7iBOSk/S.dirmngr' failed: No such file or directory
gpg: keyserver receive failed: No dirmngr

sudo apt update
...
W: GPG error: http://repo.yandex.ru/clickhouse/deb/stable main/ Release: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY C8F1E19FE0C56BD4
E: The repository 'http://repo.yandex.ru/clickhouse/deb/stable main/ Release' is not signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

sudo apt-get install -y clickhouse-server clickhouse-client
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following NEW packages will be installed:
  clickhouse-client clickhouse-server
0 upgraded, 2 newly installed, 0 to remove and 131 not upgraded.
Need to get 0 B/36.2 kB of archives.
After this operation, 131 kB of additional disk space will be used.
WARNING: The following packages cannot be authenticated!
  clickhouse-client clickhouse-server
E: There were unauthenticated packages and -y was used without --allow-unauthenticated

Try to install the package dirmngr:
sudo apt-get install dirmngr

then run
sudo apt-key adv --keyserver keyserver.ubuntu.com --recv E0C56BD4

Yes, it's help and debian not accept install without --allow-unauthenticated


Can't access my saved brews

Clicking on my name in the top right corner does nothing but bring me to the same welcome example page thing with all the instructions on it.

Can you right-click your username and select "copy link address", then paste it, what do you get?
Also, when you last logged in, did you put your username as an email address or just a name? I remember that was an issue people were having a while back where it would still log you in, but your profile was broken.

I'm having the same problem. I can't access my brews and aren't given a url link to my saved brews. The copied link sends me to the same start page.

I would ask the same of you: Can you right-click your username and select "copy link address", then paste it here?

Closing. Likely duplicate of #807


Use an appropriate hyperlink colour

Thanks for highlighting this deficiency.
I think this is something that should be defined in the one of the style files, rather than in the document itself, probably beamerthemeusyd.sty.
I think using the \hypersetup command should configure the options appropriately.
In my opinion using usydred for  hyperlinks fits more with the overall theme, although usydblue is a more familiar colour for hyperlinks.

I'm okay with usydred being used for hyperlinks... but I tend to use
usydred for highlighting rather than linking.
I don't claim to be any expert in beamer theming or its conventions.

Awesome looks great. Thanks.


fix #5: update BouncyCastle.Cryptography to 2.4.0

Thank you for the PR! However, I can update the nuget when I am back from vacation - which won't be until November 2024. In the meantime, it should be possible to reference the newer version of BouncyCastle in your host app and override the older version from this nuget.


Custom Plugins Example

Looking into plugging in autoprefixer while waiting for user agent support in the prefixer for server side rendering.
However, some examples in regards to plugin configuration would be useful.

Hey Robert, where did you expect to find this information? Chances are if
you didn't see it right away someone else will have the same problem.
On Tue, Sep 22, 2015 at 6:33 AM Robert Nagy<EMAIL_ADDRESS>wrote:

Closed #363 https://github.com/FormidableLabs/radium/issues/363.
—
Reply to this email directly or view it on GitHub
https://github.com/FormidableLabs/radium/issues/363#event-415852598.

There is an example for config.matchMedia however config.plugins is empty. We should probably add something like.
import Radium from 'radium'
import { Plugins } from 'radium'

const myCustomPlugin = ({componentName, style}) => {
  return style
}

export default function style(component) {
  return Radium({
    plugins: [
      Plugins.mergeStyleArray,
      Plugins.checkProps,
      Plugins.resolveMediaQueries,
      Plugins.resolveInteractionStyles,
      Plugins.prefix,
      Plugins.checkProps,
      myCustomPlugin
    ]
  })(component)
}

Yeah you're totally right, I forgot to add that. We'll get the docs updated asap. Your custom plugin should probably go between prefixer and checkProps, btw.


[Feature] Enable citation autocomplete on mobile by adding citation dictionary export type

Debug log ID
N/A
What happened?
Autocomplete of citations on mobile devices is a very welcome feature for those of us who use our iPads for writing as well as reading. However, there is a lack of software support for this. Obsidian user u/Feralflora has developed a workflow using Better Bibtex and Obsidian to accomplish this:
Mobile citation autocompletion guide using Quickadd and Various Complements
This workflow involves outputting a BibTeX file then processing it using an Obsidian QuickAdd template to produce a citation dictionary which Various Complements accesses on mobile. This is a real advance for those of us using our iPads to write. The workflow can be partially automated by using Better Bibtex's automatic export feature, but the QuickAdd template still has to be manually run whenever the database changes.
A more elegant solution would be for Better Bibtex to output the citation dictionary directly. The custom dictionary format is very simple:
`The data should be in a format like this:
inserted text, description, alias
In our case, we want to insert the citekey, and use the title as both description and alias, like so:
citekey, title, title
Yes, title twice. Why? The first instance of title is only for display in the dropdown, while the second one is a searchable alias. With this format, we can search for and autocomplete citekeys using either the citekey or the title.`
Would it be possible to add this feature to Better Bibtex? It is likely that other mobile software could be configured to use a citation dictionary, although the format might vary. A full implementation of this might use a template or GUI to configure the dictionary output, but a quick and dirty implementation to start would also be nice :-)

Isn't this something that can be automated on the obsidian side? I'm particularly unkeen about doing any gui work for this, I really don't like UI work, and especially not in the transition period from Zotero 6 to 7.

I think ignore/delete the GUI comment - that could always be a future request as it is certainly not necessary. The format could just be hardcoded as it would suit most users.
Also, this request should not take precedence over Z7 work!
It may be possible to automate on the Obsidian side but:

At present the dictionary conversion runs on desktop, what if the user goes straight to Obsidian mobile without using Obsidian desktop
There would need to be something constantly running to check for a changed bibtex file
It ignores other platforms (although you could argue that this doesn’t matter if no-one has identified anything specific)

So I think it is more elegant to have it located in BB as BB has the automatic export code. Perhaps see if anyone else is interested in this feature though?

3. It ignores other platforms (although you could argue that this doesn’t matter if no-one has identified anything specific)

Hi, just chipping in since this is about the script I made. Regarding this point ☝  Without a way to customize the format of the citation dictionary to autocomplete from, this is only really relevant to Obsidian users, or even more specifically, Various complements plugin users, since the citation dictionary is made according to the syntax of this plugin.
Anyway, automating the running of this script (at least on Obsidian upstart) is something that should be fairly simple to achieve in Obsidian, and something I intend to work on given the uptake of this workflow.
However, as is, this feature request is still valuable, since the script can only run in a desktop environment, as mentioned here:

At present the dictionary conversion runs on desktop, what if the user goes straight to Obsidian mobile without using Obsidian desktop

So of course, having a direct BBT option would simplify things. But again, this is very platform specific. Nevertheless, there are very few options that allow bibtex / pandoc citations on iPads (I've looked hard), so this workflow has value in that light.
The limiting factor of the script is the fs dependency, which could perhaps be eliminated by using the Obsidian API instead, but I am not sure. If so, the script could also run on mobile devices.

At present the dictionary conversion runs on desktop

OK, but so does Zotero + BBT. Something is running on the desktop in any case.
I'm trying to think of something that would generalize, because I do not want to go the path of adding bespoke translators + auto-export. I can't run most Zotero translators as auto-exports, because auto-exports happen in a limited emulation of the environment Zotero translators expect. Plus there would have to be changes to the prefs GUI to manage them. I'm open to lending a hand here, I just haven't found the right solution yet.

OK, but so does Zotero + BBT. Something is running on the desktop in any case.

Right, so not running on mobile is only really a problem if you:

add new sources to Zotero / make changes that change a citekey
open Zotero, which allows an auto-export to occur
but then don't open Obsidian, so the citekey autocompletion dictionary is not updated
Try to insert a citation about the newly added item on the iPad

Of course, since the script is just javascript, it could be run independently of Obsidian, which could also circumvent the problem. Either running on a timer, or based on file changes to the bibliography. I don't have experience with makefiles, but perhaps that is an option?

I'm trying to think of something that would generalize, because I do not want to go the path of adding bespoke translators + auto-export.

Doesn't seem like you could generalize this without that path, as far as I can tell.

I'm not too worried about the path, just the part that generates the content.

I'm not too worried about the path, just the part that generates the content.

I'm not sure what you mean by that, given what you wrote before, which seems to conflict with this sentiment. But anyway, I've documented a simple way to automatically update the citation dictionary every time Obsidian opens in the linked forum post.

Yeah I think you're right. I'd still need to do GUI work for management of the auto-exports, and it would require something special in the export dialog.


Fix the Quiz Section

You can select multiple options at a time in the quiz. Actually, you can select all of them if you want.

If you select all of them ya more than one in select genres question the music tab will not be generated.

please assign me this I would like to work on this. I have already come up with a solution.

Cool, @snahashisk Go ahead!


socks5: slightly improve error reporting

This is the start of proper socks5 error reporting. There is some
followup needed, but its pretty invasive, so wanted to get at least the
base merged. The full thing is in
https://github.com/howardjohn/ztunnel/tree/socks5/report-error and I
will keep working on it and followup.
End result, using a Go client when we have a DNS lookup error:
Before: socks connect tcp <IP_ADDRESS>:15080->echo.ecs.local:8080: dial tcp <IP_ADDRESS>:15080: connect: connection refused
After: socks connect tcp <IP_ADDRESS>:15080->echo.ecs.local:8080: unknown error host unreachable

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
c1f14fc356d338b4d3b1d9a33e34970e3440da4c
link
true
/test test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.


Dual-Switch does NOT work anymore

Dual-Switch does NOT work anymore:

What version of Windows are you running? Maybe a recent update broke it?
I've just tested on my machine - and it's working as expected. Changes when quick switching and switching via the menu

yo try to delete the config file in %AppData/Local/AudioSwitcher. Resolved the issue for me when the switching was stuck ;)

just tried, didn't help

confirming it also doesnt work on the 21H1 win10 version.. is author planning to do something about it? there wasnt an update for 3 years.. thanks

Reinstalled windows 21H2 and still happening.. Is this project really dead? Or what is happening with v2? Thx

bump, same issue


GitHub action set-output command deprecated

To fix:

Update action-gh-release with PR: https://github.com/softprops/action-gh-release/pull/266

Will wait on the PR to update

Hi! I saw this referenced in the PR to fix the other deprecation warning, so just wanted to plug you can use the vanilla gh CLI client too which I suspect will be very stable. Here's an example:
https://github.com/rubyatscale/use_packwerk/blob/main/.github/workflows/cd.yml

@alexevanczuk Thanks for your remark. I changed my code to use the gh client now - it is way better than depending on 3rd party actions :)

Changes have been implemented.


when built in debug mode, actually require all modules to ensure things execute correctly on import

addressed by https://github.com/3p3r/cdk-web/pull/141


Add an example for different budget for different disruption reasons

Issue #, if available:
Description of changes:

Adding an example of different disruption budget for different disruption reasons.

I came across this after upgrading in v1 and it's quite useful as we needed to keep the disruption quite strict to limit blast radius of situation like AMI update/EKS upgrade, and we noticed that it affected the consolidation activity.
with this now we are allowed to consolidate more efficiently while keeping the strict policy for update.
Let me know if it makes sense or if it's not that useful feel free to close it.
Thank you
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for submitting this contribution! Its great to hear how these blueprints / Karpenter v1 can a help solve real world challenges.
For users to fully benefit from your example could you share how this was tested and how we could replicate this. Although we are adding this as an example to the README.md, it would be great for users confidence to know how to validate the configuration. Thanks again.

I have recently applied this new nodepool configuration in production (after finishing v1 upgrade).
apiVersion: karpenter.sh/v1
kind: NodePool
metadata:
  name: multiple-consolidations
spec:
  disruption:
    budgets:
    - nodes: "1"
      reasons:
      - Drifted
    - duration: 14m0s
      nodes: "0"
      reasons:
      - Drifted
      schedule: '*/15 * * * *'
    - nodes: "3"
      reasons:
      - Empty
      - Underutilized
    - duration: 9m0s
      nodes: "0"
      reasons:
      - Empty
      - Underutilized
      schedule: '*/10 * * * *'
    consolidateAfter: 5m0s
    consolidationPolicy: WhenEmptyOrUnderutilized
  template:
    spec:
      expireAfter: 720h
      nodeClassRef:
        group: karpenter.k8s.aws
        kind: EC2NodeClass
        name: default
      requirements:
      - key: karpenter.k8s.aws/instance-memory
        operator: Gt
        values:
        - "16000"
      - key: karpenter.k8s.aws/instance-category
        operator: In
        values:
        - r
        - m
        - c
      - key: karpenter.k8s.aws/instance-generation
        operator: Gt
        values:
        - "4"
      - key: karpenter.sh/capacity-type
        operator: In
        values:
        - on-demand
      - key: kubernetes.io/arch
        operator: In
        values:
        - amd64
        - arm64
      - key: topology.kubernetes.io/zone
        operator: In
        values:
        - eu-west-1a
        - eu-west-1b
        - eu-west-1c
      - key: kubernetes.io/os
        operator: In
        values:
        - linux
  weight: 1

(redacted some details such as taint, and selector)
This is the metrics showing that it behaves as needed. the metric being used is karpenter_nodepools_allowed_disruptions

green line is drifted reason which is 1 node every 15 minutes (start acting for example minutes 44, 59)
read and blue represents  underutilised and empty which is 3 every 10 minutes starting for example 39, 49, 59
Let me know if you need me to share these in the doc somehow or anything needed.
Thank you

@InsomniaCoder first of all, THANK YOU so much for not just letting us know these blueprints have been useful for you, but also for making contributions as well, you rock! I have a few recommendations about this:

Can you please make this part of the "Multipe Budgets" section, and move the "Multiple Budgets" section after the "Reasons" section? That way we can keep a consistent order and going deeper every time.
As Jake suggested (and you already answered), it would be really helpful if you can incorporate what you described here, specially to show the results others will see by having this configuration in place.
Can you please break down each budget, you're already doing it partially but it was a bit hard for me to follow along. Maybe you can explain the four scenarios, then show the NodePool config, and then the results.
Can you also either add a note or directly make it explicit that the budget config will "in a given time frame, at most x nodes can be disrupting at a given moment".
Let's see how long it ends up being this blueprint, maybe it will be worth it to actually have a dedicated blueprint for this and tested (following Jake's recommendation).

We think this contributions the blueprint will end up being even more awesome :)


Is it possible to download files to google drive directly without downloading to our server

Hi , i have this question in my mind and i searched  so much about this but there is nothing related to my queries -
can we able to download the files directly to google drive after the auth done without downloading them to our server , i believe there are google api support url download , so is it possible to cut the server downloading and uploading process by directly uploading them from the original server ,

I don't see how that's possible. Who will send the file to google if not the server?

Yeah we need the server but  file is in external server and downloading that and  storing to our server  and again  uploading that to  gdrive is long process and a high bandwidth occurrs ,,when we initiate a download on external server we get the link right  using that link we usually download that to our server so rather than that can we sent it to gdrive  ,

So, you want GDrive to download file from the server? (AKAIK, GDrive
doesn't provide any such feature)
Even if it did, how'll that save bandwidth? It'll still cost you the same
as uploading to drive.
On Wed 24 Oct, 2018, 8:04 PM Arjun Ram V S<EMAIL_ADDRESS>wrote:

Yeah we need the server but file is in external server and downloading
that and storing to our server and again uploading that to gdrive is long
process and a high bandwidth occurrs ,,when we initiate a download on
external server we get the link right using that link we usually download
that to our server so rather than that can we sent it to gdrive ,
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Mrigank11/embetacloud/issues/51#issuecomment-432682558,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQF00ZANUTFcjiFUr1qO73TmZDoRF-8Uks5uoHp3gaJpZM4X3kYj
.

External server to gdrive .not downloading to our server and reupload to gdrive , there is a urlfetch in google drive API don't knoe how does it work

there is a urlfetch in google drive API

Any links?

https://developers.google.com/drive/api/v2/reference/files/get
https://github.com/J05HI/google-drive-direct-remote-upload?files=1
https://developers.google.com/apps-script/reference/url-fetch/url-fetch-app this is from app script

https://developers.google.com/drive/api/v2/reference/files/get

:x: It's for getting files from GDrive, not uploading.

https://github.com/J05HI/google-drive-direct-remote-upload?files=1

:x: He's just streaming to save space. You won't save any bandwidth.

https://developers.google.com/apps-script/reference/url-fetch/url-fetch-app

:x: Irrelevant in our use case. It's for Google apps script.
Please read carefully before posting.
I'm gonna close this.

Ok thanks.
Look like it is not possible yet ,because in cloud-torrent also this features are still pending ,hope some scripts will come in future ,


Guidance on OpenCL related packages

I would like to streamline the installation of OpenCL related SW by adding a few packages. Initially I only wanted to add the clMathLibraries projects, but because AMD recently completely removed their OpenCL SDK, I'm going overboard and want to also build the OpenCL SDK from source.
Every OpenCL SDK consist of the canonical header files which can be found on the Khronos GitHub repo, as well as the sources for the canonical ICD loader which is a system library that will load the implementations. Luckily, the Khronos ICD implementation builds using CMake.
The clMathLibraries (clFFT, clBLAS, clRNG, clSPARSE) all build using CMake as well.
What are the buzzwords I should be looking for if:

The ICD sources don't use any versioning in the GiHub repo. No tags, nothing. How should I fetch the sources if I don't know the checksum of a totally volatile repo?
The OpenCL ICD must be a dynamic library. How can I forbid it being built as a static library?
Is there some special treatment for those high quality packages that create Package Config scripts when built?

Edit: the AMD APP SDK did not vanish, they just removed all links pointing to the SDK from developer.amd.com. Already notified them and they will fix it some time in the future. Still, I'll go through with building the minimalist SDK as well, so users won't have to rely on AMD APP SDK, Intel OpenCL SDK, or Nvidia CUDA SDK, all of which are 200+GB in size solely for grabbing the 8 headers and OpenCL.lib.

Thanks for the input. Should anyone else be interested, these are the repos I'm trying to integrate in some sensible manner:

Khronos/OpenCL-Headers
Khronos/OpenCL-CLHPP
Khronos/OpenCL-ICD-Loader
clMathLibraries/clFFT
clMathLibraries/clBLAS
clMathLibraries/clSPARSE
clMathLibraries/clRNG

The first step is to package the headers and the ICD loader together in a manner that clMath libraries (and anything that uses find_package(OpenCL)) can actually use.
I'll try to make use of features wherever possible: for eg. imbue FFTW support into clFFT (handcrafted assembly code when using CPU devices instead of generic OpenCL implementations, in case FFTW is used as a dependency).

I think I mostly got the hang of submitting packages. Always poke around other ports and steal a few tips and tricks. :)


Include storybook decorators in the screenshots

Currently loki ignores the global decorators when capturing screenshots. Make loki include the global decorators in the screenshots.
Reason: button outline is truncated.

Seems like we do not really need it.