Consistenly format CLR types in exceptions messages and logging

I've been using '{0}' in the strings and just passing the Type to the formatter, this results in the fully qualified type name enclosed in single quotes... we should be consistent throughout the stack though.

Clearing up milestone so that we can see this in triage and decide the priority.


Duplicate signals fixes

At first this should fix - https://github.com/godotengine/godot/issues/12750
After some experimenting, I found duplicate_signals not copying signals at all ! When duplicate(via scripts) is called - no signals is copying to the new object. I simplify it and now it working as expected.
Also I removed unnecessary call of duplicate_signals in Node.cpp/duplicate - this call already exist in _duplicate function
I also enable to duplicate deferred or oneshot signals

@akien-mga review ?

Don't ping me for review on all your PRs please, I'm not the only dev and I do what I can already.

Sorry

Ok, I think its ready to merge, I just enabled duplication through context menu in editor and refactor a method in node a bit...


Subscription-based Events (legion-like)

What problem does this solve or what need does it fill?
Current Event system does not play well with out-of-sync readers. (systems with different FixedStep's, update on demand, etc...)
What solution would you like?
legion - like subscription based events. EventReader subscribes to EventSource. EventSource-EventReader keep track of EventData usage, and free memory after EventData was consumed.
I see 3 possible implementations. In all cases EventReader should be stored as system-local resource. EventSource can be global resource. All versions also could have unsubscribe().

First - EventSource does not store EventData queue, EventData copied into each EventReader queue.
//pseudocode in cpp
template<class Data>
struct EventSource {
     std::vector<EventReader<Data>*> subscribers;

     void trigger(Data data){
        for(EventReader* subscriber : subscribers)
           subscriber->event_list.push_back(data);
     }
}

template<class Data>
struct EventReader{
    circular_buffer<Data> event_list;
}

Simple to implement, read-efficient, easy to make thread-safe. Memory consumption and trigger event processing time grows linearly with EventReader's count.

Second - move each EventData in Arc, have Arc-pointers in EventReader - list.
//pseudocode in cpp
template<class Data>
struct EventSource {
     std::vector<EventReader<Data>*> subscribers;

     void trigger(Data data){
        auto shared_ptr = make_shared_pointer(data);
        for(EventReader* subscriber : subscribers)
           subscriber->event_list.push_back(shared_ptr);
     }
}

template<class Data>
struct EventReader{
    circular_buffer<shared_pointer<Data>> event_list;
}

Memory fragmentation, memory alloc/dealloc (can be solved with per EventSource memory-arena [custom Arc?]), Arc dereferencing. But trigger costs does not grow with EventReader's count.
N.B. Since events operate on inter-system level - it could turn out that memory fragmentation is not a problem... CPU cache probably will already be clogged with iterated components data.

Third - second version variation, using intrusive linked list and intrusive Arc.
//pseudocode in cpp
// lack of actual synchronization
template<class Data>
struct EventSource {

    struct EventData{
         Data data;
         atomic<int32> use_count;
         // intrusive linked list (indices as alternative)
         atomic<EventData*> next;
         atomic<EventData*> prev;
    }
    atomic<EventData*> event_list_begin;
    std::array<Data, 128> memory_arena;  // or in heap. Or maybe global memory arena?

   int event_readers = 0;

    void subscribe(EventReader& event_reader){
       event_reader.data_ptr= event_list_begin;
       ++event_readers;
    }

     void trigger(Data data){
        EventData* event = push_list_back(memory_arena, event_list_begin);
        event->use_count = event_readers;
     }
}

template<class Data>
struct EventReader{
    EventSource<Data>* src;
    EventData* data_ptr;  // can't be freed while in use

    optional<Data> consume(){
       data_ptr = list_next(src->event_list_begin, data_ptr);
       if (!data_ptr)
          return {};

       Data data = data_ptr->data;
       data_ptr->use_count--;
       if (data_ptr->use_count == 0){
           list_free(src->event_list_begin, data_ptr);
       }
       return std::move(data);
    }
}

Synchronized Intrusive List may be somewhat tricky to implement. Lowest memory consumption. Reader have fixed memory consumption.

https://github.com/bevyengine/rfcs/pull/17 and https://github.com/bevyengine/rfcs/pull/25 aim to solve different aspects of this problem domain. As you suspected, performance is a serious concern.

Yes, looks like that rfcs do what I want. Closing this.


Split off errors from OpcodeResolution

This implements the changes described in https://github.com/noir-lang/noir/pull/520#discussion_r1052624508.
This is sensible imo as

OpcodeResolution doesn't make sense for a circuit-level function such as PartialWitnessGenerator.solve (what does "Skip" mean when solving a circuit?)
We can define error messages here in ACVM for the failure modes of solving the circuit. Callers then don't need to understand the reasoning behind all the OpcodeResolution variants to be able to separate success vs failure.

I've lumped in a change to simplify BlackBoxFuncCall in the last commit which improves readability imo, happy to revert this if this is unwanted.

haha was thinking of doing something similar! This looks good to me overall, I've opened an issue re UnknownError, but its not a blocker for this, so happy to merge


auto-import with package.json#imports does not take moduleResolution into account

🔎 Search Terms
"subpath", "imports", "package.json", "moduleResolution"
🕗 Version & Regression Information

This changed in commit or PR #55015
This is the behavior in every version I tried, and I reviewed the FAQ for entries about imports, completions

⏯ Playground Link
alan910127/ts-60405-repro
💻 Code
// package.json
{
  "imports": {
    "#*": "./src/*"
  },
  "devDependencies": {
    "typescript": "5.6.3"
  }
}

// tsconfig.json
{
  "compilerOptions": {
    "moduleResolution": "Bundler"
  }
}

// src/foo/bar/baz.ts
export function baz() {}

// src/one/two/three.ts
// actual   - import { baz } from "#foo/bar/baz.js";
// expected - import { baz } from "#foo/bar/baz";

🙁 Actual behavior
When auto-importing the function baz (either through completion or code action), the typescript language server will insert an import statement with a .js file extension.
import { baz } from "#foo/bar/baz.js";

🙂 Expected behavior
Since we are using "moduleResolution": "Bundler", it should insert an import statement without the .js file extension.
import { baz } from "#foo/bar/baz";

Additional information about the issue
I've seen a similar issue #59200, but I cannot reproduce the issue, with the same setup. Therefore, I think my issue might be related with the subpath imports.

I don't think this is a bug. package.json#imports don't describe relative specifiers. Resolving #foo/bar/baz based on "#*": "./src/*" to ./src/foo/bar/baz.js would be incorrect.
A related comment to this by @andrewbranch :
https://github.com/microsoft/TypeScript/issues/60003#issuecomment-2364130579

Oh, I see. But TypeScript language server does not error if I use the extensionless module specifier either. Why is that the case?

The thread I linked to specifically calls out extensionless module specifiers as erroring. If they are not then perhaps that is a bug - it might be good to wait for @andrewbranch to chime in. It would be good to understand the difference between that issue and yours.

@alan910127 your repo link above 404s—did you accidentally make it private? If you share the repo, I’ll investigate. @Andarist is right; the import you listed as your expected behavior should have an error.


Error displaying widget: model not found

Describe the bug
Error displaying widget: model not found
To Reproduce
from bqplot import pyplot as plt
apple_df = pd.read_csv('https://raw.githubusercontent.com/plotly/datasets/master/finance-charts-apple.csv')
print(apple_df)
fig = plt.figure(title="Apple Mar-2020 CandleStick Chart")

fig.layout.width="800px"

ohlc = plt.ohlc(x=apple_df.index, y=apple_df[["AAPL.Open","AAPL.High","AAPL.Low","AAPL.Close"]],
                marker="candle", stroke="blue")

ohlc.colors=["lime", "tomato"]
plt.xlabel("Date")

plt.show()

Expected behavior
Chart to show up
Additional context
Add any other context about the problem here.
JupyterLab v3.2.0
/Users/kdang/anaconda3/envs/stock-cms/share/jupyter/labextensions
        bqplot v0.5.32 enabled OK (python, bqplot)
        bqscales v0.2.2 enabled OK
        jupyterlab-plotly v5.3.1 enabled OK
        @jupyter-widgets/jupyterlab-manager v3.0.0 enabled OK (python, jupyterlab_widgets)

Other labextensions (built into JupyterLab)
   app dir: /Users/kdang/anaconda3/envs/stock-cms/share/jupyter/lab

Name: bqplot
Version: 0.12.31
Summary: Interactive plotting for the Jupyter notebook, using d3.js and ipywidgets.
Home-page: https://github.com/bqplot/bqplot
Author: The BQplot Development Team
Author-email:
License: Apache
Location: /Users/kdang/anaconda3/envs/stock-cms/lib/python3.7/site-packages
Requires: ipywidgets, numpy, traittypes, traitlets, pandas
Required-by:
Note: you may need to restart the kernel to use updated packages.

Hello @wakandan , just another user here.
The times I have seen that - is when I have jupyterlab open and install bqplot, then try to run the code.
Have you tried to completely shutdown jupyterlab, then bring it up again and execute the code?
I noticed your code was not quite executable as is since it is missing the pandas import, for ease of reproducing I am posting the code with that added in:
from bqplot import pyplot as plt
apple_df = pd.read_csv('https://raw.githubusercontent.com/plotly/datasets/master/finance-charts-apple.csv')
print(apple_df)
fig = plt.figure(title="Apple Mar-2020 CandleStick Chart")

fig.layout.width="800px"

ohlc = plt.ohlc(x=apple_df.index, y=apple_df[["AAPL.Open","AAPL.High","AAPL.Low","AAPL.Close"]],
                marker="candle", stroke="blue")

ohlc.colors=["lime", "tomato"]
plt.xlabel("Date")

plt.show()


doesn't work in 'default' profile

aws configure set s3.endpoint_url https://hb.xxx.com
results in
[default]
aws_access_key_id=xxx
aws_secret_access_key=xxx
s3 =
    endpoint_url = https://hb.xxx.com

root@db0-dev:~# aws s3 ls

An error occurred (InvalidAccessKeyId) when calling the ListBuckets operation: The AWS Access Key Id you provided does not exist in our records.

irrelevant:
missed the:
aws configure set plugins.endpoint awscli_plugin_endpoint
part


Can't train on my data using a pre-trained model

Hi! I tried to use the pre-trained "Universal_rehistoGAN_v2" model to train on my data with the same picture resolution to improve the quality of the neural network, but there was a problem "self.GAN.load_state_dict":
Error(s) in loading state_dict for recoloringGAN:
size mismatch for ED.encoder_blocks.0.conv_res.weight: copying a param with shape torch.Size([36, 18, 1, 1]) from checkpoint, the shape in current model is torch.Size([16, 8, 1, 1]).
size mismatch for ED.encoder_blocks.0.conv_res.bias: copying a param with shape torch.Size([36]) from checkpoint, the shape in current model is torch.Size([16]).
size mismatch for ED.encoder_blocks.0.net.0.weight: copying a param with shape torch.Size([36, 18, 3, 3]) from checkpoint, the shape in current model is torch.Size([16, 8, 3, 3]).
size mismatch for ED.encoder_blocks.0.net.0.bias: copying a param with shape torch.Size([36]) from checkpoint, the shape in current model is torch.Size([16]).
...
I tried adding "strict=False" as described here (https://github.com/pytorch/pytorch/issues/40859), but to no avail. I suspect that the model I loaded and the target model is not identical, so the error raise to inform about mismatches of size, layers.
How to get around this problem? Thanks in advance for the answer

You are right. Please make sure to use
--network_capacity 18
when training the universal model.

Thank you!


Make travis.yml use tox environments

Travis is running only 7 jobs now, tox is running 21.

Fixed in #93


Rajarju/web invoke method

Description
Update verbs
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@purplecabbage I think it would be best to implement this with the #130


Scoring algorithm

The scoring algorithm assign a CONTAINER_AREA_BONUS to figures based on their minH - the idea seems to be that bigger figures (usually pictures) are more important and so should go first. Am I right?
If this is the case why use minH and not the area (something like Math.sqrt(mapped_container.figureSize.minH * mapped_container.figureSize.minW)? Using only the height privileges vertical pictures.
Wouldn't be better to have a bonus based on the position of the figures in the article instead of the size? Presumably whoever is creating the content knows what is more imporant and could place it at the beginning of the article.
Thanks

In my local builds I use the area, the square root is probably a better idea (same order of magnitude as one side).
I don't really see a real disadvantage to this change; depending on the content, it will change the "best grid" for someone, but that's a given.
I will clean it up and send a pull request.


Collection of PRs related to integration of bosh-azure-storage-cli

Here is a collection of PRs related to integration of bosh-azure-storage-cli:

cloudfoundry/bosh:

[x] auto bump the azure storage cli
[x] Integrate bosh-azure-storage-cli into bosh

cloudfoundry/bosh-linux-stemcell-builder

[x] Add azure-storage-cli to the stemcells

cloudfoundry/bosh-stemcells-ci

[x] Add azure-storage-cli to the stemcells

cloudfoundry/docs-bosh

[x] Adapt documentation

All PRs are merged, thx!


Wrong JVP and VJP ?

Hi,
I am trying to compute VJPs and JVPs for a simple heat equation, but it seems unless I am missing something, the output shapes are inconsistent :
function heat(u, p, t)
    gamma, dx = p
    dxxu = (circshift(u, 1) .- 2 * u .+ circshift(u, -1)) / dx^2
    return gamma * dxxu
end

Ns = 5
xmin, xmax = 0.0, 1.0
dx = (xmax - xmin) / (Ns - 1)

xs = collect(LinRange(xmin, xmax, Ns))
using FastDifferentiation
FastDifferentiation.@variables uv pv tv
uvs = make_variables(:uv, Ns)
pvs = make_variables(:pv, 2)
tvs = make_variables(:tv, 1)
fv = heat(uvs, pvs, tvs)
js = FastDifferentiation.jacobian(fv, uvs) # The Jacobian js is of shape Ns x Ns

vjps, rvs = FastDifferentiation.jacobian_transpose_v(fv, uvs) # The variable vjps is of shape Ns+2 and rvs is of shape Ns
jvps, vvs = FastDifferentiation.jacobian_times_v(fv, uvs) # The variable jvps is of shape Ns and rvs is of shape Ns+2

The jacobian seems correct but not the vjps and jvps.

Yes, it's a bug. The vjps terms include all the variables in the function, not just the variables you are differentiating with respect to, in this case uvs. Same thing for the vvs, term it includes all the variables in the functions. I'll fix it today.

I have a fix on branch brianguenter/issue60. Could you download this branch and run it on your code to verify that it is giving the results you expect? If you have any other code that exercises jacobian_times_v and jacobian_transpose_v could you run that as well and report any bugs to me?

Since I haven't heard from you will assume you are no longer having problems. I'm closing the issue.

Sorry for the late response.
Thanks a lot for your work and for this library ! It seems to be working perfectly.


VList horizontal scrollOffset is not updating in iOS Safari

Describe the bug
iOS Safari doesn't seem to have a continuous event trigger for scrolling an element. The event chain on iOS is something like touch start (triggered) --> movement (no event) --> touch end (triggered) therefore the scroll event is only updated at the end of the interaction.
To Reproduce

Create a <VList horizontal .../> component and scroll the element left to right in iOS Safari.
The scrollOffset property is only updated when the scrolling stops.

Expected behavior
Similar behavior as in other browsers. A continous update of the scrollOffset property when scrolling.
Platform:

OS: iOS
Browser: [Safari
Version of react: 18.2.0
Version of this package: 0.15.7

@feketegy
Hi, could you tell me the version of iOS Safari you tested?

@inokawa sure, iOS 17.0.1 and the full user agent for Safari on iOS is Mozilla/5.0 (iPhone; CPU iPhone OS 17_0_1 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.0 Mobile/15E148 Safari/604.1 browser

Thanks!
It look like similar version to my iOS... I might be misunderstanding something but anyway I try to reproduce it.

@feketegy
So could I ask you what the actual problem is?
Virtualization fails? or you want to use VListHandle.scrollOffset for something?
If your request is that VList should have same timing to dispatch scroll events across browsers, I think it's impossible because each browsers have different timing and VList just subscribes it.
If iOS Safari dispatches scroll event but VList missed it, it should be fixed.
If virtualization fails due to wrong logic of VList, it should be fixed.

You're right, it's more of an issue in iOS and scrolling than with the package.
On touch devices there's no real scrolling happening, instead touch swipe and movement.
Other virtualized list packages fix this by calculating the distance moved in the touch event and updating the scroll property with that value, to have a consistent functionality across browsers. I thought virtua handles it the same way.

Do you know what library supports fallback with touch event?
If it's acceptable one, it's worth to implement it for virtua also.

If I remember correctly react-virtualized and react-window do it.

Thanks!
As far as I know, they doesn't subscribe touch event but it has WebkitOverflowScrolling: 'touch' style on scrollable element.
It may solve your problem if you migrated from react-virtualized or react-window.
https://github.com/bvaughn/react-window/blob/efad3d8909753fd74aad7c47dc902b26f0919651/src/createListComponent.js#L374


SAS mode "'" malfunction after commit to fix empty string and space only string problem.

Thank you for the quick update. After testing out the empty string problem is gone, but if I use "'" to represent a string ' it cause the same problem. Before the update "'" didnt cause any problem.

Promoting a fix,
Else if (ch === '"'){
state.continueString = state.continueString == ch? Null : (state.continueString == "'" ? state.continueString : ch)
Return "string"}
Do the same for "'"

However i noticed other problems where some words are colored has numbers


Move current gohugo.io and themes.gohugo.io to Netlify

This is a technical prerequisite for getting the refreshed gohugo.io site out the door, but as the current build is broken on several levels (for one, the deployment machine fails on the Git command -- and the docs was the main motivation for adding that feature in the first place ... -- and it has a too old version of Hugo).
So I suggest we just do this quick and hard for the current static sites.
We already have a Netlify build for the main site, so the suggested game plan is:

@spf13 tries to get a theme site up and running (with the scripts and all) on Netlify. (I have no access to configure the Git link)
When 1) is OK, @spf13 make the DNS switch.

Sounds OK?

And please, please don't make this into a discussion about any future site setup. This is about the current production environment, and that only.

/cc @budparr @rdwatters

We already have a Netlify build for the main site

Which site?  Actually, it probably doesn't matter since both the docs and the news site are on Netlify.
I'm cool with whatever process you guys want. As I mentioned to @budparr, I already updated some of the styles per his quick and dirty suggestions, so the contrast between the new home and docs is more seamless.
@bep, ideally, how much time are you thinking? I ask just so that I know how much time I have to finish the 1 or 2 WIP pages. . Thanks!

Also, @digitalcraftsman and @budparr; I haven't been keeping up with the themes site as much. I can do a PR to line up the styles. I'm okay with getting this done now and refining the theme/styles in future iterations.

@rdwatters I will say this one final time: This issue is about the CURRENT gohugo.io and themes.gohugo.io.

I imagine @biilmann could be helpful with this process, and it may be helpful to get the sites on the "Pro" plan, which is free for open source projects. Let me know if there's anything specific I can do to help.

I'm cool with whatever you want to do. Still.
I agree that the the guys at Netlify would probably be of great help. I shouldn't go on Github when I wake up at 3am randomly.

I support this and will do whatever I need to do / can do to help.
On Fri, Mar 24, 2017 at 10:17 AM Ryan Watters<EMAIL_ADDRESS>wrote:

I'm cool with whatever you want to do. Still.
I agree that the the guys at Netlify would probably be of great help. I
shouldn't go on Github when I wake up at 3am randomly.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/spf13/hugo/issues/3228#issuecomment-289034517, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAKlZBsX4_3V2jo-X2W6EIHy24wcEyxJks5ro9BvgaJpZM4Mn1vj
.

I support this and will do whatever I need to do / can do to help.

@spf13 I will turn this around: I assigned this issue to you, so you will have to drive this issue. You have the needed access to your DNS provider etc., so it isn't very practical for anyone else to do it.
But I will do anything I can do to help ...

This is how I'd approach setting up themes.gohugo.io on Netlify:

I'd add https://github.com/spf13/hugoThemeSiteScript as a submodule of https://github.com/spf13/hugoThemesSite
I'd add a script in hugoThemesSite that updates the submodule, runs the script and builds the site.
I'd create the site on Netlify making that script the build command.

There might be other alternative ways to accomplish it, but this sounds like a reasonable game plan to me.

@spf13 Have added some TODO tasks for myself in the first post here; when I finished that, it should be ready for you to do the rest. I will let you know when I'm done.

@spf13 I need you to create a site from the repo:

https://github.com/spf13/hugoThemes
Branch: master
Build command: buildThemeSite.sh
Publish directory: _script/hugoThemeSite/themeSite/public

Don't worry if it fails ... I will fix that.

@spf13 I was hoping we could get this done before Hugo 0.20 (which I guess will ship on Monday), but that looks ... optimistic with this speed.

@bep I've set it up as hugothemes.netlify.com and invited you to be a collaborator.

Let us know if there's anything we can do to help at any point :)

@spf13 @biilmann that was pretty smooth sailing; my "blind scripting" worked almost right out of the box:
https://hugothemes.netlify.com/
As I see it, both sites are now set up correctly vs branches and deployments scripts and can be switched in DNS:

https://hugoio.netlify.com/
https://hugothemes.netlify.com/

And it is pretty fast as well, considering it is pulling in and building 150+ themes ...

It's great to see some progress in this area and I can second @bep that the build times are really fast.
@bep how can I help?

@digitalcraftsman the DNS updates @spf13 has to do. He should also add you as a collaborator, I guess you "own" these sites more than me ... It is pretty nice to have a log to look at ... I would "never" have found the small item in my script that failed on Linux without it.

He should also add you as a collaborator, I guess you "own" these sites more than me ...

It's more about maintenance that ownership but I get it :wink:

It is pretty nice to have a log to look at ... I would "never" have found the small item in my script that failed on Linux without it.

This is one of the most important points - hopefully we can forget the "blind man porking around in the dark"-metaphor.

@digitalcraftsman what you can do is to check that the site reflects the changes in the theme as you update them.

hopefully we can forget the "blind man poking around in the dark"-metaphor.

The good thing about that is if in the future see a job ad searching for a Jedi DevOps, you can apply and put me up as a reference :-)
A related story from back when I was a student and not loaded with money. My 17 inch CRT monotor broke down. I spent enough time on the university PC lab, so I could do without a computer, but I had a document on the harddrive that I really needed. The PC was not connected to any network (this was in the early days of computing), but it had a printer attached ...
A long story short: I finally managed to turn on the computer, open up the word processor, the document and print it ...

@biilmann the Netlify log console for the theme build is consistently making my browser unresponsive (Chrome on MacBook).  But only if I "watch it" live, so it's a minor problem.

It seems that the netlify themes site is missing two themes from the official themes site: arch and purehugo.

That's because they're blacklisted.

I'm ready to pull the trigger and move the DNS over. Just let me know when and I'll flip the switch.

@spf13 just to it whenever you please.

The transition was successful and DNS entries are updated.


Crash when spawning 132k entities with 9 components

Not sure if this is binding issue or flecs issue, the coredump is also quite useless:
* thread #1, name = 'Game', stop reason = signal SIGSEGV: address not mapped to object
  * frame #0: 0x000075d4dd47abd9 libcoreclr.so`___lldb_unnamed_symbol9858 + 249
    frame #1: 0x000075d4dd3ebd7e libcoreclr.so`___lldb_unnamed_symbol8317 + 126
    frame #2: 0x000075d4dd3ecee2 libcoreclr.so`___lldb_unnamed_symbol8333 + 514
    frame #3: 0x000075d4dd3edcf4 libcoreclr.so`___lldb_unnamed_symbol8341 + 1412
    frame #4: 0x000075d4dd3e7596 libcoreclr.so`___lldb_unnamed_symbol8272 + 374
    frame #5: 0x000075d4dd3ef647 libcoreclr.so`___lldb_unnamed_symbol8375 + 359
    frame #6: 0x000075d4dd46a90a libcoreclr.so`___lldb_unnamed_symbol9794 + 810
    frame #7: 0x000075d4dd469392 libcoreclr.so`___lldb_unnamed_symbol9791 + 242
    frame #8: 0x000075d4dd48e8fc libcoreclr.so`___lldb_unnamed_symbol9968 + 220
    frame #9: 0x000075d4dd319f34 libcoreclr.so`___lldb_unnamed_symbol5639 + 116
    frame #10: 0x000075d4dd616dde libcoreclr.so`___lldb_unnamed_symbol16085 + 510
    frame #11: 0x000075d4dd8a339d libc.so.6`___lldb_unnamed_symbol3666 + 941
    frame #12: 0x000075d4dd92849c libc.so.6`___lldb_unnamed_symbol4097 + 7

Found something:
Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
   at Systems.<>c.<ProcessOccluders>b__6_0(Iter it, Field`1 positions, Field`1 occluders, Field`1 translates) in /home/deathbeam/git/raylib-game/src/Systems.cs:line 211
   at Flecs.NET.Core.Invoker.Iter[T0,T1,T2](Iter it, IterFieldCallback`3 callback)
   at Flecs.NET.Core.BindingContext.Functions`3.IterFieldCallbackDelegate(ecs_iter_t* iter)
   at Flecs.NET.Bindings.flecs.ecs_progress(ecs_world_t* world, Single delta_time)
   at Game.Main() in /home/deathbeam/git/raylib-game/src/Game.cs:line 135
[1]    1102445 IOT instruction (core dumped)  ./bin/Release/net8.0/linux-x64/publish/Game

When i commented out the access to Occluders it crashed elsewhere.
This is my Occluder component:
public record struct Occluder(Vector2[] Points);

Most of my components have simple values but Occluder is first one it encountered with Vec2 array so I assume its getting yeeted somewhere.

Yea when I changed all my types to unmanaged only I can continue spawning entities easily:
public record struct Texture(FixedString64 Path)
{
    public Texture(string path) : this(new FixedString64(path)) { }
}

public record struct Occluder(NativeArray<Vector2> Points)
{
    public Occluder(Vector2[] points) : this(new NativeArray<Vector2>(points.Length))
    {
        for (int i = 0; i < points.Length; i++)
        {
            Points[i] = points[i];
        }
    }
}

public record struct Collider(NativeArray<Vector2> Points)
{
    public Collider(Vector2[] points) : this(new NativeArray<Vector2>(points.Length))
    {
        for (int i = 0; i < points.Length; i++)
        {
            Points[i] = points[i];
        }
    }
}

public record struct GridPath(int Progress, NativeArray<Vector2> Points)
{
    public GridPath(int progress, Vector2[] points) : this(progress, new NativeArray<Vector2>(points.Length))
    {
        for (int i = 0; i < points.Length; i++)
        {
            Points[i] = points[i];
        }
    }
}

So its definitely issue with how managed types are being handled.

Found something:
Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
   at Systems.<>c.<ProcessOccluders>b__6_0(Iter it, Field`1 positions, Field`1 occluders, Field`1 translates) in /home/deathbeam/git/raylib-game/src/Systems.cs:line 211
   at Flecs.NET.Core.Invoker.Iter[T0,T1,T2](Iter it, IterFieldCallback`3 callback)
   at Flecs.NET.Core.BindingContext.Functions`3.IterFieldCallbackDelegate(ecs_iter_t* iter)
   at Flecs.NET.Bindings.flecs.ecs_progress(ecs_world_t* world, Single delta_time)
   at Game.Main() in /home/deathbeam/git/raylib-game/src/Game.cs:line 135
[1]    1102445 IOT instruction (core dumped)  ./bin/Release/net8.0/linux-x64/publish/Game

When i commented out the access to Occluders it crashed elsewhere.
This is my Occluder component:
public record struct Occluder(Vector2[] Points);

Most of my components have simple values but Occluder is first one it encountered with Vec2 array so I assume its getting yeeted somewhere.

Can you provide a repro that I can test locally? This usually means that you have a null reference somewhere which can possibly happen if you're doing something like this:
using World world = World.Create();

world.Entity().Add<string>();              // This defaults to a null reference. Will throw exception if you try to access it.
world.Entity().Set<string>("Hello World"); // You need to provide a reference to a managed component with .Set

Yea I checked if I have nulls anywhere but its not possible, at least not explicitely. Here is minimal repro where I managed to reproduce it super easily:
Works at 100k, crashes at 150k. Without the 2 Vector2[] dummies it crashes at 1.6M. Without dummies and string even 10M works fine.
using System.Numerics;
using Flecs.NET.Core;

public record struct Texture(string Path);

public record struct Position(float X, float Y);

public record struct Color(byte R, byte G, byte B, byte A);

public record struct Velocity(float X, float Y);

public record struct Destination(float X, float Y);

public record struct Source(int X, int Y, int Width, int Height);

public record struct Translate(float X, float Y);

public record struct Dummy1(params Vector2[] Points);

public record struct Dummy2(params Vector2[] Points);

class Program
{
    private static readonly int winWidth = 1280;
    private static readonly int winHeight = 720;
    private static readonly int numEntities = 150000;
    private static readonly string entityTexture = "assets/enemies/Enemy_010.png";
    private static readonly Random random = new Random();

    private static Vector2 randomVector2(int maxX, int maxY)
    {
        return new Vector2 { X = random.Next(0, maxX), Y = random.Next(0, maxY) };
    }

    private static Color randomColor()
    {
        return new Color
        {
            R = (byte)random.Next(0, 256),
            G = (byte)random.Next(0, 256),
            B = (byte)random.Next(0, 256),
            A = 255,
        };
    }

    private static void createEntity(World world)
    {
        var pos = randomVector2(winWidth, winHeight);
        var color = randomColor();

        world.Entity()
            .Set(new Texture(entityTexture))
            .Set(new Position(pos.X, pos.Y))
            .Set(new Source(0, 0, 24, 24))
            .Set(new Destination(0, 0))
            .Set(new Velocity(0, 0))
            .Set(new Translate(-12, -6))
            .Set(color)
            .Set(new Dummy1(new Vector2(0, 0), new Vector2(0, 0), new Vector2(0, 0), new Vector2(0, 0)))
            .Set(new Dummy2(new Vector2(0, 0), new Vector2(0, 0), new Vector2(0, 0), new Vector2(0, 0)));
    }

    static void Main(string[] args)
    {
        var world = World.Create();

        for (int i = 0; i < numEntities; i++)
        {
            createEntity(world);
        }

        var system = world.System<Texture, Position, Source, Destination, Color>("RenderSystem")
            .Each((ref Texture texture, ref Position position, ref Source source, ref Destination destination, ref Color color) =>
            {
                Console.WriteLine($"Rendering {texture.Path} at {position.X}, {position.Y} with color {color.R}, {color.G}, {color.B}, {color.A}");
            });

        while (world.Progress())
        {
        }
    }
}

Fixed by #54, closing


Return FREERDP_ERROR_AUTHENTICATION_FAILED on an authentication failure

when using NLA with xfreerdp.

@bjcollins Would it not be easier just to use SetLastError(ERROR_MUTUAL_AUTH_FAILED) (or similar) instead of all the new functions?

@freerdp-bot test

@bjcollins any comment on @akallabeth suggestion?

@bmiklautz @akallabeth I think this patch could cut down, SetLastError can probably just be called a little deeper without passing the failure boolean up the call stack and I called the callback "test_function" which is not really a great name. Not sure about using ERROR_MUTUAL_AUTH_FAILED as I wanted the exit code to reflect that it was an NLA auth failure. Either way, I want to look at this again and try to rework this patch.

@akallabeth @bmiklautz  Ok, I reworked this. I wanted to make sure the xfreerdp process return code could be used to determine if process had an authentication failure - so I was unsure about trying to use ERROR_MUTUAL_AUTH_FAILED, but the PR is definitely in better shape than my first submission.

@bjcollins Looks good so far, just a question, why don't you call freerdp_set_last_error(rdp->context, FREERDP_ERROR_AUTHENTICATION_FAILED) directly instead of using the new boolean value nlaFailure?

@akallabeth I think the variable might be useful in the future and this approach keeps setting of the error in the same location it already is. I think you can definitely just set call freerdp_set_last_error directly from the callback, but I like using a variable better. I will gladly remove the variable if you want.

@bjcollins Would be nice, it usually is easier to find a bug if the error is set where it occured.

@akallabeth Ok, removed the extra variable and just set the error in the callback.

I really would like to see this pull request merged. Could please someone merge ?
It also should fix issue #2760 #2434 and helps for #2432
@bjcollins: in issue 2760 you mentioned "TLS ALERT for ACCESS DENIED". Is this the same problem reported here by Marc André ?

+1 definitely need better error handling in FreeRDP - this would be a first step.

:+1: I also join @lemonmojo and @giox069 , this is a must have for several uses cases.
Great job @bjcollins!

@giox069 yes it is the same problem. He discovered the problem and put in the server-side code to send the alert. This adds in the code to detect that "Access Denied" alert on the client side and set an exit code accordingly. Sorry, that is not obvious in the code and I did not put in any comments.
I can probably improve the "transport_ssl_cb" function to make that more clear. I am currently looking at the raw numerical value, but there is an ssl function "SSL_alert_desc_string" which can decipher that value into a 2 character string value "AD". That change along with some comments should help make this PR more readable.

@bjcollins: thanks for the explanation. I was just looking for the cause of the bad return code, and I came to the same conclusion. I asked to you just for a confirmation.
The only need I have now is... please merge this PR. I'm a Remmina developer/maintainer. I would like to update the "remmina-next Ubuntu PPA" where FreeRDP is very old (before issue #2434). But issues #2760 #2434 will eventually create issues in the PPA with remmina password saving, so I'm blocked on these issues and this PR.
Please, please, merge. Thank you :)

ok ok I'm not so happy with the thing implemented in the OpenSSL callback, but let's merge it as is.

Thank you very much :+1:  :+1: :+1:

Thanks!

@hardening Is your issue with the OpenSSL callback similar to what I said about about just using the raw value instead of using a helper function such as "SSL_alert_desc_string" to translate the code? Or is there more? I am just curious as this is something I would like to fix up a little better in the nearish future as I do not really like the callback code either.


AIP 1.1: Arbitrum Foundation Lockup Wallet

Lockup wallet as per AIP 1.1 (in snapshot stage as of posting this)

would be nice to add in a test case that forks the L2 network

added audit report

todos before submission:

Get foundation vesting wallet beneficiary
deploy / verify Arbitrum Foundation Vesting Wallet
deploy/ verify AIP1Point1Target

@fredlacs the repo already includes code for various deployed actions that haven't been used (mostly for unplanned / emergency scenarios https://github.com/ArbitrumFoundation/governance/tree/main/src/gov-action-contracts) so "in-repo" already doesn't imply "reflects on-chain vote."
I still think we should wait to merge until we can resolve the pending TODO (foundation beneficiary multisig)


[Bug report]

Version
2.0.3
Platforms
dart
Device Model
emulator
flutter info
/data/user/0/com.example.new_app/cache/43e263ae-5d36-4e92-8d71-eb39f929ea257380969216658532980.jpg
E/flutter ( 5143): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: 'package:flutter_image_compress_platform_interface/src/validator.dart': Failed assertion: line 23 pos 9: '(name.endsWith('.jpg') || name.endsWith('.jpeg'))': The jpeg format name must end with jpg or jpeg.

How to reproduce?
image path is jpg, but plugin don't comperss
Logs
No response
Example code (optional)
Future<void> pickImage({required ImageSource source}) async {
    final ImagePicker picker = ImagePicker();
    final Directory tempDir = await getTemporaryDirectory();

    final XFile? getImage = await picker.pickImage(source: source);
    if (getImage != null) {
      final XFile? image =
          await testCompressAndGetFile(File(getImage.path), tempDir.path);
      if (image != null) {
        setState(() {
          _imageAvatar = File(image.path);
        });
      } else {
        print('no image selected');
      }
    }
  }

Contact
No response

i'm having the same issue here. Does anyone fixed this issue yet?

 FlutterImageCompress.validator.ignoreCheckExtName = true; add this line, before the compress method

Thanks! this solve my problem when i try to use webp & heic format.


Here are some other links to add, and a suggestion (and mistake found)

Here are some more links to add.
https://aboutiota.info/ - All About IOTA
http://iotabalance.com/ - Minimalist IOTA Balance Lookup
http://tangler.org/ - Tangle Explorer
http://iotachina.com/ - IOTA China 首页
http://iotaitalia.com/ - IOTA Italia
http://blog.naver.com/iotakorea - IOTA 한국
http://lhj.hatenablog.jp/entry/iota - IOTA 日本
https://blog.iota.org/iota-ecosystem-fund-2-million-f6ade6a4d8ba - Ecosystem Fund Post
A mistake: Under Blogs & Sites
The title Wikipedia is linking to trustediot.org and the description is the link to wikipedia.
I think Wikipedia should be at the top in the About section.

@stockermans thanks for links. i added all of them. and moved wikipedia links to about section.


[bug] Fold code is the wrong implementation

VSCode Version: 1.10.2
OS Version: Windows10

Steps to Reproduce:
1.write code in js file
function abc() {
    var see;
    var bug2;
    var bug = `testtest
I can Cross line`;
}

folded “function abc()”
result:

expected (visual studio 2015):

then I find ...
please notice the first space

function abc() {
    var see;
    var bug2;
    var bug = `testtest
 I can Cross line`;
}

result:

function abc() {
    var see;
var bug2;
    var bug = `testtest
I can Cross line`;
}

result:

function abc() {
    var see;
    var bug2;
    var bug = `testtest
    I can Cross line`;
 }

 var config = {
    see: 0,
};

result:

other file also

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
<meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>Document</title>
 </head>
 <body>

 </body>
</html>

result:

expected (visual studio 2015):

it seems only to determine whether the first character is a space

+1. I don't use this feature, but I've observed similar bugs with code-folding. The scope of the code block doesn't seem to be respected, instead the feature seems to rely on the indentation of code.

Correct, code folding is based on indentation.

#3422 is the request to enable language based folding.


Clear inlinee callinfo from the catch of OP_TryFinally as well

When we hit an exception and we have a try finally within a try catch. We will execute finally code by calling BailOnException.
Since we inline into functions with try now, we will end up in inconsistent inlinee callinfo when we call the bailout code from finally.
So add the walker and clear inlinee callinfo. Also set a bit that the callinfo's were cleared so that OP_TryCatch does not walk again to do the same.
Fixes OS# 14336922

This unfortunately does not work with nested exception:
try {
try {
inlinee1();
} finally {
try {
inlinee2();
} catch() {
}
}
} catch() {
}
The catch in the finally will reset the threadcontext...
You can fix this by having your constructor save the previous state on the threadcontext, which gives you a stack.
Of course, this still won't be sufficient for inlining functions with try blocks...  If we ever do our own EH frames, we could treat inlinees as finallys and unwind them then.

Will close this and reopen against release/1.8


"Setting Sync" doesn't sync Tasks

VSCode Version: 1.51.1
OS Version: Windows 10

Steps to Reproduce:

Have VSCode on one computer system - PC1
Have few VSCode Tasks configured in User Settings - Task.json
Turn on Setting Sync in VSCode running on PC1
On a new computer PC2, install VSCode
Now login to the VSCode on the PC2 with the same account as PC1 and restore the settings.
Observe that all key shortcuts, settings are restored but Tasks are not restored.

Does this issue occur when all extensions are disabled?: Yes/No

/duplicate

@sandy081 Can you let me know, this is duplicate of which issue?

https://github.com/microsoft/vscode/issues/91011


💣 Wrong content-type: when PUT api/v1/observability/tenant/{tenant}/channels/{channel}, expect application/json

后端问题 https://github.com/kubegems/kubegems/issues/432


Add dependencies to documentation

 Issue by petebachant
Saturday Jan 23, 2016 at 23:26 GMT
Originally opened as https://github.com/whophil/CACTUS-SNL/issues/21

I am on an Linux Mint, for which I believe all dependencies can be installed with
sudo apt-get install liblapack-dev gfortran

Closing this, since build is now with CMake (#18), which will tell you if you're missing any build deps.


Add document for PgResult

@BillyYccc before merging your PR I need to ask you if you comply with the CLA I set on this project, which basically you need to agree with the copyright on the project and swear you did not steal any code anywhere :-)

@vietj Yes, I do agree with the CLA! :) Also, do I have to sign like that in Vert.x Projects?

it's a diffrent project, I haven't set formally anything (yet), perhaps I'll ask something more formal later.

Maybe Code of Conduct is suitable for this.

@BillyYccc no, CoC are a different things, they don't care about IP or due diligence, they care about people not being asses :-)


Props is typed correctly on vanilla linaria, but not here

Do you want to request a feature or report a bug?
bug
What is the current behavior?
In typescript, the type of props inside a styled component is "any"
If the current behavior is a bug, please provide the steps to reproduce and a minimal repository on GitHub that we can yarn install and yarn test.
https://github.com/mattoni/linaria-test
No need to start it, it doesn't work properly (another issue to file) but open up the src/badges/badge.tsx file, and you'll see that the type of props is correct when using vanilla linaria.
What is the expected behavior?
Props will be correctly typed
Please provide your exact Babel configuration and mention your Linaria, Node, Yarn/npm version and operating system.
In package.json of test repo

Thank you for the repo. I will look into this. I will be pushing a new beta update soon that will fix the start issue.

@brandonkal sorry for the late reply, and thank you for your detailed analysis of the problem. For me personally, it'd be better to have the linaria typing be the default and find another way to accept the styled component, but i can see how this may be restrictive and not the right path. It's unfortunate we have to manually do the typing like that.
Do you plan to keep working on your fork? It seems the main linaria page has seen little to no activity for months now and as am planning to stake a lot of projects on this (I love the concept) I'd be willing to use your fork and help contribute to it.

@mattoni Maintaining the fork is very important to me, so as always, contributions are appreciated.
All tests are passing now with the new features and I am planning a new release as soon as possible.
I very much love the concept here and am planning to move all my remaining Styled Components and Emotion projects to this fork.
The manual typing thing was annoying when I started with Styled Components but restricting it to only accept inline functions would be too much. I have some VSCode snippets that makes using the shortcut a non-issue for my workflow.
The next release will have much faster webpack compilation due to a vastly improved caching mechanism.

Awesome work. I'll be using your version as soon as I can get around to adding the additional typing in.

@mattoni Looking at Styled Components, I see that props argument is now typed. So it looks like it is possible. If you can figure out how to get both working, I'll accept the PR.

@mattoni I gave it yet another attempt and found a solution: You can remove a call signature from a Typescript type by doing the following:
type StyledComponentInterpolation = Pick<
  StyledComponent<any, any>,
  keyof StyledComponent<any, any>
>;

So now you should typed interpolations props arguments and support for other Styled Components as selectors.
I've pushed a release v3.0.0-beta.7 with this and many other improvements.
This should be the last beta so let me know if you find any other issues.
Closing as fixed.


Auto resize token/contract logos to 256x256

Closes #693

@fmrsabino Maybe we can do this for the config-service too

Coverage increased (+0.007%) to 88.291% when pulling ab8119380ac7f8e9abe1a5ae4028cbca9410314d on resize-and-convert-images into 95a4f8fc1eedfb6535e20546418b17b63912c0d0 on master.


Update proxy core

Pull Request Test Coverage Report for Build 197

2 of 7 (28.57%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.8%) to 73.913%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

main.go
0
2
0.0%

k8s/client.go
2
5
40.0%

Files with Coverage Reduction
New Missed Lines
%

k8s/client.go
1
84.04%

Totals

Change from base Build 195:
0.8%

Covered Lines:
136

Relevant Lines:
184

💛  - Coveralls

Can you also update the application.yml and values.yml with the new values -  https://github.com/Peripli/service-broker-proxy-cf/pull/20/files


Convert builtin errors to EvalError

Codecov Report
Attention: Patch coverage is 0% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 44.55%. Comparing base (b2f2d0d) to head (7627463).

Files
Patch %
Lines

cmd/dawn/builtins.go
0.00%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
- Coverage   45.32%   44.55%   -0.77%
==========================================
  Files          23       23
  Lines        2716     2765      +49
==========================================
+ Hits         1231     1232       +1
- Misses       1348     1395      +47
- Partials      137      138       +1

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


Error: AnimatedValue: Attempting to set value to undefined

I'm getting this error when I install and run this library for the first time.

Device: [e.g. iPhone13]
OS: [e.g. iOS16.1]

I'm assuming the error is caused by libraries clashing but I'm not sure. These are my packages that I have installed on my project.
"react-native-svg": "^12.1.0",
"react-native-reanimated": "^2.12.0",
"react-native-linear-gradient": "^2.5.6",
"react-native-gifted-charts": "^1.2.41",
Would appreciate if I could get some guidance.

Hi @Abhinandan-Kushwaha I am getting this in the project can you suggest me any solution ?


Codacy issues set2

There are 166 new issues from this. Mostly are code formatting. Fix those too and update PR. @aayusharora

#173


Newlines '\n' Isn't Getting Stripped From The Passwords Returning Bad Results With Valid Stuff.

I'm currently fixing this tool instead of deleting it. I wanted to share to hall process with you guys using the issues.

On the StartBruteAccount() Function at line 24 the program is taking the passwords from a newline-separated list without removing the newline from it. that uses a bad password won't be valid with the authentication and even if the password is valid the program won't show that it's valid since it's using a bad password. the fix will be something like this

for Password In PasswordsList:
    Password = Password.rstrip("\n")

    [SMTP Auth Here...]

4117bca

Closing this issue since it has been solved on in 40e156d please update the tools.


Release taskcat [0.9.6]

4f85cec Bump Version - Creating a new minor version
cd46ca6 Using the correct variable
4f0de83 Bump Version - Creating a new minor version
c8c80a5 Proper regex is key
39fe620 Bump Version - Creating a new minor version
3f108dd Fixing dev release fetch
52e0732 travis config updated prod release  based on tags
0434cde Update date css for docs
7f4d80d Bump Version - Creating a new minor version
38a2e10 Adjusting travis regex for prod release
65e2d44 Update date css for docs
9c798ea Bump Version - Creating a new minor version
5ab4f5b Throwing an exception if a parameter value is None. Fixes #391
7f1c33b Update e2e tests to check for default lambda source folder
77d7116 Ignore files created in legacy unit tests
7e85ded Update unit tests to check for source in default path
93ee0fa Update tests data to use default lambda_fuctions dir
09b13c9 Update package command default to look in project-root/lambda_functions
04bc0bf Increase timeout for taskcat version check - Set timeout 5 seconds
92b7b4d fix typo

/do-e2e-tests

Codecov Report

Merging #406 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #406      +/-   ##
==========================================
- Coverage   73.53%   73.49%   -0.04%
==========================================
  Files          30       30
  Lines        2724     2724
  Branches      464      464
==========================================
- Hits         2003     2002       -1
  Misses        611      611
- Partials      110      111       +1

Impacted Files
Coverage Δ

taskcat/_lambda_build.py
85.24% <0%> (-1.64%)
:arrow_down:

Continue to review full report at Codecov.

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

/do-e2e-tests


imageUrl with special characters, are not allowed in firebase messaging. But there exists a lots of image urls with different special characters.

Environment:

Operating System Version: Ubuntu 20.04.2 LTS
Firebase SDK version: 9.4.1
Firebase Product: messaging (fcm)
Node.js version: v12.21.0
NPM Version: 6.14.11

Steps to reproduce:

Following code is in typescript.

import * as admin from 'firebase-admin';

let message: admin.messaging.MulticastMessage = {
    tokens: ["<fcmtokens>"],
    notification: {
        title: "This is a test notification on fcm",
        body: "This notification is to check if image with specials chars can be sent in fcm notification",
        imageUrl: "https://s.yimg.com/uu/api/res/1.2/8e4p0MNjKRtV25O3t_qSMw--~B/aD00MjU7dz04MDA7YXBwaWQ9eXRhY2h5b24-/https://media.zenfs.com/en/nextshark_articles_509/287cce64e3a1cd49446e8751b89262ef"
    }
}
admin.messaging().sendMulticast(message)
    .then((response: admin.messaging.BatchResponse) => {
        console.log("Message has been sent...");
    })
    .catch((error: any) => {
        console.log(error);
        // This notification throws here, and flow comes in this section
        // Error is "notification.imageUrl must be a valid URL string"
    });

As we see, the above example, the imageUrl has multiple special characters, still it is an URL which can surely load an image.
But, firebase admin doesn't accept this.
Hope to see some upgrades in the pattern checking rules of imageUrl.
Thanks.

It's not really a matter of special characters. Our existing path regex doesn't allow empty path segments, and the // in the middle of the path is interpreted  as an empty path segment. Will see what we can do about that.

These days, the URLs are becoming too dynamic... However, thanks for  " Will see what we can do about that.". Really grateful if a viable solution is implemented.
Thanks.

any update on this?


Format on typing no longer works in newer versions

The most recent version that still works is v2021.3.680753044. In all the newer versions, format on typing no longer works.
1+2+3+4 should be formatted into 1 + 2 + 3 + 4 when starting a new line. But nothing happens in the newer versions.

Hi, thanks for the issue report, we are not longer support format on typing. You can use other formats like format on save or indentation format.

Hi, thanks for the issue report, we are not longer support format on typing. You can use other formats like format on save or indentation format.

But format on save doesn't seem to work either..

Can you please share your Python Output Logs or any other error that shows


Can only run tests with Unittest or Pytest running tests serially

To reproduce the the issue run pytest tests from the root directory.
The error is ' OSError: [WinError 6] The handle is invalid'
The cause is pytest by default runs tests in parallel. In parallel, the omnx logger is not able to get a handle.

Maintainers decided that will just use Unittest features for the time being. Movement to pytest specific features will require discussion. Moved to low priority. Only become high priority if test time becomes burdensome.


Scrollable Panel issue

It's definitely a great extension. I found out it can't be scroll after installing it. Using on my Yosemite iMac with Magic Mouse. Looking to see u fixing bug soon.

thanks for reply
I founded that i'll fix soon.

fixed enable scrollbar on tree view 1.0.2


fix: swap out our custom heading tokenizer to more closely follow remark-parse

Fix RM-4298

🧰 Changes
We used to have a custom tokenizer to allow headers without a space between the # char and the heading content. It was resulting in the position data being skewed, which then meant that when we tried to normalize links in headers in the editor (over here in this file) we crashed the page.
We've now adapted our tokenizer to closely follow remark-parse's approach (as defined here) but adapted the regex we match against to allow those cases without a space between the # char and the content.
🧬 QA & Testing

Broken on production.
Working in this PR app.

This PR was released!
🚀 Changes included in v6.39.3


[internal] Added TODO #131

Motivation
This is an initial PR to create a standard on how we could track betters TODOs on Github and enable the community to find possible issues to start work on.
Issues created:

https://github.com/coinbase/rest-hooks/issues/131

@ntucker let me know if you like the format

Hmm, this creates more overhead to know what the todo is. Maybe leave the first line there ?

Closing because no progress for several months. Feel free to open a new issue if you feel like!


Fixed Autofacing not happening item.UseTurn = true;

Please merge this fix when you get a chance.

Does this patch fix some kind of bug? Or does it just make every weapon auto-turn? (if so, I don't really want that, sorry)
Also, why did you delete ru-RU.lang, and mess with the .csproj file?

It makes weapons auto-turn which was missing from your version. I never told you to take all my changes just the auto-turn part - so see no reason for you to worry about how i changed my fork to make it compile on my side.

Sorry, I don't think all weapons need to auto-turn by default. Maybe this could be a config option instead.


Fresh install of @angular/cli NOT @angular-cli

Ive tried to install the @angular/cli and when I just try to check the version I get the error below.  Exact same error when I try to create a new project with ng
Bug Report or Feature Request (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ ] feature request

Versions.
Output from: ng --version.
npm --version 3.10.10
Windows 10
Repro steps.
npm install -g @angular/cli
ng -v
AppData\Roaming\npm\node_modules@angular\cli\models\config\config.js:77
throw new InvalidConfigError('Parsing .angular-cli.json failed. Please make sure your .angular-cli.json'
^
InvalidConfigError: Parsing .angular-cli.json failed. Please make sure your .angular-cli.json is valid JSON. Error:
SyntaxError: Unexpected token u in JSON at position 22
at InvalidConfigError (C:\Users\shawn\AppData\Roaming\npm\node_modules@angular\cli\models\config\config.js:10:9)
at Function.fromConfigPath (C:\Users\shawn\AppData\Roaming\npm\node_modules@angular\cli\models\config\config.js:77:19)
at Function.fromGlobal (C:\Users\shawn\AppData\Roaming\npm\node_modules@angular\cli\models\config.js:56:46)
at Object. (C:\Users\shawn\AppData\Roaming\npm\node_modules@angular\cli\commands\build.js:7:71)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)

Do you have an angular-cli.json file in your home directory?  If so, try removing it.

No my problem is there is no project yet.  I grabbed @angular/cli using npm and any command I try using ng generates the same error.  No project is ever generated.  I cant even check the version, ie: ng -v

Please check that there are no package.json files in any parent directories. That file is used to determine the the root of the application, if it finds one, it assumes you are in a CLI-based project.

The angular-cli.json file can exist in two different locations.  The first is within a project to provide the settings for the project.  The second is within a user's home directory to provide global defaults for all projects.  The stack trace provided appears to indicate that the error occurred while trying to parse the global configuration file.

Please make sure your .angular-cli.json is valid JSON. Json format error.


@W-14118931 - Update changelog

Description
Update Preview related PRs to changelog under one liner

Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

(change1)

How to Test-Drive This PR

(step1)

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[ ] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

To make sure we don't miss the change we made to the codebase, I've grouped the Preview feature related PRs under Support Storefront Preview. Only the PRs that is merged directly into develop are logged, the ones that merged into feature branch previouly is not since the code change is already reflected in the feature branch PR


chore(pingcap/tidb): bump go version to v1.21.0

Go v1.21.0 was release, we need to upgrade the CI/CD container image to adjust for pingcap/tidb repo.

/lgtm
/approve


Update README.md

Going through the process to get SoSTrades working on an OSX Mac (14.3.1), I can report the following:
For step 6, brew install jq works for me (with xcode command line tools installed and a brew environment installed)
For 3.1 Prerequisites, I downloaded the latest version of Docker (Docker version 25.0.3, build 4debf41) and it works.
But with that latest version, to get the version number of docker compose, the command is docker compose version not docker compose --version.  The reported version is Docker Compose version v2.24.5-desktop.1
3.4 invites the user to start and play with the Web UI.  I suggest that here, or in a later 3.5 or 3.6 you mention the following things I discovered (with help from the SoSTrades team):
From the "hamburger" menu, select Reference Management
In the filter, search for story
In the rightmost column (to the right of creation date), regenerate the references
From the "hamburger" menu, select Study Management
Create a New Study with a name you choose
For the Process Name, select WITNESS Coarse Dev Multi-Scenario Story Telling mda Process Climate Economics - WITNESS Repository
For the Study Name, choose your preferred study (e.g. usecase_witness_ms_mda)
For the Group Name, choose whether to make the study available to all users or SoSTrades-Dev.
After some minutes, a new study will appear
From the root of that study, you can press the Play button to begin the process of calculations
After some minutes, the status of the study will change from P (pending) to D done.
You can then pick a specific scenario (e.g. + damage -tax, fossil 100% from the mda_scenarios child of the study.
You can then visit graphs of the result in the Post Processing tab.
Note that at present, scenarios listed with - damage as the prefix encounter a fault that is not your fault.

I should also add that if the developer wants to run in the local (not a docker) environment, then petsc is needed. This can be installed on OSX using brew install petsc. It installs version 3.20, not version 3.19.

We are working on making the installation of PETSC optional and configurable as it brings some issues for some systems (like Windows)
This will be documented

Hi @MichaelTiemannOSC,
Many thanks for your feedback, this brings strong added value for the project.
We have indeed ongoing work on several points you mentionned, regarding documentation improvements, local deployment robustness (involving PETSc) as well as WITNESS studies post processing robustness.
Concerning the post processing issue, we are pleased to inform you that the development team managed to fix it. The fix is now available on the validation branches and validation platform. You can either :

check the updated study on the online platform here for example,
or update your local platform by pulling the code changes from models repositories (into ./sostrades-dev-tools/models/witness-core and ./sostrades-dev-tools/models/witness-energy ) and running docker compose up.

Local installation documentation has been updated and migrated to sphinx and deployed with readthedoc :
local platform installation link
A documentation on the use of the core execution engine (sostrades-core repository) has also been initiated and covers main principles to build a model and create a process.
core execution engine documentation link

Local installation documentation has been updated and migrated to sphinx and deployed with readthedoc :
local platform installation link
A documentation on the use of the core execution engine (sostrades-core repository) has also been initiated and covers main principles to build a model and create a process.
core execution engine documentation link

One comment about the screenshots on the readthedocs.io site: I think the screenshots need more resolution because resolution is lost in the screen capturing process.  When I scale up the size of my window, the screen captures are not very clear.


https://www.opencontainers.org/about missing space before runC repo link

It has It is available now athttps://github.com/opencontainers/runc, but should have a space after at.

Fixed!


Incorrect case in filename Core/Strings/Stringfile.cs

Fails to compile on Ubuntu 18.04, probably the case on all case-sensitive filesystems.
Renaming it to StringFile.cs seems to compile with no problems, though I'm currently unable to confirm if there's any runtime problems with that change as my Mono is throwing System.DllNotFoundExceptions on included library x64/libfluidsynth-2.so which is likely unrelated.

Bug confirmed.
Stringfile fixed, fluidsynth WIP- some dependency is missing for Linux

Added workaround for FluidMidi if fluidsynth init fails

Runs on my end now thanks, without MIDI of course. Hitting Esc at certain points throws an unhandled System.NullReferenceException at Fluid_Midi.cs line 1082 - a bValid check would probably be enough to stop that though I don't know if it would cause any leaks.


Generate declaration file when building for release

Add the declaration: true option within the configuration file when this package is built for release.
@ericvicenti This should allow this issue to be closed.
Thanks!

Hi, merged but I don't have publish access

Hey folks, I can do a publish but I just added @slorber to the repo so you can publish as you wish.
Apologies I haven't had more time to support this project, thanks for picking up the slack!

@slorber Please let me know when you have published!

@ericvicenti thanks for permissions.
I'm not familiar with release-it yet, and unfortunately I released a non-alpha version.
Do you think we can remove the alpha tag?
For alphas, do you run release-it manually or through yarn scripts? like yarn release --preRelease or something?

@evancloutier try 1.0.2-alpha.0


Improved code style.

Added second degree equation solver
math.zip

@A-Dizis You should create a separate issue for this feature request. And then make a pull request with your updated code.

@A-Dizis You should create a separate issue for this feature request. And then make a pull request with your updated code.
Originally posted by @timlg07 in https://github.com/dracoYT/MathC/pull/2#issuecomment-693673719
Hey thanks for the piece of advice. I m now more active in github, so I m currently still learning the standard processes of the platform.


Card does not appear until the camera loads

Currently, while the camera stream loads the card is not visible. Because of this, the layout of the UI changes multiple times on page load.
I suggest adding an option fallback which would take an image. This image would appear immediately and later change to a camera stream when it loads.

Since the release of 0.92, picture cards now support camera streams. I recommend using the core picture cards. I'll be deprecating this.


Clean up article and announcements

Going to not pull this in right now. It does change the routes and file names for article to be in line with announcements. I could also put announcement to the same as article if we want to.

Going to actually merge it right now. If we don't want to do it we can revert.


[Pulsar-init] Support cluster init using proxy url and protocol

Motivation
Pulsar cluster can be setup using SNI reverse proxy and Pulsar cluster metadata initialization should support cluster-metadata creation/update using proxy-protocol and URL. It will be also useful in Pulsar-helm-chart installation phase to create cluster metadata with proxy configuration as well.
Modification

Initialize Cluster metadata using Proxy URL and protocol.

To stop docbot complains, please add the following content to your PR description and select a checkbox:
- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->


fix(hooks): add ERC20ABI, ERC721ABI exports

Description
This PR resolves a runtime error caused by an incorrect import in TokenGate.tsx.
📝 Additional Information
Using @web3-ui/core caused the following error:
Error: Cannot find module '@web3-ui/hooks/src/constants'

I believe this is due to the fact that TokenGate.tsx was importing ERC20ABI using @web3-ui/hooks/src/constants which doesn't exist in the final build.
To resolve this, I added ERC20ABI as an export from @web3-ui/hooks and updated the imports in TokenGate.tsx.

@allcontributors please add @with-heart for code!


Port WithAcceptanceTest to TCK

@Mats-SX : please check this file against the original. I had to amend quite a few of the queries to fit in with the new process and want to ensure no subtleties were lost in translation...thanks

@petraselmer I cross-referenced to the original, and I find that your amendments are good improvements to the test cases. I've done many similar amendments myself, like adding labels to nodes, since we can not use the same way of asserting that exactly the same node was returned.


Re-enable async query handler unit test

Fix up and re-enable the test in tests/unit/test_create_async_query.py.

Merging this into the akislyuk-async-query-fixups branch to allow it to pass coverage checks.


Update Jackett please

Just spotted a new version of Jackett is out, could you update your script please

Script has been updated 6 Aug 2021 and will now grab the latest releases each time.


Logo is not animated

This module works (respond to almost every command I told), but no animation.
What is wrong? Thx.

Sorry, my bad. It is working now.


Npm beta version

Can we have a @next tag in npm that downloads the gh-pages version? Releasing this way is a pain. Last release is more than a month ago, i think faster releases would be better. faster bugs fixes and features that we can test and provide feedback before published into master.

That's a nice idea.
I guess it would still require publishing to npm, right? We do not have automated publishing at this time, and I think @LeaVerou is the only one who currently has the rights to publish...

Does anyone know when the next release is planned?

Where are the npm installation instructions?

@QuantumInformation npm installation instructions are on the homepage of the website.
@felixsanz I realized that if you really want to use the gh-pages version in your projects, you can already use a direct link to github (see this SO answer). So I don't think we need this @next tag. Also, I'm trying to set up regular releases, based on the change activity.


Validation skipping steps when previously visited

Repro steps:

use Fiddle
enter data in all fields of step 1, this will unlock step 2
enter data in all fields of step 2, this will unlock step 3
go back to step 2, clear any mandatory field
go to step 1
See that step 3 is accessible while step 2 does not validate.

Thanks for reporting @Doodlepants
Maybe all steps between step X and step Y should be validated if X < Y.
So in this case, when going from 1 to 3, both step 1 and 2 should be validated. What do you think ?

A good approach is often the simplest.  I don't think re-validating intermediate steps is overkill.
This validation could be done of all subsequent steps when going back to a previous step, it has the added bonus of providing each steps with an accurate feedback.  Be it Ok / Error / Unavailable due to previous step error.
I'd be happy to provide some schematic or help ( if I can, git is still very new to me )  :)

Sure. Any help would be appreciated. I'm wondering where should the validation stop in the case discussed above:
So we have
1 2 3 4
Step 2 is invalid and the user is on step 1. He wants to go straight to 3 but 2 is invalid. I am assuming that after validation he should be redirected to the first invalid step,
1 - > 3
1 2 3 4

I will find some time to do this by the end of the week.

Actually this seemed quite easy to implement (5 lines of code 😄 )

@Doodlepants See the same fiddle from above. It should validate all steps when going from a step to another where the difference between them is greater than 1 step.
https://jsfiddle.net/CristiJ/bt5dhqtf/286/
Therefore, you will not be allowed to jump to any step without validating everything which I think is good


Skins: User view & Full view

-- draft --
Story: There is a need to make simplified screen view. Considering potential patients the app should display minimum informations and minimize confusion.
Task:

Combine the 3 "authentication" checks into one icon "Authentication Status".
If all are okay, make it green with a tick.
If any item is not okay, make it either amber or red and display appropriate message depending on what items fails.

@UnHynek as discussed yesterday, does the following help clarify?

Whether we simply have a Green/ Red check mark without any text or if we include "Authentication Status" or "Verification Status" etc. is something that can be discussed. Perhaps a Red or Green check mark may suffice but this really needs to be assessed by UX experts and taking into consideration different countries/ regions.

To be discussed if we should remove the details as to why the Symbol is Green or Red i.e. do we really want to/ need to show the Serial Number, Product Status (name needs to change) and Package Verification icons. If the check mark is Red then the patient could (if they so wished) get more information as to why it is red. An alternative would be to just press the check mark if it was red and then have a reason why the check mark is red. The tech team need to assess the effort required for the different possibilities. All of this needs to be considered as part of the UX over haul, which will be a different activity.

The following 2 screen shots show the Happy Path:

The following 2 screen shots show the Un-happy Path:

The way the "Add to Cabinet" button is shown is far from ideal. Consideration should be given to having some way of enabling/ disabling this or at least aligning it with the icons that are on the screen.  When everything is enabled then it does not look that bad. As stated previously, all of this needs to be considered as part of the UX over haul, which will be a different activity.

The following shows the Happy Path when everything is enabled, in this case the icons would not be greyed out:

The attached excel file has the different combinations as to which symbol should be displayed. This needs to be reviewed by the Tech team to ensure the combination are correct.

Serial Number Exp Date Check Proposal v0.4.xlsx
@steinso2 @nhrishi @rite2bala @andrejstas @salboaie @thursbyk for your information and input.


Multi images inferencing

Hi @YuDeng , Thank you for sharing your great work.
How can I do model inferencing for the multi-image case as there is no C-Net available yet? My dataset has three images for each person and was taken from frontal and side.
You mentioned we could use the average shape for the multi-image case in #17 . I'm new to this area. Could you please expand on this?
If you could provide some example code would be much appreciated.
Thanks

Hi, you can take the average of the 'id' coefficients predicted by different images here: https://github.com/sicxu/Deep3DFaceRecon_pytorch/blob/73d491102af6731bded9ae6b3cc7466c3b2e9e48/models/bfm.py#L283, and set the 'exp' coef to zero. Then you just need to send the modified coef_dict to the reconstruction pipeline and it will return you an average face shape.

Correct me if I am wrong. I suspect the model would also need face landmarks from those additional images. If so, this may not apply to my case since other images were captured from the side and do not have landmarks like eyes, nose, etc.
Thanks

Correct me if I am wrong. I suspect the model would also need face landmarks from those additional images. If so, this may not apply to my case since other images were captured from the side and do not have landmarks like eyes, nose, etc.
Thanks

Hi @icedpanda,  were you able to run this for multi images?

@pyxploiter Haven't had a chance to run multi images yet, but a single image looks fine

@icedpanda

@pyxploiter还没有机会运行多张图片，但单张图片看起来不错
Have you found a job that fits your dataset?


Right hand drawer for Project reconfiguration

Description
Restructuring the project 'cog' to be a right-hand drawer with tabs, the first converting the modal to panels to make way for tabs for other project level reconfigurations - more complex token activities, permissions, NFT stuff.
Proposal
Here is a prototype layout via Figma - https://www.figma.com/file/ffk6DLhm4sfOdZfU8xc27u/JB-Treasury-UX-Review?node-id=464%3A80118 - Treasury Settings Page
Screenshots

Prototype
You may need to click on a project you own to see the cog.
https://bleeding-edge.juicebox.wtf/

Thanks for creating this issue @tankbottoms .
This is def a step in the right direction, however I think we should consider a few other options here.
For example, a dedicated Settings page for projects might be viable.
I think this is a good opportunity to do some "proper" user testing too.

Closed via https://github.com/jbx-protocol/juice-interface/pull/1875 MVP


ascii image converter saving formats

i have a suggestion if there's a way for program can save the a colored ascii image. Such an improvement will be very great.

@Genialngash Thanks for the suggestion! Actually I thought of that before but chose to not implement it. The colors you see with the --color flag in the terminal are actually escape codes which the terminal converts into colored text. See this gist for more info.
If one were to save this output in a .txt file, it'd look something like this:

Now, you can copy and paste this from the terminal with a command like cat and retain the results, but that's probably not the usecase most people are looking for when they want to save ascii art.
I'll think about this and leave this issue open in case someone else can come up with a workaround.

@Genialngash The recently released version 1.3.0 now supports exporting ascii art as PNG images now. I believe that should cover the lack of an option for saving colored output. Will close this issue in a day.


added 3 zk content pages

Added 3 pieces of content about ZK
Fixes on content:

Fixed author name display
Fixed table rendering (remarkGfm)

@emilyJLin95 @PrintRH thanks for reviewing! I submitted my resolutions


Implement wrapper for objectAccessControl.

This fixes #832. It is not used for much now, but will be needed
to implement the APIs to change Object Acls, and to fully
implement ObjectMetadata.

I'm curious - the text of #832 refers to a doc section for the JSON API, and your response to one of my earlier reviews also referenced that API's documentation (https://github.com/GoogleCloudPlatform/google-cloud-cpp/pull/810). Why is that?  I thought you were doing this implementation for the XML API.

PTAL. With respect to XML vs. JSON: the current proposal is to use XML only for the media read/write operations. I want to discuss this with @houglum when he returns before rubber stamping the document with that proposal.


新建的网站项目，启动报错

$ pnpm run dev

<EMAIL_ADDRESS>dev D:\code\modernjs-learn\website
modern dev

Starting the development server...
Failed to compile.
./src/App.tsx
Module parse failed: Unexpected token (9:2)
File was processed with these loaders:

./node_modules/.pnpm/registry.npmmirror.com+@pmmmwh+react-refresh-webpack-plugin<EMAIL_ADDRESS>You may need an additional loader to handle the result of these loaders.
|
| const App = () => (

|
|
Compiling...
Failed to compile.

the same problem
node version 14.17.6
pnpm version 6.20.1

File was processed with these loaders:

./node_modules/.pnpm/@pmmmwh+react-refresh-webpack-plugin<EMAIL_ADDRESS>You may need an additional loader to handle the result of these loaders.
|
| const App = () => (

|
|

和 #182 一样的问题

和 #182 一样的问题


Reduced charges per use is not parsed

Item Class: Life Flasks
Rarity: Magic
Simmering Ultimate Life Flask of the Distiller
--------
Recovers 920 Life over 3 Seconds
Consumes 7 (augmented) of 75 Charges on use
Currently has 0 Charges
--------
Requirements:
Level: 60
--------
Item Level: 66
--------
23% of Recovery applied Instantly
26% reduced Charges per use
--------
Right click to drink. Can only hold charges while in belt. Refill at Wells or by killing monsters.


Remove duplication of convertToErrorEvents

Summary
There is duplication of convertErrorToEvents between v1 and v2 message servers. This function should be moved somewhere and re-used instead of being duplicated.
See https://github.com/cosmos/ibc-go/pull/7421#discussion_r1795403833

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned
[ ] Estimate provided

could put this in internal/errors


EthStorage Devs Meeting #78 Agenda

Meeting Info

Apr 30, 2024, 11:00 UTC+8
Duration: 60 minutes
Meeting link
Anyone can comment on the agenda where they would like this to be discussed.

Agenda

Protocol/App/Research progress

Public testnet progress track
https://github.com/ethstorage/es-node/releases/tag/v0.1.14
An article about web3:// and EthStorage: https://www.notion.so/qzhodl/dApp-2f025b42dae44da2aed13b7f0e612a04

fix reviews for da phase 1: https://github.com/ethstorage/op-geth/pull/2 and https://github.com/ethstorage/optimism/pull/23
fix da-server url problem
Now investigating celestia da, reading docs from Qiang.
TODO: refine da phase 2 pr

以太坊中文周会 Proposal

fix the commit of storage-contracts-v1 pr, support automatic update of startTime and verify deployed contracts.
ethstorage-sdk supports uploading folders and fixes the problem of "nonce too high" when too many blob transactions are submitted.
the blob upload function of ethfs-cli is implemented using ethstorage-sdk.

https://docs.ethstorage.io/rollup-guide
go-rapidsnark/snarkjs tests and fixes:

fix cross-platform build error on github action
https://github.com/ethstorage/es-node/pull/281
https://github.com/ethstorage/ethstorage-doc/pull/26

Study OP e2e test
Replay zkWasm smoke test

EthStorage -- sync

Code Change:
https://github.com/ethstorage/es-node/issues/253, https://github.com/ethstorage/es-node/pull/268, network dashboard

investigation: https://github.com/ethstorage/es-node/issues/176, risc zero trusted setup, p0tion， summary

Code review

Public testnet

Modify the end-to-end test for the OP e2e

Discuss https://ethresear.ch/t/mining-attacks-on-pora-proof-of-random-access/19323

More work on: https://github.com/ethstorage/zkWasm
Complete an end 2 end script : https://github.com/ethstorage/zkWasm/blob/frank/cont_script/continuation.md
re-started studying on op-reth


Invalid input data running an ASPIF input

Dear Matthias,
I have been trying to run the diff-SAT for the probabilistic part of it (PrASP), but can't get the system to accept the aspif file. It is probably something I am doing wrong myself. I have been trying so far using your examples in the readme.
I have tested the 0.5.0 and 0.5.2 releases, getting the same behavior.
Steps to reproduce the behavior:

Create file test.lp and copy paste in it the code in example (1).
Run "clingo test.lp --trans-ext=all --pre=aspif > test.aspif"
Run "java -jar diffSAT.jar test.aspif -n 100 --solverarg diversify true --solverarg suppressAnswers true --solverarg showProbsOfSymbols true"
Stdout: "diff-SAT 0.5.2

Error [-100]: Invalid input data:"
Desktop:

OS: tested on Windows and Linux

Best Wishes
David

Here I am copy-pasting the content of test.aspif:
asp 1 0 0
1 0 1 1 0 0
1 0 1 2 0 0
1 0 1 3 0 0
1 0 1 4 0 0
1 0 1 8 0 0
1 0 1 9 0 0
1 0 0 0 1 -11
1 0 1 13 0 1 11
1 0 0 0 1 -15
1 0 1 17 0 1 15
1 0 1 5 0 2 6 7
1 0 0 0 2 10 7
1 0 0 0 2 14 6
1 0 1 10 0 1 -18
1 0 1 11 0 1 -18
1 0 1 18 0 1 -10
1 0 1 7 0 1 -19
1 0 1 11 0 1 -19
1 0 1 19 0 1 -7
1 0 1 14 0 1 -20
1 0 1 15 0 1 -20
1 0 1 20 0 1 -14
1 0 1 6 0 1 -21
1 0 1 15 0 1 -21
1 0 1 21 0 1 -6
4 7 coin(1) 0
4 7 coin(2) 0
4 19 pr(heads(1),5000) 0
4 19 pr(heads(2),5000) 0
4 3 win 1 5
4 8 heads(2) 1 6
4 8 heads(1) 1 7
4 8 tails(1) 1 10
4 8 tails(2) 1 14
0

Dear David,
thanks for reporting this behavior. It looks like an encoding issue: the aspif file generated
by clingo appears to be in UCS-2 format (which adds unrecognized extra bytes to the
file header), whereas the current version of diff-SAT expects input in UTF-8.
A quick workaround is to open the generated aspif file with Notepad++ (or some other editor which
supports switching between encodings), to change the encoding there to UTF-8, and to save the file.
Best regards,
Matthias
From: David-Tuc<EMAIL_ADDRESS>Sent: Monday, February 1, 2021 11:52 AM
To: MatthiasNickles/diff-SAT<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [MatthiasNickles/diff-SAT] Invalid input data running an ASPIF input (#2)
Dear Matthias,
I have been trying to run the diff-SAT for the probabilistic part of it (PrASP), but can't get the system to accept the aspif file. It is probably something I am doing wrong myself. I have been trying so far using your examples in the readme.
I have tested the 0.5.0 and 0.5.2 releases, getting the same behavior.
Steps to reproduce the behavior:

Create file test.lp and copy paste in it the code in example (1).
Run "clingo test.lp --trans-ext=all --pre=aspif > test.aspif"
Run "java -jar diffSAT.jar test.aspif -n 100 --solverarg diversify true --solverarg suppressAnswers true --solverarg showProbsOfSymbols true"
Stdout: "diff-SAT 0.5.2
Error [-100]: Invalid input data:"
Desktop:

OS: tested on Windows and Linux
Best Wishes
David

Here I am copy-pasting the content of test.aspif:
asp 1 0 0
1 0 1 1 0 0
1 0 1 2 0 0
1 0 1 3 0 0
1 0 1 4 0 0
1 0 1 8 0 0
1 0 1 9 0 0
1 0 0 0 1 -11
1 0 1 13 0 1 11
1 0 0 0 1 -15
1 0 1 17 0 1 15
1 0 1 5 0 2 6 7
1 0 0 0 2 10 7
1 0 0 0 2 14 6
1 0 1 10 0 1 -18
1 0 1 11 0 1 -18
1 0 1 18 0 1 -10
1 0 1 7 0 1 -19
1 0 1 11 0 1 -19
1 0 1 19 0 1 -7
1 0 1 14 0 1 -20
1 0 1 15 0 1 -20
1 0 1 20 0 1 -14
1 0 1 6 0 1 -21
1 0 1 15 0 1 -21
1 0 1 21 0 1 -6
4 7 coin(1) 0
4 7 coin(2) 0
4 19 pr(heads(1),5000) 0
4 19 pr(heads(2),5000) 0
4 3 win 1 5
4 8 heads(2) 1 6
4 8 heads(1) 1 7
4 8 tails(1) 1 10
4 8 tails(2) 1 14
0
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/MatthiasNickles/diff-SAT/issues/2 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AJWQSXD3MCFNMW6IURJ2ML3S42IUPANCNFSM4W4U3DGA .  https://github.com/notifications/beacon/AJWQSXEAT46L6E7BNVSGS2DS42IUPA5CNFSM4W4U3DGKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4L4UXHTA.gif

Dear Matthias,
Indeed that does the trick ! Thank you very much.
Best Wishes
David

Addendum: The next version of diff-SAT (planned to appear in ca. 1-2 weeks) will support input files in UCS-2 and UTF-16 encoding directly.


#158423645 expand ride details to request ride

What does this PR do?

Adds passengers' view of an expanded ride offer's details.

A passenger shuold be able to view all the details associated with that particular ride offer, with an option to confirm/submit the ride requests.

Description of Task to be completed?

Building (without functionality), the UI elements with HTML&CSS.

How the app should be tested
On a webbrowser:

Download the repo
Extract and navigate into the Ride-My-Way folder
in the UI folder, open expanded_ride_passenger.html on a browser

On a  terminal (Linux):

Install and set up git sudo apt install git

Clone the repository git clone https://github.com/artorious/Ride-My-Way.git

Run git checkout ft-passengers-expanded-ride-158423645

Run cd Ride-My-Way/UI/

Open expanded_ride_passenger.html on a browser

Any background context you want to provide?
The UI has no functionality.
Developed using HTML and CSS
What are the relevant pivotal tracker stories?
#158423645
Screenshots

Questions:

Great UI,my kind of colors

Awesome progress

@kwanj-k @xcixor @LWanjiru Thanks for the input. I've made adjustments to make the page easier on the eyes


useState troubles to update when receiving new events

Hello everyone, please help with an issue I encountered while using pusher-websocket-react-native. I'm trying to implement a small chat, and the problem is that when I receive a notification in the onEvent method and try to update my local state, the messages value always equals an empty array.
cosnt [messages, setMessages] = useState([])
useEffect(() => {
      getInitialMessagesByRestApi().then(res => {setMessages(res.data)})
}, [])

...
const onEvent = () => {
const onEvent = event => {
    const parsedData = JSON.parse(event.data)

   setMessages([...messages, event.data]) <-- messages in this place always empty
}

I already tried to get initial messages after pusher init, it works but when i am gettig few events like one by one it , messages array always equal to initial without prev message from pusher events

I'm also struggling with this. onEvent handlers seem to be completely unaware of any state changes - they only have a snapshot of state when pusher was initiated. This makes it really difficult to do anything useful with events in a modern functional component react native app...

Could you update to  setMessages(messages => ([...messages, event.data])); ? With this I was able to see the array being updated correctly.


It is prohibited to sell the material.

[Thanks for splitting testdata from libjxl]
The main page should make it a bit more clear that imagecompression.info/* files are a bit more restrictive than the top-level BSD style license.

it must be available freely under the conditions stated here. That is, it is prohibited to sell the material.

https://github.com/libjxl/testdata/blob/main/third_party/imagecompression.info/LICENSE.txt

I see a single solution for my issue: Upload a stripped version of testdata to Debian archive. I'll remove the subfolder imagecompression.info. This means I'll need to remove a portion of the libjxl test suite. Hopefully there is nothing really particular with these images.
cc: @jonsneyers

#2 prepares the move away from using any data from imagecompression.info. Leaving this open until we have updated the commit hash of the submodule testdata in libjxl/libjxl and adapted the corresponding tests.