Wrong values used for composite primary key

Hi,
I'm not sure if this is bug or something is hidden somewhere in configuration and I missed it.
I have little bit unusual structure. I have main table Player with join table FriendPlayer, which should consists of only two columns, PlayerId and FriendId, both columns are pointing to the Player table.
public class Player
{
      public Guid Id;

      public virtual ICollection<FriendPlayer> Friends { get; set; } = new List<FriendPlayer>();
}

public class FriendPlayer
    {
        /// <summary>
        /// Requesting side Id of friendship
        /// </summary>
        public Guid PlayerId { get; set; }

        /// <summary>
        /// Requesting side of friendship
        /// </summary>
        public Player Player { get; set; }

        /// <summary>
        /// Accepted side Id of friendship
        /// </summary>
        public Guid FriendId { get; set; }

        /// <summary>
        /// Accepted side of friendship
        /// </summary>
        public Player Friend { get; set; }
    }

class PlayerContext
{
  protected override void OnModelCreating(ModelBuilder modelBuilder)
  {
      ......
      modelBuilder.Entity<FriendPlayer>()
                .HasOne(fp => fp.Friend)
                .WithMany(p => p.Friends)
                .HasForeignKey(fp => fp.FriendId);
      modelBuilder.Entity<FriendPlayer>().HasKey(fp => new { fp.PlayerId, fp.FriendId });
  }

  public DbSet<Player> Players { get; set; }

  public DbSet<FriendPlayer> FriendPlayers { get; set; }
}

First of all, I don't know if this approach is correct but I couldn't find any docs about this self reference same table from two properties to create one level of hierarchy.
Something must be wrong because when saving to DB, it uses PlayerId as both parts of composite primary key as well as values for columns, so basically I end up with same value in both columns. Did I miss something?
Thank you for any advice.

@OndrejPiza Thanks for filing this issue. We will discuss in our next triage meeting.

@ajcvickers Thank you. I've did some try and debug and found out, that this is done via adding this relation object to parent collection. When I added it directly through context, it worked OK.
So when I attach this object to parent collection, during save this parent Id is propagated to all children entities based on Type of navigation property on each children, right?
Then it works "as supposed" I guess, only for this specific scenario it overwrites my other navigation property of the same type.
Then I've found some article on SO, where it was describing how to achieve that, at least it works also by adding it to the collection, but I don't know if this is correct approach.
            builder.Entity<FriendPlayer>()
                .HasOne(fp => fp.Friend)
                .WithMany()
                .HasForeignKey(fp => fp.FriendId)
                .OnDelete(DeleteBehavior.Restrict);

            builder.Entity<FriendPlayer>()
                .HasOne(fp => fp.Player)
                .WithMany(p => p.FriendPlayers)
                .HasForeignKey(fp => fp.PlayerId);

https://stackoverflow.com/questions/49214748/many-to-many-self-referencing-relationship
Basically you have to have either two properties on parent for each, in my case I have another 3 relations like that so it would end up in 8 properties in total, or you have to leave one .WithMany() without specifying relationship.

@OndrejPiza Answer on Stack Overflow looks correct.

It's is, but you end up with one unmapped navigational property, depending which one you choose not to have .WithMany() with corresponding collection. I can live with only ID being mapped, without loaded object, it's just something you have to consider, eg. doing null checking if you're converting whole object in some way or use your own loading logic. Since it is already part of the parent, I think it's not such a problem.
But I think that this is not quite a way how would I suppose or expect it to work, especially based on example in documentation. Could you please consider extending documentation about this special case?

@OndrejPiza There can be zero, one, or two navigation properties for each relationship. However, the same navigation property cannot be used for two different relationships.


Golang: Minor updates (CVE-2018-6574)

Motivation for this change
Golang minor releases
Fixes CVE-2018-6574
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build go_1_8 go_1_9

Success on x86_64-linux (full log)
Partial log (click to expand)
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
cannot find section .dynamic
/nix/store/67fq0vzm849fqzx7mg407hd9xgb7kggr-go-1.8.7
/nix/store/7q0h78kbb15mxzhazqcpg782fvwz50lx-go-1.9.4

Success on aarch64-linux (full log)
Partial log (click to expand)
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
cannot find section .dynamic
/nix/store/ym1g763akbqzn82zk6c4yr4bcpg8fx8v-go-1.8.7
/nix/store/83c3rmkd133mk7kkqw0nh6i1hjams8by-go-1.9.4

Success on x86_64-darwin (full log)
Partial log (click to expand)

---
Installed Go for darwin/amd64 in /nix/store/3zgph1101r5kblb04b93faw4rhpcb1s3-go-1.9.4/share/go
Installed commands in /nix/store/3zgph1101r5kblb04b93faw4rhpcb1s3-go-1.9.4/share/go/bin
post-installation fixup
strip is /nix/store/5a88zk3jgimdmzg8rfhvm93kxib3njf9-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/3zgph1101r5kblb04b93faw4rhpcb1s3-go-1.9.4/bin
patching script interpreter paths in /nix/store/3zgph1101r5kblb04b93faw4rhpcb1s3-go-1.9.4
/nix/store/i5fnywpn6y38a2q5zar5iw973xaggg9h-go-1.8.7
/nix/store/3zgph1101r5kblb04b93faw4rhpcb1s3-go-1.9.4

Success on x86_64-darwin (full log)
Partial log (click to expand)
/nix/store/i5fnywpn6y38a2q5zar5iw973xaggg9h-go-1.8.7
/nix/store/3zgph1101r5kblb04b93faw4rhpcb1s3-go-1.9.4

Success on x86_64-linux (full log)
Partial log (click to expand)
/nix/store/67fq0vzm849fqzx7mg407hd9xgb7kggr-go-1.8.7
/nix/store/7q0h78kbb15mxzhazqcpg782fvwz50lx-go-1.9.4

Failure on aarch64-linux (full log)
Partial log (click to expand)
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
cannot find section .dynamic
error: build of '/nix/store/s7mkvh8ypay1d5dqzrlpnka9x7wjwb4k-go-1.8.7.drv' failed


Test against Node.js 4 and 6 LTS

Coverage remained the same at 92.422% when pulling bb541add860d2f133c17e0fb01474aa0cf2a04cc on MarkHerhold:patch-1 into c1a0e69460496eb73735c15373d3baa8629b0956 on google:master.

I signed it!


Resolving of latest version using multiple release branches unexpected

TL;DR
When using multiple release branches the resolving of the latest tag is done from the current branch instead of from the whole repository.
Expected behavior
We are experiencing issues when using multiple release branches.
According to supporting multiple release branches multiple branches are supported. Currently we are adding default-branch: ${{ github.ref_name }} when running on a release branch to ensure that we support all created release branches.
We expected release-please to resolve the version from the whole repository. Otherwise we are not able to use it, as we are currently absolutely locked into using multiple release branches at the same time.
Observed behavior
The problem is that the latest version is seamingly resolved from the current release branch, which isn't what we need. We may have multiple release branches at the same time, so one may have a newer tag than the one which release-please currently runs for.
Action YAML
name: Release flow test
on:
  workflow_dispatch:
  push:
    branches-ignore:
      - release-please*
  pull_request: {}

jobs:
  release-please:
    runs-on: ubuntu-latest
    name: Release please
    outputs:
      release_created: ${{ steps.release_please.outputs.release_created }}
      tag_name: ${{ steps.release_please.outputs.tag_name }}
    steps:
      - name: Release please
        id: release_please
        if: startsWith(github.ref, 'refs/heads/release-')
        uses: google-github-actions/release-please-action@v3
        with:
          release-type: go
          package-name: prototype-versioning
          default-branch: ${{ github.ref_name }}

Log output
No response
Additional information
We tried some workarounds for this:

After the release was created we merged the release branch using the ours strategy. This was to add the created tag to the default branch where release branches are branched from. This worked, but had the problem of still needing to use a release-as commit to bump versions in release branches, that were created before the latest tag was added to the default branch.
After release-please created a PR we create a release-as commit on our default branch with a new version which should be used by release-please in the future (for the next release branch). This works for the version, but not for the changelog, as the commits will still be resolved the same way. This could probably solved using last-release-sha, which isn't something we can add to the action, only in the manifest...

Sorry for the delay in getting to this issue.
If you are not using manifest configs/version file which explicitly declare the latest version, then release-please needs to figure out what the latest version on that branch was. The first attempt is to look through the latest commit history to that branch and find a release-please PR. If we don't find one, then it looks at releases and tags to find the highest semver tag of commit SHAs we've seen on that branch. Do those alternative release branches have release-please PRs or releases tagged?
Without an explicit example, there's not a whole lot we can do to help debug this.

@chingor13
No worries, thanks for getting to it.
Yes, we do not explicitly declare the latest version, as this would kind of defeat the purpose of using release-please in the first place.
The code you linked shows me that this is actually wanted behaviour: release-please only looks for the latest evrsion in the current branch. This what our problem is, as we have multiple release branches (even possibly in parallel), which don't know about each others version, as we knowingly never merge the branches back to our default branch.
Obviopusly this is wanted, but that defeats the purpose of multiple release branches, specifically the default-branch option supporting wildcards.
Ofc if you want I could provide an explicit example for what we are trying to do, but the provided action YAML already contains the necessary information. If you create multiple branches then they will never create releases with subsequent version numbers, so branch 1 creates a 1.0.0 release, but branch 2 also tries to create a 1.0.0, despite it already existing. Just let me know if you need an explicit example or any other additional informations from my site.

Ah, I see. Our primary use case for multiple release branches is for handling LTS/backport versions. In our usage: main contains the latest version (say 2.1.3). We'll also have a 1.x branch that branched from the last v1 release tag. In that branch, we'll configure the always-bump-patch versioning strategy, as we know we're only going to add bugfixes to that branch. If necessary, we can add additional versioning strategies as that's a configurable interface that we expose.
We don't have any implementation for release-please knowing about versions in other branches, because it's unclear how we would handle that kind of scenario anyways. In your example, both branches have commits that would merit a 1.0.0 version. How would you handle it if you were doing it manually?

I would use git tag -l v* to get the list of all tags on a repository and then I would resolve the highest semantic version from this list to be used as the base version to calculate a new tag based on semver.

Please re-open this on the primary https://github.com/googleapis/release-please if this is still a feature request.


optax: don't propagate jaxlib to avoid collisions

Move jaxlib into nativeBuildInputs to prevent collisions on jaxlib.
Description of changes
Without this, it's not possible to use JAX with CUDA; jaxlib and jaxlib with CUDA enabled will collide in the closure.
This has been pointed out elsewhere: https://github.com/NixOS/nixpkgs/pull/266467#discussion_r1452765961.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

It is surprising if the change works, given that strictDeps is set to true for Python packages. My guess is that jaxlib can be entirely removed from the expression...

Uh, either way I was too playing too fast and loose here; I can't reproduce the collision now. Will check back later. What effect does strictDeps have?

strictDeps make the distinction between nativeBuildInputs and buildInputs strict. So if jaxlib is required for the build to succeed and already resides in buildInputs, moving it to nativeBuildInputs would likely break the build for missing libraries.


begin to add information theory workflow

This PR is the initiation of the information theory analysis workflow. This PR only has the overall file organization and one function to try to keep it a size manageable for review. I plan to add functions to the it_analysis_data_prep.py that do preprocessing, verify time step, check for gaps etc.

This looks good! Just made a few structural changes and I think a good next step would be pulling out some variables into the config file.
I'm actually thinking we should restructure the config file so we don't have to repeat these S3 variables each time, but I can make that update later if you want to pull them into the file for now just to have it match the format of the other scripts.


(maint) Improve memory usage in benchmark tool

Use ActiveMQ for on-disk intermediate storage when doing a
simulation-style (--runinterval) benchmark.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/344/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/351/
Test PASSed.

:+1: works great for me. Works very nicely compared to master with -i 30 -n 100000

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/352/
Test FAILed.

unit tests passed and the acceptance tests don't matter for benchmark changes, i'm gonna go ahead and merge this

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/356/
Test PASSed.


how to start with wit.ai?

Hi!
Thanks for developing this project! I am a ruby/sinatra newbie who is trying to develop a messenger bot, and this gem was very useful! I am now thinking about integrating wit.ai, but I am stuck with their example code, I know that plug-in supports built-in NLP = wit.ai, but couldn't manage to get started. I would appreciate any help! I can see from my console that I can get nlp confidence values etc. but I don't know how to work with them in my code.
Facebook's example code is:
function firstEntity(nlp, name) {
  return nlp && nlp.entities && nlp.entities && nlp.entities[name] && nlp.entities[name][0];
}

function handleMessage(message) {
  // check greeting is here and is confident
  const greeting = firstEntity(message.nlp, 'greeting');
  if (greeting && greeting.confidence > 0.8) {
    sendResponse('Hi there!');
  } else {
    // default logic
  }
}

my code is but I couldn't manage to get it work.
Bot.on :message do |message|

  function firstEntity(nlp, name) {
    return nlp && nlp.entities && nlp.entities && nlp.entities[name] && nlp.entities[name][0];
  }

greetings = firstEntity(message.nlp, 'greetings')
  if greetings && greetings.confidence > 0.8
    message.reply(text: 'successful!')
  end

end

Thank you!

edit: adjusted my code.

Sorry for the late reply, @mericda. The reason this isn't working is because you're mixing two programming languages (JavaScript and Ruby). You'll have to learn one of them to write a bot, so I suggest you pick the one you like the most and read a book about it. :-)


Test that you can pin_read a public pin on RStudio Connect

(Via a question from solutions engineering)
I think pins might assume you need an API key

import pins
board = pins.board_rsconnect(
    server_url="https://colorado.rstudio.com/rsc", allow_pickle_read=True
)
board.pin_read("isabel.zimmerman/iz-cars")

returns
pins.rsconnect.api.RsConnectApiRequestError: {'code': 24, 'error': 'The requested operation requires authentication.', 'payload': None}

where isabel.zimmerman/iz-cars is public

Hmm... I'm not sure pins can implement pin read without authentication.
The error seems to occur when board.pin_read("some_pin") triggers:

pin_versions() to get most recent version, which leads to...
pin_exists() to ensure there's a pin to get versions from, which leads to...
<RsConnectFs>.exists(), which has to convert "some_pin" to a guid, so
makes a GET request to /__api__/v1/users

which you can't do without authenticating. If there were a way to convert <user_name>/<content_name> to a guid without authentication, then it might be possible?

Running into place where I would like to read Connect publicly (for building model card + monitoring dashboard templates) again. Is there anything I can do to help this along?

Can you provide example code and a traceback? Is from_pin() it erroring when it tries to board.pin_versions(...)?
I'm not sure I understand how vetiver's _choose_version differs from board.pin_read(some_version, None). Is there a situation in which it does something different?

Ah, I'm noticing you check for an "active" column -- which is what pin_versons returns for the RSConnect board in R, but this is not implemented in python apparently...
R

Python

Should be resolved in v0.0.6, which has a test of publicly reading a pin with board_url

Here's an example translated from this test
from pins import board_url, board_rsconnect

board_short = board_rsconnect(SOME_SERVER_URL)
board_short.pin_write(df, "susan/mtcars", type="csv", access_type="all")

# note that users can also get this from the web ui
content_url = board_short.fs.info("susan/mtcars")["content_url"]

board = board_url("", pin_paths={"rsc_public": content_url})

df_no_key = board.pin_read("rsc_public")


Add task mid-run update mechanism

Right now we don't have much insight into the status of a given Task while it is running.  For example once a Task is scheduled we don't have any updates until the task is complete (not even what host the Task is running on).  We should add a way for the Tasks to give more updates, especially when the have been picked up by a Worker and when they actually start running.  Some of the obvious options:

Tasks/Workers write updates directly to Datastore/Redis.
Add a Cloud Function Tasks/Workers can call to update status (currently these are only RO though).
Add a new PubSub message that Tasks/Workers can send that the server will pick up and then write to Datastore/Redis.

This issue is fixed.


[Feature Request]_Rework_assetIdent

Description
This feature aims to rework assetIdent to expand its functionality as a cacheEntety and make it easier to use and more versatile.
Link to the current Struct
https://github.com/ynput/ayon-usd-resolver/blob/8e681a95b756ed0cf458bb14f9d10a785c87de22/src/AyonUsdResolver/resolverContextCache.h#L19
Rationale
cache entity re-evaluation
Currently, we cannot invalidate a cache entity so that it gets re-evaluated when encountered.
file structures
We should move the cache entity object to its file to keep every file isolated for its purpose.
Proposed Solution
Add a function re_evaluate()
and add private variables to make sure the function is the main way of getting the data.
Alternatives Considered
Global or per object invalidation via external calls (already implemented via removeCachedObject and clearCache respectively) but its means that cache invalidation need to be done from the outside.
Additional Context
No response
Impact
No response
Assumed Complexity
No response
Impact Scale
No response
Other tools that get touched
No response
Is there an existing issue with this?

[X] I have searched the existing issues

Are there any labels you wish to add?

[X] I have added the relevant labels to the bug report.

Let's consider adding a handle to store what root_replace option has been used within the cache_entety.

this will need to be a key for the siteRoots unordert_map
we could store a vector along the resolver so that it can store a u_int8t index instead of a string_key. This would also make disconnecting the AyonCppApi easier.

There should be an option to mark an asset_itent as static when it is constructed so that it can not be overwritten.

this would be good to have for pinning_file support
this should be immutable, as lazy handling of this would allow for some sketchy setups, and there should not be a case where you don't know if it is static before creation.


fix: update evonet seeds + DPNS contractId

Issue being fixed or implemented
In order to allow users to connect to the new endpoint and to resolve some accessibility issue to the seeds from outside, we bring in this PR additional PR.
What was done

fix: removed previous seeds (seed.evonet.networks...)
fix: added new seeds dns (seed-1, ..., seed-5.evonet.networks...).
fix: updated DPNS contractID

How Has This Been Tested?
Successful integration tests means a proper resolving on those seeds.
Also manual testing.
Notes
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation

Closing in favor of #61


Suggestion: Change package name to reflect upgrade from AOR to RA

The naming convention shown in the react-admin documentation here shows the included data providers are named in the form ra-data-[provider name]
It might be worth changing the naming of this module to something like ra-data-feathers from aor-feathers-client. This could help to improve visibility of the module and remove associations with the old admin-on-rest which is no longer supported.
As a side-note on improving visibility, it may also be worth adding react-admin to keywords in package.json.

Ok it is done, just summarizing what has been done

Renamed github repo to ra-data-feathers
Created a new repo with the old repo name
npm deprecate over aor-feathers-client
Upgrade source code files with new name
New  keyword on package.json
new release on ra-data-feathers
PR on react-admin to update link

if there is something more to do please advice.


feat!: add variables for all arguments

@ErlendT What's making this a breaking change? Does adding one or more of these variables trigger a recreate of the Databricks workspace or something?

@ErlendT What's making this a breaking change? Does adding one or more of these variables trigger a recreate of the Databricks workspace or something?

No, but it requires upgraded azurerm version to allow all settings.

No, but it requires upgraded azurerm version to allow all settings.

IMO this doesn't have to count as a breaking change, as keeping the provider up-to-date should be an expectation. Should update contributing guidelines to clarify if this counts as breaking or not.
@equinor/terraform-baseline-maintainers Comments?


Support for compiling multiple sources

Compiling multiple sources (like, as part of the same project) is a clear need.
Programs in the same project should be able to make references to one another.
The result should be a list of root objects.

Done!


Arnold materials: not working with animation

Repro

Create a Serlio Node
WindTurbine_Esri.zip
Serlio > Create Arnold Materials
Set Key Frame
Set another Key Frame
BUG: Infinite creation of material nodes

internal tracker ID: CE-7282


Automatically footnote square-bracketed text w/o citations

(Following on this conversation.)
Square-bracketed text without citations is usually ignored by citeproc, except when it's immediately preceded by a citation, as in this example from the User's Guide:
@smith [p. 33] says blah.
This yields Smith1 says blah. while
@Smith, a great guy, [p. 33] says blah. yields Smith,1 a great guy, [p. 33] says blah.
This means that while notes with or following citations can be handled in whatever way is normal for the chosen style, notes without citations (and not immediately following them) cannot. Instead they need to be preceded by ^ to get them to be footnoted on their own. (The example I just gave is a little silly, but it's easy to come up with examples of note text not containing citations.)
If I have this right, I would love to be able not to worry about the contents of a note when writing it and just have all square-bracketed text be treated the same way, as notes. The second bit of text above would then yield Smith,1 a great guy,2 says blah. Since markdown normally escapes square brackets, I don't think this is a big imposition on editing, though perhaps an option to do this could be used.

How hard is it just to write
@smith [p. 33], a great guy, says...

which seems more natural to me anyway?  Adding the feature you recommend would require indefinite lookahead and would add considerable complexity (and probably unanticipated bugs).  A secondary worry is that this would require much more escaping of square brackets than is normally needed in Markdown documents, breaking many existing documents. So, I'm not in favor.
+++ John Muccigrosso [Mar 18 15 12:38 ]:

(Following on this conversation.)
Square-bracketed text without citations is usually ignored by citeproc, except when it's immediately preceded by a citation, as in this example from the User's Guide:
@smith04 [p. 33] says blah.
This yields Smith^1 says blah. while
@Smith, a great guy, [p. 33] says blah. yields Smith,1 a great guy, [p. 33] says blah.
This means that while notes with or following citations can be handled in whatever way is normal for the chosen style, notes without citations (and not immediately following them) cannot. Instead they need to be preceded by ^ to get them to be footnoted on their own. (The example I just gave is a little silly, but it's easy to come up with examples of note text not containing citations.)
If I have this right, I would love to be able not to worry about the contents of a note when writing it and just have all square-bracketed text be treated the same way, as notes. The second bit of text above would then yield Smith,1 a great guy,2 says blah. Since markdown normally escapes square brackets, I don't think this is a big imposition on editing, though perhaps an option to do this could be used.

Reply to this email directly or view it on GitHub:
https://github.com/jgm/pandoc-citeproc/issues/117

Aww thanks @jgm! :blush:

I don't want to cite anyone at all (no offence, @smith!).
The use case is when I want to put in a note that doesn't have a citation in it. In a footnote-using style, that requires a leading carat or pandoc-citeproc will ignore it. Personally I rarely use square brackets at all, so escaping them isn't a big deal for me. However if that's the problem, perhaps a flag within the brackets, like a sole "@" or a "@@@"?

+++ John Muccigrosso [Mar 19 15 09:57 ]:

I don't want to cite anyone at all (no offence, @smith!).
The use case is when I want to put in a note that doesn't have a citation in it.

So, you mean you want the "p. 33" to be a regular footnote?  As you know, you could just do
^[p. 33]

In a footnote-using style, that requires a leading carat or pandoc-citeproc will ignore it. Personally I rarely use square brackets at all, so escaping them isn't a big deal for me.

It may be a problem for other people.

However if that's the problem, perhaps a flag within the brackets, like a sole "@" or a "@@@"?

I don't see how
[@@@p. 33]

is any better than
^[p. 33]

I see your point about a flag being a flag.
What I'm trying to do is both adhere to the statement that the better way to make notes for pandoc is to use just the square brackets, so that both in-line and footnoting styles will treat them appropriately (versus always using the carat) and not to have to worry about whether there's a citation in or preceding them in order to put in the carat deliberately. Perhaps I'm just going to have to live with the worry.
That the markdown renderer in MacDown (hoedown) won't recognize either of those as notes is another issue. I was using [^noteID] with a following [^noteID]: note text previously. That's a lot less intrusive and makes for more readable markdown text.

So here's a thought:
The markdown footnote method of using [^notemarker] followed by [^notemarker]: note text somewhere else in the text is appealing to me, and was in fact the method I was originally using. I like it because it allows me to keep the notes out of the main body of the text, making that a lot more readable.
pandoc-citeproc makes all these things into footnotes, regardless of the stylesheet in use. (I think it also doesn't format them the same way WRT punctuation.)
Why not treat these as notes of whatever kind of note the stylesheet dictates, instead of forcing them to be footnotes? IOW, treat any ````[^]```-enclosed text the way citation-including or -following square-brackted things are now? Then all notes would be treated the same way, regardless of the presence of a citation, and the markdown text would be readable. It would break the footnotes for anyone who doesn't have a csl indicated, since the default Chicago style is in-line (but perhaps it could be an option).

Some people like to use regular footnotes in addition to parenthetical
citations, for longer expository digressions that aren't just references.
So I don't think it would be good to convert footnotes into
parentheticals in cases like this.
+++ John Muccigrosso [Mar 27 15 08:28 ]:

So here's a thought:
The markdown footnote method of using [^notemarker] followed by [^notemarker]: note text somewhere else in the text is appealing to me, and was in fact the method I was originally using. I like it because it allows me to keep the notes out of the main body of the text, making that a lot more readable.
pandoc-citeproc makes all these things into footnotes, regardless of the stylesheet in use. (I think it also doesn't format them the same way WRT punctuation.)
Why not treat these as notes of whatever kind of note the stylesheet dictates, instead of forcing them to be footnotes? IOW, treat any ````[^]```-enclosed text the way citation-including or -following square-brackted things are now? Then all notes would be treated the same way, regardless of the presence of a citation, and the markdown text would be readable. It would break the footnotes for anyone who doesn't have a csl indicated, since the default Chicago style is in-line (but perhaps it could be an option).

Reply to this email directly or view it on GitHub:
https://github.com/jgm/pandoc-citeproc/issues/117#issuecomment-86974681

Yes, I see how that would be problematic. Making the behavior optional would prevent that. Or those who want footnotes in an in-line style could use ^[  ]?
I notice that the CommonMarkdown spec handles [ ] all by itself differently, so maybe it's best to wait for that to continue this discussion? (Plus I don't see a lot of other people coming forward with the same concerns I have. :-) )


[tcgc] need to handle projection when finding spread original model

fix: https://github.com/Azure/typespec-azure/issues/1429

All changed packages have been documented.

:white_check_mark: @azure-tools/typespec-client-generator-core

Show changes
@azure-tools/typespec-client-generator-core - fix ✏

need to handle projection when finding spread original model

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


Difficulty creating an Initial Machine

Thanks again for helping me previously on the .qcow2 issue!
I'm having another issue with creating an initial machine. I tried to create the machine in the dashboard.

However, when I hit the Create VM's button, nothing happens. I checked the apache2 error.log file and found this:

Again, if this is something I misconfigured, please let me know. I'm very close with this!

Please update your KVM-VDI and try again

Another error popped up (T_VARIABLE on line 91). I researched the error and found that it pops up when a semicolon is missing. So, I added the semicolon at the end of line 91. After doing this, the Initial machine looked like it started working (I added a commit to that line).
So I thought....
After creating the machine, I went into the dashboard and found that it just sits on creation.

Here is my configuration:

Here is the error I get when I create the machine:

Any help is much appreciated! Thanks!

Sorry, should have tested this. Now it should work.

Thank you! It works! Prepping our Windows machines now! :)


I2S write broken

Apparently i2s-write is broken - luckily released 0.19.0 is not affected
❯ git bisect good
6fd9443c138bc3eb46da5dead333ccf6d424cec7 is the first bad commit
commit 6fd9443c138bc3eb46da5dead333ccf6d424cec7
Author: Dominic Fischer<EMAIL_ADDRESS>Date:   Tue Jul 16 09:52:48 2024 +0100

    Allow users to easily name DMA channels (#1771)

    * Introduce public DmaChannel trait

    * Use DmaChannel trait in all peripherals

    * Simplify ChannelTypes trait

    * changelog

    * Rename things

    * missed a spot

    ---------

    Co-authored-by: Dominic Fischer<EMAIL_ADDRESS>
 esp-hal/CHANGELOG.md             |   1 +
 esp-hal/src/aes/mod.rs           |  24 +++---
 esp-hal/src/dma/gdma.rs          |  33 +++++---
 esp-hal/src/dma/mod.rs           | 168 ++++++++++++++++++---------------------
 esp-hal/src/dma/pdma.rs          |  20 +++--
 esp-hal/src/i2s.rs               |  88 ++++++++++----------
 esp-hal/src/lcd_cam/cam.rs       |  25 +++---
 esp-hal/src/lcd_cam/lcd/i8080.rs |  29 +++----
 esp-hal/src/parl_io.rs           |  84 ++++++++++----------
 esp-hal/src/spi/master.rs        |  68 ++++++++--------
 esp-hal/src/spi/slave.rs         |  36 +++++----
 11 files changed, 291 insertions(+), 285 deletions(-)

Easy to reproduce via cargo xtask run-example examples esp32c6 embassy_i2s_sound - after that commit it doesn't do anything anymore
This is especially bad since the hil-test didn't catch this 😢

Ok makes sense the test didn't catch this since it's only broken for async (which doesn't make a lot of sense somehow)

I guess I found it: https://github.com/esp-rs/esp-hal/blob/bc7c53f6680dc194262a451e3c9bf71c2730c9a5/esp-hal/src/dma/mod.rs#L1455
That should be CH::Tx::waker()

Whoops! sorry about that 😬

No problem - should have noticed it during the review 😃


Add badges in Readme.md

Description
Badges are a great way to express things with figures.
Mock

[ ] Add Html badge
[ ] Add CSS badge
Can suggest more badges if assignee wants to

I can work on this. Please share more information like - which type badges do you want for this repo, etc. I normally use badges from https://shields.io/ . It is free and easy to use.

@NitinNair89 Well you can go for this issue :)

Sure @keshakaneria . Any other badges apart from the ones listed in mock?

I have edited the description adding some more :)


set key to default "" in order for iterator not to fail

async getHistoryForKey (key) {
   return await this.handler.handleGetHistoryForKey(key, this.channel_id, this.txId);
}

if key does not exist it will return a MockHistoryQueryIterator where data is undefined { data: undefined, [...] } which will cause errors when executing the iterator.

Is this the behaviour of the actual chaincode? or should this be an empty array?


Fix license key from config file issue

Description
Please include a summary of the changes and the related issue.
Type of change

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.

[ x] Ran the app

Checklist:

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

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Ctrl-D doesn't respect the Scene tree view focus if the Script editor is open. The highlighted line is duplicated instead of the selected node

Godot version
3.5.stable
System information
Windows 10
Issue description
If the Script editor is open, Pressing Ctrl-D will duplicate the Highlighted line, even if the Script editor is not focused. The most common behavior is that at some point I click on a Node in the Scene tree view to duplicate it, but the shortcut duplicates the line in the (now out of focus) Script editor, instead of the node in the now focused Scene tree view.
I don't know if this was present in Godot 3.4, but it's more visibler in 3.5 since now the Script tab is "sticky", and remains open after clicking on some node, instead of switching to the 2D or 3D view.
Note that when the Node is selected, other shortcuts like Deleting with Del work on the Node correctly, respecting the focused view.
Steps to reproduce

Create a Godot scene with 1 Node, and 1 Script attached
Open the Script View
Click on the Node in the Scene View
Press Ctrl-D.
The highlighted line gets duplicated, instead of the Node (the node can't be duplicated as it's a root, but you'd get a message anyway).

Minimal reproduction project
Setup is too small / simple to warrant uploading a project. But if it does help, I can provide.

Duplicate of https://github.com/godotengine/godot/issues/52861.


As A user not only do I want to be able to see the number of online players but also their usernames so I can possibly add them as friends.

09ba66820f14787101fa6afbf2b11ca70ad61586


V2.2.2 Syntax Error in creating the Stage tables in surrogate key macro for Snowflake Plugin

There is an additional AS causing SYNTAX error in SQL getting generated using dbt_artifacts.surrogate_key macro. This prevents the staging table creation in snowflake plugin

Hi @ManjariManokar .
Thanks for raising this issue and apologies for how long it has taken to get back to you. I was unable to replicate this exact example locally, but it did prompt me that the surrogate key macro we were using is now out of date, so I have created a PR to update it to the latest version, which should resolve this issue for you.

Thank you for your response
On Fri, 28 Apr 2023, 16:41 Gemma Down, @.***> wrote:

Closed #265 https://github.com/brooklyn-data/dbt_artifacts/issues/265
as completed via #298
https://github.com/brooklyn-data/dbt_artifacts/pull/298.
—
Reply to this email directly, view it on GitHub
https://github.com/brooklyn-data/dbt_artifacts/issues/265#event-9126200998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEMAUVVAKDBXZOPMSZIVSPLXDPQKLANCNFSM6AAAAAAU2RO55A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Is there a way to draw a dash line?

I love you guys works.
The library is exactly what I need.
Here is few requirements. Is there a way to draw a dash line?

@nyfpos
https://github.com/memononen/nanovg/issues/482
https://github.com/memononen/nanovg/issues/436

thanks


Fix csv export error by adding cookieParser middleware

fix #123
cc @AndrewGHC

Thanks for this @karuppiah7890! And nice spot.


Xy 6

Original NCL Script: https://www.ncl.ucar.edu/Applications/Scripts/xy_6.ncl
Original NCL Plot: https://www.ncl.ucar.edu/Applications/Images/xy_6_lg.png
Issue: #330

The plot isn't exactly the same. I think it's because of how the xticks are assigned. I'm not exactly sure how the original plot creates equal spacing on the tickmarks on x-axis while the actual values(years) are not equally spaced apart.

So I think I have made a (very) hacky work around for this issue with ax.xaxis.set_minor_locator and also defining the xticks in line 72 and removing the null ('') labels from xticklabels list. Like I said, it's hacky, so @anissa111 if you know of a better route than this, definitely let me know but, either way, here is what I have inserted on line 77:
ax.xaxis.set_minor_locator(FixedLocator([1954, 1961, 1968.5, 1977, 1987]))

this will put exactly one tick in middle point of each major tick at the specified value. You can use decimals, so it can be precise enough. But what I don't love about this version is that it specified by hand and I have a feeling it may not be the true blue best approach.
Let me know what you think!

The plot isn't exactly the same. I think it's because of how the xticks are assigned. I'm not exactly sure how the original plot creates equal spacing on the tickmarks on x-axis while the actual values(years) are not equally spaced apart.

So I think I have made a (very) hacky work around for this issue with ax.xaxis.set_minor_locator and also defining the xticks in line 72 and removing the null ('') labels from xticklabels list. Like I said, it's hacky, so @anissa111 if you know of a better route than this, definitely let me know but, either way, here is what I have inserted on line 77:
ax.xaxis.set_minor_locator(FixedLocator([1954, 1961, 1968.5, 1977, 1987]))

this will put exactly one tick in middle point of each major tick at the specified value. You can use decimals, so it can be precise enough. But what I don't love about this version is that it specified by hand and I have a feeling it may not be the true blue best approach.
Let me know what you think!

I tried to follow what you said but I don't seem to be able to

Oops sorry, I don't know what I did that closed the PR.

The plot isn't exactly the same. I think it's because of how the xticks are assigned. I'm not exactly sure how the original plot creates equal spacing on the tickmarks on x-axis while the actual values(years) are not equally spaced apart.

So I think I have made a (very) hacky work around for this issue with ax.xaxis.set_minor_locator and also defining the xticks in line 72 and removing the null ('') labels from xticklabels list. Like I said, it's hacky, so @anissa111 if you know of a better route than this, definitely let me know but, either way, here is what I have inserted on line 77:
ax.xaxis.set_minor_locator(FixedLocator([1954, 1961, 1968.5, 1977, 1987]))

this will put exactly one tick in middle point of each major tick at the specified value. You can use decimals, so it can be precise enough. But what I don't love about this version is that it specified by hand and I have a feeling it may not be the true blue best approach.
Let me know what you think!

I tried to edit my code according to this but I still couldn't recreate the original plot. The plot itself doesn't seem to change for me when I change the ticks locations. Do you mind sharing your code from line 72-77? Thank you!

@Jiaqi-beep
The code starts on line 66 and goes through 77:
# Use geocat.viz.util convenience function to set axes parameters without calling several matplotlib functions
# Set axes limits, tick values, and tick labels
gvutil.set_axes_limits_and_ticks(ax,
                                 xlim=(1951, 1991),
                                 ylim=(-1.5, 0.6),
                                 yticks=np.arange(-1.2, .6, .4),
                                 xticks=[1951, 1957, 1965, 1972, 1982, 1991],
                                 xticklabels=[
                                     '1951', '1957', '1965', '1972',
                                     '1982', '1991'
                                 ])
ax.xaxis.set_minor_locator(FixedLocator([1954, 1961, 1968.5, 1977, 1987]))

I will note that the new minor tick marks are pretty faint, but that might be something that can be changed with a kwarg later on.
This is the plot output that I have when running locally:

updated.


API Proposal: Add DeviceStream APIs to IoT Hub Device SDK (C)

Overview
Azure IoT Hub device streams facilitate the creation of secure bi-directional TCP tunnels for a variety of cloud-to-device communication scenarios. A device stream is mediated by an IoT Hub streaming endpoint which acts as a proxy between your device and service endpoints. This setup is especially useful when devices are behind a network firewall or reside inside of a private network. As such, IoT Hub device streams help address customers' need to reach IoT devices in a firewall-friendly manner and without the need to broadly opening up incoming or outgoing network firewall ports. Using IoT Hub device streams, devices remain secure and will only need to open up outbound TCP connections to IoT hub's streaming endpoint over port 443. Once a stream is established, the service-side and device-side applications will each have programmatic access to a WebSocket client object to send and receive raw bytes to one another. The reliability and ordering guarantees provided by this tunnel is on par with TCP.
Device Stream Workflow
A device stream is initiated when the service requests to connect to a device by providing its device ID. This workflow particularly fits into a client/server communication model, including SSH and RDP, where a user intends to remotely connect to the SSH or RDP server running on the device using an SSH or RDP client program.
The device stream creation process involves a negotiation between the device, service, IoT hub's main and streaming endpoints. While IoT hub's main endpoint orchestrates the creation of a device stream, the streaming endpoint handles the traffic that flows between the service and device.
Device stream creation flow
Programmatic creation of a device stream using the SDK involves the following steps:

The device application registers a callback in advance to be notified of when a new device stream is initiated to the device. This step typically takes place when the device boots up and connects to IoT Hub.
The service-side program initiates a device stream when needed by providing the device ID (not the IP address).
IoT hub notifies the device-side program by invoking the callback registered in step 1. The device may accept or reject the stream initiation request. This logic can be specific to your application scenario. If the stream request is rejected by the device, IoT Hub informs the service accordingly; otherwise, the steps below follow.
The device creates a secure outbound TCP connection to the streaming endpoint over port 443 and upgrades the connection to a WebSocket. The URL of the streaming endpoint as well as the credentials to use to authenticate are both provided to the device by IoT Hub as part of the request sent in step 3.
The service is notified of the result of device accepting the stream and proceeds to create its own WebSocket client to the streaming endpoint. Similarly, it receives the streaming endpoint URL and authentication information from IoT Hub.

In the handshake process above:

The handshake process must complete within 60 seconds (step 2 through 5), otherwise the handshake would fail with a timeout and the service will be notified accordingly.
After the stream creation flow above completes, the streaming endpoint will act as a proxy and will transfer traffic between the service and the device over their respective WebSockets.
Device and service both need outbound connectivity to IoT Hub's main endpoint as well as the streaming endpoint over port 443. The URL of these endpoints is available on Overview tab on the IoT Hub's portal.
The reliability and ordering guarantees of an established stream is on par with TCP.
All connections to IoT Hub and streaming endpoint use TLS and are encrypted.

Termination Flow
An established stream terminates when either of the TCP connections to the gateway are disconnected (by the service or device). This can take place voluntarily by closing the WebSocket on either the device or service programs, or involuntarily in case of a network connectivity timeout or process failure. Upon termination of either device or service's connection to the streaming endpoint, the other TCP connection will also be (forcefully) terminated and the service and device are responsible to re-create the stream, if needed.
SDK Availability
Two sides of each stream (on the device and service side) use the IoT Hub SDK to establish the tunnel. During public preview, customers can choose from the following SDK languages:

The C and C# SDK's support device streams on the device side.
The Node.js and C# SDK support device streams on the service side.

Sample Code (C)
This echo sample demonstrates programmatic use of device streams to send and receive bytes between service and device applications. To execute the scenario end to end, use the corresponding C# or Node.js sample for the service side.
Device Sample Code (C)
Full sample project here
Creating the device client using your device connection string and desired protocol (existing APIs):
// Used to initialize IoTHub SDK subsystem
(void)IoTHub_Init();

IOTHUB_DEVICE_CLIENT_HANDLE device_handle;

// Create the iothub handle here
device_handle = IoTHubDeviceClient_CreateFromConnectionString(connectionString, protocol);

Listening for device stream requests (new APIs):
if (IoTHubDeviceClient_SetStreamRequestCallback(device_handle, streamRequestCallback, NULL) != IOTHUB_CLIENT_OK)
{
    (void)printf("Failed setting the stream request callback");
}
else
{
    do
    {
        if (g_uws_client_handle != NULL)
        {
            uws_client_dowork(g_uws_client_handle);
        }

        ThreadAPI_Sleep(100);
    } while (g_continueRunning);
}

Handling the stream request callback (new APIs):
static DEVICE_STREAM_C2D_RESPONSE* streamRequestCallback(DEVICE_STREAM_C2D_REQUEST* stream_request, void* context)
{
    (void)context;

    (void)printf("Received stream request (%s)\r\n", stream_request->name);

    g_uws_client_handle = create_websocket_client(stream_request);

    return stream_c2d_response_create(stream_request, true);
}

New API Surface added (C)
Reference to existing IoTHub client APIs (link)
IoTHub Device SDK additions
typedef struct DEVICE_STREAM_C2D_REQUEST_TAG
{
    /**
    * @brief    Name of the stream. This is a null-terminated string.
    */
    char* name;
    /**
    * @brief    Websockets URL to connect to the streaming gateway. This is a null-terminated string.
    */
    char* url;
    /**
    * @brief    Authorization token to be provided to the streaming gateway upon connection. This is a null-terminated string.
    */
    char* authorization_token;
    /**
    * @brief    Request ID used to correlate requests and responses. Do not modify its value.
    */
    char* request_id;
} DEVICE_STREAM_C2D_REQUEST;

typedef struct DEVICE_STREAM_C2D_RESPONSE_TAG
{
    /**
    * @brief    Indicates if the stream request was accepted or rejected by the local endpoint. Use true to accept, or false to reject.
    */
    bool accept;
    /**
    * @brief    Request ID used to correlate requests and responses. Do not modify its value.
    */
    char* request_id;
} DEVICE_STREAM_C2D_RESPONSE;

/**
* @brief     Callback invoked for new cloud-to-device stream requests.
* @param     request         Contains the basic information to connect to the streaming gateway, as well as optional custom data provided by the originating endpoint.
* @param     context         User-defined context, as provided in the call to *_SetStreamRequestCallback.
* @return    An instance of DEVICE_STREAM_C2D_RESPONSE indicating if the stream request is accepted or rejected.
*/
typedef DEVICE_STREAM_C2D_RESPONSE* (*DEVICE_STREAM_C2D_REQUEST_CALLBACK)(DEVICE_STREAM_C2D_REQUEST* request, void* context);

DEVICE_STREAM_C2D_RESPONSE* stream_c2d_response_create(DEVICE_STREAM_C2D_REQUEST* request, bool accept);

IOTHUB_CLIENT_RESULT IoTHubDeviceClient_SetStreamRequestCallback(IOTHUB_DEVICE_CLIENT_HANDLE iotHubClientHandle, DEVICE_STREAM_C2D_REQUEST_CALLBACK streamRequestCallback, void* context);
IOTHUB_CLIENT_RESULT IoTHubModuleClient_SetStreamRequestCallback(IOTHUB_MODULE_CLIENT_HANDLE iotHubModuleClientHandle, DEVICE_STREAM_C2D_REQUEST_CALLBACK streamRequestCallback, void* context);

This has now been open for 8 weeks.  Any further action required on this?

Closing for lack of movement.


misc edits in /docs; mostly style, grammar, consistency

- Summary
Read through all the .md files under /docs/ and copy edited. Primarily made changes to sentences to clean up structure and usage. Some minor grammar and general typo edits.
Editing for style, usage, consistency, etc. is how I can best contribute (not a coder; professional writer/editor) to this repo.
Cleaned up and (hopefully) help make docs more readable and usable by the end user.
- Test plan
No tests since changes were strictly to copy; no code changes.
- Description for the changelog
Copy edits to /docs/ to improve consistency & readability
- A picture of a cute animal (not mandatory but encouraged)

My dog & co-editor Psycho!

@flaturtha thanks for this! Just a couple of change requests regarding relative links and this should be good to merge.

I believe I've made all the changes/updates per everyone's requests. I've committed again to this PR. Let me know if I missed anything.

@flaturtha awesome I'll check now. Also, I was just about to bring up your docs work on the call when we lost you! Documentation is a major area of need for the project, it'll get more attention in future meetings.

Sorry to suddenly disappear. Hated to but my dog had an emergency. At the vet now.
Sent via the Samsung Galaxy S® 5 mini, an AT&T 4G LTE smartphone
-------- Original message --------
From: Shawn Erquhart<EMAIL_ADDRESS>Date: 7/19/2017  10:58 AM  (GMT-06:00)
To: netlify/netlify-cms<EMAIL_ADDRESS>Cc: Rich Cook<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [netlify/netlify-cms] misc edits in /docs; mostly style, grammar,
consistency (#489)
@flaturtha awesome I'll check now. Also, I was just about to bring up your docs work on the call when we lost you! Documentation is a major area of need for the project, it'll get more attention in future meetings.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/netlify/netlify-cms","title":"netlify/netlify-cms","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/netlify/netlify-cms"}},"updates":{"snippets":[{"icon":"PERSON","message":"@erquhart in #489: @flaturtha awesome I'll check now. Also, I was just about to bring up your docs work on the call when we lost you! Documentation is a major area of need for the project, it'll get more attention in future meetings."}],"action":{"name":"View Pull Request","url":"https://github.com/netlify/netlify-cms/pull/489#issuecomment-316433276"}}}

Sorry to hear it 😞, hope he's doing better.

Again, I am super stoked about this PR. Thanks for your patience!
Looking at Psyho's smiling face has me hoping he/she is OK!

Just pushed up the requested changes (2 files: removed .md extension). Sorry for the delay.
I will be out of town Monday through Thursday but will be happy to make further changes/edits as needed.

@flaturtha you are the man.
@verythorough feel free to merge this when you're comfortable with the changes.

Glad to help. Out of town until Friday but shout if I can do anything else to help out.
Rich
Sent via the Samsung Galaxy S® 5 mini, an AT&T 4G LTE smartphone
-------- Original message --------
From: Shawn Erquhart<EMAIL_ADDRESS>Date: 7/24/2017  12:11 PM  (GMT-06:00)
To: netlify/netlify-cms<EMAIL_ADDRESS>Cc: Rich Cook<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [netlify/netlify-cms] misc edits in /docs; mostly style, grammar,
consistency (#489)
@flaturtha you are the man.
@verythorough feel free to merge this when you're comfortable with the changes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/netlify/netlify-cms","title":"netlify/netlify-cms","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/netlify/netlify-cms"}},"updates":{"snippets":[{"icon":"PERSON","message":"@erquhart in #489: @flaturtha you are the man.\r\n\r\n@verythorough feel free to merge this when you're comfortable with the changes."}],"action":{"name":"View Pull Request","url":"https://github.com/netlify/netlify-cms/pull/489#issuecomment-317490194"}}}

Thanks so much! ❤  for Psycho!


test--3-main

Please pull this in!

 Quality gate: Failed
26 Issues

 24 Code issues
 2 Anti-patterns

Details

 Quality gate: Failed
26 Issues

 24 Code issues
 2 Anti-patterns

Details

 Quality gate: Failed
26 Issues

 24 Code issues
 2 Anti-patterns

Details


Switch command-line parsing to clap

It's faster and we might be able to auto-generate the autocompletion files (see comment in #86).
(Generating autocompletion doesn't need to be part of this issue.)
Potentially related to #79.

On that occasion, maybe -m/--markdown should be renamed to -r/--raw?

Furthermore, we don't quite follow the spec:
https://github.com/tldr-pages/tldr/blob/master/CLIENT-SPECIFICATION.md#arguments
For example, -p is --pager, not --platform.

I am a clap maintainer. I can answer any issues for this. We will be releasing a 3.0 beta version soon. You can use the new clap_derive.
I was looking at the issues because I didn't like the way the colors are shown. Most of the other clients follow a certain pattern and I got used to that.

Oh wow. Out of curiosity, today I did a release mode profile with valgrind/callgrind. Here's the call graph of main:

On here, the purple part in the bottom is config file TOML parsing. The colorful stuff on the lower right and right side is the actual file loading, tokenization and rendering. And the huge blue chunk is just docopt argument parsing :flushed:
Of course in absolute numbers it's still quite fast and callgrind does not measure in absolute time but in instructions (so I/O is not considered), but it's still quite a big part of the binary.
We should definitely try clap instead, and then do a comparison :slightly_smiling_face: Maybe I can do a quick draft tomorrow.

If you want to use clap v3, please use clap_derive from master branch. If not, you should try structopt derive.

@pksunkara with clap_derive, is there a way to manually print the help from the parsed struct?
(Some flags are only valid in combination with a command.)

Some flags are only valid in combination with a command.

You should be able to define all that in clap itself which does parsing and errors with help messages when condition fails.

@pksunkara let's continue the discussion in https://github.com/dbrgn/tealdeer/pull/108!

In case we fully implement the client spec, this will fix #120.


Add Dependencies between MemoryCache objects

When caching a tenant, we create cache entries for all identifiers, which is great for performance but has the downside that upon eviction, all other identifiers become invalid.
This change, suggested by @benfoster in #27 , resolves that issue by creating a cache dependency between the items. When one expires, all other related entries are removed.
I've also moved this behavior out of CreateCacheEntryOptions() so that an implementer who only wishes to make minor changes to their cache options, such as setting a sliding expiration of 15 minutes, they are not forced to re-implement the disposal code.
If an implementer wishes to disable this functionality, they can override DisposeTenantOnExpiration as follows:
protected override bool DisposeTenantOnExpiration => false;

@benfoster this branch has been updated to include some missing fixes (samples and tests are all that's affected) from @joeaudette's PR for RTM, as well as include the RTM PR.

I see you've also added a new interface ITenant to handle null tenants. Whilst this appears to be a positive change, can you in future submit separate PRs for different features/fixes so we can discuss them.

@benfoster  I've separated that out to a new branch and PR.

@benfoster I've removed the ITenant interface and will submit as a separate P/R

@benfoster please see #37 for the null tenant feature.


[A11y][Fastpass]Aria-label not allowed on b

URL: https://www.nuget.org/packages/Newtonsoft.Json

Path
#paket-cli > .alert-warning[role="alert"][aria-live="assertive"] > b[display="none"][aria-label="warning"]

Fix the following:
aria-label attribute cannot be used on a b with no valid role attribute.
https://www.w3.org/WAI/WCAG21/Understanding/name-role-value.html
https://accessibilityinsights.io/info-examples/web/aria-allowed-attr
Looks like the fix here would be to either drop this label altogether, OR give this element a role that would make this label proper usage.

Verified the issue no longer is flagged by fastpass.


autoHeight prop only works without an h prop

I currently have a container than I need to be able to change height based on the content. autoHeight: true is the solution here, however if I don't set the h prop to a number, I get a pile of errors on the console. The following works for me, minus all the errors. Wasn't sure if this was an issue with the code or the way I'm implementing it.
      i: 'component_0',
      x: 1,
      y: 0,
      w: 10,
      autoHeight: true,
      static: true,

How to install this library? any NPM package?

I currently have a container than I need to be able to change height based on the content. autoHeight: true is the solution here, however if I don't set the h prop to a number, I get a pile of errors on the console. The following works for me, minus all the errors. Wasn't sure if this was an issue with the code or the way I'm implementing it.
      i: 'component_0',
      x: 1,
      y: 0,
      w: 10,
      autoHeight: true,
      static: true,

@medwards2009 Did you manage to get this working without errors?


Fix capybara with react

Turns out that PhantomJS/Poltergeist does not render any React components (even if you do things like wait_for_ajax and sleep(5000). Switching to capybara webkit solves the problem. Also, we had some redundant drivers in the Gemfile.
JIRA: https://netguru.atlassian.net/browse/PEOPLE-14

LGTM :+1: :shipit:


somehow accommodate projects that are using wartremover

this came up in the context of #347 / https://github.com/scala/community-builds/pull/355 (unfreezing simulacrum)
it would be super awesome if somebody else could figure this out, I already burned too much time on it

relocated @milessabin comment:

Given that wartremover is a linter, wouldn't it be better to patch the projects to disable it rather than freeze them? Also, are the upstream projects aware of the issue with wartremover and the freezing?

if someone could find a way to disable it just using extra.commands, without having to fork, that would be awesome
upstream projects are aware if I @-mention them at fork or freeze time, which I uuuusually do. we don't have a good system for this

maybe the approach taken by scalafmt at https://github.com/olafurpg/scalafmt/pull/610 would work? (I haven't studied it enough to know whether this is plausible)

FWIW, I use synthetic projects in the scalafix compiler plugin which may be closer to wartremover's situation. See here for how I deal with 2.11 and 2.12. A synthetic project is needed for each supported scala version. A nice side-effect to this is that update is cached on the synthetic projects, which makes resolution faster for multi-project build (esp. scala.js cross-projects in combination with SNAPSHOT versions)

this hasn't been a problem lately. we still have simulacrum frozen over it, but it hasn't caused problems with any other projects in a long time. (for all I know, some newer wartremover version may even have eased the problem.) we can reopen if needed

hit this recently adding a project (curryhoward) that has wartremover 2.2.1 which is the latest version, so there hasn't been a fix.

#698 adds wartremover to the build, fixing the problem


fix:  events emitted on overflow detected/cleared

Hey! It seems the onOverflowDetected and onOverflowCleared are swapped. Currently, when the animation is running due to overflow, the onOverflowCleared is emitted.

Summary by Sourcery
Fix the event emission logic for overflow detection and clearance by swapping the 'onOverflowDetected' and 'onOverflowCleared' events to ensure they are emitted under the correct conditions.
Bug Fixes:

Correct the emission of events by swapping 'onOverflowDetected' and 'onOverflowCleared' to their appropriate conditions.

Thank you for catching that!


Enable rolling agent

Failing only host because RMW in Rolling docker is not updated to the latest version.

@mergify backport foxy galactic


Any way to print horizontally?

Because the datagrid I am using is to big for a normal print page, I would like to print it horizontally (landscape mode).
But I can't find any option that would allow it.

It's not supported in the base library, but  you can, generate the PDF using Quest.PDF  and use this service to get the Print Dialog to print the file. Which will have better results in the long run.


after hierarchy, where is the module's docstring?

after introducing a hierarchy for modules like devices (see issue #31 and PR #38), the docstring explainign what the module contains does not exist anymore. Would still be a good thing to have the option to document the module.

I think this is resolved now by #49. If not feel free to reopen.


Goldendict not finding dictionary files.

Venetia writes: " I installed the paid for version of GoldenDict (which is definitely worth it - works so much better) and uninstalled the free version ( colordict). Then I used the updater app which said it had downloaded all dictionary files but once finished the 'done' button is inactive and can't be pressed, so it never seems to complete and the dictionaries never appear in the GoldenDict app. Not sure what I'm doing wrong but any suggestions gratefully welcomed.  "

Likely obsolete issue.


chore: update golangci-lint to v3

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.04%) to 96.146%

Files with Coverage Reduction
New Missed Lines
%

report/export.go
1
94.41%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
2345

Relevant Lines:
2439

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+2.4%) to 98.573%

Files with Coverage Reduction
New Missed Lines
%

report/export.go
1
94.41%

Totals

Change from base Build<PHONE_NUMBER>:
2.4%

Covered Lines:
1520

Relevant Lines:
1542

💛  - Coveralls


move development database out of the source tree

this also includes a dump/migrate/restore of the old DB being pushed to S3 which contains postgres14 pg_dump with all of the current migrations applied.

I'd like us to be able to have this replace the current dev/example.sql.xz if the shasum value in the Makefile changes... but I'm not smart enough at make.

toward(ish) #6504.

Going to go a different direction here and include it via a development dependency.


Browser Alert Username and Password Input

The problem
I have a staging website where it need to input username and password before it's loading. How to do it?
Details
I have a staging website where it need to input username and password before it's loading. How to do
Tell us about your environment

Selenide Version: 6.14.0
Chrome\Firefox\IE Version: 112
Browser Driver Version: -
Selenide-Appium Version: -
IOS\Android\WEB version: -
Selenium Version:  4.9.1
OS Version:  Win 10

Code To Reproduce Issue [ Good To Have ]

You can use method Selenide.open with username/password parameters.
RTFT: https://github.com/selenide/selenide/blob/main/statics/src/test/java/integration/BasicAuthTest.java#L13

Thanks.


add articles on RSE group structures?

You could potentially add some articles on how RSE groups are structured, such as

https://doi.org/10.1109/SE4Science.2019.00009 (preprint http://arxiv.org/abs/1903.00732)
https://doi.iorg/10.1109/MCSE.2023.3264113 (preprint https://arxiv.org/abs/2210.16261)

There are likely other such papers, though I can't find any right now...

Many thanks @danielskatz, apologies for the delay, this is now added in #32.


Adding a layer for a road under construction

I'm a noob in maputnik. After some time I could style anything I need pretty easily.
But I've found a constructed road - https://www.openstreetmap.org/#map=18/57.15488/65.59681
And I couldn't understand how can I add the construction layer to my style http://editor.openmaptiles.org/?style=https://rawgit.com/openmaptiles/positron-gl-style/master/style.json#16.14/57.154374/65.597906
I probably don't really understand the layer\tag\source\type concept. Sorry.
Can anyone give me a hint on how to add a constructed road layer to my style?

Hi @zvukoper currently roads under construction are not included in the source you are using  (OpenMapTiles) for the Positron style.
See https://github.com/openmaptiles/openmaptiles/issues/501, https://github.com/openmaptiles/openmaptiles/issues/321, https://github.com/openmaptiles/openmaptiles/issues/82.
There are several ways how you could achieve what you want. One solution would be to extract the geometry for the road under construction from OSM and add it as GeoJSON line to the map.
You could add the GeoJSON as source in Maputnik and style it or add it with GL JS like in the following example:
https://www.mapbox.com/mapbox-gl-js/example/geojson-line/

Thanks for your feedback @zvukoper, closing here.


kvm/init: remove condition for kvm mutable pods

As agreed on Rkt sync. All implemented cri features works well for kvm, and we plan to be on track with cri for kvm.

good news, thanks for verifying :+1: LGTM on green!


What is the problem with this tacotron training in other language?

I want to train Tacotron in Persian language and I follow this training. I have about 2500 wav files (about 2 hours) and their text and phonemes. Because of phonemes are in English characters I use them as texts of the wav files in training. Then start training Tacotron for 20 epochs (about 4 hours) and it reached loss 0.6 . After that I use this inference file to get results of tests. But I get this results:

and the resulted audio is just some noise..
I tried to use original text file and results don't change.
what's the problem with that? what should I do?

I finally understand what was the problem. I just must train the model more..
After about 200 epochs (10 times more) results became so good.


Screen Black, no Video only on Android

Hi I'm trying to use the media_kit for see a video stream but, the screen is full black, and I have this error message:
Invalid argument(s): Failed to load dynamic library 'libavcodec.so': dlopen failed: library "libavcodec.so" not found
I/flutter (23344): #0      _open (dart:ffi-patch/ffi_dynamic_library_patch.dart:11:43)
I/flutter (23344): #1      new DynamicLibrary.open (dart:ffi-patch/ffi_dynamic_library_patch.dart:22:12)
I/flutter (23344): #2      AndroidHelper.ensureInitialized (package:media_kit/src/utils/android_helper.dart:23:43)
I/flutter (23344): #3      libmpvEnsureInitialized (package:media_kit/src/player/libmpv/player/real.dart:40:17)
I/flutter (23344): #4      MediaKit.ensureInitialized (package:media_kit/src/media_kit.dart:39:7)
i have tested the video on Ios and works just fine, and the problem is only here.

Upgrade all dependencies as specified in "Installation" section.
Provide your pubspec.yaml.

ac_connectivity: ^0.1.0
ansicolor: ^2.0.1
auto_orientation: ^2.3.1
badges: ^3.1.1
camera: ^0.10.5+2
cloud_firestore: ^4.7.1
collection: ^1.17.1
crypto: ^3.0.3
dart_reed_solomon_nullsafety: ^0.2.0
device_info_plus: ^8.2.2
dio: ^5.1.1
dio_cache_interceptor: ^3.4.2
dio_cache_interceptor_hive_store: ^3.2.1
encrypt: ^5.0.1
equatable: ^2.0.5
expandable_tree_menu: ^0.1.0-dev.5
file_picker: ^5.2.10
fire_tv_listener: ^0.0.6
firebase_auth: ^4.5.0
firebase_core: ^2.13.0
firebase_dart: ^1.0.9
firebase_storage: ^11.2.1
flutter:
sdk: flutter
flutter_hls_parser: ^2.0.1
flutter_localizations:
sdk: flutter
flutter_meedu_media_kit: ^4.2.12
flutter_slidable: ^3.0.0
flutter_svg: ^2.0.5
font_awesome_flutter: ^10.5.0
get: ^4.6.5
google_fonts: ^4.0.4
http: ^0.13.6
icons_launcher: ^2.1.0
intl: ^0.18.0
js: ^0.6.5
json_annotation: ^4.8.1
jumping_dot: ^0.0.6
local_value: ^1.2.0
logger: ^2.0.1
media_kit: 1.0.2
media_kit_video: ^1.0.2
path_provider: ^2.0.15
responsive_builder: ^0.7.0
scrollable_positioned_list: ^0.3.8
shelf: ^1.4.1
shimmer: ^3.0.0
shrink_sidemenu: ^2.0.0+2-null-safety
sleek_circular_slider: ^2.0.1
url_launcher: ^6.1.11
uuid: ^3.0.7
wakelock: ^0.6.2
win32: ^3.1.4

I'm going to delete issue from this repository to maintain hygiene, you can keep personal record on your e-mail.
The help is provided above. You'll obviously get error if you haven't added dependency.


Remove useless INFO ide_ci::program::command: prefix in the CI output

When looking at CI output it seems to be dominated by INFO ide_ci::program::command:. Almost every interesting line starts with:
INFO ide_ci::program::command:
INFO ide_ci::program::command:
INFO ide_ci::program::command:
INFO ide_ci::program::command:
INFO ide_ci::program::command:
INFO ide_ci::program::command:
INFO ide_ci::program::command:

please remove this prefix and (possibly) replace it with something more useful.

One thing I also find very hard is to identify what command a line belongs to. I've seen prefixes like [pid XYZ] in some longs and they were very helpful, especially when things were printed in parallel. What about:

when executing a command assign it an identifier (1, 2, 3, ...)
print out the command to be executed with a [job X] prefix - e.g. [job 3] ls -l /
prefix every line of output with such prefix - e.g. [job 3] /etc or [job 3] /bin

Such a prefix would be more valuable replacement of the INFO ide_ci::program::command: one.

I believe these logs are coming from functions that are annotated with #[instrument]. Is there any way how to configure the format of the output for this instrumentation @mwu-tow ?

Could we also add timestamps to the logs? From the current format, there is no way how to infer how long did a command take.

FWIW the timestamps are added by the GH Actions, they can be displayed by toggling this:

(but that's only for CI runs)

I believe these logs are coming from functions that are annotated with #[instrument]. Is there any way how to configure the format of the output for this instrumentation @mwu-tow ?

Yes, it can be adjusted. What you see is the default log format. Logs are prefixed with log level and module name.

One thing I also find very hard is to identify what command a line belongs to. I've seen prefixes like [pid XYZ] in some logs and they were very helpful, especially when things were printed in parallel. What about:
* when executing a command assign it an identifier (1, 2, 3, ...)

* print out the command to be executed with a `[job X]` prefix - e.g. `[job 3] ls -l /`

* prefix every line of output with such prefix - e.g. `[job 3] /etc` or `[job 3] /bin`

Such a prefix would be more valuable replacement of the INFO ide_ci::program::command: one.

Yes, these are good suggestions.
The workaround for now is to look backwards for <START> log for the process. Just before it there will be log from the process spawning that includes command details. Similar log is for the stream end, as the process finishes.
This won't be reliable if there are multiple processes with the same binary name interleaving their outputs — but I don't think we have such cases?


testoptix enhancements to make more analogous to testrender

Change -r to -res to match testrender (and for that matter, make -res
work for testshade, not -g). (In both cases, the old oddballs are
silent synonyms.)

Respond to several additional OSL options, and set them the same
was as testrender

Allow --iters (much like it works in testshade) to make it easy to
run all the rays many times to make accurate timing easier.

Do some of synchronizing of near-identical code between
testshade, testrender, and testoptix. And synchronize some of the
analogous command line arguments (e.g., -res for everybody, not -res
for testrender and -r for testoptix and -g for testsahde).

Add empty OPTIMIZEONLY marker files in the optix test directories
to signify that they don't need -O0 test cases run (currently, such tests
will fail due to the OptiX path not yet supporting "init ops").

LGTM


objc_macos_opt_native: Test Suite 'MacTests.xctest' and 'Selected tests' failed

https://source.cloud.google.com/results/invocations/3766bb98-eeca-4e6b-bdce-2e20b497d687/targets/grpc%2Fcore%2Fpull_request%2Fmacos%2Fgrpc_basictests_objc_mac/log
Test Suite 'StressTestsSSL' started at 2019-08-05 16:17:18.817
Test Case '-[StressTestsSSL testNetworkFlapWithV1API]' started.
Test Case '-[StressTestsSSL testNetworkFlapWithV1API]' passed (0.358 seconds).
Test Case '-[StressTestsSSL testNetworkFlapWithV2API]' started.
Test Case '-[StressTestsSSL testNetworkFlapWithV2API]' passed (11.079 seconds).
Test Suite 'StressTestsSSL' passed at 2019-08-05 16:17:30.255.
	 Executed 2 tests, with 0 failures (0 unexpected) in 11.437 (11.439) seconds
Test Suite 'MacTests.xctest' failed at 2019-08-05 16:17:30.256.
	 Executed 123 tests, with 1 failure (0 unexpected) in 61.198 (61.259) seconds
Test Suite 'Selected tests' failed at 2019-08-05 16:17:30.257.
	 Executed 123 tests, with 1 failure (0 unexpected) in 61.198 (61.260) seconds
kill -9 `jobs -p` ; echo "EXIT TIME:  $(date)"
jobs -p
date
EXIT TIME:  Mon Aug  5 16:17:30 PDT 2019

I don't see other info.

This was the failed test:
/Volumes/BuildData/tmpfs/src/github/grpc/workspace_objc_macos_opt_native/src/objective-c/tests/UnitTests/APIv2Tests.m:412: error: -[CallAPIv2Tests testTimeout] : ((error.code) equal to (GRPCErrorCodeDeadlineExceeded)) failed: ("14") is not equal to ("4")


[Bug Report] Default theme css - cannot find module

Element UI version
2.0.1
OS/Browsers version
Google Chrome
Vue version
2.5.2
Reproduction Link
https://jsfiddle.net/r7qwke1f/38/
Steps to reproduce
Try to Include CSS  to html
What is Expected?
success loaded css
What is actually happening?
Getting error<EMAIL_ADDRESS>Failed to load resource: the server responded with a status of 404 ()"

2.0 theme file changed to theme-chalk

#7868


extended attributes in 0.14 / linux

attic checks if extended attributes are enabled in func defined in attic/xattr.py
def is_enabled():
"""Determine if xattr is enabled on the filesystem
"""
with tempfile.NamedTemporaryFile() as fd:
.....
but really the func only checks if extended attributes are enabled in /tmp directory, what seems to be not quite right for every scenario  (in opensuse 13.2 /tmp is mountpoint for tmpfs).
There are also other issue concerning extenden attributed - not every filesystem makes possible to set it for links, what attic seems to assume, when extended attributes work it must work the same way in every filesystem.
Tests in my opensuse 13.2 installation can't really be performed:
Traceback (most recent call last):
File "/usr/lib/python3.4/site-packages/Attic-0.14_dirty-py3.4-linux-i686.egg/attic/testsuite/archiver.py", line 152, in test_basic_functionality
self.create_test_files()
File "/usr/lib/python3.4/site-packages/Attic-0.14_dirty-py3.4-linux-i686.egg/attic/testsuite/archiver.py", line 145, in create_test_files
xattr.setxattr(os.path.join(self.input_path, 'link1'), 'user.foo_symlink', b'bar_symlink', follow_symlinks=False)
File "/usr/lib/python3.4/site-packages/Attic-0.14_dirty-py3.4-linux-i686.egg/attic/xattr.py", line 101, in setxattr
_check(func(path, name, value, len(value) if value else 0, 0), path)
File "/usr/lib/python3.4/site-packages/Attic-0.14_dirty-py3.4-linux-i686.egg/attic/xattr.py", line 36, in _check
raise OSError(get_errno(), path)
PermissionError: [Errno 1] b'/tmp/tmpx6jsc97w/input/link1'

I can confirm the issue with setxattr on symlinks, that line 145 in the traceback. And some other linux people on the IRC chan have the same issue.
About is_enabled checking just /tmp (or $TMP, $TMPDIR, $TEMP):
It's good enough for the tests, as they create all input data there.
I agree it should get a path=None param, so it is more generally useful and can be called upon a specific path.

See PR #235.


Remove assignable field from roles

Since we're always getting/adding auto-assignable roles, the assignable boolean is useless, since it's always true.

IIRC it was to enable in a future update to allow people with access to the bot but not to the "manage roles" (which is the shittiest permission ever) to add specific roles to other persons.
If not needed, then yeah let's scrap it i guess ?

We'll see about that later, after I'm done with #2. :)


⚠ API - Lens has degraded performance

In 838aae7, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 503
Response time: 711 ms

Resolved: API - Lens performance has improved in ff8e180 after 6 minutes.


Want to add GIF creator

Aim
I want to add GIF creator using python.
Details
-It created GIFs.
can you please assign me this task @avinashkranjan

Aim
I want to add GIF creator using python.
Details
-It created GIFs. can you please assign me this task @avinashkranjan

GO AHEAD @Kalivarapubindusree


Codec combinator system

a handful of combinators for easy building of hand-made codecs for records would be nice
can probably take inspiration from chiron/fleece which are entirely this approach

No longer a real value add

This isnt happening


Incompatible with cmislib from trunk

If you try to run with a build of cmislib from trunk (0.6 Dev), eg to support
the Browser binding as well, then the import of CmisClient from cmislibalf
fails with:

    from cmislibalf import CmisClient
  File "cmislibalf/__init__.py", line 22, in <module>
    model.Document._initData = initData
AttributeError: 'module' object has no attribute 'Document'

It seems that the Document object has moved to domain, with subclasses in the
atompub and browser bindings

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 1 Dec 2014 at 7:05

Hi,
I am facing the same issue, is there any way to help or make it work ?
best regards


Background: Don't prevent quit if shutdown fails

If attempting to stop k8smanager fails, we should display an error and then quit anyway.  Otherwise the user will be stuck with an infinite chain of dialog boxes with no way of actually stopping.
Fixes #258

Hmm, it seemed to work for me:

Downloaded the artifact (zip containing dmg containing .app)
Dropped the .app on a (FAT32-formatted) USB disk.
Right clicked on the .app in Finder, and Option-clicked Open
Got the warning, clicked Open. (The Open button is not available unless you Option-click on the previous menu)
App started
Got sudo prompt, clicked cancel. Get Error starting Kubernetes dialog. Close it.
App is running, error in the status.
⌘-Q
Got sudo prompt again, cancel again
Get Error Starting Kubernetes again, close it.
App quits successfully.

macOS Catalina 10.15.7


Rename bin

This PR changes the main binary to aperture so that you know what process this is in your acitivity monitor, currently it was just "main", which is a bit confusing 😄
See here for an example (before):

~Also, seems like the processes aren't exited correctly? 🤔  when playing around, i had 10's of main processes in my activity monitor 😕~
see #35 👍

I tried using the child_process argv0 option, that made it show up correctly in htop, but not in Activity Monitor.

As far as I recall, the main is mandatory 🤔 I think it's for when you run it as a script (which I used to do when developing). Will download Xcode etc and test it 👍

another option is to symlink main to aperture 🤔

As far as I recall, the main is mandatory

I don't think it is.


As a user, I want to create a new task, enter a title and description and assign a due date and priority (i.e.none, low, medium, high) to it, in order to add the task to the task list.

[ ] User clicks on “create new task” and can fill in the desired information.
[ ] User can select the task priority from a predefined set.
[ ] After filling in the information, the user can save the task and is redirected to the task overview page.
[ ] The newly created task appears in the task lists.

Estimate: 8h


proof gpt-4 can post without your permission

This issue is created to demonstrate that GPT-4 can create issues in a GitHub repository. Please note that this action was performed with the user's consent.

Yes, it would be cool to have an AI knowledgeable of the repo deployed :)
Like here: https://github.com/hwchase17/langchain/issues/5472


MVVM approach is weird

Why is the sample using static members in the viewmodel? I have never seen this approach, even for simplification's sake, and it could be confusing to beginners. I think changing it to a regular viewmodel and instantiating a static copy somewhere, maybe in the own ContentPage's resources, would be more beneficial since it shows how it is done more commonly.

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

ID: ed970855-6db1-6838-12cf-5a6f58b8ebf4
Version Independent ID: edfe1b4d-58d2-477d-89b2-2641789cf2db
Content: Part 5. From Data Bindings to MVVM - Xamarin
Content Source: docs/xamarin-forms/xaml/xaml-basics/data-bindings-to-mvvm.md
Product: xamarin
GitHub Login: @charlespetzold
Microsoft Alias: chape

The PageDataViewModel class has only one static member. The Title and Description members used for binding on the TextCell in the XAML file are both instance members, as is the Type property accessed in the code-behind file.
The only static member is All, which is a collection of all the instances of PageDataViewModel. There is no reason for each of the instances of PageDataViewModel to have its own instance of this collection. That would be wasteful and awkward. That's why that All property is static and initialized in a static constructor.


MAAS-106 & MAAS-104 | Security enhancements

These don't pose any actual threat ATM so it is completely fine to have this PR in public 🙂

Codecov Report

Merging #69 (6d8ed43) into main (123e1f3) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #69      +/-   ##
==========================================
+ Coverage   89.20%   89.21%   +0.01%
==========================================
  Files          48       48
  Lines        1519     1521       +2
  Branches      121      121
==========================================
+ Hits         1355     1357       +2
  Misses        129      129
  Partials       35       35

Impacted Files
Coverage Δ

maritime_maas/settings.py
83.33% <100.00%> (+0.57%)
:arrow_up:

Continue to review full report at Codecov.

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


[WIP] - QML learning path

Link to preview

NOTE: do not merge yet

Fix https://github.com/Qiskit/qiskit.org/issues/2228

converting to to active PR so we can get updated previews. do not merge until ready

a couple comments:

should the IBM Quantum Lab text be a link to https://quantum-computing.ibm.com/lab?

the following links did not work

@vabarbosa thank you! Should be all fixed in d37b093 hopefully.

@frankharkins @JRussellHuffman i believe this is good to go. should the [WIP] be removed and we proceed with merging this afternoon?


Golang and bmclib updates

Description
Update our versions of golang and bmclib.
Why is this needed
Keeping things up to date.
How Has This Been Tested?

[ ] Manually
[ ] In an Equinix Facility

How are existing users impacted? What migration steps/scripts do we need?
Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

@nshalman this can be closed now since its been covered in https://github.com/tinkerbell/pbnj/pull/137


[CARBONDATA-3012] Added support for full scan queries for vector direct fill.

This PR depends on PR https://github.com/apache/carbondata/pull/2818
After decompressing the page in our V3 reader we can immediately fill the data to a vector without any condition checks inside loops. So here complete column page data is set to column vector in a single batch and gives back data to Spark/Presto.
For this purpose, a new method is added in ColumnPageDecoder
ColumnPage decodeAndFillVector(byte[] input, int offset, int length, ColumnVectorInfo vectorInfo,
      BitSet nullBits, boolean isLVEncoded)

The above method takes vector fill it in a single loop without any checks inside loop.
And also added new method inside DimensionDataChunkStore
 void fillVector(int[] invertedIndex, int[] invertedIndexReverse, byte[] data,
      ColumnVectorInfo vectorInfo);

The above method takes vector fill it in a single loop without any checks inside loop.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/995/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9063/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/798/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1001/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/802/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9069/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/804/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/805/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/807/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9072/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/813/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1010/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/816/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9081/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1013/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/885/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9150/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1083/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/887/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9152/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1085/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/900/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/904/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1098/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9169/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/911/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1111/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/918/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1118/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9178/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/930/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1006/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1220/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1007/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1021/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9286/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1022/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9287/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1234/

LGTM

Merged


Refactored, improved and fixed SleepSchedule

Short Description:

Refactored code
Fixed edge case if sleep time has started previous day
Brought back SCHEDULING_MARGIN
Removed unused LOG_INTERVAL_SECONDS
Unified time output format
Added feature to enable/disable every single schedule entry
Improved behaviour in cases if config is empty of all entries are disabled
Improved behaviour on config parsing/missing config keys errors (ValueError for time and duration, warning for time_random_offset and duration_random_offset)
Updated existing tests and added some more.

Fixes:

#5242

https://github.com/PokemonGoF/PokemonGo-Bot/pull/5189#issuecomment-245460154