Return otpServers with /project endpoint to allow speedup

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
Recent front end PRs (https://github.com/ibi-group/datatools-ui/pull/895) have tried to reduce the amount of data that is unnecessarily requested by the front end. This endpoint created a conflict bc the /project and project:id differed in their return structure (specifically in returning otpServers).
Per the comment in the backend, I'm worried about performance issues with this change:

TODO: perhaps the ProjectWithOtpServers should not be a mixin anymore?

Let me do some testing on a large project deployment and see. The TODO is separate from the performance issue, that is more just for if this fix is permanent, then it doesn't really make sense to keep the field as a mixin.

Yeah this seems to slow things down significantly for a big project. I'll do some work to maybe reduce the amount of data we need to request.


Added arpTableSolver

Branch with arpTableSolver, for use in ZeroConf funcionality
Dependencies:
"@network-utils/arp-lookup": "^1.0.4"
"net-ping": "^1.2.3"

Hey @luisllamasbinaburo
Here are some numbers according to this page, https://bundlephobia.com/:

package
MINIFIED
MINIFIED + GZIPPED

net-ping
10.7kB
3.2kB

@network-utils/arp-lookup
1.5kB
761B

ewelink-api
839kB
228.2kB

Here are more info:
https://bundlephobia.com/scan-results?packages=@network-utils/arp-lookup@1.0.4,net-ping@1.2.3,delay@4.3.0,nonce@1.0.4,random@2.1.1,request@2.88.0,request-promise@4.2.4,websocket<EMAIL_ADDRESS>https://bundlephobia.com/scan-results?packages=delay@4.3.0,nonce@1.0.4,random@2.1.1,request@2.88.0,request-promise@4.2.4,websocket<EMAIL_ADDRESS>So, looks like adding these 2 packages have a insignificant overhead.

I have a few questions here:

how do you initialize this class? something like this?

const arpTableSolver = require('./lib/arpTableSolver.js');
(async () => {
  const arp = new arpTableSolver();
  await arp.Init();
]})();

GetIp is not called on the class. what is the purpose of this method?

tried await arp.getTable() manually but always get an empty array

Hello,

I use this way

(async () => {
  const ipres = new IpResolver();
  await ipres.Init();
  await ipres.SaveArpTable('arptable.json');
})();

and once I have the 'arptable.json', at anytime
(async () => {
  let arpTable = await ipres.loadArpTable('arptable.json');
})();

But it is really strange, because this is basically the same that your did, and you are getting any empty array. I don't understand. Which OS are you using? (I tried in Win10)
On the other hand, GetIP(mac) method is provided to be used on the Ewelink connection for ZeronConf. But I didn't include in this branch, because I don't want to 'mess' your code.
I try to keep them independent, and leave this branch only for showing the arpTable
(that it seems, doesn't work for you, but i don't know why)

Found the problem:
Looks like @network-utils/arp-lookup doesn't work on macos.
The parser method doesn't understand the response from arp -a (which is different from windows)
Also, tested on linux with same result.
I'm looking for another npm package to get mac/ips

Ok, I'm going to test it

Hello @luisllamasbinaburo
I've made some changes to the class using the new library.
I was able to export a json file with the ip/mac associations.
Let me know if it works for you so we can merge and continue with the other PR.
Thanks.-

Yes, it works perfectly. (Sorry for the late reply)
Let me know if i can do anything more
Thanks!


Recognize from field

It would be nice to allow and format a from: field.
I use it to nicely show mails received when I want to keep them in my notes.

Hey @matclab,
Thank you for your suggestion. So, do you want this field from just for "Documenting" your email?
The mailto API does not support a from field (See here), and that means that this field can only be used for the preview, but not for the Mailto button. Is this your intention?

Yes, totally, just for documenting.

@matclab have fun ;)

Thank you very much @joleaf :clap:


build: change githooks for pre-commit

change hook system for a more customizable one
Behavior change, now changes detected by the hooks are applied if they can.
Hooks configured in this PR:

end-of-file-fixer, ensures a file ends with a newline (or is empty)
trailing-whitespace, removes any trailing whitespace
isort, applies isort on python imports
black, applies black formatter
commitizen, check the commit message respects conventional commits

Run poetry install to get new dependencies.
Run this to get new hooks.
poetry run pre-commit install
poetry run pre-commit install --hook-type commit-msg

Maybe add a badge to the README https://pre-commit.com/#badging-your-repository ?

It is possible to add the licence headers checker too ?

Maybe add a badge to the README https://pre-commit.com/#badging-your-repository ?

Badge added

It is possible to add the licence headers checker too ?

I tested the two license checkers I found. They ended up checking files in .git folder, and did not have options to exclude paths...

rebase on master


Make prevent teleport on headset collision optional

It may not always be desired to block teleport on headset collision
in fact it may be better off letting this be done by a separate script that hooks the two events up.

I agree.
If the headset collision with some ennemy during gameplay, the player should be able to teleport away. Otherwise any collision that occurs with game objects would block the player teleport movement.

There's a PR for this if you wanna check it
https://github.com/thestonefox/SteamVR_Unity_Toolkit/pull/515


add env.go file with ConsolidateEnvVars function

This function looks for discrepancies between the environment variables
and the fields they pertain to in Arken's main config struct. The environment
variables are chosen over the struct values (which are read from
~/.arken/arken.config at startup) if present.
This function also decides how Arken's environment variables are named:
ARKEN_<CATEGORY>_<DESCRIPTOR>
where category is the name of the substruct of struct config.go::config in
which the field was found, and the descriptor is the name of the field.
This commit also changes config.go by adding a call to the new function.

Looks good to me. Thanks for the addition @DaniloHP.


Change typing of logLevel argument to logCreator

The kafkajs log levels are numeric at runtime, but the typescript declarations for the logCreator function takes a string argument. This PR changes the argument's type and adds a test for it.

Thanks for the contribution! Just a question about whether or not we can restrict the type to the appropriate enum.

This is available in the pre-release version 1.12.0-beta.3.


Create gatsby-clay

Hi @TomasBankauskas - I realized that my theme may not be displaying because I forgot to add .md to the file name. What is the quickest way for me to fix this issue? Cheers

Hi @lilxyzz, thank you for the theme submission. I added a .md ending to your file, and you can see your theme here. One issue remains, your demo is not accessible in the iframe Refused to display 'https://clay-theme.netlify.app/' in a frame because it set 'X-Frame-Options' to 'deny'.

Thank you @TomasBankauskas, really appreciate it. It looks like Netlify is adding 'X-Frame-Options'  by default now. I will look into disabling it when I get a moment. Cheers.

Np @lilxyzz. Regarding X-Frame-Options, maybe this documentation will be useful for you -> https://docs.netlify.com/routing/headers/#syntax-for-the-headers-file


All have been fixed upstream! :)


Screenshot prevention has stopped working with iOS 17

I understand that the iOS screenshot prevention is unofficial, and is based on a bit of a hack.
It looks like this hack has stopped working as of iOS 17.

not working on ios 17

Running into the same problem now. Interested to see how we can get it working for iOS17 again.

+1

This could offer a possible solution: https://gist.github.com/ashishkakkad8/747223d68f3675dce4f05e4396baec7e

+1

HI any update on it

I've made this PR that addresses this issues. It works for me by patching this package, but I wanted to contribute with a PR.
https://github.com/killserver/react-native-screenshot-prevent/pull/31

send update in version 1.1.9. check pls and thx all )

After taking screenshot, warning pop-up is not closing & when we come to other screen it takes screenshot                                     ### here is my code                                                                                import ScreenshotPrevent, { addListener } from 'react-native-screenshot-prevent';
componentDidMount(){
// Prevent screenshots when the component is mounted
ScreenshotPrevent.enabled(true);
ScreenshotPrevent.enableSecureView();
// Add a listener to handle screenshot prevention events
this.subscription = addListener(() => {
});
}                                                                                componentWillUnmount() {
this._engine?.destroy();
ScreenshotPrevent.enabled(false);
ScreenshotPrevent.disableSecureView();
this.subscription.remove();
}

Hi! Thanks for this useful package. 👍🏼
So I have tried version v1.1.9 on iPhone X running iOS 17.2.1, but Im still able to take screenshot.
The screenshot:

Code (App.tsx):
!__DEV__ && PreventScreenshot.enableSecureView()

package.json:
"react": "17.0.2",
"react-native": "0.65.1",
"react-native-screenshot-prevent": "^1.1.9",

Any kind of help would be appreciated, thanks.

I've put up a PR that should fix this issue, the only thing is that since we didn't need the image functionality in my workplace, I didn't have time to test and keep support for it, but the main idea is to add the textfield secure layer to the app's UIWindow, and from there it should work, and we should be able to add the image as a background to the textfield so it works

Thanks @GianniCarlo !
I tried your PR and the enable/disable screenshot works as expected (tested on iOS 17.5 simulator, android not yet).


Some Core channel arguments are insufficiently documented

Among the channel args documented here I have had the following issues with understanding their functionality, specifically when trying to implement them in @grpc/grpc-js:

GRPC_COMPRESSION_CHANNEL_DEFAULT_LEVEL: What is the practical effect of setting this argument?
GRPC_ARG_MAX_CONNECTION_AGE_GRACE_MS: Is there a difference between setting this and making GRPC_ARG_MAX_CONNECTION_AGE_MS longer?
GRPC_ARG_ENABLE_DEADLINE_CHECKS: Is this an option for the client, the server, or both? What functionality exactly is "deadline checking"?
GRPC_ARG_MIN_RECONNECT_BACKOFF_MS: How does this interact with GRPC_ARG_INITIAL_RECONNECT_BACKOFF_MS?

In addition, many of the options are missing information about the default value if the argument is not explicitly set.

I think this issue should be reopen :-)

I'll subscribe to updates :-)


Update slogdet guidance for when the determinant is zero

This PR

updates the guidance in linalg.slogdet for when the determinant is zero. The original language stemmed from NumPy; however, the guidance there is more conceptual than grounded in practice due to errors in numerical precision (see https://github.com/numpy/numpy/issues/20114). Accordingly, this PR loosens restrictions around return values when the determinant is 0. Namely, the result should be sign 0 and logabsdet -inf, but may not be due to numerical precision errors.

Thanks @leofang! 😅


directly compare stderr to empty string in testRedirectionOfStreams to obtain more meaningful messages if test fails

While installing Tika 1.16, I was running into this cryptic test failure:
java.lang.AssertionError: expected:<0> but was:<42>

At first, I was a bit confused, since 42 is clearly a superior result compared to 0... [1]
After doing some digging, it turns out that I had $_JAVA_OPTIONS defined, which results in a message like this to stderr every time java is called:
Picked up _JAVA_OPTIONS: ...

I had to change the failing test to directly compare the stderr output with the empty string rather than checking the length of the strings to figure out what was wrong, so I propose to change this.
[1] https://en.wikipedia.org/wiki/42_(number)#The_Hitchhiker.27s_Guide_to_the_Galaxy

Sorry about that, and thank you for figuring out what went wrong.  I completely agree on the superiority of 42.  Does your PR actually fix your problem, though? If you're getting Picked up... on stderr...that doesn't equal "".

It doesn't fix my problem, but it makes the cause of the failing test very clear (as opposed to 0 != 42 as output from the failing test).
If not $_JAVA_OPTIONS is not defined, the Picked up ... messages is not printed to stderr, and the test passes.
So, maybe you want to undefine $_JAVA_OPTIONS in the test to avoid that the test result is determined by the environment?

Ha, ok.  It is uglier than I'd like to undefine $_JAVA_OPTIONS in the child process.  I modified the test to allow "" or a string starting with "Picked up"...a kluge, I admit, but...let me know if this fixes your problem.


Parsing of lists with content blocks

Description
foobar

- a

  - b
  - c

foobar

- a #[

  - b
  - b
]

foobar

- a
#[
  - b
  - b
]

Shouldn't all three examples render the same way? The third one doesn't.
This one is even stranger:
foobar

- a

  - b
  - c

foobar

- a
#[

  - b
  - b
]

Reproduction URL
No response
Operating system
Web app
Typst version

[X] I am using the latest version of Typst

The content block needs to be at the same level if you want it to be part of the same list
- a
  #[
    - b
    - b
  ]

(You can also forego the syntax sugar)

I don't find it intuitive because AFAICU it's the same whitespace in all three cases, but ultimately it's a decision about syntax, ok.
Now, if the reason is that content block must be at the same level to be part of the same list, why the numbering is consecutive here:
+ a
#[
  + b
  + c
]

=>

1. a
2. b
3. c

If you compare the two following cases:
+ a #[
  + b
  + c
]

+ a
#[
  + b
  + c
]

the difference is not whether the block is part of the same list or not (it is in both cases, and in some sense it is even more in the second example), but whether the block creates a sublist or not.
So maybe the rule is: the block must be in the same line of an item to create a sublist of that item?

Yes, that's what I meant. Edited my comment

Great, I find that version more intuitive, it also helps understand that indentation in not actually needed inside the block:
+ a #[
+ b
+ c
]


Update cats-effect, cats-effect-laws to 3.2.5

Updates

org.typelevel:cats-effect
org.typelevel:cats-effect-laws

from 3.2.4 to 3.2.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

labels: test-library-update, semver-patch

Superseded by #229.


bump version to 2.18.3

Wanted to get OAS3 validateSwagger in the release version. Please let me know if the release version needs to be bumped to a different version.

what else is required to get attention in this PR and merge?

I released the new version with OAS3 support.
https://github.com/int128/gradle-swagger-generator-plugin/releases/tag/2.19.0


build

Sorry, I use the wrong repo

I'm fine adding this but you'd want to build all targets. This requires msbuild and setting the variable ProjectLoadStyle=All. Check out the Readme for details

@twsouthwick I need your help. I find the DocumentFormat.OpenXml.Packaging.Tests.csproj and IsolatedStorageExceptionWorkaround.csproj and so on can not compile in NET35
"D:\a\Open-XML-SDK\Open-XML-SDK\Open-XML-SDK.sln" (default target) (1) ->
"D:\a\Open-XML-SDK\Open-XML-SDK\samples\IsolatedStorageExceptionWorkaround\IsolatedStorageExceptionWorkaround.csproj" (default target) (8) ->
"D:\a\Open-XML-SDK\Open-XML-SDK\samples\IsolatedStorageExceptionWorkaround\IsolatedStorageExceptionWorkaround.csproj" (Build target) (8:2) ->
  Program.cs(80,43): error CS0012: The type 'Package' is defined in an assembly that is not referenced. You must add a reference to assembly 'WindowsBase, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. [D:\a\Open-XML-SDK\Open-XML-SDK\samples\IsolatedStorageExceptionWorkaround\IsolatedStorageExceptionWorkaround.csproj]
  Program.cs(95,43): error CS0012: The type 'Package' is defined in an assembly that is not referenced. You must add a reference to assembly 'WindowsBase, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. [D:\a\Open-XML-SDK\Open-XML-SDK\samples\IsolatedStorageExceptionWorkaround\IsolatedStorageExceptionWorkaround.csproj]

@twsouthwick The dotnet framework and the dotnet core 1.1 will only build the main project, because they can not compile the IsolatedStorageExceptionWorkaround.csproj and so on.
The test will only run in dotnet core 2.1
What do you think?

All the task can run parallel.

See: https://github.com/dotnet-campus/Open-XML-SDK/pull/1
https://github.com/dotnet-campus/Open-XML-SDK/pull/1/checks?check_run_id=692572748

Why I do not use msbuild /m /p:configuration="release" /p:ProjectLoadStyle=All to compile?
Because it can not run successfully.
See https://github.com/dotnet-campus/Open-XML-SDK/runs/688931364?check_suite_focus=true

Looks like there are some pre-reqs that are not installed. I'll take a look later and see what's missing

I've pushed a change to allow building everything with dotnet rather than msbuild. Take a look at the status checks now - they may provide enough information that this isn't needed.

The results now show helpful info via devops:

@twsouthwick Thank you.  Waiting for your good news

With the changes I mentioned above, I'm not sure this is still needed. Thoughts? I'd rather not have duplicated checks.

@twsouthwick Do you means that the developer can find the Azure DevOps when build error?
But I do not have access.

No, you won't be able to see that, but you should be able to see the checks
for instance at https://github.com/OfficeDev/Open-XML-SDK/pull/727/checks

@twsouthwick Thank you. I know what you mean.

@twsouthwick I want to close this PR, because we can use the Azure DevOps


Workaround isnan across different environments.

Closes  #4882.  Haven't verify the fix yet.  Just trying to see general test results.

@melonki Could you please help verifying this PR fix the error you encountered?

I built it on Ubuntu 19.04 with R with cuda 9.2.

@melonki Could you please help verifying this PR fix the error you encountered? You can copy this PR into a local branch by:
git checkout -b trivialfis-fix-nan master
git pull https://github.com/trivialfis/xgboost.git fix-nan

Thank you for your solution but the errors still exist. I am not sure whether I used the PR in the right way. The details are in the attach file.
ErrorLog2.txt

@melonki Could you try clearing out your build directory (maybe created by msvc, I don't know)?  From your error log, there are lots of unrelated errors like:
“stdafx.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package\xgboost-r.
“hdfs.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package\xgboost-r.v
“curl/curl.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package\xgboost-r.
“gtest/gtest.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package

and many others.  The above listed missing header by default are not used in XGBoost, unless you explicitly asked by providing corresponding CMake parameters.

@melonki Could you try clearing out your build directory (maybe created by msvc, I don't know)? From your error log, there are lots of unrelated errors like:
“stdafx.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package\xgboost-r.
“hdfs.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package\xgboost-r.v
“curl/curl.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package\xgboost-r.
“gtest/gtest.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package

and many others. The above listed missing header by default are not used in XGBoost, unless you explicitly asked by providing corresponding CMake parameters.

okay, I'll try it again.

@melonki Could you try clearing out your build directory (maybe created by msvc, I don't know)? From your error log, there are lots of unrelated errors like:
“stdafx.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package\xgboost-r.
“hdfs.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package\xgboost-r.v
“curl/curl.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package\xgboost-r.
“gtest/gtest.h”: No such file or directory [C:\Users\B621\xgboost\build\R-package

and many others. The above listed missing header by default are not used in XGBoost, unless you explicitly asked by providing corresponding CMake parameters.

I uninstalled xgboost package in R studio and delete xgboost folder in my directory and installed it all over again, but the errors still exist.
Here are the codes I typed in CMD:
git clone https://github.com/dmlc/xgboost cd xgboost git submodule init git submodule update git checkout -b trivialfis-fix-nan master git pull https://github.com/trivialfis/xgboost.git fix-nan cd R-package "C:/Program Files/R/R-3.6.1/bin/x64/R.exe" CMD INSTALL . cd C:\Users\B621\xgboost mkdir build cd build cmake .. -G"Visual Studio 15 2017 Win64" -DUSE_CUDA=ON -DR_LIB=ON -DLIBR_EXECUTABLE="C:/Program Files/R/R-3.6.1/bin/x64/R.exe" cmake --build . --target install --config Release
The details are in the attach file.
ErrorLog3.txt

@melonki Please take a look at : https://xgboost.readthedocs.io/en/latest/build.html#installing-r-package-with-gpu-support
I don't think you need to do:
cd R-package
... CMD INSTALL

Using cmake install should be enough.

@melonki Great!


ci: Update fmf plan to add a separate job to prepare managed nodes

Add a preparation job to run on managed nodes
Update instructions to run tests locally

Signed-off-by: Sergei Petrosian<EMAIL_ADDRESS>

[citest]

@tomjelinek Why are tests failing with this? Is there an additional setup task for tests?
TASK [fedora.linux_system_roles.ha_cluster : Fail if passwords are not specified] ***
task path: /tmp/tmp.ftwKGHCLYp/ansible_collections/fedora/linux_system_roles/roles/ha_cluster/tasks/shell_pcs/check-and-prepare-role-variables.yml:43
Friday 26 July 2024  13:33:10 -0400 (0:00:00.028)       0:00:05.834 ***********
failed: [managed_node1] (item=ha_cluster_hacluster_password) => {
    "ansible_loop_var": "item",
    "changed": false,
    "item": "ha_cluster_hacluster_password"
}

MSG:

ha_cluster_hacluster_password must be specified

@tomjelinek Why are tests failing with this? Is there an additional setup task for tests?
TASK [fedora.linux_system_roles.ha_cluster : Fail if passwords are not specified] ***
task path: /tmp/tmp.ftwKGHCLYp/ansible_collections/fedora/linux_system_roles/roles/ha_cluster/tasks/shell_pcs/check-and-prepare-role-variables.yml:43
Friday 26 July 2024  13:33:10 -0400 (0:00:00.028)       0:00:05.834 ***********
failed: [managed_node1] (item=ha_cluster_hacluster_password) => {
    "ansible_loop_var": "item",
    "changed": false,
    "item": "ha_cluster_hacluster_password"
}

MSG:

ha_cluster_hacluster_password must be specified

Which test is this?

@spetrosi ah - the vault support has not been added yet

@spetrosi test script will need to implement something like https://github.com/linux-system-roles/tox-lsr/blob/main/src/tox_lsr/test_scripts/runqemu.py#L886

[citest]

Ansible 2.17 shows that password_hash filter is going to be deprecated since Ansible 2.19.
I noticed this because passlib, a requirement for password_hash, is not pulled when installing ansible-2.17 with pip.
Apparently, passlib is not being supported, three years without commits.
More info in https://github.com/ansible/ansible/issues/81949 and in https://github.com/ansible/ansible/issues/82652.
We can use some cli like mkpasswd instead.

[citest]


Test XML file

In PHPCS 2.7.1 the XML files are not being tested any more.

XML rulesets were never intentionally tested. It was an accident that they were parsed. But because they were, it meant that tests were not running in isolation (they were running with all the XML ruleset rules and exclusions applied) so they were not accurate.

This mean that if there is syntax error in the XML file or a ruleset is misspelled there will be no error. This simple line of code will help test the XML for this issue and return an error if there are any issues.

What are you actually trying to achieve here?

Sorry, I was testing an idea. Meant to make a PR to my own repo first. As a PR is already created, I have cleaned it up and add a proper description.

First off, the rulesets do not need to be tested against every combi in the matrix, once is enough. if [[ "$SNIFF" == "1" ]]

Makes sense. I have changed it.

Second, the WPCS sniff code itself is run against it's own standard in the command just below, so that will test the rulesets too - it runs against Extra (incl Core) + Docs. The only one which isn't tested that way is VIP.

It is sort of easier to run it on the whole WordPress ruleset. That also means any new rulesets that are added like WordPress-Theme are automatically tested too.
I had to create a dummy file that would not generate any issues.

Closing as fixed by https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/pull/914


Firefox x86 update package don't work with windows x86 2004 and above

tried to generate firefox update with exe and msi. on windows x86 2004 and above it don't work. with windows x86 1909 there are no problems. No Problems at all with windows x64 1909 and above
Windwos error code : 0x80240017
Event ID : 20
Windows 10 pro
WPP Release.v1.4.2103.13
WPP Release.v1.4.2008.16
WPP Release.v1.4.1906.3
thanks for your great work 👍

Hi esbiko13, this error code maps to: Operation was not performed because there are no applicable updates.
How do you make the update? What applicability rules do you declare?

hey, my traditional way is that i start a custom update to create an install.ini one time. than i create or recreate the update package by extracting the exe file downloaded from mozilla, take the setup.exe and the "core" folder and check the existing file version of installed firefox.exe for installing the new one. i also check the os achitecture (x86 x64) with WMI Query and the presents of the .ini file.
It works fine until windows x86 1909. after upgrading that test vm to 2004 the same update package don't work anymore.
with my x64 windows systems i have no trouble.
facing this issue today, i also tried to build up the package by the msi version from mozilla and only check the installed product code. same issue x86 don't work x64 works fine. :(

yes, manual install works


Collections index app

Sets up adminOnly route for /collections index page.
TODO: test the index route.

@yuki24 @l2succes ok to merge?

@l2succes the meta is included in the reaction component, what further steps are necessary for the Head?


Upgrade support

Description
Changes proposed in this pull request:

parallel-installation lib supports upgrades

Related issue(s)

/meow


add plugin to intercept markdown links

Fix #209

Verified that this does fix the bad scroll behavior described by Dan (for me locally).


ESP32 Wrover Device Crashing When Using ESP-MQTT (IDFGH-3252)

Environment

Development Kit:      ESP32-Wrover-Kit
Kit version (for WroverKit/PicoKit/DevKitC): v4
Module or chip used:  ESP32-WROOM-32
IDF version : 4
Build System:  CMake
Operating System:     Linux
Using an IDE?: PlatformIO on Visual Studio Code
Power Supply: USB

I am using Arduino as an ESP-IDF Component
Problem Description
The ESP32 device crashes when it runs esp_mqtt_client_init() function
Code to reproduce this issue
#include <stdio.h>
#include <freertos/FreeRTOS.h>
#include <freertos/task.h>
#include "sdkconfig.h"
#include "mqtt_client.h"
#include <Arduino.h>
#include <WiFi.h>
#include "esp_log.h"
#include "esp_event.h"
#include "sdkconfig.h"

const char *ssid = "1233";
const char *password = ",,,";

void setup()
{
    Serial.begin(115200);
    //WiFi Login
    WiFi.begin(ssid, password);

    while (WiFi.status() != WL_CONNECTED)
    {
        delay(1000);
        Serial.println("Establishing connection to WiFi..");
    }

    Serial.println("Connected to network");

    // MQTT Setup
    Serial.println("Connecting to MQTT Broker");
    esp_mqtt_client_config_t mqtt_config;
    mqtt_config.uri = "mqtt://123.com";
    mqtt_config.username = "abc";
    mqtt_config.password = "123";
    mqtt_config.client_id = "esp32";
    Serial.println("Initializing Client");
    esp_mqtt_client_handle_t client = esp_mqtt_client_init(&mqtt_config);
    Serial.println("Registering Callback");
    Serial.println("Connecting");
    esp_mqtt_client_start(client);
}

void loop()
{
}

Debug Logs
␀␀Establishing connection to WiFi..
␛[0;32mI (2166) tcpip_adapter: sta ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>␛[0m
Establishing connection to WiFi..
Connected to network
Connecting to MQTT Broker
Initializing Client
Guru Meditation Error: Core  1 panic'ed (LoadProhibited). Exception was unhandled.

Other items if possible
https://gist.github.com/mntolia/2cf2576b3c9ae3c6bd82b189c82dcd54

Hi @mntolia
Most of the items in the configuration struct are mutually exclusive, others must be set to zero if the feature is disabled. In other words, the init struct must be all zero's except for just few items one configures and this construct does not assure zero init:
esp_mqtt_client_config_t mqtt_config;

You can either memset() to zero or initialize using the C99 designated initializers (they kindof work from C++ as well if the fields are ordered and child objects are somehow enclosed)
Can you please update this line as suggested below and check if this fixes your issue?
esp_mqtt_client_config_t mqtt_config = { 0 };

Hi @david-cermak ,
Thanks a lot. I got the device connected. But I am facing another issue. When I run the esp_mqtt_client_register_event() function according to this example:
https://github.com/espressif/esp-idf/blob/ce2a99dc23533f40369e4ab0d17ffd40f4b0dd72/examples/protocols/mqtt/ssl/main/app_main.c
I get this error:
argument of type "int" is incompatible with parameter of type "esp_mqtt_event_id_t"
Additionally in the event handler function:
static void mqtt_event_handler(void *handler_args, esp_event_base_t base, int32_t event_id, void *event_data) {
    ESP_LOGD(TAG, "Event dispatched from event loop base=%s, event_id=%d", base, event_id);
        mqtt_event_handler_cb(event_data);

}

I get this error:
invalid conversion from 'void*' to 'esp_mqtt_event_handle_t' {aka 'esp_mqtt_event_t*'} [-fpermissive]

If you could point out where I am wrong it would be really appreciated :)

@mntolia  This is an issue with C++ build, has been fixed in https://github.com/espressif/esp-mqtt/commit/8a1e1a5a9ffce89313aaa209375e1def893a4c73, but apparently that hasn't made it yet to arduino. As a workaround you can cast the variables to the expected types or even avoid using an event loop altogether.
It is still possible to configure mqtt_event_handler_cb() as a plain callback in client config:
    mqtt_config.event_handle = mqtt_event_handler_cb;

And remove both definition of the mqtt_event_handler() and registration of the handler esp_mqtt_client_register_event()

Thanks David. I will close this ticket.


dev docker-compose up crashes

Describe the bug
docker-compose up -d crashes:
ERROR: Encountered errors while bringing up the project
ERROR: for shop  Cannot start service shop: Ports are not available: listen tcp <IP_ADDRESS>:80: bind: An attempt was made to access a socket in a way forbidden by its access permissions

To Reproduce
Steps to reproduce the behavior:

create dev docker-compose.yml
docker-compose up -d

Expected behavior
A dev version of shopware
Desktop (please complete the following information):

OS: Windows 10 Pro 19041.867
Browser Chrome 89.0.4389.90

You have already a service running on port 80.
maybe a another docker container or a local xamp/lamp/apache/nginx ...

Hi
I thought it might not be so bad to have that in the docks on the website.
here it is for you: https://docs.dockware.io/faq/error-port-not-available

awesome
thx for letting us know
i will now close this ticket :)
if u ever need anything else, just let us know


Show values to lbs

I usually use pounds(lbs) but the output is only in kg. Could you give a choice in the lovelace configuration to display lbs?

I usually use pounds(lbs) but the output is only in kg. Could you give a choice in the lovelace configuration to display lbs?

You only want the unit of measurement or the conversion ?

The conversion! I can do it by creating a bunch of template sensors, but it's not a clean solution.

La conversion! Je peux le faire en créant un tas de capteurs modèles, mais ce n'est pas une solution propre.

I found the solution. Question in French the units are never in the plural. Example 50 kilograms = 50 kg and not 50 kgs. For you I must put 50 lb or 50 lbs?

https://github.com/dckiller51/lovelace-body-miscale-card/releases/tag/v5.6.0

La conversion! Je peux le faire en créant un tas de capteurs modèles, mais ce n'est pas une solution propre.

I found the solution. Question in French the units are never in the plural. Example 50 kilograms = 50 kg and not 50 kgs. For you I must put 50 lb or 50 lbs?

lbs always

Wow! thanks a million!

@dckiller51  One minor detail. when graphing, lbs are graphed against kg baseline:

@dckiller51 One minor detail. when graphing, lbs are graphed against kg baseline:

The values have been changed. Attention these data are related to my score. You have to adapt your data with your mitfit application.
https://github.com/dckiller51/lovelace-body-miscale-card#options


MSAA causes much higher performance hit with D9VK

The performance hit for activating MSAA in pretty much any game appears to be much higher with D9VK compared to native D3D9. The testing was done on Windows, but the same performance hit is observable on Linux as well.

A couple games with MSAA off vs. on
Call of Duty 4 - No AA:

Resolution
D3D9
D9VK

2560x1080
91 fps (16% GPU)
91 fps (21% GPU)

5120x2160
91 fps (52% GPU)
91 fps (63% GPU)

Call of Duty 4 - 4xAA:

Resolution
D3D9
D9VK

2560x1080
91 fps (20% GPU)
91 fps (34% GPU)

5120x2160
91 fps (67% GPU)
87 fps (99% GPU)

Half-Life 2 - No AA:

Resolution
D3D9
D9VK

2560x1080
285 fps (26% GPU)
285 fps (28% GPU)

5120x2160
285 fps (43% GPU)
285 fps (55% GPU)

Half-Life 2 - 8xMSAA:

Resolution
D3D9
D9VK

2560x1080
285 fps (34% GPU)
285 fps (90% GPU)

5120x2160
285 fps (90% GPU)
103 fps (99% GPU)

Need for Speed: Most Wanted (2005) - No AA:

Resolution
D3D9
D9VK

2560x1080
87 fps (26% GPU)
90 fps (25% GPU)

5120x2160
87 fps (53% GPU)
90 fps (56% GPU)

Need for Speed: Most Wanted (2005) - Max AA:

Resolution
D3D9
D9VK

2560x1080
87 fps (36% GPU)
90 fps (55% GPU)

5120x2160
87 fps (92% GPU)
53 fps (99% GPU)

Need for Speed: Underground 2 - No AA:

Resolution
D3D9
D9VK

2560x1080
120 fps (30% GPU)
120 fps (28% GPU)

5120x2160
120 fps (75% GPU)
120 fps (75% GPU)

Need for Speed: Underground 2 - Max AA:

Resolution
D3D9
D9VK

2560x1080
120 fps (50% GPU)
120 fps (69% GPU)

5120x2160
85 fps (99% GPU)
54 fps (99% GPU)

Vindictus - No AA:

Resolution
D3D9
D9VK

2560x1080
60 fps (35% GPU)
60 fps (33% GPU)

5120x2160
60 fps (70% GPU)
60 fps (75% GPU)

Vindictus - 4xMSAA:

Resolution
D3D9
D9VK

2560x1080
60 fps (42% GPU)
60 fps (51% GPU)

5120x2160
56 fps (99% GPU)
37 fps (99% GPU)

Software information
Any game with MSAA
System information

GPU: RX 580 4GB
Driver: Radeon Software 19.7.5. on Windows, Mesa ACO Git on Linux
D9VK version: 9e10bca

Apitrace file(s)

hl2.7z
Probably won't be useful, I was just deactivating and reactivating MSAA in the menu.

Log files

hl2_d3d9.log

I can confirm this in Counter-Strike: Source:
no MSAA: 716fps (I unlocked the framerate)
8xMSAA: 192fps
Quite an extreme hit for MSAA in an old forward renderer. This is with RX 570 on RADV ACO.

This issue also affects DXVK. Nobody knows why it happens or how to fix it.

This issue also affects DXVK. Nobody knows why it happens or how to fix it.

I can't confirm this with Crysis 1. I set "high" quality config to equal "very high" so that more effects can be used with D3D9 vs. 10. Performance hit with no MSAA vs. 4xMSAA:
DXVK: 74fps -> 59fps
D9VK: 66fps -> 44fps

I can't replicate this on AMDVLK on Windows. I am able to hit 300fps (the cap, consistently) at 4K with 8x MSAA on Portal 2 with my Vega 56.

People reporting this problem appear to all be on Polaris. It might be a driver issue that only affects Polaris.

I can't replicate this on AMDVLK on Windows. I am able to hit 300fps (the cap, consistently) at 4K with 8x MSAA on Portal 2 at the start of sp_a1_intro2 with my Vega 56.
Which is better than native, which only gets around 260-270.

fps_max 0 to uncap.

I know 🐸 I wasn't testing anything though, I was just playing Portal 2 :b

There's basically nothing I can do about this. Sorry.
Vulkan sample counts are Vulkan sample counts, and there isn't really anything I can change -- especially if I can't repro the behaviour myself.

There is a renderstate to disable antialiasing (ie. render non-msaa to an msaa rt) but we can't implement that, and even if we had an extension there are certain limitations that games don't respect that make it unviable. (ie. sample count of depth stencil, depth write quirks wrt this on certain hw)


Compiler crash in Swift 5.7

Describe the bug
Trying to compile this repo at this tag crashes the compiler w/ the current preview of Swift 5.7 (see versions later).
When switching to tag __SWIFT_5.7_CRASHER_WORKAROUND__ there are no crashes of the compiler anymore.
To Reproduce
Steps to reproduce the behavior:

Clone the repo https://github.com/happn-app/officectl.git and checkout the tag __SWIFT_5.7_CRASHER__
Try and compile w/ Swift 5.7 (either on macOS or Linux)

Expected behavior
The compilation should report errors or should pass. It should not crash.
Environment

OS: macOS 12.4 and Linux (Docker)
Xcode Version: 14.0 beta 2 (14A5229c)
Docker image: swiftlang/swift:nightly-5.7-focal (5ea3ea9c2666)

Additional context
I think there are two bugs. There are two commits between __SWIFT_5.7_CRASHER__  and __SWIFT_5.7_CRASHER_WORKAROUND__. One fixes the first issue, the other fixes the other issue.
I don’t know what the issues are though.
I tried reducing the crash but did not succeed.

I cannot reproduce the issue with a recent 14.0 beta. It may have been fixed. Do you have a crash log?

When compiling with Xcode 14.0 beta 5 (14A5294e), I get this: https://gist.github.com/Frizlab/ce48383e4ef379b7f840eacd883ed632

Ah, looks like a SILGen bug.

Still occurring with Xcode 14.0 beta 6 (14A5294g).

Seems fixed (I did not get the crash) with toolchain org.swift.57202208291a (Apple Swift version 5.8-dev (LLVM 03bd949eeee4761, Swift 2c40a850df6f409) - Target: arm64-apple-macosx13.0).

Closing as I cannot reproduce anymore.


formatter failed

2024-08-07T17:25:27.253662543-04:00 [ERROR] failed to format via language server

Caused by:
    formatting nixpkgs-fmt command exited with 65280

my zed wrapper
  zed-fhs = pkgs.buildFHSUserEnv {
   name = "zed";
   targetPkgs = pkgs:
     with pkgs; [
       zed-editor
       nodejs_22
       nodePackages_latest.vscode-json-languageserver
       nixd
       nixpkgs-fmt
          ];
      runScript = "zed";
 };

somehow worked after reboot.


Add float cast op for JIT backend

Checklist

[x] Confirmed that run-checks all script has been executed.
[x] Made sure the book is up to date with changes in this PR.

Related Issues/PRs
Follow-up of #2483
Changes
Added float_cast implementation for burn-jit, burn-fusion and burn-router.

Removed elem type generic from JitTensor

JitTensor now contains the dtype field
Refactored all calls with generics to have the correct generic types propagation (new execute_with_dtype macro to call cubecl kernels)

Added Primitive trait for backend tensor primitives

Motivation: all primitives previously had the same trait requirements, and now we need to report the correct dtype in the Display implementation for Tensor (K::elem_type_name() would report the default floating point dtype even for tensors cast to another precision)

Added dtype checks for binary float ops (no automatic type promotion for now, so we panic for different types)

Known issue
Trying to concatenate tensors cast to a different floating point dtype will fail with the current cat_with_slice_assign implementation because the output tensor is initialized with the default floating point element precision in K::empty(...).
let tensor1: Tensor<B, 2> = Tensor::from_floats([[2., 3.], [4., 5.]], &device).cast(DType::F16);
let tensor2 = Tensor::ones_like(&tensor1).cast(DType::F16);
let tensor3 = Tensor::zeros_like(&tensor1).cast(DType::F16);

// Panics with incompatible dtype (for output)
let output = Tensor::<B, 2>::cat(vec![tensor1, tensor2, tensor3], 0);

TODO

[ ] Implement float_cast for ndarray
[ ] Remove redundant precision bridge

Testing
Added new tests for primitive trait.

@nathanielsimard


[optimizer] Bug fix

Tensor copy constructor and copy assigment operator creates a copy of the vector
This led to bug in optimizer which updated the copy of the weight than the weight itself
Fixed by refernce of weight in optimizer
Resolves #241
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #242. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

BTW,  *.in is going to be ignored in #228. Is addition of *.in in this PR intended?

BTW, *.in is going to be ignored in #228. Is addition of *.in in this PR intended?

ohh yes, updated.


Update LLN and CLT

Generally looks great. A few things:

What's going on with the title of this plot? (Ignore the dvipng issues, those are probably missing some command-line tool on my local machine).

For some reason, our version (below) of this plot looks a lot worse. Some issues in the plotting library? Maybe we can take another look.

The KernelDensity package is throwing some kind of Warning: Could not import LinearAlgebra into Interpolations.jl error. Doesn't seem to break anything...but good to keep an eye on.


Allow setting default values for optional properties closes #19

This implementation does not make any sense since the optional property is still nullable. All the logic needs to go into  the required classes.


Integration Mybatis with spring-data-jdbc,make mybatis can mixed with spring-data-jdbc in ONE repository [DATAJDBC-373]

neil4dong opened DATAJDBC-373 and commented
Integration Mybatis with spring-data-jdbc,make mybatis can mixed with spring-data-jdbc in ONE repository by annotation with @MybatisQuery
Please check out if this way is feasible ？
In my case we currently use mybatis for data access， and we are considering to migrate to spring-data-jdbc， but we have a lot of complex SQL which organized by Mybatis.
Considering the migration cost, how about use mybaitis and spring-data-jdbc together, so we can benefit form

the simplicity provided form spring-data-jdbc
another complex SQL solution provided by mybatis (which now only provided by QueryDsl)

Referenced from: pull request https://github.com/spring-projects/spring-data-jdbc/pull/152

This is nice idea.... is this in the plan?

The PR involved in this issue makes me confused. Is it trying to introduce the concept of BaseMapper similar to baomidou's Mybatis Plus to directly provide additional methods for Mapper?

I implemented such functionality by custom NamedParameterTemplate
@Configuration
public class MybatisQuerySupportConfig {
    @Bean
    public NamedParameterJdbcOperations namedParameterJdbcOperations(DataSource dataSource) {
        return new MybatisJdbcTemplate(dataSource) {@Override protected Dialect dialect() {return new MySQLDialect();}};
    }
}

then you can use like this
@Query("person.selectAllRow")
List<Person> selectAllRow();

For more details, spring-data-jdbc-demo
But this is more troublesome, I still hope you can support
another complex SQL solution provided by mybatis
you can see  mybati  only complex SQL solution provided  No other functions

This issue started as a post hoc issue for a PR I just closed and didn't really develop in a clear direction.
Therefore closing it.


Compound variants are lost when extending multiple times

Describe the bug
When extending tv objects in cascade, the compoundVariants gets lost.
To Reproduce
import { tv } from 'tailwind-variants'

const themeButton = tv({
  base: 'font-medium',
  variants: {
    color: {
      primary: 'text-blue-500'
    },
    disabled: {
      true: 'opacity-50'
    }
  },
  compoundVariants: [{
    color: 'primary',
    disabled: true,
    class: 'bg-black'
  }],
  defaultVariants: {
    color: 'primary',
    disabled: true
  }
})

const appButton = tv({ extend: themeButton })
const button = tv({ extend: appButton })

console.log('appButton', appButton())
// -> font-medium text-blue-500 opacity-50 bg-black
console.log('button', button())
// -> font-medium text-blue-500 opacity-50

Expected behavior
The appButton and button function calls should have the same results.
Desktop (please complete the following information):

OS: Mac
Browser: Chrome

@mskelton could you please take a look at this?

@jrgarciadev PR opened to fix this #170


[docs] Add a section explaining project motivation

It would be better to have a separate section explaining project motivation in the project's README (Will be useful for new contributors)

Hey, I want to work on this. Please assign this to me!

Go ahead @adityasingh10042 👍🏻


What is the reason for handcrafting QuerySet class from scratch?

There is already Query class here https://github.com/sqlalchemy/sqlalchemy/blob/master/lib/sqlalchemy/orm/query.py with all powerful staff included. I mean it could be based on that

There's 5000 lines of code that's not trivial to pick apart.
I'd totally love for someone with more experience of working in depth with SQLAlchemy to take a crack at an SQLAlchemy-style integration on top of databases, but that's not the approach I'm planning to take here.


Cannot locate specified Dockerfile

I'm getting this error when running Nodeshift for Minikube deployment:
2021-01-08T12:37:38.034Z ERROR (HTTP code 500) server error - Cannot locate specified Dockerfile: Dockerfile
2021-01-08T12:37:38.034Z ERROR Status code 500

But I have a Dockerfile in the root dir:
09:38 $ ll | grep Dockerfile
-rw-rw-r--   1 hf hf    380 jan  8 09:35 Dockerfile
✔ ~/code/nodeshift/nodejs-messaging-work-queue/frontend [kafka-nodeshift L|✚ 3…2]

Not sure if that is a bug or if I'm missing something

what is the repo you are using?   looks like it could be a bug

what is the repo you are using?   looks like it could be a bug

@lholmquist I'm using with this https://github.com/nodeshift-starters/nodejs-messaging-work-queue on the front-end part first
same behavior with npx

@lholmquist I'm using with this https://github.com/nodeshift-starters/nodejs-messaging-work-queue on the front-end part first
same behavior with npx

@helio-frota are you on a specific branch?  there isn't a Dockerfile on the main branches

@helio-frota are you on a specific branch?  there isn't a Dockerfile on the main branches

@lholmquist I created the Dockerfile locally

@lholmquist I created the Dockerfile locally

@helio-frota I think i know whats going on.   The package.json in those examples have the files property with the list of files to use.  the Dockerfile is probably not in there,  so nodeshift skips it

@helio-frota I think i know whats going on.   The package.json in those examples have the files property with the list of files to use.  the Dockerfile is probably not in there,  so nodeshift skips it

@lholmquist that makes sense. I'll try again later then I'll post the result here, thanks

@lholmquist that makes sense. I'll try again later then I'll post the result here, thanks

@lholmquist thanks for the information, like you wrote the issue is just missing to add Dockerfile in files section :+1:

@lholmquist thanks for the information, like you wrote the issue is just missing to add Dockerfile in files section :+1:


The question of the gated axial attention formula

Excuse me, for the gated axial-attention formula, I can understant the setting of the gating parameters in the three positions of Gqr, Gkr and Gr, but why do you add the gating parameters on v, which has no position information.
Thank you!

Hello @jeya-maria-jose and @wang-yu-xuan , from what I understood from the code, the gated parameters Gq, Gk, Gv1 and Gv2 is just self.f_qr, self.f_kr, self.f_sve and self.f_sv on the code is it? They appear to be nn.Parameters with value of 0.1 and requires_grad = False. Does it mean that these parameters are not updated during training? Why then does Section 3.2 call them learnable parameters? How did you decide on the value of 0.1?

Hi @charleneolive , the gate parameters are not updated for the initial epochs to give an early start to the other parameters. From experiments, we found this configuration to work better. Later the requires_grad is set as True from 10th epoch.  https://github.com/jeya-maria-jose/Medical-Transformer/blob/10fb5a705d41d4fb88575dc4c3ff78abfe415153/train.py#L169

Hi @charleneolive , the gate parameters are not updated for the initial epochs to give an early start to the other parameters. From experiments, we found this configuration to work better. Later the requires_grad is set as True from 10th epoch.
https://github.com/jeya-maria-jose/Medical-Transformer/blob/10fb5a705d41d4fb88575dc4c3ff78abfe415153/train.py#L169

Makes sense, thank you!


Support the prank() family of cheatcods

What title says. This is a slightly more complex PR. To support pranks, we needed to keep them scoped to external call contexts. To do so we hotpatch the Interpreter.run function, and add callbacks that keep track of the external call stack. Then we add (first pending and then activated) pranks as additional properties on the EEI objects that we use to identify external call frames. Additionally we interpose on the CALLER and ORIGIN opcodes to implement the prank behavior

Codecov Report

Merging #23 (feef3ed) into main (e768de5) will increase coverage by 0.16%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
+ Coverage   78.51%   78.67%   +0.16%
==========================================
  Files          19       20       +1
  Lines        1666     1768     +102
  Branches      339      360      +21
==========================================
+ Hits         1308     1391      +83
- Misses        207      217      +10
- Partials      151      160       +9

Impacted Files
Coverage Δ

src/debug/foundry_cheatcodes.ts
85.33% <76.82%> (-8.79%)
:arrow_down:

src/debug/opcode_interposing.ts
85.00% <85.00%> (ø)

src/debug/sol_debugger.ts
84.51% <100.00%> (+0.88%)
:arrow_up:

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


feat: Ensure adopted link records are fresher than previous entries

Fixes #258, fixes #562

Great work on this, I think it needs some consideration in the periphery of usage but otherwise I like it.

Updated with handling the error in non-test/looping scenarios where we want to continue (:drum:) processing if a failure occurs


[Request] Filter notifications by type

I'm a big fan of this app, but one thing I miss is being able to hide notifications for new messages.
Is this something you would consider adding whenever possible?

Good idea, I'll consider it for the next improvments

Added along with per-type notification sound


House keeping

Migrate to addons-based config
Upgrade all
Remove title from homepage

@pbauer build fixed :) I just read in discord about the beer, am I the first one? :P

@sneridagh Yes, the 🍺 is yours 😄

@pbauer :tada:
Then whoever makes the next PR beer is on me then!


[FEATURE] Document Intervals Query

Add documentation for the full-text intervals query.

@mattweber: This feature is already documented here, https://opensearch.org/docs/latest/opensearch/query-dsl/full-text/

@Naarcha-AWS no it is not, please reopen.

@Naarcha-AWS  please reopen, cc @dblock @nknize


Bump Sphinx from 1.3.5 to 1.4.6

The listed Sphinx requirement is outdated.
Pull request: https://github.com/rtfd/readthedocs.org/pull/2378

Closing as duplicate of #1853


Reverse decision to store RIDs in qualifiedName for DataStage entities

We had previously decided to store the IGC RID in the qualifiedName of certain DataStage entities (like jobs and stages), with the idea that this would give us a way to look them back up by ID.
However, we have since modified the GUID structure from IGC so that the metadata collection ID of the IGC repository is embedded in the GUIDs -- therefore, the RID alone is insufficient to do this reverse-lookup.  Furthermore, because the DataStage connector uses its own configuration (does not re-use any IGC connector), it will not be aware of what the metadata collection ID of the IGC repository is when setting these qualifiedNames.  As a result, these RIDs are probably less useful than what we're using as a qualifiedName everywhere else...
Should we therefore revert back to using the same qualifiedName as used elsewhere rather than a RID?

Agreed 24/10 call to go ahead and revert this per comment above.


Improve repo sync script

Change the Makefile.common update script to be more generic.
Sync CODE_OF_CONDUCT.md and LICENSE files.


Re-split mesh on restart

It would be incredibly useful if KHARMA could read old iharm3d restart files -- this would allow:

Verification that KHARMA fixes inversion failures in exactly the same way as iharm3d did
Extending old GRMHD simulations at the same resolution
Restarting old GRMHD runs at higher resolution, either to extend them or just to skip the initial 2-3kM+ of infalling time before quiescence.

I'm opening an issue to track progress because this eventually involves several steps:

Initialize a single meshblock of matching size
Split a single restart into an arbitrary number of meshblocks for distributed parallel operation
Interpolate the magnetic field and clean divB, to allow restarting at different resolution than the original.
(Separate issue) Interpolate the magenetic field onto faces for AMR/higher-order operation

Note uses 1-3 roughly require features 1-3 respectively.

Step 1 is done, though purpose (1) is not worth chasing IMO now we know that it's possible to complete sims with dramatically fewer fixups using resolution/wind/better floors.
For reference, here's the difference after 1M between implementations:

Everything tracked here is now supported in dev.


LICENCE is contradictory

The first line on LICENSE.md currently states:
Copyright 2011-2012 Bulletproof Networks. All rights reserved.

The rest of the file then has the standard MIT licence.
However, the All rights reserved. contradicts the MIT license.
So the MIT license is valid, the All rights reserved. should be removed.

Thanks LH - This has been resolved by @elmobp in Pull Request #22


make it compile to 1.18.2

seems to work for 1.18.2, you'll notice that i also added an executable bit to gradlew so others don't have to :)

Fixes #6 and #7 too

darn my bad i tried changing the compare branch

aw darn

i give up, i'm changing this the normal way and making a new pr


VCPKG fails to configure when running through cmake configuration on linux.

Describe the bug
$ cmake -S . -B build -DCMAKE_TOOLCHAIN_FILE=/mnt/d/dev/vcpkg/scripts/buildsystems/vcpkg.cmake
-- Running vcpkg install
-- Running vcpkg install - done
CMake Error at /mnt/d/dev/vcpkg/scripts/buildsystems/vcpkg.cmake:531 (list):
  list sub-command REMOVE_ITEM requires two or more arguments.
Call Stack (most recent call first):
  /usr/share/cmake-3.16/Modules/CMakeDetermineSystem.cmake:93 (include)
  CMakeLists.txt:3 (enable_language)

-- The CXX compiler identification is GNU 9.3.0
-- Check for working CXX compiler: /usr/bin/c++
CMake Error at /mnt/d/dev/vcpkg/scripts/buildsystems/vcpkg.cmake:531 (list):
  list sub-command REMOVE_ITEM requires two or more arguments.
Call Stack (most recent call first):
  /mnt/d/dev/project/build/CMakeFiles/3.16.3/CMakeSystem.cmake:6 (include)
  /mnt/d/dev/project/build/CMakeFiles/CMakeTmp/CMakeLists.txt:2 (project)

CMake Error at /usr/share/cmake-3.16/Modules/CMakeTestCXXCompiler.cmake:37 (try_compile):
  Failed to configure test project build system.
Call Stack (most recent call first):
  CMakeLists.txt:3 (enable_language)

-- Configuring incomplete, errors occurred!
See also "/mnt/d/dev/project/build/CMakeFiles/CMakeOutput.log".

Environment

OS: Ubuntu 20.04 LTS (WSL2)
OS: Ubuntu 20.04 LTS (docker)
Compiler: gcc 9.3.0

To Reproduce
Steps to reproduce the behavior:

$ cmake -S . -B build -DCMAKE_TOOLCHAIN_FILE=/mnt/d/dev/vcpkg/scripts/buildsystems/vcpkg.cmake
See error
Repro code when

Expected behavior
I expect vcpkg to successfully install my listed dependencies from my manifest file.
Failure logs
-(please attached failure logs)
Additional context
I have a CMake project that depends on libjpeg-turbo that has the following manifest file when this problem occurs.
{
  "name": "project",
  "version-string": "2022.1.31",
  "port-version": 1,
  "dependencies": [
    {
      "name":"libjpeg-turbo"
    }
  ]
}

I have narrowed the problem down to a recent addition to vcpkg/scripts/buildsystems/vcpkg.cmake:351
option(VCPKG_SETUP_CMAKE_PROGRAM_PATH  "Enable the setup of CMAKE_PROGRAM_PATH to vcpkg paths" ON)
cmake_dependent_option(VCPKG_USE_HOST_TOOLS "Setup CMAKE_PROGRAM_PATH to use host tools" ON "NOT VCPKG_HOST_TRIPLET STREQUAL \"\"" OFF)

if(VCPKG_SETUP_CMAKE_PROGRAM_PATH)
    set(tools_base_path "${VCPKG_INSTALLED_DIR}/${VCPKG_TARGET_TRIPLET}/tools")
    if(VCPKG_USE_HOST_TOOLS)
        set(tools_base_path "${VCPKG_INSTALLED_DIR}/${VCPKG_HOST_TRIPLET}/tools")
    endif()
    list(APPEND CMAKE_PROGRAM_PATH "${tools_base_path}")
    file(GLOB_RECURSE Z_VCPKG_TOOLS_DIRS "${tools_base_path}/*")
    file(GLOB_RECURSE Z_VCPKG_TOOLS_FILES LIST_DIRECTORIES false "${tools_base_path}/*")
    list(REMOVE_ITEM Z_VCPKG_TOOLS_DIRS ${Z_VCPKG_TOOLS_FILES})
    list(FILTER Z_VCPKG_TOOLS_DIRS EXCLUDE REGEX "/debug/")
    foreach(Z_VCPKG_TOOLS_DIR IN LISTS Z_VCPKG_TOOLS_DIRS)
        list(APPEND CMAKE_PROGRAM_PATH "${Z_VCPKG_TOOLS_DIR}")
    endforeach()
    unset(Z_VCPKG_TOOLS_DIRS)
    unset(Z_VCPKG_TOOLS_DIR)
    unset(tools_base_path)
endif()

I have been unsuccessful at getting this script to be ignored. I have tried setting -DVCPKG_SETUP_CMAKE_PROGRAM_PATH=OFF to no avail. CMakeCache.txt states it is off but vcpkg seems to ignore the state of the variable and continues to run the above script no matter how I try to turn it off.
My problem goes away when I disable the above script entirely by commenting it out.

PR: #23410


Added a new migration to change 'Of' to lowercase

#593
Just a new migration to change "Of" to "of"
Changelog:

Test fixtures updated
Migration added

 Government Of British Columbia | Update to lowercase "of" to align with BC Government web standards |1|

only needs to rename 0028 to 0029


(g)(10) Certification US Core 5.0.1 Inferno Reference Server Errors

Hello there,
In this test run https://inferno.healthit.gov/suites/test_sessions/f2dd3a69-4770-412e-bcbf-d024b1456af0#g10_certification we are executing the Inferno tests for Standalone Patient App and Single Patient API (US Core 5.0.1) against the Inferno Reference Server and there are a lot of resources that have errors or are skipped. We are using the default selections and inputs when running the tests.
I noticed that there is already an open issue for the ServiceRequest related errors (https://github.com/onc-healthit/onc-certification-g10-test-kit/issues/287) so those should be covered with that one, but there are many more unrelated to that. Until recently we worked on certification for US Core 3.1.1 and the Inferno tests against the Reference Server were all passing.
We also tried the same run on the Inferno dev instance, but we got the same output https://inferno-dev.healthit.gov/suites/test_sessions/f3f50641-3eef-4cee-84f3-231784b99099#g10_certification.
Is this behavior expected for US Core 5.0.1 or are we doing something wrong?
Thanks.

The reference server does not support US Core 4 or 5 yet.

Is there a rough timeframe for when that would be added?

We don't have a timeline yet. We are working on updating Inferno demo server to US Core v4.0.0.


Fix detection of button n°2 for Xiamo Aqara Opple Remote 6 V5

Fix detection of button n°2 for Xiamo Aqara Opple Remote 6 V5
Similar to #700

Codecov Report

Merging #904 (473db6d) into dev (470ccf9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #904   +/-   ##
=======================================
  Coverage   82.50%   82.50%
=======================================
  Files         185      185
  Lines        4589     4589
=======================================
  Hits         3786     3786
  Misses        803      803

Impacted Files
Coverage Δ

zhaquirks/xiaomi/aqara/opple_remote.py
87.39% <ø> (ø)

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build 890121196

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.502%

Totals

Change from base Build 876824514:
0.0%

Covered Lines:
3786

Relevant Lines:
4589

💛  - Coveralls


Remove "edit on GitHub" from mobile

Do it as is done on RTD

There is also an issue with system fonts on iOS devices. The "intended" look on Android is:

However, in Safari on an iPhone it doesn't look right:

It's not just Safari though (which is why I think it's about system fonts). E.g. in Chrome on an iPhone:

and Chrome on an iPad:

I hope that uncss just lopped off the CSS needed on these other devices.

The culprit was not uncss, but it was me :grinning: Somehow or another my font / glyph hacking introduced a problem.


Disable CRON

Is there any option to disable cron?  I want to manage this via scripts to support multiple backend backups.  So having a default cron will not work with my use case.

For managing restic via script it might be enough to use restic either on the host or use the official restic container directly with docker exec
The whole point of this container is automatic backup scheduling. You could set an invalid corn that will never trigger - I assume there is no check against that :)


Fix empty ref scroll

setTimeout work afte componentWillUnount, and error

Cannot read property 'style' of undefined

@Asgator can you serve a demo about this error?

@lanjingling0510 number scrolle and befor stop unmounted component
In my situation that hestory go back;

@Asgator okay.  l will publish it in next version these days.

@lanjingling0510 thank you

When will the version be released


Setup Gmail with Email Digest Agent

I would like to use my Gmail account to send notifications from Huginn.
I followed the documentation to configure the Docker container. Running the email digest agent does not report an error, but I haven't got any email. I also checked the /app/log/production.log file inside the container, but it was empty.
What did I do wrong? Why there is no error?

Hi @elvetemedve.
Do you have any messages in the Agent logs in the UI?  I assume you configured the ENV variables for the email username and password, etc?

@elvetemedve You access the logs with docker logs <container name>

Hi @cantino,
The only message I see is this: Sending digest mail to<EMAIL_ADDRESS>with events [2].
Hi @dsander,
I triggered the agent and watched the logs. I guess these are the relevant lines:
foreman stdout | 08:44:31 jobs.1 | 2016-03-04T08:44:31+0000: [Worker(host:ea13ecddd52f pid:1149)] Job ActiveJob::QueueAdapters::DelayedJobAdapter::JobWrapper (id=2018) RUNNING
foreman stdout | 08:44:31 jobs.1 | [ActiveJob] [AgentCheckJob] [5d2f54f2-90b8-4e82-b548-bc44ea55ef54] Performing AgentCheckJob from DelayedJob(default) with arguments: 5
foreman stdout | 08:44:31 jobs.1 | [ActiveJob] [AgentCheckJob] [5d2f54f2-90b8-4e82-b548-bc44ea55ef54] Performed AgentCheckJob from DelayedJob(default) in 24.14ms
foreman stdout | 08:44:31 jobs.1 | 2016-03-04T08:44:31+0000: [Worker(host:ea13ecddd52f pid:1149)] Job ActiveJob::QueueAdapters::DelayedJobAdapter::JobWrapper (id=2018) COMPLETED after 0.0282
foreman stdout | 08:44:31 jobs.1 | 2016-03-04T08:44:31+0000: [Worker(host:ea13ecddd52f pid:1149)] 1 jobs processed at 23.8401 j/s, 0 failed

However I don't see any email or SMTP related info.

That does not look like the agent attempted to deliver a mail, did the Digest Agent have event in its memory queue?

@dsander It seems to be empty at the moment.
I made few screenshot of the configuration. See it here: https://drive.google.com/folderview?id=0B2tXOm0gFdwed0tDZFVSQ3ZqSTg&usp=sharing
Can you spot any problem?

The configuration looks correct, when you re-emit the event of the RainNotifier it should be added to the email digest agens memory again. Running the digest agent afterwards should send an email, or at least show an error in the docker logs.

They key was re-emitting the event of the Rain Notifier. At the time the event was created, the SMTP setup was wrong. I amened it later, but only re-run the Email Digest agent which did not do anything, because of the empty memory queue.
So it works perfectly.
Thank you for helping me.

Glad we could help!

Hi @elvetemedve , I also used Docker to deploy Huginn, but I don't know how to load my huginn env to Docker. I use:
docker run --restart=always -it -e DO_NOT_SEED=true --env-file /home/docker/huginn/env.huginn -p 3000:3000 huginn/huginn
But I alway get error. Can you help me?
Thank you^^

Hi @soiqualang,
I don't see any obvious error with your command. Just one tiny improvement you can do is move the DO_NOT_SEED variable into the environment file.
I can't SSH into my home server remotely at the moment, so I will check how I did that in the evening.

@elvetemedve , thanks for your replying.
When I run that command, I just got this:

I also removed DO_NOT_SEED, it've showed the same result.
Sorry to bother you so much.

@soiqualang The source of the problem is shown around the middle:
Access denied for user 'root'@'localhost'

Looks like your connection settings to the MySQL database is incorrect. Did you follow the setup guide on Dockehub?

@soiqualang I have created the bash command alias below to update/setup Huginn via Docker.
alias update-huginn='sudo docker rm huginn; sudo docker pull huginn/huginn; sudo docker create -v /var/lib/huginn:/var/lib/mysql -p 3000:3000 --env-file="$HOME/huginn.env" --name=huginn huginn/huginn;'
where huginn.env is based on the sample file, but I removed the unused settings and added DO_NOT_SEED=1 to the end of the file.

@elvetemedve Try removing the DATABASE_PASSWORD="" line your .env file. You might also need to delete your database volume to ensure the default password is used.


Building a test

Current Test only does manual printing.
We need a test to check If it is working properly.

Done in v1.0.0


Request: repositioning post-end when legacyLayout = true

When legacyLayout = true, I find the post-end bar seems to be in the wrong place. See the below screenshot:

I think a few options for when legacyLayout = true:

Disable post-end in favor of art-end
Disable art-end and move post-end above the metadata (what hermit v1 has)

What do you think?  Thanks in advance

Hi @jamesbraza
Yep, my bad. I zoned out while checking the layout and failed to differentiate. Pushing a change within an hour.

I think it is better (for legacy compatibility) to have art-end disabled.

This is how layout will look.

Closing this issue. Re-open if required


Cluster installation prechecks should query ssh keys and passwordless prompts are configured as needed

The role prechecks should query all cluster nodes ssh key exchange and passwordless prompts are as expected prior to running the playbook.  This will prevent hangs and failures to ensure the environment is configured properly.
See 'mmnetverify' or other options to run the query from install node to all nodes as well as all nodes to other nodes as well as to themselves.

As per our design and discussion , we are not planning to add any hard validation in the open source code. user needs to define inventory and check node accordingly. we will have real validation and mmnet check in the toolkit.
We have already added example playbook to validate ssh and node states . user can write their own playbook using this sample playbook.
https://github.com/IBM/ibm-spectrum-scale-install-infra/blob/dev/samples/playbook_cloud.yml

As per our design and discussion , we are not planning to add any hard validation in the open source code. user needs to define inventory and check node accordingly. we will have real validation and mmnet check in the toolkit.
We have already added example playbook to validate ssh and node states . user can write their own playbook using this sample playbook.
https://github.com/IBM/ibm-spectrum-scale-install-infra/blob/dev/samples/playbook_cloud.yml


None correct type handling for slot 'rate of x11-video-mode structure

Slot 'rate in x11-video-mode structure type is double-float, but not correct handling this fact.
For correct working i changed this code:
(defmethod set-video-mode ((mode x11-video-mode))
(glop-xlib::with-current-display dpy
(glop-xlib::set-mode dpy 0 (x11-video-mode-index mode)
(x11-video-mode-rate mode))))
on it:
(defmethod set-video-mode ((mode x11-video-mode))
(glop-xlib::with-current-display dpy
(glop-xlib::set-mode dpy 0 (x11-video-mode-index mode)
(truncate (x11-video-mode-rate mode)))))
And also two call make-x11-video-mode:
(make-x11-video-mode
...
:rate rate
...)
Changed to:
(make-x11-video-mode
...
:rate (coerce rate 'double-float)
...)
Please analyzed bug report

Looks like the type change was introduced during the osx implementation.
rate slot should be integer on x11 implementation but double-float on osx implementation.
I moved the rate slot to platform specific video-mode struct instead of the base one.

Xm.. but slot "rate" in src/utils.lisp not specified:
(defstruct video-mode
(width 0 :type integer)
(height 0 :type integer)
(depth 0 :type integer)
(rate 0 :type double-float)) ; <

I pushed the change to the devel branch not to the master one.
https://github.com/patzy/glop/tree/devel


Make JTAG tests report assertions properly

JTAG tests don't capture output from verilog simulator, so, if it asserts, the console won't print anything interesting.

Fixed in 7088d5958c2355f17cbccaa3cf42dafe3eaa07e4


Tooltip needs more flexibility

Currently Tooltip only supports hovering a pointing device. It would be nice if it would support any kinds of InputEvent if someone wants it. Right now it is impossible for example to make tooltips to appear using LongPress which is the convention on touchscreen devices without hover support.
One Option would be Tomski's proposed solution: https://gist.github.com/Tom-Ski/a02f0cb692abb707ec48
This would add flexibility to extend BaseTooltip to use different EventListeners and to support any kind of behaviour.

It looks like currently a tooltip will be shown if you long press. Is that not the case?
TooltipManager could have a show(Tooltip) method if you want to programmatically trigger showing.

Nope, it's not the case, at least on Android. On the computer it seems to work, because there was a mouseMoved() before. Also I tried calling getManager().enter() however that doesn't works because there is a condition "if (Gdx.input.isTouched)) return; which prevents setContainerPosition to be called. I tried something like this: https://gist.github.com/sarkanyi/85a9dc0150da0efbed9

Well, based on this I've implemented my own tooltip, that gives me even more flexibility so I guess it's safe to close this if it's not desired on libgdx level.

Sorry, this issue got buried in the backlog. Enter is fired for mouse move on the desktop and also for each mouse down / touch on all platforms. Ultimately we need if something in tooltip needs to be improved. https://github.com/libgdx/libgdx/wiki/More-Info


UI changes

Changes on landing page as well as when it displays the data fetched.

i would like to contribute please assign


Notes release notes 0.65

I will update the docs (promise!), but here are a list of the main changes/fixes in this release. Also see the referenced issues.

Any branch node can now bundle resources (images, data files etc.), even the taxonomy nodes (e.g. /categories)
List pages (sections and the home page) can now be added to taxonomies.
draft, publishDate, expiryDate etc. now also works for list pages, and is recursive.
New _build meta keyword in front matter. See BuildConfig below. headless still works, but this is more general as it works for any page and is obviously more flexible. Can, of course, be set in a cascadeblock. Setting PublishResources=false will turn off auto-publishing of bundled resources (e.g. image originals).
.GetPage "members.md" (the Page method) will now only do relative lookups, which is what most people would expect.
A slight change of how disableKinds for regular pages: They will not be rendered on its own, but will be added to the site collections.

Some relevant fixed issues: #6312 #6087 #6738 #6412 #6743 #6875 #6034 #6902 #6173  #6590
type BuildConfig struct {
	// Whether to add it to any of the page collections.
	// Note that the page can still be found with .Site.GetPage.
	List bool // Default true

	// Whether to render it.
	Render bool // Default true

	// Whether to publish its resources. These will still be published on demand,
	// but enabling this can be useful if the originals (e.g. images) are
	// never used.
	PublishResources bool // Default true
}

The above would fits into a content file with YAML front matter:
---
title: Nothing
_build:
  list: false
  render: false
  publishResources: false
---

I see a couple of differences in taxonomies:

term pages' parent seem to be directly the home page in v0.65 (compare https://github.com/baldowl/hugo-0.64.1-vs-0.65-dev/blob/master/public-hugo-0.64.1/tags/a-tag/index.html with https://github.com/baldowl/hugo-0.64.1-vs-0.65-dev/blob/master/public-hugo-0.65-dev/tags/a-tag/index.html; navigation was generated by https://github.com/baldowl/hugo-0.64.1-vs-0.65-dev/blob/master/layouts/partials/breadcrumb.html);
taxonomies' RSS feeds seem to be empty (compare https://github.com/baldowl/hugo-0.64.1-vs-0.65-dev/blob/master/public-hugo-0.64.1/tags/index.xml with https://github.com/baldowl/hugo-0.64.1-vs-0.65-dev/blob/master/public-hugo-0.65-dev/tags/index.xml; generated by Hugo's internal RSS template).

Thanks @baldowl -- much appreciated.

Didn't got too far :
~/go/bin/hugo  -v --debug
Total in 4 ms
Error: from config: failed to resolve output format "RSS" from site config

Thanks, @divinerites -- I'll fix the RSS issue. The background is a consolidation of RSS as both a "page kind" and an output format. Conceptually it should be disabled by defining  outputs  to not include it, but we should be backwards compatible, of course (I had a test for it, not sure why It didn't fail ...).

 Thanks for pointing it.

I confirm that your fix https://github.com/gohugoio/hugo/commit/da54787cfa97789624e467a4451dfeb50f563e41 works fine on my sites if they have the conflicting configuration. Thanks.

I am still seeing the problem from #6173 .
If the main _index.md doesn't exist, the PT version can't access the Resource and just outputs (unknown)

I'm a bit confused by the new BuildConfig and the option to omit bundled resources: if publishResources is false, when is a resource published?
Using a simple-minded shortcode like {{ (.Page.Resources.GetMatch (.Get 0)).RelPermalink }} I get the expected string, but the referenced resource (which is not an image) isn't copied to the destination dir.

when is a resource published?

When you access one of the methods that require it to be published (e.g. .RelPermalink). I will do a double check that this is actually the case ... I may have a hole in the test coverage in this case.

@baldowl OK, I had a test for this, but the test was wrong (and very limited) -- this should be fixed now. Thanks for the report.

Not sure if it is a hint for a glitch/bug, but my code gives some differences between 0.61.0 & 0.65.0.
So just in case.
And I can't find a way to check what pages are generated ("hugo list" only the blogs).
$ hugo --gc --cleanDestinationDir
| FR  | EN | NL
+------------------+-----+----+----+
Pages            |  24 | 22 | 22
Paginator pages  |   2 |  2 |  2
Non-page files   |   0 |  0 |  0
Static files     |  92 | 92 | 92
Processed images | 225 |  0 |  0
Aliases          |   4 |  1 |  1
Sitemaps         |   2 |  1 |  1
Cleaned          |   0 |  0 |  0
Total in 898 ms
$ ~/go/bin/hugo --gc --cleanDestinationDir
| FR  | EN | NL
-------------------+-----+----+-----
Pages            |  23 | 21 | 21
Paginator pages  |   2 |  2 |  2
Non-page files   |   0 |  0 |  0
Static files     |  92 | 92 | 92
Processed images | 225 |  0 |  0
Aliases          |   4 |  1 |  1
Sitemaps         |   2 |  1 |  1
Cleaned          |   0 |  0 |  0
Will keep investigating.

Forget it. It is a different behaviour with "draft = true" on `_index.html". So this behaviours is correct. Sorry for the noise.


Local babel plugin not working correctly after npm run eject

I want to develop a plugin of babel , for convenience, I ejected CRA and do something as below

Add plugins path to paths.js
path.js

{
  appSrc: resolveApp('src'),
  plugins: resolveApp('plugins')
}

Add plugins to  options  in webpack.config.js
webpack.config.js

{
    test: /\.(js|mjs)$/,
    exclude: /@babel(?:\/|\\{1,2})runtime/,
    loader: require.resolve('babel-loader'),
    options: {
        babelrc: false,
        configFile: false,
        compact: false,
        presets: [
            [
                require.resolve('babel-preset-react-app/dependencies'),
                { helpers: true },
            ],
        ],
        plugins: [
            // add my babel plugin
            path.resolve(paths.plugins, 'index.js')
        ],

    },
}

Writing code in plugins/index.js
plugins/index.js

module.exports = function(){
    return {
        name:'validate-response',
        visitor:{
            ImportDeclaration(path){
            		// never arrive
                console.log(path.node)
            },
            Identifier(path){
            		if(path.node.name==='request'){
            		    // never arrive
            				console.log('request')
            		}
            }
        }
    }
}

add some code to app.js
App.js

import {request} from '@y/request'

My expectation is to print the log , but backfire, the log has never been printed.
In my [example] (https://github.com/mqliutie/example) the log can be printed

I'm sorry but we don't provide support after ejecting. It sounds like this is a question about how to write babel plugins. I'd recommend trying StackOverflow.


using table's zod validator on client side?

for form validation on client side,
and for body validation on server side,
I tried to use same schema from SomeTable.createSchema(),
but I since postgres trying to run on client,
results in below error.
ReferenceError: Buffer is not defined
    at new Writer (chunk-7EVEQ3B7.js?v=458f132b:2592:23)
    at node_modules/pg-protocol/dist/serializer.js (chunk-7EVEQ3B7.js?v=458f132b:2669:18)
    at __require (chunk-V4OQ3NZ2.js?v=458f132b:11:50)
    at node_modules/pg-protocol/dist/index.js (chunk-7EVEQ3B7.js?v=458f132b:3234:24)
    at __require (chunk-V4OQ3NZ2.js?v=458f132b:11:50)
    at node_modules/pg/lib/connection.js (chunk-7EVEQ3B7.js?v=458f132b:3303:32)
    at __require (chunk-V4OQ3NZ2.js?v=458f132b:11:50)
    at node_modules/pg/lib/client.js (chunk-7EVEQ3B7.js?v=458f132b:5897:22)
    at __require (chunk-V4OQ3NZ2.js?v=458f132b:11:50)
    at node_modules/pg/lib/index.js (chunk-7EVEQ3B7.js?v=458f132b:7179:18)

I know I've been throwing lots of schema questions these days,
but it is because I so much love the feature and want to use it everywhere.
is there any way that I can get the schema from table on client side?

The current implementation is only good for server side, because it would pull all the ORM into frontend if you try to import the schema.
Some day in the future it could generate schemas by running a cli command, so that schemas won't depend on ORM directly.

Thanks, will be waiting!
anyway the .optional() part for querySchema is still broken,
hoping it gets fixed soon.

If something is broken, please open an issue so I can know about it and will remember to fix it.


USER STORY - 06 - CREATING DATA IN SQL & CREATING API ENDPOINT FOR SIGN UP FROM AS A GRADUATE( continuing issue #12, branch- Grad-Form-SignUp-backend-DATA)#18

Story
As a CYF GRADUATE
I Want - I want to see the button for signing up and logging in -Front end part(by Sandra)
to sign up as a graduate I want to input my credentials which are
the  email, and username, and  a password
So That -
2.2 sign up with  email, username, and password
2.3  want to get a message that the account has been created / Grads registered
Acceptance Criteria

username
email
password

Implementation Notes
SQL data for the registered grads need to be created.(@KamNik0044 )
table name - "user_data" #34
Checklist:

[ ] I have defined the user story
[ ] I have defined some acceptance criteria
[ ] I have included any implementation notes
[ ] I have included all relevant links

shall I merge this?


Updates for Elm 0.16

I was hoping to address some the items remaining on the checklist but ran into a few issues after updating to Elm 0.16. These changes aim to address those.
I've labelled this as work in progress since I'm now receiving the following error, and have only found this reference to something similar so far: https://github.com/elm-lang/elm-compiler/issues/1150#issuecomment-154279787

Looks like I just need to use the relevant scripts in package.json.
I'll do this shortly.

@ohanhi this is now ready for review :pray:

@paulyoung Nice, seems to work as expected! I shall merge! :+1:


Change EKF update to registered event.

Relates to #63
Changes the EKF's update to be a kTimed event. The user must call estimator.set_next_message_time(t) to schedule an event at time t (technically, t-eps).
I'm not totally sure that this is the right approach, so please read with a critical eye. I'm also not totally sure it's better than #187
Flaws:

Since DoCalcNextUpdateTime is const, next_message_time is declared mutable so it can be reset within this method. This is probably not the right way to do this, as it breaks some of the Systems contracts regarding Context, but perhaps it's OK.
The required use of eps in the event time. It could, alternatively, be moved to simulator.AdvanceTo(t + eps), if that's any better.

The benefits from this approach are that:

context.time is always correct (up to eps)
Doesn't require superfluous states/ports to the system.

This change is

Thanks for the quick fix here! Would it be better to avoid using inf and instead _not _ reset the time. Instead, in DoCalcNextUpdateTime, just check if
context.time < next_message_time

before declaring the event?

Yeah I think that also works, with a slight modification, using context.time < next_message_time - eps. The changes are in the new commit

Flaws:
* Since `DoCalcNextUpdateTime` is `const`, `next_message_time` is declared mutable so it can be reset within this method. This is probably not the right way to do this, as it breaks some of the Systems contracts regarding Context, but perhaps it's OK.

next_message_time doesn't need to be mutable anymore in the new implementation!


feat: switch component for OMP

Type of Change

[ ] Bugfix
[x] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

Motivation and Context

How did you test it?

Where to test it?

[x] INTEG
[x] SANDBOX
[ ] PROD

Checklist

[x] I ran npm run re:build
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible

can you raise a separate PR for independent and core component changes . So that this PR only contains the business logic changes


Textures cache is drained when cocos2d::LWFNode is released

Code:
void LWFResourceCache::unloadLWFDataInternal(const shared_ptr<LWFData> &data)
{
	map<string, bool>::iterator
		it(data->resourceCache.begin()), itend(data->resourceCache.end());
	TextureCache *cache = Director::getInstance()->getTextureCache();
	for (; it != itend; ++it) {
		Texture2D *texture = cache->getTextureForKey(it->first.c_str());
		if (texture && texture->getReferenceCount() == 1)
			cache->removeTexture(texture);
	}
}

Every time we load LWF file which was loaded before - we will load (decode png, apply alpha processing) same images again.
Some ideas:

modern devices have a lot of memory
mobile devices (iOS/Android) have "memory warning" mechanism which should be used to free unused memory when it's needed
in our project, images loading takes 90% of all loading time
performance unfriendly if each view/screen has it's own LWF. each time we open some view/popup - we will spend a lot of time to load textures we've loaded before
hi-jacking (direct manipulation) of cache is bad practice

I can imagine that in some rare situations, we'd prefer to have as much free memory a possible before new LWF is loaded, especially if it uses a lot of huge textures, but it's rare situation.
Some statistics (TextureCache::addImage) as example. We have "pause" screen as separated LWF. First and second screen opening
With cache drain: 300ms/291ms
Without:               302ms/2ms
Can we have it configurable at least?

Yes, definitely. You can change the behavior whatever you want because it is open source.

Sorry, no offense here.
Just wanted to hear thoughts of author about current approach and probably to get more flexible solution in upstream.
Thank You.

I don't understand the point. You wanted to keep textures on memory? why don't you keep LWF instance or LWF data on memory?

The idea of cache is to save result of expensive work and re-use it when it will be needed once again. This is not about owning objects.
Values stored in any cache always live longer than it's users.
If You look through whole cocos2dx project you will see, that there is no any manual cache draining used, except FPS counter.
Owning all LWF objects will lock textures in cache and if memory warning will happened - they can't be released and app won't survive. It's very different from approach, when textures are kept in cache for case when they will be needed once again, but if app is out of memory - they can be easily released.
From cocos2dx srcs:
    /** Purges the dictionary of loaded textures.
    * Call this method if you receive the "Memory Warning".
    * In the short term: it will free some resources preventing your app from being killed.
    * In the medium term: it will allocate more resources.
    * In the long term: it will be the same.
    */
    void removeAllTextures();

Very detailed explanation: http://stackoverflow.com/questions/15704367/cocos2d-2-0-when-is-good-practice-to-purge-caches

I know the behavior and still I don't understand your point at all. You can rewrite the code in order to get the behavior what you want.

Thank you.
I will create PR, probably it will show the idea much better.

Thank you. But no need to create PR.


GEODE-5474: Fix test flakiness in DiskRegionAsyncRecoveryJUnitTest an…

…d DiskOldAPIsJUnitTest

Occasionally these tests will fail if the time between region close and
re-creation is so short that the PersistenceMemberID is equivalent between
the old and new values.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

I was just thinking it would be in this test

Awaitility already seems to have the right mechanisms...


enhancement: pull items directly out of the Network View to player inventory or cursor

I find myself building requester chests early game just to pull items out of the item network.
This is before personal logistics are available.
It would be nice to be able to pull items directly from the Network View dialog on the item tab.
Maybe use the same ctrl/shift/left/right click semantics used elsewhere in the game?

100% agree.

#57 gets this working.
We may need to fine-tune the various click/control/shift combos, but I'm happy with it.