Wallet - Contacts - Not saving

Hi,
Issue Impact: Minor
Device Manufacturer: Huawei
Device Model: P20Pro
Device OS: Android
Device OS Version: 10
Application Version: apk: v0.98.24 wallet version 1.1.2
What Happened?: I created and saved new contact amd despite a pop us saying the new contact was saved it did not appear on the list. I refreshed the page with no effect. I closed down the Wallet and reopened with no effect. No contact saved.
What Did You Expect To Happen? To save new contact and for it to appear in the list immediately
Steps to recreate:
1 Create contact
2.  Pop up saying contact saved
3. Refresh wallet
4. No contract saved
I still have trouble adding screenshots from phone to GitHub. I'll get @eurobuddha to add in comments.

Right. Sonit transpires I already had "Bob" in my contacts as "Boldor" not knowing that. So it looks like the Wallet knew the address was already saved wirh different contact but still popped up with a "Bob was saved to your contacts" even though it wasn't nor was the name changed.
Screenshots will shine additional light onto this 😁

This is now fixed in v1.1.3,  You'll now receive a message stating that the address already exists in your contacts if you try to re-add the same address.

Thanks Elias,
I guess this can be closed I guess 😊
R

we will close it once you have tasted 1.1.3 and tested it @rafaroberto

Thanks!
On Thu, 17 Jun 2021, 11:24 eurobuddha, @.***> wrote:

we will close it once you have tasted 1.1.3 and tested it @rafaroberto
https://github.com/rafaroberto
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/minima-global/Minima/issues/266#issuecomment-863121840,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUK4ADEMPLZWYJ2ORJ3YSPLTTHENVANCNFSM46M5ZWEQ
.

Bump


Tipping Hat emoji

Tip of the Hat as either an informal greeting or formal/military salute. Could also be Person Lifting Hat.
Petitions

change.org

2021 Emoji 11.0
https://emojipedia.org/saluting-face/


Missing information for one or both assets

This is logged:
Skipping non-image asset_id cca82518-6646-47f7-b6d4-029afe05de4a with Content-Type: application/json; charset=utf-8

There is nothing related to the ID  of cca82518... in the Immich DB (public.assets) and no file in the filesystem with that name (e.g. preview image).
Does the app handle deleted assets correctly?

Yes, when you delete an image in Immich, the image code remains in the duplicate database. If you rerun the duplicate finder, it will not be able to locate the image in Immich and will return an error. I will update the deletion code to also remove the corresponding entry from the database.

Anyhow sometime Immich return a json file. This will be skipped. Anyhow seems that you don't have duplicate. What I'll implement soon are similiar image finder. Maybe changing frontend from streamlit to react....


Error loading PokeMMO data package. The current data may be invalid or corrupt.

I always seem to be getting, even after a repair.
Error loading PokeMMO data package.
The current data may be invalid or corrupt.
Please reinstall the client to resolve this issue.

Do you want to attempt to repair your PokeMMO client?

Hi, the bot is not updated for the latest update so it starts up the older version of the game which is why you see the error.

That makes sense, thank you!


CX Reflected_XSS_All_Clients @ src/main/java/com/github/rafaelrpinto/vulnerablejavawebapp/controller/UserController.java [master]

Reflected_XSS_All_Clients issue exists @ src/main/java/com/github/rafaelrpinto/vulnerablejavawebapp/controller/UserController.java in branch master
Method processSignUpForm at line 65 of src\main\java\com\github\rafaelrpinto\vulnerablejavawebapp\controller\UserController.java gets user input for the name element. This element’s value then flows through the code without being properly sanitized or validated and is eventually displayed to the user in method href="javascript:void at line 45 of src\main\webapp\WEB-INF\jsp\include\header.jsp. This may enable a Cross-Site-Scripting attack.
Severity: High
CWE:79
Vulnerability details and guidance
Checkmarx
Lines: 65

Code (Line #65):
	String processSignUpForm(@RequestParam("name") String name, @RequestParam("email") String email, @RequestParam("password") String password,

Issue still exists.

Issue still exists.

Issue still exists.


switch bootJDK download scripts to use adoptium binaries

This fixes the failing JDK tip builds
fixes: https://github.com/adoptium/temurin-build/issues/2704
I really want to split these horrible blocks of code into a shared function but will work on that in a follow-up PR

Should we be using GH setup-java action for this? Am also worried that adoptium only pulls down subset of adoptopenjdk API

No one of the main reasons we don't is because we want to test the code that pulls down boot JDK's. I've added another check which will fall back to adoptopenjdk if required.

Test failures are unrelated to this change

I'm going to merge this as it'll help with testing any emergency PRs WRT to java 17

@karianna @gdams I can see the fallback to adoptopenjdk in the linux script here but not the others. Isn't this introducing a bit of a risk? Given that we have the forwarding from adoptopenjdk->adoptium I'm not clear on why we're pushing this through during a lockdown period.

@karianna @gdams I can see the fallback to adoptopenjdk in the linux script here but not the others. Isn't this introducing a bit of a risk? Given that we have the forwarding from adoptopenjdk->adoptium I'm not clear on why we're pushing this through during a lockdown period.

17/tip builds were failing for GH Actions which meant we didn't have PR testing on any changes we might need to push through during release week, so this was worth getting in (it certainly couldn't make it worse).


delay to collect a did:web VC

when you collect a VC with did:web example : https://talao.co/gaiax
The time to get and sign  the VC is about 10s, it would be great to have a spinner or a message to wait

maybe best solution is to set the button inactive and display a message while the wallet is signing


Support and document the new local_heap configuration property for Query configuration

https://issues.jboss.org/browse/ISPN-8034
The "ram" short name to use Lucene's RAMDirectory implementation was renamed to "local_heap" in Hibernate Search to make its function clearer in the context of Infinispan as there are many kinds of ram.
The "ram" keyword was hardcoded into the Infinispan project though, so this needs to be updated and documented.

Looks good. I'd like a note in the upgrading.adoc file too please

Looks good. I'd like a note in the upgrading.adoc file too please

Right! thanks, I didn't know about that one. Will do asap.

Updated the migration guide.

Pushed to master, thanks !


Catch redundant brackets around overloaded labels

Hlint says nothing about (#label) when it could/should be #label.

It's an issue in the underlying version of haskell-src-exts-util (although totally a bug I wrote). I've submitted a PR upstream.

Very prompt! Thanks.

Feel free to close here.


False-positive possibly unbound variable with explicit True / False check

Environment data

Language Server version: 2021.4.4-pre.1
OS and version: darwin x64
Python version (and distribution if applicable, e.g. Anaconda): 3.10.0a7
python.analysis.indexing: true
python.analysis.typeCheckingMode: strict

Expected behaviour
An explicit False check should be handled like else. Especially with pattern matching, it's not always possible to default to else.
Actual behaviour

Logs
--
Code Snippet / Additional information

simplified example
from typing import Any

class Custom:
    pass

class Derived(Custom):
    pass

def func(cls: Custom) -> Derived:
    var: Any
    if isinstance(cls, Derived):
        var = cls
    elif not isinstance(cls, Derived):
        var = Derived()
    return var  # possibly unbound

def func2(cls: Custom) -> Derived:
    var: Any
    match isinstance(cls, Derived):
        case True:
            var = cls
        case False:
            var = Derived()
    return var  # possibly unbound

These are very difficult and costly determinations for a static type checker to make, so we're unlikely to implement them unless we receive strong signal from many users that they are generally desired.
As an easy workaround, you can initialize your variable outside of the conditional/match statement or ensure that you always have a code path that executes (e.g. use an else or a wildcard pattern):

def func(cls: Custom) -> Derived:
    var: Any
    if isinstance(cls, Derived):
        var = cls
    else:
        var = Derived()
    return var

def func2(cls: Custom) -> Derived:
    var: Any
    match isinstance(cls, Derived):
        case True:
            var = cls
        case _:
            var = Derived()
    return var


Set the cooldownDueDate after randomGem is minted

we have to set the cooldownDueDate of the gem which will be minted after randomGemPack function
currently we have two types of events emitted when the randomGem is minted, one is RandomGemTransferred which is the case when there is any gem available in treasury and CommonGemMinted event which is the case when there is no gem in treasury
I think adding cooldownDueDate to those two params are a bit annoying on contract logic side so to implement this consistently on every gems' cases, i think we need to make a new event with tokenId, cooldownDueDate, etc inside transferTreasuryGEMto function as all the gems transferred from treasury to users must reset the cooldown in every cases
Please let me know if this doesn't make sense so we can have a deeper discuss

after discussion, here is the updated code : https://github.com/tokamak-network/gem-nft-contract/commit/ae84834591c14564ce0dff20e534637241221575


Might be handy to replace bookmark location

If you replace the source of the bookmark as if it was a bookmarklet you can get away with the separators not redirecting you when clicked, like so

It's a good idea. Unfortunately, this doesn't work as Chrome uses the location to determine icon for the bookmark. And with this solution you'd always get the default bookmark icon (the globe).


Without req.answer calling SSG will respond plain text.

Minimal ex:
server.routes:
  "/":
    "Hello, world!"

Works also like #20


Add option to reset sensor without generating notification

Add the option to reset the sensor without generating a notification. So notifications are generated only when the sensor is tripped.
Use cases:

When the switch is turned off right away, then you get two notifications in succession
When the switch is turned off at a later time, the sensor reset notification may not make any sense (maybe switch is reset at midnight by daily automation)

Also interested in this. Is this technically possible @tmyrden ?

I don't see why this wouldn't be possible. Happy to accept a PR for this functionality. I might have time to play with it over the holidays too, but no guarantees.


Unable to create Amazon Credentials

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Open Cloud Credentials (https://rancher.ops.ps.m953.net/g/security/cloud-credentials)
Try to add a new Amazon Credential
Fill out any required filed

Result:
Validation failed in API: a Config field must be set is required
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):

2.3.5

Installation option (single install/HA):
HA

I found the problem.
This error message is kinda misleading. Please update it.
Problem was, that i disabled any Node Drivers but vsphere Drivers.
Re-enabling the Amazon EC2 Driver solved the problem.

Faced same issue on a branch new installation of Rancher v2.5 (v2.5-07131035f51f3e264c114e9d9dde65aae37bdf4f-linux-arm64).
Guys, any ideas what can be wrong ?

Faced same issue on a branch new installation of Rancher v2.5 (v2.5-07131035f51f3e264c114e9d9dde65aae37bdf4f-linux-arm64).
Guys, any ideas what can be wrong ?

Identified the issue:
As you may see i am using ARM version, but seems like some drivers are not working there. So, please beware of ARM architecture for rancher setup.


WSL Install Hangs without error

Version
Microsoft Windows [Version 10.0.19044.2486]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.4.72
Distro Version
N/A
Other Software
No response
Repro Steps
Attempt to perform install with "wsl --install -d Ubuntu" or an update with "wsl --update"
Expected Behavior
It should install
Actual Behavior
When trying to install WSL and Ubuntu with the command "wsl --install -d Ubuntu" it just hangs. Same if I try and update kernel with "wsl --update"
I left it running overnight to be sure, nothing changed.
There is no error message, it just sits on the message "Installing: Ubuntu"
(I realize it's not much to go on, but I don't have much to go on either... if there are logs somewhere I can look for, that might help)
Diagnostic Logs
No response

/dumps

The file is too big to attach to GitHub, so I uploaded it to GD instead
https://drive.google.com/file/d/1tSXMpWs7uDdbofweKvE5kAQYQeA7FCHv/view?usp=sharing


shrink when needed

closes #18

@Sune1337 Sadly I had to remove one performance improvement in order to get it to shrink when needed. Auto height had to be called to get the correct scrollHeight again. Do you have any other idea?

Do you mean it does not shrink once it has expanded because you write in the textbox, or is it when you change the model?
What browser and version are you using?

Is the official demo working for you?`
http://embed.plnkr.co/8a3hvdzIIUs3X1nOKPtK?show=preview

Yup. The official demo is working.
I'm using ionic2 and it broke there.
I'll try to create a plunker or something to show and update #18 description.

You can try using the Ionic 2 demo at
https://github.com/fiznool/angular2-elastic-demo-ionic. I haven’t got round
to updating this demo yet to use ng-elastic, but you could fork and work
from there.

Damn. I'm sorry I have very limited internet cap this week to make this kind of test. I can help a lot more by saturday.
If anyone wants to test, I'm using Chrome 57.0.2987.133 (64-bit) and ionic-angular 2.1.0.
The html part that broke is:
<ion-item>
    <ion-label stacked>notes</ion-label>
    <ion-textarea name="notes" fz-elastic [(ngModel)]="notes"></ion-textarea>
</ion-item>

If no one else is able to reproduce, I'll make sure I have a branch showing the bug by saturday.

The issue is that this.textareaEl.scrollHeight is not reduced when deleting lines from the textarea.
I could not find the reason why this happens when running in Ionic and not when running "vanilla".
The fix you have done was the only one i could get to work as well.
The effect is that the browser needs to do some layout calculations every time you press a key instead of every time a new line is added.

Yeah. We can try to hack into ionic internals to find out why that happens and maybe post an issue on their repo. But if we want this directive to support ionic, I think this PR is a price we have to pay at least for now.

yup

I think i found the source of the error. If you add 'border: 0' to a textarea, the shrinking stop working.
Ionic does this via the 'text-input' class.
Can replicate on official demo if adding the border in dev-tools.

scrollHeight seem to behave different depending on combinations of box-sizing and border.
Setting height to 'auto' seem to be the only way to cover all cases.
Test it out:
https://jsbin.com/xoqacuy/5/edit?html,css,js,output

And the behavior is consistent between Chrome, FF, IE and Edge so it seems to be by design.

Wow @Sune1337 thanks for the research done! Great "plunker" showing this behavior.

Great detective work @Sune1337!
I guess for now this PR can be merged in so we can support Ionic.
Another possible approach could be to debounce/throttle the adjust function thus reducing the number of layout calculations if the input is updated quickly.

I think we'll be OK without the performance enhancement. On the demo site the layout calculations took 0.6ms on my laptop so it'll probably only be an issue with very complex layouts.

I'll get this merged in and released after the Easter break.
Thanks for all this!

Thanks again for the contribution. The fixes here were published in ng-elastic 1.0.0-beta.3.


CI job build-sdk-cpp-coverage fails with Gcovr 5.1

See https://github.com/trytouca/trytouca/actions/runs/2326121465/attempts/3 as an example.

After some investigation, opened a GitHub issues with gcovr/gcovr#616 to see if maintainers have any idea.

Received update that the issue is known to gcovr team and v5.2 is on the way. Should upgrade (unpin gcovr) once released.


feat: upgrade typescript 5 and adds SWC

So far looks good! Was the bot still able to build and run locally?

This PR still in Draft, not finished. Opened just for tracking purpose.


Add Support for linode.

Would like to have support for linode added to the list of providers.

That screen is essentially a frontend for docker-machine.  Linnode (or someone else) would need to contribute a driver for it to be supported.

I'm going to close this issue as there is a dependency for a driver to be added on docker-machine. When it's added, we could add support for it.

it looks like the driver is being worked on. https://github.com/docker/machine/pull/268

Docker is not accepting new drivers into machine proper anymore, they have a way they can be written as plugins.  We are working on supporting that so that you can choose to include whatever drivers you want without involving us.


tf.keras.backend.repeat_elements does not support negative indexing on tensors with dynamic shape

System information

Have I written custom code (as opposed to using a stock example script
provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 and Google Colab
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v2.2.0-0-g2b96f3662b 2.2.0
Python version: 3.7
CUDA/cuDNN version: 10.1
GPU model and memory: Colab GPU

Describe the problem
The implementation of repeat_elements behave differently whether the input tensor has static or dynamic shape.
For tensors with dynamic shape it does not accept negative indexing for the axis parameter.
For tensors with static shape it accepts negative indexing for the axis parameter.
TensorFlow follow standard python indexing rules.
There is a workaround using positive indexing.
How to reproduce
Run the repeat_elements with axis=-1 and tf.config.experimental_run_functions_eagerly(False)
Note the resulting array
Run with tf.config.experimental_run_functions_eagerly(True)
Note the resulting array
Set axis=1 and repeat 1-4
Note that input_signature parameter in tf.function is there to reproduce a scenario of a graph with the tensor x with dynamic shape.
Source code / logs
import tensorflow as tf
import tensorflow.keras.backend as K
import numpy as np

@tf.function(input_signature=[tf.TensorSpec(shape=[None, None], dtype=tf.int32)])
def f(x):
  x = K.repeat_elements(x, rep=3, axis=-1)
  return x

tf.config.experimental_run_functions_eagerly(True)
v = tf.Variable([[0, 1],[2, 3]])
f(v)

<tf.Tensor: shape=(2, 6), dtype=int32, numpy=
array([[0, 0, 0, 1, 1, 1],
[2, 2, 2, 3, 3, 3]], dtype=int32)>

@tf.function(input_signature=[tf.TensorSpec(shape=[None, None], dtype=tf.int32)])
def f(x):
  x = K.repeat_elements(x, rep=3, axis=-1)
  return x

tf.config.experimental_run_functions_eagerly(False)
v = tf.Variable([[0, 1],[2, 3]])
f(v)

<tf.Tensor: shape=(2, 6), dtype=int32, numpy=
array([[0, 1, 2, 3, 0, 1],
[2, 3, 0, 1, 2, 3]], dtype=int32)>

I am able to replicate the reported issue, please find the gist here

repeat_elements is kept for backwards compatibility right now, but you should prefer the underlying TensorFlow manipulations or tf.experimental.numpy.repeat if you need alternate behavior.


Refactor withPeerSelectionActoins

In the #4555 branch withPeerSelectionActions has 19 arguments, we should use a record to pass them.

Maybe even not a single record but multiple, if it makes sense.


Quantum bug

Hi, I have a very annoying issue with quantum,
This is the index.ts code I have
(async () => {
    const moment = await import("moment")
    console.log(moment);
 })();

Now with this simple standard config
const sharedFuseConfig = {
    homeDir: `src`,
    cache: true,
    modulesFolder: `${__dirname}/node_modules/`,
    plugins: [
      QuantumPlugin({
            target : 'browser',
            bakeApiIntoBundle : 'app',
            manifest : true,
            containedAPI : false,
            removeExportsInterop : false,
            replaceTypeOf:false,
            removeUseStrict : false,
            ensureES5 : false
        })
    ],
    output: `./dist/$name.js`
};

Now when I do node fuse, everything is fine and fuse creates a split bundle for momentjs but now if I modify my index.ts and save it fuse watcher kicks again but now I get an empty momentjs splitted file like this:
(function($fsx){
})($fsx);

The only way around this is to stop the fuse process everytime I change my local code and restart so quantum will create a none empty split momentjs file.
So to explain better, what is happening is that with quantum enabled, it works and kicks  but it creates empty split files and you have to restart fuse process in order to avoid this issue. Which is something I don’t want at all, because for me I want to see my production result immediately and I don’t want to go through the dev vs production process. That is why I enabled quantum by default.

I get it, will check it out next week, thanks!

Keep me update on your findings. This bug is really a blocking point.

I will get to it when I will have time.

Thanks for your support!
By the way, I debug it and I found out where is the bug. It takes 20 min to figure it out but the bug is special on this line: https://github.com/fuse-box/fuse-box/blob/master/src/quantum/plugin/QuantumCore.ts#L167
Specially this line is so confusing but I guess, you know what you wrote. To solve the bug, I have to add the following line after it:
this.producerAbstraction.bundleAbstractions.get(key).packageAbstractions = bundleAbstraction.packageAbstractions;

The problem is that after the first bundle, packageAbstractions from bits are empty. So, if you want the render to write content, https://github.com/fuse-box/fuse-box/blob/master/src/quantum/plugin/QuantumCore.ts#L267. You have to fullfil packageAbstractions.
Again, after debugging all of this, even i say I solve it, I feel that there is something wrong somewhere.
Up to u to understand it if this allows you to figure out it !

@leon-andria been looking at it for a while, can't find a solution just yet. If you have ideas please share ;-) It might be related to the old variables being populated

@nchanged I will look at this tonight, with the last debugging experience I got to track this issue, I have a better idea.

Hi
I am facing the same issue, with a simple setup dynamically loading a module
import("lodash").then((module)=> console.log(module));

with the fusebox config
FuseBox.init({
        homeDir: "./src",
        output: "dist/$name.js",
        cache: false,
        sourceMaps: !isProduction,
        useTypescriptCompiler: true,
        plugins: [
            QuantumPlugin({
                treeshake: true,
                target: "universal",
                bakeApiIntoBundle:"app"
            })
        ]
    });

Initial build works flawless, but after saving the file with the dynamic import of lodash (while fuse is running) , it just fails, producing
(function($fsx){
})($fsx);

in the new file generated by QuantumBit: Moving module lodash from app to 6cb2ad0d
Help appreciated! :)

FuseBox v4 Major Upgrade
As of today, the latest major version of FuseBox v4 is available in fuse-box@next. FuseBox has undergone a full re-write from scratch, learning from its mistakes in the previous versions.
There are a lot of improvements, speed and config wise.
Get started here
We are still working on the documentation and polishing before the major release. We are getting very close. And we need help testing.
npm install fuse-box@next

Most of the issues that I am closing have been resolved, however, if you find that your issue is still relevant in v4 don't hesitate to re-open or (better) create a new one with v4 configuration.


Add loggers, remove traces of persons in output + minor code stuff

Codecov Report

Merging #328 (b1e2784) into master (012e63e) will increase coverage by 0.09%.
The diff coverage is 88.00%.

@@             Coverage Diff              @@
##             master     #328      +/-   ##
============================================
+ Coverage     77.74%   77.83%   +0.09%
- Complexity      659      663       +4
============================================
  Files            98       98
  Lines          2175     2184       +9
  Branches        180      180
============================================
+ Hits           1691     1700       +9
  Misses          396      396
  Partials         88       88

Impacted Files
Coverage Δ

src/main/java/donnafin/MainApp.java
0.00% <0.00%> (ø)

src/main/java/donnafin/commons/core/Messages.java
66.66% <ø> (ø)

.../main/java/donnafin/logic/commands/AddCommand.java
61.90% <ø> (ø)

...main/java/donnafin/logic/commands/FindCommand.java
92.30% <ø> (ø)

...ava/donnafin/logic/parser/RemoveCommandParser.java
68.18% <0.00%> (ø)

...n/java/donnafin/model/person/UniquePersonList.java
92.68% <ø> (ø)

.../donnafin/storage/JsonSerializableAddressBook.java
100.00% <ø> (ø)

...in/java/donnafin/storage/JsonUserPrefsStorage.java
87.50% <ø> (ø)

src/main/java/donnafin/ui/AttributePanel.java
100.00% <ø> (ø)

src/main/java/donnafin/ui/UiManager.java
19.48% <ø> (ø)

... and 13 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 71e0100...b1e2784. Read the comment docs.


Field return issue depending on field order in same query

Describe the bug
Depending on if a field is before or after another field, its value is null or defined using updates & resolvers.
e.g
media {
  uri
}
medias @stream {
  uri
}

will return
"media": null,
"medias": [...]

while
medias @stream {
  uri
}
media {
  uri
}

returns
"medias": [...]
"media": {
  uri: ...
}

I'm not sure if that's normal behavior with my resolvers, but I'm reporting just in case it's a fixable issue / to document it for anyone else also hitting that edge case.
Reproduction
https://github.com/Banou26/urql-issue-3385/blob/main/src/main.tsx#L106-L138
Urql version
urql 4.0.5
Validations

[X] I can confirm that this is a bug report, and not a feature request, RFC, question, or discussion, for which GitHub Discussions should be used
[X] Read the docs.
[X] Follow our Code of Conduct

That is actually expected, in graphcache we execute the resolvers/directives in the order they are encountered within the document so it could be that i.e. the next field hasn't resolved yet which would make it undefined. I guess we could make resolve all the fields without user-defined logic first but that's probably quite a big byte and performance hit to take.

As per the above, just because you can access arbitrary data from a result (be that an API result or a cache result that's being built up) that doesn't mean that that's a great idea 😅
There are several issues in your config that are preventing me from being able to help you, and to put it simply, it's too complex.
The short answer here is that some fields will always be unavailable. If you have fields a and b that are read from the cache and you have resolvers for both that are accessing each other, one will always be unavailable when the other is resolved.
In other words, you must not depend on other properties being available on the parent object in most cases.
There are several things that, personally, I wouldn't recommend in your code sample, and most may come down to schema design and trying to write a configuration that's just a little too "out there" and complicated but I'll list a couple points here to summarise what I mean.

Using non-scalar values in keys. I understand that you're trying to derive keys in your configuration to have relational joined keys for your edges based on their children, but I'm not convinced that's a good idea. Episode for instance could have its own id and from a glance, the key being predictable doesn't really help you that much probably
Overriding a result value as in updates.Episode isn't a supported pattern. so any behaviour that you'll see there is unsupported. This also shouldn't ever be necessary
Deriving a field from a key, such as in your resolvers, is not recommended. While it's possible to use keys as simple strings, they're basically meant as opaque values. This isn't enforced but that's basically like deriving a value from an ID

It basically looks a little like you're trying to normalise data on the client-side that isn't normalised on the schema.
Furthermore, I wouldn't recommend to try to normalise edges. You could give them a normalised ID on your schema (after which it would also make more sense to teach Graphcache what that ID looks like if it's predictable) but that also depends on whether you really need to be able to access an edge independently of its parent, which, again, hinges on how you access these edges and how they "re-occur" in your schema.
Say, if you have a details view and a list view. If that's the only way to get from a list of entities to a single entity, and if you then have the edge entity on a separate field or fields, as long as that entity has an ID field, there isn't any need to derive a predictable key in Graphcache.
I'll close this for now, but feel free to post more questions here ✌

Thanks for the descriptive answer!
The truth is that I'm hacking together yoga & urql into being a graph database & graphql client for my project. I'm a very edge case user and I'm definitely not using urql in the conventional way.
I used Apollo Server/Client with good success but it had terrible performance issues so I'm currently moving to urql because of that.
Using @stream simplified my logic by a thousand times, even if it's dependent on undefined cache resolver behaviors.
All of my data doesn't come from an actual graphql API server with normalized data that i host/control but instead, run in yoga from the browser, that executes scrapers that returns handles that might link to the same "end resource" or not.
I need to regroup these related resources into one accessible and queryable element for my frontend, that's what I'm doing through the "scannarr" elements here. This was the simplest way for me to do that, that i found, to group related resources into a simple renderable element. (this repro doesn't have a lot of the actually more complex field grouping logic)

Overriding a result value as in updates.Episode isn't a supported pattern. so any behaviour that you'll see there is unsupported. This also shouldn't ever be necessary

I've actually tried not modifying the actual result in the updates resolver, using cache.link, cache.writeFragment, ect... but I've somehow never managed to actually modify the cache values. So i ended up trying this and it worked perfectly for my case. I'd prefer not having to do this TBH but I've lost too much time trying the "documented way".
If you're curious about the project, It's https://github.com/Banou26/scannarr (very much WIP right now since i've broken everything to debug this issue haha), that I'm using in https://github.com/Banou26/stub


emacsMacport: get rid of stdenv override, fix aarch64-darwin build

Motivation for this change
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Tested with sandboxing natively, skipped sandbox on x86_64-darwin arch (via Rosetta) because it takes painfully long time to finish.
Note: to get Rosetta working under Nix with sandbox = true, you need to configure following:
{
  nix.sandboxPaths = [
    "/private/var/db/oah" # aot files
    "/Library/Apple"      # rosetta runtime
  ];
}

OK, seems like it's unstable on aarch64.
Process 11508 launched: '/nix/store/9kznfvwsww0gy6f6hyhnr2pm0d5jq437-emacs-mac-27.2-8.2/Applications/Emacs.app/Contents/MacOS/Emacs' (arm64)
2021-09-18 18:21:25.430863+0300 Emacs[11508:927600] +[MTLIOAccelDevice registerDevices]: Zero Metal services found
Process 11508 stopped
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
    frame #0: 0x0000000191fef254 libobjc.A.dylib`objc_msgSend + 52
libobjc.A.dylib`objc_msgSend:
->  0x191fef254 <+52>: ldp    x17, x9, [x13], #-0x10
    0x191fef258 <+56>: cmp    x9, x1
    0x191fef25c <+60>: b.ne   0x191fef26c               ; <+76>
    0x191fef260 <+64>: eor    x10, x10, x1
Target 0: (Emacs) stopped.
(lldb) bt
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
  * frame #0: 0x0000000191fef254 libobjc.A.dylib`objc_msgSend + 52
    frame #1: 0x000000019201280c libobjc.A.dylib`AutoreleasePoolPage::releaseUntil(objc_object**) + 204
    frame #2: 0x0000000191ff0e0c libobjc.A.dylib`objc_autoreleasePoolPop + 212
    frame #3: 0x000000010020d37c Emacs`mac_gui_loop + 92
    frame #4: 0x000000010020d2a8 Emacs`main + 1116
    frame #5: 0x0000000192175430 libdyld.dylib`start + 4
(lldb)

Got it working, apparently launching Emacs inside app bundle from terminal is a bad idea.

FYI: https://github.com/NixOS/nixpkgs/pull/155360. I'm trying to integrate your patch and got it working but both patches still crash on startup.
It's inconsistent, so perhaps, just by chance, the time you ran it via open worked.

I wonder why our emacsMacport crashes but @railwaycat's build doesn't.
Fatal error 11: Segmentation fault
Backtrace:
0   Emacs                               0x0000000102389c24 deliver_thread_signal + 60
1   Emacs                               0x00000001023885d0 seed_random + 0
2   Emacs                               0x0000000102389ca0 handle_sigsegv + 64
3   libsystem_platform.dylib            0x000000018c6704e4 _sigtramp + 56
4   ???                                 0xffff80018c4e4664 0x0 +<PHONE_NUMBER>2870103652
5   libobjc.A.dylib                     0x000000018c4e4664 _objc_msgSend_uncached + 68
6   libobjc.A.dylib                     0x000000018c508f70 _ZN11objc_object17sidetable_releaseEbb + 228
7   libobjc.A.dylib                     0x000000018c4eb38c _ZN19AutoreleasePoolPage12releaseUntilEPP11objc_object + 200
8   libobjc.A.dylib                     0x000000018c4e7d68 objc_autoreleasePoolPop + 208
9   Emacs                               0x00000001024b90b4 mac_gui_loop + 92
10  Emacs                               0x00000001024b8fe0 main + 1116
11  dyld                                0x0000000102cf90f4 start + 520
Segmentation fault: 11

I don't need this PR anymore, someone else feel free to pick this up.


Whiteboard Client arrival activity implemented

State manager client arrival complete
communicator client arrival complete
basic operations implemented(WB)
utility data structure classes added
(P.S.: There are still some files requiring comments and logs, those will be added in future PRs. For now, implementation is required by UX)

Can you guys add some unit tests to the functions implemented

I will merge this PR now but try to implement the unit tests ASAP.


Ignite documentation adjustments

Update README:

so it matches actual variables structure
add link to StorageClass documentation, so it's easier for users to configure persistence layer

Current README has incorrect persistence variables structure description.

@viglesiasce for some reason there are no reviewers on this PR, could you please review it/assign someone? It's just minor adjustments in documentation to match actual config structure

/assign @viglesiasce


diagram for outbound flow?

Is there diagram for outbound flow?

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

ID: fefee79b-aac3-6f4d-92bb-ffd7021a85a8
Version Independent ID: 446e7928-558c-b72d-2b8f-e53f04f2ebd0
Content: Data management and integration by using data entities - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/data-entities/data-management-integration-data-entity.md
Service: dynamics-ax-platform
GitHub Login: @Sunil-Garg
Microsoft Alias: sunilg

@prodev42, thanks for posting this!
@Sunil-Garg, can you please answer this question, and update the topic, if appropriate.

prodev42, we currently don't have an outbound diagram. The flow is almost the same as shown for import. One exception I can think of the part where the export file will be uploaded by AOS.


Problem with ibex_alu.sv when synthesis with genus cadence

Observed Behavior

Hi, I am new to both cadence and genus and got those weird error when genus read ibex_alu.sv. Any help is highly appreciated
genus.log
My Environment

EDA tool and version:

Genus(TM) Synthesis Solution, Version: 22.12-s082_1
Operating system:

CentOS 7.3
Version of the Ibex source code:

668233699df9ec2a40413e69e0de0a5b10185980

The error lines in the attached log file start with:
  parameter ibex_pkg::rv32b_e RV32B = ibex_pkg::RV32BNone
                            |
Error   : Parsing error. [VLOGPT-1] [read_hdl]
        : Expecting identifier in file '/home/usr14/ibex/rtl/ibex_alu.sv' on line 10, column 29.
        : Invalid Verilog syntax is parsed, or unsupported Verilog syntax is encountered.
  parameter ibex_pkg::rv32b_e RV32B = ibex_pkg::RV32BNone

I think this is probably saying that Genus hasn't actually imported the package file before ibex_alu.sv. The list of files comes from a TCL file that I think you probably wrote (genus_script_parallel_ibex.tcl). My guess is that this needs fixing.
Adding support for Genus would be neat (but I can't help: I don't have a licence!). I'm closing this issue because I don't think there's enough information for anyone to help make progress. If you have a specific issue that we can help with, please re-open or open a new (more specific) issue.

Hi！
I'm having the exact same problem with the Genus synthesis ibex_simple_system, and I'd like to ask how to import the ibex package in Genus before synthesizing. Or where I can find a package file to import.
Thank you for your help!


Fake Telegram Copyright Report

Hi, what about fake copyright reports! My channel got copyright complaint and there is nothing related to product or company. Its commit details of my github repository!
And no one replying to emails!
Please note that our moderators had to block the following messages in your channel @mltb_official due to copyright infringement:
https://t.me/c/1210063731/106
Do u have any idea with who i should contact with expectation of getting replied?
Thanks

You should have received all details in the message about the copyright infringement. There is no available information besides that.

Bruh we know who do this. Everything is fake and no one reply.
My channel is for github updates.
I have tried to contact all Telegram emails... no one responded.
I have edited the message while to know exactly what was the message is. But I'm sure it's not related to any copyright issue.
Imagine I'm reporting here! Sorry but one responding even twitter commmets also on twitter .. telegram login support

Hi, what about fake copyright reports! My channel got copyright complaint and there is nothing related to product or company. Its commit details of my github repository!
And no one replying to emails!
Please note that our moderators had to block the following messages in your channel @mltb_official due to copyright infringement: https://t.me/c/1210063731/106
Do u have any idea with who i should contact with expectation of getting replied?
Thanks

Telegram doesn't care about you! They consider all complaints from bytescare.com as genuine because of some weird reason.

I can't help with DMCA requests either.

 But I'm asking about a way to contact someone can help me in this issue

Hi, what about fake copyright reports! My channel got copyright complaint and there is nothing related to product or company. Its commit details of my github repository!
And no one replying to emails!
Please note that our moderators had to block the following messages in your channel @mltb_official due to copyright infringement: https://t.me/c/1210063731/106
Do u have any idea with who i should contact with expectation of getting replied?
Thanks

Telegram doesn't care about you! They consider all complaints from bytescare.com as genuine because of some weird reason.

Yeah same bytescare.com but others receive from other domains also once they know the way to do fake complaints then they will do it with different domains it's very easy for them to do it. In those two weeks about 12 fake complaints for groups and channels and bots belong to my friends! It's very easy

@levlam hi, another fake report happened now.
Is there anyway to contact someone who can help ?
Telegram:
Hello, this is Telegram Support.
bytescare.com made a complaint https://t.me/mltb_official_group is infringing on their copyright.
We kindly ask you to contact the copyright holder via email<EMAIL_ADDRESS>to resolve the matter.
Hello, this is Telegram Support.
bytescare.com made a complaint https://t.me/mltb_official is infringing on their copyright.
We kindly ask you to contact the copyright holder via email<EMAIL_ADDRESS>to resolve the matter.
Please note that our moderators had to block the following messages in your channel @mltb_official due to copyright infringement: https://t.me/c/1210063731/98

They threaten and demand money in return otherwise they flag the channel and groups.
Anyone can face those people on telegram and later on all this will increase.
I start thinking about this also! Like i can gain money by threaten people while telegram devs are sleeping.

They threaten and demand money in return otherwise they flag the channel and groups.
Anyone can face those people on telegram and later on all this will increase.
I start thinking about this also! Like i can gain money by threaten people while telegram devs are sleeping.

This is Business. We should join in.

Another fake report but this time no mentioned message
Telegram:
Hello, this is Telegram Support.
bytescare.com made a complaint https://t.me/mltb_official_group is infringing on their copyright.
We kindly ask you to contact the copyright holder via email<EMAIL_ADDRESS>to resolve the matter.
Hello, this is Telegram Support.
bytescare.com made a complaint https://t.me/mltb_official is infringing on their copyright.
We kindly ask you to contact the copyright holder via email<EMAIL_ADDRESS>to resolve the matter.

@levlam now my channel and group banned due fake copyright.
Congratulations telegram.

@levlam do u really don't know who can help with this issue? Why this ignorance?

See https://en.wikipedia.org/wiki/Digital_Millennium_Copyright_Act#Title_II:_Online_Copyright_Infringement_Liability_Limitation_Act: "OSPs must ... block access to alleged infringing material. OCILLA also includes a counternotification provision that offers OSPs a safe harbor from liability to their users when users claim that the material in question is not, in fact, infringing.". You already received all relevant information. If someone tries to threaten and demand money from you, then you need to report them to the Police and not to Telegram.

See https://en.wikipedia.org/wiki/Digital_Millennium_Copyright_Act#Title_II:_Online_Copyright_Infringement_Liability_Limitation_Act: "OSPs must ... block access to alleged infringing material. ... OCILLA also includes a counternotification provision that offers OSPs a safe harbor from liability to their users when users claim that the material in question is not, in fact, infringing.". You already received all relevant information. If someone tries to threaten and demand money from you, then you need to report them to the Police and not to Telegram.

Bro u r out of the topic!
I'm telling u that there is someone who MOCK telegram and did fake reports and telegram accepted all.
I told he was reporting my commits updates channel.
Like:
"Fix yt-dlp user settings"
Like this message he reported it as copyright compliant.
I want someone to talk with him about this. And no one responding no one helping

Bro in short there is NO copyright infringement in my channel or group.  Yeah don't say its different. No it's the same! I'm posting commits titles with some details on my telegram channel.

Yes imagine u r getting copyright infringement on this commit title what u will do with github? Switching to gitlab ? Or report?

The site is the same as you can see.

There is no message that violates copyright laws.

The site is the same as you can see.

There is no message that violates copyright laws.

@levlam It's also happened to me.
In my group and updates channel, where we discuss our problems, I have also been granted fake copyright.
We all know who is responsible for all this, and there is nothing we can do about it.

Same site! I hope u r sure there is no content against copyright in ur channel ?

The site is the same as you can see.

There is no message that violates copyright laws.

I will post here more for explanation i hope he don't close this issue. I'm  sharing the link to the dead support every where.
Here some of my identical posts on my channel.

@anasty17 GitHub issues can never be a solution for anything not related to TDLib. Even Telegram developers can't do anything with your issues outside development circle.

@anasty17 GitHub issues can never be a solution for anything not related to TDLib. Even Telegram developers can't do nothing with your issues outside development circle.

What is actually ur problem?  I know what github devs have no relation with this but i have no one to ask or to report for!
My first question was: who can help with this issue.

And your last words was "I was sharing this issue link everywhere.", and my message is clear. GitHub issues is for TDLib related issues only not for other issues.

And your last words was "I was sharing this issue link everywhere.", and my message is clear. GitHub issues is for TDLib related issues only not for other issues.

Thanks for telling me thing I already know! BUT THERE IS NO OTHER SOLUTION! I was trying to know someone who can help.

Sharing a link of an unrelated issue to TDLib is not a solution. And if you got banned again for any reasons, it's not the right place to re-do what you did.

Sharing a link of an unrelated issue to TDLib is not a solution. And if you got banned again for any reasons, it's not the right place to re-do what you did.

Then gjve solution! There is no support team in telegram.
So if u r the owner of this repo and saying this so the best solution is the switching from telegram to another platform. If there is any fake complaint u r facing and no ome can restore ur channel or group so why people would use telegram.
And if I'm developing this repo and i know that there is no one can help the users then if i care to telegram and the user provides evidence for everything he said, then i will report to someone i know he can solve this problem. IF I CARE TO TELEGRAM. Its just forwarding and it will not take much time from me.

Sharing a link of an unrelated issue to TDLib is not a solution. And if you got banned again for any reasons, it's not the right place to re-do what you did.

Instead repeating this again and again give solution to who face those fake reports and i will share it in my channel.
I know what u r saying and i agree with u 100%. If there is no way to contact telegram for reporting issue like this then i will spam everywhere until telegram provide way to report issue with expectation of response.
And the guy who did this in my channel will re-do those fake strikes again. I know him but i don't have evidence on him.
So please if u have helpful word then say it. If u don't then don't reply here please.

@anasty17 congratulations on unblocking your channel.
i have similar stories to share. 🤣  since it is being unwelcomed here. i will stop with a message.

telegram team should pay attention to verify legitimacy of the complaint, if you can not verify the complaint yourself just do not take any action nor force channel/group owners to contact fake complainants through email.

I also became victim of the same fraud recently. My channel (http://t.me/data_science_enthusiasts) is suspended by Telegram on August 14, 2023 due to copyright infringement. As per their communication I did reached out to concerned party http://bytescare.com/ and have mailed an apology since they identified a post on the channel that had caused copyright violation but they did not responded back to my mail. I had mailed to Telegram support team but they too did not responded. At first I was feeling that it was a mistake from my end but after reading the thread I realised it is a case of fraud. I don't know will my channel be reinstated and if yes then when? I'm still waiting for my channel status to be restored, 20 days have passed.

@levlam channel restored, i don't how but thanks for not deleting or closing this issue.
I was sharing this issue link everywhere.
Thanks ♡

Seems they block channel for 30 days from the first copyright violation message ? Was this true in your case ?

@levlam channel restored, i don't how but thanks for not deleting or closing this issue.
I was sharing this issue link everywhere.
Thanks ♡

Seems they block channel for 30 days from the first copyright violation message ? Was this true in your case ?

Don't comment here.  Change ur channel username

@levlam channel restored, i don't how but thanks for not deleting or closing this issue.
I was sharing this issue link everywhere.
Thanks ♡

Seems they block channel for 30 days from the first copyright violation message ? Was this true in your case ?

Did your channel get blocked? We got similar message against our telegram group.

(url  ) I also became victim of the same fraud recently. My channel (http://t.me/data_science_enthusiasts) is suspended by Telegram on August 14, 2023 due to copyright infringement. As per their communication I did reached out to concerned party http://bytescare.com/ and have mailed an apology since they identified a post on the channel that had caused copyright violation but they did not responded back to my mail. I had mailed to Telegram support team but they too did not responded. At first I was feeling that it was a mistake from my end but after reading the thread I realised it is a case of fraud. I don't know will my channel be reinstated and if yes then when? I'm still waiting for my channel status to be restored, 20 days have passed.

Did your channel get blocked? Should we be concerned as we recieved similar message but doesnt really tell what copyright was infringed

@sujeetmishra89

This is still a concern. I and my friends are getting similar copyright violation without reason. Just violation.

I have the same issue and already 45 days no reply

I have the same issue and already 45 days no reply

Telegram doesn't have support team

Another fake report but this time no mentioned message
Telegram: Hello, this is Telegram Support.
bytescare.com made a complaint https://t.me/mltb_official_group is infringing on their copyright.
We kindly ask you to contact the copyright holder via email<EMAIL_ADDRESS>to resolve the matter.
Hello, this is Telegram Support.
bytescare.com made a complaint https://t.me/mltb_official is infringing on their copyright.
We kindly ask you to contact the copyright holder via email<EMAIL_ADDRESS>to resolve the matter.

Same with me my groups got fake copyright strike by aiplex.com who protects Indian websites copyrighted content
Users share them to create fake copyright strike


Funding option for SIGNA on the Signum Network

Funding option for SIGNA on the SIGNUM Network.

People could do value for value on No Agenda using SIGNA.   Currently each SIGNA is worth about 6 to 7 Sats each.
Signum is the blockchain
Signa is the cryptocurrency
SIGNA is the symbol/ticker
Truly sustainable mining
Proof-of-Commitment (PoC+) is the next evolution of the Proof of Capacity (PoC) consensus algorithm. Using available disk space, you can sustainably secure the network and boost your share of mining rewards by committing Signa on-chain.
Easy smart contracts
With our Signum SmartJ platform, developers can write, debug, and deploy smart contracts using only Java.
Smart Tokens for your applications
The Signum network is ready for a booming tokenization, with a supportive transaction framework
What is PoC+?
Signum introduced the world to the Proof of Commitment (PoC+) decentralized consensus as the evolution of the Proof of Capacity (PoC) consensus.
Different than the well-known Proof of Work (PoW) consensus used by Bitcoin and many other coins, which require special power-hungry equipment, PoC+ simply uses available disk space.
This innovative consensus offers a new way for miners to increase their effective storage capacity - committing a Signa balance (stake) in their account. This will help secure the network and improve their chances of earning mining rewards. PoC+ is a greener option since effective capacity can be increased without purchasing more equipment.
The mining process in the PoC+ consensus is so effective and has such low hardware requirements that any consumer-grade PC can be used to mine.  The user will not even notice that the mining process is happening, aside from the occasional blinking LEDs from the hard disk drive (HDD)!

Related to #444

Yes, please read #444 and follow that path.


Fix: #285

Description

Fixes #285
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)

What's Changed

[ ] react-copy-to-clipboard to change the behavior of copy-to-clipboard.tsx file.

I have updated the file


Failed to locate opened application window with appId because of RintimeBroker running first

When I am running my application using WAD on remote machine which is working slowly
Then RuntimeBroker running first
And I got an error:
HTTP/1.1 500 Internal Error
Content-Length: 191
Content-Type: application/json

{"status":13,"value":{"error":"unknown error","message":"Failed to locate opened application window with appId: '''!App, and processId: 24472"}}

And Application is runnung in suspended mode, even when this option is switched off in Windows configuration
As I understood WAD is searching for specified window for first sub-process, but when RuntimeBroker is running first and has no window, then I receive this error

I am running into the same situation. If I run the app debug... then it seems to work fine. -- but when I run normal its like the splash screen starts nut takes too long and the runtime broker suspends the process?
any help would be appreciated

Hi @IevgeniiAstanin and @HughJohnson69  ,
This could be due to the splash screen  interfering with the session. Recommend taking a look at the Wiki if you haven't already.
Can you share which process the processID is referencing?
Thanks


disable trampoline in dbundle

/cc @segiddins

:hourglass: Testing commit c90a6c1 with merge dec1548...

:broken_heart: Test failed - status

@homu retry

:hourglass: Testing commit c90a6c1 with merge b44d763...

:sunny: Test successful - status


v3rpc: bytes-key map look-up gc optimization

This change https://github.com/golang/go/commit/f5f5a8b6209f84961687d993b93ea0d397f5d5bf just got merged to go1.6.1 where Go does special optimization for x = m[string(k)] where k is []byte.

@gyuho This optimization exists since 1.3 :-) The tag go1.6.1 on the page you linked is just the latest tag that contains this commit, but not the first. The first tag with this commit was go1.3beta1.

@nuss-justin You are right. Thanks for correction!

Have you got any benchmark or profiling results to show how much performance improvements it gives us?

The code changes lgtm.

@xiang90 simple benchmark snippet here:
package main

import (
	"fmt"
	"time"
)

func main() {
	var (
		key = []byte("hello world")
		ks  = string(key)
	)
	{
		rmap := make(map[string]struct{})
		st := time.Now()
		for i := 0; i < 10000000; i++ {
			_, ok := rmap[string(key)]
			_ = ok
		}
		fmt.Println(time.Since(st)) // 62.630954ms
	}
	{
		rmap := make(map[string]struct{})
		st := time.Now()
		for i := 0; i < 10000000; i++ {
			_, ok := rmap[ks]
			_ = ok
		}
		fmt.Println(time.Since(st)) // 40.5739ms
	}
}

If we want benchmarks specific to our use case, I can add benchmark tests for func checkRequestDupKeys(reqs []*pb.RequestUnion) error {.

I am more interested in how this improves etcd overall performance. But I guess it is OK... It does not complicate etcd code anyway.

So the patch affects our Txn API where it calls checkRequestDupKeys. My simple bench script shows
With patch:

19.603312727s
19.540302976s
19.671864464s
19.51260928s
19.581460839s

avg: 19.581910057199998
Without patch:

19.674088137s
19.724849562s
19.667308366s
19.755512172s
19.688727127s

avg: 19.7020970728
Code:
package main

import (
	"fmt"
	"log"
	"time"

	"github.com/coreos/etcd/clientv3"
	"golang.org/x/net/context"
)

var (
	dialTimeout    = 5 * time.Second
	requestTimeout = 1 * time.Second
	endpoints      = []string{"localhost:2379"}
)

func main() {
	var total time.Duration
	for i := 0; i < 10000; i++ {
		if i%1000 == 0 {
			fmt.Println(i)
		}
		cli, err := clientv3.New(clientv3.Config{
			Endpoints:   endpoints,
			DialTimeout: dialTimeout,
		})
		if err != nil {
			log.Fatal(err)
		}

		kvc := clientv3.NewKV(cli)

		_, err = kvc.Put(context.TODO(), "key", "xyz")
		if err != nil {
			log.Fatal(err)
		}

		st := time.Now()
		ctx, cancel := context.WithTimeout(context.Background(), requestTimeout)
		_, err = kvc.Txn(ctx).
			If(clientv3.Compare(clientv3.Value("key"), ">", "abc")). // txn value comparisons are lexical
			Then(clientv3.OpPut("key", "XYZ")).                      // this runs, since 'xyz' > 'abc'
			Else(clientv3.OpPut("key", "ABC")).
			Commit()
		cancel()
		if err != nil {
			log.Fatal(err)
		}
		took := time.Since(st)
		total += took

		cli.Close()
	}
	fmt.Println("took:", total)
}

The performance is slightly better.


[SECURITY] Attempt to subtract with overflow in SWAP opcode

Our team (@fuzzinglabs) discovered a bug was identified in the op_swap function, we can trigger an overflow due to the lack of checks.
Root cause
pub fn op_swap(
        &mut self,
        current_call_frame: &mut CallFrame,
        op: Opcode,
    ) -> Result<OpcodeSuccess, VMError> {
        self.increase_consumed_gas(current_call_frame, gas_cost::SWAPN)?;

        let depth = (op as u8) - (Opcode::SWAP1 as u8) + 1;

        if current_call_frame.stack.len() < depth as usize {
            return Err(VMError::StackUnderflow);
        }
        let stack_top_index = current_call_frame.stack.len();
        let to_swap_index = stack_top_index
            .checked_sub(depth as usize)
            .ok_or(VMError::StackUnderflow)?;
        current_call_frame
            .stack
            .swap(stack_top_index - 1, to_swap_index - 1);

        Ok(OpcodeSuccess::Continue)
    }

stack_top_index - 1 we trigger an underflow because there is no check if stack_top_index ≠ 0.
Step to reproduce
Payload
[48, 144]

ADDRESS
SWAP1

Add to [test](https://github.com/lambdaclass/lambda_ethereum_rust/blob/main/crates/vm/levm/tests/tests.rs) :
#[test]
fn test_overflow_swap() {
    let mut vm = new_vm_with_bytecode(Bytes::copy_from_slice(&[48, 144]));
    let mut current_call_frame = vm.call_frames.pop().unwrap();
    vm.execute(&mut current_call_frame);
}

Backtrace
---- tests::test_overflow_swap stdout ----
thread 'tests::test_overflow_swap' panicked at crates/vm/levm/src/opcode_handlers/exchange.rs:32:40:
attempt to subtract with overflow
stack backtrace:
   0: rust_begin_unwind
             at /rustc/59e2c01c2217a01546222e4d9ff4e6695ee8a1db/library/std/src/panicking.rs:658:5
   1: core::panicking::panic_fmt
             at /rustc/59e2c01c2217a01546222e4d9ff4e6695ee8a1db/library/core/src/panicking.rs:74:14
   2: core::panicking::panic_const::panic_const_sub_overflow
             at /rustc/59e2c01c2217a01546222e4d9ff4e6695ee8a1db/library/core/src/panicking.rs:181:21
   3: ethereum_rust_levm::opcode_handlers::exchange::<impl ethereum_rust_levm::vm::VM>::op_swap
             at ./src/opcode_handlers/exchange.rs:32:40
   4: ethereum_rust_levm::vm::VM::execute
             at ./src/vm.rs:207:21
   5: lib::tests::test_overflow_swap
             at ./tests/tests.rs:82:5
   6: lib::tests::test_overflow_swap::{{closure}}
             at ./tests/tests.rs:79:24
   7: core::ops::function::FnOnce::call_once
             at /rustc/59e2c01c2217a01546222e4d9ff4e6695ee8a1db/library/core/src/ops/function.rs:250:5
   8: core::ops::function::FnOnce::call_once
             at /rustc/59e2c01c2217a01546222e4d9ff4e6695ee8a1db/library/core/src/ops/function.rs:250:5

Thanks! Fixed in https://github.com/lambdaclass/lambda_ethereum_rust/pull/1095.


Encoding the key properly

Simple question.  The key you get from google looks like this:

The docs say:

Note once again that the privateKeyData returned is a base64-encoded string representation of the TYPE_GOOGLE_CREDENTIALS_FILE value (JSON or P12 key/credentials). You can store the base64-decoded privateKeyData in a file with the appropriate extension (either .p12 or .json) and use it as input to client libraries or the gcloud auth activate-service-account command.

So does this mean we encrypt the entire .json file above or we copy only the private_key or private_key_id out into its own .json file and encrypt that?

Nevermind it's obviously the entire file.  I thought I was doing something wrong because it kept saying I don't have permissions and I know the roles were setup ok for that service account.  It looked like it was generating the same key every time I created a new key during trial and error and just something on my end.  Disregard.


travis: add initial checks

Starting with git commit validation.
Travis integration will still need to be enabled.
Signed-off-by: Vincent Batts<EMAIL_ADDRESS>

Lgtm

@vbatts go ahead and merge.
@caniszczyk how do admin rights work on this github org? We need to setup travis-ci

@philips I enabled Travis CI for the project


Add a way to use syntaxnet's engine as a parser

I've already installed syntaxnet but found that syntaxnet only provides POS and syntax parser, eg. it is not possible to find the lemma for words. Is there a way to connect spacy with syntaxnet as you mention in your blog post?

I tried to do this, but it's quite hard. I spent days trying to read SyntaxNet's code, and concluded I'd have to write some new C++ just to feed SyntaxNet a string from Python instead of a filename from which to read the text. Has this changed at all?

Hi, haven't read the complete source code yet. I'm piping in parser_eval.py from stdin as in the provided demo.

This would be incredible, just wondering how viable it actually is tho (since I have yet to see anyone using syntaxnet without piping between stdin and subprocess stuff)

Even more tantalising now that they released models for 40 more languages.

Honestly the BIST parser is much better. Yes it's less mature, but the
accuracy is the same and there are almost no hyper-parameters. Training on
the other Universal Dependencies corpora should be no problem.
In the meantime, it's probably easiest to pipe through SyntaxNet and set
the parses onto spaCy Doc objects with the doc.from_array method. A
deserialiser that reads syntaxnet's protobuf format would also be super
nice.
On Wednesday, August 10, 2016, Adam Mathias Bittlingmayer <
<EMAIL_ADDRESS>wrote:

Even more tantalising now that they released models for 40 more languages.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/spacy-io/spaCy/issues/432#issuecomment-238823465, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHr7ZmZCr8bBu4glT5BiHj3ra7QsrJaYks5qeaMbgaJpZM4I6TWT
.

Ps: anyone feeling a little exploratory can also find an implementation of
a feed-forward neural CRF parser in the july16 branch. This requires the
parser_nn_2016 branch of thinc. The script to train it is at
bin/parser/conll_train.py
This model is broadly similar to the most accurate syntaxnet model. I've
been using it to train very low memory models. So far I've managed to get a
parser down to around 60mb, beating the accuracy of the current linear
model. This would cut spaCy's memory use down to under 500mb.
The model naturally suports a cache, so users can control the space/speed
tradeoff at runtime. I haven't benchmarked for efficiency yet, but the
model is training in a few hours on a single CPU thread, so it can't be too
terrible.
There are really only a few outstanding issues. The main one is a memory
error --- it's crashing on exit, so I've stuffed something up somewhere.
Windows support will also be a huge mess. I've also found a couple of bugs,
and there are hyper-parameters to tune...
As I mentioned in another thread, I'll be mostly afk the next week, as I'm
out of town. And I've been making promises about this damn neural network
model for too long :). There will be tonnes to do when I get back, and some
annoying resource constraints to juggle, so I'm reluctant to promise a
release date. But I'll definitely be saying more about it when I get back
:).
On Wednesday, August 10, 2016, Matthew Honnibal<EMAIL_ADDRESS>wrote:

Honestly the BIST parser is much better. Yes it's less mature, but the
accuracy is the same and there are almost no hyper-parameters. Training on
the other Universal Dependencies corpora should be no problem.
In the meantime, it's probably easiest to pipe through SyntaxNet and set
the parses onto spaCy Doc objects with the doc.from_array method. A
deserialiser that reads syntaxnet's protobuf format would also be super
nice.
On Wednesday, August 10, 2016, Adam Mathias Bittlingmayer <
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Even more tantalising now that they released models for 40 more
languages.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/spacy-io/spaCy/issues/432#issuecomment-238823465,
or mute
the thread
<https://github.com/notifications/unsubscribe-auth/
AHr7ZmZCr8bBu4glT5BiHj3ra7QsrJaYks5qeaMbgaJpZM4I6TWT>
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/spacy-io/spaCy/issues/432#issuecomment-238838357, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAvrbWhVzsyUSjrq1terI6-QyLj8Owkuks5qebQqgaJpZM4I6TWT
.

Thanks for the guidance as always
Speaking for my own project, languages and convenience matter more than accuracy or underlying implementation, and reliable guidance as much as actual updates.
My team has tried SyntaxNet and it leaves much to be desired on these fronts (just ease of use and training on and parsing additional languages, decided to give up on reducing footprint size), so there is some opportunity here.


Improve error message for posix_spawn failure in PlatformBuilder.kt

The current implementation at PlatformBuilder.kt:165 shows the error message "io.matthewnelson.kmp.file.IOException: posix_spawn failure in pre-exec/exec step. Bad arguments?" on error. This message is not very informative and makes it difficult to debug the issue.
I suggest enhancing the error message by using strerror to get the specific error information when an error occurs. This will provide more detailed information about the cause of the error, making it easier to identify and fix the problem.
Proposed Change:
Update the error handling to include strerror for more informative error messages. For example:
val errorMessage = strerror(posix_errno)
throw IOException("posix_spawn failure in pre-exec/exec step: $errorMessage")

This change will help developers better understand the nature of the error and take appropriate action to resolve it.
Thank you for considering this improvement.

posix_spawn errno is already checked via the .check extension function.
There are 2 stages to how the system executes posix_spawn, the first of which returns an immediate error and sets errno. If there is a failure after that (indicative of pid being -1), there is no way to determine what that error was attributed to, thus the message of "Bad arguments?".
Looking at the code though, I do see an issue that could be improved upon which I.all open a separate issue ticket for.
What was the command and arguments you were running, and what platform?


Can't close keyboard in send in wallets

How to reproduce

Go to wallets then navigate to send
Enter any field
Try to close the mobile keyboard by navigating outside the field. It won't get closed.

Work Completed:

when user clicks outside the keyboard in the page, it will close.

Verified on Android 14,
Threefold-Connect: 4.0.0-184
APK:12-12-2024
The keyboard is closed normally by tapping outside the field

Related Link:
-https://app.testlodge.com/a/26076/projects/40893/suites/206114?expand_section=407473#case_4048983


SQL collection creation: Request Units have different thresholds based on fixed vs unlimited size

Fixed : 400 - 10,000
Unlimited : 1,000- 10,000
Screenshots:
Fixed-

Unlimited-

See #295

Just an implementation detail. Closing it.


slim-buster image doesn't work properly (after 3.9)

works:
FROM python:3.8-slim-buster

RUN pip install confluent_kafka

does not work
FROM python:3.9-slim-buster

RUN pip install confluent_kafka

throws errors on missing gcc
when after installing gcc, it throws errors on librdkafka, then after installing that you try installing confluent_platform. THEN you figure out that it's the python docker image thats the issue, not your dependencies...
attachment to document my steps to reproduce (using a gitlab-runner and gcr.io/kaniko)
error.log

https://github.com/docker-library/docs/tree/master/python#pythonversion
For python:slim you'll want to install gcc, do pip install, and then uninstall gcc in the same layer https://github.com/docker-library/python/issues/488#issuecomment-632744287

It sounds like confluent_kafka might not support Python 3.9 (yet?)

Yeah seems that way if the latest release is July and 3.9 was released October 5. And the gcc error would be due to the manylinux1 wheels not being compatible, such as if the version is unsupported
I don't see anything in their issues mentioning 3.9

I will redirect this issue to confluent_kafka instead. The error messages just sound relatively system-ish, to have been issues with confluent, I apologize for hitting the wrong forum.


display: -webkit-box breaks layout in newer versions of Safari

If I use a site like:
http://the-echoplex.net/flexyboxes/

When I add
display: flex

It adds the following prefixes:
display: -ms-flexbox;
display: -webkit-flex;
display: flex;

It does not include display: -webkit-box which breaks my container layout.
Instead of my boxes fitting inside my container, when -webkit-box is added, the boxes attempt to stretch out my container...only in Safari (using 8.0.5).
Is there a way to change this with Autoprefixer?

How Flexyboxes is affected to Autoprefixer?

What I mean is this:
display: -webkit-box

Is for old Safari (/* OLD - iOS 6-, Safari 3.1-6 */) but it appears that newer versions of Safari, if they see -webkit-box, it breaks the flex layout. Omitting this allows the layout to work properly in newer browsers.
Is it possible to have a setting to disable auto-prefixer from adding:
display: -webkit-box

If I add:
display: flex

Just don't use Flexuboxes and use only Autoprefixer, it will do all correctly

NVM, my bad (sorry). Looks like Safari just started to ignore -webkit-box again.


IE11 Cannot redefine non-configurable property 'includes'

Hi,
After including vue-toasted library ie11 crashes. I think it's because it's redefining includes in this block
// string includes polyfill
if (!String.prototype.includes) {
	Object.defineProperty(String.prototype, 'includes', {
		value: function value(search, start) {
			if (typeof start !== 'number') {
				start = 0;
			}

			if (start + search.length > this.length) {
				return false;
			} else {
				return this.indexOf(search, start) !== -1;
			}
		}
	});
}

Hi, I'm having the same problem on the last version of the package.
Any updates?

+1 here. The supplied PR seems to fix it.


Cannot load default config file

C02QH2D7G8WM:opencl userone$ ./histogram_opencl
Fontconfig error: Cannot load default config file
Freetype face initilization(3001)
C02QH2D7G8WM:opencl userone$ ./plot
-bash: ./plot: No such file or directory
C02QH2D7G8WM:opencl userone$ ./plot3_opencl
Fontconfig error: Cannot load default config file
Freetype face initilization(3001)

Can you provide more information about your system? Are you using Linux or OSX? And which branch of forge are you testing?

@dendisuhubdy Can you please let us know if you are still facing the issue you mentioned above.

Closing, please reopen if you are still facing the problem.


Technopig/crippling shout

Everything looks really solid aside from the fact that it applies Howl of Terror on top of Cripple. Please test that your Targets Allowed change worked then let me know.

Tested, will start making it a part of the practice going forward.  all working okay. its removing the AOE on the ability that avoids it

Well done. Not quite there yet. With a range of 0, the ability can still target the caster. So:

I would recommend changing Targets Allowed to Bridge or something.

hmm didn't notice that ill make it cripple and it should be fine


Add timing sections to /draw/world

Seems like the engine/world's Draw function has no timing.Section calls. Let's fix that for better self profiling.

	/draw/world                                      46.853434ms (calls 68*689.021µs, frames 60*780.89µs, worst frame 1.998808ms)
	/draw/world/debug                                8.69372ms (calls 68*127.848µs, frames 60*144.895µs, worst frame 405.362µs)
	/draw/world/entities                             1.332028ms (calls 68*19.588µs, frames 60*22.2µs, worst frame 85.556µs)
	/draw/world/fill                                 1.046791ms (calls 68*15.393µs, frames 60*17.446µs, worst frame 39.531µs)
	/draw/world/tiles                                25.371846ms (calls 68*373.115µs, frames 60*422.864µs, worst frame 1.209174ms)
	/draw/world/visibility_mask                      8.9587ms (calls 68*131.745µs, frames 60*149.311µs, worst frame 219.443µs)
	/draw/world/visibility_mask/compute_mask         6.375808ms (calls 61*104.521µs, frames 60*106.263µs, worst frame 157.005µs)
	/draw/world/visibility_mask/copy_outside         1.077312ms (calls 61*17.66µs, frames 60*17.955µs, worst frame 25.702µs)


Run as Android Foreground Service

I've read some of the discussions about background mode (#127 & #151).  I have a basic RN project that starts an android Foreground Service and runs BluetoothLeScanner to scan for devices.  I only need to capture advertising packets (no connections, no GATT).
Can I use this project from my RN bridge foreground service?

If I start an Android Foreground Service, then on the JS side, periodically call startDeviceScan, the JS code is only executed while the app is active even though the Foreground Service is active and shown in notifications. If the screen times out or the user switches away from my app, the JS code stops running even though the Foreground Service is active.

@rgstephens You are not calling startDeviceScan from any Component do you?

Yes, I was.  Good question.  I'm going to make a sample app and do the call the from the root.  This example in github currently starts a loop in index.js that logs the time to console.log and stores it in redux every 10 seconds. I start the loop before the root component is registered.
I have a switch on the main component that starts a foreground service.  Whether the service is started or not, the 10 second timer updates stop when the app loses focus, even though the Android Foreground Service notification shows that it is still running.
I'm going to update this loop to call startDeviceScan at a regular interval.
setInterval(function() {
    console.log('*** index.js setInterval', new Date());
    store.dispatch({ type: 'TIMESTAMP' });
  }, 10000);

@DariuszAniszewski I now have a complete example on github which calls startDeviceScan from the root in a loop every 10 seconds and has a switch to start/stop a foreground service.  The scans still stop when I switch away from the app even though the foreground service is running.

cc: @dariuszseweryn ^^

@rgstephens Do you still have an issue?

Yes, I still have the problem. The sample app posted announce does not work in the background.

still facing this same issue too


Goaccess 0.8.5 - Fatal Error with nginx default log

if i type
zcat -f /var/log/nginx/-access.log* | goaccess
got this error:
GoAccess - version 0.8.5 - Jan 16 2015 17:05:32
Fatal error has occurred
Error occured at: goaccess.c - main - 832
Nothing valid to process.
i'm sure i'm editing the right config file:
date_format %d/%b/%Y:%T %z
log_format %h %^[%d:%^] "%r" %s %b "%R" "%u"
log sample:
<IP_ADDRESS> - - [16/Jan/2015:16:53:54 +0100] "GET / HTTP/1.1" 200 3740 "http://example.com/" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.99 Safari/537.36"

Since goaccess does not use time specifiers at the moment, you can
remove them from the date_format.
date_format %d/%b/%Y
That should do it.
On Fri, Jan 16, 2015 at 11:00 AM, radar155<EMAIL_ADDRESS>wrote:

if i type
zcat -f /var/log/nginx/-access.log* | goaccess
got this error:
**GoAccess - version 0.8.5 - Jan 16 2015 17:05:32
Fatal error has occurred
Error occured at: goaccess.c - main - 832
Nothing valid to process.**
i'm sure i'm editing the right config file:
date_format %d/%b/%Y:%T %z
log_format %h %^[%d:%^] "%r" %s %b "%R" "%u"
log sample:
<IP_ADDRESS> - - [16/Jan/2015:16:53:54 +0100] "GET / HTTP/1.1" 200 3740 "http://example.com/" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.99 Safari/537.36"
—
Reply to this email directly or view it on GitHub.

Closing this. Feel free to reopen it if needed.


Store state differs between first and successive ssr in with-redux-saga example

Example name
with-redux-saga
Describe the bug
Store state differs between first and second (actually all successive) server-side renders.
To Reproduce

npx create-next-app --example with-redux-saga with-redux-saga-app
cd with-redux-saga
npm install
npm run dev
Open web browser and check the store value for placeholderData (has data)
Reload the web browser and check the store value for placeholderData (is null)

Only solution is to restart the server.
Expected behaviour
I expected placeholderData to have the same value after every browser reload.

Reverting #6300 solves this issue. In effect reverting back to.
"next-redux-saga": "3.0.0",
"redux": "4.0.0",
"redux-saga": "0.16.0"

@janjarfalk i'll have a look at it.

Well, obviously saga task is canceled on server side as seen here on lines index.js:19-22. Therefore, takeLatest(actionTypes.LOAD_DATA, loadDataSaga) is not caught and no data is loaded on second (and continuous) server side render. However the task was already canceled before this update, see index.js:24-26.
So, @bmealhouse I wonder how the task survived on server side within version "next-redux-saga": "3.0."?

Ah probably because of what Andarist mentioned here, that dispatch(END) would not propperly close the task. Then we're screwed.. The whole V3 API built on a bug and in order to run it we would have to change behaviour in "redux-saga"..

I have te solution.. facepalm will implement it this evening! @bmealhouse @janjarfalk

The error must be located in the example. See this issue in next-redux-saga for more information.

You might be right, but I didn't find this issue in the with-redux-saga example. I just used it to reproduce it. I see it in several, unfortunately private, projects. The setup in those projects is however derived from with-redux-saga in one way or another.
I am having problems geting my-little-streaming-dream (the authserver) up and running. Do you have something smaller where it works?

If I simplify the requestUserInfo in my-little-streaming-dream to...
function* requestUserInfo(action: RequestUserInfoAction) {
  yield put(setUserInfo({name:'Timon'}));
}

... it does reproduce the issue.

Huh, heavy mistake on my side.. didn't run yarn install on my laptop and was still using "redux-saga": "0.16.2"..
You're right @janjarfalk, problem still exists.

No worries! I appreciate the time and effort you are putting into this! I wish I could be of more help. My current knowledge of next-redux-saga, next-redux-wrapper and the changes in redux-saga is too little.

@janjarfalk  I got the error and I even know how to fix it. Will update the example and the documentation on next-redux-saga shortly :)

Can you test the example again @janjarfalk?

@bbortt Works like a charm! Thank you very much!

@bbortt - After looking through the PR it was a very straightforward solution. Excellent work!

Just a short thing before you go...
As I said before my knowledge of next-redux-saga, next-redux-wrapper and the changes in<EMAIL_ADDRESS>is little. So.. not knowing you would fix this so quickly (thanks again!) I decided to copy-paste-rewrite next-redux-saga, next-redux-wrapper from scratch.
I think it would make sense to drop next-redux-wrapper as a dependency. It doesn't do much and the code was easier for me to reason about when it was merge with next-redux-saga.
https://gist.github.com/janjarfalk/e0ed2566143d9ed02cae7c9a1c7bdcbb

@janjarfalk - I definitely agree with this approach. However I don't plan to make those changes myself, since I'm not using redux or saga in any of my current projects. If anyone would like to submit a PR for this I would be happy to get this in.

@janjarfalk can you submit a pr containing those changes?
I would collaborate on creating tests and I also see one or two things in your gist which I want to check first.

@bbortt There are a lot of breaking changes in the gist - no config, no debug etc. I think we can and should drop next-redux-wrapper without breaking changes. Let me find some time and I'll do a proper merge and PR.


Practically any function/method from matplotlib will raise the "reportUnknownMemberType" error

From Marketplace review:

Robert Buch
Using practically any function/method from matplotlib will raise the "reportUnknownMemberType" error, because matplotlib never type-hints the "**kwargs" parameter. In fact, you basically can't use "**kwargs" yourself unless every such parameter is of the same type. If they do have different types, you are better off just disabling the report. (Tip: Use the comment "# pyright: ignore [reportUnknownMemberType]" to disable it for just one line.)

Unable to repro. Robert, if you can provide a code sample that repros this issue, we can reopen this issue and investigate further.


Improve failure reporting on missing EC2 metadata

When application runs outside of EC2 instance and region is not specified in application properties, startup fails with IllegalStateException. This can be done in more developer friendly way using Spring Boot failure analysis which would give developer hint what's the action he has to take in order to solve the problem

@maciejwalkowiak - how we can handle it when running locally?

When running locally I would recommend providing value for static region.

@maciejwalkowiak Can I check this one?

@giova333 would be great!

Finally, I managed to check this issue :)
Since the thrown exception is IllegalStateException, I think it's a bad practice to analyze such generic exception in FailureAnalyzer. It'd great to create a dedicated exception RegionNotFoundException or something like that, and use it in all RegionProvider implementation when unable to retrieve the region. And for different RegionProvider implementation we can provide differently action messages:
StaticRegionProvider -> the region provided in cloud.aws.region.static is invalid
Ec2MetadataRegionProvider  -> application is running outside ec2 instance, set up the region explicitly using cloud.aws.region.static
DefaultAwsRegionProviderChainDelegate  -> Provide the region either via env variable AWS_REGION_ENV_VAR, system property aws.region or ~/.aws/config file.
The only thing that bothers me is that introducing a new exception is a breaking change, so
I am wondering how should we deal with that.
@maciejwalkowiak WDYT?

I like this idea. What if we make this new exception type extend IllegalStateException. In such case it would not be breaking anymore, right?

That was my first thought but unfortunately, different region providers throw different exceptions:
Ec2MetadataRegionProvider  -> IllegalStateException
StaticRegionProvider  -> IllegalArgumentException
DefaultAwsRegionProviderChainDelegate -> propagates SdkClientException


az aks scale fails when using CustomVnet + kubenet

Describe the bug
When deploying a cluster with Custom VNET + kubenet, we are unable to run the scale command on it if we provided custom values for the serviceCIDR and/or the DNS service IP.
az aks scale -g aks-20180614f --name aks-20180614f --node-count 5

> Operation failed with status: 'Bad Request'. Details: Changing property 'networkProfile.dnsServiceIP' is not allowed.

To Reproduce
Deploy the cluster with custom serviceCIDR and custom DNS Service IP and Kubenet plugin.
Try to scale the cluster using az aks scale
Expected behavior
The cluster should create new vms in the specified CIDRs
Environment summary
Azure CLI > 2.0.61 but any should be
Additional context
This problem is probably underlying somewhere deeper than in the Azure CLI

I tested this and could not repro on latest. Could you confirm if you still experience on latest?

aks


Bootstrap4

Fixed jumbotron spacing, btn-outline-secondary and about.html, register.html issues.

This is looking great!
There seems to be a spacing/sizing issue with the header and alerts. I see the following after registering:

The above is not an issue on the /users/ page.

It only happens on the home page, such as when you register or log out:

It was a login form issue, fixed it, also renamed button, since it makes more sense now.

Thanks for looking into that @Hiyorimi .
It's still not 100% correct--the alert has unnecessary spacing around it:

The alert should also take the whole width.
Expected
(this is what it was before 66b97d5)

Actual

Since the cross is stretched to far to the right, I took the responsibility to wrap alerts in the container, to make it more neat IMO.
If spacing still seems unnecessary, I can remove it.

The alerts should align with the full width (as in the "Expected" screenshot above).

Any news ?


How low can you go: Will blase work for M dwarfs?

I cloned an M-dwarf spectrum across the entire HPF bandwidth, yielding 23112 lines.  I started with a 2800 K, logg=5 PHOENIX spectrum at native resolution.  Here is a snippet of what that looked like:

The cloning process captures lots but not all of the residuals.  I think we would need Gaussian processes to define the pseudo-continuum to get much better performance, since some lines chaotically devolve into pseudo-continuum right now.
Here is a snapshot of what real data looks like for an M-dwarf with comparable photospheric properties:

Key result: the imperfections in cloning are much smaller than the data-model mismatch.  That's about what I expected, but it's useful to see it quantified. This result means tuning the blase model to the data will yield a much better fit, but the interpretability gets tricky: each line may not map onto its adjusted value correctly.  In other words, I think the M-dwarf lines may be missing or out-of-order, not simply quasistatically off in amplitude and width.
In any case it'll be easier to demo the method with warmer M-dwarfs or K stars, that do not have this mapping confusion problem.

Update, I did a demo on an ultracool dwarf with Sonora and it "works" with the same proviso that interpretation becomes tricky.  Gaussian Processes for the pseudo-continuum estimation will likely be necessary to make this category of spectra truly excellent.


before_script and after_success steps

as in travis, very useful for slow stages that shouldn't affect the build status.

I think the 0.4 yaml and plugins should definitely help with emulating the behavior of after_success, since certain plugins can already run in "forked mode" which means they run async and don't impact the build result. An example of this are the notification plugins.
How does before_script work? Does the build block until before_script has started? If before_script fails does it fail the build? It would be helpful to understand a bit more about before_script and its intended use case

I actually don't know what the semantic difference is between before_script and script. Yes, it would fail the build. I guess it's more about code cleanliness. So I only really care about after_success but it was more a general query about phases.

cool, thanks for the clarification. Let's see how far we can get with 0.4 once it is available, and then we can augment at needed. The use case definitely makes sense

@bradrydzewski this can be reopened, since 0.4 is nearly ready and this issue is still actual.


Speed up tests: Use DELETE FROM instead of TRUNCATE for clearTable

https://github.com/silverstripe/silverstripe-framework/pull/8433 got merged, and then reverted because we need to fix our core reliance on hardcoded database identifiers. @sminnee suggested that this should be a major release change.

Regarding major release: I don't think hardcoded database identifiers in tests are part of our public API. We might get away with it if we put a couple of PHPUnit example fixes in our upgrading docs? Or we could allow people to opt in to this legacy behaviour?

We've already addressed this with DELETE FROM and SET AUTO_INCERMENT = 1. Job done!

This is fixed by https://github.com/silverstripe/silverstripe-framework/pull/8437

PRs merged


Create the Landing Page for the Website [HTML+CSS+Bootstrap+JQuery]

Type
Advancement
Description
Create the Landing Page for the Website using HTML, CSS, Bootstrap and JQuery. Refer this for the UI Wireframe that you would need to follow.
You will have maximum 7 Days to work on this. You don't need to work on the integration of Dark Theme for now.
Tools

HTML
CSS
Bootstrap
JQuery

Have you read the Contributing Guidelines on Pull Requests?
Yes

@HarshCasper I would like to work on this issue.

I thought this is the same task as issue no #106 have

I am assigning it to you @devanshiv123
@nooras The Page you developed was for the React Application.

@HarshCasper I want to work on this?

@devanshiv123 Are you working on this?
If not, I am assigning this to @Adyasha8105

@devanshiv123 Are you working on this?
If not, I am assigning this to @Adyasha8105

@HarshCasper No, I am facing some problems, you can assign it to someone else.

@Adyasha8105 I am assigning this to you. Kindly confirm if you will be working on this.

Can I take up this issue? But I'm not that familiar with jquery!

I'd like to work on this.

@nikita-1801 Would you be able to finish it up in 3 Days?

I'll try my best!


Incorrect error message for string concat in select clause

Here is the sample code:
        map<anydata> keyValsMap = {foo:"sss", bar:"ffff"};
        var x = from var k in keyValsMap.keys()
                let var v = keyValsMap[k]
                select k+"dd"

        io:println(x);

The screenshot with the diagnostic:

This is fixed in beta1


on_ready() event gets called twice

The on_ready event gets called in the BaseEventHandler start function:
https://github.com/TimMcCool/scratchattach/blob/b5d6321140ea1404c8e8068895d7e7aba3042672/scratchattach/eventhandlers/_base.py#L27
It also gets called in every _updater function:
https://github.com/TimMcCool/scratchattach/blob/b5d6321140ea1404c8e8068895d7e7aba3042672/scratchattach/eventhandlers/cloud_events.py#L33
https://github.com/TimMcCool/scratchattach/blob/b5d6321140ea1404c8e8068895d7e7aba3042672/scratchattach/eventhandlers/cloud_events.py#L89
https://github.com/TimMcCool/scratchattach/blob/b5d6321140ea1404c8e8068895d7e7aba3042672/scratchattach/eventhandlers/message_events.py#L23

This is a problem I have, it prints "ready" twice which is annoying.

will be fixed

fixed in v2.1.4

Thank You!


Misc/better CI

Updated the workflow to relax the protection settings on the master branch when tagging and revert them back at the end.
Since the user who tags the release is an admin and the action runs as that user, it should be allowed to push changes.
I've already added the admin token into the secrets section

Sneaky. So it will disable the protection for a bit while doing the work? Too bad GitHub doesn't provide a better way to do this. For most projects it's probably not an issue anyway.


Least privileged permissions bug? Application.ReadWrite.OwnedBy

Bicep version
run bicep --version via the Bicep CLI, az bicep version via the AZ CLI or via VS code by navigating to the extensions tab and searching for Bicep
Bicep CLI version 0.28.1
Describe the bug
A clear and concise description of what the bug is vs what you expected to happen
I have an service principal (ccoe-devops-iac02) with the API permission Application.ReadWrite.OwnedBy. It has least priviledge as we want it to be, so it can create and manage applications created by itself.
When I create an new application in the context of ccoe-devops-iac02, using the New-AzADApplication command, the ccoe-devops-iac02 service principal becomes the owner of the newly created application. And I'm able to manage the application with ccoe-devops-iac02, I can create an service principal etc.
This doesn't work when using Bicep.
To Reproduce
Steps to reproduce the behavior:
When I do the same using Bicep and the<EMAIL_ADDRESS>provider. I'm able to create the object, but the ccoe-devops-iac02 service principal isn't the owner. I'm not able to manage it. So when I run the Bicep template again I get the error: Insufficient privileges to complete the operation. I'm also not able to create an service principal to the application. This works fine with PowerShell.
The fix is grant the ccoe-devops-iac02 the API permission Application.ReadWrite.All, but we don't want that.
Additional context
Add any other context about the problem here.

I believe this is a duplicate of https://github.com/microsoftgraph/msgraph-bicep-types/issues/134 - please use this issue to track instead.


Property decorator documentation is inaccurate?

From the official docs about property decorators:
"The return value is ignored too."
But the following example returns a new descriptor, and it works fine:
class Greeter {
  @logProperty
  greeting;
}

function logProperty(target, key): any {
    var value;

    var getter = function() {
      console.log('getter');
      return value;
    };

    var setter = function(newVal) {
      value = newVal;
    };

    return {
      get: getter,
      set: setter,
      enumerable: true,
      configurable: true
    };
}

const g = new Greeter();
const b = g.greeting;

var __decorate = (this && this.__decorate) || function(decorators, target, key, desc) {
    var c = arguments.length,
        r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc,
        d;
    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
    else
        for (var i = decorators.length - 1; i >= 0; i--)
            if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
    return c > 3 && r && Object.defineProperty(target, key, r), r;
};

The return value is indeed not ignored here, and is in fact used as input for a call to Object.defineProperty().  I think the documentation is inaccurate.
However, a potential pitfall I see: the decorated property ends up being defined on the prototype and is no longer an instance property because:
__decorate([
    logProperty
], Greeter.prototype, "greeting", void 0);

If you add an initializer to the greeting property (greeting = 1;) it ends up calling the setter as a result, because the compiler moves the initialization to the constructor.

Yes, I'm working on a new article which explains how it works under the hood. That's how I found this behavior.

I meant "pitfall" in the sense of being a footgun, people might not expect instance properties to get moved to the prototype just because they added a decorator :wink:

It's been added to the prototype anyway:
    __decorate([
        logProperty
    ], Greeter.prototype <=====, "greeting", void 0);

Right, that's what I said - it only goes to the prototype if you have a decorator, otherwise it's strictly an own property of the instance.  Seemed surprising, even if it's intentional.

Agree.

Great find @NetanelBasal :clap:

Agree. They explain why in the docs:
"This is because there is currently no mechanism to describe an instance property when defining members of a prototype, and no way to observe or modify the initializer for a property. "

In fact, I also confused about the reason that described in the docs, very grateful if anybody can tell me by plain words 😄