Tree-shaking not working properly

I've been testing a bit with the production bundling, and the tree-shaking is not working as I would expect it to. I'm new at the tool, so perhaps I am misunderstanding what it is supposed to do, but the behaviour seems odd to me.
The problem:
Classes that are neither imported nor used are still included in the bundle.
I started a completely clean angular-cli project and created a new file. In this file I created two classes, Foo and Bar. I then, inside app.component.ts import the first of these classes.
import { Foo } from './some.service';
When then creating a production build I assumed that the class Bar would be removed, since it is never imported or used. But it isn't, it's included in the minified bundle. The same happens if I split the classes into one file each and then use a barrel (e.g. index.ts) and import Foo from there.
Am I misunderstanding tree shaking or is this behaviour incorrect? Or do I have to change some setting somewhere to activate it?
This is on Mac OSX - El Capitan and using beta17.

Hi @HonnKing , AFAIK, the aot-compiler does not do tree-shaking.
Tree-shaking is done i a process later in your build pipeline.

Might be, but then the documentation needs to be updated.
"All builds make use of bundling, and using the --prod flag in ng build --prod or ng serve --prod will also make use of uglifying and tree-shaking functionality."

I'm sorry, I was talking about the aot compiler.... But yes, the Angular-CLI makes use of tree shaking. This is automatically done in Webpack (which is what the CLI is using)
I'm sorry for the confusion ;)

Tree shaking definitely should be present. This is something we need to investigate.

My test was similar to yours, I created src/app/tree-shaking-test.ts with the following content:
export class Foo {
  name = 'foo class';
}

export class Bar {
  name = 'bar class';
}

Then added the following to src/app/app.module.ts:
import { Foo } from './tree-shaking-test';

console.log(Foo);

Ran ng build and could find this block inside of dist/main.bundle.js:
"use strict";
/* harmony export (binding) */ __webpack_require__.d(exports, "a", function() { return Foo; });
/* unused harmony export Bar */
var Foo = (function () {
    function Foo() {
        this.name = 'foo class';
    }
    return Foo;
}());
var Bar = (function () {
    function Bar() {
        this.name = 'bar class';
    }
    return Bar;
}());

ng build --prod yielded a similar code block, with both "foo class" and "bar class":
function(t,e,n){"use strict";n.d(e,"a",function(){return r});var r=function(){function t(){this.name="foo class"}return t}();(function(){function t(){this.name="bar class"}return t})()}

Ok there's an issue on Webpack for this: https://github.com/webpack/webpack/issues/2899. I'll keep an eye on it.

@filipesilva Any updates?
Without tree shaking I'm ending up with a prod build for main.bundle.js of 2.5MB, so it is a very large issue.

Similarly to @trevorfrese I'm interested in any progress on this, anything new?

@trevorfrese @lbialy I don't have any updates, just watching the issue.
@TheLarkInn is there any update on your end?

any solution for the problem? Or should I use roupp after buillding the build ?

Closing this as this is a webpack issue and there are fixes being discussed on their side.

@hansl I believe this issue should be re-opened to track. angular-cli currently promises to deliver low-configuration treeshaking, which is one of our motivations for using it. If webpack doesn't get it together it's reasonable to make a call to either switch to rollup or remove the expectation of treeshaking.

The official recommendation from the Angular team is to ship custom libraries as a Flat ES Module generated with rollup. Nevertheless, even exporting libraries as a Flat ES Module, while also avoiding Kitchen Sink modules, importing one module from the FESM would result in the whole library being imported into the final bundle. I know this is particular to Webpack treeshaking not working properly, just wanted to give some input about this, it's still happening.


Correct a word in remarks of /directives.

.drective to .directive

@ROOOO : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Gettings requirements to build wheel  did not run successfully

Hello,
I'm facing an issue while installing the newest Bumble version ( > 0.0.134) on a fedora Linux 37 VM.
The following commands give me an error (see below):
pip install bumble
python -m pip install bumble

even while cloning and trying a local install with pip and python local cmd lines such as:
pip install -e .

error :
Collecting libusb-package==<IP_ADDRESS>
  Downloading libusb-package-<IP_ADDRESS>.tar.gz (408 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 408.7/408.7 kB 3.1 MB/s eta 0:00:00
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [18 lines of output]
.
.
.
        File "/tmp/pip-build-env-8ff85nug/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 335, in run_setup
          exec(code, locals())
        File "<string>", line 20, in <module>
      ModuleNotFoundError: No module named 'tomli'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

I found one way to fix it: a newest version of libusb-package (<IP_ADDRESS> works on my VMs).
In setup.cfg : libusb-package == <IP_ADDRESS>; platform_system!='Emscripten' $\implies$ libusb-package == <IP_ADDRESS>; platform_system!='Emscripten'

Thanks for reporting this. I have pushed a hot fix for it, and created a package release v0.0.140. Please let me know if that works for you.

Thanks for the hotfix, it works!


MPA sample should use the username instead of user to create a WebAuthnAuthenticator

In the MPA sample, a WebAuthnAuthenticator gets created from the variable user.
https://github.com/webauthn4j/webauthn4j-spring-security/blob/1564c361bc5659ba3a8b9e7a76b608987a2e7b9b/samples/mpa/src/main/java/com/webauthn4j/springframework/security/webauthn/sample/app/web/WebAuthnSampleController.java#L155-L158
But AssertionOptionsProviderImpl passes authentication.getName() to the WebAuthnAuthenticatorService when looking up credentials.
https://github.com/webauthn4j/webauthn4j-spring-security/blob/1564c361bc5659ba3a8b9e7a76b608987a2e7b9b/webauthn4j-spring-security-core/src/main/java/com/webauthn4j/springframework/security/options/AssertionOptionsProviderImpl.java#L161-L163
Since the in-memory implementation is used in the sample, we search for the name of the current Authentication in the map.
https://github.com/webauthn4j/webauthn4j-spring-security/blob/1564c361bc5659ba3a8b9e7a76b608987a2e7b9b/webauthn4j-spring-security-core/src/main/java/com/webauthn4j/springframework/security/authenticator/InMemoryWebAuthnAuthenticatorManager.java#L47-L49
Since the sample code stored the whole UsernamePasswordAuthenticationToken, this will never match and a user authenticated in a first step will not be presented with allowed credentials.
To allow matching to work, I would propose to change the sample code to use the users name instead of the whole user.
If you think this is the correct way, I am happy to create a PR for it.

Sorry for late response. As you pointed out, this needs to be fixed. I'll appreciate if you create a pull-request for it.


who can give me a success demo?

it not work on ios device, and it tell me "cordova" is not define in spa?

Because it's for the app and not for the Web

it not work on ios device, and it tell me "cordova" is not define in spa?

Because it's for the app and not for the Web


VS Code freezes

VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-08T23:31:51.320Z)
OS Version: Darwin x64 16.4.0
Extensions:

Extension
Author
Version

vscode-swift-language
rlovelett
0.1.0

Steps to Reproduce:

Please provide some information about the issue you are seeing and some repo steps.
Thanks

open VSC, do some work in the editor
then switch to other programs
wait for sometime, about 20 mins
want to back to VSC and continue previous work, but VSC is not response, have to restart VSC

Can you show us a screenshot of the state of VS Code when it does not respond?

@igordonxiao
I wonder if this issue is being caused by an installed extension. Can you try to run VS Code without extensions? From the command line (NOT the integrated terminal in Code), execute: code --disable-extensions and try your steps again to see if it reproduces. If you see it is an issue with the extension, please file it against the extension repository itself.

@bpasero Ok, i'll try to remove extensions and feedback this issue.

I have removed all extensions, VSC was still not response for waiting sometimes.

@igordonxiao does this even reproduce when you just have a simple file open with little content? Trying to understand if this is maybe related to the content of the editor you are seeing.

@bpasero I keep some content in editor and didn't save it as a file, i did this behavior in older VCS versions, but worked well.

I am seeing intermittent freezes while editing JS files on Windows 10, seemingly random without linkage to any user action. I have tried running without any plugins, and it still persists. Please advise.

@igordonxiao is the contents you keep in the editor very large? does it reproduce with any content?

@bpasero it's no relationship with the content is, just a few lines text. Now it did not occur again in new VSC version on my PC, maybe it has been fixed.


Update code to compile without issue when ExistentialAny enabled

Motivation:
We should feel confident that we compile clean when built in an environment that has enabled upcoming Swift features.
When compiling swift-asn1 specifying .enableUpcomingFeature("ExistentialAny") there are some minor warnings emitted by ASN1.swift.
Modifications:
Minor code changes to address surfaced warnings
Result:
Better confidence that we don't break downstreams that compile this code with these upcoming features enabled.

@swift-server-bot add to allowlist


/usr/bin for shell scripts

In all you shell scripts /bin/bash always point to /bin:
https://github.com/six2dez/reconftw/blob/main/install.sh#L1
https://github.com/six2dez/reconftw/blob/main/reconftw.sh#L1
Instead of /bin/bash I suggest to use /usr/bin/env bash , it will search for bash in $PATH and then start with the first one it can find

Done


Using network-config v1, can't have a MAC address with all numbers

We are still on CloudBase-Init 1.1.4 due to https://github.com/cloudbase/cloudbase-init/issues/147. In testing using kvm, I found that if my VM and network-config has a MAC address with only numbers, CloudBase-Init fails to parse it because "int type doesn't have lower function".
Seems there needs to be some protection that if a MAC is of int type, don't try toLower (or always force the incoming MAC to be string).
My network-config:
version: 1
config:
  # multiple interfaces or networks so we need a bond/team NIC
  - type: physical
    name: eth0
    mac_address: 52:54:00:00:00:01
    mtu: 1450
  - type: physical
    name: eth1
    mac_address: 52:54:00:00:00:02
    mtu: 1450
  - type: bond
    name: Default
    mac_address: 52:54:00:00:00:01
    bond_interfaces:
    -  eth0
    -  eth1
    params:
      bond-mode: active-backup
      xmit_hash_policy: layer2
      bond-lacp-rate: false

  # multiple networks with vlans
  - type: vlan
    name: "Default.0"
    vlan_link:  Default
    vlan_id: 0
    subnets:
    - type: static
      address: <IP_ADDRESS>/24
      gateway: <IP_ADDRESS>
      dns_nameservers:
        - <IP_ADDRESS>

My virt-install command to bring up the VM:
virt-install \
  --name win2022 \
  --memory 2048 \
  --vcpus 2 \
  --disk size=50 \
  --cdrom ./Win2022.iso \
  --os-variant win2k22 \
  --network network=default,mac=52:54:00:00:00:01 \
  --network network=default,mac=52:54:00:00:00:02 \
  --graphics spice,listen=<IP_ADDRESS> \
  --boot uefi \
  --disk path=./win2022.img,format=raw,cache=none,device=floppy

From error log:

Linux cloud-init might have a similar issue however have not tried it: https://github.com/canonical/cloud-init/blob/main/cloudinit/net/network_state.py#L738

Hello,
I could reproduce the issue and it looks like a yaml quirk, as the yaml implementation sees the unquoted MAC addresses as integers.
Cloud-init has the same implementation and from a similar bug report, see https://github.com/canonical/cloud-init/issues/3703:
cloud-init suggests in network config docs above that mac_address values should be colon-delimited values such as 04:11:20:54:23:11. Also, the network-config file is read as yaml, any integer-only values present in yaml will be interpreted as an int instead of strings. To ensure yaml sees an integer value as a string you would need either single quotes or double quotes around a value.

Thank you,
Adrian


Display placeholder during Web App loading

Is your feature request related to a problem? Please describe.
Currently, there are Telegram applications which do not display placeholder during Web App loading. This feature is rather important to make user's experience using Web Apps much better.
Describe the solution you'd like
Display loader during Web App loading.
Describe alternatives you've considered
You can find good solution made by Telegram macOS developers.
Additional context
-

Issues:
https://github.com/telegramdesktop/tdesktop/issues/25792
https://github.com/Ajaxy/telegram-tt/issues/228
https://github.com/TelegramMessenger/Telegram-iOS/issues/976


Quadratic

Enter your question -
Find the roots of the quadratic Equation
Enter link to the question(if question belongs to any online platform) -
NO link
Tags for the question(eg - Array, Basic, Stack, etc.) -
Maths, Datatypes

hello sir, kindly assign this issue to me

@Krush2311,
Go ahead, Kindly add your solution to "Basic Codes" folder.
Deadline - 30/09/2021


Email do not exist

<EMAIL_ADDRESS>that is written in the github readme do not exist.
Final-Recipient: rfc822<EMAIL_ADDRESS>Action: failed
Status: 5.0.0
Diagnostic-Code: smtp; 550-5.1.1 The email account that you tried to reach does not exist. Please try
550-5.1.1 double-checking the recipient's email address for typos or
550-5.1.1 unnecessary spaces. Learn more at
Please put a contact information for sending invitations to new members in the slack chat.

@tkapet thanks! I'm fixing the email issue at the moment. We'll send you slack invite + update the readme as well

Hey @tkapet
Here is the slack invite link
https://join.slack.com/t/vuestorefront/shared_invite/enQtMjUyMzA1NDM1ODg4LTYzZjljNjZlN2JmMmM4NzQzOWFkOTA5NTMyOTg0NzIwNjMxNTNjYTZhNzAzOWM5MjEyYzE2OWE3ZTJmOTU3OTY

E-mail issue fixed, should work now


Additional Rename Feature

Description of the new feature / enhancement
There should be an option if there are many diffrent files that you can select all of the files you want and add before / after the file name and extension of a date or something. Right now this only works if the names are identicly and you can replace them. But would love to just add the text before or after the file name
Scenario when this would be used?
Makes the workflow of working with large amout of files easier
Supporting information
No response

Sorry for the late response.
You can use "(?=.*)" to replace before the filename.

And you can use "(.)$" to replace at the end of the string.


[Stumpwm] Cannot load-modules anymore

Describe the bug
Since #208198 PM stumpwm is now up to date but I have an issue in stumpwm module loading. SBCL will return the a package conflict error between packages "CLOSER-COMMON-LISP" and "DYNAMIC-MXINS"
Steps To Reproduce
Steps to reproduce the behavior:

Add windowManager.stumpwm.enable = true; to your /etc/nixos/configuration.nix file
Download stumpwm contrib modules in an appropriate directory git clone https://github.com/stumpwm/stumpwm-contrib.git ~/.stumpwm.d/modules
Create a minimal ~/.config/stumpwm/config file:

(init-load-path "/path/to/home/.stumpwm.d/modules") ;; put the correct path to where you cloned stumpwm contrib
(load-module "battery-portable") ;; this is the line causing an issue

Either reboot using stumpwm or refresh the stumpwm config using reloadrc command (by default keystrokes are: CTRL-t ; loadrc Enter)

Expected behavior
Modules are loaded as expected
Metadata
Results of nix-shell -p nix-info --run "nix-info -m":
 - system: `"x86_64-linux"`
 - host os: `Linux 4.14.313, NixOS, 23.05 (Stoat), 23.05pre478022.d6b863fd9b7`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(charleshd): `"home-manager, nixpkgs-23.05pre476152.2362848adf8"`
 - channels(root): `"nixos-23.05pre478022.d6b863fd9b7, nixpkgs-23.05pre476152.2362848adf8"`
 - nixpkgs: `/home/charleshd/.nix-defexpr/channels/nixpkgs`

Hm, indeed. And maybe for debugging we should provide access to the «library» version of StumpWM…

We could try building using the Makefile instead of save-lisp-and-die. I did, but then there were problems with missing HOME.  Then the problem was that the stumpwm build script seems to want to remember the value of HOME from compile-time on run-time (something like "/build")
How did stumpwm work previously in nixpkgs?

Can you try with this? https://github.com/NixOS/nixpkgs/issues/228980


Fix thread values

Fix issue #648 by overriding the configured profile information with the command-line arguments. The Project object is the last thing that really knows about both CLI arguments and profiles so this seemed like the best place to override things.
The do_nothing sql is a bit funny because pg_sleep returns a void type so I couldn't just select pg_sleep(1). I assume there's a better way to do it.

hahahaha i love it. @beckjake does that test fail if you comment out the change? if so, :shipit: . nice work!

Yeah, the test fails without the fix commit.


Refactored helper function to calculate next meeting date

Leveraged native moment.js methods to calculate the next meeting date.
Updated the cron to work with updated helper.  Cron will now create mtg
issue with title "Monday All-Hands Call: MMM DD YYYY"
Resolves: #14

Note: I don't have time to maintain this, happy to have others do it. Don't wait for my review. If it works it works. Thanks for fixing it. :)

Closing this PR as this repository has been deprecated. All the functionality has been moved to https://github.com/ipfs/sprint-helper.
Thanks for this contribution, it was not fruitless! I'll try to backport it directly to sprint-helper


End of match

The re_match() routine returns the first character of the match.  Is there an easy way to return the last matching character of the string?  I'm trying to copy out only the resulting matching string but can't think of an easy way.
Great library by the way!

I did it by returning text or NULL in matchpattern, it should point to the end of match.
https://github.com/monolifed/tiny-regex-c/blob/master/re.c#L371

The length of the match can now be returned as per #36


Background Threading

@TimOliver
We LOVE to help with any issues or bugs you have!
Questions: If you have questions about how to use Realm, ask on
StackOverflow.
We monitor the realm tag.
Feature Request: Just fill in the first two sections below.
Bugs: To help you as fast as possible with an issue please describe your issue
and the steps you have taken to reproduce it in as many details as possible.
Thanks for helping us help you! :-)
-->
Goals

I want to write to my realm file in a background thread while I'm updating my file as to not hold up my UI when a user logs in and by default from my understanding I will have to also read from the file in the background thread.
I have read the swift docs and searched SO but have been unsuccessful in applying the concepts to my app specifically.
Expected Results

I expect my file to get updated from the network in a background thread while the user navigates the app without interruption
Actual Results

The file is still being written to and read from the main thread and the UI is held up for a substantial amount of time while updating data from the network.
While trying to follow the Realm Swift Docs I was only able to get to the point of crashing with the following error message "xerminating app due to uncaught exception 'RLMException', reason: 'Realm accessed from incorrect thread'"
I was able to get rid of that error add get the app running while realm seems to be executing in the background however I'm not sure if its being done correctly or reading from the updated file.
Steps to Reproduce

Run project. Realm file should be querying the network upon app launch.
Code Sample

sent full Xcode project via email from<EMAIL_ADDRESS>subject line Background Threading
Version of Realm and Tooling

Realm framework version: ?
2.5.1
Realm Object Server version: ?
Xcode version: ?
8.3.2
iOS/OSX version: ?
Dependency manager + version: ?

Thanks for getting in touch with us.
The same-thread Realm requirement is very simple to fulfill. It boils down to this:
If you create an instance of a Realm, or get a Realm object, list, results, or any other Realm type from a Realm, you can use it only on the same thread. If you want to use the same Realm, object, or whatever on another thread, you have to create a new Realm instance on that other thread and re-fetch the object.
Here are some examples of things you shouldn't do:

You get a Realm in some function. Then you call Async to dispatch a block onto another thread, but in that other block you use the same Realm instance.
You have a manager object which has a Realm property. Then you have a method that uses that Realm property, but you call that method from multiple threads.

Here are code samples:
// BAD
func doSomethingWrong() {
  // Assume doSomething is running on the main queue
  let config = // (get a Realm configuration)
  let myRealm = try! Realm(configuration: config)
  let objects = myRealm.objects(MyObject.self)
  // (do more things with myRealm)

  DispatchQueue.global(qos: .background).async {
    let otherObjects = myRealm.objects(YourObject.self)  // this queue is running on a different thread, so you can't use `myRealm` here!
  }
}

// OKAY
func doSomethingRight() {
  // Assume doSomething is running on the main queue
  let config = // (get a Realm configuration)
  let myRealm = try! Realm(configuration: config)
  let objects = myRealm.objects(MyObject.self)
  // (do more things with myRealm)

  DispatchQueue.global(qos: .background).async {
    // You *can* pass Realm configurations between threads, that's okay
    let myBackgroundRealm = try! Realm(configuration: config)
    let otherObjects = myBackgroundRealm.objects(YourObject.self)  // okay, since we opened myBackgroundRealm on the same thread
  }
}

// BAD
class MyRealmManagerWrong {
  // Assume we can get the configuration from somewhere; where it comes from isn't important
  let theRealm = try! Realm(config: getConfig())

  // What if we call this method from any thread other than the one upon which
  // MyRealmManager was instantiated? That'll cause a crash.
  func getMyObjects() -> Results<MyObject> {
    return theRealm.objects(MyObject.self)
  }
}

// OKAY
class MyRealmManagerRight {
  let theConfig: Realm.Configuration = getConfig()

  // This is okay, since we always re-open the Realm and never allow it to escape to another thread.
  func getMyObjects() -> Results<MyObject> {
    // This is cheap, since the framework caches the expensive parts of a Realm once it's been opened at least once
    let theRealm = try! Realm(config: theConfig)
    return theRealm.objects(MyObject.self)
  }
}

If you change your app's code to take these guidelines into account you should be free of any crashing issues due to accessing Realms or related instances from the wrong threads.

Thanks for getting back to me this is really helpful.  Some follow up questions I have are:

In your doSomethingRight() function is the objects constant the same as the objects in myBackgroundRealm.objects?

How does the MyObject object differ from the YourObject object?

I don't need to incorporate the ThreadSafeReference?

Your code sample applies to both reading and writing the realm file, correct?

@austinzheng

To answer your questions:

Yes, the things contained in objects and otherObjects are exactly the same. (There are some subtleties to this, but you won't run into them unless you explicitly go looking for them.)
The sample MyObject and YourObject types are irrelevant. The only point of those lines of code is to show doing something with the Realm: it could be getting objects, starting a write transaction, removing objects from the Realm, etc. If it helps, substitute them mentally with comments saying "do something with this Realm".
Thread safe references are an alternative to the approaches I showed above. For example, in the first block of code instead of opening otherObjects in the async block I could have put the original objects in a thread safe reference and then opened the reference in the block. A thread safe reference is a way to move Realm lists, objects, results, etc between threads safely. I recommend you don't worry about them for now; if you get things working with the techniques I described above you can then decide whether or not it makes sense to use thread safe references.
Yes.

Hope this helps.

Thanks for the extremely comprehensive answers @austinzheng! :)

I've updated a majority of my code but the file is still running on the main thread, so I'm wondering if I also need to send any function that calls realm to the background as well.  See the following code:
class MyRealmManager {
    let realm = try! Realm(configuration: RealmConfig.main.configuration)
}

//From this
class ViewController {
     cached = MyRealmManager()
     override func viewDidLoad() {
         cached.updateAll()
    }
}

//To this
class ViewController {
     cached = MyRealmManager()
     override func viewDidLoad() {
     DispatchQueue.global(qos: .background).async {
         let realm = try! Realm(configuration: RealmConfig.main.configuration)
         cached.updateAll()
       }
    }
}

Hope my follow up question makes sense
@austinzheng

Only the main queue is guaranteed to always execute on the same thread (the main thread). Queues and threads are distinct, and a single queue is usually served by multiple threads. (This means that, even if you open a Realm on a given background queue in a given block, you shouldn't use that same Realm in a different block even if that block is also run on the same queue.)
Hope that helps.

There has to be something I'm missing because Realm appears to be in the background thread (Thread 5) but my UI is still held up while Realm is being updated by the network.  I have attached a file for you to take a look at if you could please and let me know where I'm going wrong.
Thank you
@austinzheng
CacheRealm.swift.zip

@alexcowley unfortunately it's not possible to reproduce your case using just a source file you sent us and without the whole xCode project. It also contains a lot of unrelated code that makes the investigation very hard. However I've noticed some issues in your code that can potentially be related:
func collegeFinder(for name:String)->CollegeData {

    ...

    let backgroundRealm = try! Realm(configuration: RealmConfig.main.configuration) // <-- You create a new Realm instance on a caller thread

    ...

    let college: CollegeData
    if let found = CollegeData.find(realm: backgroundRealm, searchName: name) {
        college = found
    } else {
        college = CollegeData()
        college.name = name
        backgroundRealm.add(college) // <-- You modify realm outside write transaction
    }

    return college // <-- You return `CollegeData` instance from you local `backgroundRealm`
}

later in updateCollege:
func updateCollege(object:PFObject) {
    DispatchQueue.global(qos: .background).async {
        ...
        let backgroundRealm = try! Realm(configuration: RealmConfig.main.configuration) // <-- You create a local realm instance on background queue

        ...

        backgroundRealm.beginWrite() <-- Transaction is started on your local Realm instance
        // Find or create college
        let college = self.collegeFinder(for: collegeName) <-- `college` is fetched or created in a different realm instance from `collegeFinder`

        // Add Image
        if let image = object["image"] as? String {
            college.url = image // <-- You're modifying realm object in another realm's write transaction
            print("Successfully retrieved \(object).")
        }
        ...
    }
}

Hope this info helps you to find your issue. Please review your code or try to create a simplified version that reproduces it.

Thank you for your response @stel I have emailed my zipped project to your help email when I first opened this ticket with a subject line of "Background Threading".
@austinzheng
@TimOliver

@alexcowley did you try to fix the issues I've mentioned in my previous comment?

Try to do this:
//This method will run on background, to maintain your Realm updated, you need to say what are the process.
DispatchQueue.global(qos: .background).sync {

	DispatchQueue(label: "first").sync {
		//Run first method, create first instance of Realm.
	}

	DispatchQueue(label: "second").sync {
		//Run second method, create second instance of Realm.
	}

}

@stel yes I have, I removed the local realm instances but left the realm write transactions as is.  See attached files.
CacheRealm.swift.zip
SecondaryLaunchViewController.swift.zip

You're still getting this exception because you're starting a write transaction for realm created on a different thread. If you go down by the stack trace in debugger you will find the place in your code that causes this issue.

... you cannot have multiple threads sharing the same instances of Realm objects. If multiple threads need to access the same objects they will each need to get their own instances

please learn more in docs: https://realm.io/docs/swift/latest/#threading

@austinzheng
class databaseManager {
func fetchData() ->   person {

    Realm.Configuration.defaultConfiguration = config

    let nill=person()

    if let results =  self.database.objects(person.self).first {

        return results

    }else {

        return nill
    }

}

func saveData(ps: person) ->Bool  {

  try! self.database.write {
        print("fsdfsf",ps)

        self.database.add(ps, update: true)

        }
    return true

}
}
could some one modify this class for singleton object ??

class databaseManager {
     static let shared = databaseManager() // Use databaseManager.shared to get the singleton
func fetchData() ->   person {

    Realm.Configuration.defaultConfiguration = config
    let nill=person()\
    if let results =  self.database.objects(person.self).first {
        return results
    } else {
        return nill
    }
}

func saveData(ps: person) ->Bool  {
  try! self.database.write {
        print("fsdfsf",ps)
        self.database.add(ps, update: true)
        }
    return true
}
}

This probably isn't the place for this question. For questions instead of issues with Realm try posting on StackOverflow.com


Doesn't work with node-redis v4.0.0

I updated node-redis from v3.1.2 to 4.0.0 on my node client and socket.io stopped working. First emit is send, then it crashes.
Node client:
Error: Got an unexpected reply from Redis
    at RedisCommandsQueue._RedisCommandsQueue_shiftWaitingForReply

Socket.io server (redis 3.1.2):
node_modules\notepack.io\lib\decode.js:246
    throw new Error((buffer.length - decoder.offset) + ' trailing bytes');

Running Socket.io server with redis 4.0.0:
node_modules\@socket.io\redis-adapter\dist\index.js:64
        this.subClient.psubscribe(this.channel + "*", onError);
                       ^

TypeError: this.subClient.psubscribe is not a function
    at new RedisAdapter

Support for redis@4 was added in release 7.1.0: https://github.com/socketio/socket.io-redis-adapter/releases/tag/7.1.0


Default build fails because src/public does not exist

Not sure if there are assets missing from the repo or if this copying is supposed to be optional. I can open a PR if I know which

Mmm are you sure the build is failing? Maybe is throwing the "cp: no such file or directory...", as part of the "npm run build" command is copying assets from an optional "src/public//*"** directory. But this should not stop you from building.
For the  "npm run watch-server" command, no assets copy is needed, as it runs directly in Typescript; the thing is when compiling the JS to create a "build" directory, if you are using some assets, they should be copied over to the folder, wherever they are. As an example, I created the file called copyStaticAssets.ts, where the paths of the assets to be copied can be specified. Just for the sake of having something on it, I included 3 typical paths where people include assets, but I didn't store anything there...


No way to change increase zoom amount

Is your feature request related to a problem? Please describe.
Need a prop similar to wheelZoomRatio for zoom on slider button as the default 1 is too much in source
Describe the solution you'd like
prop similar to wheelZoomRatio
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

zoomAmount and zoomStep props add to version 1.1.0


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in f69280f.


Prepare CI for NPEFix unification

Increase maven-repair timeout, as it now includes repairnator-pipeline as a dependency.
Add MAVEN_OPTS do run-with-core.sh script.

Small and green, LGTM. But no new dependency in maven-repair.

Small and green, LGTM. But no new dependency in maven-repair.

Small and green, LGTM. But no new dependency in maven-repair.

Well, it was green because I removed ${MAVEN_OPTS} from the script. There's a test failing because of that I think.
The dependency will come in the maven-repair PR.

Small and green, LGTM. But no new dependency in maven-repair.

Well, it was green because I removed ${MAVEN_OPTS} from the script. There's a test failing because of that I think.
The dependency will come in the maven-repair PR.

Ack, updated description accordingly

Ack, updated description accordingly

Ack, updated description accordingly

OK! What I meant by that was that the script now includes installing repairnator-pipeline.

Ack, updated description accordingly

OK! What I meant by that was that the script now includes installing repairnator-pipeline.

I think there is something weird going on with some tests.
The 9394dff failed but 7cf6f5b passed, and they are exactly the same.

I think there is something weird going on with some tests.
The 9394dff failed but 7cf6f5b passed, and they are exactly the same.

The 9394dff failed but 7cf6f5b passed, and they are exactly the same.

This suggest we have a flaky test. Is that for one single test?

The 9394dff failed but 7cf6f5b passed, and they are exactly the same.

This suggest we have a flaky test. Is that for one single test?

Should be ready. Flaky test hasn't failed again.

Should be ready. Flaky test hasn't failed again.

Nice, thanks @andre15silva.
We can now rebase the subsequent PRs and proceed.

Nice, thanks @andre15silva.
We can now rebase the subsequent PRs and proceed.


Webp vs. JPEG sizes?

I am a newbie for image processing.  To my understanding, webp is supposed to have smaller size than jpeg.  However, from my experiment, this is not the case after converting to both format using Sharp.
I am actually getting smaller sizes when converting to JPEG.
Here is my configurations:
sharp().resize({ width: 300 }).jpeg({ quality: 100 })
sharp().resize({ width: 300 }).webp({ lossless: true })

Original PNG size is 937 KB.
Converted JPEG is 408.7 KB
Converted WEBP is 584.9 KB
For resized smaller images are the same result.
Am I missing something or this is expected?

Hi, JPEG is always lossy, even at a "quality" of 100.
A fairer comparison might be:
  sharp().resize({ width: 300 }).jpeg({ quality: 100 })
- sharp().resize({ width: 300 }).webp({ lossless: true })
+ sharp().resize({ width: 300 }).webp({ quality: 100 })

lovell,
Thanks a lot.  After trying
 sharp().resize({ width: 300 }).webp({ quality: 100 })

The size was reduced to 217.3 LB.  This indeed is a great reduction!


operator is not triggered on changes of resource

Bug Report
operator is not triggered on changes of resource.
What did you do?
when i run my operator in an k8s cluster, it receives and acts on a resource right-away. But when i then change something on the resource, nothing happens. Only when i then manually restart (delete pod) the operator, it is triggered once. This happens only when i run the operator inside k8s.
What did you expect to see?
the operator should react on every change of the operated resource
What did you see instead? Under which circumstances?
when testing the operator against minikube, all is fine. when running it in our k8s cluster (which also uses RBAC) then things only work on start once. the operator doesnt seem to be notified for new events. What i DO see though is this warning during startup:
{"@timestamp":"2022-10-24T09:39:08.794Z","@version":"1","level":"WARN","severity":"WARN","logger_name":"io.fabric8.kubernetes.client.dsl.internal.WatchConnectionManager","thread":"OkHttp https://<IP_ADDRESS>/...","message":"Exec Failure: HTTP 403, Status: 403 - Forbidden","application":"my-crd-operator"}
│ {"@timestamp":"2022-10-24T09:39:08.795Z","@version":"1","level":"DEBUG","severity":"DEBUG","logger_name":"io.fabric8.kubernetes.client.dsl.internal.AbstractWatchManager","thread":"OkHttp https://<IP_ADDRESS>/...","message":"Force closing the watch io.fabric8.kubernetes.client.dsl.internal.WatchConnectionManager@3acd530d","application":"my-crd-operator"}
│ {"@timestamp":"2022-10-24T09:39:08.795Z","@version":"1","level":"DEBUG","severity":"DEBUG","logger_name":"io.fabric8.kubernetes.client.informers.impl.cache.Reflector","thread":"OkHttp https://<IP_ADDRESS>/...","message":"Watch gracefully closed for url.de/v1/my-crd","application":"my-crd-operator"}

Environment
Kubernetes cluster type:
rancher
$ Mention java-operator-sdk version from pom.xml file
      <groupId>io.javaoperatorsdk</groupId>
      <artifactId>operator-framework-spring-boot-starter</artifactId>
      <version>3.2.0</version>

$ java -version
17
$ kubectl version
Client Version: v1.25.3
Kustomize Version: v4.5.7
Server Version: v1.20.8
Possible Solution
maybe some additional roleBindings necessary?
Additional context

@warmuuh could you share some code to reproduce this?
This is highly unlikely, see our End To End tests, which are deployed to a cluster and are receiving the events. Maybe could you try also those?

yes...just learned about the watch permission.thx


supporting other linkers

Unresolved questions
1. How to expose this feature?
Maybe a command line interface that looks like this:
flip-link $actual_linker_arguments --linker=arm-none-eabi-gcc

Where --linker may appear in any position. flip-link will use arm-none-eabi-gcc as the linker.
This way the user species flip-link in their config.toml like this:
# .cargo/config.toml
rustflags = [
    "-C", "linker=flip-link",
    "-C", "link-arg=--linker=arm-none-eabi-gcc", # <- may appear in any position
    "-C", "linker-flavor=gcc", # <- not passed to the linker
    # ..
]

2. How to determine the linker flavor?
flip-link passes extra linker arguments to the underlying linker.
It currently assumes the linker flavor is "ld". If you want to use gcc those extra linker arguments need to be prefixed by -Wl, (iirc), i.e. -Wl,--defsym=something instead of just --defsym=something.
flip-link will not see the -C linker-flavor passed to rustc. Should the user also pass a --linker-flavor argument to flip-link?

Just pinging this issue as I believe it would be a very useful feature ! Thanks :)

https://app.bountysource.com/issues/110784469-supporting-other-linkers

Any updates on this? Or support for windows-gnu toolchain variant?

No update so far. If you want to give it a shot I am happy to support you.


Fluid on Pubs

After we're done with camera-ready, of course!

@AmberLJC @Aetf Ping!

Just create new pull request


[Firestore] Type definitions are in conflict with @google-cloud/firestore v 3.0.0

3.0.0 was only released yesterday, so not unexpected but still
node_modules/@google-cloud/firestore/types/firestore.d.ts:23:1 - error TS6200: Definitions of the following identifiers conflict with those in another file: DocumentData, UpdateData, Firestore, GeoPoint, Transaction, WriteBatch, WriteResult, DocumentReference, DocumentSnapshot, QueryDocumentSnapshot, OrderByDirection, WhereFilterOp, Query, QuerySnapshot, DocumentChangeType, CollectionReference, FieldValue, FieldPath, Timestamp, v1beta1, v1, FirebaseFirestore

23 declare namespace FirebaseFirestore {
   ~~~~~~~

  node_modules/firebase-admin/node_modules/@google-cloud/firestore/types/firestore.d.ts:23:1
    23 declare namespace FirebaseFirestore {
       ~~~~~~~
    Conflicts are in this file.

Operating System version: Mac OS X 10.14.4
Firebase SDK version: 7.6.0
Firebase Product:  Firestore
Node.js version: 11.9.0
NPM version: 6.13.4

@arpit Thanks for the report. You seem to be using two different versions of Firestore (one from node_modules/@google-cloud/firestore and one from node_modules/firebase-admin/node_modules/@google-cloud/firestore). As shown in your errors, this breaks the TypeScript compile.
We haven't yet updated Firebase Admin to include Firestore 3.0. If you want to avoid this problem, you can install Firebase Admin from this branch:
"dependencies": {
   "firebase-admin": "git+https://github.com/firebase/firebase-admin-node.git#firestore-30"
}

@schmidt-sebastian We are currently using firebase-admin 8.10.0 and facing this issue. Does it mean firestore 3.0 is not yet into the admin module?

Firebase Admin Node depends on 3.x Firestore: https://github.com/firebase/firebase-admin-node/blob/master/package.json#L64

So any reason why its still displaying the confict?

I have seen cases like this before when two version of Firestore were included via NPM's dependency resolution. This doesn't necessarily have to be related to Firebase Admin.

Ok, Here is our scenario.
We installed both firebase-admin as well as @google-cloud/firestore in our project.
The reason why we had @google-cloud/firestore is

To connect to local firestore emulator during our tests. (emulators:exec is buggy to use)
We need to connect to a different firestore project from the current project.

What we have observed is nested node-modules folder with in the node-modules/firebase-admin has @google-cloud/firestore types too leading to the issue. We see it a general use case too as its quite common to use @google-cloud/firestore package.
This issue didn't happen earlier but only on upgrading to latest firebase-admin.
To solve it temporarily, we just deleted the types file listed in firebase-admin to avoid the conflicting error. But we see it would be great if there is a right solution.

You should be able to use @google-cloud/firestore and firebase-admin together if they point to the same version. NPM will not duplicate the package if the version can be shared.

Seeing this same error now too.
Currently have the following packages:
    "firebase-admin": "^8.12.1",
    "@google-cloud/firestore": "^3.8.0",

@asithade thank you for the workaround.

I'm facing the same issue with
"@google-cloud/pubsub": "^2.14.0",
"firebase-admin": "^9.9.0",

Is there any fix / workaround except skipLibCheck?


Convert xlf back to resx/xaml following loc handback

Customer scenario
Some strings are still not translated as they were added after last handback
Bugs this fixes:
VSO 370232
Workarounds, if any
None
Risk
Low, small number of new strings are now translated.
Performance impact
None
Is this a regression from a previous update?
No. The strings are new.
Root cause analysis:
We didn't miss anything. Strings were added after last handback.
How was the bug found?
N/A
@srivatsn @dotnet/project-system

Tagging @MattGertz for RTM approval.

Approved to merge (via 370315)


Update model interface for optimal filter

As planned in #136
Split update_particles! to update_particle_dynamics! and update_particle_noise!
DONE: Now both new functions return the observations which may not be ideal. Would it be better to separate get_particle_observations into a third function?

I pushed this last week https://github.com/Team-RADDISH/ParticleDA.jl/commit/28feb5234a87e8c3514e6a4ea741758bbc77cbe2 but forgot to open the PR :facepalm:  Yours is better, it has also tests! :smiley:

I pushed this last week https://github.com/Team-RADDISH/ParticleDA.jl/commit/28feb5234a87e8c3514e6a4ea741758bbc77cbe2 but forgot to open the PR :facepalm:  Yours is better, it has also tests! :smiley:

That's exactly the thing in my TODO! Can we merge it into this PR?

That's exactly the thing in my TODO! Can we merge it into this PR?

Yes we can! 💪

Yes we can! 💪

I'll mark this as ready for review. The other thing to do in #136 is expose more fields from the model_data struct but we can do that in another PR.

I'll mark this as ready for review. The other thing to do in #136 is expose more fields from the model_data struct but we can do that in another PR.


Feature: hook for when environment changes

I think it would be kind of nice to have a hook that is called only when a direnv environment changes. Sometimes services need to be updated/restarted for a buffer when the context shifts.  The two services that I've run into are the anaconda-mode server for Python development, and the dante-mode server for Haskell development.
If we get this hook, it would be nice if it passed along enough information to know exactly which environment variables changed.  One way of doing it would be to pass in both the old and new environment map.
I just thought of this idea, so I don't have code for a pull request yet.  But I also thought it might be good to talk through too before writing code.

what exactly is your use case? i think you want to detect "i changed between projects" events?  anything on top of that?  because maybe there's a simpler way that does not involve diffing old and new environments and so on?

Hi @wbolster, just getting to this.  Thanks for the patience.
I completely get the idea of wanting to keep the design simple on your end.  I still think my request has merit, but let me explain my thoughts with a concrete example:
Right now, for Haskell, I'm using Dante, and for Python I'm using anaconda-mode.  These tools both run background services based on command-line tools assumed to be on the PATH.  For me, Dante runs a Cabal REPL instance, based on tools defined in Nix.  And anaconda-mode runs some Python server based on the Python interpreter the project is built against, and I use Nix to manage this too.
Getting all the dependencies and versions right for these servers per-project is exactly what Direnv is great for, so in that regard, direnv-mode works out really nicely.  But these servers are kind of like live caches that can get invalidated.  What if I change my Nix expression?  Or my dependency list?  Then I have to restart these servers.  But before that, I need to do a direnv-update-environment call.  Furthermore, I'm missing out on automated notification of when my environment has changed.
So right now, I'm making proxy commands for restarting services that make sure call direnv-update-environment, but also these restarts have to be called manually when I notice things are out of sync.
If I had a hook that would trigger when the environment changed in a buffer, I wouldn't have to call my service restarts manually, and I would also not need to call direnv-update-environment, because my hook would be automatically called after the environment update.
It just makes my life so much cleaner as a consumer.  What do you think?

what information would be needed in such a hook? both the complete old and new environments?

Hello! I have a similar use-case.
I use Nix (and nix-shell) to provide development environments for my Elixir projects. I also use alchemist. To be able to jump to code in Erlang/Elixir sources (you can jump to definitions in your project's code and its dependencies by default), you need to tell alchemist where are the Erlang/Elixir sources are located.
In shell.nix I expose the source directories of Erlang and Elixir as environment variables to the Nix shell and to Emacs (via direnv):
{ pkgs ? import ./pkgs.nix # pinned nixpkgs
}:
let
  inherit (pkgs) beam;

  erlang = pkgs.erlangR21.override {
    enableHipe = false;
    wxSupport = true;
  };

  elixir = (beam.packagesWith erlang).elixir_1_8;
in pkgs.mkShell {
  buildInputs = with pkgs; [
    elixir
    erlang
    pgcli
  ];

  ERL_AFLAGS = "-kernel shell_history enabled";

  ERLANG_SRC = "${erlang.src}";
  ELIXIR_SRC = "${elixir.src}";
}

These values are not static (i.e., they are not available system-wide) and may differ from project to project.
I tried to set alchemist-goto-{erlang|elixir}-source-dir inside an elixir-mode hook but that didn't work because the hook runs before direnv-update-directory-environment, so I came up with this workaround:
  (defun set-erlang/elixir-source (&rest args)
    (when (eq major-mode 'elixir-mode)
        (when-let* ((erlang-src (getenv "ERLANG_SRC")))
          (setq-local alchemist-goto-erlang-source-dir erlang-src))
        (when-let* ((elixir-src (getenv "ELIXIR_SRC")))
          (setq-local alchemist-goto-elixir-source-dir elixir-src))))

  (advice-add 'direnv-update-directory-environment :after #'set-erlang/elixir-source)

This workaround feels kinda hackish to me, so I thought that having a hook that runs after direnv-update-... would be nice, then I stumbled upon this request.


Added werkzeug 2.2.x to requirements

For Contrast Python Developers Only:

[x] I've created a PR to update the vulnpy commit

[ ] We do not want to update to this PR's top commit.

I'm not a 100% sure this is the best solution but, we need to have Werkzeug 2.2.* for Flask 2.2.* and we can't really do it because vulnpy is expecting Werkzeug 2.1.* or lower

I'm not a 100% sure this is the best solution but we need Werkzeug 2.2 or higher for Flask 2.2 to work

Disregard that for now, I was thinking this will break the lower Flask version tests in the agent and it did

So this should be ready for review, the reason why I removed the werkzeug requirement is that for Flask 2.2.* we need Werkzeug 2.2.* and this breaks the agent requirements


Fix a typo in this doc

Closes #280

Docs Build status updates of commit 1894bd8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

TerminalDocs/customize-settings/appearance.md
:white_check_mark:Succeeded
View

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

Thanks @zadjii-msft !


Crypto implementation for shared model

What is this pull request?
Implementation for shared model crypto.
Why do you implement it? Why do we need this pull request?
Shared model does not contain crypto implementation now, only interfaces. Also this is an important step to crypto bindings.
Details/Features

Move all crypto and hash code to shared model
Add BlobImpl class as implementation of Blob interface
Inherit BlobImpl instead of Blob for public and private keys, Signed, Hash
Replace Keypair interface with implementation
Inherit Keypair from Primitive; add makeOldModel
Introduce new Seed object for storing seeds
Add Signer and Verifier as low-level crypto wrappers
Add CryptoProvider as a wrapper for all crypto operations
Add HashProvider as a wrapper for all low-level hashing operations
Use real hashes in proto Amount and AddAssetQuantity

Codecov Report

Merging #699 into feature/shared_model will decrease coverage by <.01%.
The diff coverage is 76.47%.

@@                   Coverage Diff                    @@
##           feature/shared_model     #699      +/-   ##
========================================================
- Coverage                 43.49%   43.48%   -0.01%
========================================================
  Files                       298      298
  Lines                     12022    12020       -2
  Branches                   6599     6599
========================================================
- Hits                       5229     5227       -2
  Misses                      721      721
  Partials                   6072     6072

Impacted Files
Coverage Δ

...rohad/model/converters/json_query_factory_test.cpp
5.49% <ø> (ø)
:arrow_up:

irohad/model/converters/impl/pb_block_factory.cpp
43.24% <ø> (ø)
:arrow_up:

...le/irohad/model/operators/model_operators_test.cpp
26.94% <ø> (ø)
:arrow_up:

...ryptography/ed25519_sha3_impl/internal/impl/sign.c
100% <ø> (ø)

...model/converters/impl/json_transaction_factory.cpp
46.15% <ø> (ø)
:arrow_up:

...ryptography/ed25519_sha3_impl/internal/impl/sha3.c
76.92% <ø> (ø)

.../irohad/consensus/yac/yac_crypto_provider_test.cpp
11.76% <ø> (ø)
:arrow_up:

irohad/model/generators/impl/query_generator.cpp
59.32% <ø> (ø)
:arrow_up:

irohad/model/impl/query_execution.cpp
35.48% <ø> (ø)
:arrow_up:

irohad/model/generators/impl/block_generator.cpp
54.54% <ø> (ø)
:arrow_up:

... and 37 more

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 41f1d86...8a685d6. Read the comment docs.

shared_model/cryptography/blob_impl.hpp:35

what is this function? move outside (make free function)


Chore/api auth and translation service methods

This PR adds the Auther and the ServiceMethodHandler and ServiceMethod classes. It still needs more work as the type validation is now unnecessary. I also think the Auther can be made clearer. The idea of the Auther is the typing:
Auther<Static, Dynamic> = Static -> Dynamic -> Session -> AuthResult

This looks good after the conversations are answered / fixed :))

Well I have made it slightly better now I think. (More consistent). I think this pattern is quite good at this point

There are some oddities with how forms submit data. For example a checkbox submits 'on'. We may let the validation In the service worker take in 'on' (or a bool for calls from the api) and convert it to a bool there directly. I also want to say that we should standardise some of the validation I think. Like now people write .max(number, str). This error string should be standard. So I think. in general we will need to rethink validation. We could make some custom validation stuff wrapping zod like val.checkboxOrBool().
There will always be 1:1 between handlers and methods. The handlers are not meant to be called directly, they are only split for code splitting purposes.


fix(codegen): lower the level of precedence in TaggedTemplateExpression

Current dependencies on/for this PR:

main

PR #2391   👈

This stack of pull requests is managed by Graphite.


Differences between relations available on Wikidata and the LOTUS web interface

Hello,
I may have noticed some inconsistencies between the data available by SPARQL request on Wikidata and those available on the LOTUS web interface, which I understood as a mirror of what's in Wikidata.
For instance, N(6)-(1,2-dicarboxyethyl)-AMP (Q2823236) in found produced by Valsa sordida (Q7912606) and this relation is referenced by the publication Phosphoproteomic and Metabolomic Profiling Uncovers the Roles of CcPmk1 in the Pathogenicity of Cytospora chrysosperma (https://doi.org/10.1128/SPECTRUM.00176-22). However, this relation does not seems to be available on the LOTUS interface.
Is this just an update issue ?
Thanks for your help.

Hi again @maximeDelmas,
As suggested in your other issue, please have a look at #59, you will probably find the answer there. If not, simply ping us 😊

Sorry, I think now it's clear thx a lot !


What's the difference between spring-cloud-alibaba and spring-cloud-alicloud?

Is their any top level design on such two projects?

Sping-Cloud-Alibaba includes open source components, and Sping-Cloud-Alicloud includes commercial components.

@xiaolongzuo That makes sense. Thanks!

resolved


Add support for automatically creating queues/topics in Azure ServiceBus support

Look into possibly flipping around the whole credentials management for ServiceBus. Management libraries are looking for service credentials, while today we have ConnectionString going into the queue client. Should only need one of them for both purposes.


Add .introspectSplitViewController() on iOS

This adds introspection of the UISplitViewController on iOS and thereby fixes #70.

@JannThomas Just a quick note: we should update the readme at this point to reflect the new methods and cleanup the "Cannot implement" section since it's outdated.

@ldiqual yes, we already talked about that today actually. I will take a look at that this week.


Changing CURATE header

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (0e81f65) 84.97% compared to head (7e93f9b) 85.03%.
Report is 10 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
+ Coverage   84.97%   85.03%   +0.06%
==========================================
  Files          13       13
  Lines        3101     3101
==========================================
+ Hits         2635     2637       +2
+ Misses        466      464       -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update README.md

added
https://forthebadge.com/images/badges/built-by-codebabes.svg

@qwe123coder , thank you so much for your contribution, though is it possible for you to perhaps change the badge, to  a more suitable one, like:
[![forthebadge](https://forthebadge.com/images/badges/built-by-developers.svg)](https://forthebadge.com)
thank you so much! :)

@qwe123coder , thank you so much for your contribution, though is it possible for you to perhaps change the badge, to a more suitable one, for this repo like:
[![forthebadge](https://forthebadge.com/images/badges/built-by-developers.svg)](https://forthebadge.com)
opps i put the worng on


[databroker] Implement kuksa.val.v1 service

Add kuksa.val.v1 service implementation to databroker.
Add SetDatapoints to sdv.databroker.v1.Broker interface.

Hi @argerus
I did a smoke test to verify that there are no regressions using dbc-feeder and it worked well.
What we lack - as I see it - is to update the README. A few ideas:

Extend the Interface section
Mention that databroker support 2 interfaces, the legacy SDV Databroker Proto Interface and the new unified Kuksa.VAL Proto interface
Mention that both are supported on the same port, a client can use whichever it like
Mention that the databroker-cli (still) uses the legacy API (if I understand it correctly, otherwise state that it use the new API)
Ass a summary of the new methods in https://github.com/eclipse/kuksa.val/blob/master/kuksa_databroker/proto/sdv/databroker/v1/broker.proto, similar to what we already have for the legacy API

@romainletendart @int0x27 please review as well!


NEXT-10849 - Allow the selection of a default language when installing Shopware using the cli.

Description
Currently you will not be able to specify a default system language, if you install Shopware6 using the cli. The feature is already present in the Shopware6 installer ui and should be transfered to the .psh.phar install or better the .psh.phar init command.
Possible Solution
Have the .psh.phar init command accept an environment variable with a default locale for the shop.

In the last boost day there is already some code for on https://github.com/wexo/platform/tree/feature/NEXT-8602-change-default-language based on the pull request https://github.com/shopware/production/pull/24
The latest changes there need a good testing an some info on the administration. As a language change is affecting everything a worker is needed to change all of it and this is not displayed in the administration yet.

As discussed with @pantrtxp this will already be fixed by shopware/production#24. 👍


Pytest discovery can't add two tests subdirectories

Environment data

VS Code version: 1.61.2
Extension version: v2021.10.1336267007
OS and version: Fedora 34, Linux x64 5.14.12-200.fc34.x86_64
Python version: Python 3.9.4
Type of virtual environment used: Fedora Toolbox (podman) F33 container
Relevant/affected Python packages and their versions: pytest 6.0.2
Value of the python.languageServer setting: default

Expected behaviour
I should be able to discover tests in several directories by adding each directory as a separate argument to pytest, e.g.
"python.testing.pytestArgs": [
  "tests/local",
  "tests/unit"
]

I don't want to discover all tests in tests/* as this causes pytest discovery to run into an error, some subdirectories in tests/ contain tests that require pytest plugins or special setup to run in CI, but everything in tests/local and tests/unit should be runnable locally.
Actual behaviour
Instead of invoking pytest once, with both args, to discover all tests in both dirs, the extension seems to invoke pytest twice, once with each arg, and then runs into an error when adding the discovered tests to the testing sidebar pane. I can see the error in the Python Output log (below) and in the testing sidebar pane I can only see tests in the second directory, tests/unit, and nothing from the first dir tests/local. I am guessing the error "Multiple raw data nodes had the same id" relates to the data backing the tree view in the sidebar.
Steps to reproduce:

Create a Python project with tests in two subdirs of tests/
Add both dirs as args to python.testing.pytestArgs
Enable testing using pytest and run test discovery
Only the second dir / arg will be shown in the Testing sidebar.

Logs

Output for Python in the Output panel

> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2021.10.1336267007/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir ~/Source/myproject -s --cache-clear tests/local
cwd: ~/Source/myproject
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2021.10.1336267007/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir ~/Source/myproject -s --cache-clear tests/unit
cwd: ~/Source/myproject
Error 2021-10-21 10:50:14: Multiple (2) raw data nodes had the same id: ./tests

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


Dev

Hi @mtringuyen, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Seems to be an accidental PR, so closing.


Running npm install Error

I downloaded the git repo ran npm install I get multiple errors related to git cloning. Permission denied Make sure You have correct access rights

I believe this is probably an issue with your npm/git setup. What repository do you not have access to specifically? It's possible it could be coding-standards but this is publicly accessible, your setup just might not like ssh. I will be releasing v2 soon which has a much easier development process.

I look forward to that. :)

Fixed in v2.0.0 (I hope!).


Where are the types for rnmapbox? E.G: "FeatureCollection" types?

Environment

Dev OS: [e.g. OSX 11.0.1, Win10]
@rnmapbox/maps version: [eg. 10.0.15]
React Native version: [eg. 0.72.6]
Expo version: [eg. 49.0.0]

Steps to reproduce

react-native init sample --version<EMAIL_ADDRESS>cd sample
npm install rnmapbox/maps#main --save
# or released version `npm install<EMAIL_ADDRESS>--save`
react-native run-android

I cant find the types for MapBox. Where are they? Do I have to install @types/mapbox-gl?
import { FeatureCollection } from "@rnmapbox/maps"; // cant find the module here.
I have latest typescript version.

@eybel I'm not sure we use FeatureCollection type in our interface.
Please use
import type { FeatureCollection } from '@turf/helpers';

@eybel I'm not sure we use FeatureCollection type in our interface.
Please use
import type { FeatureCollection } from '@turf/helpers';

I havent used that, is an external package?
There is this alternative, but I am not sure whats the best option.
import { FeatureCollection } from "geojson";

I'm pretty sure turfjs and geojson type definitions are compatible.

'@turf/helpers'

Alright then, thank you!


[Offloading] Support remote API Server access

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #(issue)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[ ] Test A
[ ] Test B

/build

/merge


Dependency optimization

This is an attempt at partially resolving https://github.com/gruntwork-io/terragrunt/issues/1103
I thought if there was a way we can optimize the remote state fetching to avoid recursively parsing all the dependencies, and realized that if the config follows a few conventions, this is possible.

The remote state is managed using remote_state blocks.
The remote_state block is in generate mode (the generate attribute is set).
The remote_state block itself does not depend on any dependency outputs (locals and include are ok).

When these conditions are met, we are able to generate a terraform module that only contains the remote state backend configuration, which we can use to run terraform output to get the recorded outputs from the state without retrieving dependencies. You don't even need the full module source to do this!
Note that supporting arbitrary terragrunt configuration (e.g. those managing backends with generate) is difficult as there won't be a stable API we could define to account for all the cases. For these configurations, we will have to fall back to the old method until we are ready to relax these constraints further.

Note that supporting arbitrary terragrunt configuration (e.g. those managing backends with generate) is difficult as there won't be a stable API we could define to account for all the cases.
I'm probably missing something obvious. Can you state it differently?

What I mean is that in order to support a generic solution that works for any terragrunt configuration, we need to implement some complex parsing mechanism that supports just-in-time lookups to ensure that we only get the information that is absolutely necessary.
This is because we don't have a stable API for managing remote state blocks in terragrunt: there are multiple ways one can do this between:

Using remote_state block.
Using generate block to generate the backend config.
Using extra_arguments to feed the specific backend config to init calls.
Using dependency references in generate blocks to output backend config blocks for buckets managed by terraform.

We won't be able to replicate the optimizations here for each of these use cases, until we are ready to start enforcing stricter constraints on the config similar to what we stated here.
E.g., the next one I see being possible is if we introduce a way to flag generate blocks that are used to generate backend config (with a special attribute like is_state_block = true).

Thanks for review! I'm going to merge this in now and release so we can start benefitting!


[iOS 13] Context Menus

See https://developer.apple.com/design/human-interface-guidelines/ios/controls/context-menus/
Similar to "peek and pop", which Flutter does not yet support, but doesn't use 3D touch.

See this WWDC video around 39:00 for more.

Hi @justinmc. I'm trying to implement this delegate method:
- (void)tableView:(UITableView *)tableView willCommitMenuWithAnimator:(id<UIContextMenuInteractionCommitAnimating>);
and I want to navigate to the next view (which is a view controller showing the currently selected image) after tapping that peeking view, just like the behavior in Photos on iOS13.
However this method doesn't not have a indexPath, so I don't know how to let it know which is the current selected image.
Do you know if there is any way to do it?
Thanks!

@TLiu2121999 I don't fully follow, are you attempting to make an implementation of context menus in Flutter?

Some gifs in light and dark modes for reference.

A horizontal screenshot from the video.

The current context menu will always move to the center between the screens, so it is difficult to use it in lists or other places.
like:

@YeungKC Could you open a separate issue with reproduction steps?

@justinmc #60334 OK~


mobile and webgpu: trigger redraw request when needed and improve window creation

Objective

Since #11227, Bevy doesn't work on mobile anymore. Windows are not created.

Solution

Create initial window on mobile after the initial Resume event. macOS is included because it's excluded from the other initial window creation and I didn't want it to feel alone. Also, it makes sense. this is needed for Android
https://github.com/bevyengine/bevy/blob/cfcb6885e3b475a93ec0fe7e88023ac0f354bbbf/crates/bevy_winit/src/lib.rs#L152
request redraw during plugin initialisation (needed for WebGPU)
request redraw when receiving AboutToWait instead of at the end of the event handler. request to redraw during a RedrawRequested event are ignored on iOS

But I see window creation on the Event::NewEvents(StartCause::Init) for ios and macos. (lines 367 to 391) Are those now redundant? If they don't run, we should probably remove those lines.

Right, we should!
The first commit just fixed Android, and worked on iOS due to a cached build artefact with another change.
I added a correct fix for iOS (don't request redraw during a RedrawRequested event) and fro WebGPU which was also broken (request redraw during plugin initialisation).


Create Account Fails on CLEOS

./cleos create account eosio dan EOS6MRyAjQq8ud7hVNYcfnVPJqcVpscN5So8BhtHuGYqET5GDW5CV EOS6MRyAjQq8ud7hVNYcfnVPJqcVpscN5So8BhtHuGYqET5GDW5CV
2468798ms thread-0   main.cpp:1158                 main                 ] Failed with error: Assert Exception (10)
itr != structs.end(): Unknown struct
Failed to serialize type

Setting the bios contract first will fix this.
The command succeeds, but then fails when attempting to pretty print.
We should probably catch ABI issues and dump the JSON if the pretty-print fails for some reason.

This failed to fix the issue.  I have a fix that fixes this for all problems related to cleos before any system contract is set.

Creating account without setting the bios contract beforehand now succeeds.


Have you ever tried to run your code?

I'm trying to clean this mess up in your code, but I couldn't!
Have you honestly at least one time tried to run your code? Nothing is runnable.
all docker-compose files stuck in loads of exceptions!
No offence, but you should be responsible with what you are sharing. your book is quite good, but the code is a real mess.
We cannot understand anything when we can not run the code.

@mi2shiii Sorry for the late reply. Are you still having problems? Everything should run. Please email me directly to solve the problems you are having.
Sorry for the inconvenience.


Disable cloudflare gate

@Starz0r
In the mean time you could try to reach the site through https://starz0r.github.io/AreWeAntiCheatYet

Cloudflare Gate isn't enabled on the site, nor is Cloudflare currently used in DNS Propagation (this isn't even what Gate looks like…). You might have a third-party proxy or similar that's causing this message to trigger. I would check your home connection and make sure there isn't any bad actors remotely using a device as a botnet.

You are missinformed and my connection is fine and monitored in real time.
CF is using this new UI since +- 2 months back and you are not getting it because they are able to fingerprint you. To disable it -> Turn off the DDoS Protection.
If you are concerned about DDoS then set fail2ban (instead of this MITM).

I'd rather not go back to managing draconian applications like fail2ban to have a basic functioning setup. Not that fail2ban is even the correct solution here, since that's for intrusion prevention, not stopping someone from sending a mass amount of connections to my server. That's why DDoS protection from providers like Cloudflare are very important, nonobserving fingerprint implications.
Regardless, our server provider seems to have Gate automatically enabled, without exposing any configuration to change this. You “should” be able to reach the site depending on a Round Robin configuration to the GitHub Pages hosted version. Though, I'm unsure if Gate would catch you again before reaching it, though I'd assume you should.

What Cloudflare is providing is global mass surveillance that you are helping to be part of with the CL-Gate as it only lets through users that it can fingerprint and track across the internet and bullies into submission those whom it can't..
Use different provider or if it's hosted on GitHub then just set up appropriate redirection on your domain.
See https://developer.mozilla.org/en-US/docs/Web/HTTP/Redirections
You can also do silent redirection in e.g. nginx using the HTTP::header replace Host that once the domain is loaded it changes on the github page without changing the URL in the search bar.

What Cloudflare is providing is global mass surveillance that you are helping to be part of with the CL-Gate as it only lets through users that it can fingerprint and track across the internet and bullies into submission those whom it can't..

Please abstain from these baseless accusations. Cloudflare, as far as we've seen, isn't in the business of selling user data, and has no reason to do so as they are far more profitable even without doing so. Saying this, as if it were some confirmed truth, is just inflammatory, and harms your credibility.
I consider not being able to reach the site without JavaScript & Cookies enabled a bug, so I will personally look into this further to see if I can get our provider to disable this. Apologies for the false starts on this, and I will see that this eventually gets solved.

Please abstain from these baseless accusations. Cloudflare, as far as we've seen, isn't in the business of selling user data, and has no reason to do so as they are far more profitable even without doing so. -- @Starz0r

Cloudflare is well known global surveillance company[2][3][4] who has no ethics and absolute disregard for user's privacy in selling user data[1][7] including sharing such data with the russian government during the invasion of ukraine[8] and other invasive government agencies[5][6].

I consider not being able to reach the site without JavaScript & Cookies enabled a bug, so I will personally look into this further to see if I can get our provider to disable this. -- @Starz0r

Your website doesn't allow visit without fingerprinting and tracking across the internet, the javascript and cookies are just the tip of the iceberg -> Disable Cloudflare's "DDoS protection" (cloudflare gate) to address this.
References:

https://edit.tosdr.org/points/12652
https://edit.tosdr.org/points/7101
https://edit.tosdr.org/points/23283
Security and privacy issues documented on wiki https://en.wikipedia.org/wiki/Cloudflare#Security_and_privacy_issues
https://www.cnet.com/news/privacy/at-t-lets-nsa-hide-and-surveil-in-plain-sight-the-intercept-reports/
https://en.wikipedia.org/wiki/PRISM
https://edit.tosdr.org/points/12637
Reaction to war in ukraine by cloudflare https://en.wikipedia.org/wiki/Cloudflare#Reaction_to_the_Russian_invasion_of_Ukraine

References:

edit.tosdr.org/points/12652
edit.tosdr.org/points/7101
edit.tosdr.org/points/23283
Security and privacy issues documented on wiki en.wikipedia.org/wiki/Cloudflare#Security_and_privacy_issues
cnet.com/news/privacy/at-t-lets-nsa-hide-and-surveil-in-plain-sight-the-intercept-reports
en.wikipedia.org/wiki/PRISM
edit.tosdr.org/points/12637
Reaction to war in ukraine by cloudflare en.wikipedia.org/wiki/Cloudflare#Reaction_to_the_Russian_invasion_of_Ukraine

I can't help myself but I find this really funny...
1, 2, 3, 7) are about tracking users... but users of Cloudflare - the website owners themselves on the Cloudflare website. It doesn't apply to users going through Cloudflare DDoS protections to target websites.
4) mentions compromise of CCTV (which has no impact on users using Cloudflare as proxy) and again only leaks about customers - owners of websites using Cloudflare and emails of complaints - again nothing about users actually being proxied, only those who interact with Cloudflare directly.
5, 6) These "Reference"s don't even mention Cloudflare at all... and every single ISP is required to provide data about customers if police/any other agency comes with warrant. That's simply law.
8)

including sharing such data with the russian government during the invasion of ukraine

Please... read that section again. The section says that Cloudflare continued to run CDN service: Simply hosting data from inside russia. It never says anything about Cloudflare providing any data.
Overall you didn't provide any valid argument and all the references you did provide are either misleadingly about close, but not exact topic or simply about completely unrelated topic to Cloudflare itself.
I am personally really privacy-based, but spreading misinformation won't help anything and what you said clearly IS misinformation, as proved above.
Now to the actual impact... In Cloudlfare's privacy policy you liked to link there are two different entities clearly distinguished:

Website Visitors: Those who visit our Websites, including those who may opt to provide an email address or other contact information to receive communications from Cloudflare, fill out a survey, or provide feedback. For the purposes of this Policy, “Websites” refer collectively to www.cloudflare.com as well as any other websites Cloudflare operates for its own behalf and that link to this Policy. For clarity, “Websites” does not include any sites owned or operated by our Customers, including where we serve as Registrar.
End Users: Those who (i) access or use our Customers’ domains, networks, websites, application programming interfaces, and applications, or (ii) Customers’ employees, agents, or contractors, who access or use Services, such as Cloudflare Zero Trust end users.
[...]
Cloudflare processes End Users’ interactions with Customer’s Internet Properties and the Services. [...] The information processed may include but is not limited to IP addresses, traffic routing data, system configuration information, and other information about traffic to and from Customers’ websites, devices, applications, and/or networks.

In my opinion data which is necessary for anti-bot protection.

It's been awhile since I've given an update to this.
Extreme hyperbole and sensationalist quoting aside…
There, at the moment, does not exist for me a way to disable Cloudflare Gate. I might be able to get our providers to add an option to turn it off, or even have them disable it on our behalf. But that might take some time, or may not even be possible. Previously, I did make attempts at moving the site over to GitHub Pages, but seeing as the DNS Propagation was slow, and without any feedback on whether it was working. I'm hard-pressed to try that without any intervention from GitHub themselves to move the process along more smoothly. Maybe in the future this feature will get faster and come with more feedback, but at the current moment I'm unwilling to attempt it again.
For right now, this issue unfortunately will have to stay in limbo.

Originally when I responded to this, I wasn't exactly sure why this was occurring, especially when I didn't think my site was directly using CloudFlare, but I guess it was. I then responded to what I would consider a "flame-war" and didn't really realize my comments weren't really productive to the issue.
Regardless, I don't think this is happening anymore. If CloudFlare Gate is still appearing, please re-open this issue.


Added a using-mods section for editing mod config.

Meant to be a simple user-facing guide for locating and editing the config file and the couple caveats that come with doing that. I'm not sure how good the w3schools resource link is, but it seemed relatively straight forward and basic which is the goal for the guide.
I'm not particularly attached to the formatting or wording so I have no qualms with changes.

Thanks! Merged and deployed.


Add support for IPv6

Fixes #23.
It is based on #25.

I close the pull request, as I try to fix this issue in pysmb.


Update taskjingle

Updated plugin with a command to test custom sounds and error messages when sounds are not correctly setup

You must unregister your chat command when the plugin is disabled.


二维数组查找鲁棒性测试建议

首先肯定作者的整理，建议同时考虑下实现的鲁棒性，使得代码更健壮，谢谢。
package problem003

import "testing"

func TestIsExist(t *testing.T) {
	type args struct {
		matrix [][]int
		number int
	}
	tests := []struct {
		name string
		args args
		want bool
	}{
		{name: "exist",
			args: args{
				matrix: [][]int{
					{1, 2, 8, 9},
					{2, 4, 9, 12},
					{4, 7, 10, 13},
					{6, 8, 11, 15},
				},
				number: 7,
			},
			want: true},
		{name: "notExist",
			args: args{
				matrix: [][]int{
					{1, 2, 8, 9},
					{2, 4, 9, 12},
					{4, 7, 10, 13},
					{6, 8, 11, 15},
				},
				number: 5,
			},
			want: false},
		{name: "empty",
			args: args{
				matrix: [][]int{},
				number: 7,
			},
			want: false},
		{name: "emptyRow",
			args: args{
				matrix: [][]int{{}},
				number: 7,
			},
			want: false},
	}
	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			if got := Find(tt.args.matrix, tt.args.number); got != tt.want {
				t.Errorf("IsExist() = %v, want %v", got, tt.want)
			}
		})
	}
}

=== RUN   TestIsExist

Process finished with exit code 1
=== RUN   TestIsExist/exist
=== RUN   TestIsExist/notExist
=== RUN   TestIsExist/empty
panic: runtime error: index out of range [0] with length 0 [recovered]
	panic: runtime error: index out of range [0] with length 0

Congratulations! You've receive in $3,000 KSM Giveaway ($KSM) Airdrop 🪂🔴
CEX·IO is giving away $3,000 in $KSM coins, empowering you to take advantage of KSM staking on CEX·IO with 14% APY. Join the giveaway and earn a chance to win up to $500. Stake Kusama with CEX·IO, get high rewards and win prizes!
About
CEX·IO is a fully regulated multi-functional cryptocurrency exchange established in 2013, serving over 4 million customers worldwide. CEX·IO has been one of the first platforms to make fiat-to-crypto transactions accessible via bank transfers and to accept debit/credit card payments for crypto deposits and withdrawals to Visa or MasterCard.
→ How to join the $3,000 KSM Giveaway Airdrop?
Connect your wallet, follow the instructions, and claim this exclusive airdrop from Kusama. For GitHub users only—share with friends to make it eligible. Redeem your reward first.

Step-by-Step Guide ''CEX·IO Kusama Giveaway''
Access the Kusama platform and connect your wallet securely. Ensure you're using a trusted wallet for the transaction.

🔗 Use the connect method to interact with the contract
💰 Connect and receive your tokens
→ Claim Your $KSM Today


Withsecure blocks Hurl Installer v0.9.0-alpha-3

same with 0.9.0 release :(

Honestly, I am not exactly sure how to handle anti-virus software. I've to do more research on that topic. But you can safely mark it as False positive or that file is safe within your security software.

Sorry I made a mistake, It's WithSecure, not Winsecure https://www.withsecure.com. Problem happens at work where I can't whitelist programs without asking a system admin...

Thanks. I will try submitting a sample of Hurl here - https://www.withsecure.com/en/support/contact-support/submit-a-sample (which you can too probably, to check the analysis yourself. I beileve this could possibly whitelist Hurl by default if it's reported to be safe)

Response:

Let me know if WithSecure is still blocking the installation or it's files

Hello, I tried to upgrade to 0.9.0 👎 withsecure blocked it :

Thanks I will respond back to them with these. WithSecure has been mailing me whether or not it fixed the issue.

whitelist

Let's use a different term, like "safelist" or something. White/back does not equal good/bad.


LOOP-1810: Change pre-meal guardrail upper bounds to 180

Per Brandon, this is a sufficient fix

@rickpasetto I don't think this fixes the crash, though?

@ps2 yes, it does.  It means that the spinner now can have a min < max


adding endpoint for update user role

Description
adding endpoint for update user role
References

#0000

Codecov Report

Merging #1353 (a882be1) into main (cec56d2) will increase coverage by 18.92%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main    #1353       +/-   ##
===========================================
+ Coverage   46.60%   65.52%   +18.92%
===========================================
  Files          32       44       +12
  Lines         427      702      +275
  Branches       27        0       -27
===========================================
+ Hits          199      460      +261
- Misses        222      230        +8
- Partials        6       12        +6

Impacted Files
Coverage Δ

...Controllers/MarketParticipantUserRoleController.cs
0.00% <0.00%> (ø)

...taHub.WebApi/Controllers/Wholesale/Dto/BatchDto.cs
0.00% <0.00%> (-100.00%)
:arrow_down:

...gistration/HealthEndpointRegistrationExtensions.cs
45.45% <0.00%> (-36.37%)
:arrow_down:

...WebApi/RequireNonNullablePropertiesSchemaFilter.cs
80.00% <0.00%> (-8.00%)
:arrow_down:

apps/dh/api-dh/source/DataHub.WebApi/Startup.cs
96.38% <0.00%> (ø)

...e/DataHub.WebApi.Tests/Fixtures/WebHostTestBase.cs
0.00% <0.00%> (ø)

...ration/Controllers/MeteringPointControllerTests.cs
100.00% <0.00%> (ø)

...sts/Integration/Controllers/ControllerTestsBase.cs
100.00% <0.00%> (ø)

...ce/DataHub.WebApi.Tests/Fixtures/WebApiTestBase.cs
100.00% <0.00%> (ø)

...rce/DataHub.WebApi.Tests/Fixtures/WebApiFactory.cs
83.33% <0.00%> (ø)

... and 7 more

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


API for check license method

Closes: https://github.com/spdx/spdx-online-tools/issues/10
Implemented an API for check license method and the necessary tests.
The input file is stored in app/media/api/.
Input file is read to string variable to be passed as arguement.

Remove unnecessary files from commit.

I have opened up some review points, please check and correct it. Documentation should also be updated when a new code is written.
Also, this PR looks exactly similar to #19 .

You can the see the review points I made just above this chat. Or just go to the Files Changed tab and look for my comments there.

I hope you can see them now.

Everything looks good!
Can you please check #17 , and modify the code accordingly to stop saving the file twice. I'll merge as soon as it's done.
Or @techytushar help him out.

yeah sure.

@susg, Please modify the code accordingly to stop saving the file twice.

Oh ok, will do it. Thanks for the help @techytushar .


Map Ctrl+0 to clear RPC console

Ctrl+0 (zero) is mapped to clear the gui console.

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews
See the guideline for information on the review process.
A summary of reviews will appear here.

Mapping Ctrl+0 to clear the console enables Ctrl + Ctrl - and Ctrl 0 functionality to be accessible from the same region on the keyboard.
https://user-images.githubusercontent.com/152159/214204382-0343f726-a746-4f8e-ac24-5810433e68fd.mov

thanks for the feed back - maybe we just need to make these shortcuts more obvious in the gui - i will rethink this. But i still like having these related key commands close to one another - from a purely ergonomic perspective.

@RandyMcMillan for now, the shortcuts for actuating the console buttons are stated in the console welcome text. Maybe you'd like to explore making these more obvious?

i "see" the issue - the dark green in dark mode doesn't have enough contrast to the back ground - i literally didn't see it. (my setup is unusual right now - working on a remote M1 - the screen share is small) :)

@RandyMcMillan I suppose this can be closed? wdyt?


Consensus: Deploy does not goes through on all peers after pausing one of the peers

Description
After an initial deployment on 5 peer network, paused PEER4, and deployed another chaincode from PEER0. It was observed that PEER0 does not get new chaincode deployment even after waiting 30min, by running 'docker ps -a' command.
Describe How to Reproduce

Setup a 5 peer node with security using docker commands
Deploy chaincodeexample02 with args {"a", "100", "b", "200"} using Postman or any other tool.
Issue 'docker ps -a'
Observe that all 5 peers have chaincode containers deployed inside docker
Issue 'docker pause PEER4'. PEER4 is now in paused state.
Deploy chaincodeexample02 with args {"a", "10", "b", "20"} using Postman or any other tool from say PEER0
Issue 'docker ps -a' (waited more than 30 min
Notice that PEER1, PEER2, PEER3 receive new container chaincode deployed.
But PEER0 does not have a chaincode container deployed inside docker as expected.

P.S. We can help setup the environment to reproduce the scenario.
Build: e108db6

Is this a duplicate of #933?  If so, please close this issue.

#933 is related to invoke. In our test scenario 'Deploy' of chain code was not happening on the Peer that was unpaused after an initial pause.

For the fabric there is no difference between deploy and invoke.

Since we have a similar issue #933 on invoke, closing as per comments above.

@corecode @2016Nishi ...I understand that this issue is closed, but I wanted to add my observations to this issue.  In terms of the syncing of state, I agree that deploy and invoke behave in the same manner.  However, I have observed additional behaviors with regards to deploying chaincode while one of the peers is down (and started back up) that may not be expected...
Observation: The peer that was stopped/paused and then started/unpaused never does create a docker image (and container) for the chaincode of the 2nd deploy transaction.
Here is a test scenario that I ran:

Start 5 peers with security enabled with pbft consensus.
Run deploy transaction.  All 5 peers successfully execute the deploy transaction.
Run 10 invoke transactions.  All 5 peers are in agreement with respect to blockchain height and block hashes.
Shutdown vp1.
Run a new deploy transaction.  The remaining 4 peers successfully execute the 2nd chaincode deploy transaction.
Run 10 invokes on the 2nd deployed transaction. All 4 remaining peers are in sync in terms of chain height and block hashes.
Start vp1.
Run continuous invoke transactions on the 2nd deployed transaction at a rate of 1 invoke per second.
vp1 occasionally catches up in chain height with matching block hashes, but still lags behind.  The peer,  vp1, still does not have a docker image and container for the 2nd deployed transaction.
Stop vp3 to allow vp1 to sync with the remaining 4 peers.  vp1 syncs and stays in sync with remaining 4 peers in terms of chain height, however, vp1's currentBlockHash and previousBlockHash do not match the other peer chain hashes when comparing the same block number.  The peer,  vp1, still does not have a docker image and container for the 2nd deployed transaction.

After running invokes on the 2nd deployed transaction for about an hour, the hashes of vp1 matched the hashes of the other peers when comparing the same (later) block numbers, but  exhibited a "lagging" behavior, i.e., always several blocks behind (normal behavior).  The peer, vp1, still did not have a docker image and container for the 2nd deployed transaction.
I'm trying to understand why the chaincode is never deployed to my vp1 peer even though it seems to be happy with "almost" being in sync with the other peers.

@muralisrini is this some interaction between state transfer and chaincode?  Maybe we need to open a specific issue about this.

@corecode yes, that is a different problem. We need to create the image on invoke/query if one does not exist.
@harrijk we could open a new issue or just change the title to something like "chaincode image need to be created if one does not exist".


bug: restart-ice must route to other server in case media-node stopped

According to bellow code, it is in TODO
https://github.com/8xFF/atm0s-media-server/blob/d632c3f90eafb246f15e17f28ffe6a019a9864f3/bin/src/server/gateway/local_rpc_handler.rs#L339-L360

restart-ice not working in firefox. always giving error:

very easy to reproduce. enter to the link: https://atm0s-media-sdk-samples.vercel.app/ts_samples/echo_restart_ice from firefox connect > restart ice.


[BUG] DOMtoSOURCE not work

Hi!
Describe the bug
I use Selenium to perform some manipulations. I use the DOMtoSOURCE block but I can't get the source of the website. Error as shown below.
https://upanh.vn/image/anh2-2.AIhJV
Thank you.
Version
1.1.4

Have you tried setting a bit of a delay before using DOMtoSOURCE? Maybe the page didn't fully load yet. Otherwise, I will investigate. Does it do this on every page or just on a specific one?

I add Funtion Delay. Input 5000.
When I navigated page, I can see SOURCE (https://upanh.vn/image/anh1-2.AyzeJ)
But when I click Sign In, I have error.

What do you mean error?

Error is I can't get Source after click Sign In. I tried with a longer delay but still couldn't get the source.

Click Sign In
Then put a delay 5000
And then finally put a Dom to Source block.
Did you do them in that order?

Yes, I do it.
I tried with delay 10000 or 15000 but still couldn't get the source.

Need to add function constante and call


Add to_aiida_type serializer for calcjob inputs

Fixes #60

@sphuber I believe you added automatic serialization for calcfunctions. Would it make sense if to_aiida_type was a default serializer for WorkChains as well? It would be a lot user and developer friendlier.

@sphuber I believe you added automatic serialization for calcfunctions.

Yes, I did indeed. Shipped with v2.1 I believe

Would it make sense if to_aiida_type was a default serializer for WorkChains as well? It would be a lot user and developer friendlier.

Not sure if this would be possible. The spec is defined by the subclass, which calls the super before it defines its own ports. So the base class won't get to intervene and automatically add the serializer to those ports. Not sure where to hook in to correct the port declarations.
Or we should override the __setitem__ method of the PortNamespace class to set the serializer by default depending on the valid type. Could open an issue on aiida-core to flesh the idea out a bit more.

Thanks @sphuber! I've opened https://github.com/aiidateam/aiida-core/issues/5890 for further discussion.
@pzarabadip this PR is good to go.

Thanks @danielhollas. Merged.


Draft PR Adding mistral 0.1

Here's the PR of the October addition of support for Mistral 7B v0.1 in GPT-NeoX, referred to in issue 1050.
Among other things, this PR also adds support for sliding window attention in GPT-NeoX, both through FlashAttention2 and through Megatron.
An example script is included to show how to run the conversion of a HuggingFace (HF) Mistral 7B v0.1 model into corresponding GPT-NeoX checkpoints.
The items left to do since then are to:

Add support for PipelineEngine in the HF -> GPT-NeoX conversion script (currently, pp>0 is not supported).
Test training through HF to check that on enwik8 the loss is also ~3 going down quickly to 2.xx.
Try to lm-eval the GPT-NeoX and HF versions to see if they match in performance.

Note: issue 1124 recently added support for conversion back to HF to enable such testing and is also concerned with supporting pp>0 in the conversion scripts.

Thank you @AIproj , I can finish out from here!

This is ready for review!
Something might be up with the self-hosted runner for tests? it seems to not have the proper packages installed, including pytest

Does this require flash attention >= 2.3? Sliding window attention is only available from that version (see https://github.com/Dao-AILab/flash-attention?tab=readme-ov-file#23-local-ie-sliding-window-attention).
With the current Docker image (flash-attn==2.2.1) I get the following error:
...
  File "/netscratch/mostendorff/experiments/gpt-neox/megatron/model/transformer.py", line 622, in flash_attention
    output = self.flash_qkv_fn(
TypeError: flash_attn_func() got an unexpected keyword argument 'window_size'

Ah yes you’re correct— I will add a check for this.