Investigate Embedding UI as Iframe

Investigate embedding the UI as an Iframe while still being able to communicate with the local API to capture requests and forward them to the UI.

@ilyagelman @morsdyce Should work just fine, add this to any page to test (ES6 must be supported):
/**
 * Shredder API Dummy
 */

window.Shredder = {
  sayHi: (one, two) => {
    console.log('API sayHi', one, two);
  }
};

/**
 * Add an iFrame element for the Shredder UI
 */

let iframeElm = document.createElement('iframe');
iframeElm.src = '';
iframeElm.width = '100%';
iframeElm.height = '100%';

document.body.appendChild(iframeElm);

/**
 * Get the iFrame's document, body and head
 */

let iframeRoot = iframeElm.contentDocument;
let iframeBody = iframeRoot.body || iframeRoot.querySelector('body');
let iframeHead = iframeRoot.querySelector('head') || iframeBody;

/**
 * Prepare Shredder UI Styles
 */

let uiStyle = document.createElement('style');
uiStyle.textContent = `
  h1 {
    background: lightblue;
  }
`;

/**
 * Prepare Shredder UI View
 */

let uiHtml = document.createElement('div');
uiHtml.innerHTML = `

Bad Ass API Mocking

`;

/**
 * Prepare Shredder UI Code
 */

let uiScript = document.createElement('script');
uiScript.type = 'text/javascript';
uiScript.text = `
  // Execute methods from the Shredder API
  window.parent.Shredder.sayHi('Foo', 'Bar')
`;

/**
 * Inject the Shredder UI
 */

iframeBody.appendChild(uiHtml);
iframeHead.appendChild(uiStyle);
iframeBody.appendChild(uiScript);

Open questions

The code and styles must be post transpilation, that's no issue on production, we could have them as strings and just inject them, but how would it work in development?
We would still need some CSS to position the iframe and handle showing/hiding it, this would have to be added to the original page, the iframe styles would have to be as explicit as possible to avoid external styles from affecting it.

@alexilyaev What about mocking xhr requests within iframe?

@ilyagelman You don't, you mock them outside as usual, the iFrame is just the carrier that will communicate with the Shredder API.


Identify Relevant Networks in Spatial Data

Tutorials: https://developers.sap.com/tutorials/hana-cloud-smart-multi-model-4.html
Small typo

Hi,
Thank you for letting us know. This typo will be corrected soon. I hope you found the tutorial helpful.
Regards,
Vijay

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


an input trigger keydown event with  will trigger other input focus event

Version
2.6.11
Reproduction link
https://codesandbox.io/s/mystifying-golick-vu5wu?file=/src/App.vue
Steps to reproduce
the first input add 'focus' listeners
the other input trigger keydown event with <Enter> always trigger the first input focus event
What is expected?
only trigger focus event when the first input focused
What is actually happening?
the other input always trigger the first input focus event
Version
2.6.11
Reproduction link
https://codesandbox.io/s/mystifying-golick-vu5wu?file=/src/App.vue
Steps to reproduce
the first input add  listeners
the other input trigger keydown event with <Enter> always trigger the first input focus event
What is expected?
only trigger focus event when the first input focused
What is actually happening?
the other input always trigger the first input focus event

我没能重现这个错误


Prod envs now use openforcefield 0.8.4

Description
Version bump for the openforcefield toolkit 0.8.4rc1 -> 0.8.4.
Shouldn't be much different than 0.8.4rc1, so shouldn't be dramatic.
Status

[x] Ready to go

Something about 0.8.4 appears to have broken the tests. Based on the differences between 0.8.4 and 0.8.4rc1, is it obvious what may be causing this @j-wags?

CI runs on master appear to work fine still: https://github.com/openforcefield/openff-benchmark/actions/runs/647641024

Looking into this now. Apologies for the delay.

Probably an interaction with this change. Setting up a dev environment to resolve it.

Ok, I found and fixed two problems:

OFFTK 0.8.4 pre-populates the mapped SMILES in the extras dict when it runs offmol.to_qcschema. openff-benchmark then tries to set it again using the same mapped SMILES, which causes an error. So I added a try/except that should make it work for 0.8.4rc1 (which doesn't pre-populate the mapped SMILES) AND 0.8.4 (which does pre-populate it). This fix seems to work on my local machine. @dotsdl, could you sanity-check this?
SQLAlchemy 1.4 came out a few hours ago and is causing tests to fail. I don't know much about our use of that package. So I've pinned to 1.3.

Codecov Report

Merging #66 (f632282) into master (6c37648) will increase coverage by 3.45%.
The diff coverage is 100.00%.

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

Ah, thank you for this @j-wags! Reading your resolution it's clear I should have dove into it further myself, since I was directly involved in those changes.


Error when using computed values with ember-decorators-polyfill installed or Ember 3.10.0

Environment

Ember Version: 3.6 with ember-decorators-polyfill or current 3.10.0-beta.2
Ember CLI Version: 3.6
Ember CP Validations Version: 3.5.5 and 4.0.0-beta.7

Steps to Reproduce
I tested this with my work app at 3.8 and used the dummy app at 3.6 and installed ember-decorators-polyfill
Having validations that use a form of computed for their value.
models/user-detail.js - dob validator
after: computed(function() {
  return moment()
    .subtract(120, 'years')
    .format('M/D/YYYY');
}).volatile(),

Results in the error:
Error: Assertion Failed: EmberObject.create no longer supports defining computed properties. Define computed properties using extend() or reopen() before calling create().
I tracked it in the debugger to
-private/options.js in the constructor for the Options class
constructor({ model, attribute, options = {} }) {
  const optionKeys = keys(options);
  const createParams = { [OPTION_KEYS]: optionKeys, model, attribute };

  // If any of the options is a CP, we need to create a custom class for it
  if (optionKeys.some(key => isDescriptor(options[key]))) {
    return OptionsObject.extend(options).create(createParams);
  }

  return OptionsObject.create(createParams, options);
}

I chatted with Chris Garrett in e-decorators on the Ember discord. He suggested that the computed properties were not being detected and that you would always need to extend. He said the same error would apply with Ember 3.10 and I have been able to reproduce it with the dummy app and ember-source upgraded to 3.10.0-beta.2
I attempted update the code to always return OptionsObject.extend(options).create(createParams);
Which got rid of the error but then some of the tests were not passing.
Please let me know if there is more detail that I can provide.

It seems like the issue here has something to do with the isDescriptor utility method. Do you happen to know why that might be?

So I did a little debugging and checked
ember 3.6 with ember-decorators-polyfill:

typeof "function"
isDescriptor is true
constructor dec - returned here https://github.com/pzuraq/ember-decorators-polyfill/blob/v1.0.3/vendor/ember-decorators-polyfill/index.js#L209

ember 3.10.0-beta.2

typeof "function"
isDescriptor is undefined
constructor COMPUTED_DECORATOR

@jderr-mx Did you ever find a solution to this?
@offirgolan Any ideas how to fix?

I started digging into the tests that are failing and it looks like the dependent keys for the model in theoptions object are not being recognized when the CPs are created.
I think it is caused because rather than an instance of a ComputedProperty object the computed decorator is a function. The result is there is no _dependentKeys array available when  extractOptionsDependentKeys runs.
function extractOptionsDependentKeys(options) {
  if (options && typeof options === 'object') {
    return Object.keys(options).reduce((arr, key) => {
      let option = options[key];

      if (isDescriptor(option)) {
        return arr.concat(option._dependentKeys || []);
      }

      return arr;
    }, []);
  }

  return [];
}

For reference, I have been debugging this test:
Integration | Validations | Factory - General: disabled validations - cp with dependent key

I was able to work around this issue by creating a dependentKeys object on the validation. This got all the cp related tests passing in PR @Alonski submitted.
    let Validations = buildValidations(
      {
        firstName: validator('inline', { validate: Validators.presence }),
        lastName: validator('inline', {
          validate: Validators.presence,
          disabled: not('model.validateLastName')
        })
      },
      {
        dependentKeys: ['model.validateLastName']
      }
    );

Inheritance integration tests in Factory - General are failing in Ember < 3.9 The Parent fields are not getting inherited by the child.

Getting the same error in Ember 3.10.0-beta.1. Any ideas how to fix it?

@alejandrodevs Can you try to point to this branch? #636
See if it works until we have a more permanent fix

I tried it and the error is still present.
Ember: 3.10.0-beta.1.
Ember CP Validation: Your branch

For 3.10 I had to update addon/-private/options.js to make the constructor always extend the object.
export default class Options {
  constructor({ model, attribute, options = {} }) {
    const optionKeys = keys(options);
    const createParams = { [OPTION_KEYS]: optionKeys, model, attribute };

    return OptionsObject.extend(options).create(createParams);
  }
}

@jderr-mx Could that have side effects in other versions?

I will have to see. I'll start by running the tests with ember-try which should turn up any potential issues.

For 3.10 I had to update addon/-private/options.js to make the constructor always extend the object.

This is very expensive to do and I would prefer to fix the underlying isDescriptor utility method instead if at all possible.

For 3.10 I had to update addon/-private/options.js to make the constructor always extend the object.

This is very expensive to do and I would prefer to fix the underlying isDescriptor utility method instead if at all possible.

That utility method relies on the property .isDescriptor which is no longer present in 3.10+
What would be a safe property to test?

@offirgolan from what I understood from @pzuraq we should always extend the object.

@offirgolan what they're saying is correct, the goal in the near future is to eliminate all of the the intimate/private APIs for computed properties. There are a few that have to remain around for the time being, but where there are alternatives we're trying to push people towards them. I think extending is a reasonable alternative, and it will be the only alternative in the future with native classes, so I would recommend moving to something like this now, or not using computed properties.

Any additional thoughts on how to update this addon?
Decorators have shipped in 3.10 which will cause a couple of issues if you are using a computed property in a validator, to disable it for example.

EmberObject.create no longer supports defining computed properties. issue
model.<someAttr> will need to be explicitly added to dependentKeys

For 3.10 I had to update addon/-private/options.js to make the constructor always extend the object.
export default class Options {
  constructor({ model, attribute, options = {} }) {
    const optionKeys = keys(options);
    const createParams = { [OPTION_KEYS]: optionKeys, model, attribute };

    return OptionsObject.extend(options).create(createParams);
  }
}

What about doing this to fix the first point?
EmberObject.create no longer supports defining computed properties. issue

When do we need to do this model.<someAttr> will need to be explicitly added to dependentKeys?

When do we need to do this model.<someAttr> will need to be explicitly added to dependentKeys?

See this example: https://github.com/offirgolan/ember-cp-validations/issues/635#issuecomment-485315979
The disabled attribute of lastName is set using a computed property

@jderr-mx Oh I see! at least we can get ride of that issue using dependentKeys in this moment. I think the first point should be fixed as it has a higher priority.
Any of us has worked on fixing that issue?

@alejandrodevs I have been trying to fix this. Running into a few issues. Check out this PR: #636

This fix by @GavinJoyce is working well.
https://github.com/offirgolan/ember-cp-validations/pull/636#issuecomment-494330352

Is the support for decorators in this addon widely used? If not, perhaps we could consider dropping them?

@GavinJoyce I managed to fix the issue with Decorators in your branch I think.
I will try to find time in the next few days to make a PR.
What are we going to do about versions older than 3.7 though?

@Alonski nice one, perhaps you could push up what you have?

What are we going to do about versions older than 3.7 though?

My changes work with earlier versions of Ember too.

Is the support for decorators in this addon widely used? If not, perhaps we could consider dropping them?

Or perhaps we could move the decorator support to an addon which wraps this one?

Or perhaps we could move the decorator support to an addon which wraps this one?

This looks the most reasonable. Keep decorators available without making the main addon unstable.

https://github.com/offirgolan/ember-cp-validations/pull/641 is 🍏. It drops support for decorators and supports all versions of ember from 2.16 -> 3.10

I'm running into the same issue when using the ember-intl's t CP macro inside an ember-awesome-macros CP macro.
import { and, or } from 'ember-awesome-macros';
import { t } from 'ember-intl';

This works:
  @t('assumptionGroup.options.partTime')
  partTimeName!: string;

  @t('assumptionGroup.options.pattern')
  patternName!: string;

  @or(
    and('isPartTime', 'partTimeName'),
    and('isPattern',  'patternName'),
                      'name'
  )
  nameEffective!: string;

This doesn't (but it reads much better!):
  @or(
    and('isPartTime', t('assumptionGroup.options.partTime')),
    and('isPattern',  t('assumptionGroup.options.pattern')),
                      'name'
  )
  nameEffective!: string;


No focus when typing selection

Starting with a selected item. When, to change the selection, the user starts typing, the selection moves to the end of the list and no active selection is displayed.
With the default examples, first select 'Wyoming', and then type 'a'.
Previously (and what is expected) the first item of the selection is visible and focused.
On the v4, the selection moves to the end of the list without any selection.
Previously:

On v4:

I can't seem to replicate this on the new site (v4.0.6-rc.1) - this may have been resolved as part of some of the recent fixes for other focus-related issues.


refactor(@angular/cli): remove compression plugin

Followup from #4618
BREAKING CHANGE: compressed code (.gz) is no longer generated on production builds.

I am a little surprised by this change. One thing I like about Angular-CLI is the way it handles all the steps needed to produce a production-ready deployment output.
I can see why gzipping may not always be appropriate, and as such may not strictly be a core feature of Angular CLI, but it is a step which requires some build tool on top of just naive copying between deployment folders.
This means that we'll have to add yet another build tool to the chain on top of Angular CLI to provide this step; something I am really not fond of.
Are there any plans for re-introducing this as an easy plugin?

@wulfsberg the reasoning for this change can be found https://github.com/angular/angular-cli/pull/4618#issuecomment-279175818
No plans of re-introducing it ourselves. Sounds perfectly possible with the upcoming plugin architecture.

Would not be better then to remove this feature once it's easy to do with the new plugin architecture?

https://github.com/angular/angular-cli/pull/4618#issuecomment-279175818 says that there is an easy workaround and asks for documenting how it can be enabled by end users. Where can I find such information? This PR removed the feature, but did not add any docs AFAICT.
Also, https://github.com/angular/angular-cli/pull/4702#issuecomment-280623583 mentions "upcoming plugin architecture" does this mean that the mentioned workaround is not possible until that feature lands?

At least give us a flag to enable it ╮(╯▽╰)╭

Is there any other ways to compress text files?

Hello guys!
I'm using workaround with gulp.
npm install --save-dev gulp
npm install --save-dev gulp-gzip
I changed script section of package.json
"scripts": {     "ng": "ng",     "start": "ng serve",     "test": "ng test",     "lint": "ng lint",     "e2e": "ng e2e",     "build-prod": "ng build -t production -e prod && gulp compress"   },
And I added gulpfile.js in the root directory (next to the package.json)
`var gulp = require('gulp');
var gzip = require('gulp-gzip');
gulp.task('compress', function() {
gulp.src(['./dist/.js', './dist/.css'])
.pipe(gzip())
.pipe(gulp.dest('./dist'));
});`
I run building via npm run build-prod

Hi @Anufriev,
that didn't worked perfectly. But changing to this works:
gulp.src(['./dist/*.js', './dist/*.css'])

oh man, so sad to hear, I loved the gzip feature 😞

You should guys leave it as special option with additional flag.

@Anufriev, did you have to modify the index.html file or the lazy load?  I tried your suggestion and I was not able to get it to work.

@filipesilva I wasn't able to find any documentation on the new plugin architecture for angular-cli. Is it still in development and if so will it be released soon? So far using ng eject does the job for me but I'm interested in what could be achieved with the plugin architecture :)

So, what is the "easy workaround" then? https://github.com/angular/angular-cli/pull/4618#issuecomment-279175818 talks about it but does not explain it ...


unexpected error when I set the property

Describe the bug
When I edit the value in the set property, if I type in quickly, the result will be presented very slowly. And often throw an error.

and some lg file will be deleted automatically.

Version
"3a7dbce5c902d"
Browser

[x] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior

Screenshots

Additional context

After some dig, It can only be re-pro on an old laptop + yarn build && yarn start

addressed in #2786


Updating menu bar

related to #53

Maybe remove users link from the navbar also?

LGTM. Thank you:)

Great fix, maybe delete the Users link? since we said we won't make this page, and you already changing the navbar here.

No description of the change that is intended to be made anywhere - not in the commit messages, not in the PR description and not in the issue.
How am I supposed to understand what kind of change did you intend to do? "Updating menu bar" is a far too generic title.


No GPU displayed for Ubuntu on Windows 10 WSL

GPU: "GPU NAME"
does not display and can not be enabled.

wsl still doesn't support gpu

I think this is the case, it should use lspci (if it works).


⚠ Cioos Atlantic CKAN has degraded performance

In 91a15b9, Cioos Atlantic CKAN (https://catalogue.cioosatlantic.ca/) experienced degraded performance:

HTTP code: 503
Response time: 18297 ms

Resolved: Cioos Atlantic CKAN performance has improved in b6c880d.


Introduce github actions

Add github actions

Great, thanks! Will merge when green and then we can check that it's working in a follow-up to remove Travis CI.

Codecov Report

Merging #110 (26b27de) into master (8b0f7e6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #110   +/-   ##
=======================================
  Coverage   86.39%   86.39%
=======================================
  Files          18       18
  Lines         272      272
  Branches        7        8    +1
=======================================
  Hits          235      235
  Misses         37       37

Continue to review full report at Codecov.

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

@erwan Do you want to open a follow-up PR to remove the Travis CI config, or should one of us?


Removing healthcheck function due to platform inconsistency

#34

@pebeto Can you please merge and tag a new release when ready?


having trouble setting up a local dev environment

👋 hello! this is a great project and i'd really like to become a contributor.
i'm trying to set up a local dev environment following the docs and so far not able to so. i've gotten as far as launching the frontend and the api locally with the former pointed to the latter, but i'm getting errors like this in the api:
{"level":"error","message":"Could not find sites by id in the database:  connect ECONNREFUSED ::1:5432","metadata":{"module":"/Users/simonfishel/opt/git/OpenClimate/api/orm/sites.ts","name":"SequelizeConnectionRefusedError","original":{"address":"::1","code":"ECONNREFUSED","errno":-61,"port":5432,"syscall":"connect"},"parent":{"address":"::1","code":"ECONNREFUSED","errno":-61,"port":5432,"syscall":"connect"},"service":"OpenClimate-hub-controller","stack":"SequelizeConnectionRefusedError: connect ECONNREFUSED ::1:5432\n    at Client._connectionCallback (/Users/simonfishel/opt/git/OpenClimate/api/node_modules/sequelize/src/dialects/postgres/connection-manager.js:179:24)\n    at Client._handleErrorWhileConnecting (/Users/simonfishel/opt/git/OpenClimate/api/node_modules/pg/lib/client.js:318:19)\n    at Client._handleErrorEvent (/Users/simonfishel/opt/git/OpenClimate/api/node_modules/pg/lib/client.js:328:19)\n    at Connection.emit (node:events:513:28)\n    at Connection.emit (node:domain:489:12)\n    at Socket.reportStreamError (/Users/simonfishel/opt/git/OpenClimate/api/node_modules/pg/lib/connection.js:57:12)\n    at Socket.emit (node:events:513:28)\n    at Socket.emit (node:domain:489:12)\n    at emitErrorNT (node:internal/streams/destroy:151:8)\n    at emitErrorCloseNT (node:internal/streams/destroy:116:3)"}}

so i'm guessing i need to also have a database running, but can't find any instructions on how to do that.
could you provide some instructions on the standard flow for local development? it can be high-level, i'm happy to figure out the details myself as part of learning about the tech stack. and if you're up for it i could send you a PR with updates to the contributing docs. thanks!

Hi :)
Which operating system are you using?
I've put together a PR with scripts and documentation for running k8s locally on Linux, which you can find here:
#167

oh great, i missed that k8s directory, but it's what i was looking for, thanks! i'm on a mac, so looks like i should be all set.
out of curiosity, if i'm just making changes to the frontend, is it possible/recommended to point it to the production api instead of running all of the services locally?

Yes that's possible, I've done the same thing. You should be able to replace the value of api in server.environment.ts, which is used for all API requests the frontend makes.


added raw csv files (output of to_csv.py)

"turning holders into Contributors and all sticks from the same holder into ValueSets of one Contribution" - ok, this would be done in to_cldf.py - regarding ling_area, well, 15 areas have no glottolog code, and afaik these info are a bit vague

A Language isn't required to have a glottocode. In fact it, isn't required to be anything language like at all. In the D-PLACE app, societies are modelled as Language. And considering that "linguistic areas" are the only thing in AMSD we might want to put on a map, modelling them as Language seems appropriate.

Btw. to_cldf.py would be a bit of a misnomer, right? I can't see what kind of CLDF dataset this could be turned into.

re Language: ok, re to_cldf: ok, I could write initialised.py in such a way to use the raw/*.csv directly


GPT3.5 and GPT4 Config

Allow user to specify what model they want
model = optimizedlanguage(model="gpt4", api_key="ssk")

There is a pr for this https://github.com/kyegomez/tree-of-thoughts/pull/6, you can then set api_model with gpt-4


Extract signature crate

Problem
solana_sdk::signature::Signature needs to be made available outside of solana-sdk.
Summary of Changes

Make a new solana-signature crate and move Signature, SIGNATURE_BYTES, and ParseSignatureError there
Re-export these items in solana_sdk::signature for backwards compatibility
Make the serde*, rand and ed25519-dalek dependencies optional in this crate. There a bunch of cases where we want Signature but don't want all these extras

Even though it's outside the program sdk, it's a really obvious candidate for splitting out of the non-program SDK, kind of like Hash, since it's just encapsulating 64 bytes. If I'm not mistaken, we've even had Agave devs want this, so I'll be happy to take a look through it

I noticed we could swap out GenericArray but I've made a separate PR for that in case it turns into a rabbit hole https://github.com/anza-xyz/agave/pull/2857

I noticed we could swap out GenericArray but I've made a separate PR for that in case it turns into a rabbit hole https://github.com/anza-xyz/agave/pull/2857

I deliberately didn't mention it for exactly that reason :sweat_smile:


Custom fields

For donation pages on a per-marathon basis, have custom questions via open text/checkbox/selection.
Examples:

Team Green Gaming can have a selection box with locations for trees planted
Marathon X can ask "Tick this box if you allow us to send you a newsletter 4 times a year"
Marathon Y that wants freeform text for any reason, like "random donator gets to name my baby, what's your pick?"

Oengus implementation:
(if you're fast, check out bsgo7)
https://i.imgur.com/yCntPT2.png

Current build has checkboxes and freeform inputs, select menus to come later.


Update README.md

Updated README.md to include steps for testing Stackle API
Changes proposed

Added quick steps on running the unit tests in the Stackle API
That's it

oh..yep, my bad. I'll change that real quick


Node Polyfills

This will correct "Buffer is not defined" errors in console that occur when using wallet clients from use-wallet that use Buffer.from.

Thanks for PR!
I believe you and Doug discussed this separately, and this is handled by use-wallet v3 beta 3.
Closing.

For reference, we opted for the broader solution of resolving the issue in the library itself: https://github.com/TxnLab/use-wallet/pull/153


chore(docs): update README.md

Make features section a list instead of paragraph

Hello, I wonder what's wrong with the CI. @Whoaa512


How can I get the val_loss value when training?

How can I get the val_loss value when training?
I know that when training in mmcv, the loss value is Validation, and Test returns the mAP value, not the loss value.
When I posted a question before, I was told to get the val_loss value through single_gpu_test() or multi_gpu_test() of eval_hooks.py, but I don't get a sense of it.
Then, I saw the answer in another question, it was a method to train the validation dataloader once more and return the loss value.
Reference link: https://github.com/open-mmlab/mmdetection/issues/7479
However, I haven't been able to find a way to modify the cfg pipeline.
Can you give me an example or method to get val_loss? It's so hard to know how to weave hooks

Similar: https://github.com/open-mmlab/mmdetection/issues/5694
You just need to set workflow = [('train', 1),('val', 1)] in the configuration

Thanks!,
However, that error occurs.
TypeError: forward_train() missing 2 required positional arguments: 'gt_bboxes' and 'gt_labels'
I think I need to edit the config value of data.val.pipeline,
in epoch_based_runner.py
def run_iter(self, data_batch, train_mode, **kwargs):
         if self.batch_processor is not None:
             outputs = self.batch_processor(
                 self.model, data_batch, train_mode=train_mode, **kwargs)
         elif train_mode:
             outputs = self.model.train_step(data_batch, self.optimizer,
                                             **kwargs)
         else:
             outputs = self.model.val_step(data_batch, self.optimizer, **kwargs)

Is there a way to modify the model's data.val.pipeline before executing val_step() in ?

Thanks!,
However, that error occurs.
TypeError: forward_train() missing 2 required positional arguments: 'gt_bboxes' and 'gt_labels'
I think I need to edit the config value of data.val.pipeline, in epoch_based_runner.py
def run_iter(self, data_batch, train_mode, **kwargs):
         if self.batch_processor is not None:
             outputs = self.batch_processor(
                 self.model, data_batch, train_mode=train_mode, **kwargs)
         elif train_mode:
             outputs = self.model.train_step(data_batch, self.optimizer,
                                             **kwargs)
         else:
             outputs = self.model.val_step(data_batch, self.optimizer, **kwargs)

Is there a way to modify the model's data.val.pipeline before executing val_step() inside, other than to modify the pipeline value by externally modifying config.py?

I get the same error...
Has anyone managed to get this right?


fix: include validation to author in Check PR -force update

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: caykemanjanelli
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Fix minor issues with custom_asset example

Objective

Use bevy's re-exported AsyncReadExt so users don't think they need to depend on futures-lite.
Fix a funky  error text

In the same vein, even if Reader https://docs.rs/bevy/latest/bevy/asset/io/type.Reader.html exists, it is not directly used in the signature of  AssetLoader's load: https://docs.rs/bevy/latest/bevy/asset/trait.AssetLoader.html and since AsyncRead is not reexported it might lead people to depend on futures-io.

it is not directly used in the signature of AssetLoader's load:

I may just be confused (if so, please help me out), but I think it is? https://github.com/bevyengine/bevy/blob/aaef5577cdbb5f7256d177be5fc98b0965d67753/crates/bevy_asset/src/loader.rs#L35
And in the custom_asset example:
https://github.com/bevyengine/bevy/blob/aaef5577cdbb5f7256d177be5fc98b0965d67753/examples/asset/custom_asset.rs#L40

Ah, I guess it's rustdoc that expands it then :(

I had to add futures-io (AsyncRead) and futures_lite ( Stream) to dependencies in order to implement custom AssetReader  + import futures_lite::StreamExt to iterate on result from read_directory. It would be nice if all those types could be reexported


#166 Add rehydrated callback

This PR adds rehydrated callback after store.replaceState has been called.
P.S. Why does it seem like nobody actually uses prettier to format the code?

Thanks for the PR. Could you remove the changes you've done by "fixing" prettier? Now it's quite hard to see the actual change you've made with all the indentation changes. The prettier config was just fine, nothing needed to be changed there.

Could you please elaborate the purpose of the callback? And the option is also missing in the README.

The thing I am trying to achieve here is mainly targeted to Nuxt.js users.
Since Nuxt.js users would have to implement window.onNuxtReady() to make the persisted state from localStorage to work, there is a slight time gap between when Nuxt is ready and when persisted state is re-hydrated.
Therefore, if one wishes to access data that is stored in Vuex during the mounted lifecycle, one would get a Nuxt re-hydrated value, instead of the localStorage persisted value.
By providing a rehydrated callback, users can commit a mutation to the store, changing a stored value, say isLoaded, to true, which can be a signal that indicates the state has been retrieved from localStorage.
I hope that I have explained my motive to implement this. I will make changes to the README file later.

@nelsliu9121  rehydrated isLoaded Can you tell me how to use in NuxtJs?
I encountered problems during use. When i use plugins read store state, it's empty . Because load data after window.onNuxtReady()
so i want to know  When is it loaded before reading the data?
Can you give me some examples?
Thank you very much.

Awesome! I'd like to see this merged.

Another usecase is when using navigation guards for authentication and redirection in Vue. The beforeEnter guards need a way to know when rehydration is finished so it knows when it's safe to check.

Any reason this hasn't been merged in yet?

I also need this functionality to be able to use this package.

@nelsliu9121 thanks for your work!


Failed to Start Application (LogMessage)

On one test system (running a 0.14 release), this error started to occur. ChocolateyGUI was previously functional but now it crashes immediately upon startup.
2017-08-31 07:46:28.563 -04:00 [Fatal] Failed to start application.
System.ServiceModel.FaultException`1[System.ServiceModel.ExceptionDetail]: The operation 'LogMessage' could not be completed because the sessionful channel timed out waiting to receive a message.  To increase the timeout, either set the receiveTimeout property on the binding in your configuration file, or set the ReceiveTimeout property on the Binding directly. (Fault Detail is equal to An ExceptionDetail, likely created by IncludeExceptionDetailInFaults=true, whose value is:
System.ServiceModel.CommunicationObjectAbortedException: The operation 'LogMessage' could not be completed because the sessionful channel timed out waiting to receive a message.  To increase the timeout, either set the receiveTimeout property on the binding in your configuration file, or set the ReceiveTimeout property on the Binding directly.

Server stack trace:
   at System.ServiceModel.Channels.ServiceChannel.ThrowIfIdleAborted(ProxyOperationRuntime operation)
   at System.ServiceModel.Channels.ServiceChannel.Call(String action, Boolean oneway, ProxyOperationRuntime operation, Object[] ins, Object[] outs, TimeSpan timeout)
   at System.ServiceModel.Channels.ServiceChannelProxy.InvokeService(IMethodCallMessage methodCall, ProxyOperationRuntime operation)
   at System.ServiceModel.Channels.ServiceChannelProxy.Invoke(IMessage message)

Exception rethrown at [0]:
   at System.Runtime.Remoting.Proxies.RealPr...).

Hallo @TheVakman , thanks for the report - can you be a bit more specific concerning the version of ChocolateyGUI you're using? (maybe even a commit hash?) - where did you install from?

I installed from the MyGet repository at the time. I can again update now.
0.14.0-unstable.290+Branch.develop - not sure if that's good enough.
Also, thanks for adding the versions back in!

Is chocolatey itself still performing normally?

Sorry, I forgot to reply. Yes, it is - completely functional.
However, I think we can close this. I was able to reproduce consistently but the .300 release doesn't seem to have this issue for me. It's also possible that the uninstall/reinstall fixed the issue.


Discord logo missing from titlebar

Describe the bug
The discord logo is completely missing from the titlebar, other icons work.
To Reproduce
It happens when you install Slate by default.
Screenshots

Desktop (please complete the following information):
Discord channel: PTB
OS: Windows
Mod: BetterDiscord
Additional context
This can be somewhat fixed by adding this CSS:
.typeWindows-1za-n7:before {
  content: '';
  position: fixed;
  background: url('https://discord.com/assets/28174a34e77bb5e5310ced9f95cb480b.png') center/cover no-repeat;
  width: 20px;
  height: 20px;
  left: 12px;
  top: 4px;
}

I cannot recreate.

For some reason it didn't load for me, maybe due to occurrence of other themes.


Default terminal is sh, not bash

Current the terminal in qhub opens to sh. Per @dharhas:
Jupyter terminal isn’t an actual interactive shell so you have to make a .profile that then sources .bashrc

conda init doesn’t work otherwise because .bashrc isn’t sourced when a terminal is opened

fixing it to open bash and setting up the .profile to source .bashrc should be added to qhub

This is also related to an error when running conda activate default from the default terminal:
$ conda activate default
/bin/sh: 5: /home/conda/environments/default/etc/conda/deactivate.d/geotiff-deactivate.sh: [[: not found
/bin/sh: 5: /home/conda/environments/default/etc/conda/deactivate.d/gdal-deactivate.sh: [[: not found
/bin/sh: 11: /home/conda/environments/default/etc/conda/deactivate.d/gdal-deactivate.sh: [[: not found
/bin/sh: 6: /home/conda/environments/default/etc/conda/activate.d/gdal-activate.sh: [[: not found
/bin/sh: 10: /home/conda/environments/default/etc/conda/activate.d/gdal-activate.sh: [[: not found
/bin/sh: 27: /home/conda/environments/default/etc/conda/activate.d/gdal-activate.sh: [[: not found
/bin/sh: 6: /home/conda/environments/default/etc/conda/activate.d/geotiff-activate.sh: [[: not found

Properly activate bash and source the .profile to load conda properly

@prasunanand please make this change for qhub and then apply it to all our current qhub deployments (quansight, datum and training)

@dharhas Working on it.


Proposal: add support to expose functions on a public domain

Each function could have a new boolean attribute named exposed, besides that, the server could accept connection on a public domain specified by the environment variable.
All exposed function may be accessible onhttp://${EXPOSABLE_HOST}/${namespace}/${functionID}, avoiding to expose the private API on the same domain.

Coverage decreased (-1.0%) to 89.462% when pulling 41b402a0e5f7b5db3004dcf340db2598af4d2cf1 on http-invocator into 06e4845c2735858703aaa108f97eb456f4e525bd on master.


admin cli package

centralized location for scripts used by dittofeed developers and system administrators

Neat! Worth updating the README to mention this feature.

@samlader  I've included a link to our Contributing docs in the README, and added a note to those docs!


Issues with rendering :nth-of-type

:nth-of-type(n + {int}) renders as :nth-of-type(+{int}) (where int is just a number).
I was expected to see :nth-of-type(n + {int}) instead.

It should - fix is coming.

Fixed in 4.0.8.


Google Tag Manager support

This PR adds Google Tag Manager support.

Google Tag Manager embedding snippet
utility function to push to dataLayer: lib/gtm.js
emits custom events like routeChangeStart, routeChangeComplete, and dataLoaded so that developers can build triggers on that

Forgot to mention that this PR also disables static-page optimization for landing page (pages/index.js), as discussed with @LucienLee  in 11/20 meeting.


Keep notification pop-ups with tool instructions visible at all times

When I pressed on the "draw walls" button, a notification popped up explaining how to use this tool. However, after clicking around for a bit, the notification disappeared and I forgot what I had to do to exit the tool 😅
I think it would be more helpful to keep this notification (or some similar info display) visible at all times while using the tool, so the user doesn't have to remember what was initially written on the popup.

There is the Help toggle (question mark button in the toolbar) that has togglable and more detailed tool instructions. I agree it is a bit hard to find and thus bad UX. If anyone would love to try their hand at improving the UX and making the help button more obvious (or even improving the instructions), I would be very open to the idea and you can be as creative as you want as long as it fits the rest of the app stylistically.
I'll leave this as help wanted with an ui/ux focus. :)


Intersect a ray with triangulate using gpu

Hello, i am a student interesting in ray intersection. I desires to know how can i intersect a ray with triangulate using gpu. Is there any code abort ray intersection in this project?

Hi, the code you are looking for is in the OpenCL kernels (.cl files) in the following subdirectory:
RadeonRays_SDK/RadeonRays/src/kernels/CL/.cl
You will find the code for the ray triangle intersection in common.cl in the function:
INLINE float fast_intersect_triangle(ray r, float3 v1, float3 v2, float3 v3, float t_max)
The other files (intersect_.cl) contain the code used for the traversal of the different bounding volume hierarchy variants supported by RadeonRays.
One word of warning though: At the moment it looks like AMD likely abandoned this project. So you probably won't get any "official" responses/support from AMD.

I appreciate your help. Thank you so much.


Use jQuery consistently

Here's your first pull request! This updates a few lines to use jQuery instead of the native equivalents. It also creates the audio element in one place instead of on every function call.
Nice work!

The following creates new objects as opposed to div's.
var node = $('<div>', { 'class': 'circle' }, '</div>');

Correction, the following code creates objects as opposed to div's:
var node = $('<div>', { 'class': 'circle' });


Unable to open VSCode from WSL

Version
10.0.22581.1
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu  20.04
Other Software
VSCode 1.65.2
VSCode Remote - WSL v0.64.2
Repro Steps
Open a Windows Terminal into WSL, enter the command "code ."
Expected Behavior
Opens VSCode on my host machine.
Actual Behavior
Does nothing, command just hangs indefinitely.
Diagnostic Logs
Prior to updating to the win11_22581.1 ni_release, when I used "code ." in WSL it opened VSCode in an elevated Windows Terminal and gave me a permission error in a non-elevated terminal.  Now the command does nothing in both.

I'm not sure if this is related, but I have the same problem for bringing up VSCode from an arrangement of Slackware under WSL. But it works elsewhere.

Please provide the logs as mentioned here https://github.com/microsoft/WSL/blob/master/CONTRIBUTING.md

@aedryan If you have installed vscode inside the wsl, then you should uninstall it and install vscode in your host (windows) machine. Then restart the wsl and try again. Also, vscode remote should be automatically downloaded and installed. So, no manual steps shall be required.

Please provide the logs as mentioned here https://github.com/microsoft/WSL/blob/master/CONTRIBUTING.md

Thanks, I went to do this and suddenly it started working again, I'm confused but this may not be an issue or whatever state I managed to get my system in that caused the issue eludes me.

@aedryan If you have installed vscode inside the wsl, then you should uninstall it and install vscode in your host (windows) machine. Then restart the wsl and try again. Also, vscode remote should be automatically downloaded and installed. So, no manual steps shall be required. Important: Vscode in wsl is only available in terminal, not in any vnc applications.

VSCode is installed on my host machine.  I already had VSCode and the remote extension installed, this happened after a Windows upgrade.  I was indeed invoking this from Windows Terminal.

I believe I run into the same issue on a daily basis. During my daily development routine (running Windows Terminal, GIT, Minikube, Lens IDE, VS Code, and Docker) I'll run into a system state where certain actions (code ., get fetch -ap, explorer.exe $(wslpath -w .) to name the most common ones) just hang indefinitely through the shell. When I discover this hanging for any single command it is a clue the whole system is in trouble. Previously WSL issues (a month or two prior) were resolved with a wsl --shutdown but now things get so stuck I need to do a full system reset to resume using the Linux half of my computer. Based on the timing I'd guess the progression of this issue came from swapping to the Windows Store version of WSL.
Here are a few system details if it helps provide a focus.
C:\Users\user>cmd.exe --version
Microsoft Windows [Version 10.0.22000.556]

user@user:~$ uname -a
Linux user <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Wed Mar 2 00:30:59 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

C:\Users\user>winget list
Name                              Id                                Version           Available    Source
---------------------------------------------------------------------------------------------------------
Docker Desktop                    Docker.DockerDesktop              4.6.1                          winget
Windows Terminal                  Microsoft.WindowsTerminal         1.12.10393.0                   winget
Windows Subsystem for Linux Prev… MicrosoftCorporationII.WindowsSu… <IP_ADDRESS>
Microsoft Visual Studio Code      Microsoft.VisualStudioCode        1.65.2                         winget
Windows Subsystem for Linux Upda… {8BC9BA1B-F6F3-471D-8773-5283F0C… <IP_ADDRESS>
Windows Subsystem for Linux WSLg… {E04B0005-A349-4BCC-9662-CA01320… 1.0.26

I'll follow this issue and if @aedryan is unable to reproduce it I could always create a new issue with details from what I've seen.

I can vouch for having Docker running in my configuration when the issue is reproduced as well as making use of WSLG.

I am running into this issue daily it seems.
Here is a link to my strace output when running the command.  https://gist.github.com/mlohstroh/c0f6a9325377069a0fb18027e8c49894
The reverse also happens where I cannot access the WSL filesystem when trying to go through the network drive that WSL makes for the windows side.
So this is much bigger than just vscode. It's really not allowing any cross OS file system interaction to happen.

I would agree with @mlohstroh. It feels a problem generalized to networking between the systems. Local programs in WSL continue to work (nano for example) but any program that calls between the systems seems to hang indefinitely. The last time my system got in this state I tried accessing \\wsl.localhost\home from Windows explorer and it became unresponsive. Interestingly enough I was able open VSCode through the Windows search and view + change files within WSL while the system was in this broken state.

Sorry if this is off topic for this issue but if it is indeed more generalized I thought it might be worth describing.  Since updating to win11_22581.1 ni_release I have gotten my system into such a state that I cannot access WSL at all and even gotten into a state where running wsl --shutdown locked up my system entirely.  I was able to recover with a reboot in both instances.

/dupe 7883


Various enhancements

This PR contains various enhancements/cleanups, e.g.

Remove whitespace/uncommented code
Remove annotations without effect
The created and modified properties of PersistentObject will be ignored when being serialized by Jackson (@JsonIgnore). This can be overwritten in subclasses (see Application)
Make use of @JsonIdentityInfo and @JsonIdentityReference annotations to serialize objects (or collections) as simple values
Rename column definitions in hibernate annotations to avoid problems with Oracle (30 char identifier limit)
Use ToStringBuilder.reflectionToString everywhere

Please review and feel free to merge.

For the next batch: I personally prefer multiple PRs with ine dedicated topic. Not a blocker.

I added some minor comments, which you may want to address. Otherwise looks good. Thanks @buehner

I added two commits. The first one fixes the wrong JavaDoc. The second one replaces INDEX with IDX. You're right @marcjansen . This should avoid conflicts with reserverd words.

Good to go now

Thanks!


Floating point exception in MaxPool3DGrad

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
tf 2.9 and 2.11.0-dev20220828
Custom Code
No
OS Platform and Distribution
Linux Ubuntu 20.04
Mobile device
No response
Python version
3.10
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
An FPE can be triggered when the channel of `orig_input` is zero, which may cause a DoS.

Standalone code to reproduce the issue
import tensorflow as tf
with tf.device("GPU:0"):
    orig_input = tf.random.uniform([16, 28, 30, 10, 0], dtype=tf.float32)
    orig_output = tf.random.uniform([16, 2, 1, 2, 0], dtype=tf.float32)
    grad = tf.random.uniform([16, 2, 1, 2, 0], dtype=tf.float32)
    ksize = [1, 22, 12, 17, 1]
    strides = [1, 16, 30, 7, 1]
    padding = "SAME"
    data_format = "NDHWC"
    res = tf.raw_ops.MaxPool3DGrad(
        orig_input=orig_input,
        orig_output=orig_output,
        grad=grad,
        ksize=ksize,
        strides=strides,
        padding=padding,
        data_format=data_format,
    )

Relevant log output
Floating point exception (core dumped)

Facing the same issue.

@shijy16,
Could you please confirm where the issue persists when the code is executed in GPU mode only or CPU too? Thank you!

@tilakrayal
Thanks for response. This issue seems to be GPU mode only.

@shijy16,
Looks like this is a duplicate of issue #57497. Could you please confirm and close this issue, since it is already being tracked there? Thank you!

@tilakrayal
Hi, issuse #57497 is a check-fail, but this one is a Floating-Point-Exception, the input are different too. So I donot think they are the same issue.

@gadagashwini,
I was able to reproduce the issue on tensorflow-gpu v2.8, v2.9 and nightly where it was executed on tensorflow-cpu without any error. Kindly find the gist of it here.

@shijy16,
I tried to execute the mentioned code on tf-nightly(2.15.0-dev20230918) and it was executed without any error/issue or the crash. Kindly find the gist of it here. Thank you!


PLAT-80872: Support Expandable Item in VirtualList

Checklist

[X] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[X] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Needed to expand item height dynamically.
Resolution
Support Expandable Item in VirtualList.
Links
PLAT-80872

LGTM for logic, and we may need to discuss APIs. For now, LGTM.


UIOPT Enable Toggling between Mandara Dark and Light Themes

Improved:
Implemented:
Documented:
Completed:
Reverted:
Fixed:
(OFBIZ-)
Explanation
UIOPT-61 Enable Toggling between Mandara Dark and Light Themes
Thanks:

Merged, since review was already performed


dotnet-sdk: Add dotnet-sdk 3 and update dotnet-sdk2

[x] add dotnet-sdk3-lts
[x] update dotnet-sdk-lts

Closes #250

jsonpath does not support hyphens(-), will work on that later.

Closed in favour of #186


[#53] Update DeveloperGuide.md

Use cases

#53

Codecov Report

Merging #72 (cd7588a) into master (0373258) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #72   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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


[ENHANCEMENT] Reduce the bundle size impact of upgrading from 2.9.2 to v3

Description
BundlePhobia shows a 63% bundle size increase in 3.0.0.

It would be great if the SDK's module/tree shaking design allowed consumers to only incur bundle growth for features that they use.
Benefits
It would make it more likely for consumers to stay up to date with the latest versions of the SDK.
As it is, I'm recommending that we stay on 2.9.2 for our projects.
Detail
No response
Examples
No response
Risks/Downsides
No response

Agree. 💯
Now that our functional requirements are satisified for this new version, we'll be diving into reducing the underlying JavaScript SDK size and then find opportunities to reduce the React SDK as well.


Option to specify query in command line

xtt-stats --query="where isWork group by $workspace"

done


Calculator not working: number-handler.js not found

I have discovered Script Kit yesterday and have been really enjoying it. I couldn't imagine a better way to quickly create, manage and use personal scripts and integrate them into my workflow and OS.
Thank you for building this!
I have been experiencing one issue: when trying to run the built-in calculator I get the following error message:

As I have just pressed the key "2", the error shows number-handler.js 2 not found.
Is this a known problem, or is this functionality still work in progress?

@daanlenaerts Sorry, it's a known issue. You can try out the latest pre-release if you'd like to use it today:
https://github.com/johnlindquist/kitapp/releases/tag/v1.49.5

@johnlindquist Thank you very much for your swift reply. I have tried it out in the pre-release, and now it does indeed work. Thanks a lot!


Add instrumentation

There's no metrics yet, it should really have them.

I plan to contribute a beginning of this, the first steps I have in mind are:

include Prometheus client library;
add counters for error rates;
add request duration histograms for the http endpoints.

What are your thoughts?

Sounds good.


Custom Widget on Survey

Are you requesting a feature, reporting a bug or asking a question?
Question: Hello I'm trying to create a custom widget, I've been looking at several examples specially this one https://next.plnkr.co/edit/FTNXuTmI9RwgCRjc?preview , however the widget is not visible on the survey toolbar. Could you please help me please? thanks.
What is the current behavior?
Custom widget not visible on toolbar
What is the expected behavior?
Custom widget visible on toolbar
How would you reproduce the current behavior (if this is a bug)?
Showing code
https://repl.it/repls/PristineWrathfulScreenscraper
Specify your:
browser: Google Chrome
browser version:78.0.3904.70 (Official Build) (64-bit)
surveyjs platform: react
surveyjs version: latest

SurveyCreator uses survey-knockout library
I've updated your sample - https://repl.it/repls/InferiorSeparateRobot
import * as SurveyKo from "survey-knockout";
import init from './widget';
init(SurveyKo);


Add regular expression filters

This PR adds regular expression filters per #1.
A significant concern is the balancing of user input sanitization vs. flexibility. Regular expressions could cause a ReDoS vector without filtering if the tool is used upstream in other projects. However, filtering removes groups, or operators, * extensions, and more, essentially making regular expressions useless.
I have used timeouts to prevent long-running regular expressions (e.g., preventing ReDoS through execution limits). This required the non-standard-library package regex, which provides timeouts.
The new filters are:

/ -> REGEX
^/ -> NOT_REGEX

The new info page is:

+----+
|NOTE|
+----+
Jockey is a work-in-progress currently only supports querying:
    units
    machines

+-------+
|FILTERS|
+-------+
Filters have a three-part syntax:
    <object type><filter code><content>

<object type> can be any supported Juju object types or their equivalent
abbreviations (see "SHORT NAMES", below).   I will get to that shortly.

Closing: this will be included in a larger overall refactor I'm developing.

Okay.  What are you planning to refactor?

#47 complex caching and deeper introspection


[dogstatsd] fix replay speed

What does this PR do?
Change timing logic. Read is now aware of total capture/replay time.
Motivation
dogstatsd-replay is slower than expected, replaying a 10s capture takes 16s.
time agent dogstatsd-replay -f /opt/dsd_capture/prod
Replaying dogstatsd traffic...
clearing agent replay states...
replay done

real	0m16.802s
user	0m0.613s
sys	0m0.446s

Additional Notes
Current logic sleep for inter-message interval and does not keep track of total time (accumulating error over time), leading to slower than capture message replay due to:

time.Sleep not being nano second accurate
time.Sleep pauses the current goroutine for at least the duration d

Possible Drawbacks / Trade-offs
dogstatsd-replay produce higher through output.

Describe how to test/QA your changes
Run: time ./bin/agent dogstatsd-replay -c bin/agent/dist/datadog.yaml -f $DSD_CAPTURE

/merge


Keymap not recognised

Checkhealth
health#compe#check
compe:snippet

OK: snippet engine detected.

compe:mapping

OK: compe#complete is mapped: (i  <C-Space>   * compe#complete())
INFO: compe#confirm is not mapped
OK: compe#close is mapped: (i  <C-E>       * compe#close('<C-E>'))
OK: compe#scroll is mapped: (i  <C-D>       * compe#scroll({ 'delta': -4 }))

Describe the bug
I cant get snippets to work for the life of me...
The FAQ says just use the mapping for compe#confirm.
I did but it doesnt work for some reason. Check health also says i have not mapped it , when i have. (Just copied the mappings)
I even followed the LSP snippets in the faq but nothing changed.
Expected behavior
Screenshots (optional)
Additional context (optional)

Could you provide your vimrc?

Could you provide your vimrc?

I am no OP, but am having this issue. My configuration for nvim-compe bindings can be seen here. Only control J/K are not working.

I guess you are using CR mapping. You should check if another plugin use the mapping key you have defined.
Run :imap to see the current mapping
The plugin 'cohama/lexima.vim' was the issue in my case

I guess you are using CR mapping. You should check if another plugin use the mapping key you have defined.
Run :imap to see the current mapping
The plugin 'cohama/lexima.vim' was the issue in my case

Oh I forgot about this issue, I moved to just using tab and s-tab.


How do I change the starting show location?

How do I fix Popover starting position by depending on the left or right, rather than from the center
如何让 Popover  起始位置固定是依赖左边或者是右边，而不是从中心位置显示

You can do the following:

Adjust the view's frame, which will be shown in the popover
Adjust the popover type .type(.up) or something similar.


Redeploy will change from ahub to payasyougo

Added the below note column:

[!NOTE]
If you already have your SQL VMS on Azure and they are converted from Payasyougo to AHUB and if there is a requirement to add this machines to Availability Set, then you can redeploy the VMS and attach existing disks. This will change your license back to payasyougo so you need to again change back to AHUB post redeployment

Icm for same is at:
https://icm.ad.msft.net/imp/v3/incidents/details/113765561/home

@ujjwalpatel910 : Thanks for your contribution! The author, @xujing, has been notified to review your proposed change.

Hi @xujing-ms - Could you review the proposed changes? Also, when you have a chance, could you update the article's metadata so that the author value is your GitHub user name and the ms.author value is your Microsoft alias? Thanks.


Setting to disable sound when using backspace in Integrated Terminal?

This could just be me, but when using the Integrated Terminal with "terminal.integrated.shell.windows"  set to powershell.exe and deleting characters with backspace there is a high-pitched, old-school sound warning when there are no more characters to delete back. It would be wonderful if there is a setting to disable that sound, like "terminal.integrated.sound": "false"
I also recorded the audio here

VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-09T00:26:45.394Z)
OS Version: Windows_NT ia32 10.0.14393

You'll need to do this by configuring PowerShell or changing the shell http://superuser.com/a/1113450

How about making is an option still ... and invoking that as part of creating a powershell session from vscode? - Save people having to dig.

That sort of feature should probably belong in the powershell extension https://github.com/PowerShell/vscode-powershell

Here's another vote for this feature to be added.  It's unpleasant and is different behavior from all other terminals.


Fix IntelliSense

can someone please merge this?

@u3u There are two ways to declare module types 1) Top Level, and 2) Ambient Modules.
Chalk is using top level declarations.
This PR adds an ambient module definition that tries to augment the top level module's definition of the export.

Apologies, I should have actually checked the code before suggesting a PR. I assumed the type syntax would (correctly) be as follows:
declare module "chalk" {
  export const enum Level {
    None = 0,
    Basic = 1,
    Ansi256 = 2,
    TrueColor = 3
  }

  export interface ChalkOptions {
    enabled?: boolean;
    level?: Level;
  }

  export interface ChalkConstructor {
    new (options?: ChalkOptions): Chalk;
    (options?: ChalkOptions): Chalk;
  }

  export interface ColorSupport {
    level: Level;
    hasBasic: boolean;
    has256: boolean;
    has16m: boolean;
  }

  export interface Chalk {
    (...text: string[]): string;
    (text: TemplateStringsArray, ...placeholders: string[]): string;
    constructor: ChalkConstructor;
    enabled: boolean;
    level: Level;
    rgb(r: number, g: number, b: number): this;
    hsl(h: number, s: number, l: number): this;
    hsv(h: number, s: number, v: number): this;
    hwb(h: number, w: number, b: number): this;
    bgHex(color: string): this;
    bgKeyword(color: string): this;
    bgRgb(r: number, g: number, b: number): this;
    bgHsl(h: number, s: number, l: number): this;
    bgHsv(h: number, s: number, v: number): this;
    bgHwb(h: number, w: number, b: number): this;
    hex(color: string): this;
    keyword(color: string): this;

    readonly reset: this;
    readonly bold: this;
    readonly dim: this;
    readonly italic: this;
    readonly underline: this;
    readonly inverse: this;
    readonly hidden: this;
    readonly strikethrough: this;

    readonly visible: this;

    readonly black: this;
    readonly red: this;
    readonly green: this;
    readonly yellow: this;
    readonly blue: this;
    readonly magenta: this;
    readonly cyan: this;
    readonly white: this;
    readonly gray: this;
    readonly grey: this;
    readonly blackBright: this;
    readonly redBright: this;
    readonly greenBright: this;
    readonly yellowBright: this;
    readonly blueBright: this;
    readonly magentaBright: this;
    readonly cyanBright: this;
    readonly whiteBright: this;

    readonly bgBlack: this;
    readonly bgRed: this;
    readonly bgGreen: this;
    readonly bgYellow: this;
    readonly bgBlue: this;
    readonly bgMagenta: this;
    readonly bgCyan: this;
    readonly bgWhite: this;
    readonly bgBlackBright: this;
    readonly bgRedBright: this;
    readonly bgGreenBright: this;
    readonly bgYellowBright: this;
    readonly bgBlueBright: this;
    readonly bgMagentaBright: this;
    readonly bgCyanBright: this;
    readonly bgWhiteBright: this;
  }

  const chalk: Chalk & { supportsColor: ColorSupport };

  export default chalk;
}

@leebenson In what scenario does import chalk from "chalk"; not work? The only difference I see with your typings is an Ambient Module declaration which theoretically is no diffrent than a top level export, since those are scoped to the module they were pulled in with (eg. chalk/index.js)


Is your feature request related to a problem? Please describe.
An Open Graph image is an image that is associated with a webpage and is used when the webpage is shared on social media platforms like Facebook, Twitter, LinkedIn, etc. Open Graph images are also called social sharing images or social media images.
Describe the solution you'd like
Create an Open Graph image.
having an open graph image increases the click-through rate of a website link when share across social media.

Add a reference image for your project

Ok, I will work on it


Move gym from habitat_baselines to habitat

Motivation and Context
In order to separate habitat from habitat_baselines, we move the gym environments to habitat. This enables users to quickly start using the habitat environments without having to install the habitat_baselines trainers.
Reward and success definitions have been moved from baseline configuration to task configuration.
Modified configs creation script to allow modification of the task config from the baseline config.
How Has This Been Tested
Tested that training starts for all of the registered gym environments tasks
Added a unit tests that runs one episode of each gym environment
properly behaves.
Types of changes

Docs change / refactoring / dependency upgrade
Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@vincentpierre Pre-commit is now working properly again. Feel free to pull form master to resolve.


foxglove_compressed_video_transport: 1.0.0-1 in 'jazzy/distribution.yaml' [bloom]

Increasing version of package(s) in repository foxglove_compressed_video_transport to 1.0.0-1:

upstream repository: https://github.com/ros-misc-utilities/foxglove_compressed_video_transport.git
release repository: https://github.com/ros2-gbp/foxglove_compressed_video_transport-release.git
distro file: jazzy/distribution.yaml
bloom version: 0.12.0
previous version for package: null

foxglove_compressed_video_transport
* initial commit
* Contributors: Bernd Pfrommer

Holding for Jazzy sync


Feature: Make the back button hide the UI when paused

By back button I mean the button my TV remote, the UI obstructs a significant portion of the image when paused.
Better yet, maybe don't show the UI at all until the user navigates down to it.

This is intentional #195

Why though? There doesn't seem to be any rationale behind it.

It's simply about the fact that I'm customizing the player only around my needs and wishes. The usage on TV might be slightly different but at least on phones, I ended up strongly disliking the tap to toggle UI controller feature. I realized I don't need that extra tap and do not watch any content that would require that. There are hundreds other players out there so I went all in on being different.

I understand your stance, and it is perfectly valid because it's your project after all.
However I would like to request addition of the functionality dismiss it by using the back button, that way both you and us the users would be satisfied :smile:
When watching a video or a movie, some scenes show text only for a brief moment, and sometimes we want to read it by pausing. Currently this is impossible... but we can easily make this possible by making the UI dismissable with the back button, and I'm assuming it would be a very easy task to implement!
I assure you that many users will be delighted to have this feature, thank you in advance! :pray:

There are hundreds other players out there so I went all in on being different.

Indeed, but in my experience none of them even come close to the perfection and simplicity in this player! Especially for a TV, this is one of the few players which can properly send Dolby Atmos audio without transcoding it.


More than 5 items in the front page listings

On my 1366x768 screen the last item is always in a row of its own and it looks weird. We should have more resources there.

How about 7 resources? On bigger monitors it should display as either all 7 or 5 + 2, and on smaller ones it'll display as 4 + 3.


Removing old jsartoolkit5 version from AR.js

Do you want to request a feature or report a bug?
This a feature request and also an enhancement of AR.js

What is the current behavior?
Actually we have two types of AR.js libs as described in the AR-js-org docs

Image Tracking and Location Based  >> (ar-nft.js and aframe-ar-nft.js)
the old style lib for Pattern Marker; Barcode Marker... >>  (ar.js  and aframe-ar.js)

The first one use the newest jsartoolkit5 lib, the second use the old jsartoolkit5 lib with .min and .api bundled together.
Why this? artoolkit.api.js is required otherwise you will get this issue on Mobile device (portrait mode) see https://github.com/AR-js-org/AR.js/pull/60#issuecomment-600755818 and also pay attention to all the discussion in the PR https://github.com/AR-js-org/AR.js/pull/60
We need to remove the old lib because we can not add new features to jsartoolkit5.
If the current behavior is a bug, please provide the steps to reproduce.
remove artoolkit.api.js from the compilation (Makefile) ,
- cat vendor/jsartoolkit5/build/artoolkit.min.js \
- vendor/jsartoolkit5/js/artoolkit.api.js >> build/ar.js

+ cat vendor/jsartoolkit5/build/artoolkit.min.js >> build/ar.js

run the script, test the basic example or any other example on portrait mode (Mobile)

Please mention other relevant information such as the browser version, Operating System and Device Name
The bug i mentioned occurs only in portrait mode (Mobile)
What is the expected behavior?
Eliminating artoolkit.api.js should behave in the same way.
If this is a feature request, what is motivation or use case for changing the behavior?
Upgrading AR.js to the new jsartoolkit5 ( not only for nft).

I solved the problem, but i think we have to find a better solution. I will explain now all the aspect of the issue. As mentioned above, the problem arises when you are in 'portrait' mode (on Mobile) without the artooolkit.api.js.
Why this?
In the old and also in the new (with NFT)  artoolkit.api.js at the beginning, when a new ARController is instantiated theere is:
https://github.com/AR-js-org/AR.js/blob/da5ea830211c323a41ebd55a97b99f85add1cd40/three.js/vendor/jsartoolkit5/js/artoolkit.api.js#L31
this is the default setting. When you run a jsartoolkit5 code and you are in portrait mode this.orientation becomes 'portrait' because in ARController.getUserMediaARController
https://github.com/AR-js-org/AR.js/blob/da5ea830211c323a41ebd55a97b99f85add1cd40/three.js/vendor/jsartoolkit5/js/artoolkit.api.js#L1292-L1296
if the video width is less than height this.orientation = 'portrait', but because getUserMediaARController is never used inside AR.js ( treex-arcontroller, threex-artoolkitsource and so on...) this.orientation is always landscape !! see my comment here https://github.com/AR-js-org/AR.js/pull/60#issuecomment-601671876
So what is the solution?
This issue was already in the old Ar.js lib see this changes in this PR in artoolkit.api.js about the portait mode inside the _copyImageToHeap
I tested adding that changes to the new artoolkit.api.js in this way:
  if( (image.nodeName === 'IMG' && image.width > image.height ) ||
	      		(image.nodeName === 'VIDEO' && image.videoWidth > image.videoHeight) ){
			        // if landscape
	                        this.ctx.drawImage(image, 0, 0, this.canvas.width, this.canvas.height); // draw video
		        }else{
			        // if portrait
			        this.ctx.clearRect(0, 0, this.canvas.width, this.canvas.height);
		        	var scale = this.canvas.height / this.canvas.width;
			        var scaledHeight = this.canvas.width*scale;
		        	var scaledWidth = this.canvas.height*scale;
			        var marginLeft = ( this.canvas.width - scaledWidth)/2;
			        this.ctx.drawImage(image, marginLeft, 0, scaledWidth, scaledHeight); // draw video
		         }

instead of:

               if (this.orientation === 'portrait') {
			this.ctx.translate(this.canvas.width, 0);
			this.ctx.rotate(Math.PI/2);
			this.ctx.drawImage(image, 0, 0, this.canvas.height, this.canvas.width); // draw video
		} else {
			this.ctx.drawImage(image, 0, 0, this.canvas.width, this.canvas.height); // draw video
		}

Apllying that changes and rebuilding the jsartoolkit5 libs solve this issue. Anyway i will prefer to not apply that cahnges to the jsartoolkit5 and find another solution.
More to come...

If someone want to test the modified jsartoolkit5  version, as i explained in the previous comment, may grab the artoolkit.min.js from this branch https://github.com/kalwalt/jsartoolkit5/tree/fix-copyImageToHeap, and modify the Makefile as i explained in the comment at the top. After test the threejs basic example. other examples require to delete the artoolkit.api.js script import.

i have a testing AR.js branch  on my repository, testing is welcome!!

I suppose, because i didn't the first implementation of the lib, that the reason is Aframe, probably require a particular set-up, but i'm not a great expert with Aframe. Or maybe a deliberate code design choice?


Exponential disk equilibrium

IC.expdisk doesn't do a good job of equilibrium. Fix. Also feed in rotation curve instead of halo force.

Slightly improved


Copy addresses to clipboard

Add a button to briefing views to copy one or all addresses to clipboard

Or fwd to a page which lists the addresses in a table which you can filter , drop columns, etc and export to excel or csv or clipboard.


How to not add constraint for many2many?

Your Question
Since gorm2.0, it add constraint automatically for many2many, however I don't want to use foreign keys, is there any way not to add them at creation time?
The document you expected this should be explained
skip foreign key not mentioned.
https://gorm.io/docs/many_to_many.html
Expected answer
I don't want foreign key constraints in many2many related tables.

we hope to get an answer for our question.

https://gorm.io/docs/gorm_config.html#DisableForeignKeyConstraintWhenMigrating


"go tidy" action uses go1.18, which cannot work with go.mod containing go1.19

https://github.com/dims/community-images/blob/572f1f579be0a510e026f70b8173abb2cbd44465/.github/workflows/go.yml#L23
go: go.mod file indicates go 1.19, but maximum version supported by tidy is 1.18

https://github.com/johanneskastl/community-images/actions/runs/4303393509/jobs/7503072657

@johanneskastl can you update the workflow to use:
- name: Setup Go
  uses: actions/setup-go@v2
  with:
    go-version: ‘1.19.0’


Youtube Data API forUsername doesn't work with cyrillic alphabet?

Hello guys, I think there's an issue with youtube data api as it doesn't return any results when I'm trying to find a channel with "forUsername" query, value = "Хулиномика", link to example:
https://developers.google.com/youtube...
I'm sure it's existing channel, link to channel itself works fine:
https://www.youtube.com/c/Хулиномика
But response I'm receiving always like this:
{
  "kind": "youtube#channelListResponse",
  "etag": "oFbaWspmyqYyvIx-nop9GTK-gHE",
  "pageInfo": {
    "totalResults": 0,
    "resultsPerPage": 5
  }
}

I tried to use use php function urldecode() to decode "%D0%A5%D1%83%D0%BB%D0%B8%D0%BD%D0%BE%D0%BC%D0%B8%D0%BA%D0%B0" to "Хулиномика" but it didn't work out, it doesn't work either when I'm passing url encoded channel username or decoded
So it's not an issue specifically related to this php library but I'm not sure where would be better to report this?

Nevermind, I got it.
It's not because of cyrillic, it's because "short url" != "username", related issue on stackoverflow:
https://stackoverflow.com/questions/37267324/how-to-get-youtube-channel-details-using-youtube-data-api-if-channel-has-custom

Nevermind, I got it.
It's not because of cyrillic, it's because "short url" != "username", related issue on stackoverflow:
https://stackoverflow.com/questions/37267324/how-to-get-youtube-channel-details-using-youtube-data-api-if-channel-has-custom


Upgrade to Angular 6

Hello @oferh,
Do you have plans to  support Angular 6? Kindly let me know if you are planning or have already started working on the upgrade.
I  am will to work on the upgrade.

+1

@karthick-manoharan haven't had time recently to work on it would appricate a PR

I opened a PR.

Can you accept his PR for Angular 6?

Any news on this ?

This is by no means a permanent solution, but I need to be able to build this and push it up to a development server so I altered my configuration to remove aot and it builds properly. The bundle produced is still substantially smaller than a simple ng build, but aot is clearly disabled.
Still looking forward to building this with aot enabled, but don't think we can do that until this module has been upgraded.
ng build --prod --configuration=production2
            "production2": {               "optimization": true,               "outputHashing": "all",               "sourceMap": false,               "extractCss": true,               "namedChunks": false,               "aot": false,               "extractLicenses": true,               "vendorChunk": true,               "buildOptimizer": false,               "fileReplacements": [                 {                   "replace": "src/environments/environment.ts",                   "with": "src/environments/environment.prod.ts"                 }               ]             }

Eagerly looking for update for Angular 6

published as ng2-completer@next

Any update on Angular 6? Eagerly looking for the same..
Thank you!

Might have to use Angular 6's native auto complete.  https://www.encodedna.com/angular/autocomplete-textbox-in-angular-6-using-web-api.htm

@danielnajeraRTC that's Material and it's an excellent auto complete solution.


how Can we create Slug of an array in eloquent Sluggable ?

Please be sure you include all the relevant information in your issue so we can help you:

[ ] steps taken to reproduce your issue
[ ] your configuration file(s), where relevant
[ ] a copy of your Models that use or extend this package (just the relevant parts!)
[ ] any other code we might need to help

Please use a descriptive title for your issue.  "Why doesn't this work?" doesn't provide
other users much information if they are scanning the list of issues.
Also, please use fenced code blocks
when pasting more than one line of code.  It makes it so much more readable for everyone!
Thank you!

If you mean an array or list of model attributes, you can do that by providing them all in an array in the config:
'source' => ['firstname', 'middlename', 'lastname'],

If you have a model attribute that is an array, then I'd suggest making a separate getter to convert it to a string and use that as the source:
'source' => 'slug_source',

...

public function getSlugSourceAttribute()
{
    return implode(' ', $this->myArray);
}


Add argument to select LLVM mode (minified or full)

Bare-metal targets are failing due to the changes in Timer, once https://github.com/esp-rs/esp-template/pull/15 gets merged this issue will be fixed

Update https://github.com/esp-rs/esp-template/pull/15 got merged, rerun failed test and they are now passing


ESP32 DMX Libary packet size read problem

esp dmx library very help full for me but i noted one issue please solve this is issues - issues when read dmx so packet size read but when run greater then more packet after run less packet size value of packet size before data tranfered packet size. for more understand give example - when i run 200 channel packet size so read 200 but after run more then 200 like a 250 but read packet size 200 if i send data less then 200 like 150 then read packet size so read 150 but after then mote then 150 but in actual shw 150 packet size. so not increass but decrease packet size. if i reset esp32 then ok to read more packet size.
Thanks you

Thanks for the question. Which version of this library are you using and what version of ESP32 are you using?

I used latest version 4.1.0 esp_dmx library and I used ESP32 Devkit module and esp32 board version 2.0.17.

Hello! I have a same problem with latest version.

Hello someweisguy. This is library for me very usefull. Please solve this issue.


Disables heart mage trick when running under "Respawn Penalty"

When the run is under "Respawn Penalty", the heart mage projectile will now only half the current HP (instead of max HP).
For context, if someone has very low max HP when a teammate dies, the run will be ended for everyone. Therefore it is possible for players to (intentionally or accidentally) lower their max HP using Glass Cannon or Heart Mage and kills the run.
The description of the heartache effect has been changed to "[NT Fix] Your health is temporarily halved." in case people get confused why the trick no longer works.

@ParasectW while this is nice, If I were fixing this the approach I would attempt would be to disallow the run from ending if the user had glass cannon or heart mage effect. Maybe I could still add it in and just added as a game mode requirement that could be enabled to disable heart mage trick as a separate setting separate from respawn penalty. Not entirely sure at the moment which way would be better

Thanks for the PR! I will go ahead and close this as it should be approached different. I think we should still allow it but change how the health checks are done to prevent heart mage and glass cannon issues for NT. It could be cool to make this a separate mod though :)


No output logs even with specified reporters with Xcode 7.3

Logs show up when running build-tests, but none show up, even with -reporter json-stream (or pretty/plain) or when using run-tests or test with Xcode 7.3.

What test framework(s) are you using? Nimble (Quick/Nimble#270), Cedar (pivotal/cedar#383) and KIF (kif-framework/KIF#819) all ran into this issue with Xcode 7.3.

I'm using KIF. I'll definitely give what they suggest in that link a shot.

Merging into https://github.com/facebook/xctool/issues/682.


feat: generate different loggers per level to avoid allocations.

Thank you for contributing to Coraza WAF, your effort is greatly appreciated
Before submitting check if what you want to add to coraza list meets quality standards before sending pull request. Thanks!

Make sure that you've checked the boxes below before you submit PR:

[ ] My code includes positive and negative tests.
[ ] I have an appropriate description with correct grammar.
[ ] I have read Contribution guidelines, maintainers note and Quality standard.
[ ] My code is properly linted and passes pre-commit tests.

Thanks for your contribution :heart:

Closing this in the context of https://owasp.slack.com/archives/C02BXH135AT/p1677616370248079


MMCA-4829 | Refactor PostponedVatViewModel and ImportPostponedVatRequestedStatements

Acceptance Criteria:

Ensure the PostponedVatViewModel and ImportPostponedVatRequestedStatements.scala.html are refactored

Dev Tasks:

[x] Refactor PostponedVatViewModel
[x] Refactor ImportPostponedVatRequestedStatements.scala.html
[x] Amend / add unit tests

I've noticed some errors (shown below) when compiling. Do we have this highlighted to fix in another ticket? If they haven't been it may be be worth fixing it now if it's simple or getting it added as tech debt ticket down the road.
It would fail on pattern case: (_, _)

I've noticed some errors (shown below) when compiling. Do we have this highlighted to fix in another ticket? If they haven't been it may be be worth fixing it now if it's simple or getting it added as tech debt ticket down the road.
It would fail on pattern case: (_, _)

We have a scheduled ticket for this, MMCA-4884, will be addressed as part of that ticket.


[wxwidgets] Build with OpenGL for Windows ARM

What does your PR fix?
This PR builds wxWidgets with OpenGL for Windows ARM.

Which triplets are supported/not supported? Have you updated the CI baseline?
This changes the wxWidgets configuration for arm-windows-* and arm64-windows-*. There are no changes to the CI baseline.

Does your PR follow the maintainer guide?
Yes

@microsoft-github-policy-service agree [company="Alphabet"]

@microsoft-github-policy-service agree company="Alphabet"


Ask user for account ID during "Sign in with Ledger"

@marcinbodnar what would be the best/easiest way to test this?

@Patrick1904 Best would be to test it locally, you can set conts accountId = [] here so it will mimic a situation where Ledger is not able to find any account:
https://github.com/near/near-wallet/blob/master/src/utils/wallet.js#L469


Added Zamnesia as hard

Added Zamnesia as hard
https://www.zamnesia.com/

Thanks for your contribution!


Discover feed no longer working.

Describe the bug
Discover feed no longer works on Modern homepage. Turned it off and on and changed source, but that didn't fix it.
To Reproduce
Open new tab.
Expected behavior
A list of various news articles in the Discover feed.
Smartphone (please complete the following information):

Device: [Amazon Fire HD 10 tablet]
Android Version : [Android 9]
Kiwi Browser Version 112.0.5615.137

Additional context
Discover feed still works if the homepage is set to Classic homepage.

Issue confirmed, happens from a month or so, no Kiwi updates in the meantime AFAIK, must be something server-side

Kiwi v116 ( Github release) got this fixed .


Updates dependencies

uuid breaking changes

refactor v1 internal state and options logic
refactor v7 internal state and options logic, fixes
Port to TypeScript
update node support matrix (only support node 16-20)
This library always aims at supporting one EOLed LTS release which by this time now is 12.x which has reached EOL 30 Apr 2022.
Remove the minified UMD build from the package.
Drop support for browsers that don't correctly implement const/let and default arguments, and no longer transpile the browser build to ES2015.
Although in practice this is currently a noop since the resulting build does not change, the build will no longer transpiles future changes for Node.js 8.x targets, so semantically this is still a breaking change.
Deep requiring specific algorithms of this library like require('uuid/v4'), which has been deprecated in uuid@7, is no longer supported.
The default export, which used to be the v4() method but which was already discouraged in v3.x of this library, has been removed.
Explicitly note that deep imports of the different uuid version functions are deprecated and no longer encouraged and that ECMAScript module named imports should be used instead. Emit a deprecation warning for people who deep-require the different algorithm variants.
Remove builtin support for insecure random number generators in the browser. Users who want that will have to supply their own random number generator function.
Remove support for generating v3 and v5 UUIDs in Node.js<4.x
Convert code base to ECMAScript Modules (ESM) and release CommonJS build for node and ESM build for browser bundlers.

Merged manually, thank you! 🎊


Make sure all flags get pulled during init.

Not all the flags inits were being pulled in to the main framework. Adding the flags for Istio deployment as well.
This required breaking a dependency cycle by changing and moving SetupFn function to the resource package.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
ea3929915b79b3255e46a0b09ecf1eebb5786aaa
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
ea3929915b79b3255e46a0b09ecf1eebb5786aaa
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
ea3929915b79b3255e46a0b09ecf1eebb5786aaa
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [ozevren]
tests/integration/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, nmittler, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [nmittler,ozevren]
tests/integration/OWNERS [nmittler,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Update workflow for using coursier/setup-action

Changes

Update workflow
Fix badge URL in README.md

Codecov Report

Merging #700 (0226200) into main (785cbb0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              main      #700    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           66        56    -10
  Lines         2861      1970   -891
  Branches       475       322   -153
==========================================
- Hits          2861      1970   -891

Impacted Files
Coverage Δ

packages/recheck/src/lib/worker-pool.ts

packages/recheck/src/lib/pure.ts

packages/recheck/src/lib/native.ts

packages/recheck/src/browser.ts

packages/recheck/src/lib/agent.ts

packages/recheck/src/lib/exe.ts

...ges/eslint-plugin-redos/src/rules/no-vulnerable.ts

packages/recheck/src/lib/env.ts

packages/recheck/src/lib/java.ts

packages/recheck/src/main.ts

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add default value for strings in pb_map _mergeEntry to fix issue #508

Disclaimer - I'm no expert in this code base, but I was hit with the issue https://github.com/dart-lang/protobuf/issues/508 and decided to try and fix it today.
In the null_safety release, deserialising a protobuf message that includes empty strings within a map results in an error, as they aren't serialised which results in null values. This is then causing an error when field_set _meta function is called. This can be avoided by ensuring a default value of '' for strings in the _mergeEntry function

@googlebot I signed it!

@mraleph – who should review this?

@jamiemm – any tests we could add here?

Try rebasing on the latest null_safety branch

Actually, @jamiemm – rebase on master now 😄

I have done a few quick checks on the master branch, and it looks like the problem is solved there :)
Thanks!


feat(wash-cli): Add --watch flag to wash app list for displaying live info

Feature or Problem
This flag perpetually displays changes to application status until Ctrl+c is invoked
Related Issues
#2512
Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

https://github.com/user-attachments/assets/c27351f4-7fb9-4573-a192-244783a352dc
Here's the progress for wash app list --watch

Update 2:
https://github.com/user-attachments/assets/d9ca6eae-d30f-4970-b1ed-cf440a354b9b
The request timed out error came after I ran wash down -- all and the output is 90% responsive
The live info adapts to the terminal size and it runs the same in the vscode terminal too

This worked pretty well for me 😄
I had one supernit&request, but my real requested change here may be more of a personal preference. This implementation output the refreshed information every second, but what I expected was more of an implementation like watch where the screen is cleared & new information is written to the terminal.
Did you find this as an option when looking into crossterm, or give it a try and find issues? Curious on your thoughts there

By clearing the screen do you mean clearing out the history of the terminal and Displaying the table on a clean terminal or logging of new information, because the current implementation does clear the table and re-displays the info after a sleep duration.
I tweaked watch_application in a way that keeps the terminal history intact instead of clearing it out.
Do let me know if you'd prefer the former or correct me if I'm wrong of my understanding.
At first I tried the alternate terminal method but the output was disoriented and the labels weren't properly spaced each time, at first I thought it must've been cause of zsh , but then I tried it with sh and bash , which gave the same results

@Aditya1404Sal here's what it looks like for me, it seems like in an empty terminal or a mostly empty terminal it does the re-clearing, but sometimes it just continually outputs to the screen 🤔 This is made a little bit worse with tmux
https://github.com/user-attachments/assets/2af0ea28-12ed-45f5-b5ac-3150e067ef6c
Is there anything you can think of that would cause this behavior? What I see in the beginning of the video is exactly what I wanted, and I'm totally okay with clearing the previous output like you were doing before if it redraws in the same place

@Aditya1404Sal here's what it looks like for me, it seems like in an empty terminal or a mostly empty terminal it does the re-clearing, but sometimes it just continually outputs to the screen 🤔 This is made a little bit worse with tmux
wash.mov
Is there anything you can think of that would cause this behavior? What I see in the beginning of the video is exactly what I wanted, and I'm totally okay with clearing the previous output like you were doing before if it redraws in the same place

Yeah I have noticed it bugging out very rarely, I think it has something to do with the cursor tracking when the terminal isn't  in vscode but i'm not too sure.
I'll be happy to proceed with the previous version of clearing out the entire history and then redrawing in the same place.

I'll be happy to proceed with the previous version of clearing out the entire history and then redrawing in the same place.

@Aditya1404Sal I think that will work well 👍🏻 let's give that a try, and if you can test it out in another terminal I'll give it a test when you push up as well.

@brooksmtownsend I've modified the code to clear the entire terminal, The issue of multiple tables being printed out is gone (as far as i've tested) However the terminal if resized on specific intervals gives strange output, the table get's disoriented if the terminal's width is changed while the command runs.

However the terminal if resized on specific intervals gives strange output,

I think this is fine, there's really nothing we can do to handle folks resizing their terminal a bunch. I don't think it's too much to ask to re-run a command if you do this

However the terminal if resized on specific intervals gives strange output,

I think this is fine, there's really nothing we can do to handle folks resizing their terminal a bunch. I don't think it's too much to ask to re-run a command if you do this

I'll still try to find a solution for this issue :rocket:


metric-tank crash in production

2015/12/23 06:00:22 ESC[1;33m[W] failed to save chunk to cassandra after 1 attempts. <chunk T0=1450849800, LastTs=1
450850351, NumPoints=10, Saved=false>, gocql: no response received from cassandra within timeout periodESC[0m
2015/12/23 06:00:22 ESC[1;33m[W] failed to save chunk to cassandra after 1 attempts. <chunk T0=1450849800, LastTs=1
450850351, NumPoints=10, Saved=false>, gocql: no response received from cassandra within timeout periodESC[0m
2015/12/23 06:00:22 ESC[1;33m[W] failed to save chunk to cassandra after 1 attempts. <chunk T0=1450849800, LastTs=1
450850357, NumPoints=10, Saved=false>, gocql: no response received from cassandra within timeout periodESC[0m
2015/12/23 06:00:22 ESC[1;33m[W] failed to save chunk to cassandra after 1 attempts. <chunk T0=1450849800, LastTs=1
450850351, NumPoints=10, Saved=false>, gocql: no response received from cassandra within timeout periodESC[0m
2015/12/23 06:00:22 ESC[1;33m[W] failed to save chunk to cassandra after 1 attempts. <chunk T0=1450849800, LastTs=1
450850357, NumPoints=10, Saved=false>, gocql: no response received from cassandra within timeout periodESC[0m
fatal error: runtime: out of memory
runtime stack:
runtime.throw(0xa19490, 0x16)
        /usr/local/go/src/runtime/panic.go:527 +0x90
runtime.sysMap(0xc8144bc000, 0x8000, 0xc996770000, 0xc7c438)
        /usr/local/go/src/runtime/mem_linux.go:143 +0x9b
runtime.mHeap_MapBits(0xc5cf60, 0xc996870000)
        /usr/local/go/src/runtime/mbitmap.go:144 +0xcc
runtime.mHeap_SysAlloc(0xc5cf60, 0x100000, 0x21bd)
        /usr/local/go/src/runtime/malloc.go:424 +0x186
runtime.mHeap_Grow(0xc5cf60, 0x8, 0x0)

full log has been saved to /root/issue88/metric_tank.log.1.gz on metric-tank-2-prod

Might need to increase the cassandra timeout value. It is using the default of 600ms

The server still had ~20Gb of memory free.   Looks like it could be due to this bug.
https://github.com/golang/go/issues/12233

well luckily for us the fix was included in the recent 1.5.2 release , https://github.com/golang/go/issues?q=milestone%3AGo1.5.2 so can we recompile with that?

The changes introduce in PR #95 should have resolved this as we are no longer creating hundreds of thousands of goroutines.
If this re-occurs then we will need to modify our circleci scripts to use the Go1.5.2 instead of 1.5.1 which is provided in the default image.