Is it possible to add intellisense on toml file?

Currently, the  toml file is treated as a plain text. I hope it could get some highlight on some keywords, also when typing ctrl+/ in vscode, I hope the line where cursor at could be commented.

It's possible and planned, but not being worked on.
In the meanwhile you can try https://marketplace.visualstudio.com/items?itemName=bungcip.better-toml, https://marketplace.visualstudio.com/items?itemName=tamasfe.even-better-toml (it claims to be better, but I don't like the defaults) and https://marketplace.visualstudio.com/items?itemName=serayuzgur.crates.

Closing as a duplicate of #3565 to keep things tidy.


VScode: Chat scrolls to top for each new message

Every time I input a new chat message, the scoll jumps to the top of the chat history.
Cody: 0.14.0
Version: 1.83.0 (user setup)
Commit: e7e037083ff4455cf320e344325dacb480062c3c
Date: 2023-10-03T16:12:16.321Z
Electron: 25.8.4
ElectronBuildId: 24154031
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621

Yeah I faced this too all the time


Vue 3 support

Vue 3 support
Changed:

replace vue-template-compiler with @vue/compiler-sfc
upgrade vue to version 3.0.11
upgrade demo to Vue 3

@kazupon IMO it should be merged into next branch, and released as next version
Closes #139

@ardier16
I am very happy to you for your Vue 3 support.
We are planning to release vue-i18n-locale-message v1 first for Vue3 support.
After releasing that, we hope to provide a Vue 3 support version.
I would like you to send me your PR at that time.

@ardier16
Hi! I've just released v1 for Vue 2
I think I'll release v2 for Vue 3.
Could you resolve the conflict of this PR! 🙏

@ardier16
Hi! I've just released v1 for Vue 2
I think I'll release v2 for Vue 3.
Could you resolve the conflict of this PR! 🙏

Hi, @kazupon!
Just resolved a conflict, it's ready to be merged.
Is there a next branch for this PR?

@ardier16
Your PR is OK.
Thank you so much!


predictions-to-bigquery: Update FilterPredictionToDictFn.

Updates process method to parse the element (PredictionLog) and extract the input tf.Example as dict and combine it with the prediction results.
Fixes #218

It's a good idea to open an issue first for discussion.

[ X ] Tests pass
[ X ] Appropriate changes to README are included in PR

@hanneshapke thanks for the approval.
@rcrowe-google @theadactyl I believe one your approvals is needed as well to merge the PR. Please have a look when you have time. I have a follow on PR that I'm planning to submit after this one.

/lgtm

@cfezequiel Do you mind updating the CODEOWNERS file?
There is a typo /tfx_addons/predictions_to_biquery, otherwise I could sign off on it.
No need for Thea's approval

/lgtm
/merge

@cfezequiel @rcrowe-google Who is making the CODEOWNER change?


Can't resolve 'react-usefavicon' – are the main, browser etc properties in package.json correct?

I was having trouble using react-usefavicon when trying:
import { useFavicon } from "react-usefavicon";

In the end I had to use this approach, but that means no automatically imported types:
import {
  drawTextBubble,
  useFavicon
} from 'node_modules/react-usefavicon/dist/react-usefavicon.modern.mjs';

Are the browser, main etc properties in package.json correct? It looks like they're missing a react- prefix in each of their filenames:

Good catch, thanks for making an issue. I just updated the export properties and published a new version. Could you check if that fixes it?


Any strategies for writing function convert_(dataset_name)_data for a custom dataset on hugging face?

Hello, would you mind telling me some strategy suggestions for writing the convert_(dataset_name)_data functions in reform_datasets.py on a dataset from huggingface? Thank you so much!

Hey @AnnyShen55 -- we're moving a bit away from that script, but it looks like these functions.
https://github.com/allenai/open-instruct/blob/42c1fa3c10d0f39112e40146c0cebfcce88346f2/open_instruct/utils.py#L59
Then you can call
data_formatted = dataset.map(format_fnc)

Got it, any more insights that you may kindly give? Thank you!

Make copies of datasets in your preferred format :)


Edmx taking to much time to generate model

From VS Feedback:
when ever trying to generate model from data base using einity framework 6.0 it is taking over an hour to update.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Re-adding SqlUpdatesWithProcedures (with build fixes)

Same as #37 but with fixes for the build (previous pull request was missing the latest commit from master, and so there was a build error around apps/SqlUpdatesWithProcedures.java:[69,3] method does not override or implement a method from a supertype).
Old PR message from #37:

Adding a new workload that is similar to SqlUpdates but that uses sql procedures in order to batch the updates together (see batching of updates in yugabyte/yugabyte-db#5257).

As mentioned in the comments of #37, SqlUpdatesWithProcedures differs from SqlUpdates in that this workload will only send num_writes unique updates (the uniqueness is to avoid transaction conflicts). Thus this workload can also be used for performance timing, as it will exit after it has finished all its updates.
Typical use case of this workload is via first running SqlInserts:
java -jar yb-sample-apps.jar \
--workload SqlInserts \
--nodes a:5433,b:5433,c:5433 \
--num_unique_keys 5000000 \
--num_reads 0 \
--num_threads_write 128 \
--uuid 'ffffffff-ffff-ffff-ffff-ffffffffffff'

Then by running SqlUpdatesWithProcedures:
java -jar yb-sample-apps.jar \
--workload SqlUpdatesWithProcedures \
--nodes a:5433,b:5433,c:5433 \
--uuid 'ffffffff-ffff-ffff-ffff-ffffffffffff' \
--max_written_key 5000000 \
--num_writes 1000000 \
--num_reads 0 \
--num_threads_read 0 \
--num_threads_write XXX \
--update_batch_size YYY

A few things to note:

Notice that num_writes is lower than max_written_key, this is mostly to ensure that we don't have issues randomly selecting the last remaining keys to update at the end.
With the way the workload is currently written, the max update_batch_size is 50 since postgres only allows procedures to have at most 100 parameters, and we are currently just passing in key/value pairs to be updated (100/2 = 50).
Performance can also be improved by forcing all writes to a single tablet, for example by running \c postgres and create table postgresqlkeyvalue(k text primary key, v text) split into 1 tablets; in ysqlsh prior to running SqlUpdates

LGTM!!!


Explicitly set OSX_DEPLOYMENT_TARGET in cmake flags, as matrix.config.env is not propagated into job's env variables.

Fixes https://github.com/clangd/vscode-clangd/issues/228

According to objdump min macosx version is now 10.9:
Load command 9
      cmd LC_VERSION_MIN_MACOSX
  cmdsize 16
  version 10.9
      sdk 11.1

Thanks! We could also propagate matrix.env into the environment, but this seems less magic.

yeah i tried that too but couldn't find a nice way to represent that in the yaml file, especially when the matrix.env could be empty. also not relying on env variables sounded more robust in the end.


Fix activate script in liblalpulsar

The activate script was originally written to go in a package named lalpulsar so when the logic to write it was moved into liblalpulsar, things got screwed up.

Checklist

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

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


Another project for CSV-to-YNAB conversion

Hello Omar!
I've seen that you added a YNAB repository to GitHub, even if it's two years ago. Have you seen that there are many similar projects? There is one in particular that could be very interesting for you: "bank2ynab"
https://github.com/torbengb/bank2ynab/blob/develop/README.md
If you could provide a sample CSV export (anonymized) then we could easily create the converter for your bank, too! We would also like to know the default filename for the exported CSV, so we can identify it and use the proper converter.
Let me know what you think!
Best regards from snowy Austria,
Ben

Hi Ben,
I failed to notice this on time, sorry. I am using a different bank by now and I haven't looked into ynab either for years. I will archive this project, but thanks for reaching out anyway.
Regards
Omar


Proper coloring for Rust example code

https://thegraph.com/docs/en/cookbook/substreams-powered-subgraphs/

Not annotated

Enabling rule to enforce code blocks have language will avoid it in the future

@B2o5T – You worked on this in #358, right?

@benface Yes, I already worked on adding this rule, but my pr didn’t receive approvals


gulp main task fail at Windows.

Issue Overview
First build after clone gives error: "A project cannot be used in two compilations at the same time. Create multiple projects with createProject instead"
Issue Description
After clone I issued:
npm install
bower install

Next command:
gulp

Gives me error:
Compiling C:\Users\gpolis\Documents\work\ang2\angular2-starter\app\app.component.ts...
C:\Users\gpolis\Documents\work\ang2\angular2-starter\node_modules\gulp-typescript\release\main.js:71
throw new Error('gulp-typescript: A project cannot be used in two compilations at the same time. Create multiple projects with createProject instead.');
^
Error: gulp-typescript: A project cannot be used in two compilations at the same time. Create multiple projects with createProject instead.
at compile (C:\Users\gpolis\Documents\work\ang2\angular2-starter\node_modules\gulp-typescript\release\main.js:71:19)
Reproducables
git clone
npm install
bower install
gulp
Information

Operating System
Windows

Node version
4.x

NPM Version
3.x

Environment
Browser

I think this is the plugin issue

I can't reproduce the issue, can you also post the issue to the plugin repo?
Thanks

Ok. Will do.

Related issue - https://github.com/ivogabe/gulp-typescript/issues/322

Nice, you should post the typescript.js in tasks too

Probably you get this message with latest gulp-typescript?
As stated at they message, you need to create own project for each compilation task.

i see, but i didn't think one would call those tasks at the same time though, then
I will refactor the compile tasks soon, for the moment, you can move the createProject statement into each task as they said

Check my pull request. Worked for me.

that was fast! thank you

Merged!
Thank you @gintsgints for PR! :+1:


go_router_builder should support latest analyzer 6

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
I want to use go_router_builder with other current Flutter packages that require analyzer 6.
A first-party package should provide timely updates.
Sadly the previous issue was closed without addressing the problem: https://github.com/flutter/flutter/issues/132115
Proposal
Update go_router_builder to support analyzer 6.

Thanks, missed this somehow.


APU Topo Issues from ttlworks

APU > DPCM > frequency counter LFSR > order of Bits
Considering the wikipedia page about LFSRs,
for the DPCM frequency counter LFSR the naming order of Bits is swapped.
The LFSR Bit tied to the input feed should be named FR0 instead of FR8,
please check.
Note, that when changing the naming Bit order of the LFSR,
this also changes the naming Bit order of the DPCM decoder outputs.

APU has a frequency LFSR in the DPCM section,
and a frequency LFSR in the Noise section.
Control signals for both LFSRs are labeled FLOAD,FSTEP in your schematics.
That might cause a problem to the "end user".
In my schematics, I now renamed the control signals for the DPCM frequency LFSR to DFLOAD,DFSTEP.
Please check.


Show screen not displaying correctly

When pressing the " SHOW " after generating. Not sure what does this screen shows but it seems to be tucked behing the left panel. I can scroll but nothing displays

Also would you add bubbles or description on what setting does?

@cojarbi I am unable to reproduce this bug (tested on IE/FF/Chrome on Window) but i just pulled a fix which could solve the problem (please clear your cache before testing).
P.S: The window should show the produced GCode, I add a more clearly label for the buttons.

Ok i was using Chrome for mac and as shown above it did not display the middle scroll bar. On windows it does and i can see the generated gcode.  Not sure if your last fix has something to do with it

I just tested on Chrome and Safari for mac and this is fixed

Cool, Thanks for the feedback :)

Also would you add bubbles or description on what setting does?

I did what I could with my bad English, not perfect, but it's done. someone has to correct and improve ;)


Extended completion should contain extension and static members

1>Z:\GoogleDrive_anighost\FsPickler\src\FsPickler\Combinators\Tuple.fs(338,40): error FS0039: The field, constructor or member 'Item2' is not defined.
1>Z:\GoogleDrive_anighost\FsPickler\src\FsPickler\Combinators\Tuple.fs(339,40): error FS0039: The field, constructor or member 'Item3' is not defined.
1>Z:\GoogleDrive_anighost\FsPickler\src\FsPickler\Combinators\Tuple.fs(340,40): error FS0039: The field, constructor or member 'Item4' is not defined.
1>Z:\GoogleDrive_anighost\FsPickler\src\FsPickler\Combinators\Tuple.fs(341,40): error FS0039: The field, constructor or member 'Item5' is not defined.
1>Z:\GoogleDrive_anighost\FsPickler\src\FsPickler\Combinators\Tuple.fs(342,40): error FS0039: The field, constructor or member 'Item6' is not defined.
1>Z:\GoogleDrive_anighost\FsPickler\src\FsPickler\Combinators\Tuple.fs(343,40): error FS0039: The field, constructor or member 'Item7' is not defined.
1>Z:\GoogleDrive_anighost\FsPickler\src\FsPickler\Combinators\Tuple.fs(344,42): error FS0039: The field, constructor or member 'Rest' is not defined.

System.Tuple variable could not just directly get each item by .Item.
It requires "variable.ToValueTuple().ItemX" to get item

And got a lot of errors when compiling projects:
Error	FS0756	'new' may only be used with named types

@ingted wrong issue?...

I think it is. Because my projects with extension method invocation are failed to build after upgrading to VS 15.4.

VS is unable to recognize extension method's return values...

@ingted how these problems are related to this issue?

When I use Visual FSharp project, extension method doesn't work with intelligence...
If this is not related, I will delete my comments.

Could you provide a minimal repro?

Just upgrade VS from 15.2 to 15.4 and start a new FSharp Project.
In source code,
let a = new Tuple<int, int> (1,2)
let b = a.Item1
This could be built in 15.2 but not in 15.4.
On Thu, Nov 16, 2017 at 5:55 PM, Vasily Kirichenko<EMAIL_ADDRESS>
wrote:

Could you provide a minimal repro?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/issues/3948#issuecomment-344872504,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEFyiUqxHWCM2vfWWL8FKXIU0msnYdGeks5s3AaSgaJpZM4QezR5
.

@ingted The issue you're referring to isn't related to this issue. #3729 is the demonstration of breaking .ItemX properties from System.Tuples coming directly from F# code.  In this case, you should change your code to use F# tuple syntax rather than the .ItemX properties:
let (a, b) = Tuple.Create(1, 2)

Thanks so much, should I delete the previous comments (sorry about that I didn't google the correct post)?


Are alfa/beta libraries best practice

Hello Google
Considering project-level build.gradle where several app dependencies are stated, there are currently several jetpack libaries in beta and alfa:
        // App dependencies
        appCompatVersion = '1.1.0-beta01'
        constraintLayoutVersion = '2.0.0-beta1'
        .....
        fragmentVersion = '1.1.0-alpha09'
        ....
        lifecycleVersion = '2.2.0-alpha01'
        .......
        recyclerViewVersion = '1.1.0-alpha05'
       ......

Are these versions best practice or should I use stable?
Why is this project using alpha and beta dependencies?

Of course  stable is stable, but alphaxx is often good enough, but it may
contain flaws, which may break during both build and execution. For Google
it is very valuable to report incidents if you find them. You can
"downgrade" to beta or stable if it breaks, but you may loose latest
enhancements.
RG
man. 22. jul. 2019, 19:10 skrev Jørgen Svennevik Notland <
<EMAIL_ADDRESS>
Hello Google
Considering project-level build.gradle where several app dependencies are
stated, there are currently several jetpack libaries in beta and alfa:
    // App dependencies
    appCompatVersion = '1.1.0-beta01'
    constraintLayoutVersion = '2.0.0-beta1'
    .....
    fragmentVersion = '1.1.0-alpha09'
    ....
    lifecycleVersion = '2.2.0-alpha01'
    .......
    recyclerViewVersion = '1.1.0-alpha05'
   ......

Are these versions best practice or should I use stable?
Why is this project using alpha and beta dependencies?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googlesamples/android-sunflower/issues/489?email_source=notifications&email_token=ACVSTECGZIDRR4QFS2JU4NTQAXSY3A5CNFSM4IF2R4DKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HAV6QOA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACVSTEBRWUXGGZ3WLGCCXILQAXSY3ANCNFSM4IF2R4DA
.


[DO NOT MERGE] Add API for adding external storage account configurations to a web app

Redo of #3452, which was merged accidentally. The API is not live yet.
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-go

@ebencarek Let me know when this is ready to merge

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

@ebencarek Please let us know when this is ready to be merged. Will close this week if there is no activity on the PR.

Can one of the admins verify this patch?

@dsgouda This should be ready to merge on Friday.

Please post on the PR once ready.

@ebencarek gentle almost-Friday ping

@dsgouda Go ahead and merge. Thanks!

@azuresdkci Retest this please

@AutorestCI rebuild azure-sdk-for-python


Plugins being called twice on startup (dev)

Environment
See stackblitz reproduction
Reproduction
https://stackblitz.com/edit/github-cesuxd?file=nitro.config.ts,plugins%2Ftest.ts
Describe the bug
Plugins are being run twice on startup. Is this behaviour as expected since there is not much documentation on nitro plugins and their behaviour ?
Additional context
No response
Logs
No response

Yes, because it will be automatically registered by just putting the file in the plugins folder and you add it "manually" again there.
Example only with auto-registering: https://stackblitz.com/edit/unjsnitroissues766?file=nitro.config.ts,plugins%2Ftest.ts
This could be improved in the docs I'd say

(PR welcome to improve docs)

Currently triage and cleaning up stalled issues and honestly it is more an edge case we might be able to simply fix in nitro ~> https://github.com/unjs/nitro/issues/2450


terraform-fmt updates files, optionally skipped via flags

Restore terraform-fmt behaviour from v0.1.11. Fixes #48.
By default, changes are written back to source files as usual for pre-commit hooks, but can be optionally disabled by providing a --no-autofix flag to terraform-fmt.

@brikis98 , @yorinasub17 please let me know if this is OK to be reviewed, or if you'd prefer that we don't revert the default behaviour -- rather just expose the flag to switch to the old one.

Any movement on this? Seems a few folks would like old behavior

+1 on this change. Printing the diff is not as useful as auto-correcting the styling.

Is there anything what I can do to make that change happen?

while this is not complete its better then the current fuctionality

Can we get some traction on this. The previous functionality is much nicer.

Also waiting on this functionality

Would be good to see this functionality

I found this functionality in another repository:
https://github.com/antonbabenko/pre-commit-terraform?tab=readme-ov-file#terraform_fmt
    - repo: https://github.com/antonbabenko/pre-commit-terraform
      rev: v1.92.2
      hooks:
          - id: terraform_fmt


Write a "best practices" guide

Esp, write down all of the "knobs" that can be tweaked (and then, let's try to automate as many as possible / provide the option to automatically tune)

Good idea!

More practical: we'll just try our best to integrate this in the new tutorial


Application from two repositories at once

Hey guys,
I'm evaluating spring cloud config for a multi-tenant environment and my current idea is build something like this:
repo-template

keep the settings for each specific system with their default values

repo-values

keep the set values for each specific service/tenant

My goal is bring the default value from the repo-template always when a specific value was not set on the repo-values by creating files like app_name-template.yml (on the template repo) + app_name-tenant_code.yml (on the repo-values) loading the profiles template and tenant_code together.
This approach works fine when I have the template file on the same repository of the repo-values but since we are planning use the git-flow on the template to promote new settings (and we dont want to do that with the repo-values since they are env specific in some cases) they should be separate.
When I configure two repositories with the same pattern just the first one is selected and then I have loaded just the template or just the values.
Is there a way to achieve what I'm trying to do or I need to think in another solution?
Thanks in advance

Using composite worked like a charm.


chore(main): release 0.2.1

:robot: I have created a release beep boop
0.2.1 (2022-08-04)
Bug Fixes

code structure cleanup (12075d0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/apigee/terraform-modules/releases/tag/v0.2.1 :sunflower:


Can not work on object type

Thanks for your feedback, but what do you mean by "object type"?

Thanks for your feedback, but what does "object type" stand for?

@iendeavor  state some field is an object

Could you provide a codesandbox or reproduce steps?
I created an example for the object state and it seems to work well:
https://codesandbox.io/s/pinia-plugin-persistedstate-2-issues-5-y3dmb
src/stores/counter.ts:
import { defineStore } from 'pinia'

export const useCounterStore = defineStore('counter-store', {
  state: () => {
    return {
      path: {
        to: {
          count: 0,
        },
      },
    }
  },
})

src/app.vue:
<script setup lang="ts">
import { useCounterStore } from './stores/counter'

const counterStore = useCounterStore()
</script>

<template>
  <button type="button" @click="counterStore.path.to.count++">
    count is: {{ counterStore.path.to.count }}
  </button>
</template>

I found this issue when I was dealing with a similar-sounding problem.  Not sure if it is what the original poster meant or not, but this plugin doesn't support instantiated ES6 classes in the store.  They lose the class during serialization.
If the serializer for this plugin could be specified per-store, then it would be easy to build the objects based on known classes for the known data structure.
For me, I worked around it another way, but wanted to drop this comment in case anyone else came here with this problem.

@plainlystated  The per-store serializer option is defined in common options 😅:
https://github.com/iendeavor/pinia-plugin-persistedstate-2#common-options

@iendeavor Oh, nice!  I didn't realize you could use those common options on the stores.  Thanks for letting me know!

I refactored my code and got it working.  I like the approach.  Putting it below in case it's helpful to anyone that comes across this in the future.  (My JS/typescript is pretty rough but it works.)
  persistedState: {
    serialize(state: any): string {
      const simpleState: {items: CartItemParams[]} = {
        items: state.items.map((item: CartItem) => ({slug: item.product.slug, quantity: item.quantity}))
      }
      console.log('serializing', simpleState)
      return JSON.stringify(simpleState)
    },
    deserialize(value: string): {items: CartItem[]} {
      const ret = JSON.parse(value)
      ret.items =  ret.items.map((params: CartItemParams): CartItem => new CartItem(params))
      console.log('loading', ret)
      return ret
    }
  },

@iendeavor  Thanks for the help (and the project itself)!


Is it necessary to required by a settled RxSwift & ObjectMapper version

My podfile is

pod 'RxSwift',    '~> 3.0'
pod 'ObjectMapper', '-> 2.2.1'
pod 'Moya', '8.0.0-beta.5'
pod 'Moya-ObjectMapper'
pod 'Moya-ObjectMapper/RxSwift'

I got this error when pod install
[!] Unable to satisfy the following requirements:

ObjectMapper (~> 2.2.1) required by Podfile
ObjectMapper (~> 2.2.1) required by Podfile
ObjectMapper (= 2.2.2) required by Podfile.lock
ObjectMapper (= 2.2.1) required by Moya-ObjectMapper/Core (2.2)
Moya (= 8.0.0-beta.5) required by Podfile
Moya (= 8.0.0-beta.5) required by Podfile
Moya (= 8.0.0-beta.5) required by Podfile.lock
Moya (= 8.0.0-beta.3) required by Moya-ObjectMapper/Core (2.0.1)
Moya/RxSwift required by Podfile
Moya/RxSwift required by Podfile
Moya/RxSwift (= 8.0.0-beta.5) required by Podfile.lock
Moya/RxSwift (~> 5.3.0) required by Moya-ObjectMapper/RxSwift (1.1.0)
RxSwift (~> 3.0) required by Podfile
RxSwift (~> 3.0) required by Podfile
RxSwift (= 3.0.1) required by Podfile.lock
RxSwift (~> 3.0) required by Moya/RxSwift (8.0.0-beta.5)
RxSwift (~> 3.0) required by RxCocoa (3.0.1)
RxSwift (~> 3.0) required by RxDataSources (1.0.0)
RxSwift (= 2.0.0-beta.4) required by Moya-ObjectMapper/RxSwift (1.0.2)

This should be fixed by now, closing it.


Add subject: English language and literature

This is a common combined degree subject which has been entered 199 times as a UK degree subject.

Ideally we'd be able to list BOTH Hesa codes for the combined subject, but returning an array probably breaks a few things...

As this is a separate repo for reference data, I think this could include both HECOS codes now and Apply can decide to disregard them.

I still think this should output two HECOS codes before it gets merged.

@edwardhorsford I've added a comment to this for now, and have opened issue #14 to suggest allowing multiple codes.


feat: add NEIRO token

The image should be your token's logo rather than the axelar logo. Also your salt value is empty why is that? It looks to me that your salt value should be 5A1D9D229D60019372FC72391EB26FBA30FE36EABA97BA172FD1716454DAC962. Just from checking the deployTokenManager tx logs

Also missing coinGeckoId. See here for example

The image should be your token's logo rather than the axelar logo. Also your salt value is empty why is that? It looks to me that your salt value should be 5A1D9D229D60019372FC72391EB26FBA30FE36EABA97BA172FD1716454DAC962. Just from checking the deployTokenManager tx logs

Thanks for the review! I've done the requested changes


Database name should be required

DATABASE_NAME needs to be required because users can't make assumptions about 'default' database name. If we make them to specify the database name, then we don't need them to point to documentation for the default one.

@jhadvig fyi

@mfojtik the default name is production.  But if we decide to go this path Im fine with it.


Expose cpu usage in terms of cores

Cpu limits are exposed as milli cores, but cpu usage is in terms of nanoseconds. This makes visualizing these metrics together difficult.

CAdvisor is currently being updated to expose instantaneous usage.

Finally! What is the time interval that is used in the calculation of the instantaneous CPU metric?

cc @mvdan
On Fri, Sep 18, 2015 at 2:00 PM, Alexandros Mavrogiannis <
<EMAIL_ADDRESS>wrote:

Finally! What is the time interval that is used in the calculation of the
instantaneous CPU metric?
—
Reply to this email directly or view it on GitHub
https://github.com/kubernetes/heapster/issues/568#issuecomment-141567964
.

See https://github.com/google/cadvisor/pull/861. They are derived from the accumulated cpu usage, so they are updated exactly as often as the accumulated cpu usage is.

This is fixed in new Heapster.


Ramp to off will send Zigbee update at the start of the ramp

This is an interesting quirk of how C-Bus ramps work, @geoffwatts.
When ramping from on to a level of zero, a current level value of zero is sent at the beginning of the ramp, and also at the end of the ramp. (Which does not always happen. If the starting level is, say 10, then only one update is received.) This is the prime reason that I utilised an 'MQTT final' event script for acMqtt, which needs to be set to not execute during ramping.
The script will send zero to Zigbee only once, because this line, but it makes my eye twitch that it occurs at the start of the ramp and not the end.
if value == zigbee[event.dst].value then return end

There must be a simple solution, so it's on the to-do list.

And as expected there is a simple solution. I'll fix it.


Intent to drop the vendor/ folder

I'd like to drop the vendor/ folder containing copies of all required libraries used by restic. It was nice to be able to build restic from a checkout of the repo (using build.go) without having to fetch external dependencies, here's a list of downsides:

It blows up the repository and the source code release
It makes reviewing pull requests which add or update dependencies harder because many different files are changed
It creates merge conflicts for many files (in vendor) when master and a PR both change dependencies, in contrast to conflicts in just two files (go.mod and go.sum)

Before Go Modules, vendoring (directly or via a tool like dep) was required to manage dependencies. We don't need that any more once we raise the Version required for restic to 1.11. All libraries are fetched for the build process on demand, their integrity is verified by the Go compiler using the hashes contained in go.mod, so that's not an issue. For Go >= 1.13, the Go module proxy is automatically activated and we can add instructions the README for the other versions, GitHub repos going away is not a problem any more.
Several other tools have dropped the vendor/ folder since, and I'm not aware of any incidents or negative user feedback. Did you receive any for Caddy @mholt?
Any thoughts on that?

@ncw you have a vendor/ directory for rclone, what are your thoughts on that?

I think that while the reasons you outline for having had the vendor/ folder are noble, and I personally don't mind the size of the repo being large (I value it being completely standalone, not depending on anything else but Go), it's fine to make this change if you think it would clean things up.

I certainly think the writing is on the wall for the vendor directory!  In go1.13 it doesn't even use the vendor directory unless you set GO111MODULE=off or use go build -mod vendor.
For the rclone project I like to support 4 go versions which is 2 years of releases.  This lets people on older distros still compile rclone.  So when go1.14 drops, I'll be supporting go1.13, go1.12 and go1.11 which all support modules and then I can drop the vendor directory.
If I was just supporting go1.13 I'd probably drop the vendor directory now as it is actually quite painful to use.  I get contributors to make separate vendor patches so they can be dropped and rebuilt to help with the merge problems. In a long lived PR that sometimes happens several times. And as I said above, go1.13 doesn't touch the vendor directory by default so already your users will be downloading code to satisfy go.mod (unless you do something about that in the build process - I didn't check).
To summarise that ramble: if you don't want to suport go1.10 and earlier then I would drop the vendor directory :-)

So, Caddy was a bit unique because of its plugin model, and the transition to modules did cause a lot of grief for us because of that. But I think for most repos, it's pretty smooth.
Note that using modules is not mutually exclusive with using the vendor folder.
A couple things to consider, that we've had problems with (not just in Caddy, but at least in any of my projects):

Even with modules, dependencies can break you by force-pushing or re-tagging. We had this happen once. Naughty of them, but still required an (easy) update to our go.mod and/or our code to stay compatible.

go mod vendor can vendor your modules. Maintaining this, as @ncw mentioned, can be a bit of a bother.

Vendoring does not provide you any "supply chain" safety guarantees that are superior to Go modules. The only benefit I can think of to using vendoring and modules together is to guarantee that you can compile if dependency repos disappear or violate convention (i.e. force push or re-tag). But modules alone will prevent compilation if your checksums mismatch.

Having a vendor folder did cause bugs for us at times, due to how Go changes its identifier resolution logic in the presence of a vendor folder.

I personally am not a fan of vendor as a maintainer, but as a user I guess I don't care as long as it works and is well-documented (I actually don't remember all the time how to use go.mod with a vendor folder...)

Thanks for the feedback!

I certainly think the writing is on the wall for the vendor directory! In go1.13 it doesn't even use the vendor directory unless you set GO111MODULE=off or use go build -mod vendor.

Ah, that'll change for 1.14, it will use the vendor/ automatically if it is there: https://github.com/golang/go/issues/33848
So, overall I think we should drop the vendor dir. Let's see what users say ;)

I'm far far away to be a go expert but for what is worth I would go with the cleanest and easiest path for the project maintainers/developers.
Reading the comments from others maintainers it seems the smoothest path is to remove the vendor dir so +1 from me.

You could still make go mod vendor be a part of the source packaging procedure so that builds from source archives at least still are self contained and can easily be compiled when disconnected from the internet if that is something someone wants.

Good point, but before we do that (which makes the build and release process slightly more complicated) I need to know who actually uses that. So let's wait for user feedback. We can always revisit this decision later.

I really like the vendoring approach because everything is self-contained. When reading about removing vendor/ this came to my mind: https://www.sciencealert.com/how-a-programmer-almost-broke-the-internet-by-deleting-11-lines-of-code
In terms of security it might not be the best solution to keep local copies of unpatched code but this is more an issue of a package maintainer than the developers.

The Go universe has recently introduced a so-called "Module proxy", the most prominent instance is running at https://proxy.golang.org. It caches dependencies to achieve two goals:

During build, Go doesn't have to check out the code from each version control system
If the upstream goes away, the package stays in the Model proxy

It feels to me that Go has learned from npm and the leftpad disaster :)

Yeah, the only insecurities probably are google having issues or user not being able to access google services (china?) or google deciding to shut down their go mod proxy completely.

Yeah, the only two main insecurities are probably

Both go away if the source tarball is bundled with the dependencies in /vendor (a bot can upload this tarball for example for each release). The git repo IMO can stay without /vendor.

Since I've just looked it up, there are at least two different Module proxys independent of Google:

https://goproxy.io/ (apparently hosted by Alibaba)
https://goproxy.cn

Those can be configured via GOPROXY, e.g. export GOPROXY=https://goproxy.io,https://goproxy.cn,https://proxy.golang.org,direct.
When downloading code through these proxys, the Go compiler verifies the integrity of the code. Pretty cool :)


http2: removes environment variable-triggered nghttp2 logging

This logging is not covered by tests, and is less relevant with the HTTP/2 codec switchover.
The stringent code coverage limits have caused problems for past pull requests, so might as well eliminate one source of uncovered lines.
Commit Message: http2: removes environment variable-triggered nghttp2 logging
Additional Description:
Risk Level: low
Testing: ran integration tests and http/2 unit tests
Docs Changes:
Release Notes:
Platform Specific Features:

I found it useful to enable nghttp2 logs occasionally. Is there a way to enable detailed logs for oghttp2?
/wait-any

I found it useful to enable nghttp2 logs occasionally. Is there a way to enable detailed logs for oghttp2?
/wait-any

Yes, there's a QUICHE flag that can be enabled:
https://github.com/google/quiche/blob/7af0a9247627b3dea30d46605773179f1f994f9a/quiche/http2/adapter/oghttp2_session.cc#L346
I'm not sure how to set QUICHE flags for a locally running Envoy, but setting that flag should get you wire format events logged at the INFO level.


illegal characters in path

my windows build number = Creators Update
I want to debug linux application with WSL in visual studio
after following all tutorial video verbatim, none showed this error/ it meant to compile my hello world
Exception occured
while copying
sources to remote
machine-illegal characters
in path

Is your project location c:\connect and the project name ConsoleApplication2 per the video?

@Zedonboy - Could you please fill out our issue template?  We do not have enough information to reproduce this issue.

@benhillis The repro is create a project name with a space in it, in Visual Studio. That error message comes is in the MSVS output view. As dogfood, whoever maintains WSL's init could use MSVS for Linux, change the project name to wsl init (with resulting space in the executable), and you'll get the fail.

@therealkenc - Ok, so this isn't a WSL issue than but a Visual Studio issue.

Full disclosure I couldn't 100% confirm the behaviour on Real Linux because I upgraded my VM to Zesty, which doesn't appear to be supported for gdb reasons. But yes it sounds like the MSVC guys are just missing some quoting in their ssh command(s).

Well i have solved it........................
first i change "~/project" to "project" at the project environmental variables at the Remote directory
Previously i used Echo_Server as the name of my app....i guess thats a wrong naming convention.
i removed the hyphen it becomes "EchoServer"
Thank you @benhillis  @therealkenc. am grateful for your concern. :()

Might be worth posting over in the Visual Studio Feedback Center, if you feel like it. They do appear to act on the feedback there. Echo_Server is of course a perfectly valid name.

actually if you change the environmental variable only, another error will be generated
cannot create directories mkdir exit code 1
soo thats why i removed the hyphen from the name


Jira status update, management scripts and engagement description.

Jira Updates:

Added review process for validating tickets have been fixed correctly, if not the ticket can be re-opened
Created a management script for marking tickets to be reviewed and setting an alert

Management Script:

Export to csv will export the findings as a flat file

Engagement Description:

Added an optional engagement description field

Will followup with another PR for updating documentation.

Travis will probably fail b/c of S3 issues with AWS.

@devGregA Let me know if ok to merge.


Generate CMake files supporting all targets in realm-core

Allows you to build all realm-core targets without regenerating
CMake files.

Abandonned


feat: add slider

@iqbalem This is amazing, thank you! The only thing missing would be accepting multiple thumbs. Is that something you'd like to contribute with as well?

@TGlide sure, I'm on it but it may take time

Thank you! And no worries 😄

@TGlide done, also two new parameters added, orientation and disabled

@iqbalem I'm amazed, thank you! And of course, you can add the documentation whenever you like 😄

@TGlide if you don't like something about this pull request, please let me know.


Add current limiter and soft-start

https://au.element14.com/c/semiconductors-ics/power-management-ics-pmic/power-distribution-switches
https://www.digikey.com/products/en/integrated-circuits-ics/pmic-power-distribution-switches-load-drivers/726
https://www.digikey.com/product-detail/en/stmicroelectronics/VN7140ASTR/497-15994-6-ND/5431502
Example circuit

Schematics done in https://github.com/oxplot/fpx/commit/06e6602907f6ecd1e7ba80dc628277a3dda371b2


Typo in path to correspond with src/nls/root/strings.js

Thanks @dvorapa for catching this one.
Minor issues like this would have gone unnoticed.


NERDTreeFind on a new file in a new directory fails with E718: Funcref required

Steps to reproduce:

open some file
open nerdtree (e.g. with :NERDTreeFind)
open a new file with :e new-directory/new-file.txt
create the directory with :!mkdir -p %:h
create the file with :w
try to locate the file with :NERDTreeFind

What I expect:

nerdtree refreshes the parent directories and locates the file

What I get:
Error detected while processing function <SNR>107_findAndRevealPath[42]..236[23]..236:
line   23:
E716: Key not present in Dictionary: reveal(a:path, opts)
E15: Invalid expression: n.reveal(a:path, opts)
Error detected while processing function <SNR>107_findAndRevealPath:
line   44:
E718: Funcref required

This bug has been confirmed.

This was fixed as part of PR #785.  Things should work if you update the NERDTree.  Re-open if that's not the case.


Move ERC-7739 to Validators

Moved ERC-7739 from Nexus to validators (K1, Mock)

Added ERC7739Validator base contract
712 domain is taken from smart account in a validator when calculating hashes
Added supportsNestedTypedDataSign()  that iterates over validators

TODO:

[x] test supportsNestedTypedDataSign
[x] fix tests for k1 and enable mode that fail because validators now expect 7739 sigs
[x] Optimize hashing name and version in ERC7739Validator
[ ] Add test with module enable mode for account deployment userOp

needs to resolve conflicts too.

review i might need some help in debugging tests #PTSD

all tests pass. no debug needed.
all conflicts so far resolved as well I believe


String size issue

Hey! I've faced the issue today. You can't encode String more than 4046 symbols (In my case it's a JSON configuration file). After using a function on the Java side I do not receive the initial String, only part of it. Could you explain why and If possible provide some workaround?

Hi @dorozhans thanks for your message.
1/ Can you confirm me you succeed to get a smaller key without any issue ?
2/ I will investigate on large Strings to see what is possible to do.
3/ A possible workaround : separate your String in smaller Strings and then concat them ?

Yes, confirmed.
Thanks! :)
Yes, I thought about that, but I've started to find an issue in C code, probably you provide a small buffer. Will review your code too.

@dorozhans the issue does not appear to com from the Kotlin side, it can generate sha256 for 5000 characters string without any problem.
So it must be the C code as you thought, if you have any idea of a fix let me know (you can open PR).
We did not build this plugin to encode long files, but things like api keys. Can I ask you what is your use case that need to encode a full json file instead of obfuscated only the keys ?

I am receiving JSON string with configuration through the launching .sh script before app builds.
Yes, it was a mistake in the C code. I will provide a PR a bit later.

hello @dorozhans .
I think I found the fix to your issue by solving issue #26
From my new test encoding 5000 characters is now working. The issue was in the sha256 in the c++.
Could you try this solution and tell me if it is also fixed for you please ?
From the PR : #21 you can update your project by copying into your files :
in secrets.cpp function getOriginalKey
in sha256.cpp function void sha256(const char* input, char buf[2*SHA256::DIGEST_SIZE + 1])
From my tests it should solve your issue, can you try and tell me if I am correct ?
Thanks !

hello @dorozhans .
I think I found the fix to your issue by solving issue #26
From my new test encoding 5000 characters is now working. The issue was in the sha256 in the c++.
Could you try this solution and tell me if it is also fixed for you please ?
From the PR : #21 you can update your project by copying into your files :
in secrets.cpp function getOriginalKey
in sha256.cpp function void sha256(const char* input, char buf[2*SHA256::DIGEST_SIZE + 1])
From my tests it should solve your issue, can you try and tell me if I am correct ?
Thanks !

Fixed in release 0.1.1


SRS-06

Implement API click shorteners (visitors shorteners count will be updated each link is clicked)
Improve Generic Consumer Queues Functions (support multi queues running separated by different goroutines)
Add field timestamp to users & shorteners collections

LGTM 💯


Suggestion: Who's Talking?

Something that is quite helpful, is knowing who's speaking via voice in-game. Not too many scripts out there that have this functionality anymore, and those that do, don't exactly work the best. Implementing this in with the menu system would be super ideal. Perhaps even make it so the prefix can be customized when displaying in-game.

This is currently implemented in the menu when toggling the player id’s


context.json issues

Two:

[ ] Should be https (not http://bibdata.princeton.edu/context.json) in serialization
[ ] Should include Access-Control-Allow-Origin: *

curl -I https://bibdata.princeton.edu/context.json


Enable maxExpandFrames config option

@ofrobots PTAL.

LGTM once tests pass. Suggestion about error StatusMessage can be addressed in a follow on PR.

LGTM w/ comments.

Ugh. You'll have to use util._extend. I forgot that Object.assign has only been there since V8 4.5.


Support for dotLottie?

Hi, currently there is no way to use dotLottie files with lottie react native.
i think implemention is simple cause native runtimes support it.

This would be amazing, we have a lot of lottie's in our application and the size has nearly gotten to 26mb. So supporting dotLottie would help us reduce our bundle size drastically

Hi team, which version does this feat support?


Does not work  with Bulma UI

the css styles are clashing and I do not know where exactly

solved, it was a ul css rule


Nested group field inside a group field

Hopefully someone can help me out.
How do you grab the fields from a group nested inside another group?
The following works if you only have the one group:
{% set group = post.get_field('my_group') %}
{{ group.title }}

Could you share with us your ACF structure?

Hi Palmiak,
I've figured it out. Groups are within a repeater which had me stumped.
While in a repeater I had to make sure all my groups were within context.
If I wanted to grab the 'title' field from group2 I would do something like this:
{% for item in item.my_repeater %}
{{ item.group1.group2.title }}
{% endfor %}
Thanks for responding


CSS problems with several input types

I created a page and inserted custom HTML for a form, but the rendered result is unusable. You can see an example in the screenshots below.
With the following CSS snippet I was able to fix the width and padding a bit, but appearance doesn't work because it's not overridable.
input[type='radio'], input[type='checkbox'], label>input
{
    width: auto !important;
    padding: 0px !important;
    appearance: auto !important;
    -webkit-appearance: auto !important;
}

Screenshots

Thanks for the heads up! This is already fixed in the latest version.

Great! I'll try it out, thank you 👍


Share Plugin : Android Support for optional title, media URL and dialog title

Android support for sharing an optional title, and media url like external image url. Also supports optional title for Share dialog.
This is my first ever contribution. So please review and let me know your comments.

I signed it!

Can anyone please review and merge?

Can someone please review the PR?

@mravn-google @jakobr-google are you available to review this one?

@Purus Welcome to Flutter! We value your contribution and we'll review as quickly as we can :-)

@mravn-google PTAL

@Purus Any news on this? I see a few "Done" items above, but no new commits since the initial one.

@Purus Please let us know if you are able to upload the updated branch per the comments above; otherwise, no worries, we will take the PR over and land it in due course. Thanks for your contributions so far!

I'm going to close this PR since @Purus hasn't been able to update it. It's marked so that we will eventually get to it ourselves when we have the time. Thanks for your contributions; if you ever have the chance to get back to it, please don't hesitate to file a new PR and comment on this one pointing to it!


Research notebooks for SGD sampling with learning rate scheduling, hflip augmented sampling, and SGD Init + RmsLookahead finetune in sampling.

This is a rather interesting experiment where I applied learning rate scheduling to the sgd-based sampling to generate resulting images. This greatly improves upon the the baseline of just using a constant base learning rate, and it works well with 10 iterations.

Updating title to reflect the fact that I've added more research notebooks to this commit.

One more notebook:  SGD Init + RmsLookahead finetune in sampling.


Check box location incorrect if it is in live preview mode and generated by Obsidian tasks plugin

As title mentioned.
It is fine if I switch to the reading mode or if the tasks are not shown up by the query of this plugin. Fine if I use dataview plugin too.
The actually clickable box located somewhere in-between the visual box and tasks.

@yuiasora1024 Could you provide an example of this? Screenshots and markdown code to replicate the issue.

I actually need to click the space in front of the task to check it off.

There is a lot weird with the checkboxes in read mode, especially when the lists are indented:

Maybe the entire read mode code needs to be reviewed, but I don't know why one even uses the read mode after live edit came out. 🤔

It is not happens in read mode: if I switched to read mode everything is
fine.
But if I do it in live preview mode, it becomes weird.
「Emil Sundin @.***>」在 2022年11月7日 週一，上午9:30 寫道：

There is a lot weird with the checkboxes in read mode, especially when the
lists are indented:
[image: image]
https://user-images.githubusercontent.com/5908498/200208289-5807fc55-a123-4fa9-ac2a-1a1a8f5fa13a.png
Maybe the entire read mode code needs to be reviewed, but I don't know why
one even uses the read mode after live edit came out. 🤔
—
Reply to this email directly, view it on GitHub
https://github.com/bennyxguo/Obsidian-Obsidianite/issues/67#issuecomment-1304972136,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV6VDEWTQG25UEZ3BYZ7RF3WHBLSVANCNFSM6AAAAAARHDJ6HI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I can't reproduce this anymore. Now the clickable area around the checkbox is actually larger than the checkbox itself 😅


system-monitoring-center broke us

The URL lost a few  "_" so we lost the released version.

fixed in #523


Improve accessibility

make ARC usable for non-python users
reviewers have suggested GUI

After some consideration, a GUI seems to have several drawbacks in our case. Most importantly, arc has pretty minimal dependencies and is a relatively simple application that should run on anyones laptop. Adding a GUI would make the dependencies much more complicated, especially regarding platform support. Further, they tend to look rather unpolished if not maintained by a team with a graphic designer. There is not much to display anyway, accept maybe a window with logs and the output.

We instead opt for a simple command line interface with an extended help page and a pdf "cheat sheet".


[Content]:

What article on segment.com/docs is affected?
https://segment.com/docs/connections/sources/catalog/libraries/website/javascript/querystring/#:~:text=ajs_aid,call.
What part(s) of the article would you like to see updated?
This information in the doc isn't technically correct, or is at least potentially misleading, because using the word 'call' implies something happens as a result, but technically that's not always true. It returns a value when the function is called, but can also assign the value if slightly modified; both are really useful information to make visible to customers.
Current info :
ajs_aid | The anonymousId to set for the user. | This triggers an analytics.user().anonymousId() call.
What actually occurs when this function is triggered :
Returns current anonymousId value :
analytics.user().anonymousId()
=>'967dca40-48d6-435b-9d44-ef4d32057cf3'
Assigns anonymousId as the value passed in the function
analytics.user().anonymousId('1234-fghj-678')
=>'1234-fghj-678'
The table might be best updated to have the info above included. "This function returns the current anonymousId. When a value is passed, this function sets the anonymousId's value."
Additional information
CSE Liz Kane<EMAIL_ADDRESS>

@lizkane222 Thanks for this suggestion; we've added it to the docs.


Add async webhooks for transactionItem

Problem
We don't have async webhooks that could allow us to track the changes that happen on transactionItem.

### Tasks
- [ ] TBD

1. General Assumptions
The assumption of this RFC is adding the webhooks that would allow tracking the changes on transactionItem.
We will add 4 new webhooks:

TRANSACTION_AUTHORIZED - called when a transaction has been authorized
TRANSACTION_CHARGED - called when a transaction has been charged
TRANSACTION_REFUNDED - called when a transaction has been refunded
TRANSACTION_CANCELLED - called when a transaction has been canceled

Acceptance Criteria

[ ] Saleor has 4 new webhook events: transaction-authorized, transaction-charged, transaction-refunded and transaction-canceled
[ ] Each of the new webhook has a defined subscription
[ ] transactionInitialize, transactionProcess can trigger TRANSACTION_AUTHORIZED and TRANSACTION_CHARGED, based on the transactionEvent received from the payment-app
[ ] transactionEventReport triggers the webhook event based on the received input
[ ] transactionCreate, transactionUpdate trigger the event based on received input
[ ] Processing transaction-request-action (transactionRequestAction mutation), triggers the webhook.

2. API changes
Webhooks
Transaction-authorized
type TransactionAuthorized implements Event @doc(category: "Payments") {
  """Time of the event."""
  issuedAt: DateTime

  """Saleor version that triggered the event."""
  version: String

  """The user or application that triggered the event."""
  issuingPrincipal: IssuingPrincipal

  """The application receiving the webhook."""
  recipient: App

  """Look up a transaction."""
  transaction: TransactionItem!

  """ Amount related to the event"""
  amount: Money!
}

Transaction-charged
type TransactionCharged implements Event @doc(category: "Payments") {
  """Time of the event."""
  issuedAt: DateTime

  """Saleor version that triggered the event."""
  version: String

  """The user or application that triggered the event."""
  issuingPrincipal: IssuingPrincipal

  """The application receiving the webhook."""
  recipient: App

  """Look up a transaction."""
  transaction: TransactionItem!

  """ Amount related to the event"""
  amount: Money!
}

Transaction-refunded
type TransactionRefunded implements Event @doc(category: "Payments") {
  """Time of the event."""
  issuedAt: DateTime

  """Saleor version that triggered the event."""
  version: String

  """The user or application that triggered the event."""
  issuingPrincipal: IssuingPrincipal

  """The application receiving the webhook."""
  recipient: App

  """Look up a transaction."""
  transaction: TransactionItem!

  """ Amount related to the event"""
  amount: Money!
}

Transaction-canceled
type TransactionCanceled implements Event @doc(category: "Payments") {
  """Time of the event."""
  issuedAt: DateTime

  """Saleor version that triggered the event."""
  version: String

  """The user or application that triggered the event."""
  issuingPrincipal: IssuingPrincipal

  """The application receiving the webhook."""
  recipient: App

  """Look up a transaction."""
  transaction: TransactionItem!
}

Additional use case / event: TRANSACTION_METADATA_UPDATED to inform about metadata changes. Its used to sync with external systems that some custom fields were written to transaction event (e.g. payment details like card vendor)


topbar menu-icon does not display on Android devices

There is a bug on Android devices
box-shadow with no blur does not display
so instead of using the foundation box-shadow method to display the hamburger icon I just use the fa-bars icon from font awasome
hope it helps
regards
Serge

Adding the -webkit prefix doesn't solve it either. Looking into another suitable fallback.

MULTIPLE box shadows are not supported in 2.3

It looks like addressing this would be a breaking change where the mixin and markup would need to be written. We did add the vendor pre-fix to for good measure e4403ce
In Foundation 6 we are addressing this to make sure Android 2.3+ displays the icon.


NetworkMessages not being found in Writer in Unity 2020.3.0

Describe the bug
I've raised this concern on discord and someone had responded in letting me know that this was a known issue, and since I did not see it on github, I thought it may save someone some time in knowing it is a known issue.
Firstly, when updating from 2019.4.22f1 to 2020.3.0f1 I noticed that my tests were failing as it couldn't find ANY of Mirror's writers/readers. As in things like SpawnMessage would say implement a custom writer.
How to reproduce the issue, step by step
Have a simple test that uses Mirror in version 2020.3.0f1
Anything that uses a Command
We will 100% look into every bug report. but if you aren't around and we can't figure out how to reproduce your bug then we can't fix your bug.
Expected behavior
Expected Mirror NetworkMessage types to be weaved and populate the Writer singleton.

Is this the same as #2579 ?  If so, please close this.

Yea, that’s the same issue. Closing

Thanks...if you have any more information to contribute, please comment on that issue.

To everyone following this ticket, please try master branch as soon as possible to confirm we have this fixed, and come to Discord and tell us if not. Thanks!

To everyone following this ticket, please try master branch as soon as possible to confirm we have this fixed, and come to Discord and tell us if not. Thanks!


[backport] Add env var to disable RPATH

Backport of #7691 by @kmaehashi

/test mini

Test failures are unrelated.


HIVE-27537 : Backport of HIVE-19416

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

Is the change a dependency upgrade?

How was this patch tested?

@sankarh @vihangk1 Can you please review this

@amanraj2520

Noticed several partial cherry-picks. Can you add details of trade-offs, what is skipped in cherry-picks and why is it not impacting overall functionality?
Why there are custom test output fixes? Share justification for the same.
Re-generate thrift files instead of cherry-picking those changes to ensure it is truly compatible with new thrift version.
Update the title, description to list all the JIRA IDs involved in this backport.

I will add them.
There are changes because the column size has changed. Because of a new column of write_id. But will add justifaction for some of them.
This is already done.
Sure will do this.


key combinations would be nice, such as Shift+a to represent capitals and line breaks

Could you add key combinations so that we can get the capital letters and things like hidden line breaks (shift enter)?
a: 97
Shift+a = A: 65

What you are talking about is the charCode property which represents the unicode value of the pressed key, which is considerably different to the keyCode property, a mistake easily (and often) made 😄
Check out https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/charCode and https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/keyCode

Closing this for now, but we might add this feature in future release.
Regards!


Programming exercises: Fix an issue with user independent repository urls in double blind assessments

Several improvements for participations

Significantly reduce response size for participation component (fixes #4488)
Calculate the user independent repository URL server side (fixes #4489)
Improve code Quality in ParticipationResource
Remove lots of client side boilerplate code

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added @PreAuthorize and checked the course groups for all new REST Calls (security).
[x] I implemented the changes with a good performance and prevented too many database calls.
[x] I documented the Java code using JavaDoc style.

Client

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.

Changes affecting Programming Exercises

[ ] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[ ] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context

Description

Steps for Testing

Prerequisites:

1 Instructor
1 Student
1 Tutor
1 Programming Exercise with manual assessment enabled

Instructor: Create a programming exercise
Student: Participate in the programming exercise (check the json call does not include exercise or course objects)
Instructor: Navigate to the participations page and check everything is working correctly (including Go to repository)
Tutor: Navigate into the manual assessment of the programming exercise and check that the url behind Go to repository does not include a username

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage
no real changes (might slightly increase because unused code was deleted)

Tested this on TS2.
2: The call POST https://artemistest2.ase.in.tum.de/api/courses/3816/exercises/9809/participations sent when starting the exercise still got the exercise, course and student objects in the response from the server. Intended?
3: There is no "Go to respository" in the participation list view, but the Repository link works (no user prepended), Commit Hash links in individual submissions you can list from there work fine too.
4: As a tutor, the "Go to repository" link seems to be fine. No user prepended (checked in dev tools as well).

I further improved the code quality in ParticipationResource. Had to do some refactoring for it. Would be great to get some additional tests

Testing steps work as expected on a Gitlab/Jenkins setup.
However, there is an issue when updating the presentation score or individual due date on the Participation page: The server endpoints expect that the exercise is set in the participation (ParticipationResource lines 242 and 282).

Either the check on the server can be removed and be replaced by only checking that the participation is part of the correct exercise after fetching from the database.
Alternatively, the known exercise can be added back into the participation in participation.component.ts in the methods addPresentation, removePresentation, and saveChangedDueDates.

Thanks, I fixed it in the last commit and now reconnect participation and exercise in the client service. Please have another look and let me know if it works correctly now.

@dfuchss @b-fein Thanks for the feedback, I applied it, please have another look and approve if everything is fine now. Thank you!


logs not showing correctly when using sam

Description
When using sam invoke, the output of console.log is not displayed correctly as described in this issue.
Steps to reproduce
console.log('line one\rline two\rline three\nline four\nline five')
will output

START RequestId: ac1ef293-d504-16a9-6b21-20a04c7f98e6 Version: $LATEST
line fiveeT14:12:51.527Z        ac1ef293-d504-16a9-6b21-20a04c7f98e6    INFO    line one
END RequestId: ac1ef293-d504-16a9-6b21-20a04c7f98e6

and
console.log(json.stringify({one: 'line one', two: 'line two', three: 'line three', four: 'line four'}, null, 2))
is just as problematic. One can only read the first and last lines, usually { and }
According to @mhart

This is due to some unfortunate internal changes in the runtimes where console.log is overwritten by a function that transforms all newlines to \r.

Additional environment details
os: OSX 10.14.6
sam cli versions tested: 1.3.0, 1.12.0, 1.13.2

I'm also seeing this, but NOT related to use of SAM. Just using a docker image (extended from FROM public.ecr.aws/lambda/nodejs:latest) and manually using curl to test as described here.
The JSON object being logged (in my case the event) seems to be fine, it's just missing several lines of the multi-line log message (and seems to be overwriting lines that have already been written previously), so it seems to only be an output problem (everything functional is fine).

This one statement is supposed to be logging a JSON event object containing multiple layers of data. As seen in the screenshot:

It is clearly not a valid JSON object being output
The "closing" braces appear EARLIER on the same line of a wrapped url string, when in reality they should be lines later in the output.
The prefix "EVENT" doesn't even make it into the final output.

I'm also seeing this, but NOT related to use of SAM. Just using a docker image (extended from FROM public.ecr.aws/lambda/nodejs:latest) and manually using curl to test as described here.
The JSON object being logged (in my case the event) seems to be fine, it's just missing several lines of the multi-line log message (and seems to be overwriting lines that have already been written previously), so it seems to only be an output problem (everything functional is fine).

This one statement is supposed to be logging a JSON event object containing multiple layers of data. As seen in the screenshot:

It is clearly not a valid JSON object being output
The "closing" braces appear EARLIER on the same line of a wrapped url string, when in reality they should be lines later in the output.
The prefix "EVENT" doesn't even make it into the final output.

Any update on this? Quite annoying to see logs printed like this.

I'm also seeing this issue under this environment:
SAM CLI, version 1.19.1
OSX 10.14.6
It would be great if this issue could be fixed asap. Thanks

also seeing this -
SAM CLI, version 1.20.0
OSX 11.2.2

As a workaround:
sam local [invoke | start-api] 2>&1 | tr "\r" "\n"  fixes the issue for me. Found the solution here: https://github.com/aws/aws-sam-cli/issues/1359#issuecomment-744469252
Thanks to @code-is-art for the workaround

Is it possible to use this workaround in windows?

also seeing this -
SAM CLI, version 1.20.0
OSX 11.2.2

As a workaround:
sam local [invoke | start-api] 2>&1 | tr "\r" "\n" fixes the issue for me. Found the solution here: aws/aws-sam-cli#1359 (comment)
Thanks to @code-is-art for the workaround

Anyone know if it's possible to use this workaround in windows? When I run it in CMD it does not start and I get no output.

cross posting sorry https://github.com/aws/aws-sam-cli/issues/1359#issuecomment-831837585
Hi there,
I fixed the issue for us by adding "test": "sam local invoke ... 2>&1 | tr "\r" "\n"" in our package.json script section.
However, this issue has been opened for two years and it seems like it should be fixed, can you either give a reason not fix the issue and add a gentle note to the sam cli documentation (with that workaround), or fix it?
If not planning to fix, adding a note to the docs will save time to figure it out -- many people like me could waste a lot of time thinking they are doing something wrong.
thanks

Looking for a workaround for windows as well. tr isn't a natively available command in windows shell/powershell.

sam local [invoke | start-api] 2>&1 | % {$_.ToString().replace("`r`n","`n")}
If I put into package.json, it does not work:
  "dev": "sam local start-api 2>&1 | % {$_.ToString().replace('`r`n','`n')}"

npm run dev  --------> Error

Possible workaround from codebase side is just replace console.log by process.stdout:
process.stdout.write(JSON.stringify(event, null, "  ") + "\n");

process.stdout still doesnt work

This is still an issue in Feb 2023 with macOS Ventura 13.1 and SAM version 1.73.0

Just commenting that this still seems to be an issue. Not using SAM just running it through a regular docker container it appears that console.log get's cut off at one line. It does seem that process.stdout.write is a work around but of course less than convenient.

This is still an open issue. Hoping we can get a fix?

In Windows, still not working :/

So it turns out.. the good old print statement works just fine.


add task result to task_command output

Thank you for contributing to the CF CLI! Please read the following:

Please make sure you have implemented changes in line with the contributing guidelines
We're not allowed to accept any PRs without a signed CLA, no matter how small.
If your contribution falls under a company CLA but your membership is not public, expect delays while we confirm.
All new code requires tests to protect against regressions.
Contributions must be made against the appropriate branch. See the contributing guidelines
Contributions must conform to our style guide. Please reach out to us if you have questions.

Where this PR should be backported?

[ ] main - all changes should by default start here
[ ] v8
[ ] v7

Description of the Change
We must be able to understand the design of your change from this description.
Keep in mind that the maintainer reviewing this PR may not be familiar with or
have worked with the code here recently, so please walk us through the concepts.
Why Is This PR Valuable?
What benefits will be realized by the code change? What users would want this change? What user need is this change addressing?
Applicable Issues
List any applicable GitHub Issues here
How Urgent Is The Change?
Is the change urgent? If so, explain why it is time-sensitive.
Other Relevant Parties
Who else is affected by the change?

V8 PR for:
https://github.com/cloudfoundry/cli/pull/2675

Hi @saescha,
Looks like some integration tests are failing, can you please have a look?


#1 Prevent dead users from shooting. Refactor irEvent

I only made shots register if they dealt damage - I didn't see the use case otherwise.
Please let me know if this was a wrong assumption

I dont see any problem with the irEvent refactor currently.


Random Battle: Tweak movesets slightly

Lycanroc-Midday learns Drill Run.
Lycanroc-Midnight learns Fire Punch.
Tapu Fini learns Defog.

Is it worth running Defog though? It wants Calm Mind, and if it's not Calm Mind then it wants Specs.

Defog is standard on utility Fini, but I guess it isn't that great for randos. Took it out and kept the Lycanroc changes.


License information update

PR Description:
This PR is to update the license information of this project along with its dependencies
License file information in README file is updated accordingly
Project license is updated from MIT to Apache Version 2.0 considering the project use cases and deployment scenarios.

@swaminathanvasanth @abhi4578 Please have a look

@swaminathanvasanth @abhi4578 Dependency licenses file updated with all the licenses related to servers and client packages, pls review


fix: upgrade to version 3.0.604

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rajdavies
You can assign the PR to them by writing /assign @rajdavies 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

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
4fb5ed07e68977d569e10b17d234868a71915c41
link
/test gsm

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 jenkins-x/lighthouse repository. I understand the commands that are listed here.


Added the option to numerate lessons

I added the option to number the lessons in the order they were uploaded on webex.
It's similar to the prepend_date but it uses the total number of lessons uploaded (from recordings.totalCount) instead of the date and time.
It uses a variable tat you can place in the courses section (putNumberLesson).

I imagine this could be helpful for ordering and quickly finding the lessons but as of now there are some edge cases that wouldn't be covered.
I've seen professors which sometime delete some recordings, or periodically delete everything. With no absolute reference to the lesson number, in the best case, as when all past recordings are deleted, there are going to be duplicates having the same index. In the worst case, as when only a lesson is deleted, all the lessons afterwards change index and are therefore re-downloaded as the existence check is based only on filename.

There is also to consider what to do with filtered recordings.
recordings.totalCount is the total number of recordings with the filtered ones included, but recordings.recordings doesn't include filtered recordings and could therefore be smaller in size.
If the course filter where to change, the indexes prepended to the lessons could change to and the re-download problem mentioned in the previous problem would arise again.
Do you have a particular use case for these incremental indexes?

Yes, i didn't think about the possible edge scenarios because i wanted just to download the whole last-year course (because i would like to be sure to have the lessons available even if the professors decided to remove them) and have them in order without my intervention.
I used it in some courses that had some lessons removed and it caused no problems as it skipped the missing recordings while still counting correctly.
And yes, it isn't retroactive and it doesn't have any knowledge on the previously downloaded lessons.
The intention of this feature is just to be able to view the lessons in a chronological order when using the file explorer.
I didn't even think of the course filter🥲
Also, I had another """"feature"""" in mind (that I know is just for my for the files that are unavailable to download: create a simple text file with the name of the recording that caused the error so that you could then look for it on webex without having to look at the log

Just to be clear:
The use case is this:
I need to download the whole course. So i need all the lessons available. They won't change as the course has already ended. I need them downloaded because i don't want to rely on webex and his cloud to stream them. I also don't want to manually rename them in the correct way because it would be too time consuming (i downloaded using this feature 10 courses)

I see, wouldn't the prepend_date feature already sort the lessons chronologically? As they are formatted in YearMonthDay an alphabetical sort should work fine. Doesn't it work in your file explorer?
Here is an example:
20210929-Lezione_29_settembre-20210929.MP4
20211001-Lezione_1_ottobre-20211001.MP4
20211006-Lezione__6_ottobre-20211006.MP4
20211015-Lezione_15_ottobre-20211015.MP4
20211020-Lezione_20_ottobre_-20211020.MP4

Most of the time yes, but my beautiful professors decided to split the same day lesson in two parts with the same name (and thus webex adds an index to the end).
I found it more convenient to just add the number in front (also because this files will be shared to my friends and i wanted a fool-proof method to know which lesson is before the other.

Also, I had another """"feature"""" in mind (that I know is just for my for the files that are unavailable to download: create a simple text file with the name of the recording that caused the error so that you could then look for it on webex without having to look at the log

Just to be clear, the tool already downloads all possible recordings, resorting to downloading the stream in case the download feature has been disabled. Other than errors due to connection problems in the middle of the download, the only lessons not downloaded are the one for which both viewing and downloading has been disabled.
The initial idea was so that by running the tool again you could retry to download the missing recordings which previously failed. A log file for only failed downloads, to be saved for each course separately in its download folder; could be done. Containing only recordings not viewable/downloadable, would the log still be of help?

Most of the time yes, but my beautiful professors decided to split the same day lesson in two parts with the same name (and thus webex adds an index to the end).

Oh right! That also happens... In my case the professor left the same name to both part of the lesson therefore they are sorted correctly anyway.
20220302-Teoria_2_-_NomeDocente-20220302_0950-1.MP4
20220307-Teoria_3_-_NomeDocente-20220307_0920-1.MP4
20220307-Teoria_3_-_NomeDocente-20220307_1041-2.MP4
20220309-Teoria_4_-_NomeDocente-20220309_0956-1.MP4

In your case, did the name change? If you could share the recordings names used, it would be helpful insight. Anyway, the incremental index doesn't convince me, but looking through webex metadata, each lesson has assigned an incremental id. As I imagine it's managed by webex, I see no problem on prepending that as a more stable alternative. What do you think of it?
Note: for the recordings listed above, the id is a 7 digit number in the range 4000000-5000000

The IDs I'm referring to are visible in the response of the initial request to https://lti.educonnector.io/api/webex/recordings

The initial idea was so that by running the tool again you could retry to download the missing recordings which previously failed

Oh, ok that was wayyy easier. As it would resolve the missing download issue. (or the too many requests error that i hit while downloading way too many recordings all at once).
Probably would be better a txt log file with the entries of the error messages of the various recordings.
But I still think that the number up front is the simpliest and most user friendly option to understand which lesson is first. Though the prepend_date has the same identical purpose.

Just to be clear, the tool already downloads all possible recordings, resorting to downloading the stream in case the download feature has been disabled. Other than errors due to connection problems in the middle of the download, the only lessons not downloaded are the one for which both viewing and downloading has been disabled.

Yes of course. But

Oh right! That also happens... In my case the professor left the same name to both part of the lesson therefore they are sorted correctly anyway.

Yes this is the default approach i think webex uses to sort this kind of problems.

Oh right! That also happens... In my case the professor left the same name to both part of the lesson therefore they are sorted correctly anyway.

Also true

the incremental index doesn't convince me, but looking through webex metadata, each lesson has assigned an incremental id. As I imagine it's managed by webex, I see no problem on prepending that as a more stable alternative. What do you think of it?
Note: for the recordings listed above, the id is a 7 digit number in the range 4000000-5000000

Yea i saw those when i tried doing this myself before finding this incredible tool.
Regarding the index issue, the use of the webex index would be awesome as it resolves the issue of the deleted lessons while maintaining a chronological order. But for my classmates i will still use the incremental id as it's easier to understand.
Couldn't be possible to create like a json file to place in the same download folder to account for this kind of deletion so that it could keep track of the last downloaded lesson and e.g. assume that's the last one and to not redownload the whole course if one was deleted. (I know that this is way overengeneered and that by using the webex ids or just the date the problem would be insignificant, but i reeeeeally like the incremental IDs)

Thanks, nice to see that the tool is also of help to others!
Sure, it would be possible to keep track and rename all files accordingly, but I don't think it would be worth it as it adds a complex layer prone to errors and changes.
The incremental IDs that would be added are volatile and not unique as the timestamp or the webex assigned id. It works for one-time downloads of already archive courses, but not for the ongoing ones. Also, trying to specify in the documentation all the edge cases to use this feature correctly, would make the configuration step less user-friendly.
On a note, just realized that using the webex IDs would mean relaying on the fact that webex properly assigns them in order and never modifies them. I don't really like this dependency.
As for the solution I'd add the option to choose between prepending:

Only the date
Date + Time

The latter solves any ordering problem that comes to mind rn.
The incremental IDs is something that you could do externally with a script; after a course finished, and before archiving it.

Probably would be better a txt log file with the entries of the error messages of the various recordings.

To prevent further confusion, tracking the suggestion here #11 . I'll get around it soon, hopefully

Sure, it would be possible to keep track and rename all files accordingly, but I don't think it would be worth it as it adds a complex layer prone to errors and changes.

Not worth the effort I agree.

On a note, just realized that using the webex IDs would mean relaying on the fact that webex properly assigns them in order and never modifies them. I don't really like this dependency.

Right

As for the solution I'd add the option to choose between prepending:
Only the date
Date + Time

Why not add also the option for incremental IDs? and add in the documentation that it should be used at your own risk and that it can break things if a course was to be removed.

The incremental IDs is something that you could do externally with a script; after a course finished, and before archiving it.

I agree but why using an external script when you are already renaming them in the first place. It consolidates all in one single config file and process


Add reqwest pass through features

Thanks for your contribution! All looks good to me!
What do you think, @ggriffiniii ?

First thanks for the PR. I'm curious what your personal use case is for the features as I could see a few different possibilities here. While adding pass-through features like in this commit, there are still several configuration settings that users may want to provide for a reqwest client that are configured via a Builder at runtime and I would like to provide a mechanism for users to be able to control both.
One possibility would be to provide a Client::with_reqwest function, that accepts a Reqwest::Client provided by the user. The user would then be able to provide a Client with whatever features they choose (both compile time and runtime). This would make cargo compile reqwest with the union of the features specified by the client library and whatever features were enabled by the end user. To allow users to avoid an openssl dependency I would opt the client to use the rustls feature of reqwest by default.
Would the above satisfy your requirements?

I just committed https://github.com/google-apis-rs/generator/commit/7504e310e8c4c361aca998ea14c309d5c46967cc which implements Client::with_reqwest_client as described above. Let me know if that solves your issue. I'm perfectly willing to reconsider that approach, but it was straightforward to implement so I figured it would be nice to have something more concrete than my description above. Does it satisfy your requirements?

I am using this in a project that already uses reqwest (with rustls) that has 350+ dependencies and takes ~3 minutes to cold compile (debug) on an 8 core machine with 32gb of RAM and 3-10 seconds to run check (which my editor does via rust-analyzer on save)... so I am just trying to pick low hanging fruit to reduce my dependencies to keep compile time manageable. It is an uphill battle though because not every pruned dependency actually reduces compile time... 😆
That being said, adding Client::with_reqwest is awesome to have regardless because then we can reuse the same client (for connection pooling etc...).

It would still be handy to turn rustls-tls on and off I think.


Can the bot stay longer?

The number of viewers goes up and down quickly. Can I keep this for a long time?

just buy expensive proxy list


accounts CSP issues

In accounts there are CSP issues, probably are related with https://github.com/inveniosoftware/invenio-admin/pull/57

Screenshot?

@drjova do you think we can close this issue now?

@hachreak IMHO we have to double check first if it is related with the flask-admin CSP issue


remove getHashFromHeight & updates for getBlockHash

Issue being fixed or implemented

remove getHashFromHeight from insight and core APIs ( it's never used in dapi-client and it has the same functionality as getBlockHash)
remove getBlockHash from insight to not support unused code

What was done
only getBlockHash from core API remained.
What has been tested
test code updated, integration tests were run
What needs to be tested
no additional verification required ( unit and integration tests should pass)

@abvgedeika Is it ok to just remove methods without deprecation process?

@shumkov what is our deprecation process? these methods are internal(getHashFromHeight never called via rpc calls to dapi, getBlockHash uses core API and don't use insight API)

@abvgedeika Oh, my bad! You right!


Support new TEST_OUTPUT task result

Fall back to the old name if required to make the transition more smooth. Eventually we can remove the deprecated name entirely.
JIRA: HACBS-2159

Codecov Report

Merging #449 (2a6f960) into main (01a391c) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 2a6f960 differs from pull request most recent head 5ef45db. Consider uploading reports for the commit 5ef45db to get more accurate results

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #449   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           67        67
  Lines         1917      1928   +11
=========================================
+ Hits          1917      1928   +11

Impacted Files
Coverage Δ

policy/release/lib/attestations.rego
100.00% <100.00%> (ø)

policy/release/lib/attestations_test.rego
100.00% <100.00%> (ø)

Let's merge to unblock https://github.com/redhat-appstudio/build-definitions/pull/392 .


-cah / -cah help

Using -cah and -cah help as alliases to -help cah would be nice.

done in https://github.com/jonas747/yagpdb/commit/a95d0519a6552a9b4fe246b68d8f17153966e8f5


Issues with Changing Music / Intro Dialogues

Description of Problem or Question
There appears to be a bug in 2.1.5 with how music works (that was not there in previous versions).
If you play a custom music track, if another event attempt to play while it is "fading out" it will not play and will default to the default music (or the last playlist you played).
I have attached an example scenario.
Valkyrie Version
2.1.5
Attach Save, Quest or Scenario
Example, however this is happening during the intro of "A Strain on Reality" too
SoundBug.zip

It's fixed.
I'm pretty sure it never worked properly.


How to set background color of custom badge?

I wanted to try out custom badges and tried to set a different background color.
I failed and don't know why.
My documentation contains:
|Styled differently|by custom HTML classes|class:version|

My docs/overrides/extra.css contains
.version {
  background-color: #FF0000;
  color: #FF0000;
}

My mkdocs.yml contains the theme attribute custom_dir: docs/overrides.
But still my badge doesn't change:

What am I doing wrong?

Hi, I have added a page detailing how to customise the styles to the documentation. If that does not work for you/ you find any errors, please let me know by commenting here :)

This helped and is working, thanks a lot!
In my case the custom css was wrong.
It had to be
.badge.myclass .title {
  background-color: #FF0000;
  color: black;
}


Move Gydo to alumni ...

Was already making more changes. Will take those over.
What about your Stanford affiliation?

True, it is actually a joined position with Schrodinger, Stanford and UCSF.
Still, I think this should do it, my contract after all is with them …
On 4 Oct 2016 7:53 a.m., "Alexandre Bonvin"<EMAIL_ADDRESS>wrote:

Was already making more changes. Will take those over.
What about your Stanford affiliation?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/haddocking/haddocking.github.io/pull/122#issuecomment-251367204,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFNIFsb5HtGLH8uY9s6umGGfM1PPUru9ks5qwj5MgaJpZM4KNlCy
.

Ok - whatever you prefer, but showing Standord and UCSG might be good :-)

I guess youre right :-). The details of Stanford can be copied from Joao,
and for UCSF
UCSF, San Francisco, CA.
:-)
On 4 Oct 2016 8:00 a.m., "Alexandre Bonvin"<EMAIL_ADDRESS>wrote:

Ok - whatever you prefer, but showing Standord and UCSG might be good :-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/haddocking/haddocking.github.io/pull/122#issuecomment-251368427,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFNIFkT7p6apawm_BBWE68o9EDAvQjYZks5qwj_pgaJpZM4KNlCy
.


"is not a function" errors

Thanks for this brilliant project.
I encountered some errors trying to start a service.  Found the missing definitions in your other repo `webapp.  Specifically, language.js
Will you be including language.js to the web-service repo?
In my entry.js, I added the following to get my app going:
global._development_ = true;
global.log = { error: console.error };

// https://github.com/halt-hammerzeit/webapp/blob/master/code/common/language.js#L253
Object.defineProperty(Array.prototype, 'is_empty',
{
  enumerable: false,
  value: function() {
    return this.length == 0
  }
});

// https://github.com/halt-hammerzeit/webapp/blob/master/code/common/language.js#L347
Object.defineProperty(String.prototype, 'ends_with', {
  enumerable: false,
  value: function(substring) {
    let i = this.length
    let j = substring.length

    if (j > i) {
      return false
    }

    while (j > 0) {
      i--
      j--

      if (this[i] !== substring[j]) {
        return false
      }
    }

    return true
  }
});

// https://github.com/halt-hammerzeit/webapp/blob/master/code/common/language.js#L3
var exists = what => typeof what !== 'undefined';
global.exists = exists;

Wow, thanks )
Sure, i'll fix the bugs you found, that helps.
I guess it will take about 10 minutes.

@once-ler Okay, I seem to have fixed all the bugs you've found so far. Try npm install web-service@latest and tell me if it works for you now.

@halt-hammerzeit Thanks for (super) quick response.
Works out of the box with web-service@latest


CPLP-613: implemented keycloak api call and limiting count of passwor…

https://jira.catena-x.net/browse/CPLP-613

a more general remark: please ensure proper formating. Indentation shall always be 4 chars, properly aligned.


Interoperability with TaylorSeries

This is to introduce a TaylorSeries.jl extension.
No changes are expected on Ephemerides.jl, just some overloads.
FYI @PerezHz

Many thanks @andreapasquale94! We'll take a look into this and report back! cc: @lbenet @LuEdRaMo


Add http connection pool between replicas

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
I don't know what is better default for per endpoint connection pools?

I don't know what is better default for per endpoint connection pools?

I also don't know. But I think that 4 parallel transfers should be enough to utilize 10 GBit network even with moderate packet loss.


The dist folder is missing in the npm package for v1.7.0

I downgraded to v1.6.0 and it showed up fine. Not sure if it something wrong on my side, but it costs nothing to warn.
cc @xavierfoucrier

@leonardoventurini 👋
Woopsi! Good catch! 🔥
I will publish a minor release soon to fix that.
Thanks for using mojs 🎉


Sui name service  - devnet

error message when registering name

keepsake -  issue says all items already purchased but are not :)

hi randall
tried this today and its still getting same message as above :)

This is an issue with the SuiNS Dapp, I'd recommend reaching out to them for support.


Options not working since 11.0

2x Blizzard_Settings_Shared/Blizzard_Setting.lua:9: 'name' for variable 'AutomaticRoleCheck_Options.Enabled' requires string type.
[string "=[C]"]: in function error' [string "@Blizzard_Settings_Shared/Blizzard_Setting.lua"]:9: in function <Blizzard_Settings_Shared/Blizzard_Setting.lua:7> [string "@Blizzard_Settings_Shared/Blizzard_Setting.lua"]:205: in function Init'
[string "@Blizzard_SharedXMLBase/Mixin.lua"]:28: in function CreateAndInitFromMixin' [string "@Blizzard_Settings_Shared/Blizzard_Settings.lua"]:184: in function RegisterProxySetting'
[string "@AutomaticRoleCheck/AutomaticRoleCheck_Panel.lua"]:11: in function `RegisterCheckbox'
[string "@AutomaticRoleCheck/AutomaticRoleCheck_Panel.lua"]:15: in main chunk
Locals:
(*temporary) = "'name' for variable 'AutomaticRoleCheck_Options.Enabled' requires string type."

@TheAzkas
Thank you for your issue. Please note that the current version does not support 11.0.0 (the active patch as of writing this comment) due to significant API changes in the new patch.
However, it does support 11.0.2, which is not yet an active retail patch. This means your issue will be resolved between now and the release of TWW.
Until then, the addon is functional, though it does generate errors for the options screen located in your Addons. You likely do not need to use this screen if you use the commands instead.
Sorry for the inconvenience, and thanks for using the addon.

@TheAzkas FYI: The 11.0.2 patch is now an active retail update, so the issue should be resolved!