System.Net.WebRequest in beta 4?

Is there a workaround for getting System.Net.WebRequest functionality in beta 4?
code:
 System.Net.HttpWebRequest objrequest = (System.Net.HttpWebRequest)System.Net.WebRequest.Create(post_url);

results in:
Severity	Code	Description	Project	File	Line
Error	CS0234	The type or namespace name 'HttpWebRequest' does not exist in the namespace 'System.Net' (are you missing an assembly reference?)	  (The WebRequest is for posting credit card transactions to Authorize.net)
Thanks,
Todd
project.json:
{
  "webroot": "wwwroot",
  "userSecretsId": "aspnet5-TSICV5-6aec2867-e934-4174-9f5f-d9c6befae354",
  "version": "1.0.0-*",

  "dependencies": {
    "EntityFramework.SqlServer": "7.0.0-beta4",
    "EntityFramework.Commands": "7.0.0-beta4",
    "Microsoft.AspNet.Mvc": "6.0.0-beta4",
    "Microsoft.AspNet.Mvc.TagHelpers": "6.0.0-beta4",
    "Microsoft.AspNet.Authentication.Cookies": "1.0.0-beta4",
    "Microsoft.AspNet.Authentication.Facebook": "1.0.0-beta4",
    "Microsoft.AspNet.Authentication.Google": "1.0.0-beta4",
    "Microsoft.AspNet.Authentication.MicrosoftAccount": "1.0.0-beta4",
    "Microsoft.AspNet.Authentication.Twitter": "1.0.0-beta4",
    "Microsoft.AspNet.Diagnostics": "1.0.0-beta4",
    "Microsoft.AspNet.Diagnostics.Entity": "7.0.0-beta4",
    "Microsoft.AspNet.Identity.EntityFramework": "3.0.0-beta4",
    "Microsoft.AspNet.Server.IIS": "1.0.0-beta4",
    "Microsoft.AspNet.Server.WebListener": "1.0.0-beta4",
    "Microsoft.AspNet.Session": "1.0.0-beta4",
    "Microsoft.AspNet.StaticFiles": "1.0.0-beta4",
    "Microsoft.AspNet.Tooling.Razor": "1.0.0-beta4",
    "Microsoft.Framework.ConfigurationModel.Json": "1.0.0-beta4",
    "Microsoft.Framework.ConfigurationModel.UserSecrets": "1.0.0-beta4",
    "Microsoft.Framework.CodeGenerators.Mvc": "1.0.0-beta4",
    "Microsoft.Framework.Logging": "1.0.0-beta4",
    "Microsoft.Framework.Logging.Console": "1.0.0-beta4",
    "Microsoft.VisualStudio.Web.BrowserLink.Loader": "14.0.0-beta4"
  },

  "commands": {
    "web": "Microsoft.AspNet.Hosting --server Microsoft.AspNet.Server.WebListener --server.urls http://localhost:5000",
    "gen": "Microsoft.Framework.CodeGeneration",
    "ef": "EntityFramework.Commands"
  },

  "frameworks": {
    "dnx451": {
    },
    "dnxcore50": {
    }
  },

  "exclude": [
    "wwwroot",
    "node_modules",
    "bower_components"
  ],
  "publishExclude": [
    "node_modules",
    "bower_components",
    "**.xproj",
    "**.user",
    "**.vspscc"
  ],
  "scripts": {
    "postrestore": [ "npm install", "bower install" ],
    "prepare": [ "gulp copy" ]
  }
}

You can use System.Net.Http.HttpClient, which I think is generally the preferred HTTP client on .NET.

Thanks for responding Eilon, will do.
T
On Mon, Jun 29, 2015 at 10:01 AM, Eilon Lipton<EMAIL_ADDRESS>wrote:

You can use System.Net.Http.HttpClient, which I think is generally the
preferred HTTP client on .NET.
—
Reply to this email directly or view it on GitHub
https://github.com/aspnet/Mvc/issues/2758#issuecomment-116761852.

Quick question.
I'm wrestling a bit with HttpClient, not for MVC reasons dependency
reasons, but Authorize.net reasons
One option I have is using Authorize.nets library from nuget
(AuthorizeNet), but as expected I get DNX Core 5.0 compiler complaints.
Can you point me in the right direction on how to incorporate asp.net 4.5
libraries into an MVC 6.  I've tried the usual "dependencies" in
project.json
"frameworks": {
"dnx451": {
},
"dnxcore50": {
}
},
but I can't get it to work.  I'm sure this an understanding deficit on my
part and I just need a point in the right direction on how to incorporate
4.5 libraries
Thanks,
T
On Tue, Jun 30, 2015 at 9:05 AM, Eilon Lipton<EMAIL_ADDRESS>wrote:

Closed #2758 https://github.com/aspnet/Mvc/issues/2758.
—
Reply to this email directly or view it on GitHub
https://github.com/aspnet/Mvc/issues/2758#event-344395125.

You use dnx451 if you want to run on .NET Framework 4.5.1, and dnxcore50 if you want to run on .NET Core. Or of course you can target both at once (which is what the project templates do).
If you're using a library that works only on .NET 4.x, you'll need to remove the dnxcore50 from the project.json.

One free round of golf for you in Sedona AZ!
That's a gold mine tip, thanks!
T
On Tue, Jun 30, 2015 at 9:36 AM, Eilon Lipton<EMAIL_ADDRESS>wrote:

You use dnx451 if you want to run on .NET Framework 4.5.1, and dnxcore50
if you want to run on .NET Core. Or of course you can target both at once
(which is what the project templates do).
If you're using a library that works only on .NET 4.x, you'll need to
remove the dnxcore50 from the project.json.
—
Reply to this email directly or view it on GitHub
https://github.com/aspnet/Mvc/issues/2758#issuecomment-117247989.

It's hot enough in the Seattle area already, thanks :smile:

@Eilon Were you able to get the Authorize.Net Library working with .NET Core?

@adamreed90 I am not familiar with that library. Can you contact its authors to see what they have planned?

Tagged wrong person I'm sorry.  @toddtsic


anomaly_test

can I use a lot of picture doing the anomaly_test rather than one picture

One of the limitation of this model, AnoGAN.
In test time, AnoGAN can process one image per a inference time.


Module 3 : XRAY code not well explained

The lab explains how to solve the performance issue, but it does not explain how to implement XRAY code
https://github.com/aws-samples/aws-serverless-workshops/tree/master/DevOps/3_XRay#1-fix-code-bug-1

Thanks for your feedback.  We've update the DevOps Serverless Workshop with new content: https://cicd.serverlessworkshops.io/


Replace cmds with PowerShell

[Enter feedback here]
The section below:
sc config SQLBROWSER start= auto
After it is enabled, run the following command from the command prompt to start the service:
net start SQLBROWSER
should be:
Set-service sqlbrowser -StartupType Auto
Start-service sqlbrowser
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7aa839a1-df21-b566-b798-bf17159c4c74
Version Independent ID: 97d386f4-c657-eaba-a62c-404ced145cb2
Content: Install SQL Server on Server Core - SQL Server
Content Source: docs/database-engine/install-windows/install-sql-server-on-server-core.md
Product: sql
Technology: install
GitHub Login: @cawrites
Microsoft Alias: chadam

@MarcoEnxuto -- Marco, thank you for your feedback.  I am looking into this article.

I see its still not updated, is there any issue using PowerShell to invoke those cmdlets?

@cawrites -- Cheryl, please review this change in private PR 19185.

@MarcoEnxuto thank you for contacting Microsoft. Your change was reviewed and should appear in the next content refresh.


windows10 host build Hypervisor-101-in-Rust setup

1、add proc-macro2 = "1.0.66" content in the project root directory *.toml
2、add use proc_macro2 as _; content to xtest/main.rs
3、compile is successful

Thanks for reporting the issue. Resolved with b3f60f37257db8e77112b4ffbb436458f022cae6 by updating the dependencies.

Thanks for reporting the issue. Resolved with b3f60f3 by updating the dependencies.

This is what I should do, do you have any good ways to improve the compilation speed in rust? There seems to be little information in this area

I am not sure as I have not felt the need of optimization. This project clean builds for debug in about 11 seconds on my primary environment, which is fine to me.

I am not sure as I have not felt the need of optimization. This project clean builds for debug in about 11 seconds on my primary environment, which is fine to me.

Through my efforts, I found some tools that can reduce the compilation time, which is very helpful during the development of rust.
https://github.com/mozilla/sccache
https://www.msys2.org/
[build]
rustc-wrapper = "C:/Users/botao/.cargo/bin/sccache.exe"
[target.x86_64-unknown-linux-gnu]
rustflags = ["-C", "linker=C:/msys64/mingw64/bin/ld.lld.exe"]


Remove linting trees from build

This was posed as an RFC to ember-cli and merged (but not implemented.
The idea is that we have standard ways to lint and it complicates (and
slows) our build, so add an yarn test command that includes ember test && yarn run lint:js && yarn run lint:template
For reference the RFC is here:
https://github.com/sangm/rfcs/blob/master/active/0000-remove-ember-cli-eslint.md

This PR necessitates changes to the blueprint in glimmer.js repo.  Those changes need to be:

[ ] to include the eslint and ember-template-lint packages as dev dependencies
[ ] to add package.json scripts to run:

lint:js -> eslint .
lint:template -> ember-template-lint .
Have yarn test -> ember test && yarn run lint:js && yarn run lint:template

@rwjblue I think likely this is a breaking change.  Users will need to lint on their own.  New blueprinted applications won't be a problem, but existing users will need to know to expect a change.


Add support for Linux ARM

Thanks for the contribution, but like #107, this is blocked on https://github.com/electron/electron/issues/2343. Also the tests are probably going to fail.

Superceded by #474.


Reset presence lock automatically

The motivation behind presence lock is to keep the light from going out if there is not much movement in an area. However, if forgotten It will keep the lights on indefinitely.
Discover how it could be reset automatically:

based on time (after x minutes/hours)
x hours after last presence was detected
on the next day (sunrise)

Additionally it should be disabled if:

sleep mode is turned on
lights are turned off manually (?)

Perhaps in addition or instead of this, there should be a delay when turning off binary_sensor.area_presence_{area_name} after all motion sensors clear.

Perhaps in addition or instead of this, there should be a delay when turning off binary_sensor.area_presence_{area_name} after all motion sensors clear.

Yep. I have solved that by configuring the "occupancy timeout" of my motion sensors. Takes a bit of time to find a good value based on amount and placement of sensors, though. (Or if that's not possible in one's setup it should be possible to create a template sensor - but I haven't tried that yet).
I'd like to keep the logic and configuration options to a minimum. Preferably it would not be a time-based trigger but based on some user interaction (if possible).

IMO it's impossible to get a functionality like this right for everyone (or even most people). As a consequence I have decided not to implement this as a feature of Auto Areas. It is always possible to create an automation which would fit the user's use-case better.


[WIP] Disallow certain markdown characters in stream and user names.

fix for issue #6534.
step 1:  disallowing the following four characters from stream names and userprofile names: *, #, @, and `

I haven't read much of the Casper codebase to be able to review. (A LGTM from @umairwaheed would be more solid here.)

@rht @umairwaheed Here I have finished the task of disallowing the special characters in the new entries of stream and user names. Also written frontent tests for that.
I need help with migrations here so as to change all the existing names with special characters where we replace them with spaces.
I have written this function, which when run in ./manage.py shell, we remove all the existing special chars from db.
def remove_special_chars_from_streamname():
    for entry in Stream.objects.all():
       stream_name = entry.name
       if(set(stream_name).intersection(Stream.NAME_INVALID_CHARS)):
          for i in Stream.NAME_INVALID_CHARS:
             stream_name = stream_name.replace(i,' ')
       entry.name = entry.name.replace(entry.name, stream_name, 1)
       entry.save()

I am confused about to proceed with writting function in the models file, as this is just an external script.

On the Travis note, there is an err at https://travis-ci.org/zulip/zulip/jobs/277274952#L2156-L2164. This might be solved w/ a rebase.

@umairwaheed Can you review the changes?

@rheaparekh, reviewed.

Hold on. I think it was my fault. Checking the code. Yeah, so we should not delete this line:
143         casper.fill('form#add_new_subscription', {stream_name: '  '});

I previously thought that we were creating a stream with a blank name again but on looking at the code again it became clear that this field is used for filtering/searching. Sorry for this confusion.

Thanks for all your work on this, @rheaparekh! I've pushed an updated version to https://github.com/rishig/zulip/tree/review-6591; it would be great if you could review and force push that to this branch.
It has a few changes:

I changed the commit message of the first commit
Some minor stylistic changes (e.g. if list(set(..)) can just be if set(..)), and I removed 1 or 2 newlines
I replaced the deduplication code in the migration with the code I suggested since the existing code as written didn't handle the "name is now longer than 60 characters" case.
I added a backend test for the invalid characters (surprised the test coverage didn't ask for it)

As followup, it would be great to move the check for bad characters from "when you click Create" to "on keypress in the Stream Name input box".
(You can ignore the IRC followup.)
@timabbott I did not review this carefully, but skimmed the code and did a quick manual test. But I think ready for review.

@rishig Thank you so much for your review! I'll keep in mind about adding backend tests the next time. I'll also get started on the followup!

@showell Can you review this?

merged, thanks @rheaparekh!  We'll want to keep our eyes out for Casper flakes here.


Minor updates to openthread API signatures

Can someone give travis a little nudge here? This succeeded on my branch.

Nudged :)


Test PR

!test

!test

!test

!test NOVA_URL=https://github.com/lurk-lab/Nova NOVA_COMMIT=ea4f75c225cb29f523780858ec84f1ff51c229bc NOVA_TEST_NAME=solidity_compatibility_e2e_pasta


Update README.md

Added Codecov badge

Codecov Report
All modified lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:loudspeaker: Thoughts on this report? Let us know!.


[Feature] Async returns

:wave: I really like abstractions but I'm missing an async return, probably a channel, that would send results as they've been processed. So far the pools return all or none results. Is this something that would fit this library?

Hi @jeschkies! I think it's unlikely that a feature like that would be added to this library since it's focused on scoped concurrency. In particular, if we add facilities for async returns, there needs to be another goroutine outside the pool reading from the channel or there is an opening for deadlocks. IMO, this is too easy to misuse to add as a first-class feature of the library, but it should be easy enough to build this functionality on top of conc.
For example:
func main() {
	p := pool.New()

	ch := make(chan int)

	// Spawn a single reader that reads 10 items
	p.Go(func() {
		for i := 0; i < 10; i++ {
			println(<-ch)
		}
	})

	// Spawn 10 writers
	for i := 0; i < 10; i++ {
		i := i
		p.Go(func() {
			ch <- i
		})
	}

	p.Wait()
}

it's focused on scoped concurrency

That's a fair point.


Add Open Source Resources

Pull Request Template
Please ensure all items below are checked before creating a pull request:

[x] My changes were made in the resources folder, not in the auto-generated README.md file or db folder
[x] My submission is formatted according to the guidelines in the contributing guide
[x] My submission is ordered alphabetically based on the resource name
[x] My submission has a useful description
[ ] I have searched the repository for any relevant issues or pull requests

Thank you!


Minimal OCert counter increments in Praos

In TPraos, we require that the operational certificate counters are monotonically increasing:
https://github.com/input-output-hk/cardano-ledger/blob/030c3b12f128f22b9d721a31b6b5ae1b75211d68/libs/cardano-protocol-tpraos/src/Cardano/Protocol/TPraos/Rules/OCert.hs#L111
In Praos, we will additionally require that the certificate counters increase by at most one:
m == n || m + 1 == n ?! InvalidCounterOCERT m n

Note that Praos is implemented in the ouroboros-network repo.

closed by https://github.com/input-output-hk/ouroboros-network/pull/3595


Forking

Hi @ronnnnnnnnnnnnn ! I loved your work on the etekcity scale and am trying to adapt it to work with the Vitafit scale. Hope you don't mind my forking the repo. If you'd rather expand your repo's scope beyond etekcity to include Vitafit (and other future scales) I'd be happy to contribute to yours.

Hi @prabhjotsbhatia-ca! I don't mind at all, it's cool. I'm glad you like it and that you find it helpful as a starting point to your repo. Let me know if you've got any questions or if there's any way I can assist.


[Compute] Pointless error message in vm identity assign

As #17619
Command Name
az vm identity assign
If cloud profile is old, CLI will report a NoneType error
'NoneType' object is not callable
Traceback (most recent call last):
python3.8/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 654, in execute
    raise ex
cli/core/commands/__init__.py, ln 718, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 711, in _run_job
    six.reraise(*sys.exc_info())
...
cli/command_modules/vm/custom.py, ln 648, in assign_vm_identity
    assign_identity_helper(cmd.cli_ctx, getter, setter, identity_role=identity_role_id, identity_scope=identity_scope)
cli/core/commands/arm.py, ln 1057, in assign_identity
    resource = setter(resource)
cli/command_modules/vm/custom.py, ln 642, in setter
    vm.identity.user_assigned_identities[identity] = VirtualMachineIdentityUserAssignedIdentitiesValue()
TypeError: 'NoneType' object is not callable

compute

#17619

#14287


Add docs for new SVM free chunks of disk

Update HAPI documentation
Add SVM for free chunks and description

Signed-off-by: Rachel Heaton<EMAIL_ADDRESS>

Amber is out this week so we don't need her to review.

Thanks, @rheaton. Will take a look.

@kbakas-piv We took a look and made a few changes.
We'll hold off on merging until we get the okay from you

I was just able to review them. Please go ahead and merge @mlimonczenko


Fix case on pgbouncer=true param

Describe this PR
Fix the case of the param pgbouncer in page "Connection management".
It must be lowercased, this page currently has it with a uppercased B.
All other pages have already the correct lowercased version.
Changes

Fix case on param pgbouncer in page "Connection management"

Thanks for catching this! 🙏


add enhancement reroute

add the possibility to use resizing like in standard ComfyUI, by placing the cursor to the bottom right corner of the node and resize it by dragging and dropping.
and in your implementation there are no additional menus :

as well as a severe size limitation :

Thank you so much for your custom nodes

There was good reason for no labels and no resizing at the time (specifically because labels don't show in the multi-directions the node allows), but with a bit more work we can probably add them... and Done.
There's now both a "Allow/No Resizing" menu item, and a "Show/Hide Title Label" menu item on right click (grouped with the connection layout, static sizes, etc).
Unfortunately, this is only on the dev branch for now at  https://github.com/rgthree/rgthree-comfy/tree/dev which has a ton more features coming but requires some optimization in comfyui itself. Once https://github.com/comfyanonymous/ComfyUI/issues/1502 is fixed and/or https://github.com/comfyanonymous/ComfyUI/issues/1502 is merged, then this can land in rgthree/rgthree-comfy main.

if it is relevant, I would like to ask for 1 more point that I may not have emphasized as a good practice.
in a normal reroute you can separately enable show title and separately enable the show type label . your version is already almost perfect both in terms of functionality and end-user friendliness.
Maybe this part (show type automatically indicates the type of the connected line instead of title) is missing for full delight.
huge sps for your hard work, the major dev20 update is excellent.


update with testing url

Checklist

[ ] Circle CI is passing (code compiles and passes tests).
[ ] There are no breaking changes to public API.
[ ] New test cases have been added where appropriate.
[ ] All new code has been commented with doc blocks ///.

@vapor-bot

@vapor-bot

@vapor-bot

@vapor-bot

@vapor-bot

@vapor-bot

@vapor-bot

@vapor-bot test performance

@vapor-bot test performance

@vapor-bot

@vapor-bot

@vapor-bot

@vapor-bot

@vapor-bot

@vapor-bot

@vapor-bot test performance

hello

hello mr bot?


Highlighting selected values in the drop-down list

Hello, i'm using element-plus type of your plugin, and found small bug, when you manually select any value in dropdown it's highlighting, but, when i just open page and i have already pre defined value, this value not highlighted in dropdown menu.
Thanks!

Hello!
Thanks for bringing this to my attention. Please let me know if the latest version fixes this bug.

Hello! Thanks for bringing this to my attention. Please let me know if the latest version fixes this bug.

Hello, thank you for fast update!


Fails to understand oneOf React prop-types

I have a component with prop-types:
class OptionPane extends React.Component {
  static defaultProps = {};

  render() {
    return (
        ...
    );
  }
}

OptionPane.propTypes = {
  option: Types.oneOfType([Types.string, Types.bool]),
};

OptionPane.defaultProps = {
  option: false,
};

export default OptionPane;

Then later when I use this component I get error:

The same happens when I declare props like this:
type Props = {
  option: string | boolean,
}

class OptionPane extends React.Component {
  static defaultProps = {
    option: false,
  };

  props: Props;

I tried to recreate the bottom code example that's just flow. I get the error you describe in v0.52, but it works correctly in v0.53.1.
If you need this to work and your codebase is stuck in v0.52, you can also provide explicit information about the types for the component's default props and props, like this (Note that you need to switch Try Flow to using v0.52 for that to not error.)
The jump from 52->53 forces you to be explicit about the types of the props, but as a result fixes a bunch of bugs or confusing behavior that happened when Flow inferred the prop and default prop types before.


Package new igbinary version

Hello!
Is it possible to release latest igbinary for php7? It includes pretty important fix for igbinary/igbinary#58 that might resulted in segfaults (possibly related issue gplessis/dotdeb-php#144)
Thanks!

Done


Error for bin file

Hello i am receiving this error at the moment.

i am trying to run the command on my 0.bin file. The format is not in either kitti or nuScenes. Is there a way i am able to run this as a pcd file?

You'd better reformat your bin file similar to our provided example file, or you may encounter multiple unexpected errors caused by the inconsistency of model settings (such as the input feature dimension).


drop scala 2.11 support

Summary
Scala 2.11 is no more supported. Last supported version is 0.2.8-SNAPSHOT
PR Type: Feature
Status: Ready to review
Breaking change? Yes

Good riddance
Le 8 janvier 2022 14:06:06 Hayssam Saleh @.***> a écrit :

Summary
Scala 2.11 is no more supported
PR Type: Feature
Status: Ready to review
Breaking change? Yes
You can view, comment on, or merge this pull request online at:
https://github.com/starlake-ai/starlake/pull/102
Commit Summary
bc8f6b5 drop scala 2.11 supoport
File Changes
(2 files)
M build.sbt (10)
M project/Dependencies.scala (30)
Patch Links:
https://github.com/starlake-ai/starlake/pull/102.patch
https://github.com/starlake-ai/starlake/pull/102.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Win 2024

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Are there any other comments for this?
I think I addressed the major concerns of before.
I rebased and rerendered just now.


Update dependency: deps/kevm_release

@PetarMax after the KCFG transformation PR we're now getting quite big substitutions in the KCFGs (please see commit). Is it expected? Or should they be minimized?

If the substs are identity substitutions, they should be minimized. Let me check.


vehicle assignment is not happening - when vehicle travels from start node to end node without any nodes in between.

What version of OR-Tools and what language are you using?
Version: 9.0.9048
Language: Python
**Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)?
Routing Solver
What operating system (Linux, Windows, ...) and version?
Windows 10
What did you do?
I copied the exact example code from the following link:
https://developers.google.com/optimization/routing/routing_tasks#setting-start-and-end-locations-for-routes
I changed following in create_data_model() function:
data['distance_matrix'] =  [
                                             [0 , 10],
                                             [10,  0]
                                          ]
 data['num_vehicles'] = 1
 data['starts'] = [1]
 data['ends'] = [0]

Basically,  the vehicle starts at node 1 and ends at node 0.
Added below code @ line 34.
print("Is Vehicle used ?"+ str(routing.IsVehicleUsed(vehicle=vehicle_id, assignment=solution)))
Then I added below code @ line 53. (i.e. after this line -->  routing = pywrapcp.RoutingModel(manager))
for vehicle_id in range(data['num_vehicles']):
        routing.ConsiderEmptyRouteCostsForVehicle(True, vehicle_id)

What did you expect to see
Objective: 1010
IS Vehicle used ?True
Route for vehicle 0:
1 -> 0
Distance of the route: 10m
Maximum of the route distances: 10m
What did you see instead?
Objective: 1010
IS Vehicle used ?False
Route for vehicle 0:
1 -> 0
Distance of the route: 10m
Maximum of the route distances: 10m
Doubt: Why vehicle assignment is not happening for this route??
Make sure you include information that can help us debug (full error message, model Proto).
I have attached the full code
VRP issue.txt
Anything else we should know about your project / environment

Vehicle travelling from its start node to its end node is not (and will never be) considered as used even if start and end are different.
As a workaround, if you are not satisfied with this definition/implementation, you can try to post process the solution to modify this behavior in your code...


Revert "Support building DWARF-only (no .dSYM) Swift testcases."

This reverts commit b8a750f175b436bc4b66ad03cd7dfb6c07eddd07.
It broke linux.

@swift-ci test and merge


add CORS support

If imageproxy is being run behind a web server, then this of course could be configured there.  If not, then an option to set CORS headers on responses would be helpful (it looks like a handful of downstream forks have done this: b6b3e3354e5a58fa58a69a9d705c5061fa92c5aa, aa8b4611e4f60000943fafe7f58e4dbdd4f07175).  Rather than setting this to "*", we may want to make it configurable?
more info: https://developer.mozilla.org/en-US/docs/Web/HTTP/Access_control_CORS
/cc @bnmnetp @danisla

added in #96


rename to: K-12 OER Collaborative

Matt - will you please rename this project to:
K-12 OER Collaborative
Many thanks,
Cable

Done.


Open in a Trusted Digital Wallet option is not functional

Repro Steps:
-Install the BC Wallet app to accept terms of use and create pin
-Receive the email invitation for BC VC pilot app
-Proceed all through the invitation and accept
-The QR code screen displays
-Select the "Open in a Trusted Digital Wallet" option right below the QR code
Actual Result:
"Open in a Trusted Digital Wallet" option is not functional

Resolved by: https://github.com/bcgov/trust-over-ip-configurations/pull/106


Erorr While Building App

Current behavior (how does the issue manifest):
Getting following error while trying to build app linux/arm. Flogo is installed on aws-ec2 instance

2019-05-01T16:49:49.226Z - warn: Engine does not exist. Creating...
2019-05-01T16:49:49.227Z - info: Exec command: flogo create -flv github.com/TIBC                                                                             OSoftware/flogo-contrib/activity/log@v0.5.8,github.com/TIBCOSoftware/flogo-lib/a<EMAIL_ADDRESS>-f /tmp/flogo-web/build/server/local/engines/exported-app-bui                                                                             ld.json -vendor /tmp/flogo-web/build/server/local/engines/flogo-web/src/flogo-we                                                                             b/vendor app-build in /tmp/flogo-web/build/server/local/engines
2019-05-01T16:49:49.232Z - info: run command: flogo create -flv github.com/TIBCO                                                                             Software/flogo-contrib/activity/log@v0.5.8,github.com/TIBCOSoftware/flogo-lib/ap<EMAIL_ADDRESS>-f /tmp/flogo-web/build/server/local/engines/exported-app-buil                                                                             d.json -vendor /tmp/flogo-web/build/server/local/engines/flogo-web/src/flogo-web                                                                             /vendor app-build
engine:create: 9142.782ms
2019-05-01T16:49:58.370Z - info: New engine created
2019-05-01T16:49:58.370Z - info: EngineInit duration=10105ms
2019-05-01T16:49:58.373Z - info: [log] Build flogo: "flogo build -o,-e" compileO                                                                             pts:
2019-05-01T16:49:58.377Z - info: run command: flogo build -o -e
2019-05-01T16:50:18.202Z - warn: command exited with code 1: flogo build -o -e
2019-05-01T16:50:18.203Z - error: # app-build/vendor/go.uber.org/zap
vendor/go.uber.org/zap/global_prego112.go:26:7: _stdLogDefaultDepth redeclared i                                                                             n this block
previous declaration at vendor/go.uber.org/zap/global.go:34:29
FATAL: command "build" failed: exit status 2

Expected behavior:
App should get build successfully.
Minimal steps to reproduce the problem (not required if feature enhancement):
Build an app
Please tell us about your environment (Operating system, docker version, browser & web ui version, etc):
linux aws-ec2 machine
Docker version 18.06.1-ce, build e68fc7a215d7133c34aa18e3b72b4a21fd0c6136
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.X.X
latest
Additional information you deem important (e.g. issue happens only occasionally):

Hi @exceed007, please export and share the application that you're trying to build.

{
  "name": "myApp",
  "type": "flogo:app",
  "version": "0.0.1",
  "appModel": "1.0.0",
  "triggers": [
    {
      "id": "timer",
      "ref": "github.com/TIBCOSoftware/flogo-contrib/trigger/timer",
      "name": "Timer",
      "description": "Simple Timer trigger",
      "settings": {},
      "handlers": [
        {
          "action": {
            "ref": "github.com/TIBCOSoftware/flogo-contrib/action/flow",
            "data": {
              "flowURI": "res://flow:hello_world"
            }
          },
          "settings": {
            "repeating": "true",
            "notImmediate": "true",
            "startDate": "2018-01-01T12:00:00Z00:00",
            "seconds": "1"
          }
        }
      ]
    }
  ],
  "resources": [
    {
      "id": "flow:hello_world",
      "data": {
        "name": "helloWorld",
        "tasks": [
          {
            "id": "log_2",
            "name": "Log Message",
            "description": "Simple Log Activity",
            "activity": {
              "ref": "github.com/TIBCOSoftware/flogo-contrib/activity/log",
              "input": {
                "message": "hello world",
                "flowInfo": false,
                "addToFlow": false
              }
            }
          },
          {
            "id": "actreply_3",
            "name": "Reply To Trigger",
            "description": "Simple Reply Activity",
            "activity": {
              "ref": "github.com/TIBCOSoftware/flogo-contrib/activity/actreply",
              "input": {}
            }
          }
        ],
        "links": [
          {
            "from": "log_2",
            "to": "actreply_3"
          }
        ]
      }
    }
  ]
}

@fcastill - please see above

same here

building via CLI is working ok!!

Same here with a fresh 0.5.8 and just build a simple Hello World.
Same on linux/amd64 (Ubuntu) as we as on darwin/amd64 (MAC).

Can you try building on 0.9.0 of the webui? https://hub.docker.com/r/flogo/flogo-docker/ the latest tag has been updated.

Hi all, has the issue been resolved with the latest flogo webui?


Add namespace patching if given Avro schema has invalid name for record type

Some generated Avro schemas have name record which is not allowed in
Python Avro parser. Patch this by adding a namespace to the root if it does not exist. This
creates enclosing namespace also for nested elements.
Why this way
Java Avro parser accepts the record as a name. The specification only mentions that primitive types are not allowed to be used as name. Python Avro parser has stricter validation and rejects if record or any other complex type name is used.
As I know only of Flink generating such schemas and the issue is only on Avro type name record this PR does not try to mitigate any other complex types as name.
I think the patching by adding namespace if namespace does not exist is least intrusive option.
Other options are to:

Add the patching to the name itself, patched.name.record.
Change the name by uppercasing/adding lodash the first character, _record, Record.

One issue of this change I can think of is if Avro validation is made stricter to not allow type names at all. This renders the patching with namespaces invalid.

question: is this backwards compatible?

question: is this backwards compatible?

That's a good question.  This change modifies the schema being stored and returned, so it depends on how the schema is being used.  It looks promising:
Avro schema resolution ignores namespaces when matching records.  If the user was using a common "reader" schema to read mismatched files, it should work transparently.  This is true if the reader schema is either the patched or unpatched schema.
Names and namespaces aren't ever present in serialized binary data, so the binary data serialized with the unpatched schema is always deserializable with the patched schema and vice versa.
However, if the user is comparing schema metadata, there could pose a problem.  For example writing Avro files from a Kafka topic using the unpatched schema, then switch to Karapace with the patched schema.  The two files will have different schema metadata, which might be unexpected.
It's tricky to know exactly what the issues might be without actually trying.  That being said, if this is a completely new job and installation, Karapace will have never accepted and served the unpatched schema.

I've asked for a 1.11.1 release on Avro, we should probably be seeing release candidates with the upstream fix in the next few weeks.

Avro 1.11.1 has been released, it would be nice if Karapace can be upgraded to it so the issue that this PR tried to fix is resolved (commenting here since I cannot find an issue for this).

I've raised this issue #439 -- a PR will be incoming and we can close this.
The next version of Avro will accept the record name (https://issues.apache.org/jira/browse/AVRO-3370) but it will also forbid invalid enum defaults (https://issues.apache.org/jira/browse/AVRO-3229)

Closing this PR, the Avro 1.11.x is used in Karapace.


If several charts are presented on page zoom-reset-icon is shown only on first one.

If several charts are presented on page zoom-reset-icon is shown only on first one.

Hi @chemdnepr
thanks for reporting this issue! I will try to fix this as soon as possible!
Best.
Christoph


Try to catch exception in conda-build-all

Looks like this is working now, at least it got past the py34 jypye exception.

Do you know what the reason for the error in the first place is?

Issues in the jypye recipe build.sh.  However, I think this change is still worthwhile in that it prevents bad recipes from taking out our entire CI system.

Issues in the jypye recipe build.sh.

Are you sure? This change just modifies loading the metadata, not the actually running of the build.sh

Look at the Travis output for this previous PR: https://travis-ci.org/omnia-md/conda-recipes/builds/47024208
Here is a copypaste:
BUILD START: jpype1-0.5.7-np19py34_0

Fetching package metadata: ...

Solving package specifications: .

The following NEW packages will be INSTALLED:

numpy: 1.9.1-py34_0

openssl: 1.0.1k-0

python: 3.4.2-0

readline: 6.2-2

setuptools: 11.3.1-py34_0

sqlite: <IP_ADDRESS>-0

system: 5.8-1

tk: 8.5.15-0

xz: 5.0.5-0

zlib: 1.2.8-0

Linking packages ...

[ COMPLETE ] |##################################################| 100%

Removing old work directory

Source cache directory is: /home/travis/miniconda/conda-bld/src_cache

Found source in cache: JPype1-0.5.7.tar.gz

Extracting download

Package: jpype1-0.5.7-np19py34_0

source tree in: /home/travis/miniconda/conda-bld/work/JPype1-0.5.7

+ echo 'building ./jpype1'

building ./jpype1

found egg: /home/travis/miniconda/envs/_build/lib/python3.4/site-packages/setuptools-11.3.1-py3.4.egg

compiling .pyc files...

number of files: 123

BUILD END: jpype1-0.5.7-np19py34_0

# If you want to upload this package to binstar.org later, type:

#

# $ binstar upload /home/travis/miniconda/conda-bld/linux-64/jpype1-0.5.7-np19py34_0.tar.bz2

#

# To have conda build upload to binstar automatically, use

# $ conda config --set binstar_upload yes

Traceback (most recent call last):

File "./conda-build-all", line 221, in <module>

main()

File "./conda-build-all", line 65, in main

execute(args, p)

File "./conda-build-all", line 202, in execute

metadata = MetaData(recipe)

File "/home/travis/miniconda/lib/python2.7/site-packages/conda_build/metadata.py", line 241, in __init__

assert isdir(path)

AssertionError

Well sure, so that implies sometimes that recipe is not a directory.

The issue is that cluster-utils is broken, not jpype.
-
-        try:
-            metadata = MetaData(recipe)
-        except AssertionError as e:
-            print("Skipping recipe due to issue grabbing metadata.")
-            print(e)
-            continue
+        print(recipe)
+        metadata = MetaData(recipe)

$ ./conda-build-all --python 27 --check-against omnia cluster-utils
Building recipes in the following order:
cluster-utils

cluster_utils
Traceback (most recent call last):
  File "./conda-build-all", line 223, in <module>
    main()
  File "./conda-build-all", line 65, in main
    execute(args, p)
  File "./conda-build-all", line 203, in execute
    metadata = MetaData(recipe)
  File "/Users/rmcgibbo/miniconda/lib/python2.7/site-packages/conda_build/metadata.py", line 241, in __init__
    assert isdir(path)
AssertionError

Roping in @danielparton to fix.


how to using this interface with C/C++

I'm not familiar with python but c, how can I using this interface  with c/c++ to get all these data,
can u give the final GET or POST urls with parameters

I'm not familiar with python but c, how can I using this interface with c/c++ to get all these data,
can u give the final GET or POST urls with parameters

you can try this
https://tushare.pro/document/1?doc_id=130

@huanghyw
tks, I tested it with curl, and just could not parse the json results with unicode, it does not show the chinese correctly

You can try redirecting to jq. For example below.
curl -H "charset=UTF-8" -X POST -d '{"api_name": "stock_basic", "token": "be16e3141d93fce250c7273bdcb06f77d8f65cc31294551692e261fa", "params": {"list_stauts": "L"}, "fields": "ts_code,name,area,industry,list_date"}'
 http://api.waditu.com | jq


Less magic in the command lines options.

From what I understand, subclassing CommandLineOption is not yet intended to be used by end users. I would like to take this chance to use less magic in the codebase, so that if we expose it in the API, it will be intuitive to users.

Coverage decreased (-0.03%) to 85.279% when pulling 0906f6583f5e41917eca75167ec98488680063d8 on gabrieldemarmiesse:less_magic_for_command_line into d3fe102ff5e0dbe12e4b1735ec391f63e2ce7fb7 on IDSIA:master.

Alas I believe this is already user facing https://sacred.readthedocs.io/en/latest/command_line.html?highlight=CommandLineOption#custom-flags . We might take a similar approach as with the host_info_gather decorator and make this an optional argument of Experiment.

My bad. I'm really bad at searching in the docs.

We might take a similar approach as with the host_info_gather decorator and make this an optional argument of Experiment.

I'm totally in favor of that. It's way less magic than before. Now the question is how to transition without breaking everyting.

Closing as this PR won't be merged unless there are major changes.


writing unsigned integers and vectors

We are converting an HDF5 file to a root file using uproot3 with python.   We need to convert some data types that uproot3 does not support yet.
At this point we are thinking of using pyROOT or write a C++ program to handle "unsigned integers" and "vector".
Do you plan to add functionality to uproot4 to write "unsigned integer" and "vector" to a root file?
Do you think this would be available this spring release?

Writing ROOT files (at all) is a goal for this spring. Unsigned integers will definitely be included as will jagged arrays—meaning a variable number of integers per entry. These are also available in Uproot 3: documentation on writing basic TTrees and writing jagged arrays, unfortunately only documented by a PR #477.
However, if you really need the data type to be std::vector<unsigned int>, rather than unsigned int[] (which is variable), that isn't planned. They differ in that the latter has an extra 10 bytes per entry that would have to be understood and written correctly (4 of those bytes are the length of the vector, but the other 6 are a kind of header). If your goal is to have accessible jagged array data in a ROOT file, then unsigned int[] would be sufficient. If your project actually needs std::vector<unsigned int>, then we'll have to look at that.

In addition to Uproot 3, another option that exists right now is RDataFrame in PyROOT:

install ROOT with the Python bindings
open Python and import ROOT
create an RNumpyDS to point to NumPy arrays derived from h5py (sorry that I can't give more detailed instructions on that—I haven't done it)
create an RDataFrame (lots of tutorials on this)
its snapshot method writes to ROOT files.

I am also working on something like this.  I am in need of the std::vector type.  Is it possible to get this on the list of to-do items?

Okay, I'll look into the std::vector headers when I work on file-writing.


mention that input in the graylog server should be gelf udp

Great plugin :+1:
It would be really useful if somewhere in readme.md you'd mention that logback-gelf creates upd gelf messages. I had to look in the sourcecode and figure out that it's creating datagrams and I should configure the graylog server to listen to upd (create a gelf upd input).

Yep, I'm working on a new version where TCP is supported as well so I'll make it clear as part of that release. Back in my day, graylog only supported UDP ;)

Greetings,
It helps me to sent messages for Graylog udp inputs. As said, let us know if it covered the TCP and TCP over SSL inputs.
Thanks in advance.
~Regards

Here's the branch that I'm working on. In this new flow, Adding an SSL TCP appender should be pretty easy. I probably won't include one on the first release, but you're welcome to create a PR once this is release.
https://github.com/Moocar/logback-gelf/tree/the-great-appender-rejig

I just finished the branch, any chance you'd be interested in testing it @treaz and @mundrusatish? I've done a fair bit, but it would be great to get some feedback on the new approach. You'll need to build both socket-encoder-appender and logback-gelf from scratch as they're not on maven central yet
https://github.com/Moocar/socket-encoder-appender
https://github.com/Moocar/logback-gelf/tree/the-great-appender-rejig

Sorry Anthony, I'm already caught up with another project and I don't have any more time to play with graylog.

version 2.0beta1 with UDP and TCP support is now released. Closing this issue.


Add licence and accreditation information of MSP to a prescription

As a pharmacy employee when I get information from a prescription I want to see licences and accreditation information of MSP where the prescription was prescribed.
So, add to all requests about prescription attributes licenses and accreditation of legal_entity
MUST: get by id
Responses for all other endpoints might be also updated if the same view is used.
@AlekzandrV spec should be updated for all the endpoints.
{
       "licenses": [
         {
           license_number:
           "issued_by":
           issued_date:
           expiry_date:
           active_from_date:
           "what_licensed":
           "order_no":
         }
       ],
       "accreditation": {
         "category":
         issued_date:
         expiry_date:
         "order_no":
         order_date:
       }
     },

Done


Algorand extension is missing a signLogicSig(program) method

✨ Feature Request
Could you add the signLogicSig(program) method to the Algorand extension? It is a feature that can not be polyfilled with signTransaction and it is provided by other signing wallets.
🧩 Context
singLogicSig or lsig.sign is used to delegate signature authority which is a core feature of the algorand blockchain.
💻 Examples
An equivalent functionality can be fund in the My Algo Wallet, which is a similar third party signing tool.

Usage example: https://github.com/randlabs/myalgo-connect#sign-teal
Implementation example: https://github.com/randlabs/myalgo-connect/blob/f42ca7e52aa24a5cd98c6300ee6e5df8c9fda48e/lib/main.js#L337

cc @harryEth

Hi! Any update on this? I would be happy to help if you can point me in the right direction.

@solkimicreb Following up here and updating you. This feature request isn't one that's currently on our roadmap, but we will re-open the issue when it is.


Various updates to parallel mode

Replace partition_via_api with a requests.post, and pass on any relevant headers using the FastAPI request.
add smoke test to compare parallel and single mode.
xfail jpeg in tests until we fix the strategy=fast issue - https://github.com/Unstructured-IO/unstructured/pull/735 (Nevermind)

To verify, the test instructions from https://github.com/Unstructured-IO/unstructured-api/pull/115 still apply.

@cragwolfe when you have a moment can you take another look?

Yep! Added it as an optional param here: https://github.com/Unstructured-IO/unstructured-api-tools/pull/175


Escaping plus sign

This is a valid version character according to semver

Codecov Report

Merging #24 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   62.42%   62.42%
=======================================
  Files          28       28
  Lines        1030     1030
=======================================
  Hits          643      643
  Misses        387      387

Impacted Files
Coverage Δ

...n/java/net/evendanan/bazel/mvn/api/Dependency.java
89.36% <100%> (ø)
:arrow_up:

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 2e08bd5...7bba116. Read the comment docs.


Calendar Left swipe issue

Hi, In Big-Calendar,  swipe is not working smooth in mode of month. When I swipe right, swipe smooth but on left swipe it's hang after 3-4 swipes. I upload video and code here,
#screen_recording
https://user-images.githubusercontent.com/102415580/210559221-5d9b0df5-7e97-414f-bf22-a34209788c54.mov
#code
<Calendar
height={Dimensions.get("window").height - 60}
events={calendarEvents}
mode={mode}
swipeEnabled={true}
eventCellStyle={getEventColor}
/>

Are you working for the Web? Also, please make sure you're using the latest version (3.3.2)

Thank You for your response. I am working for mobile. I upgraded latest version and problem resolved.

@ronakbuoyancy Thank you for checking the new version!


Feature request: Add content to Sidenav Backdrop

Bug, feature request, or proposal:
feature request: Ability to add content to sidenav backdrop.
What is the expected behavior?
https://github.com/angular/material2/blob/master/src/lib/sidenav/sidenav-container.html
Perhaps add another <ng-content>:

     <ng-content select="md-sidenav-backdrop, mat-sidenav-backdrop"></ng-content>

<ng-content select="md-sidenav, mat-sidenav"></ng-content>

  <ng-content></ng-content>

What is the current behavior?
Not supported.
What is the use-case or motivation for changing an existing behavior?
At the moment backdrop is blank, but it would be nice if we could insert some content there...

What kind of content would you want to add inside of a backdrop?

Anything really (any component). Let's say sidenav has a list of items and on hover over each of them I want to play a video, or show description with images in the backdrop on large screens...

This doesn't sound like the sidenav should handle. The backdrop is meant to be static and only be there to catch clicks. The way it's set up at the moment is that it doesn't even get processed by Angular. Thoughts @mmalerba?

Take a look at the editor at https://sway.com/. This type of interface would be easy to create with sidenav, something like this:
<md-sidenav-container>
  <md-sidenav>
    <md-sidenav-container>
      <md-sidenav>
        <CARDS_COMPONENT></CARDS_COMPONENT>

      </md-sidenav>
    </md-sidenav-container>

    <STORYLINE_COMPONENT></STORYLINE_COMPONENT>

  </md-sidenav>

  <md-sidenav-backdrop>
    <PREVIEW_COMPONENT></PREVIEW_COMPONENT>
  </md-sidenav-backdrop>

</md-sidenav-container>

This isn't something we plan on adding; we consider the backdrop to primarily be an indicator that the content is inactive. Adding additional content would be trying to add too much behavior to this.


Supplemental changes of druntime/pull/72 - Issue 1824 - Object not const correct

http://d.puremagic.com/issues/show_bug.cgi?id=1824
https://github.com/D-Programming-Language/dmd/pull/387
https://github.com/D-Programming-Language/druntime/pull/72
https://github.com/D-Programming-Language/phobos/pull/262
This pull request does not change compiler, but fixes only dmd test suites.
2011/10/15
This change also fix issue 3780, because it requires const equality comparison for member classes.
requires: druntime git://github.com/9rnsr/druntime.git constApply
requires: phobos git://github.com/9rnsr/phobos.git constApply

This change now also fix issue 3780, because it requires const equality comparison for member classes.


Home Page - Add tab bar

As a registered professor, I want to quickly navigate to the Teams, Clients, or Projects page, so that I can manage my teams.
Add a tab bar with Teams (landing page), Clients page, and Projects page.
You do not need to design the pages. This will be done in another issue.

When I was redesigning and creating the new landing page I went ahead and added a tab bar. However, it doesn't have functionality yet. I just left a placeholder there for when we do implement it. Check out react's materials-ui tab bar!


Unwanted placeholders being placed in Android

I need to parse a string which has 3 percent (%) symbols. However, if I put more than one % symbol, the % becomes a placeholder, hence displaying a wrong text in Android. Since I'm sharing this strings.txt file with an iOS app, I would like not to create different string for each platform for better maintainability.
Strings.txt
[testString]
        es = This string has only one percentage sign. 1% symbol.
[testString2]
        es = This one has two. 1% to 2%.
[testString3]
        es = This one has three. 12%. From 13% to 15%.

Android strings.xml
<string name="testString">This string has only one percentage sign. 1% symbol.</string>
<string name="testString2">This one has two. 1%1$ to 2%2$.</string>
<string name="testString3">This one has three. 12%1$. From 13%2$ to 15%3$.</string>

Am I doing something wrong?
Regards!

As these strings all support printf notation, % symbols must be escaped. Try using %%.


Parser can't parse cast of a function call

foo : u32() {
    69;
};

bar : @u32 = (foo() as! @u32);

0;

So this code throws an error while parsing, because it can't parse the cast properly after a function call

This happens because in the parser, the postfix operators like "as" are getting parsed (l.617) before empty perens  are parsed into a call (l.665)
(Without the perens around the cast the vardecl is gonna be cast, so its gonna be even more uselesss)

What happens if you do (foo) as! ... instead of foo() as! ...

5head, that works. So is this the way to cast function calls now? Doesn't look to bad tbh, but at least I didn't really find it idiomatic especially the case without the outside perens, where the vardecl is cast.

In general, a function call is (function ...args), like LISP. No parens for arguments are needed. And, the parens are just a paren expression, for grouping.

So maybe also label this with somekind of fix when somebody feels like it label? Or write it in todo
Would just be dumb if issues get rediscovered over and over again (I see myself doing that) and I don't want to search in closed issues for things that are potentially still issues

There is no issue here to fix. foo() is not a function call, it's a function followed by a paren expression

What is this then?

It looks like a screenshot from one of the parser source files of the compiler, all of which I did not write and am not working on right now.


mitochondrial complex III deficiency nuclear type 1 MONDO:0007415 def  query

https://monarchinitiative.org/disease/MONDO:0007415
mitochondrial complex III deficiency nuclear type 1 MONDO:0007415
Description
Mitochondrial complex III deficiency is one of several conditions caused by dysfunction of mitochondria , which are specialized compartments in cells that generate more than 90% of the energy required by the body. It is a severe, multisystem disorder that includes features such as lactic acidosis , hypotonia , hypoglycemia , failure to thrive , encephalopathy , and delayed psychomotor development. Involvement of internal organs, includingliver diseaseand renal tubulopathy, may also occur. Symptoms typically begin at birth. Many affected individuals die in early childhood, but some have survived longer. It is generallycaused by mutations in nuclear DNA in the BCS1L , UQCRB and UQCRQ genes and inherited in an autosomal recessive manner.However,it may also be caused bymutations in mitochondrial DNA in the MTCYB gene, which is passed down maternally or occurs sporadically and may result in a milder form of the condition. Treatment generally focuses on alleviating symptoms and/or slowing down the progression of the disease,and effectiveness can vary among individuals.

this def has a lot of formatting errors
Is this the correct def for type I?
I thought type I was specific for BCS1L?
For example
UQCRB  is type 3  https://monarchinitiative.org/disease/MONDO:0014064
and  UQCRQ  is type 4 https://monarchinitiative.org/disease/MONDO:0014065

Also the def should say that this is a dysfunction of
"mitochondrial respiratory chain complex III "  to distinguish from the myriad of other syndromes which affect energy production and have similar symptoms.
( I think this definition was maybe meant for the parent)?

Actually, I think this term should exist, so
https://github.com/monarch-initiative/mondo/issues/1552 is out or date.
The problem is the incorrect definition assigned

Out of curiosity when was the def assigned? I can't find  a ticket mentioning it, but we are not picking this up in PomBase, so I just wondered if it was recent
@kimrutherford
https://www.pombase.org/term/MONDO:0007415

so I just wondered if it was recent

The definition is not in the mondo.obo from the latest release.

Strange - I don't see this definition in the mondo-edit.obo file.
@cmungall and @kshefchek do either of you have any idea where this definition came from in Monarch?

It looks like it is from https://rarediseases.org/gard-rare-disease/8295/mitochondrial-complex-iii-deficiency/
I've never seen a case those where there is a def in Monarch but not in the Mondo file.
If I add this def to Mondo, will it show up in Monarch?

Monarch is currently running http://purl.obolibrary.org/obo/mondo/releases/2020-03-05/mondo.owl , I can see this def there but must have been fixed in the last release.  I can close when we push our next data release (likely a few weeks out)

Oh yeah, I see the def there. It must have been removed in the latest release. Thanks @kshefchek!

@ValWood this term is currently undefined in the current Mondo file, I'll add a definition. (There was a def on the GARD website, but it looks like it is down at the moment, so I'll come back to this.)


ffmpeg post processing CLI issue

`C:/Users/Jean-Michel/AppData/Local/labDV.com/Video 360 - Tube/bin/youtube-dl/youtube-dl.exe --verbose -i --playlist-items 2 -o D:/Users/Me/Videos/Tube/the best of  the gladiators/%(title)s.%(ext)s --format bestaudio -x --audio-format mp3 --audio-quality 320k --postprocessor-args -strict experimental --ffmpeg-location C:/Projects/Dev/App/bin/applications/zeranoe/ffmpeg-x64/bin/ffmpeg.exe https://www.youtube.com/watch?v=re2-2j-q6Q8&list=RDre2-2j-q6Q8#t=4
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'-i', u'--playlist-items', u'2', u'-o', u'D:/Users/Me/Videos/Tube/the best of  the gladiators/%(title)s.%(ext)s', u'--format', u'bestaudio', u'-x', u'--audio-format', u'mp3', u'--audio-quality', u'320k', u'--postprocessor-args', u'-strict experimental --ffmpeg-location', u'C:/Projects/Dev/App/bin/applications/zeranoe/ffmpeg-x64/bin/ffmpeg.exe', u'https://www.youtube.com/watch?v=re2-2j-q6Q8&list=RDre2-2j-q6Q8#t=4']
[debug] Encodings: locale cp1252, fs mbcs, out None, pref cp1252
[debug] youtube-dl version 2015.11.10
[debug] Python version 2.7.10 - Windows-8-6.2.9200
[debug] exe versions: ffmpeg N-72443-g2e15f07, ffprobe N-72443-g2e15f07
[debug] Proxy map: {}
[generic] ffmpeg: Requesting header
WARNING: Could not send HEAD request to C:/Projects/Dev/App/bin/applications/zeranoe/ffmpeg-x64/bin/ffmpeg.exe:
[generic] ffmpeg: Downloading webpage
ERROR: Unable to download webpage:  (caused by URLError(u'unknown url type: c',))
File "youtube_dl\extractor\common.pyo", line 329, in _request_webpage
File "youtube_dl\YoutubeDL.pyo", line 1873, in urlopen
File "urllib2.pyo", line 431, in open
File "urllib2.pyo", line 454, in _open
File "urllib2.pyo", line 409, in _call_chain
File "urllib2.pyo", line 1265, in unknown_open
[youtube:playlist] Downloading playlist RDre2-2j-q6Q8 - add --no-playlist to just download video re2-2j-q6Q8
[youtube:playlist] RDre2-2j-q6Q8: Downloading Youtube mix
[download] Downloading playlist: Mix - the best of  the gladiators
[youtube:playlist] playlist Mix - the best of  the gladiators: Collected 26 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[youtube] qbd_orEZHqU: Downloading webpage
[youtube] qbd_orEZHqU: Downloading video info webpage
[youtube] qbd_orEZHqU: Extracting video information
[youtube] {22} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {43} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {18} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {5} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {36} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {17} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {137} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {248} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {136} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {247} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {135} signature length 39.40, html5 player en_US-vflJENbn4
[youtube] {244} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {134} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {243} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {133} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {242} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {160} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {278} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {140} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {171} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {249} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {250} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] {251} signature length 41.40, html5 player en_US-vflJENbn4
[youtube] qbd_orEZHqU: Downloading DASH manifest
[debug] Invoking downloader on u'https://r10---sn-hgn7zn7s.googlevideo.com/videoplayback?id=a9b77fa2b1191ea5&itag=141&source=youtube&requiressl=yes&pl=16&mv=m&ms=au&mn=sn-hgn7zn7s&mm=31&ratebypass=yes&mime=audio/mp4&gir=yes&clen=63097025&lmt=1429714603316862&dur=1979.059&upn=ZXvK6GbJyaU&fexp=9408241,9408710,9414764,9416126,9417683,9420452,9422596,9422618,9423662&sver=3&mt=1447401196&signature=4843F3BBB8E8969717D4C55D6C6A3F96E22A5F61.80694DFC4587C400296A1171FD34627A7D26285A&key=dg_yt0&ip=<IP_ADDRESS>&ipbits=0&expire=1447422956&sparams=ip,ipbits,expire,id,itag,source,requiressl,pl,mv,ms,mn,mm,ratebypass,mime,gir,clen,lmt,dur'
[download] D:\Users\Me\Videos\Tube\the best of  the gladiators\The Gladiators - 1982 - Back To Roots (FULL ALBUM).m4a has already been downloaded
[download] 100% of 60.17MiB
[ffmpeg] Correcting container in "D:\Users\Me\Videos\Tube\the best of  the gladiators\The Gladiators - 1982 - Back To Roots (FULL ALBUM).m4a"
[debug] ffmpeg command line: ffmpeg -y -i 'file:D:\Users\Me\Videos\Tube\the best of  the gladiators\The Gladiators - 1982 - Back To Roots (FULL ALBUM).m4a' -c copy -f mp4 -strict experimental --ffmpeg-location 'file:D:\Users\Me\Videos\Tube\the best of  the gladiators\The Gladiators - 1982 - Back To Roots (FULL ALBUM).temp.m4a'
ERROR: Error splitting the argument list: Option not found
Traceback (most recent call last):
File "youtube_dl\YoutubeDL.pyo", line 1712, in post_process
File "youtube_dl\postprocessor\ffmpeg.pyo", line 457, in run
File "youtube_dl\postprocessor\ffmpeg.pyo", line 158, in run_ffmpeg
File "youtube_dl\postprocessor\ffmpeg.pyo", line 154, in run_ffmpeg_multiple_files
FFmpegPostProcessorError
[debug] ffmpeg command line: ffprobe -show_streams 'file:D:\Users\Me\Videos\Tube\the best of  the gladiators\The Gladiators - 1982 - Back To Roots (FULL ALBUM).m4a'
[ffmpeg] Destination: D:\Users\Me\Videos\Tube\the best of  the gladiators\The Gladiators - 1982 - Back To Roots (FULL ALBUM).mp3
[debug] ffmpeg command line: ffmpeg -y -i 'file:D:\Users\Me\Videos\Tube\the best of  the gladiators\The Gladiators - 1982 - Back To Roots (FULL ALBUM).m4a' -vn -acodec libmp3lame -b:a 320k -strict experimental --ffmpeg-location 'file:D:\Users\Me\Videos\Tube\the best of  the gladiators\The Gladiators - 1982 - Back To Roots (FULL ALBUM).mp3'
ERROR: audio conversion failed: Error splitting the argument list: Option not found
Traceback (most recent call last):
File "youtube_dl\YoutubeDL.pyo", line 1712, in post_process
File "youtube_dl\postprocessor\ffmpeg.pyo", line 279, in run
PostProcessingError`
[debug] ffmpeg command line: ffmpeg -y -i 'file:D:\Users\Me\Videos\Tube\the best of  the gladiators\The Gladiators - 1982 - Back To Roots (FULL ALBUM).m4a' -vn -acodec libmp3lame -b:a 320k -strict experimental --ffmpeg-location 'file:D:\Users\Me\Videos\Tube\the best of  the gladiators\The Gladiators - 1982 - Back To Roots (FULL ALBUM).mp3'
ERROR: audio conversion failed: Error splitting the argument list: Option not found
--ffmpeg-location is in ffmprg command !!!
'file:D:\Users... seems to be wrong as well

Soory, my mistake, a typo iun my post processors options...


Reply to this email directly or view it on GitHub:
https://github.com/ron2111/Hacktoberfest2021_awesome-projects/pull/258#issuecomment-932816387


False positive of react/prop-types with destructuring

See https://gist.github.com/idan/1d31a706d5bce36d1ada#file-index-jsx-L45
The relevant bit is a handler method, which errors like so:
45:18  error  'target' is missing in props validation  react/prop-types
                 complains here
                 ↓
  handleChange ({target: {value}}) {
    this.setState({filterString: value})
    this.emitFiltered(value.trim())
  }

Not quite sure what additional information would be useful for running this down — let me know how I can help and I'll do so gladly.
Thanks for eslint-plugin-react! :heart:

Do you use<EMAIL_ADDRESS>? If yes I think this is duplicate of #443.

@yannickcr herp derp, indeed it is! Closing as duplicate.


Single player footer fix

Part of my ticket was to remove the footer in the single player cards. A different group has since updated the footer and made the footer more in line with the Figma. I don't think that removing the footer is going to be the direction that we are wanting to go. But it was in there, so it is completed.
@derekjpeters What direction do we want for the footer ☝🏻
Changed the text when submitting a file from "Save" to "Submit"
Changed the "I'd rather draw" and "I'd rather write" buttons to be antd button components. They now stand out a lot more and are very obviously buttons.
After submitting a file, the submission field was not clearing. Added some logic that clears that now.
https://www.loom.com/share/5ad17e2913414bed936361c60434dfe3
https://trello.com/c/KVwUj2dd/714-single-player-gameplay-mission-write-see-checklist
https://trello.com/c/GF8uxVBQ/712-single-player-gameplay-mission-read-see-checklist
https://trello.com/c/YegIug3J/713-single-player-gameplay-mission-draw-see-checklist

 Single Player - gameplay/mission/draw
SEE CHECKLIST

 Single Player - gameplay/mission/read SEE CHECKLIST

 Single Player - gameplay/mission/write
SEE CHECKLIST

The Loom video gave clear idea of all the changes made and everything looks good Tristan.

I think the alignment is better.  We can remove this PR w/o merging.


In Update, selecting and arrow keys not working

I'm using Windows 10. When I run npm-check -u I get the expected screen below, but pressing space or using the arrow keys does not update the screen. What I've found does work is to run npm-check -u, press space and then enter. There is no asterisk indicating the selection, but it does the update. I then repeat for each package.

And after waiting weeks of this behavior occurring, I mistype a command line option and now it starts working again. I was trying to turn colors off and mistyped the option. Got an error. Afterwards, it now works. So strange.


Delete Officer, Search For Officer, Horizontal Scroll

Describe the feature
Delete Officer, Search For Officer, Horizontal Scroll
Can we have these options added to the CAD for the Manage Units page
Additional Context
No response

A forced active horizontal scrollbar. Most people don't realize they need to hold SHIFT to scroll to the right.


Set 0d shape to python constant tensors

From #1916, it is found that the undefined shape disables the functionality of IsScalar op: https://github.com/microsoft/onnxscript/blob/561a6006ff48b744c7a03d80994fd74ea06be5f7/onnxscript/function_libs/torch_lib/graph_building/_graph_building_torch.py#L357 This PR annotates python scalar to be 0d tensor.

I would suggest doing this differently. The shape info may not be needed?

This may be low priority. It all starts https://github.com/microsoft/onnxscript/pull/1916 that I re-wrote aten_narrow to adapt the change that I moved YracedONNXFunction to promote python constants to tensors before function call. With doing that, IsScalar loses its functionnality because we didn't assign shape to tensors. That's why I create this PR.

Leave it as it is for now. For torchlib, we keep the implementation, and CI tests will be deprecated soon with OpSignature PR.


Update README.rst

Added how to enter debug mode

Thanks, but I don't think this belongs in the README file

Okay I just felt the need that it should be...nevermind

It's here: http://flask.pocoo.org/docs/1.0/quickstart/#debug-mode
The README is really just show a very simple example on how a Flask app works.

okay


Add option to magnify a geometry block

Description
Given how maps need to regularly be inspected, this would be most useful

This option should be added to the rest of the page blocks that don't have it, not only to geometry block

For today:

write down the plan you made for this
make a list of all the page blocks that need to be updated (update the original description, if you can't, write as a comment)

Actually a CL entry is not needed here since this is an improvement of a new feature.  Moving the card to Done.


tailwind css and gatsby setup

I was reading this: https://www.gatsbyjs.org/docs/tailwind-css/#option-2-css-in-js
and was trying to setup tailwind css in my gatbsy project
Compiling...
error There was an error trying to load the config "tailwind" for the macro imported from "tailwind.macro. Please see the error thrown for more information.
 ERROR  Failed to compile with 1 errors

has anyone setup tailwind css in a gatsby project using css in js?

https://github.com/bradlc/babel-plugin-tailwind-components#options


Water issues

Forge 2800
Dsurround: <IP_ADDRESS>
There seem to be some issues with how water is being handled in certain circumstances. Here's a link to information provided by someone that knows more than I do: https://github.com/Angry-Pixel/The-Betweenlands/issues/785

Bump

Yeah.  I think this is a bug I am fixing for the next update.  I will mark fixed.

Pushed DynamicSurroundings-1.12.2-<IP_ADDRESS> to CurseForge.


[move-vm] Add strict/paranoid runtime verification

Motivation
This is a duplicate of #493 . The intention is we will first land this version on Aptos branch while we discuss what would be the right runtime checks to add for main branch.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
cargo test

Please change the target where you want to merge into to aptos. There should be dropdown at the top of your PR. Once you do the diff will be against that branch.

@chen-robert FYI

The remaining work items are:

Get benchmarks from @vgao1996
Force annotating the return value of native functions.

@runtian-zhou some additional ideas for checks which I think would make sense to get merged in here

Function visibility check
Check ty_args against function type parameter visibility requirements
Hot potato Pack and Unpack tracking

Right now the two tests are failing:

language/move-prover/interpreter-testsuite/tests/concrete_check/pack_unpack.move
language/move-prover/interpreter-testsuite/tests/concrete_check/destroy.move

IMHO the failure here are as expected because the test function is returning struct has zero capabilities and thus causing the struct to be dropped unexpectedly...

I like this solution more than #493, I think this solution will make me sleep a little more soundly😂. Expect it to be merged into the main branch.

To expand on the issue @tnowacki was mentioning on the other PR with an example:
let bad_signer = ... // turn bytes into a signer via exploiting a vulnerable instruction (e.g., VecPack). This will still have a vector<u8> type tag, so the attacker must use it carefully
let vec = vector::empty();
vector::push_back(&mut vec, bad_signer); // note: this is a VecPushBack bytecode instruction, not a function call--no checks are performed
let any = any::pack(vec) // this will check the type of `vec`, but not `vec[0]`
let laundered_vec = any::unpack(any); // now vec[0] has the `signer` tag, can be used freely

To expand on the issue @tnowacki was mentioning on the other PR with an example:
let bad_signer = ... // turn bytes into a signer via exploiting a vulnerable instruction (e.g., VecPack). This will still have a vector<u8> type tag, so the attacker must use it carefully
let vec = vector::empty();
vector::push_back(&mut vec, bad_signer); // note: this is a VecPushBack bytecode instruction, not a function call--no checks are performed
let any = any::pack(vec) // this will check the type of `vec`, but not `vec[0]`
let laundered_vec = any::unpack(any); // now vec[0] has the `signer` tag, can be used freely

So if any is the function in aptos-framework it would be fine because we check the inner fields of struct recursively at serialization time.
If we check all type matches the declaration of Pack, we would be able to argue that in the MoveVM with paranoid mode on, no malformed value can ever be created. However, due to the limited timeline that we have I intensionally cut that corner but I would still argue that this would offer us enough safety. In this version, a module can create a struct that has bad field elements in it. But there's no way to exploit from this fact from a practical standpoint given that we check the struct types to match at function call as well.

What is going on with this PR? I thought it was already ready to land the day before yesterday. @davidiw


Looks somewhat like:
# smbclient -U foo -L //localhost
lp_load_ex: changing to config backend registry
Password for [WORKGROUP\foo]:
dos charset 'CP850' unavailable - using ASCII                      <-------- HERE
session setup failed: NT_STATUS_LOGON_FAILURE

It's minor but unnecessary as it can be eliminated by setting an smb.conf option (dos charset AFAICT).
Let's set that option and eliminate an annoyance.

We can certainly do that but I am wondering why the CP850 codepage (which is typically compiled in) is not available in the first place.

Let me know if there are any commands we can run to investigate. Otherwise, I'd like to try and squish the annoying message and then we can open a new issue to investigate deeper.

We can certainly do that but I am wondering why the CP850 codepage (which is typically compiled in) is not available in the first place.

I dug into this little deeper and here are my findings:
CP850 and IBM850 can be used interchangeably as they are aliases to each other.
charset detection is mostly driven by an on-demand initialization function get_conv_handle() in Samba which basically attempts to find DEFAULT_DOS_CHARSET(CP850) unless explicitly specified via dos charset smb.conf option. DEFAULT_DOS_CHARSET is a compile/build time macro defined to indicate the charset found. Further in its absence DEFAULT_DOS_CHARSET is set to ASCII.
I verified using sample program that independent library files, in this case for IBM850, are required for individual charsets. In Fedora glibc-gconv-extra package provides IBM850.so. Even though Samba rpms are built with IBM850 our smbclient images based on CentOS Stream 8(and server images based on Fedora 35 too) doesn't have this package installed by default and thus we end up with the warning for smbclient(and in server logs).

Wow. Love the in depth analysis @anoopcs9.  Between all that, and this seems very much an smb1 thing I'm strongly in favor of suppressing the error message by defaulting to ascii. Hopefully as samba continues to deprecate/drop support for smb1 this goes away entirely.
In the meantime I'm planning on merging #229 later today unless I hear some clear NAKs.

@anoopcs9 Impressive!


Output template playlist_index issue

Lets say I am downloading playlist with 15 videos and after 6 videos it was interrupted. So when I restart with --playlist-start 7, the index doesn't add 0 by default to the playlist_index. It creates problem if I have created playlist or want to sort videos in playlist.
What I get is: 7. filename, 8. filename, 9. filename
I want it like: 07. filename, 08. filename, 09. filename

Duplicate of #5185.


Fixed two typos in transactional-outbox.md

Fixes two typos

The one change was already submitted by an earlier PR, and the attempt change is not valid.

Nm, bad diff, merged. Thanks.


RxJava (w/ Android?)

https://github.com/Netflix/RxJava
https://github.com/Netflix/RxJava/tree/master/rxjava-contrib/rxjava-android

Using it now.


fix cef mp.events.add(...)

according to this location of the doc https://wiki.rage.mp/index.php?title=Getting_Started_with_Events#Using_events


Re-land integrate testWidgets with leak tracking.

Original PR: https://github.com/flutter/flutter/pull/138057
Revert: https://github.com/flutter/flutter/pull/140502
Issue: https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android flutter_test_performance/12787/overview
Exception: flutter test rendered unexpected output (1 bad lines)
Explanation: leak tracker adds tear down even when there is no leak tracking, because at the moment of adding tear down it is unclear if leak tracking will be used for some tests.
Fix: add enabling flag for leak tracker and make creation of tear down conditional.
Prerequisites:

[x] Add enabler: https://github.com/dart-lang/leak_tracker/pull/197
[x] Refresh leak tracker in G3: http://cl/594657360
[x] Upgrade leak tracker in flutter: https://github.com/flutter/flutter/pull/140758

Google testing passed.


[ Can't DownLoad or UpLoad files/attachements ]

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Just display an URL where a user can DL/UL a file or attachement like a mail client.
click on the link as you would normally do, a mail attachment for exemple.
Just nothing happens :s
Expected results
expect a Download to start
Actual results
Nothing at all
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

@Gobmichet Thanks for the report, but I am unclear as what exactly the issue is. Are you reporting this against url_launcher package ?
Can you provide us complete minimal reproducible code sample along with flutter doctor -v and platform on which you are seeing this behavior, so that we can properly address the reported behavior.


Consider using a GPL-compatible license

I am amazed by {sassy} system and the functionality it provides! However, it's choice of license makes it hard to integrate it into other projects, as Creative Commons licenses (besides CC0) should not be used for software:

We recommend against using Creative Commons licenses for software. […] [O]ur licenses are currently not compatible with the major software licenses, so it would be difficult to integrate CC-licensed work with other free software.
— Creative Commons FAQ

In fact, most packages from the {sassy} system are CC0 licensed, which is great, but the {sassy} meta package itself signals differently. So it would be best to choose a GPL-compatible license for {sassy} and {reporter} as well, as to ease the inclusion in larger projects.

Florian:
I deeply appreciate this feedback.  I was wondering if this was a problem, but no one had said anything.  Based on your feedback, I will change it.  Give me a few days.
David

That's really great, thank you! To reiterate: I believe CC0 is a fine choice, so it might be easiest to switch {sassy} and {reporter} to that as well.


Fix docker continuous deployment

This is a followup to #1234, which accidentally broke the continuous
deployment of docker containers to the github container registry.
This happened because we are now generating our docker files based
on the sbt build configuration, but the current github workflow expects
a Dockerfile in the root of the project. The proposed fix here (to be
tested once the PR is merged into unstable) is to generate the docker
file using our make target, and then point the github action to the
"context" (directory) where the build file and its related artifacts are stored
in our artifact dir.
This patch fixes that buy first building the docker image, before
invoking the github action to upload the image to the registry

Good idea. I'll do that, since it's already passed CI and review, it's not really risky to push the container to our registry. Thanks for the idea!

Nice. Now we can be confident the fix is actually a fix :D https://github.com/informalsystems/apalache/pkgs/container/apalache/13764782?tag=shon-fix-docker-cd
Very happy to err on the side of correctness here.

Codecov Report

Merging #1250 (13acde1) into unstable (722d22b) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 13acde1 differs from pull request most recent head 3eed3ae. Consider uploading reports for the commit 3eed3ae to get more accurate results

@@            Coverage Diff            @@
##           unstable    #1250   +/-   ##
=========================================
  Coverage     74.23%   74.23%
=========================================
  Files           325      325
  Lines         10850    10850
  Branches        526      526
=========================================
  Hits           8055     8055
  Misses         2795     2795

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 722d22b...3eed3ae. Read the comment docs.


Silience G++ sequence point warnings

Clean up initialisation of a number of class member variables to quiet NaN. Use C++ default member intialisers instead of chained assignment expressions which is a lot clearer to read, I think you'll agree. It also has the advantage of putting the initialisation right next to the member variable declaration.

The original code has the constructors use a chain of assignments expressions to set each member variable to std::numeric_limits<double>:::quiet_NaN() in the more compact form:
var1 = var2 = var3 = var4 = var5 = var6 = std::numeric_limits<double>::quiet_Nan();

However, because the variables have long names and there are so many of them, duplicates have crept in. eg:
var1 = var2 = var3 = var1 = var4 = var5 = var6 = std::numeric_limits<double>::quiet_NaN();

This will produce a sequence point warning because one of the variables (in this case, var1) is repeated in the expression. While it would be easy to fix this by removing any duplicates, I think that initialising each variable in the structure declaration is clearer and more maintainable.

No problem. Thanks for contributing.
On Tue, Jun 16, 2020 at 10:54 PM Ben Elliston<EMAIL_ADDRESS>wrote:

@bje- commented on this pull request.
In tcs/csp_solver_core.h
https://github.com/NREL/ssc/pull/399#discussion_r441279655:

};

struct S_csp_cr_inputs
{

double m_field_control;			//[-] Defocus signal from controller (can PC and TES accept all receiver output?)

int m_input_operation_mode;		//[-]

   double m_adjust;                //[-] Field availability / adjustment factor

S_csp_cr_inputs()

{

	m_field_control = std::numeric_limits<double>::quiet_NaN();

	m_adjust = std::numeric_limits<double>::quiet_NaN();

	m_input_operation_mode = -1;

}

double m_field_control =		//[-] Defocus signal from controller (can PC and TES accept all receiver output?)

  std::numeric_limits<double>::quiet_NaN();

int m_input_operation_mode = 1;		//[-]

Sorry for the mixup. 😄
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/NREL/ssc/pull/399#discussion_r441279655, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHICFBYPQQVNYLMWQEPDWDDRXBEARANCNFSM4N3EPPWQ
.


onActivityResult is not called

I start the image picker with:
ImagePicker.Companion.with(getActivity())
.start();

I can successfully select an image, but onActivityResult(…) in same Activity is not called.
When I use
 Intent action = new Intent(Intent.ACTION_OPEN_DOCUMENT);
            action.setType("image/*");
            action.addCategory(Intent.CATEGORY_OPENABLE);

        startActivityForResult(Intent.createChooser(action, getActivity().getResources().getString(
                R.string.chooseImage)), 1);

I do get an result back.
There is not much in Logcat.
I am using ButterKnifeController which encapsulates activity.

It seems that the problem is "who" is calling startActivityForResult(…).
In my case this must be done via the Controller, and not via activity.
Switching
startActivityForResult(Intent.createChooser(action, getActivity().getResources().getString(
                R.string.chooseImage)), 1);

to
getActivity().startActivityForResult(Intent.createChooser(action, getActivity().getResources().getString(
                R.string.chooseImage)), 1);

results in the same problem.

I am facing the exact same issue, it worked for a few days but then suddenly stopped. Any solutions?

@tobiasKaminsky If you're starting the ImagePicker from Fragment, instead of getActivity() you need to pass fragment instance.
ImagePicker.Companion.with(this)
.start();
In ImagePicker, Two methods are available:

with(activity: Activity)
with(fragment: Fragment)

I believe you're using "with(activity: Activity)" inside the fragment, while you should use with(fragment: Fragment).
Hope this answered your question. Please let me know if you're still facing the issue


work correctly info analysis args

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

To soIve https://github.com/Homebrew/brew/issues/6677, I add the check args.days and args.category for brew info --analytics to handle extra args.

If brew info is going to accept --category and days for formula analytics, then output_formula_analytics should be checking that their values are valid just like output_analytics currently does. It should also be checking for and displaying the categories with hyphens instead of underscores (i.e. "install-on-request" and "build-error") for consistency.

@EricFromCanada

If brew info is going to accept --category and --days for formula analytics, then output_formula_analytics should be checking that their values are valid just like output_analytics currently does.

I understood. I make output_formula_analytics check whether args.days includesvalid_days like output_analytics

It should also be checking for and displaying the categories with hyphens instead of underscores

You mean that like below code?
> brew info solr --analytics --category="install-on-request"
1     | solr                                                                                | 9,645 |  100.00%
==> install_on_request (30 days)
Index | Name (with options)                                                                                | Count |  Percent
-----:|-----|------:|--------:
1     | solr    |   710 |  100.00%
==> install_on_request (90 days)
Index | Name (with options)                                                                                | Count |  Percent
-----:|------|------:|--------:
1     | solr     | 1,821 |  100.00%
==> install_on_request (365 days)
Index | Name (with options)                                                                                | Count |  Percent
-----:|------|------:|--------:
1     | solr      | 8,673 |  100.00%

Thanks so much for your first contribution!  You rock, @hyuraku!

Thank you @MikeMcQuaid .