affected/package: net/http

What version of Go are you using (go version)?

$ go version
go version go1.19.3

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
Windows 10 x64
What did you do?
http.Get("https://some.website.intranet")
What did you expect to see?
HTTP Response or HTTP error
What did you see instead?
Get "https://some.website.intranet": dial tcp 172.X.X.X:443: connectex: An attempt was made to access a socket in a way forbidden by its access permissions.
If i do the same for external website (e.g google.com) I don't have this issue. I suspect it's because it has to go through my VPN (I use Win 10 VPN client) but I don't know why I have this error because http get shouldn't try to bind to a port of my Windows right?
Could you please enlighten me on this issue?

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Panning capabilities to move side menu (UIPanGestureRecognizer)

It would be great if this great component could open and close according to user finger location - UIPanGestureRecognizer

@LudaFuxPlaybuzz This would be great!! .. have you modify to do it or something ?? @evnaz looks no having much time at the moment.. :/


Feature/job paramter

hello @appleboy ,
i will open a PR for feedback.
The tests are running in my lokal setup well and also in my drone.ci.
You found an CSRF handling; the options to push values into the job and an full scripted jenkins test setup.
Are there also changes in the plugin interface necessary?

Hi @geopossachs Thanks for your effort. Can you fix the testing failure?

Hi @appleboy, can you help me? Do you use the .drone.yml file in your pipeline?
To build an image for the jenkins test service i need access to the docker.socket so my ci job run with the trusted flag.
https://drone.speedy-likeness.de/geopossachs/drone-jenkins/36
And can a get an example for an AppVeyor CI? I could not found a running example :-(
( drone is so easy )


Missing installer folder

The folder where the necessary code to build the windows installer and the deb package is not present in the repository. Although this is a mistake from the source management point of view, solving this issue has no implications related to the releases.

Issue solved just modifying the .gitignore file, which avoided the management of the complete installer sub-folder.


feat: verify release policy Task bundle references pinned to digest

Description
feat: add test case to cover ec verifies release policy Task bundle references pinned to digest
fix: unhold commentsin   ./tests/build/build_templates.go and ./tests/build/chains.go tto retrigger verify-enterprice-contract since https://issues.redhat.com/browse/RHTAPBUGS-352 is fixed.
chore: comments nit in test "verifies ec cli supports the current policy config fields used by the ec-policies rego rules"
Issue ticket number and link
https://issues.redhat.com/browse/HACBS-1846
Type of change

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

How Has This Been Tested?
./bin/e2e-appstudio --ginkgo.focus-file tests/enterprise-contract/contract.go --ginkgo.vv
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/hold

/unhold

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e

https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/redhat-appstudio_e2e-tests/608/pull-ci-redhat-appstudio-e2e-tests-main-redhat-appstudio-e2e/1672251951808516096#1:build-log.txt%3A12 fails the following
Summarizing 5 Failures:
  [FAIL] [e2e-demos-suite] Deploy RHTAP sample application into a Kubernetes cluster provided by user [It] waits component pipeline to finish [byoc]
  /tmp/tmp.CgUQlU31CZ/tests/byoc/byoc.go:223
  [FAIL] [e2e-demos-suite] DEVHAS-234: creates an application with springboot component from RHTAP samples [It] waits public component springboot-component pipeline to be finished [e2e-demo]
  /tmp/tmp.CgUQlU31CZ/tests/e2e-demos/e2e-demo.go:191
  [FAIL] [e2e-demos-suite] DEVHAS-234: create an application with quarkus component [It] waits public component quarkus-devfile pipeline to be finished [e2e-demo]
  /tmp/tmp.CgUQlU31CZ/tests/e2e-demos/e2e-demo.go:191
  [FAIL] [e2e-demos-suite] DEVHAS-234: creates quarkus application(with dockerfile but not devfile) which is not included in AppStudio starter stack [It] waits public component status-quarkus-io pipeline to be finished [e2e-demo]
  /tmp/tmp.CgUQlU31CZ/tests/e2e-demos/e2e-demo.go:191
  [FAIL] [e2e-demos-suite] multi-component scenario with components with devfile or dockerfile or both [It] waits application mc-two-scenarios components pipelines to be finished [e2e-demo, multi-component]
  /tmp/tmp.CgUQlU31CZ/tests/e2e-demos/multi-component.go:225

https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/redhat-appstudio_e2e-tests/608/pull-ci-redhat-appstudio-e2e-tests-main-redhat-appstudio-e2e/1673321987662942208 failes the following
Summarizing 8 Failures:   [FAIL] [e2e-demos-suite] DEVHAS-234: creates an application with python component from RHTAP samples [AfterAll] checks if component  health [e2e-demo]   /tmp/tmp.Tz5lxwN5er/tests/e2e-demos/e2e-demo.go:88   [FAIL] [e2e-demos-suite] DEVHAS-234: create an nodejs application without dockerfile [AfterAll] checks if component  health [e2e-demo]   /tmp/tmp.Tz5lxwN5er/tests/e2e-demos/e2e-demo.go:88   [FAIL] [release-suite [HACBS-1571]test-release-e2e-push-image-to-pyxis] [BeforeAll] Post-release verification verifies that Component 1 can be created and build PipelineRun is created for it in dev namespace and succeeds [release, pushPyxis, HACBS]   /tmp/tmp.Tz5lxwN5er/tests/release/e2e-test-push-image-to-pyxis.go:132   [FAIL] [e2e-demos-suite] Deploy RHTAP sample application into a Openshift cluster provided by user [AfterAll] checks if component  was deployed in the target cluster and namespace [byoc]   /tmp/tmp.Tz5lxwN5er/tests/byoc/byoc.go:115   [FAIL] [build-service-suite Build service E2E tests] test PaC component build when a new component without specified branch is created [It] a related PipelineRun and Github webhook should be deleted after deleting the component [build, HACBS, github-webhook, pac-build, pipeline, image-controller, pac-custom-default-branch]   /tmp/tmp.Tz5lxwN5er/tests/build/build.go:181   [FAIL] [release-suite [HACBS-738]test-release-service-default-pipeline] [BeforeAll] Post-release verification verifies that a build PipelineRun is created in dev namespace and succeeds [release, defaultBundle, HACBS]   /tmp/tmp.Tz5lxwN5er/tests/release/e2e-test-default-bundle.go:99   [FAIL] [release-suite [HACBS-1199]test-release-e2e-with-deployment] [BeforeAll] Post-release verification verifies that a build PipelineRun is created in dev namespace and succeeds [release, withDeployment, HACBS]   /tmp/tmp.Tz5lxwN5er/tests/release/e2e-test-default-with-deployment.go:88   [FAIL] [e2e-demos-suite] DEVHAS-234: create an golang application [AfterAll] checks if component  health [e2e-demo]   /tmp/tmp.Tz5lxwN5er/tests/e2e-demos/e2e-demo.go:88

https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/redhat-appstudio_e2e-tests/608/pull-ci-redhat-appstudio-e2e-tests-main-redhat-appstudio-e2e/1674778180667314176 failes the following
Summarizing 2 Failures:
  [FAIL] [e2e-demos-suite] DEVHAS-234: create an nodejs application without dockerfile [It] deploys component  using gitops [e2e-demo]
  /tmp/tmp.em20BSzeh2/tests/e2e-demos/e2e-demo.go:227
  [FAIL] [e2e-demos-suite] DEVHAS-234: creates an application with python component from RHTAP samples [It] deploys component  using gitops [e2e-demo]
  /tmp/tmp.em20BSzeh2/tests/e2e-demos/e2e-demo.go:227

https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/redhat-appstudio_e2e-tests/608/pull-ci-redhat-appstudio-e2e-tests-main-redhat-appstudio-e2e/1674840274464935936 fails
Summarizing 1 Failure:
  [FAIL] [build-service-suite Build service E2E tests] test PaC component build when the component is removed and recreated (with the same name in the same namespace) [BeforeAll] should no longer lead to a creation of a PaC PR [build, HACBS, github-webhook, pac-build, pipeline, image-controller]
  /tmp/tmp.hmZ15tybkA/tests/build/build.go:472

/retest

/lgtm


Add Monoid and Eq for Style

This adds Monoid and Eq for Style in the cats package.

Codecov Report

Merging #188 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #188      +/-   ##
==========================================
+ Coverage   98.06%   98.08%   +0.01%
==========================================
  Files           7        7
  Lines         414      418       +4
  Branches       32       32
==========================================
+ Hits          406      410       +4
  Misses          8        8

Impacted Files
Coverage Δ

...rc/main/scala/org/typelevel/paiges/instances.scala
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

👍


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/soef/ioBroker.firetv

[ ] [E019] Invalid repository URL: https://github.com/soef/iobroker.firetv. Expected: git+https://github.com/soef/ioBroker.firetv.git
[ ] [E114] No adapter are allowed in the repo without admin3 support
[ ] [E115] No license found in io-package.json
[ ] [E802] No topics found in the repository. Please go to "https://github.com/soef/ioBroker.firetv" and press "Manage topics"
[ ] [E603] NO "## Changelog" found in README.md
[ ] [E604] No "## License" found in README.md
[ ] [E701] No actual year found in LICENSE. Please add "Copyright (c) 2016-2020 soef<EMAIL_ADDRESS>at the start of LICENSE

I have also found warnings that may be fixed if possible.

[ ] [W113] Adapter should support compact mode

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Do you need help fixing the bugs?


Remove references to defunct npm registry

Should we remove the "generate npmrc option" completely?

I agree we should remove the option completely

Can we really just delete the option? It is listed as 'deprecated' in the --help output, but imo this would technically be a breaking change..
      --generateNpmrc                Deprecated. If set to true the generator
                                     will generate an .npmrc file specifying a
                                     registry for @sap scoped dependencies. This
                                     is not necessary anymore and will be
                                     skipped by default.

I think it was internal so removing is possible.

"internal" in what sense? The npmrc function has an @internal annotation, is this what you mean? What's the (semantic) meaning of this? It does not refer to the cli option in any way, does it?

"internal" in what sense? The npmrc function has an @internal annotation, is this what you mean? What's the (semantic) meaning of this? It does not refer to the cli option in any way, does it?

This means that the method is not exported on root level and is not part of our public API. This means we can refactor, rename or delete methods like this as we please. Sometime we add a compatibility note but this is not needed in this case.


Text Sets: Easier/smarter text font & color styling

Summary
This PR ensures that any text set added to the page has good contrast with its background.
Uses either black/white color or semi-transparent black/white scrim (see Figma).
Example:

Relevant Technical Choices
Firefox doesn't support backdrop-filter: blur(), should we leave it as is or try to mimic that effect? The same applies for text presets.

To-do

[ ] Tests

User-facing changes
Text sets, when inserted (via click or drag) will change their colors based on the background.
Testing Instructions
QA
Enable feature flag:

For easier testing, I've added a green dot on the text sets that are qualified for this feature because some of them are not obvious (in other words: exclude text sets with any predefined text color).

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT

[ ] UAT should use the same steps as above.

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[ ] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #7995

@googlebot I consent.

@barklund @miina With Marcin OOO & with QA passed, can one of you remove the green dots before merging?

@googlebot I consent.

Thanks @miina. I've also added one simple test.


feature: add_picture() supports PDF

Is there support for inserting .pdf files or other image files that support vector graphics? Sorry if I've missed it, but the docs only have direct references to .png .jpeg, and .gif, and in my testing I've only gotten add_picture() calls to work with .png files. When I try a *.pdf file there is no effect.

Not presently. I'll leave this one open as a feature request for this.

I don't think PPTs can include PDFs. There will probably need to be some intermediate process to convert to an image format.

My current solution is to convert prior to calling python-pptx. This is somewhat annoying because it creates intermediate image files only for this purpose, but for my purposes it isn't a significant inconvenience.
I'm not sure what's going on under the hood, but you can certainly import pdf files into powerpoint without  rasterizing them. Perhaps they are converted to some other vector format, but they remain vector-based in their behavior (arbitrary resizing without loss of quality).

Not to do with the solution per se, but is there a reason you couldn't use in-memory stream objects like 'StringIO' instead of disk files for the intermediate image streams? Or are you doing that step with something other than Python?

Good point, I should be able to bypass the disk file creation. For my purposes most of the images are generated with matplotlib.

@petebachant turns out to be quite right about this. While the PowerPoint application allows a user to insert PDF and EPS images, they are converted to placeable WMF/EMF before being written to the .pptx package. I'm strongly inclined to avoid adding a new dependency to the library, so for the time being folks will be on their own to do the conversion with another available Python library. Closing for now.


Cannot find module 'tvmjs' or its corresponding type declarations EVEN has ‘npm install tvmjs’ done

(gh_web-llm) amd00@asus00:~/llm_dev/web-llm$ npm run build

<EMAIL_ADDRESS>build
rollup -c

src/index.ts → lib/index.js...
[!] (plugin rpt2) Error: src/chat_module.ts:1:24 - error TS2307: Cannot find module 'tvmjs' or its corresponding type declarations.
1 import * as tvmjs from "tvmjs";
~~~~~~~
src/index.ts
at error (/home/amd00/llm_dev/web-llm/node_modules/rollup/dist/shared/rollup.js:198:30)
at throwPluginError (/home/amd00/llm_dev/web-llm/node_modules/rollup/dist/shared/rollup.js:21718:12)
at Object.error (/home/amd00/llm_dev/web-llm/node_modules/rollup/dist/shared/rollup.js:22672:20)
at RollupContext.error (/home/amd00/llm_dev/web-llm/node_modules/rollup-plugin-typescript2/src/context.ts:35:17)
at /home/amd00/llm_dev/web-llm/node_modules/rollup-plugin-typescript2/src/diagnostics.ts:70:17
at Array.forEach (<anonymous>)
at printDiagnostics (/home/amd00/llm_dev/web-llm/node_modules/rollup-plugin-typescript2/src/diagnostics.ts:42:14)
at typecheckFile (/home/amd00/llm_dev/web-llm/node_modules/rollup-plugin-typescript2/src/index.ts:67:3)
at Object.<anonymous> (/home/amd00/llm_dev/web-llm/node_modules/rollup-plugin-typescript2/src/index.ts:257:5)
at Generator.next (<anonymous>)

(gh_web-llm) amd00@asus00:~/llm_dev/web-llm$ tvmjs

@SeekPoint
Same problem here...

If you would only like to use webllm, you do not need to run npm run build, instead, follow instructions in examples to directly install WebLLM from npm.
See https://github.com/mlc-ai/web-llm/tree/main/examples/get-started and other examples https://github.com/mlc-ai/web-llm/tree/main/examples
Try this only if you would like to develop WebLLM package. Please follow https://github.com/mlc-ai/web-llm#build-webllm-package-from-source (there are steps before you can run npm run build)

@tqchen
Thanks for the reply...  But...
><EMAIL_ADDRESS>start
> parcel src/get_started.html  --port 8888

Server running at http://localhost:8888
🚨 Build failed.

@parcel/core: Failed to resolve '@mlc-ai/web-llm' from './src/get_started.ts'

  ....../web-llm/examples/get-started/src/get_started.ts:1:25
  > 1 | import * as webllm from "@mlc-ai/web-llm";
  >   |                         ^^^^^^^^^^^^^^^^^
    2 |
    3 | function setLabel(id: string, text: string) {

@parcel/resolver-default: Could not load './lib/index.js' from module '@mlc-ai/web-llm' found in package.json#main

  ....../web-llm/examples/get-started/node_modules/@mlc-ai/web-llm/package.json:5:11
    4 |   "description": "Hardware accelerated language model chats on browsers",
  > 5 |   "main": "lib/index.js",
  >   |           ^^^^^^^^^^^^^^ './lib/index.js' does not exist, did you mean './src/index.ts'?'
    6 |   "types": "lib/index.d.ts",
    7 |   "type": "module",

Looks like it's a parcel issue.
And if I move examples folder out of web-llm itself, I'm still getting:
➜  get-started npm start

><EMAIL_ADDRESS>start
> parcel src/get_started.html  --port 8888

Server running at http://localhost:8888
🚨 Build failed.

@parcel/core: Failed to resolve '@mlc-ai/web-llm' from './src/get_started.ts'

  ....../examples/get-started/src/get_started.ts:1:25
  > 1 | import * as webllm from "@mlc-ai/web-llm";
  >   |                         ^^^^^^^^^^^^^^^^^
    2 |
    3 | function setLabel(id: string, text: string) {

@parcel/resolver-default: Cannot find module '@mlc-ai/web-llm'
💡 Did you mean '@mlc-ai/web-llm'?

And, I can see @mlc-ai/web-llm has been installed locally.
➜  get-started npm list @mlc-ai/web-llm
<EMAIL_ADDRESS>....../examples/get-started
└──<EMAIL_ADDRESS>-> ./../..

Any further suggestions? Thank you ...

Make sure you run npm install first to install web-llm, you would need to change the web-llm dependency to 0.2.0 instead of file as being suggested.
If you have already done that differently, remove the node_modules folder locally first

@tqchen
Great... It finally runs successfully....
However, the same as https://mlc.ai/web-llm/ :
Init error, Error: Cannot find adapter that matches the request
[System Initalize]
Init error, Error: Cannot find adapter that matches the request

Any further suggestions??
Thanks

That likely mean that you do not have a device that have enough GPU RAM

@tqchen
What? I got a Geforce 3090 24G. Won't that do?
And, BTW, you know any llm that can be easily deployed on a cellphone?
Thanks...

hmm, that seems to be sufficient, maybe you can try out https://webgpureport.org/ to see if you can find the GPU.
That could be an issue of your browser, make sure you use the latest chrome
For mobile deployment, checkout https://mlc.ai/mlc-llm/

Do you have your parameters downloaded into the specified path?

still failed:
(base) amd01@mz32-01:~/llm_dev/web-llm/examples/get-started$ npm install
npm ERR! Maximum call stack size exceeded
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/amd01/.npm/_logs/2023-07-09T04_15_49_122Z-debug.log
(base) amd01@mz32-01:~/llm_dev/web-llm/examples/get-started$

(gh_web-llm) amd00@asus00:~/llm_dev/web-llm$ npm run build（gh_web-llm） amd00@asus00：~/llm_dev/web-llm$ npm run build

<EMAIL_ADDRESS>build<EMAIL_ADDRESS>构建
rollup -c 汇总 -c

src/index.ts → lib/index.js... [!] (plugin rpt2) Error: src/chat_module.ts:1:24 - error TS2307: Cannot find module 'tvmjs' or its corresponding type declarations.[!]（插件 rpt2）错误：src/chat_module.ts：1：24 - 错误 TS2307：找不到模块“tvmjs”或其相应的类型声明。
1 import * as tvmjs from "tvmjs";1 从“tvmjs”导入 * 作为 tvmjs; ~~~~~~~
src/index.ts
at error (/home/amd00/llm_dev/web-llm/node_modules/rollup/dist/shared/rollup.js:198:30)
at throwPluginError (/home/amd00/llm_dev/web-llm/node_modules/rollup/dist/shared/rollup.js:21718:12)
at Object.error (/home/amd00/llm_dev/web-llm/node_modules/rollup/dist/shared/rollup.js:22672:20)
at RollupContext.error (/home/amd00/llm_dev/web-llm/node_modules/rollup-plugin-typescript2/src/context.ts:35:17)
at /home/amd00/llm_dev/web-llm/node_modules/rollup-plugin-typescript2/src/diagnostics.ts:70:17
at Array.forEach (<anonymous>)
at printDiagnostics (/home/amd00/llm_dev/web-llm/node_modules/rollup-plugin-typescript2/src/diagnostics.ts:42:14)
at typecheckFile (/home/amd00/llm_dev/web-llm/node_modules/rollup-plugin-typescript2/src/index.ts:67:3)
at Object.<anonymous> (/home/amd00/llm_dev/web-llm/node_modules/rollup-plugin-typescript2/src/index.ts:257:5)
at Generator.next (<anonymous>)

(gh_web-llm) amd00@asus00:~/llm_dev/web-llm$ tvmjs（gh_web-LLM） amd00@asus00：~/llm_dev/web-llm$ tvmjs

The same issue persists even after installing the previous command. Have you resolved it?


exifread return incorrect value

Hi, i'm trying to extract the exif valies from a raw image, the exifread function process_file seems to work, but some tags are incorrect.
I report an example below:
exif = exifread.process_file(file, detail=False)
if I type
exif['Image Orientation']
I obtain the following output: (0x0112) Short=Rotated 90 CW @ 78
but if I type exif['Image Orientation'].values
I obtain the following ouput: [6]

What about exif[Image Orientation].printable ?


Fix error saving files when streaming using autofilter and sheet protection

Summary
Hello. Recently I came across a problem when, when recording a file in a stream, you set a password and autofilter, the file breaks and cannot be opened. Having slightly changed the source code, I was able to solve this problem by changing the order in which data was written to the file.
Test plan
The code below on the current version of the library creates a file that does not open and causes an error

After commenting
worksheet.autoFilter = { from: 'A1', to: 'C1' };

or
await worksheet.protect('test', { formatColumns: true, formatRows: true, autoFilter: true, pivotTables: true });

out the error disappears
import { stream } from 'exceljs';

async function main() {
    const workbook = new stream.xlsx.WorkbookWriter({ filename: './test.xlsx', useStyles: true });

    const worksheet = workbook.addWorksheet('test');
    worksheet.addRow([3, 'Sam', new Date()]).commit();
    worksheet.autoFilter = { from: 'A1', to: 'C1' };
    await worksheet.protect('test', { formatColumns: true, formatRows: true, autoFilter: true, pivotTables: true });
    worksheet.commit();
    await workbook.commit();
}
main();

Changes to the pull request correct this error.

Any updates on this?


No error messages when invalid combinations are raised

Requesting production of certain items (like yellow science in a grey machine) or "5b" in a box instead of "b5" makes the calculation vanish without a trace. (Worthy of note - invalid combinations are still possible with some calculations)
An error message detailing the problem would be nice so the user can quickly see and fix the problem

Indeed, that would be nice, good suggestion!


add developer docs to this repo

should have everything relevant to onboarding new developers to tfchain codebase

https://github.com/threefoldtech/tfchain/tree/development/docs

Verified https://github.com/threefoldtech/tfchain/blob/development/docs/development/development.md.


Yaml file understanding

Hey ! I would like help for understand format file and where i put the table name. Can you help me ? Thank's

Hi ! If i understand, in example_schema: customer and customer_address are table name. And raw is name of one column name ?

All columns of the specified tables are dumped and anonymized by default. If you want to just copy some of the columns without anonymization, you must explicitly list them in the raw property (that's why it's called raw). Please refer to the README for details about the anonymization process.


Updates to README, RELEASING, and build.gradle

Codecov Report

Merging #405 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #405   +/-   ##
=========================================
  Coverage     70.86%   70.86%
  Complexity      623      623
=========================================
  Files           134      134
  Lines          2863     2863
  Branches        215      215
=========================================
  Hits           2029     2029
  Misses          756      756
  Partials         78       78

Continue to review full report at Codecov.

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


Add Window Attribute for Metrics

resolves #5721
Description
Adding the attribute window to the metric definition as an optional string. No tests have been added yet as this is merely a draft.
Produced Manifest
I tested this out in a development project and was able to confirm that "window": "14 days" was correctly represented in the single metric where it was defined. For all others "window": "" was present.
Checklist

[X] I have read the contributing guide and understand what's expected of me
[X] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

@callum-mcdata once you are ready for an Engineering team member to review (test passing, name solidified), feel free to add the ready_for_review label to the PR so we know to take a look

Hey @leahwicz this PR is ready for review! I updated the tests to ensure that they pass. If the PR gets the 🟢 from the core team I'll make the corresponding updates to the docs repo before we merge!

Thanks @callum-mcdata! I just added @emmyoop as a reviewer since she is on support rotation right now

@emmyoop made the changes that you mentioned in above resolved conversations!

@gshank could I get approval to merge in this PR? Looks like I'm not an authorized user to merge into main


Add Google Analytics

I think that knowing more about our site and visitors would be very nice. Even knowing the hits our resources have.
https://www.gatsbyjs.org/docs/adding-analytics/

we do have it set up :) just added you to the account 🤓


Set particular locales for application (#135)

Coverage increased (+0.2%) to 99.76% when pulling e91579306ed14d211a5cda66362b66fb1582d27f on defself:master into c24a2ed0488e2c3b71749d548d6954b843f50268 on kslazarev:master.

Coverage increased (+0.2%) to 99.76% when pulling e91579306ed14d211a5cda66362b66fb1582d27f on defself:master into c24a2ed0488e2c3b71749d548d6954b843f50268 on kslazarev:master.

Coverage increased (+0.2%) to 99.76% when pulling d5146ac2ce4bfabdd2903575017672d2cf4ff275 on defself:master into c24a2ed0488e2c3b71749d548d6954b843f50268 on kslazarev:master.

Coverage decreased (-0.1%) to 99.403% when pulling 2bfec0d386fca248cfd59f2e6310002be61ff8e8 on defself:master into c24a2ed0488e2c3b71749d548d6954b843f50268 on kslazarev:master.

This works but I don't love it. It feels like it requires complicated gymnastics like require: false and manual loading. There's probably a simpler solution to delay setting I18n.locale_file setting and/or to unload whatever is there when the locales are assigned and replace with new values.
I think as a developer I'd want to do this, which is similar to other gems:
NumbersAndWords.configure do |config|
   config.locales = [:en, :ru]
end

Coverage decreased (-32.9%) to 66.624% when pulling 399d9cfdc1aa6571676e3c448b613e4bdc8a6f62 on defself:master into c24a2ed0488e2c3b71749d548d6954b843f50268 on kslazarev:master.

Coverage increased (+0.2%) to 99.76% when pulling 416815355ef2069f6378d2ac26954ab1ea9b309f on defself:master into c24a2ed0488e2c3b71749d548d6954b843f50268 on kslazarev:master.


Double check email address extraction

For this paper
http://tazendra.caltech.edu/~acedb/daniel/00062455_Huang22.pdf
only one author's email was extracted<EMAIL_ADDRESS>

I've debugged the code and Tim's email is extracted but discarded since it contains an extra "." at the end. I can improve the extraction by removing extra characters.

Fixed code in wbtools to remove additional "." in email addresses
Added code to extract email addresses on multiple lines broken at the @ sign, like in the following example:

you@
email.com

Updated requirements file in AFP codebase to point to new wbtools version


feat(Topology/Order/UpperLowerSetTopology): Introduce the Upper Set (or Alexandrov) topology and the Lower Set topology

Introduces the Upper Set topology, which is the canonical example of an Alexandrov topology and its dual, the Lower Set topology.

An Alexandrov topology is a topology where the intersection of any collection of open sets is open. The UpperSetTopology is an Alexandrov topology, and, given any Alexandrov topology, a Preorder may be defined on the underlying set such that the UpperSetTopology of the Preorder coincides with the original topology.
However, our motivation for introducing the UpperSetTopology is its use in the construction of the Scott Topology (#2508). Beyond demonstrating that the set of open sets is closed under arbitrary intersections, we have not sought to further develop the theory of Alexandrov topologies in this PR. However, we hope that by breaking the Upper Set topology out into a separate file / PR, we may facilitate anyone who does wish to develop mathlib4 in that direction.
See also the discussion about naming here
For completeness we also introduce the dual Lower Set topology.

[x] depends on: #5873

Could you add comments (I'd say both in the module docstrings and in the docs of the definitions) warning that this is not the dual of the LowerTopology? Maybe even add an example in the module docstring.

Could you add comments (I'd say both in the module docstrings and in the docs of the definitions) warning that this is not the dual of the LowerTopology? Maybe even add an example in the module docstring.

I've added the warning and also UpperSetTopology.Monotone_to_LowerTopology_Dual_Continuous which explains the relationship between these topologies.

I think I know your answer, but why not also introduce LowerSetTopology? In particular I feel like having the full collection would make it easier to emphasize that LowerTopology is different from LowerSetTopoloy, without resorting to dualizing the order.
I don't have a use case in mind, but I know that I have a use case for the dual version of LowerTopology (which we still don't have) so it wouldn't surprise me if such a case appears in the future.

I think I know your answer, but why not also introduce LowerSetTopology? In particular I feel like having the full collection would make it easier to emphasize that LowerTopology is different from LowerSetTopoloy, without resorting to dualizing the order. I don't have a use case in mind, but I know that I have a use case for the dual version of LowerTopology (which we still don't have) so it wouldn't surprise me if such a case appears in the future.

I got my fingers burnt early on introducing something just because we could which then caused trouble later on, so I've been a bit cautious since. If you have a use for UpperTopology (the dual version of LowerTopology) then maybe we should just go ahead and add these things in.
Would I just create a file called LowerSetTopology.lean, define
def lowerSetTopology : TopologicalSpace α :=
{ IsOpen := IsLowerSet,
...

and restate/prove all the results, or can I infer them all somehow using OrderDual?

I'd say you can have them in the same file (UpperLowerSetTopology.lean). You can either separate them into two sections or mix them, it doesn't really matter (except maybe for managing namespaces). You'll have to restate all the results, but the proofs should hopefully be one line applications of the upper version

I'd say you can have them in the same file (UpperLowerSetTopology.lean). You can either separate them into two sections or mix them, it doesn't really matter (except maybe for managing namespaces). You'll have to restate all the results, but the proofs should hopefully be one line applications of the upper version

I've made a start on adding UpperTopology: https://github.com/leanprover-community/mathlib4/tree/mans0954/UpperTopology
I've added def toOrderDualHomeomorph [Preorder α] : WithLowerTopology α ≃ₜ WithUpperTopology αᵒᵈ - is that the right way to express this?

I've made a start on adding UpperTopology: mans0954/UpperTopology
I've added def toOrderDualHomeomorph [Preorder α] : WithLowerTopology α ≃ₜ WithUpperTopology αᵒᵈ - is that the right way to express this?

Could you please open a PR for that? It'll be easier to make suggestions. Also if you want to dualize this particular file in a separate PR that would work for me, do as you wish.

@ADedecker I've added the statements for the lower set topology to this file/PR.
Is the next step to prove:
instance  [Preorder α] [TopologicalSpace α] [LowerSetTopology α] : UpperSetTopology (αᵒᵈ) := sorry

so that
theorem LowerSetTopology.IsOpen_sInter {S : Set (Set α)} (hf : ∀ s ∈ S, IsOpen s) : IsOpen (⋂₀ S) := by
  apply @UpperSetTopology.IsOpen_sInter αᵒᵈ _ _ _ _ (fun s a ↦ hf s a)

etc?

Yes, I think this is the perfect approach! In particular note that you don't actually need an homeomorphism with the order dual, this instance does all the work.

This PR/issue depends on:

leanprover-community/mathlib4#5873
By Dependent Issues (🤖). Happy coding!

Could you merge master?

Could you merge master?

Done.

bors r+


[RFE] support PTP (hardware clock) for time synchronisation

Description
PTP is recommended in various environments, particularly in Azure. Their recommendation is to use chronyd, which is not included in Flatcar's minimal fs, and would be an exercise to bundle in a container with host devices.
It seems that systemd-timesyncd does not support PTP, so an issue has been opened there: https://github.com/systemd/systemd/issues/22828
Impact
systemd-timesyncd defaults to a pool.ntp.org org address, even though a precision time protocol device is available on the host.
Environment and steps to reproduce

Set-up:  Flatcar VM on Azure (gen2, standard B1's)

vbatts@vbatts-Lemur-Pro:~$ ssh -l core <IP_ADDRESS>
Last login: Tue Mar 22 19:20:48 UTC 2022 from <IP_ADDRESS> on pts/0
Flatcar Container Linux by Kinvolk alpha 3165.0.0 for Microsoft Azure
core@vb-ptp ~ $ ls /dev | grep ptp
ptp0
ptp_hyperv

Expected behavior
timedatectl to reflect PTP sync, or similar

As a stop-cap solution an alpine / chrony container would serve as a work-around:
FROM alpine

RUN apk add chrony
RUN echo "log statistics measurements tracking" > /etc/chrony/chrony.conf
RUN echo "driftfile /var/lib/chrony/drift" > /etc/chrony/chrony.conf
RUN echo "makestep 1.0 3" > /etc/chrony/chrony.conf
RUN echo "maxupdateskew 100.0" > /etc/chrony/chrony.conf
RUN echo "dumpdir /var/lib/chrony" > /etc/chrony/chrony.conf
RUN echo "rtcfile /var/lib/chrony/rtc" > /etc/chrony/chrony.conf
RUN echo "rtcdevice /dev/ptp0" > /etc/chrony/chrony.conf
RUN echo "refclock PHC /dev/ptp0 poll 3 dpoll -2 offset 0 stratum 2" > /etc/chrony/chrony.conf

then
docker build -t chrony .
sudo systemctl disable --now systemd-timesyncd
docker run -it --cap-add=SYS_TIME --device=/dev/rtc:/dev/rtc --device=/dev/ptp_hyperv:/dev/ptp0 chrony chronyd -s -d

right on. And since you disabled the system service, you might should make that container a --restart=always and make sure that docker.service is enabled, and not just docker.socket.
Ideal fix would be PTP feature in systemd-timesyncd.

right on. And since you disabled the system service, you might should make that container a --restart=always and make sure that docker.service is enabled, and not just docker.socket. Ideal fix would be PTP feature in systemd-timesyncd.

Actually, we can pass this via an ignition config and run the chrony container as a systemd service. Please find the config below - just a braindump for now so w can turn it into documentation later:
CT config (human-readable, for editing and adjusting): use_chrony.yaml
storage:
  files:
    - path: /opt/chrony-build/Dockerfile
      mode: 0644
      contents:
        inline: |
          FROM alpine
          RUN apk add chrony
          RUN echo "log statistics measurements tracking" > /etc/chrony/chrony.conf
          RUN echo "driftfile /var/lib/chrony/drift" > /etc/chrony/chrony.conf
          RUN echo "makestep 1.0 3" > /etc/chrony/chrony.conf
          RUN echo "maxupdateskew 100.0" > /etc/chrony/chrony.conf
          RUN echo "dumpdir /var/lib/chrony" > /etc/chrony/chrony.conf
          RUN echo "rtcfile /var/lib/chrony/rtc" > /etc/chrony/chrony.conf
          RUN echo "rtcdevice /dev/ptp0" > /etc/chrony/chrony.conf
          RUN echo "refclock PHC /dev/ptp0 poll 3 dpoll -2 offset 0 stratum 2" > /etc/chrony/chrony.conf
systemd:
  units:
    - name: systemd-timesyncd.service
      mask: true
    - name: prepare-chrony.service
      enabled: true
      contents: |
        [Unit]
        Description=Build the chrony container image
        ConditionPathExists=!/opt/chrony-build/done
        [Service]
        Type=oneshot
        RemainAfterExit=true
        Restart=on-failure
        WorkingDirectory=/opt/chrony-build
        ExecStart=docker build -t chrony .
        ExecStartPost=touch done
        [Install]
        WantedBy=multi-user.target
    - name: chrony.service
      enabled: true
      contents: |
        [Unit]
        Description=Chrony RTC time sync service
        After=docker.service prepare-chrony.service
        Requires=docker.service prepare-chrony.service
        [Service]
        TimeoutStartSec=0
        ExecStartPre=-/usr/bin/docker rm --force chrony
        ExecStart=/usr/bin/docker run --name chrony -i --cap-add=SYS_TIME --device=/dev/rtc:/dev/rtc --device=/dev/ptp_hyperv:/dev/ptp0 chrony chronyd -s -d
        ExecStop=/usr/bin/docker stop chrony
        Restart=always
        RestartSec=5s
        [Install]
        WantedBy=multi-user.target

Run this to convert to ignition config:
cat use_chrony.yaml | docker run --rm -i ghcr.io/flatcar-linux/ct:latest > ignition.json

Use this with az vm create [...] --custom-data ignition.json
ignition.json
{"ignition":{"config":{},"security":{"tls":{}},"timeouts":{},"version":"2.3.0"},"networkd":{},"passwd":{},"storage":{"files":[{"filesystem":"root","path":"/opt/chrony-build/Dockerfile","contents":{"source":"data:,FROM%20alpine%0ARUN%20apk%20add%20chrony%0ARUN%20echo%20%22log%20statistics%20measurements%20tracking%22%20%3E%20%2Fetc%2Fchrony%2Fchrony.conf%0ARUN%20echo%20%22driftfile%20%2Fvar%2Flib%2Fchrony%2Fdrift%22%20%3E%20%2Fetc%2Fchrony%2Fchrony.conf%0ARUN%20echo%20%22makestep%201.0%203%22%20%3E%20%2Fetc%2Fchrony%2Fchrony.conf%0ARUN%20echo%20%22maxupdateskew%20100.0%22%20%3E%20%2Fetc%2Fchrony%2Fchrony.conf%0ARUN%20echo%20%22dumpdir%20%2Fvar%2Flib%2Fchrony%22%20%3E%20%2Fetc%2Fchrony%2Fchrony.conf%0ARUN%20echo%20%22rtcfile%20%2Fvar%2Flib%2Fchrony%2Frtc%22%20%3E%20%2Fetc%2Fchrony%2Fchrony.conf%0ARUN%20echo%20%22rtcdevice%20%2Fdev%2Fptp0%22%20%3E%20%2Fetc%2Fchrony%2Fchrony.conf%0ARUN%20echo%20%22refclock%20PHC%20%2Fdev%2Fptp0%20poll%203%20dpoll%20-2%20offset%200%20stratum%202%22%20%3E%20%2Fetc%2Fchrony%2Fchrony.conf%20%0A","verification":{}},"mode":420}]},"systemd":{"units":[{"mask":true,"name":"systemd-timesyncd.service"},{"contents":"[Unit]\nDescription=Build the chrony container image\nConditionPathExists=!/opt/chrony-build/done\n[Service]\nType=oneshot\nRemainAfterExit=true\nRestart=on-failure\nWorkingDirectory=/opt/chrony-build\nExecStart=docker build -t chrony .\nExecStartPost=touch done\n[Install]\nWantedBy=multi-user.target\n","enabled":true,"name":"prepare-chrony.service"},{"contents":"[Unit]\nDescription=Chrony RTC time sync service\nAfter=docker.service prepare-chrony.service\nRequires=docker.service prepare-chrony.service\n[Service]\nTimeoutStartSec=0\nExecStartPre=-/usr/bin/docker rm --force chrony\nExecStart=/usr/bin/docker run --name chrony -i --cap-add=SYS_TIME --device=/dev/rtc:/dev/rtc --device=/dev/ptp_hyperv:/dev/ptp0 chrony chronyd -s -d\nExecStop=/usr/bin/docker stop chrony\nRestart=always                                                                        \nRestartSec=5s                                                                         \n[Install]                                                                             \nWantedBy=multi-user.target                                                            \n","enabled":true,"name":"chrony.service"}]}}


Improvement: Add ECDSA support into Convert-PemToPfx

When talking about PKI, for decades RSA was the de-facto algorithm. Since it is no more, adding support for EC-algorithms would be advised.
Suggestions for implementation can be taken from my tool https://github.com/HQJaTu/RDP-cert-tools/blob/main/update-RDP-cert.ps1. It has readily implemented functions __composeEcDsaPrivateKeyBlob and __attachECPrivateKey. Also there exists enough logic for guessing if RSA or ECDSA material should be imported from a set of PEM-files.
It should be relatively easy to implement something similar into PSPKI.

Fair enough. However, this in most cases is kind of tricky, because private key must have NCRYPT_ALLOW_PLAINTEXT_EXPORT_FLAG flag enabled and which is not always set for exportable key. That is, you can export key in PFX bag, but cannot extract raw key material. This happens when you import certificate from PFX. For some reasons this flag is not enabled and the function fails and even NCryptSetProperty doesn't help.

Yes, I agree on what you said about PKCS #12 and #7 -import.
However, the code is generic and can import EC-certificate and key from a set of PEM-files too. All of these sources can be supported with relative ease, as I've implemented them into my RDP-tooling.
My suggestion is: Being the popular option, PSPKI should have this support also for everybody in The Net to enjoy.


Top or bottom of page cut off. Only one can be shown. - Google Chrome - Android

When I load the page, it is showed OK, but when I use it, top or bottom cut off.
I use swipe to change volume so I can not swipe to access to top or to bottom.
This behaviuor started with recent updates. In previous versions worked ok.
It happens in mobile mode.
Server:
Version: 8.3.1 -<PHONE_NUMBER> @ Fri Feb 17 06:33:29 WEST 2023
Operating system: Windows 10 - EN - cp1252
Platform Architecture: 8664
Material Skin (v3.3.4)
Client:
Android 10
Google Chrome: 114.2.5735.130

Sorry, but I cannot recreate this on my S20FE - but this has Android 13.
You say it was working before, so I'm going to need you to locate the last version that did work - so that I can do a diff and see what might cause this.
All releases are on Github. What you need to do is download a release (e.g. https://github.com/CDrummond/lms-material/releases/download/3.3.0/lms-material-3.3.0.zip), unzip its contents over the MaterialSkin folder in your LMS install, restart LMS, clear Chrome's cache, kill the WebApp, and reload.
To make things easier I suggest you try each minor version first - so 3.2, 3.1, 3.0 Then if 3.1.0 works, but not 3.2.0, then try 3.1.1, 3.1.2, etc.

Thanks for the quickly response.
I found that WebApp display parameter set in Fullscreen causes the issue.
If I set to Standalone, material skin works ok.

Yes, but that is only a work-around. It does not solve the actual issue you are seeing. A better work-around is to use the APK (https://f-droid.org/en/packages/com.craigd.lmsmaterial.app/) - this is a WebView wrapper. However, it'd be better to solve the actual issue. For that I will need your help as described.

Of course, it is a workaround but the bug is more accurate. I will help you to solve this issue.
I will try to fdroid app too... I did not know about it .


Error in compute_mean_contributions_over_time

Hi all,
Ran into a small error in one of the functions in the base MMM class of the 0.2.0 release.
I tried using the compute_mean_contributions_over_time method and received this error:
ValueError: Length of values (1) does not match length of index (200)
Looks like the function is trying to assign the intercept contribution and that is returning a single value as an np.array and pandas isn't happy about it.
line 583 of base.py
         contributions_intercept_over_time = (
            az.extract(
                mmm.fit_result,
                var_names=["intercept"],
                combined=True,
            )
            .mean("sample")
-->       .to_numpy()
        )
#array([0.0083757])

        all_contributions_over_time = (
            contributions_channel_over_time.join(contributions_control_over_time)
            .join(contributions_fourier_over_time)
-->       .assign(intercept=contributions_intercept_over_time)
        )

Probably not the best solution, but I removed the .to_numpy() and used .values[0] to get it to work. Pandas filled the column.
contributions_intercept_over_time = (
            az.extract(
                mmm.fit_result,
                var_names=["intercept"],
                combined=True,
            )
            .mean("sample")
            .values[0]
        )

        all_contributions_over_time = (
            contributions_channel_over_time.join(contributions_control_over_time)
            .join(contributions_fourier_over_time)
            .assign(intercept=contributions_intercept_over_time)
        )

False alarm. I realized the issue here.
In setting the Intercept prior in the model_config I passed an np.array().  Therefore an np.array() was coming out of the AZ.extract.
Thanks all!


Can't update fakeroot-tcp

Please fill out the below information:
Describe the issue
When updating fakeroot-tcp, it does not update
To Reproduce
Use pikaur or equivalent to update fakeroot-tcp from AUR
Expected behavior
fakeroot-tcp gets updated correctly
Screenshots
If applicable, add screenshots to help explain your problem.
Enviroment:

Windows build number: Microsoft Windows [Version 10.0.19041.172]
Security Software: Windows Defender
WSL version: WSL2
ArchWSL version. <IP_ADDRESS>
ArchWSL Installer type zip/appx/other clean/update: appx
Launcher version wsldl, version 20032600

Additional context
Build output:
libtool: install: (cd /home/rdb/.cache/pikaur/build/fakeroot-tcp/pkg/fakeroot-tcp/usr/lib/libfakeroot && { ln -s -f libf
akeroot-0.so libfakeroot.so || { rm -f libfakeroot.so && ln -s libfakeroot-0.so libfakeroot.so; }; })
libtool: install: /usr/bin/install -c .libs/libfakeroot.lai /home/rdb/.cache/pikaur/build/fakeroot-tcp/pkg/fakeroot-tcp/
usr/lib/libfakeroot/libfakeroot.la
libtool: warning: remember to run 'libtool --finish /usr/lib/libfakeroot'
 /usr/sbin/mkdir -p '/home/rdb/.cache/pikaur/build/fakeroot-tcp/pkg/fakeroot-tcp/usr/bin'
  /bin/sh ./libtool   --mode=install /usr/bin/install -c faked '/home/rdb/.cache/pikaur/build/fakeroot-tcp/pkg/fakeroot-
tcp/usr/bin'
libtool: install: /usr/bin/install -c faked /home/rdb/.cache/pikaur/build/fakeroot-tcp/pkg/fakeroot-tcp/usr/bin/faked
make[2]: Nothing to be done for 'install-data-am'.
make[2]: Leaving directory '/home/rdb/.cache/pikaur/build/fakeroot-tcp/src/fakeroot-1.24'
make[1]: Leaving directory '/home/rdb/.cache/pikaur/build/fakeroot-tcp/src/fakeroot-1.24'

When using WSL2, can I use fakeroot? (Instead of fakeroot-tcp)?

Yes.
WSL2 does not require fakeroot-tcp.

@yuk7 could you update this in the wiki pls? https://github.com/yuk7/ArchWSL/wiki/Known-issues
😄

@codemonkey800
I have not confirmed this problem yet.
Did it occur in your environment?

@yuk7 Oh no I meant about WSL 2 not requiring fakeroot-tcp. I had trouble installing google-cloud-sdk from aur because it was hanging on the package() stage. I came here for help and found your answer here 😄
Reinstalling fakeroot fixed the problem, so it would be nice to have a note in the wiki saying WSL 2 doesn't require fakeroot-tcp so that people don't install it when they don't need it, and so it also prevents weird problems like mine from happening lol

@codemonkey800 I see.
I'll document it later👍

I found a solution to issues with fakeroot
just install base-devel and it should fix issues with that


formatter: wrongly eats newline for multi-lines call with a block

# (1) This formats OK:
method(
  1
)

# (2) But when I write:
method(
  1
) do
  puts "hello"
end

# It gets formatted to:
method(
  1) do
  puts "hello"
end

Which makes it a bit difficult to distinguish between the end of the arguments and the start of the block.
Like (1), I think (2) shouldn't change when formatted.
(if accepted, any way to get this before next release?)

Awesome, thank you @MakeNowJust


Remove IDocumentMappingService.GetLanguageKind(...) and make it an extension method on RazorCodeDocument

For a long while, the GetLanguageKind(...) method that determines whether an index into a document falls within Razor, C# or HTML has been a bit of a wart on the IDocumentMappingService. It really isn't part of document mapping, and its implementation is completely distinct. In fact, making the actual change is quite simple, so why hadn't it been done yet? The answer is mocking.
There are several tests that mock IDocumentMappingService.GetLanguageKind(...) to lie about test inputs. In my not-so-humble opinion, this represents an abuse of mocking. Instead of setting up tests to have the necessary inputs that ensure GetLanguageKind(...) would return a real and correct result, the inputs would often be garbage and an IDocumentMappingService mock would lie about the GetLanguageKind(...) result at a particular location. This makes moving GetLanguageKind(...) off of IDocumentMappingService a much larger change than it needs to be. This is why there are substantial test changes in this PR.
Don't misunderstand me as a mocking hater! Mocking libraries are definitely useful! In fact, there are new mocks used in this very PR! However, mocks should be used judiciously and thoughtfully, and in this case, a mock was used to write lazy tests.

Fixes https://github.com/dotnet/razor/issues/8774


Error = Failed to retrive mp4 video url

Some videos are getting error while playing videos.
iOS version 15.6.1
error response : - Error = Failed to retrive mp4 video url
tried with both methods given in repo

@nithintwobase  did you find any solutions?

@nithintwobase do you use fetchVideoURLFrom(url: ) or fetchVideoURLFrom(id: ) ?

I tried both methods
On Sun, Oct 2, 2022, 23:08 Nyan Lin Tun @.***> wrote:

@nithintwobase https://github.com/nithintwobase do you use fetchVideoURLFrom(url:
) or fetchVideoURLFrom(id: ) ?
—
Reply to this email directly, view it on GitHub
https://github.com/superm0/HCVimeoVideoExtractor/issues/13#issuecomment-1264695597,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW6RKV3GCYXVYBR4RLP6Y3DWBHCBVANCNFSM6AAAAAAQFPHKWA
.
You are receiving this because you were mentioned.Message ID:
@.***>

for me using the fetchVideoURLFrom(url: ) solve the issue

@nithintwobase @nyan-lin-tun  Since yesterday this library stop working  (only for new videos) , old videos are working, they are already on Vimeo cloud) , I ll give you example.
I checked the old videos JSON data and compare with the new video's JSON , check the screenshots.
old Videos JSON data  in which we get the videos URL from this path progressive

now if you upload a new video on Vimeo server and trying to get the JSON  using HCVimeoVideoExtractor  then you will find  progressive this key return empty data.
here is the screenshot

i get an error like this mentioned below.
Printing description of data:
▿ 3 elements
▿ 0 : 2 elements
- key : "title"

- value : Sorry

▿ 1 : 2 elements
- key : "view"

- value : 7

▿ 2 : 2 elements
- key : "message"

- value : This video does not exist.

but the same vimeo video work in Android & web
Thanks & Regards,
Nithin Shenoy s
Software Engineer
Mobile: +91<PHONE_NUMBER>
Email: @.***
Web: www.2basetechnologies.com
[image: Join 2base at Facebook]
[image: Join 2base at Twitter] http://www.twitter.com/2basetech
[image: Join 2base at LinkedIn]
http://www.linkedin.com/companies/2base-technologies
On Mon, Oct 3, 2022 at 9:30 AM Swift Developer @.***>
wrote:

@nithintwobase https://github.com/nithintwobase @nyan-lin-tun
https://github.com/nyan-lin-tun Since yesterday this library stop
working (only for new videos) , old videos are working, they are already on
Vimeo cloud) , I ll give you example.
I checked the old videos JSON data and compare with the new video's JSON ,
check the screenshots.
old Videos JSON data in which we get the videos URL from this path
progressive
[image: Screen Shot 2022-10-02 at 21 07 21]
https://user-images.githubusercontent.com/1904874/193498996-69303e1c-9f7b-48c3-b40c-fd48aefda0f0.png
now if you upload a new video on Vimeo server and trying to get the JSON
using HCVimeoVideoExtractor then you will find progressive this key
return empty data.
here is the screenshot
[image: Screen Shot 2022-10-02 at 21 07 46]
https://user-images.githubusercontent.com/1904874/193499192-f7684be7-94b2-408c-b11d-1ae9c8167ac5.png
—
Reply to this email directly, view it on GitHub
https://github.com/superm0/HCVimeoVideoExtractor/issues/13#issuecomment-1264898969,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW6RKV4RB2Y2XH5MFLAHGE3WBJK6LANCNFSM6AAAAAAQFPHKWA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@nivritgupta seem there are some changes from the Vimeo

@nithintwobase Did you get its solutions. I am also facing same issue. Working fine for older videos but not for new videos.

this issue has been resolved with the latest update. for new Vimeo videos, the video URL can be retrieved through .Quality1080p or .QualityUnknown

@superm0  the issue is not solved yet , I checked the retrieve the videos with .Quality1080p or .QualityUnknown only and found  this node from JSON still return  nil data
progressive =         (
);

@nivritgupta can you share your vimeo url?

@nivritgupta can you share your vimeo url?

Never mind the problem is fixed now , thanks


Missing dependency for aiohttp 0.22.0a0 :  https://pypi.python.org/pypi/multidict

multidict is published on PyPI


Copy database from assets which has multiple tables with content

Hi,
@simolus3
I have already prefilled SQLite DB which I want to use in my flutter application with few new tables.
My existing DB has 4-5 tables with around 1000 rows.
So I have followed the below link;
https://moor.simonbinder.eu/docs/other-engines/vm/#using-moor-ffi-with-an-existing-database
Assets DB has copied successfully in the application, But when I query the tables, which were already existing, it shows 0 records.
My database file looks like below;
String dbPath;
String dbName = "data.sqlite";

class AppDatabase extends _$AppDatabase {
  AppDatabase() : super(_openConnection());

  @override
  int get schemaVersion => 1;

  getAllTables() async {
    for (var table in allTables) {
     await select(table).get().then((data){
       AppDebugLog.println('table : ${table.actualTableName} : ${data.length}');
     });
    }
  }
}

LazyDatabase _openConnection() {
  try {
    return LazyDatabase(() async {
      var dbFolder = await getApplicationDocumentsDirectory();
      dbPath = p.join(dbFolder.path, dbName);

      final file = File(dbPath);
      if (!file.existsSync()) {
        // copy the file from an asset, or network, or any other source
        // Should happen only the first time you launch your application
        AppDebugLog.println('Copy DB from assets : ');

        // Make sure the parent directory exists
        await Directory(dirname(dbPath)).create(recursive: true);

        // Copy from asset
        var data = await rootBundle.load(join('assets', dbName));
        List<int> bytes =
            data.buffer.asUint8List(data.offsetInBytes, data.lengthInBytes);
        //AppDebugLog.println('Assets DB : $bytes');

        // Write and flush the bytes written
        await file.writeAsBytes(bytes, flush: true);
      } else {
        AppDebugLog.println('Open existing DB :');
      }
      var db = VmDatabase(file);
      return db;
    });
  } catch (ex) {
    AppDebugLog.println('Exception in _openConnection : ' + ex.toString());
  }
  return null;
}

Thanks,
Nikhil Sathawara.

Your code looks alright to me. Does the actualTableName match the table name in your pre-existing database?

Hi,
@simolus3
You were right, the auto generated actualTableNames were different than mine exiting table names.
I have to manually modified them.
After that I got my records.
Thanks for the help.

Glad I could help!

I have to manually modified them.

If you had to change the generated code, note that you can also override the tableName getter directly in your table class. Moor will then pick it up directly, which means that you don't have to change your code every time after running the builder.

@simolus3
Yes, You are right we can override tableName and moor will assign that name to tables.
I have changed code as per that.
Thanks.


async/await

Do you have native support of async/await ?

I'm interested into that feature too :)

I apologize for not understanding, but I'm not sure what is meant by "native support of async/await"?
Is there any chance you can explain what you mean? I don't really see how async/await and a serialization framework are connected.
If you don't mind, please clarify and perhaps I can provide some help.

It would be nice if we could read or write from tokio::io::AsyncRead and tokio::io::AsyncWrite streams.


typo fix

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks!

Although I will miss the image of someone clucking to unsubscribe 🐔


[mgmt] fabric ga

https://github.com/Azure/sdk-release-request/issues/5418

API change check
API changes are not detected in this pull request.


Override fill and stroke through props

I've added the option to replace fill and stroke values of the component through props. Fallback is the original fill/stroke. Is omitted through --fillProp false and --strokeProp false option.

Added width and height also.


Make this even more system independent by allowing the user to specify another system.

This is most likely makes sense together with associated mypy types, though they are of course not necessary as they are only needed for type checking, and can be specified to be None for code that doesn't use mypy.

Solved with e53a117f7a6c37a6584a0398776b8b12e3571bd0

Solved with e53a117f7a6c37a6584a0398776b8b12e3571bd0


Reliable way of linking to kirby folder & kirby bootstrap file

In worker.php, bootstrap.php and the path to the Kirby folder are based on the default locations and cannot be altered.
This will give problems to people who have a non-default folder setup, or use composer.
Find a way to make this better.

One solution, at least for the cron usage, would be setting the cron with an argument, like /var/whatever/worker.php /../path/to/root/, and then using the (optional) argument when requiring bootstrap.php, like this:
$kirbyPath = __DIR__ . (count($argv) > 1 ? $argv[1] : '/../../../../');
require "{$kirbyPath}kirby/bootstrap.php";


Update validation.py

Fixes #<issue_number_goes_here>

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

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

Take a look


Add default target settings for resource bundles

Resolves: https://github.com/tuist/tuist/issues/4568
Short description 📝

On Xcode 14, the default CODE_SIGNING_ALLOWED setting has changed to YES whereas it used to be NO on Xcode 13
This causes archiving to fail for cases where iOS applications use bundle targets to host resources, as code signing fails

Code signing isn't required for resource bundle target (as far as I'm aware)

Solution 📦

To mitigate this on Xcode 14 and maintain backwards compatibility with Xcode 13, a new default target setting is being added to keep this set to NO

References 📚

https://github.com/tuist/tuist/pull/4588
https://developer.apple.com/forums/thread/708659

Assuming a project that has an iOS resource bundle target called A_A, we can examine the resolved settings by Xcode when no settings are specified for CODE_SIGNING_ALLOWED
$ xcrun xcodebuild -showBuildSettings -workspace App.xcworkspace -scheme A_A | grep CODE_SIGN
    AD_HOC_CODE_SIGNING_ALLOWED = NO
    CODE_SIGNING_ALLOWED = NO
    CODE_SIGNING_REQUIRED = YES
    CODE_SIGN_CONTEXT_CLASS = XCiPhoneOSCodeSignContext
    CODE_SIGN_IDENTITY = iPhone Developer
    CODE_SIGN_INJECT_BASE_ENTITLEMENTS = YES
    EXPANDED_CODE_SIGN_IDENTITY =
    EXPANDED_CODE_SIGN_IDENTITY_NAME =
    __CODE_SIGNING_ALLOWED_appletvos = NO
    __CODE_SIGNING_ALLOWED_iphoneos = NO
    __CODE_SIGNING_ALLOWED_watchos = NO

$ DEVELOPER_DIR=/Applications/Xcode14-beta2.app/Contents/Developer/ xcrun xcodebuild -showBuildSettings -workspace App.xcworkspace -scheme A_A | grep CODE_SIGN
    AD_HOC_CODE_SIGNING_ALLOWED = NO
    CODE_SIGNING_ALLOWED = YES
    CODE_SIGNING_REQUIRED = YES
    CODE_SIGN_CONTEXT_CLASS = XCiPhoneOSCodeSignContext
    CODE_SIGN_IDENTITY = iPhone Developer
    CODE_SIGN_INJECT_BASE_ENTITLEMENTS = YES
    EXPANDED_CODE_SIGN_IDENTITY =
    EXPANDED_CODE_SIGN_IDENTITY_NAME =

Alternatives
We could alternatively keep this default setting within Tuist's DefaultSettingsProvider if we think this isn't generally applicable.

Codecov Report

Merging #697 (257737e) into main (e02240f) will decrease coverage by 2.62%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #697      +/-   ##
==========================================
- Coverage   84.92%   82.30%   -2.63%
==========================================
  Files         157      157
  Lines        9109     9131      +22
==========================================
- Hits         7736     7515     -221
- Misses       1373     1616     +243

Impacted Files
Coverage Δ

...ources/XcodeProj/Utils/BuildSettingsProvider.swift
50.70% <100.00%> (-3.26%)
:arrow_down:

...deProjTests/Utils/BuildSettingsProviderTests.swift
100.00% <100.00%> (ø)

...ces/XcodeProj/Objects/Targets/PBXProductType.swift
11.42% <0.00%> (-42.86%)
:arrow_down:

...Workspace/XCWorkspaceDataElementLocationType.swift
57.14% <0.00%> (-22.23%)
:arrow_down:

...deProjTests/Extensions/XCTestCase+Assertions.swift
50.00% <0.00%> (-16.67%)
:arrow_down:

...es/XcodeProj/Extensions/Dictionary+Enumerate.swift
61.53% <0.00%> (-15.39%)
:arrow_down:

...es/XcodeProj/Objects/BuildPhase/PBXBuildRule.swift
80.00% <0.00%> (-14.00%)
:arrow_down:

Sources/XcodeProj/Errors/Errors.swift
34.72% <0.00%> (-12.50%)
:arrow_down:

...urces/XcodeProj/Scheme/XCScheme+PathRunnable.swift
87.50% <0.00%> (-12.50%)
:arrow_down:

...ces/XcodeProj/Objects/Files/PBXFileReference.swift
77.55% <0.00%> (-12.25%)
:arrow_down:

... and 64 more

Continue to review full report at Codecov.

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

We could alternatively keep this default setting within Tuist's DefaultSettingsProvider if we think this isn't generally applicable.

I am bit leaning to this - shouldn't the settings here try to adhere to Xcode's defaults? Or do we have a history of deviating when convenient? @kwridan

Agreed, was also unsure - will move it to Tuist where we have better context on what is being attempted rather than here where it should reflect Xcode defaults 👍


[FlatButton] Add labelPosition prop.

At the moment FlatButtons always render the label before any passed children.
I've added a labelPosition prop to allow changing the order of the elements.
Defaults to right as I thought the most common use is adding a FontIcon or similar which is normally placed on the left hand side of the button.

@hai-cea good catch, fixed!

+1 Rebased, renamed to "before" and "after" and made beforedefault.

Thanks @vaiRk

I have just tested the feature. I have two issue labelPosition should default to after and I can't make the property work because a float: left style applied. I'm gonna submit a PR to fix this.


Windows packaging is very slow

My making two changes, one in configuration and one in the source code of this plugin, I have reduced my packaging time from over 30s to less than 10s.
As the number of dependencies increases this difference would grow. I only have 7 dependencies and 1 dev dependency at present.
I have created a PR to use junction instead of copy https://github.com/UnitedIncome/serverless-python-requirements/pull/347

Closing as fix was merged and released; happy to reopen if there are still issues.


Further plan for revising the source code of the paper

Hi, Zhongqi! Thank you for this interesting work. Do you have a plan for releasing the source code of the paper?
Look forward to your reply.

Hi! @mo666666,
Sorry for replying late. We promise to release our data and testing/training code of detection, localization and mitigation before 2024/9/15.
Thanks.

Zhongqi, thank you for your reply. The revision plan is OK for me.


[Archi 5] Support ArchiMate 3.2

This is an umbrella issue for implementing the changes in Archi for ArchiMate 3.2.
Changes between 3.1 and 3.2 - https://pubs.opengroup.org/architecture/archimate32-doc/#_Toc112155137

Improved the definitions of several concepts, most notably outcome, constraint, business function, and product
Made the Physical elements part of the Technology Layer chapter instead of a separate chapter
Changed the Technology Layer metamodel, making device, system software, facility, and equipment no longer subtypes of node but of technology internal active structure element, and adding composition and aggregation relationships with node.
Added composition and aggregation relationships from plateau to outcome
Added realization from material to equipment
Improved the restrictions on derivation rules described in Section B.3.5
Added a derivation rule for grouping
Changed the icon notation of the meaning and communication network elements, added an icon notation for work package, and added new box notations for meaning, value, business object, contract, representation, work package, and deliverable
Changed the default color of the plateau and gap elements to the same pink as the other Implementation and Migration elements

New branch archimate32 created.
Things to do:

[x] Change default colours of Location, Gap and Plateau
[ ] Add alternate figures for Business Object, Contract, Representation, Product, Grouping, Data Object, Meaning, Value, Work Package, Deliverable
[x] Change some first type figures to rectangles
[x] Swap some default figure types from first position to second position (e.g. Application Component)
[x] Add Model Compatibility Handler to handle swapped default figure types
[ ] Update Relationships table
[ ] Update Hints texts

I'll take care of the relationship table and suggest to replace hints content by only the definition coming from the glossary. This should make it easier for people to translate Archi because they'll be able to reuse the official glossary's translation (and if none exist, they could create one and I'll manage to get it through the internal review process of the Open Group).
We should also anticipate a bit the fact that the ArchiMate Model Exchange File Format will change in the (hopefully) near future and make sure we can importe a file which claims to be from the new version (without changing the import code). This will make sure that people can at least import a model created with another tool that create "3.2" AMEFF files.

@jbsarrodie I've had a go at the new small icons and the new alternate figures (Work Package, Communication Network).
I failed at creating thought bubbles for the Meaning figure, so left it as it is.
My maths is terrible!

replace hints content by only the definition coming from the glossary.

I can't find the 3.2 glossary. Is it published?

I failed at creating thought bubbles for the Meaning figure, so left it as it is.
My maths is terrible!

😂
I'll manage it. It might be better to either keep the current figure as the icon version or used a slight different one to maximize the avalable space for the text inside the bubble (e.g. having only one small bubble in the lower left corner).

I can't find the 3.2 glossary. Is it published?

No, but I can share it with you in private.

For the hints, the text is rather short. Should we keep the Category ad Examples parts?
For example:
A Business Actor represents a business entity that is capable of performing behavior.
Category:
Business. Active Structure.
Examples:
A Customer, Marketing & Communications department, Director of Finance, Secretary, Admissions Department, Product Development.

Should we also keep the Category and Examples parts?

Yes, that make sense and there's no changes on these parts

@jbsarrodie I've had a go at the new small icons and the new alternate figures (Work Package, Communication Network).
I failed at creating thought bubbles for the Meaning figure, so left it as it is.
My maths is terrible!

Hello @Phillipus
I'm not sure if this is still an outstanding issue, but in case you are interested, I have created a 16x16 'Meaning' transparent PNG (the GitHub Issue editor does not allow me to include the .ICO file) that approximates the new ArchiMate 3.2 version (attached).

Regards,
Andrew Greff

HI Andrew, thanks for that. The palette requires a 16x16 and a 32x32 version. The actual figures are created in Draw2d code (example here)

Here is the new relationship.xml file (created from the official Excel version of the table, thus the order of concepts and characters is a bit different):

Thanks for that! In order to get a readable diff between versions I need to put it all in alphabetical order. Can you think a way to do this? Maybe a jArchi script? ;-)

Thanks for that! In order to get a readable diff between versions I need to put it all in alphabetical order. Can you think a way to do this? Maybe a jArchi script? ;-)

I can easily do it for concepts' names (I just have to reorder some columns and rows), but that's another story for letter combinations depincting allowed relationships :-(

I'm working on a Java routine to do it...

This one is missing:
<target concept="Junction" relations="afnortv" />

This one is missing:
<target concept="Junction" relations="afnortv" />

Yes, that's because things changes a bit:

All relationships can be used with Junction
From ArchiMate 3.2 - 5.5.1 Junction: a junction (which connects some relationships), may also be aggregated or composed in a plateau, grouping, or location element. If that junction connects other aggregation or composition relationships, it should be interpreted without the aggregation or composition to this containing element. The composition or aggregation merely states that the junction is part of a plateau, grouping, or location. Without that relationship, the junction must fulfill the same conditions as above: it connects relationships of the same type, with at least one incoming and one outgoing relationship.

But for us, maybe it still makes sense to add it to the xml file.

We have to add it to the xml file because of the way the code works. So should it be this?
<target concept="Junction" relations="acfginorstv" />
And this:
    <source concept="Junction">
        <target concept="ApplicationCollaboration" relations="acfginorstv" />
        ...more...

@ModelDeveloper Thanks for that. The icons in the tree and palette are indeed pngs. See here.  I tried your Meaning icons and I think it might work better with only one small bubble and more rounded main bubble.
For reference - https://github.com/archimatetool/archi/blob/master/com.archimatetool.editor/img/archimate/meaning%402x.png

So should it be this?

Yes, I'll re-generate the XML tomorrow.

It's easier for me to manually edit it...

That's quite easy for me and I can re-order concepts at the same time ;-)

That's quite easy for me and I can re-order concepts at the same time ;-)

Noooooooooooooooooooooooooooo!
https://github.com/archimatetool/archi/commit/79b2e0903824ce0b358637390606b679b33c8d32

Archi 5 is out!

@fdh54 The was changed because new figures were introduced in Archi 5.2 for each concept and the figure types had to be standardised. The Archi file format is internal and subject to change.


Sequelize example in no typescript enviroment?

It there same example of sequelize without typescript?
Get NoResourceAdapterError: There are no adapters supporting one of the resource you provided , if run in expressjs without  typescript.

Sorry , finally I change to use it in typescript.


Sometimes failed with Unable to capture screenshot (-32000)

What versions are you running?
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.7.4
$ google-chrome --version
Chrome/107.0.5304.107
$ go version
go version go1.19 linux/amd64

What did you do? Include clear steps.
I was use headless way to screenshot.
actx, cancel1 = chromedp.NewRemoteAllocator(c, ws)
defer cancel1()

ctx, cancel2 := chromedp.NewContext(actx, chromedp.WithDebugf(log.Printf))
defer cancel2()
var res []byte
err := chromedp.Ran(ctx,chromedp.Navigate("https://www.baidu.com"),chromedp.FullScreenshot(res, 100))
fmt.Println(err)

What did you expect to see?
Screenshot always success.
What did you see instead?
Sometime the screenshot was successed,bug sometimes failed with the message Unable to capture screenshot (-32000)

I fixed it! When the headless container run had not  set shm-size


Clock.prototype.start doesn't "reset the clock" with respect to elapsed()

Since the elapsed function now just returns this._elapsed (instead of computing it as before), calling the start method on a clock doesn't change the returned value of the elapsed function.
My use case is enforcing a minimum amount of time between some user-generated events. When the event is called, I reset the clock by calling myClock.start(). Then before the event is called again, I check myClock.elapsed().
My fix was to add this._elapsed = 0; into the start function, but I haven't thought through any other ways of using the elapsed function, so it's probably not a quick fix except for me.

The start() function should indeed reset elapsed time. In fact, elapsed time should be 0 on the frame in which a reset occurs. In start(), I've added _elapsed=0 and reset _lastMasterElapsed and _currentMasterElapsed to the master, which results in the desired behavior.


Use centralized function/objects for all network access

The network calls are scattered in many places and this is problem.

this is technical debt
we cannot log things easily #590
we should disable etags but this is in many places #321
we cannot easily add proxy support at once #559
we cannot implement API politeness at once #506 #568

I skimmed the code and found a fetch_response utility function in vulnerabilities/utils.py, more fetch_* functions/methods purpose-built for their respective importers, and fetch methods various VersionAPI subclasses.
Before I try to fix this issue, I would like to know if this is what you're talking about as well? Perhaps, you know of some more?
My plan to fix this:

Understand how each importer/VersionAPI fetches data.
Separate what's general to every importer/VersionAPI from what's special to each.
Design a base Fetcher class (analogous to the Importer class).
Specialize (through inheritance) the Fetcher class for an importer/VersionAPI as necessary.

I'm contributing for the first time and will appreciate all the advice and constructive criticism I get get. :)


Test123

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

could you please describe what is this aiming to do?


can't run example/version.go on Window 10

1.download ffmpeg dev
2.download pkg-config
3.set pkg *.pc file
4.go run version.go on goav/example
is it work only on linux os?
return
..\avformat\avformat.go:270:45: cannot use _Ciconst_sizeof_char * _Ctype_ulong(proto_size) (type _Ctype_ulong) as type _Ctype_ulonglong in argument to _Cfunc__CMalloc
..\avformat\avformat.go:273:53: cannot use _Ciconst_sizeof_char * _Ctype_ulong(authorization_size) (type _Ctype_ulong) as type _Ctype_ulonglong in argument to _Cfunc__CMalloc
..\avformat\avformat.go:276:48: cannot use _Ciconst_sizeof_char * _Ctype_ulong(hostname_size) (type _Ctype_ulong) as type _Ctype_ulonglong in argument to _Cfunc__CMalloc
..\avformat\avformat.go:279:44: cannot use _Ciconst_sizeof_char * _Ctype_ulong(path_size) (type _Ctype_ulong) as type _Ctype_ulonglong in argument to _Cfunc__CMalloc
..\avformat\avformat.go:300:43: cannot use _Ciconst_sizeof_char * _Ctype_ulong(buf_size) (type _Ctype_ulong) as type _Ctype_ulonglong in argument to _Cfunc__CMalloc
..\avformat\avformat.go:321:43: cannot use _Ciconst_sizeof_char * _Ctype_ulong(buf_size) (type _Ctype_ulong) as type _Ctype_ulonglong in argument to _Cfunc__CMalloc

How did you set pkg *.pc file? I'm trying to accomplish this but unfortunatelly still getting No package '...' found


Addition of Feedback Option to the Website.

Description:
A feedback option has been added to the website to allow users to share their experiences, suggestions, and issues directly. This feature is intended to help gather valuable user insights and improve the website based on real-time user feedback.
Purpose:
The feedback option enables users to provide input on website usability, content quality, and any challenges they encounter. This input is essential for continuous improvements and ensuring the website meets user expectations.
Implementation Details:
Design: The feedback option is prominently placed for easy accessibility, with a simple and user-friendly interface.
Content: The feedback form may include fields such as user satisfaction rating, comments, and suggestions.
Submission Process: Feedback can be submitted directly through the form and will be sent to the website’s admin or support team for review.
Additional Suggestions:
Feedback Categories: Include categories (e.g., usability, content, features, bugs) to help users specify the nature of their feedback.
Anonymous Feedback Option: Allow users to submit feedback anonymously, encouraging honest responses.
Thank You Message: Display a “Thank you” message after submission to acknowledge the user's contribution.
Regular Review: Periodically review feedback submissions to identify recurring issues and prioritize improvements.
Expected Benefits:
The feedback option allows the team to stay connected with user needs and expectations, facilitating targeted improvements. This can lead to enhanced user satisfaction and a more tailored user experience on the website.

Well this feature is already present


Link incorrect.

Was linked here from AzureAD security page with title "How password hash synchronization works" which isn't what's here.

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

ID: aad82713-d562-5778-fbf8-035e24f9c470
Version Independent ID: 6370cca0-c78e-4738-ba67-8fb14594b0f2
Content: What is password hash synchronization with Azure AD?
Content Source: articles/active-directory/hybrid/whatis-phs.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

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

@oddhair what page were you linked from?


backend: dont log warnings if not necessary

this is a follow-up for https://github.com/puppetlabs/puppetlabs-haproxy/pull/442
with the current code in the main branch, a warning is logged to the
puppetserver.log for every backend, if httpchk is set. This is very
noisy and provides 0 benefit, if the user set the
sort_options_alphabetic to false.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@0a937ee). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #449   +/-   ##
=======================================
  Coverage        ?   16.66%
=======================================
  Files           ?        1
  Lines           ?        6
  Branches        ?        0
=======================================
  Hits            ?        1
  Misses          ?        5
  Partials        ?        0

Continue to review full report at Codecov.

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

Hi @bastelfreak, thank you for your contribution!
kind regards,
@adrianiurca


clown sleep mimimimimni is so fkin loud

Description

Really annoying
Reproduction

Screenshots

Additional context

Honestly should really quiet it down or split it into smaller sounds so it's not so long

i agree

intentional :godo:

This issue can be closed


How to include multiple condition within ONE sub aggregation?

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v7 (for Elasticsearch 7.x)
[x] elastic.v6 (for Elasticsearch 6.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v2 (for Elasticsearch 1.x)
May I have instructions on how to write code with subaggregation to achieve something like this?

I have total and success calculated and I used a bucket script to do the final calculation.
But the SubAggregation method can only contain one name. Please advice!

I managed to do it by adding subsequent SubAggregation. Thanks


Support for EGL_KHR_no_config_context

Description

This is a mix between a bug/feature. My application makes use of EGL_KHR_no_config_context, but this is not currently supported by RenderDoc. The bug part of it, I guess, is that EGL_KHR_no_config_context is still listed as a supported extension when querying eglQueryString( ..., EGL_EXTENSIONS ), so applications checking this value will get an incorrect result because the RenderDoc hooks do not support the extension.
The feature request would be to support the EGL_KHR_no_config_context extension, since it is quite useful.
The current behavior when attempting to call eglCreateContext with EGL_NO_CONFIG_KHR is that the call generates EGL_BAD_CONFIG.
The error occurs because of this call, since EGL_NO_CONFIG_KHR is an invalid argument for the call. I'm not familiar enough with the renderdoc source, but maybe these calls could be moved to a hook for eglCreatePlatformWindowSurface or eglCreateWindowSurface?
Environment

RenderDoc version: 1.24
Operating System: Linux Mint 20.3
Graphics API:  EGL API version: 1.5 (while instantiating an OpenGL 4.6 context)

I don't have any plans to add support for new EGL extensions that require implementation work, but the query string is a problem I hadn't considered. Modifying the string is not hard but I will need to check to see what to do with extensions. I'm wondering if it's best to do a block list of extensions that fail instead of an allow list since EGL being a platform-specific API it may have platform undocumented extensions that should be preserved, I'm concerned about the unforeseen problems that would come from some random Android device stopping working if an extension disappears.

I went through the EGL extensions, only at a quick look given the number & complexity but it seems like this is the only extension which intersects with the subset RenderDoc cares about enough to need to be blocked.
I don't have an android test program that I can easily modify to test with, if that is still reporting the extension as supported please let me know and I can look into it.


Bug with Angular 1.4.3 and Restangular 1.5.2

TypeError: _.includes is not a function
TypeError: _.includes is not a function
    at Object.b.isSafe (http://wip.dev/bundles/app_web/bower_components/restangular/dist/restangular.min.js?v=v1:6:534)
at Object.B (http://wip.dev/bundles/app_web/bower_components/restangular/dist/restangular.min.js?v=v1:6:18327)
at Object.C (http://wip.dev/bundles/app_web/bower_components/restangular/dist/restangular.min.js?v=v1:6:18597)
at Class.extend.getAllToday (http://wip.dev/bundles/app_warehouse/scripts/src/InventoryMovementModule/services/InventoryMovementResources.js?v=v1:18:99)

InventoryMovementResources
            return this.api.one('date').one('today').withHttpConfig({timeout: canceller.promise}).get();

After change to use back Restangular 1.5.1, all works

Hi @tom10271, I guess I know the problem. Can you have a look with lodash version you're using?

I think I am using underscore only, is it because it uses some functions missing from underscore?

Yep. Seems like underscore doesn't have _.includes. See #1314 why we switched from _.contains (that's present with underscore!) to _includes. It seems like the lodash dependency is one of the biggest problems of Restangular.

@tom10271 upgrade to underscore 1.8.  It has an alias for .includes to .contains

Thank you for providing solutions! I really appreciate it.


Fix build without jemalloc

libs/libglibc-compatibility/libglibc-compatibility.a(glibc-compatibility.c.o): In function `explicit_bzero':
/home/amos/git/chorigin/build-dev/../libs/libglibc-compatibility/glibc-compatibility.c:151: multiple definition of `explicit_bzero'
contrib/ssl/crypto/libcrypto.a(explicit_bzero.c.o):/home/amos/git/chorigin/build-dev/../contrib/ssl/crypto/compat/explicit_bzero.c:16: first defined here
collect2: error: ld returned 1 exit status

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Ok, but I don't understand how it is related to jemalloc
(it is related to glibc and libressl)


Add vcpkg instructions to README.md

This seems like a handy thing to add to the README

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.
After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @wheezil isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
CLA GitHub bot

I've completed the CLA.  Full steam ahead!

@jtattermusch, are you still interested in pursuing this?

I'm going to close this PR for now, but let's keep tracking the situation around gRPC vcpkg support in https://github.com/grpc/grpc/issues/10699 and once a well-tested and usable vcpkg exists, we will be happy to advertise that in the how to use instructions: (e.g. here https://github.com/grpc/grpc/tree/master/src/cpp#packaging-systems)


glTF 2.0 animations don't play

Still investigating this, but I'm having trouble getting any glTF 2.0 animation to play in master.  Even the "animated triangle" official sample model won't budge.
The animation code is being exercised, particularly in ModelAnimationCollection it is running every frame and calculating a localAnimationTime and such.  Just visually, nothing appears to move on the screen.

It seems to work for me in the 3D Models sandcastle demo.
Are you passing in shouldAnimate : true to the viewer?

The 3D Models demo uses glTF 1.0 models.  Did you try this with glTF 2.0?  Which model?

Scratch that.  It's working now in Sandcastle, for certain models.


The observation value "[Object]" is not subscribable. Make sure you are returning the Query, and not fetching it yourself.

I can't seem to be able to query Parse. I'm using the ParseReact and React packages. Here's my code:

I get a this error and undefined in the render() {} block for tweets object. Any lead on how to resolve this would be great

Put parens around new Parse.Query('Tweet').

Hey @TylerBrock, I did  as your suggested (and you're right about constructing the query outside the return value), however it still consoles out undefined meaning that somehow I can't get the instance to subscribe to tweets

Are there any tweets in Tweet to observe?
It seems like you have errors compiling your JS, you might want to drop this into your webpack config:
  node: {
    fs: 'empty',
    child_process: 'empty',
    readline: 'empty'
  },

I have > 1.9k tweets in my parse core. I updated my webpack config but it still gives me the same error.
Here's my updated code.

Full Code

Issue 140

The issue is that you are using two different versions of the Parse SDK.
require('parse-react') pulls in the normally-named 'parse' npm module, but you're constructing promises with 'parse-node'.
If you switch to using require('parse'), you'll be fine.