cmd/scollector: Optimise performance for linux stat collection.

On a big server with lots of network interfaces, disks, processes, VMs, etc... the stat collection is very heavy. After running pprof I saw the biggest offending functions and I have stream lined them.
Replacing simple regex with strings methods.
Replace IsDigit and IsAlNum with strconv methods.
Use /proc/mounts and Statfs() instead of 'df', simplifies the code. Less
string processing since it returns int.
Handle filesystems with spaces, use internal functions
Rewrote isPseudo() as a closure. At the moment /proc/filesystems
is read for every filesystem being checked, now it is only read once per
check.
removed IsDigit and IsAlNum, replaced them with strconv methods.
Use Sysinfo() instead of /proc/(uptime|loadavg), faster and less string
processing. But it results in losing linux.uptime_now and
linux.loadavg_runnable metrics.
Changed "linux.loadvg_" to "linux.loadvg."

Updated the pull-request with better rebase/squash.
The original had a bug where it would skip checking filesystems.
This one works correctly.

Update with check for number of fields in /proc/mounts.

This might fix https://github.com/bosun-monitor/bosun/issues/1069


:bug: Select All option in Autocomplete component disappears on scroll

Describe the bug
When you have 30 items or more, the Select All option disappears when scrolling. I assume the issues is caused by useVirtualizer removing the option.
https://codesandbox.io/p/sandbox/eds-autocomplete-select-all-disappearing-xrn9qm
Steps to reproduce the bug

Open https://codesandbox.io/p/sandbox/eds-autocomplete-select-all-disappearing-xrn9qm
Scroll far down
See the Select All option disappear

Expected behavior
It should not disappear.
Specifications

Version: 0.39.0
Browser: N/A
OS: N/A

Additional context
https://codesandbox.io/p/sandbox/eds-autocomplete-select-all-disappearing-xrn9qm

This makes sense as the select all "item" is injected into the rest of the options and handled by the virtualizer.  I remember @yusijs tried a bunch of stuff and this was the solution that played nice with the virtualizer. We'll have to take another look if there is something we can do here about this 👍


test: add labels to issues

Blocked by

[ ] https://github.com/mikedeboer/node-github/issues/587
[x] https://github.com/node-nock/nock/pull/979
[x] https://github.com/node-nock/nock/pull/980

mikedeboer#587 won't get fixed anytime soon, skipping the test


Help with using collections

While trying to use collections I am running across errors like the following:
Class not registered for JS export: System.Collections.ObjectModel.ReadOnlyCollection`1[System.String]
Class not registered for JS export: System.Collections.Generic.List`1
Class not registered for JS export: System.Collections.ArrayList

I suspect I am doing something wrong, but a bit hard to find out what the correct process is, can you please help out?
Here's a snippet of my code (the full test project is available here: (https://github.com/rijulg/node-api-dotnet-issue)
using System;
using System.Collections;
using System.Collections.Generic;
using System.Collections.ObjectModel;

public class HelloDll
{

    internal List<String> _items = new List<String>();
    public ReadOnlyCollection<String> items
    {
        get
        {
            return new ReadOnlyCollection<String>(_items);
        }
    }

    public void AddItem(string item)
    {
        _items.Add(item);
    }
}

import { Injectable } from "@nestjs/common";
import { ConfigService } from "@nestjs/config";
import { load as loadDLL } from "node-api-dotnet";

@Injectable()
export default class DLLRepository {
  private readonly helloClass: any;
  constructor(configService: ConfigService) {
    const dllPath = configService.getOrThrow("DLL_PATH");
    const dll = loadDLL(dllPath);
    this.helloClass = new dll.HelloDll();
  }
  addItem(item: string) {
    this.helloClass.AddItem(item);
  }
  getItems(): any {
    return this.helloClass.items;
  }
}

Calling getItems in JS fails.

Hi, thanks for the issue report.
ReadOnlyCollection<T> and IList<T> are supposed to work in this kind of scenario, although ArrayList isn't currently supported. I will investigate and get back to you.

Oh, you need to use the collection interfaces (IReadOnlyCollection<T> or IList<T>), instead of the classes (ReadOnlyCollection<T>, List<T>) on public properties consumed by JavaScript.
The classes may be supported later. But it's a good practice anyway to use collection interfaces for public properties.

The relevant code is in JSMarshaller.BuildFrom/ToJSCollectionExpressions(). Going from .NET to JS (for a property get) should be a pretty small change, since the class can be handled exactly the same way as the interface. The opposite direction (for a property set) will be much more difficult, but maybe you don't need that.
You may find the code difficult to understand though, since it is building expression trees that then get either emitted at build time or compiled at runtime. I have been thinking about writing some technical documentation about how all of that works internally, but have not done it yet.
Your other alternative is to build a .NET adapter for the JS interop layer.

Thanks, I'll try both and share my findings here in either case.

I tried using AOT process (building a .Net Adapater) but unfortunately the issue with classes and interfaces persists there. I will be trying to modify the library now to accommodate the missing classes. From my investigation so far I can tell that the classes are never exported because they are seen as System classes in NodeApi.DotNetHost.AssemblyExporter.ExportClassDependencies.  But based on Jason's feedback I'll try to modify the marshaller to export the classes.
For logkeeping I get the following error:
Debug_String
[]
[ 0, 1, 2 ]
/home/workspace/workdir/srcext/js/main.js:14
    console.log(dll.TestBinding.listList);
                                ^

Error: Class not registered for JS export: System.Collections.Generic.List`1[System.Int32]
    at Microsoft.JavaScript.NodeApi.Interop.JSRuntimeContext.GetClassConstructor[T]()
    at Microsoft.JavaScript.NodeApi.Interop.JSRuntimeContext.<>c__DisplayClass28_0`1.<GetOrCreateObjectWrapper>g__CreateWrapper|0(T obj)
    at Microsoft.JavaScript.NodeApi.Interop.JSRuntimeContext.<>c__DisplayClass28_0`1.<GetOrCreateObjectWrapper>b__1(Object _)
    at System.Collections.Concurrent.ConcurrentDictionary`2.AddOrUpdate(TKey key, Func`2 addValueFactory, Func`3 updateValueFactory)
    at Microsoft.JavaScript.NodeApi.Interop.JSRuntimeContext.GetOrCreateObjectWrapper[T](T obj)
    at Microsoft.JavaScript.NodeApi.Generated.Module.get_dotnet_TestBinding_listList(JSCallbackArgs __args) in /home/workspace/workdir/srcext/dotnet/Microsoft.JavaScript.NodeApi.Generator/Microsoft.JavaScript.NodeApi.Generator.ModuleGenerator/dotnet.NodeApi.g.cs:line 63
    at Microsoft.JavaScript.NodeApi.JSNativeApi.InvokeCallback[TDescriptor](napi_env env, napi_callback_info callbackInfo, JSValueScopeType scopeType, Func`2 getCallbackDescriptor)
    at main (/home/workspace/workdir/srcext/js/main.js:14:33)
    at Object.<anonymous> (/home/workspace/workdir/srcext/js/main.js:17:1)
    at Module._compile (node:internal/modules/cjs/loader:1254:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
    at Module.load (node:internal/modules/cjs/loader:1117:32)
    at Module._load (node:internal/modules/cjs/loader:958:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:23:47

Node.js v18.16.0
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Here are the files I have
// /home/workspace/workdir/srcext/dotnet/TestBinding.cs
using Microsoft.JavaScript.NodeApi;

namespace dotnet;

[JSExport]
public class TestBinding
{
    public static List<int> listList { get; set; } = new List<int>(new int[] { 0, 1, 2 });
    public static IList<int> listIList { get; set; } = new List<int>(new int[] { 0, 1, 2 });
    public static int[] listArray { get; set; } = new int[] {0, 1, 2};
    public static string debugString { get; set; } = "Debug_String";
}

// /home/workspace/workdir/srcext/dotnet/dotnet.csproj
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net6.0</TargetFramework>
    <ImplicitUsings>enable</ImplicitUsings>
    <Nullable>enable</Nullable>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.JavaScript.NodeApi" Version="0.2.26" />
    <PackageReference Include="Microsoft.JavaScript.NodeApi.Generator" Version="0.2.26" />
  </ItemGroup>

</Project>

// /home/workspace/workdir/srcext/dotnet/NuGet.Config
<?xml version="1.0" encoding="utf-8"?>
<configuration>
  <packageSources>
    <add key="local-packages" value="/home/workspace/workdir/node-api-dotnet/out/pkg" />
  </packageSources>
</configuration>

// /home/workspace/workdir/srcext/js/main.js

const modPath = require.resolve("node-api-dotnet");
console.log(modPath);

const dotnet = require('node-api-dotnet');

function main() {
    const dllPath = process.argv[2];
    const dll = dotnet.require(dllPath);
    console.log(dll.TestBinding.debugString);
    console.log(dll.TestBinding.listIList);
    console.log(dll.TestBinding.listArray);
    console.log(dll.TestBinding.listList);
}

main()

Run process
cd /home/workspace/workdir/srcext/dotnet
dotnet build

cd /home/workspace/workdir/srcext/js
node main.js /home/workspace/workdir/srcext/dotnet/bin/Debug/net6.0/dotnet.dll

The reason I suggested creating the .NET adapter module is so that you can precisely control the APIs that are exported to JavaScript, and therefore you can avoid any aspect of interop that is difficult or problematic. Even if you ultimately need to call a 3rd-party library that returns a concrete collection class like ReadOnlyCollection<string>, the public members of your adapter module can export it via the collection interface IReadOnlyCollection<string> instead of the class.
[JSExport]
public class HelloDllAdapter
{
    private readonly HelloDll instance = new();

    public IReadOnlyCollection<String> Items => instance.items;
}

yes, that's a good point; I am just trying to adapt a large DLL almost automatically without having to write a lot of binding code. Will revert to other methods if fixing just a couple of collections is not enough.

Closing in favor of #242 which has a more succinct description of the issue.


unkown configuration for /DsdtFilePath

So I followed the installation, i put in the windows 10 iso and tried to start the VM but then this message pops up:

Failed to open a session for the virtual machine vm0.Unknown configuration value '/DsdtFilePath' found in the configuration of acpi instance #0 (VERR_CFGM_CONFIG_UNKNOWN_VALUE).

Result Code:
E_FAIL (0x80004005)

Component:
ConsoleWrap

Interface:
IConsole {872da645-4a9b-1727-bee2-5585105b9eed}

The loader was executed in an administrator vm, actually i dont even know if the problem lies within the steps of the guide or if it would happen even if i wouldnt try to harden it. I made a clean install everything should be fine.
I use the Oracle virtual box version 6.1.36 and downloaded the windows iso from their website
I have seen another post on this problem here, but i didnt understand the answer really and it didnt seem like i did something wrong.

This project is not maintained since 08.2020. Last VBox version it was checked against was 6.1.2-6.


Renaming a file using the arrow keys to navigate words in its name, will start selecting other files

Description
When I'm renaming a file, if I use the arrow keys to navigate between words of its name, the file rename will abort as soon as I navigate to the last or first characters of the name (including extension) and Files start selecting nearby files.
Steps To Reproduce

Rename file
Use left arrow key continuously to rename first characters of filename or right arrow key to rename last characters
Files aborts rename operation
Files starts selecting other files/folders with the arrow keys

Expected behavior
Files should only leave the rename operation after an explicit Save or Abort by the user
Files Version
Versão: <IP_ADDRESS>
Windows Version
OS Version: 10.0.22621.317 (Win11 22H2 Release Preview)
Relevant Assets/Logs
debug.log
debug_fulltrust.log

Also #5824

Ive noticed it also does it on the search bar with all arrow directions deleting the search bars content.
With file names i've only had the up and down arrow do it now which I believe normally just goes to start and end of text.

Left and right also cause the issue in Grid View. @carlosvsilva what layout mode are you using?

this bug seems to only affect Files when the View is set to Tiles, (Mosaicos), the second of the list, my favourite :)


Audio stream duration doubled?

I'm using YouTubeKit to get the audio url. and feed it to AVPlayer. It plays fine, except audio's total length seems doubled.
For a 1-minute video, we can get an audio url, but the total duration of the stream is 2 minutes. After playing 1 minute, the player keeps playing for another 1 minute, but there is no sound. I try copying the audio url and paste it in browser to play, also showing double audio length.
Has anyone seen this issue before?
Not sure if this is YouTubeKit specific or YouTube making it that way.

Yes, some audio streams have this issue, but it's not something I can fix without actually downloading the audio and trimming it to the correct length. For my own project, I have been using the duration from YouTube's API to display it to the user.

We probably also need a workaround like that.
Thanks for the info!


No source maps in production mode

Bug report
What is the current behavior?
Unable to create sourcemaps in production mode.
If the current behavior is a bug, please provide the steps to reproduce.
So we have 3 environments:
development  (dev build with source maps)
staging           (prod build with source maps)
production     (prod build without source maps)
This was working fine with webpack 4.41.0, but something changed after 4.42.0 -> 4.46.0.
It kinda works, it generates source map for stuff in the node_modules, but not our src folder.
webpack --progress --env.staging --mode production --env.extractCss --optimize-minimize --devtool source-map
But if i change mode to development, like:
webpack --progress --env.staging --mode development --env.extractCss --optimize-minimize --devtool source-map
Then it generates sourcemap for both node_modules and our src folder.
But then we get development source code. Not minified etc. Not a production build.
What is the expected behavior?
Generate source map when i tell it to generate source maps.
Even if i build a production build.
What am i missing ? :)
Other relevant information:
webpack version:   4.46.0
Node.js version:  14.15.4
Operating System: Windows 10
Additional tools:

If the current behavior is a bug, please provide the steps to reproduce.

Sorry command line arguments without configuration can't help to help you, nobody know, won't fix

Humm this kinda fixed it self when upgrading:
ts-loader, @types/node or sass-loader i think..
Some strange dependencies here..


chore: update repo

update version to 3.3.7-1
Issue: https://github.com/linuxdeepin/developer-center/issues/5731
Log: update repo

/approve


Snap has been created for slides

Changes Introduced

README has been updated to reflect snap installation
Snap directory and yaml created
Snap has been published the Snap Store

I can continue to support the snap if you'd like but if you'd rather take full ownership and support it yourself, we can discuss steps to do so (very easy).
This is an awesome application and I can see myself using it to impress the geekier people out there. :)

Thanks so so much! ❤ I don't use snap (I'm on MacOS) so it would be awesome if you want to continue to support it!

Thanks so so much!  I don't use snap (I'm on MacOS) so it would be awesome if you want to continue to support it!

Totally! It's pretty straight forward to change directions if you ever decide you'd rather take care of it. Thanks for this amazing application.

Thanks for this amazing application.

No problem at all! It means a lot that you are using it! 🙂 Good luck with your presentations!


Refactor non-code-path resources from FS => FX + Add License

Summary

Updated readme file and matched its format with the provided sample readme file.
Removed all occurrences of full stack with Optimizely Feature Experimentation
Refactor required due to rename of Full Stack product to Feature Experimentation.
Added Apache 2.0 License

Issues
FSSDK-8960

@russell-loube-optimizely - ready for review

LGTM


Bootstrapping new mattermost-docker does not work

I've followed the instructions described at Production Docker Setup on Ubuntu exactly on a current / fresh install of ubuntu 18.04  server and get "stuck" right after the docker-compose up -d command.
The containers are running and a curl http://localhost/ returns 502 Bad Gateway
The folder volumes/app/mattermost/logs is empty.
Could you help me and which useful informations can i provide?
Grateful,
CompileNix

Hi @compilenix.
I'm sorry, it seems that this documentation page is not up to date. I will not be able to fix it before next weeks, but to unblock you, I invite you to check at the README especially the part about UID/GID which may be your problem.

Hi @pichouk! The doc PR mentioned by @compilenix is ready for final dev review: https://github.com/mattermost/docs/pull/2027

Hi @amyblais, I answered on the PR. It's perfect :ok_hand:


[BUG] - unix /var/lib/kubelet/plugins/block.csi.vultr.com/csi.sock not accessible (for rook.io)

Describe the bug
Using rook.io, the rook pods rook-ceph-osd-prepare- fails to setup a PersistentVolumeClaim.
"describe pod" finally reports the event (warning) "MapVolume.SetUpDevice failed for volume "pvc-c869a0057b0c4904" : kubernetes.io/csi: blockMapper.stageVolumeForBlock failed to check STAGE_UNSTAGE_VOLUME capability: rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing dial unix /var/lib/kubelet/plugins/block.csi.vultr.com/csi.sock: connect: connection refused"" from the kubelet.
To Reproduce
Steps to reproduce the behavior:
(NOTE: This setup works fine on Azure AKS, with only the storageClassName adjusted.)

Create a fresh kubernetes cluster (probably 1 worker node is sufficient for reproduction)
For basic Rook setup: From the files in https://github.com/rook/rook/tree/master/deploy/examples, run kubectl apply -f for crds.yaml, common.yaml and operator.yaml, this creates CRDs, Roles and a rook-ceph-operator deployment/pod
Run kubectl apply -f for the following CephCluster yaml:

apiVersion: ceph.rook.io/v1
kind: CephCluster
metadata:
  name: rook-ceph
  namespace: rook-ceph
spec:
  cephVersion:
    # NOTE: see cluster.yaml in <https://github.com/rook/rook.git> for up-to-date image version
    image: quay.io/ceph/ceph:v17.2.1
    allowUnsupported: false
  dataDirHostPath: /var/lib/rook
  skipUpgradeChecks: false
  continueUpgradeAfterChecksEvenIfNotHealthy: false
  waitTimeoutForHealthyOSDInMinutes: 10
  mon:
    count: 3
    allowMultiplePerNode: false
  mgr:
    count: 2
    allowMultiplePerNode: false
    modules:
      - name: pg_autoscaler
        enabled: true
  dashboard:
    enabled: true
    ssl: true
  storage:
   storageClassDeviceSets:
    - name: set1
      # NOTE: change this to the number of nodes that should host an OSD
      count: 1
      portable: false
      tuneDeviceClass: false
      encrypted: false
      volumeClaimTemplates:
      - metadata:
          name: data
        spec:
          storageClassName: vultr-block-storage-hdd
          accessModes:
            - ReadWriteOnce
          # NOTE: rook seems to expect a raw, unmounted device "volumeMode: Block"
          volumeMode: Block
          resources:
            requests:
              storage: 40Gi

See events in kubectl -n rook-ceph describe pod rook-ceph-osd-prepare-* after the pod(s) got stuck.

Expected behavior
The pods rook-ceph-osd-prepare-* should disappear after a short time, and instead corresponding rook-ceph-osd-* pods (without -prepare-) should remain.
Additional context
I was using VKE with Kubernetes 1.23.x.

Hi, I have encountered the same situation.
I have a question about this matter.

do you plan to support volumeMode: Block in the future?
is it possible to support volumeMode: Block by changing this CSI implementation?

Thank you in advance.

I have a question about this matter.

do you plan to support volumeMode: Block in the future?
is it possible to support volumeMode: Block by changing this CSI implementation?

Unofficially: I'd expect this should be possible (with changes). When attaching block devices to vultr instances normally, you get the raw device and can do stuff to them (create LVM volumes, basic filesystems, use LUKS, etc).


Configure Babel for Jest to work with ES6

In the .babelrc file configure so that Jest will work with ES6 syntax.

We're already doing this.


CS-50: using export of tokens studio for figma, all assets sets, generat assets sets folders and files

This only has to make it through Thursday meeting, and stays on this branch. I'll move it to draft.


Changes for Memory-Space Aware Packing

Basically just adding a checkTouch function to determine whether the buffer has been touched in the current memory space so we can avoid unneeded host-packing when a buffer is moved back to host for some other read-only purpose.

When would you use this instead of just array.move( space, false )?

That would modify previousSpace if there was an actual move, right?
I still need to check if this fully fixes the issue I was seeing, which is that a pack was taking place on host due to a blueprint (or silo) output event triggering to move the data back to host on the first timestep prior to the collection event triggering.
The collection then happened on host, which should be fine, but uninitialized values were coming out for that collect, so I wanted to see if collecting from the device buffer instead fixed the issue.
Okay I just checked and this does fix the issue I was seeing with that first collection. Trying to think if possibly the bug is still in the collection operation, setting an offset incorrectly or something.

Hopefully this actually winds up not being needed and I can just throw this branch away, just need to figure out the root of the bug in that particular case.

That would modify previousSpace if there was an actual move, right?

Pretty sure previousSpace is only updated when the data is touched in a new space.
https://github.com/LLNL/CHAI/blob/d4a0a5700caed3858a8f285abc710cf7ac35fe59/src/chai/ManagedArray.inl#LL421C11-L421C11
https://github.com/LLNL/CHAI/blob/d4a0a5700caed3858a8f285abc710cf7ac35fe59/src/chai/ArrayManager.cpp#L204

Ah okay, I figured out the bug regardless and will be able to drop this branch once I am able to put together the fix in geos.
So if I have a buffer touched on device and invoke move(host, false) it will move the data to host, but not modify previousSpace. If I then subsequently -- in say a separate geos event that doesn't know if any other event has fired to move the data -- move(host, false) again there would again be data movement, since nothing in the underlying state is modified which would prevent the erroneous extra move. Basically I was hoping there was a mechanism to get the 'set of valid spaces' where the data is currently up to date (for a read-only operation), but looking at the chai record that probably isn't trivially recoverable.


Create CODE_OF_CONDUCT.md

Kindly review the Code of Conduct and provide your feedback. If you are in agreement, please accept the pull request.

Looks good. well done guys


Wrong default sortBy behaviour - uppercased strings go first

Describe the bug (required)
While using the default sorting method, the uppercased string goes before lowercased. People usually
Provide an example via Codesandbox! (required)
Check codesandbox to reproduce the issue.

Steps To Reproduce (required)
Steps to reproduce the behavior:

Click on sort arrow on status column
See that uppercased examples goes before lowercased

Expected behavior (Recommended)
People usually uses javascript .localeCompare() method to perform comparision between different strings. I believe that adding this option to sorthMethods would be appreciated.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Windows
Browser - all browsers, js logical issue

Additional context
I would like to create a PR with the solution

U can provide your custom sort method for the same.

Sure but I thought that it would be nice add this option to sortTypes.js in order to support sorting uppercased & lowercased strings out of the box.

I have the same problem with react sorting, is there a default that react-table handles this, or is it just by providing a custom sort method?


⚠ Bofrost.de has degraded performance

In 5872619, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1307 ms

Resolved: Bofrost.de performance has improved in 860e313.


Allow integer defaults for double fields

Fixes #150.

Ebert has finished reviewing this Pull Request and has found:

1 possible new issue (including those that may have been commented here).

You can see more details about this review at https://ebertapp.io/github/pinterest/elixir-thrift/pulls/151.


WIP: Regenerate with latest gir and trust upstream nullable annotations for return values

See https://github.com/gtk-rs/gir/pull/970 and https://github.com/gtk-rs/gir/pull/972
CC @sophie-h and @GuillaumeGomez

gio is blocked by https://gitlab.gnome.org/GNOME/glib/-/merge_requests/1708 , I'm not going to add overrides in Gir.toml for all these.


Spinbox still can be clicked even when mouse_filter set to Ignore

Godot version:
Godot 3.2.stable
OS/device including version:
Windows 10
Issue description:
The lineedit part of the spinbox still can be clicked even when mouse_filter set to Ignore. All element of the spinbox should not receive input when mouse_filter set to Ignore.
Steps to reproduce:
Make new scene
Add spinbox
Set the mouse_filter to Ignore
Click and also edit the spinbox
Minimal reproduction project:
Just do reproduction step.

I'll check into it.

The 'editable' property of line_edit in spinbox doesn't change even if mouse_filter is set to IGNORE in editor.  The changed value in the editor isn't getting transferred to the node's(line_edit) property.

I fixed this issue by making Control's set_mouse_filter a virtual function and overriding it in SpinBox.
@theoway If you can recommend some other way to solve this without using the virtual function or signal, then I will be glad to hear it.

@simpuid Your solution is just fine. Can't think of any other way. Good job ;)


Lack of 2 step ownership in the set_fee_to_setter

Github username: @JJtheAndroid
Twitter username: --
Severity: minor
Description
No 2 step ownership transfer could render the function useless if the setter made a mistake
Attack Scenario\
        #[ink(message)]
        fn set_fee_to_setter(&mut self, fee_to_setter: AccountId) -> Result<(), FactoryError> {
            self._only_fee_setter()?;
            self.fee_to_setter = fee_to_setter;
            Ok(())
        }

the following function allows the current fee_to_setter to change the setter. However the lack of a 2 step process i.e. having the proposed setter accept the transfer, could result in the original setter making a mistake and sending to a random address, making the function useless
Attachments

Proof of Concept (PoC) File

Run the following test in the factory contract
  fn initialize_works() {
            let accounts = default_accounts::<ink::env::DefaultEnvironment>();
            let mut factory = FactoryContract::new(accounts.alice, Hash::default());
            assert_eq!(factory.fee_to, None);
            set_caller(accounts.alice);
            //intends to set the fee setter to bob but instead makes a mistake and sets it to a dormant account (eve), making the function useless
            factory.set_fee_to_setter(accounts.eve);
            assert_eq!(factory.fee_to_setter, accounts.eve);

        }

Thank you for participation. After carefully reviewing the submission we've concluded that this issue is INVALID.
This is a conscious design choice, not a vulnerability.
We hope you participate in the future audits of ink!.


add the ability to run terraform plan only

Thanks @myjaffoo, this is cool!  I do have a tiny issue with a "dry run" which I'll explain:
One of the reasons I love terraform plan is the fact that what you see is what you get. However, that plan is only guaranteed if you apply that same "plan.out".  If dry run is set, a plan is shown. The next time dry run is not set, TFO will generate a new plan and apply it. And unfortunately, there could have been changes to the targets which means the plan might look different than before and it will be applied anyway.
This brings up a good conversation. The way the TFO volumeMount is set, it keeps all the plans in unique directories.  For instance:

Generation 1 (ie a brand new TFO k8s-resource created) will store the plan in $HOME/generations/1/main/plan.out.
When changes are made to the TFO k8s-resource, generation 2 will store the plan in $HOME/generations/2/main/plan.out.

One could be clever store the last successful "dry run" generation in "$HOME/dryrun.index. We can make the following assumptions to make use of this file.  The dryrun.index file contains a single line and keeps a single index. But then how does the user tell TFO that the next run should consume the plan from the dry-run?  I haven't given much thought to this yet.
My initial thoughts before this PR was planning on using a "postPlanScript" to pause TFO. The script would accept some event (that I also haven't determined) to continue or just error out which stops the execution.
I'm going to brew on this PR for a while. If you think of a good way to use the dry-run's plan.out so a new plan isn't generated, I'd love to hear from you.
Maybe something like
...
kind: terraform
spec:
  plan: "DryRun"
  #plan:  "UseDryRun" # looks for `dryrun.index`
  #plan: "" #  when not matching above, or null, defaults to generate plan and apply

I'm not sold on my ideas for this lol


Maya looks: support for native Redshift texture format

Feature
As we support OpenImageIO .tx files for Arnold, we should also add support for Redshift native texture support to avoid double conversions from .tx to .rstexbin.
I suggest using similar mechanism we use for conversion of .tx files - calling oiio maketx, but using redshiftTextureProcessor instead. This must be checked for existence and should reside in $REDSHIFT_COREDATAPATH/bin path (defined by redshift plugin in Maya).
[cuID:23yx6ke]

Documentation

Redshift Texture Processor Tool documentation

Example usage
On command-line
redshiftTextureProcessor S:/texture.png

Through Python subprocess
import subprocess
import os

if "REDSHIFT_COREDATAPATH" not in os.environ:
    raise RuntimeError("Must have Redshift available.")

exe = os.path.expandvars("${REDSHIFT_COREDATAPATH}/bin/redshiftTextureProcessor")
filepath = "S:/texture.png"
cmd = [exe, filepath]
subprocess.Popen(cmd)

Notes
Some file textures need to be flagged for certain conversion. For example spherical HDRI files will need the -isphere flag and forcing -l (force linear gamma: recommended for floating point textures) or -s (force sRGB gamma: recommended for integer textures) flags depending on the source files.
Looking at #2601 this might get involved - it might be non-trivial to decide how to interpret a source file. Should we need tools in-place to maybe allow artists to tag certain files for conversion to e.g. .rstexbin
The Render Time Considerations mentioned in the docs is also something to be aware of.


feat(startInstance): add overlay to Cloud start instance

Closes #2693
Note: to implement a 2-step approach for deploy+start instance like we do for platform, the deployment needed to be tweaked to be able to be controlled with start instance (get deploy config -> get start instance config -> deploy -> run instance)

Also I think some more basic tests could be added. E.g., follow along https://github.com/camunda/camunda-modeler/blob/develop/client/src/plugins/camunda-plugin/start-instance-tool/tests/StartInstanceToolSpec.js (there we for example have test for the unhappy path like user cancellation which I think are missing here)

I added some more tests on this one. They won't have as many as the platform because the platform handles the deploy part more manually. I tried to delegate this to the DeploymentPlugin as much as possible for Cloud.


Parent() example not recursive

import lark

t = lark.Tree("A", [lark.Tree("B", [lark.Tree("C", list())], list())], list())

# From https://github.com/lark-parser/lark/blob/master/docs/recipes.md#recipes
class Parent(lark.Visitor):
    def visit(self, tree):
        for subtree in tree.children:
            if isinstance(subtree, lark.Tree):
                assert not hasattr(subtree, 'parent')
                subtree.parent = tree

Parent().visit(t)
print(t)
print(t.pretty())
print(t.children[0].parent)
print(t.children[0].children[0])
print(t.children[0].children[0].parent)

Fixed. Thanks!

Thanks for the quick patch! Hope I did the minimum example right this time, now that you showed me it's no problem to create lark.Tree's like this. 😄

Hehe yes, your example made it very easy for me to find the problem and fix it. Well done :)


Consider using Flask mail extension for clock process

http://pythonhosted.org/Flask-Mail/

This issue was moved to Dallinger/Dallinger#175


put: pull-request removes .git/resource/changed_files

when you have a pr resource with params:  {list_changed_files: true} and you only have a plan with get and then task, in the task you get the .git/resource/changed_files (introduced in 0.17).
But, if there's a put:  action between the get and task (typically with params: { status: pending }, the .git/resource/changed_files is then missing in the task action.

Hey @gdamjan and thanks for the issue! I think perhaps the implicit get after put in Concourse is what might be the root problem for you here, and I think the following workaround can help you (read more about it in README.md):
put: update-status <-- Use an alias for the pull-request resource
resource: pull-request
params:
    path: pull-request
    status: pending
get_params: {skip_download: true}

Should put default to the above if concourse breaks the config without skip download?
Still failing with this option for me slighly different error.
update-status
2019/10/19 13:26:57 put failed: failed to read version from path: open /tmp/build/put/pull-request/.git/resource/version.json: no such file or directory

Closing this as I think it has been answered :)


invalid load key

{"error": "invalid load key, '\\x00'.", "prediction_id": "d3dfa92e-40c9-4866-b854-1437dd8b264a", "logger": "cog.server.runner", "timestamp": "2023-09-19T02:31:29.259351Z", "severity": "INFO", "message": "prediction failed"}
{"logger": "cog.server.runner", "timestamp": "2023-09-19T02:31:29.362471Z", "exception": "Traceback (most recent call last):\n  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/site-packages/cog/server/runner.py\", line 113, in handle_error\n    raise error\n  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/multiprocessing/pool.py\", line 125, in worker\n    result = (True, func(*args, **kwds))\n  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/site-packages/cog/server/runner.py\", line 334, in predict\n    return _predict(\n  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/site-packages/cog/server/runner.py\", line 370, in _predict\n    for event in worker.predict(input_dict, poll=0.1):\n  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/site-packages/cog/server/worker.py\", line 118, in _wait\n    ev = self._events.recv()\n  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/multiprocessing/connection.py\", line 251, in recv\n    return _ForkingPickler.loads(buf.getbuffer())\n_pickle.UnpicklingError: invalid load key, '\\x00'.", "severity": "ERROR", "message": "caught exception while running prediction"}

what does this error means? i think the error is not due to my predict code, but why is this happen?

Reformatting for readability:
Traceback (most recent call last):
  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/site-packages/cog/server/runner.py\", line 113, in handle_error
    raise error
  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/multiprocessing/pool.py\", line 125, in worker
    result = (True, func(*args, **kwds))
  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/site-packages/cog/server/runner.py\", line 334, in predict
    return _predict(
  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/site-packages/cog/server/runner.py\", line 370, in _predict
    for event in worker.predict(input_dict, poll=0.1):
  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/site-packages/cog/server/worker.py\", line 118, in _wait
    ev = self._events.recv()
  File \"/media/74nvme/software/miniconda3/envs/goods_inpaint_dev/lib/python3.8/multiprocessing/connection.py\", line 251, in recv
    return _ForkingPickler.loads(buf.getbuffer())
_pickle.UnpicklingError: invalid load key, '\\x00'.", "severity": "ERROR", "message": "caught exception while running prediction

This looks like a problem unpickling related to a null byte. It's a common problem that has to do with how the file is saved, loaded, and processed. I don't have any specific insight into why this is happening, though. You could start by searching Stack Overflow, et al for "pickle invalid load" ^1.
Could you share your model's repo or the predict.py function to get a better idea about what's happening?


New version: LinwoodCloud.Butterfly.Nightly version 2.1.0-beta.2

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

2024-03-11 05:23:25.917 [CLI ] No suitable installer found for manifest Microsoft.VCRedist.2015+.x64 with version 14.38.33135.0

(Automated response - build 869.)

How can I fix this?

@wingetbot waivers Add Validation-Installation-Error

I didn't get the same error in a Windows 10 VM.

it first appears in https://github.com/microsoft/winget-pkgs/pull/141032 for me, but I don't know why.
I upgraded to flutter 3.19 maybe this is the issue?


Maintenance

Hello!
Is this project still maintened?

@iamcryptoki , i think there is still interest but i have no idea who the leading developers are.this was migrated from kuvbernetes to the cloudstack project to prevent it from being deleted completely. Are you volunteering?

Thank you for your reply!
Not at the moment but I would be more than happy to volunteer in the very near future if the project is no longer maintained.

@iamcryptoki any help and volunteering is welcome!

@iamcryptoki Thanks for the offer!
The project is very much alive, but still needs some love from a few sides.
If you have any issues, input or pull requests, please post them on the Github tracker.
And if you want to try the provider for yourself, you can always build it locally.
I recommend merging #8 , #10 , #11 and #9 as they contain various fixes and the IP whitelisting feature that I just finished. Any feedback on #9 is highly appreciated.

@iamcryptoki You can also take a look at https://hub.docker.com/repository/docker/swisstxt/cloudstack-kubernetes-provider - I'm occasionally pushing development versions there.

Release management and CI is being addressed in #7  and #16 .
We have access to Dockerhub now, and containers will be pushed there.
Please stay tuned for the v1.0 release.

Closing this, @iamcryptoki pl follow #7 and #16  and wait for v1.0 something soon (in coming weeks/months).


Locations with hyphens can't be chosen

The following countries/territories can't be selected if you type the hyphen:
Guinea-Bissau
Ras al-Khaimah
Saint-Martin (French part)
Umm al-Quwain

Before you type the hyphen, you can choose an item:

As soon as you type the hyphen, the results vanish:

Probably also related to https://github.com/alphagov/govuk-country-and-territory-autocomplete/issues/40 @nickcolley

Similar happens for searching on people's - the apostrophe breaks it. Just people matches against North Korea for example.


Conflict with jQuery File Upload

I've got jQuery File Upload on another file input that is uploading files directly to S3. When I drag and drop an image into EZDZ (does not happen when file is selected by clicking the drag area), it automatically triggers the file uploading on the other file input.
To clarify, the form has two file inputs, both with different names, classes, and ids. The only thing that makes them the same is that they are both input type="file". One file input is controlled by ezdz, the other by jQuery File Upload.
The only way I could get around this, was by disable, then enabling jQuery File Upload using ezdz events.
This is more of a heads up. Is there something I'm missing instead of this monkey patch?
		enter:  function() {
			$('.file-direct-upload').fileupload('disable');
		},
		accept: function(){
			$('.file-direct-upload').fileupload('enable');
		},

Hello,
Ezdz has no real drag&drop feature. This is the native behavior of any input type file (except on IE).
So I doubt there is something I can do here.
If you provide a JSfiddle or something I could look at it one day.


[GSSOC'23] - QuoteBot for Telegram

I want to add QuoteBot into the repo. The "QuoteBot" is a Telegram bot that delivers a daily dose of inspiration through a collection of random motivational quotes. By simply typing the /quote command, users can receive a beautifully crafted quote from renowned personalities across various fields.
These quotes are carefully selected to uplift spirits, provide encouragement, and ignite motivation in users' lives. With its easy-to-use interface and seamless integration with Telegram, the QuoteBot aims to inspire users and help them maintain a positive mindset as they navigate their daily challenges. Join the QuoteBot and let its uplifting quotes brighten your day and fuel your drive towards success.

Please assign me this issue under GSSOC'23!! I would love to work on this.

I am excited to work on this project. Maybe I can help in further implementation of the project. I request you to please assign me this project.

Please assign me this issue under GSSOC'23!! I would love to work on this.


cfo: build all fuzzers first

For some reason, all VOPR runs timed out on CFO:
https://github.com/tigerbeetle/devhubdb/blob/9ee578f1f39530573dafcf5dfa1c856f312ef1bc/fuzzing/data.json#L530-L617
(note "ok": false and "seed_timestamp_end":<PHONE_NUMBER>) for each one of the seeds.
I wasn't able to reproduce this locally or on the CFO machine, but my hypothesis here is that the build system deadlocks with a lot of concurrency.
Restore the behavior of building all code once before fuzzing, even if we don't need some of the fuzzers.
Note that I've unintentionally didn't remove building of minor fuzzers, so those dont suffer concurrent build issues, and perhaps are not timing out for this reason.

Sorry for flipping my mind back and forth here @sentientwaffle :rofl:


Add support for manual log rotation.

I wanted this for logging a stream of JSON events where a certain event should trigger a rotation.
This adds a new ContentLimit that never rotates the log automatically. To force a rotation the user has to call FileRotate::rotate() which is now part of the public API.

Thanks for the PR. Could you add some tests to show that it works and so we avoid future regressions?

Test case added :)


Open Dropdown options are not refreshed when multiple select value changed programmatically

Prerequisites

[x] I have searched for similar issues in both open and closed tickets and cannot find a duplicate
[x] The issue still exists against the latest master branch of Select2
[x] This is not a usage question (Those should be directed to the community)
[x] I have attempted to find the simplest possible steos to reproduce the issue
[ ] I have included a failing test as a pull request (Optional)

Steps to reproduce the issue

Open a multiple select dropdown.
Change the value programmatically: $('select').val('AK').change();

Expected behavior and actual behavior
When I follow those steps, I see...  The options in the input area are updated.  The options in the dropdown are not updated.
I was expecting...  The options in the dropdown to be updated to reflect the new value of the select element / the items that have been selected programmatically.
Environment
Browsers

[x] Google Chrome
[ ] Mozilla Firefox
[ ] Internet Explorer

Operating System

[ ] Windows
[x] Mac OS X
[ ] Linux
[ ] Mobile

Libraries

jQuery version: 1.11.2, 3.1.1
Select2 version: 4.0.3

Isolating the problem

[x] This bug happens on the examples page
[x] The bug happens consistently across all tested browsers
[ ] This bug happens when using Select2 without other pluigns
[ ] I can reproduce this bug in a jsbin

Notes
I've implemented a Select All / None feature in my custom dropdown.  I need the dropdown items to be updated immediately because the dropdown is open and the options are displayed at the time I change the select value programmatically.
I've searched for a report of this problem everywhere I can think of and I can't find a report of it, nor can I find a workaround.  I would really appreciate any help in finding a fix/workaround for this.

The only way I can think of is to trigger the back end select toggle, reload Select2 (normal procedure after changing options programatically) and then trigger the open command again - It'd be a dirty way hacky way of doing it but it'd be the quickest method I can think of without having to rebuild parts of Select 2 to allow this .

For those interested, I found a workaround.  After I change the value of the select element, I update the aria-selected attributes of the results, like so:  $('select').data('select2').$results.children().attr('aria-selected', [true|false]);
BTW, is this the best way to get a reference to the select2 instance: $('select').data('select2') ?


Channel tab not refreshed when settings change via setConfig()

Copied bug from: https://github.com/OfficeDev/TeamsFx/issues/10077
Details from original post:
Describe the bug
When changing the settings of a tab from the tab's settings page & calling the setConfig() method, the Teams client does not reload the tab to update based on the new changes. However, navigating off & back to the tab does pick up the saved changes, proving the changes were saved.
To Reproduce
Steps to reproduce the behavior:

create a new tab project with a new channel tab
add a tab that displays the current
add a configuration page that changes the entityId when the settings dialog's SAVE button is clicked
observe the tab does not refresh and shows the old value
navigate away from the tab and come back, observe the entityId value has changed, proving the save settings page worked correctly

Expected behavior
Channel tab is refreshed by the Teams client when the settings are saved on the tab.
Screenshots
See the following video that demonstrates the issue:

GH - OfficeDev/TeamsFx Issue#10077 - Watch Video

VS Code Extension Information (please complete the following information):

OS: macOS
Version v5.2

n/a---
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: b743e43d-95cc-3d1e-b142-31a16369f97e
Version Independent ID: b743e43d-95cc-3d1e-b142-31a16369f97e
Content: Teams Toolkit Overview - Teams
Content Source: msteams-platform/toolkit/teams-toolkit-fundamentals.md
Service: msteams
GitHub Login: @zyxiaoyuer
Microsoft Alias: zhany

Please Track the question from here-https://github.com/OfficeDev/microsoft-teams-library-js/issues/1963

This is still happening with teams-js 2.17.0

@LeonYasoon -We have already raised the bug for the same. We will inform the engineering team that issue happens with 2.17.0 as well.
Could you please track this issue from here for updates-https://github.com/OfficeDev/microsoft-teams-library-js/issues/1963


Using Go code in C++ via cgo (LLVM)

What version of rules_go are you using?
1.23.0
What version of gazelle are you using?
0.40.0
What version of Bazel are you using?
bazel 7.4.1
Does this issue reproduce with the latest releases of all the above?
What operating system and processor architecture are you using?
Linux, x86
Any other potentially useful information about your toolchain?
toolchains_llvm 1.2.0
llvm 19.1.0
What did you do?
Hi,
I'm trying to call Go code from my cc_library but I'm struggling to define the dependencies properly.
Is there a canonical example on how to achieve that? I was following roughly along this example
cc_library(
  name="mylib",
  deps = [ ":mygolib.cc"],
)

go_binary(
  name="mygolib",
  cgo = True,
  linkmode = "c-archive",
  srcs = ["mygolib.go"],    # contains code marked with // export
)

When using mylib in a cc_test I'm getting linker errors:
ld.lld: error: undefined symbol: __res_search
>>> referenced by cgo_unix_cgo_res.cgo2.c

I suppose that due to using llvm while the cgo library seems to be compiled with a different compiler (though here I'm mostly guessing). I tried playing around with adding copts = ["-lstdc++ -lc++abi"] but couldn't make it work.
Do you have any tips what to do here?

It's working now by adding linkopts = ["-lresolv"] to the cc_test or cc_binary that consumes the mylib library.


posix-compatible lowercasing

The bash substitution is not portable and will not work on macOS runners (old version of bash?). Using tr is portable and should work across both POSIX systems.
Signed-off-by: Matthew Fisher<EMAIL_ADDRESS>

confirmed working in https://github.com/deislabs/krustlet/pull/218/checks?check_run_id=616242004.


crypto unit tests

Feature request template
Feature description
All functions in https://github.com/spacemeshos/go-spacemesh/tree/master/crypto should have unit tests.
Reasons for adding this feature
You can never have enough tests.

Back again.  Added randg_test.go ( PR #92 ), Travis didn't like the overall effect.  Won't fix stuff outside of crypto tests ( which seemed to work in Travis).  API and p2p are blowing up
Notice that "findnode.go:131:15: undefined: "github.com/spacemeshos/go-spacemesh/crypto".MinInt32" so probably something will be changing in crypto soon anyway.
Will check in some time this week and take another look.  Next target probably getting keys_test.go nearer 100%

Looking at the develop branch - we have pretty good coverage - we should add a bit more tests to keys.go

I'm closing this as I think we are at a good place regarding coverage. Please reopen if you feel like adding additional tests.


Deleted unused file Buckets.png

There is an unused file in the repository
When cloning repo on a mac, there is a case insensitivity issue too:
warning: the following paths have collided (e.g. case-sensitive paths
on a case-insensitive filesystem) and only one from the same
colliding group is in the working tree:

  'docs/images/Buckets.png'
  'docs/images/buckets.png'

buckets.png appears to be used in the readme, but Buckets.png is not

LGTM, thanks for this update Jason.


feat: top pages and backlinks from ahrefs

Add top pages and backlinks queries from Ahrefs to be used by the SEO pre go-live assessments and others.
http client to use can be passed via constructor and the usual fetch is passed in the createFrom. This is required as we'd want to reuse the client in the pre go-live scripts, where we use node-fetch-cache to cache the responses.
Includes also changes from https://github.com/adobe/spacecat-audit-worker/pull/215 as well

:tada: This PR is included in version 1.13.0 :tada:
The release is available on:

v1.13.0
GitHub release

Your semantic-release bot :package::rocket:


Cannot replicate active_learning\qnIPV_experiment.py with WISKI model

Hi,
Sorry to bother you. I see your excellent paper which is related to WISKI. I have been trying to replicate WISKI model but I ran into bugs that I cannot deal with.  I wrote an issue in WISKI github but there has no reply.
When I run active_learning\qnIPV_experiment.py with WISKI model, it says that
  File "C:\Users\.conda\envs\env01\Lib\site-packages\botorch\optim\closures\model_closures.py", line 176, in closure log_likelihood = mll(                      ^^^^   File "C:\code\online_gp-main\online_gp\mlls\batched_woodbury_marginal_log_likelihood.py", line 49, in __call__     for _, prior, closure, _ in self.named_priors():         ^^^^^^^^^^^^^^^^^^^^ ValueError: too many values to unpack (expected 4)
So I rewrite the FixedNoiseOnlineSKIGP in active_learning\qnIPV_experiment.py Line 84.:
model = FixedNoiseOnlineSKIGP(             init_x,             init_y.view(-1, 1),             init_y_var.view(-1, 1),             GridInterpolationKernel(                 base_kernel=ScaleKernel(                     MaternKernel(                         ard_num_dims=2,                         nu=0.5,                         lengthscale_prior=GammaPrior(3.0, 6.0),                     ),                     outputscale_prior=GammaPrior(2.0, 0.15),                 ),                 grid_size=30,                 num_dims=2,                 grid_bounds=torch.tensor([[0.0, 1.0], [0.0, 1.0]]),             ),             learn_additional_noise=False,             likelihood=ExactMarginalLogLikelihood         ).to(device) .
I define Likelihood. But I tried batchedWoodburylikelihood and exactmarginalloglikehood, errors are shown as Traceback (most recent call last):   File "C:\code\online_gp-main\experiments\fixed_noise_regression\wiski_regression.py", line 194, in <module>     main(args)   File "C:\code\online_gp-main\experiments\fixed_noise_regression\wiski_regression.py", line 99, in main     mll = mll_type(model.likelihood, model)           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\chenyiran\.conda\envs\env01\Lib\site-packages\gpytorch\mlls\exact_marginal_log_likelihood.py", line 33, in __init__     raise RuntimeError("Likelihood must be Gaussian for exact inference") RuntimeError: Likelihood must be Gaussian for exact inference.
I have tried to replicate but I ran into errors that I cannot deal with. Sorry to bother you. Could you tell me how to run qnIPV_experiment.py or Wiski_regression.py succeessfully? Thanks a lot!

please follow the link below to configure the environment variables and path, and reproduce WISKI model.
https://github.com/wjmaddox/online_vargp


Some words that an RSE has a special obligation for making results explainable.

to sort out #275 , #266 .

Hi all,
I tried to incorporate all your feedback. Let's see how this round works...

I got some text & references but just couldn't find the time to put it here yet. I'd be thankful if we could delay that until the next meeting, so if have a chance to present some text.

yes, I think that would be very helpful

OK, since @hvwaldow already has a follow-up PR, we merge this one.


Is there a support for the parser httptools?

Hi,
I use the sanic framework, a Flask-like Python 3 framework with asyncio. This framework uses the httptools parser and I would like to known if there is an equivalent module in webargs for this parser.
If it's not the case, do you have some docs which explain how I can add a supported "framework" to webargs. I think this parser is quite similar to sanic, e.g. https://github.com/channelcat/sanic/blob/master/docs/sanic/request_data.md
Thanks,
Damien G.

I check httptools. It's the http parser used for the Sanic web framework. A sanicparser should be great. I'll try to check the core.Parser to understand how doing that.

I'm not familiar with httptools.
I recommend taking a look at the existing Parser subclasses to see implementation examples. If you end up implementing webargs-sanic, let me know, and I can link to it in the docs.


wrong path to utils.sh if there is a third level like in Extras scripts

If I run the spotify script from the menu then the dialog_handling.sh cannot find den utils.sh because there is one more layer.
output from dialog_handling.sh:
CURRENT_DIR: /home/dashorty/.tmux/plugins/tmux-menus/items/extras
SCRIPT_DIR : /home/dashorty/.tmux/plugins/tmux-menus/items/scripts

https://github.com/jaclu/tmux-menus/blob/7a388b5f9eff159fc79e629112e7d567332cb921/scripts/dialog_handling.sh#L436-L443

I tested it on several systems, so the checks should hopefully work on all platforms! Im closing this now, but please re-open if the issue was not resolved!


Should complete code coverage

I added a couple tests that should entirely cover didUpdateHook. Codecov reports 100% on my fork, however I'm very new to code coverage, so I hope this pull request is not in vain.. 🙏

Alright thanks for your PR. I'm going to merge it and update pull request :)

Yahoo! Thanks a ton!


User groups

NewCantus needs to have users with different permissions. Copied from an email from Debra:

Here, briefly is what I'm thinking right now:
--Contributor
--Co-contributor
--Editor/Proofreader (just one level for this, with access only when provided by Project Manager or Admin)
--Debra or "Project Manager" (no access to anything dangerous, i.e., no programming code sections)
--System Administrator (full control, but make sure they don't change any content)

For the editor/proofreader level, my interpretation is that the Project Manager would assign specific sources to that user, for which they'd have access to either edit or proofread the source. If the user has editor access to a source, they'd be presented with all of the Fulltext, Volpiano, metadata fields but no "proofread" checkboxes when editing a chant, while if a user has proofreader access to a source, they would have all of those fields plus the checkboxes.
Thus, for each editor/proofreader, there would be some sources for which they cannot edit anything, some sources for which they have limited editor permissions, and some sources for which they have extended proofreader permissions.

this has been implemented! closing.


Skip signature files (#37)

This fixes #37 by skipping any .fsi signature files that are found in the project. It also updates Ionide.ProjInfo in paket.lock, as I was having issues loading my test project example with 0.48.
Tested by running the CLI tool against the sample file in #37, which now exits successfully.

Thanks for this!

Thanks, released in version 0.9.0.


Move celery to optional requirements

This looks like a much bigger diff than it actually is because I also alphabetized the extra requirements.
Real changes:

move celery & flower to optional requirements
move statsd to optional requirements
remove version caps for celery/flower (honestly prompted because I had far fewer flower crashes when I was running 0.8x than 0.7x)

@mistercrunch not sure how you manage your install -- but would you want celery to be added to the devel config?


On safari (when using pickadate) the page scrolldown when you open the datepicker

Currently on our site the only place you can see this happen is safari on mobile (IOS) since we only use pickadate for our mobile datepicker.
gozengo.com
I have no idea why this only happens in safari.
Any ideas?

I think I have the same issue. Position ends up slightly too low, i.e covering the "today" button until you scroll up. Maybe it's due to the IOS way of dynamically hiding the address bar or something?

It happens on regular safari as well (on our site) so I don't think it is
an iOS issue
On Wednesday, February 17, 2016, Fredrik Blomqvist<EMAIL_ADDRESS>wrote:

I think I have the same issue. Position ends up slightly too low, i.e
covering the "today" button until you scroll up. Maybe it's due to the IOS
way of dynamically hiding the address bar or something?
—
Reply to this email directly or view it on GitHub
https://github.com/amsul/pickadate.js/issues/836#issuecomment-185261718.

Having the same issue as well

This issue has been a huge pain for a site I'm working on. Page scrolls down to the bottom on Safari and then can sometimes not open again until you focus out of the window. I've resorted to turning off the translateY and transitions in the CSS and setting pointer-events to none unless it has the open or focus classes


ASN table should also allow entry of BGP community tags

Many networks are just one ASN, and in that case the ASN table in its current form is a bit useless.
Propose to add a "bgp_comm_tag" column to this table and associated APIs to allow networks to use BGP community tags to associate customer networks with cachegroups.

@jhg03a - any thoughts on this one?

Both ASN and BGP Community Tags are arbitrary mappings to locations.  So for some implementations it may make sense to use one over another.  They could be used as part of an implementation of a czf generator.  I'm not sure I'd just add it to the ASN table though as it would break the 1:1 mapping of cachegroup to ASN.  I think it would be easier/better for it to be a peer structured like the ASN table, then it can have its own 1:1 relationship of cachegroup to BGP Community Tag.  It's more flexible to have it, but it's not a huge priority for me personally.

Also for what it's worth, this request predates the implementation of deep_czf as an option which may be used similarly if the generator implementation chooses to use this information.

Also for what it's worth, this request predates the implementation of deep_czf as an option which may be used similarly if the generator implementation chooses to use this information.

sounds like we have a workaround! i think i'll close. @knutsel please reopen if u feel strongly about this request.


Adding More Transport Protocols - A Proposal

What follows is a thought dump, so sorry if it rambles a bit.
This module is in a pretty good state with the HTTP Protocol, so i started thinking about what the next protocol that could be added.
As I thought about it,  I realized a couple things.
This first is we probably need to make our top level Receiver and Emitter a little more generic.  At the moment,  they have things related to the HTTP Protocol in them.
This second thins is when we add a new transport protocol in, we are also potentially going to increase the dependencies for each one added.  My assumption would be that we would use a library related to that transport protocol for emitting events.  For example,  if we added MQTT then we might use the mqtt module.
The problem,  and maybe it isn't a problem, is that a user might only want or need 1 protocol,  and if we had multiple protocols implemented,  then they would be getting more dependencies than needed.
One solution to this could be creating new modules that depend on this module as a base for each of the new Protocols.   I would assume we would just keep the HTTP protocol in the base module since, and i have no data to back this up, it might the most common to use.  So if a user wanted the MQTT protocol, they could just install that module.  However; if they wanted to use multiple protocols,  I could see this becoming a little messy.
Another solution i thought about was some sort of plugin architecture,  but i not really sure of the UX of it. with this case we would still be creating separate modules to plugin,  but there would be one "entry point" to creating new Emitters and Receivers
Again,  this was a thought dump to start a discussion on our best path forward.

@cloudevents/sdk-javascript-maintainers and anyone else

@lholmquist thanks for bringing this up! As you know, I'm interested in moving ahead with MQTT protocol support in the near future.
One thing to consider is that we don't really need to support the MQTT protocol itself, we just need to provide a means for developers to express a CloudEvent as an MQTT publish message. In fact, I would say with regard to HTTP, our choice to use Axios to send over the wire may not actually be the best thing. The module could be smaller and more concise if we simply provided methods similar to this.
// send a cloud event
const ce = new CloudEvent({ /** attributes */ });
const structuredHTTPMessage = HTTP.asStructured(ce);

// then it's up to the developer to actually send these using whatever HTTP module they like
axios.request( {
  method: POST,
  data: structuredHTTPMessage.body,
  headers: structuredHTTPMessage.headers
});

// receive a cloud event using express
app.post('/', (req, res) => {
  const incomingEvent = HTTP.receive(req.headers, req.body);
});

In fact, that's essentially how our Receiver works today. It doesn't really know about the HTTP protocol itself - it just accepts some headers and a body. It's a little strange to me that we do it so differently by actually sending an event over the wire when emitting an event.
I say all of this to emphasize that we don't really need to worry about the MQTT protocol. We just need to provide a way to transform a CloudEvent instance into an MQTT publish message, and to read an MQTT publish message and convert it into an event. This is good, because the mqtt module is not small. Including all of its dependencies, it runs about 10MB.

It's a little strange to me that we do it so differently by actually sending an event over the wire when emitting an event.

One reason i can see having an Emitter, is to make the users life a little easier with not having to worry about setting the correct method and those other tiny details.    But I think that making this library smaller,  and focused on just formatting(receieving and sending) the CloudEvents might be a good direction to go in
That would allow a user to use their http lib of choice(or other protocol lib of choice).
This would also solve the issue #314 about the Emitter being static,  since we wouldn't have an emitter function anymore


Proposal to add IconButtonThemeData

Why don't we have IconButtonThemeData just like ElevatedButtonThemeData or TextButtonThemeData? We do have IconThemeData to provide consistent icon styling across the whole app, but when it comes to styling IconButton, it is a problem. It inherits styling properties from the default values of ThemeData such as focusColor, disabledColor and  highlightColor. I don't think it's convenient to build custom icon buttons using Theme Data.
Use case
The most common use case that I can think of is to change the icon color of icon button when it is pressed.
Proposal
There will be IconButtonThemeData and its style property as ButtonStyle. The implementation will be similar to the following code:
final ThemeData myTheme = ThemeData(
  iconButtonTheme: IconButtonThemeData(
    style: ButtonStyle(
      // Icon Color
      foregroundColor: MaterialStateProperty.resolveWith<Color>((states) {
        if (states.contains(MaterialState.pressed))
          // Make icon color grey when it is pressed
          return Colors.grey;
        // Default color
        return Colors.blue;
      }),
      // We can also change the icon based on material states
      iconData: MaterialStateProperty.resolveWith<IconData>((states) {
        if (states.contains(MaterialState.hovered))
          // Change icon
          return Icons.cancel;
        // Default icon
        return Icons.add;
      }),
    ),
  ),
);

IconButtonThemeData is now available which includes a ButtonStyle property

Assigning @QuncCccccc as they fixed the issue.


Add Dissonance theme

Dissonance is my version of Symphonic, with red cursor and full black background

Thank you


[Outlook] (autolaunch) Only simple CORS supported

Docs Build status updates of commit f221aea:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 6a3eddc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[WIP]: Remove apply patch

This PR addresses the diffing of arrays of objects bug in the applyPatch
To reproduce the bug try create a Single container app:

Add 1 local volume
Add 1 external volume
! Change something in the JSON, this is important to update the base config
Try adding one more local volume
Notice that it gained the external field in the JSON

Then pull the branch and validate that the problem is gone.
On top of that, since we have a battle tested diffing algo we don't need deleting KEY_VALUE_FIELDS anymore, jsondiffpatch handles that as well. The issue was that an attempt of editing an env var in the json would duplicate the var.
Less magic is better. Less tests in the test suite is better as well.
🙇

Konfliktz!

@nLight what are we planning to do with this guy?


[fix] correct ReadOnlyFormData generator implementation

#1830 prompted me to check the actual code and I found some oddities, it will now follow the written types correctly.

Is there some kind of test we can add for this?

Good idea, I've added some unit tests


Using a unique IV for each encryption / using PBKDF2

Currently your IV is set on class instantiation. So each encryption will use the same IV. I would commend you to use a new unique IV for each encryption run.
As second I would suggest to use hashlib.pbkdf2_hmac to make you package more secure.
Best regards

Thank you for letting be notice, I will fix that as soon as possible. Thank you again!


Wiki rename flashes modal that wiki has moved

Steps

Go to a project you are an admin on
Go to the the wiki tab
Create a new wiki
On the new wiki, click on the name of the wiki in the upper left above the wiki menu
Edit or rename the wiki
Click to save the edit
Look at the screen

Expected
I would expect the rename to be saved, nothing flashing on the screen
Actual
After saving the edits to the wiki name, there is a brief flash of a modal that says the wiki has been moved.

I think this is a desired behavior. If the user's internet is slow or under some lag spike, the pop up modal gives user a sense for what's going on. It also block the page to prevent user doing other action when server is handling user's rename wiki request. @erinmayhood any thought?

Closed via https://openscience.atlassian.net/browse/OSF-4225


Make editor text size configurable

The text in the editor is now much smaller than it used to be. I much prefer it the way it was.
Can this be made configurable?

Font is much bigger in my case...

I support this :-) especially for hi-res monitors a configurable fontsize would be great.

Fixed


Header detection not working

Hi,
First of all, congrats for this project.
Below my Request Headers and configurations
// initalizer
export default function (i18next, i18nextMiddleware) {
  i18next
  .use(Backend)
  .use(i18nextMiddleware.LanguageDetector)
  .init({
    backend: {
      // path where resources get loaded from
      loadPath: path.join(__dirname, '..', '/locales/{{lng}}/{{ns}}.json'),
      // path to post missing resources
      addPath: path.join(__dirname, '..', '/locales/{{lng}}/{{ns}}.missing.json'),
      // jsonIndent to use when storing json files
      jsonIndent: 2
    },
    fallbackLng: config.locales.fallbackLng,
    whitelist: config.locales.whitelist,
    preload: config.locales.whitelist
    //, debug: debug.enabled
  })
  debug('i18next initialized')
}

initializer.i18next(i18next, i18nextMiddleware)
app.use(i18nextMiddleware.handle(i18next))
```

Everytime i request with custom `accept-language` header (see below), the language in `req.language` rolling back to my `fallbackLng`

```
Content-Type: application/json
Accept-Language: en
Cache-Control: no-cache
```

However, It's working with queryString paramater like lng=en

Any solutions relative to my issue?

not sure but Accept-Language should be something like: `Accept-Language: da, en-gb;q=0.8, en;q=0.7``
see:  http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html 14.4
detection is like: https://github.com/i18next/i18next-express-middleware/blob/master/src/languageLookups/header.js#L18

if not the case PR would be welcome to support header not providing weighted languages (pure strings in format en) or i could add this and publish new version.

Seems like I have the same problem. My req.headers['accept-language'] is
de-DE,de;q=0.8,en-US;q=0.6,en;q=0.4

The found variable in header.js returns: [ 'de-DE', 'de', 'en-US', 'en' ] but anyway req.language / req.languages just return the fallbackLng in my routes (en).

can you try with v0.1.1 i just published....should fix the issue.

Great, thank you - that works.

@jamuhl
hi, does not looks to work for me :
TypeError: Cannot read property 'forEach' of undefined
    at /Users/userName/Documents/projects/projectName/node_modules/i18next-express-middleware/lib/LanguageDetector.js:103:19
    at Array.forEach (native)
    at LanguageDetector.detect (/Users/userName/Documents/projects/projectName/node_modules/i18next-express-middleware/lib/LanguageDetector.js:97:22)
    at /Users/userName/Documents/projects/projectName/node_modules/i18next-express-middleware/lib/index.js:36:96
    at Layer.handle [as handle_request] (/Users/userName/Documents/projects/projectName/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/Users/userName/Documents/projects/projectName/node_modules/express/lib/router/index.js:312:13)
    at /Users/userName/Documents/projects/projectName/node_modules/express/lib/router/index.js:280:7
    at Function.process_params (/Users/userName/Documents/projects/projectName/node_modules/express/lib/router/index.js:330:12)
    at next (/Users/userName/Documents/projects/projectName/node_modules/express/lib/router/index.js:271:10)
    at jsonParser (/Users/userName/Documents/projects/projectName/node_modules/body-parser/lib/types/json.js:100:40)

@jamuhl
Found a bug, see below :
Issue code : https://github.com/i18next/i18next-express-middleware/blob/master/src/LanguageDetector.js#L56
let detections = this.detectors[detectorName].lookup(req, res, this.options);
if (typeof detection === 'string') detections = [detections];

should be
if(!detections) return
if (typeof detections === 'string') detections = [detections];


FixCommandHelp - fix typo

:+1:
(but please regenerate the README, @ see contributing ;) ;)  wondering why this passes on 5.3 though :P)

i fixed the readme file directly, then i thought i should fix it in php generator, then i commited only later :D

because it's skipped:

Unsupported symfony/console version, Symfony\Component\Console\Output\BufferedOutput was added in 2.4.

i fixed the readme file directly, then i thought i should fix it in php generator, then i commited only later :D

nice one : D


Add enable_private_network & enable_ip6 options

I'm going to merge this, though I'm not actively maintaining this project.
Would you like to become vagrant-vultr maintainer?

Sure. Thanks
On 23 Jan 2017 07:57, "Alex Rodionov"<EMAIL_ADDRESS>wrote:

I'm going to merge this, though I'm not actively maintaining this project.
Would you like to become vagrant-vultr maintainer?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/p0deje/vagrant-vultr/pull/6#issuecomment-274421600,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEr1t3_97wN2U0VsDu0MjpzVtmmNVW79ks5rVF1sgaJpZM4LqJgE
.


Ws2022

ws2022

@nwoodmsft could you please take a quick look when you get a chance/review?

Is there a test pass that shows this working?

Also, are you going to revert the code with the "revert" comment?

Also, are you going to revert the code with the "revert" comment?

reverted those changes (they were needed for the testing)

Is there a test pass that shows this working?

yes https://msazure.visualstudio.com/msk8s/_build/results?buildId=68497418&view=logs&s=16339c14-0211-5cee-da04-6888d48254ee

@nwoodmsft @pradipd @apluchik need help to merge this PR, I don't have write access


Support Visual Studio 2022

It is said that currently the API doesn't work with the newest Visual Studio. We should support it somehow.

I had recent problem with mingw64 being in my path (environment variables).  Something in protobuf build was looking for unistd.h, ?abseil?
First error was: Build failed ?protobuf-d.vcxproj ( i forget exact message)
Removing mysys64/*/mingw* from path fixed build issues for me, as zlib was no longer found by CMake.
Suggest we need output of:
cmake -B build -G "Visual Studio 17 2022"
and/or
Build -> Solution (in Visual Studio)
to troubleshoot further, as I cannot repro now that I cleaned up my path pollution.

Hi, @Nickrader
I believe it was broken long before integration with mingw. Might be fixed by update of the protobuf submodule, could also require fixes of new compilation issues.

Yeah, seems to be working now.  Uninstalled 2019, rebuilt 'build' folder, rebuilt in VS.  Examples and tests run. Property pages showing latest release for MSVC, 14.37.32822

Done.


Unregister monkey-run

Recently, I changed my organization name in Github from Monkey-Run to MonkeyRun.
Would you please unregister my bower package "monkey-run"?
I want to create a new package pointing to the new repo address.
Thanks!

done


chore: Switch Silverblue, Kinoite & Sway base images to official quay URL

The only difference in packages is some addition of virtualization qemu packages iirc, don't know the exact changelog (might be the regular update though, P5 says there's no change in packages).

Closing until Timothee advises us to switch https://discord.com/channels/1202304549294190602/1202304549944316007/1316104338040553534


fix(cli): Fix tauri init

Switch to Tauri lib instead of require to import toml.

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[x] No

The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
Fix Error: Cannot find module '../../../../tauri/Cargo.toml'

Closing in favor of https://github.com/tauri-apps/tauri/pull/1374
The require() call was wrong, it shouldn't be dynamic paths like that.


Cat loader2

Hello @williamFalcon! Thanks for opening this PR.

In the file flash/data/data_utils.py:

Line 11:5: E121 continuation line under-indented for hanging indent
Line 11:5: E125 continuation line with same indent as next logical line

Do see the Hitchhiker's guide to code style


chore(deps): update lscr.io/linuxserver/lidarr docker tag to v8

This PR contains the following updates:

Package
Update
Change

lscr.io/linuxserver/lidarr
major
1.0.2 -> 8.1.2135

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will ignore this upgrade and you will not receive PRs for any future 8.x releases. However, if you upgrade to 8.x manually then Renovate will reenable minor and patch updates automatically.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


Paper question: no use of validation set during U2Net training

Hi,
in section 4.3 of the paper you write:

We train the network until the loss converges without using validation set

From that sentence, I understand that you wait for the training loss to flatten out (converge), without using any validation set to check for over/underfitting.
Am I getting this right?
Thanks and have a great one!

Yes, that's true. Actually, we did use MSRA2.5k as the validation. But all
the other test sets have different characteristics from MSRA2.5K. Good
performance on MSRA2.5K won't guarantee better generalizations on different
datasets. We then remove the validation set and train to converge. If you
want, you can plot the -log(loss). The flat curve of -log(loss) should make
more sense because the loss decreases very slight in the later training
stage and -log(loss) is able to amplifying the decreasing trend, which
provides you more recognizable changes in the loss.
On Mon, Apr 26, 2021 at 12:20 PM Francesco Pochetti <
@.***> wrote:

Hi,
in section 4.3 of the paper https://arxiv.org/pdf/2005.09007.pdf you
write:
We train the network until the loss converges without using validation set
From that sentence, I understand that you wait for the training loss to
flatten out (converge), without using any validation set to check for
over/underfitting.
Am I getting this right?
Thanks and have a great one!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/xuebinqin/U-2-Net/issues/194, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSGOROIMOE4KLGX5QNH5D3TKUO5PANCNFSM43SLW72Q
.

--
Xuebin Qin
PhD
Department of Computing Science
University of Alberta, Edmonton, AB, Canada
Homepage:https://webdocs.cs.ualberta.ca/~xuebin/

Understood. Thanks!

May I ask how big is the dataset you trained on?
I don't seem to find this info in the paper.

Please refer to the Sec. 4.1 https://arxiv.org/pdf/2005.09007.pdf
On Mon, Apr 26, 2021 at 6:53 PM Francesco Pochetti @.***>
wrote:

May I ask how big is the dataset you trained on?
I don't seem to find this info in the paper.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/xuebinqin/U-2-Net/issues/194#issuecomment-826899948,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSGORKP6RIX3GFESXT74TLTKV47DANCNFSM43SLW72Q
.

--
Xuebin Qin
PhD
Department of Computing Science
University of Alberta, Edmonton, AB, Canada
Homepage:https://webdocs.cs.ualberta.ca/~xuebin/

Ouch sorry, that was hidden in plain sight!
Apologies for missing it and thanks


Generate API Documentation

It should be possible to generate an API overview page and endpoint explorer! There are many ways to approach this and needs to be designed out.

@amiller-gh Do you have endpoints that you want documented? I'm new to writing API documentation, but I know how to document endpoints!

I'm more thinking that Loll, as part of discovering API endpoints on disk at startup, can also auto-generate a documentation page for each user drafted endpoint and serves it up. Would love to use an existing API docs generator that can leverage something like JSDoc, typescript types, etc.


Unavoidable logcat warning

When building AndroidKeysetManager the first time I see a warning like this:
AndroidKeysetManager: keyset not found, will generate a new one
    java.io.FileNotFoundException: can't read keyset; the pref value <name> does not exist
        at com.google.crypto.tink.integration.android.SharedPrefKeysetReader.readPref(SharedPrefKeysetReader.java:71)
        at com.google.crypto.tink.integration.android.SharedPrefKeysetReader.readEncrypted(SharedPrefKeysetReader.java:89)
        at com.google.crypto.tink.KeysetHandle.read(KeysetHandle.java:105)
        at com.google.crypto.tink.integration.android.AndroidKeysetManager$Builder.read(AndroidKeysetManager.java:311)
        at com.google.crypto.tink.integration.android.AndroidKeysetManager$Builder.readOrGenerateNewKeyset(AndroidKeysetManager.java:287)
        at com.google.crypto.tink.integration.android.AndroidKeysetManager$Builder.build(AndroidKeysetManager.java:238)

As far as I can see this warning should not be logged since it will always happen the first time and there is no way to fix around it.

This is useful because it allows us to know when Tink decided to generate a brand new keyset.
Why is a warning a problem?

It's annoying to see a big yellow chunk of warning on startup. We keep our logcat lean to be able to quickly detect issues.
This should probably be hidden behind a debug flag. Or you can use logcat properties to decide if the log tag should be used.
This is especially bad for production since apps should ideally not print to logcat in production because it's noisy as it is.

@thaidn by all means, keep it as a verbose log message, it's just warning look bad in release builds on the logcat. My lead asked me to see if there was anyway of getting rid of it today - I linked him this thread.

Got ya. I'm fixing this.

Got ya. I'm fixing this.

When creating the encrypted shared preferences I got this exception for the first time when file gets created, I saw in the GitHub code that log.w() is written with the exception object, this big chunk of code doesn't not look good , its better to keep only log message without the exception object
9638-9638/com.example.ssp W/AndroidKeysetManager: keyset not found, will generate a new one
java.io.FileNotFoundException: can't read keyset; the pref value androidx_security_crypto_encrypted_prefs_key_keyset does not exist
at com.google.crypto.tink.integration.android.SharedPrefKeysetReader.readPref(SharedPrefKeysetReader.java:71)
at com.google.crypto.tink.integration.android.SharedPrefKeysetReader.readEncrypted(SharedPrefKeysetReader.java:89)
at com.google.crypto.tink.KeysetHandle.read(KeysetHandle.java:105)
at com.google.crypto.tink.integration.android.AndroidKeysetManager$Builder.read(AndroidKeysetManager.java:311)
at com.google.crypto.tink.integration.android.AndroidKeysetManager$Builder.readOrGenerateNewKeyset(AndroidKeysetManager.java:287)
at com.google.crypto.tink.integration.android.AndroidKeysetManager$Builder.build(AndroidKeysetManager.java:238)
at androidx.security.crypto.EncryptedSharedPreferences.create(EncryptedSharedPreferences.java:155)
at androidx.security.crypto.EncryptedSharedPreferences.create(EncryptedSharedPreferences.java:120)


Record where user has crossed paths to avoid repeated notifications

Description
This fixes the issue where notification is fired on every intersection. I have created a seperate store to add the user's intersections and help avoid redundant checks and notifications. #432
The intersections store is indexed via the user’s location time where the contact happened. The inner object represents the infected cases.
1588132102236: {
<PHONE_NUMBER>236: {
        "latitude": -33.849211, "longitude": 150.906321
      },
<PHONE_NUMBER>426: {
        "latitude": 37.33182281, "longitude": -122.03038342
      }
},

When asyncCheckIntersect() runs, we keep track of the exposure period of all new intersections, only when that counter is within a set threshold, the notification is fired.
How to test

Record locations with exposure period for 10 minutes or longer
Monitor notification pop up
Reload app

@tremblerz  - samikhalildev is working to clear the auto-check and then this will be ready for your review.

@kenpugsley @tstirrat Please review my update PR.

@tstirrat  Thanks for the feedback! :)

@samikhalildev I think this is waiting on you to resolve the conflicts.


ci-gke: adjust junit file names to matrix properties

Currently, the name of the junit files no longer matches the name of the job including the values of the matrix properties.
This is kind of a requirement for external tooling to match them against a specific job.
This commit fixes this and uses (${{ join(matrix.*, ', ') }}) to keep the names in sync.

/test

/test

/test

/test


[AMBARI-23275] - Upgrade maven dependency for jdeb and for rpm-maven-plugin

What changes were proposed in this pull request?
Upgraded maven dependency for jdeb and rpm-maven-plugin. Some configurations are not picked because of outdated dependency. Also the spec file creation timr has improved in latest plugins
How was this patch tested?
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1224/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1226/
Test PASSed.

@oleewere Can you please review, too?  Thanks.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1331/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1332/
Test FAILed.
Test FAILured.

Can someone help in getting it merged.


[DO NOT MERGE YET] Dev - test playback alignment

@suhlrich and @AlbertoCasasOrtiz, just tried out and there is still a little delay sometimes. After discussing, we figured that not automatically playing the animation might solve the problem. @AlbertoCasasOrtiz will work on that. Thanks.
This session is a good one to try things out: https://dev.app.opencap.ai/session/98a97501-692e-4900-be59-dfa028e32aec

I do like the auto play function. Is there another solution? Perhaps we can discuss tomorrow
On Jan 30, 2023, at 5:06 PM, Antoine Falisse @.***> wrote:

@suhlrichhttps://github.com/suhlrich and @AlbertoCasasOrtizhttps://github.com/AlbertoCasasOrtiz, just tried out and there is still a little delay sometimes. After discussing, we figured that not automatically playing the animation might solve the problem. @AlbertoCasasOrtizhttps://github.com/AlbertoCasasOrtiz will work on that. Thanks.
This session is a good one to try things out: https://dev.app.opencap.ai/session/98a97501-692e-4900-be59-dfa028e32aec
—
Reply to this email directly, view it on GitHubhttps://github.com/stanfordnmbl/opencap-viewer/pull/20#issuecomment-1409595845, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKOYGJ3PKYVNIBFPBYIRZ7LWVBQQ7ANCNFSM6AAAAAAULT7U4U.
You are receiving this because you were mentioned.Message ID: @.***>

Sure, I think we should discuss this today. I let the link to the function that is probably causing this:
https://github.com/stanfordnmbl/opencap-viewer/blob/3ae194241f676dadba1e838ada4a610d0f997ccb/src/components/pages/Step5.vue#L672

@AlbertoCasasOrtiz is this PR ready to be merged? Should I test something?


Array initialisation short-form

In MS Small Basic, arrays can be initialised with a string:
arr = "a=100;b=999;"
TextWindow.WriteLine(arr["b"]) ' prints 999

This doesn't seem to be possible with ISB - is there an alternative for that?

The code doesn't work as expected in the online editor of Microsoft Small Basic: http://superbasic-v2.azurewebsites.net/
ISB is a derived project of https://github.com/sb/smallbasic-editor, which is the source code of the online editor.

Understood, that's too bad. It would be great if it'd be possible to create arrays on a single line to simulate a variable amount of method parameters. Anyway, it's not too important, especially as I could just parse the string myself.


Dependabot downgrades instead of stated upgrade (related to double tagging?)

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
Go
Package manager version
No response
Language version
1.21.6
Manifest location and content before the Dependabot update
https://github.com/sylabs/singularity/blob/637e3eeeaf0265f11814006f271cc00c33532e64/go.mod
dependabot.yml content
version: 2
updates:
  - package-ecosystem: "gomod" # See documentation for possible values
    directory: "/" # Location of package manifests
    schedule:
      interval: "daily"
    target-branch: main
  - package-ecosystem: "gomod" # See documentation for possible values
    directory: "/" # Location of package manifests
    schedule:
      interval: "daily"
    target-branch: release-4.1
  - package-ecosystem: "gomod" # See documentation for possible values
    directory: "/" # Location of package manifests
    schedule:
      interval: "daily"
    target-branch: release-4.0
  - package-ecosystem: "gomod" # See documentation for possible values
    directory: "/" # Location of package manifests
    schedule:
      interval: "daily"
    target-branch: release-3.11

Updated dependency
github.com/opencontainers/image-spec from 1.1.0-rc5 to 1.1.0-rc.6
What you expected to see, versus what you actually saw
The github.com/opencontainers/image-spec project tagged a new RC.
It appears that this was tagged as both 1.1.0-rc.6 and 1.1.0-rc6 ~(in error?)~, as queried with them here: https://github.com/opencontainers/image-spec/issues/1159
Dependabot opened a PR at https://github.com/sylabs/singularity/pull/2557 which announces that it is updating github.com/opencontainers/image-spec from 1.1.0-rc5 to 1.1.0-rc.6. Instead it is downgrading to the 1.1.0-rc3 tag, and causing a large number of other dependencies to be downgraded also.
See - https://github.com/sylabs/singularity/pull/2557/files#diff-33ef32bf6c23acb95f5902d7097b7a1d5128ca061167ec0716715b0b9eeaa5f6
Native package manager behavior
Native go get -u updates to v1.1.0-rc6, with no other go.mod changes.
Note that it picked v1.1.0-rc6 and not the v1.1.0-rc.6 that dependabot reported attempting.
02:26 pm $ go get -u github.com/opencontainers/image-spec
go: downloading github.com/opencontainers/image-spec v1.1.0-rc6
go: upgraded github.com/opencontainers/image-spec v1.1.0-rc5 => v1.1.0-rc6

02:26 pm $ git diff
diff --git a/go.mod b/go.mod
index 0944c7074..0c22d4b52 100644
--- a/go.mod
+++ b/go.mod
@@ -33,7 +33,7 @@ require (
        github.com/gosimple/slug v1.13.1
        github.com/moby/buildkit v0.12.4
        github.com/moby/term v0.5.0
-       github.com/opencontainers/image-spec v1.1.0-rc5
+       github.com/opencontainers/image-spec v1.1.0-rc6
        github.com/opencontainers/runc v1.1.11
        github.com/opencontainers/runtime-spec v1.1.0
        github.com/opencontainers/runtime-tools v0.9.1-0.20230914150019-408c51e934dc
diff --git a/go.sum b/go.sum
index 82ef1fb40..ed362a295 100644
--- a/go.sum
+++ b/go.sum
@@ -379,6 +379,8 @@ github.com/opencontainers/go-digest v1.0.0/go.mod h1:0JzlMkj0TRzQZfJkVvzbP0HBR3I
 github.com/opencontainers/image-spec v1.0.1/go.mod h1:BtxoFyWECRxE4U/7sNtV5W15zMzWCbyJoFRP3s7yZA0=
 github.com/opencontainers/image-spec v1.1.0-rc5 h1:Ygwkfw9bpDvs+c9E34SdgGOj41dX/cbdlwvlWt0pnFI=
 github.com/opencontainers/image-spec v1.1.0-rc5/go.mod h1:X4pATf0uXsnn3g5aiGIsVnJBR4mxhKzfwmvK/B2NTm8=
+github.com/opencontainers/image-spec v1.1.0-rc6 h1:XDqvyKsJEbRtATzkgItUqBA7QHk58yxX1Ov9HERHNqU=
+github.com/opencontainers/image-spec v1.1.0-rc6/go.mod h1:W4s4sFTMaBeK1BQLXbG4AdM2szdn85PY75RI83NrTrM=
 github.com/opencontainers/runc v1.0.0-rc90/go.mod h1:qT5XzbpPznkRYVz/mWwUaVBUv2rmF59PVA73FjuZG0U=
 github.com/opencontainers/runc v1.1.11 h1:9LjxyVlE0BPMRP2wuQDRlHV4941Jp9rc3F0+YKimopA=
 github.com/opencontainers/runc v1.1.11/go.mod h1:S+lQwSfncpBha7XTy/5lBwWgm5+y5Ma/O44Ekby9FK8=

Images of the diff or a link to the PR, issue, or logs

https://github.com/sylabs/singularity/pull/2557
https://github.com/sylabs/singularity/pull/2552
https://github.com/sylabs/singularity/pull/2550

Smallest manifest that reproduces the issue
No response

Dependabot is just running Go CLI commands.  If I then run go mod tidy then it downgrades github.com/opencontainers/image-spec to v1.1.0-rc3.
It looks like the Go tooling is getting tripped up by the prerelease format change.

Dependabot is just running Go CLI commands.  If I then run go mod tidy then it downgrades github.com/opencontainers/image-spec to v1.1.0-rc3.

That maybe true following a go get with the -rc.6 tag. However, doing a go get -u github.com/opencontainers/image-spec, i.e. just asking for an upgrade, and not specifying the rc.6 resulted in a clean bump from rc-5 up to rc-6, avoiding any unrelated downgrades.
I guess this means that dependabot is examining the dependency tags available, choosing a specific tag itself, (in this case preferring rc.6 over rc-6) and then using a specifig go get <module>@<version> rather than using go get -u to perform the upgrade?
Regardless, we were able to solve it simply with a manual go get -u github.com/opencontainers/image-spec, and this is a weird enough situation (project changing tag format and keeping both formats) that I don't think it's critical to fix.