Change default domain from OPNsense.local to OPNsense.internal

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Please consider changing the default domain for new installations to OPNsense.internal following the IANA proposition.
https://itp.cdn.icann.org/en/files/root-system/identification-tld-private-use-24-01-2024-en.pdf
I know the decision for the domain is not final by the IANA but maybe you can already do preparations or identify possible problems with this change.

For future reference this should be a single line pull request:
https://github.com/opnsense/core/blob/5220c4a1a684adbf8b6b74431a3a8693f685478a/src/etc/config.xml.sample#L207
But I suggest waiting for this to be actually a standard or at least gain a wider recognition first.
Cheers,
Franco


COCO dataset

Thanks for the nice repository!
Is this available to do transfer learning with coco dataset?

You can try to directly apply the model on the coco dataset or any 2D image tasks, but I am not sure wether it works if you would transfer between semantickitti and coco dataset, because the domains are quite different. However it is still very interesting to transfer between 2D and 3D domain.


[Q] How to force stop runs?

In wandb.ai I clicked stop for some runs but it won't stop even I waited few hours.
It just keeps showing stopping.

is there any way to force stop those runs?
I want to delete it but It won't.

Hey there, we are investigating the issue at the moment. Could you share the link to the run?

Hey @Yoonho-Na, are you still experiencing this?

I haven't stopped any runs after that.
I'll inform you when it happens again.
thank you.

Hi, I run into the same issue, the run path is lucienwang/mvtec/1fgsjbvc.

Hey @lucienwang1009, did you delete the run? I can't find a run with the specified id.

Hi @armanhar, yes I deleted it as it stopped somehow. Thank you.

@armanhar Hi, in wandb, the log state is always running no matter whether the project is finished or not. And the stop running bottom does not work

I am facing the same issue too. I don't want to delete the run but it just dosn't stop.

I run into the same issus: https://wandb.ai/littlecatx/huggingface/runs/mrk8sx6n/overview


Install stone docker image

Summary:
added document for getting stone docker images for issues #24

Thank you for the opportunity ser @dmirgaleev  if there any others you'd need me to help with im here


Spanish translation

Here´s the spanish translation :)
I noticed there are some strings that are not used, they are shown in english even though there´s a translated string available. I. e. "- To add a BitTorrent-Magnet URL, use the add URI option and add it there. " in the Add by torrent modal.

I have fixed the - To add a BitTorrent-Magnet URL, use the add URI option and add it there.  translate bug. Let me know if you find more! @jdarias

Also, this looks good. You did it!! :)

Great!
Indeed i found several of those, i´ll make a list and post them here.

In connection settings modal:
_(Requires an appropriate webserver to be configured.) _
In the about modal:
Or you can open the latest version in browser through:
Under the Manage menu:
Shutdown server
Here are ones in a screenshot:
Imgur
http://imgur.com/KZqTVq5

Thanks @jdarias for compiling the list of errors! I have fixed most of them and they are pushed to master. The only one I did not is Shutdown server as most translation except English translation define this. Have you defined it in the Spanish translation?

No, I haven´t,
I am adding a new line with this in es_ES.js.
    'Shutdown Server': 'Desactivar servidor',


Not all files being uploaded

I'm running into an issue where gatsby-plugin-s3 only uploads some of my files (124 to be precise), leaving the website broken (index.html is missing, etc).
I build my site and trigger gatsby-plugin-s3 on AWS CodePipeline. It was working until recently.
There are no errors in the logs:

Date
Message

2021-04-05T14:22:40.444+10:00
[Container] 2021/04/05 04:22:40 Running command yarn run deploy

2021-04-05T14:22:40.444+10:00
yarn run v1.22.5

2021-04-05T14:22:40.444+10:00
$ gatsby-plugin-s3 deploy

2021-04-05T14:22:42.526+10:00
- Retrieving bucket info...

2021-04-05T14:22:42.526+10:00
✔ Synced.

2021-04-05T14:22:42.526+10:00

2021-04-05T14:22:42.526+10:00
Your website has now been published to:

2021-04-05T14:22:42.526+10:00
xxx-www

2021-04-05T14:22:42.526+10:00

2021-04-05T14:22:42.526+10:00
Done in 1.52s.

I have added ls and find commands to the script, and I can see the files on disk, they're just not being uploaded to S3 for some reason.
e.g.
$ ls -l public
...
-rw-r--r-- 1 root root 5706 Apr 5 04:22 index.html
...

I have tried deleting all files from S3, but only the same 124 files re-appear after re-running the script.
Interestingly, the same script is working just fine for a different deployment environment (different S3 bucket, different AWS account).
Can you suggest how I can debug this?

Any tips on debugging? This is only happening in our staging environment right now, but I'm concerned that if we don't figure it out, it could suddenly start happening in production too.

Hi, sorry for the late response.
I haven't encountered this issue before, and sadly don't have time to look into it myself. But I'll happily help guide you through some troubleshooting.
I would suggest modifying bin.ts to log the path to files here. That should help you find out if klaw is actually seeing the files.
Assuming that klaw is picking up the files, take a look here to see if the files are being detected as unchanged (and therefore skipped).
Best of luck, please let me know if you have any questions.


Fixed #25230, Fixed #30685 -- Optimized distinct().count().

This still needs a battery of test as the distinct().count() coverage is relatively poor in the test suite but this will hopefully address ticket-25230 and ticket-30685.
Missing test coverage

[ ] Expression.deterministic
[ ] distinct().count() with Random() annotation
[ ] distinct().count() with existing aggregation
[ ] distinct(*on).count() coverage

Another case that must be covered here is that COUNT(DISTINCT col) cannot be used if col is nullable.
https://dbfiddle.uk/HBf0068t

I guess I'll eventually come back to this but I'm not actively working on it.


Removing old Test-Solution and Adding PHPUnit Tests + Fix of some little potential bugs

[REMOVED]
Testing with custom solution (Nest)
[ADDED]
EmptyDataException; phpunit; phpunit-watcher; PHPUnit unit tests (incomplete)
[FIXED]
No Exception was Thrown if trying to save empty data -> EmptyDataException;
The shorthand array syntax ([]) for Symmetric array destructuring just provided in PHP >= 7.1

Wrong branch


Screen shifts to the beginning of the buffer in vim

steps:
open vim
anytime you try to append using the "a" command, the whole screen shifts upwards

I only get this in split windows.


Fix syntax error for endCaptures

We'd like to use this grammar to highlight Hack code on GitHub.com (cf. github/linguist#4466). Unfortunately, in its current version, the grammar is rejected by our compiler due to a syntax error. This pull request fixes it.

 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.

cc @PranayAgarwal


Speed up build_conda_pkg

Similar to #2365 and #2404 , build_conda_pkgis now a long pole in the CI tent.  Even though the rest of the CI jobs take ~10-12 minutes to run, this one is 30+ minutes and should either be moved to merges to main or effort put into shortening it.

Unfortunately, we don't have a lot of control over the mechanics of the build other than what unit tests we run during the process.
I think these are the levers we can tweak:

Don't run the full set of unit tests in build_conda_pkg - feel like this defeats the purpose of the check
Run the check on a beefier machine so that we can make use of -j 8 - I think we're trying to avoid self-hosted runners
Like you said, run after merge to main only - I like this. It's what we had before but it does miss the 1% of prs where this check can prevent merging something that will break the conda build.
Run only on prs that change requirements files change, e.g. core-requirements.txt, requirements.txt, test-requirements.txt, minimum/latest deps, etc - I like this because I think the only prs that really have a chance of failing build_conda_pkg are those that update the requirements in any way.

I think we can do both 3 and 4 and vote we proceed with that. What do you think @chukarsten ? Are there any other quick options I'm missing ?

I think #4 is a really clever solution and the most robust.  Do you think that can be done?  Like a GH action dependent on whether one of the reqs.txt files is changed?

Yes, I believe so!


1.4.1 hotfix

Description
Issue 1.4.1 hotfix for issue below.
Issue reference
Please reference the issue this PR will close: #3699
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Do not merge until the problem is figured out. Issue https://github.com/dapr/dapr/issues/3699 remains unsolved.

As per @pkedy's comment, this hotfix is correct:
https://github.com/dapr/dapr/pull/3703#issuecomment-925400915


feat(html): support env replacement

Description

Support %ENV_NAME% syntax in HTML files. Close https://github.com/vitejs/vite/issues/3105
Additional context

I made it a pre-hook for transformIndexHtml as otherwise normal-hook transforms like vite-plugin-html may get a parse error.
It also supports %MODE%, %PROD%, etc, which are a little short, but maybe that's fine.
It doesn't support %SSR% as I don't think it makes sense in HTML files.

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Isn't this a breaking change?

It would break if someone happens to use %something% and has an env named "something". If not %something% would be left as-is. I think it's quite unlikely that this would happen and break things, we also used to merge features like this in the past.

It would break if someone happens to use %something% and has an env named "something". If not %something% would be left as-is.

What do you think if we also parse the remaining %VITE_...% placeholders and emit a warning about this env being missed? I think it will be a common mistake and we could still be safe if we only check for the env prefixed ones.

Additionally, SvelteKit uses %% in a different manner, but it might not matter since they have their own HTML template parsing and HTML generation.

What do you think if we also parse the remaining %VITE_...% placeholders and emit a warning about this env being missed? I think it will be a common mistake and we could still be safe if we only check for the env prefixed ones.

Yeah I think this is good to have too. I'll add a quick warning for this, and maybe we could improved later if needed.

Is this feature available in 4.1.4?

It's not merged yet, so no.

hi! I know this PR was just merged like 2 days ago, but since it's in main branch, we should expect some release for this? ty!

@lucassonsinlima it is already released as<EMAIL_ADDRESS>Next week 4.2 stable will go out if there aren't regressions and the current issues in ecosystem-ci are resolved

@lucassonsinlima it is already released as<EMAIL_ADDRESS>Next week 4.2 stable will go out if there aren't regressions and the current issues in ecosystem-ci are resolved

thank you!!

@patak-dev Is there anything incorrect about trying to use it in the following way:
    <link rel="icon" href="%VITE_FAVICON_URL%" />

It works well in the development environment, but when I try to run build I get the following:

Thanks for the report @aardvarkk, https://github.com/vitejs/vite/pull/12404 should fix your issue.

That's so cool! updated vite to use it here.
there is some way to check an env variable (an if statement) to dynamically ad some script on it?

Thank you, guys.  🎉

Does this enable conditionals too, i.e. the logical equivalent of the following ejs syntax:
<% if (process.env.REACT_APP_ENV !== 'production') { %>
  <meta name="robots" content="noindex,nofollow" />
<% } %>

@fritzfelix  I feel that vite may not natively support it, perhaps you can use a similar plugin to pre-parse the ejs module.
{
  enforce: 'pre',
  name: 'ejs-inject',
  transformIndexHtml: {
    order: 'pre',
    handler(html) {
      const template = ejs.compile(html);
      process.env.REACT_APP_ENV = 'production';
      return template(process.env);
    }
  }
}

Yeah, conditionals are not planned for Vite and is documented at https://vitejs.dev/guide/env-and-mode.html#html-env-replacement. We don't want to add a potential proprietary templating style to the HTML by default.

I ended up using ViteEjsPlugin, I couldn't figure out how to use this feature 🤷♂


avoid the use of pkg_resources when importlib.metadata is available

pkg_resources is removed from Python 3.12, and its use raises a warning.
importlib.metadata can be used to get distribution info.

@d9pouces
I've fixed the CI issues in the recent changes
Can you update your PR so I can review and merge it?

@d9pouces I've fixed the CI issues in the recent changes Can you update your PR so I can review and merge it?

Done!
Can I also expect the merge of #737?

Conflicts seem to have disappeared and this branch can be merged.

Thanks @d9pouces


ACC-273.2 (Level A)Kyma-dashboard_Button Focus lost.

1.1 Internal Message Details
Internal Message:
Test Plan:	ACC SAP Kyma runtime 2024
Message Priority:	High
Affected Check Point:	ACC-273.2
Affected UI:	Button
Short text:	ACC-273.2 (Level A)Kyma-dashboard_Button Focus lost.
1.2 Application Details
Application Name	Kyma-dashboard
Application URL:	https://canary.cockpit.btp.int.sap/cockpit#/globalaccount/7304343e-1ce0-412b-a955-5948bab9d2c8/subaccount/e0ec1134-2770-40a2-910b-2b8e32ec8f60/subaccountoverview
User:	SSO
Technology:	SAP UI5
UI5 Version:	1.124.1
1.3 Test Environment Details:
GitHub	https://github.com/kyma-project/busola/issues
Web browser:	Google Chrome - Version 127.0.6533.73 (Official Build) (64-bit)
Operating System:	Windows 11 Enterprise Version 22H2
Screen Reader:	JAWS 2024.2405.83 ILM 64 bit for Windows
2. Screen Flow
•	Login into the application using the URL provided.
•	URL: https://canary.cockpit.btp.int.sap/cockpit#/globalaccount/7304343e-1ce0-412b-a955-5948bab9d2c8/subaccount/e0ec1134-2770-40a2-910b-2b8e32ec8f60/subaccountoverview.
•	Activate ‘Console URL:Link to dashboard’ in overview page.
3. Issue
Synopsis: Focus lost.
Observed Behavior:
On closing ‘Error’ dialog, focus is lost and moves to browser UI elements instead of staying on ‘Create’ button within ‘Deployments’ page.
Expected Behavior:
Focus should not be lost on activating an ui, the focus should be retained on the button or on the next active UI element.
When losing the focus on the current element the reading/interaction is stopped. The user is then forced to relocate the focus back to the previously used UI element. This adds much additional mental and physical work and shall therefore be avoided.

Kindly refer to attached error and issue processing document for more details.
Regards,
Huda

ACC-273.2 (Level A)Kyma-dashboard_Button Focus is lost.docx
Issue processing-09.2021 (1).docx


Added new goog:chromeOptions key.

Starting with Selenium 3.8 the default chromeOptions are ignored on selenium chrome instances, instead goog:chromeOptions is the new key.
https://github.com/elgalu/docker-selenium/issues/201
This adds the new goog:chromeOptions key while keeping the older chromeOptions key, selenium ignores the extra parameters sent to it and uses only the correct one.

Codecov Report

Merging #114 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #114      +/-   ##
=========================================
- Coverage    0.31%   0.31%   -0.01%
=========================================
  Files           4       4
  Lines         954     955       +1
=========================================
  Hits            3       3
- Misses        951     952       +1

Impacted Files
Coverage Δ

selenium.go
0% <0%> (ø)
:arrow_up:

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 e9cd906...cf3aaf2. Read the comment docs.

Thank you!


[Question]: conn.RemoteAddr() null pointer

Actions I've taken before I'm here

[X] I've thoroughly read the documentations about this problem but still have no answer.
[X] I've searched the Github Issues/Discussions but didn't find any similar problems that have been solved.
[X] I've searched the internet for this problem but didn't find anything helpful.

Questions with details
gnet版本v1.6.6
在React()方法中存储客户端连接gnet.Conn连接到sync.Map中， 然后在其它协程（清理心跳超时的连接）中取出sync.Map的连接conn，并调用conn.RemoteAddr().String() 会产生 panic
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x74f2f1]
在另一个issue中看到gnet 需要回收 localAddr 和 remoteAddr 的内存， 不确定跟这个有没有关系
https://github.com/panjf2000/gnet/issues/544
onClosed的时候有从sync.Map中删除conn
https://github.com/panjf2000/gnet/issues/126
Code snippets (optional)
No response

RemoteAddr() 不是线程安全的方法，不能在 gnet 事件函数 之外调用，用户需要预先保存起来：https://pkg.go.dev/github.com/panjf2000/gnet/v2#Conn
还有，建议不要使用 v1 了，切换到 v2，v1 目前不维护了。

🤖 Non-English text detected, translating ...

RemoteAddr() is not a thread-safe method and cannot be called outside the gnet event function. Users need to save it in advance: https://pkg.go.dev/github.com/panjf2000/gnet/v2#Conn
Also, it is recommended not to use v1 and switch to v2. v1 is currently not maintained.

gnet.Conn已经保存到了sync.Map中， 为什么还会空指针呢

🤖 Non-English text detected, translating ...

gnet.Conn has been saved in sync.Map, why is there still a null pointer?

@panjf2000

🤖 Non-English text detected, translating ...

@panjf2000

gnet.Conn已经保存到了sync.Map中， 为什么还会空指针呢

@panjf2000 可以解答下吗，谢谢

🤖 Non-English text detected, translating ...

gnet.Conn has been saved in sync.Map, why is there a null pointer?

@panjf2000 Can you answer this question? Thank you.

不是因为 gnet.Conn 是空指针，而是因为其内部的 remoteAddr 字段是空，这里之所以会 panic 是因为调用了 RemoteAddr().String().

🤖 Non-English text detected, translating ...

It's not because gnet.Conn is a null pointer, but because its internal remoteAddr field is empty. The reason why there is a panic here is because RemoteAddr().String() is called.


adds documentation for passthrough options

This PR adds documentation on a frequently asked question about how "passthrough options" work with Connect-Query.

This would definitely prevent me from asking about how to add options in slack. LGTM!


fix: updated node-tdd

:tada: This PR is included in version 2.12.10 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Libclang in WASM

Is the libclang interface exposed in the clang WASM?
Is is even possible to do so?

This might be of help https://github.com/TheComputerM/libclang-wasm

Thanks @TheComputerM 🙈🙈🙈


SPI delay improvement

It seems the SPI transmission(8 bits data) added about 3us delay, this can be get rid of by moving the actual transmission to another time, and only indicate the daughter board to switch to next LED using NSS pin as generic GPIO. However, the bl_set_led function call itself takes about 6us which is too long. We can try to directly set GPIO or write to BSRR to see if that will be much quicker.

By using NSS pin as GPIO indicator and directly write to GPIO registers, we minimised the delay, and now it's about 200-300ns which should be good enough. And we can see from the diagram that the LEDs on mother and daughter board lights almost at same time, the delay is within 1 sample period.


loadingComponent property from defineAsyncComponent is ignored on SSR

Vue version
3.2.45
Link to minimal reproduction
https://stackblitz.com/edit/github-1aowkn?file=src%2Fpages%2FHome.vue&terminal=dev
Steps to reproduce
I encountered an issue with defineAsyncComponent loadingComponent property, it seems that the component provided is ignored when using SSR and ClientOnly tricks.
<script setup>
import { ref, defineComponent, h, defineAsyncComponent } from 'vue';

const ClientOnly = defineComponent({
  setup(_, { slots }) {
    const show = ref(false);

    onMounted(() => {
      show.value = true;
    });

    return () => (show.value && slots.default ? slots.default() : null);
  },
});

const ComponentA = () => h('div', { style: 'background: red' }, 'Component A loading');
const ComponentB = () => h('div', { style: 'background: blue' }, 'Component B loaded');

const AsyncComponent = defineAsyncComponent({
  loader: () => new Promise((res) => setTimeout(res, 5000)).then(ComponentB),
  loadingComponent: ComponentA,
  delay: 2000,
});
</script>

<template>
  <ClientOnly>
    <ComponentA />
    <ComponentB />
    <AsyncComponent />
  </ClientOnly>
</template>

What is expected?
In the example above, using AsyncComponent should render nothing for 2 seconds (delay) then render ComponentA for the next 5 seconds (the loadingComponent) and then, after a total of 7 seconds, display the ComponentB
What is actually happening?
Actually, it renders nothing for 7 seconds (delay + loading time) and then displays the ComponentB
System Info
  System:
    OS: Linux 5.15 Ubuntu 20.04.5 LTS (Focal Fossa)
    CPU: (16) x64 AMD Ryzen 9 5900HS with Radeon Graphics
    Memory: 9.14 GB / 15.33 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 18.9.0 - ~/.nvm/versions/node/v18.9.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.9.0/bin/yarn
    npm: 8.19.1 - ~/.nvm/versions/node/v18.9.0/bin/npm
  Browsers:
    Chrome: 108.0.5359.124

Any additional comments?

Found the issue when using Nuxt 3.0.0, also created an issue on the repo https://github.com/nuxt/framework/issues/9765
Maybe related to https://github.com/vuejs/core/issues/6482

That's because it's nested in <Suspense>.


Set custom UserAgent on go-vinyldns client

As a maintainer of terraform-provider-vinyldns, I'd like terraform-provider-vinyldns send a custom User-Agent on requests such that usage can be tracked in aggregated logs and operational visibility metrics. I'm imagining a UserAgent string such as terraform-provider-vinyldns/<version>.

Hi! I'd like to help, but I don't have an idea where I can get the version from, can you give me a clue?

@marekfilip Hi there and thanks for the offer to help! Currently, the version is stored in the GNUMakefile here and used here at build time as well as in generating terraform-provider-vinyldns GitHub releases.
Does that clarify?

@mdb Great! To make sure is my plan good. I will pass version from main to vinyldns package, if version would be empty I set UserAgent as 'terraform-provider-vinyldnswithout a version otherwise it will be liketerraform-provider-vinyldns/. In this way version will be set on build time and released application will introduce herself always with the proper UserAgent`.
Is that make sense?

@marekfilip Yep; if I understand correctly, I think that sounds perfect!

I'll deal with it on Saturday :v:

Well, it not saturday but here it is. Sorry for delay.
Thanks in advance for review and information what to change/improve.

Thanks @marekfilip !


docs: Changed README with specified output

Description
I have updated the README because after seeing the output with terraform console, I see that it's "cgw_ids" and not just "ids".
Motivation and Context

Breaking Changes

How Has This Been Tested?

[ ] I have tested and validated these changes using one or more of the provided examples/* projects

It is ids in the code - https://github.com/terraform-aws-modules/terraform-aws-customer-gateway/blob/master/outputs.tf#L1
You probably looked into the VPC module where it is called cgw_ids - https://github.com/terraform-aws-modules/terraform-aws-vpc/blob/master/outputs.tf#L371

Sorry, there has been an error, I was looking at the 2.0 documentation while using 1.0, sorry for the inconvenience.


Inference with other well known dataset

Hi, Thank you for your great work.
I tried to use NeuralRecon to reconstruct "Totem pole" data at  here.
What I did is mimicking ScanNet data set style and test as a data of ScanNet. Therefore, I resized color images and changed file extension from "png" to "jpg" and get camera trajectory data. When I test using open3d to find out trajectory is correct, it seems nice.
However, It didn't work. and I find that the array occ has all negative values.
I want to know that what i did wrong and how to test data like "Totem pole".
Thanks.

We haven't experimented with this dataset before, but it should work. There are two general rules of thumb to adapt NeuralRecon to new datasets:

Make sure the coordinate system and format of the camera poses are aligned with the ScanNet/ARKit demo.
Make sure that the ground plane is correctly configured, as detailed in here.

Hi!
First congrats on your amazing work! I reopen this issue because I'm struggling with exactly the same problem: occ has all negative values when I ran it with my custom data.
I'm pretty sure the coordinate system and format of the camera poses are aligned with the ARKit demo. I think the problem is related with the offset that you apply to the x-y plane of the camera poses. I tried using an offset to put x-y plane at 1.5 meters over the ground, but it didn't work. With some random with some specific numbers NeuralRecon is able to reconstruct a couple of frames, but then the same problem appear.
I've seen in other issues that we have to configure the offset to simulate the conditions of ScanNet data during training, but you never specify which is that criterion. Therefore I wanted to ask you that, which is the criterion you follow to determine you need that +1.5 meters?
Thanks! :)


Datepicker - Clear should set active date to null

Describe the bug
Clear button from Datepicker is not updating the active date to null.
Expected behavior
Clear the active date when the Clear button is pressed.
Actual behavior
The active date is updated to the current date when the clear button is pressed

Ok, I was testing and the problem is that we don't allow to clear the date, that's important to be able to notify that confirmation

I'm wondering if we can revert the date selection if the cancel button is pressed after the clear button, what do you think folks? <3

Actually we define the Datepicker dialog as sort of a "step by step date creator". When you choose a date by picking a day it's just a first step. Then you can still choose cancel to prevent date change in input, or choose OK to submit picked date. Also, you can still set clear to cancel the selection (which is not same as cancelling the date change state). At this moment, we don't want to make any changes to pickers as it's too soon after last release. But will be monitoring the situation and issues.


Swap order of name and toolCallId

https://github.com/microsoft/vscode/pull/231788#discussion_r1808469301

We did thsi


[Request] Shortcut to go to the end of tv episode name when adding .torrent file

When the metadata for the torrent file name and tv episode names are too long it is not possible to see which tv episode you are selecting when opening .torrent file for season packs. A shortcut to go to the end of the tv episode name will be very useful.

When you manually add torrent file for season pack it opens a window with a list of the episodes, and if you have long names for the torrent metadata - like torrent name, tv show name - you can see only the beginning of the selection. But to see which episode you have actually selected, you have to wait the carousel to reach the end of the selection which is a very slow thing.
example: https://i.imgur.com/UnK3vTv.png

That dialog is managed and showed by Kodi.
Elementum can only give list of options to display (what it actually does).
I don't know of any way to control that window.


Commercetools/refactor cms contentful

@lauraluiz please, review
#28
#23
I created one branch for docu, javadoc & refactoring because I started on javadoc and during refactoring it was already outdated

@katmatt I adjusted the code according to your remarks

@piobra Please merge your PR 👍 Or let me know if you don't have write permissions and then I'll merge your PR.

Great job @piobra ! The documentation is awesome!

@lauraluiz @katmatt  Thanks :)


upgrade test vm image to pg 16

The latest supported Postgres version at Neon is 16. This pull request bumps the Postgres version of the test image so that people can test autoscaling with the latest version of supported Postgres in Neon.

A couple remaining lines referencing pg14-disk-test in README.md, but other than that, lgtm - thanks!
cc'ing the others in case there's anything else to consider here.

Fixed, thanks!

Let's also update the image we use on CI, currently it's Postgres 15:
https://github.com/neondatabase/autoscaling/blob/79a96a1286e8272b51b9bc2ddbacfead57e8c8d7/.github/workflows/build-test-vm.yaml#L40
https://github.com/neondatabase/autoscaling/blob/79a96a1286e8272b51b9bc2ddbacfead57e8c8d7/.github/workflows/build-test-vm.yaml#L31
$ git grep postgres-15
.github/workflows/build-test-vm.yaml:      vm-postgres-15-bullseye:
.github/workflows/build-test-vm.yaml:        value: ${{ jobs.tags.outputs.vm-postgres-15-bullseye }}
.github/workflows/build-test-vm.yaml:  IMG_POSTGRES_15_BULLSEYE: "neondatabase/vm-postgres-15-bullseye"
.github/workflows/build-test-vm.yaml:      vm-postgres-15-bullseye: ${{ steps.show-tags.outputs.vm-postgres-15-bullseye }}
.github/workflows/build-test-vm.yaml:          echo "vm-postgres-15-bullseye=${{ env.IMG_POSTGRES_15_BULLSEYE }}:${{ inputs.tag }}" | tee -a $GITHUB_OUTPUT
.github/workflows/build-test-vm.yaml:      - name: build ${{ needs.tags.outputs.vm-postgres-15-bullseye }}
.github/workflows/build-test-vm.yaml:          ./bin/vm-builder -src postgres:15-bullseye -spec tests/e2e/image-spec.yaml -dst ${{ needs.tags.outputs.vm-postgres-15-bullseye }}
.github/workflows/build-test-vm.yaml:      - name: docker push ${{ needs.tags.outputs.vm-postgres-15-bullseye }}
.github/workflows/build-test-vm.yaml:          docker push ${{ needs.tags.outputs.vm-postgres-15-bullseye }}
.github/workflows/e2e-test.yaml:          TEST_IMAGE: ${{ needs.build-test-vm.outputs.vm-postgres-15-bullseye }}

@bayandin thanks! just added a commit https://github.com/neondatabase/autoscaling/pull/810/commits/66c01faae2d057dfffa7cd9d3a92323f076a0d3a. I assume this workflow will be called from somewhere else, and given that the output artifact id has been changed to postgres-16, probably there is something else to update in the cloud repo?

well, it seems that we only triggered the workflow once so I assume it's not actively used for now.
https://github.com/neondatabase/autoscaling/actions/workflows/build-test-vm.yaml

assume this workflow will be called from somewhere else

Nah, workflows in autoscaling repo are called withing the repo itself only. Afair.
Will double-check it once get back to the laptop

Will double-check it once get back to the laptop

Yep, workflows from this repo are used only in this repo.


Reactive regression on 3.4.15

Vue version
3.4.15
Link to minimal reproduction
https://play.vuejs.org/#eNqNUttqGzEQ/RWhF7c0XhHcvhg3pC2Btg9taQt90ct2NbGV6MZo5BjM/nukXe9641wICKS5njNntOefQqi2CfiSr2KDOtCFdLALHokpuK6TIbaXjjFVU/3mbf9mDIESusFibFubBEs2+wrG+FnvbcvVnklX7sbbkAjUcqixcX1sxxhtdKy6LuzdRzZj/zwadWg0wh2TpgjS5bMSI/tsENhgaoJsMbbanF/s9wWQte1KZKukT1L4Gde2TDy3dahuondZjY6ZPASi5CNxybNcxZZ8QxTiUohGuVymwOgtVg5IuGDFZU4TmBxpC3Pl7eWiel+dfxBKR5r6K4h2/h/9XQTMXSTPio04Iju3gHMEpwABX4t7UvYA+yT2CH8QNctCsfHuWq9PRCnL1AbwZyDt3UNx6vwB7r53PsIE4zDNBprbJ/w3cdcP9QuhYzYRgGpcA/Xhqz8/YJffY9B6lcxhEc8Ef0P0JhWOfdrn5FSmPcnr2H7rdqzd+m+82hG4OAxViA6f+LCQLy+MfqS7qBajiu09Yuwe7Q==
Steps to reproduce
Just open the reproduction and see the recursive updates in action.
What is expected?
The same as son v3.4.14 and lower: Only one update should occur.
What is actually happening?
The getter modifies the state before returning it, and this triggers reevaluation of the getter after the call, getting it caught in an infinite update loop.
System Info
System:
    OS: macOS 14.2.1
    CPU: (14) arm64 Apple M3 Max
    Memory: 308.45 MB / 36.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.6.1 - ~/.nvm/versions/node/v20.6.1/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v20.6.1/bin/yarn
    npm: 9.8.1 - ~/.nvm/versions/node/v20.6.1/bin/npm
    pnpm: 7.30.0 - ~/.nvm/versions/node/v20.6.1/bin/pnpm
    bun: 1.0.11 - ~/.bun/bin/bun
  Browsers:
    Chrome: 120.0.6099.129
    Chrome Canary: 122.0.6225.0
    Firefox Nightly: 113.0a1
    Safari: 17.2.1
    Safari Technology Preview: 17.4

Any additional comments?
I am fairly confident that this regression was introduced by:
#10123 / bcdecb7b1a5b6eb55a2b5975f2f73df69c52099a

Computed was updated in recent versions, and I believe this is the expected result in your reproduction. You are not supposed to make effects in your computed function.

Expected. Such usage is explicitly documented as a bad practice: https://vuejs.org/guide/essentials/computed.html#getters-should-be-side-effect-free

I strongly disagree. I don't think such massive changes of behavior, even if the pattern itself is bad practise, should be rolled out in a patch release. This goes against all decent practises of semantic versioning.

Sorry for replying to a locked issue, but I just checked my unread email and found that I had forgotten to give a follow-up. This is no longer an issue since 3.4.16.


Change builders coordinates for non Java buildpacks

The samples need to start using latest builders coordinates.
For example:
https://github.com/paketo-buildpacks/samples/blob/main/.github/workflows/test-all-samples.yml#L81 shoudl instead use paketobuildpacks/builder-jammy-full


Indent lines sometimes shown inside Treesitter Context window

Problem
When scrolling inside deeply nested structures, sometimes indent-blankline draws guides inside the Treesitter Context window.
It does reproduce either with Onedark theme, and default Neovim theme.
I'm using Neovim 0.10.3, and latest version of the plugin.
https://github.com/user-attachments/assets/0acdbc6f-b6e2-43b6-830c-bc96c82fe7c9
https://github.com/user-attachments/assets/4a4bcab7-1640-4b5b-92c1-27404bb0748c
Steps to reproduce
I'm openin file neovim/src/nvim/mouse.c from Neovim repo on commit 487c48ec86.

Put cursor at line 530
Scroll down with  till the line ~538
Observe indent guides inside TSContext window

I've the following configs for indent-blankline, and for treesitter-context.
  {
    "lukas-reineke/indent-blankline.nvim",
    main = "ibl",
    opts = {
      scope = { enabled = false },
      debounce = 10,
      indent = {
        char = "│",
        tab_char = "║",
        smart_indent_cap = false,
      },
    }
  },

require("treesitter-context").setup({
  enable = true,
  max_lines = 10, -- How many lines the window should span. Values <= 0 mean no limit.
  patterns = { -- Match patterns for TS nodes. These get wrapped to match at word boundaries.
    default = {
      "function",
      "method",
      "for",
      "while",
      "if",
      "switch",
      "case",
    },
    rust = {
      "loop_expression",
      "impl_item",
    },
    typescript = {
      "class_declaration",
      "abstract_class_declaration",
      "else_clause",
    },
  },
})

Expected behavior
indent-blankline guides shoud not be drawn inside treesitter-context window.
Neovim version (nvim -v)
0.10.3

I am not sure whether this is an issue with treesitter context, or with the indent-blankline, and I would appreciate any help towards resolving the issue.

This is not a bug in indent-blankline, IBL does not know anything about other plugins.
You can take a look at :help ibl.config.exclude to see how to disable IBL for specific buffers. Not much else I can help you with, sorry.

Ok, got it. Thanks anyway

Okay, so I've checked the buffer that Treesitter Context creates. Its buftype is "nofile", which should be excluded from the indent-blankline. I have default exclude options. When I added exclude buftype "nofile", the issue was still present.
https://github.com/user-attachments/assets/6f93973e-7048-4ccb-b6a3-25d6b0a3be4e


[RHTAPBUGS-434] Update dockerfile to fix github action error

What does this PR do?:

Updates the copy command to copy just the tini binary, not the entire folder
Which issue(s)/story(ies) does this PR fixes:

https://issues.redhat.com/browse/RHTAPBUGS-434
PR acceptance criteria:

[ ] Unit/Functional tests

[ ] Documentation

[ ] Client Impact

How to test changes / Special notes to the reviewer:
Manually retested the image on a local cluster to ensure feature flagging still worked as expected.  Confirmed manager binary is still running under tini with these new changes

/retest

/retest

/retest


🌱 Fix go mod vulnerabilities according to different CVEs

What this PR does / why we need it:
Backport of #157.
Fixes# https://github.com/advisories/GHSA-r48q-9g5r-8q2h , https://github.com/advisories/GHSA-69cg-p879-7622 and https://github.com/advisories/GHSA-hp87-p4gw-j4gq

/test-ubuntu-integration-release-1-2

/hold
this is missing hack/tools/go.mod update

/retest


🚧 Add support for stock market

This PR will add a new feature to the game - that is the stock market. With this feature, we are going to introduce support to buy shares or invest in other people's companies, receive dividends (not only for the owner but also the shareholders), and many things related with real business world. To implement these changes we identified the following as some goals to fulfill.

[ ] Allow a player to create more than one company (this'd be easy to implement)
[ ] Create the stock market dashboard to display trends in people's companies. We are going to use my LineGraph library
[x] Adding support for data and time. This is to update trends in stock market in continuous time intervales. We are planning to use my timers library for better support
[x] Creating methods to invest and receive dividens in other people's companies

This closes #50 and closes #47

At this point (with the stock market dashboard), the map looks like this

We are happy to announce that the dashboard is functioning now (after 9767ac9). Here are some screenshots


fixes and partial travis/docker/coveralls/tests

shell=True is temporary, as it is not recommended. This should be removed shortly.
Travis/Docker/Coveralls integrations and tests are partial. The rest will be added before next release.

Coverage increased (+12.5%) to 37.255% when pulling c555722c7bb3db47a683b0b61e6d81179fff2d3b on dev into 896c2fb2178642d987060a153bc299d10fa4c682 on master.


evp_extra_test: Do not manipulate providers in default context

Otherwise the with OPENSSL_TEST_RAND_ORDER following tests will
be broken. There is also no real need to do that.
Fixes #14070
Checklist

[ ] documentation is added or updated
[x] tests are added or updated

@paulidale  - there is another failure shown here in ecdsatest
just doing
test/ecdsatest -test x9_62_tests -iter 1

Shows a problem..
This does some weird stuff.
the fake_rand_provider_start happens followed by fake_rand_set_callback(), but then inside the generate cal lfake_rand_newctx() gets called which clears the cb. Hench the random comes from the wrong place..

There's a fix for that.  It's waiting for the 24 hour timer to run down, although neither @t8m nor I would object to it being declared urgent.

Merged to master. Thank you for the reviews. Hopefully this fixes the most common intermittent failures in CI. If you spot some more, please create a new issue.


Visualize joint axis and display type and limits

Show joint properties in left hand panel and optionally show joint axis as arrow in the 3d view.

Can you post some screenshots which show the behavior you've added. I believe I know what you intend from the code changes, but trying it locally I'm not seeing what I would expect. It would also be helpful if you could describe how you test it out so that we're using the same baseline for comparison.

Load a robot model with joints that have axes to the parameter server, then add a RobotModel display to rviz, then expand Links, change Link Tree Style to Joints in Alphabetic Order or Tree of links and joints, then find a continuous, revolute, prismatic, or planar joint, then click the Show Joint Axis checkbox to show a green arrow for the joint.

... and it looks like unchecking the box doesn't make the arrow disappear, I need to fix that.

Using rrbot from gazebo_ros_demos:
roslaunch rrbot_description rrbot_rviz.launch

Ok, I was missing the part where you have to select a different tree view mode. lgtm


App doesn't throw error when null value is provided to GridView.builder and ListView.builder and IDE will be 50-80% slower

@ Moderator || please change Title when this one is not right. I was also thinking about Title: Can't provide null to GridView.builder or ListView.builder.
When using an GridView.builder or ListView.builder (I only have tested it with those, but logical every similar Widget should be effected) and null is provided to the Widget, then no exception is thrown. The itemCount is nullable so, I expect, null can be provided. While that issue leaves the screen blank when building new or freezes, the preferred IDE, in my case VS Code, but effects also Android Studio, will get slower for about 50 to 80%.
Found on latest Stable, Master and Dev
Steps to Reproduce

Clone reproducible issue from https://github.com/TheGlorySaint/GridViewListViewNoError
Execute flutter run on the code sample
Code should run fine
Now uncomment line 50 to 63 and/or 78 to 84
Save File
Hot Reload, Hot Restart or build App completely new
IDE (VS Code or Android Studio) will get 50-80% slower, for example when wanting to terminate Simulator, with "Debugger Stop Button"

Expected results:
Either that you can provide null to the ListView/GridView.builder or that you can't provide null and an error/exception is thrown white the null value used text.
Actual results:
You can't provide null to ListView/GridView.builder and/or there is no exception/error thrown.

Code sample
https://github.com/TheGlorySaint/GridViewListViewNoError

Logs
flutter doctor -v
[✓] Flutter (Channel master, 3.9.0-22.0.pre.28, on macOS 13.3 22E252 darwin-arm64, locale de-DE)
    • Flutter version 3.9.0-22.0.pre.28 on channel master at /opt/homebrew/Caskroom/flutter/3.7.9/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 1a6a20cdc6 (8 hours ago), 2023-04-05 06:39:09 +0100
    • Engine revision 4219c72ac8
    • Dart version 3.0.0 (build 3.0.0-408.0.dev)
    • DevTools version 2.23.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/dqss/Library/Android/sdk
    • Platform android-33-ext5, build-tools 33.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.77.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.62.0

[✓] Connected device (4 available)
    • iPhone 14 Pro (mobile)        • 549F21F9-460D-46A6-BE18-0D4F52F67873     • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • Chrome (web)                  • chrome                                   • web-javascript • Google Chrome 111.0.5563.146

[✓] Network resources
    • All expected network resources are available.

• No issues found!

The problem is that you have shrinkWrap (which requires building all children) and an infinite number of children (null itemCount and the itemBuilder never returns null). So flutter will continue to build children infinitely. I don't think there is any possible way to detect this condition before hand :laughing:.
See https://en.wikipedia.org/wiki/Halting_problem

@moffatman You can also set shrinkWrap to false (adjust the Column) and this issue also exist.
Shouldn't then the itemCount be required?

@TheGlorySaint Please share a code sample with column adjusted as you say. I can use the following without any issue:
      Column(
        children: [
          // ...
          Flexible(
            child: ListView.builder(
              itemCount: null,
              itemBuilder: (BuildContext context, int index) {
                return Text('$index');
              }
            )
          ),
          // ...
        ]
      )

I have tried applying Flexible (to the provided sample code above) as Moffat suggested and see it works:

@TheGlorySaint Can you try and confirm it also works for you?

@moffatman @huycozy I can confirm that wrapping it into a Flexible and null is provided to the itemCount works.
Still, there is the issue that when shrinkWrap = true is provided, no exception is thrown. Nowhere is stated that either shrinkWrap = true with an itemCount must be provided or wrapped into a Flexible Widget.

@huycozy
At this example, you need to provide a shrinkWrap = true else you get an Vertical viewport was given unbounded height. error. You can't wrap the GridView with an Flexible or Expanded Widget else you get "RenderFlex children have non-zero flex but incoming height constraints are unbounded."
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({
    super.key,
  });

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int? nullLengthGridView;
  int? workingLengthGridView = 4;
  int? nullLengthListView;
  int? workingLengthListView = 4;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(widget.title),
      ),
      body: CustomScrollView(
        slivers: [
          SliverToBoxAdapter(
            child: Padding(
              padding: const EdgeInsets.all(15),
              child: DecoratedBox(
                decoration: BoxDecoration(
                  color: Colors.white,
                  borderRadius: BorderRadius.circular(15),
                  boxShadow: const [
                    BoxShadow(
                      color: Colors.black38,
                      spreadRadius: 0,
                      blurRadius: 10,
                    ),
                  ],
                ),
                child: Padding(
                  padding: const EdgeInsets.all(15),
                  child: Column(
                    children: [
                      const Align(
                        alignment: Alignment.topLeft,
                        child: Text(
                          'Grid',
                        ),
                      ),
                      const SizedBox(height: 20),
                      GridView.builder(
                        physics: const NeverScrollableScrollPhysics(),
                        gridDelegate:
                            const SliverGridDelegateWithFixedCrossAxisCount(
                          crossAxisCount: 2,
                          crossAxisSpacing: 10,
                          mainAxisSpacing: 10,
                          childAspectRatio: 2,
                        ),
                        shrinkWrap: true,
                        itemCount: 5,
                        itemBuilder: (BuildContext context, int index) {
                          return Text(
                            '$index',
                            style: const TextStyle(
                              fontSize: 20,
                            ),
                          );
                        },
                      ),
                    ],
                  ),
                ),
              ),
            ),
          ),
        ],
      ),
    );
  }
}

You could refactor this to use a sliver grid instead here.
I don't think it's possible to add an error. Since it's valid to use shrinkWrap=true, itemCount=null, and itemBuilder which only returns widgets for some limited values of i.

The possibility to leave itemCount null is provided because there are cases in which a ListView can expand indefinitely, by building dynamically all the elements - I want to point out that a ListView in Flutter is RecyclerView in classic Android development.
So, having the possibility to let the list just create the items when you have theme on the screen, allows you to have an infinite but unrendered number of them.
Adding shrinkWrap is the problem, because you are requiring the ListView to have a finite size.
I think that the way we could have to recognize this error and stops silent fails of the behavior would be just to check in the constructors that shrinkWrap is false if itemCount is null - this for @moffatman stating that it's difficult to prevent this error.

@emiliodallatorre
You can use itemCount: null and still have a finite number of items. The ListView will stop building more items the first time you return null from the itemBuilder.
So this is valid:
ListView.builder(
  itemCount: null,
  shrinkWrap: true,
  itemBuilder: (BuildContext context, int index) {
    if (index > 10 {
      return null;
    }
    return Text('Item $index');
  }
)

While this will run forever:
ListView.builder(
  itemCount: null,
  shrinkWrap: true,
  itemBuilder: (BuildContext context, int index) {
    return Text('Item $index');
  }
)

And there's no way from within the ListView we can tell them apart, since they are just arbitrary opaque callbacks. Of course in this trivial example, we know we could have put itemCount: 11. But there may be some more complex use cases where the number of items is expensive to compute at the beginning. So that's why I believe we can't take any action on this change. Except for maybe updating the documentation of shrinkWrap?

@emiliodallatorre
You can use itemCount: null and still have a finite number of items. The ListView will stop building more items the first time you return null from the itemBuilder.
So this is valid:
ListView.builder(
  itemCount: null,
  shrinkWrap: true,
  itemBuilder: (BuildContext context, int index) {
    if (index > 10 {
      return null;
    }
    return Text('Item $index');
  }
)

While this will run forever:
ListView.builder(
  itemCount: null,
  shrinkWrap: true,
  itemBuilder: (BuildContext context, int index) {
    return Text('Item $index');
  }
)

And there's no way from within the ListView we can tell them apart, since they are just arbitrary opaque callbacks. Of course in this trivial example, we know we could have put itemCount: 11. But there may be some more complex use cases where the number of items is expensive to compute at the beginning. So that's why I believe we can't take any action on this change. Except for maybe updating the documentation of shrinkWrap?

Wow...!
I thought an ItemBuilder should return a Widget, not a Widget?. This is a complete game-changer for me; thank you so much!


Support hapi v15 server events

[ ] server.event()
[ ] server.on()

If anyone wants to work on this and HC is still pre-v2, just branch from https://github.com/devinivy/haute-couture/tree/extend-manifest.

Can you add a little more detail?  As more of a newb to haute-couture and hapi, I'm not sure what you're asking for, but would like to get my paws dirty soon.

haute-couture maps (file structure <-one-to-one-> hapi plugin methods).  For example, calling server.register([plugin1, plugin2], callback) is the same as creating filesplugins/plugin1.js and plugins/plugin2.js when using haute-couture.  In hapi v15 two new plugin methods were added– server.on() and server.event().  We want haute-couture to support those two methods by mapping the contents of some files to calls to those methods.


⭐ Verify iCloud Sign in method

Looks like I have to enter manually my password for now, it's a bit annoying, we'd like to work with FaceID or TouchID to simplify payment.

According to Apple, it is expected, Testflight doesn't use FaceID, it has to be release in production, closing. Will re-open if we see the problem in production


C# vs C++

Proposed title of article
C# vs C++
Proposed article introduction
C# is a multi-paradigm general object-oriented programming language and is designed for common language infrastructure (CLI) language whereas C++ is a C++ is a general object-oriented programming language with imperative and generic programming features. It is an extension of the C programming language.
In this article, C++ and C# will be discussed in depth with focus on the way they work, similarities,key differences and when to use each language.
This information shall be written in a way a beginner shall understand and can be used comprehensively by high end programmers.
Key takeaways
1.What is C# and C++?
2.Features and properties of C# and C++.
3.Head To Head Comparison Between C++ and C# (Infographics).
4.Key differences between C++ and C# with examples.
5.C++ vs C# Comparison
6.Cross-platform - Which of the languages is easily compatible to other platforms
7.Project - Where does each language suit the best. i.e, suited for complex web-based concurrency projects or game development projects
8.Syntax - which one offers developer friendly syntax
9.Conclusion.
Article quality
I plan to cover in depth C# and C++ programming languages and clear off misconceptions and false ideologies that other ingenious have made programmers misunderstand.
This concept shall also be in a way that a beginner will understand and still remain relevant to high end programmers.
References
N/A

Thank you for your response and we thank you for submitting your topic. After some careful consideration it struck us that this topic may be a bit over saturated throughout other blog sites and official documentations as previously mentioned on your topic form by our content moderator.
We typically refrain from publishing content that is covered widely on the net or other blogs. As we're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts.
We believe this is the best way for students to build a great portfolio (for potential employers) is by building what does not exist and what can provide the most value.
You are more than welcome to pursue another more in depth topic - or continue working on this one by flushing it out a bit more (i.e. making it more in depthand/or providing more detail.


Fix issues with extended dataformat

Description:
Currently luminosity shows a value in UI, but in raw data it shows null.
This should be null/hidden in UI.
"luminosity": null,

Also I don't see these in localisation, I updated the localisation otherwise.

@io53 I don't have a lux sensor, but I see this value still even data reports "null".
It should be hidden from view when sensor is not present, am I correct?

Yes, but e0 didn't have invalid values support. It has it now.

Works for me


Fix Overflow of Text in Button

Issue Description
Problem
The leaderboard button on the home page is experiencing text overflow, causing the text to extend beyond the boundaries of the button's border on certain smaller screen sizes. This issue negatively affects the visual presentation and readability of the button.
Reproduction Steps
To reproduce the issue, follow these steps:

Open the website in Google Chrome.
Open the Chrome Developer Tools by right-clicking on the page and selecting "Inspect" or by using the keyboard shortcut Ctrl + Shift + I.
In the Developer Tools panel, click on the "Toggle device toolbar" button or press Ctrl + Shift + M to enable the device emulation mode.
Choose the device type "iPad Air" or "Samsung Galaxy S8+" from the device dropdown or enter the corresponding dimensions manually.
Navigate to the home page and locate the leaderboard button.
Observe that the text within the button overflows outside its border.

Expected Behavior
The text within the leaderboard button should remain within the button's border on all screen sizes, ensuring readability and a visually pleasing appearance.
Proposed Solution
To fix this issue, the following steps can be taken:

Adjust the CSS styles for the leaderboard button to allow for responsive behavior.
Use appropriate CSS techniques such as setting maximum widths, using ellipsis for long text, or wrapping the text when necessary.
Test the changes across various devices and screen sizes to ensure the button's text is displayed correctly without overflow.

Please feel free to assign this issue to yourself if you would like to work on it. If you have any questions or need further clarification, don't hesitate to ask.

Can I do this?


Create CONTRIBUTING.md

Hi! I've created a contributing.md, as I couldn't find one in your project.

Also, with adding this Contributing.md, I would recommend for new people to the repo, you could add a link in the Readme.md that will link directly to this, currently the readme just mentions that it is here, but for people who want to contribute, it would help to have it linked.

This looks really good! Thank you :)


Bump node version to 20

Fixes deprecation warning in github
Summary of PR changes
Bump node version from 16 to 20 in github action using statement in order to fix the github deprecation warning related to node 16.
PR Requirements

[ ] For major, minor, or breaking changes, at least one of the commit messages contains the appropriate +semver: keywords.

See the Incrementing the Version section of the repository's README.md for more details.

[ ] The action code does not contain sensitive information.

NOTE: If the repo's workflow could not automatically update the README.md, it should be updated manually with the next version.  For javascript actions, if the repo's workflow could not automatically recompile the action it should also be updated manually as part of the PR.

Thank you for your PR.  I'm currently working on a branch that adds some tests to this action prior to updating Node 20.  When I get those in place, I'll make sure to include this update.  Hopefully that should happen next week.

This has now been fixed via another PR, thank you :)

The new version that supports node 20 is v2.4.0


add CI

@homunkulus r+

:pushpin: Commit 66e7897 has been approved by japaric

:hourglass: Testing commit 66e7897 with merge 66e7897...

:sunny: Test successful - status-travis
Approved by: japaric
Pushing 66e7897df90f60f7440131b30cf89e5b5c929146 to master...


[patch] EHealth-745: added dictionary mapping for declaration_request

Changes Unknown when pulling 18a6b2eddbf6ed7f1da0cbfc459d86d748221a29 on EHealth-745 into ** on master**.


Hide repository separator

After upgrading to the latest version, use scm management to find out something like a separator.

That's by design -- just a visual separator of the current branch and context from the rest.

Looks weird.....

@eamodio Is it possible to add a configuration to control its display and hiding, which in my opinion affects my visual enjoyment.

@tiansin if this was helpful to you, consider giving a reward to the issue solver using Speedoss.com


add stick to materials

default:stick could be replaced by dry_brush, sapling or similar in non default games.
Maybe this could be added with groups PR -> group:stick instead.

Ids:
MTG default:stick
MCL: mcl_core:stick


Annotated code examples?

In your blog post on flo you link to an annotated version of the hpaste.org codebase. Sadly that link leads only to a 404 today.
Would you possibly still have that code around somewhere? Or are there possibly other examples of annotated code available?

The link just needed updating. I've fixed the link.

Cheers!


Rating system

Implement a content rating system or user feedback feature to gather user opinions and ratings for blogs or articles.

its not currently needed for website


Bug: back button on mobiles

To replicate
Load search page: family.services.govt.nz
Select category (eg. Youth Services).  Results display.
Select phone/browser back button
Expected result: blank search
Actual result: Category continues to be selected, but search summary states “0 results”

Released to UAT.
Keyword and category have been added to the React state (that's just an implementation detail), both should be tracked properly now through any number of back and forward navigations.

Looks good @Br3nda and @lamorrison.
Only comment on the fix is that (for the initial test scenario above) is you have to tap the screen to remove the light highlight/border on the Youth Service category.
But that's small beans!  The page no longer displays zero results and the prompt "Make a search or choose a category above" should make it clear to people that they aren't looking at search results now.

Agree with Sarah - small beans and if you are starting a new search you would tap the screen by default)


Fixes an issue with sun_cc lacking the __global attribute for enums

Fixes an issue with sun_cc lacking the __global attribute for enums, reported as an issue in #213
https://docs.oracle.com/cd/E19205-01/819-5267/bkaed/index.html

In branch hotfix/visibility-warnings together with other visibility issues. Testing, thanks!

Merged to master, closing. Thanks!


fix: allow empty string for condition reason

Fixes #N/A
Description
Allows empty strings for status condition reasons in the nodeclaims. This is a gap in the #1254 where we still fail validation when the reason is set to an empty string.
How was this change tested?
Manual validation: installed v0.36.2, spun up nodes, upgraded to HEAD, expired existing nodes
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 77.94%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/node/termination/terminator/eviction.go
2
89.72%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
8271

Relevant Lines:
10612

💛  - Coveralls


[Question] Angular 18 compatibility?

I would like to know what the forecast is for the release of the version compatible with Angular 18, as we are already close to version 19 as well.

8.0.0-rc.11 should be fairly stable, but we are coordinating its official release around the release of our 9.3.0 Enterprise Server and 5.0.0 renderer, which is expected in the next month or two.

Thanks


Bugfix/GitHub release status check

Superagent version updated and api changed.
This change is to reflect that.

@juliangarritano Thank you for your work. It landed and should work with the next release which will come today.


before_session hook failed: NoMethodError: undefined method `chomp' for nil:NilClass

I'm using rails-pry to create a breakpoint with a binding.pry statement. This has worked just fine for just about ever. However now there's this fun new error message I'm not even sure how to being debugging the debugger:
before_session hook failed: NoMethodError: undefined method `chomp' for nil:NilClass
usr/local/bundle/gems/pry-0.12.2/lib/pry/pager.rb:116:in `block in write'
(see _pry_.hooks.errors to debug)

The result is the binding.pry statement is ignored and the code I intent to inspect continues to run.

I got a similar error message
before_session hook failed: NoMethodError: undefined method each' for nil:NilClass`

+1
From: /concierge-render/app/controllers/api/v1/files_controller.rb @ line 3 API::V1::FilesController#destroy:

     2: def destroy
 =>  3:   binding.pry
     4:   file_name = params[:id]
     5:   type = file_params[:type] || 'uploads'
     6:   if type == 'outputs'
     7:     # Only admins can delete outputs they do not own directly.
     8:     job_id = file_params.require(:job_id)
     9:     job = Job.find(job_id)
    10:     unless current_user == job.owner || current_user.admin?
    11:       return not_found_error(file_name)
    12:     end
    13:     full_path = File.join(job.owner.data_dir, type, file_name)
    14:   else
    15:     full_path = File.join(current_user.data_dir, type, file_name)
    16:   end
    17:
    18:   if File.exists? full_path
    19:     if File.file? full_path
    20:       File.delete full_path

<page break> --- Press enter to continue ( q<enter> to break ) --- <page break>
before_session hook failed: NoMethodError: undefined method `chomp' for nil:NilClass
/usr/local/bundle/gems/pry-0.12.2/lib/pry/pager.rb:116:in `block in write'
(see _pry_.hooks.errors to debug)
[1] pry(#<API::V1::FilesController>)> F

Failures:

  1) API::V1::FilesController DELETE #destroy when the file can be found from the file name param when the file name represents a file removes the file from the user's uploads directory
     Failure/Error: expect { subject }.to change { File.exists?(file_path) }.from(true).to(false)
       expected `File.exists?(file_path)` to have changed from true to false, but did not change
     # ./spec/controllers/api/v1/files_controller_spec.rb:240:in `block (5 levels) in <top (required)>'
     # ./spec/rails_helper.rb:111:in `block (3 levels) in <top (required)>'
     # /usr/local/bundle/gems/database_cleaner-1.8.4/lib/database_cleaner/generic/base.rb:16:in `cleaning'
     # /usr/local/bundle/gems/database_cleaner-1.8.4/lib/database_cleaner/configuration.rb:87:in `block (2 levels) in cleaning'
     # /usr/local/bundle/gems/database_cleaner-1.8.4/lib/database_cleaner/configuration.rb:88:in `cleaning'
     # ./spec/rails_helper.rb:110:in `block (2 levels) in <top (required)>'

Finished in 1.01 seconds (files took 1.98 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./spec/controllers/api/v1/files_controller_spec.rb:239 # API::V1::FilesController DELETE #destroy when the file can be found from the file name param when the file name represents a file removes the file from the user's uploads directory

Still getting the same error and the execution never stops. Any lead on this?
<page break> --- Press enter to continue ( q<enter> to break ) --- <page break> before_session hook failed: NoMethodError: undefined method 'chomp' for nil:NilClass /usr/local/bundle/gems/pry-0.12.2/lib/pry/pager.rb:116:in 'block in write' (see _pry_.hooks.errors to debug)

Hey folks - sorry for necro'ing a 2 year old thing here, but the fix for this issue is to allow a psuedo-tty and to keep stdin open on Docker when running your containers. If you're running via docker-compose, make sure to add these lines to your Ruby/Rails service:
stdin_open: true
tty: true


L10N: Do not have permission to delete an asset.

This is when an user doesn't have Manage Asset permission but tried to delete an asset. (You need to remove the permission while the user have signed in to MTiOS to display Edit Asset screen)
@yuji Should I create this case to fbz instead of here?

hmm.. This message came from Data API...
Move to FogBugz. https://movabletype.fogbugz.com/f/cases/113531


fix hangs

It turned out that it is not problem on our side.
Run this:
Enum.each 1..1000, fn(i) ->
  IO.inspect i
  IO.inspect HTTPoison.head("/invalid")
end

On my mac it hangs somewhere between 40-60


remove youtube_player_flutter and flutter_inappwebview to fix common related issues in flutter_quill_extensions

We're having many issue and reports to issues that caused by flutter_inappwebview which is required and needed by youtube_player_flutter which is a dependency in flutter_quill_extensions
Embedding platform Webview in the editor causes performance issues, rebuilding the platform widget consistently makes the video reload many times on some platforms, making the editor unstable even when not interacting with the platform widget, increasing the bundle size, requiring more transitive dependencies and more manual set up for web.
We did plan on introducing this change in the next major release or by introducing a separate package (#2276), or by using the download URL to load the video without the iframe (#1916), directly using a normal video player widget (which violates the terms of service of YouTube) and some other solutions by it seems that we're getting more reports outside of GitHub, and I did have some issues while working on an outdated branch.
Removing those dependencies was something we planned for a while ago, pubspec by default use the latest compatible version of any dependencies as it assumes it confirms to Semantic Versioning and they have introduced macOS and Windows support in a minor version, which causes to have build failure for the users and for maintainers working in outdated branches, we do find it a bit confusing since we didn't manually updated the version of any related dependency.
It seems that flutter_inappwebview_windows and flutter_inappwebview_macos was introduced as dependencies of flutter_inappwebview 6.1.0, where they were experimental and causing build failure even on Android (on Windows machine). It seems that they working hard to introduce fixes as fast as possible, see
flutter_inappwebview #2294 and flutter_inappwebview #2288.
I'm grateful for their efforts to the open source community and the support for macOS and Windows even if it introduce many issues, however, I do think those dependencies should be included in the user project and not as part of this project, especially we don't heavily depend on it, if this was a payment package to a web service (that doesn't have public web API) or something related then it would be critical to have it, the solution we have is not ideal for many users, and more users detaching flutter_quill_extensions and eventually flutter_quill.
We're currently trying to focus on the current issues we have, we prefer to not have more on the surface for more issues at the moment.
We have four choices:

Do nothing for now and wait for issues to be fixed, then remove it once we're ready for a major breaking change which we are not and should not introduce a breaking change release for this change for many reasons we have discussed across multiple issues, see #2259
Introduce a breaking change in non-minor version 10.8.0 and include notes for users in CHANGELOG.md, provide a quick experimental solution to allow users to introduce their own solution, currently, we can't plan for a stable solution as we have more many issues that need time and introducing a good solution is not a priority, we also have discussed about this across multiple issues, see #2270
Separate the version of flutter_quill_extensions right away and only introduce a breaking change for the extensions package. Introducing a breaking change for flutter_quill is currently not an option due to the reasons we have discussed in many issues (we don't have a specific issue to keep track of those discussions due to the lack of time), this will break our current workflow which also have issues, I prefer to not use such solutions as we still busy with other issues and need better planning
Use a different package or load the video using the download URL, possibly violate YouTube's terms of service, but not necessarily, previously I saw some apps that use the download URL without violating the terms of service, I'm not sure if there are use cases or if what they meant that we should not download videos to the user device using the download URL or if we're allowed to only play the video in video player indicating that this is a youtube video with the youtube video URL (not the download URL) and with some indicators to make it clear this is not our video, for web this can be solved by simply using iframe with platform-specific check which still cause issues even on Flutter web.

If fixing this issue with a breaking change in a minor issue is okay for most users, then we might introduce it, it shouldn't cause any issues for libraries and packages in pub.dev (see packages that depend on flutter_quill_extensions), we also could simply make it so that it would revert the video as a video link, allowing the user to press the link and play it on Youtube app as they expect, also it's might not very common that users copy video URL that's YouTube specific and then insert it in the editor.
See #1888, #2276, #1916, #2280, #989 for related issues.

@singerdmx @CatHood0 @AtlasAutocode Need to know which method you prefer to handle this situation before introducing any change.


Can appending '.min' be optional?

I noticed you rename the output file rather than update the file in-place. Can this be optional?
https://github.com/ksmithut/metalsmith-uglify/blob/master/lib/normalize-options.js#L106

Yes, seems reasonable. I believe I used to have it as an option, but removed it in a major version release because I didn't like the way I was doing it. I'll add this back in.

@chaucerbao I have added a nameTemplate option which allows you to customize the filename. The default option is [name].min.[ext]. You can change it to [name].[ext] replace the original file in the generated output. Let me know if this fills your need.

Nice! Thanks.


Add approximately pattern

Given https://unicode-org.atlassian.net/browse/CLDR-11431, the following is how we could add the approximately pattern to Intl.NumberFormat:
new Intl.NumberFormat({
  signDisplay: "approximately"
});

Expected outputs:

Input
en-US result with signDisplay: "approximately"

5
~5

0
~0

-0
-0

-5
-5

Alternatively, we could throw RangeError on the negative numbers, since the behavior with them is not well-defined in CLDR.
Related: #6

@sffc This seems like a nice feature. Were you saying there are currently questions about the data quality here?

@sffc This seems like a nice feature. Were you saying there are currently questions about the data quality here?

I'm working on the data quality issues upstream in https://unicode-org.atlassian.net/browse/CLDR-11431

See https://github.com/tc39/proposal-intl-numberformat-v3/pull/24

See https://github.com/tc39/proposal-intl-numberformat-v3/pull/24

2021-01-14: We agreed to use the approximately sign as the format when a range collapses down to one number, but not to expose the approximately sign as another option on signDisplay.  If we want to add an approximately format in the future, it should be its own option on Intl.NumberFormat, not an overload of signDisplay.

2021-01-14: We agreed to use the approximately sign as the format when a range collapses down to one number, but not to expose the approximately sign as another option on signDisplay.  If we want to add an approximately format in the future, it should be its own option on Intl.NumberFormat, not an overload of signDisplay.


Fix issue 'double spaces between struct field prefix and identity when using long attributes

Fixes #4671 .

Thanks again!


Read-only version of the Schema Registry UI

Hello,
What configuration / ENV vars would be necessary in order to make this UI read only (no edit/create/delete abilities)?
Thanks in advance.

Hi @kalbmj. For now we only support disabling schema deletion via the option allowSchemaDeletion in env.js. If you use our docker image, you can use the environment variable ALLOW_DELETION=0 instead.
We don't support disabling the edit and create schema functionality.
Please note though that even if we supported them, it would just be a false sense of security. In order for Schema Registry UI to work, your browser has to be able to access the Schema Registry itself. So even if the UI does not allow you to edit a schema, you can always fire up your terminal and use curl or any other application that can do POST http requests and do it manually.

Thanks @andmarios

@andmarios While I understand that implementing read-only does not improve security, I would argue that it is a reasonable feature to add. What's the reason behind implementing ALLOW_DELETION but not create/edit? What makes deletion so special? In my understanding, create/edit/delete are all "write" actions, and disabling deletion alone is not very useful.

Hello @yuha0
As per #87 we now have a READ ONLY version thanks to @fstaudt.
The default value is false but you can control it with the docker var READONLY_MODE or readonlyMode in the env.js file.

@jglambed  Thanks. I didn't notice that issue. I tried the environment variable and it didn't work. I can still create new schema and edit existing one:
> docker run --name registry-ui --rm -p 9005:8000 -e SCHEMAREGISTRY_URL=http://my-schema-registry-url:8081 -e READONLY_MODE=1 -e P
ROXY=true schema-registry-ui:latest
Landoop Schema Registry UI 0.9.5
Visit <https://github.com/Landoop/schema-registry-ui/tree/master/docker>
to find more about how you can configure this container.

Enabling proxy.
Enabling readonly mode.
Setting Schema Registry URL to /api/schema-registry.
Note: if you use a PORT lower than 1024, please note that schema-registry-ui can
now run under any user. In the future a non-root user may become the default.
In this case you will have to explicitly allow binding to such ports, either by
setting the root user or something like '--sysctl net.ipv4.ip_unprivileged_port_start=0'.

Option will only be available in next release (0.9.6?), it is not yet available

@fstaudt Thanks! I will wait for the release then.
That option is in the documentation in the docker image repo and I saw Enabling readonly mode. in the log. Those are kind of misleading though...

Latest image of schema-registry-ui has been generated 13 days ago when my PR was accepted.
I had not seen it but it's possible that it already contains the readonly feature.

So it is not implemented correctly? I can confirm I am using the latest tag, which was pushed 13 days ago.

Logs still show version 0.9.5.
I don't think the latest image already include the feature.
It is indeed misleading.
I only provided the PR, I'm not aware of the release process


Update copyright

Signed off by @jnpacker

Add missing copyright
FF trigger test
/auto-cc

/cc @hanqiuzh

/lgtm


XCode7 (GM): compiler warnings

14 compiler warnings like "RKMIMETypeSerialization.m:41:17: Method override for the designated initializer of the superclass '-init' not found" found in RK 0.25

A PR with a fix would be much appreciated!

Haven't been using RestKit with Swift, but it would be ok to remove the NS_DESIGNATED_INITIALIZER from the init declarations? Either this or implementing the designated init from the parent classes, calling the implemented designated inits, which does not make much sense
- (instancetype)init
{
    self = [self initWithMIMEType:nil serializationClass:nil];
    return self;
}

No, the designated initializers should stay, and no args initializers need to be added that either throw an exception or call super.

The thing is, you really have to call the init marked as designated to get rid of the warning, so super is not enough. What you be a decent implementation for this inits with no args?
ex:
- (instancetype)init
{
    self = [self initWithMIMEType:nil serializationClass:nil];
    @throw [NSException exceptionWithName:NSInternalInconsistencyException
                                   reason:[NSString stringWithFormat:@"-init is not a valid initializer for the class %@", NSStringFromClass([self class])]
                                 userInfo:nil];
    return self;
}

Yeah, except you'd want to throw the exception before delegating to the other initializer.

If you do that, you get a "Code will never be executed" warning in the line of the other initializer :/


Localize the particular refinement condition that fails

See e.g.

@nilehmann this is now ready for you to look at, thanks!

awesome! I'll take a look ASAP

Keeping spans consistent through normalization and simplification looks hard to get right. For example
#![flux::defs {
    fn inc1(x: int) -> int {
        x + 1
    }
}]

#[flux::sig(fn() -> i32[inc1(0)])]
fn f() -> i32 {
    2
}

The current behavior is
error[FLUX]: postcondition might not hold
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:9:5
  |
9 |     2
  |     ^ return site
-Ztrack-diagnostics: created at flux-refineck/src/lib.rs:105:37
  |
  = note: a postcondition cannot be proved at this return site
note: this is the condition that cannot be proved
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:3:9
  |
3 |         x + 1
  |         ^^^^^

I'm not convinced that should be the expected behavior. I think the error should say
error[FLUX]: postcondition might not hold
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:9:5
  |
9 |     2
  |     ^ return site
-Ztrack-diagnostics: created at flux-refineck/src/lib.rs:105:37
  |
  = note: a postcondition cannot be proved at this return site
note: this is the condition that cannot be proved
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:7:25
  |
3 |         inc1(0)
  |         ^^^^^^^

This also feels questionable
#[flux::refined_by(p: int -> bool)]
struct S;

#[flux::sig(fn(s: S[@p], x: i32{p(x)}))]
fn f(s: S, x: i32) {}

#[flux::sig(fn(s: S[|x| x > 0]))]
fn g(s: S) {
    f(s, 0);
}

error[FLUX]: precondition might not hold
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:9:5
  |
9 |     f(s, 0);
  |     ^^^^^^^ call site
-Ztrack-diagnostics: created at flux-refineck/src/lib.rs:98:37
  |
  = note: a precondition cannot be proved at this call site
note: this is the condition that cannot be proved
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:7:29
  |
7 | #[flux::sig(fn(s: S[|x| x > 0]))]
  |

The first one I agree - it should point at inc(0) — that should be fixable
by tracking the span of the source expression during expansion.
the second one though seems correct to me? Or is your point that the whole
of x > 0 span should be blamed and not only the 0?
On Sat, May 20, 2023 at 5:35 PM Nico Lehmann @.***>
wrote:

This also feels questionable
#[flux::refined_by(p: int -> bool)]struct S;
#[flux::sig(fn(s: @.***, x: i32{p(x)}))]fn f(s: S, x: i32) {}
#[flux::sig(fn(s: S[|x| x > 0]))]fn g(s: S) {
f(s, 0);}
error[FLUX]: precondition might not hold
--> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:9:5
|
9 |     f(s, 0);
|     ^^^^^^^ call site
-Ztrack-diagnostics: created at flux-refineck/src/lib.rs:98:37
|
= note: a precondition cannot be proved at this call site
note: this is the condition that cannot be proved
--> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:7:29
|
7 | #[flux::sig(fn(s: S[|x| x > 0]))]
|
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/453*issuecomment-1556051785__;Iw!!Mih3wA!D6fPRFu18AQj0dItCcB3EptxqD-ZF-I5txUyzHE43hVzfV1DbrOpx1i5077Pw0sOovWBDMlcrgTsErZf_PWMe5uX$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AAMS4ODGVOQWXG6QKLWE373XHFPMNANCNFSM6AAAAAAYIFWBVA__;!!Mih3wA!D6fPRFu18AQj0dItCcB3EptxqD-ZF-I5txUyzHE43hVzfV1DbrOpx1i5077Pw0sOovWBDMlcrgTsErZf_H4x8wnp$
.
You are receiving this because you authored the thread.Message ID:
@.***>

--

Ranjit.

Ah no I see your point re second one. Yes I suppose again that ought to be
fixable by appropriately tracking spans through substitution.
On Sat, May 20, 2023 at 7:40 PM Ranjit Jhala @.***> wrote:

The first one I agree - it should point at inc(0) — that should be fixable
by tracking the span of the source expression during expansion.
the second one though seems correct to me? Or is your point that the whole
of x > 0 span should be blamed and not only the 0?
On Sat, May 20, 2023 at 5:35 PM Nico Lehmann @.***>
wrote:

This also feels questionable
#[flux::refined_by(p: int -> bool)]struct S;
#[flux::sig(fn(s: @.***, x: i32{p(x)}))]fn f(s: S, x: i32) {}
#[flux::sig(fn(s: S[|x| x > 0]))]fn g(s: S) {
f(s, 0);}
error[FLUX]: precondition might not hold
--> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:9:5
|
9 |     f(s, 0);
|     ^^^^^^^ call site
-Ztrack-diagnostics: created at flux-refineck/src/lib.rs:98:37
|
= note: a precondition cannot be proved at this call site
note: this is the condition that cannot be proved
--> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:7:29
|
7 | #[flux::sig(fn(s: S[|x| x > 0]))]
|
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/453*issuecomment-1556051785__;Iw!!Mih3wA!D6fPRFu18AQj0dItCcB3EptxqD-ZF-I5txUyzHE43hVzfV1DbrOpx1i5077Pw0sOovWBDMlcrgTsErZf_PWMe5uX$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AAMS4ODGVOQWXG6QKLWE373XHFPMNANCNFSM6AAAAAAYIFWBVA__;!!Mih3wA!D6fPRFu18AQj0dItCcB3EptxqD-ZF-I5txUyzHE43hVzfV1DbrOpx1i5077Pw0sOovWBDMlcrgTsErZf_H4x8wnp$
.
You are receiving this because you authored the thread.Message ID:
@.***>

--

Ranjit.

--

Ranjit.

If we have something like this
#![flux::defs {
    fn p(x: int) -> bool {
        x > 0 && x < 10 && x % 2 == 0
    }
}]

#[flux::sig(fn(x: i32{ p(x) }))]
fn assertp() {}

fn test() {
    assertp(11);
}

I guess it would be useful to know the conjuncts inside p that fails so maybe the error could be
error[FLUX]: precondition might not hold
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:10:4
   |
10 |     assertp(11);
   |     ^^^^^^^^^^^ call site
-Ztrack-diagnostics: created at flux-refineck/src/lib.rs:105:37
  |
note: this is the condition that cannot be proved
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:3:25
  |
3 |         x < 10
  |         ^^^^^^
note: inside this function call
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:6:25
  |
6 |         p(x)
  |         ^^^^

error[FLUX]: precondition might not hold
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:10:4
   |
10 |     assertp(11);
   |     ^^^^^^^^^^^ call site
-Ztrack-diagnostics: created at flux-refineck/src/lib.rs:105:37
  |
note: this is the condition that cannot be proved
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:3:25
  |
3 |         x % 2 == 0
  |         ^^^^^^^^^^
note: inside this function call
 --> /home/nlehmann/phd/flux-rs/flux/attic/playground.rs:6:25
  |
6 |         p(x)
  |         ^^^^

But this sounds hard, so maybe ok to collapse them in the same error.

Misc updates to track "expansion" through defs and predicates.
Example localize02.rs
#![flux::defs {
    fn funky(x: int) -> bool {
        0 < x && (x < 10 && x % 2 == 0)
    }

    fn chunky(y:int) -> bool {
        funky(y) && 2 <= y
    }
}]

#[flux::sig(fn(x: i32{ chunky(x) }))]
fn assertp(_x: i32) {}

fn test() {
    assertp(12);
}

produces the error

Example localize03.rs

#[flux::refined_by(p: int -> bool)]
struct S;

#[flux::sig(fn(s: S[@p], x: i32{p(x)}))]
fn f(s: S, x: i32) {}

#[flux::sig(fn(s: S[|x| x > 0]))]
fn g(s: S) {
    f(s, 0);
}

produces the error

In principle, the ESpan thing could probably be generalized (easily) to a full "stack" but that seems like overkill perhaps?


Package Initialization Error

Diagnostics
Request: GET https://api.teletype.atom.io/protocol-version

Versions
Teletype version: v0.13.3
Atom version: 1.29.0
Platform: linux

Thanks for the report! I'm going to go ahead and close as a duplicate of #372 so you can subscribe there for updates if you'd like. If you can, please let us know if you're consistently seeing this error or if you saw it just once.  It would also be useful to know if you use access the internet via a proxy.


leapp upgrade 8.10 -> 9.4, Grub boot failure

Hi
I tried to upgrade Oracle Linux 8.10 to 9.4 with leapp upgrade --oraclelinux but it ended up not booting with vmlinuz-upgrade and only the grub prompt was shown.
I loaded vmlinuz-upgrade and initrd-upgrade at the grub prompt manually and got the upgrade and OS running.
It seems like the grub menu.cfg was missing and installing them sorted it out:
 grub2-mkconfig -o /boot/grub2/grub.cfg
grubby —info ALL now show the menu entries.
I think this is a problem within leapp.

Hello! Just so that i can reproduce the issue correctly.

this is a legacy ( not UEFI, but BIOS system ) ?
upgrade was done NOT in OCI ?

I in fact discovered an issue with latest grub2 update that could have caused /boot/grub2/grub.cfg missing, but it should have been seen only after first stage of upgrade is done. Did you see on a serial console if before you have been dropped to grub prompt some upgrade actions were performed or not ?

Hi
[hasch@fftest ~]$ [[ -d /sys/firmware/efi ]] && echo UEFI || echo BIOS
UEFI

I'm not sure what OCI is. Please explain.
In the VMware serial console i did see the upgrade was performed. I tried several things so I could remember wrong, but first boot after leapp upgrade gave the grub prompt, and then I loaded the vmlinuz-upgrade kernel and packages was installed.

Well, as this is a test server I could ask the VMware-admin to restore the server back to 8.10, and run the upgrade again.
Is there anything special I should take a note on during the process before/after?
Otherwise I will just run the usual:

yum upgrade
reboot
leapp preupgrade --oraclelinux
leapp upgrade --oraclelinux
reboot

BTW, the only third party package I install was Postgresql but I removed that before leapp upgrade.

Very much appreciate your help!
The issue is most likely caused by grub2 update that changed the order of config file processing and updated system ends up with no config in /boot/grub2.
If this is a test system you could run:
yum upgrade
reboot
leapp preupgrade --oraclelinux
leapp upgrade --oraclelinux

After that verify:
find /boot |grep grub.cfg
cat /boot/efi/EFI/redhat/grub.cfg |grep blscfg
ls /boot/loader/entries

i need output of last 3 commands.

find /boot |grep grub.cfg
/boot/efi/EFI/redhat/grub.cfg

cat /boot/efi/EFI/redhat/grub.cfg |grep blscfg
# The blscfg command parses the BootLoaderSpec files stored in /boot/loader/entries and
insmod blscfg
blscfg

ls /boot/loader/entries
706645d081b64f3c8c1bd3ed80b205e6-0-rescue.conf
706645d081b64f3c8c1bd3ed80b205e6-4.18.0-513.18.1.el8_9.x86_64.conf
706645d081b64f3c8c1bd3ed80b205e6-4.18.0-553.5.1.el8_10.x86_64.conf
706645d081b64f3c8c1bd3ed80b205e6-4.18.0-553.el8_10.x86_64.conf
706645d081b64f3c8c1bd3ed80b205e6-5.4.17-2136.329.3.1.el8uek.x86_64.conf
706645d081b64f3c8c1bd3ed80b205e6-5.4.17-2136.331.7.el8uek.x86_64.conf
706645d081b64f3c8c1bd3ed80b205e6-5.4.17-2136.332.5.2.el8uek.x86_64.conf
706645d081b64f3c8c1bd3ed80b205e6-upgrade.x86_64.conf

The server has NOT been rebooted after leapp upgrade.

Yeah, you were hitting grub2 config manipulation issue that led to system being unbootable post-upgrade.
Note: we have pulled initial fix out of the repos, since it introduced a regression, updated version is passing testing right now.
I'll update this thread, when update is live.
Thanks a lot!

updated version is passing testing right now
Sounds good.

But then I will just roll back my test server to last snapshot and wait for next try.

I think I am also getting similar issue. I did a fresh deployment of RHEL 8.9 Server and performed leap upgrade to 9 and failed. It landed on grub menu after the upgrade

@digilkans if you have performed the upgrade of RHEL 8.9 instance, i am afraid this is not our scope of work.


⚠ Cioos National CKAN Sitemap has degraded performance

In d3d2060, Cioos National CKAN Sitemap (https://catalogue.cioos.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 6847 ms

Resolved: Cioos National CKAN Sitemap performance has improved in 95a1f74 after 5 minutes.


2024 wad challenge submission.

Thank you for your great work!
I want to participate in the CVPR'24 4D occupancy forecasting challenge.
If I want to save the output of a pretrained 4D-OCC-forecasting baseline model in the format specified as /path/to/submission, what code should I use? If there isn't any, could you provide some reference code that I could refer to?

Script to compute the metrics in the CVPR '23 paper and the leaderboard for the 4D occupancy forecasting challenge. Usage: python evaluate.py --annotations /path/to/annotations --submission /path/to/submission.

Another question, I can't acess the submission guidance url(https://eval.ai/web/challenges/challenge-page/1977/submission). Error message: Page Not Found 404.

Hey, thanks for your interest! I'm not sure why you are not able to view the link. The submission file instructions are already there. Maybe you are not logged in the Eval AI portal? Can you try deleting cookies etc, or logging into Eval AI in an incognito window?

Oh, thank you.
Now I can access the page.
I have another question. Is it possible to use the extrinsic parameter in order to train my own model? or is it cheating?

you can use the extrinsics to train your model, as long as given a ray origin and direction at inference you can get the distance travelled by the ray from your model. This means that at inference, your model should not be dependent on an "extrinsics matrix".

Thank you for your reply!


Add community documentation section

This adds links to two recent efforts at documenting fp-ts outside of the main repository.

@tstelzer Is it ok to link to your documentation project yet? I would propose to add it under a new section called "community documentation", what do you think?

@grossbart I'm currently drafting the first part, once it is in a presentable form, I'd open a PR to add it!

@tstelzer Sure, I removed the link for now!


Nebra Hotspot not found on onboarding server error

New Nebra hotspot.  Get hotspot not found on onboarding server when trying to connect.

Please send a mail to<EMAIL_ADDRESS>


Fix Validators of RPC implementation in Ostracon

Description
Closes: https://github.com/line/lbm-sdk/issues/703
Here are the bugs:

On RPC

Validators can't use LoadVoters since it needs VoterParams and LastProof. But Validators supports latestUncommittedHeight. It doesn't save VoterParams and LastProof for that.

https://github.com/line/ostracon/blob/v1.0.6/rpc/core/consensus.go#L21
https://github.com/line/ostracon/blob/v1.0.6/rpc/core/consensus.go#L26
https://github.com/line/ostracon/blob/v1.0.6/state/store.go#L182-L185
https://github.com/line/ostracon/blob/v1.0.6/state/store.go#L193-L201

Voters can't support latestUncommittedHeight

https://github.com/line/ostracon/blob/v1.0.6/rpc/core/consensus.go#L70

On LightClient

LightBlock shouldn't use Validators of RPC for getting ValidatorSet and VoterSet

https://github.com/line/ostracon/blob/v1.0.6/light/provider/http/http.go#L128

LightBlock doesn't carefully validate related ValidatorSet

https://github.com/line/ostracon/blob/v1.0.6/types/light.go#L22
https://github.com/line/ostracon/blob/v1.0.6/types/block.go#L400

Solutions

On RPC:

Add ValidatorsWIthVoters: https://github.com/line/ostracon/pull/478/commits/9f48fe93e56403c24f0051cd86a2e4a2b570db2e
Fix Voters: https://github.com/line/ostracon/pull/478/commits/a240b3172d1581317320336bce046eadd1aaf358
Fix Validators: https://github.com/line/ostracon/pull/478/commits/a4a30202def49e39bb60985395e6985e638fcbaf

On LightClient:

Fix LightBlock: https://github.com/line/ostracon/pull/478/commits/e5296c5d74bbec5bb7557c5ca04c8cfbf8aa1d45

LigthBlock:

Add the validation: https://github.com/line/ostracon/pull/478/commits/065d0486551da3eecd41e4bfdf71c838e18b1fcc
Remove unnecessary calling: https://github.com/line/ostracon/pull/478/commits/d2eebe66115bced9cb503fcd6cf78784a3579148

In addition
Some of the changes are duplicates updated by the other PRs

https://github.com/line/ostracon/pull/513
https://github.com/line/ostracon/pull/510
https://github.com/line/ostracon/pull/504

Is the cause of this issue the inability to get voters in latest uncommitted height?

Yes, it is.

I recognize that latest uncommitted height is current committing block height.

No, it isn't. Basically, storing the Validators of the state is the current committing block height. And storing the NextValidators of the state is the uncommitted block height.

In this height we should get voters for consensus.

If we can get Voters from DB, we can get it with the current committing block height.
The issue of https://github.com/line/lbm-sdk/issues/703 happened with the uncommitted block height.