Unused arguments cause buildEngineDistribution to generate invalid caches and prevent tests requiring the affected file from running

I have noticed a bug with serialization of org.enso.compiler.core.ir.expression.warnings.Unused$FunctionArgument. At first it looked innocuous - just preventing some IR caches from loading - but of course it should just then parse the original source and all should be fine.
Well - unfortunately I have found that for some reason instead of doing that and recovering - in some scenarios (described below) the invalid cache causes the program to fail altogether.
The repro is following:

Checkout develop and buildEngineDistribution in sbt.
Add a function foo bar = 42 at the end of File.enso - the bar argument is unused causing a warning. See patch at the bottom.
Again, do buildEngineDistribution.
Run .\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\bin\enso --run .\test\Tests\src\System\File_Spec.enso.

The (4) run fails with:
C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso: error: The symbol Existing_File_Behavior (module, type, or constructor) does not exist in File.
C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso: error: The symbol File_Access (module, type, or constructor) does not exist in File.
C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso: error: The symbol File_Permissions (module, type, or constructor) does not exist in File.
C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso: error: The symbol Write_Extensions (module, type, or constructor) does not exist in File.
C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso:47:14: error: The name `Vector` could not be found.
   47 | file_types : Vector
      |              ^~~~~~

(...)

C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso:802:39: error: The name `Nothing` could not be found.
  802 |         if file.is_directory.not then Nothing else
      |                                       ^~~~~~~
C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso:804:40: error: The name `Option` could not be found.
  804 |             options = children.map c-> Option c.name c.name.pretty
      |                                        ^~~~~~
C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso:805:13: error: The name `Widget` could not be found.
  805 |             Widget.Single_Choice values=options display=Display.Always
      |             ^~~~~~
C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso:805:57: error: The name `Display` could not be found.
  805 |             Widget.Single_Choice values=options display=Display.Always
      |                                                         ^~~~~~~
C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso:807:5: warning: Unused function argument bar.
  807 | foo bar = 42
      |     ^~~
Aborting due to 186 errors and 1 warnings.
Execution finished with an error: Compilation aborted due to errors.

We can see that constructors like Nothing are not resolved correctly - but they are obviously imported at the top of the File.enso...
Apparently, adding the unused argument has broken the whole file.
For context, this is what is written out when running the second buildEngineDistribution:
(...)

[info] Assembly jar up to date: runner.jar
[info] Regenerating manifest for [distribution\lib\Standard\Base\0.0.0-dev].
[info] Generating index for built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base
C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\System\File.enso:807:5: warning: Unused function argument bar.
  807 | foo bar = 42
      |     ^~~
[ERROR] [2023-11-24T13:29:49+01:00] [enso.org.enso.interpreter.caches.ModuleCache] Failed to save cache for Standard.Base.System.File.
java.io.IOException: No persistance for org.enso.compiler.core.ir.expression.warnings.Unused$FunctionArgument
        at org.enso.runtime/org.enso.persist.PerMap.searchSupertype(PerMap.java:63)
        at org.enso.runtime/org.enso.persist.PerMap.searchSupertype(PerMap.java:56)
        at org.enso.runtime/org.enso.persist.PerMap.searchSupertype(PerMap.java:56)
        at org.enso.runtime/org.enso.persist.PerMap.forType(PerMap.java:73)
        at org.enso.runtime/org.enso.persist.PerGenerator.writeIndirect(PerGenerator.java:67)
        at org.enso.runtime/org.enso.persist.PerGenerator$ReferenceOutput.writeObject(PerGenerator.java:104)
        at org.enso.runtime/org.enso.compiler.core.ir.IrPersistance$PersistScalaList.writeObject(IrPersistance.java:192)
        at org.enso.runtime/org.enso.compiler.core.ir.IrPersistance$PersistScalaList.writeObject(IrPersistance.java:180)
        at org.enso.runtime/org.enso.persist.Persistance.writeInline(Persistance.java:139)
        at org.enso.runtime/org.enso.persist.PerGenerator$ReferenceOutput.writeInline(PerGenerator.java:99)
        at org.enso.runtime/org.enso.compiler.pass.analyse.PersistGatherDiagnostics_DiagnosticsMeta.writeObject(PersistGatherDiagnostics_DiagnosticsMeta.java:18)
        at org.enso.runtime/org.enso.compiler.pass.analyse.PersistGatherDiagnostics_DiagnosticsMeta.writeObject(PersistGatherDiagnostics_DiagnosticsMeta.java:4)
        at org.enso.runtime/org.enso.persist.Persistance.writeInline(Persistance.java:139)
        at org.enso.runtime/org.enso.persist.PerGenerator.writeIndirect(PerGenerator.java:72)
        at org.enso.runtime/org.enso.persist.PerGenerator$ReferenceOutput.writeObject(PerGenerator.java:104)
        at org.enso.runtime/org.enso.compiler.core.ir.IrPersistance$PersistScalaMap.writeObject(IrPersistance.java:225)
        at org.enso.runtime/org.enso.compiler.core.ir.IrPersistance$PersistScalaMap.writeObject(IrPersistance.java:210)
        at org.enso.runtime/org.enso.persist.Persistance.writeInline(Persistance.java:139)
        at org.enso.runtime/org.enso.persist.PerGenerator$ReferenceOutput.writeInline(PerGenerator.java:99)
        at org.enso.runtime/org.enso.compiler.core.ir.IrPersistance$PersistMetadataStorage.writeObject(IrPersistance.java:385)
        at org.enso.runtime/org.enso.compiler.core.ir.IrPersistance$PersistMetadataStorage.writeObject(IrPersistance.java:370)
        at org.enso.runtime/org.enso.persist.Persistance.writeInline(Persistance.java:139)
        at org.enso.runtime/org.enso.persist.PerGenerator$ReferenceOutput.writeInline(PerGenerator.java:99)
        at org.enso.runtime/org.enso.compiler.core.ir.PersistModule.writeObject(PersistModule.java:29)
        at org.enso.runtime/org.enso.compiler.core.ir.PersistModule.writeObject(PersistModule.java:4)
        at org.enso.runtime/org.enso.persist.Persistance.writeInline(Persistance.java:139)
        at org.enso.runtime/org.enso.persist.PerGenerator.writeObject(PerGenerator.java:51)
        at org.enso.runtime/org.enso.persist.PerGenerator.writeObject(PerGenerator.java:33)
        at org.enso.runtime/org.enso.persist.Persistance.write(Persistance.java:173)
        at org.enso.runtime/org.enso.interpreter.caches.ModuleCache.serialize(ModuleCache.java:147)
        at org.enso.runtime/org.enso.interpreter.caches.ModuleCache.serialize(ModuleCache.java:26)
        at org.enso.runtime/org.enso.interpreter.caches.Cache.saveCacheTo(Cache.java:127)
        at org.enso.runtime/org.enso.interpreter.caches.Cache.lambda$save$0(Cache.java:97)
        at java.base/java.util.Optional.flatMap(Optional.java:289)
        at org.enso.runtime/org.enso.interpreter.caches.Cache.save(Cache.java:85)
        at org.enso.runtime/org.enso.interpreter.runtime.TruffleCompilerContext.saveCache(TruffleCompilerContext.java:186)
        at org.enso.runtime/org.enso.interpreter.runtime.SerializationManager.$anonfun$doSerializeModule$1(SerializationManager.scala:611)
        at org.enso.runtime/org.enso.interpreter.runtime.SerializationManager.$anonfun$doSerializeModule$1$adapted(SerializationManager.scala:591)
        at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:317)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642)
        at java.base/java.lang.Thread.run(Thread.java:1583)
        at org.graalvm.truffle/com.oracle.truffle.polyglot.SystemThread.run(SystemThread.java:68)
[info] Engine package created at built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev
[success] Total time: 67 s (01:07), completed 24 lis 2023, 13:29:51

The problem with this bug is that merely adding a function with unused argument which is just a warning completely prevents the program from running after the indexes are rebuilt by buildEngineDistribution.
The 'workaround' is to ensure there are no unused variables in std-lib (which is of course the desired state for production anyway) - but during development it is a bit problematic as sometimes one may want to test a partially implemented solution.

The biggest issue IMO is not that a warning is not serializable (although ideally it should be), but that somehow our caches get into a state in which instead of invalidating them and re-parsing the file, they are loaded but they are malformed and the program fails.
This is actually similar to intermittent IR caches issues I've been seeing from time to time - but I was never able to reliably reproduce that. This time it seems that the repro is completely deterministic.
Before fixing the missing serialization logic, it may be worth to investigate what happens so that the malformed cache is not invalidated - as that could potentially also affect the other, more intermittent, failures that we cannot easily reproduce.

Thanks for the reproducer. I'll take a look.

The problem is caused by Base.bindings cache being stored, but File.ir cache not. One possible fix would be to avoid storing .bindings cache when any of its related .ir files is corrupted. Still investigating why the missing File.ir is causing the import resolution to fail.

One possible fix would be to avoid storing .bindings cache when any of its related .ir files is corrupted.

Should we instead invalidate the .bindings cache on load? I guess not storing it is also good, but even if it is stored but the related .ir file is missing (e.g. someone deleted the .ir file, but did not delete the .bindings file), I guess the program should still work just fine (probably just 'eagerly' invalidating all caches that may be only 'partially valid').


Create CONTRIBUTING.md

Y'all need this.

Whilst I think you provided some good points, I'm not entirely sure that it was worded appropriately, and outside of that it contains many grammatical errors that might make it harder for others to read.

I realize this. I am not so very good at some grammatical things and am generally pretty blunt. I just am trying to make a statement. You can use a creative license.

Not worded appropriately with several typos. I think people contributing have a general idea on how to contribute and what to contribute; pull requests that we don't see fit for the repository are closed.


[bug report] carousel renders El carousel item according to array. After partial data of array changes, El carousel item does not switch left and right according to array order

Element UI version
2.13.2
OS/Browsers version
window 10 / chrome 70
Vue version
3.0.0-beta.20
Reproduction Link
https://codepen.io/jian-yin/pen/ExPLpjW?editable=true%3Dhttps%3A%2F%2Felement.eleme.cn%2F
Steps to reproduce
1、data 定义数组 list =  [1,3,5,7]
2、mounted 后 this.list = [1,2,3,4]
What is Expected?
Carousel 切换顺序 1,2,3,4
What is actually happening?
Carousel 切换顺序 1,3,2,4

$children 并不保证顺序，所以 Carousel 中 items 顺序 错误
https://github.com/vuejs/vue/issues/2275


this string is not fully expanded

expected
➤ fish --version
fish, version 3.0.2
➤ echo emote_collector.{{extensions.{bingo.commands,emote}},utils.{bingo,converter}}
emote_collector.extensions.bingo.commands emote_collector.extensions.emote emote_collector.utils.bingo emote_collector.utils.converter

actual
🐍 import braceexpand
🐍 braceexpand.__version__
'0.1.2'
🐍 list(braceexpand.braceexpand('emote_collector.{{extensions.{bingo.commands,emote}},utils.{bingo,converter}}'))
['emote_collector.{extensions.{bingo.commands,emote}}', 'emote_collector.utils.bingo', 'emote_collector.utils.converter']

this is my fault. I should have used emote_collector.{extensions.{bingo.commands,emote},utils.{bingo,converter}} instead

Thank your for the report. I see you already found the solution to your problem, but your example helped me uncover an unrelated issue.
Bash expands your first pattern to this:
$ echo emote_collector.{{extensions.{bingo.commands,emote}},utils.{bingo,converter}}
emote_collector.{extensions.bingo.commands} emote_collector.{extensions.emote} emote_collector.utils.bingo emote_collector.utils.converter

But braceexpand would not further expand the pattern {bingo.commands,emote} that is wrapped with an extra level of braces. This is now fixed in master.


Unable to cache files on separate drives

Hi,
I just started using vmtouch and it's been working great for the most part. However, I am unable to cache files that are stored on a separate drive using the 9P filesystem. The files either are cached and then immediately removed when touching them or FATAL (Cannot allocate memory) errors are thrown when trying to lock the files in memory.
Do you have any idea of workarounds?
Thanks,
Vincent

Hi thanks for writing. I have no experience with the 9P filesystem. Which OS are you on, and which driver are you using? Does it use FUSE? Are there any special mount options on the FS?
I have however used S3FS before which is a FUSE mount for amazon's S3 system. It interacts with the page cache in a strange way: It creates a special temp directory on the filesystem and these are the files that back the page cache for the filesystem, so vmtouch needs to be run on them, not the files that appear in the mounted directory.
Regarding the FATAL errors when locking, does this occur when run as the root user as well? Most OSs have resource limits and/or system-wide limits on "wired memory". You can read more about those here: https://github.com/hoytech/vmtouch/blob/master/TUNING.md

I'm running Ubuntu 14.04 LTS on QEMU host with VIRTIO transport. I don't think I am using FUSE or any special mount options either. I am just using the 9p with the 9P2000.L protocol.
I see there are some options regarding cache:
cache=mode      specifies a caching policy.  By default, no caches are used.
none = default no cache policy, metadata and data
alike are synchronous.
loose = no attempts are made at consistency,
intended for exclusive, read-only mounts
fscache = use FS-Cache for a persistent, read-only
cache backend.
mmap = minimal cache that is only used for read-write
mmap.  Northing else is cached, like cache=none
I am not using this flag, so I guess this is cache=none. Perhaps this flag needs to be set?
Also, I tried running as root and it was successful (and vmtouch showed that the file was cached completely). However, when I ran my programs using the file, the performance was not as good as when the file was on the local SSD so I don't think it quite worked as intended.

Ahh interesting. I would try to set the cache to "fscache" if you want vmtouch to be able to control it. Hopefully you can run vmtouch directly on the files instead of the temporary backing files like with S3FS.
Please let us know what you find out. :)

It seems to have fixed the issue, thanks a lot for all your help!

Thanks for letting me know. I've added a brief note about 9P to the TUNING.md file: https://github.com/hoytech/vmtouch/blob/master/TUNING.md


Detected integration that called async_setup_platforms instead of awaiting async_forward_entry_setups

Detected integration that called async_setup_platforms instead of awaiting async_forward_entry_setups; this will fail in version 2023.3.
Please report issue to the custom integration author for frigate using this method at custom_components/frigate/init.py, line 289: hass.config_entries.async_setup_platforms(entry, PLATFORMS)

Already fixed, see pinned issue: https://github.com/blakeblackshear/frigate-hass-integration/issues/421 . Will be in the next build.


Is source-map-loader necessary for the typescript example?

In the webpack / typescript documentation it says that the source-map-loader is necessary for sourcemaps to work.  I don't believe it is (I've been happily debugging TypeScript with sourcemaps and webpack for about 2 years now).  I was just curious if I was missing anything?  As far as I know this section:
 module: {
   rules: [
     {
       enforce: 'pre',
       test: /\.js$/,
       loader: "source-map-loader"
     },
     {
       enforce: 'pre',
       test: /\.tsx?$/,
       use: "source-map-loader"
     }
   ]
 },

Isn't necessary at all.
You just need "sourceMap": true in the tsconfig.json and devtool: 'inline-source-map' in the webpack.config.js.
I just wanted to know if there was anything else source-map-loader brings to the party?  AFAIK this isn't required...

Related #969

Can you take a look at the changes in #969 and let me know if the guide is valid so we can push it forward?

The changes in #969 look valid to me - I don't think source-map-loader is required.

I followed the guide, but run Babel after TypeScript  and it looks I can only get the source map for my code after TypeScript run.
E.g. I have a file like this (foo.tsx):
import React from 'react';
import { NameComponent, ImageTestComponent } from 'some-module';

/**
 * This is our app component.
 */
export const AppComponent = () => (

    <NameComponent a={1} b={2} name="foo" />
    <ImageTestComponent />
    {
      process.env.NODE_ENV === 'production'
        ?
Production Build

        :
Dev Build

    }

);
console.log('test');

'test' is logged correctly to the console and it shows foo.tsx:9 (correct name, wrong line). Clicking on foo.tsx shows this file:
import React from 'react';
import { NameComponent, ImageTestComponent } from 'some-module';
/**
 * This is our app component.
 */
export var AppComponent = function AppComponent() {
    return React.createElement("div", null, React.createElement(NameComponent, { a: 1, b: 2, name: "foo" }), React.createElement(ImageTestComponent, null), process.env.NODE_ENV === 'production' ? React.createElement("p", null, "Production Build") : React.createElement("p", null, "Dev Build"));
};
console.log('test');

//////////////////
// WEBPACK FOOTER
// ./src/app.tsx
// module id = 84
// module chunks = 0

(Here is the console.log on line 9.)
Any idea why?

Do you have these set correctly?

You just need "sourceMap": true in the tsconfig.json and devtool: 'inline-source-map' in the webpack.config.js.

Well you can check out our setup, but it's quite big. We have a similar tool like create-react-app called ws (linked to v2 branch). It also contains examples.
The webpack config for an single page app is created here.
Already cloned ts-loader earlier today, because of the examples 😄 Will try out react-babel-karma-gulp.

Great - thanks for reporting back. Be great to hear what the difference is (if you find it)

BTW you might find the fork-ts-checker-webpack-plugin / happypack examples useful if you'd like faster build times. I'm planning to blog about these but haven't got round to it yet

Found the bug https://github.com/Va1/string-replace-loader/pull/28.

Great work!


Tip: Switch between projects

I work on many side-projects, and navigating between them takes seconds. I wondered if i could make a "fast project switcher" using fzf.

The code is
cd ~/`cd ~; { find workspace -maxdepth 1 & find dev -maxdepth 1; } | fzf`

Where ~ is the grandparent directory for your project directories, and workspace and dev are directories where each subfolder is a project. So, if all of your projects were in ~/projects, you wouldn't need to combine the output of two commands, you could just do:
cd ~/`cd ~;  find projects -maxdepth 1  | fzf`

I'm not sure if an issue is the best place for this, but thought i'd share this tip with others. Feel free to close.

Thanks for sharing. Adding a preview window would be a nice touch.
# Showing the content of README.md.
# You can use programs for syntax highlighting instead of cat.
#   See: https://github.com/junegunn/fzf#preview-window
cd ~/`cd ~;  find projects -maxdepth 1  | fzf --preview 'cat {}/README.md'`

# Using tree command to show the project structure
cd ~/`cd ~;  find projects -maxdepth 1  | fzf --preview 'tree {} | head -200'`


Decoding router params.

Hello,
I found error on decoding params from url_template. This is examle of decoding GET param and params in url.
import falcon

from wsgiref import simple_server

class ThingsParam:
    def on_get(self, req, resp):
        print(req.get_param('query', True))
        resp.status = falcon.HTTP_200

class ThingsQuery:
    def on_get(self, req, resp, query):
        print(query)
        resp.status = falcon.HTTP_200

app = falcon.API()

app.add_route('/things', ThingsParam())
app.add_route('/things/{query}', ThingsQuery())

if __name__ == '__main__':
    httpd = simple_server.make_server('<IP_ADDRESS>', 8000, app)
    httpd.serve_forever()

Output will be:
<IP_ADDRESS> - - [09/Jul/2015 16:03:53] "GET / HTTP/1.1" 404 0
<IP_ADDRESS> - - [09/Jul/2015 16:03:53] "GET /favicon.ico HTTP/1.1" 404 0
ąę
<IP_ADDRESS> - - [09/Jul/2015 16:04:10] "GET /things?query=%C4%85%C4%99 HTTP/1.1" 200 0
ÄÄ
<IP_ADDRESS> - - [09/Jul/2015 16:04:17] "GET /things/%C4%85%C4%99 HTTP/1.1" 200 0

Thanks for the report. I tried this under CPython 2.7.10 on OS X and it printed out the correct string, "ąę", each time. However, under CPython 3.4.3, the string is misinterpreted.
Digging into this further, I found that when constructing the WSGI environ dict under CPython 2.7.10, wsgiref and gunicorn both decode the escaped path segment to a UTF-8 encoded instance of str. On the other hand, they both leave the query string alone (the percent encoding is preserved).
Under CPython 3.4.3, the characters are copied directly to an instance of str as well, but under Py3k, str is a native Unicode type, and when you print it out, the code points are not assumed to be UTF-8.
# Bytes posted from curl
b'/things/%C4%85%C4%99'

# Interpreted by the WSGI server
u'/things/\xC4\x85\xC4\x99' == u'/things/Ä\x85Ä\x99'

# What a correctly decoded string would look like
u'/things/\u0105\u0119' == u'/things/ąę'

@jmvrbanac Thoughts on what we should do about this?

[fixed]


New frontend for faucet

@silasdavis I removed the CDNs and am now using local images. Let me know if any further changes are required.

Second thoughts I'll handle this since I need to bump version anyway, nice work @vwakesahu


datree: 1.5.25 -> 1.6.13

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/datreeio/datree/releases.
meta.description for datree is: "CLI tool to ensure K8s manifests and Helm charts follow best practices as well as your organization’s policies"
meta.homepage for datree is: "https://datree.io/"
meta.changelog for datree is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.6.0 with grep in /nix/store/24kxd3dpanx04lr1dp695zd8g4fzrzfy-datree-1.6.0
found 1.6.0 in filename of file in /nix/store/24kxd3dpanx04lr1dp695zd8g4fzrzfy-datree-1.6.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
datree

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/24kxd3dpanx04lr1dp695zd8g4fzrzfy-datree-1.6.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A datree https://github.com/r-ryantm/nixpkgs/archive/68c9fa2abac35daa069b6a6936fd17117f058431.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/24kxd3dpanx04lr1dp695zd8g4fzrzfy-datree-1.6.0
ls -la /nix/store/24kxd3dpanx04lr1dp695zd8g4fzrzfy-datree-1.6.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

datree

Maintainer pings
cc @jceb for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/datree/versions.
meta.description for datree is: "CLI tool to ensure K8s manifests and Helm charts follow best practices as well as your organization’s policies"
meta.homepage for datree is: "https://datree.io/"
meta.changelog for datree is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.6.6 with grep in /nix/store/768rxf7b4wfbxw3hwmmspb7f7klhfh6m-datree-1.6.6
found 1.6.6 in filename of file in /nix/store/768rxf7b4wfbxw3hwmmspb7f7klhfh6m-datree-1.6.6

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
datree

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/768rxf7b4wfbxw3hwmmspb7f7klhfh6m-datree-1.6.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A datree https://github.com/r-ryantm/nixpkgs/archive/6efed05e0b2d5b31e052887a0855cda70b87d4a5.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/768rxf7b4wfbxw3hwmmspb7f7klhfh6m-datree-1.6.6
ls -la /nix/store/768rxf7b4wfbxw3hwmmspb7f7klhfh6m-datree-1.6.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

datree

Maintainer pings
cc @jceb for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/datree/versions.
meta.description for datree is: "CLI tool to ensure K8s manifests and Helm charts follow best practices as well as your organization’s policies"
meta.homepage for datree is: "https://datree.io/"
meta.changelog for datree is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.6.13 with grep in /nix/store/xr6r1s0xbd57nxmcyn8zf6c5mm5v2wfk-datree-1.6.13
found 1.6.13 in filename of file in /nix/store/xr6r1s0xbd57nxmcyn8zf6c5mm5v2wfk-datree-1.6.13

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
datree

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xr6r1s0xbd57nxmcyn8zf6c5mm5v2wfk-datree-1.6.13 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A datree https://github.com/r-ryantm/nixpkgs/archive/7e97189ac8c73cea1788700d9b6a8058ff60aef3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xr6r1s0xbd57nxmcyn8zf6c5mm5v2wfk-datree-1.6.13
ls -la /nix/store/xr6r1s0xbd57nxmcyn8zf6c5mm5v2wfk-datree-1.6.13/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

datree

Maintainer pings
cc @jceb for testing.

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

1 package built:

datree


GUACAMOLE-203: Add support for SSH Keep Alive

This is for the minor changes required on the client side to support configuring the SSH Keep Alive interval for SSH connections.

Another commit to add support for keepalive configuration for SFTP connections in VNC and RDP.

Ping...any takers for review on this?

@mike-jumper: Is there any way with the implementation of the NUMERIC_FIELD type to force it to be >= 0?

@mike-jumper Ping...any further action needed, here?  Any comments on my previous question Re: NUMERIC_FIELD and forcing values to be 0 or positive?

Okay, negative numbers dealt with on Guacamole server side, so nothing really to do, here, although it still seems like adding options for these NUMERIC fields for forcing value ranges might be worthwhile.


ConfigError raised when used with django-polymorphic

https://github.com/ericls/django-hashids/blob/master/django_hashids/field.py#L50-L57
django-polymorphic or via model inheritance

Taking a look

I can see how this can be a problem but can't reproduce the issue on my end.
This is what I'm doing:
class TestBase(Model):
    class Meta:
        abstract = True

    hashid =  HashidsField()
    # This also works
    # hashid =  HashidsField(hashids_instance=Hashids(salt="123"))

class InheritanceModel(TestBase):
    pass

class InheritanceModel2(TestBase):
    pass

And I'm able to test it with success:
def test_model_inheritance():
    from tests.test_app.models import InheritanceModel, InheritanceModel2

    instance = InheritanceModel.objects.create()
    InheritanceModel.objects.filter(hashid=instance.hashid).get() == instance

    instance2 = InheritanceModel2.objects.create()
    InheritanceModel2.objects.filter(hashid=instance2.hashid).get() == instance2

Could you please provide some reproducing steps?

Sure, I saw the issue with django-polymorphic if that helps. I’ll try to setup a test to replicate the issue.

Got it. I'm going to give it a try as well.

Hi @jackton1, I think #25 is related to this and #26 might fix it. I still hasn't being able to reproduce this issue but would you like to give the new patch a try?

Should be fixed in version 0.6.0(11e25d02779d41e16c72dea0a7c58e1261b2b41f)


Windows terminal doesn't open in other directory

Environment
Windows 1909: build 18363.836
Windows Terminal 1.0.1401.0

Steps to reproduce
Run the windows terminal in a directory

or try to open it from another directory

Expected behavior
The windows terminal should open in that directory

Actual behavior
The terminal opens in C:\Users[user]

Thanks for the suggestion! Please make sure to search for duplicates when filing new issues.  We wanted to make wt behave like cmd or powershell here, but unfortunately, there were just too many odd edge cases to make this work right. There are more details over in #878 (comment) if you want more reading.
/dup #878

Thanks for the suggestion! Please make sure to search for duplicates when filing new issues. We wanted to make wt behave like cmd or powershell here, but unfortunately, there were just too many odd edge cases to make this work right. There are more details over in #878 (comment) if you want more reading.
/dup #878

Sorry, I only searched the open issues. Thank you for the feeback.
For everyone else who doesnt want to look tough the previous threads the answer to make it work would be.
wt -d .


[Blood Magic] Issue with Alchemy Table

It would appear that there is a bit of a circulatory recipe issue. The Alchemy Table required Scorned Bricks and Scorned Bricks require  Spectral Dust. However spectral dust requires an Alchemy Table. Therefore making it currently impossible to get the Alchemy Table. This is on version 0.10.0 of the pack.

Spectral dust is obtained by killing undead creatures with an arthana.

Spectral dust is obtained by killing undead creatures with an arthana.

Athame*?


Sync rhs/ceph-csi:devel with ceph/ceph-csi:devel

syncing downstream devel from upstream devel to get latest fixes,
/cherry-pick release-4.10

/assign @agarwal-mudit

@nixpanic commitlint is failing due to dependent bot commits. not sure how it was passing earlier, should we disable/skip commitlint in downstream?

@nixpanic commitlint is failing due to dependent bot commits. not sure how it was passing earlier, should we disable/skip commitlint in downstream?

Not sure if it is required to pass all GitHub Workflows.
/override commitlint

/override commitlint/commitlint
Just trying :)

/override tide
Maybe this allows it to get merged?

@openshift-cherrypick-robot help

@openshift-merge-robot help

/override commitlint
Currently merged is bocked with:
tide Pending — Not mergeable. Job commitlint has not succeeded.

Marked several status contexts as required passing, excluding commitlint in the repository settings. Restarted commitlint for this PR, maybe tide picksup the status?

Marked several status contexts as required passing, excluding commitlint in the repository settings. Restarted commitlint for this PR, maybe tide picksup the status?

That didnt work, maybe need to change the settings in the openshift/release repository?

/override commitlint

If we are ready to override commitlint, why don't we merge it manually. Other tests have passed?

If we are ready to override commitlint, why don't we merge it manually. Other tests have passed?

Yes, that is probably what I would do next as well

Yeah, this can be a manual merge

/cherry-pick release-4.10


Production of quantitative datasets and expansion of models

I would like to ask if the quantified dataset should be in a wiki format or similar to training data. If you want to create it yourself, what format should you use; Secondly, can this framework quantify other models?I would like to ask if the quantified dataset should be in a wiki format or similar to training data. If you want to create it yourself, what format should you use; Secondly, can this framework quantify other models?

The converter interprets the input Parquet file as a blob of text (by concatenating all the text fields it contains) which is then chunked up into rows of equal length as defined by the command-line arguments.
The quantizer then uses this data as the input to one large forward pass through the model, recording the intermediate hidden state before and after every linear layer that is to be quantized. These states are used as a reference to adjust the quantization grid and to perform the GPTQ error compensation.
This means the model becomes "specialized", to some degree depending on how aggressive the bitrate is, in reproducing the internal states produced by the calibration data. It will still work with other data, it will just work more precisely when dealing with inputs similar to what it was calibrated on.
I don't think there's a perfect consensus on what the calibration dataset should look like, but you most likely want the calibration data to resemble data that you intend to feed the quantized model later on.

The converter interprets the input Parquet file as a blob of text (by concatenating all the text fields it contains) which is then chunked up into rows of equal length as defined by the command-line arguments.
The quantizer then uses this data as the input to one large forward pass through the model, recording the intermediate hidden state before and after every linear layer that is to be quantized. These states are used as a reference to adjust the quantization grid and to perform the GPTQ error compensation.
This means the model becomes "specialized", to some degree depending on how aggressive the bitrate is, in reproducing the internal states produced by the calibration data. It will still work with other data, it will just work more precisely when dealing with inputs similar to what it was calibrated on.
I don't think there's a perfect consensus on what the calibration dataset should look like, but you most likely want the calibration data to resemble data that you intend to feed the quantized model later on.
As for other models, the framework is written for Llama models specifically. That includes Llama, Llama-2, CodeLlama and Mistral, as well as any models that derive from them. I may add support for more architectures over time, especially if something ends up really standing out or gaining a lot of adoption. But I also have limited time, and I'm not looking to (and couldn't hope to) create and maintain an alternative to HF Transformers.

Thank you for your contribution to the quantification work. I have tried the quantification of llama2-13b and the acceleration effect is very obvious. I look forward to supporting more models in this project. I hope that in terms of quantifying datasets, we can achieve a format similar to that in autogptq. Thank you again for your contribution.


How to customize windows location position

What OS are you using (uname -a, or Windows version)?
Windows  10

What programming language are you using (C/C++/Go/Rust)?
Go

What did you expect to see and what you saw instead?
Windows initial position is right 0 and bottom 0

@zserge please close, this is a duplicate of #642


New Agar.io client protocol 14 (Text and FFA lb adjustment)

Hi,
credit goes to user https://github.com/cowpits who put information about packet under #1157. This fixes both Text based leaderboard and standard FFA.

Please wait with merging, I think some clean-up need to be done, we have big mess in UpdateLeaderboard.js some of the blocks can be merged, since there are only 1-3 changes inside. Will do it tomorrow, day after, will see.
If anyone have pressure to use this code, just take it ;)

It's ready @FantasyIsBae


Add bitcoode parsing support

I'm currently working on adding bitcode parsing support in https://github.com/moosichu/zar,  as milestones are reached I will update this branch to mirror those changes so they can be more easily tracked against zld.
For now I will focus purely on the needs of the archiver (as you can see, there are some unrelated changes in this PR), with the intent of cleaning things up once those needs have been met.

Since there has been no progress on this, I hope you won't mind if I close it for now. Feel free to reopen if you update the branch, or open a new PR.


Support arrays as constants

PHPCS version: 2.9.1
PHP version: 7.1.12
WPCS version: latest
In the past, PHP has not supported constants that were arrays. That lack of support is no longer relevant, so I'm hoping the WPCS team can make a ruling on the field: is WPCS going to support this feature, or not?
Personally, I'm not crazy about constants as arrays. But my company uses constants to death, so this is a bit of a constant debate.

@holisticnetworking There is no reason why WPCS shouldn't support it as it is a PHP feature (since PHP 5.6/7.0).
Are there any particular sniffs in which you are missing support for this ?
Any specific issues with the existing sniffs which you are running into ?
Some example code and some info on what you would expect to happen when running WPCS over that code and what currently happens, would help to clarify the issue.

Fair enough!
So I'm checking into a Git repository through SourceTree. The repository is setup to automatically check the code standards on checkin. This is the message I get:
Errors parsing docroot/wp-content/plugins/company-plugin/common/common.php```

Line 96 reads:
```} elseif ( YOUTUBE_API_STATUS['UPLOAD_VIDEO_SUCCESS'] === $video_status )```

If I alter that line to assign the array YOUTUBE_API_STATUS to a local variable $youtube_status, there is no error message.

@holisticnetworking Ok, that clears things up.
Based on the info provided, this is what I think is happening:
You are either using the WordPress or the WordPress-Extra ruleset. These rulesets contain the Generic.PHP.Syntax sniff which is just a simple sniff linting the PHP in each file.
As long as you are running the sniffs on a system which is using a PHP version which is equal to or higher than your minimum supported PHP version, that should be fine and you will not see any notices (unless there are parse errors of course).
In your case, based on the info provided, running the sniffs on PHP 5.6 or 7.0 or higher, should therefore be fine.
As you are seeing the parse error, I suspect that the SourceTree checks are being run on an older PHP version.
So there are two things you can do:

Check with SourceTree if there is anything you can do in the configuration to make sure that the PHP version your checks are being run on is in line with the minimum supported PHP version for your code.
Exclude the Generic.PHP.Syntax sniff from your custom ruleset or via the command-line.

Side-note: the error message coming from the sniff should actually look slightly different, so I wouldn't be surprised if SourceTree has a native PHP linting check on check-ins as well.
Solution 1 should solve both issues in that case.

I knew I should have tried checking in w/o SourceTree!
Thanks so much!


Add dynamic fluid information

What does this change?
This changes the fluid properties to an interface called FluidInformation this means that mods that want to dynamically change things such as the tint or texture can as of right now they are stored once and cant be changed.

RESOURCEFUL BEES ON FABRIC BABY


feat: report exceptions when sending

report exceptions to laravel's exception handler instead of logging
update test to confirm that exceptions are still logged

Closes #32

Codecov Report

Merging #33 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #33   +/-   ##
=========================================
  Coverage     98.18%   98.18%
  Complexity       78       78
=========================================
  Files             9        9
  Lines           220      220
=========================================
  Hits            216      216
  Misses            4        4

Impacted Files
Coverage Δ
Complexity Δ

...rc/Console/Commands/SendScheduledNotifications.php
100% <100%> (ø)
3 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


UMG: unsupported value type DelegateProperty for property TextDelegate

Using a PyUserWidget as an overall root UI, I was trying to setup a binding for a TextBlock in UMG, such that the TextBlock would reflect whatever I typed in a separate InputBox.
I saw that there is the FGetText Delegate in Widget.h which is used for the binding and the TextDelegate in the TextBlock class.
I assume I could probably subclass PyUserWidget, and make custom widgets with C++ backing some BindDynamic calls, but that feels cumbersome.
Any thoughts on supporting the DelegateProperty type, and being able to bind methods via BindDynamic in Python?

hi, i will work on it next week (now we are at the reboot conference in dubrovnik)

Hi, so i have added the delegate_bind_ufunction(object, function_name) method:
text_block.delegate_bind_ufunction('TextDelegate', widget, 'FunctionName')

but for widgets i think it is better to use the  included binding system:
from unreal_engine.structs import DelegateEditorBinding
widget.Bindings = [DelegateEditorBinding(PropertyName='Text',ObjectName=text_block.get_name(),FunctionName='YourFunctionName')]

ensure your function has the required signature (must return a Text in your case and must be pure)


Fix version for bitflags - bump MSRV

Using bitflags = "~1.2" will only allow bitflags versions matching 1.2.x
Using bitflags = "1.3.2" will only allow versions above 1.3.2.
And as mentioned here cargo does not allow multiple semver compatible versions of the same crate, attempting to do so will fail to build.
This all means that if another dependency adds bitflags = "1.3.2" (a very common way to specify dependencies) then cargo update/cargo run will fail:

The easiest way to resolve this is to just bump the MSRV to support bitflags 1.3.
An alternative to the approach in this PR would be finding a way to remove the bitflags dependency entirely, feel free to close this PR and do that if you want.

Oh, and it would be great if you could make a note of the MSRV bump in the changeling.

Ay, I was not aware of this. I thought that cargo will always just install multiple versions.

Yeah its a bit of a hidden footgun :/
Thanks for the feedback, glad we can go ahead with this, everything should be addressed now.

Thanks for the PR! FYI, I have just published 0.10.1 which includes your fix.


ERROR WHEN UPDATE DEPENDENCIES

Please make sure these boxes are checked (put an 'x' inside them) before creating the issue.

[x ] I am using Python 3.5 or higher (run python --version on the command line)
[ x] I have followed the official guides to install the bot for my system
[ ] I have updated my dependencies to the latest version using the appropriate update_dependencies script
[x ] I am running version 1.9.6_3 of MusicBot

Description of issue
this happens when I start update_dependencies.bat
Steps to reproduce

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here.

this problem

Based on the launcher fail check in the background, it looks like you have a bad install. Make sure you installed correctly:
https://just-some-bots.github.io/MusicBot/#installationwindows

I did all how it said in the guide but this happens, so should I try to reinstall it?

I did all how it said in the guide so there r still this problem

I got wat the problem, the new version of a bot doesn't work with 8 windows, cuz new version of plugins which update deps downloads doesn't work with 8 windows


Update resource name, display name and description when duplicating custom serving runtime

Closes #1500
Description

When duplicating an OOTB custom serving runtime, change the preset value to:
metadata.name -> ${metadata.name}-copy
displayName -> Copy of ${displayName}
description -> description (unchanged)

How Has This Been Tested?

Go to settings -> Serving runtimes page
Choose an OOTB serving runtime and click Duplicate in the dropdown menu
Check the metadata.name field and metadata.annotations['openshift.io/display-name'] field to make sure they are updated as described above

Test Impact

N/A
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/lgtm


[Relay][Frontend] Span Filling ONNX

Set node name as the source name of span during the conversion of ONNX model.
Assign node name to a node based on op type when it is empty.
To get the reference of renamed nodes. Add a function to export the ONNX model after conversion.
Add structural_equal comparisons with and without set_span to the existing test cases.
Add span test cases for frequent conversions.
Add span test case for exporting model parameter.

Hi community,
This is a part of the RFC, TVM Explorer Infrastructure. After this PR, the source node information of ONNX will be tagged to the converted relay.
Note that the node names might not exist in the some model. Therefore we did the following changes for span mapping:

Assign new node names based on their op types when they are missing during the conversion.
Add a parameter to from_ONNX to export the model after the conversion.

Node names of the exported model can be a reference to the source_names of spans. Feel free to give us any comment. Thank you! :D
For your reference, here is the related links.
PreRFC in forum
apache/tvm-rfcs#92
#13116
@haowhsu-quic, @zack-ch


Runing migration even if the migrations are not loaded from module service provider

Versions:

laravel-modules Version: 11.1.0 - 11.1.1
Laravel Version: 11.23.5
PHP Version: 8.3.6

Description:
I commented loadMigrationsFrom function to ensure that no migration should be run by module itself
and let the tenancy provider to do so because i have a modular system and is multi-tenanted database.
but after updating the composer all my module migration are going to central database and this is not what we pretend.
but after downgrade to version 11.0.11 everything   works fine.
Thanks.
Steps To Reproduce:
 # ModuleServiceProvider
        //$this->loadMigrationsFrom(module_path($this->moduleName, 'database/migrations'));

php artisan migrate

…

Hi,
In version 11.1.0, we introduced auto-registration of migrations for all enabled modules (#1889, #1945).
You can sync your config file with the new version and disable this feature if needed by updating the following section:
/*
|--------------------------------------------------------------------------
| Auto Discover of Modules
|--------------------------------------------------------------------------
|
| Configure the auto-discovery of modules here.
| This simplifies module service provider management.
|
*/
'auto-discover' => [
    /*
    |--------------------------------------------------------------------------
    | Migrations
    |--------------------------------------------------------------------------
    |
    | This option automatically registers migrations.
    |
    */
    'migrations' => true,

    /*
    |--------------------------------------------------------------------------
    | Translations
    |--------------------------------------------------------------------------
    |
    | This option automatically registers language files.
    |
    */
    'translations' => false,
],

Thanks that works


HmIP-PDT contains invalid control

I'm trying out the new integration with my setup and i think i found an issue.
For the HmIP-PDT, it seems like there is an invalid control, i believe a wrong channel is shown (HmIP-PDT 000...:4), so it's probably channel 4

I also think that actual temperature is not needed as it doesn't contain any value.

Hi
actual_temperature has already been removed from integration, so this is a leftover in your installation. Just click on the entity and click remove.
Stehlampe should be the correct on channel 3 and the other entity could als be deleted.
After a restart the entities should not come back.

Hi,
i just made a fresh install of the integration and now everything looks fine:

I installed the integration shortly after it was available in hacs and updated it since then, just didn't find any time to try it out.
But now everything is set up correctly for a new installation, thanks.


DMonitoring Inputs

Hello,
I am trying to run dmonitoring_model.keras on my computer. I am just wondering what is the input format for this model and where can I find code that processes inputs for it?
Thanks

you can check out
https://github.com/commaai/openpilot/blob/f3fb406eee85b072c727ccaf027bd2dacd3aa69a/selfdrive/modeld/models/dmonitoring.cc#L37


(PUP-3645) Add support for configurable url prefix

This commit support for a configurable setting master_url_prefix, which
defaults to /puppet, that the master's http endpoints are mounted at. All
requests generated by the agent honor this prefix. This will allow us to mount
the Puppet HTTP endpoints in a namespaced URL space alongside other apps
running on the same webserver.

CLA signed by all contributors.

CLA signed by all contributors.

+1 ; will leave this open for a few more days in case other folks want to review it as well, but will plan on merging by mid-week unless anyone raises concerns.

@rlinehan I was surprised when I discovered that I didn't need an exact match with the URL prefix.  For example, I started my master with a prefix of foo and then did an agent run with a prefix of foo1 expecting it to fail but it worked.  I suppose it's still a "prefix" and not an exact match...
Is this desirable? Or is it unavoidable? Is there a real-world scenario in which this could cause problems, like if there were multiple masters with prefixes like ps, ps1, ps2 etc.?

@nwolfe hah yep, you have found the hidden easter egg, or something. This is fixed/worked around in the next PR (#3464) by adding a slash to the end of the prefix when the routes are defined, because this was causing problems when the CA routes were separated out with a default prefix of /puppet-ca.

:+1: looks good! Verified master_url_prefix is honored by agent and CLI tools (certificate, certificate_revocation_list, etc).

:+1: Verified the prefix is honored by the agent.


Document the way to use default and custom formatters

I remembered that MessageFormat supported formatting of dates and numbers, but when I tried that, I got errors like fmt.date is not a function. I had to dig into the code to find that even the default formatters have to be included separately.
It would be great to mention that in a more visible part of https://messageformat.github.io/

You're right. I'll take a look at the documentation, and try to make this a bit more obvious. Unfortunately, node 0.10 and IE 10 are still widely enough used that defaulting to code that would fail on them is not really viable.


Error in dps calculation

My dps in game and 34k
Path building the dps is 39k
The error approaches 7% and as the damage increases
The dps error also increases

fall of oriath

Well, you have to provide your PoB profile at least.


训练一个LongformerForSequenceClassification模型，报错

训练一个模型，报错：
代码大致如下：
"""
Erlangshen Longformer
"""
from datasets import Dataset
import pandas as pd
from fengshen.models.longformer import LongformerForSequenceClassification
from transformers import BertTokenizer
from transformers import DataCollatorWithPadding
import evaluate
import numpy as np
from transformers import TrainingArguments, Trainer
import torch
input_field = "content"
model_name = "IDEA-CCNL/Erlangshen-Longformer-110M"
tokenizer = BertTokenizer.from_pretrained(model_name)
max_input_size = 1536
accuracy = evaluate.load("accuracy")
data_collator = DataCollatorWithPadding(tokenizer=tokenizer)
id2label = {0: "正面", 1: "中性", 2: "负面"}
label2id = {"正面": 0, "中性": 1, "负面": 2}
model = LongformerForSequenceClassification.from_pretrained(
model_name, num_labels=len(id2label), id2label=id2label, label2id=label2id
)
def preprocess_function(examples):
return tokenizer(examples[input_field], truncation=True, max_length=max_input_size)
def compute_metrics(eval_pred):
predictions, labels = eval_pred
predictions = np.argmax(predictions, axis=1)
return accuracy.compute(predictions=predictions, references=labels)
train = pd.read_csv("data/Train_DataSet.csv")
train_label = pd.read_csv("data/Train_DataSet_Label.csv")
train=train.merge(train_label,on='id',how='left')
train['label']=train['label'].fillna(-1)
train=train[train['label']!=-1]
train['label']=train['label'].astype(int)
train["content"] = train["content"].fillna("无")
train["title"] = train["title"].fillna("无")
dataset = Dataset.from_pandas(train[[input_field, 'label']])
dataset = dataset.train_test_split()
tokenized_ds = dataset.map(preprocess_function, batched=True)
inputs, labels = tokenized_ds['test'][input_field], tokenized_ds['test']['label']
torch.autograd.set_detect_anomaly(True)
training_args = TrainingArguments(
output_dir="longformer_chinese_sentiment",
learning_rate=5e-4,
per_device_train_batch_size=1,
per_device_eval_batch_size=1,
num_train_epochs=4,
weight_decay=0.01,
evaluation_strategy="epoch",
save_strategy="epoch",
load_best_model_at_end=True,
push_to_hub=False,
optim='adamw_torch',
)
trainer = Trainer(
model=model,
args=training_args,
train_dataset=tokenized_ds["train"],
eval_dataset=tokenized_ds["test"],
tokenizer=tokenizer,
data_collator=data_collator,
compute_metrics=compute_metrics,
)
trainer.train()
报错如下：
[2023-08-25 13:35:24,607] [INFO] [real_accelerator.py:158:get_accelerator] Setting ds_accelerator to cuda (auto detect)
Map: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 5505/5505 [00:19<00:00, 286.92 examples/s]
Map: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 1835/1835 [00:06<00:00, 281.76 examples/s]
/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/transformers/modeling_utils.py:900: FutureWarning: The device argument is deprecated and will be removed in v5 of Transformers.
warnings.warn(
/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/torch/autograd/init.py:200: UserWarning: Error detected in BmmBackward0. Traceback of forward call that caused the error:
File "ErLS_LF.py", line 90, in
trainer.train()
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/transformers/trainer.py", line 1555, in train
return inner_training_loop(
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/transformers/trainer.py", line 1837, in _inner_training_loop
tr_loss_step = self.training_step(model, inputs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/transformers/trainer.py", line 2682, in training_step
loss = self.compute_loss(model, inputs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/transformers/trainer.py", line 2707, in compute_loss
outputs = model(**inputs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/fengshen-0.0.1-py3.8.egg/fengshen/models/longformer/modeling_longformer.py", line 2064, in forward
outputs = self.longformer(
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/fengshen-0.0.1-py3.8.egg/fengshen/models/longformer/modeling_longformer.py", line 1880, in forward
encoder_outputs = self.encoder(
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/fengshen-0.0.1-py3.8.egg/fengshen/models/longformer/modeling_longformer.py", line 1472, in forward
layer_outputs = layer_module(
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/fengshen-0.0.1-py3.8.egg/fengshen/models/longformer/modeling_longformer.py", line 1396, in forward
self_attn_outputs = self.attention(
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/fengshen-0.0.1-py3.8.egg/fengshen/models/longformer/modeling_longformer.py", line 1331, in forward
self_outputs = self.self(
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/fengshen-0.0.1-py3.8.egg/fengshen/models/longformer/modeling_longformer.py", line 779, in forward
attn_output = self._compute_attn_output_with_global_indices(
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/fengshen-0.0.1-py3.8.egg/fengshen/models/longformer/modeling_longformer.py", line 1113, in _compute_attn_output_with_global_indices
attn_output_only_global = torch.matmul(
(Triggered internally at ../torch/csrc/autograd/python_anomaly_mode.cpp:114.)
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
Traceback (most recent call last):
File "ErLS_LF.py", line 90, in
trainer.train()
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/transformers/trainer.py", line 1555, in train
return inner_training_loop(
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/transformers/trainer.py", line 1837, in _inner_training_loop
tr_loss_step = self.training_step(model, inputs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/transformers/trainer.py", line 2693, in training_step
self.accelerator.backward(loss)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/accelerate/accelerator.py", line 1923, in backward
loss.backward(**kwargs)
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/torch/_tensor.py", line 487, in backward
torch.autograd.backward(
File "/root/miniconda3/envs/deeplearning/lib/python3.8/site-packages/torch/autograd/init.py", line 200, in backward
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.cuda.FloatTensor [12, 1536, 1]], which is output 0 of AsStridedBackward0, is at version 1; expected version 0 instead. Hint: the backtrace further above shows the operation that failed to compute its gradient. The variable in question was changed in there or anywhere later. Good luck!
请帮忙看一下问题出在哪里

I also encountered the same problem, have you solved it?

I also encountered the same problem, have you solved it? @ganzhiruyi @hejunqing May I ask how to solve this problem?

I also encountered the same problem, have you solved it? @ganzhiruyi @hejunqing May I ask how to solve this problem? Thanks a lot!


Simple website and a Logo for installer.to

Done some work for issue #7 and #9
#9
simple logo for installer.to and also simple brand design for what to do what not to do.
Symbol(Logo), two down arrows head represent the "download or something installing to a computer"
Primary color - #0077EE1
Secondary color - #9B9B9B
Dark mode - #1C1C1C

#7
Completely Mobile friendly responsive simple website for documentation purposes to use as installer primary website.
** Used libraries -**

Bootstrap 3.3 for the responsive purpose.
JQuery
Waypoints
JQuery flex-slider
line-awesome as the icon lib
animate.css
etc..

Important
Contend need to be added.

Nice UIs and the logos looks awesome. Good work @asithc


Improve GraphQL utilities

We need to enhance our GraphQL API by implementing custom resolvers that manipulate the data returned from the queries to match the specific data types and structures required by the SDK and its consumers.
A prime example is the GraphQL type for UTXOs: InputCoin which needs to be manipulated to the corresponding SDK type: Resource.
We need to make this change to improve things not only for the internal use of the SDK but also for the SDK consumers.
Having said that, a deep analysis should be made not only in the SDK but also on the block explorer to identify all possible resolvers that can be created to reduce the data manipulation after queries, as the block explorer executes a lot of GraphQL queries.

Do you mean that we write resolvers such that e.g. a call to provider.operations.getCoinsToSpend would give the same result as the current provider.getCoinsToSpend?

Do you mean that we write resolvers such that e.g. a call to provider.operations.getCoinsToSpend would give the same result as the current provider.getCoinsToSpend?

@nedsalk Not exactly. These resolvers should take care only of the data manipulation that happens before returning the data. For this specific case I believe would be this part https://github.com/FuelLabs/fuels-ts/blob/53dafb1c6e5e02e47f805c111a8f348278101a5f/packages/providers/src/provider.ts#L898-L927.
In this specific case, it could be a little different, I did not go deep into it.


PullAsync: Incremental sync will sync the last created item over and over because updatedAt uses ge

I'm currently getting instances where PullAsync with a unique queryId returns the last item every time PullAsync is called because the filter for updatedAt uses ge (greater or equal) rather than just greater than). As you can see below, the requests have (updatedAt ge datetimeoffset'2015-12-13T04:28:12.8160000%2B00:00') while the updatedAt of the Response matches that "updatedAt":"2015-12-13T04:28:12.816Z" therefore always syncing the last created item and creating two requests to be sent from the one call as shown below.
This also causes to requests to be sent to the server which should be avoided.
Requests:
2015-12-22 11:55:01.147 AppiOS[1694:7409279]
Request: https://app-code.azurewebsites.net/tables/Child?$filter=(updatedAt ge datetimeoffset'2015-12-13T04:28:12.8160000%2B00:00')&$orderby=updatedAt&$skip=0&$top=50&__includeDeleted=true

2015-12-22 11:55:01.643 AppiOS[1694:7409279]
Response:   [{"deleted":false,"updatedAt":"2015-12-13T04:28:12.816Z","createdAt":"2015-11-17T22:45:21.475Z","version":"AAAAAAAAJyY=","id":"c1816d8d-046a-4144-9e30-1eba6fbf7888","photo":"51ec63a3-5a50-4e7b-82db-edc5bd13831e.jpg","displayOrder":0,"identityColor":"#ef222f","lastName":"Last","firstName":"First"}]

2015-12-22 11:55:01.659 AppiOS[1694:7409279]
Request: https://app-code.azurewebsites.net/tables/Child?$filter=(updatedAt ge datetimeoffset'2015-12-13T04:28:12.8160000%2B00:00')&$orderby=updatedAt&$skip=1&$top=50&__includeDeleted=true

2015-12-22 11:55:01.733 AppiOS[1694:7409279]
Response:   []

My method
public async Task<ObservableCollection<Child>> ChildrenGetAsync()
		{
			if (!initialized)
				await Init();

			if (await ConnectivityService.IsConnected())
			{
				var children = new ObservableCollection<Child>();

				using (new PerformanceTester("ChildrenGetAsync"))
				{
					await childSyncTable.PullAsync("ChildrenGet" + AzureService.Instance.CurrentMobileServiceUser.UserId, childSyncTable.CreateQuery());
					List<Child> items = await childSyncTable.ToListAsync();
					children = new ObservableCollection<Child>(items.OrderBy(x => x.FirstName));
				}
				return children;
			}
			return new ObservableCollection<Child>();
		}

We currently use ge, as we can not guarantee we received all the data at the last updatedAt we got (could be more on page 2, etc)  see: https://github.com/Azure/azure-mobile-services/issues/680 for a better option, not sure where the tracking issue is on the -apps versions to link here

Tracking this in the new client SDK repo as Azure/azure-mobile-apps-net-client/issues/51


Add PowerShell installer for Windows

Add PowerShell installer for Windows.

OK.

@itok01 Can you update README.md for it?

Yes, I'll do it.

Thanks.


codegen: add support for abstract classes

Abstract classes only contain static methods (they implement a static interface) and the code generation was failing because we were trying to generate their signature. The signature is used to create new instances of generic classes, so we can skip its generation on abstract classes.
The commit also removes the class struct and its constructor, since they were unused.
fixes #79

Thanks a lot! I spent some time trying to get this to work, but it helps if you're familiar with the codebase. In the meantime, I figured out I don't actually need WinRT for my use-case though, so I might not end up using this. Still, I'm sure someone else will!


Move suffix from timestamp to readable format

as asked in issue #6  from paf1:

Well , this is fixed now, but would you generate human number format suffix (<PHONE_NUMBER> eg:? ), pls.,
regs. Pavel

Hello,
fixed, runing OK, with no problems .
Still one note, would you delete seconds from string ? Not needed.
regs.
Pavel


Scss import not working in react MFE

I am using the single spa react typescript SSR. For the CSS we were using styled components. Currently, we have a requirement that there are some third-party components that need to be integrated.
The issue with those components is the styling part which is using the SCSS import for their components. Attached is the screenshot.

So I need to set the support for SCSS in webpack file and I have added the configuration for the same but SCSS failed to run in server-side rendering and loaded on the client side, but my requirement is to run the SCSS imports from the server side. Below is the webpack file  and
webpack.config.js
const webpackMerge = require('webpack-merge');
const singleSpaDefaults = require('./build-config/webpack/webpack-react-ts.config');
const path = require('path');
const EsmWebpackPlugin = require('@purtuga/esm-webpack-plugin');
const Dotenv = require('dotenv-webpack');
const UglifyJsPlugin = require('uglifyjs-webpack-plugin');
const MiniCssExtractPlugin = require("mini-css-extract-plugin");
module.exports = webpackConfigEnv => {
rules: [
{
test: /.mjs$/,
include: /node_modules/,
type: 'javascript/auto'
}
];
const defaultConfig = singleSpaDefaults({
orgName: 'bjs',
projectName: 'static-micro-frontend',
webpackConfigEnv
});
const serverConfig = singleSpaDefaults({
orgName: 'bjs',
projectName: 'static-micro-frontend',
webpackConfigEnv
});
defaultConfig.plugins = defaultConfig.plugins.filter(
p => p.constructor.name !== 'CleanWebpackPlugin'
);
serverConfig.plugins = serverConfig.plugins.filter(
p => p.constructor.name !== 'CleanWebpackPlugin'
);
const alias = {
'@components': path.resolve(__dirname, 'src/components'),
'@assets': path.resolve(__dirname, 'src/assets')
};
return [
webpackMerge.smart(defaultConfig, {
resolve: {
alias
},
optimization: {
minimizer: [new UglifyJsPlugin()]
},
devServer: {
compress: true,
port: 8085
}
}),
webpackMerge.smart(serverConfig, {
// modify the webpack config however you'd like to by adding to this object
target: 'node',
mode: 'development',
entry: path.resolve(process.cwd(), 'src/node-entry.tsx'),
module: {
rules:[{
test: /.(scss|sass)$/,
use: [
MiniCssExtractPlugin.loader,
          "style-loader",
          {
            loader: 'css-loader',
            options: {
              modules: true
            }
          },
          {
            loader: 'sass-loader',
            options: {
              implementation: require('node-sass')
            }
          }
        ]
    }]
  },
  output: {
    library: 'mf',
    libraryTarget: 'var',
    filename: 'server.mjs'
  },
  devServer: {
    compress: true,
    port: 8085
  },
  externals: defaultConfig.externals.concat(/react-dom\/.+/).concat(/^@bjs\/?.*$/),
  plugins: [
    new EsmWebpackPlugin({
      moduleExternals: true
    }),
    new MiniCssExtractPlugin(),
    new Dotenv()
  ],
  resolve: {
    alias
  }
})

];
};
Below is the error

Need help with this error

Hi, did you solve the issue?

Hey MilanKovacic,
The issue has been resolved.
In the webpack configuration, I removed the style-loader and MiniCssExtractPlugin.loader and replaced them with an isomorphic style loader, which successfully resolved my issue.
Thank you!


chore: deprecate network URLs, add asset ID and chain ID for mainnet

Closes #2509

Release notes
In this release, we:

Deprecated the two network URLs, added asset ID and chain ID for mainnet

Summary
This PR deprecates the two network URLs currently exported by the SDK, and adds the asset ID and chain ID for mainnet.
Checklist

[x] All changes are covered by tests (or not applicable)
[x] All changes are documented (or not applicable)
[x] I reviewed the entire PR myself (preferably, on GH UI)
[x] I described all Breaking Changes (or there's none)

I haven't deprecated the old URLs like the original issue suggests because I think people will still need them, at least the testnet one for sure. Thoughts? cc @petertonysmith94 @arboleya

I personally believe deprecation would be the best way forward.
Users managing their own endpoints seems like a reasonable ask to me. Also removes coupling our SDK to specific endpoints.

@maschad In this case, they must be defined under the internal namespace and can only be used by tests.
Is that what you had in mind?

@arboleya yes I think so, my main point is essentially we no longer need to export these constants, but they are still useful for our tests.

Great suggestion @maschad. Done


[skia] Adding support for other graphics APIs

Updating skia to add support for other APIs other than OpenGL.

What does your PR fix?

Which triplets are supported/not supported? Have you updated the CI baseline?
Unchanged from previous. CI baseline is updated.

Does your PR follow the maintainer guide?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Please run command "./vcpkg x-add-version skia --overwrite-version" then commit the changes again

@dave-juicelabs ,Please run command "./vcpkg x-add-version skia --overwrite-version" then commit the changes again

@dave-juicelabs ,Please run command "./vcpkg x-add-version skia --overwrite-version" then commit the changes again

@dave-juicelabs ,Could you please take a look:
[799/799] link skia.dll skia.dll.lib skia.dll.pdb
FAILED: skia.dll skia.dll.lib skia.dll.pdb
"C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC/Tools/MSVC/14.29.30133/bin/HostX64/x64/link.exe" /nologo /IMPLIB:./skia.dll.lib /DLL /OUT:./skia.dll /PDB:./skia.dll.pdb @./skia.dll.rsp
   Creating library ./skia.dll.lib and object ./skia.dll.exp
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_blob_create referenced in function "class std::unique_ptr<struct hb_blob_t,struct SkFunctionWrapper<void __cdecl(struct hb_blob_t *),&void __cdecl hb_blob_destroy(struct hb_blob_t *)> > __cdecl to_blob(class sk_sp<class SkData>)" (?to_blob@@YA?AV?$unique_ptr@Uhb_blob_t@@U?$SkFunctionWrapper@$$A6AXPEAUhb_blob_t@@@Z$1?hb_blob_destroy@@YAX0@Z@@@std@@V?$sk_sp@VSkData@@@@@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_blob_destroy referenced in function "public: __cdecl <lambda_1b826c87cf9bf7a1c0fcf379fd41c482>::operator()(void const *,void *)const " (??R<lambda_1b826c87cf9bf7a1c0fcf379fd41c482>@@QEBA@PEBXPEAX@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_blob_get_data referenced in function "class sk_sp<class SkData> __cdecl to_data(class std::unique_ptr<struct hb_blob_t,struct SkFunctionWrapper<void __cdecl(struct hb_blob_t *),&void __cdecl hb_blob_destroy(struct hb_blob_t *)> >)" (?to_data@@YA?AV?$sk_sp@VSkData@@@@V?$unique_ptr@Uhb_blob_t@@U?$SkFunctionWrapper@$$A6AXPEAUhb_blob_t@@@Z$1?hb_blob_destroy@@YAX0@Z@@@std@@@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_set_add referenced in function "public: __cdecl <lambda_7aa85c9c4411a3e1c4254e48c2302f10>::operator()(unsigned int)const " (??R<lambda_7aa85c9c4411a3e1c4254e48c2302f10>@@QEBA@I@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_face_create referenced in function "class sk_sp<class SkData> __cdecl subset_harfbuzz(class sk_sp<class SkData>,class SkPDFGlyphUse const &,int)" (?subset_harfbuzz@@YA?AV?$sk_sp@VSkData@@@@V1@AEBVSkPDFGlyphUse@@H@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_face_destroy referenced in function "public: void __cdecl SkFunctionWrapper<void __cdecl(struct hb_face_t *),&void __cdecl hb_face_destroy(struct hb_face_t *)>::operator()<struct hb_face_t * &>(struct hb_face_t * &)const " (??$?RAEAPEAUhb_face_t@@@?$SkFunctionWrapper@$$A6AXPEAUhb_face_t@@@Z$1?hb_face_destroy@@YAX0@Z@@QEBAXAEAPEAUhb_face_t@@@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_face_reference_blob referenced in function "class sk_sp<class SkData> __cdecl subset_harfbuzz(class sk_sp<class SkData>,class SkPDFGlyphUse const &,int)" (?subset_harfbuzz@@YA?AV?$sk_sp@VSkData@@@@V1@AEBVSkPDFGlyphUse@@H@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_subset_input_create_or_fail referenced in function "class sk_sp<class SkData> __cdecl subset_harfbuzz(class sk_sp<class SkData>,class SkPDFGlyphUse const &,int)" (?subset_harfbuzz@@YA?AV?$sk_sp@VSkData@@@@V1@AEBVSkPDFGlyphUse@@H@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_subset_input_destroy referenced in function "public: void __cdecl SkFunctionWrapper<void __cdecl(struct hb_subset_input_t *),&void __cdecl hb_subset_input_destroy(struct hb_subset_input_t *)>::operator()<struct hb_subset_input_t * &>(struct hb_subset_input_t * &)const " (??$?RAEAPEAUhb_subset_input_t@@@?$SkFunctionWrapper@$$A6AXPEAUhb_subset_input_t@@@Z$1?hb_subset_input_destroy@@YAX0@Z@@QEBAXAEAPEAUhb_subset_input_t@@@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_subset_input_glyph_set referenced in function "class sk_sp<class SkData> __cdecl subset_harfbuzz(class sk_sp<class SkData>,class SkPDFGlyphUse const &,int)" (?subset_harfbuzz@@YA?AV?$sk_sp@VSkData@@@@V1@AEBVSkPDFGlyphUse@@H@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_subset_input_set_drop_hints referenced in function "public: static class std::unique_ptr<struct hb_face_t,struct SkFunctionWrapper<void __cdecl(struct hb_face_t *),&void __cdecl hb_face_destroy(struct hb_face_t *)> > __cdecl SkPDFHarfBuzzSubset<struct hb_subset_input_t *,void>::Make(struct hb_subset_input_t *,struct hb_face_t *)" (?Make@?$SkPDFHarfBuzzSubset@PEAUhb_subset_input_t@@X@@SA?AV?$unique_ptr@Uhb_face_t@@U?$SkFunctionWrapper@$$A6AXPEAUhb_face_t@@@Z$1?hb_face_destroy@@YAX0@Z@@@std@@PEAUhb_subset_input_t@@PEAUhb_face_t@@@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_subset_input_set_retain_gids referenced in function "public: static class std::unique_ptr<struct hb_face_t,struct SkFunctionWrapper<void __cdecl(struct hb_face_t *),&void __cdecl hb_face_destroy(struct hb_face_t *)> > __cdecl SkPDFHarfBuzzSubset<struct hb_subset_input_t *,void>::Make(struct hb_subset_input_t *,struct hb_face_t *)" (?Make@?$SkPDFHarfBuzzSubset@PEAUhb_subset_input_t@@X@@SA?AV?$unique_ptr@Uhb_face_t@@U?$SkFunctionWrapper@$$A6AXPEAUhb_face_t@@@Z$1?hb_face_destroy@@YAX0@Z@@@std@@PEAUhb_subset_input_t@@PEAUhb_face_t@@@Z)
pdf.SkPDFSubsetFont.obj : error LNK2019: unresolved external symbol hb_subset referenced in function "public: static class std::unique_ptr<struct hb_face_t,struct SkFunctionWrapper<void __cdecl(struct hb_face_t *),&void __cdecl hb_face_destroy(struct hb_face_t *)> > __cdecl SkPDFHarfBuzzSubset<struct hb_subset_input_t *,void>::Make(struct hb_subset_input_t *,struct hb_face_t *)" (?Make@?$SkPDFHarfBuzzSubset@PEAUhb_subset_input_t@@X@@SA?AV?$unique_ptr@Uhb_face_t@@U?$SkFunctionWrapper@$$A6AXPEAUhb_face_t@@@Z$1?hb_face_destroy@@YAX0@Z@@@std@@PEAUhb_subset_input_t@@PEAUhb_face_t@@@Z)
.\skia.dll : fatal error LNK1120: 13 unresolved externals
ninja: build stopped: subcommand failed.

@damcclos ,It seems that the CI error has not changed from last time，after finish your suggesstion changes,please run command "./vcpkg x-add-version skia --overwrite-version" then commit the changes again

@dave-juicelabs ，Is this PR being worked on?

@JonLiu1993 Yes, I will be updating this PR.

@dave-juicelabs ,I tested feature skia[]:x64-windows and skia[]:x64-windows-static locally but failed :
ERROR at //BUILD.gn:17:3: Unable to load "F:/test-pr/skia/vcpkg/buildtrees/skia/src/fa8ee834ed-4d075674a2.clean/third_party/externals/dawn/scripts/dawn_features.gni".
  import("third_party/externals/dawn/scripts/dawn_features.gni")
  ^------------------------------------------------------------

Tested all the different features with https://github.com/dave-juicelabs/skia-vcpkg-integration.

Please get failure logs here.

Separated gn update from this PR, #21114 . Failures will be fixed there first before this should be merged.

@dave-juicelabs ,Could you please take a look:
[20/881] compile ../src/fa8ee834ed-2b62b425cf.clean/src/ports/SkFontMgr_fontconfig.cpp
FAILED: obj/src/ports/fontmgr_fontconfig.SkFontMgr_fontconfig.o
c++ -MD -MF obj/src/ports/fontmgr_fontconfig.SkFontMgr_fontconfig.o.d -DSK_R32_SHIFT=16 -DSK_ENABLE_SKSL -DSK_GAMMA_APPLY_TO_A8 -DSK_ALLOW_STATIC_GLOBAL_INITIALIZERS=1 -DGR_TEST_UTILS=1 -DSKIA_IMPLEMENTATION=1 -I../src/fa8ee834ed-2b62b425cf.clean -Wno-attributes -fstrict-aliasing -fPIC -fvisibility=hidden -g -Wall -Wextra -Winit-self -Wpointer-arith -Wsign-compare -Wvla -Wno-deprecated-declarations -Wno-maybe-uninitialized -Wno-psabi -Wno-unused-parameter -std=c++17 -fvisibility-inlines-hidden -fno-exceptions -fno-rtti -Wnon-virtual-dtor -Wno-noexcept-type -Wno-redundant-move -c ../src/fa8ee834ed-2b62b425cf.clean/src/ports/SkFontMgr_fontconfig.cpp -o obj/src/ports/fontmgr_fontconfig.SkFontMgr_fontconfig.o
../src/fa8ee834ed-2b62b425cf.clean/src/ports/SkFontMgr_fontconfig.cpp:27:10: fatal error: fontconfig/fontconfig.h: No such file or directory
   27 | #include <fontconfig/fontconfig.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

Builds on #21114, which should be merged first.

Now depends on #21498

Please merge to master first.

Looks okay to me.

@dave-juicelabs ,Thanks for your hard working, I tested all features locally but failed:
F:\test-pr\skia\vcpkg\buildtrees\skia\build-x64-windows-dbg-out.log:
ninja: Entering directory `F:/test-pr/skia/vcpkg/buildtrees/skia/x64-windows-dbg'

F:\test-pr\skia\vcpkg\buildtrees\skia\build-x64-windows-dbg-err.log:
ninja: error: obj/third_party/spirv-cross/spirv_cross.ninja:11: multiple rules generate obj/third_party/externals/spirv-cross/spirv_cross.spirv_cfg.obj [-w dupbuild=err]

Please use vcpkg spirv-cross port

@JonLiu1993 What command line did you use to generate that error?

@JonLiu1993 What command line did you use to generate that error?

./vcpkg install skia[*]:x64-windows

@dave-juicelabs, I tested all features again on different machines and failed because of the same error.

I've been caught up in other work but I have been able to reproduce this error. I will be getting back to this shortly.

@dave-juicelabs , any updates?

Not sure what the failure is or how to debug it at the moment.

Not sure what the failure is or how to debug it at the moment.

Please make sure you're up to date with master.

The regression is a python3 issue which is not related to this PR.

Depends on https://github.com/microsoft/vcpkg/pull/23525.

/azp run

/azp run

Close and reopen this PR to re-trigger the ci test.

@dave-juicelabs ,Thanks for your hard working, I tested all features locally by the command "./vcpkg install skia[*] "but failed:
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:158 (message):
    Command failed: "C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" -C F:/Feature-test/skia/vcpkg/buildtrees/skia/x86-windows-dbg :skia
    Working Directory: F:/Feature-test/skia/vcpkg/buildtrees/skia/src/fa8ee834ed-0fb838c932.clean
    See logs for more information:
      F:\Feature-test\skia\vcpkg\buildtrees\skia\build-x86-windows-dbg-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_build_ninja.cmake:20 (vcpkg_execute_build_process)
  scripts/cmake/vcpkg_build_ninja.cmake:41 (z_vcpkg_build_ninja_build)
  installed/x64-windows/share/vcpkg-gn/vcpkg_gn_install.cmake:123 (vcpkg_build_ninja)
  ports/skia/portfile.cmake:279 (vcpkg_gn_install)
  scripts/ports.cmake:145 (include)

Error: Building package skia:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `git pull` and `.\vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+skia
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[skia]+Build+error
including:
  package: skia[core,dawn,direct3d,gl,metal,vulkan]:x86-windows -> canvaskit/0.29.0

build-x86-windows-dbg-out.log

@JonLiu1993 Seems to be the Python 3 build issue again.

@dave-juicelabs, Could you please merge the master? I have no permission to push your branch

/azp run

@dave-juicelabs ,Is there any progress on this pr?

Closing this PR since it seems that no progress is being made. Please reopen if work is still being done.


samples: Bluetooth: add code-sample tags to hci_ samples

Add zephyr:code-sample header to the samples exposing the controller over HCI.
They should now be linked to the hci_raw documentation page, along with a nicer overlay when referring to them from other places in the docs.

I don't have time for this, sorry..


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 73 ms

Resolved: ClamAV performance has improved in b686f22.


Doc Check Request: Exchange 2019 is not mentioned as a supported version, yet is mentioned in the article as a link.

In step 2 of the pre-requisites, we are linking Exchange 2019, yet we don't mention it as a supported version in step 1. Can we please verify and correct this?

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

ID: 5033cf14-8912-471d-66b7-61ca7e29d752
Version Independent ID: a7c05a79-cd65-1717-d4a4-0dde3c368263
Content: Connect to Exchange Server (on-premises) - Power Platform
Content Source: power-platform/admin/connect-exchange-server-on-premises.md
Service: power-platform
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

@HugoDurana
Thanks for bringing this to our attention. This has been corrected.


Replace detox with parallel tox

Tox 3.7 is out with parallel mode, see https://tox.readthedocs.io/en/latest/changelog.html#v3-7-0-2019-01-11 That should make the tox.ini even simpler.

Thanks for the heads up, done in 31de3b1


Export Gaussian Quality

The rendering effect looks good in the NerfStudio viewer when I train with Gaussian in NerfStudio. However, when I export the .ply file and edit it using Super Splat, I find the rendering effect deteriorates. What could be the reason for this?
nerfstudio：

supersplat：

This is a known issue. The rendering pipeline of nerfstudio's cuda rasterizer is not exactly the same as many web viewer which uses webGL/webGPU shader implementation. There are several factors that contributes to the issue (e.g. different color clamping schema, ignoring SH high order components, etc.).
The plan I have in mind is to develop a web viewer compatible training of splafacto. This feature is WIP.

Thank you very much for your reply, looking forward to it

There is a parallel effort to build a nerfstudio compatible viser viewer: https://github.com/nerfstudio-project/viser/pull/110


click to start new game

It's annoying the way you have to click in the square to start a game, since you end up with the mouse pointer in the way.
I thought maybe it was done this way to get the browser to activate the widget somehow but...
it's doubly annoying that you have to do this between each game. Should be able to press space/enter to restart

+1

Closing due to PR #5 fixing this :)


⚠ Processors has degraded performance

In ab4b910, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 57 ms

Resolved: Processors performance has improved in eaa7370.


Scaden simulate, data input

Hi!
First of all, thanks for sharing your work :)
According to the documentation, when simulating the data from our single-cell dataset, we should normalize these counts by the library size. However, the example counts file I see is not strictly normalized, so I was wondering if a strict normalization (i.e., dividing the raw counts by the sum for that sample) would work, since these data would only contain values from 0 to 1.
Thank you so much in advance!

Hi!
Sorry for the late reply!
The counts are normalized exactly to this range inside Scaden's processing scheme, with a normalization method chosen for this. So while you could normalize your counts to 0-1 and it would probably still work, I wouldn't advice doing that.
Hope that answers your question?
Cheers,
Kevin

Yes! It does answer it!
Thank you so much,
Gemma

Sorry to bother you again... Our single cell data should anyway be library normalized before using scaden simulate, shouldn't it? I think you were using scanpy to get your single cell data library normalized counts but instead of being summed to 1, they added to the median of the library size before normalization. Am I right?


Implement Bucket for kv feature

[x] new()
[x] contains(&self, _key: Any)
[x] get(&self, _key: Any)
[x] set(&self, _key: Any, _value: Any)
[x] remove(&self, _key: Any)
[x] iter(&self)
[x] iter_range(&self, a: Any, b: Any)
[ ] iter_prefix(&self, a: Any)
[ ] watch(&self, key: Any)
[ ] prev_key(&self, key: Any)
[ ] next_key(&self, key: Any)
[ ] pop(&self, key: Any)
[ ] pop_back(&self)
[ ] pop_front(&self)
[x] len(&self)
[x] is_empty(&self)

This issue breaks down to #50 #51 #52 #53 #54 #55 #56


Same code which connects to Azure SQL database with enabling TLS encryption works in Windows 10 but fails in Ubuntu 14.04.5 LTS

Issue
I use below code to connect to a Azure SQL database with enabling TLS encryption. It works in Windows 10 but fails in Ubuntu 14.04.5 LTS. The error in Ubuntu is as below. Expect your help!
 My Linux VM is inside Azure so the connection will have a connection policy of Redirect by default. After I change the policy of Azure SQL server to proxy. It works.
https://docs.microsoft.com/en-us/azure/sql-database/sql-database-connectivity-architecture


Website readme is out of date

The command
deno -A —watch main.ts
should be replaced with
dev.ts

I assume you are talking about /www/README.md file in this repository. This is already being fixed by https://github.com/denoland/fresh/pull/448.
deno run -A --watch=static/,routes/ dev.ts is already the default command for newly generated projects.
By the way, I would suggest you format the code in issues with ` `. You can also use ``` ``` for multi-line blocks of code. This really helps by making the code more readable.

Thanks


[FEATURE] Continue Buttons Style Improvement

Description
The Continue buttons on the home page of Milan webpage is not styled yet and kept primary so CSS features may be added to it.
good first issue
Screenshots

Additional information
No response

It's great having you contribute to Milan ! This is an auto-generated textThe maintainers/owner will look into the issue soon, add relevant tags and proceed further, please have paitence. Meanwhile feel free to support star the repo and share it with your friends. :nerd_face: :rocket:

Hi @IAmTamal ,
Could you please assign me this issue ?

Hi there, the issue won't add any real value/changes. We 'd prefer the button as it is !


feat(http): Add decompression support

~The code is horrible since I've never done compression & I have very little experience with buffers. While I could split the code into functions and make it more readable I don't know if I'm even on the right track.~
~I hope to receive some guidance on how it should be done before I tidy it up.~
The decompression is implemented with aggregate -> reader instead of AsyncRead. I don't know how big of a problem this is, but Body -> AsyncReader was very hard to implement (unless I'd pull in tower) upstream issue)
I've tested the code and it's working.

The new features will also need to be added to the readme

I plan to switch to brotli using async-compression. Will hopefully be done this weekend. Please wait until then for code review

Ready for review unless we'd like to hold of until the decompressor is async (shouldn't matter since it's fast enough to not be blocking + would add too much code imo).

I am not a big fan of mixing in unrelated cleanup in a pull request it makes it all a bit messy, escially if we later have to go back and figure out why some behavior changed.

I am not a big fan of mixing in unrelated cleanup in a pull request it makes it all a bit messy, escially if we later have to go back and figure out why some behavior changed.

Same here, I'd like to have the unrelated changes reverted before approving

The "unrelated" changes will just rot :/ the PR modifies code that's is close enough that I felt it justifiable to include.

Those changes are welcome, just not in this PR. If you would like to make another PR specifically for cleanup, I'm all for that.

Brotli internally uses a buffer which defaults to 4096 bytes and I added a buffer to twilight of 256 bytes. The brotli and twilight buffer should probably be the same size and a good default size should be found before this is merged, 256 was arbitrary.


An Observable.FromEvent observable throws ArgumentException upon subscription

Bug
An Observable.FromEvent<EventHandler, EventArgs> observable throws ArgumentException "Cannot bind to the target method because its signature is not compatible with that of the delegate type" upon subscription.

Which library version?

5.0.0

What are the platform(s), environment(s) and related component version(s)?

Windows, .NET 6.0.5

What is the use case or problem?

Get an observable from an event, convert it to a task and await the task

What is the expected outcome?

Task completes when the event is triggered

What is the actual outcome?

ArgumentException: "Cannot bind to the target method because its signature is not compatible with that of the delegate type"

What is the stacktrace of the exception(s) if any?

System.ArgumentException: Cannot bind to the target method because its signature is not compatible with that of the delegate type.
   at System.Reflection.RuntimeMethodInfo.CreateDelegateInternal(Type delegateType, Object firstArgument, DelegateBindingFlags bindingFlags)
   at System.Reflection.RuntimeMethodInfo.CreateDelegate(Type delegateType, Object target)
   at System.Reactive.ReflectionUtils.CreateDelegate[TDelegate](Object o, MethodInfo method) in /_/Rx.NET/Source/src/System.Reactive/Internal/ReflectionUtils.cs:line 18
   at System.Reactive.Linq.ObservableImpl.FromEvent`2.GetHandler(Action`1 onNext) in /_/Rx.NET/Source/src/System.Reactive/Linq/Observable/FromEvent.cs:line 181
   at System.Reactive.Linq.ObservableImpl.EventProducer`2.Session.Initialize() in /_/Rx.NET/Source/src/System.Reactive/Linq/Observable/FromEvent.cs:line 315
   at System.Reactive.Linq.ObservableImpl.EventProducer`2.Session.Connect(IObserver`1 observer) in /_/Rx.NET/Source/src/System.Reactive/Linq/Observable/FromEvent.cs:line 268
--- End of stack trace from previous location ---
   at Program.<Main>$(String[] args) in C:\FromEvent\Program.cs:line 14

Do you have a code snippet or project that reproduces the problem?

I might be missing something but this seems to be a very trivial use case of Observable.FromEvent:
using System.Reactive.Linq;
using System.Reactive.Threading.Tasks;

try
{
    var testSource = new TestSource();

    var task = Observable.FromEvent<EventHandler, EventArgs>(
        h => testSource.TestEvent += h,
        h => testSource.TestEvent -= h).FirstAsync().ToTask();

    ThreadPool.QueueUserWorkItem(_ => testSource.Fire());

    await task;
}
catch (Exception ex)
{
    Console.WriteLine(ex);
    Console.ReadLine();
}

class TestSource
{
    public event EventHandler? TestEvent;

    public void Fire() => TestEvent?.Invoke(this, EventArgs.Empty);
}

Thanks for looking into it! 🙏

Having looked at the implementation of  FromEvent, I now realize I should be just use be using FromEventPattern instead, which deals with delegates taking both object? sender and EventArgs-derived args.
This must be a FAQ, but somehow I couldn't really find anything searching for "Cannot bind to the target method because its signature is not compatible with that of the delegate type" + FromEvent.

To my future self: not that we should do it, but it's still possible to use the more generic version FomEvent (vs FromEventPattern), like this:
var task = Observable.FromEvent<EventHandler, EventArgs>(
    h => (object? sender, EventArgs e) => h(e),
    h => testSource.TestEvent += h,
    h => testSource.TestEvent -= h)
    .FirstAsync()
    .ToTask();


Migrate helm chart to ghcr.io

Overview of feature request
We had originally planned to move our Helm Chart to ghcr.io as part of publicizing the repo. At the last minute we ran into issues pulling and installing the chart in the CLI, so we reverted the changed and kept pulling from ACR.
Acceptance criteria
Support pull/install in rad init from GHCR.
Remove publish to ACR (might want to delay).
Update documentation.
AB#9841

Reopening because we reverted the PR that fixed this