bad parentage for hydrogen peroxide terms

GO:0042542    response to hydrogen peroxide
has
GO:0046677    response to antibiotic
ancestor

GO:0042743    hydrogen peroxide metabolic process
has
GO:0016999   antibiotic metabolic process

These will be fixed if we act on my proposal to stop reasoning over roles.
This table shows all of the relationship changes for various hydrogen peroxide terms that will occur if we drop reasoning over roles. The two relationships @ValWood listed are indicated with stars in those rows.

Term
ID
Relationship change
Related term

cellular response to hydrogen peroxide
GO:0070301
Removed SubClassOf
cellular response to antibiotic

cellular response to hydrogen peroxide
GO:0070301
Removed SubClassOf
cellular response to drug

cellular response to hydrogen peroxide
GO:0070301
Removed SubClassOf
cellular response to toxic substance

hydrogen peroxide biosynthetic process
GO:0050665
Removed SubClassOf
antibiotic biosynthetic process

hydrogen peroxide biosynthetic process
GO:0050665
Removed SubClassOf
cofactor biosynthetic process

hydrogen peroxide biosynthetic   process
GO:0050665
Added SubClassOf
cellular biosynthetic process

hydrogen peroxide catabolic process
GO:0042744
Removed SubClassOf
antibiotic catabolic process

hydrogen peroxide catabolic process
GO:0042744
Removed SubClassOf
cofactor catabolic process

hydrogen peroxide catabolic process
GO:0042744
Removed SubClassOf
drug catabolic process

hydrogen peroxide catabolic   process
GO:0042744
Added SubClassOf
cellular catabolic process

**hydrogen peroxide metabolic process
GO:0042743
Removed SubClassOf
antibiotic metabolic process**

hydrogen peroxide metabolic process
GO:0042743
Removed SubClassOf
cofactor metabolic process

hydrogen peroxide transmembrane transport
GO:0080170
Removed SubClassOf
cofactor transport

hydrogen peroxide transmembrane transport
GO:0080170
Removed SubClassOf
drug transmembrane transport

hydrogen peroxide   transmembrane transport
GO:0080170
Added SubClassOf
transmembrane transport

negative regulation of hydrogen peroxide   metabolic process
GO:0010727
Removed SubClassOf
negative regulation of cofactor metabolic process

negative regulation of hydrogen   peroxide-induced cell death
GO:1903206
Removed SubClassOf
negative regulation of cellular response to drug

positive regulation of hydrogen peroxide   metabolic process
GO:0010726
Removed SubClassOf
positive regulation of cofactor metabolic process

positive regulation of hydrogen   peroxide-induced cell death
GO:1905206
Removed SubClassOf
positive regulation of cellular response to drug

regulation of hydrogen peroxide metabolic   process
GO:0010310
Removed SubClassOf
regulation of cofactor metabolic process

regulation of hydrogen peroxide-induced   cell death
GO:1903205
Removed SubClassOf
regulation of cellular response to drug

**response to hydrogen peroxide
GO:0042542
Removed SubClassOf
response to antibiotic**

response to hydrogen peroxide
GO:0042542
Removed SubClassOf
response to drug

response to hydrogen peroxide
GO:0042542
Removed SubClassOf
response to toxic substance

This table summarizes the 24 changes related to hydrogen peroxide terms with respect to some general categories. 21 relationships involving the roles antibiotic cofactor drug, and toxin will be removed. 3 relationships to general terms for cellular metabolism or transport will be added.
While I haven't looked at everything in as much detail as I've just done for these hydrogen peroxide terms, I think this is a good representative sample of the kinds of changes that comprise a large portion of the changes that will occur if we stop reasoning over roles.

category
Added SubClassOf
Removed SubClassOf

metabolism, general
2

transport, general
1

antibiotic

5

cofactor

7

drug

7

toxin

2

Grand Total
3
21

This was fixed by this ticket:
cessation of reasoning over ChEBI roles, i.e. stop inferring relationships like "ATP binding" is_a "drug binding" #19479


Add Code of Conduct file

I added a standard CODE_OF_CONDUCT.md file

Sorry @ronsigal - I somehow oversaw this PR and I added CODE_OF_CONDUCT.md file already in the meantime.


Installing failed

here is the output

<EMAIL_ADDRESS>install C:\Users\Weiwei\AppData\Local\Temp\apm-install-dir-116222-12908-114irv3\node_modules\term3\node_modules\ptyw.js
node-gyp rebuild

C:\Users\Weiwei\AppData\Local\Temp\apm-install-dir-116222-12908-114irv3\node_modules\term3\node_modules\ptyw.js>if not defined npm_config_node_gyp (node "C:\Users\Weiwei\AppData\Local\atom\app-1.6.0\resources\app\apm\node_modules\npm\bin\node-gyp-bin\....\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node  rebuild )
�ڴ˽��򷽰���һ������һ����Ŀ����Ҫ���ò������ɣ������ӡ�/m�����ء�
MSBUILD : error MSB3428: δ�ܼ��� Visual C++ ������VCBuild.exe����Ҫ���������⣬1) ��װ .NET Framework 2.0 SDK��2) ��װ Microsoft Visual Studio 2005���� 3) ��������������װ��������λ�ã��뽫��λ�����ӵ�ϵͳ·���С� [C:\Users\Weiwei\AppData\Local\Temp\apm-install-dir-116222-12908-114irv3\node_modules\term3\node_modules\ptyw.js\build\binding.sln]
MSBUILD : error MSB3428: δ�ܼ��� Visual C++ ������VCBuild.exe����Ҫ���������⣬1) ��װ .NET Framework 2.0 SDK��2) ��װ Microsoft Visual Studio 2005���� 3) ��������������װ��������λ�ã��뽫��λ�����ӵ�ϵͳ·���С� [C:\Users\Weiwei\AppData\Local\Temp\apm-install-dir-116222-12908-114irv3\node_modules\term3\node_modules\ptyw.js\build\binding.sln]
npm WARN deprecated<EMAIL_ADDRESS>react-tools is deprecated. For more information, visit https://fb.me/react-tools-deprecated
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs version 3 and before will fail on newer node releases. Please update to<EMAIL_ADDRESS>as soon as possible.
npm ERR! Windows_NT 6.2.9200
npm ERR! argv "C:\Users\Weiwei\AppData\Local\atom\app-1.6.0\resources\app\apm\bin\node.exe" "C:\Users\Weiwei\AppData\Local\atom\app-1.6.0\resources\app\apm\node_modules\npm\bin\npm-cli.js" "--globalconfig" "C:\Users\Weiwei\.atom\.apm\.apmrc" "--userconfig" "C:\Users\Weiwei\.atom\.apmrc" "install" "C:\Users\Weiwei\AppData\Local\Temp\d-116222-12908-mgejb7\package.tgz" "--target=0.34.5" "--arch=ia32"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the ptyw.js package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls ptyw.js
npm ERR! There is likely additional logging output above.

+1

><EMAIL_ADDRESS>install C:\Users\user\AppData\Local\Temp\apm-install-dir-116412-6448-1gmvkxp\node_modules\term3\node_modules\ptyw.js
> node-gyp rebuild

C:\Users\user\AppData\Local\Temp\apm-install-dir-116412-6448-1gmvkxp\node_modules\term3\node_modules\ptyw.js>if not defined npm_config_node_gyp (node "C:\Users\user\AppData\Local\atom\app-1.7.3\resources\app\apm\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node  rebuild )

npm WARN deprecated<EMAIL_ADDRESS>react-tools is deprecated. For more information, visit https://fb.me/react-tools-deprecated
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\Users\\user\\AppData\\Local\\atom\\app-1.7.3\\resources\\app\\apm\\bin\\node.exe" "C:\\Users\\user\\AppData\\Local\\atom\\app-1.7.3\\resources\\app\\apm\\node_modules\\npm\\bin\\npm-cli.js" "--globalconfig" "C:\\Users\\user\\.atom\\.apm\\.apmrc" "--userconfig" "C:\\Users\\user\\.atom\\.apmrc" "install" "C:\\Users\\user\\AppData\\Local\\Temp\\d-116412-6448-2zmcf\\package.tgz" "--target=0.36.8" "--arch=ia32"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the ptyw.js package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls ptyw.js
npm ERR! There is likely additional logging output above.

Same issue here. node-gyp rebuild fails. +1

This is super frustrating for me, as I can't reproduce it on any system. I've tried my Mac, my Ubuntu laptop, my Windows machine, and even other people's macs. None of them have had any issues installing the plugin.

Anyone got the fix for this?  I experienced the same issue as mentioned.
It seemed the installation of ptyw.js failed.  But if the ptyw.js installed fine if installed by npm.  however, it still failed if I tried to install using apm ...
I was able to install the term3 on my Mac.  But just not Windows
gyp ERR! System Windows_NT 6.1.7601

I ran into similar problems. Here is how I believe I got it working:

Install MS components needed for build:  I used windows-build-tools

Download and install .Net Framework 4.5 (needed for windows 7 machine)
install windows build tools
npm install --global windows-build-tools
In my case, I also ran into an issue windows-build-tools#12, which I fixed by letting the install (WindowsBuildTools_Run.exe) finished, and did  the following

npm config set msvs_version 2015
npm config set python "%USERPROFILE%\.windows-build-tools\python27\python.exe"

After step 1 was completed, I still ran into __pfnDliNotifyHook2 redefinition error during compilation. It is fixed by upgrading npm with npm install -g npm . See: https://github.com/Microsoft/nodejs-guidelines/blob/master/windows-environment.md#compiling-native-addon-modules

ptwy.js could finally be built successfully, and thus atom-term3.


Document how to make bare package list

If you're like me, you probably just want to install the latest version of a package, whatever it is. You could write a quick script to do it, or you could use grep to filter out the version information:
apm list --installed --bare | grep '^[^@]\+' -o > my_atom_packages.txt


Authentication and Encryption

In early analysis of 0MQ security underlying DCell, it seems the best way to bring authentication and encryption to DCell is by implementing CurveZMQ:
https://github.com/zeromq/libcurve
Research the API; if possible and wise: integrate libcurve to provide security for DCell.

[x] Research API
[x] Write integration
[ ] Test implementation
[ ] Write applicable tests
[ ] Update documentation
[x] Write example code

Coverage:

[ ] Encryption
[ ] Authentication

czmq is about to get official ruby bindings, so my wrapper won't receive any updates anymore: https://github.com/zeromq/czmq/tree/master/bindings/ruby/lib/czmq/ffi

Will reapproach once bindings land.


Fix cookie issue

Ref:

https://github.com/exactlab/fractal-containers/issues/3

See:

https://kit.svelte.dev/docs/hooks#server-hooks-handlefetch

From the SvelteKit documentation (https://kit.svelte.dev/docs/hooks#server-hooks-handlefetch):

Credentials
For same-origin requests, SvelteKit's fetch implementation will forward cookie and authorization headers unless the credentials option is set to "omit".
For cross-origin requests, cookie will be included if the request URL belongs to a subdomain of the app — for example if your app is on my-domain.com, and your API is on api.my-domain.com, cookies will be included in the request.
If your app and your API are on sibling subdomains — www.my-domain.com and api.my-domain.com for example — then a cookie belonging to a common parent domain like my-domain.com will not be included, because SvelteKit has no way to know which domain the cookie belongs to. In these cases you will need to manually include the cookie using handleFetch:
src/hooks.server.js
/** @type {import('@sveltejs/kit').HandleFetch} */
export async function handleFetch({ event, request, fetch }) {
  if (request.url.startsWith('https://api.my-domain.com/')) {
    request.headers.set('cookie', event.request.headers.get('cookie'));
  }

  return fetch(request);
}

This matches with the implementation proposed via https://github.com/fractal-analytics-platform/fractal-web/commit/acdea0c4a5f1cd0435e727414765ed10ef155960, where we include the cookie every time an API call is made to fractal-server.

Note: we did not hit this issue before, because we typically have fractal-server and fractal-web run on the same machine.
The issue appeared in https://github.com/exactlab/fractal-containers, where the fractal-web and fractal-server are running from different containers (the same would hold for different machines).


Wrap np.tril and np.triu in jit.

Avoids materializing potentially large triangular mask constants.
May help with #1644

Closing because we'll include this in #1719.


0x23081007 message type support for PM sensors

This is a minimal set of changes needed to add support for PM sensors on RCXAZ air quality sensors which include particulate matter sensor values.
I honestly can't say for sure if this starts in 9-in-1 or 11-in-1 models because they count things like clocks and calendars in the number of available "sensors".
It appears that models with newer sensors just send the data from the added sensors in different message types. This means newer models will share sensor message formatting for previously-supported sensors. There's no guarantee it'll always work this way, but it's helpful as is.
One interesting thing to note is that ESPHome constants include ICON_MOLECULE_CO2 but not ICON_MOLECULE for some reason.

Linking to issue #3 which is the reason for this PR and provides a lot of technical data.

Thanks for your continued efforts!


Pulling Fuel from Central complains about missing Result, due to missing version

Bug Report
Description
When using central and using Fuel this way:
repositories {
    mavenCentral()
}
dependencies {
    implementation("com.github.kittinunf.fuel:fuel:+")
    implementation("com.github.kittinunf.fuel:fuel-coroutines:+")
    implementation("com.github.kittinunf.fuel:fuel-gson:+")
}

Gradle complains:
* What went wrong:
Execution failed for task ':compileKotlin'.
> Could not resolve all files for configuration ':compileClasspath'.
   > Could not find com.github.kittinunf.result:result:2.2.0.
     Searched in the following locations:
       - https://repo.maven.apache.org/maven2/com/github/kittinunf/result/result/2.2.0/result-2.2.0.pom
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project : > com.github.kittinunf.fuel:fuel:2.2.3

This is caused by the fact that the first version of Result published on Central (thanks to kittinunf/result#70) is 3.0.1.
Proposed solution
I suggest tackling this in two rather straightforwards steps:

Make the current Fuel depend on Result 3.0.1
Release a new version of Fuel

Since Central enforces a no-retract policy, it's impossible to change the current version to point to Result 3.0.1, a new release is thus necessary.
Alternate solution
Manually releasing result 2.2.0 on Central would work as well, but that requires manual intervention, and thus precious human time.
Temporary workaround
Who is currently using Fuel via Central can work around the issue by enforcing a compile-time dependency on Result (warning: it taints the dependency tree)
dependencies {
    implementation("com.github.kittinunf.result:result:3.0.1")
}

Example work around here: https://github.com/kittinunf/fuel/issues/757

I will take care of this 🙇 sorry for inconveniences 🙏

Thank you @kittinunf :pray: :bow:

Hello
Any way to help? I cannot override the dependency as a workaround, as I'm using Kotlin Scripting.

@nfrankel Let me help you out by manually pushing 2.2.0 to the maven central 🙇

Thanks a lot @kittinunf! However, in the meantime, I switched to OkHttp because I was blocked: the override workaround doesn't work in the context of Kotlin Scripting

@nfrankel Oh no, sorry for the delay :( I will try to be more promptly support next time 🙇

@kittinunf No worries! This is your personal Open Source project, I don't expect timely support.
Keep up the good work

@kittinunf any update on this? I am facing the same error for version 2.2.3

We have released new version 2.3.0 and it will reference the Result library (3.1.0) from the mavenCentral. It should take a few hours or so for everyone to be able to fetch! 🙇 🙇 🙇

I think this should solve all of the problem regarding to version on mavenCentral. 🙇 🙇 Feel free to re-open if this doesn't fix your issue. Otherwise, please let me close this. :)


Bump ansi-regex for workbench

Signed-off-by: Joshua Li<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #975 (36ff978) into 2.x (2a16227) will decrease coverage by 34.79%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##                2.x     #975       +/-   ##
=============================================
- Coverage     97.55%   62.76%   -34.80%
=============================================
  Files           303       10      -293
  Lines          7815      658     -7157
  Branches        507      119      -388
=============================================
- Hits           7624      413     -7211
- Misses          190      192        +2
- Partials          1       53       +52

Flag
Coverage Δ

query-workbench
62.76% <ø> (?)

sql-engine
?

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

Impacted Files
Coverage Δ

...g/opensearch/sql/data/model/ExprIntervalValue.java

.../opensearch/sql/planner/logical/LogicalWindow.java

...rotocol/response/format/JsonResponseFormatter.java

...ch/sql/expression/aggregation/NamedAggregator.java

...nsearch/sql/expression/window/WindowFunctions.java

...ch/sql/expression/aggregation/CountAggregator.java

...ql/planner/optimizer/rule/PushFilterUnderSort.java

.../sql/planner/physical/PhysicalPlanNodeVisitor.java

...ch/sql/opensearch/planner/physical/ADOperator.java

...h/sql/opensearch/data/type/OpenSearchDataType.java

... and 303 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


V3 Upgrade : ESM, TypeScript 4.9 & Node.js support

feat(upgrade): migrate to ESM
feat(upgrade): add exports
feat(upgrade): upgrade to TypeScript 4.9
feat(upgrade): add node.js support
feat(upgrade): adjust parse.getMessages signature

cc @vishwam

Here is the package 🎉 : https://www.npmjs.com/package/@fortaine/fetch-event-source

Hey, @gfortaine, you've done a perfect job. Is it strictly required to use "node": ">=18.12" I want to use your package while official is in progress, but I use node16. Can you downgrade this requirement?

@lexich Done ✅ : https://www.npmjs.com/package/@fortaine/fetch-event-source/v/3.0.4?activeTab=explore

@firedog1024 Wouldn't Microsoft mind to prioritize this PR, please ?

@gfortaine On Node 16.15.1, with a node-fetch polyfill, it gives the following error:
TypeError: stream.getReader is not a function
    at getBytes (file:///.../node_modules/@fortaine/fetch-event-source/lib/esm/parse.js:2:27)
    at create (file:///.../node_modules/@fortaine/fetch-event-source/lib/esm/fetch.js:54:23)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

would be awesome to get this merged, IMO

@waylaidwanderer Your fix works a treat!
I added that fix to @gfortaine's fixes and published them here: https://www.npmjs.com/package/fetch-event-source-hperrin

@vishwam can we get this merged please?

@gfortaine can you add this fix to your PR please (and ideally your package also, since I'm currently using that until this PR is merged)

If this can be be fixed and released, many people, including me, would be helped !

@gfortaine can you add this fix to your PR please (and ideally your package also, since I'm currently using that until this PR is merged)

Fontaine's lib would work well under NodeJS 18.17.1. Even without the fix that you referred to. Guess Native Fetch implementation has been improved, and connection header seems to be not forbidden as well

@vishwam @firedog1024 has this project been abandoned? What's the issue with getting this merged? Clearly a lot of people are eager for it, given the comments here

You might want to check out extended-eventsource as potential new replacement. It's codebase looks proper.

What's the status of this PR?

Ok but what would be the right way to know if this is moving forward? Sorry for the noise

You would see it happening here.


⚠ ClamAV has degraded performance

In b3bc8dc, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 203 ms

Resolved: ClamAV performance has improved in a95bbf8.


Document in the readme all possible parameters in the Bicep files

I've reviewed the current parameters and made some fixes with 40b3400fa2f7fe6f720f811fde2d4e73f6afbb77 commit. I think we are good now.


Changing indentation settings is not effective

Expected Behavior
When indentSize and continuationIndentSize are set to value other than 4 it should take it into consideration.
Current Behavior
When I set indentSize and continuationIndentSize to 2, detekt-formatting still expects 4-space indentation.
Steps to Reproduce (for bugs)

Change indentation: indentSize and continuationIndentSize to 2 in default-detekt-config.yml

  Indentation:
    active: true
    autoCorrect: true
    indentSize: 2
    continuationIndentSize: 2

Write code with 2-space indentation.
Run gradle detektCheck
Detekt reports error

Ruleset: formatting - ...
	Indentation - [PsiWhiteSpace] at ...

Context
It's not possible to change default indentation.
Your Environment

Version used: 1.0.0.RC8
Operating System and version: Mac OS

Example project:
detekt-indentation-bug.zip

Run
./gradlew detektCheck

The result is:

Ruleset: formatting - 5min debt
        Indentation - [PsiWhiteSpace] at /Users/user/projects/detekt-indentation/src/main/kotlin/Main.kt:1:32

although indentation in line 2 is correct - 2 spaces.
Line 6 indented with 4 spaces is correct.

I have decreased MaximumLineLength to 60, to have a proof that configuration file default-detekt-config.yml is taken into consideration.

Thanks for raising this issue and providing a sample. That's a bug.
The configuration option for the Indentation rule is never read from the detekt-config.yaml file.
The same is true for the MaximumLineLength rule.


ociTools: Fix invalid cgroup mount option

Description of changes

Fix a typo in cgroup mount options: realatime -> relatime
Things done
Created an OCI image using pkgs.ociTools.buildContainer and ran it with runc. This was not possible before:

ERRO[0000] runc run failed: unable to start container process: error during container init: error mounting "cgroup" to rootfs at "/sys/fs/cgroup": mount cgroup:/sys/fs/cgroup (via /proc/self/fd/6), flags: 0xf, data: realatime: invalid argument

Relevant tests did not exist (otherwise they would have failed), but ran some in any case. I would love to add some automated tests, but would need some help adding them. I've already added an example that can be manually tested here.

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@aib I think a test would make sense. But I also need to look how to best do this. We can do this in another PR as I like to merge this already to get the fix in.


"InvokeSet should not be used"

InvokeSet is the only way I can see that allows you to set the value of an attribute that currently has no value.  The Properties collection can only be used to update attributes that have a value.  Even if you specify additional properties to return in the DirectorySearcher, the entry will not contain them and there is no way to set them.
There is a method "Add" but you cannot use it because the collection is a fixed size.
If there is a better way for this scenario, the documentation should specify how because InvokeSet is the only way I can see to do it...

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

ID: 1db88fe4-b886-0ba0-6d0a-08abc2e836ac
Version Independent ID: 88779c68-f022-dcbc-d242-7e9c2be1bd07
Content: DirectoryEntry.InvokeSet(String, Object[]) Method (System.DirectoryServices)
Content Source: xml/System.DirectoryServices/DirectoryEntry.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

@tquerec, @josephisenhour, @bongiovimatthew-microsoft can you help with this feedback? Can you please let us know if there's something we should add to improve the docs for this API? Thanks.

It should work if you try it this way. You should of course replace the ***** with your real values.
`            var de = new DirectoryEntry("LDAP://.net/CN=,CN=Users,DC=,DC=net",
"", // Replace with username
"******", // Replace with password
AuthenticationTypes.Secure);
        de.Properties["Description"].Add("New Description");
        de.CommitChanges();

`

That code did the trick.  Thank you.

From: Joe Isenhour<EMAIL_ADDRESS>Sent: Tuesday, September 25, 2018 10:49:36 AM
To: dotnet/docs
Cc: Timothy Carroll; Author
Subject: Re: [dotnet/docs] "InvokeSet should not be used" (#7811)
It should work if you try it this way. You should of course replace the ***** with your real values.
` var de = new DirectoryEntry("LDAP://.net/CN=,CN=Users,DC=,DC=net",
"", // Replace with username
"******", // Replace with password
AuthenticationTypes.Secure);
    de.Properties["Description"].Add("New Description");
    de.CommitChanges();

`
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/docs/issues/7811#issuecomment-424373878, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFM6_z4qzyFz6PY1MXi-YGJZzv-PX3xEks5uekKAgaJpZM4WzQep.


Module for shipping to LogDNA's REST API

This is a feature request for adding an adapter for the LogDNA REST API so it can be used with Docker.
Here's there API documentation: https://docs.logdna.com/docs/logdna-cli
Basically the docker log line just needs to be posted to the /logs/ingest endpoint with the following content:
{
   "lines":[
      {
         "timestamp":1464041337000,
         "line":"This is an awesome log statement",
         "file":"example.log"
      }
   ]
}

I'll try to give it a shot myself but I don't have that much time to spend on it. If anyone wants to give it a shot that'd be great 💯

You can create a third party module for this feature. I don't think it belongs in logspout core.

@dewey can't you use logdna's syslog support (https://docs.logdna.com/docs/rsyslog) ?

@henridf Yea, they added that now so it's not necessary any more.


Insert fails with   'url argument is missing'

Bug report

[x ] I confirm this is a bug with Supabase, not with my own application.
[ x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
Im running this insert:
    const res = await supabase.from("waitinglist").insert({
      email:email
    })

And im getting this error
  error: {
    code: 'P0001',
    details: null,
    hint: null,
    message: 'url argument is missing'
  },

When changing the same lines to
    const res = await supabase.from("proposed_features").insert({
      idea: email
    })

It works. Why?
I even worked some days ago, without any bigger changes done to the app.
To Reproduce
I dont know what the error is about, google does not return anything. And I dont know what makes this reproducable without sharing secrets
Expected behavior
Expected to insert the expected value.
System information

OS: macOS
Version of supabase-js: 2.39.0
Version of Node.js: 16.20

Additional context
None really

So this is some of the complexities of having things a bit to much abstracted from each other in supabase. It seemed I have initated a trigger and that was having some issues. I guess I have never completed the trigger and added it for fun. But it affected my total flow and without any clear warnings.
What got me on the path was a google search that talked about functions. That got me on the right path to find what might have happened. This could have been solved if terraform was possible or some other IaC tool. Then it would be clear what has been made.
I will leave this open for about a week, in hope that it can be used as a user feedback entry and help the supabase team

This happened to me as well today. The insert was working fine until I added a DB trigger to it.

Maybe there is needed some better DX on this? Better error handling or some information to share so that one knows that the trigger affecets things?

I will close this for now

@vongohren Why did you close this? This is a serious bug that does not allow the use of newly created triggers.

Oh! I thought it was only me who miss configured something and did not dive into it. As my main issue was DX of how the error manifested itself and made it difficult for me to read.
But if you are sure it affects the product itself, I will leave it open a bit more.

I confirm that a recently created http trigger is preventing connected DB inserts with the following error: {"code": "P0001", "details": null, "hint": null, "message": "url argument is missing"}

I looked deeper into this, and believe the reason is based on the trigger you are choosing. If you choose the built in HTTP trigger, the database row have to have a column with value URL and some others. Because that is how triggers work, the row itself has the values it needs, and the specific trigger selected has a dependency on URL. You can make your own function that dont depends on url, and specifies this in the function or some other place.
So based on this I will close this issue, as its not an issue, just dependent on the configuration a person chooses.

Hi. I was facing this exact issue today and it seems there have been changes made in the UI. So at first I had to create a webhook with the edge function selected which created the database trigger automatically. Then it started to work fine.


Update RPC OpenRPC spec

Really Nice!

Hello! Is this PR still a WIP? Or are we ready to merge? @aditya1702

@aditya1702 if you want to take a crack at moving the changes into that directory, /openrpc/README.md can help with some guidance and tips.
On the other hand, if you want, I can help migrate the changes into those files. I know it's a tricky setup, and I'd hate for you to have to duplicate the work

Hello! Is this PR still a WIP? Or are we ready to merge? @aditya1702

@briwylde08 Actually we do want these changes to go in. However, we had to revert these changes temporarily in rpc for the current release. So it makes more sense to do them in the next one when they actually go through. I will close this PR for now and create a new one then


dynamic rebalancing and resharding

To implement the feature of dynamic shard splitting and migration

Yes，It is very important that we will do it as soon as possible。


Reload apps the same way as domains and databases

Reload deprovisioning apps the same way we do for domains and databases
to be consistent with how they do it. This also means that we do not
need to reload the operation when deprovisioning an app, which should
also fix #715.
cc @sandersonet @krallin

👍


feat: made poll choices be multiple options instead of one pipe separated string

closes #249

You guys really haven't approved these yet huh


Updated Italian translation to 19.43

Includes translations for 19.42 and missing ones for 19.41

Thanks!


Unaccessible content while someone is seeding

Operation System: MacOS, Linux, Windows
Beaker version: Many
Today I was in class with my students and everyone  published a page using Beaker’s “Create new from folder”. Once published, only a couple of dat URL were accessible to some of the people in the class (not all). As far as I can remember, once a content is public, anybody can connect to it directly? Why do we get the “Add to watchlist” page?

Everybody has Beaker open so seed actually existed
I personally checked if the web project were “published”
This is not the first time I meet this issue, I already had to wait hours before a friend could access my newly made dat site.

Is this a known issue?

That's a shame. It sounds like you were all on the same LAN, yes? I wonder if we have a LAN discovery issue. Is there any chance the LAN would be hostile to multicast traffic?

Same LAN: Yes we were. The school’s.

Is there any chance the LAN would be hostile to multicast traffic?
I have no idea. I can make some tests next time. How can I do so? Can hashbase help to solve that temporarily?

Yeah, Hashbase can help by being generally more reliable. There's a really good chance the school is blocking the multicast traffic. We just haven't had the resources to work around that scenario.

Good to know. I will use Hashbase next time. Thanks!

Yeah. Sorry for the issues!


Ability to use wildcards when disabling cops

Is your feature request related to a problem? Please describe.
When it is necessary to write a method that RuboCop decides is too complex, it tends to trigger multiple cops. If there is no easy way to rewrite the method in a reasonable amount of time, it is necessary to disable multiple cops on one line:
def complex_method # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity

This is a lot of typing! However, disabling all cops for the method with a simple rubocop:disable is overkill.
Describe the solution you'd like
It would be useful to have wildcard support:
def complex_method # rubocop:disable Metrics/*

or
def complex_method # rubocop:disable Metrics

(arguably, the first is more explicit and guards against typos better)
Describe alternatives you've considered
Obviously, I considered breaking the method down into simpler code 😁 , but it doesn't seem to be particularly easy in this case, and the required dev time is a factor.
Supporting bash-style globbing would be nice, but would probably be a lot more hassle to implement for something that only makes the line slightly shorter:
def complex_method # rubocop:disable Metrics/{AbcSize,CyclomaticComplexity,MethodLength,PerceivedComplexity}

Ideally, it would be possible to define groups in rubocop.yml, and then disable those:
Groups:
  ComplexityChecks:
    - Metrics/AbcSize
    - Metrics/CyclomaticComplexity
    - Metrics/MethodLength
    - Metrics/PerceivedComplexity

def complex_method # rubocop:disable Groups/ComplexityChecks

but I suspect that will be a lot of work to implement, and I'm not sure if there's enough demand for this feature to justify it.
Additional context
none

def complex_method # rubocop:disable Metrics

It seems good to me that it can be specified by department. because there are similar specification options such as rubocop --only department and enabling/disabling by department in .rubocop.yml. It does not bring in new complexity.
The only concern is that the specified department's cops hidden in the department may not be able to detect an offense. However, it shouldn't be the issue as user will accept the risk and specify it.

I'm working on the class that's responsible for this thing. So further I can implement that. @koic could you assign this issue to me?

@AndreiEres Done.

https://github.com/rubocop/rubocop/pull/9630

Added department checking to Registry
Added list of names for department to Registry

I think it can be done with https://github.com/rubocop/rubocop/pull/9626

Added checking for department only to directive comment.
Added note about new feature to documentation.

@bbatsov or other, could you take a look?

Guys, I discovered that disabling of all department has consequences. How we can deal with redundant department disabling? I realised that the best way — do not use RedundantCopDisabling, but create another cop — RedundantDepartmentDisabling. So we will ignore department disabling in RedundantCop processing and check it only for RedundantDepartment.
What do you think?

Don't they have to work together? As I mentioned in #9563, if a department is disabled, disabling a specific cop below it is redundant.
# rubocop:disable Metrics
...
# rubocop:disable Metrics/Foo <-- redundant

Oh, it's true. If we split that cops we can't catch that interference. Hard thing :-)


Error while downgrading to 2.1.0

I've tried around 7 times now and I keep getting the Nintendo "An error has occured. Hold down the POWER Button to turn off the power, then turn it on and try again." error.
I've done everything exactly as stated, what is it that I'm missing?
cheers

Uninstall plaisysupdater and reinstall and try aging. If that don't work try to reinstall your TWL(it's in the guide on how to do it)

On Jun 22, 2016, at 8:23 AM, subree<EMAIL_ADDRESS>wrote:
I've tried around 7 times now and I keep getting the Nintendo "An error has occured. Hold down the POWER Button to turn off the power, then turn it on and try again." error.
I've done everything exactly as stated, what is it that I'm missing?
cheers
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Post your log files like the troubleshooting page instructs.

Closing this for inactivity


Files Plugin has wrong order of arguments

In the function doWriteFile, the arguments to writeFile should be swapped. FilePath should come before contents.

To be consistent with how Haskell normally orders them? Good idea.


Flutter sdk & packages upgraded to the last versions.

Flutter version & network info plus package version upgraded.

@diantahoc @E-MRE Thanks a bunch for both of your great work & help on this.
@diantahoc It'd be great if we can merge this PR. This works even with the latest version released yesterday at Google IO 2024.
Until this is merged, I'm using the below (hope this helps someone):
flutter_pos_printer_platform_image_3:
git:
url: https://github.com/diantahoc/flutter_pos_printer_platform.git
ref: 9624e1cea4a4bf669ca31f5ff7f45b251581a512

Hello.
Trying to use this fix, the build failed on Codemagic in the generation step:
> flutter.BAT build windows --release

windows/flutter/CMakeLists.txt does not use FLUTTER_TARGET_PLATFORM, updating.
Building Windows application...
CMake Error:
  Error evaluating generator expression:

    $<TARGET_FILE:flutter_pos_printer_platform_image_3_plugin>

  No target "flutter_pos_printer_platform_image_3_plugin"

CMake Error:
  Error evaluating generator expression:

    $<TARGET_FILE:flutter_pos_printer_platform_image_3_plugin>

  No target "flutter_pos_printer_platform_image_3_plugin"

CMake Error:
  Error evaluating generator expression:

    $<TARGET_FILE:flutter_pos_printer_platform_image_3_plugin>

  No target "flutter_pos_printer_platform_image_3_plugin"

@E-MRE can you explain what we're doing wrong?

I've solved my problem by renaming my fork of flutter_pos_printer_platform_image_3 to flutter_pos_printer_platform again.
Apparently, for some reason the _image_3 bit breaks the Windows build.
You can check our fix (or workaround?) here: https://github.com/in4matic-srl/flutter_pos_printer_platform/tree/renamed
The cascade forking is going out of control, so I don't know if I should submit a PR and where.

Getting problems when upgrading  my android project to target targetSdkVersion of 34. For some reason bluetoothService is not being initialized. Problem is happening literary on permission requests.

Also getting similar initialization issue specifically with the bluetooth and usb service. below is the thrown issue
E/MethodChannel#com.sersoluciones.flutter_pos_printer_platform( 6189): Failed to handle method call
E/MethodChannel#com.sersoluciones.flutter_pos_printer_platform( 6189): kotlin.UninitializedPropertyAccessException: lateinit property bluetoothService has not been initialized
E/MethodChannel#com.sersoluciones.flutter_pos_printer_platform( 6189): 	at com.sersoluciones.flutter_pos_printer_platform.FlutterPosPrinterPlatformPlugin.verifyIsBluetoothIsOn(FlutterPosPrinterPlatformPlugin.kt:307)
E/MethodChannel#com.sersoluciones.flutter_pos_printer_platform( 6189): 	at com.sersoluciones.flutter_pos_printer_platform.FlutterPosPrinterPlatformPlugin.onMethodCall(FlutterPosPrinterPlatformPlugin.kt:221)
E/MethodChannel#com.sersoluciones.flutter_pos_printer_platform( 6189): 	at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage(MethodChannel.java:267)

Getting problems when upgrading my android project to target targetSdkVersion of 34. For some reason bluetoothService is not being initialized. Problem is happening literary on permission requests. @E-MRE @diantahoc this is kind of urgent because of google play target policy described in https://support.google.com/googleplay/android-developer/answer/11926878

So how can I fix this If updated targetSdkversion from 33 to 34?

Getting problems when upgrading my android project to target targetSdkVersion of 34. For some reason bluetoothService is not being initialized. Problem is happening literary on permission requests. @E-MRE @diantahoc this is kind of urgent because of google play target policy described in https://support.google.com/googleplay/android-developer/answer/11926878

Hello, the problem is now fixed as of version 1.2.4


 is not a valid SSR component.

Hi,
New to svelte here. I tried using one of TailwindUI components and got the error below.
TransitionRoot is not a valid SSR component. You may need to review your build config to ensure that dependencies are compiled, rather than imported as pre-compiled modules

From https://github.com/sveltejs/sapper/issues/774 it looks like the library might need to be installed as a devDependency. Can you try that and see if it fixes your issue?
Just to make sure, are you using the right component? The public interface for transitions is <Transition> and <TransitionChild>
If neither of those is the issue, it'd be great if you could create a repository that reproduces this so I can investigate more-- hard to debug these build issues without it.

Looks like I was using the incorrect component.


#159623447 Create API documentation

What does this PR do?
Creates complete API documentation for existing API endpoints based on Swagger
Description of Task to be completed

Create documentation
Ensure consistency of validation messages in documentation and API

How should this be manually tested?

Pull branch into local repo and run npm run dev
Point browser to http://localhost:3000/api/docs
Documentation will be deployed to https://my-diary-api.herokuapp.com/api/docs

What are the relevant pivotal tracker stories?
#159623447

Pull Request Test Coverage Report for Build 111

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 80.87%

Totals

Change from base Build 109:
0.2%

Covered Lines:
144

Relevant Lines:
170

💛  - Coveralls


Adjust Agent to use v2 REST API

Background
The new version of Server REST API has been designed.
Goal
Change Atum Agent to utilize v2 REST API endpoints
### Tasks
- [ ] #222

Depends on #140

This item depends on:

AbsaOSS/atum-service#140
By Dependent Issues.


Added group to FastStart AppScalefile.

:+1:

There are a number of AppScale deployments with the same "group" parameter in the logs currently, so I don't think that would be an issue. I tested it by a. running the script multiple times on the same machine and b. by having multiple instances running with the same group defined (faststart-AWS). No unusual behavior in either scenario.


TestNSCAndICMPWebhookLocal failing in CI

Expected Behavior
CI should reliably pass on master
Current Behavior
TestNSCAndICMPWebhookLocal is failing in CI: https://circleci.com/gh/networkservicemesh/networkservicemesh/47149

Reproducible https://circleci.com/gh/networkservicemesh/networkservicemesh/48789?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-checks-link


💡[Feature]: The aim of the project is to analyze and predict the disease based on the given dataset.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
To analyze the dataset of Cardiovascular Disease and build and train the model on the basis of different features and variables.
Use Case
To analyze the dataset of Cardiovascular Disease and build and train the model on the basis of different features and variables.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@sanjay-kv  please assign this to me


Launchpad: Show empty and non-empty patterns on Pattern page

It's currently not possible to tell if a pattern on the Pattern-select page is empty or not.
Suggest lighting non-empty patterns in a dimmed colour, leaving empty patterns unlit.

This is a good suggestion, however, I have to think what exactly qualifies as an empty sequence. But probably it would just be it's initial state.

Cool. I think one with no steps set to "on" for Note tracks.
Not sure what would define an "empty" Curve track.


bachify QPs

Hey,
thank you for your great work, I am interested in it a lot. I have a question about parallelization. Is it possible to solve many ups at the same time, as mini baches in deep learning? Thank you.

you could. but depending on your use case, relu-qp may or may not be what you are looking for. for general (batched) qp solvers on GPUs please checkout jax and pytorch implementations of the primal-dual interior-point method.


Fix code coverage

Seems like we have code coverage that is supposed to run in CI, but it doesn't seem to work properly:

So either we should fix it or remove it.

Merged codecov 5.0.2

The same error still occurs though?

Updated the token, looks like it works now


Billboards not working on planes

Godot version:

Godot 3.2.3, 3.2.4 Beta 5
OS/device including version:

RTX2080S GLES3
Issue description:

Y billboard breaks planes from external software I would expect it to make the x axis always face the camera as described in the docs.
Billboard off

y billboard on

The example shows y billboard but it seems the issue effects all billboard modes.
Steps to reproduce:

import a plane from blender
add it to mesh instance add assign a material
change material to y billboard.

Minimal reproduction project:

BillBoardError.zip

This is because the billboard expects a different rotation for the plane.
Consider using a MeshInstance with a QuadMesh instead.

This is because the billboard expects a different rotation for the plane.
Consider using a MeshInstance with a QuadMesh instead.

Tried rotating the plane that doesn't work.

Quad mesh does work but I need the pivot to be at the top like in the first screen shot I showed.

Tried rotating the plane that doesn't work.

Quad mesh does work but I need the pivot to be at the top like in the first screen shot I showed.

Ok for the pivot you can add a spatial then add you mesh as a child that allows you to get the same effect as setting the pivot. Im going to close this as it seems this is intended. Perhaps this should be added to the docs though.

Ok for the pivot you can add a spatial then add you mesh as a child that allows you to get the same effect as setting the pivot. Im going to close this as it seems this is intended. Perhaps this should be added to the docs though.


support downloading through corporate proxies

https module does not support proxy.
We use the request module, which has sam'ish API, and support proxy via standard 'https_proxy' environment variable

Hey @tardyp, thank you for the PR!
The request module is quite heavy on dependencies, it adds in 47 packages. We'd like to keep our dependencies as lean as possible, so I'd rather have proxy support implemented manually atop of node's https module.

Indeed, I did not realize the hell of dependencies on top of request. I would however avoid to reinvent the wheel at maintaining proxy handling code inside this repo.
So we have to find the right minimalist npm module, which does proxy and streaming.

quick search shows that there is a global-tunnel package which is more minialist, but it looks unmaintain.
request has 4M downloads per week, so I would expect most people already have those deps is their yarn cache.
related link:
https://github.com/nodejs/node/issues/8381
https://github.com/nodejs/node/issues/1490
interesting comment for me is:

Speaking from prior experience where proxy support was added to an existing product, you start > with HTTP CONNECT but you end up supporting everything from SOCKSv5 to HTTP POST, > basic/digest/and-so-on auth, Kerberos, NTLMv1 and v2, custom CA chains, client certificates, certificate fingerprinting, etc., etc.

That's why there is a request package which handles this for you.

@tardyp thank you for the links, those were really interesting to read.
Here are my findings:

According to this issue, socks proxy is not supported by request/request. So I'd assume we're dealing specifically with HTTP proxy here.
HTTP proxying seems to be doable with node's http module  and https://github.com/Rob--W/proxy-from-env to parse env variables (check out Rob's example).

Let me know if you're interested in implementing this approach. If not, I'd rather go with a graceful failure that would explain how to download needed chromium revision manually.

Could also use caw, which does most of the job automatically. But then again, it also brings some dependencies, although very small ones.

Closing in favor of #455


Add HierarchicalSearch

Implementation of Issue #149

You've copied some of the pseudocode, but you haven't defined what a HLA is, nor a HLA library, nor how a HLA is replaced by a refinement, nor shown an example of the code actually solving a problem. Good start, but needs a lot more work.

@norvig Thanks for the feedback Sir!.
I will be working on this to rectify the mistakes you've mentioned.
Just a question, Do i have to implement a Action class for a HLA or is their some other preferred way?Need your inputs here.
Thanks!!

Good questions, @MaVericKWareZ , and I'm glad you're sticking with this. In Figure 11.4 from https://github.com/aimacode/aima-pseudocode/blob/master/algorithms.pdf we see that a Refinement has an HLA, a list of steps, and optionally a precondition. The HLA has an operator and arguments, and can have variables. You could use the Expr class from logic.py for HLAs; probably a subclass of Expr.

@norvig Sir,Please do have a look at the Code now.
Am i moving in the right direction? Awaiting your feedback and further inputs on this issue.
Also i have tried to add the test_planning.py file for the pytests required for hierarchicalSearch().

There is some prob with my fork of aima-python.So i a closing this PR.
Will follow up with an updated one.


Update announcement.html

Updating the amount of stars in the repository for 3.8k

I'd like to work on this issue ; can you please assign me this  issue or I can just raised the PR ?

I'd like to work on this issue ; can you please assign me this issue or I can just raised the PR ?

I think you could probably just open a PR. It seems like a very small change.
Maybe phase it in such a way that it is clear that change was added after the post war originally written (ie, "(Aug 27 edit: Now 4k stars)"

ok ,I will change it to  4k

I updated it to 6.3k

Refactored for 7.5k Stars

haha best of luck! 1 star from me!

Announcement is a point in time page and represents state of things at a time of the announcement. Dynamic stars counter is already available on our manifesto page. Hope it makes sense.


Error on tfx login after update to 0.1.9

After login to the latest version I now get the following error on tfx login:
Message: Failed Request: Not Found(404) - https://mytenant.visualstudio.com/DefaultCollection/My%20Collection/_apis/connectionData

It worked with the previous version with the same collection url / access token

hmmm, I tried yesterday.  What url are you supplying in tfx login prompts?
It seems your url has Default Collection and My Collection

Yes, that was it. Thanks.
If I pass https://mytenant.visualstudio.com/DefaultCollection it works, if I pass https://mytenant.visualstudio.com/DefaultCollection/My%20Collection the error is thrown. This worked in the previous version, but since it asks for collection url and not project url, this makes sense

OK - just to be clear, is My Collection the name of your project?
Perhaps we can error with a better message if you don't:

supply a path segment of 1 segment or
it has two segments and the first one is 'tfs' (on-prem case)

What do you think?

Yeah, My Collection is the name of the project.
Improved error message are never a bad idea :) For VSO case you might also check for the https:// and the visualstudio.com part.

Yeah, we'll add more robustness and checking here.
We'll refrain from checking for visualstudio.com since we have test int envs etc... fragile.
Thanks!

better error messages: f4c1d9906f68ea7b6a4317e4ca1702e04197fe0b


Corrected TAG > RESET in oms_setup.sh

When the oms_setup.sh script prints the configured parameters it printed "TAG" which was actually a copy paste error and should have been "RESET"
The line changed actually explains it pretty well


⚠ Spryr has degraded performance

In 24be19e, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3184 ms

Resolved: Spryr performance has improved in e493f75 after 9 minutes.


Class not found defined in dbObj:

i met this problem......help me  sos sos sos

It's not clear from the description what the problem is, so I'm going to close this issue.


buildMethod fails on iOS

Bug description
Using buildMethod along with customParameters fails on iOS.
It doesn't seem to be a directly game-ci issue, but something with the docker image.
This is the error that appears:
Aborting batchmode due to failure:
executeMethod class 'Foo' could not be found.
Argument was -executeMethod NameSpace.With.Several.Segments.Foo.Bar

This only happens with iOS specifically.
The platforms StandaloneWindows64, StandaloneOSX, Android, and WebGL work as intended.
I'm using a matrix strategy, so the workflow job for each platform is exactly the same.
Workflow is using the image unityci/editor:ubuntu-2021.3.20f1-ios-1. Unsure if it is just this specific image with the issue, haven't tested with other Editor versions.
Example:
      - name: Build asset bundles
        id: build
        uses: game-ci/unity-builder@v2
        env:
          UNITY_EMAIL: ${{ secrets.UNITY_BASE_EDITOR_EMAIL }}
          UNITY_PASSWORD: ${{ secrets.UNITY_BASE_EDITOR_PASSWORD }}
          UNITY_SERIAL: ${{ secrets.UNITY_BASE_EDITOR_SERIAL }}
        with:
          targetPlatform: ${{ matrix.targetPlatform }}
          gitPrivateToken: ${{ secrets.UNITY_BASE_GITHUB_TOKEN }}
          buildMethod: NameSpace.With.Several.Segments.Foo.Bar
          customParameters: -slackAccessToken ${{ secrets.SLACK_BOT_TOKEN }} -slackChannelId ${{ vars.SLACK_ABB_CHANNEL_ID }}

How to reproduce

Create a new workflow that checks out a unity project using actions/checkout@v3
Unity project needs a namespaced, static editor method to call
Unity project should be version 2021.3.20f1
Use target platform iOS
Call a static editor method with buildMethod
Run workflow

Expected behavior
Unity should find the method and call it.
Additional details
The editor method I'm calling is installed as a git dependency (and later will be made into a package).
It's in an Editor folder in an Editor assembly. The assembly definition has Editor as its only platform.

After researching further I still have the issue, but I believe it is the fault on this side and not on game-ci.
I'll open another issue if it is the case.


Courses

Redesigned Courses

The PR is unfortunately dirty, there are a lot changes which shouldn't be there and it is very hard to do a code review.
Could you please:

Remove commits which do not belong to this PR
Squash commits to get a better commit history ("Fixed icon space", "Small layout changes" - it will be very hard to distinguish what has happend there later)

If you need guidance how to write commit messages, Flynn has a very clean and solid commit history: https://github.com/flynn/flynn/commits/master

Your local branch is probably very messy. I suggesst to pull the current master on top of your local branch, do a diff with origin/master, create a new branch based on origin/master (make sure to fetch first) and the patch and decide how you want to partition the commits.
Here's a guide how to do that: https://github.com/flynn/flynn/pull/1449#issuecomment-96402181

Don't forget to push with: git push myfork localbranch:courses. Oh and feel free to PR against master, I'll merge my changes on top of yours.


on/off labels

hi
its nice to have on/off labels in the ITSwitch . like this picture:

Probably not gonna happen. This switch was developed as a part of a project that I'm no longer working on. Thus I have no time/reason to extend it. If you do I'll gladly accept a pull-request though. 👍

oh okay , thanks for the response though.
ps. i dont have the experience to do it in obj. c , maybe if it was in swift i could try though :b , do you know any similar project thats written in swift?

I don't think there is a production-ready one in Swift. The code should be understandable though even if you don't write lots of Objective-C.

obj-c is like Chinese to my eyes :b i just use ready classes  in my swift projects. :(
i found some switches like yours but they are for iOS , i hope i can tweak them so i can use them for macOS.

Thanks Ilija for this project. I've created a Swift version of ITSwitch with a bunch of styling options such as knob images:
https://github.com/OskarGroth/OGSwitch
I don't really think switch On/Off labels are necessary, but if you need it, it should be trivial for you to add. Don't forget to send a PR :)

I'm closing this as OGSwitch offers a solution to this 🙂


C API

#21
I wouldn't merge this just yet - there are a few ugly hacks in this. Wanted to run some ideas by you first.
So, the first thing is that I had to redefine all of the structs and typedefs. This is because they're scattered throughout the codebase in files with C++ code. My suggestion is that these structs and typedefs are instead put in some sort of XXX-Structs.h file or something.
Also, for many of the structs, I had to strip out the C++ convenience functions:

typedef struct q3Mat3
{
	q3Vec3 ex;
	q3Vec3 ey;
	q3Vec3 ez;
        // C++ convenience functions were here
} q3Mat3;

This would be an easy fix, just add #ifdef __cplusplus things around the convenience functions.
I was also not able to properly expose the classes to C, since classes aren't supported in C. I had to typedef them to void to get it to work:
// bootstrapped types

typedef void q3Body;
typedef void q3Scene;
typedef void q3BoxDef;
typedef void q3Quaternion;
typedef void q3ContactListener;
typedef void q3Render;

I might suggest changing these classes into structs, since it seems like OOP isn't being used anyway.
This last one I don't have any idea on how to tackle. Basically, I wrote the C API functions using a bunch of macros that basically called the original function in C++ (the EXPOSE macro). However, there's a bit of an issue with the pass-by-reference functions (with the & in the arguments). I can't generate them using EXPOSE. So I wrote a workaround for if the function had only pass-by-reference arguments (the PXPOSE macro, P for pointer). But unfortunately there were about 5 or 6 functions that had mixed pass-by-reference and pass-by-value, which I'd have to write by hand (and would be hell to maintain in the future).
My request is that all functions get rid of this pass-by-reference functionality and instead use regular pointers, unless if there's another solution I haven't thought of.

BTW, I should clarify, this compiles and works. But it's ugly.

Hi there! Very cool, I'm glad you took some time to make this. I'll have some time next week to look into it in detail. You're right, OOP is not used and so all member functions can be either hidden away or just converted to C code fairly easily. I think the idea of adding in a structs header is a good idea, but it might not be necessary if the opaque typedefs work out.
As for pass by value/reference, I actually really don't like references and would be completely happy to see them gone. Passing pointers and strictly by-value everything else would be a positive change.

No problem! Thanks for this awesome library :D
And cool, I'll start implementing these when I have time. This C API works for my use case so it may be a while before getting around to it.

Took a closer look. The macros look practical, but I would not want to pull them in in the current state. I think a proper solution would be to just re-write some of the source files in C form. This would be more or less a C port at that point, and someone is working on a C port right now for the Citro3D library (https://github.com/tommai78101/citro3d/tree/physics/source/physics). Personally I don't think I can dedicate time to a C port, and since you got it working for your purposes, all seems well for now :)

@rweichler I would encourage you to continue with your own C implementation port. My library is ported for the Nintendo 3DS family systems, and it is not for PC/Mac.


Remove vault dead code

This has been broken and unused for nearly 2 years. Maybe we will add
vault back at some point, but no need to keep dead code in the mean time
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
505b8553d24683ecb830526ef0ca7e5f9e81864b
link
/test gencheck_istio

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

@howardjohn: PR needs rebase.

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

@JimmyCYJ ptal


Support open files in VSCode-insiders

VSCode-insiders may be the same as VSCode，Its command is  "code-insiders ". Could you please support for it? Thanks.

Sure thing, I can do that.

VS Code Insiders support available in v0.5.1. Enjoy!


Add standard Makefile target documentation

Description

List standard Makefile targets.
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

I don't see push-manifest target, can you include that as well?

CC @tomdee who has been thinking a bit about this.

CC @tomdee who has been thinking a bit about this

Good point.
This largely came out of conversations I have had with @fasaxc , who has been channeling your thoughts to me. Good to have a single place for it though.

@mkumatag wrote:

I don't see push-manifest target, can you include that as well?

Oh yes. I left it out because we don't yet have agreement that multi-arch manifest will be supported. @fasaxc is working to get it from product management, but it depends on agreeing to:

drop support for docker <1.10
drop support for quay.io

Once that is there, happy to add it in (really open a new PR).

Once that is there, happy to add it in (really open a new PR).

make sense.. waiting period.. :)

Based on the discussions and PRs that were merged, adding tag-images targets.
Can we move ahead on this?

@fasaxc updated.


doubling in certain tracks

there is a visual overlay of the transcription on top of itself. Has occurred in Dard's transcription of Hafiz Ali Khan Yaman. When this occurs, user is unable to save.
console error:
TypeError: Cannot read properties of null (reading 'ownerDocument')     at new pt (transform-31f0ced6.js:1:5848)     at Ie (transform-31f0ced6.js:1:6395)     at v.qe [as data] (transform-31f0ced6.js:1:7164)     at Proxy.codifiedAddPlucks (EditorComponent-cdd4b18a.js:187:70213)     at Proxy.codifiedAddArticulations (EditorComponent-cdd4b18a.js:187:68915)     at Proxy.codifiedAddTraj (EditorComponent-cdd4b18a.js:187:91829)     at EditorComponent-cdd4b18a.js:187:91979     at Proxy.forEach (<anonymous>)     at EditorComponent-cdd4b18a.js:187:91951     at Proxy.forEach (<anonymous>) Ua @ index-116aa40f.js:1 transform-31f0ced6.js:1 Uncaught TypeError: Cannot read properties of null (reading 'ownerDocument')     at new pt (transform-31f0ced6.js:1:5848)     at Ie (transform-31f0ced6.js:1:6395)     at v.qe [as data] (transform-31f0ced6.js:1:7164)     at Proxy.codifiedAddPlucks (EditorComponent-cdd4b18a.js:187:70213)     at Proxy.codifiedAddArticulations (EditorComponent-cdd4b18a.js:187:68915)     at Proxy.codifiedAddTraj (EditorComponent-cdd4b18a.js:187:91829)     at EditorComponent-cdd4b18a.js:187:91979     at Proxy.forEach (<anonymous>)     at EditorComponent-cdd4b18a.js:187:91951     at Proxy.forEach (<anonymous>) editor:1 The keyword 'slider-vertical' specified to an 'appearance' property is not standardized. It will be removed in the future. editor:1 The keyword 'slider-vertical' specified to an 'appearance' property is not standardized. It will be removed in the future. DevTools failed to load source map: Could not load content for chrome-extension://fheoggkfdfchfphceeifdbepaooicaho/sourceMap/scripts/iframe_form_check.js.map: System error: net::ERR_BLOCKED_BY_CLIENT DevTools failed to load source map: Could not load content for chrome-extension://fheoggkfdfchfphceeifdbepaooicaho/sourceMap/scripts/iframe_form_detection.js.map: System error: net::ERR_BLOCKED_BY_CLIENT

I think this has been solved


Rendering CSS that isn't inlined

I can render css on my inserted HTML but only if it's an inline style. So far I've not been able to get it to work with any css in a style tag. Is this possible?

I don't think so. Note that HTML in .docx files comes with very severe limitations is only supported by Microsoft Word, and not e.g. LibreOffice as far as I know. Best to check the spec to see if your use case is possible. Maybe you can find that on this site http://officeopenxml.com/WPdocument.php


Predefine sort order for certain keys.

Create a new option which includes an array of strings. If a given key is in the array, it will be displayed before the others, in the order specified in the configuration. Otherwise, the key will be displayed after.
Example:
jsonSortForceOrder:
  - f
  - o
  - u
  - b
  - a
  - r

This JS object:
{
  a: null,
  b: null,
  f:  null,
  z: null,
  k: null,
  u: null,
  o: null,
  r: null,
}

Would be sorted like so:
{
  f: null,
  o: null,
  u:  null,
  b: null,
  a: null,
  r: null,
  k: null,
  z: null,
}

This can already be done with #30 (as seen with the foubar text case) but this would provide a more convenient means of performing this same function without having to create an additional file.

Interesting idea! This sounds useful.
One possible enhancement would be to use a special syntax to denote classes of characters, like regex character classes maybe. So then you could pass in this:
jsonSortForceOrder:
  - f
  - \d
  - r

to force the object { r: null, f: null, 9: null } to be sorted as { f: null, 9: null, r: null }.
Other nice-to-have qualities would be:

The ability to match one character in a class or any number of characters in a class
The ability to specify that a class of characters should be sorted in reverse order
Maybe more advanced regex syntax?
What happens when classes overlap? This should be made clear

This is now supported by the jsonSortOrder option implemented in #92. Let me know what you think!


Docs C-1 Home Page Bad Link

Describe the bug
C-1 Personnel Home Page on the Documentation. The How to Recruit someone link doesn't link to a page.

Made this issue so I remember to fix in next update push.


How do you set up a delay between downloading images?

Mainly for sankaku, but it may be applicable to other booru's as well.
Thanks.

Since 4.4.0; you can just edit the source's settings (from the "Sources" window).
In the "Download" tab, you can set the values for various throttles (thumbs, images, pages, tags, and delay before a new try in case of an error).


Replaced obsolete QGLWidget with QOpenGLWidget

Description
Replaced an obsolete QGLWidget class.
From: http://doc.qt.io/qt-5/qglwidget.html
"This class [QGLWidget] is obsolete. It is provided to keep old source code working. We strongly advise against using it in new code."
List of changes

replaced QGLWidget with QOpenGLWidget class
replaced QGLFormat infrastructure with QSurfaceFormat infrastructure

FYI: I'm in the middle of rewriting all example windows in #271


Solve: angular config for urlpath to REST-API uses default value instead of external config.

This PR solves the issue #295
I put the config data into the ngrx store and then the components are able to wait until the configuration loaded.

Also solves duplicated #310


Allow modifier "ReplacePath" to replace path with a text template.

Description

Extended modifier ReplacePath, allowed to replace path with a text template.
For example: Replace origin path /whoami/** to /who-am-i/**
Usage: set rule value for frontend to ReplacePath:{{Replace . /whoami /who-am-i 1}}, when request path is /whoami/users/1, replaced path will be /who-am-i/users/1
Replace just is an alias for strings.Replace.

Hi @Tiscs
Thanks for your contribution :)
We think it would be better to use regex instead of go templates for this feature (go template are know by few people). WDYT? Do you want to implement this ?

Ping @Tiscs

@emilevauge
Thanks a lot for your response.
I will implement it, it's useful for my project.
Any other suggestions for the syntax?

@emilevauge
I implement regexp based rule and the syntax likes {regexp} $> {replacement}, operator $> means replace to, WDYT?
Or we can add a property "regexp" to "ReplacePath"?

Could you create a decidate middleware ReplacePathRegex ?

@ldez
Added the new middleware ReplacePathRegex and unit tests.

Ping @Juliens

Likes HeadersRegexp, should be ReplacePathRegexp here?

I think in ReplacePathRegex is a bug.
I would like to do the following.
https://cloud.xxxxx.de:51443/webhook_654364/geo -> xxx/webhook/geo
rule = "Host:cloud.xxxxx.de;ReplacePathRegex: ^/webhook_654364/(.*) /webhook/$1"
But the rule is interpreted as a wildcard, because I reach the page now under all paths
https://cloud.xxxxx.de:51443/test3432sadfasfasf
https://cloud.xxxxx.de:51443/testccxswwwsf
https://cloud.xxxxx.de:51443/webhook_654364/geo


:seedling: Bump CodeQL Action version to 3.x and remove whitespace

What kind of change does this PR introduce?
(Is it a bug fix, feature, docs update, something else?)

[x] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
I don't see Dependabot updating the github/codeql action in this particular action workflow, though it's updating a) other actions in this workflow and b) github/codeql in other actions.
Just a while speculation, but this is the only one with the whitespace between the actual name key and the uses? Just in case, firing this to see if the next Dependabot scan will start patching the codeql action in this file also. (and if so, it may infer a minor text checking bug in the upstream Dependabot algorithm?)
What is the new behavior (if this is a feature change)?**
Should allow Dependabot to bump the codeql action in this file.
While here, I also bumped the actions manually to the latest v3.x series (which also includes Node 20, which should help eliminate any actions warnings about Node deprecations).
Note the only other call is this in a different workflow:
https://github.com/ossf/scorecard/blob/e780e089f512f12cd1fc3d090a2424f186ac1a78/.github/workflows/scorecard-analysis.yml#L53-L54

[ ] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes

Special notes for your reviewer
An attempt to help dig more into https://github.com/ossf/scorecard-action/pull/1354.
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

Indeed, but notice that PR is updating the codeql action ONLY in the Scorecard Analysis workflow? It's not updating it in the CodeQL Workflow? (if I'm reading it correctly)

What do you think about trying the v3.x series? (only difference being Node 20, which eliminates a GitHub Runner warning about them deprecating Node 16)

What do you think about trying the v3.x series? (only difference being Node 20, which eliminates a GitHub Runner warning about them deprecating Node 16)

Interestingly I see that commit listed as the ref for the "CodeQL Bundle v2.16.5" release
https://github.com/github/codeql-action/releases
Where as if you look at https://github.com/github/codeql-action/tags:
v3.24.8 uses a different hash: 05963f47d870e2cb19a537396c1f668a348c7d8f
I'm good with trying @05963f47d870e2cb19a537396c1f668a348c7d8f # v3.24.8, just please make sure to update the version in scorecard-analysis.yml too

here's an example of dependabot picking up the updates properly. so I recommend mirroring what we do here:
https://github.com/ossf/scorecard-action/pull/1355/files

@adamdmharvey are you still interested in patching this (based on v3.24.10 at this point, you can see https://github.com/ossf/scorecard-action/pull/1360)? If not, I would be fine putting it as a good-first-issue for someone else to pickup.

Yep!  I'll take a stab.


Feature/e2e fr version

Description
Fixes 1715
Notes (out of scope, known isues, hints for reviewing code, ...)  (optional)
...
Test cases

[auto-generated message] E2E test report on: https://algorea-static.s3.eu-central-1.amazonaws.com/branch/feature/e2e-fr-version/playwright-report/index.html

[auto-generated message] E2E test report on: https://algorea-static.s3.eu-central-1.amazonaws.com/branch/feature/e2e-fr-version/playwright-report/index.html


releases-index.json: add .NET Framework (4.x)

URL(s)
https://github.com/dotnet/core/blob/main/release-notes/releases-index.json
Description
This is a list of .NET from 1.0 to 3.1 (skipping 4.x) and from 5.0 all the way to 9.0.
Considering .NET Framework 4.x is built-in Windows, I don't see a reason why releases-index.json would not include this specific version, especially when it's still considered supported versions.
I think .NET Framework should be included.

Does Json files have to be given permission or downloaded by the owner? Thx
On Sat, Jul 13, 2024 at 3:53 AM Obegg @.***> wrote:

URL(s)
https://github.com/dotnet/core/blob/main/release-notes/releases-index.json
Description
This is a list of .NET from 1.0 to 3.1 (skipping 4.x) and from 5.0 all the
way to 9.0.
Considering .NET Framework 4.x is built-in Windows, I don't see a reason
why releases-index.json
https://github.com/dotnet/core/blob/main/release-notes/releases-index.json
would not include this specific version, especially when it's still
considered supported versions
https://dotnet.microsoft.com/en-us/download/dotnet-framework.
I think .NET Framework should be included.
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/9408, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BA7FZAROCJABFPAK3HR5WJ3ZMEBLJAVCNFSM6AAAAABK2FSVJ2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQYDMOBUGQ2TAMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@rbhanda could you please help with this issue? Thanks!

@billgothacked -- Check out:
https://github.com/dotnet/core/blob/main/release-notes/license-information.md

This repo is dedicated to .NET/.NET Core. We're not going to host .NET Framework content here.
We have (more limited) release notes for .NET Framework in another repo: https://github.com/microsoft/dotnet/tree/main/releases.
What would you do with such a file?

Do you download KBs for that? If you rely on Windows/Microsoft Update, you will get that automatically.

OK. How would this file help with that?

How about this site?
https://dotnet.microsoft.com/download/dotnet-framework

I shared this issue with the folks on our team that manage .NET Framework.


Finding Integration within HACS?

I'm trying to figure out how to install this in HA. The instructions are quite straightforward:

Method 1. HACS > Integrations > Plus > WebRTC > Install

But, WebRTC isn't available. I've added the repository https://github.com/AlexxIT/hassio-addons to the addon store, but that only adds go2rtc as an available option, not WebRTC.
What have I missed?

I should add, the installation does work by copying the files manually. But it's just not as user friendly for beginner users and it's harder to keep up-to-date!

I have also tried using the altnernative method and the files manually. This does create the WebRTC Camera in the integrations menu, but does not create the WebRTC addon in HA.

Installation did work when copying the files manually (did have to clear my browser cache to get it to work). Would still like to know how to add it via HACS store for better update management (and perhaps this can be clarified in the documentation for beginners like myself).

This is not addon, but integration. HACS is not Home Assistant, but Home Assistant Community Store

Thanks. So stupid, it does indeed show in integrations I was clicking addons.

"browse repositories" has been removed... you have to simple search it, it took me a while until i've found it :)
had to login in incognito too see this msg


Auto-create Discord Channels

Have the bot create necessary channels upon getting added to a server so that admins don't need to fuss with permissions.

Also auto-create webhook and provide admin with the secret key


from which dir does oen have to run your codes?

from src?
from root? but then one has to change the file paths in Compile.jl?

Steps on how to run the code added. Module structure in progress


Origin of URL

https://github.com/google/python-laurel/blob/76c703f10a1857dbe0a4cd2ef59f7be03a38efce/laurel/__init__.py#L26
How did you acquire this authentication URL? Who hosts it?

I used https://mitmproxy.org/ and a phone running Android 6. You set up an access point with the proxy and enroll the fake certificate in the phone, and then all API calls that the app makes are recorded by mitmproxy. You can also disassemble the apk with https://github.com/skylot/jadx and look for stuff there. xlink is a Chinese company who provide an IoT management platform, so I assume that GE contract out the cloud service to them.

I love your approach to mapping capturing the requests. I attempted it but used  wireshark and was inundated with packets. Spent most of my energy getting the filters set.


jQuery 1.6.4

Is there a reason the built-in jQuery is so old? Also, the rails.js is fairly limited. I've dropped in the full version on my fork of the project, and it seems to be quite stable. I wanted to check here to see if there was a specific reason this wasn't done in the first place, before I submitted a pull request.

See #468.

Seems like this is a duplicate, mind if I close this?


Schemas may not be named after primitives

Exception in thread "main" org.apache.avro.AvroTypeException: Schemas may not be named after primitives: string

The FHIR schema defines it's own types for some values that conflict with AVRO's primitives.
We need to sanitize record names.

boolean
int
long
float
double
bytes
string

https://avro.apache.org/docs/1.10.1/spec.html#schema_primitive

There are two parts to this problem.

Sanitize the record name itself.
Also up date any AvroRef so it continues to point to the correct thing.


[731] Feedback from hiring staff is sent to Google Sheets in a daily bulk upload

Trello card URL:
https://trello.com/c/MtL5EUEU/731-feedback-from-hiring-staff-should-be-downloadable-rather-than-sent-to-a-google-sheet
Changes in this PR:
This builds on #757 to add details of feedback responses to a Google Sheet on a daily basis, rather than as soon as they are recieved. This means we hit to Google Sheets API less often, and means we're queueing up fewer jobs
This is in draft at the moment, as #757 needs to be merged first

 Feedback from hiring staff should be downloadable rather than sent to a Google Sheet


Half hour pricing

Thank you for what looks like a good start on an excellent utility. From your three repositories I get the strong impression you are in Ireland. Currently there are pricing by the hour. In England the pricing is by half hour. For example Octopus GO has cheap rate from 0030 to 0430, and the Octopus Agile is by half hour slots.
It would be great if the pricing structure could be enhanced to the half hour

I am indeed. Thanks for the feedback and suggestion!
Which are you using? I've been concentrating on the Android app for a while now. The python AlphaESS integration is broken for example. It's not a very high priority, and there are a lot more capabilities in the Android -- you could think of the python as a proof of concept. Load profile and costs can be loaded from the python config directory to the Android app, they use the same json format.
I do want to make this applicable to more markets... After I get it working the way I want in a small market. To do this well, support from folk in those markets will be needed. I will be working on the costs later in the year.
I need to add limitations to the costs, 'fair usage' types, for cheaper 'EV' rates. I was not aware that the UK used 1/2 hr intervals. That's a bit of a change as it was a pretty early decision to balance usability and flexibility. It's encoded in the DB, the json format, and the algorithms for costing and validation. Not a problem as such, just a lot of little changes. I will look at the two together.
Right now I'm taking a break to get my garden sorted, should be back in a few weeks ago going well.

The android is currently only available in Ireland, so I cannot look at that from the play store (Not sure if the APK is available)
I did look at the code with altering it in mind, but did find it was very heavily embedded into integer hours, plus I have a limited familiarity with json.
My main aim is to simulate Octopus Agile which is a tariff that varies including negative. The prices are published 12 hours in advance in half hour price slots (not verified.) Using historic data I wanted to build on what you have written to see if with a battery (and weather information for solar, later) I could write a clever way to to charge the battery, which could also be used on historic data to calculate optimum battery size. With this in mind python is probably more useful than android so that it can be integrated into things like Home Assistant and ESPhome. However, I totally understand that the android app is very useful for people that do not what to go that far, therefore probably does have more appeal to more people.
Oh yes the garden, that will be taking my time as well.
Thanks for all your hard work

No problem, it's a fun diversion when I don't let it take all my time. I will take a look at this, no promises on timeline...
Two options for the Android app:

Join the alpha test, I need the email address of the email used to access the play store
Download and install android studio, clone the repo, build and install on a phone via adb

I have added minute pricing to the android app (V1.32) -- and pushed to alpha testing.
It should provide a good insight into how the python can also be adapted, if you are still interested.

Thank you
This is very delayed, yes I'm still following and interested in using, other projects became more important, but maybe winter will lead to more computer time


LCOM4.xsl:65-69: LCOM4:...

The puzzle 216-5540d942 from #216 has to be resolved:
https://github.com/yegor256/jpeek/blob/a0588eabe8dae3337baa22e930c7f0e346a722fb/src/main/resources/org/jpeek/metrics/LCOM4.xsl#L65-L69
The puzzle was created by rultor on 07-Jan-19.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The job #295 assigned to @rajanamistry/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@rajanamistry/z this job was assigned to you 10days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

The user @rajanamistry/z resigned from #295, please stop working. Reason for job resignation: It is older than 10 days, see §8

Resigned on delay, see §8: -30 point(s) just awarded to @rajanamistry/z

The job #295 assigned to @thomas-melville/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

Resigned on delay, see §8: -30 point(s) just awarded to @thomas-melville/z

The user @thomas-melville/z resigned from #295, please stop working. Reason for job resignation: It is older than 10 days, see §8

The job #295 assigned to @uurl/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@uurl/z this job was assigned to you 10days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat wait for #343

@0crat wait for #343 (here)

@Protagores The impediment for #295 was registered successfully by @protagores/z

@verkhovin @paulodamaso @yegor256 The notCommonAttributesWithAllArgsConstructor test is already enabled in the org.jpeek.metrics.Lcom4Test class, and it passes successfully. Do you think the puzzle is no longer needed in this case?


90° is better than 270°

In my opinion more degrees mean more hardware consumption
I hope it's better not 270°, but two opposite 90°
I generally use clockwise or counterclockwise to refer to 90° Rotation
CW 90° and CCW 90°
0 and 180°

There is not going to be any performance delta for rotating by 270 degrees and rotating by -90 degrees.


modify changelog format

This PR changes the format in which changelogs are generated. We use the @changesets/changelog-github package, which is a better default than we have now. It includes the PR number and author id. and has links to commit/PR/author.
Example changelog - https://github.com/changesets/changesets/blob/main/packages/cli/CHANGELOG.md


Bogus claimid tests

This adds a test which checks that someone can't just set claimId to whatever they want when creating a claim update (for example, the claimId of the current winner) and get their claim inserted immediately (or ever).

ACK, not sure why 40735d9 is there though, that one should be merged already


Support for Redactor field type

Support adding embedded assets to a Redactor field type

Actually, scratch this idea. You really shouldn't be inserting assets into a rich text editor.


[DOCFIX] Update Configuring-Alluxio-with-S3.md of Chinese version

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14927/
Test PASSed.

Merged build finished. Test PASSed.


setState is not being called after wrapping component into Radium.Enhancer

Either my tests are messy (they worked fine before) or Radium does some crazy stuff here, I'm not sure.
I have a simple flux-react app based on the v0.13 syntax and my tests are now failing at the point where it should call this.setState on _onStoreChange:
class Container extends React.Component {

  static getStateFromStores() {
    return {
      firstScreen: ScreensStore.getState().get('screens').first()
    }
  }

  constructor(props) {
    super(props)
    this._onStoreChange = this._onStoreChange.bind(this)
    this.state = Container.getStateFromStores()
  }

  componentWillMount() {
    ScreensStore.addChangeListener(this._onStoreChange)
  }

  componentWillUnmount() {
    ScreensStore.removeChangeListener(this._onStoreChange)
  }

  _onStoreChange() {
    this.setState(Container.getStateFromStores())
  }

  render() {...}

}

export default Radium.Enhancer(Container)

FAILED TESTS:
  Container.jsx
    _onStoreChange()
      ✖ should call setState when being called
        PhantomJS 1.9.8 (Mac OS X)
      Expected spy setState to have been called.

This worked fine before wrapping it into Radium.Enhancer.
The test:
describe('_onStoreChange()', () => {

    beforeEach(() => {
      Container = require('../Container.jsx')
      ContainerEl = React.createElement(Container)
      renderedRootElement = TestUtils.renderIntoDocument(ContainerEl)
      spyOn(renderedRootElement, 'setState')
    })

    afterEach(() => {
      renderedRootElement.componentWillUnmount()
    })

    it('should call setState when being called', () => {
      expect(renderedRootElement.setState).toBeDefined()
      expect(renderedRootElement.setState.calls.count()).toEqual(0)
      renderedRootElement._onStoreChange()
      expect(renderedRootElement.setState).toHaveBeenCalled()
      expect(renderedRootElement.setState.calls.count()).toEqual(1)
    })

  })

Were you using class syntax before wrapping with Radium.Enhancer? With the new syntax, React no longer autobinds handlers, so this is a common issue.

Nope, I have been using the class-syntax before as well. Would you recommend to try .bind-ing my stuff where needed? Working on some other stuff now, wondering if that could fix my issue?

NVM, looks like you are already calling this._onStoreChange = this._onStoreChange.bind(this). I'll see if I can find a simple repro.

 Thats what my code is based on: https://github.com/mrtnbroder/isomorphic-react-webpack-boilerplate

Might this work? http://stackoverflow.com/questions/16502267/issues-with-jasmines-spyon-tohavebeencalled-on-a-prototypal-method

One more question; is it only the tests failing, or does it also fail when you test it in the browser?

It works with .prototype. so
      spyOn(renderedRootElement, 'setState')

is now
      spyOn(Container.prototype, 'setState')

not sure if that is the correct way to do it though... but it looks like it is?

Thanks for taking a look @ianobermiller, feel free to close this if you want. Or dig in a bit deeper if you want.
It worked in browser btw.

I believe this is expected, yes. Thanks for trying that! The only possible thing Radium could do is (dangerously) modify your component class instead of wrapping it.


pypy, pypy3: 7.3.2 → 7.3.3

Motivation for this change
Upgrade pypy and pypy3 from 7.3.2 to 7.3.3.
Things done

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

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

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

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

2 packages built:

pypy (pypy2 ,pypy27)
pypy3 (pypy36)


[terra-tag] additional features

Enhancement
Adding additional features detailed here https://zpl.io/2Ewmqke

removable version of tag with a close/"X" icon
potentially removing the custom icon/decorated version if not being used by any group
adding a second visual style for use over alternate backgrounds (photos, darker solid colors, etc.)
add additional interaction styles for "pressed" state (new)

I had a couple of questions regarding the implementation of this:

How should the implementation of the Close/"X" icon be :

Similar to the close icon of Action Header ?, i.e. a button : https://github.com/cerner/terra-core/blob/master/packages/terra-action-header/src/ActionHeader.jsx#L86-L102
Similar to the tag of multi select of terra-form-select ?, i.e. a span : https://github.com/cerner/terra-core/blob/master/packages/terra-form-select/src/shared/_Tag.jsx

I have some confusion w.r.t. to the hyperlink/navigation part as per the https://zpl.io/2Ewmqke:
The basic construction indicates that the Tag Label is hyperlink, whereas the Interaction styles indicates that the Tag Container can be pressed.
So should the navigation occur on the Tag click (similar to a button) "or" the Tag Label click (similar to a hyperlink) ?
Also i agree with the removal of the custom icon, since after a quick search in our internal github : here. It is clear that the custom icon is not being used.
So should we go ahead and remove that feature?
I quite did not understand the following requirement:
"adding a second visual style for use over alternate backgrounds (photos, darker solid colors, etc.)"
For what scenario is the dark style required?
If we do go ahead with the dark style, how do we want to implement it? (i.e. Taking input in the form of a prop)?
Also does the wireframe need to be revisited, since it was last updated in 2018?


Docker: a new apollo_dev image include libtorch-gpu rebuilt to fix the perception core dump

The final image was 16.9G, which was too big. Try rebuild it from our cyber_image rather than previous dev.image.

I have rebuilt the docker image, and compressed the size to 13.3G.


map drift a lot,loop not found

Hi koide3 ,
I tested hdl graph slam by robosense 16 lidar, but i found that there is something wrong with the map.
--- loop detection ---
num_candidates: 8
matching........ done
best_score: 36.815    time: 0.000[sec]
loop not found...

tf between map and odom,
At time<PHONE_NUMBER>.112
- Translation: [-5.383, -2.814, 16.107]
- Rotation: in Quaternion [-0.001, -0.031, -0.034, 0.999]
            in RPY (radian) [-0.001, -0.062, -0.068]
            in RPY (degree) [-0.034, -3.564, -3.881]

Hi @liuyf5231 ,
NDT-based methods may not work well with 16-line LIDARs because it is not very robust to sparse point clouds. Could you try to use GICP or GICP_OMP instead of NDT_OMP (by changing parameters in the launch file)? If you are using ROS melodic, GICP should work well (PCL on Kinetic or earlier has a bug). You can also try to slow down the rosbag playback speed:
bash rosbag play --clock -r 0.2 bagfilename.bag

Hi @liuyf5231 ,
NDT-based methods may not work well with 16-line LIDARs because it is not very robust to sparse point clouds. Could you try to use GICP or GICP_OMP instead of NDT_OMP (by changing parameters in the launch file)? If you are using ROS melodic, GICP should work well (PCL on Kinetic or earlier has a bug). You can also try to slow down the rosbag playback speed:
bash rosbag play --clock -r 0.2 bagfilename.bag

ok, I will try.

In that case, maybe you can use NDT_OMP since the target point cloud (map cloud) is dense.


Only 1 minion can be ping(ed) sometimes

After vagrant up, I ssh to the salt master and run: salt "*" test.ping, then I got this:
root@saltmaster:/srv/salt# salt "*" test.ping
minion1:
    True
minion2:
    Minion did not return. [No response]

I've tried this 20 times (restart, vagrant up, ssh to master, run test.ping), 2 minions return ping True only 2 times.
After vagrant up, my computer still have 3GB free of RAM, 50GB HDD free.

I'm having trouble reproducing this with the latest changes from the repo.
Can you try running this again after verifying you're using the very latest code from this repo? I just ran a test.ping 20 times and I got responses from both minions every time. We did make a couple changes yesterday that did improve the responsiveness of the salt master.

I cannot re-produce this issues today.
Thank you! 😃

I had the same issue too. Either minion1 or minion2 randomly failed to return.
It seems to be related to "ubuntu/trusty64" because after I changed xxx.vm.box to "bento/centos-7.2" in Vagrantfile, it just works smoothly.


implement  PointerEvent

Feature type
Others
The spec link.
https://w3c.github.io/pointerevents/
How much important for you
#1450
implementation  PointerEvent and use the pointerType to distinguish between mouse and gesture

https://github.com/openwebf/webf/issues/89 该pr合入此问题不复存在