Communication Channel opened by import.meta

Part of the discussion was about a communication channel opened by import.meta. It centers on the cached object being reused in a second evaluation of the module as part of Steps 3 and 4.f.
However, the module that this object is cached on is not reused another evaluation of the module. GetActiveScriptOrModule() grabs the active module from the execution context. I don't believe there can ever be a second evaluation of the same module that would reuse this execution context.
With that, I don't see import.meta being any different than a module-scoped const declaration at the beginning of the module. Something like:
const __SPECIAL_MODULE_SCOPED_IMPORT_META__ = { url: 'https://example.com' };

function foo() {
  // import.meta.url
  __SPECIAL_MODULE_SCOPED_IMPORT_META__.url
}

I don't think there are any communication channels opened by this proposal.
/cc @erights

I'm also confused why any callsite-based mechanism to produce a mutable object is problematic.
In fact, the current semantics of tagged template literal arrays - being cached per callsite and not per realm - seems to suggest that they don't, in fact, need to be frozen.
Can someone help me understand the hazard here?

Before the Tagged Template Literal revision, there was an absolute need to freeze the shared literal. This is because it could be retrieved (with the same identity) from any module at any point:
// a.js
const strings = (s => s)`foo`;
strings.bar = 1;

// b.js
const strings = (s => s)`foo`;
assert.equals(strings.bar, 1)

I don't believe import.meta is exposed in the same way, so it should be fine.

Agreed, but with that revision, there seems to be no further need - similarly, import.meta has no need.

Closing as proposal is now stage 4


Possibility to modify message on publish up/downstream callbacks.

Hello,
At this line, you have decided that the callbacks' signatures on onPublishUpstreamMessage and onPublishDownstreamMessage as Message -> IO ()s. Is it possible to create pull request to change it into Message -> Message?
So on this line, instead of
publishUpstream :: Broker auth -> Message -> IO ()
publishUpstream broker msg = do
  onPublishUpstream (brokerCallbacks broker) msg
  publishDownstream broker msg

the library use:
publishUpstream :: Broker auth -> Message -> IO Message
publishUpstream broker msg =
  onPublishUpstream (brokerCallbacks broker) msg >>= publishDownstream broker

There's a possibility that in a custom broker, the user wants to modify the message that being passed around because of some reasons.
For an example, in hummingbird' Internal.hs, instead of
, Broker.onPublishUpstream   = \_->Prometheus.incCounter (Prometheus.hummingbird_publications_upstream_total metrics)

One could use
, Broker.onPublishUpstream  msg = do
    Prometheus.incCounter (Prometheus.hummingbird_publications_upstream_total metrics)
    pure msg

Or when user wants to modify the message
, Broker.onPublishUpstream  msg = pure $ msg { msgTopic = "some/other/topic" }

There are a few trade-offs here:
Pros:

A little more flexible library.

Cons:

Added computation process.
The above computation could be abused by adding too many computation so this library's (and/or custom broker's) performance would drop a little (or much depends on the load).

https://github.com/Lupino/haskell-mqtt/commit/28f197bce5ee7a5c2c8572eaa5cfaa03f5f5d0a2


change maintainer property

Updated maintainer property from<EMAIL_ADDRESS>to https://dcos.io/community
Haven't updated these packages:
Kafka
Cassandra
Spark
Marathon
Marathon-lb
tunnel-cli
Chronos
DSE
Exhibitor
Jenkins
@triclambert @keithchambers @ssk2 let me know if these should be changed too

I'd remove Exhibitor and DSE as well, but looks good beyond that.

Lgtm as long as @graham-murphy and @keithchambers are ok with it.

@ssk2 the existing elastic packages are maintained by someone else entirely.  Our forthcoming new package should follow this community model initially.

looks like I got a quorum. @ssk2 can I ask you to merge it? dont want to merge my own PR.

Woohoo, merge commit.


Maven deploy must be skipped for all modules other than the add-on itself

Either the POMs in the project or the CI configuration must make sure that only the add-on module is deployed to a staging repository.

Fixed with build running deploy only for the add-on module


Support for API 10

Hey, your library should work on API 10 with no big issues. So why not to lower minSdkVersion?

Because it's 2015 :)


Move loader functions into seperate file

https://github.com/angry-goose-initiative/wiki/issues/45

LGTM

I think that the loaders are useful utilities that shouldn't depend on the memory class. The way that we could do this is by:

Removing the memory.h include
Making the loader functions standalone (not part of a class and in their own namespace)
Passing a callback function to them which is how they write bytes to whatever is calling them

I'm good to merge this for now and I can work on this in a follow up PR.

Agreed that there should likely be a separate loader.h for this.
Also perhaps some of the assertions about corrupted ELF fields could be replaced by exceptions that can be gracefully caught?
But this can be done later.


define setting items in const

Put setting modules in a const for more user-friendly editing.

Summary by CodeRabbit

New Features

Improved settings page structure by consolidating multiple settings modules into a dynamic array-driven approach.
Enhanced maintainability and scalability of the settings interface.

Bug Fixes

Ensured that the "Danger Zone" module retains its functionality for account deletion.

Refactor

Optimized code by reducing redundancy and improving readability through the use of an {#each} block for rendering settings modules.

Thanks but I'm going to pass on this change (see my comment on the other PR for more detailed reason).
The code before was more idiomatic svelte, and using a component removed any repetitive code. Adding a data structure isn't needed.


iPad 3 can pinch zoom past last zoom level

"if you move the map, the zoom level is restored to maximum zoom level" /via @nuky89

I believe this has been fixed in 1.0.0b2.


Allow to use self-hosted gitlab instances with own CA cert chain

Is there an existing issue for this?

[X] I have searched the existing issues

Description of the bug
Hi,
currently it's not possible to use git-cliff with a self-hosted GitLab instance that uses an own CA for example with self-signed certificates.
A possible solution would be to use the rustls-tls-native-roots feature instead of rustls-tls for reqwest or provide a similar CLI argument like uv in https://github.com/astral-sh/uv/pull/2362
Steps To Reproduce

Setup a GitLab instance with a self-signed CA
Set GITLAB_REPO, GITLAB_TOKEN and GITLAB_API_URL environment variables
Run git-cliff

Expected behavior
git-cliff can request information from a self-hosted GitLab instance using an own CA cert chain
Screenshots / Logs
 WARN  git_cliff_core::changelog > You are using an experimental feature! Please report bugs at <https://git-cliff.org/issues>
 ERROR git_cliff_core::changelog > Failed to lookup project! HTTP client with middleware error: `Middleware error: Request error: error sending request for url (https://xxx/api/v4/projects/xxx)`
thread 'main' panicked at git-cliff-core/src/changelog.rs:452:18:
Could not get gitlab metadata: HttpClientMiddlewareError(Middleware(Request error: error sending request for url (https://xxx/api/v4/projects/xxx)
Caused by:
    0: error sending request for url (https://xxx/api/v4/projects/xxx)
    1: client error (Connect)
    2: invalid peer certificate: UnknownIssuer

Software information

Operating system: Debian Linux
Project version: 2.5.0 installed from PyPI

Additional context
No response

Hey, thanks for reporting!
The possible solution you recommended looks solid, would you be down to implement it? 🐻 I can also try it out but it would be easier if you already have an environment to test it out.

Sorry missed your answer. Sadly I have no experience with ruff yet and therefore I don't think I can provide a good PR :-(


Take alpha antialising options into account when setting up materials

This restores the functionality of the alpha antialising options
Fixes: https://github.com/godotengine/godot/issues/64501
https://github.com/godotengine/godot/pull/58954 made it so that alpha scissor and alpha hash materials always go through the opaque pipeline. However, alpha antialising is more like depth prepass in that it can be used for depth pass and shadows, but still needs to be drawn with a transparent pipeline so that proper MSAA can be done.
Image comparison - cropped and scaled
Before:

After:

comparison slider for fun
As a bonus:
Alpha edge antialising is a very important tool to combat aliasing that occurs when using alpha scissor or alpha hashing. Alpha scissor allows you to take advantage of the depth prepass to get proper sorting of semi-transparent objects (like leaves or grass) within a mesh.
Plain alpha
Ordinary alpha looks nice and smooth, but suffers from a chaotic layering of the individual branches (branches that are near the bottom that should be hidden are drawn over top) and casts no shadows.

Alpha scissor allows us to properly order the branches and cast shadows but the edges are rough looking and the aliasing is even worse when the camera moves

Depth prepass alpha looks nice and smooth like alpha, and even casts shadows, but lower branches are still getting drawn over upper branches

Alpha scissor with Alpha blend retains the positive characteristics of alpha scissor, but smooths out the aliasing artifacts.

Comparison

Thanks!


Performance fix account GraphQL endpoint

Describe your changes

Strategy is to iterate and filter over internal commands until there are 'enough'
Iteration favours lower block heights first. This could be tweaked to favour higher blocks first, if desired

Link issue(s) fixed
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have verified new and existing tests pass locally with my changes.
[x] I verified whether it was necessary to increment the database version.

Possible fix for #1339


How exactly do I use 'jshintConfig' inside package.json

Aim: To use custom configuration for jshint, which is installed using npm.
OS: Windows-7 (I know, I am sorry -_-)
Current progress: I have already tried and succeeded in using --config command to load a custom config file.
Problem: Based on documentation here and many other places on the web we have to add a property jshintConfig in package.json file. I added this property as such:

Also, I have added my jshintrc file parallel to package.json.
My basic jshintrc file looks like this:
{
  "undef": true,
  "unused": true,
  "esversion": 6,
  "globals": {
    "MY_GLOBAL": true,
	"$": true
  }
}

When I use command jshint dummy.js, it does not use the config file.
Although, it does use it if I use the command jshint --config jshintrc dummy.js, while placing jshintrc file parallel to my dummy.js.
I understand I should be making .jshintrc file, but that is not allowed in windows afaik.
Edit: I have also tried adding json directly as value of jshintConfig property, but it was still not picked up.

@fmartin5 Updating jshintConfig property directly should have worked, but it didn't. Maybe I'm updating the wrong package.json
Creating .jshintrc using cmd is a good way I guess, but even if I get it working, I would have to create these files for different projects or add this file in file system root(so that jshint eventually finds it).

Yup, that must be it.
My project does not have a package.json and I was indeed modifying NPM package's JSON.
So, how should I use my custom configuration for JSHINT?
I was looking for a global approach rather than having multiple .jshintrc files.

You can still create a package.json file for your project (e.g. by running npm init in its directory) and put the jshintConfig therein, but it will be local to that project only.
Afaik, the only global approach is having a .jshintrc file somewhere in the file system above all your different projects; this is not necessarily the system root.
From what I've seen this way of using dotfiles for configuration is very common, it also applies to other linters.

Yeah. So, now have a .jshintrc file in system root. This works for me for me.
Thanks.


shopify-cli: 2.14.0 -> 2.32.0

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable: is a ruby gem, none of these seem to have tests?

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package built:

shopify-cli

Thank you!


Neighbors - Iterate over a number of items from the current while only advancing the interator once

Although this might not be the best name, here's the idea.
let mut v = Vec::new();
for (a, b) in (1..5).neighbors(2) {
    v.push((a, b));
}
assert_eq!(v, vec![(1, 2), (2, 3), (3, 4), (4, 5)]);

let mut v = Vec::new();
for (a, b, c) in (1..5).neighbors(3) {
    v.push((a, b, c));
}

assert_eq!(v, vec![(1, 2, 3), (2, 3, 4), (3, 4, 5)]);

As far as typing goes, I'm not entirely sure how to implement it for an arbitrary number of items.

Welp, I suppose I should have looked a bit harder at the docs. This is basically tuple_windows


feat: supporting expressions to query metrics data

Videos / Screenshots
https://github.com/apache/skywalking-booster-ui/assets/20871783/e41eda74-0ba4-410d-a489-573a9ef44f46

https://github.com/apache/skywalking-booster-ui/assets/20871783/be275d3f-7ecb-4026-88a2-03ede7232b57

Signed-off-by: Qiuxia Fan<EMAIL_ADDRESS>

Let's support summary exp and detail exp for table widget.
Permanently remove calculation features from the expression option.
We are deprecating this from the main stream since 9.5.0. Thanks.

Let's support summary exp and detail exp for table widget. Permanently remove calculation features from the expression option. We are deprecating this from the main stream since 9.5.0. Thanks.

Done

Could you make these two in one line to indicate these are for one column? Then put a title for two. The first should be required, the second is optional.
And, did you miss the place to set up the title of this column? How about a title setup and two expressions are in the same line? Or in one block?

Another question is, it looks like both General and expression are working at the same time, how to determine the order of  left to right?

Updated
https://github.com/apache/skywalking-booster-ui/assets/20871783/8025fb36-43db-419b-a8b8-ffa68325c3a0

Could you share how to set up column title? Is the calculation disabled?

Could you share how to set up column title? Is the calculation disabled?

Sure.

@wankai123 Do we still need subscript? I think we could format all through expression, right?

subscript

yes, we can calculate through expression.

@Fine0830 Please remove it as well.

Done


blog entry in docs ("can be problematic") / "A Better Pip Workflow" needs newlines in  blocks

I am filing this and considering it a bug, since this is an article linked to in the docs. As per subject, the <pre> blocks in the linked-to article (https://kennethreitz.org/essays/2016/02/25/a-better-pip-workflow) need to be cleaned up, so that the newlines are actually visible, vs all the text being squeezed together on one line.
This is a screenshot of what things look like right now

:

@kennethreitz Can fix this if he wants, but this is not a pipenv issue per se.

@kennethreitz Can fix this if he wants, but this is not a pipenv issue per se.

would you consider this a pipenv documentation issue?

got it, thanks. @kennethreitz if you could fix the formatting in the blog entry (the official docs, as well as the README.md in this repo link to it; "can be problematic" in both places) that would be great.


build: :rocket: Remove Autotools and MSVC build systems

The ultimate goal :)

Unit tests create an artifact in the source tree:
share/rpcauth/__pycache__/rpcauth.cpython-312.pyc

Unit tests create an artifact in the source tree:
share/rpcauth/__pycache__/rpcauth.cpython-312.pyc

Fixed.

Rebased. More stuff have been removed from the src/univalue .

ACK 045f267577abc0f2788311d34d1c31d4b6493ffb

Why restore src/minisketch/.gitignore and why drop commit 3e6cf60806 test: Do not write Python bytecode to source directory?

Why restore src/minisketch/.gitignore..?

The src/minisketch is a subtree and shouldn't be modified in this PR.

and why drop commit 3e6cf60 test: Do not write Python bytecode to source directory?

It is still not clear which way is preferred. For more details, please refer to https://github.com/bitcoin/bitcoin/pull/30533.

Rebased.

Looks like this is now bitcoin#30731 and bitcoin#30664, so reviewers of this change may want to go there.

Closing this PR.


error: package app.xplatform.capacitor.plugins does not exist

Hey there,
Getting the above error while building using android studio. Seems to be a naming issue.
Thanks

did you import app.xplatform.capacitor.plugins package in your app MainActivity.java file.
if possible please provide more details.

There were some library issues in gradle which caused this. No longer getting the error.

im not sure how the import statement works in java. can someone point me to the right direction. Im getting the same error

Im also getting the same error.

I am also facing this problem...

Can you please share more details. such as package.json your capacitor version and node and your OS

package.json
{
  "name": "---",
  "version": "0.0.1",
  "author": "---",
  "homepage": "---",
  "scripts": {
    "ng": "ng",
    "start": "ng serve",
    "build": "ng build",
    "test": "ng test",
    "lint": "ng lint",
    "e2e": "ng e2e"
  },
  "private": true,
  "dependencies": {
    "@angular/animations": "^8.2.1",
    "@angular/common": "^7.2.14",
    "@angular/core": "^7.2.14",
    "@angular/fire": "^5.1.3",
    "@angular/forms": "^7.2.14",
    "@angular/http": "^7.2.14",
    "@angular/platform-browser": "^7.2.14",
    "@angular/platform-browser-dynamic": "^7.2.14",
    "@angular/router": "^7.2.14",
    "@capacitor/android": "^1.1.1",
    "@capacitor/cli": "^1.1.1",
    "@capacitor/core": "^1.1.1",
    "@capacitor/ios": "^1.1.1",
    "@fortawesome/angular-fontawesome": "^0.3.0",
    "@fortawesome/fontawesome-svg-core": "^1.2.21",
    "@fortawesome/free-regular-svg-icons": "^5.10.1",
    "@fortawesome/free-solid-svg-icons": "^5.10.1",
    "@ionic-native/core": "^5.5.0",
    "@ionic-native/email-composer": "^5.5.1",
    "@ionic-native/facebook": "^5.5.0",
    "@ionic-native/google-plus": "^5.5.1",
    "@ionic-native/network": "^5.5.0",
    "@ionic-native/secure-storage": "^5.5.0",
    "@ionic/angular": "^4.7.1",
    "@ionic/storage": "^2.2.0",
    "@ngx-translate/core": "^11.0.1",
    "@ngx-translate/http-loader": "^4.0.0",
    "angular2-swing": "^0.14.0",
    "capacitor-admob": "0.0.9",
    "cordova-ios": "^5.0.1",
    "cordova-plugin-device": "^2.0.2",
    "cordova-plugin-email-composer": "0.9.2",
    "cordova-plugin-googleplus": "^8.0.0",
    "cordova-plugin-ionic-webview": "^3.1.2",
    "cordova-plugin-network-information": "2.0.1",
    "cordova-plugin-whitelist": "^1.3.3",
    "cordova-sqlite-storage": "3.0.0",
    "cordova-support-android-plugin": "1.0.1",
    "cordova-support-google-services": "1.3.1",
    "core-js": "^2.5.4",
    "firebase": "^5.11.1",
    "fsevents": "^2.0.7",
    "hammerjs": "^2.0.8",
    "ngx-image-cropper": "^1.3.9",
    "rxjs": "~6.3.3",
    "zone.js": "~0.8.29"
  },
  "devDependencies": {
    "@angular-devkit/architect": "~0.12.3",
    "@angular-devkit/build-angular": "^0.13.8",
    "@angular-devkit/core": "~7.2.3",
    "@angular-devkit/schematics": "~7.2.3",
    "@angular/cli": "~7.2.3",
    "@angular/compiler": "^7.2.14",
    "@angular/compiler-cli": "^7.2.14",
    "@angular/language-service": "^7.2.14",
    "@ionic/angular-toolkit": "^1.4.1",
    "@ionic/lab": "^1.0.24",
    "@types/jasmine": "~2.8.8",
    "@types/jasminewd2": "~2.0.3",
    "@types/node": "^10.12.30",
    "codelyzer": "~4.5.0",
    "jasmine-core": "~2.99.1",
    "jasmine-spec-reporter": "~4.2.1",
    "karma": "^4.1.0",
    "karma-chrome-launcher": "~2.2.0",
    "karma-coverage-istanbul-reporter": "^2.0.5",
    "karma-jasmine": "~1.1.2",
    "karma-jasmine-html-reporter": "^0.2.2",
    "node-sass": "^4.12.0",
    "protractor": "~5.4.0",
    "ts-node": "^8.0.3",
    "tslint": "~5.12.0",
    "typescript": "~3.1.6",
    "xcode": "^2.0.0"
  },
  "description": "Like My Ride",
  "cordova": {
    "plugins": {
      "cordova-sqlite-storage": {},
      "cordova-plugin-whitelist": {},
      "cordova-plugin-device": {},
      "cordova-plugin-ionic-webview": {
        "ANDROID_SUPPORT_ANNOTATIONS_VERSION": "27.+"
      },
      "cordova-plugin-network-information": {},
      "cordova-plugin-email-composer": {
        "ANDROID_SUPPORT_V4_VERSION": "27.+"
      },
      "cordova-plugin-googleplus": {
        "PLAY_SERVICES_VERSION": "11.8.0"
      }
    },
    "platforms": [
      "ios"
    ]
  }
}

Capacitor Version: 1.1.1
OS: MacOS Mojave
Node: v12.6.0

Did you check admob-demo app.? is it work.? I am going to check what wrong. it will be great if you check the demo  app and let me know.

I just tested the demo app. It is running perfectly fine... hm :(

Please use this command for your app maybe it will fixed the problem! or ignore the IDE error and try to build you app i am not sure but maybe it will work. 🙂
ionic build & npx cap copy & npx cap sync & npx cap update & npx cap open android

I already did that command :( - btw. i thought npx cap sync combines update and copy?
Funny. I built it before i posted this error and it did not work. Now i tried to build again and it worked. Now it works.. haha i hate that mobile developing game :D
Thank you for really fast help! 🥇

i thought npx cap sync combines update and copy?
Yes you are right. but i did not have any problem using all of this command.  When i develop a plugins using all of this command individually work great for me.
Thank you for your report. 🙂

I have got another question, i absolutely need iOS integration.. When is iOS ready?

I am also excited to implement this plugins for iOS platform, but unfortunately currently i did not have a MAC machine, but i am trying to publish iOS integration within this month.


port,ip

how to set port ip and change the listening port?

you can set in source code or use command line
---Original---
From: @.>
Date: Mon, Sep 27, 2021 20:07 PM
To: @.>;
Cc: @.***>;
Subject: [qwqdanchun/HVNC] port,ip (#1)
fow to set port ip and change the listening port?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

thanks for your fast response .But where in the command line?

for example
client.exe <IP_ADDRESS>
you can't change the port in command now
---Original---
From: @.>
Date: Mon, Sep 27, 2021 20:20 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [qwqdanchun/HVNC] port,ip (#1)
thanks for your fast response .But where in the command line?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

thanks! But how to change the ports?

in source code
---Original---
From: @.>
Date: Mon, Sep 27, 2021 20:29 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [qwqdanchun/HVNC] port,ip (#1)
thanks! But how to change the ports?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

in source code
…
---Original--- From: @.> Date: Mon, Sep 27, 2021 20:29 PM To: @.>; Cc<EMAIL_ADDRESS>Subject: Re: [qwqdanchun/HVNC] port,ip (#1) thanks! But how to change the ports? — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

yeah but where?

Oh i found the client port but i still cant find the listening

oh find the other one.Thanks for your time!

https://github.com/qwqdanchun/HVNC/blob/1a15ec46bd6a5521b3adddb13937ed8251d3ad0a/Client/hiddenDesktop.cpp#L38
------------------ 原始邮件 ------------------
发件人:                                                                                "qwqdanchun/HVNC"                                                                                @.>;
发送时间: 2021年9月27日(星期一) 晚上8:43
@.>;
<EMAIL_ADDRESS>主题: Re: [qwqdanchun/HVNC] port,ip (#1)
in source code
…
---Original--- From: @.> Date: Mon, Sep 27, 2021 20:29 PM To: @.>; Cc<EMAIL_ADDRESS>Subject: Re: [qwqdanchun/HVNC] port,ip (#1) thanks! But how to change the ports? — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.
yeah but where?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


Bitmap not defined in System.Drawing

Tried to follow your guide, but using MatchTemplate() results in a build exception:
Error	CS7069	Reference to type 'Bitmap' claims it is defined in 'System.Drawing', but it could not be found
I'm using Visual Studio 2019 on Windows 10

Had the same issue - not sure what the best solution is, but I got it working by setting the target framework to .NET 4.7.2.

Had the same issue - not sure what the best solution is, but I got it working by setting the target framework to .NET 4.7.2.


Parallel build of netcdf-fortran-4.4.0 fails with netcdf-c-4.5.0-rc1

Environment Information
64-bit linux with mpich 3.2 and gnu tool chain, using autotools build.
Summary of Issue
Building fortran with parallel fails with netcdf-c-4.5.0-rc1.
make[5]: Entering directory '/home/ed/Downloads/netcdf-fortran-4.4.4/examples/F90'
PASS: simple_xy_wr
PASS: simple_xy_rd
PASS: sfc_pres_temp_wr
PASS: sfc_pres_temp_rd
PASS: pres_temp_4D_wr
PASS: pres_temp_4D_rd
FAIL: run_f90_par_examples.sh
PASS: simple_xy_nc4_wr
PASS: simple_xy_nc4_rd
Steps to reproduce the behavior
CC=mpicc LD_LIBRARY_PATH=/usr/local/netcdf-c-4.5.0-rc1_mpich-3.2/lib FC=mpifort CPPFLAGS=-I/usr/local/netcdf-c-4.5.0-rc1_mpich-3.2/include LDFLAGS=-L/usr/local/netcdf-c-4.5.0-rc1_mpich-3.2/lib ./configure --enable-parallel-tests --prefix=/usr/local/netcdf-fortran-4.4.4_mpich-3.2
make check

Thanks Ed.  Our parallel tests on our end haven't reported an error, so I'm taking a closer look at this!

I just checked this with netcdf-c 4.5.0 and netcdf-fortran-4.4.4 and unfortunately got the same issue.
When I run the test on its own I get this output:
ed@mikado:~/Downloads/netcdf-fortran-4.4.4/examples/F90$ bash run_f90_par_examples.sh
 *** SUCCESS writing example file simple_xy_par.nc!
 *** SUCCESS reading example file simple_xy_par.nc!
 NetCDF: Attempt to extend dataset during NC_INDEPENDENT I/O operation. Use nc_va
 NetCDF: Attempt to extend dataset during NC_INDEPENDENT I/O operation. Use nc_va
 NetCDF: Attempt to extend dataset during NC_INDEPENDENT I/O operation.  Back in the office, the PR aggregation should go up shortly.


added background color setters to DrawerFrameLayout

Sent PR as you requested ;)

Make sure you don't change the code style too much as this makes it difficult to me to see the real changes

Cmon, it's just couple spaces ;)

I'll merge it by hand, when I'm at home later ;)

I thought it would be better to have the whole drawer themable so I'm currently developing some DrawerTheme class and a theme engine.

The theme engine will allow you to set themes for your drawer and each item/profile. And you can change it in runtime

Rather than merging many small code changes for theming I made a theming engine that is more flexible.


make tlog tests work without need for iyo env variables

Currently one cannot run any tests which create a 0-stor client, without defining valid values for the three iyo variables: iyo_organization, iyo_client_id and iyo_secret.
0-Stor seems to use a PEM certificate and public key for this, as can be seen in:

https://github.com/zero-os/0-stor/blob/master/client/client_test.go#L27
https://github.com/zero-os/0-stor/blob/master/client/client_test.go#L68

That could be a solution. Either way, we need something so travis and developers can run tests again without requiring to set environment variables.

@zaibon were you able to take a look at this?

Not yet


Support for LMStudio

Feature Request
https://lmstudio.ai/
Could LM Studio support be added. LM Studio is much more versatile than ollama and allows you to download and run GGUF models directly from Hugging Face
Motivation
Ollama simply doesn't work with my Nvidia GPU. It utilizes the CPU only so it's extremely slow.
Your Contribution
Perhaps we could look at how AnythingLLM integrates with LM Studio

Relevant messages I found from LM Studio's discord

Hey @EDLLT
How are you?
If I understand correctly, all you have to do is update the base url in the OpenAI component. I set it to http://localhost:1234/v1 and it worked first try.

Hey @EDLLT
How are you?
If I understand correctly, all you have to do is update the base url in the OpenAI component. I set it to http://localhost:1234/v1 and it worked first try.

Oh. Apologies, I had tried setting it before but not setting the API key was the causing it to not work.
Thanks for your help!

Hey @EDLLT
How are you?
If I understand correctly, all you have to do is update the base url in the OpenAI component. I set it to http://localhost:1234/v1 and it worked first try.

Hey, so I managed to get the LM Studio's LLMs working but what about embedding models?
Unlike the OpenAI LLM component, the OpenAI embeddings don't allow me to change the base url

@duso
Hello, are you there

@dosu
Hello, are you there?

@dosu
Why am I getting a CI error?
CI error doesn't make sense to me
Error: src/backend/base/langflow/components/models/LMStudioModel.py:57:14: RUF005 Consider iterable unpacking instead of concatenation
Error: Process completed with exit code 1.

Line it's complaining about
57    inputs = LCModelComponent._base_inputs + [
58        IntInput(
59            name="max_tokens",
60            display_name="Max Tokens",
61            advanced=True,
62            info="The maximum number of tokens to generate. Set to 0 for unlimited tokens.",
63            range_spec=RangeSpec(min=0, max=128000),
64        ),
...

Every other component I'd seen(OpenAIModel.py, OllamaModel.py) are structured like that but they don't get errors..?

@dosu
src/backend/base/langflow/components/models/LMStudioModel.py
Comment on lines +139 to +140
if output_schema_dict:
output = output.with_structured_output(schema=output_schema_dict, method="json_mode")  # type: ignore
Contributor
@ogabrielluiz ogabrielluiz 2 hours ago
I'm using it today (loving it, btw) and it seems it has to be json_schema. We need to update this (and the OpenAI component) to make this work better with BaseModel
What did ogabrielluiz mean by that comment? What should be changed?


Use messense/rust-musl-cross as custom images in cross? [feature request]

Hi,
I was wondering about difference between cross and messense/rust-musl-cross images.
For sure there is big one - cross doesn't support openssl. Are there any other?
I am asking because maybe it would be good idea to make  messense/rust-musl-cross images work with cross.  See custom docker images in cross.

Is it not working now? Could you paste the error message here?

➜ cross build --target armv7-unknown-linux-musleabihf
error: could not create home directory: '/.rustup'
error: caused by: Permission denied (os error 13)

➜ cat Cross.toml
[target.armv7-unknown-linux-musleabihf]
image = "messense/rust-musl-cross:armv7-musleabihf"

I think we need to find out what is trying to create the home directory /.rustup.
This issue might be somewhat related: https://github.com/rust-embedded/cross/issues/399


Question: What is the proper way to construct a dynamic query for EF Core using DateTime or Nullable DateTime?

With EF 6 and Dynamic Linq I could just do the following predicate:
"PooledDate >= new DateTime(2019,1,1,0,0,0)"
Now with EF Core 2.2 this does not work. It results with SqlException "Conversion failed when converting date and/or time from character string."
Here is the expression that is generated:
(Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryable`1[TracSeq.Data.Models.PoolData]).Where(Param_0 => (Param_0.PooledDate >= Convert(1/1/2019 12:00:00 AM)))
I've tried string literals, and get the following:
(Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryable`1[TracSeq.Data.Models.PoolData]).Where(Param_0 => (Param_0.PooledDate >= 1/1/2019 12:00:00 AM))
Any help that could point me in the right direction would be greatly appreciated.

closing


(Question / FeatureRequest) HTTP - unable to create assertion in session exec

Hi,
I'm unable to make assertion in exec block of code (Scala test) that fails the test if it's not passed.
My Gatling version I'm using is '3.7.3'
I'm doing request to http separately like:
 val request = exec {
    http("GET SOMETHING")
      .get(<some endpoint>)
      .header(<some header>)
      .check(status.exists, status.saveAs("get.statusCode"))
}

And then I'm using it in definition of function like:
def someFunction = {
     exec(
        request.during(<some time>) {
           exec(
             session => {
              allRequests += 1
              if (session("get.statusCode").as[Int] == 200) {
                successes += 1
              }
              session
            }
          )
})

I'd like to add some assertion in this someFunction to check, if percentage of successful requests (statusCode == 200) is not less than 10.
I've tried adding something like:
        assert((successes*100)/allRequests >= 10))

But it does not fail the test scenario, when it's below that 10%.
I've tried using exitHereIfFailed or any of those types from Gatling documentation (but with no luck).
Moreover, I also added .assertions(forAll.failedRequests.count.is(0)) in setUp section. This also did not work (test is still ending with Build Successful).
I'm aware, there is also possibility to use an assertion for global requests (assertions(global.successfulRequests.percent.gt(10))), but I need my custom one.
Also I do not want to use any check, because counters (allRequests and successes) must be in request.during(<some time>) block.
Is there any option to use my custom counters and check percentage of successful request as in my example?

We want assertions to be:

serializable over the wire (hence use an AST, not arbitrary functions)
computed against Gatling metrics (not arbitrary ones you would compute manually)

In short, you would like Gatling to have custom metrics (counters, but others might want durations, often called timers) and that they support assertions.
That's something we're considering, but it will probably only make its way in Gatling Enterprise.

Okey, I see. Thanks for so fast response.
In this situation, is there any other way to check if e.g. at least 10% requests return proper value?

Can't you explicitly check for status 200, instead of using the default one?

I was thinking about it. But the counters are not working properly in this http request exec. So that, I assumed that I need to use them in request.during(<some time>) block. Am I doing sth wrong?


missing characters

hi. i used your program and understood that in fast speed typing two problem occurs. first some characters missed and second
sometimes uart1 give synchronization error.

Can you test this temp firmware? send me feedback, (still beta, but better)
20241231084058.zip
Thx @1250364116

Thanks. The problem was not for code. It was a parsing problem not related
to this code.
در تاریخ سه‌شنبه ۳۱ دسامبر ۲۰۲۴، ۱۱:۱۳ Dreg @.***>
نوشت:

Can you test this temp firmware? send me feedback, (still beta, but better)
20241231084058.zip
https://github.com/user-attachments/files/18280038/20241231084058.zip
Thx @1250364116 https://github.com/1250364116
—
Reply to this email directly, view it on GitHub
https://github.com/therealdreg/okhi/issues/1#issuecomment-2566210036,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2VUWY7BG2EWNGWUXDVTP632IJDJ3AVCNFSM6AAAAABQBNZFXOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRWGIYTAMBTGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


runiq: 2.0.0 -> 2.0.0-unstable-2024-08-19

Description of changes
Build from git to allow compiling with rust 1.80, until there is an upstream release containing a642926f6ec09d4faeebebb563d4aed89e0d36fb.
Things done
Switched source to the upstream git repo and vendored the lock file (since there is no Cargo.lock in that repo).

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

cross referencing https://github.com/NixOS/nixpkgs/issues/332957


Pod oci-onm-mgmt-agent-0 can't start

Hello.
Could tell me why oci-onm-mgmt-agent-0 pod cannot start without oci-onm-mgmt-agent-container-registry-key secret please?
I have installed chart like this
helm install oci-k8s-mon charts/oci-onm --values override_values.yaml
But I see a error in a pod:
Unable to retrieve some image pull secrets (oci-onm-mgmt-agent-container-registry-key); attempting to pull the image may not succeed
I didn't see any requirements in README.md about building mgmt-agent image or enabling a secret value during chart installation.
Could you tell me please how can I  fix it?
Thank you in advance.

hi @art102 , Though this warning appears, the oci-onm-mgmt-agent-0 pod would start successfully. Could you cross check the pod status please ?

HI.
Unfortunately this pod doesn't start...
Also I would like to know what the right way to install OKE monitoring solution is?
For instance I have pulled this repo and do this
cd oke_oci_monitoring/oci-kubernetes-monitoring
helm install oci-onm-k8s --values ../override_values.yaml charts/oci-onm
and I get this error
Error: INSTALLATION FAILED: An error occurred while checking for chart dependencies. You may need to run helm dependency build to fetch missing dependencies: found in Chart.yaml, but missing in charts/ directory: oci-onm-common, oci-onm-logan, oci-onm-mgmt-agent
Can you  change the README to have the right manual to use this solution please?

@prasebha @niravg75 , could you help @art102 here w.r.t Management agent pod related issues ?
As mentioned in the README, please download the chart from releases page instead of using the source code, so that you do not need to run dependency update. Otherwise you need to run the helm dependency update in each of the folders before running the helm install.


domainlab from PyPI under Python 3.11

The newest version of domainlab cannot be installed with pip install domainlab or pip install domainlab==0.1.9 under Python 3.11 due to the following dependency issue:
ERROR: Could not find a version that satisfies the requirement torchvision<0.14.0,>=0.13.0 (from domainlab) (from versions: 0.15.1, 0.15.2, 0.16.0, 0.16.1)
ERROR: No matching distribution found for torchvision<0.14.0,>=0.13.0

It can still be installed when the --no-deps option is used with pip, and torch and torchvision can be installed separately in that case (i.e., pip install --no-deps domainlab==0.1.9 and then pip install torch torchvision). However, the versions of torch and torchvision installed in that way under Python 3.11 (specifically, '2.1.1+cu121' and '0.16.1+cu121') are not compatible with some of the existing domainlab code:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[8], line 1
----> 1 exp.execute(num_epochs=2)

File /storage2T/software/.pyenv/versions/3.11.0/envs/domainlab-pip/lib/python3.11/site-packages/domainlab/exp/exp_main.py:71, in Exp.execute(self, num_epochs)
     69 for epoch in range(1, num_epochs):
     70     t_before_epoch = t_c
---> 71     flag_stop = self.trainer.tr_epoch(epoch)
     72     t_c = datetime.datetime.now()
     73     logger.info(f"after epoch: {epoch},"
     74                 f"now: {str(t_c)},"
     75                 f"epoch time: {t_c - t_before_epoch},"
     76                 f"used: {t_c - t_0},"
     77                 f"model: {self.visitor.model_name}")

File /storage2T/software/.pyenv/versions/3.11.0/envs/domainlab-pip/lib/python3.11/site-packages/domainlab/algos/trainers/train_mldg.py:64, in TrainerMLDG.tr_epoch(self, epoch)
     62 self.inner_trainer.model.load_state_dict(self.model.state_dict())
     63 # update inner_model
---> 64 self.inner_trainer.before_epoch()  # set model to train mode
     65 self.inner_trainer.reset()  # force optimizer to re-initialize
     66 self.inner_trainer.tr_batch(
     67     tensor_x_s, vec_y_s, vec_d_s, others_s, ind_batch, epoch)

File /storage2T/software/.pyenv/versions/3.11.0/envs/domainlab-pip/lib/python3.11/site-packages/domainlab/algos/trainers/train_basic.py:31, in TrainerBasic.before_epoch(self)
     26 def before_epoch(self):
     27     """
     28     set model to train mode
     29     initialize some member variables
     30     """
---> 31     self.model.train()
     32     self.counter_batch = 0.0
     33     self.epo_loss_tr = 0

File /storage2T/software/.pyenv/versions/3.11.0/envs/domainlab-pip/lib/python3.11/site-packages/torch/nn/modules/module.py:2398, in Module.train(self, mode)
   2396 self.training = mode
   2397 for module in self.children():
-> 2398     module.train(mode)
   2399 return self

File /storage2T/software/.pyenv/versions/3.11.0/envs/domainlab-pip/lib/python3.11/site-packages/torch/nn/modules/module.py:2398, in Module.train(self, mode)
   2396 self.training = mode
   2397 for module in self.children():
-> 2398     module.train(mode)
   2399 return self

File /storage2T/software/.pyenv/versions/3.11.0/envs/domainlab-pip/lib/python3.11/site-packages/torchvision/models/feature_extraction.py:324, in DualGraphModule.train(self, mode)
    321 # NOTE: Only set self.graph if the current graph is not the desired
    322 # one. This saves us from recompiling the graph where not necessary.
    323 if mode and not self.training:
--> 324     self.graph = self.train_graph
    325 elif not mode and self.training:
    326     self.graph = self.eval_graph

File /storage2T/software/.pyenv/versions/3.11.0/envs/domainlab-pip/lib/python3.11/site-packages/torch/nn/modules/module.py:1695, in Module.__getattr__(self, name)
   1693     if name in modules:
   1694         return modules[name]
-> 1695 raise AttributeError(f"'{type(self).__name__}' object has no attribute '{name}'")

AttributeError: 'GraphModule' object has no attribute 'train_graph'

Not an urgent issue since it's easy to just create a virtual environment under Python 3.10, and then pip install domainlab works just fine.

@agisga ,   we have to increae the python verion i guess, see here
https://github.com/snakemake/snakemake/issues/2607#issuecomment-1890979155
https://github.com/marrlab/DomainLab/pull/741
I have tested before on my cluster, newer version of torch works well with domainlab

https://github.com/marrlab/DomainLab/pull/743
@agisga


[Bug]: POST method is used instead of PATCH for previewing subscription change

What happened?
Call to PreviewSubscription() failed with an error Decoding issue in call to POST /subscriptions/<redacted>/preview (HTTP status 405): found nil paddle.Error when one was expected
Steps to reproduce
There're no special steps to reproduce other than calling this method.
This documentation says that method should be PATCH while source code does POST.
What did you expect to happen?
API call succeeds or at least Paddle own parsing would not fail with that weird error.
Logs
No response

Hey @ribtoks, thank you for raising this issue, I can confirm we consider this as a high priority and will be looking to get a fix for this as soon as possible.

Hey @mikeymike How is that high priority work going?

Hi @ribtoks I wanted to give you an update on this before the weekend as some blockers in the code generation prevented a release today. I'll update here again on Monday with progress details on this, apologies for the delay!

Hey @ribtoks this is now released in v0.4.0 👍


Received error - failed to load and summarize the video[Summarize YouTube Videos with AI] ...

Extension
https://www.raycast.com/iKasch/summarize-youtube-video-with-ai
Description
Error:
Error: Error response received. HTTP Status: 429 You have been rate limited. Please slow down!

Steps To Reproduce

Open Raycast
Type "Summarize..." to run extension
Enter URL: https://www.youtube.com/watch?v=RcYjXbSJBN8&t=26s
3b. tried without the time stamp: https://www.youtube.com/watch?v=RcYjXbSJBN8

All failed to provide a summary.
This is a 2 hour interview, so perhaps it's too long? I wanted to compare it with a summary from Eightify, to see how/if this extension was comparable. This extension returned errors all 3 attempts.
Current Behaviour
Does not provide summary of YouTube video
Expected Behaviour
Should provide summary of YouTube video

Same error here. 12min. video: https://www.youtube.com/watch?v=Qz-TEbUj9sE
Commenting to follow any updates.

This works again with the recent fix #12108
@raycastbot close this issue


Shared data schema for backend and frontend

We should define the data types shared between the backend and frontend in a shared schema file so that changes to the underlying data model are propogated to the other codebases.
Some options:

JSON schema

We already use JSON for the APIs, so would be easier to integrate
Use generators for TypeScript types

Protobuf

Binary data format, so slightly more efficient (but maybe not necessary)
Can generate type definitions and mode classes for TypeScript and JavaScript

Adding graphql to the mix.


Cannot add missing ORCID IDs to Publication Details

When I enter a DOI, it pulls in metadata including authors. Some fields that are not populated can still be edited on the Publication Details page. This does not include the ORCID ID for the authors populated. It would be nice if I was able to put in people's ORCID IDs if CrossRef doesn't supply them. This would give us the opportunity to enrich the metadata from CrossRef with more connections.

we COULD capture it, but we wont be able to use it for anything.
not enough time for features that would utilize this data in this launch.

confirmed this is still an issue 1/2/2019


Update BoardConfig.h "spec" to do button mapping by pin

The GPIO mapping refactor means that protobuf can assign any pin to any button, but the refactor did not update BoardConfig.h because it was too much change all at once. Now that the refactor is settling in 0.7.6, we think we should tackle the BoardConfig.h and change the button/extra defines to instead be pin defines, so that any combination of one to many buttons can be expressed in the base configs.
The thought right now is just to simply have a define for each of PIN_00 through PIN_29 in some fashion, probably assigning them directly to GpioActions?

This has been completed and is in main.


Expose Linux specifics and libnuma interop

Expose Linux-specific API functions and libnuma interoperability behind suitable cfg flags with associated doc(cfg)/cfg(doc):

[ ] Linux: https://hwloc.readthedocs.io/en/v2.9/group__hwlocality__linux.html
[ ] libnuma:

ulong: https://hwloc.readthedocs.io/en/v2.9/group__hwlocality__linux__libnuma__ulongs.html
bitmask: https://hwloc.readthedocs.io/en/v2.9/group__hwlocality__linux__libnuma__bitmask.html

libnuma support should also be gated behind a suitable feature, and will depend on the availability of a minimally working binding.

The Linux part of this should be done soon (hopefully next week), but the libnuma part requires libnuma bindings whose masks expose *const c_ulong interop. The current Rust libnuma bindings don't do this and seem unmaintained, so before I tackle this issue, someone else must take over the maintenance of Rust libnuma bindings and expose this functionality.


MINOR: Update outdate documentation for the SubscriptionState

The current documentation indicates two positions are tracked, but these positions were removed a few years ago. Now we use a single position to track the last consumed record. Updated the documentation to reflect to the current state.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks!


client and server don't have same version

I do not find my containers when one engine is added. And I got output
$ curl -X GET http://host1:4243/v1.15/version
{"ApiVersion":"1.16","Arch":"amd64","GitCommit":"5bc2ff8","GoVersion":"go1.3.3","KernelVersion":"3.13.0-32-generic","Os":"linux","Version":"1.4.1"}

$ curl -X GET http://host2:4243/v1.13/version
{"ApiVersion":"1.13","Arch":"amd64","GitCommit":"d84a070/1.1.2","GoVersion":"go1.2.2","KernelVersion":"3.10.58","Os":"linux","Version":"1.1.2"}

After looking into docker daemon logs, I found shipyard send request to host1 with the wrong URL (v1.13 shoud be right bellow). I think the Docker API shoud work for both v1.13 and v1.15
$  curl -X GET http://host2:4243/v1.15/version
client and server don't have same version (client : 1.15, server: 1.13)

Additionally, I saw too much log output in my docker daemon:
INFO[80466] GET /v1.15/version
INFO[80466] +job version()
INFO[80466] -job version() = OK (0)
INFO[80472] GET /_ping
INFO[80472] GET /v1.15/version
INFO[80472] +job version()
INFO[80472] -job version() = OK (0)
INFO[80474] GET /_ping
INFO[80474] GET /v1.15/version
INFO[80474] +job version()
INFO[80474] -job version() = OK (0)
INFO[80476] GET /_ping
INFO[80476] GET /v1.15/version
INFO[80476] +job version()
INFO[80476] -job version() = OK (0)

Yes Shipyard expects the latest stable release.  There is an issue open to
allow older versions however some things will not be compatible.
On Monday, January 5, 2015, Sean Chow<EMAIL_ADDRESS>wrote:

Additionally, I saw too much log output in my docker daemon:
INFO[80466] GET /v1.15/version
INFO[80466] +job version()
INFO[80466] -job version() = OK (0)
INFO[80472] GET /_ping
INFO[80472] GET /v1.15/version
INFO[80472] +job version()
INFO[80472] -job version() = OK (0)
INFO[80474] GET /_ping
INFO[80474] GET /v1.15/version
INFO[80474] +job version()
INFO[80474] -job version() = OK (0)
INFO[80476] GET /_ping
INFO[80476] GET /v1.15/version
INFO[80476] +job version()
INFO[80476] -job version() = OK (0)
—
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/394#issuecomment-68673826.

Shipyard v3 has been released and is a major refactor. Could you please check to see if this is still an issue? If so, please re-open. Thanks!


Test Extension Cross-Functionality

The extension should be able to handle any chrome based browser. Check that it does

Firefox and Chrome have been checked.


change progressSteps color?

Hi!
Is there any option to change the progressSteps color property? I've been looking at the methos at github, and I can not find one.
I'm using a color scheme for a site and it will be a lovely addition.
Thanks.

Theming of different parts will be fixed properly in the next major. For now, you can use something like this: https://jsfiddle.net/ad3quksn/129/

Thanks!


[Uptime Incentives]: Add authorized uptime validation for new NoLock gauges

Closes: #7367
What is the purpose of the change
This PR adds the relevant validation logic for authorized uptimes on CL gauge durations. Given how this logic touches existing state and a heavily depended on component of our codebase, the changes have heavy comments explaining the reasoning and implications.
This PR also makes a change to how CL gauges are linked to pool IDs in state. It is important to note that this breaks the previous setup where there was a "general linking" and then an additional link for internal gauges, reducing this to a single link based on gauge duration. The implications and next steps to ensure the safety of this are outlined in great detail in #7375.
Testing and Verifying
Tests covering the validation logic are added to gauge_test.go. The key changes are covered by existing tests, which were how the issue was discovered in the first place.
Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[x] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[x] Code comments?
[ ] N/A

I just had a question about this statement. Why should we fall back in this case? Shouldn't we be able to control this so that distribute never comes across an invalid or unauthorized duration?

#7369 has a breakdown of the reasoning. Tldr is this 1. minimizes the surface for existing gauges to unintentionally trigger chain halts/block incentives, and 2. is a clever way for us to ensure uptimes can be "unauthorized" in the future.
There are probably marginal gains we can make on these fronts with a meaningfully more complex implementation, but this approach seems quite elegant (and secure) to me given the current tradeoff space. As always, feel free to push back!


Angular2 support planned?

Hi, this has been a very useful package for me so far and I'm currently experimenting with an Angular2 app. Any plans to support Angular2?
Unfortunately I don't trust my JavaScript skills enough to do that, but if you'd be interested in it, I could try implementing the support, while rewriting it to typescript.

+1

I haven't looked into Angular 2 very much yet. I know there are ways to use Angular 2 code in an Angular 1 app (ng-upgrade, ng-forward), but don't want people to have to include an extra lib to use Angulartics on v1.
Perhaps this should be done in a branch for now? Thoughts @angulartics/core

+1

@timelf123 I believe it is best to do a all new repository for say "angular2-angulatics" as the current code base is very tightly connected to angular1. For angular2 most of the code can be redone in plain javascript classes. I am happy to help setting up the project and doing the core and GA implementations. Don't have the time for all other integrations though.
As for angular1 with the new component router, this could be done on a branch for now. It is difficult to know if this is even needed as it depends if users will go straight to angular2 or will try angular1 with new router... Probably best to invest on consulting the community to better understand if there any need for this feature.

quick example if you want to do some testing to track route changes on angular2.
import {Injectable} from 'angular2/core';
import {Router, RouterLink} from 'angular2/router';

@Injectable()
export class StatusListener {
	constructor(
		router: Router,
		status: status
	) {
		router.subscribe((route) => {
			console.log(route);
			console.log(router.recognize(route));
		});
	}
}

Awesome work @JonnyBGod!
UI-router 1.0 alpha came out recently and their goal for the next release is to support both Angular 1 and 2. Is it possible to do this in the same repo? I have zero Angular 2 experience.
There is a repo created https://github.com/angulartics/angulartics2 - feel free to contribute there

Will do. Need some time to study the current implementation better and understand all needs.
As for hybrid version, it might be possible. The routers are very similar in functionality and from what I saw probably about 70% of code is similar. This might be possible with a simple wrapper for angular1 version but needs further study.
I would focus on angular2 for now. The benefits from moving to angular2 are too good to just stay in angular1 and save a few weeks of work. (that my opinion) So we should provide an easy way to do so by supporting angular2 as soon as possible.
Do you have any idea of what are the most used integrations?

A hybrid version would be amazing but I agree that it doesn't hurt to get out a purely Angular 2 version in the mean time.
I would guess google analytics, mixpanel, kissmetrics, and segment are the most used plugins

I just release an initial implementation of angulartics for angular2 https://github.com/angulartics/angulartics2
Lets bring al angular2 discussion to the new repo.

:+1:


addRecordBackListener not working when passing params in startRecorder function

The addRecorderBackListener stops working when we pass params in startRecorder function
const onStartRecord = async () => {
const path = 'hello.m4a'; // Set the path for the audio file
const audioSet = {
AudioEncoderAndroid: AudioEncoderAndroidType.AAC,
AudioSourceAndroid: AudioSourceAndroidType.MIC,
AVEncoderAudioQualityKeyIOS: AVEncoderAudioQualityIOSType.high,
AVNumberOfChannelsKeyIOS: 2,
AVFormatIDKeyIOS: AVEncodingOption.aac,
};
const meteringEnabled = true;

const result = await audioRecorderPlayer.startRecorder(
  path,
  audioSet,
  meteringEnabled,
);
audioRecorderPlayer.addRecordBackListener((e) => {
  setRecordSeconds(e.currentPosition);
  setMetering(e.currentMetering);
  // console.log('metering==', e.currentMetering, e.currentPosition);
});
console.log('res==', result, metering, recordSeconds);

};
"react-native-audio-recorder-player": "^3.6.6",
"react-native": "^0.72.7",
issue on both platforms
the console log statement in the listener should work
the console log statement in the listener doesnt work

What android version are you using in the simulator?

iOS callback addRecordBackListener not working too
Example work:
const result = await audioRecorderPlayer.current.startRecorder()

{"e": {"currentMetering": 0, "currentPosition":<PHONE_NUMBER>54195, "isRecording": true}, "state": {"recordSecs":<PHONE_NUMBER>54195, "recordTime": "00:00:52"}}

Not working:
const result = await audioRecorderPlayer.current.startRecorder(uri, audioSet)

iOS callback addRecordBackListener not working too when use path prop.
Example success work:
const result = await audioRecorderPlayer.current.startRecorder()

{"e": {"currentMetering": 0, "currentPosition":<PHONE_NUMBER>54195, "isRecording": true}, "state": {"recordSecs":<PHONE_NUMBER>54195, "recordTime": "00:00:52"}}

Not working:
const uri = `${RNFS.CachesDirectoryPath}/sound.aac`

const result = await audioRecorderPlayer.current.startRecorder(uri)
// const result = await audioRecorderPlayer.current.startRecorder(uri, audioSet)

May be problem is directory path, if use only name it work,
https://github.com/hyochan/react-native-audio-recorder-player/blob/2816abab9b049b104ff29b2c8ac1792a8e19822b/ios/RNAudioRecorderPlayer.swift#L46
    const result = await audioRecorderPlayer.current.startRecorder(
      'sound.aac',
      audioSet,
      meteringEnabled,
    )

Thank you for your solution. This worked perfectly.


Kdoc fixer false-negative: not generating a Kdoc for several public functions

diktat 1.0.0-RC
    fun hasNoChildren() = children.size == 0
    fun getFirstChild() = children.elementAtOrNull(0)

/**
 * @return
 */
fun hasNoChildren() = children.size == 0
// <------ where is a Kdoc?
fun getFirstChild() = children.elementAtOrNull(0)

I'm adding autofix label, because we still emit a warning, even though autofix doesn't work as expected

We can enhance the fixed logic further and not ask @return on simple delegating getters:
    @GetMapping("/projects")
    fun getProjects() = projectService.getProjects()

There is no sense to write @return all projects here, bu since 1.0.0-rc.3 we suggest this


Changed ordering of created pipelines to creation date instead of status

Codecov Report

Merging #90 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #90   +/-   ##
=======================================
  Coverage   64.44%   64.44%
=======================================
  Files          22       22
  Lines        1887     1887
=======================================
  Hits         1216     1216
  Misses        539      539
  Partials      132      132

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 374cc8d...f4198b8. Read the comment docs.


Add Backups page for lisk service

Description
Add a page to explain how to backup lisk service node for v0.7.0
Motivation
After running a lisk node for a while, there will be indexed data in the lisk service similar to snapshot in Lisk node.
Lisk service should be able to start from the 'backup' data, and the steps to generate the backup and restore the backup for the lisk service should be explained.
This issue in now being handled in https://github.com/LiskHQ/lisk-service/issues/1751

This issue in now being handled in https://github.com/LiskHQ/lisk-service/issues/1751


Update msodbc constants

Updated msodbc constants based on the latest header file from ODBC driver, in particular the ones for Column Encryption

Coverage increased (+23.5%) to 75.332% when pulling 48478125e0de9597159978040a7297bdb3a8d041 on yitam:updateConstants into 45d87b7a96cb049e0cedb9e6302428a2ad1050a4 on Microsoft:dev.

Coverage increased (+23.09%) to 74.957% when pulling 7c105ac08e217d2dbf0e6a93eb552aa4a86b0b78 on yitam:updateConstants into 45d87b7a96cb049e0cedb9e6302428a2ad1050a4 on Microsoft:dev.

Coverage increased (+0.3%) to 52.162% when pulling 1afc5091257a703aa92e75ce3b7bb77f8a6f893a on yitam:updateConstants into 45d87b7a96cb049e0cedb9e6302428a2ad1050a4 on Microsoft:dev.

Coverage increased (+23.4%) to 75.235% when pulling 25e7705249bea5e9da8a1be15bb2bee6b7b452bc on yitam:updateConstants into 45d87b7a96cb049e0cedb9e6302428a2ad1050a4 on Microsoft:dev.


Fix exception using unformatted string

Thanks, for consistency can you change to an f-string please.

Done!


problem about the result

Hello, thanks for your good jobs!
When I use your codes, i find almost all types of loss can reach the result that rank1 is over 0.60. But when i read the paper, if the method is over 0.60(rank1), it can be seen as a good method. Even the contrastive loss can achieve rank1 over 0.60. Why? Thanks

Network: Inception V2
Freeze BN
Dimension: 512
Adam 1e-5 learning rate
And utilizing all the pairs (different from the lifted structure randomly sampling)
All these together is enough to make good performance with contrasstive loss.

Thanks very much !

Network: Inception V2
Freeze BN
Dimension: 512
Adam 1e-5 learning rate
And utilizing all the pairs (different from the lifted structure randomly sampling)
All these together is enough to make good performance with contrasstive loss.

Wow, i used the contrastive loss in Cub200 and  then got rank1 0.68(but highest baseline in your chart is 0.66). When you train the model , have you use the data augmentation 'rand_crop' which is in your codes?
Thanks for your reply!

yes

Found in my experiment, why BNInception in your code If there is no freeze BN-layer's result will be about two points lower than after freeze.  Why is there no such phenomenon in the InceptionBN network in other people's papers?
Looking forward to your reply.


Batch the skinning compute shader

This PR  showed that we have benefit of batching all the avatar matrix job into one instead of  multiple one.
The same optimization could be done for the compute shader part of skinning; since right now we have one compute shader skinning per avatar

Pending: https://github.com/decentraland/unity-explorer/pull/1855 (pending merge)


[KYUUBI #3327] Replace the contributing link in the PR template

fix: #3327
Why are the changes needed?

The contributing link in the PULL_REQUEST_TEMPLATE is out-of-date, so this patch replaces it with the new contributing link.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

thanks @YesOrNo828, merged to master


[WIP] Use asset pipeline path helpers for virtual tour files

The SWF and XML files are in the asset pipeline already, but we refer to
them directly.  This means the tour only works on dev (where assets don't
get compiled into hash urls) or on production (where the unadorned files
have been uploaded to the whitehall frontend machines directly).
If we use asset_path helpers instead the tour should work in all
environments.
Spotted while trying to test https://trello.com/c/ni5o7K2J/70-changes-to-10-downing-street-history-page on integration

Marked as WIP as the only way to test if this works is to deploy and see what breaks.  Will remove WIP once we have a version that actually works.

Ofc as soon as I add the WIP tag the version that's up proves to be the one that works.  Ready to review now.


make install error in Ubuntu 18.04 LTS

I'm running Ubuntu 18.04 LTS under Windows Subsytem for Linux.  I cloned the repo without issue, as root via sudo su.
Cloning into 'nostril'...
remote: Enumerating objects: 264, done.
remote: Counting objects: 100% (7/7), done.
remote: Compressing objects: 100% (6/6), done.
remote: Total 264 (delta 1), reused 6 (delta 1), pack-reused 257
Receiving objects: 100% (264/264), 72.30 KiB | 796.00 KiB/s, done.
Resolving deltas: 100% (137/137), done.
root@xxx:/home/xxx/git-repos# cd nostril
root@xxx:/home/xxx/git-repos/nostril# make install
cc configurator.c -o configurator
./configurator > config.h
Reinitializing submodules deps/secp256k1 ...
Submodule 'deps/secp256k1' (https://github.com/bitcoin-core/secp256k1) registered for path 'deps/secp256k1'
Cloning into '/home/xxx/git-repos/nostril/deps/secp256k1'...
Submodule path 'deps/secp256k1': checked out '694ce8fb2d1fd8a3d641d7c33705691d41a2a860'
cc sha256.c
cc nostril.c
cc aes.c
cc base64.c
cd deps/secp256k1; \
./autogen.sh
./autogen.sh: 3: ./autogen.sh: autoreconf: not found
Makefile:36: recipe for target 'deps/secp256k1/configure' failed
make: *** [deps/secp256k1/configure] Error 127

I've updated Ubuntu via apt update followed by apt upgrade, with no change in behavior.
Any ideas what I'm doing wrong (aside from running Windows lol)

same here

Try,
sudo apt-get install libtool -y;
sudo apt-get install autoconf -y;

./configurator

make;


fix: Use correct re-exports

The re-exports in this project were wrong. The workers/ (plural) directory actually exists in the project, not worker/ (singular).

Thank you very much.


[5.3] [WIP] Support for default value in explicit bindings

Description
I would like to be allowed to enter a default value when using model aliases, like I am allowed in implicit bindings.
Implicit Binding has firstOrFail or first, if default value is present.
https://github.com/laravel/framework/blob/5.3/src/Illuminate/Routing/Router.php#L805
I am not 100% sure, but I think explicit bindings goes though model on Router.php which does not have default value check.
https://github.com/laravel/framework/blob/5.3/src/Illuminate/Routing/Router.php#L926
What do I expect:
I expect that 'hello world' should be returned, because a default value for name is set.

This test doesn't even make sense. That's not how explicit model binding works (it doesn't bind to a variable name).

Ok I'll give up. :+1:


Meeting Minutes

Meeting Notes for Jan 12, 2021 Lab Session #1
Items addressed:

Self Introduction
Create repo
Add license in the repo
Teamwork contract, Code of Conduct
Selected dataset

PR:
Discussed and agreed that we will create a new branch every time for each task assigned. Once completed, we will push it to our own fork, and then do a pull request from our own fork to the main repo.
We will also assign reviewers when pushing a PR, and any one of the reviewer can merge the PR as they see fit.
Dataset chosen:
https://www.kaggle.com/mathurinache/world-happiness-report?select=2020.csv
To-do:

Contribution Guidelines - Nicholas will upload it later
proposal writeup (https://docs.google.com/document/d/1A2-vTrzkiibF8Atw_0BTbK7SqB0rMm61o7UE1GBT8PA/edit)
Description of your app
Sketch of your app

Tasks splits:
proposal:
Motivation and purpose: William
description of data: Deepak
research questions: Nicholas
usage:Nicholas
sketch: (to be done with PowerPoints)
Zeliha
William
Next Meeting:
Wednesday Jan 13, 2021 (on slack)
Thursday Jan 14 7:00 p.m. (zoom link TBD)

Meeting Notes for Jan 14, 2021 7:00pm
Items Addressed:
Dataset:  decided to merge cost of living index and population by country files with happiness report.
Motivation and purpose - done
Description of data - done.
Sketch - reviewed and agreed that we will add two dropdown menu for user to select region and country. Also provided other feedback and Zeliha will finalize tonight.
To-do:

Research questions and usage scenarios - Nicholas has provided an idea and discussed with team members. Will finish writing before next meeting.
Description of your app - William will try to finish writing once Zeliha finalize the sketch based on group feedback.
Sketch - Zeliha to finalize
Dataset - Deepak to merge all three dataset into one which will include happiness report, population and cost of living expenses.

Next Meeting:
Friday Jan 15, 2021 11:00 a.m.

Meeting Notes for Jan 14, 2021 7:00pm
Items Addressed:
Dataset:  decided to merge cost of living index and population by country files with happiness report.
Motivation and purpose - done
Description of data - done.
Sketch - reviewed and agreed that we will add two dropdown menu for user to select region and country. Also provided other feedback and Zeliha will finalize tonight.
To-do:

Research questions and usage scenarios - Nicholas has provided an idea and discussed with team members. Will finish writing before next meeting.
Description of your app - William will try to finish writing once Zeliha finalize the sketch based on group feedback.
Sketch - Zeliha to finalize
Dataset - Deepak to merge all three dataset into one which will include happiness report, population and cost of living expenses.

Next Meeting:
Friday Jan 15, 2021 11:00 a.m.

Meeting Notes for Jan 14, 2021 11:00a.m.

Finalized sketch.png
Reviewed and edited "Usage and Research" & "Description of the App"
Put together proposal.md and updated readme.md

To-do:
Please check M1 instruction carefully one more time and if you want to make any necessary changes, communicate on Slack. A release will be created and submission to canvas will be at 11:00 p.m. tonight.

Meeting Notes for Jan 14, 2021 11:00a.m.

Finalized sketch.png
Reviewed and edited "Usage and Research" & "Description of the App"
Put together proposal.md and updated readme.md

To-do:
Please check M1 instruction carefully one more time and if you want to make any necessary changes, communicate on Slack. A release will be created and submission to canvas will be at 11:00 p.m. tonight.

Meeting Notes for Jan 15, 2021 2:30pm
Items Addressed:
Milestone to-do list walkthrough and set expectations
Meeting planned
Action Plans:
Work on the Dash file

Meeting Notes for Jan 15, 2021 2:30pm
Items Addressed:
Milestone to-do list walkthrough and set expectations
Meeting planned
Action Plans:
Work on the Dash file

Meeting Notes for Jan 22, 2021 11:00am
Items Addressed:
Milestone to-do list walkthrough and set expectations
Added available charts to the app.py
Deployed the app
Action Plans:
Work on the reflection file

Meeting Notes for Jan 22, 2021 11:00am
Items Addressed:
Milestone to-do list walkthrough and set expectations
Added available charts to the app.py
Deployed the app
Action Plans:
Work on the reflection file

Closing meeting minutes before milestone 4 release


--hostdisplay fails if X server runs without cookie authentication

x11docker is checking the XAUTHORITY (Hostxauthority) variables and correctly reports
x11docker WARNING: Your host X server runs without cookie authentication.

when the X Server runs without cookie authentication.
But when given the --hostdisplay option, following commands are run
Hostcookie=$(xauth -n -i -f $Hostxauthority list | grep :$Hostdisplaynumber | awk '{print \$3}')
xauth -n -i -f $Xclientcookie add $Newdisplay MIT-MAGIC-COOKIE-1 \$Hostcookie

which of course fail, because $Hostxauthority is unset/empty.
Using the --xauth=no option is also not helping, since it just unsets the XAUTHORITY variable (which in our case is already by default).

Encasing the commands in a guard checking for unset Hostxauthority would mitigate this issue, but this may also happen in other positions, since I get the general feeling that x11docker is always asuming xauth functionality.

Thank you for the report!
I'll check this some day.


Password Policy Issue

Hi,
Whenever I am trying to set user password - using password policy.
It does not validate. here is my my code. With reference of code
https://docs.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.identity.passwordoptions.requireduniquechars?view=aspnetcore-3.1
services
               .AddIdentity<Users, IdentityRole>(options =>
               {
                   options.User.RequireUniqueEmail = true;
                   options.Password.RequiredLength = 8;
                   options.Password.RequiredUniqueChars = 3;
                   options.Password.RequireDigit = true;
                   options.Password.RequireLowercase = true;
                   options.Password.RequireUppercase = true;
               })

Here is my test Case
AdMmid@123 - Valid
Admmid@123 - Not Valid
May I know why first case is valid and another case is not. As I assume in both cases there is more than 3 unique characters.

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support group like Stack Overflow with the asp.net-core tag.


Second connection seems to fail in Internet Explorer 9, 10, and 11

Steps to reproduce:

In Internet Explorer 9, 10, or 11, connect to Dropbox in the Kloudless widget.
Refresh page.
In Kloudless menu, click to view your accounts.
Click the Box icon.
Provide credentials and connect.

Expected:  Box window closes and you see the file list of the service.
Observed:  The Box window closes, but the view does not update.  The Box service is not added to your list of accounts.  If you then refresh the browser and go back to Kloudless, you'll see Box is already successfully connected.

Thanks for identifying these issues. We'll take a closer look at IE / sorting within the next 2 weeks.

@jpplangrid A fix has been released for this. Please let me know if you continue to encounter any issues here.

Thanks @vinodc!  I verified this works in IE9, IE10, and IE11.


ES7 objectRestSpread bug

[x] check the documentation of ESDoc

https://esdoc.org/manual/configuration/config.html
https://esdoc.org/manual/usage/tags.html
https://esdoc.org/manual/faq/faq.html

[x] your input data (codes, config and more)
https://github.com/ratson/bug-esdoc

class Bug {
  constructor() {
    const o = {}
    this.a = {
      ...o,
    }
  }
}

[x] write output that you obtained (log or screenshot)
TypeError: Cannot read property 'name' of undefined
[x] if possible, write your expectation for the input
Should generate docs without error
[x] your environment

OS type and version
macOS 10.10
Node.js version
v7.3.0
ESDoc version
v0.5.1

@ratson Sorry for late reply 🙇
I merged and improved to guess type.
https://github.com/esdoc/esdoc/pull/364
https://github.com/esdoc/esdoc/commit/b613502c8eaa010979f3344012a2543801c18d38
And I will release it in next version


Re-implement NPE checks and division by 0 checks in the ILCodeGenerator

Instead of the NasmCodeGenerator, so they'll work on all architectures (!)

This is causing issues with temps being read more than once (once for the null or division by 0 check, and once for where it's actually used.)

Blocked by #248

And index checks


v5 + emotion - increase in Cumulative Layout Shift

[x ] The issue is present in the latest release.
[ x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

I notice significant increase in CLS after upgrading project from v4 to v5 and refactoring makeStyles/useStyles to emotion.
Increase is from acceptable 0.059 on mobile and 0.006 desktop to 0.574 mobile and 0.869 desktop.
Project is next.js, the only change made is makeStyles => emotion refactoring.
It was done using css prop
example
const style = {     link: css'         color: inherit;     ' } <NextLink href={href}>{children}</NextLink>
Opted to use css prop as it is the most similar to previous format using makeStyles
Did anyone else noticed this metric to increase?
There is another similar issue at emotion's git Emotion causes large Layout Shift (with critical CSS extraction) #2154
, but not completely the same, i don't have any local font for example, i have google fonts which add only 0.0something to cls
I didn't do any specific configuration or setup, it is and should work out of the box with nextjs and mui
Your Environment 🌎

Next.js, mui v5, emotion

Thanks for the feedback.
Without a reproduction there's nothing we can do about this other than refer to the existing emotion issue.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-next), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

i agree, was wondering to see if there are more cases.
i will try to setup some example

Hi @eps1lon,
I'm also facing the same layout shifting issue, I have bootstrapped my app using MUI NextJS example, On top of that I have added a color mode switcher (Dark vs Light).
Here is my code https://github.com/tmkasun/me/tree/nextMe
and here is the deployed app https://nextme.vercel.app/
I ran a lighthouse performance test on the above deployment and got the following result

Cumulative Layout Shift
1.229

App has only @emotion & @mui packages installed other than next & react

I see the same problem in my website

Please provide more details on what's the root cause of this Cumulative Layout Shift.


Merlin not recognizing reason syntax on Windows

I got almost everything working on Windows. I did the following:

Installed ocaml from here.
Installed reason via opam following the instructions on the readme.
Installed merlin via opam.
Installed the vscode-reasonml extension.

Then I cloned react-reason-example. Almost everything worked fine, refmt was working, the app was built without errors, but the vscode extension isn't recognizing the reason syntax. It seems to be recognizing it as ocaml code instead of reason code. Here is a screenshot of it:

Here's the result of where refmt ocamlmerlin ocamlmerlin-reason:
C:\OCaml64\home\fhelwanger\.opam\4.02.3+mingw64c\bin\refmt.exe
C:\OCaml64\home\fhelwanger\.opam\4.02.3+mingw64c\bin\ocamlmerlin.exe
C:\OCaml64\home\fhelwanger\.opam\4.02.3+mingw64c\bin\ocamlmerlin-reason.exe

Here's the result of ocamlmerlin -version and refmt --version:
The Merlin toolkit version 2.5.3, for Ocaml 4.02.3

Reason 1.13.0 @ dbe5d86

you can try https://github.com/reasonml/reasonml.github.io/issues/109
it mostly works for me out of the box, only 1 requirements you need to set this in vscode settings
 "reason.path.ocamlfind": "ocamlfind.cmd",
  // The path to the `ocamlmerlin` binary.
  // "reason.path.ocamlmerlin": "ocamlmerlin.cmd",
  // The path to the `opam` binary.
  "reason.path.opam": "opam.cmd",
  // The path to the `rebuild` binary.
  "reason.path.rebuild": "rebuild.cmd",
  // The path to the `refmt` binary.
  "reason.path.refmt": "refmt.cmd",
  // The path to the `refmterr` binary.
  "reason.path.refmterr": "refmterr.cmd",
  // The path to the `rtop` binary.
  "reason.path.rtop": "rtop.cmd",

Yes! Actually, I created https://github.com/fhelwanger/ocaml-on-windows because of this issue but forgot to update it 😄
Thanks!

I think that in the last vscode-reasonml version you don't even need to configure your reason paths anymore. It was fixed here https://github.com/freebroccolo/ocaml-language-server/pull/31


InstanceContainer#copy() does not work for unloaded instances

If you use an InstanceContainer that has not yet been loaded when copying it, the resultant InstanceContainer will be empty. To get around this, I can use InstanceContainer#loadChunk() but imo InstanceContainer#copy() should do this automatically

This is expected behavior. You need to load the chunks you want to copy.
Loading them automatically does not make much sense. In most cases there is not a simple way to know all of the chunks, and loading them all is unnecessary at best, problematic at worst if it's a big world.


G1 SaveGame loading from ZenKit causes a crash

The attached save game causes ZenKit to crash when calling new SaveGame().LoadWorld().
savegame13.zip
The last logged message is:
level=Warning, name=ReadArchive, message=Object not fully loaded: zCEventManager
(No further details on C# side then. The SaveGame seems to contain some invalid data for NPCs. When I load a SaveGame near the start area it works (less NPCs, or at least not the broken NPC?)
Stacktrace:
========== OUTPUTTING STACK TRACE ==================

0x00007FFEBF10FE4C (KERNELBASE) RaiseException
0x00007FFE4BAD38D1 (zenkitcapi) ZkWorld_addNpc
0x00007FFE4BC02489 (zenkitcapi) ZkWorld_addNpc
0x00007FFE4BABAD62 (zenkitcapi) ZkWorld_addNpc
0x00007FFE4BA41E2B (zenkitcapi) ZkWorld_addNpc
0x00007FFE4BA37716 (zenkitcapi) ZkWorld_addNpc
0x00007FFE4BA550D8 (zenkitcapi) ZkWorld_addNpc
0x00007FFE4BAAD2A6 (zenkitcapi) ZkWorld_addNpc
0x00007FFE4BAA3110 (zenkitcapi) ZkWorld_addNpc
0x00007FFE4BA550D8 (zenkitcapi) ZkWorld_addNpc
0x00007FFE4BA8E5D2 (zenkitcapi) ZkWorld_addNpc
0x00007FFE4B9FF3F5 (zenkitcapi) ZkSaveGame_loadWorld
0x000001D50219DAA6 (Mono JIT Code) (wrapper managed-to-native) ZenKit.Native:ZkSaveGame_loadWorld (uintptr,string)
0x000001D50219D253 (Mono JIT Code) ZenKit.SaveGame:LoadWorld (string)
0x000001D50219966B (Mono JIT Code) [C:\Users\jaxt0\Documents\Gothic-UnZENity-Project\_git\Gothic-UnZENity\Assets\UnZENity-Core\Scripts\Debugging\SaveGameDebugger.cs:58] GUZ.Core.Debugging.SaveGameDebugger:CompareSaves ()

...

level=Warning, name=ReadArchive, message=Object not fully loaded: zCEventManager
UnityEngine.StackTraceUtility:ExtractStackTrace ()
UnityEngine.DebugLogHandler:LogFormat (UnityEngine.LogType,UnityEngine.Object,string,object[])
UnityEngine.Logger:Log (UnityEngine.LogType,object)
UnityEngine.Debug:LogWarning (object)
GUZ.Core.Util.Logging:OnZenKitLogMessage (ZenKit.LogLevel,string,string) (at Assets/UnZENity-Core/Scripts/Util/Logging.cs:27)
ZenKit.Logger:_nativeCallbackHandler (intptr,ZenKit.LogLevel,string,string)
ZenKit.SaveGame:LoadWorld (string)
GUZ.Core.Debugging.SaveGameDebugger:CompareSaves () (at Assets/UnZENity-Core/Scripts/Debugging/SaveGameDebugger.cs:58)
GUZ.Core.Debugging.SaveGameDebugger:OnValidate () (at Assets/UnZENity-Core/Scripts/Debugging/SaveGameDebugger.cs:37)

What was done before saving the game?

Started the game 1min ago
Marvin mode teleportation into Gomez room (FP_CAMPFIRE_GOMEZ1)
Hit "Save"

Ooooooo faaaancyy! You discovered a missing puzzle piece! You were in a cutscene when you hit save, right? Well the thing is: ZenKit doesn't yet handle cutscenes for world saves, because I never found a save which was saved during one (or at least had the object actually contain something).
I'll have to implement that and I'll let you know once its done.
See also: EventManager::load().

With pleasure. 🙇♂
I wasn't inside a cutscenes, but as you said: maybe it gets filled at a later stage, I hit somehow.
Thanks for checking. Can't wait to go on.

Fixed. Thank you.


Update name and type for dataset_deployments

I've never understood why dataset_deployments is an array since it simply contains the url for the CXG permanent link:
    "dataset_deployments": [
      {
        "url": "https://cellxgene.cziscience.com/e/aad97cb5-f375-45ef-ae9d-178e7f5d5180.cxg/"
      }

I would recommend the following changes:

The type should be a string (url) and not an array.
The name is not self-documenting. What is a "dataset deployments" to a curator?

@jahilton suggests:
visualization_url or explorer_url

For consistency with existing portal APIs, the name should be explorer_url.

Presumably this was introduced at a time when we anticipated datasets being hosted in various ways across multiple services.

For consistency with existing portal APIs, the name should be explorer_url.

But it is consistent with the UI API... do you still want to make the change?

Yes please. The change to name and typing is consistent with newer APIs such as https://api.cellxgene.cziscience.com/dp/v1/datasets/index used in filter by metadata.

The change was made to explorer_url of type 'string'


collapseAll() issue.

This is what i have done but CollapseAll is not working. Can you please check?
var grid, dialog;
$(document).ready(function () {
grid = $('#grid').grid({
    primaryKey: 'SNo',
    dataSource: '/OrderedChemicalss/GetOrderedChemicals',
    selectionType: 'multiple',
    resizableColumns: true,
    selectionMethod: 'checkbox',
    grouping: { groupBy: 'OrderNo' },
    columns: [
        { title: 'SNo', field: 'SNo', hidden: true },
        { title: 'Order No', hidden: true,field: 'OrderNo', sortable: false },
        { title: 'Order Date', field: 'OrderDate', sortable: true, type: 'date', format: 'dd/mm/yyyy' },
        { title: 'Staff Name', field: 'StaffName', sortable: true },
        { title: 'RefDocNo', field: 'RefDocNo', sortable: true },
        { title: 'Supplier Name', field: 'SupplierName', sortable: true },
        { title: 'Chemical Code', field: 'ChemicalCode', sortable: true },
        { title: 'Chemical Name', field: 'ChemicalName', sortable: true },
        { title: 'Suppliers Code', field: 'SupplierCode', sortable: true },
        { title: 'Quantity', field: 'Quantity', sortable: false },
        { title: 'UOM', field: 'UOM', sortable: false },
        { title: 'Price', field: 'Price', sortable: false },
        { title: 'Status', field: 'Status', sortable: true },
        { tmpl: 'edit', align: 'center', events: { 'click': Edit } },
        { tmpl: 'delete', align: 'center', events: { 'click': Delete } }

    ],
    pager: { limit: 5 }
});

grid.on('dataBound', function (e, records, totalRecords) {
    alert('dataBound is fired. ' + records.length + ' records are bound to the grid.');
    grid.collapseAll();
    alert('dataBound is fired. ' + records.length + ' records are bound to the gridsssssssss.');
});

grid.on('dataBinding', function () {
    grid.collapseAll();
});

});

I think that you need to use .expandAll() instead of .collapseAll() in order to expand all rows on dataBound. Could you try that.

Yes, i am trying, I guess it takes primary key in java script as an argument.
Actually by default i want to collapse all instead of expand all.

No its not working.

By Default i want to display them like this

This problem is fixed in the new development build. You can find the new development build in the dist folder at github. You have to use dataBound event and collapseAll method in order to do that.

Thanks, its working fine, Can you please do this in https://cdnjs.cloudflare.com/ajax/libs/gijgo/1.7.0/combined/js ?
using this icons doesn't load up.


Malformed update JSON file

The forge update JSON file, fetched from https://raw.githubusercontent.com/puggan/mc-colorbricks/1.16/master/update.json has a syntax error on line 22 (missing comma).
{
  "homepage": "https://github.com/puggan/mc-colorbricks",
  "1.14.4": {
    "1.14.4-1.1.1": "v1.1.1"
  },
  "1.15.2": {
    "1.15.2-1.1.1": "v1.1.1"
  },
  "1.16.1": {
    "1.16.1-1.1.1": "v1.1.1"
  },
  "1.16.2": {
    "1.16.2-1.1.1": "v1.1.1"
  },
  "promos": {
    "1.14.4-latest": "1.14.4-1.1.1",
    "1.14.4-recommended": "1.14.4-1.1.1",
    "1.15.2-latest": "1.15.2-1.1.0",
    "1.15.2-recommended": "1.15.2-1.1.1",
    "1.16.1-latest": "1.16.1-1.1.1",
    "1.16.1-recommended": "1.16.1-1.1.1" <--------- here
    "1.16.2-latest": "1.16.2-1.1.1",
    "1.16.2-recommended": "1.16.2-1.1.1"
  }
}

Thanks, fixed.


Attach debug info from source language

Feature
Currently, wasmtime's error in wat text files are based on the compilation offset.
When an error occurs, it is very confusing where the error occurred.
I would like to attach some source language location information to point out the error location.
Similar to:
;; @url: file://path/position.wasm:line:column
(type $test-type (externref))
;; @url: file://path/position.wasm:offset
(func $test-function
    ;; @file: main.rs:12:24
    (block $test-block

    )
)

binaryen supports similar features
at: https://github.com/WebAssembly/binaryen/blob/921644ca65afbafb84fb82d58dacc4a028e2d720/test/lit/debug/full.wat#L49-L64
Benefit
Clearer error location
Implementation
Unknown, perhaps we can parse custom instructions and mixin them into the final binary wasm with debugger info.
Alternatives
Wait for source language debugging.
But I'm not quite sure how this works, or how to integrate it with my pipeline.
Many times I write wat files manually, without any toolchain.

For solution 1, I first think about when it is necessary to write wat manually

Learn how to use wat
Communicate or experience new proposed features
Write test cases
Debug optimizer pass or compiler artifacts

Generally speaking, these codes are small in size, and it is not difficult to infer the wrong instructions based on the error message

Decompile highly optimized released wasm and try to find out where the problem is in wat

Marking the wrong instructions is of great help at this time, but I doubt whether this thing itself is meaningful.
Therefore, I think that from the perspective of usage scenarios, maintaining a separate set of wat format error reporting benefits is not obvious.
From the implementation point of view, the parser needs to track the line column of each instruction and pass it layer by layer at runtime, and the changes to the backend are also relatively large.

For solution 2, the biggest problem is that there is no standardization. Other tools may also use special tags independently, just like binaryen does now.
Another hidden problem is whether wasm cg is willing to standardize. Maybe people have no motivation to do so: https://github.com/WebAssembly/debugging/issues/19#issuecomment-1984894109
From the implementation point of view, binary debugging is a determined goal. The change is that the front-end needs to parse some additional custom information, and the runtime and back-end do not need to be changed.
When changes occur or corresponding proposals appear, the cost of changes is relatively small.

In summary, I prefer option 2 when choosing between the two, unless there is a better solution

Another unknown question is what does the wat product of decompiling wasm with DWARF information look like?
Will debugger info be erased directly?
If not, is it appropriate to use the annotations proposal?
If possible, this is very suitable as a bridge for two-way conversion

Currently with wasm-tools print if you've got a wasm with dwarf it'll print (@custom (after data) ".debug.XXXX" "...") which is basically just there to show you it exists, it doesn't actually work. That debuginfo is almost certainly broken once the text goes back to binary (since the dwarf handles binary offsets and binary -> text -> binary likely changes binary offsets, e.g. reencoding lebs into their minimal size)
Otherwise though the annotations proposal is likely the way to go here, but it's still a question of formats. Wasmtime supports DWARF as a means of converting binary offsets to filenames/line numbers but DWARF has no definition in the wasm text format with annotations, and supporting the entirety of dwarf with annotations would be a pretty ambitious project. On the web there's source maps which is not DWARF and Wasmtime doesn't support. I'm not personally familiar with source maps, but the challenge again would be representing all of source maps in the text format which may or may not be feasible.
There's the other alternative of creating a third means of mapping filenames/line numbers to binary offsets. That could be customized just for the wasm text format and have a clear definition with the annotations proposal. The downside of this though is that it's a third alternative and support would need to be added to all consuming/producing tools, which is likely not trivial.
I would hazard a guess that source maps might be the way to go here. That would require investigating if it's reasonable to represent source maps as annotations (which I'm not sure if it is) in the text format and then adding support to Wasmtime/wasm-tools/etc. I'm also not sure who actually produces source maps originally (e.g. tools that do this, I just know that they exist)

If you're going to add support for generating debug info to wasm-tools, use DWARF instead of source maps. You don't have to emit all the info that DWARF supports emitting (e.g. full source variable recovery) and can emit just the bits that are equivalent to what you'd put in a source map (the .debug_line section). But DWARF generally has a much higher quality of toolchain support for reading/writing it and is much more compact (binary encoding rather than base64 strings).

Ah yes that's a good point, and sorry yeah I shouldn't say one approach should definitely be used over the other. Adding limited support for annotations<->DWARF to tooling I think could also be reasonable.

If I want to add support for .debug_line in wat where do I start?
I tried tracing from this call
https://github.com/bytecodealliance/wasmtime/blob/f40aaa51a1df835498985acfc91ade33240d02b5/tests/all/traps.rs#L817-L828
I found
https://github.com/bytecodealliance/wasmtime/blob/f40aaa51a1df835498985acfc91ade33240d02b5/crates/environ/src/compile/module_environ.rs#L792-L825
But I don't know how to make the front end fill in dwarf.debug_line based on the wat@custom section

For my answer below I'm assuming that the scope of the implementation you're interested in is to add the ability to have (@foo ...) annotations in the text format, probably interspersed with actual instructions, which ends up getting the .debug_* sections filled out.
To do that you'll want to take a look at https://github.com/bytecodealliance/wasm-tools, specifically the wast crate located at crates/wast. I'd recommend reading over https://github.com/bytecodealliance/wasm-tools/pull/1394 as well which implements branch hints which is the closest (ish) feature to what you're looking to implement. The various steps involved will be:

Invent a desired syntax, for example (@debug_line "foo.rs" 10 3) or something like that.
Add parsing support to wast to parse these annotations. It'll probably be around expr.rs and you'll store metadata around here.
Update encoding of expressions to return not only branch hint metadata but additionally debug info metadata (this is the location where actual binary offsets are created since it happens during emission)
After functions are encoded you'll check to see if there's any debuginfo present and if so emit the requisite .debug_* custom sections. For this you'll probably want to use the gimli crate to create dwarf sections.

I don't know enough about DWARF myself to be able to suggest enough about what the annotations should look like or what dwarf sections are going to be emitted. You probably won't have to emit every section in a gimli::Dwarf though as much of those aren't related to just filename/line number info I think.

FWIW, there is a simple DWARF-writing example here: https://github.com/gimli-rs/gimli/blob/master/crates/examples/src/bin/simple_write.rs#L106