mapping between downloaded images and pandas data frame rows

This page has belwo code sample to download images in a dataset and also to convert to pandas dataframe. I like to confirm if indexes in animal_pd (pandas dataframe) and download_path (list) match each other. For e.g. will download_path[0] always correspond to first entry in animal_pd ?
Is there a guaranatee that orders of rows in to_pandas_dataframe() will be same as order of elements in list returned by .download() ? Without this ordering, we cannot really map between dataframe and downloaded images.
import azureml.core
from azureml.core import Dataset, Workspace

# get animal_labels dataset from the workspace
animal_labels = Dataset.get_by_name(workspace, 'animal_labels')
animal_pd = animal_labels.to_pandas_dataframe()

# download the images to local
download_path = animal_labels.download(stream_column='image_url')

import matplotlib.pyplot as plt
import matplotlib.image as mpimg

#read images from downloaded path
img = mpimg.imread(download_path[0])
imgplot = plt.imshow(img)

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

ID: c02d3f15-dbc8-a1d7-6f95-2f3527d002f0
Version Independent ID: 6c17f89c-c984-c4f5-d9f8-b943521938fc
Content: Create and explore datasets with labels - Azure Machine Learning
Content Source: articles/machine-learning/v1/how-to-use-labeled-dataset.md
Service: machine-learning
Sub-service: mldata
GitHub Login: @Blackmist
Microsoft Alias: larryfr

@kiranpradeep
Thanks for your feedback! We will investigate and update as appropriate.

@Blackmist
Could you please review and add your comments on this, update as appropriate.

@kiranpradeep
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Thanks for the question @kiranpradeep
@kvijaykannan and @sdgilley can you provide any information on whether there are any guarantees on the order of rows .to_dataframe_pandas() and .download() methods of datasets?

#reassign:sdgilley

@kiranpradeep The order of the the images should be the same as the execution follows identical path
However there is a way more ergonomic way of achieving this goal which would not even need to download the files in the first place. Your 'image_url' column contains file pointer class StreamInfo that object implements open() method which is a File-like object and could be used by any python library that expects file objects
so in effect you can change you code to look like this:
`
import azureml.core
from azureml.core import Dataset, Workspace
get animal_labels dataset from the workspace
animal_labels = Dataset.get_by_name(workspace, 'animal_labels')
animal_pd = animal_labels.to_pandas_dataframe()
import matplotlib.pyplot as plt
import matplotlib.image as mpimg
#read images from downloaded path
img = mpimg.imread(animal_pd['image_url'].iloc(0).open())
imgplot = plt.imshow(img)
`
This will let you stream from remote storage directly and has an additional side benefit of not requiring the whole dataset to fit on disk (streaming processing)
As you can see in this case there is no risk of mismatched records as both image_url and labels would be coming from the same dataframe record

Thanks, @anliakho2!  I'm updating our document to use your modification of the code.

Additionally if files need to be accesses multiple times it will make sense to control caching and parallelization degree for downloading each file stream. Here is what it exposed to make it possible:
from azureml.dataprep.rslex import BufferingOptions, Downloader, CachingOptions, FileCacheOptions
file_cache_options = FileCacheOptions(file_cache_size=10 * 1024 * 1024 * 1024) # use up to 10 GB on disk to cache opened files
caching_options = CachingOptions(memory_cache_size=512 * 1024 * 1024, file_cache_options) # use up to 512 MB of RAM to cache file contents
downloader = Downloader(block_size=1024, read_threads=8, caching_options)
buffering_options = BufferingOptions(1, downloader).

import azureml.core
from azureml.core import Dataset, Workspace

# get animal_labels dataset from the workspace
animal_labels = Dataset.get_by_name(workspace, 'animal_labels')
animal_pd = animal_labels.to_pandas_dataframe()

import matplotlib.pyplot as plt
import matplotlib.image as mpimg

#read images directly from remote stream using configured buffering options
img = mpimg.imread(animal_pd['image_url'].iloc(0).open(buffering_options))
imgplot = plt.imshow(img)

@sdgilley  as for documentation I think a scenario where images are processed in a loop would be more appropriate.
Also take not of updated comment when images are accessed as those are no longer downloaded, so comment needs to change too


refactor: remove the all option

Summary
Closes https://github.com/biomejs/biome/issues/4663
Unfortunately, there's no migration path other than removing the option.

Test Plan
Removed all snapshots and tests that were using the option.

I agree with removing the top-level all property, but is it necessary to remove it for all categories as well? If so, what will be the replacement for enabling a category?

To enable a group, a user will be able to assign true to the entire group e.g. "style": true.
While this will be possible, we should discourage its usage because it has the same pitfalls of the previous logic

Makes perfect sense! Is the plan still on to have environment groups or whatever we end up calling them? Something like "react": true should be valid and not have conflicting rules within its group, I think.

Yes that's the plan. The proposed name for this concept was "domains"


fix formatting issues

closed and sent PR to current working branch 4.5.x
https://github.com/wso2/carbon-kernel/pull/2098


MTP connection fails with USB3 drivers

If XHCI USB drivers are enabled from bios, windows and linux will be unable to access the phone via MTP protocol. EHCI drivers just works. Also confirmed this issue by second computer.

Do you know whether it works on Sony Android? on other Sailfish devices?

Yes I have been tested that before flashed as also moved some files to xz3 succesfully.

As USB3 works for me on Linux for MTP, closing this as stale. We have switched bases meanwhile and it probably has to be reevaluated if it is still true


Updating subscription email template

Issue #: https://github.com/bcgov/met-public/issues/2182
Description of changes:

Updating subscription email template to add unsubscribe path
Updating subject on verification email

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #2270 (7cedea1) into main (c1b874d) will decrease coverage by 0.01%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main    #2270      +/-   ##
==========================================
- Coverage   68.47%   68.47%   -0.01%
==========================================
  Files         472      472
  Lines       15602    15605       +3
  Branches     1191     1191
==========================================
+ Hits        10684    10685       +1
- Misses       4712     4714       +2
  Partials      206      206

Flag
Coverage Δ

metapi
76.46% <40.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

met-api/src/met_api/config.py
96.75% <ø> (ø)

...src/met_api/services/email_verification_service.py
72.72% <40.00%> (-0.73%)
:arrow_down:


Maven: add repositories

@pascalgrimaud Should I create a variant for snapshotRepository?

@DanielFran : if you think it can be usefull in short roadmap yes. Otherwise, I'd think it's better to add new methods only when needed :)

@pascalgrimaud Bug bounty claimed: https://opencollective.com/generator-jhipster/expenses/60925

@DanielFran : approved


Issue 14 IE 11 Fix

Removed CustomEvent polyfill which seemed to cause issues with other
mdl components. Replaced with Helper method. Updated references to
CustomEvent defintion with calls to helper method instead.

@ahlechandre could you merge this?

Sorry for the long time but I'm very busy lately and did not have time to check the PR code.

Currently, I can't merge this because the changes was in the bundle file instead of in the entry point.
.
+-- src
|     +-- stepper
|           +-- stepper.js (entry) <-- You should put it here.
+-- stepper.js (bundle) <-- Where you did.
+-- stepper.min.js (bundle)

The JavasScript bundle files (placed in the root directory) are automatically generated by a transpiling process using Babel and Webpack. To contribute properly, you should write the changes in entry file and commit so then I can merge this PR.
Optionally, If you want to generate the bundles locally, place your changes in entry point and follows:
# Inside mdl-stepper/ folder.
# Install dev dependencies.
npm install
# Build a development version of javascript changes.
npm run dev
# Build a production version of javascript changes.
npm run prod
# Build a production version of javascript changes on WINDOWS.
# npm run prod-windows

Look at scripts in package.json to see what internally means each command.

My bad. I don't often work with transpiling processes. I'm holding off for a few though, as I am running into an issue with the stepper actions not triggering that I believe is in relation to my changes.

Alright latest commit is to the entry point and is working in IE 11, Edge, and Chrome browsers. Events are triggering as expected.

I just made some changes to adjust it to Google code style.
Thanks @Rathxxx for contributing to v1.1.6.


[WIP] Java 9 Support

related to https://github.com/Jasig/uPortal/pull/1007

closing while Jasig/uPortal#1007 is stalled


2.7.2 timeouts

just upgraded from 2.7.1 to 2.7.2 and get a lot of these errors:
            File "C:\Users\your_username\.local\pipx\venvs\ofscraper\lib\site-packages\ofscraper\utils\download.py", line 227, in main_download_downloader
              async for chunk in r.iter_chunked(1024):
            File "C:\Users\your_username\.local\pipx\venvs\ofscraper\lib\site-packages\aiohttp\streams.py", line 35, in __anext__
              rv = await self.read_func()
            File "C:\Users\your_username\.local\pipx\venvs\ofscraper\lib\site-packages\aiohttp\streams.py", line 385, in read
              await self._wait("read")
            File "C:\Users\your_username\.local\pipx\venvs\ofscraper\lib\site-packages\aiohttp\streams.py", line 303, in _wait
              with self._timer:
            File "C:\Users\your_username\.local\pipx\venvs\ofscraper\lib\site-packages\aiohttp\helpers.py", line 721, in __exit__
              raise asyncio.TimeoutError from None
          asyncio.exceptions.TimeoutError

TRACEBACK  [download.main_download_downloader:240]

and i noticed it downloads many more things at once. around 27 videos i counted were downloading at the same time, which seems risky to me and could be related to the timeouts.
also shift+right stopped working for me in the menu

I'm curious  why it timing out
But the changes I pushed weren't ready for prime time
I realize now the command I ran did just push the development branch rather then the branch I made specifically for the 2.7.2 release
2.7.3 should have the code base I original want to push


Consolidate Toolbar in Manuscript Editor

At the moment we have some icons, one is already problematic ('$') because not common and has ambiguous meaning (inline vs display version).
With the rather huge number of content types the flat icon tool approach won't hold.
Thus I suggest to add a dropdown menu as we have it in MetadataEditor in first place, and use such tools only contextually (e.g. Table tools) and for 'formatting' annotations.

@michael I think this is important, because it changes the Toolbar drastically. I would like to have a solution that is good enough for the whole thing at the end.

Fine with me. Will also result in a less crowded toolbar.
I wonder if we should call it "Insert" instead of "Add". Maybe call it "Insert" also in the MetadataEditor.

I am also thinking about a dedicated ToolGroup for the contextually available tools (e.g. Table tools)

Moving this to 2.0 as not critical. This requires a different implementation of InsertFigure tool, currently we can't render custom tools inside menu items.

It seems to be more critical, after the first round of feedback.

This is exactly what we need to do.

I can take care of the UI parts and arranging things in the insert menu. I may need your help to fix FigureTool.

Wait, we also need a workflow for stuff we add from the manuscript panel (Modal) or we jump to the metadata panel at the right position. But then we would need some jump logic. I'm not sure we want to open that box so late in the game...? But no question we need to act here soon. Just the question is for 1.0 or 1.1 ?

I don't want to do the jump thing, for the beginning.
First iteration should be to restructure the toolbar.
You can start with a concept first, without the impl.
Then we should change kit/ui/Menu. One simple way could be to specify the MenuItemClass in the itemSpec.  We need to think if there is a mechanism to e.g. embed the Tool into a MenuItem, or somehow different, so that we can easily switch between a Tool and a Menu oriented UI.
Then we would add everything which is 'straight-forward', just as to consolidate.
After that I would open a new issue, which is about adding more things, e.g. add author - let's discuss details then there.

Re: reuse of tool impl. Probably it would be good to have an 'ActionItem' or something like this, used either by ToggleTool or MenuItem, so it should not add to many things, more the action stuff, and a render method so that it can add the native input forinstance.

Re: when to do it. Let's start with the easy things, and shift the complicated ones.

@michael let's come up with a plan first. Could you sketch a UI? Don't hesitate to introduce new requirements regarding Menus and Tools. I will be happy to address that.
Having that we can decide when and how to approach this.


Enh/control child name mangling

The Python name (via attributes) of a child device is foo.bar.baz, however
for a number of historical reasons (allowed keys in mongo and for attribute
access in Pandas data frames with columns named from the data keys in an event
stream) ophyd-sync has replaced '.' with '_' in the names.  This leads to
ambiguity when looking at the (default) name to which component on which device
the data came from as there may be _ in the attribute names.  Ophyd-async
used '-' which does allow for round tripping.
This makes it controllable on per-instance basis in ophyd-sync so that users
can either make a different set of trade offs.
xref bluesky/ophyd-async#666

I probably should move the doc commit to its own PR...


By default Use temporary working branch with git scm

Signed-off-by: Olblak<EMAIL_ADDRESS>By default Use temporary working branch with git scm
Requirement for https://github.com/updatecli/updatecli/pull/744

Test
To test this pull request, I used the following manifest to manifest that in one case we have a temporary branch and not in the second situation
title: "Test git scm resource"

scms:
  tmpDir:
    kind: git
    spec:
      url: https://github.com/updatecli/updatecli.git
      disableworkingbranch: true
      branch: main
  noneTmpDir:
    kind: git
    spec:
      url: https://github.com/updatecli/website.git
      branch: master

sources:
  default:
    kind: jenkins

targets:
  tmpDir:
    name: "tmpdir"
    kind: file
    scmid: tmpDir
    spec:
      file: README.adoc
  noneTmpDir:
    name: "testnonetmpdir"
    kind: file
    scmid: noneTmpDir
    spec:
      file: README.md

Additional Information
Tradeoff
Up to know the default behavior for git scm was to not use a temporary branch such as "updatecli_xxxx". To keep the current manifest working we could provide an ugprade path where automatically set "disableworkingbranch: true" for version older than 0.27, and we had a warning message
Potential improvement
Now that I am experimentation with gitea support, I realize that scm of type other than "svn, or mercurial" doesn't add any value and quite the opposite make the code more difficult to maintain. On mid term, I am considering deprecating the github scm kind

Missing the migration warning

Missing the warning message

Up to know the default behavior for git scm was to not use a temporary branch such as "updatecli_xxxx". To keep the current manifest working we could provide an ugprade path where automatically set "disableworkingbranch: true" for version older than 0.27, and we had a warning message

Well in fact, I don't see any other way than keeping the current behavior when used without pullrequest as I don't want to introduce breaking changes.
So I am going  to check if a scm of kind "git" is used alone, if it's the case then I just print a warning message saying that the behavior will be modified in coming releases so we give enough time to run the migration tool updatecli manifest upgrade

I am putting this PR on old until we finish the work introduced by @dduportal
https://github.com/updatecli/updatecli/pull/713

I think I'll have another look to this but I am considering changing the parameter name from disableworkingbranch to workflow
Where workflow could accept the following parameters

"centralized", means commit directly to the branch defined in the manifest
"feature-branch", means commits to a temporary branch
"github/gitea", means commits to a temporary branch and open a pullrequest

In

Can you clarify, it is not really clear?

Can you clarify, it is not really clear?

No nevermind, I'll try to have a look to your PR and the issue you created.
You better captured and explained the problem.


Duplication Glitch from Redstone Control and Drive Disconnecting

Issue description:
Odd Duplication Glitch, with different functions on Server/SP
What happens:
Duplicates items. If you disconnect drives (untested with other storage options) and use redstone control on the controller, it creates these "ghost items" which you can take.
What you expected to happen:
Not duplicate items. When you disconnect drives, it shouldn't show the items.
Steps to reproduce:
On Server:

Disable Controller using Redstone Control
Disconnect Drives from System
Re-Enable Controller
Take items and done! System re-syncs if you re-connect cables or place/break controller. (Possibly also syncs if you do anything to the system itself, untested)

On SP:

Disable Controller using Redstone Control.
Disconnect Drives from System
Take Storage Disks out from Drives
Re-Enable Controller
Take items!
Place Disks back in Drives and re-connect Drives.

In SP, if you don't take the storage disks out from the drives, the ghost items still appear, but taking them out still takes them out of the drives. (Even though the drives are NOT connected to the system). Taking Disks out from Drives still allows you to duplicate items.
...
Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: <IP_ADDRESS>32
Refined Storage: 1.5.34

Does this issue occur on a server? [yes/no]

Yes, it occurs on Servers and Single Player, with slight differences.

Fixed, thx for reporting!


boundary: 0.2.3 -> 0.3.0

Motivation for this change
Bump boundary to 0.3.0
Things done

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

[X] 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 nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
21.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

[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 126500 run on x86_64-linux 1

1 package built:

boundary


Add new post on release automation

@fmvilas thanks a lot for a detailed review man! helpfull suggestions!
all changes applied, you can have another look 👁


[gql-tag-operations-preset] Generated index.ts should contain \n regardless of whether source contains LF or CRLF

First of all, I would like to thank the creators of gql-tag-operations-preset, it's a really clever solution to have type safety while avoiding explicit imports.
Describe the bug
TL;DR
Generated index.ts will cause TypeScript compilation to fail when source files contain CRLF (e.g. when using Windows).
Long description
When source file contains LF, generated index.ts will contain \n and TypeScript will compile successfully.
When source file contains CRLF, generated index.ts will contain \r\n and TypeScript will not compile.
When source file contains CRLF, and generated index.ts is manually changed to contain \n, TypeScript will compile successfully.
It seems that generated index.ts should contain \n regardless of whether the source contains LF or CRLF.
To Reproduce
NOTE: I have created a minimal reproduction example. It contains  schema, codegen.yml and operations.
Steps to reproduce the problem (using Linux):
1. Modify src/app/app.tsx by replacing LF with CRLF:
$ unix2dos src/app/app.tsx
unix2dos: converting file src/app/app.tsx to DOS format...

New lines are now CRLF:
$ cat -A src/app/app.tsx
import {gql} from "@my/gql";^M$
import {useQuery} from "@apollo/client";^M$
import React from "react";^M$
^M$
const PET_LIST = gql(`^M$
  query Get_Pet_List {^M$
    petList {^M$
      id^M$
      identificationNumber^M$
    }^M$
  }^M$
`);^M$
^M$
export const MyComponent = () => {^M$
  const {data} = useQuery(PET_LIST);^M$
^M$
  return <>{data?.petList?.[0]?.id}</>;^M$
};^M$

2. Run graphql-codegen by executing npm run generate:
$ npm run generate

><EMAIL_ADDRESS>generate
> graphql-codegen

(node:9963) ExperimentalWarning: stream/web is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
  ✔ Parse configuration
  ✔ Generate outputs

The generated index.ts file will now contain \r\n:
$ cat src/gql/index.ts
/* eslint-disable */
import * as graphql from './graphql';
import { TypedDocumentNode as DocumentNode } from '@graphql-typed-document-node/core';

const documents = {
    "\r\n  query Get_Pet_List {\r\n    petList {\r\n      id\r\n      identificationNumber\r\n    }\r\n  }\r\n": graphql.Get_Pet_ListDocument,
};

. . .

3. Running tsc will now result in a compilation error:
$ tsc
src/app/app.tsx:15:27 - error TS2345: Argument of type 'unknown' is not assignable to parameter of type 'DocumentNode | TypedDocumentNode<any, Record<string, any>>'.
  Type 'unknown' is not assignable to type 'TypedDocumentNode<any, Record<string, any>>'.

15   const {data} = useQuery(PET_LIST);
                             ~~~~~~~~

Found 1 error.

4. Now, if you go to generated index.ts and manually change all \r\n to \n and run tsc again - it will compile.
Environment

"@graphql-codegen/cli": "^2.3.1",
"@graphql-codegen/gql-tag-operations-preset": "^1.1.7",
"@graphql-codegen/typescript": "^2.2.2",
NodeJS: 16.13.1

Additional info
This ticket suggests configuring .gitattributes to force using LF instead of CRLF on Windows. However, in my opinion it is not reasonable to ask users to change their CRLF strategy in order to work around this, it seems too invasive. Also, unless I'm missing something, it won't seem to help with source files created on Windows machine.

We had a similar issue reported, and the following seems to be the most reasonable approach: https://github.com/dotansimha/graphql-code-generator/issues/6811#issuecomment-997936255
Do you agree with that, would you like to help out documenting that or exploring alternative solutions?

Hi @n1ru4l
I don't think https://github.com/dotansimha/graphql-code-generator/issues/6811#issuecomment-997936255 is the best solution because it won't help when a new source file is created on Windows.
I can offer my help working on this issue.
Empirically it seems that outputting \n works for both CRLF and LF source files. I will look into it more deeply. If you have any thoughts on that - it would be most helpful.

It's seems that TypeScript does not differentiate between CRLF and LF - both become \n.

// This template literal contains CRLF as line breaks
type CRLF = `one
two
`;

// This template literal contains LF as line breaks
type LF = `one
two
`;

const test1: CRLF = 'one\r\ntwo\n'; // TS2322: Type '"one\r\ntwo\n"' is not assignable to type '"one\ntwo\n"'.
const test2: LF = 'one\r\ntwo\n'; // TS2322: Type '"one\r\ntwo\n"' is not assignable to type '"one\ntwo\n"'.
const test3: CRLF = 'one\ntwo\n'; // compilation passes
const test4: LF = 'one\ntwo\n'; // compilation passes

I'll work on a PR.

TypeScript Playground Links for context: https://www.typescriptlang.org/play?#code/PTAEBUAsEsGdQC4FMC2AHANgQ2aD1kAnLDUAYwHsA7BLaK+AYQCUAZAMVC3nyqVABGhJFgDWsALAAoBAE80-Fh1ABeUAANqSaQgDuFaeoDc06SAgx4ydNlz4iJctVr14y7nnr8hI8Tvn8ymqafDr6hiZS0pQMCIhIsAgAjABcoEqcagDkWgA6hLk0+oVZRqDm4ADKAEwAzNXVaeABoFkARHkFRRSFbVmgcKBUFHHcsNAA5lRYAhj8CBSILe153b1ZAHTR1InxiY2gQa2dhXo9VKXlYFV1DU3LHXz5p8VUfQPww6Ow41Mzc4hFnIFK1HkgXuc+lspDFdshErU0hlVMcnmsLmVzJR0NBbNBqKA0GMEttYnsEAAWNJHHJos4lTFgbFoXE4fFUQnE2BAA

Fixed via https://github.com/dotansimha/graphql-code-generator/pull/7369

I am having a similar issue, is this happening again?


Add 2 papers on one shot learning and medical image analysis

Issue fix: #304
Add interesting papers read this week implementation of insights needs to be done

Thanks for the comment, I have removed the files and add my brief notes. I will try to implement these over the weekend:)


Added a description on the readme of what the project is about.

Just some minor copy and pasting, so a casual visitor can know what this project is about.

cool, thank you very much!


No such file or directory during "kubeadm"

vm creation is ok.
if fails to copy  the kubeadm during the task: "download : kubeadm | Copy kubeadm binary from download dir" with the following error:
TASK [download : kubeadm | Copy kubeadm binary from download dir] **************
Tuesday 16 July 2019  09:16:33 +0200 (0:00:01.932)       0:05:29.185 **********
fatal: [k8s-kubespray-master-0 -> <IP_ADDRESS>]: FAILED! => {"changed": false, "cmd": "sshpass", "msg": "[Errno 2] No such file or directory", "rc": 2}
and /tmp/release is ok:
ll /tmp/releases/
total 290144
-rwxr-xr-x. 1 root root  30863968 Jul 16 07:55 calicoctl
-rwxr-xr-x. 1 root root  15385105 Jul 16 07:55 cni-plugins-amd64-v0.6.0.tgz
-rwxr-xr-x. 1 root root 211257536 Jul 16 07:55 hyperkube
-rwxr-xr-x. 1 root root  39591200 Jul 16 07:55 kubeadm

did u fix it?

yep thanks. it was a ansible.cfg param os a sshpass config issue. thanks

@tdovan what was the fix ?


Group function on cursors

Hey there, just want to say this is a great JSON file storage package!
I would love if you guys implemented something along the lines of:
db.find({}).group({field})

Me too.


pythonPackages.root_numpy: add setuptools dependency

Motivation for this change
noticed it was broken in #75569
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @
[2 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/75641
2 package were built:
python27Packages.root_numpy python27Packages.rootpy

@GrahamcOfBorg build python27Packages.root_numpy python27Packages.rootpy


Use minimatch instead of glob for mac os x support.

Thanks for this; I was a bit worried about adding a dep that required compilation.
Will test & push a fix this weekend (sunday or after, most likely).
Cheers,
D


SyntaxError: Identifier '$' has already been declared

Describe the bug
I am creating a component library using the following configuration
export default defineConfig({
    plugins:[
        vue({
            reactivityTransform: true
        }),
    ],
    ...
})

After compiling, this module is introduced in a new project and reactivityTransform is also turned on
This error occurs. I tried #7854, but it didn't work
https://github.com/vitejs/vite/issues/7854
Reproduction
...
Steps to reproduce
No response
System Info
"vue": "^3.2.45",
"@vitejs/plugin-vue": "^3.2.0",
"vite": "^3.2.5"

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

@zz541843 Would you provide a repository that describes what you are doing? (doesn't need to reproduce the issue)

@zz541843 Would you provide a repository that describes what you are doing? (doesn't need to reproduce the issue)

I may not be able to provide it, but I think I can make it clear. What I'm doing is encapsulating a component library xx-ui using vue3 and packaging it with vite, enabling reactivityTransform, then creating a new project pnpm create vue and also enabling reactivityTransform, Then introduce xx-ui, and it is possible to get an error for $, because the compiled code of xx-ui has a chance to have a function called $. How to avoid this problem?

I need to know the precise steps and the actual config file.

@sapphi-red this is repo => https://github.com/zz541843/vite-issuse-11355
I was just simulating a problem, so i didn't set build.rollupOptions.external include “vue"

git pull
2.pnpm i -r
3.cd packages/xx-ui  && pnpm build
4.cd packages/xx-web && pnpm dev


Profile and speed up global illumination correction

@starcalibre reports that global illumination correction is slow. Global feature detection will probably also be slow. But both of these methods are theoretically fast, so it's worth figuring out what is slowing them down.

Closed by #78


fix conda develp error message

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Proposing to remove conda develop . after partially reading a long discussion on conda github about whether to deprecate or remove it in projects. Based on my (partial) understanding of our project, this is a good mitigation
Tests performed by the developer:
Tips for QA testing:

Darn it, done mixed things up ain't I? Being dumbo-fingered is no fun :'(


Utils repair style

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

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

repair style of bash files in utils for brew style with shellcheck --enable=all
https://github.com/Homebrew/brew/blob/fa3af8d24d0335641e58a4355fa9de6d42ae4982/Library/Homebrew/style.rb#L188
related #11086

Thanks again @hyuraku!


[New Content]: Indexers section

What would you like us to add?
A distinct section discussing indexers. Specific content has been prepared and is currently awaiting approval to proceed with opening the PR.
How important it is?
Indexers enable dApps to access past on-chain data such as transaction histories, smart contract states, and event logs more easily. This speeds up data searches and supports real-time updates, ensuring smooth operation for various apps and platforms.

This is a good suggestion, and we're looking at adding developer tooling and infra available on Scroll soon, including links to external guides, tutorial and documentation.
We of course can't feature a specific indexer to the exclusion of other indexers, but once we provide an area for this content, we'll encourage subquery to provide useful content that bring value to Scroll developers.

Going to leave this issue open for now -- we're not planning an "indexer" section per se, but hopefully it will remind us to reach back out.


defmt-decoder: DWARF analysis should consider DW_TAG_compile_unit

https://github.com/knurling-rs/defmt/blob/5ef0ba7979e0eb75f981e512c2dd42b7df88fcdf/decoder/src/elf2table/mod.rs#L290-L306
this part of the analysis is only correct when the DWARF contains a single "compile unit", i.e. the DW_TAG_compile_unit does not appear in the DWARF information.
when DW_TAG_compile_unit appears in the DWARF info, the namespaces (= Rust module paths) may not start at depth 0. instead the linked logic should use the depth at which DW_TAG_compiler unit appears as the start depth of the module path; IOW, a new module path should be constructed when DW_TAG_compiler_unit is seen.

when DW_TAG_compile_unit appears

I was not able to come up with a repro for this that depends on defmt but have seen this issue with some other code that analyzes DWARF info to extract location info like defmt does. that other code uses the dev profile with opt-level=s and lto=thin (nightly-2021-12-16) but those compiler settings did not produce more than one DWARF "compile unit" when I tried them with app-template

PS pretty sure this is unrelated to issue #649

I was wrong here and mixing data from entries that come from different "units" in my implementation. The defmt implementation is correct in that DW_TAG_compile_unit only appears once per "unit" (and there are as many "unit"s in an ELF file as the number of codegen-units (rustc setting) the program was compiled with)


Add more SLAP to Model for event related commands

Codecov Report

Merging #114 (f9d1f55) into master (ef98523) will decrease coverage by 0.10%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #114      +/-   ##
============================================
- Coverage     68.47%   68.36%   -0.11%
- Complexity      595      596       +1
============================================
  Files           100      100
  Lines          2106     2099       -7
  Branches        259      259
============================================
- Hits           1442     1435       -7
  Misses          576      576
  Partials         88       88

Impacted Files
Coverage Δ

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...dress/logic/commands/schedule/AddEventCommand.java
72.72% <100.00%> (-4.20%)
:arrow_down:

...ss/logic/commands/schedule/DeleteEventCommand.java
70.83% <100.00%> (-4.17%)
:arrow_down:

...ress/logic/commands/schedule/EditEventCommand.java
96.15% <100.00%> (-0.52%)
:arrow_down:

...rc/main/java/seedu/address/model/ModelManager.java
98.24% <100.00%> (+0.16%)
: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 ef98523...f9d1f55. Read the comment docs.

LGTM for me, @hanqinilnix @markbiju wdyt?

Looks good to me too.


Create Standard AppDev-Styled Launchscreen

See Eatery / BRShuttle
Background Color: Icon Background Color
Centered Graphic: Icon Graphic
Footer Graphic: AppDev Icon with appropriate tint (possibly with Cornell AppDev word mark below)

What we talked about this briefly in pod meeting! Maybe design can mock something up maybe based on some of the above ideas


When using s3gof3r as a library, is there a progress bar feature?

When uploading a large blob, it would be nice to see progress bar.

See #13 for discussion of this. It's not possible on upload due to streaming.


Marker Error: TypeError: this.getNativeComponent is not a function (it is undefined)

Summary
After installing react-native-maps, I cannot implement markers. The MapView works, but the marker does not.
Error:
ERROR Warning: TypeError: this.getNativeComponent is not a function (it is undefined)
Here is my babel.config.js file:
module.exports = {
  presets: ['module:metro-react-native-babel-preset'],
  env: {
    production: {
      plugins: ['react-native-paper/babel'],
    },
  },
  plugins: [
    [
      'module-resolver',
      {
        root: ['./src'],
        alias: {
          '@': './src',
        },
      },
    ],
    ['@babel/plugin-transform-class-properties', { loose: true }],
    ['@babel/plugin-transform-private-methods', { loose: true }],
    ['@babel/plugin-transform-private-property-in-object', { loose: true }],
    [
      'module:react-native-dotenv',
      {
        moduleName: 'react-native-dotenv',
        path: '.env',
      },
    ],
  ],
};

Running on React Native v0.75.4
React Native Maps version: 1.20.1
Reproducible sample code
<View style={{ flex: 1 }}>
			<MapView
				style={StyleSheet.absoluteFillObject}
				initialRegion={INITIAL_REGION}
				showsUserLocation
				showsMyLocationButton
				ref={mapRef}
				onRegionChangeComplete={onRegionChange}
			>
				{markers.map((marker, index) => (
					<MapMarker
						key={index}
						title={marker.name}
						coordinate={marker}
						onPress={() => onMarkerSelected(marker)}
					>
						<Callout onPress={calloutPressed}>
							<View style={{ padding: 10 }}>
								<Text style={{ fontSize: 24 }}>Hello</Text>
							</View>
						</Callout>
					</MapMarker>
				))}
			</MapView>
		</View>

Steps to reproduce
Just try to call the Marker component
Expected result
I expect to display markers on the map
Actual result
ERROR Warning: TypeError: this.getNativeComponent is not a function (it is undefined)

React Native Maps Version
1.20.1
What platforms are you seeing the problem on?
Android, iOS (Apple Maps), iOS (Google Maps)
React Native Version
0.75.4
What version of Expo are you using?
Not using Expo
Device(s)
iPhone 15 Pro Max
Additional information
No response

Same here. @olipda Any solution?

I have exactly the same issue

I am also get the same error

Me too , this issue

Facing same issue in RN 0.76.3

@olipda please don't use this lib with the latest version of react-native, for now Fabric is not supported yet (in progress)
see: https://github.com/react-native-maps/react-native-maps/issues/5206#issuecomment-2504728236
and main readme


Quick reference window for a task

Is your feature request related to a problem? Please describe.
Love this mod! I built a little extension for another todo list mod to open up task details in a little quick reference window like so:

That way I can look at nested tasks more easily and understand what's happening without having the main tasks window open and having to visually parse all the indentation and nesting.
Describe the solution you'd like
Replicating the screenshot above for this mod (basically just breaking out one single task + all its subtasks + edit/add buttons into a new window)
Additional context
Add any other context or screenshots about the feature request here.

Unfortunately this will be non-trivial, but I will add it in the future.

Getting there. I decided that rather than trying to generalize the checkboxes component, I would just duplicate all of the code. Not as "clean" as generalizing it, but much less of a headache as well.

After putting this off for a long time, I've come to the realization that this probably isn't going to happen. I am really unhappy with the implementation, and I don't have the time or energy to refactor it into something I can live with. I'm sorry.

I have pushed the task-details-gui branch if you want to use it yourself. Keep in mind that it's incomplete, though.

Thanks! I'll take a look and see


create minimal test script for eeg-expy experiments

started a minimal test script, (probably) to be used outside of the github CI, for checking experiments are running ok.

merging now, will continue to build this up in future PRs.


GPC-NONE: Add hmpo dev template

Might be easier to put the HMPO policy in the manual stack and export the queue vals?


Cannot advance to the news task

Challenge Change the Color of Text has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_1) AppleWebKit/601.2.7 (KHTML, like Gecko) Version/9.0.1 Safari/601.2.7.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 Github issues are not for help, so please visit the Help Chat if you need assistance. Thanks and happy coding!

https://www.ibm.com/developerworks/community/blogs/cloudview?sortby=0&maxresults=10&order=asc&lang=es


removes Fast Property caching so app can start up.

The fast property endpoint is no longer valid and was throwing an exception that was preventing the app from starting up.
Since we do not use any of the Fast Property methods that reliy on this cache we are just loading it with empty lists.

asgard-pull-requests #527 FAILURE
Looks like there's a problem with this pull request


refactor: move bot direct hardware control over to DeepHorOpMode

For easy implementation of initializtion, auto & manual control, we can move all hardware initialization, movement commands (drive, arm, slide, swing) to an abstract class that we can refrence.
This lets us

Easy hardware porting across autos & teleops
Practicing DRY
Easy Initalization
And more!


Optional fields get left at default values even if assigned

Hi,
I'd like to report strange behaviour when I have a schema with optional fields.
Here's my schema:
syntax = "proto2";

message GpsPayload {
	required bool gps_fix = 1;
	required int64 epoch = 4;
	required int32 battery = 5;
	optional float latitude = 2;
	optional float longitude = 3;
	optional float speed_kph = 6;
	optional float heading = 7;
	optional float altitude = 8;
}

Here's my arduino code:
  // Buffer for storing the encoded protocol buffer message
  uint8_t buffer[128];
  GpsPayload payload = GpsPayload_init_zero;
  pb_ostream_t stream = pb_ostream_from_buffer(buffer, sizeof(buffer));
  // Set GPS data
  payload.gps_fix = gps_fix;
  payload.latitude = latitude;
  payload.longitude = longitude;
  payload.speed_kph = speed_kph;
  payload.heading = heading;
  payload.altitude = altitude;
  // Set metadata
  payload.epoch = now();
  uint16_t batt_percent;
  if (fona.getBattPercent(&batt_percent)) {
    payload.battery = batt_percent;
  }
  Serial.flush();

  Watchdog.reset();

  // Serialise Protocol Buffer
  bool status = pb_encode(&stream, GpsPayload_fields, &payload);
  uint8_t pb_length = stream.bytes_written;

With this schema, the stream.bytes_written is always 10 bytes whether the optional fields are set or not.
When I decode the message in Python I get this (N.B. Latitude, Longitude, Altitude should not be 0)
payload: \x08\x01\x20\xd3\xf5\xb9\xf4\x05\x28\x2f
GPS Fix: True
Epoch:<PHONE_NUMBER>
Battery: 47 %
Latitude: 0.0
Longitude: 0.0
Speed km/h: 0.0
Heading: 0.0
Altitude: 0.0

However, when I change the schema so that optional fields are required, but make no changes to the C code it works as intended.
syntax = "proto2";

message GpsPayload {
	required bool gps_fix = 1;
	required int64 epoch = 4;
	required int32 battery = 5;
	required float latitude = 2;
	required float longitude = 3;
	required float speed_kph = 6;
	required float heading = 7;
	required float altitude = 8;
}

With this schema, the stream.bytes_written is significantly larger, 35 bytes.
When I decode the message I get this:
payload: \x08\x01\x15\xcd\x4f\x4b\x42\x1d\xcd\x08\x49\xbe\x20\x97\xf6\xb9\xf4\x05\x28\x31\x35\x00\x00\x00\x00\x3d\x00\x00\x00\x00\x45\x33\x33\x6d\x42
Decoding message:
GPS Fix: True
Epoch:<PHONE_NUMBER>
Battery: 49 %
Latitude: 50.82793045043945
Longitude: -0.19632263481616974
Speed km/h: 0.0
Heading: 0.0
Altitude: 59.29999923706055

Any ideas why this is occurring?

In proto2 syntax and C, there is no direct way to detect when an optional field is set or not set. Nanopb generates separate fields bool has_altitude etc. in your struct, that you have to set to true if they are present.
(In C++, this is solved with setter methods. In proto3 syntax, fields are present if their value is non-zero.)

Thank you :)

It worked. Thanks!


Roll engine bb8e6e4dbcbb..0345967a1964 (1 commits)

git@github.com:flutter/engine.git/compare/bb8e6e4dbcbb...0345967a1964
git log bb8e6e4dbcbb..0345967a1964 --no-merges --oneline
2019-08-22<EMAIL_ADDRESS>Include Java stack trace in method channel invocations (flutter/engine#11361)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (liyuqian@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5125419289804800

Commit queue failed; closing this roll.


Add the ability to import local fonts

Your idea:
Currently, to make a new theme (dynamically), I can get away with it without real code. The style editor allows me to add a background image, change font size, colors, images etc
But I cannot add new fonts. So I am locked to the fonts already existing.
My suggestion is for the possibility to import fonts by uploading them (locally on the client) e.g. .ttf and .otf
If it's too hard given uploading local image is not possible, then linking to a font website should work imo.
Until then, I would be thankful to anyone who can tell me how to import new fonts to the browser via developer tools.

Its not a matter of it being difficult, its a matter of we are not a hosting site. You can upload a font to fontsquirrel or even github, or get google fonts, and use them in your brew. This will not be implemented as you suggest.
How to use custom fonts:
If the default/web-safe fonts are not satisfactory, and Google Fonts doesn’t help either, you can generate your own web font kit using FontSquirrel.com with some effort.
Follow the terms of the font’s license.

Download a font, generally a .zip file (extract it if so).

At the Fontsquirrel Generator, check “legally allowed to upload” and click Upload Font. Navigate to your font file and upload. Choose “Expert” option.

Set following options: WOFF2 format, WebOnly protection, Base64 Encode & Style Link. “Download Your Font”.

Open your new font folder (“webfontkit-[date]-…”) and open the “stylesheet.css” file in a text editor. Most of it is gibberish, copy and paste the entirety into your Style Editor. Set the desired element’s font-family to match the name found in the stylesheet.css.

OR

Upload the .woff2 file (or .ttf, .otf, whatever you want) to a Github account, push that file’s URL through Githack, and @import the prod URL into your brew. Set your font-family for the desired element.

In the brew, the way to use a custom font is to import it, using the @font-face rule, example as follows:
@font-face {
  font-family: "Trickster";
  font-style: regular;
  font-weight: normal;
  src: url("trickster.otf") format("opentype") ,
    url("trickster.ttf") format("truetype"),
    url("trickster.woff") format("woff");
}

Note: you only need to specify a font style and font weight if you want to use a bold or thin version (font-weight) or italics (font-style).
If you need anything else, you should probably ask on reddit or discord better. But this is plenty to go on. Closing this.

Also, if you have the fonts installed on your computer, you should still be able to use them— but when others try to view your brew online, it won’t work because the font isn’t hosted.
But if you don’t care about sharing your brew as a Homebrewery link, you can just print it to PDF and share that, and it should embed the font.

I thank you very much for the detailed guide. I luckily need it to work only locally (to generate the PDF) as you described in your latest message, so I skipped the part where I upload fonts. And it works :)
I just installed the font locally, rebooted and it works on the css styling!
Again, thank you for covering both use-cases since I wasn't clear on which one I wanted, and I honestly didn't expect such a swift response time


Use FactoryServer.sh for daemon execution

FactoryServer.sh sets up a few environment variables and provides the path to the server binary for launching Satisfactory Dedicated server. I do not believe this env vars are used during execution.
With the migration to UE5 imminent for the experimental channel, this should handle changes to the game files that may include a new path to launching the server.

Whilst this change should (in theory) handle the experimental branch drop next week, I'd propose an additional change that replicates the functionality of this script with binary auto-detection once names are known, as well as option parsing in the case that newer binaries have different or alternate launch options (to better support running newer/older update branches).

This looks good, thank you! I hadn't come across FactoryServer.sh yet as I've been busy with work. Really appreciate you taking the initiative :) I ran the server with your changes and everything looked good 😎


Creation of duplicate nodes

I don't know exactly what goes on with it, but it appears to create duplicate nodes at times.
Look at these two changesets: in the second one, several months afterwards, the waste basket was added again in the same spot; and I did not do it. 129200972, 138303681
Same happened with this: 138293099, 138303681
I would never have realized if it was not for this osmose correction: 138497435

Perhaps this is a faulty one https://github.com/Zverik/every_door/issues/540
I see in https://osmcha.org/changesets/138511312 / objects created on 2023-03-23

Try clearing uploaded objects in the settings. Fix https://github.com/Zverik/every_door/issues/540 It was a month ago

I've encountered this same issue yesterday, but in this changeset   I didn't move any node, I just wanted to create a gate and the app created two.

created_by... Vespucci <IP_ADDRESS> ?

Ups, I was totally sure I used EveryDoor

@deevroman I have cleared the downloaded objects and the cache; but in the settings it still displayed an uploading queue with a lot of old edits I had made.
I manually deleted all of those from the queue and I have used EveryDoor to map a little bit afterwards.
I'll later have to check if any duplicate node is still being added to the map or if this problem was solved.
If this problem is solved, it may still be a good idea to let people know it may happen and to clear the upload queue after updating to #540.

The edits had all been sent already; among the things left to upload were the waste basket and the travel agency I previously mentioned.
The app considered those as not uploaded, and I imagine it would have uploaded them again the next time I made an edit.

Could you provide a log from menu / About Every Door 3.1 / System Log / send? (mention this issue ID) after this duplicate upload happens (would be good to intentionally upload duplicates twice in a row).
I guess it might prove informative

As far as I can see, no duplicate nodes have been created since I cleared the upload queue. I am thus unable to reproduce the problem anymore.
All items in the upload queue were things that had been uploaded previously.

Can confirm. I mapped from a fresh slate, deleted downloaded data beforehand. I added some POIs earlier today, uploaded. Went out tonight, mapped new things, uploaded, then noticed duplicates of all the things I had mapped earlier.
Now I check the upload queue, and there are things in there from two weeks ago saying Pending. I checked it, and there are three of it now (since deleted).
I did use the app to delete the duplicates, and it seems to have successfully done that and removed them from the queue, but it looks like new additions stick around.

@pkoby Are you able to submit a log reproducing the error before clearing the upload queue? I am unfortunately unable to reproduce the problem.

Oops, I already cleared the queue. I have been trying to duplicate the issue, or at least I'm keeping track of what things I'm doing in the app to try to troubleshoot if it happens again, but so far no luck(?).
So far, events that haven't triggered it are:

Adding a POI, closing app, reopening, then upload
Adding a POI, moving a bunch, then upload
Adding a POI, leaving it for a bunch of time, then upload
Adding a POI, clearing the downloaded data, redownload, then upload

I think I have a lead.
Let's start with @pkoby's edits

It can be seen that the edits resulted from a mechanism for splitting large edits into parts.
A bit later there are more edits that arose in this way and led to duplicates, but let's move on to @davidoskky edits
It seems to be a matter of splitting edits here too, but there's more weirdness:

If it’s clear from the last empty edit: the connection is probably broken, then where the edits come from without comments is unclear
What can be done:

understand in the code why empty edits occur
add a warning that there are unsubmitted edits. This would be a useful message to the user
It would be useful to add a tag to edit packages created by splitting edits that would inform other mappers of this.

As a note, I remember that sometimes I would open the app and it would tell me there were changes to upload before I made any change.

here is reproducible example: https://github.com/Zverik/every_door/discussions/676
perhaps user can be instructed to gather logs?

perhaps user can be instructed to gather logs?

An osc file with the changes that EveryDoor is trying to submit would probably be helpful.

one more case:
https://www.openstreetmap.org/changeset/144179402
https://www.openstreetmap.org/changeset/144376523
(duplicated these rock and clock, heh)

How to try to reproduce. (for the tests I rebuilt EveryDoor and used the sandboxed API)
Maybe it won't reproduce the first time and I'm missing some mandatory steps, but I was able to reproduce twice roughly like this:

Download a zone that has a POI that is not tied to a way (in my case it was a tree)

Open iD and attach this tree to some line (I attached it to the border of an industrial zone). Send it to the server.

In EveryDoor add some objects.

Remove the tree

Add new objects.

Add some objects a few kilometres away from the tree (so that Every Door splits the edits)

Send the edits.

EveryDoor will show that there are no edits not uploaded (!)

Go to the edit queue and find the conflict.
9.1 If you download .osc, it will show that there is one edit left.

Close EveryDoor and unload it from memory.

Open it again.

The edits are back!
12.1 The .osc edits are back as well

Video from step 8:
https://github.com/Zverik/every_door/assets/25708359/b872bfef-dde7-4d87-973f-1fa94823e644

What's also interesting is that you don't see any warning about broken edits. But if you try to make more edits and preferably within a few kilometres of each other you get:

Moreover now the deletion of loaded data is broken:
https://github.com/Zverik/every_door/assets/25708359/145c3871-70e9-40d3-a12d-7c338a4f1417


Cannot generate Docute document (error message)

Hello,
Thanks for this interesting project. I have tried to set up a Docute document, but I'm failing using the gen command every time using vuese gen --include="src/pages/search/*.vue".
The error reads
⏳  Start creating markdown files...
TypeError: Cannot read property 'children' of undefined
    at Object.comment (/usr/local/share/.config/yarn/global/node_modules/vue-template-compiler/build.js:2575:21)
    at parseHTML (/usr/local/share/.config/yarn/global/node_modules/vue-template-compiler/build.js:345:23)
    at parse (/usr/local/share/.config/yarn/global/node_modules/vue-template-compiler/build.js:2402:3)
    at baseCompile (/usr/local/share/.config/yarn/global/node_modules/vue-template-compiler/build.js:4267:13)
    at Object.compile (/usr/local/share/.config/yarn/global/node_modules/vue-template-compiler/build.js:4242:22)
    at sfcToAST (/usr/local/share/.config/yarn/global/node_modules/vuese/dist/vuese.js:28:47)
    at parser$1 (/usr/local/share/.config/yarn/global/node_modules/vuese/dist/vuese.js:402:20)
    at files.map (/usr/local/share/.config/yarn/global/node_modules/vuese/lib/index.js:30:23)
    at <anonymous>

Anyone knows why this happens?
Also, I'd like to know if the command parses all subdirectories automatically?
Thank you in advance

You need to know which one of the .vue files to handle is wrong and provide the contents of the file to me. There is no better way now, I am very sorry, then I will think about a better way to reproduce.

I also encountered the same problem, and in addition, in many pages, props, slots, methods, events can not be identified, resulting in the corresponding part of the generated document is empty.

Can you give me a reproduced source file? Can be sent to my email, I will provide an online recurring solution as soon as possible

Add one,but mine is a private project,just follow the issue.

I console.log something in vuese/dist/vuese.js, line 116:
            comments.split('\n').forEach(c => {                 if ((matchs = c.match(leadRE))) {                     currentKey = matchs[1];                     res[currentKey] = res[currentKey] || [];                     console.log(currentKey, ' | ', res[currentKey])                     res[currentKey].push(c.replace(leadRE, '').trim());                 }                 else {                     res.default.push(c);                 }             });
and then:
D:\www\APP_BASE_TEMPLATE>vuese gen --include="src/components/*.vue" ? Select the target to generate Docute - The fastest way to create a documentation site for your project. ∞ Start creating markdown files... param  |  [] param  |  [ 'x String X轴坐标' ] param  |  [ 'x String X轴坐标', 'y String X轴坐标' ] **constructor  |  function Object() { [native code] }** TypeError: res[currentKey].push is not a function     at comments.split.forEach.c (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\vuese\dist\vuese.js:117:37)     at Array.forEach (<anonymous>)     at commentNodes.forEach (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\vuese\dist\vuese.js:112:34)     at Array.forEach (<anonymous>)     at getComments (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\vuese\dist\vuese.js:93:18)     at ObjectProperty.properties.forEach (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\vuese\dist\vuese.js:245:37)     at Array.forEach (<anonymous>)     at ObjectProperty (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\vuese\dist\vuese.js:244:24)     at NodePath._call (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\@babel\traverse\lib\path\context.js:53:20)     at NodePath.call (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\@babel\traverse\lib\path\context.js:40:17)     at NodePath.visit (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\@babel\traverse\lib\path\context.js:88:12)     at TraversalContext.visitQueue (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\@babel\traverse\lib\context.js:118:16)     at TraversalContext.visitMultiple (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\@babel\traverse\lib\context.js:85:17)     at TraversalContext.visit (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\@babel\traverse\lib\context.js:144:19)     at Function.traverse.node (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\@babel\traverse\lib\index.js:94:17)     at NodePath.visit (C:\Users\ferrinweb\AppData\Local\Yarn\Data\global\node_modules\@babel\traverse\lib\path\context.js:95:18)
so i think you can check the value obtained from 'res[currentKey]', pay attention to the bold words in the error stack above.

@ferrinweb
I guess your comment should look like:
/**
 * @param xxx
 * @constructor xxx
 */

But only @arg is really recognized.
Read the documentation carefully:
https://github.com/HcySunYang/vuese#write-a-document-for-your-component

Another error stack:
TypeError: Cannot read property 'children' of undefined
    at Object.comment (C:\Users\zuoyi\AppData\Local\Yarn\Data\global\node_modules\vue-template-compiler\build.js:2575:21)
    at parseHTML (C:\Users\zuoyi\AppData\Local\Yarn\Data\global\node_modules\vue-template-compiler\build.js:345:23)
    at parse (C:\Users\zuoyi\AppData\Local\Yarn\Data\global\node_modules\vue-template-compiler\build.js:2402:3)
    at baseCompile (C:\Users\zuoyi\AppData\Local\Yarn\Data\global\node_modules\vue-template-compiler\build.js:4267:13)
    at Object.compile (C:\Users\zuoyi\AppData\Local\Yarn\Data\global\node_modules\vue-template-compiler\build.js:4242:22)
    at sfcToAST (C:\Users\zuoyi\AppData\Local\Yarn\Data\global\node_modules\vuese\dist\vuese.js:28:47)
    at parser$1 (C:\Users\zuoyi\AppData\Local\Yarn\Data\global\node_modules\vuese\dist\vuese.js:402:20)
    at files.map (C:\Users\zuoyi\AppData\Local\Yarn\Data\global\node_modules\vuese\lib\index.js:30:23)
    at <anonymous>

Does a multi-line comment (such as the one below) cause the error?
<!-- Author: ferrinweb -->
<!-- Create Time: 2018/9/30 14:34 -->
<!-- Description: image-editor -->
<!-- How to use?
import as component, and then:
<image-editor class="image-editor hw100"
    v-if="showEditor"
    :source="editSource"
    @image-edit-done="showNewImage"
></image-editor>
-->

@ferrinweb
Very weird behavior, are you writing this comment outside of the root element of the template? You try to write it inside the root element, this problem will not occur, this may be a problem with vue-template-compiler .

I have test, and you are right. thanks!
Thank you for this amazing work, again!

same problem... i use the demo
`

title

`

Close inactive


langchain: fix syntax error in code comment for create_tool_calling_agent

PR message:

Description: Corrected a syntax error in the code comments within the create_tool_calling_agent function in the langchain package.
Issue: N/A
Dependencies: No additional dependencies required.
Twitter handle: N/A

Thank you!


[terraform] Don't use quotes in the makefile

With the quotes in place, the underlying terraform vars would end up
with quotes in them, breaking a lot of the logic.  After we fix this,
we should probably undo
https://github.com/chef/chef-server/commit/e59d9cfc1141a42fc250359d1de92b195fa66812
which was me not realizing what was actually happening.
Signed-off-by: Steven Danna<EMAIL_ADDRESS>

good catch!


[Aio] Fix typo in async graceful shutdown example

Background: https://github.com/grpc/grpc/pull/26622

@donnadionne

@XiaoConstantine Thanks for the fix!
@yashykt Can you help to review this PR? As we need 2 maintainer reviews?


Chemical Fixation - Capitalisation issue

The temperature, duration, and instrument headers should be capitalised.

This issue have been fixed and is ready to be reviewed in localhost:3002

reviewed - looks good.

closed as part of merge request #22


How to implement custom titlebar?

I cann't find method to set titlebar background color, but I find description in src about custom titlebar.
Anyone have advices on this?
I don't like white color titlebar with dark theme as below:

I think you should hide titlebar use show_titlebar

But I need a title bar with mouse drag, resize and minimize/maximize/restore button.

If you want to change how the buttons in the menu look like, you have to change the env: Wrap your menu inside an env_scope.
This is a list env-keys which determine how the controls of druid look like.
An example how to change the values:
use druid::WidgetExt;
use druid::theme;

let menu_bar_widget = ...;

menu_bar_widget.env_scope(|env, data|{
    env.set(theme::BUTTON_DARK, Color::grey8(100));
    env.set(theme::BUTTON_LIGHT, Color::grey8(150));
    env.set(theme::BUTTON_BORDER_RADIUS, 0.0);
});

But I need a title bar with mouse drag, resize and minimize/maximize/restore button.

You have to make your custom widget do these actions( mouse drag, resize, minimize and so on  )

I implemented a custom title bar. Code is not great but works :sweat_smile:

Resize is handled for free, but the rest has to be handled manually. Relevant docs: https://docs.rs/druid/0.7.0/druid/struct.WindowHandle.html
The docs say handle_titlebar is Windows-only, but I think it's actually cross-platform, at least with Druid@master.

Closing as probably resolved. Please feel free to comment with any more questions.


Copying fails

Whenever I try to use this plugin to copy/move file from one folder to another, the file gets copied but the content of the file gets modified. All that copied file contains is [object Object]. Tried debugging the code but no solution...

Seeing the same here.


Inconsistencies that need to be fixed

make it a setting environment variable

reddit agree doesn't work

doesn't work yet

doesn't work
GET<EMAIL_ADDRESS>net::ERR_ABORTED 403
geolocation.html:23          GET<EMAIL_ADDRESS>net::ERR_ABORTED 403
index.html:1 Refused to execute script from 'https://smm-tester.ncsa.illinois.edu/graphql-doc/schema/scripts/focus-active.js' because its MIME type ('text/html') is not executable, and strict MIME type checking is enabled.
geolocation.html:34 Uncaught ReferenceError: ol is not defined
at geolocation.html:34:32

image crawler does not work


vscode:// url redirect does not open vscode. No alternatives offered.

Issue Type: Bug

On Ubuntu run snap install vscode to install from the snap store
Start vscode using the icon in the dashboard
Install the GitHub Pull Requests plugin and click the popup to login
Login to GitHub and click the "Continue" button
If the browser prompts to run xdg-open approve the request (this is the Linux mechanism to launch URL handlers
VSCode does not receive the URL trigger because snaps don't provide an interface for this feature yet.
Try to find an alternative way to trigger vscode with the URL directly, such as on the commandline, but realise there is no documentation on how to do such things :-)
That's it.

The vscode cannot be triggered via the URL handling mechanism as mentioned because of snap limitations currently. There is no alternative mechanism to get the URL to vscode that is readily identifiable to a user. It would be handy to provide a command line snippet that will achieve the same as the URL handler for cases where the URL handler fails.
Extension version: 0.2.0
VS Code version: Code 1.28.0 (431ef9da3cf88a7e164f9d33bf62695e07c6c2a9, 2018-10-05T14:52:58.945Z)
OS version: Linux x64 4.15.0-36-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz (8 x 3834)

GPU Status
2d_canvas: unavailable_softwarechecker_imaging: disabled_offflash_3d: unavailable_softwareflash_stage3d: unavailable_softwareflash_stage3d_baseline: unavailable_softwaregpu_compositing: unavailable_softwaremultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: unavailable_softwarevideo_encode: unavailable_softwarewebgl: enabled_readbackwebgl2: unavailable_off

Load (avg)
2, 2, 2

Memory (System)
31.30GB (0.40GB free)

Process Argv
.

Screen Reader
no

VM
0%

Sorry we didn't mention that anywhere but you can use personal access token for authentication by setting
    "github.hosts": [
        {
            "host": "https://github.com",
            "username": "",
            "token": ""
        }
    ]

could you give it a try?

We can add a copy-paste fallback for if the protocol handler fails.

Just tried with adding the token but unfortunately still experiencing same problem.

What permissions are required when using a PAT?

Hi! I think this flow should be fixed by https://github.com/Microsoft/vscode-pull-request-github/pull/659
The uri handler may not open, but the auth server and newest version of the extension now provide an alternative route.
Please open a new issue if there are any problems.

@pdf — It needs read/write on all your repos, public and private.  I'll update the docs with that information.


send predict request to triton tensorflow2, but meet data type not expect error.

I send a predict request to model mesh, but meet data type not expect error.
version: 0.9
os: macos
the request data for imput image tensor as blew
&ModelInferRequest_InferInputTensor {
  Name: "input_image",
  Datatype: "BYTES",
  Shape: [1],
  Contents: {
     BytesContents: [[1,2,3,4]] // image bytes.
  }
}

saved_model shape information.
The given SavedModel SignatureDef contains the following input(s):
  inputs['input_image'] tensor_info:
      dtype: DT_STRING
      shape: (-1)
      name: serving_default_input_image:0
  inputs['iou_thres'] tensor_info:
      dtype: DT_FLOAT
      shape: (-1)
      name: serving_default_iou_thres:0
  inputs['score_thres'] tensor_info:
      dtype: DT_FLOAT
      shape: (-1)
      name: serving_default_score_thres:0
The given SavedModel SignatureDef contains the following output(s):
  outputs['score_map'] tensor_info:
      dtype: DT_FLOAT
      shape: (-1, 6)
      name: StatefulPartitionedCall:0
Method name is: tensorflow/serving/predict

schema.json
{
    "inputs": [
        {
            "name": "input_image",
            "datatype": "BYTES",
            "shape": [
                1
            ]
        },
        {
            "name": "iou_thres",
            "datatype": "FP32",
            "shape": [
                1
            ]
        },
        {
            "name": "score_thres",
            "datatype": "FP32",
            "shape": [
                1
            ]
        }
    ],
    "outputs": [
        {
            "name": "score_map",
            "datatype": "FP32",
            "shape": [
                -1,
                6
            ]
        }
    ]
}

error log

@meijies it looks like you're sending the request as gRPC not REST, is that correct?
If so this problem shouldn't have anything to do with modelmesh, it's more how Triton is interpreting the input. You could try:

Loading the model in a standalone Triton container and sending it the same gRPC request
Updating to use the latest version of Triton
Open a similar issue in their github repo if you still have the problem

@njhill  thank you , I find the shape of input tensors should be (-1, 1), because batch requests are supported by default.


remove MDN links from English challenges

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #37827

@Sky020 You might try to search https://freecodecamp.org/news to see if you can find a good article there.  Many of the most popular Guide Topics were transported to a corresponding news article.
Currently, there are only 4 challenges with MDN links:

Increment a Number with JavaScript - The MDN information does not give any further incite to using the ++ operator, so I think we just remove this link all together.  Also, it would not make sense to create an entire news article for this, though we could create a Guide topic on the forum I suppose and make it a wiki.

Record Collection - The MDN link relates to the push method which is already introduced way before this challenge, so I think we can just delete this link.

Change Text Inside an Element Using jQuery - The MDN link here (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/em) actually has some merit.  I think we could either create a news article or a forum Guide topic with similar information and examples and link to it instead.

Use Assert.isOK and Assert.isNotOK - The Truthy and Falsy MDN links could be replaced by a link to the Falsy Bouncer challenge or a news or forum article covering the information in more detail.

The above is just my thoughts on the links.  If at all possible, I think the links to MDN should be removed.

Originally posted by @RandellDawson in https://github.com/freeCodeCamp/freeCodeCamp/issues/37827#issuecomment-665854174
Todo

[x] Increment a Number with JavaScript
[x] Change Text Inside an Element Using jQuery
[ ] Use Assert.isOK and Assert.isNotOK

What verbiage do we want here?

[ ] Record Collection

@RandellDawson , That is excellent! In this case, I suggest because other members were in agreement in the linked PR, we open this to help wanted (or first-timers), and do as you have suggested above.
One point on my side:

Record Collection - The MDN link relates to the push method which is already introduced way before this challenge, so I think we can just delete this link.

Instead of removing the link, I suggest it be changed to direct to the challenge that introduces the push method.
So, if you agree, Randell, I leave it to you to open this to first-timers or otherwise.
Originally posted by @Sky020 in https://github.com/freeCodeCamp/freeCodeCamp/issues/37827#issuecomment-665927113

Which push challenge is being referenced here? Want to ensure I replace this MDN link with the correct challenge link.

Hi @brignano, thanks for taking the time to create this PR.
For Record Collection, the first challenge to introduce push is here: https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures/basic-javascript/manipulate-arrays-with-push
We could link to that challenge instead of removing the link. For verbiage, maybe something like:
The `push` array method will be helpful here. Check out our Manipulate Arrays With push() challenge to review how it works.

Thanks for the quick response @scissorsneedfoodtoo :partying_face:
Record collection should be all set now as per your feedback.
Just want to confirm the verbiage I used for Use Assert.isOK and Assert.isNotOK,
https://github.com/freeCodeCamp/freeCodeCamp/blob/0a1a2b133f554db7df3705d2bc22965983971e84/curriculum/challenges/english/06-quality-assurance/quality-assurance-and-testing-with-chai/use-assert.isok-and-assert.isnotok.english.md#L15
Also, as per https://github.com/freeCodeCamp/freeCodeCamp/issues/37827#issuecomment-666066362, please let me know if there's anything I can contribute regarding the news article referenced in the below comment.

Change Text Inside an Element Using jQuery - The MDN link here (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/em) actually has some merit. I think we could either create a news article or a forum Guide topic with similar information and examples and link to it instead.

Thanks again!

@brignano, awesome, thanks for updating that so quickly. There wasn't a great resource on  elements on News, so I updated an existing article: https://www.freecodecamp.org/news/html-elements-explained-what-are-html-tags/#em-element
Would you mind adding that to your PR?
Also, I think the verbiage for the Use Assert.isOK and Assert.isNotOK sounds great. What do you think @RandellDawson?

Great @scissorsneedfoodtoo thanks for sharing the updated News article :+1:
I have updated both files as per your latest comments.

@scissorsneedfoodtoo If you are happy with the changes, I think it is ready to merge.

:sparkles: :sparkles: :sparkles: Congratulations on your first contribution to freeCodeCamp! :sparkles: :sparkles: :sparkles:
Thank you for this quick fix, @brignano. Looking forward to your next contribution.


configuration, consider removing the "enabled" field

in the below, the rule is actually disabled,
    "prefix_is_current_class": {
      "omitMeInstanceCalls": true
    },

It shouldn't:
https://github.com/abaplint/abaplint/blob/216e81a70d05af7668f447177720613c28a90a48/test/config.ts#L23-L37
But overall I agree. I'd remove applyUnspecifiedRules too, it's kind of a gimmick which makes your config unstable.

getEnabledRules: hmm, perhaps I did something wrong
applyUnspecifiedRules: yea, agree

Hi, I am currently updating my abaplint config files and noticed applyUnspecifiedRules got removed. Are unspecified rules now disabled by default or how do you get that behavior without that option now?

unspecified rules are disabled by default


CB-10025. Remove getObject requirement from cdp logging policy (S3).

See detailed description in the commit message.

@oleewere Is there anything that prevents putting this to "ready"?

@oleewere Is there anything that prevents putting this to "ready"?

i had to make sure this is still working, i have just validated it so i can open it for review

Super. 👍


Email Notify not working on .NET Core Standard 7

This is a fresh install of Elmah on .NET Core Standard 7
I set up as follows.   I do get a SQL report, however, I do not get an email notify.  I do not see any errors in the event log.   Running a packet sniffer, I see no attempt to contact Gmail.
            services.AddElmah<SqlErrorLog>(options => {
                options.Path = "~/ErrorTrap";
                options.ConnectionString = _connectionStrings.SqlServer;
                options.SqlServerDatabaseSchemaName = "Errors";
                options.SqlServerDatabaseTableName = "Error";
                options.Notifiers.Add(new ErrorMailNotifier("Custom", new EmailOptions()
                {
                    SendYsod = true,
                    AuthPassword = "<Redacted>",
                    AuthUserName = <Redacted>",
                    MailRecipient = "<Redacted>",
                    MailSender = "<Redacted>",
                    MailSubjectFormat = "Error ({1}): {0}",
                    ReportAsynchronously = false,
                    SmtpPort = 587,
                    SmtpServer = "smtp.gmail.com",
                }));
                options.ApplicationName = _programSettings.Customer + " API " + _programSettings.Site;
           });

I think you meant to report this to ElmahCore/ElmahCore?

Oh, I apologize, I did pick the wrong repository.


Change tray icon

Add transparency to tray icon and icon change when camera is on

Hey, that's awesome! Thanks for the PR, from a quick skim it seems very cool, I'll merge first and ask questions later, cheers!


Feature Request: add getName and getAvatar to identity kit

Describe the solution you'd like
Currently the identity package has useName and useAvatar hooks. It would be great to expose this logic outside of a hook for use in the server.
For example getName and getAvatar, then the hooks are wrappers around these.
Describe alternatives you've considered.
No response

Yeah you right, let me figure out how to get this going.


Specifying "no_default_flags(true)" doesn't actually disable all default flags when compile CUDA code with nvcc

Currently, I am trying to compile CUDA code with nvcc in my build script and export it to a .ptx file. The desired compile command is rather simple and looks like this.
nvcc -ptx ./cuda/example.cu

My build.rs looks like this
extern crate cc;

fn main() {
    cc::Build::new()
        .cuda(true)
        .no_default_flags(true)
        .flag("-ptx")
        .file("cuda/example.cu")
        .compile("example.ptx")
}

However it appears that even with "no_default_flags" enabled, my compilation command is this:
 nvcc -ccbin=c++ -Xcompiler -Wall -Xcompiler -Wextra -ptx -o /home/user/CLionProjects/CudaEx/target/debug/build/CudaEx-82cc37462807794c/out/cuda/example.o -c cuda/example.cu

How can I disable ALL default flags?

The explicit purpose of the cc-rs and the .compile() method in particular is to produce a static library and make arrangements so that it's then passed to rustc for linking into the final executable object. If you want to produce anything else, cc-rs is not the tool. Just use std::process::Command...

Wextra appears to not be a valid flag when compiling for Windows. Is it not possible to remove unless skipping cc?


Changing role to ClusterRole and generating a new CSV results in bad CSV

Bug Report

What did you do?
I changed an operator's role to a clusterRole in deploy/role.yaml. I would have rather added it with limited permission, but I understand having both is not currently supported. Then I generated a new CSV.
What did you expect to see?
I expected the new CSV to have a spec.install.spec.clusterPermissions value, but not a spec.install.spec.permissions value.
What did you see instead? Under which circumstances?
I saw both spec.install.spec.clusterPermissions and spec.install.spec.permissions. I expect the second set was pulled over from the previous CSV.

Ansible

Possible Solution
When generating a CSV from another, filter out the permissions and only consider the deploy/role.yaml as the source of truth for all permissions. Anything else may result in an unexpected level of permissions given to the operator. I'm imagining the potential of removing something from role.yaml, but that merging with the previous CSV resulting in the CSV still having the old permissions. That'd be easy to miss, and potentially problematic for security.
Additional context
Add any other context about the problem here.

Hi @flickerfly,
When we gen the CSV by the tool,  it will use the resources files in the deploy dir to generate the CSV. However, if we change these files and would like to reflect this change in the integration with OLM we need re-generate the CSV (usually it will mean a new version/tag of it).
So, I am not sure if I follow up what exactly you are pointing here in this issue. Were you expecting that change the roles would automatically change the CSV previous generated?  Could you please confirm my understanding?

Sorry, let me see if I can make it more clear.
I changed the deploy/role.yaml from a Kind: Role to a Kind: ClusterRole and generated CSV 2 noting CSV 1 as the CSV to be replaced. The result was CSV 2 with both the Role and the ClusterRole in it.

@flickerfly is this still an issue with a more recent version? This may have been fixed by an unrelated PR.

Closing this, feel free to re-open if your issue is not fixed in the most recent applicable version.


"make install" to a custom directory

Normally with a cmake project you can do:
cmake -DCMAKE_INSTALL_PREFIX=install/to/this/directory
make
make install

...and it will install binaries, headers, and libs to install/to/this/directory.  I don't see a way to do this with flatcc, but maybe I'm missing something obvious.  How would I go about installing to a custom directory?

I'm not a CMake expert on what is customary. By default flatcc installs into a local bin and lib dir, but there are variables that can be overridden. If you can come up with a more correct way of achieving this without breaking current behaviour please do.
The following PR may be relevant to this discussion: https://github.com/dvidelabs/flatcc/pull/109

It seems that AUR packages uses install prefix:
https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=flatcc
build() {
  cd "${srcdir}/${pkgname}-${pkgver}"

  mkdir -p build && cd build

  cmake \
    -DBUILD_SHARED_LIBS=on \
    -DFLATCC_ALLOW_WERROR=off \
    -DCMAKE_BUILD_TYPE=Release \
    -DCMAKE_INSTALL_PREFIX=/usr \
    -DFLATCC_INSTALL=on \
    ..

  make
}

Thanks, that answers my question.  Closing issue.


cmake support

I tried adding this myself, but encountered errors:
[  1%] Linking CXX executable jsonxx
/data/data/com.termux/files/usr/bin/../lib/crtbegin_dynamic.o: In function `_start':
crtbegin.c:(.text+0x84): undefined reference to `main'
clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation)        make[2]: *** [lib/jsonxx/CMakeFiles/jsonxx.dir/build.make:95: lib/jsonxx/jsonxx] Error 1
make[1]: *** [CMakeFiles/Makefile2:86: lib/jsonxx/CMakeFiles/jsonxx.dir/all] Error 2
make: *** [Makefile:130: all] Error 2

I'm guessing you used add_executable instead of add_library, although seeing the code you used might help more. Not sure what hjiang's position on cmake is, though.

Oh, i did use add_executable  😂
The annoying thing about cmake is that it replaces the current Makefile.


Setting "window.menuBarVisibility" should be "window.menuBarVisibility.windows"

Setting item "window.menuBarVisibility" should be "window.menuBarVisibility.windows", and exists on all platforms(windows, linux, osx).
When I copy the setting file (contains "window.mentBarVisibility") from windows to mac, vscode would give a warning: "Unknown configure setting". This almost drive me crazy.
I hope this configure setting would exists on all platforms, including windows, linux and osx, so that the warning would not exist.
However, this setting is only valid on windows. So I think it would be change to "window.menuBarVisibility.windows".

@roblourens since this applies to many settings

Dupe of https://github.com/Microsoft/vscode/issues/5595


Improve the warnings in addTilesetImage()

This PR

Adds a new feature

I tried to clarify the two warning messages, and the second one now prints the valid tilesets if you pass a bad tileset name. Should save some developer anguish.

:v:


Change entity to real char to avoid breaking type resolution

Hi,
The < entity after return type is breaking the phpstorm resolver:

Not sure if it's the right way to go as I'm not sure why there are entities and what'd happen if we remove them.
There are other functions where this is the case too (for example, strcoll), If this commit is validated, I'll modify the others

It looks like a bug for me. There is a space between type and description that distinguishes them. But PhpStorm somehow allows spaces in types declaration around | and &, even though that's not following PhpDoc standards and any popular PhpDoc parser would parse that differently.

@orklah Thank you very much for pointing this out! This is definitely the bug and I've filled it in YT. https://youtrack.jetbrains.com/issue/WI-50310, please feel free to vote. So I'm closing this PR as it's not related actually with stubs but rather is a bug in phpdoc processing.


Added a faked observation file for a new ctest for recently fixed lateral boundary halo points

This PR is in complementary to (depends on ) the PR https://github.com/JCSDA-internal/fv3-jedi/pull/1110 to Adding a hofx ctest for the fixed lateral boundary halo points treatments.
This PR would add a new small-size observation file with the (only) faked observation close to (on ) the lateral boundary).

add a purpose explicitly, to prevent any potential breakdowns in future development related to the halo handling?

Maybe instead of adding this obsfile you could generate one obs on the fly, using the GenList function in ufo?

@cmgas  Thanks for this suggestion, I will try this genlist function first.

@HuiLiu-NOAA  I will make this PR purpose explicit as you suggested if I still need this PR .

@cmgas , I just had a quick try to use genlist function in my fv3jedi test.  But it seems , such genlist needs  parameters outside of obsspace (from recordhandler.yaml as an example) to set up the values of the faked observation.  And, hence, from my understanding, it is unable to be used in fv3jedi tests (non-specific like test_RecordHandler.x"  to create an ob on fly by use of the obsspace yaml block. Please correct me if genlist could be used in apps like hofx and that will be great if you can give an example for that.

@TingLei-daprediction you can use it in a single ob test with FV3-JEDI, @DavidNew-NOAA has exercised this and I think @shoyokota was going to try it

@CoryMartin-NOAA  Thanks.  I will see if and how @DavidNew-NOAA and @shoyokota to do it.  For being now, I will change this PR to Draft PR (and the related PR on fv3-jedi).  Thanks

An update  , I said the obs values needed to be defined outside of obsspace for genlist type.  That is wrong.
IODA tests's yaml like iodatest_obsspace_mpi_reader_pool.yaml gives an example to specifiy obs values.  I will try that.

I have a run with genlist working and I will use it for the planned test without needs to add an obs file.
Thanks a lot to all the comments/suggestions I got here.
I will close this PR.


Add option to automatically expand lazy getters

@andrewbranch requested an option to automatically expand getters in https://github.com/microsoft/vscode-js-debug/issues/447. We should add an option to automatically expand getters in core now that there's native lazy variable support.

@roblourens great, that works nicely!


Update mrfutils.py

add connector thingy

This is a fine adjustment but I'm not sure if limiting to 10 connections is a good idea. Should we make this a param that the user passes in or...?
10 seems really low to me.


Internationalize NetNewsWire

Note: This PR is up-to-date with main and includes the SwiftUI rewrite of certain iOS screens in #3778. As such, the #3778 PR should be handled first which will remove many of the commits listed below.
This PR provided a base for future localizations of NetNewsWire.

Internationalizes NetNewsWire to the standards discussed in #78 (both keys, comments)

Each target has its own Localizable.strings (and .stringsdict)
There is no separation of certain strings, e.g., button titles, into separate tables

Includes base language (en) and English UK (en-GB) localizations.

Converts the widget to these standards and removes SwiftGen from its Build Phases
Includes a Localizations.md Technote on standards and procedures to update
The Sharing Service now uses ListFormatter instead of .joined(separator: ) to ensure the correct local separator is used.
Commit b7dbe5c fixes #3857

There are a few areas that remain exclusively English that require further design discussion:

The Keyboard Shortcuts help screen on the Mac (this is HTML and not picked up by Localizable.strings)
The default feeds on an initial install (these are English)

@stuartbreckenridge This has some conflicts now, but I’d like to get it merged. Would you update the PR with resolutions for those conflicts? Thanks!

@brentsimmons Conflicts resolved.


Ansible Galaxy - Error installing multiple roles from requirements.yml

Description
Running - $ ansible-galaxy install -r requirements.yml
from root of the project throwing
[WARNING]: - roles was NOT installed successfully: Content has no field named 'owner'
Due to missing roles I am facing "the role 'geerlingguy.mysql' was not found" error while executing
$ ansible-playbook playbook.yml
Steps to Reproduce
Steps to reproduce the behavior:

Go to 'https://github.com/apigee/devportal-kickstart-drupal-ansible-installer#apigee-devportal-kickstart-for-drupal-ansible-installation-scripts' and follow installation steps.
After cloning repo and installing Ansible.
Run  $ ansible-galaxy install -r requirements.yml

Actual Behavior
$ ansible-galaxy install -r requirements.yml

downloading role 'roles', owned by
[WARNING]: - roles was NOT installed successfully: Content has no field named 'owner'

Expected Behavior
$ ansible-galaxy install -r requirements.yml
-downloading role 'php', owned by geerlingguy

geerlingguy.php (###) was installed successfully

Screenshots

##Note
I am working on windows machine and using Cygwin64 Terminal for installing Apigee Kickstart developer portal for Drupal using Ansible.

A workaround to resolve the error when running 'ansible-galaxy install -r requirements.yml'  is to individually install all the roles that are mentioned in requirement.yml file.
eg :  ansible-galaxy install geerlingguy.php

I followed instructions and did not get any errors, are you running a old version of ansible? The collection subcomand only exists in ansible 2.9+.
% ansible-galaxy --version                                                                                (master)devportal-kickstart-drupal-ansible-installer
ansible-galaxy 2.10.3
 % ansible-galaxy install -r requirements.yml                                                                                (master)devportal-kickstart-drupal-ansible-installer
Starting galaxy role install process
- downloading role 'php', owned by geerlingguy
- downloading role from https://github.com/geerlingguy/ansible-role-php/archive/4.2.0.tar.gz
- extracting geerlingguy.php to /Users/chrisnovak/Downloads/devportal-kickstart-drupal-ansible-installer/roles/geerlingguy.php
- geerlingguy.php (4.2.0) was installed successfully
- downloading role 'nginx', owned by geerlingguy
- downloading role from https://github.com/geerlingguy/ansible-role-nginx/archive/2.7.0.tar.gz
- extracting geerlingguy.nginx to /Users/chrisnovak/Downloads/devportal-kickstart-drupal-ansible-installer/roles/geerlingguy.nginx
- geerlingguy.nginx (2.7.0) was installed successfully
- downloading role 'repo-remi', owned by geerlingguy
- downloading role from https://github.com/geerlingguy/ansible-role-repo-remi/archive/1.2.0.tar.gz
- extracting geerlingguy.repo-remi to /Users/chrisnovak/Downloads/devportal-kickstart-drupal-ansible-installer/roles/geerlingguy.repo-remi
- geerlingguy.repo-remi (1.2.0) was installed successfully
- downloading role 'repo-epel', owned by geerlingguy
- downloading role from https://github.com/geerlingguy/ansible-role-repo-epel/archive/1.3.0.tar.gz
- extracting geerlingguy.repo-epel to /Users/chrisnovak/Downloads/devportal-kickstart-drupal-ansible-installer/roles/geerlingguy.repo-epel
- geerlingguy.repo-epel (1.3.0) was installed successfully
- downloading role 'drupal', owned by geerlingguy
- downloading role from https://github.com/geerlingguy/ansible-role-drupal/archive/3.0.1.tar.gz
- extracting geerlingguy.drupal to /Users/chrisnovak/Downloads/devportal-kickstart-drupal-ansible-installer/roles/geerlingguy.drupal
- geerlingguy.drupal (3.0.1) was installed successfully
- downloading role 'mysql', owned by geerlingguy
- downloading role from https://github.com/geerlingguy/ansible-role-mysql/archive/3.1.0.tar.gz
- extracting geerlingguy.mysql to /Users/chrisnovak/Downloads/devportal-kickstart-drupal-ansible-installer/roles/geerlingguy.mysql
- geerlingguy.mysql (3.1.0) was installed successfully
- downloading role 'composer', owned by geerlingguy
- downloading role from https://github.com/geerlingguy/ansible-role-composer/archive/1.7.6.tar.gz
- extracting geerlingguy.composer to /Users/chrisnovak/Downloads/devportal-kickstart-drupal-ansible-installer/roles/geerlingguy.composer
- geerlingguy.composer (1.7.6) was installed successfully
- downloading role 'git', owned by geerlingguy
- downloading role from https://github.com/geerlingguy/ansible-role-git/archive/3.0.0.tar.gz
- extracting geerlingguy.git to /Users/chrisnovak/Downloads/devportal-kickstart-drupal-ansible-installer/roles/geerlingguy.git
- geerlingguy.git (3.0.0) was installed successfully

@shishir-intelli I will close this issue, please reopen if this did not solve your problem.


Load from wandb logger

Adds support to load checkpoints from wandb logger.
Testing:
Adding unit tests for this is a pain because you need to start saving and loading checkpoints from wandb, so you need to throw in wandb credentials and every time tests are run, we read and write from there. I spent a bit of time playing around with this, but I'm strongly in favor of not adding a unit test for this behavior. It's not ideal, but I think it's far better than constantly writing to wandb with each github action
To verify it works, I ran two runs which save and load from there. Run 1:
composer -n 1 examples/run_composer_trainer.py -f composer/yamls/models/resnet56_cifar10.yaml
        --train_dataset.cifar10.datadir /localdisk/CIFAR10/
        --val_dataset.cifar10.datadir /localdisk/CIFAR10/
        --loggers progress_bar wandb
        --save_folder experiments
        --save_overwrite True
        --loggers.wandb.log_artifacts True
        --loggers.wandb.entity mosaic-ml
        --loggers.wandb.name ckpt-test
        --loggers.wandb.project ckpt

which saves to wandb. Run 2:
composer -n 1 examples/run_composer_trainer.py -f composer/yamls/models/resnet56_cifar10.yaml
        --train_dataset.cifar10.datadir /localdisk/CIFAR10/
        --val_dataset.cifar10.datadir /localdisk/CIFAR10/
        --loggers progress_bar wandb
        --save_folder experiments
        --save_overwrite True
        --loggers.wandb.log_artifacts True
        --loggers.wandb.entity mosaic-ml
        --loggers.wandb.name ckpt-test
        --loggers.wandb.project ckpt

which loads from wandb and continues training. Run 2 uses a slightly different yaml which includes
load_logger_destination:
  wandb:
    entity: mosaic-ml
    name: ckpt-test
    project: ckpt

The resulting output is that it continues training from the checkpoint! Woo

Also, I'm not really sure what's going on, but when you checkpoint with wandb in the unit tests, the checkpoints are corrupted. My hunch is if you don't provide proper credentials to wandb and start using it, it doesn't nicely catch this scenario and give you an error, and instead continues onward stumbling along in a broken fashion.
For example, if you try to save a checkpoint after adding the Wandb logger in the unit tests, it becomes corrupted since wandb.run.project returns something weird which torch doesn't like when loading. I spent some time trying to figure out what it's returning, but I have no idea what's gong on...

Also, I'm not really sure what's going on, but when you checkpoint with wandb in the unit tests, the checkpoints are corrupted. My hunch is if you don't provide proper credentials to wandb and start using it, it doesn't nicely catch this scenario and give you an error, and instead continues onward stumbling along in a broken fashion.
For example, if you try to save a checkpoint after adding the Wandb logger in the unit tests, it becomes corrupted since wandb.run.project returns something weird which torch doesn't like when loading. I spent some time trying to figure out what it's returning, but I have no idea what's gong on...

In the unit tests, we set the variable WANDB_DISABLED, so it may be incompatible with that setting? So long as you validated this works manually, then we're all good! It would probably be worthwhile adding in a wandb key into jenkins so we can run integration tests, but that's an issue for another day :)


feat: change sqlite to any

Change sqlite to any so it can be reused in the atoma-proxy with postgres.

apparently the any driver is not supporting sqlite bool value