HostDB mocking

This PR serves to showcase what extensive mocking would look like in a module. It's not perfect (comments/naming/organization could be improved), but it's more-or-less how I envisioned it.
Nearly all of the hostdb's dependencies have been mocked. (The exceptions are: filepath.Join, sync.RWMutex, os.MkdirAll, crypto.Perm, rand.Int, time.Now, types.Currency, and possibly one or two others that I missed. These are pretty inconsequential, since most do not return errors.)
The New/newHostDB pattern in particular has made it very easy to unit test the hostdb's methods. Most of our module tests have fully initialize the modules, meaning they spawn goroutines, write to disk, mine blocks, etc., all just to test a fairly simple function. By testing with newHostDB (which takes all dependencies as parameters and has no side effects) instead of New, it becomes trivial to pass in only the dependencies that will be called and mock them accordingly. See the newly-added tests for examples.
Finally, @VoidingWarranties and I talked in person about modifying this code to reduce duplication of types. The basic premise is to define shared "stub" types for each module in the modules package instead defining "restricted interfaces" internal to each module. As an example, the hostdb now defines the following type:
hdbTransactionPool interface {
    AcceptTransactionSet([]types.Transaction) error
}

which is a parameter to newHostDB. Since modules.TransactionPool implements hdbTransactionPool, it can be passed to newHostDB. And during testing, tests which require AcceptTransactionSet must define a type that implements hdbTransactionPool.
In Jordan's proposal, we would instead define a stub type for the transaction pool like so:
// in modules/transactionpool.go
type TransactionPoolStub struct{}
func (TransactionPoolStub) AcceptTransactionSet([]types.Transaction) error { return nil }
func (TransactionPoolStub) IsStandardTransaction(types.Transaction) error  { return nil }
func (TransactionPoolStub) PurgeTransactionPool()                          { }
// ... etc

then, when testing the hostdb, we would define our testing type like so:
type tpoolTest struct {
    modules.TransactionPoolStub
}
func (tpoolTest) AcceptTransactionSet([]types.Transaction) error {
    // ... test-specific logic ...
}

And finally, we would throw away the hdbTransactionPool type, and have newHostDB take a modules.TransactionPool instead. Since tpoolTest embeds modules.TransactionPoolStub, it inherits its methods and thus implements the full modules.TransactionPool interface. We can then override any methods as needed during testing.
Part of the rationale here is that we will likely want to define shared stubs anyway, for dependencies like time.Sleep and net.Dial.
The code reduction is compelling, but something doesn't sit quite right with me about this approach. I prefer to pass modules the smallest interface they require. Also, embedding types all over the place may also be confusing or error-prone.
I guess my primary objection is that I'm averse to writing generic code until at least two specific cases exist. I feel that we should adopt the pattern in this PR for now, and consider abstracting it only after we've mocked another module to a similar degree. It will be easier to see the parallel and evaluate what makes sense once the code is on the page instead of in our heads.

There is some disagreement as to whether tests should always been run with a "real" HostDB, i.e. one that has goroutines flying around and concurrently mutating state.
My opinion is that unit tests such as TestActiveHosts should not be required to use a "real" HostDB. Unit tests are intended to be run in isolation. Mandating that unit tests spawn the HostDB's scanning threads turns them into integration tests, and introduces extra code (and headaches) for little clear gain.
I see parallels here to dependency mocking; they both make you think carefully about boundaries. When you're in mocking mode, you are drawing a tight circle around your object. Anywhere the circle is broken (e.g. where a dependency is tightly coupled), you use an interface to close the circle. This allows you to test your object in isolation.
With unit tests, the circle is broken by impurity. You want to test the function in as pure an environment as possible, so that you can verify that it produces the correct output without interference from hidden state, concurrent mutations, nondeterminism, etc. When testing pure functions, we don't have to worry about this at all. But where we do have impure functions, it makes sense to control them to the greatest extent possible. For example, ActiveHosts could be converted to a pure function like so:
func (hdb *HostDB) ActiveHosts() []modules.HostSettings {
    hdb.mu.RLock()
    defer hdb.mu.RUnlock()
    return activeHosts(hdb.activeHosts) // implementation omitted
}

Then we could test activeHosts without needing a HostDB at all. But in practice this is pattern is probably infeasible, as it doubles the number of functions we have to test.
However, in TestActiveHosts we can more-or-less replicate this purity by creating a HostDB with only activeHosts, initialized with testing values. While the purity guarantees aren't as strong, they are better than nothing.
So my general recommendation is:

Within reason, try to unit-test every function, and do so in isolation. Use table-driven tests where possible, and use mocking to ensure that dependencies are called correctly.
Note where each function interacts with other functions, and write integration tests for each interaction. In general, these interactions happen when one function writes to a module field and another reads from it -- state is being shared. Test that a write followed by a read produces the correct result. And don't forget that concurrency (namely channels and mutexes) almost always implies shared state.

As an example, hdb.insertHost sends a value down the hdb.scanPool channel, while hdb.threadedProbeHosts reads from it. So we need two unit tests and one integration test. First, we test hdb.insertHost in isolation, manually reading values from hdb.scanPool to confirm that the send is occurring. Then we test hdb.threadedProbeHosts by manually sending values down hdb.scanPool and confirm that it is reading and processing the values correctly. Finally, we write a test that spawns hdb.threadedProbeHosts and calls hdb.insertHost, and confirm that they interacted properly. Now we can say the hdb.insertHost<--->hdb.threadedProbeHosts pairing has been tested to the best of our ability. Other integration tests can include this pairing as part of an even larger test.

My biggest concern is how useful the tests will be. When you test a function, you need to have strong certainty that the values you are testing correctly map to all ranges of values that will be provided to the function in a production environment.
If you consider the state to be part of the input, then you need to make sure that when simplifying the behavior of the state, you are simplifying it such that the tests are still representative of what will happen when the state is not simplified. I am worried that certain types of mocking over-simplify things, and therefore reduce the utility of the test.
I'm not making a general statement about mocking or that unit testing is bad. I'm just stating that I've seen unit tests which I would estimate are nearly worthless, and I want to be sure that we are writing unit tests which have actual utility.

I agree that integration tests should try to emulate real-world conditions. That is absolutely important. But I think that stipulation makes less sense for unit tests, for the reasons given above.
Perhaps our strategy should be to write integration tests first, and unit tests only as a means of improving test coverage or checking for bugs revealed by the integration tests.
Another thing to consider is that integration tests can be just as fallible as unit tests when it comes to simulating reality. As you pointed out earlier, full code coverage does not imply absence of bugs. Just like unit tests, integration tests can wind up testing only a subset of the inputs that are possible in real-world conditions. The only difference is that with an integration test, you are guaranteed to be testing at least one code path that will occur in production.

One thing I definitely want to see is the strict use of the prefixes TestUnit and TestIntegration in the tests for all packages where mocking and 100% coverage has been achieved. I think it's important to be able to see what code isn't touched by integration tests, so that a judgement call can be made deciding whether more integration tests are necessary or whether the unit tests are being thorough enough.
A good example would be uploading. If you have 100% code coverage on uploading but none of it is an integration test with the renter, I would feel like an integration test would be necessary to add. But you might not realize that you are missing that coverage because the unit tests are saying that all the code is reached. OTOH, if you see that integration testing is covering all code except some of the unlikely error handling, it's probably good enough to accept that only unit tests are exploring the error handling code.

Most concerns addressed. There are a few unfortunate bits left (like walletShim and modWallet), but they can be resolved later. I want to move on to more pressing concerns, so if you have any final comments, they should be "must-haves."

So, should drop 2 TODOs and should change the need for
modWallet github.com/bla/bla/bla

doing so would require renaming the wallet type. What is a better name?


add letsencrypt capabilities

Need more tests but here it what it will look like.
Let's encrypt only for in production environment so I force it to production environment only, users that was https for development mode will have to create self signed certificate and it will be used for all env except production.
Let me know what you think about it. Also if you can make some test with this branch see if letsencrypt is working. I'll do some on my side but I'm quite new to it

This looks pretty good! Here are my thoughts. We are ending into a highly opinionated area here.  While greenlock is a good thing, I'm not a 100% sure every dev will agree with it.  There are also a bunch of plugins that this accepts, that would be a pain to add support for each.  My preferences would be to be able to load this is externally.  Perhaps we allow trailpack-express to accept a server parameter or loader instead of hard coding it in?

You're right, my idea was to try to keep it really simple for the users, but I guess with a plugin system it could be as simple a this too. Let's agree on some interface for the plugins in order to achieve the same things then. For me the best way is to have under web.js a field externalConfig like:
externalConfig: (trailsApp, expressApp) => {
   ...
}

But the problem of that will be that you need to duplicate the logic under lib.Server.start in case of non production env
Any other idea ?

Well, we already have config/env, so I'd say that it could be done there?

@scott-wyatt so basically here nothing more than this right ?

Yeah, that's pretty much it :-)

So if it's ok let's merge this and create a trailpack-letsencrypt then :)


Displaying cached data?

I'm noticing a weird behavior but I can't seem to nail down the cause.
I've got two sheets I'm displaying. Each sheet is a kids homework for the week.
In the config file, I have them as two different modules displaying in different areas of the screen.
They were working great until this week when one of them decided to stop updating and continues to display last week's data despite the Google Sheet being updated.
But wait, it gets weirder. I figured it was just a network hiccup, and reloaded the page and all looked well. Then at some point, it reverts to the old data again! It did this today. I verified the URL in the config file, reloaded, and then an hour or so later noticed it was showing old data again.
Side notes that may or may not be relevant- Each Google sheet contains a Sheet 1 & Sheet 2.
Sheet 1 is populated from various ImportHTML statements
Sheet 2 is populated from some transpose statements to pull from Sheet 1.

@gonzonia I can't seem to replicate your issue. If the data in your sheet(s) is not private / sensitive, do you mind posting the links to the sheets with what you expect to see and what you're actually seeing?

At some point this resolved. I'm going to close it. If I notice again I'll re-open and try to post more info.


Downloading collections

[x] basic streaming zip download functionality
[x] handle recursive collections
[x] handle double-upped resources (don't have 1+ of the same resource)
[x] download in sprite2 format
[x] download in sb2 format
[x] gui for selecting how to download (sprite or project; etc)
[x] support resume 🎉

Yay all items complete


Resaved all if at least one is installed

Hello.
As i see, when i call Setting::get they loaded all settins from database
select * from `settings
If I call once Settings::set / Settings::forget etc (any changes in settings) all settins will resaved in database. For example if i have 200 settings key in database, and change 1 key, it's generate 200 updates query to database. This creates an extra useless load on the database.
code:
Setting::set('debug',TRUE);
Setting::save();

logs:
$ cat www.query-2019-07-16.log | grep settings | grep select | wc -l
2
$ cat www.query-2019-07-16.log | grep settings | grep -o '\[update settings set value = ? where key = ?\]'  | wc -l
205

I think you need to track only the changed keys, and only cause updates for them.

I fixed this issue in my code, by adding 4 lines in code
$ diff SettingStore.php SettingStoreFix.php
35c35,43
< 	/**
---
>     /**
>      * Updated settings
>      *
>      * @var array
>      */
>     protected $updated = [];
>
>
>     /**
46d53
<
71a79
>
77a86
>                 array_push($this->updated, $k);
80a90
>             array_push($this->updated, $key);

$ diff DatabaseSettingStore.php DatabaseSettingStoreFix.php
173a174
> 		    if ( FALSE == in_array($key, $this->updated) ) { continue; }

Hi @rayderua thanks for spotting, do you mind putting in a PR?

update previous comment patch for 0.8.3
What is PR?
in any case, if that fixes issue, then I don't mind :)

May be this issue is a duplicate for
https://github.com/anlutro/laravel-settings/issues/100


Add client extension projects to coupon-with-object-actions sample

Fixed https://github.com/liferay/liferay-localdev/pull/51


Fixed Grom Hellscream soundset path

On 1.36, "Hellscream" doesn't seem to exist.

I didn't look for the reason as to why, but I remember some odd change about Hellscream name for reforged, maybe it's related?
https://us.forums.blizzard.com/en/warcraft3/t/why-do-we-hate-grom-now/14474

Thanks


Use the msgspec library for IMUDataPacket classes

Closes #7
To be merged after #6 and testing running/installation/building on the raspberry pi

oh, but we had to check if it works on the pi..

do you mean if the library works? or just if our code works

yeah I guess it would have been better to test if it worked before merging it, but I wanted us to get consolidated. We can work out all the bugs and kinks on tuesday


CIF-1364: Added class for results count

Description
New CSS class for product count in productcollection
Related Issue
CIF-1364
Motivation and Context
Need to show product count in PLP and search page
How Has This Been Tested?
Manually
Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes and the overall coverage did not decrease.
[x] All unit tests pass on CircleCi.
[x] I ran all tests locally and they pass.

Sorry just saw it's like that in the ticket ... hmmm, well, then let's have it there ... :-(


Create test1

Collaborator: Review for this pull request was created. Link to review

Collaborator cannot find remote user mapping for pull request creator, review created by system administrator


How can i get list of galleries?

Is there any way to get list of all galleries?

Yeah you can…

Create a new page from the CMS tab. Now under Markup put

{% for gallery in galleries %}

{{ gallery.name }}

{% else %}

No galleries available

{% endfor %}

And under Code put
function onStart()
{
    $galleries = new \Mohsin\MagnificGallery\Models\Gallery;
    $galleries = $galleries -> all();
    $this['galleries'] = $galleries;
}

No need to include any component, this directly tells the system to create a Gallery Model’s instance. So, you can use this to show a list of galleries if you need. Although, this isn’t really what the plugin was designed for. Let me know if you have any other queries. I’m closing this now.


if directories does not exists, create it

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

One or more commits are not associated with a GitHub user. The email used in the Git commit must match a verified email address in a GitHub user profile. For more information, please see https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/

Regards,
CLA GitHub bot

fixed my email.

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

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

Regards,
CLA GitHub bot

fix email strike 2

This looks good to me! happy to merge when this CLA situation is fixed

strike 3

Fixes #102

thanks a lot!


nixos/network-interfaces: reflect negative settings of proxyARP

Currently, setting proxyARP to true enables proxy_arp, but setting it to false doesn't disable it. This is surprising and stateful.

@bluescreen303 @kfiz

I completely agree.
Good catch.

Good catch, indeed. Fine by me.


When git commit from the worktree, neogit window shows the main directory

Description
When you open vim with the path of the worktree and make a commit, neogit window switches to the state of the initial folder (more details in steps to reproduce).

Neovim version
NVIM v0.8.1
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by<EMAIL_ADDRESS>Features: +acl +iconv +tui
Operating system and version
macOS 13.0
Steps to reproduce
Example with the current repo:

git clone /Users/m1/IdeaProjects/vim/neogit/README.md;
cd neogit;
git worktree add -b emergency-fix ../temp master;
vim ../temp/README.md;
Make changes to the file, open neogit and make a commit;
In the neogit window you don't see your commit.

One thing I noticed. It works if you do it like this:

same as above
same as above
same as above
vim .;
Open directory for the worktree. Inside vim: :e ../temp/README.md;
Set current directory as root: :cd %:p:h<CR>;
Make changes to the file, open neogit and make a commit;

Expected behavior
Neogit state should not be switched.
Actual behavior
Neogit state switched.
Minimal config
any config

@D00mch , this is happening for me too but in a different way.
For me it shows the branch from where I have created the worktree after committing in the worktree's branch and then I will close the neogit popup and then open neogit again. Then it shows the branch in which I have committed.
It started happening recently around 1 month. Not sure though.


wandb 0.8.16 is incompatible with graphql-core==0.3.0

Error: version conflict, wandb cannot start with graphql-core==0.3.0:
wandb --version
Traceback (most recent call last):
  File "/usr/local/bin/wandb", line 5, in <module>
    from wandb.cli import cli
  File "/usr/local/lib/python3.6/dist-packages/wandb/__init__.py", line 45, in <module>
    from wandb.apis import InternalApi, PublicApi, CommError
  File "/usr/local/lib/python3.6/dist-packages/wandb/apis/__init__.py", line 7, in <module>
    from gql.client import RetryError
  File "/usr/local/lib/python3.6/dist-packages/gql/__init__.py", line 2, in <module>
    from .client import Client
  File "/usr/local/lib/python3.6/dist-packages/gql/client.py", line 3, in <module>
    from graphql import parse, introspection_query, build_ast_schema, build_client_schema
ImportError: cannot import name 'introspection_query'

wandb --version && python --version && uname

Weights and Biases version:

# pip show wandb
Name: wandb
Version: 0.8.16
Summary: A CLI and library for interacting with the Weights and Biases API.
Home-page: https://github.com/wandb/client
Author: Weights & Biases
Author-email<EMAIL_ADDRESS>License: MIT license
Location: /usr/local/lib/python3.6/dist-packages
Requires: shortuuid, nvidia-ml-py3, watchdog, GitPython, sentry-sdk, docker-pycreds, six, subprocess32, python-dateutil, gql, Click, configparser, requests, psutil
Required-by:

Python version:

# python --version
Python 3.6.8

Operating System:

# uname -a
Linux job-4ab92e1c-250c-42bf-9d64-2f610d7a6c64 4.14.137+ #1 SMP Thu Aug 8 02:47:02 PDT 2019 x86_64 x86_64 x86_64 GNU/Linux

Description
Dependencies:
wandb requires gql requires graphql-core:
# pip show wandb
Name: wandb
Version: 0.8.16
Summary: A CLI and library for interacting with the Weights and Biases API.
Home-page: https://github.com/wandb/client
Author: Weights & Biases
Author-email<EMAIL_ADDRESS>License: MIT license
Location: /usr/local/lib/python3.6/dist-packages
Requires: six, watchdog, configparser, shortuuid, docker-pycreds, requests, nvidia-ml-py3, python-dateutil, subprocess32, sentry-sdk, GitPython, psutil, Click, gql
Required-by:

# pip show gql
Name: gql
Version: 0.1.0
Summary: GraphQL client for Python
Home-page: https://github.com/graphql-python/gql
Author: Syrus Akbary
Author-email<EMAIL_ADDRESS>License: MIT
Location: /usr/local/lib/python3.6/dist-packages
Requires: six, promise, graphql-core
Required-by: wandb

# pip show graphql-core
Name: graphql-core
Version: 3.0.0
Summary: GraphQL implementation for Python, a port of GraphQL.js, the JavaScript reference implementation for GraphQL.
Home-page: https://github.com/graphql-python/graphql-core-next
Author: Christoph Zwerschke
Author-email<EMAIL_ADDRESS>License: MIT license
Location: /usr/local/lib/python3.6/dist-packages
Requires:
Required-by: gql

Note, wandb==0.8.16 works well together with gql==0.1.0 and graphql-core==2.2.1.
Solution:
If wandb really needs graphql-core<0.3.0, I propose to add this requirement explicitly to wandb's setup.py in addition to exitsting dependency on gql as gql itself does not have this restriction.

Thanks for reporting!  We saw this as well over the weekend.  We have a new release that should go out today or tomorrow, you can install with: pip install --upgrade git+git://github.com/wandb/client.git@task/pin_graphql#egg=wandb

We just pushed a fix for this in 0.8.17.


when i download vs code on mac and open it the black screen open and stay

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

Does this issue occur when all extensions are disabled?: Yes
VS Code Version: 1.78.2
OS Version: mocOS ventura 13.2.1
Steps to Reproduce:
install vs code from official website
open it
lanuch black screen

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


CUDNN 6.1 is not available for download on https://developer.nvidia.com/rdp/cudnn-archive

CUDNN 6.1 (version listed in README.md) is not available for download on https://developer.nvidia.com/rdp/cudnn-archive
Is there another version I can use that would work?

I have just tried with cuDNN v6.0 (April 27, 2017), for CUDA 8.0 and the unit tests pass. I will update REAME.md because 6.1 doens't seem to appear on nvidia website.
I need to update to Managed Cuda 9.1 / cuDNN 7.1

Thank you. I will try cuDNN v6.0 (April 27, 2017), for CUDA 8.0.

I think it is working... but not sure if I am comparing things right.
MnistDemo vs MnistDemo.GPU - GPU is getting through more examples but is converging slower than CPU version.
GPU
Epoch #7
Loss: 2.136335 Train accuracy: 28% Test accuracy: 38%
Example seen: 70656 Fwd: 0.27ms Bckw: 0.01ms Updt: 0ms

CPU
Epoch #1
Loss: 1.70971740924065 Train accuracy: 51% Test accuracy: 53%
Example seen: 680 Fwd: 16.44ms Bckw: 63.97ms

There are some small differences between MnistDemo CPU and GPU.

float vs double
different batch size (20 vs 1024)
GPU version doesn't use L2 regularization and Momentum (SgdTrainer)

You should get similar results If you set all those parameters to the same values. GPU should be faster if batchsize is big enough.


[Backport release-v0.2] Update awesome docs

Description
Backport of #20 to release-v0.2.
relates to
original author: @afritzler

Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin backport-20-to-release-v0.2
git worktree add --checkout .worktree/backport-20-to-release-v0.2 backport-20-to-release-v0.2
cd .worktree/backport-20-to-release-v0.2
git reset --hard HEAD^
git cherry-pick -x 1edb36d5bc5b245b3eb4431aa71593b74ce7176a
git push --force-with-lease


corrected the alignment of the nav bar

@PriyaGhosal
issue closes #1831
i have corrected the nav bar .
before:

after:

can you please merge this ?

hey @PriyaGhosal
can you please review this ?


Master rework

create-react-app integration complete

still a couple of problems we need to fix:

indent level is not working like it should. maybe something to do with how webpack includes CSS v how the build setup from punchcardjs dealt with it;
source maps don't work for me, they only go back as far as the (compiled) ES6, not the written TS

anyway still merging.


Generate a benchmark dataset for the features module

Steps:

Modify the ICA-AROMA code, making sure all intermediate results are not wiped out.
Run that derived code from ICA-AROMA locally with FSL installed
Create a zipfile with intermediate results, including all the masks generated along the way as well as the conversion (resampling) of the initial BOLD in standard space. Make sure all the masks that the feature extraction requires are all kept in the standard space. Include in the zip the dataframe of features you obtain (could be done together with #31)
Write unit tests of the features module using these masks.

It would be interesting that this dataset and #29 were the same so that when we take the leap of working on native space it is easy to adjust the new thresholds to obtain close results.

I would stick to one single dataset for all tests if possible. One in native space, one in MNI space.

I think @tsalo would very much appreciate having this ready for when he starts hacking today. Unfortunately, I don't have the bandwidth to do it today - @eurunuela do you think you could find someone to get this done soonish?

I can try to do it myself. I'll see if I find enough time for this.

Yup, considering the momentum @tsalo is showing, I think anything we can do to cater to him will get us a long way.

I completely agree! He's on 🔥 !


Revert "Fix compilation warnings in extended test suite"

Reverts apple/swift-corelibs-libdispatch#290
Sorry to revert, but this breaks Swift CI incremental bots and PR testing (so it's blocking all other work)
swift-corelibs-libdispatch/os/linux_base.h:16:10: fatal error: 'config/config_ac.h' file not found
#include <config/config_ac.h>
^
https://ci.swift.org/job/oss-swift-incremental-RA-linux-ubuntu-16_04/4182/console

This was the wrong PR to revert. I'll manually reapply the compilation warning fixes.
This is the PR I meant to revert: https://github.com/apple/swift-corelibs-libdispatch/pull/286


Create bone transform for Figure

I want to have an optional option for Figure to create a list of children transforms that follow the figure animation joints.

#37
Fixed in v0.0.6


add new composer global binary path

@robbyrussell fixed by https://github.com/robbyrussell/oh-my-zsh/commit/4fba92e04fa9b62b2259abc45eb92ca6a74f1639


The variable number in the decomposition does not match the number in the model

Dear support team,
I use pyGCGopt 0.3.1, GCG 3.5.5, SCIP 8.1.0, pySCIPopt 4.4.0 to detect the following instance. The instance has 4200 variables (200 bin, 0 int, 0 impl, 4000 cont) and 7209 constraints, but I find the decomposition only has 3209 variables.
The instance
Uploading instance.txt…
(you may want to change the .txt to .mps before using this instance)
starting detection
 Consclassifier "nonzeros" yields a classification with 32  different constraint classes
 Consclassifier "constypes" yields a classification with 1 different constraint classes
 Consclassifier "constypes according to miplib" yields a classification with 2 different constraint classes
 Consclassifier "gamsdomain" yields a classification with 1  different constraint classes
 Conspartition "gamsdomain" is not considered since it offers the same structure as "constypes" conspartition
 Consclassifier "gamssymbols" yields a classification with 1  different constraint classes
 Conspartition "gamssymbols" is not considered since it offers the same structure as "constypes" conspartition
 Varclassifier "gamsdomain" yields a classification with 1  different variable classes
 Varclassifier "gamssymbols" yields a classification with 1  different variable classes
 Varpartition "gamssymbols" is not considered since it offers the same structure as "gamsdomain"
 Varclassifier "vartypes" yields a classification with 2 different variable classes
 Varclassifier "varobjvals" yields a classification with 2 different variable classes
 Varpartition "varobjvals" is not considered since it offers the same structure as "vartypes"
 Varclassifier "varobjvalsigns" yields a classification with 2 different variable classes
 Varpartition "varobjvalsigns" is not considered since it offers the same structure as "vartypes"
 the current consclass distribution includes 32 classes but only 18 are allowed for GCGconshdlrDecompCalcCandidatesNBlocks()
start finding decompositions for original problem!
 in dec_consclass: there are 3 different constraint classes
  the current consclass distribution includes 32 classes but only 9 are allowed for propagatePartialdec() of cons class detector
 the current constraint classifier "constypes" consists of 1 different classes
  the current constraint classifier "constypes according to miplib" consists of 2 different classes
 dec_consclass found 4 new partialdecs
dec_densemasterconss found 1 new partialdec
dec_neighborhoodmaster found 1 new partialdec
POSTPROCESSING of decompositions. Added 1 new decomps.
Found 10 finished decompositions.

Detection Time: 4.03

The codes are as follows
instance = "instance.mps"
m = gcg.Model()
m.readProblem(instance)
m.detect()
decomps = m.listDecompositions()
d = decomps[0]
print(d.getNVars())

All the best
Regards

Hallo @haoyangliu123,
Unfortunately, I am not able to download the instance. Can you please send me the instance to the email<EMAIL_ADDRESS>.
Best wishes,
Jurgen

This is not a bug. This decomposition is based on the model after presolve and the presolved model has exactly 3209 variables.


Episode idea: Service Url

Describe a topic you want to learn (required)
There are multiple options to set a
Why do you want to learn this topic? (required)

Topic lookup logic
What is the difference between web service url and broker service url?
How to configure the service url?
DNS vs Multi-hosts service url

Reference (optional)

Fixed by #16


About the converstion with own dataset

I tried to convert my own dataset with convert.py but it failed. Specifically, when preparing  100-500 images from any direction, it produced distorted images in the image directory. Is there any good way to correctly convert our own dataset for gaussian-grouping format?

Hi, convert.py is used in original Gaussian Splatting for converting custom dataset. Its usage of COLMAP only supports undistorted datasets (PINHOLE or SIMPLE_PINHOLE cameras).
You may refer to some of the issues in Gaussian Splatting like this one, or check the dataset and some usage of COLMAP.


Getting this error while configuring WVD IN AZURE PORTAL POWERSHELL Add-RdsAccount: The term 'Add-RdsAccount' is not recognized as a name of a cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[ ] The relevant Azure service or technology.
[ ] A link to the published documentation article that you have feedback about.
[ ] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[ ] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[ ] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/forums/34192--general-feedback.

@azureabhay Thanks for reaching out!
In general, you would see that error if the cmdlet's module is not installed / imported.
In your case, the cmdlet Add-RdsAccount is not recognized probably because it's module WindowsVirtualDesktopPowershell is not installed / imported. So I would recommend you to make sure the module is installed and then try out the cmdlet.
Also, FYI, this azure-docs repository deals about feedback related to particular Azure document page (like correcting doc bugs, doc enhancements, product issues related to wrong doc instructions, etc.) so going forward please share the Azure document link to which your feedback is related. Also, please elaborate what's your feedback regarding an Azure document.
If you need help in resolving a product related issue (but not the document) then I suggest you to ask the question in public forums like Microsoft Q&A, etc. but not here in this azure-docs repository. Hope this clarifies.


[Forge 1.20.1] Config file for the mod

Would be nice if we could configure the output rate of the Frost Dynamo and stat boosts each augment gives.
KubeJS support for the Frost Dynamo would be great as well.
I also found that the Frost Dynamo's break time is much longer than standard Dynamos.

Any reason why you won't support KubeJS? What about CraftTweaker?
I just want a way to customize the dynamos & whatnot.

custom dynamo fuel can be done with datapack

custom dynamo fuel can be done with datapack

ok, but how? What's the folder structure? Is there an example file anywhere?!
not to be rude or anything, but i'm so damn tired of mod devs going "just use a datapack lulz" and giving you 0 direction for wtf you're meant to do with it. At least link to a file and have a line of text listing the folder structure the datapack should use.

https://github.com/Mrthomas20121-Mods/ThermalExtra/tree/1.20.1/src/main/resources/data/thermal_extra/recipes/machine/dynamo/frost


Feature: Weak Points

Description
Weak points are areas which a line can be crossed through to kill the Kaiju.

The student will use the weak points to kill the Kaiju and see where the function should go

Functional Requirements

[ ] It should be able to draw a circle around the weak point to signal the crossable area
[ ] It should be able to fire a UnityEvent when the weak point is hit

Acceptance Criteria

[ ] There should be a weak point prefab

Dependencies

#182
#183

Definition of Done

[ ] Follows the naming conventions
[ ] Follows the coding conventions
[ ] Meets the acceptance criteria
[ ] Pull request has been opened

Need ROBB-E and function line controller


[Feature] Ollama vision support

Describe the feature
Hello ! I saw the outperforming results this models does in OCR with the demo on hugface, I wonder if it will be possible to add it  to the vision API of ollama ? Only text version is available for now https://ollama.com/internlm/internlm2.5
Thanks you, if not could you explain why I am curious :)
Will you implement it?

[ ] I would like to implement this feature and create a PR!

InternLM 2.5 does not support vision input, while InternLM-Xcomposer and InternVL supports. You may raise issue in their corresponding projects.
https://github.com/OpenGVLab/InternVL
https://github.com/InternLM/InternLM-XComposer


Deployment fails because not enough disk space

info
when i deploy my code to a deploymentslot the deployment fails often because there is not enough space on the disc
I can manually delete the old zip-packages in C:/home/data/SitePackages/*.zip and then i can run my github-action again. now the deployment works.
is it possible to delete the old packages with the deployment?
or is there a better solution to my problem?
code
this is my github action
name: Build and deploy Node.js project to Azure Function App

on:
  push:
    branches:
      - main
  workflow_dispatch:

# CONFIGURATION
# For help, go to https://github.com/Azure/Actions
#
# 1. Set up the following secrets in your repository:
#   AZURE_FUNCTIONAPP_PUBLISH_PROFILE
#
# 2. Change these variables for your configuration:
env:
  AZURE_FUNCTIONAPP_NAME: 'apppname'   # set this to your function app name on Azure
  AZURE_FUNCTIONAPP_SLOTNAME: 'dev'   # set this to your function app name on Azure
  AZURE_FUNCTIONAPP_PACKAGE_PATH: '.'       # set this to the path to your function app project, defaults to the repository root
  NODE_VERSION: '18.x'                      # set this to the node version to use (e.g. '8.x', '10.x', '12.x')

jobs:
  build-and-deploy:
    runs-on: windows-latest
    environment: dev
    steps:
    - name: 'Checkout GitHub Action'
      uses: actions/checkout@v3

    - name: Setup Node ${{ env.NODE_VERSION }} Environment
      uses: actions/setup-node@v3
      with:
        node-version: ${{ env.NODE_VERSION }}

    - name: 'Resolve Project Dependencies Using Npm'
      shell: pwsh
      run: |
        pushd './${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}'
        npm install
        npm run build --if-present
        npm run test --if-present
        popd

    - name: 'Run Azure Functions Action'
      uses: Azure/functions-action@v1
      id: fa
      with:
        app-name: ${{ env.AZURE_FUNCTIONAPP_NAME }}
        slot-name: ${{ env.AZURE_FUNCTIONAPP_SLOTNAME }}
        package: ${{ env.AZURE_FUNCTIONAPP_PACKAGE_PATH }}
        publish-profile: ${{ secrets.AZUREAPPSERVICE_PUBLISHPROFILE }}
        respect-funcignore: true

LogFile in Kudu
the last entry after a failed deployment says this:
There is not enough space on the disk.
...
<EventData>
<Data>3006</Data>
<Data>A parser error has occurred.</Data>
<Data>12/6/2022 1:19:01 AM</Data>
<Data>12/6/2022 12:19:01 AM</Data>
...
<Data>Full</Data>
<Data>/</Data>
<Data>
C:\Program Files (x86)\SiteExtensions\Kudu\99.41021.5971\
</Data>
<Data>10-30-2-204</Data>
<Data/>
<Data>5880</Data>
<Data>w3wp.exe</Data>
<Data>IIS APPPOOL\gpjoperations</Data>
<Data>HttpParseException</Data>
<Data>
There is not enough space on the disk. at System.Web.Compilation.AssemblyBuilder.AddBuildProvider(BuildProvider buildProvider) at System.Web.Compilation.BuildProvidersCompiler.ProcessBuildProviders() at System.Web.Compilation.BuildProvidersCompiler.PerformBuild() at System.Web.Compilation.BuildManager.CompileWebFile(VirtualPath virtualPath) at System.Web.Compilation.BuildManager.GetVPathBuildResultInternal(VirtualPath virtualPath, Boolean noBuild, Boolean allowCrossApp, Boolean allowBuildInPrecompile, Boolean throwIfNotFound, Boolean ensureIsUpToDate) at System.Web.Compilation.BuildManager.GetVPathBuildResultWithNoAssert(HttpContext context, VirtualPath virtualPath, Boolean noBuild, Boolean allowCrossApp, Boolean allowBuildInPrecompile, Boolean throwIfNotFound, Boolean ensureIsUpToDate) at System.Web.Compilation.BuildManager.GetVirtualPathObjectFactory(VirtualPath virtualPath, HttpContext context, Boolean allowCrossApp, Boolean throwIfNotFound) at System.Web.Compilation.BuildManager.CreateInstanceFromVirtualPath(VirtualPath virtualPath, Type requiredBaseType, HttpContext context, Boolean allowCrossApp) at System.Web.WebPages.BuildManagerWrapper.CreateInstanceOfType[T](String virtualPath) at System.Web.WebPages.VirtualPathFactoryExtensions.CreateInstance[T](IVirtualPathFactory factory, String virtualPath) at System.Web.WebPages.VirtualPathFactoryManager.CreateInstanceOfType[T](String virtualPath) at System.Web.WebPages.VirtualPathFactoryExtensions.CreateInstance[T](IVirtualPathFactory factory, String virtualPath) at System.Web.WebPages.WebPageHttpHandler.CreateFromVirtualPath(String virtualPath, IVirtualPathFactory virtualPathFactory) at System.Web.WebPages.WebPageRoute.DoPostResolveRequestCache(HttpContextBase context) at System.Web.WebPages.WebPageHttpModule.OnApplicationPostResolveRequestCache(Object sender, EventArgs e) at System.Web.HttpApplication.SyncEventExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute() at System.Web.HttpApplication.ExecuteStepImpl(IExecutionStep step) at System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously) There is not enough space on the disk. at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath) at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost) at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share) at System.Web.Compilation.AssemblyBuilder.CreateCodeFileWithAssert(String generatedFilePath) at System.Web.Compilation.AssemblyBuilder.CreateCodeFile(BuildProvider buildProvider, String& filename) at System.Web.Compilation.AssemblyBuilder.AddCodeCompileUnit(BuildProvider buildProvider, CodeCompileUnit compileUnit) at System.Web.WebPages.Razor.AssemblyBuilderWrapper.AddCodeCompileUnit(BuildProvider buildProvider, CodeCompileUnit compileUnit) at System.Web.WebPages.Razor.RazorBuildProvider.GenerateCodeCore(IAssemblyBuilder assemblyBuilder) at System.Web.WebPages.Razor.RazorBuildProvider.GenerateCode(AssemblyBuilder assemblyBuilder) at System.Web.Compilation.AssemblyBuilder.AddBuildProvider(BuildProvider buildProvider)
</Data>
<Data>...
</Data>
<Data>/ProcessExplorer/</Data>
<Data>...</Data>
<Data/>
<Data>False</Data>
<Data/>
<Data>IIS APPPOOL\gpjoperations</Data>
<Data>32</Data>
<Data>IIS APPPOOL\gpjoperations</Data>
<Data>False</Data>
<Data>
</Data>
</EventData>
...

I ran into the same issue... you can clearly see it in VSCode:

You don't see it so easily when deploying via a github action, however after applying the setting:

Redeploying the GH workflow, worked without any other changes.


webpackPreload not working at all

What is the current behavior?
I am experiencing issues while using webpackPreload.
No script with rel="preload" is added to the <head>.
Steps to reproduce
const HtmlWebpackPlugin = require('html-webpack-plugin');

module.exports = {
  mode: 'none',
  plugins: [
      new HtmlWebpackPlugin({template: './src/index.html'}),
  ]
};

export function divide(a, b) {
    return a / b;
}

document.addEventListener("DOMContentLoaded", () => {
    const button = document.querySelector('#divideButton');
    button.addEventListener('click', () => {
        import(
            `./utilities/divide`
            /* webpackPreload: true */
            /* webpackChunkName: "utilities" */
            )
            .then(divideModule => {
                console.log(divideModule.divide(6, 3)); // 2
            })
    });
});

<!DOCTYPE html>
<html>
    <head>
    </head>
    <body>
        <button id="divideButton">Click</button>
    </body>
</html>

I am using<EMAIL_ADDRESS>What is the expected behavior?
I expect the <link rel="preload" as="script" href="utilities.js"> to be added.
Other relevant information:
webpack version:  4.16.5
Node.js version: 8.11.3
Operating System: Ubuntu 18.04

I think this is not yet supported by the html-plugin. You could file an issue there.
Anyway using preload in your example is incorrect. It should use prefetch instead.
Note that prefetch doesn't require html-plugin support and should work.

@sokra The docs do not mention the need to use html-plugin for preload.
Prefetch is working without problems.
https://webpack.js.org/guides/code-splitting/#prefetching-preloading-modules

That's true. It's mentioned here: https://medium.com/webpack/link-rel-prefetch-preload-in-webpack-51a52358f84c in the FAQ
Please send a PR to the docs adding it.

@sokra "When prefetch/preload is used in import()s in a entry chunk the html generation is resposible for adding the  tags to the HTML. "
The prefetch works without problems for me though. Is it connected to that PR?
https://github.com/webpack/webpack/pull/7444

Is it connected to that PR?

Yes. This PR allows to use prefetch without need to generate <link> tags in the HTML. It will be injected by the js bundle instead. This is possible because prefetching starts after the initial load. Preloading starts in parallel to the js bundle, so this is not possible for preload.

@sokra I understand. I will work on a PR to the docs. Thanks ;)

@webpack-bot move to webpack/webpack.js.org

@sokra The docs do not mention the need to use html-plugin for preload.
Prefetch is working without problems.
https://webpack.js.org/guides/code-splitting/#prefetching-preloading-modules

That's true. It's mentioned here: https://medium.com/webpack/link-rel-prefetch-preload-in-webpack-51a52358f84c in the FAQ
Please send a PR to the docs adding it.


Error Building with CMake

tl;dr:
the library png16 doesn't get found.
I tried to install libpng16-dev, but that resulted in a linking error
prerequisites:

Ubuntu 16.04
The package tensorflow-cpp-generic_1.13.1fix2.git20190225.6612da8951.ppa1_amd64.deb
Noted dependencies (libdouble-conversion-dev libfarmhash-dev libre2-dev libgif-dev libpng-dev libsqlite3-dev libsnappy-dev liblmdb-dev libicu-dev libjsoncpp-dev)
How i installed the package:

sudo apt install ./tensorflow-cpp-generic_1.13.1fix2.git20190225.6612da8951.ppa1_amd64.deb
This gives an dependency error (libpng-dev) , so I installed it (Apt notes that it installed libpng12-dev).
Then i installed the package again, but this time gives the error:
N: Can't drop privileges for downloading as file '/home/simon/Downloads/tensorflow-cpp-generic_1.13.1fix2.git20190225.6612da8951.ppa1_amd64.deb' couldn't be accessed by user '_apt'. - pkgAcquire::Run (13: Permission denied)
but the files get installed.
Project setup:
example.cpp:
#include <tensorflow/core/protobuf/meta_graph.pb.h>
#include <tensorflow/core/public/session.h>

int main() { }

CMakeLists.txt;
cmake_minimum_required(VERSION 3.5)

set (CMAKE_CXX_STANDARD 11)

set(CMAKE_PREFIX_PATH /usr/lib/cmake/tensorflow-cpp/ )

FIND_PACKAGE(TensorFlowCpp REQUIRED PATHS /usr/lib/cmake/tensorflow-cpp/)

INCLUDE_DIRECTORIES(${TENSORFLOWCPP_INCLUDE_DIRS})

ADD_EXECUTABLE(example example.cpp)

TARGET_LINK_LIBRARIES(example ${TENSORFLOWCPP_LIBRARIES})

Error:
Then building the dummy project This error appears:
[ 50%] Building CXX object CMakeFiles/example.dir/example.cpp.o
[100%] Linking CXX executable example
/usr/bin/ld: cannot find -lpng16
collect2: error: ld returned 1 exit status
CMakeFiles/example.dir/build.make:97: recipe for target 'example' failed
make[2]: *** [example] Error 1
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/example.dir/all' failed
make[1]: *** [CMakeFiles/example.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2

Informations:
ldconfig -p | grep png

	libpng12.so.0 (libc6,x86-64) => /lib/x86_64-linux-gnu/libpng12.so.0
	libpng12.so.0 (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libpng12.so.0
	libpng12.so.0 (libc6,x86-64) => /usr/lib/libpng12.so.0
	libpng12.so (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libpng12.so

apt list --installed | grep png

WARNING: apt does not have a stable CLI interface. Use with caution in scripts.

libpng12-0/xenial-updates,xenial-security,now 1.2.54-1ubuntu1.1 amd64 [installed]
libpng12-dev/xenial-updates,xenial-security,now 1.2.54-1ubuntu1.1 amd64 [installed]

Tried solutions:
Since the png16 library can't be found and ubuntu uses libpng12-dev package for the libpng package, i tried to install the libpng16-dev package instead of the pnglib-dev package.
Then installing the libpng16-dev packge, the tensorflow and libpng12-dev package gets removed( :smile: )
So i installed the tensorflow package again, without the libpng-dev dependency.
sudo dpkg -i --ignore-depends=libpng-dev tensorflow-cpp-generic_1.13.1fix2.git20190225.6612da8951.ppa1_amd64.deb
Then building now, the following linking error appears.
Error:
simon:~/Downloads$ make
[ 50%] Linking CXX executable example
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `ucnv_setToUCallBack_60'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::UnicodeString::append(int)'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::UnicodeStringAppendable::~UnicodeStringAppendable()'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::UnicodeString::~UnicodeString()'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::UnicodeString::toUTF8(icu_60::ByteSink&) const'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to<EMAIL_ADDRESS>/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `ucnv_close_60'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `ucnv_reset_60'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `std::thread::_M_start_thread(std::unique_ptr<std::thread::_State, std::default_delete<std::thread::_State> >, void<EMAIL_ADDRESS>/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::StringCharacterIterator::~StringCharacterIterator()'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::ErrorCode::~ErrorCode()'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::UCharCharacterIterator::next32PostInc()'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::UCharCharacterIterator::hasNext()'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `ucnv_getNextUChar_60'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to<EMAIL_ADDRESS>/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `ucnv_open_60'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `vtable for icu_60::UnicodeString'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::ByteSink::GetAppendBuffer(int, int, char*, int, int*)'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `UCNV_TO_U_CALLBACK_SUBSTITUTE_60'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `typeinfo for icu_60::ByteSink'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `typeinfo for<EMAIL_ADDRESS>/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::ErrorCode::reset()'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `vtable for icu_60::ErrorCode'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `vtable for icu_60::UnicodeStringAppendable'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::UMemory::operator delete(void*)'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::ByteSink::Flush()'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::UnicodeStringAppendable::appendCodePoint(int)'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::UnicodeString::countChar32(int, int) const'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::StringCharacterIterator::StringCharacterIterator(icu_60::UnicodeString const&)'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to<EMAIL_ADDRESS>/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `icu_60::ByteSink::~ByteSink()'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to `uscript_getScript_60'
/usr/lib/tensorflow-cpp/libtensorflow-core.so: undefined reference to<EMAIL_ADDRESS>collect2: error: ld returned 1 exit status
CMakeFiles/example.dir/build.make:97: recipe for target 'example' failed
make[2]: *** [example] Error 1
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/example.dir/all' failed
make[1]: *** [CMakeFiles/example.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2

Error informations::
libpng16-16/xenial,now 1.6.20-2 amd64 [installed,automatic]
libpng16-dev/xenial,now 1.6.20-2 amd64 [installed]
libpng16-devtools/xenial,now 1.6.20-2 amd64 [installed,automatic]
libpng16-tools/xenial,now 1.6.20-2 amd64 [installed,automatic]

ldconfig -p | grep png
	libpng16.so.16 (libc6,x86-64) => /lib/x86_64-linux-gnu/libpng16.so.16
	libpng16.so.16 (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libpng16.so.16
	libpng16.so.16 (libc6,x86-64) => /usr/lib/libpng16.so.16
	libpng16.so (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libpng16.so
	libpng12.so.0 (libc6,x86-64) => /lib/x86_64-linux-gnu/libpng12.so.0
	libpng12.so.0 (libc6,x86-64) => /usr/lib/x86_64-linux-gnu/libpng12.so.0

i build tensorflow from source

You can build Tensorflow from source, if you want to get a working package of TF 1.13.1, but the package you tried was built for Ubuntu Bionic while your system is Ubuntu Xenial. That is the explanation of the linker errors. The last version what I built and published was TF 1.7:
https://github.com/kecsap/tensorflow_cpp_packaging/releases/tag/Tensorflow_1.7.0_git20180411_079539b

Ok, thank you for the explanation


bar charts

Hi, I've added graphType=bar to Graphite on request of my collegues. I thought it would be nice to share. Please let me know what you think,
Regards, Piotr

I won't be finishing this, Grafana does this way better than I could ever built in Graphite.


New Field: Usage Type

We miss a field in "Property Objects":
field name: usageType
description: The type of usage of the financing property: self, let
not required
string
enum: [self, let]

added at the same fields in the yaml than #58


wmi_logical_disk_size_bytes

Recently i have installed wmi_exporter in my Windows servers. I have noticed that the value of the wmi_logical_disk_size_bytes is not correct. Both of my C and E drives has the same value in Prometheus, While they are not on the same size.
Please see the attached screenshots.
What could be the issue?
Please advise.

can you open a powershell window and run the following:
Get-WmiObject -Query "select * from Win32_PerfRawData_PerfDisk_LogicalDisk" | ft Name, PercentFreeSpace, PercentFreeSpace_Base

Done. Please see the attached screenshot.
What could be the issue?

the wmi_exporter is using the same wmi query as above to get the metrics and provides that for prometheus.  Your issue is not with wmi_exporter, per se, but something is not correct on that server.
If that OS is Windows 2003 or 2008, it might be related to this:
http://ossmall.ro/the-win32perfrawdataperfdisklogicaldisk-wmi-class-returns-an-incorrect-percentage-of-free-disk-space-in-windows-vista-and-in-windows-server-2008/

OK. Thanks!

@supertwister Did this resolve your problem?

I still have the issue with the wrong values of the drives size, But this issue is not related the exporter. It seems like a bug in Windows.
In order to workaround this issue, I have added in Prometheus the drive sizes (hard coded), And now i can monitor the drives's free space as i wanted.

I still have the issue with the wrong values of the drives size, But this issue is not related to the exporter. It seems like a bug in Windows.
In order to workaround this issue, I have added in Prometheus the drive sizes (hard coded), And now i can monitor the drives's free space as i wanted.

how do you deal with the problem

The link explaining the issue is dead. This link here might help though:
https://support.microsoft.com/en-us/help/961435/some-base-values-are-not-updated-when-you-use-the-win32-perfrawdata-pe


Add parallel algorithms

https://devblogs.microsoft.com/cppblog/using-c17-parallel-algorithms-for-better-performance/
https://en.cppreference.com/w/cpp/experimental/parallelism/existing


fix: Typo in points for #716

Fix the typo in points as per the discussion in https://github.com/embeddings-benchmark/mteb/pull/716#issuecomment-2130437694

cc @isaac-chung @dokato


How can I fix the port numbers for multiple GPU backends? I don't want to change the port numbers every time I restart the backend

Your question

I have multiple GPUs and I want to fix the port number and don't want to change it every time.

Generally you shouldn't touch the backend ports directly at all? So main question is, why are you even doing that? You're probably doing something wrong if so.

Generally you shouldn't touch the backend ports directly at all? So main question is, why are you even doing that? You're probably doing something wrong if so.

I found that the port number would change every time I restarted the backend service. Because I was making a workflow using NetDis, I needed to frequently install custom nodes during the process. Each installation required restarting the backend service, which caused the Comfyui port number to change. This invalidated my workflow and required me to re-set the port number in NetDis.

NetDis?


Implement the usage of pyansys-tools-versioning

As title says.

This is now possible using ansys-api-geometry version 0.3.8

We will have to implement some try/except behavior though to avoid failing past versions. But in those cases, we will assume 24R1.

Done through #1024


LS: Add completions of struct members

Stack:

#6566
#6565
#6567 ⬅
#6563

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

✓ Commit merged in pull request #6566


⚠ Bofrost.fr has degraded performance

In ef67550, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1358 ms

Resolved: Bofrost.fr performance has improved in 0542e88 after 6 minutes.


Update aws-core, http-client-spi, ... to 2.20.51

Updates

software.amazon.awssdk:aws-core
software.amazon.awssdk:http-client-spi
software.amazon.awssdk:netty-nio-client

from 2.20.47 to 2.20.51.
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!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #1165.


BK8s datasource middleware -- PoC for A records

Initial PoC for kubernetes middleware.
Implementation is rough but demonstrates A record lookup based on kubernetes service name. The CoreFile defines the zone that maps to the kubernetes middleware. For example, the zone "coredns.local". If a service is launched in kubernetes with the name "mynginx" in the namespace "demo", then the container IP will be resolved by looking up "mynginx.demo.coredns.local".
More details provided in the file middleware/kubernetes/README.md
Many outstanding TODO items are described in the README. Additionally: code cleanup, removing remaining etcd-focused code, and providing unittests.

Yes, this looks nice. I'll comment on some style issue.
The main thing is that there is so much code duplication between the etcd middleware and this, that we should probably abstract that to an interface (i.e. #59) and implement this on top of that. Needs some thinking about the directory structure middleware/etcd does not make much sense anymore then.
I might do the etcd abstraction today, as again that's just lifting code from skydns.

I'm moving to a separate branch for on-going enhancements. I think the issues in this PR are addressed.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/coredns] BK8s datasource..." ]

I'm moving to a separate branch for on-going enhancements. I think the issues
in this PR are addressed.

Ack.
What's your plan? Merged when works, then abstract etcd backend in a kv
(keyvalue) backend and move this into that middleware?
Or
Abstract etcd middleware first and apply this directly on top of that?
/Miek
--
Miek Gieben

Shall we merged now, so we can (both) iterate on it?


[vm] Assert in reload on closures_without_captured_variables_test

Commit https://github.com/dart-lang/sdk/commit/e9b56794fcc050a43425c4c1023ce8a03382e9de.
The tests
vm/dart_2/isolates/closures_without_captured_variables_test Crash (expected Pass)

are failing on configurations
dartk-reload-linux-debug-x64

log: https://dart-ci.appspot.com/log/vm-kernel-reload-linux-debug-x64/dartk-reload-linux-debug-x64/9202/vm/dart_2/isolates/closures_without_captured_variables_test
../../runtime/vm/object.cc: 2646: error: expected: vtable() == builtin_vtables_[cid]
version=2.15.0-edge.01c4999b318e3a6f1475510d577d473377434340 (be) (Fri Sep 3 23:52:48 2021 +0000) on "linux_x64"
pid=25507, thread=25559, isolate_group=main(0x55f115616800), isolate=(nil)((nil))
isolate_instructions=55f112daa3e0, vm_instructions=55f112daa3e0
  pc 0x000055f1131a3c1c fp 0x00007fda6ad7d9a0 dart::Profiler::DumpStackTrace(void*)+0x7c
  pc 0x000055f112daa594 fp 0x00007fda6ad7da80 dart::Assert::Fail(char const*, ...) const+0x84
  pc 0x000055f1130b7176 fp 0x00007fda6ad7dac0 dart::Object::CheckHandle() const+0x1a6
  pc 0x000055f11312eb51 fp 0x00007fda6ad7db00 dart::Instance::GetTypeArguments() const+0xa1

The cause is independent of that commit, it just happens to trigger under rare circumstances. I've uploaded a fix for it in cl/212585


[Postgres] Test and document support for YugabyteDB

As a sibling suggestion to https://github.com/launchbadge/sqlx/issues/198 . I would be willing to help.

From a skim it seems like it should "just work" with the postgres driver. We should test and document this.
Next, it seems like there is actually additional (optional) complexity to the protocol where it would make sense to have a sqlx::yogabyte module that is based on the sqlx::postgres module. That should be split into a low priority task. I don't see us circling back to enhance support for spin off databases for a while. I'd love to accept a PR here though.


Gluetun in infinite crash loop after automatic update by watchtower

Hello,
Gluetun has been running perfectly for several months on my TS-364 QNAP (32GB RAM, 2x4TB in raid1, firmware 5.1.5 20240219), using  wireguard by NORDVPN and shadowsocks.
On 19-03-24 watchtower updated the gluetun docker container. Following, the container entered an infinite loop of starts and crashes.
The log report:

Running version latest built on 2024-03-18T17:49:37.003Z (commit 3d16798)
🔧 Need help? https://github.com/qdm12/gluetun/discussions/new
🐛 Bug? https://github.com/qdm12/gluetun/issues/new
✨ New feature? https://github.com/qdm12/gluetun/issues/new
☕ Discussion? https://github.com/qdm12/gluetun/discussions/new
💻 Email<EMAIL_ADDRESS>💰 Help me? https://www.paypal.me/qmcgaw https://github.com/sponsors/qdm12
2024-03-19T11:37:33+01:00 INFO [routing] default route found: interface eth0, gateway <IP_ADDRESS>, assigned IP <IP_ADDRESS> and family v4
2024-03-19T11:37:33+01:00 INFO [routing] local ethernet link found: eth0
2024-03-19T11:37:33+01:00 INFO [routing] local ipnet found: <IP_ADDRESS>/22
2024-03-19T11:37:33+01:00 INFO [firewall] enabling...
2024-03-19T11:37:33+01:00 INFO [firewall] enabled successfully
2024-03-19T11:37:34+01:00 INFO [storage] merging by most recent 17820 hardcoded servers and 17820 servers read from /gluetun/servers.json
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x760835]
goroutine 35 [running]:
github.com/qdm12/gluetun/internal/configuration/settings.(*ServerSelection).validate(0xc0004cd130, {0xc00002c015, 0x7}, {0x1098740?, 0xc0024517a0?})
github.com/qdm12/gluetun/internal/configuration/settings/serverselection.go:149 +0xa55
github.com/qdm12/gluetun/internal/configuration/settings.(*Provider).validate(0xc0004cd128, {0xc00002a021?, 0x9?}, {0x1098740, 0xc0024517a0})
github.com/qdm12/gluetun/internal/configuration/settings/provider.go:47 +0x2fc
github.com/qdm12/gluetun/internal/configuration/settings.(*VPN).Validate(0xc0004cd118, {0x1098740, 0xc0024517a0}, 0x0?)
github.com/qdm12/gluetun/internal/configuration/settings/vpn.go:30 +0x165
github.com/qdm12/gluetun/internal/configuration/settings.(*Settings).Validate.func1()
github.com/qdm12/gluetun/internal/configuration/settings/settings.go:52 +0x2b
github.com/qdm12/gluetun/internal/configuration/settings.(*Settings).Validate(0xc0004cce00, {0x1098740?, 0xc0024517a0}, 0x0)
github.com/qdm12/gluetun/internal/configuration/settings/settings.go:57 +0x9cd
main._main({0x109d840, 0xc0004c2000}, {{0x1096954, 0x6}, {0x1096cd8, 0x7}, {0x1098470, 0x18}}, {0xc0000220a0, 0x1, ...}, ...)
./main.go:240 +0xf9d
main.main.func1()
./main.go:92 +0x12c
created by main.main in goroutine 1
./main.go:91 +0x5e5

When creating the containers manually using  "docker compose up -d",  resulted in following error:

Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to create new parent process: namespace path: lstat /proc/10977/ns/net: no such file or directory: unknown

I've used following docker compose file:
services:
  gluetun:
    image: qmcgaw/gluetun
    container_name: gluetun
    cap_add:
      - NET_ADMIN
    devices:
      - /dev/net/tun:/dev/net/tun
    ports:
      - 8186:8080 # gluetun
      - 8888:8888/tcp
      - 8388:8388/tcp
      - 8388:8388/udp
      - 9091:9091 # transmission-vpn
      - 51413:51413
      - 51413:51413/udp
      - 5800:5800 # jdownloader-vpn
      - 3129:3129
      - 8181:8080 # sabnzbd-vpn
    volumes:
      - /share/CACHEDEV1_DATA/appdata/gluetun:/gluetun
    environment:
      - SHADOWSOCKS=on
      - SHADOWSOCKS_PASSWORD=xxxxxxxxxxxxxxxxx
      - VPN_SERVICE_PROVIDER=nordvpn
      - VPN_TYPE=wireguard
      - WIREGUARD_PRIVATE_KEYxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
      - SERVER_COUNTRIES=Netherlands,Belgium,Germany
      - SERVER_REGION=Europe
      - TZ=Europe/Amsterdam
    restart: unless-stopped

  transmission:
    image: ghcr.io/linuxserver/transmission:latest
    container_name: transmission-vpn
    network_mode: "service:gluetun"
    environment:
      - PUID=1000
      - PGID=1000
      - TZ=Europe/Amsterdam
      - USER=xxxxx
      - PASS=xxxxxxxxxxx
    volumes:
      - /share/CACHEDEV1_DATA/appdata/transmission:/config
      - /share/CACHEDEV1_DATA/Download/completed:/downloads
    restart: unless-stopped

  jdownloader-2-vpn:
    image: jlesage/jdownloader-2:latest
    container_name: jdownloader-2-vpn
    network_mode: "service:gluetun"
    environment:
      - MYJDOWNLOADER_EMAIL=xxxxxxxxxxxxxxxxxxxx
      - MYJDOWNLOADER_PASSWORDxxxxxxxxxxxxxxxxxxxxx
      - PUID=1000
      - PGID=1000
      - TZ=Europe/Amsterdam
    volumes:
      - "/share/CACHEDEV1_DATA/appdata/jdownloader-2-vpn:/config:rw"
      - "/share/CACHEDEV1_DATA/Download/completed:/output:rw"
    restart: unless-stopped

  sabnzbd:
    image: lscr.io/linuxserver/sabnzbd:latest
    container_name: sabnzbd-vpn
    network_mode: "service:gluetun"
    environment:
      - PUID=1000
      - PGID=1000
      - TZ=Europe/Amsterdam
    volumes:
      - /share/CACHEDEV1_DATA/appdata/sabnzbd/config:/config
      - /share/CACHEDEV1_DATA/Download/completed:/downloads #optional
      - /share/CACHEDEV1_DATA/Download/incomplete:/incomplete-downloads #optional
    restart: unless-stopped

What to do to resolve the issue?

Same here. Downgrading to v3 is a temporary fix. Latest commit pushed yesterday (March 18th) seems to have broken something.

@Orbocom  - you might want to open this issue in https://github.com/qdm12/gluetun instead of the wiki.
PS: The issue seems to have been fixed in the latest commit pushed an hour ago. Try to pull the latest image from docker hub.

Thank you for your advice. You are absolutely right. My apologies.

Great. Please close the ticket if the issue has been fixed for you :)

Thanks for prompt response


Broken link in the readme

Describe the bug
When you click on the contributor guidelines in the readme section, it redirects you to a 404 not found error
To Reproduce
Steps to reproduce the behavior:

Go to 'contributions'
Click on 'contribution guidelines'
See error

Expected behavior
Redirection to the contribution guidelines
Screenshots

Desktop (please complete the following information):

OS: macOS
Browser chrome
Version 124.0.6367.208

I would like to work on this issue and fix it

Thanks for Highlighting, you can redirect into https://github.com/Recode-Hive/Stackoverflow-Analysis/tree/main/Readme

Hello! I opened a pull request for it! Please check it out and tell me if it is okay.

I have a doubt. Wouldn't it be better to redirect users to "https://github.com/Recode-Hive/Stackoverflow-Analysis/blob/main/Readme/Contributing.md" i.e., the contributing file directly?

https://github.com/Recode-Hive/Stackoverflow-Analysis/tree/main/Readme this would be the right location

okayy!


Compute unsealed sector CID syscall

Summary of changes
Changes introduced in this pull request:

Implements the compute_unsealed_sector_cid syscall in the runtime

Interfaces through the filecoin-proofs-api

Updates commcid interface to take in references (was no need to move variables before, but assumed they wouldn't be needed after)

Dependent on #357 so I will remove other commits when that comes into mater and open this PR up
Reference issue to close (if applicable)

Closes
#303
Other information and links

line 23 in vm/src/piece/mod.rs
pub fn validate(self) -> Result<(), &’static str> {
Can probably take a ref. I couldnt comment on that line so posting here

line 23 in vm/src/piece/mod.rs
pub fn validate(self) -> Result<(), &’static str> {
Can probably take a ref. I couldnt comment on that line so posting here

Nah it's just a u64 so better to just move in. Pretty sure clippy warns on this


More optimizations

Comparing the values in the test, it seems a few things got slower and others faster. Not happy overall with this PR.


[PROJECT] integrate with accounting tools and practices via exports and reports

Overview
Open Collective does not desire to become a fully featured accounting platform (like QuickBooks or Xero), but our users, especially larger fiscal hosts with established financial processes, need to integrate Open Collective with their accounting operations. We also do not desire to do a deep technical integration (and want to enable users to choose their own accounting tooling, not only the ones we've integrated with), so we want to focus on data export formats to better match standard accounting formats.
Related issues: #3983 #3340 #3875 #3552 #4039 #3961 #3689 #4366 #4018 #3934 #4382 #4361 #2798 #4300
Owner
TBD... probably needs @znarf because of the ledger implications, and also someone from the host admin side of the team.
Prioritization

Does this serve our mission?

Open Collective is an interface between collaborative Collectives and the outside world, seeking to enable them to retain their essence as circles, while being able to operate in the world of accounting, taxes, and legal entities. Backoffice tooling for this function is essential.

Will this help to make us financially sustainable within a year?

Our main revenue source now, and the area we see the most future potential, is large scale hosts, all of whom need to do tax reporting and integrate with accounting systems.

Can this make us move faster?

It will feel slower at first because these are mostly invisible features only host admins care about, but it will definitely help more hosts decide to use Open Collective faster and onboard more smoothly.

Does this increase quality?

Yes, this is a key area of ledger-debt and technical-debt in the platform.

Does this add technical debt?

No, it will reduce technical-debt.

Is it feasible to build and maintain with our current team and runway?

It would take a decent amount of time and attention from Francois and either Pia or Alanna, to go deep on both the ledger and accounting functionality sides. But if we decide that's worth it, then we can manage it.

What insight or research is this project based on?

Our own experience operating hosts and customer development discussions with new hosts.

Is this problem already solved elsewhere?

We currently solve this in our first party hosts with a messy combination of manually cleaning our current data exports and trying to achieve what we need with metabase. This takes a lot of time from developers and accountants (expensive!) and is quite error prone (we have major issues every time with taxes/audit exports). Existing accounting software does solve this but our data has to match up with them.

How will we measure our success? Do we have a baseline today that we can compare this against?

Less time spent on tax and audit data crunching for our own hosts.
Wider adoption of OC as a system by existing orgs becoming new hosts.
Faster onboarding of new hosts who need to set up their accounting processes.

How well do we understand the complexity of the problem, feature to be added or user need to be addressed?

There's still a fair amount of discovery to be done. We're clear about the desired end-state but need to figure out all the details.

What is the estimated timeframe for delivering this project?

?
Kick-off meeting
?
Roadmap & Timeframe
?
Success metrics
Less time spent on tax and audit data crunching for our own hosts.
Wider adoption of OC as a system by existing orgs becoming new hosts.
Faster onboarding of new hosts who need to set up their accounting processes.
Release planning
?
Estimated delivery date
?

Which aspect(s) of the platform does this change impact?
Ledger, host settings, data exports.

Which group(s) of users will benefit from this change?
Hosts

Is this project in response to a user-requested feature? If so who requested it (link to feature request if available)?
Yes - all new hosts ask about integration with their accounting systems, plus our own experience with our hosts.

Is this project related to any similar work we have done, or are planning on doing in the next three months?
Not really?

Next steps

[ ] Open up any appropriate project issues
[ ] Open a documentation issue, link it to this one
[ ] Open a release issue, link it to this one
[ ] Create a project board, add this issue to the description, add any issues created above

Project Closure

Close all relevant project issues
Open up issues for improvements
Post comm's according to comm's plan (most likely in release issue)
Post project screencast on documentation slack channel
Post project closure in newsletter slack channel
Celebrate!!

Adding #4497 to this

Adding local tax stuff here as we cannot align with accounting if we don't handle tax correctly #4493 #4532 #4494 #1552 #4366 #vat

Related: #6121

closing in favour of #3983


fixes IBM-Swift/Health#8 Add check to avoid crash because if negative…

… UInt64 numbers
Because of different rounding behaviour in DateFormatter.string and UInt64(date.timeIntervalSince1970 * 1000.0) it is possible that an older date has more milliseconds than a newer date. To avoid it I check if the current date is later than the old date.
The problem is described in #8

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #9 into master will increase coverage by 0.7%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master       #9     +/-   ##
=========================================
+ Coverage   87.25%   87.96%   +0.7%
=========================================
  Files           3        3
  Lines         102      108      +6
=========================================
+ Hits           89       95      +6
  Misses         13       13

Flag
Coverage Δ

#Health
87.96% <100%> (+0.7%)
:arrow_up:

Impacted Files
Coverage Δ

Sources/Health/Health.swift
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

any update on this PR?

Thanks for this! Sorry for the delay, merged and tagged as 1.0.4 👍


Adding types for webrick, nats, grpc

Hello!
I am interested in adding types for webrick, nats and grpc to this repository ☺.
If you think that this is a good idea, please, write me and I will start working on a PR with webrick types to demonstrate how it would look like.

Thanks for your contribution!

If you think that this is a good idea,

Sure! Please write the types 👍

I'll close this issue. Feel free to open PRs on this repository.


chore: add gcs folder to ci exec img.

gcs is a supported path in kfp, adding it here for parity

/lgtm
/approve


#18331 Have BrowserKit\Client create and accept a Crawler from another source

Issue on symfony/symfony#18331
It probably won't get in, but I need my PR patch.

Thanks for contributing this pull request. However, the repositories of individual Symfony Components are read-only, so pull requests are ignored. Please, submit this pull request to the main Symfony repository at https://github.com/symfony/symfony/pulls
Thank you!


Feature Request: Allow users to provide their own NetworkDispatcher and extract Ktor dependency into a separate artifact

Ktor can be a pretty heavy dependency in apps that already have networking stacks
We're using this approach to tie networking within KMP to our existing networking stack
It would be great to be able to provide my own NetworkDispatcher implementation and avoid Ktor

@way2jatin
What do you think about removing ktor dependency and consuming library users implementions of NetworkDispatcher?

So, according to the request, 'ktor-client-core' dependency should be removed. This also leads to removing @DelicateCoroutinesApi annotation, because 'ktor-client-core' transitively depends on 'kotlinx-coroutines-core-jvm'.

@DelicateCoroutinesApi annotation is used through the entire our codebase.
@ankushg @way2jatin Do you prefer to get rid of this annotation or to leave it and have 'kotlinx-coroutines-core' dependency?

@Bohdan-Kim
Can you please change GBSDKBuilderAppconstructor api and pass networkDispatcher as a param. And also make CoreNetworkClient class public. It will allow us to exclude ktor library dependency from our app and also Proguard will remove unused CoreNetworkClient class.

Sure
Should it look like this?

In this pull request networkDispatcher was moved to GBSDKBuilder params.
If app already has networking stack NetworkDispatcher implementation can be passed through GBSDKBuilder parameters. ktor dependency from GrowthBook Kotlin SDK will be optimized and removed in such case.


Change Twitter to X.

I know i know, nobody needs it, but Elon does it.
So it would be nice to update to X instead of Twitter.

@tobiasquadflieg Hello Brother
can you help me run this project locally in my laptop
i am not able to understand from contributing.md


ROX-25620: Switch to new Konflux task repos

Context is in the ticket.
Figured digests similarly to https://github.com/stackrox/collector/pull/1780

chown: changing ownership of '/etc/pki/entitlement': Operation not permitted

issue seems persistent. I'll report.

Ok, that chown issue does not get reproduced on plain ubi or ubi-minimal, 8 or 9.
This is due to our stuff here:
https://github.com/stackrox/scanner/blob/499765062d7f5c1246870d88e0ffadc0f05da47f/image/scanner/rhel/konflux.Dockerfile#L74
Need to figure out why it exists.


ffe-searchable-dropdown-react: wrong colors for dark variant of component

Which package is this issue related to?
ffe-searchable-dropdown-react
Describe your issue (screenshots welcome!)
Compared to the dark variant of input and dropdown, the dark variant of searchable dropdown does not get the proper colors set. Please note that this is the dark variant of the component used for internal applications.

What is the expected behavior?
The searchable dropdown should have the same general styling as the regular dropdown component.
What is the actual behavior?
The searchable dropdown has the wrong background color in the input field, wrong field label color, wrong icon color etc.
Steps to reproduce
Visit https://design.sparebank1.no/styleguidist/index.html#!/SearchableDropdown and compare the dropdown at the very bottom of the page to the dropdow at the bottom of https://design.sparebank1.no/styleguidist/index.html#!/Dropdown.

Darkmode searchable dropdown #817


not able to open saved session from macOS in Windows

Description
Given I had file.proxymansessionv2 that exported from proxyman mac version
When I open this file in Windows proxyman
Then app throws error

Steps to Reproduce

Save session from proxyman MacOS

Shared to windows user

open in proxyman Windows

Current Behavior
App throw error and unable to load the file
Expected Behavior
the file proxymansessionv2 should be compatible and platform agnostic
Environment

App version: Version 2.14.0
Windows/Linux version:

Version	23H2
OS build	22631.3527
Experience	Windows Feature Experience Pack 1000.22700.1003.0

Hi @FachrulCH
This feature has not been implemented yet. You can track the progress in our roadmap to see when this feature will be completed 👍🏻
This is our roadmap https://github.com/ProxymanApp/proxyman-windows-linux/projects/1
And this is the proxymansessionv2 ticket https://github.com/ProxymanApp/proxyman-windows-linux/projects/1

@FachrulCH you can right click on the folder, app, domains -> Export -> ProxymanLog
Then, importing it to Proxyman Windows.
ProxymanSession hasn't implemented yet on Windows.

ah sure,, well noted 👍


[2374] - feat(course preview) - display financial incentives

Context
The course preview page (/organisations/[PROVIDER_CODE]/2020/courses/[COURSE_CODE]/preview)will now display financial incentives (scholarship, bursary or early career payments) depending on whether this information is returned from the backend API
This replicates what is shown on search-and-compare-ui
Changes proposed in this pull request

Re-enable financial incentives partial
Introduces new CourseDecorator logic to determine which financial incentives are displayed

Guidance to review

In the 'early career payments` section the subject name displayed depends on how may subjects a course has. If more than one then it reverts to the course name (discussed with @defong)

Screenshots
A course has no financial incentives whatsoever.
Is this even possible?

A course with scholarship only

A course with scholarship and bursary

A course with scholarship, bursary and early career repayments

Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Early Career Payment Link is hardcoded

did you change the bit at the top as well

refereneced
https://find-postgraduate-teacher-training.education.gov.uk/results/filter/subject?subjects=3&qualifications=QtsOnly,PgdePgceWithQts,Other&fulltime=False&parttime=False&hasvacancies=True&senCourses=False
why £3500

This was old copy in the partial I re-enabled.
Will update.

did you change the bit at the top as well

Nope. I'll update

@defong - funding options fixed

Looks good to me once the boolean is corrected and Sau's comments are addressed 👍

Don't know how much like for like is needed or where to draw the line.
What's implemented is mainly scholarship and bursary where its fine to show a bullet list with 2 items, that's mainly good to go, minus default text not shown
It will look odd to have a a bullet list for one item.
From a data point of view it's probably a given that if there is a scholarship then it will have a bursary.

Guidance to review
* In the 'early career payments` section the subject name displayed depends on how may subjects a course has. If more than one then it reverts to the course name (discussed with @defong)

Misquoted, but since you mentioned 'early career payments`, I doubled checked

if the course has two subjects and they are Mathematics & Physics secondary subjects then there is no 'early career payments`

https://github.com/DFE-Digital/search-and-compare-ui/blob/34ca77118ebd756ec056155f4c7093646355f930/shared/ViewModels/FinanceViewModel.cs#L70-L83
But ignore that for now as I don't think its right @dankmitchell any idea?

This was the bit I was refering to
HasBursary & HasScholarshipAndBursary is overridden if
course name starts with and has 2 subjects associated with it
                "Drama",
                "Media Studies",
                "PE",
                "Physical Education",

see
https://github.com/DFE-Digital/search-and-compare-ui/blob/903e35ea800ccc4cf0b06a18a36e037ab813e229/shared/ViewFormatters/CourseExtensions.cs#L109-L158


Windows: "Error: unknown sync backend 'gitmock'

Error occurs when cloning repo with gopass (gopass clone ) with git installed through scoop (https://github.com/lukesampson/scoop) and available through the PATH. I'm not really sure where to start looking for a fix for this.

This is related to a major refactoring in the 1.7 development cycle. Errors related to the different backends are expected and we plan to track them down before the next release. Sorry for the inconvenience. Will try to fix this soon.


Feedback: Install CockroachDB on Mac

Page: https://cockroachlabs.com/docs/v24.2/install-cockroachdb-mac.html
What is the reason for your feedback?
[ ] Missing the information I need
[ ] Too complicated
[ ] Out of date
[ X] Something is broken
[ ] Other
Additional details
The docs say, docker pull cockroachdb/cockroach:v24.2.0; but that tag doesn't exist on DockerHub. I would have just fixed the doc, but I'm not actually sure which tag should currently be recommended.
Jira issue: DOC-10949

The docs must have been published early, but v24.2.0 is now released and the Docker image is published.


Compiled config vs runtime config

💚 The custom environment variables feature.
This feature leverages Webpack's DefinePlugin which is explained as:

…global constants which can be configured at compile time.

The issue is not all configuration should be compiled into an application.
Things that are stable between environments (dev, staging, production) of an app make sense to compile-in:

version number
commit sha/number
browser support flags

Things that change between environments would be better provided dynamically from the current runtime:

URLs of an APIs (may change for each environment)
secret tokens (may change for each user or request)

Ideally a bundle could be tested in CI, then promoted through environments to production, but compiling all of these values into the bundle means that every promotion requires rebuild.
I've used Node to serve single-page apps in the past with a base HTML template that gets environment variables set via inline

After running npm run build, people would find their env.js file untounched under build/env.js. They would then be able to edit this file manually directly on the deployment machine or through a small bash script to change API_URL to https://api.my-app.com for instance.
Should I submit a PR documenting this technique or even adding this env.js file to the public folder of the boilerplate ?

@tibdex I like this solution, but did you test via the npm run test with the jest. seems will have some error and it can not get in the test level. any advice?

Writing configuration values into a file is antithetical to 12-factor. Instead, always use runtime environment variables for configuration.
create-react-app [CRA] does not provide a production runtime. It only builds a static bundle. Runtime environment is not something solvable by CRA, unless it becomes prescriptive about deployment which is unlikely given its wide range of applications.
Solving this problem is about finding a way to set those variable values in the React app at runtime instead of buildtime. I know of two ways to accomplish this:

Render runtime environment values into global variables in a

which ends up like this in the build output
You need to enable JavaScript to run this app.

Loading the env.js this way is completely synchronous. env.js will be loaded before the main bundle and you can just assume the configuration is there.
You could however (as @forrest-rival also pointed out) use an asynchronous request to fetch some json (env.json) too right when your application is loading. As mentioned, you would have to take care that the configuration is loaded before showing the app (by wrapping your app as mentioned).
I have been using both approaches in the past and personally, I found a synchronously loaded env.js file to be sufficient most of the time. Loading the configuration dynamically gives you some flexibility with regards to configuration loading but comes with a bit complexity.
You can also combine both approaches of course.
In any case, at some point you might have to consider what to do with the app loaded up in your user's browsers when configuration changes. How to signal configuration changes to the browsers? Will you poll for changes? Have a socket open? SSE? And in case your configuration changed, is it enough to just load the new values or do you need to reload the whole application?


Issue with macro in pico/binary_info.h

This caused my project to stop compiling:
My code includes pico/binary_info.h; when I include this line of code, however:
bi_decl(
      bi_3pins_with_func(SPI_RX_PIN, SPI_TX_PIN, SPI_SCK_PIN, GPIO_FUNC_SPI)
);

(which tells the BI about my SPI configuration, I believe)
the build fails with this message:
[build] /Users/ME/src/pico-sdk/src/common/pico_binary_info/include/pico/binary_info/code.h:22:55: error: '_error_bi_is_missing_enclosing_decl_93' was not declared in this scope; did you mean '_error_bi_is_missing_enclosing_decl_92'?

That line of code uses a macro that embeds the __LINE__ number, and clearly it's being calculated wrong, and I have no idea why - I'm not a super experienced C++ programmer.
This happens with arm-none-eabi 9.21 and 7.2.1, and I'm using CMake 3.23.2.
Any ideas?

Looking at https://github.com/raspberrypi/pico-examples/search?q=bi_3pins_with_func&type=code I wonder if your error might be caused by your bi_decl and bi_3pins_with_func being on different lines? That's a bit of a random guess though :wink:

@lurch oh good lord. that is, in fact, the funkiness at my end. Of course: what puzzles me is that this used to compile. Anyhow, mystery solved, let's close this and never talk of it again.


shellcode generation

Hi, I would like to ask whether the client generation can support the generation of shellcode, so that it can be packaged into various executable files through its own loader, and the anti-virus etc. can also be customized.

Hi there this has already been asked.
https://github.com/NHAS/reverse_ssh/issues/128
The shellcode for RSSH would be way way bigger than anything that is practical to use. RSSH is a post exploitation tool. If you want to generate your own shellcode/loader for it then awesome!
However it is currently outside the scope of this project


print() on Flutter apps/packages does not allow usage of stdout colors

When using pure Dart packages, it is possible to colorize the message printed in the stdout. Libraries like colorize or colorize_lumberdash rely on this.
However, when using them in a flutter context, this is what gets printed instead:
flutter: \^[[33mmy warning<…>

That same code is printed in yellow without flutter: \^[[33mmy in a pure Dart app/package.
Steps to reproduce:

pubspec.yaml

name: example
description: A new Flutter project.
version: 1.0.0+1

environment:
  sdk: ">=2.0.0-dev.68.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

  cupertino_icons: ^0.1.2
  colorize: 2.0.0

dev_dependencies:
  flutter_test:
    sdk: flutter

flutter:
  uses-material-design: true

main.dart

import 'package:flutter/material.dart';
import 'package:colorize/colorize.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {

    final message = Colorize('my warning');
    message.yellow();
    print(message);

    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: Text('Home Page'),
    );
  }
}

Expected: a my warning message yellow printed in stdout
Recevied: flutter: \^[[33mmy warning<…>
Also reproducible with colorize_lumberdash on Flutter packages
Works as expected on pure Dart packages

Seems like this issue might be fixed using ansicolor: https://pub.dev/packages/ansicolor
Closing this issue for now