Support git config

First, excellent gulp plugin!  Really helpful and useful so thanks for your effort.
Second, is it possible to add support for 'git config' to the plugin?
Here's the git man page for it: http://git-scm.com/docs/git-config
Here's an example of how it could be used:
git.config({args: 'core.sparsecheckout true'}, function (err) {
        if (err) throw err;
    });

Adding more non-file based git commands would create a much harder to manage massive plugin. This goes against gulp guidelines.
Referring to #56, use the git.exec function: https://github.com/stevelacy/gulp-git/blob/master/lib/exec.js

@stevelacy Ok. Thanks for the info! I searched the issues for 'config' and nothing came up so I created this one.  In the end, I resulted to using node.process_child.exec which it looks like git.exec does internally.  I missed the git.exec as it was down the page and past the 'init' documentation section which is where I was looking to setup the git repo after the 'init' call.

Ah, yes it is rather hard to notice.

I 'd like push a PR to do this.
const git = require('gulp-git')({
  gitDir: gitDir, // `--git-dir` from `git`.
  pager: false,   // `--no-pager` from `git`.
  config: {       // `-c` from `git`, key-value pairs
    "core.sparsecheckout": true,
    "user.email"<EMAIL_ADDRESS>    "user.name": "Your Name",
  }
});

git.commit();

alessioalex/git-spawned-stream#4

I do not want to increase the API of this project.


Figure out how we can use the previous blog image assets paths.

Before launch, I want to introduce a bucket-based asset system, but for the old content, I'd like to figure out if I can load the images from the static folder.

Closing in favor of #95


Provide an action to navigate to derived resource

I have an ancient visual studio habit of pressing ctrl ~ to flip between the .cpp and .h files.
In my projects, I also often have .inl files, and it's always annoyed me that ctrl-~ doesn't include those while flipping between files.
Could I suggest, adding ctrl-~, but cycling between all associated files (that is, files with the same name, but different file extension).

:+1:

Closing as this can be implemented by an extension


feat: optionally run migrations as an init contianer

As covered in a few other issues and PRs, there's a deadlock situation with the chart when deploying with helm --wait and/or ArgoCD:

The OpenFGA Pod won't start until the migrations have been applied
The Migration Job applies the migrations, and is configured as a Helm Hook
When using helm --wait (or ArgoCD) hooks aren't run until all Pods are healthy
Deadlock: Job won't start because the Pod isn't healthy yet, Pod will never become healthy until the Job runs

Description
This PR introduces an alternative mechanism for running the migrations: an init container on the OpenFGA pod. Instead of waiting for a Job to run, the migrations are ran when the Pod first starts as an init container.
A new migrationType value is added, which can be set to job or initContainer and defaults to job (so is a backwards-compatible change). If set to job the chart behaves as it currently does, but if set to initContainer then the following happens:

the Job is not created
the wait-for-container init container is not included on the Deployment
a new migrate-database init container is included instead

References

https://github.com/openfga/helm-charts/issues/100
https://github.com/openfga/helm-charts/issues/120
https://github.com/openfga/helm-charts/issues/131
https://github.com/openfga/helm-charts/pull/99
https://github.com/openfga/helm-charts/pull/138

Review Checklist

[x] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[x] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

I run into this issue too. The PR will solve it, is there any way to speed up the review?


Fix dbus docs

I have trying to connect using dbus but, for example, on arch there is no pydbus. I have to use import dbus.
The first example in docs could be:
import dbus
bus = dbus.SessionBus()

layout = bus.get_object(bus_name='org.way-cooler', object_path='/org/way_cooler/Layout')

active_id = layout.ActiveContainerId(dbus_interface='org.way_cooler.Layout'))

#dbus-send --session --print-reply --type=method_call --dest=org.way-cooler /org/way_cooler/Layout org.way_cooler.Layout.ActiveContainerId

Last line is the call using dbus-send program

You should install pydbus using pip (I'm also using Arch).
Since there is no dbus library in the standard library, I went with what I saw was the most popular / best maintained.
I'll be adding more ways to represent the examples in the future though. I'll probably target Lua, Ruby, Rust, and the shell (e.g with the dbus-send you showed).
If you'd like to help though, feel free to open a PR adding the shell examples in the docs or add a disclaimer somewhere that for the python examples the user should install pydbus.

Ok. Thanks


Text Editor slow with this extension enabled in large solutions

The Text Editor becomes border-line unusable on a large solution if I enable this extension. Every change to a text file causes VS to use 100% of a core for about 0.5 seconds. Deleting characters gets close to 1sec delay for every character.
This is on Visual Studio 2015 Pro, Windows 10, with a large C++ heavy solution (ex. all of boost), using latest version of VSColorOutput 2.1.1.
Great extension otherwise so I'm sad I have to disable it.

It's not that I don't believe you, it's just that the extension has nothing to do with the text editor. Can't even begin to think why this would be the case.
Still I downloaded boost and tried a couple of projects. I didn't see any slow downs.

My apologies, I spoke too soon. Although the plugin does aggravate the issue, I'm fairly sure at this point that it only does so because of the Visual Studio features it uses. I ran into the same problem later in the loading process with the VSColorOutput extensions uninstalled.
The problem does not seem to happen in SafeMode but I'm still trying to figure out which extension is the bugger.
Anyway, sorry for the false alarm. Closing issue.

No worries.


Kotest Plugin Incompatible with Android Studio Arctic Fox

Hello! There seems to be an issue with installing the Kotest plugin on newer versions of Android Studio [seemingly on Arctic Fox, but I can't say for sure]. Whenever I try to install the plugin I get the below error

Here is my Android Studio install information:

Please let me know if there's anything you can do to help. Appreciate your time!

This might help https://github.com/kotest/kotest-intellij-plugin/issues/133


Add Slovak language

Numbers are spaced out, which is not 100% grammatically correct. It should be easy to adapt should you decide that you want to support the "compact" form as well.
Slovak numerals are declinable, which means that if toSlovak was parametrizable with the grammatical case and gender, the various forms could be generated on demand.
Any plans to implement parsers/generators for ordinal numbers?

 You should sign our Contributor License Agreement in order to get your pull request merged.

Thank you for your interest in contributing to Numsense. The code looks good, and it compiles without warnings on my machine, and all tests pass :+1:

Numbers are spaced out, which is not 100% grammatically correct. It should be easy to adapt should you decide that you want to support the "compact" form as well.

We're having a discussion on a similar topic about the Spanish contribution. Is this something that makes sense to take into account here as well?

Slovak numerals are declinable, which means that if toSlovak was parametrizable with the grammatical case and gender, the various forms could be generated on demand.

We could make it parametrisable. Danish and English doesn't make the distinction, but I'm aware that other languages do. We had a similar discussion about Russian, but here we decided that it wouldn't constrain further options to start with only the masculine form. This option may or may not be reasonable for Slovak; you tell me :smile:

Any plans to implement parsers/generators for ordinal numbers?

Not at the moment.
Given the quality of the code, I'd be happy to proceed. The next step would then be a language review from someone who reads Slovak.
In light of all of the above, though, is there anything you want to change, or should we proceed with the language review?

We're having a discussion on a similar topic about the Spanish contribution. Is this something that makes sense to take into account here as well?

It does. With that said, I think that it would be fine to merge this as is. Once it has been decided on how to approach the problem, I could revisit this in a different PR. You'd have to be a language purist to be offended by this :)

We had a similar discussion about Russian, but here we decided that it wouldn't constrain further options to start with only the masculine form.

It will be a fairly large undertaking to introduce cases, but I can't say whether the structure of the code would have to go through a major overhault or not at this point.
Unless you're of a different opinion, I'd have it reviewed in the current state. I'll be happy to push this further in the future.

As I don't read Slovak, I'd like someone who does to review that part of it, if at all possible. I don't expect any errors, but it's always good with a second pair of eyes :smile:

I solicited a review on Twitter: https://twitter.com/ploeh/status/694537645456756736
Please retweet and spread the word :smile:


[FLOC-1059] [FLOC-2081] [FLOC-2279] [FLOC-2402] [FLOC-2447] [FLOC-2524]

Fixes https://clusterhq.atlassian.net/browse/FLOC-1059
Fixes https://clusterhq.atlassian.net/browse/FLOC-2081
Fixes https://clusterhq.atlassian.net/browse/FLOC-2279
Fixes https://clusterhq.atlassian.net/browse/FLOC-2402
Fixes https://clusterhq.atlassian.net/browse/FLOC-2447
Fixes https://clusterhq.atlassian.net/browse/FLOC-2524
This removes the dependency declared on Docker from the native packages.  Since this means that installing the Flocker packages no longer drags in the Docker package, it also updates the provisioning code to explicitly install Docker.  It uses the Docker-supplied script for doing so (and therefore lets the version of Docker float based on the whim of that script).  Some tests which were broken by that change and were highly duplicative of the implementation they tested have also been deleted.
It also updates the installation documentation to instruct users to install Docker.

The four failing client/(aws|rackspace)/ubuntu-1[45].04 tests can be ignored, since those builders have gone away and been replaced by simpler ones.

Since we're not pinning the Docker version, why is it preferable to use the Docker downloads rather than using the CentOS docker package from the extras repo (currently 1.7.1) and the Ubuntu 15.04 docker.io package (currently 1.5) for testing?  If we used the wget approach for Ubuntu 14.04, then we would have a good range of Docker versions being tested. This would seem to give us a better chance of finding problems before users.

CentOS 7 acceptance test on AWS is failing. running the steps manually gives:
# systemctl start docker.service
Job for docker.service failed. See 'systemctl status docker.service' and 'journalctl -xn' for details.
# systemctl status docker.service -l
docker.service - Docker Application Container Engine
   Loaded: loaded (/usr/lib/systemd/system/docker.service; enabled)
   Active: failed (Result: exit-code) since Wed 2015-08-12 11:24:11 UTC; 17s ago
     Docs: https://docs.docker.com
  Process: 8013 ExecStart=/usr/bin/docker daemon -H fd:// (code=exited, status=127)
 Main PID: 8013 (code=exited, status=127)

Aug 12 11:24:11 ip-172-31-13-88 docker[8013]: /usr/bin/docker: relocation error: /usr/bin/docker: symbol dm_task_get_info_with_deferred_remove, version Base not defined in file libdevmapper.so.1.02 with link time reference
Aug 12 11:24:11 ip-172-31-13-88 systemd[1]: docker.service: main process exited, code=exited, status=127/n/a
Aug 12 11:24:11 ip-172-31-13-88 systemd[1]: Failed to start Docker Application Container Engine.
Aug 12 11:24:11 ip-172-31-13-88 systemd[1]: Unit docker.service entered failed state.

I think the problem with CentOS on AWS is related to the device-mapper-libs package problem listed in https://rhn.redhat.com/errata/RHBA-2015-0993.html
This could be another reason to use the OS package for CentOS, at least.

It seems like we should have some check in the agent service for the existence of Docker, and for the minimum required version, to ensure that any error specifically identifies the problem of a missing or too-old Docker.  I will try to see what error is currently given if Docker is missing, but in the meantime, returning this to Code Backlog for addressing the above comments.

why is it preferable to use the Docker downloads

Probably so that users get the newest version of Docker which seems to be what we want users to get.  For example, 1.7.1 still has the firewalld mis-interaction but 1.8 fixes this.

OK, I am happy for the docs to suggest that users install the latest Docker using the Docker downloads. But I expect some users will use OS packages to install Docker, probably before they come anywhere near Flocker (e.g. the people who use Ansible, Puppet, etc. to keep their systems uniform). Hence, I still think it is worth testing using the different methods described above for each supported OS. (3 tests in total for 3 platforms, not multiple combinations of platform and install method).

I will try to see what error is currently given if Docker is missing, but in the meantime, returning this to Code Backlog for addressing the above comments.

If Docker is not installed, there is no error until a client attempts to start a container. For such a singular error, it would be better for the Flocker agent to check, on startup, its connectivity to Docker, verify the version, and exit with a specific error if it is not happy.  This would work for the container agent, not sure how it translates to the dataset agent.

Points not yet addressed:

I haven't changed the behavior of the container agent in the case where Docker isn't installed.  I agree the current behavior isn't ideal.  Considering our plans for the container agent and the lack of a general way to do centralized error reporting, I'm not sure there's much that can be done about this that's actually worthwhile.
I haven't done anything to expand testing to cover the different Docker packages.  I'm reluctant to undertake any non-trivial BuildBot reconfiguration at the moment given our plans for BuildBot.  Without knowing more about which packages users actually want to use (not just which packages they happen to have arbitrarily selected) I'm also not sure how much value there is in testing additional configurations.  It would be nice to have a systematic way to determine which of these packages users might actually benefit from using so we can focus our efforts on those.

Re container agent, we could add a "Requires=docker.service" for systemd in the [Unit] section, and equivalent upstart thing. And then it would fail to start in a meaningful way. E.g. if I configure Requires=xxx.service then I get failure on systemctl start flocker-container-agent:
Failed to issue method call: Unit xxx.service failed to load: No such file or directory.

Re testing, I'd say something like "Test with current and previous major release" and that's about it, given pace of development. But insofar as we're switching from hard-coded version we can do that as follow-up.

I did only a minimal review insofar as it's been through some rounds.

Address minor comments.
Add the Requires= thing for flocker-container-agent systemd unit file, and equivalent for upstart if there is one. Mysteriously non-working clusters are bad.
Then merge when green.

Oh, and:
4. Add appropriate news entry to documentation.

And I guess:
5. Does anything need to be done to upgrade an existing install? If so document it.

Add appropriate news entry to documentation.

I added to the release notes even though you said "news entry".  There's a NEWS file but I think you meant the release notes.  If not, well, oops.

Does anything need to be done to upgrade an existing install? If so document it.

Probably not.  Although maybe we should suggest that people do upgrade to a newer version of Docker?  Dunno.
Really, we should have some kind of testing in place to verify it's possible to upgrade our software...


add option to configure which sql strategy should be used to reserve

Hi,
in #89 my colleague Christian provided a clean interface to allow us to "configure" the used sql-strategy to lookup the record to be processed next by a monkey patch. We have this monkey patch now in production for a long time and would like to now go a step further and turn this into a configuration option and replace the monkey-patch:
Before:
module Delayed
  module Backend
    module ActiveRecord
      class Job
        def self.reserve_with_scope(ready_scope, worker, now)
          reserve_with_scope_using_default_sql(ready_scope, worker, now)
        end
      end
    end
  end
end

After:
Delayed::Backend::ActiveRecord.configure do |config|
  config.reserve_sql_strategy = :default_sql # default is still :optimized_sql
end

What do you think?
Best, Klaus

:+1:

Looks like some people are using it in production already.
Is there a chance for getting this merged? Ping @sferik

Thanks for merging this!


DOM Manipulation properties and functions intellisense problem

When I write DOM properties. It was working well even after the new release it worked properly but from few hours it had a problem and that pictures will explain the problem
When I type style property it doesn't show anything

getElementsByClassName return an Element (you can check this by hovering) We do not know for sure that it will return an html element with a style property.
You need to add a jsdoc cast to tell VS Code that you really do have html Elements:
const elements = /** @type {HTMLCollectionOf<HTMLDivElement>} */(document.getElementsByClassName('bla'));


support for nix

A really cool project!
I tried signing up with my config, but it fails on step 2, because it can't find the init file.
My init.lua doesn't exist in the dotfiles, because it's generated by nix (which I use to manage plugins).
I'm not sure if it would be feasible to add support for nix, but I've opened this issue just in case 😄

I on purpose try to limit users from adding invalid configs, I currently pick the root of your config (supporting monorepos) based on which init file you pick (init.lua / init.vim / .vimrc). Picking an init file is currently required in order to prevent users from adding completely irrelevant repositories.
Do you have any ideas on how your config could be validated and an init file can be selected?

Would it be possible to just parse the flake.nix file to figure out all plugins? Is there a way to validate that it is a config and not an unrelated nix project?
Thanks for making the issue btw :D It's hard to build software for use cases that you don't know exist!

Would it be possible to just parse the flake.nix file to figure out all plugins?

That's why I'm not sure if it's feasible to add support for nix.
There are quite a few different ways you can manage plugins with nix (mine is a less conventional one).

You can declare plugins as flake inputs (like I do).
Or you can get them from nixpkgs.
You can also use a module system like nixvim.
They don't necessarily have to be declared in a flake.nix. They could be in any nix file that is directly or indirectly imported by the flake.nix.
In Nix, attributes can't always be named the same as plugins are. For example, plenary.nvim becomes plenary-nvim in nix.  You would have to evaluate the nix expression to determine the actual plugin name. This might make automatic plugin detection a lot harder for nix.

Do you have any ideas on how your config could be validated and an init file can be selected?

Technically, you don't need an init file. You could also just have files in <nvim-config-dir>/ftplugin/<filetype>.lua or in another directory that neovim picks up as part of the runtimepath (see :help runtimepath).
Perhaps detecting the presence of any of those directories would help? Although this wouldn't work in all cases either.
I guess some other options for nix would be to grep for a neovim configuration like programs.neovim.
I think it would be a good idea to collect different nix use cases here for a while (or to collect unsupported use cases somewhere) before attempting to implement support.
I'll see if I can find some more use cases after work.

I've just came here report a similar issue. I also think that it is not an easy task to support nix based configurations.
I agree that the best would be to collect various use-cases and maybe start from the simplest ones that could be added with the least effort.
My configuration lives here https://github.com/ghostbuster91/dot-files/tree/master/programs/neovim . It is a mix of different approaches. Some of the plugins are taken directly from the nixpkgs, some are defined as overlays (to override existing ones) and some as new derivations.
I have the init.lua but it gets inlined into home-manager configuration.

One option you could maybe go for is to optionally support a ready-made metadata dump stored in the repo. (i.e. something like .dotfyle-neovim.json in the root of the repo) That file has all of the information you'd want to find, already rendered. Thus any weird setups or different structures can be supported without you having to build a hundred different parsers, assuming the other person is willing to put in the work to generate such a file.

@nogweii See https://github.com/creativenull/dotfyle-metadata.nvim - it's a WIP and not yet supported by Dotfyle, but I will build a parser for it when I'm back from vacation.

Any updates on this?


Email beta testers

When PlanHub is fully open, we should send a thank-you email to everyone who participated in the beta. This tells them:
a. thank you
b. intermittent bugginess and major issues have been solved now because obviously we'd start using a staging environment then
c. other people can use it now, go tell your friends

I'm not sure if this should go on the milestone or not...

@thatoddmailbox Nope but still needed

@medensqui @nywillb @willweinstein YAY I MADE A GDOC FOR THIS

Closing... This is not needed


Use correct color for checkboxes (disabled)

Expecting these checkboxes and check marks to be using "disabled" color

From https://github.com/etn-ccis/blui-react-native-themes/issues/57

https://github.com/callstack/react-native-paper/blob/c190a95e6b2b3842d372ba8f75477752d1042034/src/components/Checkbox/utils.ts#L131 need to update onSurfaceDisabled color code to match design

This will get fixed when https://github.com/etn-ccis/blui-react-native-themes/pull/68 changes are published.

This is fixed


Allow members to temporarily rename their group

Idea
Allow members of a group to rename it.
Keep the guild group name as a default, and place restrictions on the new names.
Implementation proposal
Add a new timer command groupname <new name> which renames the current group.
The name's aren't cached anywhere, so this should be a matter of getting the current timer and setting the name attribute.

The following snippet was hotpatched into a running instance and tested without issue.
An additional check for name duplicates might be useful, although not essential
from cmdClient import cmd

@cmd("rename",
     group="Timer",
     desc="Rename your group.")
async def cmd_rename(ctx):
    """
    Usage``:
        rename <groupname>
    Description:
        Set the name of your current group to `groupname`.
    Arguments::
        groupname: The new name for your group, less than `20` charachters long.
    Related:
        join, status, groups
    """
    timer = ctx.client.interface.get_timer_for(ctx.author.id)
    if timer is None:
        return await ctx.error_reply(
            "You need to join a group first!"
        )
    if not (0 < len(ctx.arg_str) < 20):
        return await ctx.error_reply(
            "Please supply a new group name under `20` characters long!\n"
            "**Usage:** `rename <groupname>`"
        )
    timer.name = ctx.arg_str
    await ctx.embedreply("Your group has been renamed to **{}**.".format(ctx.arg_str))


I need helo

Hello, i dont know anything about programming and things, and i didnt understand how you need to install it, is there a tutorial? thank you in advance

It's in read.md bruh, tutorial is given with specific commands to intall them


Live CSS editor

Is your feature request related to a problem? Please describe.
Editing and testing customcss can be annoying.
Describe the solution you'd like
A live editor. Detects if editing user or global css. Global CSS can save live, but user css should be saved only with a click of a button.
Additional context
Predecessor #64

Well go with just the preview feature currently being implemented.


Instructions for non-optional configuration

I just deployed my first suspenders app and was first greeted by an SMTP configuration error. After removing the SMTP config, I received error: KeyError: key not found: "APPLICATION_HOST".
In #532 @tute mentioned suspenders is not meant to be plug and play. Does it make sense to add something to the README or after suspenders finishes to explain what still needs configuring?

We've been documenting through exceptions until now, which feels very unnatural to me. In https://github.com/thoughtbot/suspenders/pull/722#issuecomment-210556755 I state that I wish for a suspended app not to fail upon deploy to Heroku, but I've had some pushback in moving towards that direction.
Specifically for this case, your PR fixes the APPLICATION_HOST issue (thank you!). SMTP could be conditional or not, but if we don't make it so, I vote for having what's needed in the README.

Uses for APPLICATION_HOST:

Used in Rack::CanonicalHost. This way, requests to other hostnames (such as the Heroku hostname) will redirect. This helps with TLS certs, SEO, and other problems.
Used as the default config.action_controller.asset_host, though can be overridden by setting ASSET_HOST. This value is used in e.g. image_url, stylesheet_link_tag, etc. This ensures that it makes the request using the canonical hostname, which helps with TLS and browser security checks.
Used as the production Action::Mailer host. This way emails will appear to come from the correct domain, which helps ensure reliable delivery.

^ This might all be useful for writing documentation on this.

APPLICATION_HOST is now fixed, SMTP variables are next (failing tests in https://github.com/thoughtbot/suspenders/pull/722 are targeting that).
I'm not sure what variables will fail next, but I do plan to move #722 forward, which leads to fixing this issue.
If in the end we get green CI in different environments, but developer input is still needed (for example, SMTP credentials), then I'll add post-generate instructions so that developers know what to expect.
Thank you all for your input.


Reder problem for Copy button

Hi, here's my content:
:::debug[🪲 DEBUG]
tp.system.**prompt**("Input content：", clip, false, true)
$$\sum_{i=0}^n i^2 = \frac{(n^2+n)(2n+1)}{6}$$
:::

In Read mode, the copy button dispeared:

Hello. Wow, a very interesting bug... I'll look into it as soon as possible 👍

Same happens if theres code in the text.


A11y_SQL Azure Data Studio_Add an Azure account_Add an Azure account_Reflow:On zooming screen content at 400% then  deploy a data tier radio button name is not completely visible.

"Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier."
GitHubTags: #A11yMAS;#A11yTCS;#A11ySev2;#BM_SQLAzureDataStudio_Win32_Jun2021;#SQL Azure Data Studio;#Win10;#FTP;#DesktopApp;#WCAG1.4.10;
Pre-Requisite:
Set display resolution
Environment Details:
Application Name: SQL Azure Data Studio
Windows Version: Win10
Repro Steps:

Launch the application-Azure Data Studio.
Select the Connections icon (if not already selected) from left bar.
Hover over to the 'Azure' header and select the '+' icon.
Hover over to the Azure header in the blade that opens up and click on the "+" icon.
Tab navigate to deploy a data tier radio button and observe its not visible completely on 400% zoom.

Actual:
On zooming screen content at 400% then  deploy a data tier radio button name is not completely visible.
Expected:
On zooming screen content at 400% then  deploy a data tier radio button name should be completely visible.
User Impact:
Low vision user will miss some information if the radio button name is not visible on zooming 400%.

closing the issue, Azure Data Studio requested exception for WCAG 2.1: https://ceacc.visualstudio.com/ceaccex/_workitems/edit/42844/

GitHubTags:#Closed;


Better error messages for bad conditional references

Is your feature request related to a problem? Please describe.
Resolving conditional references as part of transaction bundle processing was implemented under #1329
When the conditional reference cannot be resolved (no matches) or resolves to multiple target resources, the server will now reject the create/update.
However, the error message is a fairly generic "conditional reference could not be resolved" or "conditional reference resolves to more than one resource" (or something like that).
Instead, it would be nice if we could include the path to the offending conditional reference (and/or the conditional reference’s value itself).
Describe the solution you'd like
At minimum: An expression in the OperationOutcomeIssue that specifies the path to the offending conditional reference
Bonus:  the url of the conditional reference that failed in the detail message
Describe alternatives you've considered
Acceptance Criteria

GIVEN a transaction bundle with a condition reference
AND the conditional reference is bogus
WHEN the bundle is posted
THEN an operation outcome is returned
AND it contains an issue that includes a path to the bogus conditional reference

GIVEN a transaction bundle with a condition reference
AND the conditional reference resolves to multiple resources
WHEN the bundle is posted
THEN an operation outcome is returned
AND it contains an issue that includes a path to the bogus conditional reference

Additional context

Confirmed issues created for conditional references in a transaction bundle for the following cases:

reference does not resolve
reference resolves to multiple
reference search is invalid

in all cases the resulting issue contains an expression indicating the location (Bundle.entry[x])
confirmed varied location of failure within the bundle were represented accurately
example:
{     "resourceType": "OperationOutcome",     "id": "ac-11-0-2-1aafb23b-2ef4-4798-bb94-8fc5fe053f66",     "issue": [         {             "severity": "fatal",             "code": "not-found",             "details": {                 "text": "Error resolving conditional reference: search 'Patient?_id=PatientBogus' returned no results"             },             "expression": [                 "Bundle.entry[0]"             ]         }     ] }


SwiftAudio is not linked on 1.2.0

SwiftAudio is not linked on 1.2.0
It is required to build on ios, it is possible to add it to dependency ?

https://github.com/react-native-kit/react-native-track-player/pull/802 Pr pending


Report bug upstream: all_strings value length check inconsistent

There are 2 implemented _all_strings methods and in one the len(value) > 0 check occurs only when if strip is True.
This is NavigableString
        value = self
        if strip:
            value = value.strip()
        if len(value) > 0:
            yield value

This is Tag (where multiple descendants are looped over)
            if strip:
                descendant = descendant.strip()
                if len(descendant) == 0:
                    continue
            yield descendant

Is this a bug?
If so it would give inconsistent handling between NavigableString vs. Tag
Potentially should be reported upstream.

Fixing it doesn't make any tests pass so either this isn't a genuine bug (no effect) or it is but the test suite doesn't cover it.

don't think this is a bug after all, empty string inside a tag is simply not parsed as any text (due to looping thru descendants)


[Discussion] 3.0 - Removing Microsoft.AspNetCore.Mvc.WebApiCompatShim

The Microsoft.AspNetCore.Mvc.WebApiCompatShim (WebApiCompatShim) package provides partial compatibility in ASP.NET Core with ASP.NET Web API 2 to simplify migrating existing Web API implementations to ASP.NET Core. However, apps using the WebApiCompatShim do not benefit from the API related features shipping in recent ASP.NET Core releases, such as improved Open API Spec genertaion, standardized error handling, and client code generation. To better focus our API efforts in 3.0, we plan to remove WebApiCompatShim. We recommend that existing applications using the WebApiCompatShim migrate to the newer [ApiController] model.

This package was downloaded about 600 000 times. It's a fairly high number when you know it's not referenced by the Microsoft.AspNetCore/Microsoft.AspNetCore.All metapackages.
Are you sure that removing this package - that shouldn't be that time-consuming to maintain in its current state - is worth the pain it will cause to the ones relying on it, who will be forced to choose between migrating away from the Web API 2 primitives or staying on 2.2 forever?

@pinpointtowners: 2.x is an LTS release. This gives a few years to remove the shim dependency from your code.

@Sebazzz sure, but it doesn't make the migration less painful: you just have more time to do it.
This package was a (great) way to encourage Web API 2 users to migrate without having to dedicate massive human/financial resources to rewrite all their controllers (apps with hundreds of controllers are not that rare).
Removing it at this point is like abandoning them in the middle of the road and telling them they must now migrate the hard way, for a maintenance gain that seems inexistant to me (the history seems pretty clear: https://github.com/aspnet/Mvc/tree/release/2.2/src/Microsoft.AspNetCore.Mvc.WebApiCompatShim).

I've been looking at the usage of this and at this point I still think we will remove it in 3.0. But there's still time to change that, I'm not locked in.
I'm willing to talk about either not removing it or porting some of the functionality in the shim. Some of it could become features you can turn on when using ApiController if we had a better understanding of what parts of the shim were most useful to folks. If you have opinions or thoughts on that then let me know.

Some of it could become features you can turn on when using ApiController if we had a better understanding of what parts of the shim were most useful to folks.

IMHO, the ability to return instances of HttpResponseMessage and throw HttpResponseException exceptions (independently of whether it's a good or bad practice) exactly like the way you did in Web API 2 is probably one of the most important aspects, as you don't have to rewrite all your controllers to work with the new world model.
This part could certainly be replaced by something like a Roslyn-based rewriter, that would help move from the old pattern to the new IActionResult-based one. I'm a bit surprised this wasn't mentioned and proposed as a way to make the migration less painful once the shim package is gone.

@PinpointTownes I've filed a couple of follow up issues (https://github.com/aspnet/Mvc/issues/8703, https://github.com/aspnet/Mvc/issues/8704) based on this discussion.
A rewriter (Roslyn codefix?) certainly sounds like an interesting idea, I'm not entirely sure how well it'd for all but the most trivial of user codes. That said, the nice part about writing Roslyn extensibility is that the community can step up and contribute something.


feat: Use Fetch instead of XHR when available

Took best of both worlds from:

https://github.com/getsentry/raven-js/pull/1156
https://github.com/getsentry/raven-js/pull/1115

and added tests coverage.

Tests are broken on IE10/11 and mobile chrome. I'll fix them before merging.


change(types): ModifyThread.locked has been renamed to onlyModeratorsUnarchive

Pending until this name change is finalized

They decided to keep it locked


License question

The repo says the license is Apache 2.0, but the decNumber library seems to be GPL. Is it really possible to license this library as Apache 2.0?

Afaik decNumber is dual licensed as gpl and bsd.
On Dec 14, 2016 12:07, "Simon Andersson"<EMAIL_ADDRESS>wrote:
The repo says the license is Apache 2.0, but the decNumber library seems to
be GPL. Is it really possible to license this library as Apache 2.0?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alkis/decimal/issues/22, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQO_djiLC4Vs6HIUGx5SEeipDHhy5obks5rH83mgaJpZM4LMzE-
.

Thanks @alkis! Do you have a link that supports this claim?

Here is something in the header: https://sourceware.org/svn/gcc/tags/stack-last-merge/libdecnumber/decNumber.h

The copies in your repo says 'ICU License', where does these files come from?

http://speleotrove.com/decimal/decnumber.html
Lists two sources. One from ICU and one from gnu gcc. The licences are ICU
and GPL perspectively.
On Dec 14, 2016 12:24, "Simon Andersson"<EMAIL_ADDRESS>wrote:

The copies in your repo says 'ICU License', where does these files come
from?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alkis/decimal/issues/22#issuecomment-267010457, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAQO_WlY0KYuou7QVQknXr7Uu3bUA3yLks5rH9HrgaJpZM4LMzE-
.

Thanks! So the ICU license is compatible with Apache 2?

Yes. Read this:
https://www.apache.org/legal/resolved#category-a
On Wed, Dec 14, 2016 at 12:31 PM, Simon Andersson<EMAIL_ADDRESS>wrote:

Thanks! So the ICU license is compatible with Apache 2?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alkis/decimal/issues/22#issuecomment-267011581, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAQO_bawSaDj1QRibSXR2jTL4Pb9ivmQks5rH9N-gaJpZM4LMzE-
.

Thank you, good link! My question is answered.


January is not recognized by date_extractor?

Heya,
This is a wonderful library. Hoping to use it to extract dates in a project, but I noticed that January consistently fails to be properly extracted in tests. I'm wondering if this is a subtle bug with indexes / accidental type coercion of 0 to false in date_extractor.coffee.
Would be happy to help you track down the issue if you have trouble.
Here is an example I just tried on master:
nlp.value("Today is January 7, 2015").date()
{ month: null,
  day: 7,
  year: 2015,
  to_day: null,
  to_year: 2015,
  to_month: null }

Hey, you know what, I think I figured it out:
parseInt("0") || null
returns null
Dat rough JavaScript lyfe.

How about we replace || with ? and call it a day? Gonna submit a pull request in a little bit after I do some tests.

 Will merge any fix, thanks for the nice words
On Wed, 7 Jan 2015 7:27 pm Ethan Sherbondy<EMAIL_ADDRESS>wrote:

How about we replace || with ? and call it a day? Gonna submit a pull
request in a little bit after I do some tests.
—
Reply to this email directly or view it on GitHub
https://github.com/spencermountain/nlp_compromise/issues/15#issuecomment-69076739
.

Well, I tried to fix things, but something went awry.
https://github.com/inboxapp/nlp_compromise
January works, but now half the time I call date(), null is returned.
There seems to be some bit of state that is persisting between calls, which is quite unfortunate.
Let me know if you get to the bottom of this, or manage to successfully cherry-pick a subset of these changes into a working fix!

hey man, this is fixed in v0.3.0


single letter word removed when numbering

Hi there, great tool so far; thanks for producing it and making it available.
I have noticed that if I start a title with a "one letter word" it is removed on numbering. for example, I have a heading "G Suite" and when numbering the numbering is applied correctly but the "G" disappears so the title becomes "X Suite" X being the heading number.
Thanks!

Thanks for the note. Yes, this is known issue with single letters starting a heading. I am thinking that the best way of dealing with this is to remove the option to do heading numbers with letters. I added that feature in haste, without really thinking it through.

Thanks. I see. I have only used headings with numbers so far but I would
guess there might be others using the heading numbers letters so adding the
feature might cause more harm than good? I mean, it's a minor issue I can
live with :-)

I am thinking of releasing a v2 of this plugin that will have some cool features like the ability to skip certain heading names and not number certain heading numbers etc.  In this version I will remove the ability to use letters for heading numbers, since I am not sure people use it.  But this will be Markdown-Tools-v2 so it will not impact any users of the current tool.  Let me know if something like this would work for you and if you have any features you would like to see.

Again, no strong opinion myself as I haven't faced such use case. Skipping
would be good. I wish Google would also do so in the ToC of the GDocs. I
have upvoted the request for them to implement.
El vie., 8 may. 2020 a las 19:34, jordan2175<EMAIL_ADDRESS>escribió:

I am thinking of releasing a v2 of this plugin that will have some cool
features like the ability to skip certain heading names and not number
certain heading numbers etc. In this version I will remove the ability to
use letters for heading numbers, since I am not sure people use it. But
this will be Markdown-Tools-v2 so it will not impact any users of the
current tool. Let me know if something like this would work for you and if
you have any features you would like to see.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jordan2175/markdown-tools/issues/7#issuecomment-625930834,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJPOGRKHUWYZ22SG6UOUNDRQQ7D5ANCNFSM4M4IUPOA
.

A possible workaround is to use Punctuation space in headings after single letters at the beginning. The AddOn does not treats that character as a space so it thinks it is not a single letter.
In fact it'd be more convenient with the no-break-space character which is more widely known, but unfortunately that one is treated just like a regular space.
So for now I'm using this character: " " (You can copy from here!) in my headings when my heading starts with a single letter, which in fact happens a lot as "a" is the definite grammatical article in Hungarian and used a lot as the first word of titles. But I think also in English it may happen fairly often to have a heading that starts with "a", as the indefinite grammatical article.
So I'll just leave it here for other folks facing the same issue: Use unicode 2008 punctuation space in the headings and it'll ensure the A (or other single letters) are not removed.

Interesting one! Thanks for looking into this again.
El mié., 17 jun. 2020 a las 22:33, Bence Szalai<EMAIL_ADDRESS>escribió:

A possible workaround is to use Punctuation space in headings after single
letters at the beginning. The AddOn does not treats that character as a
space so it thinks it is not a single letter.
In fact it'd be more convenient with the no-break-space
https://en.wikipedia.org/wiki/Non-breaking_space character which is
more widely known, but unfortunately that one is treated just like a
regular space.
So for now I'm using this character: " " (You can copy from here!) in my
headings when my heading starts with a single letter, which in fact happens
a lot as "a" is the definite grammatical article in Hungarian and used a
lot as the first word of titles. But I think also in English it may happen
fairly often to have a heading that starts with "a", as the indefinite
grammatical article.
So I'll just leave it here for other folks facing the same issue: Use unicode
2008 punctuation space https://www.compart.com/en/unicode/U+2008 in the
headings and it'll ensure the A (or other single letters) are not removed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jordan2175/markdown-tools/issues/7#issuecomment-645608993,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJPOGQNDIHXMMJ4WAPYVF3RXESAPANCNFSM4M4IUPOA
.

Yes, Hungarian (Magyarul) does use a lot of "a".  I like the idea of using that special space character.  I should try and add that to the code so it does that automatically. I have been working on a newer version of the add-on with a lot more features.


Sneak action self references

The Sneak action (UUID Compendium.pf2e.actionspf2e.Item.VMozDqMMuK5kpoX4 ) links to itself in the second paragraph instead of a Stealth roll.

No sign of this.


refactor(storage): add dedicated error type for StorageTable

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?

Return deserialize row error for deserialization errors.
Otherwise just return the existing storage error.
Cleanup some unused variants (ArrayError in StorageError, StateTableError in RwError).

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)
closes #5272

Actually, I don't think we should have a dedicated error type for both tables: the Table is actually at a layer above the state store, but it's still in the storage crate, which is the error type exposed to other crates like stream and batch. We've made HummockError a variant of StorageError, and the error generated from tables is a superset of HummockError, including some new types of (de)serialization errors. So why not simply make these new types some new variants of the storage error enum (like StorageError::Deserialize(xx)) and not erase the type of StorageError in a dedicated error type?
By the way, there's no need to use different variants just because they're generated from different places: we have the backtrace to show which line exactly caused the error.
cc @wcy-fdu

Should we merge errors in table layer in HummockError?

Actually, I don't think we should have a dedicated error type for both tables: the Table is actually at a layer above the state store, but it's still in the storage crate, which is the error type exposed to other crates like stream and batch. We've made HummockError a variant of StorageError, and the error generated from tables is a superset of HummockError, including some new types of (de)serialization errors. So why not simply make these new types some new variants of the storage error enum (like StorageError::Deserialize(xx)) and not erase the type of StorageError in a dedicated error type?
By the way, there's no need to use different variants just because they're generated from different places: we have the backtrace to show which line exactly caused the error.
cc @wcy-fdu

Make senses to me, I will refactor the state table errors later.

Should we merge errors in table layer in HummockError?

We may have some variants along with the HummockError.
pub enum StorageError {
    #[error("Hummock error: {0}")]
    Hummock(
        #[backtrace]
        #[from]
        HummockError,
    ),

    Deserialize(xxx),
    Serialize(xxx),
}

So if the error is from the state store (or keyspace), it should already be a StorageError, we can simply rethrow it. If it's some error generated by the table itself, it should be wrapped to these new variants and become a StorageError.

Should we merge errors in table layer in HummockError?

We may have some variants along with the HummockError.
pub enum StorageError {
    #[error("Hummock error: {0}")]
    Hummock(
        #[backtrace]
        #[from]
        HummockError,
    ),

    Deserialize(xxx),
    Serialize(xxx),
}

So if the error is from the state store (or keyspace), it should already be a StorageError, we can simply rethrow it. If it's some error generated by the table itself, it should be wrapped to these new variants and become a StorageError.

Actually most errors except deserialize is from keyspace,  so I believe when debugging in table layer, HummockError is enough.

Should we merge errors in table layer in HummockError?

We may have some variants along with the HummockError.
pub enum StorageError {
    #[error("Hummock error: {0}")]
    Hummock(
        #[backtrace]
        #[from]
        HummockError,
    ),

    Deserialize(xxx),
    Serialize(xxx),
}

So if the error is from the state store (or keyspace), it should already be a StorageError, we can simply rethrow it. If it's some error generated by the table itself, it should be wrapped to these new variants and become a StorageError.

Actually most errors except deserialize is from keyspace, so I believe when debugging in table layer, HummockError is enough.

Do you mean for StateTable, there's no need to add anymore variants since these are already part of HummockError, apart from StorageError::DeserializeRow?


makefile: Ignore errors on clean test

Makefile can ignore commands if failed. clean-test target deletes
the namespaces. But if the namespace does not exist then kubectl
command fails and so does the clean process.
This commit ignores the error returned by kubectl. And continues
with the rest of the commands.

cc: @indradhanush

@indradhanush done.

Merged on the command line.


Adding lockfile to the project

Latest NPM create a lockfile during install. Adding it here.
Additions

NPM Lockfile

Testing

Run npm install, check that the lockfile didn't change (@contolini, how will we handle changes if that happens between PR submission and testing?)
Run npm run cf-link (This has a side effect of taking care of the sub-dependencies in each component package.json)
Run gulp build Ensure it builds without errors

Review

@contolini

Notes

cfgov-refresh now requires the newest versions of Node and NPM. Upgrading other projects so there is parity.

Checklist

[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows the standards laid out in the front end playbook
[x] Passes all existing automated tests
[x] New functions include new tests
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged
[x] Visually tested in supported browsers and devices
[x] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

DO NOT MERGE THIS. I ran gulp build, remembering it's part of the release process, even if we don't typically run it within this project, and it's riddled with errors. Gotta do some more exploration here.

Realized my error when testing, updated testing instructions.

This is super weird, running npm install removes the comments from the package.json files, except for one line. Does anyone know if comments within package.json are now limited?

@contolini you and me both


crash when deleting ets table.

I get this CRASH when deleting an ets table:
CRASH REPORT Process <0.4972.42> with 0 neighbours crashed with reason: bad argument in call to ets:delete(11567156) in folsom_sample_exdec:delete_and_rescale/4 line 122

Folsom has moved, please resubmit your issue at https://github.com/folsom-project Thanks!


Cannot update version

Body
can not update code
Version
0.2251
What browsers are you seeing the problem on?
Microsoft Edge
What OS are you seeing the problem on?
Windows
Anything else?

Same here

#999


test with python 3.9

Scipy and matplotlib are still build from source so this will need either to wait or setup caching first

@astafan8 I think we can merge this as is. Matplotlib is still build from source but that takes less than 3 min so that seems acceptable


Allow bulk assignments to decide variant

/domain @aburgel
/platform @jmileham
This allows for variant to be left blank on bulk assignments. If it is blank, the bulk_assignment will run the variant calculator and assign visitors as if they'd actually experienced the split. It also will not override existing assignments since these would have been deterministically decided already, or overridden via the chrome plugin which we want to honor.

Closing per offline discussion, needs more thought about long term solutions for eagerly assigned splits.


update to ATT&CK v12

Hi,
I tried to upload the Azure and GCP json from your official website to the ATT&CK Navigator:

Microsoft Azure Security Control Mappings to MITRE ATT&CK®
Google Cloud Platform Security Control Mappings to MITRE ATT&CK®)

Then I hit the export Button on the Navigator and re-imported. When prompted, I choose yes, upgrade to ATT&CK Navigator v12 for both of the files. I need a common version because otherwise I cannot run a comparison with the Attack Navigator between the two Azure&GCP.
The GUI (Attack Navigator) represents the content correctly but when I hit export in the Navigator and then directly re-import that export, some entries in the matrix disappear (e.g. in Azure the Phishing in the third row is completely empty).
I verified that this issue is not present after exporting/importing when I choose "no" to not upgrade to ATT&CK Navigator v12.
But by not upgrading to a common Version I cannot use the Mitre Att&ck Navigator comparison feature.
PS: I need the export because I wrote a python script that deleted everything but the features in the "protect" category, so I can compare the two cloud providers. Thatwhy I am dependent of the export/import feature.
PPS: Before I wrote the python script, I studied your github Tool section. You have a nice option in your python script to filter on --category Protect but unfortunately this is only for direct console output. It is not possible to have the Navigator JSON exporten with a filter for "only category=protect". Would be nice if this is possible - so I would not need my own python script.

I am able to reproduce the problem, but unsure what the root cause is. I noticed that if I open the layer without upgrading to v12 then the data is captured fine, so I suspect this is an issue/limitation with the way Navigator upgrades layers. Azure is v8 → v12 and GCP is v10 → v12. @clemiller can you weigh in on this?
Ideally we will get the mappings updated to v12 this year, but no firm timeline on that.

@MoD01 I talked with @clemiller (who knows the Navigator tool much better than I do) and I think the solution is to use the Navigator upgrade wizard to copy annotations over from the old versions to new. This document explains how to handle annotations: https://github.com/mitre-attack/attack-navigator/blob/master/USAGE.md#upgrading-a-layer-to-the-current-version
Here's a screenshot showing the process e.g. for T1189 Drive-by Compromise. You can click the "Show annotated techniques only" checkbox to make this process faster.


command not found

When I try to run the command I get the following error:

I'm currently on Pop!_OS 18.04 LTS, and my VSCode info is:

Hey! I had this issue in an earlier 0.0.x versions, but it should work in versions 0.1.0 and above.
Can you check which version of the plugin you have installed?

VS Code System Information
OS Information

key
value

arch
x64

platform
linux

type
Linux

release
4.15.0-34-generic

EOL
"\n"

endianness
LE

Process Information

key
value

arch
x64

versions

key
value

http_parser
2.7.0

node
8.9.3

v8
6.1.534.41

uv
1.15.0

zlib
1.2.11

ares
1.10.1-DEV

modules
57

nghttp2
1.25.0

openssl
1.0.2n

env

key
value

LANG
en_US.UTF-8

VS Code Information

key
value

version
1.27.2

env

key
value

appName
Visual Studio Code

language
en

extensions
VSCode HackerTyper

open in marketplace
open in vscode

key
value

id
jevakallio.vscode-hacker-typer

isActive
true

name
vscode-hacker-typer

version
0.1.0

displayName
VSCode HackerTyper

description
Hacker Typer extension for looking cool while live coding

publisher
jevakallio

categories
["Other"]

That's odd! I'll have to investigate the root cause, Works On My Machine ™

Haha yeah, I figured it was one of those situations. I'm not sure what OS you run, but maybe it's a linux issue? Thank you!

Can you try to install v0.1.1 and try again? I noticed there was a casing issue in one of my imports, which will work on OSX case-insensitive file system, but will very likely fail on a case-sensitive fs.
I haven't tried it on Linux, don't have any VMs on this machine, so if you could give it a shot and report back, that would be great!

I have exact the same issue on macos mojave. Plugin was installed today

Somehow I have a similar problem, with the "save macro" command.
VS code system info:
Version: 1.30.1 (user setup)
Commit: dea8705087adb1b5e5ae1d9123278e178656186a
Date: 2018-12-18T18:12:07.165Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
OS: Windows_NT x64 6.1.7601
I will try later on today on a different PC (non-work PC) to see if that somehow makes a difference.

Not running on my VSCODE

https://github.com/jevakallio/vscode-hacker-typer/pull/23 fixed the issue for me.

Using version 0.2.1 of nodename.vscode-hacker-typer-fork, I attempted to record a macro and got: Command 'HackerTyper: Record Macro' resulted in an error (command 'nodename.vscode-hacker-typer-fork.recordMacro' not found). I've tried restarting VSCode several times, as well as uninstalling/reinstalling this extension.
VSCode info:
Version: 1.44.2 (user setup)
Commit: ff915844119ce9485abfe8aa9076ec76b5300ddd
Date: 2020-04-16T16:36:23.138Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363 [Windows 10]

Perhaps it would be best to discuss issues about the fork at https://github.com/nodename/vscode-hacker-typer/issues .
I will say please have the status bar open (View -> Appearance -> Show Status Bar) while you reproduce this issue, and tell me at that issues page what you see. Thanks!


Update lpamiodarone.json

It mentions "is in line with NICE Guidance CG180" (taken from NHS E website) - but on checking this was replaced in 2021 by NG196. Presume we should reference/link to the appropriate guidelines.

@chrisjwood16 my view is that we are quoting "NHS England states"...  there are a lot of issues with their wording, and if we try and correct we'll tie ourselves in knots, as these things aren't mentioned in later NICE guidelines.  I've got a list that I will email them with, but let's use their wording for now

@chrisjwood16 my view is that we are quoting "NHS England states"... there are a lot of issues with their wording, and if we try and correct we'll tie ourselves in knots, as these things aren't mentioned in later NICE guidelines. I've got a list that I will email them with, but let's use their wording for now

Ah just pushed an update. They do link to the correct guidance further down, so think it's justified here to put correct link?

we're going to come across this time after time - they are often referring to (@brianmackenna 's?) original work here, so as we're quoting I suggest we stick with what they're saying


Wrong instance in typespec used in package

If we declare a type in package A, and use it in package B (e.g. as a global variable), the type's vpiInstance is set to B instead of A.
Example. For this code:
package A;
   typedef enum int { x, y } C;
endpackage

package B;
   A::C d;
endpackage

Surelog produces this UHDM:
\_package: A (A::) bug.sv:1:1: , endln:3:11, parent:work@M
  |vpiDefName:A
  |vpiName:A
  |vpiFullName:A::
  |vpiTypedef:
  \_enum_typespec: (A::C), line:2:30, endln:2:31
    |vpiName:A::C
    |vpiBaseTypespec:
    \_int_typespec: , line:2:17, endln:2:20
      |vpiInstance:
      \_package: A (A::) bug.sv:1:1: , endln:3:11, parent:work@M
    |vpiEnumConst:
       ...
    |vpiInstance:
    \_package: B (B::) bug.sv:5:1: , endln:7:11, parent:work@M
      |vpiDefName:B
      |vpiName:B
      |vpiFullName:B::
      |vpiVariables:
      \_enum_var: (B::d), line:6:9, endln:6:10, parent:B::
        |vpiName:d
        |vpiFullName:B::d
        |vpiAutomatic:1
        |vpiVisibility:1
        |vpiTypespec:
        \_enum_typespec: (A::C), line:2:30, endln:2:31

Notice that the vpiInstance in our typespec A::C is pointing to package B, whereas it should be pointing to A.
Same thing happens with a struct instead of an enum, and also if C is used as a field in a struct in B, not as a global variable.

Fixed by #1372


Added support for CommonJS

Hello!
I've introduced a CommonJS support according to Angular.js best practices. This allows to use the module with modern build tools like WebPack and Browserify.
Could you please merge it and create an immediate patch release?
Thank you!

Merge it please...

Yea, I'd love to see this merged...

UP!
https://github.com/angular-ui/ui-router/issues/3031


Back Slash () is Not Correctly Escaped in WYSIWYG Mode.

Describe the bug
The Backslash is Not Escaped Correctly in WYSIWYG Mode.
To Reproduce
Steps to reproduce the behavior:

Open the editor in here: https://ui.toast.com/tui-editor
Switch to WYSIWYG mode.
Type the following content: \\\\ (4 backslash)
Switch to Markdown mode, the escaped text is: \\\\\\ (6 backslash), the display text in WYSIWYG mode is \\\ (3 backslash)
Switch back to WYSIWYG mode, the input text become \\\ (3 backslash)
With any amount of backslash greater than 2, this bug happens.

Expected behavior
Switch back and forth between 2 modes will keep the same amount of backslash: \\\\ (4 backslash)
Screenshots

+1 I am having the same issue with this.

@gigawave07
Thanks for reporting a bug. I'll fix it!

Hello @jajugoguma!
Is there some progress on this?
I've encountered the same problem.

여전히 문제네요 위지웍모드에서 마드다운전환시 \가 추가됩니다.


Documentation Issue and Python 2.7

There is an issue https://github.com/SharePoint/sp-dev-docs/wiki/Setup-your-machine.
The installation of Python is only required on Windows but not on other systems.
In addition you need to install a C++ compiler or VC++ runtime on Windows. SPFX use node-gyp an doesn't work on Windows without those compilers.
When you have Visual Studio installed you don't have a problem because it installs a C++ compiler/runtime.
Node-gyp has a detailed documentation of prerequesits for Windows, Mac and Linux. see node-gyp

Chaks - can you clean up the "setting up your machine" steps?

Another issue that might make it hard for people that just getting started with nodejs and yeomen. I always write in my documentation the full command instead of the abbrivation.
npm install -g yo gulp
instead of
npm i -g yo gulp
Might makes it more clearer what the command does when someone just started with NodeJS and especially Yeoman.

installing the generator with npm i -g @microsoft/generator-sharepoint --no-optional gets rid of the missing python and c++ stuff - and as far as I can tell, I can run everything just fine.
FYI: I don't have python installed on my machine, and as far as C++ is concerned I might have some runtimes as they where installed by some other (MSI-based) installers.

@nyn3x  --no-optional works fine on Windows tried it a couple of times but the problem is that some compilation times are much slower then they should be.
SASS compilation on large projects using the native Ruby-based implementation of SASS can cause 10 seconds or more.
The same source compiled with a native C-compiled SASS extension only took about 10 milliseconds.
Same thing applies to other components too. The reason for node-gyp is simply performance improvment with some extensions.

hmm - I see. That's not good, I haven't noticed as most my environments are missing python - so I don't know what I'm missing out on :)
OT: are the bad compilation times because of the missing node-gyp or because of the --no-optional flag?

I added a known issues doc highlighting the python issue. I tested in a fresh Windows machine and Mac to make sure they work.

@nyn3x Node-gyp is always installed when you install NodeJS. The question is will it work or break because of so called missing "development dependencies'. In other words Python and C++.
The rest is a little bit tricky to explain.
I think SASS in this case is a good example for this. You can install ruby and then use SASS the native way. The performance is somewhat good but you have the dependency on ruby.
To make SASS independant from the ruby installation the SASS team wrote a native C++ component that does exactly the same job but faster. Naturally it is a C compiled code. This libsass library is used by most IDE such as VS Studio, VS Code, Atom, Sublime, ... you name it.
To be platform independent you have to compile the C++ source for each platform you are on or precompile. NodeJS solve this issue a little bit different because you can integrate any C++ or Phython writen code into NodeJS. This is exactly what node-gyp does. Compile native C++ or Python Scripts and make it available in NodeJS but compile it on the platform you are on.
The benefit: It just compiles no matter which version you are on.
You can even use Yeoman on a raspberry pie if you like. Pyhton and C++ are installed ther by default.
The problem is on Windows systems they are not installed there.
Node-gyp is pretty common in NodeJS development and is used by a lot of tools.
If you switch to --no-optional the native NodeJS code will be used instead of the performance optimized C++/Python code.
If you like to get the full experience on Windows you have to install Python 2.7 and C++. If not use -no-optional switch. In future the goal of Microsoft is to support it on Windows trough Visual Studio or if you don't like to use Visual Studio you can run a Linux subsystem on Windos aka 'Bash on Windows'

@chakkaradeep You might want to tweak the Phantomjs known issue to clarify that both the python directory and the scripts subdirectory need to be on the PATH.
I had 2.7.10 installed to c:\tools\python2 and that directory was already on the system PATH. I still got the error. Adding c:\tools\python2\scripts to the PATH cleared it up.
Incidentally, the 2.7.12 MSI installer adds both to the PATH if you select that option.

@jshowatt Thanks! Updated here and here.

Closing this issue as the documentation has been updated.


ConvertTo-PodeWebPage -Module DBATools : You cannot call a method on a null-valued expression

I am new to using Pode.web but not new to Powershell. I am very interested in using Pode.web to automatically build a web interface to execute powershell functions in a module.
I just ran across ConvertTo-PodeWebPage. This appears to work perfectly for the example of the Powershell Module 'Pester'.
https://badgerati.github.io/Pode.Web/Tutorials/Pages/
However, When I try to use the same code and use the Module 'DBATools', I get the following error:
InvalidOperation: C:\Users\jmill\Documents\PowerShell\Modules\Pode.Web\0.4.1\Public\Pages.ps1:457:9
Line |
 457 |          $paramDefs = $ast.FindAll({ $args[0] -is [System.Management.A …
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | You cannot call a method on a null-valued expression.

Here is the code I am running:
Import-Module -Name Pode.Web

Start-PodeServer {
    # add a simple endpoint
    Add-PodeEndpoint -Address localhost -Port 8080 -Protocol Http

    # set the use of templates
    Use-PodeWebTemplates -Title 'DBATools'

    # convert module to pages
    ConvertTo-PodeWebPage -Module Pester -GroupVerbs
}

Ooo that is interesting, it would suggest there are functions in that module with no ScriptBlocks - or it could be detecting aliases possibly 🤔 I'll try and grab the DBATools module this weekend and test.
Thanks!

Thanks! There are over 500 public functions in that one. It is a great team that develops it. I can definitely work with them if something needs to be modified on that side. They are very meticulous in writing a very standardized and full-featured PowerShell module with very good test coverage.

The above commit should now fix it 😄 It wasn't playing ball with cmdlets or functions containing nested functions where parameter names were the same, but should handle both gracefully now - I installed the module and the page building works for me :)

I apologize. I use bitbucket normally (business). I do not see any issues with the change you made. However, I cannot find a good way to test your commit myself in Github. I do not have an issue with you merging to master with the code you have. However, if you have some feedback on how I can test the commit, I will be more than happy to.
Thanks again!

I have tested with dbatools locally and the pages do render now, so if you're happy with that I can push it with the next release. However, if you'd like to test you should be able to copy the content of the file (https://github.com/Badgerati/Pode.Web/blob/db517c6674257f7e343ad1201ae9aef45e5fef72/src/Public/Pages.ps1), and paste it into your installed Pode.Web module's /Public/Pages.ps1 file :)

I tried this in Powershell 7.1.3 and Windows Powershell 5.1. Both produced the same error below. I tried remove-module pode.web and reimporting again even with -force.
Get-PodeWebPagePath: C:\Users\jmill\Documents\PowerShell\Modules\pode.web\0.4.1\Public\Pages.ps1:320:16
Line |
 320 |          Url = (Get-PodeWebPagePath -Name $Name -Group $Group)
     |                 ~~~~~~~~~~~~~~~~~~~
     | The term 'Get-PodeWebPagePath' is not recognized as a name of a cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was
     | included, verify that the path is correct and try again.

Oh! I've just realised the version of Pode.Web you're using - if you're able to, upgrade to the new v0.5.1 and paste the page in again. Or, if you can't then you'll have to manually copy in the edits from the commit's diff into the Pages.ps1 file.

Thanks and good catch. That was the issue. I updated both Pode and Pode.web locally and that resolved the issue.
This is working properly for DBATools.
I really appreciate the quick responses.


append

Hi first of all let me start saying your tool looks very promising especially the level of description and details written. Before i jump into installing every thing and trying I have few simple questions and your answer will be highly appreciated .

Does it support append as output sql ?
Does it support innodb ?

Hi @mandevx,
Yes, the output is all SQL.
Yes, innodb should be supported (both databases you are comparing should use the same storage engine otherwise this will also be noted in the output SQL diff!)


Include namesdir.com url in Username Search Engines folder

See the following 2 points for how namesdir.com can help when doing OSINT:
-- Given a name (first name or last name), namesdir can help suggest usernames that someone with that name could likely use. So if you have been able to OSINT a name to a platform, namesdir can provide good username suggestions. To see these username suggestions, go to namesdir.com, search for a name, click on the name and scroll down to "Typos & Misspells"
-- Imagine this, you're on an OSINT hunt, you need to determine the full name of your OSINT target person, you come across an image with a name. You have a good hunch that this is the name of your target person. However, the image is blurred out or some characters of the name in the image are unreadable. You notice you are only able to read the first name perfectly as Jake. The last name has 10 characters and you can only read 3 (Mag) out of the image. In this case, namesdir.com can help you determine a good guess of the remaining 7 characters of the last name. Go to namesdir.com, search for Jake, select "Jake given name" and just Ctrl F the results for "Jake Mag". Quickly, you realize there is a last name called Magallanes making you figure out the full name of your target person as Jake Magallanes.

So, What;s important error or flag point in this site (Please describe as discuss further) ?
I readed about this message in email.


Mobile device support - Bug touches

Hi !
Nice plugin :-)
Just one bug on mobile, when we try to drag, I have one error in my console.
On line 188 I just change :
var mouseX = parseInt((e.touches[0].pageX - parentOffset.left));
by
var mouseX = parseInt((e.originalEvent.touches[0].pageX - parentOffset.left));

Mobile device support - Bug touches


[IOTDB-777]Missing dot between deviceId and sensorId in chunkMetadaCache

Issue description: https://issues.apache.org/jira/browse/IOTDB-777
In ChunkMetadataCache, key of this map LRULinkedHashMap<AccountableString, List> lruCache should be "file path dot deviceId dot sensorId", which the key of chunkMetadataCache was missing the separator between deviceId and sensorId.

Hi, you have to modify the write and read process of this cache together. What you modify is just the read process, in this way, you can not get any ChunkMetaData from this cache.


improve python api website ux - make toc sticky

Description
Make the table of content widget on Python docs site sticky so that it won't be cut off by the footer when scrolling down to the bottom. @ChaiBapchya
Current TOC:

New sticky TOC:

Changes

[x] Make TOC widget sticky
[x] Fixed general version dropdown style breaks in Safari by adding height to the caret

Comments

Preview: http://ec2-18-236-134-13.us-west-2.compute.amazonaws.com/api/python/docs/tutorials/getting-started/crash-course/1-ndarray.html

@mxnet-label-bot add [Website]

@mxnet-label-bot add [pr-awaiting-merge]


chore(main): release 19.0.1

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

parser: Added support for extra info on the code block start line (7e0ad40), closes #77
parser: Added support for labels in links (1fc5d90)
renderer: [] are now counted when rendering tables (c4f3d54), closes #75
renderer: Added option to disable top/bottom border & fixed a rendering error that occured when disabling "use_virt_lines" (31d36e9)
renderer: Added support for language names in code blocks (5db0e8e), closes #72
renderer: Fixed a big causing numbered lists to use virtual text (6404094)
renderer: Made icons optional (6671dd4)
renderer: Table cells now respect the column count & the cell width (a0d8a5d), closes #75

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v19.0.1
:sunflower:


Optional parameters not respected in 3.0.0

When I upgrade from 3.0.0-beta-2 to 3.0.0 I'm getting loads of
error TS2554: Expected 1-2 arguments but got 0.
for all my actionFactory<void> & actionFactory.async<void, SomeType>.
Reverting back to 3.0.0-beta-2 solves it. I'm on TypeScript 3.1.6.

Can you please provide some code to reproduce?

Sorry I've missed that, was bust/holidays :-)
I will do when I find some time next week...

Closing this - I can't reproduce it any more (we've upgraded to TypeScript 3.5.3 since, maybe that has solved it, not sure ;-) )
Thanks for the lib, it rocks ;-)


Unicode errors (invalid skip) at compiling

Godot version:3.21

Windows 10
Nvidia gtx1050ti GLES3, Vulkan
Basicly i get a Unicode errors (invalid skip) error when trying to compile a programm that can run outside of the engine!
i yust compile something all the files that the log says to install are installed.
Its not the projeckt but some error!

@Test4436 What do you mean by "compiling" here? Are you exporting a project?
Please upload a minimal reproduction project to make this easier to troubleshoot.

I have a similar problem
Godot version: 3.2.2 (mono 64x)
Windows 10 x64
Nvidia gtx1060 ti

I am getting a similar issue but it is from a game I made in Godot 2.x then ported into Godot 3.2.2.  It happens both when I run the game and when I open any scene for editing.  It just spits out a ton of unicode errors.
I noticed when I run the game and it changes scenes, it will spit out a long list of these errors then finish loading.  This, of course, causes huge performance hits.


The Charging Button Status Doesn't Update Based on the Diagnostic

I'm using a Smappee EV Line Charger and got OCPP enabled from them.
However, with this integration, the Charge Control switch is not in sync with the charger status as seen below.
Expected behavior
The charge control switch should be off when the connector status is suspected
Screenshots
If applicable, add screenshots to help explain your problem.

From central system perspective charging session is still ongoing, even though energy flow is (temporarily) suspended.


How to change the tag's description

Hi,
My spec gets generated like this,
"tags": [
{
"name": "AcmeService",
"description": "External Acme Controller"
}
],

I have two controllers with the same base URL, with different operations in it.
AcmeController - /lcm/acmes
ExternalAcmeController - /lcm/acmes

Looks like I have to do it like this, but what is the alternative, it looks like deprecated.
@Api(tags = "VDIService", ~~description~~ = "VDI Controller")

This is totally a work around. Like you pointed out the description is deprecated. There is already a story to fix this, preferably using the docket to set the tags up as opposed to using @SwaggerDefinition. Duplicate of #936.

Closing this as its a duplicate


NEXT-4852 - fixed baseUrl for hot-reload mode

:// is wrong and the files won't be loaded - It has to be //.

1. Why is this change necessary?
The JS files in hot-reload mode won't be loaded because :// is a wrong path. It has to be //.
2. What does this change do, exactly?
Fixed the path to the js files for hot reloading
3. Describe each step to reproduce the issue or behaviour.
If you're using the hot-reload for the storefront ./psh.phar storefront:hot JS files are loaded like ://url:9999/js/app.js. Because of the the wrong path ( :// ) the JS files can't be loaded. The path hast to be //.
4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/NEXT-4852
5. Which documentation changes (if any) need to be made because of this PR?
6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hey there,
thank you for the PR. Is the REQUEST_SCHEME empty/null in your case? The code should work fine as long as there is a valid REQUEST_SCHEME like http or https. If we remove the : it breaks on all setups with a valid REQUEST_SCHEME. Can you provide some information about your setup?
Best regards
Jens

Is the REQUEST_SCHEME empty/null in your case?
Yes, REQUEST_SCHEME returns null in my case. My url is http://development.test/.

@jenskueper I installed the shopware/development now also on my dev server which is handled by laravel forge - SSL enabled. Same issue. Files are loaded with ://development.test:9999/js/app.js" in the hot-reload mode.

I'm guessing the problem is the Apache/Nginx version. According to https://stackoverflow.com/a/23177714, $_SERVER['REQUEST_SCHEME'] was introduced in Apache 2.4. My macOS MAMP stack uses Apache 2.2 and REQUEST_SCHEME is not available.
REQUEST_SCHEME is not mentioned on https://www.php.net/manual/en/reserved.variables.server.php, probably it would be safer to use $_SERVER['HTTPS'].
As far as I know app.request.getScheme() is based on $_SERVER['HTTPS'] and does exactly what we'd need here.

@Alex--C Yes, seems to work - However i am not using Apache, i am on macOS with nginx / valet. I updated the PR.

maybe we could also use app.request.getHost here? instead of getting SERVER_NAME directly

IIRC getHost contains the port (:8888 for example). That would mess up hot reloading as we would get URLs like http://localhost:8888:9999/js/app.js.

works in my case too with http & https with a dev url like development.test. I adjusted the PR again.

@ChristopherDosin could you also rebase your branch to current master? there is a conflict

@mitelg Sure, sorry about that - done.

thanks for your contribution @ChristopherDosin :+1: :tada: :blue_heart:


Update Google interactions to prefer the service account where viable

Commits

update googleActions and related files to use service-level auth where viable

Iterative approach to changes required for Google document stubs, part 1. See issue #1838
This only changes the GoogleActions so that they will primarily use the google service account loaded by the application for interacting with drive-based documents.
Notes:

Users who are not the main author will not see the brew on their user page
Deletion and Creation are the two pieces that still require a valid oauth token

Ok, so essentially we have 3 different Authorization types (for my own notes here):

API_Key Has limited access but doesn't require login. Can view other users' files created with HB, but not edit them.
Service Account The Homebrewery's "Google Account". Has full edit access to all files created with HB as long as the files have the appropriate "write" permissions applied.
User Account Has full access to all files created with HB as long as the files have the appropriate "write" permissions applied. Difference is User account is needed to apply the "write" permissions in the first place on a new file. We could theoretically use User Accounts in place of Service Accounts, but then people not logged in with Google would have no access.

So.... API_key shouldn't be needed anymore since it can't edit. Tasks should be split by:
User Account - Things only the original user should be able to do
getGoogleFolder - Locates the users' HB folder, or if not found, creates a new one.
newGoogleBrew - Creates a new brew in the users' HB folder, then applies write permissions so others can edit it via HB. We may want to change the permissions from anyone to only our Service Account email, although a quick test with this didn't work, and it didn't work in the past either for unknown reasons.
listGoogleBrews - Finds all documents a user owns that were created via HB. For a user, this is his own creations. A co-author would not see that file in the list. The Service Account would see no files at all, since it doesn't own any files. We can add a "driveId" to list files from a specific drive, or q : 'folderID' in parents to list files from a specific folder to view files that are not owned but shared to the user, in which case the Service Account can see those files. Thus: viewing brews on a different user's profile page requires either storing their driveId or folderId somewhere, or Ids of the individual files (in stubs). Viewing co-authored files that you are not the original owner of, won't work with the driveId or folderId route because that requires users storing lists of other people's drives to search, but stubs would still work. Conclusion: Don't use Service Account here because we won't be storing driveIds when stubs does the job. Keep User Accounts so un-stubbed brews are still visible and can be stubbed when visited.
deleteGoogleBrew - Unless.... we need to automatically handle brew ownership transfer down the line.
Service Account - Things other users should be able to do
existGoogleBrew - Just checks if a brew exists, and only used before updating. Probably not needed; error handling for missing files can happen in updateGoogleBrews itself.
updateGoogleBrew - Yep, needed for co-authors.
readFileMetadata - Needed to view brews that you don't own. Prolly should rename to getGoogleBrew.
increaseView - Needed so strangers can update the view count.


[Bug]: Cannot create/edit role assignment rule with 'is one of/is none of' value

Plugin Version
7.0.0
PHP Version
8.2
Shopware Version
<IP_ADDRESS>
Installation method
Composer
Identity provider
Microsoft Azure OIDC
What happened?
I'm trying to setup role assignment rules with the condition:
Group IDs is one of 'value of group id'
When I save it, the value hasn't being saved. The condition is there, but with an empty value. In the database is has [] as value.
I don't see any value in the PATCH request to /api/heptacom-admin-open-auth-client/id.
This happens for every 'is one of/is none of' rule condition that i've tested.
It also happens for the Google Cloud login.
But the condition rule 'always valid', 'date range', 'time range', 'Authenticated HTTP request' works fine.
Relevant log output
No response

@JoshuaBehrens friendly reminder. Or do you want me to create a shopware ticket for this?

Hi @strixnl-ronpeeters the GitHub notifications are working fine :) we already have this in our list to check and verify. I assume as we are enrolled as free plugin you might not be able to create a ticket at Shopware. As this is a free plugin we follow up in our regular product maintenance cycle. You can hit me up by mail or slack if you want to bring up any details about it :)


[DON'T MERGE] Support llama-2

Step 1: python convert_llama.py /pasteur/u/yuhuiz/data/LLAMA2/llama-2-7b/consolidated.00.pth /pasteur/u/yuhuiz/data/LLAMA2/llama-2-7b/consolidated.00.converted.pth
Step 2: verify correctness by running verify_converted_llama.ipynb
Note: change /pasteur/u/yuhuiz/data/LLAMA2/llama-2-7b/consolidated.00.pth etc to the real path in your system.
⚠⚠⚠ DON'T MERGE! JUST FOR DEMONSTRATION PURPOSE ABOUT HOW TO USE LLAMA-2 IN OPEN_LM.

Thanks a lot this looks really great!!! Thoughts on modifying this so that it is merge-able? Could be an even more useful resource!


Guzzle 6

Is there any chance that you upgrade nexmo-client to the 6-th version of Guzzle?

:+1:

I've created PR here https://github.com/ConnectCorp/nexmo-client/pull/21

Merged #21

@vood great! Waiting for new release to add it my projects.


Parameters not cleaned up in ParameterExpression

Information

Qiskit Terra version: 0.18
Python version: 3.8
Operating system: MacOS

What is the current behavior?
Arithmetic of ParameterExpression does not clean up the content of .parameters if the ParameterExpression is not parametrized anymore. Might be confusing and lead to an accumulation of irrelevant parameters in longer computations.
Steps to reproduce the problem
a
Out[47]: ParameterExpression(1.0*x)
d
Out[48]: ParameterExpression(5.0*x)
z = d/a
z
Out[50]: ParameterExpression(5.00000000000000)
z.parameters
Out[51]: {Parameter(x)}

t
Out[79]: ParameterExpression(2*x)
u
Out[80]: ParameterExpression(2*x)
y = t-u
y
Out[82]: ParameterExpression(0)
y.parameters
Out[83]: {Parameter(x)}

What is the expected behavior?
a
Out[47]: ParameterExpression(1.0*x)
d
Out[48]: ParameterExpression(5.0*x)
z = d/a
z
Out[50]: ParameterExpression(5.00000000000000)
z.parameters
Out[51]: set()

t
Out[79]: ParameterExpression(2*x)
u
Out[80]: ParameterExpression(2*x)
y = t-u
y
Out[82]: ParameterExpression(0)
y.parameters
Out[83]: set()

I think that comment is to avoid the following: assume you construct a circuit with some parameters and it just happens that these cancel out (like two Rx rotation angles with opposite sign), if SymPy would remove this parameter, binding it would fail since it doesn't exist anymore. But the user might be unaware that is was optimized away so that's weird 😄

In general this is a full reference-counting problem here; it's not enough to say "if this expression loses parameters, remove them from the symbol table", because there might be other expressions that also use the parameter.  I think here it shouldn't be too bad to implement a basic ref-counter for the parameters in ParameterTable because there's no such thing as a cyclical reference when you're talking about atomic parameters.  Still, it's a non-trivial addition, and I didn't have time to look at it properly now, or when I was doing #6802 :(


layers: Cleanup core_validation physical device properties structs

Pull physical device memory properties into layer_data and make sure it's
correctly populated at vkCreateDevice() time.
Clarify a few ambiguous "my_data" variables as "dev" or "instance".
Rename physical device properties to be lower case. This is the start of
a slow move to unify code on common standard with lowercase variable names.

Force pushed an update that addressed Chris's comment and also added an additional CL with a bunch more clean-up.

:+1:

cherry-picked and pushed to master


Get all pasta times from all producers

Data completeness, correctness and freshness are the biggest issues for data-driven applications.
Three ways to face them:

talking directly with producers to obtain official data (static files or APIs) released under an open license
developing scrapers to extract data from official websites and keep them up to date
handmade crowd-sourcing

All strategies must take into account issue #9 .
We can start listing all Italian pasta producers:

Barilla
Garofalo
De Cecco
...

Felicetti
Voiello
Divella
Arrighi
Italpasta

@abonte I already wrote to Barilla, Garofalo and De Cecco using their official contact forms asking for a release of requested data, can you please write to your suggested producers? Thanks!


Enhanced depends logic not working with omitted dag task

Pre-requisites

[X] I have double-checked my configuration
[X] I can confirm the issues exists when I tested with :latest
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened/what you expected to happen?
When a DAG task is Skipped/Omitted the dependent task does not run.
  O
 /  \
A    B <- B is skipped
|    |
C    D <- D gets omitted
     |
     E  <- (D.Succeeded || D.Omitted || D.Skipped)

In this scenario I would expect E to still run, but it doesnt.
Version
v3.4.3
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  name: broken-depends
  generateName: broken-depends-
  namespace: argo
spec:
  entrypoint: main
  templates:
    - name: main
      inputs:
        parameters:
          - name: input_var
            default: 'true'
      dag:
        tasks:
          - name: A
            template: pass
          - name: B
            template: pass
            when: '{{ inputs.parameters.input_var }} == false'
          - name: C
            template: pass
            depends: A.Succeeded
          - name: D
            template: pass
            depends: B.Succeeded
          - name: E
            template: pass
            depends: (D.Succeeded || D.Omitted || D.Skipped)
    - name: pass
      container:
        image: 'alpine:3.7'
        command:
          - sh
          - '-c'
          - exit 0
    - name: fail
      container:
        image: 'alpine:3.7'
        command:
          - sh
          - '-c'
          - exit 1

Logs from the workflow controller
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

Logs from in your workflow's wait container
kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

Bumping as this is still an issue

still looking for this


coco segfaults on 32bit machines

running 26 tests
test deque::tests::smoke ... ok
test deque::tests::stampede ... ok
test deque::tests::destructors ... ok
test deque::tests::steal_push ... ok
test deque::tests::no_starvation ... ok
test deque::tests::stress_pinned ... ok
test deque::tests::stress ... ok
test epoch::garbage::tests::buffering ... ok
test epoch::garbage::tests::destroy_array ... ok
test epoch::garbage::tests::drop_array ... ok
test epoch::garbage::tests::count_destroy ... ok
test epoch::garbage::tests::count_drops ... ok
test epoch::garbage::tests::smoke ... ok
test epoch::garbage::tests::drop_garbage ... ok
test epoch::thread::tests::flush_local_garbage ... ok
test epoch::thread::tests::garbage_buffering ... ok
test epoch::garbage::tests::flush_pending ... ok
test epoch::garbage::tests::incremental ... ok
test epoch::thread::tests::pin_reentrant ... ok
test stack::tests::is_empty ... ok
test stack::tests::push_pop ... ok
test stack::tests::smoke ... ok
error: process didn't exit successfully: `/builddir/build/BUILD/coco-0.3.1/target/release/deps/coco-8743c10fa74862ca` (signal: 11, SIGSEGV: invalid memory reference)

Happens with i686/armv7hl, doesn't happen with ppc64/ppc64le/x86_64/s390x

Thanks for reporting! Could you perhaps try running tests one by one to check which one actually causes a segfault?

I tried to go with tests one-by-one, but on the machine I tried to -- it worked fine. Will somehow try on builders.


Building für OpenELEC on Odroid

Hi, great Software, used in on my PI.
Could you give me a few hints how to build this for OpenELEC on Odroid U3?
OpenELEC has no package management, i can use the toolchain from OpenELEC build and pull the dependencies manually.
Or could i build and run hyperion it in a chroot?
Your howtos are very rPI specific.

Hi Janhleber,
Any new info about how to compil for Odroid ??
I want buy an Odroid C1+ but I have to be sure that I can use Hyperion on it

Hi,
it worked then but I use Embest MarS Board now with freescale/NXP i.MX6 SOC.
Am 17. Februar 2016 13:53:06 MEZ, schrieb Mitchou<EMAIL_ADDRESS>
Hi Janhleber,
Any new info about how to compil for Odroid ??
I want buy an Odroid C1+ but I have to be sure that I can use Hyperion
on it

Reply to this email directly or view it on GitHub:
https://github.com/tvdzwan/hyperion/issues/240#issuecomment-185186455

--
Diese Nachricht wurde von meinem Android-Mobiltelefon mit K-9 Mail gesendet.

You have compil from scratch or use Rpi install ??

Can't remember. But on my i.MX with OpenELEC there are the libs missing for hyperion, like QT...
So I copied an Arch Linux ARM rootfs on the SDcard and used it with chroot from OpenELEC to compile and run hyperion.
There is the easy way, not the clean one ;)
Am 17. Februar 2016 14:17:47 MEZ, schrieb Mitchou<EMAIL_ADDRESS>
You have compil from scratch or use Rpi install ??

Reply to this email directly or view it on GitHub:
https://github.com/tvdzwan/hyperion/issues/240#issuecomment-185200376

--
Diese Nachricht wurde von meinem Android-Mobiltelefon mit K-9 Mail gesendet.

Closed due to age. Please open a new one, if problems still exists.
Thank you