YAMLException: can not read a block mapping entry; a multiline key may not be an implicit key at line 5, column 1

I get some cryptic error, trying to inject some code into existing src/apps/auth/api/index.js file
HYGEN_TMPLS=.templates hygen new "api"

? API Method Name: test
? API Method URL: test/method
? App: auth
? method: GET
? Params (separated by ,):

Loaded templates: .templates
YAMLException: can not read a block mapping entry; a multiline key may not be an implicit key at line 5, column 1:

    ^

Available actions:
new: api, app, component

.templates/new/api/prompt.js
const _ = require('lodash');
const fs = require('fs');

function getDirectories(path) {
  return fs.readdirSync(path).filter(function (file) {
    return fs.statSync(path+'/'+file).isDirectory();
  });
}

module.exports = [
  {
    type:    'input',
    name:    'name',
    message: 'API Method Name:',
    validate(value) {
      if (!value.length) {
        return 'API method must have a name.';
      }
      return true;
    },
  },
  {
    type:    'input',
    name:    'url',
    message: 'API Method URL:',
    validate(value) {
      if (!value.length) {
        return 'API method must have an URL.';
      }
      return true;
    },
  },
  {
    type:    'checkbox',
    name:    'appname',
    message: 'App:',
    choices: getDirectories('src/apps'),
  },
  {
    type:    'checkbox',
    name:    'method',
    message: 'HTTP method:',
    choices: [
      {
        name:    'GET',
        value:   'get',
        default: true,
      },
      {
        name:    'POST',
        value:   'post',
        default: false,
      },
      {
        name:    'PUT',
        value:   'put',
        default: false,
      },
      {
        name:    'DELETE',
        value:   'delete',
        default: false,
      },
    ],
  },
  {
    type:    'input',
    name:    'params',
    message: 'Method Params (separated by ,):',
  },
];

.templates/new/api/api.ejs.t
---
to: "src/apps/<%= h.inflection.dasherize(appname).toLowerCase() %>/api/index.js
inject: true
skip_if: <%= name %>
before: "export default {"
---
<%
const appName = h.inflection.dasherize(appname).toLowerCase();
const apiName = h.inflection.dasherize(name).toLowerCase();
%>

/**
 * Documentation
 * @param {String} code - Explanation
 * @returns {Promise}
 */
function <%= apiName %>({ <%= params%>, headers = {}, }) {
  return axios.<%= method%>(`<%= apiURL%>`, { headers, });
}

this is src/apps/auth/api/index.js where I try to inject new method
/**
 * Backend Auth API Endpoints
 */

import { helpers as h, } from '@/lib';
import axios from '@/api/axios';

/**
 * Activate the user by invitation code
 * @param {String} code - Invitation code
 * @param {String} agent - Verified agent information
 * @returns {Promise}
 */
function activate({
  code = h.mandatory('invitation code'),
  agent = h.mandatory('agent'),
  headers = {},
}) {
  return axios.put(`verification/${code}`, agent, { headers, });
}

export default {
  login,
  logout,
  resetPassword,
  verification,
  activate,
};

Hi!
Sorry about these cryptic messages, when an error happens in YAML, then the error comes from the YAML parser.
In your case looks like you're missing a closing ":
---
to: "src/apps/<%= h.inflection.dasherize(appname).toLowerCase() %>/api/index.js
                                                                                `--- missing " here
inject: true
skip_if: <%= name %>
before: "export default {"

thanks Jondot, applied the fix, but now the error is different
> HYGEN_TMPLS=.templates hygen new "api"

? API Method Name: test method
? API Method URL: test/method
? App: auth
? HTTP method: GET
? Params (separated by ,): payload

Loaded templates: .templates
TypeError: ejs:1
 >> 1| src/apps/<%= h.inflection.dasherize(appname).toLowerCase() %>/api/index.js

str.replace is not a function

Available actions:
new: api, app, component

Sadly this error doesn't say much. If you can push the entire set of templates to some temporary repo I'll try to fix it for you.

Also, I just noticed appname is a checkbox input, instead of input input:
{
    type:    'checkbox',
    name:    'appname',

Maybe this is the problem?

@jondot
done - https://github.com/dmitry-saritasa/hygen-tmp
{
    type:    'checkbox',
    name:    'appname',
    message: 'App:',
    choices: utils.getDirectories('src/apps'),
  },

so choices is array returned by utils.getDirectories function and it seems it's working as this is correct output

and src folder has two sub-folders auth, projects listed.

I've found the problem: appname is actually an array. So you need to either just take the first selection like appname[0] or use a radio select (not checkboxes).
Good luck!

cleaning up. please reopen if still relevant

wow,you guys,I have the same problem,thanks for your conversation

wow,you guys,I have the same problem,thanks for your conversation

I had a similar issue
Just in case it could help anyone, I forgot to escape special characters in my frontmatter properties
I solved it by escaping
Before
---
to: myFile
inject: true
before: [hygen-marker] my marker
---

After
---
to: myFile
inject: true
before: \[hygen\-marker\] my marker
---


Add workaround to assure that the weights of the convolution op are on the host

Currently, the ttnn conv2d operation requires that the weights of the convolution must be on host. This logic is currently added to the lib/Dialect/TTNN/Transforms/TTNNLayout.cpp, but probably should be moved to the newly made workaround pass.

FYI: @sdjordjevicTT

Thanks for pointing this out, I will assign this one to myself and pick it up!


Should Motion blur be centered around frame time?

Currently the motionblur centers around the middle of the frame (i.e blur from 1->2 for frame 1)
Might it make more sense to center it around the frame time?  In this case would be from .5 -> 1.5 for a motion length of 1.  This would match what RenderMan for Maya does and most plates as well.  Also if you turn on motion blur it would match the static render a bit closer.

Sounds good.


Sprint 2 Review

Assignment
Points
Grade
Evidence

Pull Request (Integration)
2
1.65

Relevancy Checklist (Peer)
2
79*90/2=1.75
https://nighthawkcoders.github.io/portfolio_2025/csa/unit3-p1/unit3-7

Lesson (Group)
1
0.8
https://kaydenle.github.io/KaydenStudent2025/csa/unit3-p1/unit3-7

Homework, Popcorn Hacks
1 x 8
(unit2)+0.9+0.9+0.9+0.94+0.91+0.95+0.725+(from our teamteach grading))

Individual Contribution
1
0.8

Personal Notebooks / Blogs
1
0.9
https://kaydenle.github.io/KaydenStudent2025/2024/10/10/notes_IPYNB_2_.html

Total
12

Skill
Points
Grade
Evidence

Work Habits (Analytics)
1
0.88

Team Planning (Issue)
1
0.89
https://github.com/alishahussain/alishanew2025/issues/3#issue-2577352434

Presentation Memories
1
0.9

Grading and Feedback
1
0.89
Added the Rubric now https://docs.google.com/spreadsheets/d/1W9nuXvCk3F3iMjvrlYa6nhQNuXGSQoxPxa-UzZm2o-g/edit?usp=sharing

Beyond Perfunctory
1
0.85

Total
5
4.41

Assignment
Weightage
Grade
Comments (Dinesh Sahai/Tara Sehdave)

College Board Coverage
20
(18+18)/2=18
Covers all the lesson material/Goes over all necessary topics.

Java Examples
30
(27+29)/2=28
Lots of in-depth examples/Good Java examples

Popcorn Hack Usage
10
(8+7)/2=7.5
Hack could be more interactive/Hacks could be more interactive instead of asking an open ended question. Have people answer with code.

Homework
10
(9+9)/2=9
Very thorough and covers all topics well/Homework reflects lesson

Grading Plan
10
(9+8)/2=8.5
Standard grading plan w/ explanations/Standard grading plan

Original an Creative
10
(8+8)/2=8
Including a theme or game would be nice/Could include theme to make more interesting for students

Total
90
79

I like how concise your notes are. Good job for improving with your commits.

I like your sprint 2 notes from your team teach, It's very organized and good to refer back to.

Pull request was moderate.  Did a boolean table.  Team teach was comparing objects.  He took notes on team teaches.  Picking up on his commits.  Grading did a part.

Clean and concise organization of review page was nice to see and the Boolean table was also a nice addition to see as well.

I really liked the organization of your blog, and your beyond perfunctory showed your class ( boolean table)
Also good blog.

I like how you included our boolean table and how it connected back to our exact team teach. I also like how you included specific screenshots in your table. I liked how you also talked about improvement too.

I like how you didnt try to say your team teach was complicated and complex and simply said it was easy to understand.

Blog - References stuff about blogging
Memory - Talks about memroies grading
Relevancy - used Boolean tables to represent
Beyond Perfunctory - Something to lear from fixed his commits

Your message on how you went from weekly commits to daily commits really shows your initiative to learn!
Your sprint 2 notes were great and this was well done in being concise.
Your organization and boolean table was  great to hear of

Hi Kayden,
I liked your improvement in the frequency of your Github commits.

I really like your presentation it was very straight to the point and had a lot of useful information.

Great blog Kayden!!! I loved the blog, the features were awsome.


Fix two crash-on-invalid involving extensions

A perennial problem with the declaration checker is that we have it bail out early or skip some checks entirely on an invalid declaration, which means the invalid declaration does not satisfy all the invariants we expect. Later on, we type check something else that references the invalid declaration, and crash because of the broken invariant.
This PR fixes some issues with extensions:

If an extension's extended declaration was resolved (getExtendedNominal()) but the extended type has a problem (getExtendedType()), we did not build the extension's generic signature. This means that DeclContext::isGenericContext() could return true (because the extension had a generic parameter list) but DeclContext::getGenericSignatureOfContext() could return null (because we skipped building the signature). This was rdar://problem/46604393.

If an extension was (incorrectly) nested inside of another declaration, which can happen if a user accidentally omits a closing brace for instance, we skipped binding the extension altogether, because binding only walks top-level declarations when looking for extensions. So the extension did not get a generic parameter list. However we still would set the extension's extended type to a generic type, which again caused problems. This was https://bugs.swift.org/browse/SR-9017.

The total line change in this PR is a tad big, but to make it easier to review, look at the patches separately. All of them are small, except for the second-to-last one which moves code from Sema to AST without actually changing it.

@swift-ci Please smoke test

@swift-ci Please test source compatibility


no PDF is generated

Hi, here is the edited action in my repo https://github.com/Yingjie4Science/SDGdetector/blob/main/.github/workflows/draft-pdf.yml. There is no error, but no PDF is found in the repo, see https://github.com/Yingjie4Science/SDGdetector/tree/main/paper. Any advice? Thanks!

I have a similar issue with the following error message:
Run openjournals/openjournals-draft-action@master
  with:
    journal: joss
    paper-path: paper.md
/usr/bin/docker run --name openjournalsinaralatest_e1c88f --label 8d5581 --workdir /github/workspace --rm -e "INPUT_JOURNAL" -e "INPUT_PAPER-PATH" -e "GIT_SHA" -e "JOURNAL" -e "HOME" -e "GITHUB_JOB" -e "GITHUB_REF" -e "GITHUB_SHA" -e "GITHUB_REPOSITORY" -e "GITHUB_REPOSITORY_OWNER" -e "GITHUB_RUN_ID" -e "GITHUB_RUN_NUMBER" -e "GITHUB_RETENTION_DAYS" -e "GITHUB_RUN_ATTEMPT" -e "GITHUB_ACTOR" -e "GITHUB_TRIGGERING_ACTOR" -e "GITHUB_WORKFLOW" -e "GITHUB_HEAD_REF" -e "GITHUB_BASE_REF" -e "GITHUB_EVENT_NAME" -e "GITHUB_SERVER_URL" -e "GITHUB_API_URL" -e "GITHUB_GRAPHQL_URL" -e "GITHUB_REF_NAME" -e "GITHUB_REF_PROTECTED" -e "GITHUB_REF_TYPE" -e "GITHUB_WORKSPACE" -e "GITHUB_ACTION" -e "GITHUB_EVENT_PATH" -e "GITHUB_ACTION_REPOSITORY" -e "GITHUB_ACTION_REF" -e "GITHUB_PATH" -e "GITHUB_ENV" -e "GITHUB_STEP_SUMMARY" -e "GITHUB_STATE" -e "GITHUB_OUTPUT" -e "RUNNER_OS" -e "RUNNER_ARCH" -e "RUNNER_NAME" -e "RUNNER_TOOL_CACHE" -e "RUNNER_TEMP" -e "RUNNER_WORKSPACE" -e "ACTIONS_RUNTIME_URL" -e "ACTIONS_RUNTIME_TOKEN" -e "ACTIONS_CACHE_URL" -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/edecob/edecob":"/github/workspace" openjournals/inara:latest  "-o pdf" "paper.md"
YAML parse exception at line 19, column 1,
while parsing a block mapping:
did not find expected key

I was also confused by this as it's not very clear from the documentation. The pdf isn't supposed to appear in the repository, it will appear as an 'artifact' of the github action.
For your repository, click the actions tab, then click the latest successful action. Then scroll down to the bottom of the screen and you should see a section marked artifacts that contains something with the name paper. Click on that and it'll download a zipped copy of the pdf.
For your latest successful build you can see the paper at the bottom of this link, for example: https://github.com/Yingjie4Science/SDGdetector/actions/runs/3316984180
(note: On my machine the download is failing for some reason, both for your repo and my own, but I think this is a temporary issue as it worked yesterday)

Thank you so much! Now I see the generated paper.pdf, and I can download it on my side, which is great! Thanks again!

Thanks @simonrp84 I was confused by this also. I wonder why it cannot save to the folder directly? Would it not be simpler? I wonder if there is a way to automatize this action in the GHA so that we also always have access to the latest PDF in the folder without having to go to the workflow run, download the zip locally, unzip the file, and then reupload the PDF to github... If someone knows a solution I would be happy to hear it.

Ok got it! Just add the following as the last step of the GHA:
- name: save pdf to repo
  uses: stefanzweifel/git-auto-commit-action@v4
  with:
    commit_message: Saved new PDF of paper

Hi I am receiving an error generating a pdf file and I am not sure what the reason is. Would it be possible to consult someone to check if my markdown syntax and my yml files are correct?

@rempsyc I run into the same problem. I didn't want to commit the paper.pdf from my local built so I added it to the .gitignore. Do you know whether your addition to the github action is in conflict with that? I'm also just looking for an automated link where I can read the current state of the paper.md as pdf

I think I had not added paper.pdf to .gitignore, so when the action ran, it just overwrote my local paper.pdf. Perhaps it conflicts. You should try removing it from .gitignore to see if it solves the issue (just in case). Note that this approach ended up a bit cumbersome as I had to pull changes (to receive the new PDF) every time I made a new commit and sometimes forgot so had to merge, etc.

Okay will give it a try, it would be nice if the pdf could just be deployed similar to the documentation, with the actual pdf outside of the pull request

For the record: Yes, having the paper.pdf in the .gitignore will effectively disable the github action commiting the paper.pdf that same pull request

In case of further elaboration on this https://github.com/openjournals/openjournals-draft-action/issues/15#issuecomment-1290478150


Crash on blu spell

Sorry not a lot of context here, just downloaded your latest alpha, this SS is right after i zoned into SSR II for an alex farm. After talking to the pole etc, casted battery charge and blamo! All my settings etc are default with your addon. VERY excited about your project here, I'll try to duplicate and provide more details if needed.

Just saw your hotfix, and now I cannot duplicate, huzzah!

Glad it was related with that!
Let me know if you run into any other issue 👍


Tool Durability Inconsistency between Client and Server

Server thinks the tool is broken one block earlier than it actually is, causing the last block you mine to vanish into thin air if it's not something that can be broken by your hands. Stats are known to be the same between the two, and original issue was resolved on the client side, so this can probably be resolved by either fixing the inconsistency between the two (the ideal solution), or boosting the durability of the tools on the server side by 1 (the bodgy solution).

No, this is the same issue where the block drop is calculated after you destroy it.

GitHub, I didn't click "close with comment", why did you do that.

Maybe GitHub made the right call because I just found a fix.


Samples, documentation images updates.

ConversationLearner Samples - Documentation- images updates for the memoryManager  changes.

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


expansion of cutpoint vignette (activity counts cutpoints)

Fixes #771 => Some text to explain how to apply cutpoints based on activity counts in GGIR in the cut-point vignette.

Checklist before merging:

[ ] Existing tests still work (check by running the test suite, e.g. from RStudio).
[ ] Added tests (if you added functionality) or fixed existing test (if you fixed a bug).
[ ] Updated or expanded the documentation.
[ ] Updated release notes in inst/NEWS.Rd with a user-readable summary. Please, include references to relevant issues or PR discussions.
[ ] Added your name to the contributors lists in the DESCRIPTION and CITATION.cff files.

Codecov Report

Merging #781 (c2e535a) into master (543d2bf) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head c2e535a differs from pull request most recent head 749d6bf. Consider uploading reports for the commit 749d6bf to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master     #781   +/-   ##
=======================================
  Coverage   74.14%   74.14%
=======================================
  Files          88       88
  Lines       10870    10870
=======================================
  Hits         8060     8060
  Misses       2810     2810


Incomplete comment for AnyAction in index.d.ts

Do you want to request a feature or report a bug?
Bug -- Sort of... it's an incomplete comment in index.d.ts
What is the current behavior?
The comment above AnyAction is incomplete. See here:
/**
 * An Action type which accepts any other properties.
 * This is mainly for the use of the `Reducer` type.
 * This is not part of `Action` itself to prevent users who are extending `Action.
 */
export interface AnyAction extends Action {
  // Allows any extra properties to be defined in an action.
  [extraProps: string]: any
}

This sentence never finishes, failing to explain the reasoning for the AnyAction type to exist.
This is not part of `Action` itself to prevent users who are extending `Action. ...???
What is the expected behavior?
This comment should finish its sentence and be more clear.

I can submit a PR once told what this is supposed to say, but I'm not quite sure just by looking at it... It's obvious the rest of the sentence was not included, but not obvious what it was going to say.
However, honestly, I'm not quite sure having AnyAction makes much sense here. It appears to only be adding an index signature to the base Action type, but I think we actually want that index signature on Action too, as we should be able to do things like someAction['type'].... and, if that's the case, what does AnyAction even do for us here? TypeScript does not enforce exactly matching properties in object types. It only requires us to meet the requirements of the interface/type, ignoring any extra properties. So, we can still include extra properties with the Action type.
This leaves me wondering... do we even need this AnyAction type at all?

Yes, we need AnyAction. Here's the PR that added it and here's the specific commit that added it. In fact, there's even a comment specifically about AnyAction, which should be particularly helpful.
As for the comment in question, I'm not actually sure where that was going. @aikoven, any chance you remember?

I think that it has to be something like
This is not part of `Action` itself to prevent types that extend `Action` from having an index signature.

AnyAction is indeed not the same as Action.

The actual PR that added AnyAction: https://github.com/reduxjs/redux/pull/2467

You could easily add the same index signature to Action though. In other words, Action and AnyAction could easily just be one type. I don't really see the benefit of not doing this.
interface Action<T = any> {
  [key: string]: any
  type: T
}

I actually don't really understand the reasoning behind defaulting T to any there either. It seems to have been done for convenience so that users don't have to supply a generic, but that seems somewhat non-idiomatic to me. It's not actually useful, because there's not really ever a case where we want type to be seen as any... particularly because we want to be able to narrow down types in reducers.
Right now, the tests in tests/typescript only use it in this way. They don't supply a generic. But, in real life usage, you'd most likely always want to provide the generic... You'd usually either make your type that extends Action also take the same generic and pass it through or just hard-code/partially apply the generic to Action.
interface MyType<T> extends Action<T> {
  someOtherProp: number
}

or
interface MyType extends Action<string> {
  someOtherProp: number
}

I think the types & the tests for the types could be other ways too, but that's probably a separate conversation.
This is not part of Actionitself to prevent types that extendAction from having an index signature.
Why do we want to do this? You could be able to to access the type property via index even on an action that only contains type & no other properties. Ex:
const someActionType = action['type']

Yes, you don't need an index signature to do that, but I don't see a reason why it shouldn't be supported. Type is a property on the object just like any other. I would expect all actions to have an index signature.

You could also go further:
interface Action<T, Values = any> {
  [key: string]: T | Values
  type: T
}

if you wanted to allow the user to better lock down the possible value types in the action. This seems like a better use case for defaulting a generic to any to me.

I don't really see the benefit of not doing this.
interface Action<T = any> {
 [key: string]: any
 type: T
}

This would kill type safety. Imagine you had an action like:
interface MyAction extends Action {
  payload: number;
}

With index signature, TS won't catch errors like this:
const action: MyAction = ...
action.payloa; // mistake, not caught by the compiler

It's not actually useful, because there's not really ever a case where we want type to be seen as any... particularly because we want to be able to narrow down types in reducers.

Redux may be used in lots of different ways, and it's not true that you'd always want to narrow the type of type. There are even cases when possible values for type are unknown in advance. You could also use numbers for types, and that's fine. That's why any.

You could also go further:
interface Action<T, Values = any> {
 [key: string]: T | Values
 type: T
}

This way we'd make actions look like dictionaries, which they are generally not.

...
This would kill type safety. Imagine you had an action like:
interface MyAction extends Action {
payload: number;
}
With index signature, TS won't catch errors like this:
const action: MyAction = ...
action.payloa; // mistake, not caught by the compiler

Ah, sorry. You're right. I just tried this. I didn't realize TS would allow any extra misc properties here. I was thinking it was solely about index access for annotated properties & the types of the values assigned to them for some reason. I guess the index signature is more about using Records to dynamically perform lookups based on IDs,  like string UUIDs.

Redux may be used in lots of different ways, and it's not true that you'd always > want to narrow the type of type.

I'm not sure I follow you here. Why would you ever not want the capability to narrow down action types in reducers (based on the type property)? In a statically typed environment, you want to know which action you're working with so that your update logic is safe. You may not always need to narrow down the action types, depending on your update logic (like if you want to do the same thing in response to a variety of different actions), but I'm having trouble thinking of how always having that capability would be a bad thing.

There are even cases when possible values for type are unknown in advance.

What do you mean? The value may be unknown (if dynamically generated), but you would, at least, know the type of that value.
Side-note -- It is possible to dynamically generate unique types with some type system trickery to simulate nominal typing. See here if interested: https://gist.github.com/joshburgess/729a2aeb7a9d6c09e3ac5a95cd2071be

You could also use numbers for types, and that's fine. That's why any. A generic > is needed when you want your actions to be more strict.

Why would any be needed? The generic could be supplied as string | number, like:
interface MyAction extends Action<string | number> {}

Or, if you wanted specific literals (more useful, because it allows you to narrow down the action types in reducers), something like:
interface MyAction<T extends string | number> extends Action<T> {}

const actionA: MyAction<'TEST_ACTION'> = { type: 'TEST_ACTION' } // ok
const actionB: MyAction<'TEST_ACTION'> = { type: 'OTHER_ACTION' } // type error
const actionC: MyAction<'0'> = { type: '0' } // ok
const actionD: MyAction<'0'> = { type: '2' } // type error

These things could even be inferred via action creators generically if the action creator functions themselves used generics and passed them along to the return type.
I think it would be extremely rare to want type to be any... I'm not sure I can think of a good use case for it. Wouldn't we want the user to supply the rare case rather than default to it?

You could also go further:
interface Action<T, Values = any> {
[key: string]: T | Values
type: T
}
This way we'd make actions look like dictionaries, which they are generally not.

Values can be a discriminated union here. For example, it could be your Payload type or Meta type.
type Payload = { users: Users[], count: number }
type Meta = { lastModified: Date }

interface Action<T, Values = any> {
 [key: string]: T | Values
 type: T
}

interface MyAction extends Action<'MY_ACTION', Payload | Meta> {
  payload: Payload
  meta: Meta
}

However, you were right with the first point about this index signature allowing extra properties, and that definitely is a problem & a good reason not to include that index signature on the base Action type.
Sorry if this is too off topic vs the original post. Maybe, I should make a PR to fix that AnyAction comment and discuss potential updates to the typings in a separate issue or PR?

Here is that PR for the AnyAction comment: https://github.com/reduxjs/redux/pull/3316


Add "mapping" category

The current verbs are based around data transformations, but we also need to support data mappings. These do not necessarily lead to a change in the underlying source or target data, but express a relationship between them.
The verb name can be mapping (or maybe something else?). It is up to clients to determine what they do with this information.

The current verbs are based around data transformations, but we also need to support data mappings. These do not necessarily lead to a change in the underlying source or target data, but express a relationship between them.

This is nice!
Have you though about how you will align this verb with the others in terms of return value when multiple verbs are present in the same data migration file? I say this because the current verbs are called because of their side effect to transform the underlying data. However for mapping you need metadata about the match as a return value to be able to do something afterwards.
As an example while working on https://github.com/fjuniorr/flowmapper/pull/70 initially I was using migrate_datasets to transform the flows. However I needed some way to extract the conversion_factor from a change such as:
  {
    "source": {
      "name": "Ammonia, as N"
    },
    "target": {
      "name": "Ammonia"
    },
    "conversion_factor": 1.2142857142857142
  }

for additional processing.
Another different but related question is what would the target node would represent in this new verb (or even if it will be present at all).

The verb name can be mapping (or maybe something else?).

How about match?

@fjuniorr I am tempted to mark this as "won't fix", as I am worried about scope creep. Here is our current state:

I came full circle on your idea to use JSONPath - I have updated randonneur to include a mapping section which shows where the attributes in the original files are located. See the README for examples (and randonneur_data).
flowmapper is only for biosphere flows, and there I do think there is a fundamental underlying ontology for all biosphere flows. See my fork for details. But I should add XPath/similar to the fork to show how the values are imported - ideally this would work automatically, but not all data formats support structured expression languages.

Sorry @cmutel I missed the original notification!
I don't have the full context currently loaded in my brain but I'm sure you are making the right call here.

No worries, and thanks again! Your ideas were super helpful, even if I did resist some of them at first :)
In the end we allowed each file to define its own data schema, and user XPath or JSONPath as appropriate, depending on the input data source, see https://github.com/brightway-lca/randonneur_data/blob/main/randonneur_data/data/simapro-9-ecoinvent-3.8-biosphere-manual-matches.json#L23.


feat: add Gumloop flow trigger pipe

name: pull request
about: submit changes to the project
title: "[pr] Add Gumloop flow trigger pipe"
labels: ''
assignees: ''

description
This pull request introduces a new pipe to trigger Gumloop flows. you can configure the pipeline to execute Gumloop flows based on configured intervals, daily, or hourly schedules. The pipe send the data captured by Screenpipe to the Gumloop API for processing.
you can create a gumloop flow on your own for any type of flow making or Gumloop pipeline
related issue: #365
/claim #365
type of change

[ ] bug fix
[x] new feature
[ ] breaking change
[ ] documentation update

how to test

Ensure that the pipe.ts script is running with the new Gumloop flow trigger tasks.
Verify that the scheduler tasks trigger correctly based on the configured intervals (e.g., every minute, hourly, or daily).
Confirm that the flow is successfully triggered by checking the logs and verifying the data sent to the Gumloop API.
Test the behavior with various configurations in pipe.json (such as different frequencies and flow times).

if relevant add screenshots or screen captures to prove that this PR works to save us time.
checklist

[ ] MOST IMPORTANT: this PR will require less than 30 min to review, merge, and release to production and not crash in the hand of thousands of users
[ ] i have read the CONTRIBUTING.md file
[ ] i have updated the documentation if necessary
[ ] my changes generate no new warnings
[ ] i have added tests that prove my fix is effective or that my feature works

additional notes

@divanshu-go nice!!
so what is this doing? how did you use it?
was wondering if we should make it usable in two ways:
either send all raw data to gumloop (mic & screen)
pros:

do whatever from gumloop with the data
cons:
data privacy risks
more pricy?

or have a local AI that capture specific details of your day (logs) and then send this to gumloop
pros:

more data privacy
cons:
potential loss of data, less quality, need to be more specific about the use case
more resource usage

https://github.com/mediar-ai/screenpipe/issues/365#issuecomment-2459990707


Compression type XXX is not linked with the binary

in Mac OS and ubuntu 18.04, I have install rocksdb(v5.18.3) as README include snappy/lz4, and python-rocksdb-0.7.0, Mac OS works fine, but in ubuntu 18.04, error happen, code as follow:
Python 3.6.7 (default, Oct 22 2018, 11:32:17)
[GCC 8.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import rocksdb
>>> options = rocksdb.Options()
>>> options.create_if_missing = True
>>> options.compression = rocksdb.CompressionType.snappy_compression
>>> db = rocksdb.DB("test.db", options)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "rocksdb/_rocksdb.pyx", line 1636, in rocksdb._rocksdb.DB.__cinit__
  File "rocksdb/_rocksdb.pyx", line 82, in rocksdb._rocksdb.check_status
rocksdb.errors.InvalidArgument: b'Invalid argument: Compression type Snappy is not linked with the binary.'
>>>

Did you install the snappy library?

yes, I did

@Lbqds  have you solved?
I have install
apt-get install -y librocksdb-dev libsnappy-dev zlib1g-dev libbz2-dev libgflags-dev liblz4-dev
I use it in docker ubuntu


Standalone Web Scanner Update

ITS-Pro time category: https://sourcecode.ucsd.edu/projects/WTS/repos/web-barcode-scanner/
This is complete when:

[ ] The standalone web scanner is updated to display the duplicate scan message

https://sourcecode.ucsd.edu/projects/WTS/repos/web-barcode-scanner/

Done.

@a6wu Build and deploy to CMS QA.


the cpu limits in exec pod

I run the pi test spark-py-pi.yaml
kubectl get pods -n system
NAME                READY   STATUS    RESTARTS   AGE
pyspark-pi-driver   1/1     Running   0          2m57s

and get the error from the driver as the Resource Quotas have been setted in namespace system
20/02/13 12:15:10 ERROR Utils: Uncaught exception in thread kubernetes-executor-snapshots-subscribers-1
io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: POST at: https://kubernetes.default.svc/api/v1/namespaces/system/pods. Message: Forbidden!Configured service account doesn't have access. Service account may have been revoked. pods "pyspark-pi-1581596092621-exec-1" is forbidden: failed quota: system-resourcequota: must specify limits.cpu.

I have check the pyspark-pi-driver  and pyspark-pi-1581596745205-exec-1 pod spec where I deploy the pi test spark-py-pi.yaml in default namespace
pyspark-pi-driver
    resources:
      limits:
        cpu: 1200m
        memory: 896Mi
      requests:
        cpu: "1"
        memory: 896Mi

pyspark-pi-1581596745205-exec-1
    resources:
      limits:
        memory: 896Mi
      requests:
        cpu: "1"
        memory: 896Mi

so. the exec pod don't set cpu limits? how to solve this when the Resource Quotas have been setted in namespace

soveld by  change the yaml to
  driver:
    cores: 1
    coreLimit: "1200m"
    memory: "512m"
    labels:
      version: 2.4.4
    serviceAccount: spark
  executor:
    cores: 1
    coreLimit: "1200m"
    instances: 1
    memory: "512m"
    labels:
      version: 2.4.4


Scheduler doesn't apply to different days settings

Hi,
I have a problem and I'm not sure if it is a bug or it is a bad settings... ;)
I want to activate motion during specified times, and video capture in case of detection. I want this during the night (from 06:30PM to 08:00AM the next morning) and during the whole weekend
I did this :

First line to 'disable' the motion and capture on Monday-Friday days
Second line to enable it Sunday & Saturday in Days
Third line to enable everytime after 06:30PM
Fourth line because it looks like after 00:00, the green case (top left in the picture above) is lost. So I "forced it"...
With this it looks like it doesn't work on weekends, it continues just like a week day !
Any idea ? :)

Time periods must be in increasing order for them to work.
Also they should be unique, so the second 8:30 will be ignored. Make it 8:31

Thank you so much !
I did this, can't wait to see this weekend !!


feat: Implement std::error::Error for window_vibrancy::Error

Error trait is a basic trait to indicate it is an error type. This is useful for crates which provide generic error handling like anyhow since it can be converted into anyhow::Error.
use window_vibrancy::{apply_vibrancy, NSVisualEffectMaterial};

fn main() -> anyhow::Result<()> {
    let window = ...;
    apply_vibrancy(window, NSVisualEffectMaterial::HudWindow, None, None)?;
    Ok(())
}

Currently an error like below happens on coercing the error type at the ?:
error[E0277]: the trait bound `window_vibrancy::Error: StdError` is not satisfied
   --> src/foo.rs:10:88
    |
156 |         apply_vibrancy(webview.window(), NSVisualEffectMaterial::HudWindow, None, None)?;
    |                                                                                ^ the trait `StdError` is not implemented for `window_vibrancy::Error`
    |
    = help: the following other types implement trait `FromResidual<R>`:
              <Result<T, F> as FromResidual<Yeet<E>>>
              <Result<T, F> as FromResidual<Result<std::convert::Infallible, E>>>
    = note: required for `anyhow::Error` to implement `std::convert::From<window_vibrancy::Error>`
    = note: required for `Result<WebView, anyhow::Error>` to implement `FromResidual<Result<std::convert::Infallible, window_vibrancy::Error>>`

For more information about this error, try `rustc --explain E0277`.

In the future, would be helpful if you sign your commits.

Sure. I did the setup for commit signing but I didn't enable it by default. I'll do git config commit.gpgsign true on my local repository so that all commits will be signed by default.


Is there an Update to handle dicts as input?

Have you developped an Update which uses python-docx-replace(doc, dict) ?
Would be awesome

I'm not sure if I got your point, can you give an example? If you want to pass a dict, you can use something like this:
my_dict = {"name":"Ivan", "phone": "123"}
python-docx-replace(doc, **my_dict)

Sorry, I wasn't aware of the ** convention. Thank you


api.auth.local.validate.url returns any http code between 400 - 499 , user should be notified of the exact error code.

Rancher Version: v1.2.0-pre4-rc4
Steps to reproduce the problem:
Set api.auth.local.validate.url to the password validator url.
Enable local auth.
Change password for existing account  and simulate password validator to send any error code between 400-499.
Password change fails and user is presented with the exact error message  that password validator sent. But user is not presented with the http error code that was received.
Request URL:http://<ip>:8080/v2-beta/passwords/1c46/?action=changesecret
Request Method:POST
Status Code:404 Not Found
Remote Address:<ip>:8080

No?  HTTP status codes are meaningless to users.

@mrajashree mentioned that for error code between 400-499 , the exact error codes from the authenticator are preserved.
@vincent99 If these are meaningless to users , why are we preserving these error codes we get from authenticator

We're not preserving it anywhere;  The entire 4xx range means that there was an error that's the client's fault (the user's password is bad in this case).  5xx means it's the server's fault (like we couldn't reach the validation service).
So the specification for someone to implement a validator service is just "return a 4xx status if the password is unacceptable", and the reason is human-readable text in the response.  There's no reason to require an exact status code, or to show the user the one that they happened to have returned.


fix: [NPM] V2 NPM with linux DP bug fix after first run

Reason for Change:

After running conformance test, fix finding issues.

Observed multiple restarting NPM. Checked log and there is nil pointer errors in NPMSimpleError.

Issue Fixed:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests

Notes:
TODO list.

For the first error, we need to UT to catch this nil error.
I will resolve lint warning in next PR.

close this PR since we made decision to deal with this in https://github.com/Azure/azure-container-networking/pull/1120


Allow using a react component as an icon for Widgets when writing UiItemsProvider

AbstractWidgetProps icon should probably allow for an HTML element as well. Right now you're stuck using the webfont icons or suing the Icon utilities to create an IconSpec from the svg?sprite or web components.
For Toolbar items we've been creating CustomItemDefs, which allow for a ReactNode as the iconSpec.
Clearly we can't use React.ReactNode in appui-abstract but should allow for some type of HTML element, or provide a helper like we do for CustomItemDefs
This request is being driven by DR
Here is how we currently use iconSpecs for toolbar items: https://github.com/iTwin/viewer-components-react/blob/master/packages/itwin/geo-tools/src/GeoToolsItemDef.tsx#L24

Right now, you can use the IconHelper.getIconData in core-react to "hide" React icons on a definition that can be used by AbstractWidgetProps. In 4.0 we're planning to make it easier.

It looks like we moved all the icons in our test apps to using SVGs. I've made a change to appui-test-providers to specify the floating ViewAttributes widget icon using an icon from itwinui-react-icons. Until it's pushed, here's the important code:
            const internalData = new Map<string, any>();       const widgetIconSpec = IconHelper.getIconData(<SvgWindowSettings />, internalData);       widgets.push({         id: "appui-test-providers:ViewAttributesWidget",         label: "View Attributes",         icon: widgetIconSpec,         internalData,         defaultState: WidgetState.Floating,         floatingContainerId: "appui-test-providers:ViewAttributesWidget",         isFloatingStateSupported: true,         getWidgetContent: () => { // eslint-disable-line react/display-name           return <ViewAttributesWidgetComponent />;         },         canPopout: true,       });

thansk @NancyMcCallB


Render time in 1 world

V11 DnD5e
Disabled all modules bar Tokenizer and issue persists.
In World 1, Tokenizer takes 40+s to load its window with all modules enabled or disabled.
In World 2 with all modules enabled or disabled Tokeniser loads in 2 seconds.
Console only states render beggining and suggests willReadFrequently attribute be set to true and that is all.

Can you try resetting your tokenzer settings in the bad world:
Use the dev console (F12) or create a script macro:
game.settings.settings.forEach((s) => {
  if (s.namespace !== "vtta-tokenizer") return;
  game.settings.set(s.namespace, s.key, s.default);
});

This has worked, tyvm
I had tried turning off and uninstalling tokenizer
then reinstalling
re-enabling but the issue persisted
This Macro has indeed fixed the issue, which i think means likely something on my machine certainly broke it.


Faster enum impl

If a value is failed to be found, will just return the inputed value. This way, if Discord adds new fields, it wont break.

Co-authored-by: @nekokatt
Checklist

[x] I have run nox and all the pipelines have passed.
[x] I have made unittests according to the code I have added/modified/deleted.

Related issues
Closes #212
Closes #182
Closes #151

Do we have some performance metrics on how faster this is?

Do we have some performance metrics on how faster this is compared to the stdlib enum class?

Edited the top comment to include them

lgtm, just gonna have to fix that merge conflict

This kinda links in to what I said in #201 or one of the related issues about whether we are misusing enums for sone stuff.
Lgtm though.

Just fix da merge conflicts and we can see.
I'd be interested to see how this affects the startup overhead with asyncio_debug=1

@nekokatt rebased :+1:


With 1.0.0 "broccoli-babel-transpiler is opting out of caching due to a plugin that does not provide a caching strategy: () => __dirname."

From 0.4.4 to 1.0.0 and I receive this:
$ ember s
Livereload server on http://localhost:49153
broccoli-babel-transpiler is opting out of caching due to a plugin that does not provide a caching strategy: `() => __dirname`.

two times.
I'm using:
"ember-cli-babel": "^6.6.0",
"ember-cli-babili": "~0.1.4",

This also requires a bump of broccoli-babel-transpiler, can you do a yarn upgrade broccoli-babel-transpiler to get the latest and let us know if that resolves the issue?

Solved the issue for me, thanks!

Awesome, thank you for confirming!
We need to update our peerDependency minimum for this I guess....

Fixed thanks.


package: only send relevant files to npm

This will make npm ignore the test/ folder when uploading the package. Saves space and bandwidth for all users :+1:
This is especially important on embedded products like the Tessel where every byte counts.

I prefer to ship my module with the documentation and tests included. I often work offline and having documentation available in node_modules is very helpful. Saving a few KB is a bad tradeoff.
(And I say this as a person with only a 256GB disk in my laptop.)
Related discussions:
https://github.com/nodejs/readable-stream/pull/100
https://github.com/substack/node-browserify/issues/1093
https://github.com/crypto-browserify/sha.js/issues/5

256GB? Luxury! I have 32MB on my Tessel that I have to contain myself to.
I'm not sure that I personally see the need but if you say you use it, I believe you. Maybe npm should have a requiredFiles field which can be used to remove file after the package has been saved...

I think you'll have a hard time getting everyone to remove these files from their modules, especially if you have a deep dependency tree. That's a lot of PRs!
Maybe you can running a command like rm -rf node_modules/**/test node_modules/**/*.md after install? You can probably craft a command that catches most of the worst file size offenders.

Yeah probably :)
I wish there was some sort of standard for how to publish packages to npm. Kind of like your excellent standard package. Either everyone should use the files or none should.
I've been pondering the idea of actually minifying my code before putting it on npm as well. I think that I see it more like "binary distribution" instead of source distribution, especially since it (should) have link back to the repository with the source.
It would be super cool if npm contained something like WebAssembly packages! But I think that we're a long way from that. Well, a man can dream :night_with_stars: :stars:
Thank you for taking time with this issue :+1:


Made sixel use -I

Made the sixel image backend use high color mode (-I) as it gives better image quality

Thank you for your contribution!
This PR is merged into hyfetch since this repo (dylanaraps/neofetch) seems no longer maintained.
HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.
Read the "Running Updated Original Neofetch" section for more info!


logaggregator/buffer: Skip duplicate log lines

Duplicate log lines can be sent when flynn-host is reconnecting.
Closes #2056

LGTM


TimerHandle callback fixes/improvements

This accomplishes a few related things:

Implements the user-callback for the rest of the supported, generic TIM instances (TIM2, TIM3, TIM4)
Fixes bug where DAC DMA/Callback usage would crash due to null Impl::Config object in internal hardware init
Adds example of using the new TimerHandle::PeriodElapsedCallback

Tested:

Daisy Patch SM Envelope example (uses TIM6 based DAC callback)
new TIM5 based callback example
Tested modified example using TIM4 peripheral
Tested modified example using TIM3 peripheral
Tested modified example using TIM2 peripheral*

Re. TIM2
Opened new issue (#524) to describe weirdness with attempting DeInit prior to Init.
For clarity, when using TIM2 (which is internally used by the System class), I had to Init the TIM2 peripheral, then DeInit before initializing again to have a clean state to start a callback from.
This particular use is outside the scope of this PR.


Nicer install.sh

-> Makes the Message a bit nicer
-> Fixes a typo (!!1!)

That is not a typo
Please read the contributing guidelines

Typo fixes
That one in install.sh is not a typo. Please don't open pull-requests for that. More info here http://knowyourmeme.com/memes/the-1-phenomenon


Possibly unsafe check in test

In the test https://github.com/libuv/libuv/blob/v1.x/test/test-udp-send-hang-loop.c there is the assignment:
req->handle = NULL;

followed later by:
ASSERT_NULL(send_req.handle);

I wanted to verify if this check is actually safe since I believe uv_udp_send_t::handle is a private field and shouldn't be manually set by the user.

Looks like it's documented as public - https://docs.libuv.org/en/v1.x/udp.html#public-members

Thanks. I had extended the /* private */ comment in UV_REQ_FIELDS into the uv_udp_send_t struct. So it looked private after resolving the macros.


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #77.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #77

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


Pre-train DAMSM

How to decide which epoch has good performance to choose when training DAMSM + CL for 600 epochs ?

There are word loss, sentence loss and valid loss when training DAMSM + CL ?


LiDAR mount + cone deflector

First rev done!

Drawing needs to be checked.
Bars need to be ordered.

5005 Ally sheet
Order in square stock in plain ally, 20mm
Talk to Austin for sheets
Final review

Requested quote from:
https://www.shapealuminium.net.au/aluminium-extrusion-suppliers-australia/shape-aluminium-square-box-tube-hollow/19x1.6-radius-aluminium-square-tube


image_picker fixes: file suffix, permissions redirect, video resize

This PR is for three bugs relating to image picker.  I found the latter two when fixing the first.  I'm happy to split this up if that's easier.

(android) image_picker saves both captured images and captured video with a .png extension.  I'm not 100% sure that .jpg and .mp4 are correct -- I couldn't find an authoritative source -- but that's what most information seemed to point towards.
(android) if the initial request from an app is for either video capture or the video gallery, after permission is granted, image_picker redirects to image capture or the image gallery.
(android) image_picker attempts to resize video if any of the resize parameters (maxWidth, maxHeight) are provided

Hi @wreppun, thanks for contributing; have you had a chance to consider the review feedback?

Hey, yes, thanks for the feedback.  I don't mind breaking those up -- I should be able to look at this and turn something around on Tuesday.

Published


Fix transforms for static objects

Static rigidbodies don't update their transforms.
I made a commit that resolves by setting the initial transforms state (body.pos, body.quat) used in .getPosition() & .getRotation() to be equal to body.position and body.orientation.

And i added Plane and Particle to exports. Anyway they are used in the code so why not to export them. Personally i would like to see them in Oimo.js)

yes thank for transform fiw but i made many change since you fix
i include in last revision
for plane and particle is not yet implemented
do collision for each shape is hard

@lo-th okay, can you point me to those changes so I can update my fork? Also I would like to contribute to this project (I think about possibility of adding new shapes and improving performance)

cool ok, i think update soon
i fix sphere cylinder / on world i fix remove i add intenal update for main demo

mm is not good pos need to be change at setParent
for correct world scale

@lo-th You know, i am making a physics module based on oimojs for my framework. I want to improve current integration situation in web's 3D (make it simpler). So i will contribute to Oimo.js and would like to improve it.
If you'd like to contribute to my project - let's discuss it)
I have some questions about Oimo.js can we discuss them in real-time chat? Facebook messanger / slack / discord ?

ok is done

oimo is far from possibility of ammo for the moment
maybe with next update from saharan

@lo-th That's why I develop a physics module to wrap it. I already have a one for ammo and I want to also have a  simple lightweight library like Oimo.js. Wrapping with physics module means that they will have near the same module API for whitestormjs. So people will be able to easily switch from one to another and choose the one that fits

yea i see, i make something like that
with https://github.com/lo-th/Ammo.lab
code editor is near


FR: Change polling time

Would love the ability to modify the polling time from within the options menu
I find it is far too often for my liking
I feel this could be implemented in a similar way to the "Days until..." option, simply as a number of minutes between polls

Yes, preferably one that requires support for polling times

This is implemented in v1.5.0
Let me know if there's any issues


Hide Video Conferencing when muted

Summary of the Pull Request

PR Checklist

[x] Closes: #14123 & #18548
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments
Hide Video Conferencing toolbar when mic and camera are muted
This PR adds to the feature of VideoConferencing that allows us to hide the toolbar when Mic and Camera are both unmuted. This checkbox is now a select box where the user can select the condition on which to hide the toolbar.
There are three options:

Never
When both Mic and Camera are unmuted
When both Mic and Camera are muted

The options are self-explanatory to how they modify the toolbar behaviour upon selecting them

Validation Steps Performed

Wait a minute, does this PR fix https://github.com/microsoft/PowerToys/issues/18548?

@AnonymousWP It's supposed to. But I am very new to programming in C# and C++ 17. Right now I am having trouble saving the settings that I have created. I am trying to figure out how it works and only then I will be able to tell if my modifications actually even work

@htcfreek made the change that you suggested but kept unmuted in the second position
@AnonymousWP The code is in working condition. Worked in my local environment.
Please review the code and leave some comments. I am sure the code could've been better since this is my first time using these languages.
I would also like comments on whether the feature implemented here seems logical and easy to understand? Especially the behaviour related to when "Camera is not in use".

@AnonymousWP It's supposed to. But I am very new to programming in C# and C++ 17. Right now, I am having trouble saving the settings I created. I am trying to figure out how it works, and only then I will be able to tell if my modifications even work

That's great, thanks! I was hyped for the Video Conference Mute feature, but I don't use it due to this feature missing. Glad you're attempting to add it. Unfortunately I don't have enough programming knowledge of C# to contribute/help.
I think you should also take a look at the PR Checklist at the top of this PR.

@akabhirav , please make sure to let us know when we should give another review here ;) Thank you!

Posting this as a general comment as this affects this PR greatly
I had an idea when I started taking a look at it again. Tell me if this sounds intuitive to you guys

Hides when both camera and microphone are muted

If the camera was previously not in use then everything automatically gets muted(or maybe it should just change the camera's state)
If the camera was previously in use then nothing changes

Hides when both camera and microphone are unmuted

If the camera was previously not in use everything gets unmuted(or maybe it should just change the camera's state)
If the camera  was previously in use nothing changes

I have added these changes in my latest commit. Please check it out

@jaimecbernardo, @yuyoyuppe This PR is ready for review

Thank you for the update, we have a lot of stuff pending for the next release, so we'll take a look once it's rolled out.

@yuyoyuppe if that one item is all that is needed can we merge and just do the fox in a new PR asap?

I think so, yes. Should we wait for @akabhirav confirmation?

@yuyoyuppe Hey, I am sorry about this, I have been very busy with my work lately, and I am not free before the first week of November. If you guys want to, you can merge this change and make the fix, or I will do it the after I come back


promote getting-started content

/cc: @richlander @BillWagner changed tutorial titles in toc to getting started, added link to tutorials at top of primer


[#1315]Add RPCHook construct method for MQPullConsumerScheduleService

What is the purpose of the change
Add RPCHook construct method for MQPullConsumerScheduleService
Brief changelog
Add RPCHook construct method for MQPullConsumerScheduleService
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

LGTM

Coverage increased (+0.01%) to 50.531% when pulling 75d8bb3be6bf901d8fd82e51a5041cadde14c2d9 on duhenglucky:SchedualPullPolish into 92262928fef9727f1e3ec0066e9dd5edb74d7bb5 on apache:develop.


[docker] sw-nvdl-docker/pytorch:19.10-py3 is not available

Could you please public the docker image?

It seems our reference is wrong,  I ll fix the reference today, it should be pulling from the public version from link below.
https://ngc.nvidia.com/catalog/containers/nvidia:pytorch/tags

Can we use the official Pytorch docker image rather than NGC?


portable version ceased working

Before I updated from version 5.5.2 to 7.0.1 I could use grabber portable. Now it won't open, I get this in the log:
[Warning] [Qt][default] QString::arg: Argument missing: Error posting message: %s, Host www.google-analytics.com not found
to use portable I moved the contents from C:/Users/%USERNAME%/AppData/Local/Bionus/Grabber to my install location

now this is really embarrassing. I think it works now, I copied the contents from the wrong user, I didn't use the program for so long that forgot I about this (I somehow have two user accounts)


Support custom  labels for controlPlane's job

Component(s)
controlplane, helm
Is your feature request related to a problem? Please describe.
My company uses Istio mesh, so we usually use labels in the namespace to inject the sidecar istio proxy in each pod.
helm installs or upgrades are failing since istio-proxy are injecting in pods from Jobs.
because the pods don't finish in a completed state.
Describe the solution you'd like
if jobs support custom labels, I am going to have more control which labels are injecting on Job Pods.
It will also help to identify attributes of objects that are meaningful and relevant.
also I can tell to istio controller to don't inject istio-sidecar
Describe alternatives you've considered
This is my proposal solution. You will notice that I added a new variable called customLabelsJobsControplane
# The otelcollector provides the necessary database schema for the migration
{{- if .Values.global.otelcollector.enabled }}
apiVersion: batch/v1
kind: Job
metadata:
  name: "{{ include "controlplane.fullname" . }}-clickhouse-migration-{{ now | unixEpoch }}"
  labels:
    {{- include "controlplane.labels" . | nindent 4 }}
    {{- if .Values.customLabelsJobsControplane }}{{ toYaml .Values.customLabelsJobsControplane | nindent 4 }}{{- end }}
  annotations:
    # Run migration job before helm install/upgrade
    "helm.sh/hook": post-install,post-upgrade
    "helm.sh/hook-delete-policy": before-hook-creation,hook-succeeded
    # Less priority than the migration job so the secret is created before the migration job
    "helm.sh/hook-weight": "-5"
    # Support for k14s.io. This annotation ensure that the job is run after clickhouse deployment
    {{- if .Values.global.clickhouse.enabled }}
    kapp.k14s.io/change-rule.clickhouse: "upsert after upserting cosmo.apps.clickhouse.wundergraph.com/deployment"
    {{- end }}
    {{- if .Values.global.otelcollector.enabled }}
    kapp.k14s.io/change-rule.otelcollector: "upsert after upserting cosmo.apps.otelcollector.wundergraph.com/deployment"
    {{- end }}
    # Create a new change-group so the controlplane can be deployed after the clickhouse migration
    kapp.k14s.io/change-group: "cosmo.apps.controlplane.wundergraph.com/ch-migration"
spec:
  backoffLimit: 10
  parallelism: 1
  template:
    metadata:
      name: "{{ include "controlplane.fullname" . }}-clickhouse-migration"
      labels:
        {{- include "controlplane.labels" . | nindent 8 }}
        {{- if .Values.customLabelsJobsControplane }}{{ toYaml .Values.customLabelsJobsControplane | nindent 8 }}{{- end }}
    spec:
      restartPolicy: OnFailure
      {{- with .Values.imagePullSecrets }}
      imagePullSecrets:
        {{- toYaml . | nindent 8 }}
      {{- end }}
      containers:
        - name: seed
          securityContext:
            {{- toYaml .Values.securityContext | nindent 12 }}
          image: "{{ include "controlplane.image" . }}"
          imagePullPolicy: {{ .Values.image.pullPolicy }}
          env:
            - name: CLICKHOUSE_DSN
              valueFrom:
                secretKeyRef:
                  name: {{ include "controlplane.secretName" . }}
                  key: clickhouseMigrationDsn
          args:
            - "/app/dist/bin/ch-migrate.js"
{{- end }}

Additional context
No response

Hi @esparta86 thanks for the report. We'll take a deeper look and come back. Would you be open to create a PR after we have agreed on a solution?

of course, I made those changes in my local and tested it.
let me know when you have an agreement solution.

Hi @StarpTech any update on this? I'll appreciate it

Hi @esparta86, sry for the late response. It's on my list. I'm going to come up with a solution asap.

hi @StarpTech any updates on it?  would like to know if there a plan to have custom labels for all components

+1 on this.
It would be very useful the support for custom labels.

Hi @esparta86, @keadex we're addressing in the coming week.

Hi @esparta86 thank you for coming up with this,
I released https://artifacthub.io/packages/helm/cosmo-platform/cosmo/0.9.0.
You can now pass over additional labels to the jobs.
Have a look here https://github.com/wundergraph/cosmo/blob/f2a6ba5f0b25a2f2093536fceb3c865090ba4f5a/helm/cosmo/values.yaml#L105
e.g.:
controlplane:
  additionalJobLabels: # added to all jobs
    sidecar.istio.io/inject: false
...
  jobs:
      seedOrganization:
        additionalLabels: {} # added e.g. to the seed job

Let me know if this works out for you.

Hi @AndreasZeissner we can close it. I tested that version and it is working as expected.


Add flat list proof support [ECR-3608]

Overview

Add flat list proof support.

See:

https://jira.bf.local/browse/ECR-3608
https://wiki.bf.local/display/EXN/Flat+list+proofs

Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Currently, only unit tests are implemented, hence WIP status. The unit tests in FlatListProofTest can be reviewed.

@slowli Would review please if the most important scenarious are covered?

I like tests so far :+1:

The implementation is complete. We might consider extracting verification of each type of proof in separate classes to share some data used during verification (e.g., treeHeight, proofEntriesByHeight, etc.)

Pushed #getHeight down from the interface: https://github.com/exonum/exonum-java-binding/pull/1156/commits/cb286cba40a066c8221675e3199b86bfe5c6a3f6


Bump plutus dep to 1.16

Routine bump, lets us get rid of some instances, and supports newer nothunks.

@michaelpj We prefer rebase instead of commits that merge master into ledger
This results in a nice linear history
I've fixed this in this PR by merging this PR with a squashing, but just letting you know for the future.

Ah right, I just hit the "update branch" button because it was there!


Possibility to connect multiple parents to a child

I wonder if it would be possible to implement an option to be able to connect mutliple parents into the same child somehow? I apologize if that's already possbile and I've missed that.

Same needed here!! This feature would be fantastic!

This feature exists using NgFlowchart.Options.manualConnectors = true to draw arrows from parents to a single child.


[Bug]: Broken links across the repository

Language
Javascript/Typescript, C#, Python
Version
latest
Description
I ran a broken link check on the repo and found that there are over 50 broken links some of which  return HTTP error 404, 403.
Attaching the report file with this issue.
teams-ai-broken-links.txt
Reproduction Steps

install link checker npm install -g markdown-link-check
cd into teams-ai
run for file in $(find . -type f -name '*.md'); do markdown-link-check "$file"; done
Observe the results (attaching a simplified report)

I did not know about this package, and I love it! Thank you for the heads up! Re-reviewing your PR.


How to get visible date range?

I am hoping to fetch events for the given view from the server with every "navigate" event. Is it possible to retrieve the visible date range from the component? Thanks!

onNavigate is called with two argumente: current date and the view (week, agenda etc.). Check it like so
onNavigate={(date, view) => {
            console.log('#### onNavigate');
            console.log('#### date=', date);
            console.log('#### view=', view);
            //this.setState({currentDate: date});
          }}

It is possible to calculate range like so:

if view is day: from date to date;
if view is week: from moment(date).startOf('isoWeek') to moment(date).endOf('isoWeek');
if view is month: from moment(date).startOf('month').subtract(7, 'days') to moment(date).endOf('month').add(7, 'days'); i do additional 7 days math because you can see adjacent weeks on month view (that is the way how i generate my recurrent events for the Big Calendar, but if you need only start-end of month - just remove that math);
if view is agenda: from date to whatever you want, for instance moment(date).add(7, 'days');

Don't forget onNavigate is not called when user changes only view, so you also need to handle onView like so:
onView={(view) => {
            console.log('#### onView');
            console.log('#### view=', view);
            //this.setState({currentView: view});
          }}

Thanks! Your approach was where I ended up. The "adjacent weeks on month" part was what I wasn't sure was appropriate (thinking there might be a more direct way of getting that info).

For future viewers, I created a full example Gist using @greenya's method above
https://gist.github.com/stevenkaspar/23098c255689cc623f82141564dc41cc

You should be able to use the library's own utils to retrieve the first and last visible day:
import dates from 'react-big-calendar/lib/utils/dates';

...
    onNavigate = (date, view, action) => {
        console.log(dates.firstVisibleDay(date), dates.lastVisibleDay(date));
    }
...

When displaying as week view using moment(date).startOf('isoWeek') to moment(date).endOf('isoWeek')  yields a week spanning Mon-Sun but my calendar is showing Sun-Sat. In the jQuery implementation of Full Calendar it is trivial to access the start and end dates of the current view e.g. https://stackoverflow.com/questions/32777017/how-to-get-current-displayed-date-range-in-full-calendar-jquery. Does the same concept not exist for React-Big-Calendar?

@ash-PS have you tried with .startOf('week') ?

@felgeekpe yes that works well thanks! I'm still a little surprised that users need to reimplement this logic every time they use Big Calendar, I'm assuming the date range is already getting calculated somewhere to drive the rendering of the calendar.


An unexpected error has occurred.

Demo Show Error : "An unexpected error has occurred" . how to view it?

https://github.com/mui-org/material-ui/commit/cd1be834ebe0369268cc5709549fdf1d33638e67

@oliviertassinari Thanks

It's live, it's working.

It is most definitely NOT live. I still have to visit this link to get it to work: https://material-ui.com/?lang=en

@krozett What page have you having issue with? There is a language picker now:

@oliviertassinari probably a duplicate of #13705?

@oliviertassinari Clearing my cache did not help. I was getting 404s when trying to load JS files. The solution was to delete my IndexedDB. You may want to look into this since other people were reporting similar things in the issues above.


Fix build of catch.hpp.

This is a temporary fix while you work on a migration to the newest version of catch.
This change is analogous to what was done here: https://github.com/catchorg/Catch2/commit/42e368dd0aedc122798008d8c4f583fea3296a97#diff-624ebe102997b0fba85c212e9f4b14373aeef3b18a394320e9651d15f7cecce6R10930

Replaced by pulling cach2 v3 directly from the official repo. Thanks for the PR!


Error: @drawable/ic_action_content_new

I followed the steps on the readme but I get this error, can you help me? Thank you.
Error:(24, 26) No resource found that matches the given name (at 'src' with value '@drawable/ic_action_content_new').

Sorry, it was my fault.


Better animation when adding or removing data

In order to simulate real-time chart updates (i.e. horizontal animation), it's necessary to distinguish a removed or added value from a simple update. The dataset controller now hooks array methods that alter the data array length to synchronize metadata accordingly. Also remove the duplicate calls of updateBezierControlPoints() for line and radar charts.
Examples: http://playground.abysscorp.org/chartjs/livecharts/
Plunker: http://plnkr.co/Imxwl9OQJuaMepLNy6ly
Related issues: #1997 #3126

I am 👍  to merge this. I don't know if you prefer waiting for any other comments or not

Added data cleanup (datasetController.destroy() method) and associated tests:

Can we merge this?


Fix for #257, #256 and #254

Fix for #257, #256 and #254.

I updated package.json with both changes on my Mac and still get mostly playback errors.

The errors is location based according to the nearest youtube server


fix: update color

ok

:tada: This PR is included in version 1.2.13 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


WIP: Try to use libtasn1

Checklist

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

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Blocked by https://github.com/conda-forge/libtasn1-feedstock/pull/8.

@conda-forge-admin, please rerender

Thanks, @sachalau! Shall we get it in?


Support UMD for a more universal use of modules in "dist" and "devdist" folder

Basic information

Version v3.2.0
Build

Actual behaviour
Currently it is not possible to use previously built javascript modules directly without npm.
For example, the following tag inside a static HTML won't provide the expected "datarestructor" Namespace:
<script src="../../node_modules/data-restructor/devdist/datarestructor.js">

Expected behaviour
Maybe this could be solved using parcel build option --target set to "node" instead of the default value "browser". Additionally it might be needed to use --force-node-modules-bundling e.g. for the internet explorer compatible variants.
Value
This could be very useful for a static local "SpecRunner.html" (jasmine unit tests) and for anyone,
who wants to use the modules without npm and/or parcel.

It turns out to be neither easy nor the right way to build vanilla js files without any module system just to support direct browser use. There are already tools that address this more elegant like browserify.
Nonetheless does it make sense to optimize build output to support UMD and tweak some parcel build options especially for "devdist" build.
For simple vanilla javascript applications, the source files can still be accessed directly, even through "node_modules".


Failed to load resource

I followed install instructions, but I receive ...
http://localhost:3000/angular2-text-mask Failed to load resource: the server responded with a status of 404 (Not Found)
I then tried manually specifying .js file from nodemodule but then it failed on some core module.

i've the same issue... can somebody help me?

Could you provide a little more info? The configuration used would help alot.

I followed install instructions as show in  project home page.
Error: (SystemJS) XHR error (404 Not Found) loading http://localhost:3030/angular2-text-mask(…)
configuration files...
`* System configuration for Angular samples

Adjust as necessary for your application needs.
*/
(function (global) {
System.config({
paths: {
// paths serve as alias
'npm:': 'node_modules/'
},
// map tells the System loader where to look for things
map: {
// our app is within the app folder
app: 'app',
// angular bundles
'@angular/core': <EMAIL_ADDRESS>'@angular/common': <EMAIL_ADDRESS>'@angular/compiler': <EMAIL_ADDRESS>'@angular/platform-browser': <EMAIL_ADDRESS>'@angular/platform-browser-dynamic': <EMAIL_ADDRESS>'@angular/http': <EMAIL_ADDRESS>'@angular/router': <EMAIL_ADDRESS>'@angular/forms': <EMAIL_ADDRESS>// other libraries
'rxjs':                      'npm:rxjs',
'angular-in-memory-web-api': 'npm:angular-in-memory-web-api',
},
// packages tells the System loader how to load when no filename and/or no extension
packages: {
app: {
main: './main.js',
defaultExtension: 'js'
},
rxjs: {
defaultExtension: 'js'
},
'angular-in-memory-web-api': {
main: './index.js',
defaultExtension: 'js'
}
}
});
})(this);
 {
"compilerOptions": {
"target": "es5",
"module": "commonjs",
"moduleResolution": "node",
"sourceMap": true,
"emitDecoratorMetadata": true,
"experimentalDecorators": true,
"removeComments": false,
"noImplicitAny": false
}
}
`

Sorry, I'm a bit confused but what are you expecting to find when you load http://localhost:3000/angular2-text-mask?

I'll close this for now, but will be happy to continue to answer questions or reopen the issue if needed. Thanks!


Request: add delete multiple channels

Could you add the ability to delete multiple channels. Thanks

would be useful to clear all messages from a certain workspace for example.


Can't use shift+home to select to beginning of line

VSCode Version: 1.4.0
OS Version: Windows 10 v1511 b10586.494

Steps to Reproduce:

Open a file (I have tested this with .rb, .json, and .html files)
Click on the middle of a code line with indentation
Press home once & observe that the caret to the beginning of the text, after the indentation whitespace (correct)
Press home once more & observe that the caret moves to the beginning of the line, before the indentation whitespace (correct)
Press home once more & observe that the caret moves to the beginning of the text, after the indentation whitespace (correct)
Press shift + home & observe that nothing happens (incorrect). What should happen is that the caret should be at the beginning of the line with indentation whitespace selected.

It's possible there's a setting for this somewhere, but I didn't see it.

I forgot I had TabSanity installed. Uninstalling it fixes the problem, so... oops.


Fix arity of the function parameter in the map method of Map collections

Tries to fix #68.
But I get the following error when I try compiling with scalac (cc @odersky ):
[error] collection-strawman/src/main/scala/strawman/collection/Map.scala:84: name clash between defined and inherited member:
[error] def map[B](f: ((K, V)) => B): strawman.collection.Iterable[B] in trait IterablePolyTransforms and
[error] def map[K2, V2](f: ((K, V)) => (K2, V2)): C[K2,V2] at line 84
[error] have same type after erasure: (f: Function1)Object
[error]   def map[K2, V2](f: ((K, V)) => (K2, V2)): C[K2, V2] = mapFromIterable(View.Map(coll, f))
[error]       ^
[error] collection-strawman/src/main/scala/strawman/collection/Map.scala:86: name clash between defined and inherited member:
[error] def flatMap[B](f: ((K, V)) => strawman.collection.IterableOnce[B]): strawman.collection.Iterable[B] in trait IterablePolyTransforms and
[error] def flatMap[K2, V2](f: ((K, V)) => strawman.collection.IterableOnce[(K2, V2)]): C[K2,V2] at line 86
[error] have same type after erasure: (f: Function1)Object
[error]   def flatMap[K2, V2](f: ((K, V)) => IterableOnce[(K2, V2)]): C[K2, V2] = mapFromIterable(View.FlatMap(coll, f))
[error]       ^

Maybe try without mapPolyTransforms? Define it directly in Map and you should be OK.

Maybe change the definition of MapPolyTransforms from
trait MapPolyTransforms[K, +V, +C[X, Y]]
to
trait MapPolyTransforms[K, +V, +C[X, Y] <: Map[X, Y]]

Have you tried that? (The intention is that this would then be erased to Map instead of Object).

Yes, nice trick!

The trick you proposed solved the erasure error but now we have troubles at use site: we can not use the partial function syntax to deconstruct the tuple (see the tests).
Maybe the following patch is supposed to help? https://github.com/scala/scala/pull/5698 (cc @szeiger )


Bug: Project screen NaN/infinite margin

If you define a subcontractor + rate, without the client rate you get Infinity:

If you define no rates you get NaN

Not a complete fix


Do a release before SunPy 1.0

@nabobalis @dpshelio where are we on this? Does it work with master at the moment? Should we do a release now? Do we have a way of knowing if we break this with sunpy master?

Yes. Yes if nothing will break in sunpy 1.0 in the last two main PRs. The CI will tell you, we can schedule it?

@nabobalis can you take the conn on getting a release of this out the door? and scheduling a cron with webhook into the infrastructure room?

Sure and Sure but no room.

Thanks

Up on PyPi now, feedstock updated.


Update adding_params.md to use ignite

The team at ignite responded to our feature request but it's not available yet: https://github.com/ignite/cli/issues/3684.
We should update to it at some point.

The only error is the flaky leveldb panic as seen here.
Merging since this is just a comment update.


yosys-config reports incorrect paths in prebuilt binaries

yosys-config reports paths that are baked-in at build time. In the prebuilt releases generated on GitHub, all paths are prefixed with /root/synlig/synlig/out/release, which makes the yosys-config bundled with Synlig nonfunctional unless the user extracts the archive in exactly that path.
oss-cad-suite overcomes this problem by patching yosys-config (replacing the original build path with a path relative to yosys-config's location). Perhaps something similar could be included in Synlig, either in the main build systems (Make/CMake) or at least in the GitHub workflow that generates the prebuilt binaries.
Ideally, this modification would be included in Yosys's build system, but the way yosys-config is currently generated makes it quite difficult since external library include paths can't be easily separated from Yosys's own at that point in the Makefile.

Sounds good, we just need to make sure the "plugin only" releases are unaffected (we want them to work with external yosys installs)


Dataset title

This issue is in some way linked to #2.
At the moment we name this checklist as "List of Invasive Alien Species of Union concern". However, this could be a source of concerns with EU. Maybe can we add the "LIFE RIPARIAS" prefix? Any idea?

I see your point, but it is not only the "RIPARIAS Union List", it is one we used at INBO for baseline reporting, for TrIAS indicators, for EU reporting etc. And essentially that is just what it is. So I would just leave that title as is and make sure the metadata explain that this is our work and our interpretation of mapping the Union list taxonomy to the GBIF backbone.

Agree.


Slow build speed with punica after #1804

After #1804, the build speed of vLLM becomes significantly slower. Most of the time was spent on /home/zhuohan/vllm/vllm/csrc/punica/bgmv/bgmv_all.cu. Should we turn off punica by default?
cc @Yard1

Should we make it depend on an env var that can be set when calling setup.py/pip install?

Duplicated by https://github.com/vllm-project/vllm/issues/2604 and now resolved by #2605?


gitea loss of avatars after update

Gitea version (or commit ref): update to 1.4.3 from 1.3.2
Git version: 2.15.2
Operating system: ubuntu 14.04
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No
[ ] Not relevant

Log gist:

Hello!
After update gitea container to 1.4.3 from 1.3.2 we have loss all avatars.
When adding avatars manually again, i found out that they are saved in /app/gitea/data/avatars.
In this folder (app/gitea/data)  i also saw "indexers" and "sessions" folders.
We have "sessions" folder in /data/data dir (the dir use for store data as described in the documentation with docker use
volumes:
- ./gitea:/data )
I thout that to save all the necessary data for the operation of the gitea (sessions, users, avatars and other), we need to use the "data" folder on the host (or named volume). But this is not onough, wee still need save data from many folders (/app/gitea/data/avatars or /app/gitea/data).
Please specify the full list of folders that we need to save, for a successful update of the gitea if using docker and store all data (sessions, vatars and others working data)
Best regards,
Danilin Maxim

all /app/gitea/data should be backed up

There is nothing in the documentation about this, right?

@lafriks : last time I tried (couple of weeks ago), gitea dump command did not back this up, even if documentation says otherwise


Allow arguments to property bind

Title. A weird side effect is that x@y() is now equivalent to x@y.
Closes #954

Looks great, thanks!


window.requestIdleCallback

From @bsvipas on January 19, 2018 7:0

VSCode Version: 1.19.2
OS Version: macOS

There's no suggestion for window.requestIdleCallback (rIC) only for window.requestAnimationFrame (rAF).
Copied from original issue: Microsoft/vscode#41850

Too bad.

Posting this here for people who want to copy/paste into their projects:
type RequestIdleCallbackHandle = any;
type RequestIdleCallbackOptions = {
  timeout: number;
};
type RequestIdleCallbackDeadline = {
  readonly didTimeout: boolean;
  timeRemaining: (() => number);
};

declare global {
  interface Window {
    requestIdleCallback: ((
      callback: ((deadline: RequestIdleCallbackDeadline) => void),
      opts?: RequestIdleCallbackOptions,
    ) => RequestIdleCallbackHandle);
    cancelIdleCallback: ((handle: RequestIdleCallbackHandle) => void);
  }
}


fixes #1

Join and mark as html safe so it stops getting outputted as as pesky string.

Thanks for your help!


can you do a tutorial / detailed readme on how to use the source? (i can pay for this)

my discord Suwup#0001

Recently I have a lot of spare time so I'll probably just update the read me file & include a rough sketch of the DB as I lost the original  for it

Recently I have a lot of spare time so I'll probably just update the read me file & include a rough sketch of the DB as I lost the original for it

can you add me on discord: Suwup#0001 and help me set up this? or just tell me how to set up the sql database

any updates? that sql file would be nice to have ;)

Mostlikely not gonna happen


Incorrect bounding box placement in dashboard display and downloaded files

System:

OS: Arch Linux x86_64
Kernel: 5.8.14-arch1-1
Python: 3.8.6
wandb: 0.10.4

Description:
Some image samples have incorrectly displayed bounding boxes, as displayed in the images below:
Correct bounding boxes, drawn using using PIL and ImageDraw and uploaded to wandb:

Bounding boxes generated by wandb, after downloading the logged image:

Bounding boxes displayed in the wandb dashboard UI, obtained via screenshot in fullscreen/expanded mode:

Interestingly, in both wandb samples, some bounding boxes are correct, but some are misplaced. Moreover, the downloaded image seems to have mainly boxes with correct x coordinates bu incorrect y coordinates, and the dashboard screenshot shows the converse.
Steps to reproduce:

Log an image and its bounding boxes e.g. from COCO or GQA to wandb
Compare to the known bounding box positions from the dataset.

The following code snippets were used to log the boxes:
"""Tools for visualising images, bounding boxes and attention maps."""
from typing import Dict, Optional, Sequence, List, Tuple
import random

import wandb
from torch import Tensor
from PIL import Image, ImageDraw
import torchvision.transforms as T

class BoundingBox:
    """Class wrapper for storing bounding box information."""

    def __init__(
        self,
        min_x: int,
        min_y: int,
        max_x: int,
        max_y: int,
        label: Optional[int] = None,
        score: Optional[float] = None,
    ) -> None:
        """Initialise a bounding box instance."""
        self._coords = (min_x, min_y, max_x, max_y)
        self._label = label
        self._score = score

    @property
    def min_x(self) -> int:
        """Get the minimum x value of the box, measured from the left of the image."""
        return self._coords[0]

    @property
    def min_y(self) -> int:
        """Get the minimum y value of the box, measured from the top of the image."""
        return self._coords[1]

    @property
    def max_x(self) -> int:
        """Get the maximum x value of the box, measured from the left of the image."""
        return self._coords[2]

    @property
    def max_y(self) -> int:
        """Get the maximum y value of the box, measured from the top of the image."""
        return self._coords[3]

    @property
    def label(self) -> Optional[int]:
        """Get the bounding box's label."""
        return self._label

    @property
    def score(self) -> Optional[float]:
        """Get the bounding box's score."""
        return self._score

def plot_image(
    image: Tensor,
    boxes: Optional[Dict[str, Sequence[BoundingBox]]] = None,
) -> None:
    """Plot and save an image with optional bounding boxes to file."""
    tfm = T.ToPILImage()
    img = tfm(image)
    draw = ImageDraw.Draw(img)
    if boxes is not None:
        for bbox_seq in boxes.values():
            for bbox in bbox_seq:
                color = (
                    random.randint(0, 255),
                    random.randint(0, 255),
                    random.randint(0, 255),
                )
                draw.rectangle((bbox.min_x, bbox.min_y, bbox.max_x, bbox.max_y), outline=color)
    return wandb.Image(img)

def wandb_image(
    image: Tensor,
    caption: Optional[str] = None,
    boxes: Optional[Dict[str, Sequence[BoundingBox]]] = None,
    object_to_index: Optional[Dict[str, int]] = None,
) -> wandb.Image:
    """Create a list of wandb images for logging with optional bounding boxes."""
    class_labels = (
        {idx: key for key, idx in object_to_index.items()}
        if object_to_index is not None
        else None
    )
    wandb_boxes = (
        {
            key: {
                "box_data": [
                    {
                        "position": {
                            "minX": box.min_x / image.size(2),
                            "maxX": box.max_x / image.size(2),
                            "minY": box.min_y / image.size(1),
                            "maxY": box.max_y / image.size(1),
                        },
                        "class_id": box.label,
                        "box_caption": class_labels[box.label]
                        if class_labels is not None and box.label is not None
                        else box.label
                    }
                    for box in keyed_boxes
                ],
                "class_labels": class_labels,
            }
            for key, keyed_boxes in boxes.items()
        }
        if boxes is not None
        else None
    )
    return wandb.Image(image, boxes=wandb_boxes, caption=caption)

The above functions were called like so:
Please excuse the unreferenced functions and classes. I hope this is enough to give some context.
        visualisations: Dict[str, Any] = {"images": [], "custom": []}

        dataloader = DataLoader(
            torch.utils.data.Subset(
                self.datasets.val, range(0, 10)
            ),
            batch_size=1,
            num_workers=0,
            collate_fn=VariableSizeTensorCollator(),
        )
        with torch.no_grad():
            for sample in dataloader:
                # Load data
                dependencies = sample["question"]["dependencies"].to(self.device)
                graph = sample["scene_graph"]["graph"].to(self.device)
                image = self.datasets.images[
                    self.datasets.images.key_to_index(
                        sample["scene_graph"]["imageId"][0]
                    )
                ]
                # Propagate data forward
                self.model(question_graph=dependencies, scene_graph=graph)

                print(f'{sample["scene_graph"]["boxes"][0].size()=}')
                print(f'{len(sample["scene_graph"]["boxes"][0].tolist())=}')
                print(f'{image.size()=}')

                # Add image with labeled ground truth bounding boxes
                visualisations["images"].append(wandb_image(
                    image,
                    caption=sample["scene_graph"]["imageId"][0],
                    boxes={
                        "ground_truth": [
                            BoundingBox(box[0], box[1], box[2], box[3], label=lbl)
                            for box, lbl in zip(
                                sample["scene_graph"]["boxes"][0].tolist(),
                                [self.preprocessors.scene_graphs.object_to_index[obj_lbl[0]] for obj_lbl in sample["scene_graph"]["labels"]],
                            )
                        ]
                    },
                    object_to_index=self.preprocessors.scene_graphs.object_to_index,
                ))
                visualisations["custom"].append(plot_image(
                    image,
                    boxes={
                        "ground_truth": [
                            BoundingBox(box[0], box[1], box[2], box[3], label=lbl)
                            for box, lbl in zip(
                                sample["scene_graph"]["boxes"][0].tolist(),
                                [self.preprocessors.scene_graphs.object_to_index[obj_lbl[0]] for obj_lbl in sample["scene_graph"]["labels"]],
                            )
                        ]
                    },
                ))

Hi @alexmirrington , thanks for flagging this issue. We're looking into this.

Hi @alexmirrington
Could you please let us know if the issue still persists?
We have had quite a few upgrade with the package lately. :smile:

Hi @ariG23498,
I've been working on other portions of my code base lately but when I get back to data visualisations I will let you know. Hopefully, I'll have a solid answer by the end of the week

Hi @ariG23498, The problem seems to be fixed now when using wandb==0.10.12, so I will close the issue :)


ci: validate async delete

Reason for Change:

adds a script to validate pod async delete when cns is down
only testing in swift for now, will add to cilium stages once azure-ipam changes are reverted back

Issue Fixed:

Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] relevant PR labels added

Notes:

cc @tyler-lloyd


Clean up & break out the LESS file

The less file is grown huge and there's bunch of tags in there that aren't being used anymore.

This is kind of done


The documentation is hard to read!

** READ THIS FIRST! **
Are you looking for help?
Reminder: The issue tracker is not a support forum.
Issues should only be filed in this project once they are able to be reproduced
and confirmed as a flaw in the software or incorrect information in associated
documention.
If you are encountering problems integrating this module into your application,
please post a question on the discussion forum
rather than filing an issue.
Is this a security issue?
Do not open issues that might have security implications.  Potential security
vulnerabilities should be reported privately to<EMAIL_ADDRESS> Once any
vulerabilities have been repaired, the details will be disclosed publicly in a
responsible manner.  This also allows time for coordinating with affected parties
in order to mitigate negative consequences.
If neither of the above two scenarios apply to your situation, you should open
an issue.  Delete this paragraph and the text above, and fill in the information
requested below.

Expected behavior

Actual behavior

Steps to reproduce

// Format code using Markdown code blocks

Environment

Operating System:
Node version:
passport version:

Could you please describe what you find difficult or offer some clues as to what you think would help improve the documentation?  That would make this issue actionable.  Otherwise, this issue comes across as a general complaint, and is unlikely to result in any improvements.  Thanks.


Custom element doesn't exist: ha-card-weather-conditions

Hi, I just installed the card (version c913ad8) via HACS (1.2.2) on my HomeAssistent (0.112.4) without error, but when I try to use it, I get the error "Custom element doesn't exist: ha-card-weather-conditions". The files are present on the system.
What might be wrong here? What and where can I have a look to get more details about the problem?

Same here...
Here it is what i can see in the console:

Sale same😀

Installed latest release but got nothing new...
Console says:

I renamed all of the *.json files to *.json1 and it now seems to work.

I renamed all of the *.json files to *.json1 and it now seems to work.

Didn't work for me, any chance you may have done something else? I'm also seeing the json1 not found but changing it didn't do anything

pi@home-assistant-pi4:~/.homeassistant/www/community/ha-card-weather-conditions/transl $ ls -lrt
total 64
-rw-r--r-- 1 pi pi 699 Jul 21 17:56 en.json
-rw-r--r-- 1 pi pi 747 Jul 21 17:56 fr.json
-rw-r--r-- 1 pi pi 736 Jul 21 17:56 de.json
-rw-r--r-- 1 pi pi 711 Jul 21 17:56 pt.json
-rw-r--r-- 1 pi pi 719 Jul 21 17:56 nl.json
-rw-r--r-- 1 pi pi 727 Jul 21 17:56 it.json
-rw-r--r-- 1 pi pi 714 Jul 21 17:56 sr-latn.json
-rw-r--r-- 1 pi pi 730 Jul 21 17:56 es.json
-rw-r--r-- 1 pi pi 699 Jul 21 18:08 en.json1
-rw-r--r-- 1 pi pi 727 Jul 21 18:09 it.json1
-rw-r--r-- 1 pi pi 747 Jul 21 18:10 fr.json1
-rw-r--r-- 1 pi pi 736 Jul 21 18:10 de.json1
-rw-r--r-- 1 pi pi 711 Jul 21 18:10 pt.json1
-rw-r--r-- 1 pi pi 719 Jul 21 18:10 nl.json1
-rw-r--r-- 1 pi pi 730 Jul 21 18:11 es.json1
-rw-r--r-- 1 pi pi 714 Jul 21 18:11 sr-latn.json1
This is all I did.

I had the same issue. I renamed the .json files to .json1 and it still didn't work. Instead, as @jmutnick showed, I copied the files so that I have both .json and .json1 files. It still didn't work, but after restarting HA, clearing browser cache and logging back in, it's working again.

I'm really sorry. It is a little bug fixed in the newly released version.
Thanks to all for your support.
Renato

Thanks you Grazie a te Renato
Il 22 lug 2020, 21:09 +0200, Renato Rossi<EMAIL_ADDRESS>ha scritto:

I'm really sorry. It is a little bug fixed in the newly released version.
Thanks to all for your support.
Renato
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Any ETA on the publication of the fix ?
According to HACS I am already on the latest version

I'm really sorry. The newly released version fixed this bug.
Renato

Thanks, the update has fixed my problem

Working again with the latest version. Thanks 👍🏻