Fixed Nested Docs URL conflicts

Description
The issue (ref #5879) existed due to the code was only considering the last part of the slug to fetch the page. ref examples\nested-docs\next-app\app\[...slug]\page.tsx
The fix was to use the full path of the slug to fetch the page.

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

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

Checklist:

[x] Existing test suite passes locally with my changes

@denolfe i belive the issue is solved can you please review it and tell me if any changes are required.
Thanks

Examples will be reworked for 3.0. If still an issue after, feel free to open a new PR.


Expose allow_unknown_field from XGBoost loader

Treelite offers an optional flag allow_unknown_field when parsing an XGBoost JSON model. This flag should be exposed to Triton-FIL.

Just manually kicked off CI. Not sure why it didn't go through before. Will merge tomorrow, assuming it passes.


About when Configuration's cacheEnabled is ture, SqlSession#select run later, other query-operation's result will be not accurate.

MyBatis version
master
Database vendor and version
Mysql 8.0.18
Test case or example project
In a SqlSession Object, when I use SqlSession#select(string, ResultHandler) to call a certain statement, all subsequent calls to the same statement and parameters cannot obtain normal results. I checked the source code in the org.apache.ibatis.executor.BaseExecutor class. In line 152, if the ResultHandler parameter is not null, the cache will not be read.But in line 329,it will be cached and does not check if it is empty. Is it a bug? Or is it like this in design?
Steps to reproduce
Example Code:
SqlSessionFactory factory = new SqlSessionFactoryBuilder().build(reader);
try (SqlSession session = factory.openSession()){
    List<User> user = session.selectList("club.kingon.ln.dao.UserDao.selectUsers");
    // print three result
    System.out.println(user);

    List<User> user2 = new ArrayList<>();
    session.select("selectUsers", (ctx) -> {
        user2.add((User) ctx.getResultObject());
    });
    // print three result
    System.out.println(user2);

    List<User> user3 = session.selectList("club.kingon.ln.dao.UserDao.selectUsers");
    // print empty. I think maybe the ResultHandler should also be null checked when writing to the cache and to print three result ?
    System.out.println(user3);
}

Code pic:

Maybe problem source code pic:

Hello @521274311 ,
ResultHandler is not a handy class to allow writing a lambda expression.
It is designed to provide a way to write a custom mapping logic that is too complex for a result map.
So, you should not use the same statement with and without ResultHandler.
They could return different results (see the doc for the details).
There was a similar question on StackOverflow a while ago.

@521274311 Please refer to @harawata 's comment! We close this issue. Also you can reopen this if need.


Delete file while editing in the tree view does not remove it from working file list

VSCode Version: 1.2.0
OS Version: Windows 10

Steps to Reproduce:

Open 'test.txt' file in directory
Edit and do not save
Delete 'test.txt' in tree view
create same file in the console (e.g: type nul > test.txt)
back to code, open it again in the tree view
it still show the last edited state.

We have to do this so that you do not loose your unsaved work in the editor.

Ok, It is understood. But in visual studio, in such case, application will ask me to choose three options

Discard : Discard current edited state
Overwrite : Ignore file system change but use my edited state
Cancel : do nothing just let it open in edited state (VS Code case)

Yes, we have another issue to confirm in that case.


How many concurrent watchers can sentinl run at same time

How many concurrent watchers can sentinl run at same time.

@seshu3564 Sentinl relies on Elasticsearch search thread pool. By default, it is 1000 concurrent requests (if server hardware is powerful enough), also this value can be configured. Thus theoretically, by default, we can support 1000 watchers running at the same time. Though Sentinl was never tested with this number of watchers.
One more thing, in Sentinl configuration there are hard limits for number watchers and alarms, look the links below. If you need to increase the limits you must specify it in kibana.yml.
https://github.com/sirensolutions/sentinl/blob/master/index.js#L85
https://github.com/sirensolutions/sentinl/blob/master/index.js#L86
We plan to remove these limits in the nearest future.

Added to wiki/FAQ, closing


Fediverse replies to bidirectional comments get “blackholed”

Quick summary
We're seeing a peculiar issue with the bidirectional commenting supported by the latest version of the WordPress plugin: (some) Mastodon comments to a blog comment to a previous Mastodon comment fail to appear in the proper place, breaking the chain post > Mastodon comment > blog comment > Mastodon comment.
Apparently the second Mastodon comment gets assigned blog-side to the “main page” rather than the original comment, “disappearing” blog-side, since the main page generally doesn't actually support comments itself.
An example of this in action can be seen here
https://www.matteozenatti.net/2024/01/10/delle-laudi-della-musica-09/
vs
https://sociale.network/@oblomov/111731248300928414
This may or may not be related to #638
Steps to reproduce

Write a post on the blog.
Comment from the Fediverse.
Reply to the Fediverse comment from the blog
Reply to the blog comment from the Fediverse

What you expected to happen
The reply at step 4. appears in the comment thread starting from 2., on the blog.
What actually happened
The threading seems to work correctly Fediverse-side, but not blog-side.
Impact
Some (< 50%)
Available workarounds?
No but the platform is still usable
Logs or notes
No response

The blog owner confirmed that he could see the comment, approve it, even reply to it, but it was attached to the wrong object (main page rather than the blog comment it was a reply to).

Does he also use webmentions in combination with fed.brid.gy?

Hi, I'm the owner. I did't use webmentions neither fed.brid.gy. Now I installed Webmentions plugin for wordpress. But I don't get what I'd do with fed.brid.gy

No.

Edit: i got wrong what I read before, so now I disinstalled Webmentions and can answer: no, i don't use webmentions with fed.brid.gy

Ok, today same issue: an answer to an answer gets this
"Un nuovo commento all'articolo "Main Page" è in attesa della tua approvazione https://www.matteozenatti.net/": it address to the home page, and not to the page of the post to comment.


Implement reporting feedback

Description
Creating this as a placeholder since we may not implement feedback until the new year
WG feedback

[ ] team may not use report without formatting
[ ] Is there an OIPC review attached to the file?
[ ] include # of days overdue / when LDD has been missed
[ ] indicate that LDD includes extensions applied

Jamie and Andrew:

[ ] 3 big stages recommended by Jaimie to include: CFR, Gathering, Sign off
[ ] For closed requests: Percent on time, average days overdue
[ ] For received requests: Frequency/on time
[ ] Date range retention policy - 5 fiscal years. Going forward, keep CRTS data but don't include it. Just have AXIS data.
[ ] Test if data can run during day - any timeout errors? (Run time has diminished since moving from Oracle to Sequel)
[ ] Enthusiasm for ability to sort by RBAC

@m-prodan @arielleandrews feel free to add any reporting feedback from the pilot meetings or working group session

Assumption discussed at Jan 7/22 - Those who want data manipulation option and those who want a finished, formatted option

Closing this ticket, Andrew will be reviewing and giving feedback on redash this sprint


Update 1.7.2

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found some lint.
Here's what I've got...
For recipe/meta.yaml:

❌ requirements: build: swig<4.2 must contain a space between the name and the pin, i.e. swig <4.2

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Hi @conda-forge/core team, this package appears unmaintained. Could I get a review on this update?

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like there was nothing to do.
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11991831312. Examine the logs at this URL for more detail.

One thing that's a bit silly is that each job builds the same libfdt and dtc
# job for 3.9
anaconda upload \
    /home/conda/feedstock_root/build_artifacts/linux-64/libfdt-1.7.2-hb9d3cd8_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/dtc-1.7.2-hb9d3cd8_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/pylibfdt-1.7.2-py39h8cd3c5a_0.conda
# 3.10
anaconda upload \
    /home/conda/feedstock_root/build_artifacts/linux-64/libfdt-1.7.2-hb9d3cd8_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/dtc-1.7.2-hb9d3cd8_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/pylibfdt-1.7.2-py310ha75aee5_0.conda
# 3.11
anaconda upload \
    /home/conda/feedstock_root/build_artifacts/linux-64/libfdt-1.7.2-hb9d3cd8_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/dtc-1.7.2-hb9d3cd8_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/pylibfdt-1.7.2-py311h9ecbd09_0.conda
# 3.12
anaconda upload \
    /home/conda/feedstock_root/build_artifacts/linux-64/libfdt-1.7.2-hb9d3cd8_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/dtc-1.7.2-hb9d3cd8_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/pylibfdt-1.7.2-py312h66e93f0_0.conda

This is not the end of the world, in the sense that all of them have exactly colliding build strings, meaning it'll only get uploaded once (by whichever job finishes first).
What would be better would be to collapse them all into a single job per platform that builds the lib and then 4 x pylibfdt. Given that he build scripts are already separated here, that should actually be pretty easy.

@conda-forge-admin, please rerender

One thing that's a bit silly is that each job builds the same libfdt and dtc

Contrast to all-in-one after updates 🙃
anaconda upload \
    /home/conda/feedstock_root/build_artifacts/linux-64/libfdt-1.7.2-hb9d3cd8_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/dtc-1.7.2-hb9d3cd8_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/pylibfdt-1.7.2-py312h66e93f0_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/pylibfdt-1.7.2-py310ha75aee5_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/pylibfdt-1.7.2-py39h8cd3c5a_0.conda \
    /home/conda/feedstock_root/build_artifacts/linux-64/pylibfdt-1.7.2-py311h9ecbd09_0.conda

Thanks for consolidating the build outputs, that's a nice improvement 😄. Thank you for reviewing my PR


Proxy options in fabric8 maven plugin

Description
Proxy options in fabric8 maven plugin
Info
openshift environment sits behind a corporate proxy and on deploying with 'mvn fabric8:deploy', we can see there are 502 error code,
[WARNING] F8: Cannot access cluster for detecting mode: Unexpected response code for CONNECT: 502
[INFO] F8: Running in Kubernetes mode
[INFO] F8: Running generator spring-boot
[INFO] F8: spring-boot: Using ImageStreamTag 'fis-java-openshift:2.0' from namespace 'xxx-xxx' as builder image
[INFO] F8: using resource templates from C\examples\examples-ocp-archetype\fuse70-spring-boot\src\main\fabric8
[INFO] F8: fmp-service: Adding a default service 'fuse70-spring-boot' with ports [8080]
[INFO] F8: spring-boot-health-check: Adding readiness probe on port 8081, path='/health', scheme='HTTP', with initial delay 10 seconds
[INFO] F8: spring-boot-health-check: Adding liveness probe on port 8081, path='/health', scheme='HTTP', with initial delay 180 seconds
[INFO] F8: fmp-revision-history: Adding revision history limit to 2
[INFO] F8: f8-icon: Adding icon for deployment
[INFO] F8: f8-icon: Adding icon for service
[INFO] F8: validating fuse70-spring-boot\target\classes\META-INF\fabric8\openshift\fuse70-spring-boot-deploymentconfig.yml resource
[ERROR] Failed to load json schema!
java.net.ConnectException: Connection timed out: connect
        at java.net.DualStackPlainSocketImpl.connect0(Native Method)
        at java.net.DualStackPlainSocketImpl.socketConnect(DualStackPlainSocketImpl.java:79)
        at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:350)
        at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:206)
        at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:188)
        at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:172)
        at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
        at java.net.Socket.connect(Socket.java:589)
        at sun.security.ssl.SSLSocketImpl.connect(SSLSocketImpl.java:668)

f-m-p version :
Maven version (mvn -v) : 3.5.2
Kubernetes / OpenShift setup and version : ose v3.9.0 and kubernetes v1.9.1

Maven proxy settings to by-pass the proxy does not help much.
mvn clean -DskipTests fabric8:deploy -Popenshift -Dfabric8.generator.fromMode=istag -Dfabric8.generator.from=namespace/fis-java-openshift:2.0 -Dhttp.proxy Host=xxxxx -Dhttp.proxyPort=xxxx -Dhttp.nonProxyHosts=hostname -Dfabric8.mode=openshift
The only option we see when openshift master is behind a proxy is to manually deploy importing YAML or JSON file, generated using the fabric:resource, but on production/development environments, manually deploying does not seem to be an elegant approach.
We need a way in the plugin to have some options to filter and by-pass the proxy if openshift cluster is setting behind.

@rhuss : Does it make sense to provide these kinds of options to by-pass proxy in the plugin itself? WDYT?

@rhuss, practically speaking, Maven respects the usual Java properties http.proxyHost and others. So, if the plugin is Maven based, it would be nice to have a way to respect the properties in the plugin as well.

I agree that we should honour proxy options, too, if the plugin has to reach out.
But for this particular issue at end I would much more favour a solution where the schema to validate is not downloaded everytime. I wonder whether it wouldn't be possible to include the Json schema directly within the plugin (maybe for all API versions known) and reach out to the net only for unknown APIs (e.g. those which are introduced after the plugin was released).
This would help in speeding up fmp and also help for offline mode to not unnecessarily breaking the build.

The fix to use a embedded version of the schema has been added a while ago by @hrishin and should be present in latest version.
@vrlgohel, are you using a old version?

@nicolaferraro, i am using fabric8-maven-parent version, 3.5.33.

@vrlgohel : Could you please try with latest version i.e. 3.5.41?

@rohanKanojia , testing with the latest version 3.5.41, there is a different WARN message now,
[INFO] --- fabric8-maven-plugin:3.5.41:build (default) @ fuse70-spring-boot ---
--
[INFO] F8: Using OpenShift build with strategy S2I
[WARNING] Error reading service account token from: [/var/run/secrets/kubernetes.io/serviceaccount/token]. Ignoring.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 6.675 s
[INFO] Finished at: 2018-08-28T09:43:37+01:00
[INFO] Final Memory: 71M/912M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal io.fabric8:fabric8-maven-plugin:3.5.41:build (default) on project fuse70-spring-boot: Execution default of goal io.fabric8:fabric8-maven-plugin:3.5.41:build failed: Failed to parse ipAddresskubernetes.default.svc: Unknown host kubernetes.default.svc -> [Help 1]

Has there been any updates to this issue since closed?

@JackMj : Are you facing this error too? On which version?


[Discussions and Proposals]Is there any way to control focus move to specific button on android?

Description
Sometimes, there will be some requirements that specify the focus. For example，4 buttons on the page,  ABCD；I want to move focus from A to D directly;

TouchableHighlight nextFocus are inconvenient to use。

I think you have to use the nextFocus implementation.
If not, you can try on the event blur of the component A to set the hasTVPreferredFocus to true on the component D.
I've tried this before.But the button has default behavior, for example，foucs is on A now and click right, B gets the focus first, and then D gets the focus.

I think you have to use the nextFocus implementation.
If not, you can try on the event blur of the component A to set the hasTVPreferredFocus to true on the component D.

I've tried this before.But the button has default behavior, for example，foucs is on A now and click right, B gets the focus first, and then D gets the focus.

I think you have to use the nextFocus implementation.
If not, you can try on the event blur of the component A to set the hasTVPreferredFocus to true on the component D.

I've tried this before.But the button has default behavior, for example，focus is on A now and click right, B gets the focus first, and then D gets the focus.

I see. Thats the expected behaviour, I think.
But! for the nextFocus that should not happen, as I see it it should focus directly the D component

I see. Thats the expected behaviour, I think.
But! for the nextFocus that should not happen, as I see it it should focus directly the D component

Can you provide some code examples about how to use nextFocus？I was confused with [https://reactnative.dev/docs/view#nextfocusdown-android](react native document).
Thanks~

I think there's an example on https://github.com/react-native-tvos/react-native-tvos/blob/740fecdaef5ebc3460e312685228f6501efe53db/packages/rn-tester/js/examples/DirectionalNextFocus/DirectionalNextFocusExample.js
Basically, what you need to do is store the ref of the component you want to force focus to, in this case D:
import {
  TouchableOpacity,
  findNodeHandle,
} from 'react-native'

...

<TouchableOpacity
 ref={(component) => {
            const nextFocus = findNodeHandle(component)
          }
        }}
/>

And from the component you want to force the next focus, in this case A:
<TouchableOpacity
 nextFocusDown={nextFocus}
/>

I recommend having a global state manager so that this is easily achievable from different components nested in your components tree..
I was able to manage it by having a focus context and saving the forced next focus directions with this schema:
{
    up: null,
    right: null,
    down: null,
    left: null,
}

And on the component:
<TouchableOpacity
 nextFocusDown={focus.down}
/>

for example..
Hope it helps!

I think there's an example on https://github.com/react-native-tvos/react-native-tvos/blob/740fecdaef5ebc3460e312685228f6501efe53db/packages/rn-tester/js/examples/DirectionalNextFocus/DirectionalNextFocusExample.js
Basically, what you need to do is store the ref of the component you want to force focus to, in this case D:
import {
  TouchableOpacity,
  findNodeHandle,
} from 'react-native'

...

<TouchableOpacity
 ref={(component) => {
            const nextFocus = findNodeHandle(component)
          }
        }}
/>

And from the component you want to force the next focus, in this case A:
<TouchableOpacity
 nextFocusDown={nextFocus}
/>

I recommend having a global state manager so that this is easily achievable from different components nested in your components tree..
I was able to manage it by having a focus context and saving the forced next focus directions with this schema:
{
    up: null,
    right: null,
    down: null,
    left: null,
}

And on the component:
<TouchableOpacity
 nextFocusDown={focus.down}
/>

for example..
Hope it helps!

It works well! I think the point is how to store each ref the button want to move and how to update it.
Thanks!

Yes! The main thins is being clean storing the refs.
Glad it helped!


Remove Microsoft Azure deployment section from README.md until it's implemented

It seems azure deployment was listed as a feature in the docs before it was actually supported by now-cli.
I'm also new to now-cli and just become confused by the same error message and found the related issue:
https://github.com/zeit/now-cli/issues/787
It looks like it was intended to be removed a couple months ago, but never was.  It should be say to revert this commit and add this section back to docs when the feature is added.

Good call @mattmazzola. Thank you!


Unable to create labels when shipping to Germany

From https://github.com/JouwWeb/sendcloud/pull/5#issuecomment-724139672:

@bockhauzen: There also seem to be an issue with:
client->createLabel
When trying to use DE as country code it seems to fail.
Im trying to debug it, probaly my brains are just failing.

@bockhauzen Could you please provide a code example that causes this error? When I use below code I am able to create a label without problem:
use JouwWeb\SendCloud\Client;
use JouwWeb\SendCloud\Model\Address;

$client = new Client('xxx', 'xxx');
$parcel = $client->createParcel(
    new Address('P. Erson', 'C. Ompany', 'S. Treet', '29A', 'C. Ity', '14199', 'DE'<EMAIL_ADDRESS>'+31612345678'),
    null,
    null,
    null,
    null,
    null,
    null
);
// dd($client->getShippingMethods());
$shippingMethodId = 137; // UPS Standard 0-3kg, Unstamped letter = 8
$label = $client->createLabel($parcel, $shippingMethodId, null);

dd($label);

Note that if you create a parcel in Sendcloud and it fails to create a label, you can go to your created parcels on their dashboard to see if the created parcel has any address errors.

You can close this issue
Had a call with sendcloud, apparently 41 can't be used for shipping from germany to the Netherlands.


Could not establish connection hive2

When I try to create a connection (Apache hive) I get an error:

is it still actual?

there is no update on the issue for a long time
additional info is needed to reproduce an issue.
ticket will be closed. feel free to reopen an issue and add info that could help to investigate it.


Android is hard to integrate

I am new to Android development and we are doing a cross platform game. I find your instructions to be confusing and it seems much more bloated than other platform integrations. Why don't you simply make a simple implementation and read a string from "/proc/sys/kernel/random/uuid" to avoid all the java integration?

Is your problem with the API or actually integrating into your build system? Regarding the API all you have to do is call initJni() once before you ever call newGuid().
The goal of this project was to go the opposite direction of other "cross platform" UUID libraries. Instead of implementing one portable guid generator I wanted to use the officially recommended uuid generation technique on the platform. I couldn't find any documentation that says "do not read from /proc/sys/kernel/random/uuid, use the java function instead" but from what I read they do recommend the java function. Since Google seems to intend that you do it that way, that is the way I did it.
The libuuid function used on the gnu linux version and the java randomUUID function used in the android version are possibly doing what you suggest behind the scenes. The reason I didn't do it directly is to defend against changes in the platform, like if android Q removes /proc/sys/kernel/random/uuid or something like that.
The downside to doing it the way you suggest is pretty minor though and I can see the benefit for sure. I wouldn't be opposed to a new mode like GUID_LINUX_KERNEL or something like that using your technique. Unless there is some permission issue that stops your from doing that on Android.


Fix static library build

Installing currently does not work if one wants to include the static library.
Tiro should support two modes: BUILD_SHARED_LIBS: true/false
The static library needs extra care, as it needs to be partially linked: dependencies like absl or fmt should be bundled and their symbols should be hidden.
Details:

https://cristianadam.eu/20190501/bundling-together-static-libraries-with-cmake/

Example:
$ ld -Ur --version-script=api.version.gcc  --whole-archive libtiro_static.a --no-whole-archive ../_deps/fmt-build/libfmtd.a [...]

Closing this for now because using absl as a submodule (via FetchContent) is problematic combined with installation of static libraries. The static configuration has been removed for now to simplify things.


fix(docs): update installation steps for windows and linux

This PR updates the installation instructions for Windows and Linux.
We didn't have Windows installation steps and the Linux one was using brew (needs another extra installation);
sort of an anti-pattern from how we install the rest of the ecosystem tools like containerd, etc.

ctr run works okay but getting this when I run nerdctl run, let me fix that and update the README too:
nerdctl run --rm -it mcr.microsoft.com/windows/nanoserver:ltsc2022 cmd
time="2023-03-20T04:32:35Z" level=fatal msg="failed to create default network: needs CNI plugin \"nat\" to be installed in CNI_PATH (\"C:\\\\Program Files\\\\containerd\\\\cni\\\\bin\"), see https://github.com/containernetworking/plugins/releases: exec: \"C:\\\\Program Files\\\\containerd\\\\cni\\\\bin\\\\nat\": file does not exist"

@AkihiroSuda -- thanks for the review, going to make the changes.
I finally have nerdctl running but the way I've set up the CNI-plugin is hacky hence I get this warning:
level=warning msg="default network named \"nat\" does not have an internal nerdctl ID or nerdctl-managed config file, it was most likely NOT created by nerdctl"

What is the best way of setting up the CNI plugin for windows.
I was trying to adapt this bash script here (which I think should be done in PowerShell instead, working on that) - https://gist.github.com/profnandaa/e9ab7bd7e069ce2728840b041ce4c300

@profnandaa don't worry about that warning message, even if you didn't explicitly ask for network settings for your container, it's due to the fact that nat is the network name nerdctl defaults to on Windows.
This special treatment of the default network is the result of https://github.com/containerd/nerdctl/pull/1538, after which nerdctl can tell whether a network was initially created by itself as the default network by setting a special label on it.

I was trying to adapt this bash script here

That's perfect since it's basically the "reference configuration" which is used in both the containerd CI, as well as being basically 1:1 lifted to be used in the nerdctl CI too.
Although there's basically no reason to worry about that warning message, and I wouldn't necessarily bother mentioning it in the docs, but you could dodge it by opening the network config file shown in nerdctl network list and either:

renaming that network to something other than nat (NOTE: the type should still stay nat!)
keep the nat name but add the following line to the top-level of the config:

{
  ...
   // NOTE: this ID field is all you'll really need, and you can invent your own ID if needed.
  "nerdctlID": "d919a100ce6b45524d415d52d088d5817587c6dd8c3691b03b8063c44d043523",
  "nerdctlLabels": {
    "nerdctl/default-network": "true"
  },
  ...
}


QR Code Camera Activity: Fullscreen button Does Not Work

Hey @llaske  ,
I found this issue in QR code Camera Activity where the full screen button Does not do anything.
Is it supposed to have such behaviour or is it a bug?
if it is meant to not work in the camera activity we can hide the Full screen Button .
If it is bug , I would like to fix it.
Kindly share your thought on this.
https://user-images.githubusercontent.com/34246727/223625993-0359be82-9f7d-47a8-a333-6bd3bb3fb6d7.mp4

Which OS? Which browser?

Can't reproduce on my side.
What's this logo?

Do you use a specific camera?
Are you sure your browser has access to it? Does the video works into another web app?

Hey @llaske  the camera works properly fine ,
the issues lies in the full screen toggle Button .
Screenshot

The  fullscreen Toggle Button Does not work.
Is it supposed to have such behaviour or is it a bug?
if it is meant to not work in the camera activity we can hide the Full screen Toggle  Button .

Oh okay. Not an issue, the fullscreen button is just to have the QR code in fullscreen.


TP2000-884: Add quota edit form for validity period, category

TP2000-884: Add quota edit form for validity period, category

Why

Users need a way to edit quota order numbers
This lays the groundwork for editing certificates and geographical areas that will be done in another PR

What

Adds an edit page and form for order numbers allowing users to change validity period and category

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.01 :tada:

Comparison is base (dfb7fd2) 92.51% compared to head (dc11464) 92.52%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #921      +/-   ##
==========================================
+ Coverage   92.51%   92.52%   +0.01%
==========================================
  Files         414      417       +3
  Lines       28901    29087     +186
  Branches     2200     2209       +9
==========================================
+ Hits        26738    26913     +175
- Misses       1749     1758       +9
- Partials      414      416       +2

Impacted Files
Coverage Δ

quotas/forms.py
100.00% <100.00%> (ø)

quotas/models.py
96.89% <100.00%> (-3.11%)
:arrow_down:

quotas/tests/test_forms.py
100.00% <100.00%> (ø)

quotas/tests/test_views.py
100.00% <100.00%> (ø)

quotas/validators.py
96.00% <100.00%> (+0.34%)
:arrow_up:

quotas/views.py
99.36% <100.00%> (+0.10%)
:arrow_up:

... and 15 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Select a part from building in web map (Cesuim)

Hello,
I'm working on 3D Cadastre, I want to select different parts (lots) from a building and display attribute it.
Can you  help me for this?
I don't want to select whole of a building, Do you have code for selecting different parts from building?
Thank you,
Sabah

No, we don't have open source code for this functionality. It is only supported by the commercial 3D web application developed by the company virtualcitySYSTEMS

Closing this issue. Feel free to re-open if you have further questions.


Does installing tweak void my camera's warranty?

Just like the title:
Does installing tweak void my camera's warranty?
I want to remove the "running on ntsc" on startup, but I dont want my camera's warranty is void.
*Sorry for my bad english

Short answer: Nobody knows.

Just like the title:
Does installing tweak void my camera's warranty?
I want to remove the "running on ntsc" on startup, but I dont want my camera's warranty is void.
*Sorry for my bad english

This tweak does not void your warranty (i think) because at difference of the Magic lantern, openmemories enable functions arleady existing
Srry 4 my bad english but i am italian


Show a rationale when using a default DialogOnDeniedPermission

#57

Just to remember we already reviewed this PR 👍 Thanks for your help and time @Mariovc :)

Thanks for the extra effort @Mariovc


Reapply Reverted "Add NodeTransformer support for nil-literal, global-var-def, optional and if-else"

Purpose
This reverts commit 32d2973b
Reapply reverted PR https://github.com/ballerina-platform/ballerina-lang/pull/22889
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closing in-favor of https://github.com/ballerina-platform/ballerina-lang/pull/23041


Does this not support Vista keys?

I'm going through a saved document from a long time ago where I had kept a number of Windows licenses and a few of the ones I know are Vista and a few I'm uncertain of are coming back as invalid.

The old Version: 1.0.5730 supports XP I have been told so maybe it will - I have not developed for anything since win1 so don't know.


Launch omnibus using the python_executable instead of "python"

Description
Launch Omnibus in the launcher using the python_executable variable instead of "python".
There was one part where this was inconsistent which caused the launcher to fail for me on Windows due to missing dependencies.
Side note: If omnibus launching fails, you can't seem to ctrl+C out of the launcher

Issue N/A
Developer Testing

Here's what I did to test my changes:

Ran the launcher on Windows

Reviewer Testing

Here's what you should do to quickly validate my changes:

Run the launcher, preferably on Windows

This change is

I don't mind waiting, but I think this should literally make no difference on anything other than Windows

It looks good for me on macOS (M1) Chip with Python 3.12.3!

If I have time, I will test on Debian and NixOS to check if it works later today :) (It's not really necessary)


Unable to save png on a headless server

Hello when attempting to save a dataframe as such:
import dataframe_image as dfi
from IPython.display import display, HTML
...

        percent_columns = ['Precision', 'Recall', 'F-1']
        print('INTENT ARGS EVAL:')
        output = pd.DataFrame(exp_intent_args['accuracy'] - prod_intent_args['accuracy'])
        output = output.style.applymap(lambda x: 'color : green' if x>=0 else 'color : red')
        display(output)
        output.export_png('intent_args_diff.png')

I get the following error only on my server:
Traceback (most recent call last):
  File "./utterance_understanding/eval/run_evaluation.py", line 279, in <module>
    main()
  File "./utterance_understanding/eval/run_evaluation.py", line 115, in main
    get_baseline_diff(args.baseline_path, os.path.join(evaluation_output_dir, 'debug'))
  File "./utterance_understanding/eval/run_evaluation.py", line 142, in get_baseline_diff
    get_exp_diff(exp_intent, exp_intent_args, prod_intent, prod_intent_args)
  File "./utterance_understanding/eval/run_evaluation.py", line 124, in get_exp_diff
    output.export_png('intent_args_diff.png')
  File "/miniconda/lib/python3.7/site-packages/dataframe_image/_pandas_accessor.py", line 24, in export
    return _export(obj, filename, fontsize, max_rows, max_cols, table_conversion, chrome_path)
  File "/miniconda/lib/python3.7/site-packages/dataframe_image/_pandas_accessor.py", line 73, in _export
    img_str = converter(html)
  File "/miniconda/lib/python3.7/site-packages/dataframe_image/_screenshot.py", line 167, in run
    img = self.take_screenshot()
  File "/miniconda/lib/python3.7/site-packages/dataframe_image/_screenshot.py", line 119, in take_screenshot
    img = mimage.imread(buffer)
  File "/miniconda/lib/python3.7/site-packages/matplotlib/image.py", line 1490, in imread
    with img_open(fname) as image:
  File "/miniconda/lib/python3.7/site-packages/PIL/ImageFile.py", line 121, in __init__
    self._open()
  File "/miniconda/lib/python3.7/site-packages/PIL/PngImagePlugin.py", line 676, in _open
    raise SyntaxError("not a PNG file")
SyntaxError: not a PNG file```

Did you fix it yet? Have the same problem

Here is the monkey patch to get it to work
import dataframe_image as dfi
from dataframe_image._screenshot import Screenshot

import subprocess
import io
from pathlib import Path
from tempfile import TemporaryDirectory
from matplotlib import image as mimage

def take_screenshot_override (self):
    temp_dir = TemporaryDirectory()
    temp_html = Path(temp_dir.name) / "temp.html"
    temp_img = Path(temp_dir.name) / "temp.png"
    with open(temp_html, "w") as f:
        f.write(self.html)

    with open(temp_img, "wb") as f:
        args = [
            "--enable-logging",
            "--disable-gpu",
            "--headless",
            "--no-sandbox"
            ]

        if self.ss_width and self.ss_height:
            args.append(f"--window-size={self.ss_width},{self.ss_height}")

        args += [
            "--hide-scrollbars",
            f"--screenshot={str(temp_img)}",
            str(temp_html)
            ]

        subprocess.run(executable=self.chrome_path, args=args)

    with open(temp_img, 'rb') as f:
        img_bytes = f.read()

    buffer = io.BytesIO(img_bytes)
    img = mimage.imread(buffer)
    return self.possibly_enlarge(img)

Screenshot.take_screenshot = take_screenshot_override

This argument needed to be added  "--no-sandbox"

which file exactly?

this is a monkey patch, you just need to run this before you call the export function.
it's patching the take_screenshot function of Screenshot
from dataframe_image._screenshot import Screenshot - I am hesitant to commit this change back to the library but if a contributor wants to more deeply understand the no-sandbox flag please feel free

@zer0blockchain can you make this commit to the main repo? Doing a monkey patch is messy and I am trying not to clutter my code a lot

What worked for me a ubuntu container was to install chrome, I had installed chromium before which is a requirement for PIL but that didn't solve the problem, having the official chrome package installed worked.
wget https://dl.google.com/linux/direct/google-chrome-stable_current_amd64.deb
sudo apt install ./google-chrome-stable_current_amd64.deb

There is no official chrome for arm devices. So it wouldnt work on my Raspberry Pi 4

Here's what I did to fix on ubuntu...Already mentioned here but i'll provide more steps.
install latest version of chrome -  https://tecadmin.net/install-google-chrome-in-ubuntu/

Then do there here to your chrome  - vi usr/bin/google-chrome-stable - to get there and edit
You should be able to open chrome now on ubuntu...
Now it ran without an issue  - dfi.export(df, "df_styled.png")

@zer0blockchain can you please make this commit to the main repo? Doing a monkey patch is messy and I am trying not to clutter my code a lot

@zer0blockchain , Is there a simpler approach to get dfi.export(pandas_dataframe, table_conversion='chrome') working on ubuntu/docker?
(This is the last piece that I need for the app I'm building...)

Running as root without --no-sandbox is not supported.
So, try to add this argument to function which calling chrome.

Hi,
Facing the same problem when running the program using the ubuntu systemd service
can you help me out with the issue?

Looks like @zer0blockchain's patch has been added to master:
https://github.com/dexplo/dataframe_image/blob/ff1fa4853c480cda1021229e009cdd6e0740c915/dataframe_image/_screenshot.py#L110-L130
Works for me on headless Ubuntu 22.04LTS

resolved by update dataframe-image-0.1.1 to dataframe-image-0.1.5

@Jbiloki @PaleNeutron
This issue can be closed, right? --no-sandbox has already been merged and deployed.

I do not know sure why but --no-sandbox has been commented out https://github.com/dexplo/dataframe_image/commit/40f0d8f18e4ce3e6012682ff8120b566aeca059c. I am facing the same issue with dataframe-image==0.1.11.

@muupan See https://github.com/dexplo/dataframe_image/blob/f4787e1676145f8eb47dd299dc2a0104b64b4058/dataframe_image/_screenshot.py#L130-L131 , --no-sandbox is added only with root. Normal user do not need this and will lead to problem.

@PaleNeutron
I missed that conditional --sandbox, thanks. Now I understand why it has been commented out.
However, I still needed --no-sandbox to make it work as a non-root user when I used the library inside a kubernetes container.  If always adding --no-sandbox is not a good idea, how about adding an option to force --no-sandbox or an additional condition that checks if it is inside a container?

@PaleNeutron Thanks! My workaround for now is to make an executable shell script like google-chrome --no-sandbox "$@" and specify the chrome_path argument of dataframe_image.export to it.


Adding two small datasets with log artifacts from Meterpreter

Log samples from a machine with Meterpreter actively running remote commands on it.  Also, per @patel-bhavin suggestion, creating a new directory for attack tools since meterpreter is a suite of commands to achieve various objectives (discovery, delivery, etc.).

Per Jose's suggestion, found an appropriate technique.  Also able to download logs in Windows log format


Added option for opt-out of TFS merge replicating

Sometimes TFS guys do merge changesets apparently at random, while in git merge-commit has meaning that all commits in the parent branches are merged. Thus preserving all of the merges sometimes do more harm than good. New setting is tfs.replicateTfsMerges and could be "0" or "1". This setting is turned on by default (i.e. if there's no such value in the git config file - it treats as "1") and is global (not tied to particular tfs remote/branch - mostly because on the per-tfs-remote basis it can do much confusion if configured differently in different places.
Also init command now has -m+ or --replicateTfsMerges+ option, that is true by default, but can be opted out like -m- or --replicateTfsMerges- (not sure it is the best syntax, but by the NDesk.Options docs it is the one used for bools).

Cool, this looks good, and it's pretty simple. Could you add the new stuff to the docs, at least for init and clone and config?

Also init command now has -m+ or --replicateTfsMerges+ option, that is true by default, but can be opted out like -m- or --replicateTfsMerges- (not sure it is the best syntax, but by the NDesk.Options docs it is the one used for bools).

I don't like that. Maybe --no-replicate-tfs-merges?


RFC: Do we need to create a source zip in a release?

Description of Problem
In a release, several artefacts are created, including a source .zip file, e.g. "DNN_Platform_<IP_ADDRESS>_Source.zip".
Is this still needed? GitHub creates a sources.zip automatically on a release, the repo contains the complete source anyway.
Proposed Solution Option 1
Remove the generation of the source .zip file for every release.
Affected version

[x] 9.3.0
[x] 9.2.2
[x] 9.2.1
[x] 9.2
[x] 9.1.1
[ ] 9.1
[ ] 9.0

@ohine Can you correct me if I'm wrong, but the generated source zip includes more than what is just in a GitHub download correct?

The source package is an installable website package, not just a zip of source.  So long as it still works and isn't causing problems, I think it's a helpful learning and debugging tool

I appreciate a "finished" source product which freezes a release. The automated GH download does not include external source packages, nor does it "know" how our project runs or is organized. If we can preserve it, I'd love to see it carried forward.

The source zip does contain the external extensions which are part of the default Dnn Platform distribution, and as @bdukes mentioned also an installable website package. We can easily adjust what's in the zip, maybe include some basic documentation as well to help people understand what's what. Happy to adjust it if necessary.

I agree we should keep it, at least for now.

Given that we have build processes that support this creation, and it archives other elements as well closing this for now as it doesn't introduce a burden.
A suggestion to the documentation team has been made as well to improve the documentation


Allow .tippecanoe-version to specify entire source URL

We need to apply some of our own hacks to prepare tippecanoe for compilation in Heroku's build environment. (extra flags and such)
This change allows the entire source to be specified in the .tippecanoe-version file in a repository.

no longer needed, found a smaller and better workaround


x/playground: 1.19 is missing

Currently the playground lists "Go 1.20", "Go 1.21", and "Go dev branch". 1.19 is missing, and 1.21 was not released yet.

Thanks for reporting.
It might be because CL 467955 (CC @petergeorgas) removed the -prev flag, which is still used, and https://goprevplay.golang.org/version ends up deploying tip rather than 1.19.7.
CC @findleyr.

Fix has been deployed.


🚧 Dev ➾ Provide a means of overriding all docker images

🔗 Epic|Feature Link
No response
🚥 Definition of Done

[x] 🎯 Planning and Design
[ ] 🏎 Dev Implementation Complete
[ ] ♾ PR Merged
[ ] 🏆 Product Acceptance

✅ To Do

[ ] Determine if we need to display any message when an envVarName is being used to provide an image. If we do, does it need to be dismissible?
[ ] Determine whether each plugin that uses a docker image should provide a "show-image-name" task.

⚖ Acceptance Criteria
✨ Functional:

[ ] Add a function to the Taqueria SDK like, getDockerimage(imageName: string, envVarName: string)
[ ] Adjust flextesa plugin to use this method
[ ] Adjust ligo plugin to use this method
[ ] Adjust archetype plugin to use this method
[ ] While we're editing relevant code, we may as well adjust any image names to use the latest version tag of the docker image.

🔭 Scope
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Is this something you can take on next sprint @mweichert ?

Should I take on this?

I think you are the best person to do this. We'll schedule it for an upcoming sprint. As discussed, this won't work for Flextesa so it's just the compilers
It also implicitly solves the Ligo ticket for pinning a particular version of Ligo #ligo #1343


[FLINK-1392] Add Kryo serializer for Protobuf

I've checked the added dependencies and its not overriding any versions and no transitive dependencies are added.

Per the discussion on the dev mailing list (http://mail-archives.apache.org/mod_mbox/flink-dev/201501.mbox/browser), should this be registered by the code that analyzes the generic type and sets up the serializer for the encountered (nested) types? [FLINK-1417]

I will make this change part of a new pull request for https://issues.apache.org/jira/browse/FLINK-1417.


SerializeEverything

Is it possible when using foldEverything to add an option to serialize all fields in the group?
Saves having to add the  [SerializeField] attribute to every line.
Thanks!

Hi there! I didn't test that out and currently I'm super busy with finishing our game Until We Die. So I barely can support anything :)
Hope for your understanding

Hey, no probs at all. Good luck with your game, I've wishlisted it for you looking forward to playing it!
If you have any ideas please let me know. Can we set this as a feature request for now?

Sure!


Clarify the cost per GB in the documentation

The description 'fck-nat per GB: $0.00' is misleading in the context of the final AWS billing, as there is still a charge for regional traffic. It should be clarified that the $0.045/GB NAT charge is reduced to $0.01/GB for internal traffic, which provides substantial savings, though not a complete 100% reduction.
While, technically, the processing fee is 0.00$/GB, the Data Transfer costs are 0.01$/GB

The docs are specifically referring to per GB processing charges for the NAT appliance. There can be egress, but NAT isn't always to egress. AWS data transfer charges are a mangled web and I don't think it's sensible to try and describe the diagram below in detail.
Case in point, the charge structure you describe in the issue is also incomplete. Data egress to the internet is $0.09 per GB for the first 10TB, but you're describing cross region traffic which is a much less common use case for NATs.
I feel pretty strongly that the current wording is sufficient and it is not the responsibility of the fck-nat docs to give users a crash course on the complete data transfer pricing structure. NAT Gateways charge per GB, fck-nat does not, it's that simple.

Appreciate your diagram.
I'm not describing the cross-region or even the cross-AZ. We see that traffic in the same region and AZ that was passing through NAT Gateway before is evaluated with fck nat as internal traffic inside the VPC and charged for 0.01/GB in Virginia.
However, while writing this, I realized that we have the ElasticIP attached to the EC2, and this operation has the same billing rate as cross-AZ traffic. ($0.010 per GB in Virginia - regional data transfer - in/out/between EC2 AZs or using elastic IPs or ELB)
Thank you for being so helpful! I appreciate it very much.


Added a MatrixFTCoronagraph subclass of OpticalSystem

This enables more efficient WFIRST SPLC propagation, and could benefit other coronagraph designs when the semi-analytical / Babinet superposition approach is not valid.

Nice. This makes sense as an addition and the code looks good.
Could you put together some example code for using this? It would be good to have a test case that has some simple coronagraph for which this is suitable, and which shows that you get the same result (or close enough) by using the regular brute force FFT approach and by using MatrixFTCoronagraph.

Good idea - would definitely make sense to provide a simple example of this. The issue is, I don't have a ready-made SPLC design constructed with built-in (analytic) optic functions, as opposed to linking to some mask files. I don't mind sharing some masks publicly, but then the demo is no long self-contained within the code package. It would need to be a ring-type SPLC design, since those are defined based on a set of concentric annuli, which the optics.py functions already can make.

Does it have to be an SPLC? Couldn't we use something really basic like a simple classical Lyot coronagraph with a finite field stop at the intermediate image plane?

True, much easier to implement and it would still be a solid demo.

Related to the demo, this should probably be accompanied by a test case. (POPPY uses py.test, see the "test_*.py" files in poppy/tests/.) It doesn't have to be rigorous like doing an analytic solution and cross checking or anything, just some test of agreement with existing code so we get alerted if the two diverge (and so it is regularly exercised by our CI process)

Hi Neil,
FYI we are likely to be making a new release version of poppy within the next two weeks (no later than end of November). I think there is no rush on this MatrixFTCoronagraph class so I am inclined to leave it until a future release - but if you want to put together a test case and example code in the next week or so, we could include this in the upcoming release.

I added a demo notebook for a simple test case:
https://github.com/neilzim/poppy/blob/mftcoron/notebooks/MatrixFTCoronagraph_demo.ipynb

Hi @neilzim,
I wanted to get back to this PR before it got too stale. I'm happy to merge this but I do want to make sure we have at least one automated test case that exercises this code.  Your example notebook has all the ingredients for this. Could you work with @josePhoenix to make a unit test (in poppy/tests/test_core.py)? After that I think we will be ready to merge this in to master.

I believe this is in 7704168, so all that's necessary is moving it into test_core. (And, of course, verifying that it passes for all of us!)

bah, how did I miss that? My bad.  I'm fine with leaving it in test_matrixdftcoronagraph. Never mind then, looks like this is good to merge. :-)

test passes for me!
I'll probably tweak it a little to make the oversampling factor adjustable (and the tolerance threshold along with it), since right now this adds about 40% to the total test suite run time for poppy. But that's a minor thing and I can do it after this is merged. @josePhoenix do you think you can verify that the test also passes for you today, and then I'll go ahead and merge?

Sorry I missed the earlier part of this convo. Glad the test script checks out for you.

Test case passes for me!


enable use of markdown

Addressing #3
Enabled the use of Markdown in the jinja2 templates of the portal.
The result can be viewed on the dev site: http://lb.desc-web-dev.production.svc.spin.nersc.org/
How this works:

All the markdown files from https://github.com/LSSTDESC/DC2-Public-Release/tree/main/docs have been copied to this repo under web/portal/templates/doc.  Flask has some expectations about the location of files and for now, this was the easiest place to put these. Anyone who desires to update the content can edit the markdown directly.
There are similar Flask expectations for the location of images.  The directions for downloading data using Globus includes images which are stored with the rest of the static image in web/portal/static/img.  These are referenced in the corresponding download.md file and are successfully rendered.
There are new templates for every markdown page located in the web/portal/templates directory, with the exception of getting_started.md and resources.md.  Those templates include the corresponding markdown file via a block that looks like:

   {% markdown %}
   {%include "doc/install_gcr.md"%}
   {% endmarkdown %}

The getting_started.md and resources.md files are used to populate those sections on the home page via the web/portal/templates/home.jinja2 template.
The routes are then defined in web/portal/views.py.

Note that so far bullet points are not rendered using jinja_markdown, not sure why, but is not a showstopper.
The plan is to put this up for a quick review and then merge to main and push to production.

Yes, that sounds good.  Let me try it and see what happens

Yes, that sounds good.  Let me try it and see what happens

Is there a way to automatically generate the per-doc-page templates?

Is there a way to automatically generate the per-doc-page templates?

Yes, that's exactly when Yao suggested.  Now implemented and is working.

Yes, that's exactly when Yao suggested.  Now implemented and is working.


Additional notes not converted to Pulumi docs

As a workaround/fix for #328, the schema object needs to be fully qualified and each part in the string value needs to be quoted. This is documented in the import section of the corresponding TF resources:

https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/0a77383ff51a13be40db72f4d0462fb92d0a4fbd/docs/resources/grant_privileges_to_database_role.md?plain=1#L247
https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/0a77383ff51a13be40db72f4d0462fb92d0a4fbd/docs/resources/grant_ownership.md?plain=1#L298
https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/0a77383ff51a13be40db72f4d0462fb92d0a4fbd/docs/resources/grant_privileges_to_account_role.md?plain=1#L342

But the Pulumi docs doesn't show this note:

https://www.pulumi.com/registry/packages/snowflake/api-docs/grantprivilegestodatabaserole/
https://www.pulumi.com/registry/packages/snowflake/api-docs/grantownership/
https://www.pulumi.com/registry/packages/snowflake/api-docs/grantprivilegestoaccountrole/

The quoting is also explicitely mentioned as the solution in the upstream issue: https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/1999#issuecomment-1802459148
Please make sure this note is properly converted into the Pulumi docs.

Thank you for filing this, Ringo - we'll take a look here at what options we have to preserve this information during docs transformations.


Select right arguments to test functions

Currently, a function like hasEdge is tested with arguments from take 2 edgesNotInGraph. But this take the two first edges not in the graphs, likely (0,0) and (0,1). This is not a good thing, we want to test for edges everywhere and the graph.
How can we select them to have a representative population, and how many ?

I think you shouldn't automate this. A benchmark for hasEdge should include separate lists of existing and non-existing edges, that should be specified manually and purposefully.

Ok, I agree with you. About a separate list, I think it can be generated from a pattern per graph kind, using the inputs field from the Suite data. It will certainly require that inputs become of type
GenericGraph -> [(Name,i)]

instead of
Edges -> [(Name,i)]

On the same topic, is benchmarking hasEdge over one edge we find representative is a good idea ? Maybe it can be good to make an average from a little list a of benchmark concerning similar edges. It will allow to answer questions like "How many time will take searching for an edge I know it can only be in beginning of this path" ?

@nobrakal If you look at https://github.com/snowleopard/alga/issues/14#issuecomment-370242263, you'll see that I originally suggested to make vertex/edge access patterns part of the graph descriptions, not algorithms.

I think inputs are a good thing, because users can write something like

Ouch!
Doing a quadratic amount of work (number of benchmark graphs times number of algorithms) will not scale in many ways: it is hard to write, it is hard to explain, it is hard to understand. I don't think this is a viable option. I advise to think about how you can decompose benchmarking into orthogonal concerns.

It looks orthogonal to me... An Algorithm is needing a graph and arguments to produce something, so we need to map each algorithm to each graph to each arguments to bench all possible cases. Arguments can be produced from the graph, and thus are not orthogonal with graphs, so my idea is to provide this relation between graphs and arguments.
I can't see how you can do less than

number of benchmark graphs times number of algorithms

But I have certainly missed something if you are thinking this is not even viable ^^ !


How to stop/terminate a workflow via HTTP?

Summary
Hello guys -
I am trying to use the k8s HTTP API to stop an Argo workflow.
Our project began prior to the existence of Argo server and so I'd rather just port the semantics of "Argo, stop this workflow" into a "Kubernetes, patch this workflow resource so as to terminate/stop it". Hopefully this distinction I'm making makes sense to you.
I'm struggling to figure out how to issue an "Argo, stop that workflow" command as a k8s patch HTTP request - here are some of the parameters I've tried so far:
➜  concierge-backend git:(matt-brian-argo-render) ✗ KUBECONFIG=./kubeconfig k patch workflow render-fgjxd -p '{"shutdown": "Terminate"}'

Error from server (UnsupportedMediaType): the body of the request was in an unknown format - accepted media types include: application/json-patch+json, application/merge-patch+json, application/apply-patch+yaml
➜  concierge-backend git:(matt-brian-argo-render) ✗ KUBECONFIG=./kubeconfig k patch workflow render-fgjxd --type='json' -p '{"shutdown": "Terminate"}'

Error from server (BadRequest): json: cannot unmarshal object into Go value of type jsonpatch.Patch
➜  concierge-backend git:(matt-brian-argo-render) ✗ KUBECONFIG=./kubeconfig k patch workflow render-fgjxd --type='json' -p '{"spec": {"shutdown": "Terminate"}}'

Error from server (BadRequest): json: cannot unmarshal object into Go value of type jsonpatch.Patch
➜  concierge-backend git:(matt-brian-argo-render) ✗ KUBECONFIG=./kubeconfig k patch workflow render-fgjxd --type='json' -p '[{"spec": {"shutdown": "Terminate"}}]'

The request is invalid
➜  concierge-backend git:(matt-brian-argo-render) ✗ KUBECONFIG=./kubeconfig k patch workflow render-fgjxd --type='json' -p '[{"spec": {"shutdown": "Terminate"}}]'

The request is invalid
➜  concierge-backend git:(matt-brian-argo-render) ✗ KUBECONFIG=./kubeconfig k patch workflow render-fgjxd --type='json' -p '[{"shutdown": "Terminate"}]'

The request is invalid

However as you can see, the requests are invalid.
Can you offer any suggestions on how I might go about reverse engineering the appropriate patch payload to shutdown/stop the desired workflow? Thank you kindly, I am very stuck so very much appreciate your comments!

I found the Argo "terminate a workflow" patch definition here: https://github.com/argoproj/argo/blob/591f649a306edf826b667d0069ee04cb345dcd26/workflow/util/util.go#L722-L726

This worked for me:
$ kubectl patch wf steps-4bnx4 --type merge --patch '{"spec": {"shutdown": "Terminate"}}'
workflow.argoproj.io/steps-4bnx4 patched

Wow, you are the MAN @simster7 - thank you!

Glad to help


pyJac has high accracy, why?

It can be found that pyJac has high accuracy compared with Standard in OpenFOAM. But why? Is the accumulated roundoff error ?

Hi, accuracy has many levels. If you refer to the elementwise accuracy of the Jacobian matrix, pyjac provides an analytical expression for each element so it results in high accuracy. In the past openfoam did not use detailed analytical solution in each element which caused some deviation. In the latest OF-Foundation version, this has been revised though.
If you refer to some other aspect of accuracy, like overall solution of kinetics, then it's not only pyjac which plays a role but how you compute your RHS of the equations and which ODE integrator you use for the solution.

Thanks for your explaination. I refer to the elementwise accuracy of the Jacobian matrix. It is well konwn that pyJac can provide accurate expressions of omega and Jacobian for a given mechanism. However, in OpenFOAM after v6, the fully analytic Jacobian is also provided.  And there is a significant gap between OF6+Standard and CHEMKIN, while a small gap between OF6+pyJac and CHEMKIN is found although using the same ODE solver and tolerances.
Is the high accuracy of pyJac can be attributed to the accurate expressions leading to low roundoff error?

I can't really comment on why the built-in chemistry model in OF6 differ from Chemkin/pyJac - I haven't seen any sort of verification of that.
I would expect pyJac and Chemkin to agree closely, with some small numerical differences due to slight differences in the implementations - though the commercial version of Chemkin is closed source and therefore cannot really be checked. I would not treat Chemkin as the definitive version :)


Update metrics to version 0.17

Motivation

This is part of the update to use Tokio version 1 (#2200), and must be merged together with the other PRs.
The version of the metrics crate used by Zebra doesn't support the new version of Tokio. More specifically, metrics depends on Hyper 0.14, which has the same issue. Just like in #2936, either a new version of the runtime must run beside the old version of the runtime, or the dependency has to be updated.
Solution

Update metrics to 0.17.0. This also requires the metrics-exporter-prometheus to be updated to 0.6.1.
Review

Anyone from the team can review this, but it shouldn't be merged yet.
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work

This PR is missing metrics updates in some Zebra crates. This is a merge blocker.

Oops, I had fixed this in my full-update-to-tokio-1.12.0 branch but forgot to cherry-pick into this PR :(

Merged through #2994.

Once this is merged, we will be able to delete https://github.com/ZcashFoundation/metrics 🎉

I archived the ZcashFoundation/metrics repository, so that builds of older Zebra releases would continue working.
If we need to use it again, we should rename the main branch, so we keep the commits that older Zebra releases depend on.


Add private secret manager example and test

Description
A private only secrets manager example needs to be created and used in a test in the pr_test.go.
New or affected modules

By submitting this issue, you agree to follow our Code of Conduct

@kierramarie Since there are already 2 examples in this repo, maybe convert one of them to a private one


xmlrpclib.ProtocolError: <ProtocolError for blog.example.com/xmlrpc.php: 403 Forbidden>

I stumbled upon this problem. According to this post on stackoverflow it's due to the agent not being accepted. So I subclassed the Transport class into this:
class SpecialTransport(Transport):
    user_agent = ('Mozilla/5.0 (Windows NT 6.0) AppleWebKit/537.31 (KHTML, like Gecko) Chrome/26.0.1410.43 Safari/537.31')

However I now get the error described in issue #65, despite having https in the url. So I'm not sure what's happening really.

For HTTPS connections, you should use SafeTransport as your base class. Per the documentation for xmlrpclib this is what Python does by default.

Ok, SafeTransport solved that problem. The 403 forbidden problem is still there though.

Issue solved, apparently it was a password issue.


Change name of our json-package at the right place.

I didnn't realize that the name of our json package is not coming from the nlohmann json source tar, but from our own Bazel file we made ourself. So changing the name is much simpler there than providing a patch :)
Remove the patch from #2120 again and just change the name there.
For the work in #2121 though this means, that it would probably better to use our bazel file (as it uses the single-header version as well as does proper settings for exceptions)

@lromor this is for you to review


Endpoint to expose functionality developed in Python libraries

Develop alpha version of an endpoint that exposes functionality developed in https://github.com/orgs/EOEPCA/projects/4/views/1?pane=issue&itemId=83302214&issue=EOEPCA|roadmap|255 in a unified REST interface.

Designed the first version of the REST API for health check management/scheduling, and implemented a mock version of it in https://github.com/EOEPCA/resource-health/pull/32.

Put the implementation behind a library API and made the rest API call it, among other improvements.

We have a first functioning version of the health check APIs (including towards K8s), but not deployed on the development cluster yet. Basic version of the telemetry access in the same state.

Containerized this

A functioning health check API has been deployed on the development cluster.

API endpoints have been developed as part of:

https://github.com/EOEPCA/resource-health/tree/main/check_manager
https://github.com/eoepca/python-opentelemetry-access


Examples panic on macOS 10.14.6, fuse 3.10.4

I have a panic with the two examples. I have installed fuse first with brew (with a reboot), then with the package provided on the website, but the behavior is the same:
Am I missing anything? I was following the readme at the project's root.
$ RUST_BACKTRACE=1 cargo run --example hello
    Finished dev [unoptimized + debuginfo] target(s) in 0.03s
     Running `target/debug/examples/hello`
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', examples/hello.rs:94:22
stack backtrace:
   0: backtrace::backtrace::libunwind::trace
             at /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.40/src/backtrace/libunwind.rs:88
   1: backtrace::backtrace::trace_unsynchronized
             at /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.40/src/backtrace/mod.rs:66
   2: std::sys_common::backtrace::_print_fmt
             at src/libstd/sys_common/backtrace.rs:77
   3: <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt
             at src/libstd/sys_common/backtrace.rs:59
   4: core::fmt::write
             at src/libcore/fmt/mod.rs:1052
   5: std::io::Write::write_fmt
             at src/libstd/io/mod.rs:1426
   6: std::sys_common::backtrace::_print
             at src/libstd/sys_common/backtrace.rs:62
   7: std::sys_common::backtrace::print
             at src/libstd/sys_common/backtrace.rs:49
   8: std::panicking::default_hook::{{closure}}
             at src/libstd/panicking.rs:204
   9: std::panicking::default_hook
             at src/libstd/panicking.rs:224
  10: std::panicking::rust_panic_with_hook
             at src/libstd/panicking.rs:472
  11: rust_begin_unwind
             at src/libstd/panicking.rs:380
  12: core::panicking::panic_fmt
             at src/libcore/panicking.rs:85
  13: core::panicking::panic
             at src/libcore/panicking.rs:52
  14: core::option::Option<T>::unwrap
             at /rustc/b8cedc00407a4c56a3bda1ed605c6fc166655447/src/libcore/macros/mod.rs:10
  15: hello::main
             at examples/hello.rs:94
  16: std::rt::lang_start::{{closure}}
             at /rustc/b8cedc00407a4c56a3bda1ed605c6fc166655447/src/libstd/rt.rs:67
  17: std::rt::lang_start_internal::{{closure}}
             at src/libstd/rt.rs:52
  18: std::panicking::try::do_call
             at src/libstd/panicking.rs:305
  19: __rust_maybe_catch_panic
             at src/libpanic_unwind/lib.rs:86
  20: std::panicking::try
             at src/libstd/panicking.rs:281
  21: std::panic::catch_unwind
             at src/libstd/panic.rs:394
  22: std::rt::lang_start_internal
             at src/libstd/rt.rs:51
  23: std::rt::lang_start
             at /rustc/b8cedc00407a4c56a3bda1ed605c6fc166655447/src/libstd/rt.rs:67
  24: hello::main

Ah, reading the source, it seems it requires a mount path. I assumed it would take . by default. That means it works. Maybe the example can be either more descriptive (print a better error message) or automatic (if nothing is provided, mount .)


Rename Limited trait to Clamp

Rename is_valid to is_within_bounds
Update the assert_ranges macro to use clamped instead of limited
Update relevant documentation and examples
Closes #209

Oh, excellent! I wasn't expecting such a quick response, but it's a most welcome one! And thanks for going the extra mile with the assert_ranges macro.  This looks good, so let's merge!
bors r+


Extend context-type type handling

Based on project feedback, I removed the differentiation between context and find. The project now has to decide which of the context-types can be a top-level context-type and which context-types are allowed as sub-contexts of a particular context-type in the hierarchy.
You have to run migrations.
Please review @eScienceCenter/spacialists
TODO

[x] Drag & Drop
[x] After adding a new context-type, the form is empty (reload required)
[x] Add (De)Select-All buttons in DME

Should be fixed now


Revert engine back to 06448ee8165f4ff6579ef9510ac69322eda896ac

https://github.com/flutter/engine/pull/50737 broke the tree

CC @flar @zanderso @rmistry
FYI: The auto-roller is stopped so the change doesn't roll back in.

@flar Looks like you're the engine sheriff. It's not clear to me when we can turn the autoroller back on? The next roll needs to pick up https://github.com/flutter/engine/commit/0f3ad23b84ed4a1f6631dd5cdfde4f6acb85a007 so the framework tree doesn't break again.


Fix broken tests

See https://github.com/elgentos/magento2-cypress-testing-suite/actions/runs/3053615539/jobs/4924464391

[x] Can visit the category page and filters on color red
[x] Can find a single product
[x] Can see suggestions when entering search terms

1 broken test remaining! https://github.com/elgentos/magento2-cypress-testing-suite/issues/86


Add support for developing on Replit

Hey! I wanted to see if we could get this repo working on Replit.
I added a couple config files to properly setup LSP and provide go and just in the environment and I wired up the Run button to run the full test suite. You can try this out by forking my repl here: https://replit.com/@ConnorBrewster/web5-go or, if this is merged, you can import this repo directly into a new repl and start coding right away.

Replit is cool BTW, I've played around with it a couple of times.

does this have the right version of go and can merge?

Not yet, we don't have Go 1.22 available on Replit yet but we should next week


Make revision optional when integrity is set in precache manifest

Library Affected:
workbox-build, v6.4.1
Browser & Platform:
All browsers
Issue or Feature Request Description:
The precache manifest that's injected has entries that look like this:
{"revision":null,"url":"/bundles/v1/myScript.f3e2692d7ee7e59d5def.js"}

If you set revision to undefined instead of null, it would be left out of the JSON entirely and make a smaller bundle.
export declare interface PrecacheEntry {
    integrity?: string;
    url: string;
    revision?: string | null;
}

Might be better as
revision?: string | undefined;

At some point this might change, but for the time being, we want an explicit null there as a signal that the revision field has been deliberately set, rather than someone manually creating a precache manifest entry without taking verisoning into account at all.
(This is obviously not foolproof, but it's part of a best-effort to prevent developers from mistakenly, e.g., adding {url: '/index.html'} to their precache manifest and then getting very confused when they make changes to index.html, but caches are never updated.)
In Workbox v7, I anticipate we're going to start generating integrity properties (for subresource integrity) via the build tools. If we see that integrity is set, a missing revision would probably be fine.

Makes sense, thanks for the explanation.

Hi there,
Workbox is moving to a new engineering team within Google. As part of this move, we're declaring a partial bug bankruptcy to allow the new team to start fresh. We realize this isn't optimal, but realistically, this is the only way we see it working. For transparency, here're the criteria we applied:

For PRs, we closed everything that doesn't target the current v7 branch.
For Issues, we closed everything that was created before 2024.

Thanks, and we hope for your understanding!
The Workbox team


Revert "feat(impl):[TRI-677] new module for esr microservice"

Reverts catenax-ng/product-item-relationship-service#304

This is not supposed to go into main, but on a separate branch instead.


Flag of europe is missing

https://en.wikipedia.org/wiki/Flag_of_Europe
Flag of europe is missing in your list.
Can you please make it available.
Thanks.

That flag is already present under the names European-Union.png and EU.png


Updated DataDog dashboard to use updated App.Metrics counter types

Works with yet-unreleased App.Metrics 4.2.0-preview* - there should be a new release soon which contains https://github.com/AppMetrics/AppMetrics/pull/669, which is required for these dashboards to render properly in Datadog.


(Ecto.ConstraintError) constraint error when attempting to insert model: * unique: id

I've been reading Platformatec's guide on adding embedded models. However, I am getting a constraint error when I try and insert the new embeds via changesets using Mongo.Ecto.
This is what I followed in the guide:
defmodule MyApp.Permalink do
  use Ecto.Model

  embedded_schema do
    field :url
    timestamps
  end
end

defmodule MyApp.Post do
  use Ecto.Model

  schema "posts" do
    field :title
    field :body
    has_many :comments, MyApp.Comment
    embeds_many :permalinks, MyApp.Permalink
    timestamps
  end
end

# Generate a changeset for the post
changeset = Ecto.Changeset.change(post)

# Let's track the new permalinks
changeset = Ecto.Changeset.put_change(changeset, :permalinks,
  [%Permalink{url: "example.com/thebest"},
   %Permalink{url: "another.com/mostaccessed"}]
)

# Now let's insert the post with permalinks at once!
post = Repo.insert!(changeset)

I get the following error when I post = Repo.insert!(changeset):
iex(8)> post = MyApp.Repo.insert!(changeset)
[debug] INSERT coll="posts" document=[body: "This is the body", _id: #BSON.ObjectId<1dfafe19149351241a15b1b1>, inserted_at: #BSON.DateTime<2015-12-09T14:58:33Z>, permalinks: [[_id: #BSON.ObjectId<1dfb04191493512880ee927d>, inserted_at: #BSON.DateTime<2015-12-09T15:24:09Z>, updated_at: #BSON.DateTime<2015-12-09T15:24:09Z>, url: "example.com/thebest"], [_id: #BSON.ObjectId<1dfb04191493512880ee927e>, inserted_at: #BSON.DateTime<2015-12-09T15:24:09Z>, updated_at: #BSON.DateTime<2015-12-09T15:24:09Z>, url: "another.com/mostaccessed"]], title: "Test Post", updated_at: #BSON.DateTime<2015-12-09T14:58:33Z>] [] ERROR query=0.7ms
** (Ecto.ConstraintError) constraint error when attempting to insert model:

    * unique: _id_

If you would like to convert this constraint into an error, please
call unique_constraint/3 in your changeset and define the proper
constraint name. The changeset has not defined any constraint.

      (ecto) lib/ecto/repo/model.ex:274: anonymous fn/5 in Ecto.Repo.Model.constraints_to_errors/3
    (elixir) lib/enum.ex:1387: Enum."-reduce/3-lists^foldl/2-0-"/3
      (ecto) lib/ecto/repo/model.ex:91: anonymous fn/10 in Ecto.Repo.Model.do_insert/4
      (ecto) lib/ecto/repo/model.ex:14: Ecto.Repo.Model.insert!/4

Was unsure whether this was a bug, or something I have done wrong?
Thanks!

I can't really reproduce it. I've created an example project with embedded models here: https://github.com/michalmuskala/mongodb_ecto/blob/embedded_example/examples/embedded_models/lib/embedded_models.ex
Is there something different in your setup? Do you have indexes on your database?

Thanks Michał. Your version works perfectly.
I'm getting the error I previously mentioned when starting a new Phoenix app with mongodb defined. I'll retrace my steps and will follow up in detail.

@sevfurneaux Did you manage to resolve your problem?

@sevfurneaux , I am getting the same error even for top-level (non embedded) models. Even after using unique_constraint(:fieldname) in the changeset.
I created a new app with phoenix.new .... --mongodb and created the unique index on the table using mongo shell.

it's by Ecto design. You cannot use such constraints with embeds.

Since there was no activity here for a long time, I'm going to close this. If the issue is still relevant feel free to reopen or ping.


Chopped logs don't drop items

Chopped logs don't give item drops, excluding the chopped log that the player last hit. For example, the tree in the screenshot is 6 logs tall, but only drops 3 logs when cut at the bottom.

Minecraft: 1.18.1
Forge: 39.0.59
Mods: TreeChop-1.18.1-0.15.5

Thanks for reporting, I'll get it fixed

Fixed in 0.15.6


Xcode 13 Support

Waiting until Xcode 13 is released for final testing and merging.

Any Updates on this one?

Moving this to a new PR: https://github.com/GEOSwift/GEOSwift/pull/232


Genre filter

Turn off the radio button for genres and it'll submit a bad request

Fixed by commit f7a403b


Deploying precompiles on zksync

Hello,
We are currently making a use of zkSNARK Pairing library and would like to use the precompiles prior of deployment to zksync.
https://gist.github.com/chriseth/f9be9d9391efc5beb9704255a8e2989d
@ilitteri Would it be possible to deploy them now?
Thanks.

Hi! Every precompile that is in main but ModExp is working correctly as precompiles, if you want to deploy them as independent contract you need to add a return(0, 0) to their constructor (we're pushing a PR with this fix today though).

There should not be any troubles, let me know if there's any though. We have an example of deploying one in testnet in #203

@ilitteri The static calls and outputs would work exactly as the precompiled ones right?

Yep, again, let me know if they don't. One thing it take into account (I don't think this would be your case though), if you use our era-test-node, be aware that the output is modified in our favor (the gas used is at the starting of the output).

@ilitteri Got this strange error while deploying EcPairing.yul, I have added return(0, 0) to the constructor like you told me.
  code: 'CALL_EXCEPTION',
  action: 'estimateGas',
  data: null,
  reason: null,
  transaction: {
    to: '0x0000000000000000000000000000000000008006',
    data: '0x9c4d535b00000000000000000000000000000000000000000000000000000000000000000100309f6271bc3cceacd8564e1cd00d5ac78a9294d283b886813bdf14dd200500000000000000000000000000000000000000000000000000000000000000600000000000000000000000000000000000000000000000000000000000000000',
    from: '0xfDbA3B7Db2253cD099A479d11230057389cCBa0D'
  },
  invocation: null,
  revert: null,
  shortMessage: 'missing revert data',
  info: {
    error: {
      code: 3,
      message: 'exceeds limit for published pubdata',
      data: '0x'
    },
    payload: {
      method: 'eth_estimateGas',
      params: [
        {
          type: '0x71',
          nonce: '0x395',
          from: '0xfdba3b7db2253cd099a479d11230057389ccba0d',
          to: '0x0000000000000000000000000000000000008006',
          data: '0x9c4d535b00000000000000000000000000000000000000000000000000000000000000000100309f6271bc3cceacd8564e1cd00d5ac78a9294d283b886813bdf14dd200500000000000000000000000000000000000000000000000000000000000000600000000000000000000000000000000000000000000000000000000000000000',
          eip712Meta: { gasPerPubdata: '0xc350', factoryDeps: [Array] }
        }
      ],
      id: 82,
      jsonrpc: '2.0'
    }
  }
}

Using default deployment arguments
const {
  ethers,
  zksyncEthers,
} = require('hardhat');
const {
  loadArtifact,
  deployContract,
} = zksyncEthers;

const EcAdd = await deployContract(await loadArtifact('EcAdd'), [], owner);
  await EcAdd.waitForDeployment();

  const EcMul = await deployContract(await loadArtifact('EcMul'), [], owner);
  await EcMul.waitForDeployment();

  const EcPairing = await deployContract(await loadArtifact('EcPairing'), [], owner);
  await EcPairing.waitForDeployment();

  console.log({
    EcAdd: EcAdd.target,
    EcMul: EcMul.target,
    EcPairing: EcPairing.target,
  });

Here is the compiled artifact from zkSolc, I have modified from yours
https://gist.github.com/zerotwodao/d24de3d22ca1c779def4a3c01fdf9f52

@ilitteri Seems like a compiled bytecode size for EcPairing.yul is too big, would it mean it is impossible to deploy?

@ilitteri Seems like a compiled bytecode size for EcPairing.yul is too big.
Finds out the compiled bytecode for EcPairing exceeds 200kb and the known limit for bytecode size limit is 24kb, would it be impossible to deploy?

Hm, that's bad news. I'm not sure if it'd be possible. Are you trying to deploy on testnet or in a local-net?

Something that could work (don't know the amount of work it'd involve) is to modularize the precompile (e.g. finite field arithmetics could be part of another library, also the curve arithmetics could be part of another library).

@ilitteri Tried to deploy on testnet, and it seems like it is impossible unless we lift off the https://eips.ethereum.org/EIPS/eip-170 size limit, or like you said we could try modularizing the precompile.

Do you think it would be possible to request zksync team for the deployment of those precompiles on the different addresses for the testing purpose on mainnet and testnet? It would make our contract work eventually

Have you tried to compile it with different compilation flags?

@ilitteri Which flags are available?

If you run zksolc --help you can find this:
-O, --optimization <optimization> Set the optimization parameter -O[0 | 1 | 2 | 3 | s | z]. Use `3` for best performance and `z` for minimal size

In your case I'd use -Oz flag.

@ilitteri Sorry for late reply I have successfully compiled and deployed the precompiles with ~40kb using the flag -oZ ( I don't know why I have used -O3 before ) . After the deployment I have also checked that the precompiles work with the example verifier contract that I have posted the above. Do you know when it will be live on the mainnet?

And seems like it is gas costy than the one available from ethereum mainnet would the gas cost for calling function can be decreased if they are deployed as a precompile?

@ilitteri Sorry for late reply I have successfully compiled and deployed the precompiles with ~40kb using the flag -oZ ( I don't know why I have used -O3 before ) . After the deployment I have also checked that the precompiles work with the example verifier contract that I have posted the above.

Amazing! Good luck then with your library.

Do you know when it will be live on the mainnet?

I have no answer for this. The zkSync team is reviewing if it'd be better to have them as circuits instead of as Yul contracts. For the moment, using them as contracts in testnet.

Closing the issue as it seems to be solved as said here.

@ilitteri Sorry for late reply I have successfully compiled and deployed the precompiles with ~40kb using the flag -oZ ( I don't know why I have used -O3 before ) . After the deployment I have also checked that the precompiles work with the example verifier contract that I have posted the above. Do you know when it will be live on the mainnet?

Could you please share deployment address in zkSync main or testnet

@chervyachok Sure
EcAdd: '0xbF3F280F177Befe658A6973E0E57c38d62d1bCc8'
EcMul: '0x8277B56C128e62356306C5a702E1fC2c3dB7192F'
EcPairing: '0x49a31d9776B80450d7E5D514D9a753C99D800E9d'

@chervyachok Have you tested with the ethereum testnets for your code? With my case with the gist posted above it works as intended


[WIP] Finalizing Gossip Service

This PR finalizes the Gossip service for replicas.

Codecov Report

Merging #62 (755e26c) into v1.1 (c639b08) will not change coverage.
The diff coverage is 66.66%.

:exclamation: Current head 755e26c differs from pull request most recent head 5bcf2cd. Consider uploading reports for the commit 5bcf2cd to get more accurate results

@@           Coverage Diff           @@
##             v1.1      #62   +/-   ##
=======================================
  Coverage   20.86%   20.86%
=======================================
  Files          26       26
  Lines        2856     2856
=======================================
  Hits          596      596
  Misses       2124     2124
  Partials      136      136

Impacted Files
Coverage Δ

pkg/gds/store/store.go
24.13% <ø> (ø)

pkg/gds/store/leveldb/leveldb.go
36.02% <66.66%> (ø)

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 c639b08...5bcf2cd. Read the comment docs.


forge script with input parameters, result in application panicked (crashed) error.

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (49f35c7 2022-06-30T00:31:40.009286Z)
What command(s) is the bug in?
forge script
Operating System
macOS (Apple Silicon)
Describe the bug
I have a script setup as:
function run(address a) public {
  //... do something with a
}

And I executed command as
forge script <script-file> --rpc-url <url> --private-key <private_key> --etherscan-api-key <api_key> --boradcast -- <some address, as script arguments>

But got errors:
The application panicked (crashed).
Message:  Function signature not found in the ABI
Location: cli/src/cmd/forge/script/mod.rs:390

This is a bug. Consider reporting it at https://github.com/foundry-rs/foundry

  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
   1: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
   2: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
   3: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
   4: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
   5: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
   6: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
   7: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
   8: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
   9: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
  10: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
  11: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
  12: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
  13: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
  14: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
  15: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
  16: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>
  17: __mh_execute_header<unknown>
      at <unknown source file>:<unknown line>

So now I have to put arguments variable explicitly into script file's attributes in order to execute the script.
Thank you very much for your time and your help.

You have to pass --sig "run(address)" for it to understand how to encode the arguments


autofix for no-unescaped-entities

i'm not 100% sure if this makes sense - it seems like this check is more for finding unintentionally closed jsx tags? it would be super useful though to make the recommended replacements automatically, i'd be happy to do the pr if it's in the spirit of the original rule.

An autofix would absolutely silence the warnings, but the point of this rule is to catch likely bugs - and the likely bug is that the entities are supposed to be in a jsx context. That can't be autofixed; a human needs to determine the fix.

I can see that automatically escaping < and > would be risky (I would never expect that to work!), but perhaps it could be applied for certain other characters? For example ' and " are commonly written out in JSX (though they shouldn't be!) but they don't typically indicate a bug - just someone being too lazy to escape them in my case 😳
render() {
  return (
Benjie's bullish on "eslint --fix"
);
}

I've never written an auto-fixer for ESLint so not sure if partial auto-fixing is okay.

There's discussion around this in https://github.com/yannickcr/eslint-plugin-react/issues/1263#issuecomment-309327876

In the case of straight quotes, those are typographically incorrect - the proper fix there is to use curly quotes if it’s prose. Again, i don’t think that’s something that can be safely programmatically fixed.

Of course it's not always prose; but discussions around correct typography aside, could you give an example of where ' being replaced with ' in the child (text) context would not be safe to fix programatically? Even  handles it correctly
render() {
  return (
echo 'eslint --fix "path/to/file"' >> ./autofix
);
}

vs
render() {
  return (
echo 'eslint --fix "path/to/file"' >> ./autofix
);
}

At the moment the rule seems to relate to entities being unescaped; perhaps there should be a different rule about making typographical recommendations (or maybe the error message for this rule should be change to suggest using typographical quotes to make its intent clearer)?

My main motivation for this to be auto-fixed is due to GitHub's JS code formatting messing up when only one single quote is present, e.g.
render() {
  return (
echo 'eslint --fix
);
}

(Workaround is to use the jsx code formatting (e.g. by renaming your files), or to just correctly escape the quotes.)

for me personally i've never had this rule catch what it's supposed to catch - instead it forced me to learn the ampersand codes for single and double quotes. i suppose it makes more sense for me to disable it since i'm spending more time fixing non-buggy code - i guess this can be closed out then?

@modosc all that's required to fix it is that you take your literal text, and wrap it in explicit jsx curlies, and one of the JS quote styles. If it's catching human-intended prose, then you should be replacing your straight quotes with curly quotes.
This:
render() {
  return (
echo 'eslint --fix "path/to/file"' >> ./autofix
);
}

should instead be, for example:
render() {
  return (
{`echo 'eslint --fix "path/to/file"' >> ./autofix`}
);
}

@benjie this is not intended to catch your example of "code in a pre" - it's intended to catch mistakes like in the rule documentation.

ok, i'm good then. thanks

Can the error message print more info? EG:
- `>` can be replaced with `>`
- `"` can be replaced with `"`, `“` or `”`	- `"` can be replaced with `"`, `“`, `"` or `”`
- `'` can be replaced with `'`, `‘` or `’`	- `'` can be replaced with `'`, `‘`, `'` or `’`
- `}` can be replaced with `}`	- `}` can be replaced with `}

So that I don't have to look it up every time.

@stevemao sure, that sounds like a great improvement - a PR would be most welcome.

@ljharb I just submitted a PR 😄

Did anyone come up w/ simple ways to auto-fix? Neither eslint nor prettier seem to have options to auto-fix non-smart quotes in prose to smart quotes.

There could be a plugin setting which way to replace the escape entity for auto-fix :thinking:
e.g.
"settings": {
    "eslint-plugin-react": {
      "version": "detect",
      "no-unescaped-entities": {
        "\"": """
      }
    }
  }

There could be a plugin setting which way to replace the escape entity for auto-fix :thinking:
e.g.
"settings": {
    "eslint-plugin-react": {
      "version": "detect",
      "no-unescaped-entities": {
        "\"": """
      }
    }
  }

@talentlessguy it is very atypical and rare to have settings exclusively for autofix; any autofix, with or without an option, must be something that's safe - and this one wouldn't be.

@talentlessguy it is very atypical and rare to have settings exclusively for autofix; any autofix, with or without an option, must be something that's safe - and this one wouldn't be.


Modify setup.py so that it can be installed under conda environment

Since cmake is not able to find Python-libraries under anaconda environment,
-DPYTHON_LIBRARY and -DPYTHON_INCLUDE_DIR flags should be appended to cmake command with  respective folders.
I did this by appending them to cmake_extra_arch.

Thanks for the PR.  It looks good.  Is there a reason not to add these paths for all python builds?  It seems like they might help locate the correct python files in other situations as well.  I haven't done any testing though.

To add these paths for all python builds? How to do that?
Actually I'm not familiar with python environment variables. I was just not able to install dlib today and finally found this solution workable. =P

Ah, ok.  It looks like the code might work with other python builds.  But I
need to test it as I'm not familiar with the python environment variables
either.  In any case, thanks for the PR, I'll merge it and figure out if
the trick applies to other python builds later.


Write image digest metadata file.

IIRC the image digest is only set after the image is pushed to a registry

@fxb will you address the test failures?


Option to join new video chat with Jitsi Meet

Is your feature request related to a problem? Please describe.
Right now it seems like the video room URL is constructed from the IRC channel and network name, meaning that the link shared is always to the same video room on Jitsi Meet. This is useful for some cases, but not so useful for others.
Describe the solution you'd like
When trying to share a video chat room, a modal should be shown asking the user if they want to join the default channel room (current behaviour) or create a new video room with a random url and share that with the IRC chat instead.
Additional context
Having this conditional modal would also avoid people accidentally sharing links to video room by mis-clicking, as currently there is no confirmation dialogue at all.

I'm not going to implement that. The reason why it's the same is that you are in the same room and therefor the video button have to join the same meeting. Or else there's no way for Convos to know which room to join.
The "mis-clicking" issue was much greater before, but I haven't really seen it recently after it was moved to the submenu.
If I knew that this issue would affect many users, I would consider doing some changes, but I don't think that is the case.

Or else there's no way for Convos to know which room to join.

But the link to the room is shared in the chat, so anyone can just click on it and join.
Sometimes it is desirable to have multiple parallel chats as break out rooms, so I think having this as an second option would be really nice.


[Bug] in Thala For A Reason

Description
While entering 0 as a first digit it is calculating result which may
or may not be 7.
But it should give an alert Invalid Input.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[X] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Proceed @GarvVirmani


fix strncpy bound error

This patch fixes the following error while using gcc 11
error: 'char* __builtin_strncpy(char*, const char*, long unsigned int)' specified bound depends on the length of the source argument [-Werror=stringop-overflow=]
Signed-off-by: Khem Raj<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@kraj Thanks for this fix.  Sorry we did not get around to reviewing it until now.  You have the sign the CLA before we can accept the change.  Please do that, and I will merge if all of the tests pass.

@kraj Thanks for this fix. Sorry we did not get around to reviewing it until now. You have the sign the CLA before we can accept the change. Please do that, and I will merge if all of the tests pass.

its a one liner, and I don't want to sign CLA for a trivial change like this. You can abandon it.


[EBPF] kmt: fix parent tests of flaky tests marked as failed

What does this PR do?
Fixes test-json-review to stop it from reporting parent tests as failed when a child one is marked as flaky.
Motivation
Describe how to test/QA your changes
Unit tests included.
Possible Drawbacks / Trade-offs
Additional Notes

/merge