nnf-dm + clientmountd cpu usage

We've observed that the nnf-dm and clientmountd daemons generate a surprising amount of system noise on the computes even when they should be idle.   For reference, over the last 2 weeks they've been lightly used yet have wracked up ~10 minutes of cpu time each.  This is compared to most other idle system daemons which report <5 seconds of cpu usage over the same time period.  The usage nnf-dm and clientmountd usage is similar across compute nodes.
root       33954       1  0 Nov14 ?        00:09:44 /usr/bin/nnf-dm <...args...>
root       34061       1  0 Nov14 ?        00:10:11 /usr/bin/clientmountd <...args...>
                                           ^^^^^^^^^

Corosync which is required for gfs2 generates even more noise on the compute nodes.  One possible mitigation would be to only start/stop the pacemaker service on computes when a gfs2 filesystem has been requested.  This could either be done by Flux when setting up the computes or the clientmountd which is already running there.

We've been investigating this. Early signs point to Go Garbage Collection and HTTP2 Health Checks. Both can be tuned.

@behlendorf which version of go are you using the build the daemons?

We're building with the RHEL 8.9 version of go.
# rpm -q golang
golang-1.20.10-1.module+el8.9.0+20382+04f7fe80.x86_64

# go version
go version go1.20.10 linux/amd64

We have found that we can reduce the CPU usage by approximately 66% by tuning garbage collection and the frequency of the HTTP2 health checks. In comparison of your original observations of 10m of CPU time over 14 days, we were able to get CPU usage down to 3m26s over 12 days (22-Dec to 02-Jan). This can be done by setting the following environment variables in the systemd unit file:
Environment=GOGC=off
Environment=GOMEMLIMIT=20MiB
Environment=GOMAXPROCS=5

Output of systemctl stauts with CPU accounting enabled:
[root@x9000c3s0b0n0 ~]# systemctl status clientmountd
● clientmountd.service - Data Workflow Service (DWS) Client Mount Service
   Loaded: loaded (/etc/systemd/system/clientmountd.service; enabled; vendor preset: disabled)
  Drop-In: /etc/systemd/system/clientmountd.service.d
           └─override.conf
   Active: active (running) since Wed 2023-12-20 13:24:23 CST; 1 weeks 5 days ago
 Main PID: 7840 (clientmountd)
    Tasks: 10 (limit: 1646821)
   Memory: 19.0M
      CPU: 3min 25.808s

The current solution is to start/stop these daemons at will. Flux will do that: https://github.com/flux-framework/flux-coral2/issues/166


[Bug]: Exported PDFs are neither searchable nor selectable

Bug Description
When I use the built in "Export to PDF" function with Shimmering Focus set as the theme, the exported PDF neither has selectable text nor searchable content. With any other theme I tried, this is not the case. Also the size of the created PDF file is 10x bigger than the one created with the default theme.
I tried analyzing the PDF with poppler's pdfinfo. This is what it returns on exactly the same exported note when the Shimmering Focus theme is enabled:
Creator:        Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) obsidian/1.0.3 Chrome/100.0.4896.160 Electron/18.3.5 Safari/537.36
Producer:       Skia/PDF m100
CreationDate:   Tue Apr 25 16:12:20 2023 CEST
ModDate:        Tue Apr 25 16:12:20 2023 CEST
Tagged:         no
UserProperties: no
Suspects:       no
Form:           none
JavaScript:     no
Pages:          1
Encrypted:      no
Page size:      594.96 x 841.92 pts (A4)
Page rot:       0
File size:      367473 bytes
Optimized:      no
PDF version:    1.4

and this is the output with the default theme (or any other theme I tried:
Creator:        Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) obsidian/1.0.3 Chrome/100.0.4896.160 Electron/18.3.5 Safari/537.36
Producer:       Skia/PDF m100
CreationDate:   Tue Apr 25 16:21:11 2023 CEST
ModDate:        Tue Apr 25 16:21:11 2023 CEST
Tagged:         no
UserProperties: no
Suspects:       no
Form:           none
JavaScript:     no
Pages:          1
Encrypted:      no
Page size:      594.96 x 841.92 pts (A4)
Page rot:       0
File size:      31190 bytes
Optimized:      no
PDF version:    1.4

Relevant Screenshot
If required I can provide the PDFs.
To Reproduce

Create a simple note
From the file menu choose "Export to PDF"
Open the created PDF
try searching in it.

Sample Text
No response
Obsidian Version
1.1.16
Installer Version
1.0.3
In which mode(s) does the problem occur? (if applicable)

[ ] Source (Edit)
[ ] Reading (Preview)
[ ] Live Preview

Which operating system?

[ ] macOS
[ ] Linux
[X] Windows
[ ] iOS
[ ] Android

Checklist

[X] I updated to the latest version of the theme (Settings → Appearance → Check for updates).
[X] If your installer version is much lower your app version, you should update it by re-downloading Obsidian from the Obsidian website. The installer version can be seen in the Obsidian settings (About → App → Installer Version).
[X] If the bug occurred after updating the theme, I read the last couple of entries in the changelog to make sure the bug is not an intentional change. (In that case, there will most likely be a style setting to turn it off again.)
[X] I made sure that the issue is related to this theme by checking that the problem does not occur with the default theme.
[X] In case I am missing an UI element, I have checked that it cannot be re-enabled with the Style Settings plugin.
[X] My issue concerns Obsidian version 1.0+ and I am not using the Legacy Editor (CM5). (Older Obsidian versions and the Legacy Editor are not supported by this theme anymore.)

Sorry, pdf export options are unfortunately not something that I plan to do. Reason being that I do not use PDF exports, and at the same time they are pretty time intensive to implement.
In case you find a solution, I'll gladly add it to the theme, but given all the other issues for this theme (and my other repos), PDF exports is unlikely something I have the time resources to implement on my own, sorry!

I actually accidentally stumbled upon the cause of this issue. should be fixed in the latest update


Formik component doesn't support mapPropsToValues prop.

Current Behavior
Using Formik component render prop doesn't execute mapPropsToValues
Desired Behavior
Formik component render props should expose the same functionality as the withFormik HOC
Suggested Solution
Adding mapValuesToProps to Formik component render prop
Who does this impact? Who is this for?
All users
Describe alternatives you've considered
using withFormik, although in my opinion this is somewhat a more intrusive implementation of Formik (adding props to parent component propTypes)

If you look at the Readme example of Formik component, there's a note:

The benefit of the render prop approach is that you have full access to React's state, props, and composition model. Thus there is no need to map outer props to values...you can just set the initial values, and if they depend on props / state
then--boom--you can directly access to props / state.

If you have a component that wraps
<Formik../>  then you can just initialize initialValues with your props. Along with enableReinitialize={true} you can update the Form with new props...

@cdock1029 Thanks!

I think this is related to a problem I'm having, I'm trying to read prop values that I'm passing in from a parent, which is then used with a react-select component that has similar code to https://codesandbox.io/s/73jj9zom96. I am able to read the props, but then my react-select component doesn't work at all.
``
class Post1 extends React.Component {
render() {
const { classes } = this.props;
return (

<Formik
enableReinitialize={true}
initialValues={{
category: this.props.initValues.category,
type: [],
}}
...
``
If I use:
          mapPropsToValues={props => ({              category: this.props.initValues.category,              type: [],            })}
instead of enableReinitialize and initialValues, my initial props don't get read at all. Could someone point me in the right direction?

@jaredpalmer  Using enableReinitialize is abruptly updating the form values and causing a rerender on the component. For example a dropdown component is rerendered upon selecting a value and if initialValues is updated due to this onChange event. I had to fix this by switching back to withFormik. As I never really needed to use Formik component, it worked for me. It would be nice to have mapPropsToValues.


fix: fix completion for pkg alias name

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y #747

2. What is the scope of this PR (e.g. component or file name):
lsp/completion.rs

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):
complete pkg alias name if not none instead of pkg name

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

36 of 38 (94.74%)  changed or added relevant lines in 2 files are covered.
241 unchanged lines in 8 files lost coverage.
Overall coverage decreased (-15.1%) to 72.692%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kclvm/tools/src/LSP/src/completion.rs
33
35
94.29%

Files with Coverage Reduction
New Missed Lines
%

compiler_base/error/src/diagnostic/diagnostic_message.rs
3
0.0%

compiler_base/error/src/diagnostic/components.rs
6
0.0%

compiler_base/error/src/errors.rs
8
0.0%

compiler_base/error/src/diagnostic/diagnostic_handler.rs
11
0.0%

compiler_base/session/src/lib.rs
26
58.06%

compiler_base/error/src/diagnostic/mod.rs
27
0.0%

compiler_base/error/src/diagnostic/style.rs
41
0.0%

compiler_base/error/src/emitter.rs
119
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-15.1%

Covered Lines:
42370

Relevant Lines:
58287

💛  - Coveralls


Using it without a keyboard not working

Bug Description
maybe I am missing something there but I cannot find way to use the plug-in on my tablet without a keyboard. I cannot make text editable.
Expected Behaviour

Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

@injainja Sorry to hear you're experiencing an issue. Hopefully, we can resolve it.

What type of tablet device are you on?
What browser (and version) are you using?

I am having this issue with all mobile devices I own and all browsers I use. E.g. LGV30 with chrome:
I tap into a text box which selects the box itself. But there is no way to make the text editable - neither via a second, nor a double tap.

Hello @injainja! Thank you for your patience in waiting for a response. Mobile phone support is out of scope, but tablet support is on the roadmap. We are hoping to work on this, so please feel free to reference GitHub issue #1686 for the latest information.

Thank you. Tablet will suffice 🤗


Post-jam goals

Hi visitors!
This project was made for the GameOff 2021 event. However, the end of that jam (in 2 days) won't mean the death of this project! :smiley: Those who rated this game and commented here and here gave useful feedback. Thanks in advance to those who took the risk to try this game! :heart:
I found some important issues/improvements (starting with the creepy music! :confused:) basing on those comments. Also, since I'm the owner and played lots of times, I noticed more improvements by myself.
To keep everything in the same place, I made a project board to track the goals we want to cover after the jam ends: https://github.com/DiddiLeija/diddi-and-the-bugs/projects/1
Feel free to comment here if you want more improvements soon :wink:

@HarshNarayanJha, I think these goals can help you if you want to contribute after the jam :) Also, if you wanna add other goals, feel free to tell me here!

@HarshNarayanJha, I think these goals can help you if you want to contribute after the jam :) Also, if you wanna add other goals, feel free to tell me here!

Sure, ready to help.

Sure, ready to help.

Ok. I'll cut a 1.0.0 (see #8). After that, contributions will be fully open :)

Ok. I'll cut a 1.0.0 (see #8) release. After that, contributions will be fully open :)

I released a 1.0.0 release a few minutes ago. Feel free to send your contributions!

Closing this, since we already cutter post-jam releases. I will close https://github.com/DiddiLeija/diddi-and-the-bugs/projects/1 soon, too.


[BUG] SkiaSharp 2.88.8 (Xamarin) crashes randomly with Mali GPUs

Description
Even after upgrading to 2.88.8 (Xamarin Android), we are still seeing the following crash with Mali GPUs (from Google Play Console):
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
pid: 0, tid: 21941 >>> com.soundmindentertainment.gnollhack <<<

backtrace:
  #00  pc 0x000000000004ee70  /apex/com.android.runtime/lib64/bionic/libc.so (__memcpy+192)
  #01  pc 0x0000000000762b40  /vendor/lib64/egl/mt6789/libGLES_mali.so
  #02  pc 0x000000000078b718  /vendor/lib64/egl/mt6789/libGLES_mali.so
  #03  pc 0x000000000076aa90  /vendor/lib64/egl/mt6789/libGLES_mali.so
  #04  pc 0x0000000000707d34  /vendor/lib64/egl/mt6789/libGLES_mali.so
  #05  pc 0x000000000029e450  /data/app/~~SsY_xaj2CQJm0UOb0lSueQ==/com.soundmindentertainment.gnollhack-ce759vY1qUNsToHUXChJuA==/split_config.arm64_v8a.apk!libSkiaSharp.so

Only happens with SKGLView.
Code
https://github.com/hyvanmielenpelit/GnollHack
Expected Behavior
SkiaSharp and SKGLView do not crash with Mali GPUs.
Actual Behavior
On Android, SkiaSharp crashes randomly when SKGLView is used with Mali GPUs. (But seems a bit rarer than with 2.80.3, which we used before.)
Version of SkiaSharp
2.88.3 (Current)
Last Known Good Version of SkiaSharp
Other (Please indicate in the description)
IDE / Editor
Visual Studio (Windows)
Platform / Operating System
Android
Platform / Operating System Version
Android 13, as reported by Google Play Console. Also, a couple of Android 14 phones.
Devices
Motorola Vicky (moto g72), as reported by Google Play Console. Also, a couple of Google Tensor / Pixel phones.
Relevant Screenshots
No response
Relevant Log Output
No response
Code of Conduct

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

Are you able to reproduce this crash at all? And maybe try SkiaSharp 3.x?
Not much to go on there, but if the crash is in the GLES library, could it be some Android issue?
Are you able to get the symbol in libSkiaSharp.so that is the start of the crash?

I actually happen to have Google Pixel 6a, which has Mali GPU, and I managed to get it crash sometimes with SkiaSharp 2.80.3, so there's a fair chance that I can get it crash also with 2.88.8. The game used to crash when zooming out to minimap scale, so I can try that again.  (In minimap zoom scale, bitmaps get scaled very small in the game so that you can see the entire map.) My experience though is that the crash in minimap zoom happens much less frequently these than in 2.80.3, so I was expecting the bug to have been fixed, but apparently not entirely so.
How do I get the symbol(s) from libSkiaSharp.so normally? Should I get Android logs or is there a way to get further debug data e.g. in debug configuration using SkiaSharp? Tommi has been playing around with building SkiaSharp, too, but getting a custom SkiaSharp in the game could be a bit much, but in theory might be possible.
It is obviously some Android / OpenGL / Mali driver issue, or some kind of an incompatibility between those.
SkiaSharp 3.0x does not seem to be available on Xamarin. On that note, we are still stuck with Xamarin because of the iOS memory corruption problem mentioned https://github.com/mono/SkiaSharp/issues/2840 which is the most serious problem we have encountered with SkiaSharp so far. My guess is that that corruption problem may have something to do a threading problem / incompatibility between .NET MAUI layout cycle (perhaps works now differently from Xamarin) and SKGLView rendering cycle (which seems to be oddly slow like it would cause .NET layout cycle to wait for OpenGL to finish drawing or the like, see https://github.com/mono/SkiaSharp/issues/2842). I suspect that because the corruption starts to happen after I open many game menus / pages / layouts with SKGLViews (say, 40 times), and after that suddenly things break down with SkiaSharp.

 At least the function names seem to be now there, and this is 2.88.8 for sure.
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
pid: 0, tid: 5047 >>> com.soundmindentertainment.gnollhack <<<

backtrace:
  #00  pc 0x0000000000057370  /apex/com.android.runtime/lib64/bionic/libc.so (__memcpy_aarch64_simd+304)
  #01  pc 0x0000000000948e80  /vendor/lib64/egl/libGLES_mali.so (gles_vertexp_copy_client_buffers+112)
  #02  pc 0x0000000000974268  /vendor/lib64/egl/libGLES_mali.so (gles_vertex_prepare_nx+1256)
  #03  pc 0x00000000009506a0  /vendor/lib64/egl/libGLES_mali.so (gles_drawp_draw_common+992)
  #04  pc 0x00000000008f4380  /vendor/lib64/egl/libGLES_mali.so (gles2_draw_draw_range_elements+80)
  #05  pc 0x000000000029e450  /data/app/~~m_pX_fPRZKRV0SSeX4pqzA==/com.soundmindentertainment.gnollhack-yqHSB1HKmOlCuV-gHBJD5A==/split_config.arm64_v8a.apk!libSkiaSharp.so


Orbiter Finance exploit

Name
Orbiter Finance exploit
Description
Discord account hacked. A scam website is being placed under announcements.
Addresses
0xd7d1D6692E0612A632A78fAb5a82E9c767d518bF
0x9fA7bB759641FCd37fe4aE41f725e0f653f2C726
Meta Article URLs
https://twitter.com/Orbiter_Finance/status/1664141758241050627?s=20
https://etherscan.io/address/0x9fA7bB759641FCd37fe4aE41f725e0f653f2C726

Hi, thank you for reporting these addresses. This falls under a "phishing scam" though, which we don't include in this repository.


slicereg enforces boundaries

ugly example below:

since we cannot use a gap, we should mirror the data.

problem actually comes from the smoothing kernel, which should be larger, otherwise the "search" area is not large enough for these large deformations.


enhancement idea - implement minimum file size

Idea extracted from #4

Right now the only minimum file size support present is:
https://github.com/michaelkrisper/duplicate-file-finder/blob/master/duplicatefilefinder.py#L121
I.e. 1 or more bytes.
include_empty option could be implemented by new min-filesize


Website look weird, no styles loading.

Website looks weird, no styles loading.

@kevinfilteau I'm not seeing this. It's useful to include browser versions etc, and possibly check if your network panel is loading the CSS correctly.
Either that or it is resolved now.

I've tested this in multiple different browsers and can not replicate the issue. If you are still seeing this issue, could you please post a screenshot and the browser version you are using.

Hi,
I tested with Safari, Chrome and Brave and Safari on my iPhone, same issues
everywhere
[image: Screen Shot 2020-06-05 at 09.41.37.png]
—
Kevin Filteau vous guide vers l'autogestion de votre commerce en ligne.
418-254-7193 de 10h à 16h du lundi au jeudi
<EMAIL_ADDRESS>Si vous désirez ne plus recevoir de courriel de ma part, demander ou faire
un changement à vos informations personnelles ou formuler une plainte, veuillez
immédiatement m'en informer par courriel.
On Fri, Jun 5, 2020 at 9:38 AM Jake Champion<EMAIL_ADDRESS>wrote:

I've tested this in multiple different browsers and can not replicate the
issue. If you are still seeing this issue, could you please post a
screenshot and the browser version you are using.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Financial-Times/polyfill-service/issues/2251#issuecomment-639485452,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKELF5DND7FFXK4RWKXMBTRVDYNRANCNFSM4NNNEMLA
.

@kevinfilteau the screenshot didn't work, sorry.
Can you check your network tab in the browser and see if there are errors. I feel like this might just be you.

https://www.dropbox.com/s/dsobidmwhvzrljk/Screen Shot 2020-06-05 at 09.41.37.png?dl=0
—
Kevin Filteau vous guide vers l'autogestion de votre commerce en ligne.
418-254-7193 de 10h à 16h du lundi au jeudi
<EMAIL_ADDRESS>Si vous désirez ne plus recevoir de courriel de ma part, demander ou faire
un changement à vos informations personnelles ou formuler une plainte, veuillez
immédiatement m'en informer par courriel.
On Fri, Jun 5, 2020 at 9:57 AM Antony Jones<EMAIL_ADDRESS>wrote:

@kevinfilteau https://github.com/kevinfilteau the screenshot didn't
work, sorry.
Can you check your network tab in the browser and see if there are errors.
I feel like this might just be you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Financial-Times/polyfill-service/issues/2251#issuecomment-639497868,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKELFZGJKFBODUJFASVQMDRVD2UDANCNFSM4NNNEMLA
.

Back online : )
—
Kevin Filteau vous guide vers l'autogestion de votre commerce en ligne.
418-254-7193 de 10h à 16h du lundi au jeudi
<EMAIL_ADDRESS>Si vous désirez ne plus recevoir de courriel de ma part, demander ou faire
un changement à vos informations personnelles ou formuler une plainte, veuillez
immédiatement m'en informer par courriel.
On Fri, Jun 5, 2020 at 10:40 AM Jake Champion<EMAIL_ADDRESS>wrote:

Reopened #2251
https://github.com/Financial-Times/polyfill-service/issues/2251.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Financial-Times/polyfill-service/issues/2251#event-3411651496,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKELF6ICEGUBWGQG7UCCKTRVD7VVANCNFSM4NNNEMLA
.

Hate to be the bearer of bad news, but it's happening to me now, on Chrome latest.
Seems to be a caching / deployment issue:

Looks like a few refreshes bring it back - but it's a totally different CSS hash.
I think there's some overly-aggressive caching going on for index.html


Knowledge Retrieving

User story
As a group G,
I want to decide which tool to use to extract knowledge graphs from natural language,
So that I can use that information to prompt my language model.
Acceptance Criteria

Find a tool that can extract keywords so data can be found in a knowledge graph

Ex. spaCy

In the attached photo, this issue would be the "Knowledge Retriever"
Make sure that no tool breaches the Nordjyske NDA
Has to understand Danish
Document pros and cons for each tool

Resources
Llama 2 installation + test
Notes

~The Main tool in contention is the Llama 2 LLM (implementation being tested)~
Upon reflection, Spacy seems like the best tool to use as knowledge retriever
SpaCy is showing decent entity recognition, however it might not be strong enough to recognise entities in data which it is not trained on. fx. Nordjyske articles.

Review 1 af Nichlas - Review 2 af Thomas

Looks good @barth010may, needs testing before merging and done.


FR: UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: Cannot set property 'id' of null

Hello,
So I'm try to call the key in the snapshot but I get this error:
UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: Cannot set property 'id' of null
I'm using firebase-admin: 4.2.1 and nodejs 6.9.1, the firebase product is storage, I'm using MacOx and my code is:
var db = admin.database().ref('events/-KlGpV1rBu9iACpwdluZ').orderByKey(),
        deferred = Q.defer(),
        childData = null,
        info = [];

    db.once("value").then(function(snapshot) {
        snapshot.forEach(function(childSnapshot) {
            console.log('key', childSnapshot.key);
            console.log('val', childSnapshot.val());

            info.push(childSnapshot.key);
        });

        deferred.resolve(info);
    });

    return deferred.promise;

I have been reading all documentation about the key and I didn't found any issue in the code. Is possible that the problem is that the documentation is not updated or has issues?
Thanks

I think this has something to do with your use of Q. It doesn't seem to be a problem with the Firebase database client. I'm testing with the following bit of code:
var db = admin.database().ref('events').orderByKey();
db.once("value").then(function(snapshot) {
        snapshot.forEach(function(childSnapshot) {
            console.log('key', childSnapshot.key);
            console.log('val', childSnapshot.val());
        });
});

My database has the following nodes:
+- events
   `----- k1: "v1"
    |---- k2: "v2"

I get the output:
key k1
val v1
key k2
val v2

Seems all right to me.

I tried with the Q library as well, and it seems to work fine.
var db = admin.database().ref('events2').orderByKey();
var deferred = Q.defer();
var info = [];
db.once("value").then(function(snapshot) {
        snapshot.forEach(function(childSnapshot) {
            console.log('key', childSnapshot.key);
            console.log('val', childSnapshot.val());
            info.push(childSnapshot.key);
        });

        deferred.resolve(info);
}, function(error){
  deferred.reject(error);
});

deferred.promise.then(function(data){
  console.log(data);
});

Output:
key k1
val v1
key k2
val v2
[ 'k1', 'k2' ]

I will close this ticket, since I cannot seem to find a bug here in the Admin SDK code. Please feel free to reopen with more information if you continue to run into problems.


Opening a grib file leaves a .idx in the directory

What is your issue?
I don't remember this being an issue before, but when I open a .grib2 file a .idx file remains in the directory when the script is finished running. I'm using:
with xr.open_dataset(filepath) as ds:
    ...

I'm using xarray version 0.19.0 on python version 3.8.13

Please could you confirm on the latest release of xarray, include the xr.show_versions(), and a MCVE?

here is the xr.show_versions()
>>> xr.show_versions()

INSTALLED VERSIONS
------------------
commit: None
python: 3.8.13 | packaged by conda-forge | (default, Mar 25 2022, 06:06:49)
[Clang 12.0.1 ]
python-bits: 64
OS: Darwin
OS-release: 21.4.0
machine: x86_64
processor: i386
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: ('en_US', 'UTF-8')
libhdf5: 1.12.1
libnetcdf: 4.8.1

xarray: 0.19.0
pandas: 1.3.2
numpy: 1.20.3
scipy: 1.7.1
netCDF4: 1.5.7
pydap: installed
h5netcdf: 1.0.0
h5py: 3.6.0
Nio: None
zarr: 2.11.3
cftime: 1.5.1
nc_time_axis: None
PseudoNetCDF: None
rasterio: 1.2.9
cfgrib: <IP_ADDRESS>
iris: None
bottleneck: None
dask: 2021.10.0
distributed: 2021.10.0
matplotlib: 3.4.3
cartopy: 0.20.0
seaborn: None
numbagg: None
pint: None
setuptools: 52.0.0.post20210125
pip: 21.0.1
conda: None
pytest: None
IPython: 8.1.1
sphinx: None

Here's 2022.grib.zip a file you can use for testing. And I believe this code should satisfy the MCVE request
import xarray as xr

fp = '2022.grib'

with xr.open_dataset(fp, backend_kwargs={'filter_by_keys': {'shortName': 'i10fg'}}) as ds:
    m = ds.sel(time=ds.time.dt.month.isin([3]))

this code produced this 2022.grib.02ccc.idx.zip file and it didn't delete it once the code finished running.
Please let me know if you need any more info!

I can repro, thanks @isodrosotherm .
It looks like we're passing this through to cfgrib.open_file: https://github.com/pydata/xarray/blob/main/xarray/backends/cfgrib_.py#L118.
Is this an xarray issue though? Or should upstream be cleaning up that file, assuming it should be cleaned up?

cc @alexamici

You could point the index-file location to a temporary-folder, something along the lines of https://github.com/ecmwf/cfgrib/issues/275#issuecomment-1010002087.

@isodrosotherm setting indexpath=='' will suppress the use of the external index file.

Should we by default use a temp file? or is there some benefit to retaining the index file locally? (asking as someone with very little context{

Should we by default use a temp file? or is there some benefit to retaining the index file locally? (asking as someone with very little context

Opening the a GRIB for the first time requires reading the whole file and saving the index to disk makes all subsequent open much faster. I took the decision to default to writing, but IMO the real issue is that it is not well documented how to switch it off 😅

Cool, if you think we should document it better (or write to a temp file by default, less surprising perhaps?), then that's great, let's add the issue; it doesn't have to be you who implements.

@isodrosotherm setting indexpath='' will suppress the use of the external index file.

Unfortunately, even with this argument, the following command seems to be looking for the indexfile? I am unable to read the file for the second time.
I get the following error:
raise EOFError("No valid message found in file: %r" % self.path)
EOFError: No valid message found in file:

My versions are cfgrib==<IP_ADDRESS> and xarray==0.16.2.
I am not using the latest version as I found here that the newer combination has this issue. Kindly advise.
ds_sea = xr.open_dataset(
            filename,
            engine="cfgrib",
            backend_kwargs={
                "filter_by_keys": {"typeOfLevel": "meanSea"},
                "indexpath": "",
            },
        )

I confirm having bumped into this issue of idx file not playing nicely. I ended up adding a little function to remove the idx before doing any new downloads.
In particular what happened to me was something slightly different.
It happened to me when I was crawling through GFS files, those grib files don't have the date in the file name but the date is in the folder  (i.e. https://ftpprd.ncep.noaa.gov/data/nccf/com/gfs/prod/gfs.20230923/00/atmos/). so I can have multiple files named gfs.t00z.pgrb2.0p25.f058 coming from different dates.
My workflow was: downloading - opening grib (either xarray or cfgrib)- compute whatever - delete file. When I then moved to the next date,  the idx file that was present in the folder would not play nicely. Sometimes even if I used filterbykeys and the shortname I'd get the wrong variable. it was very intermittent and not consistent. It was reusing the idx file because it was there and the idx file was wrong because it had been generated for another file.
Just reporting it as a FYI. Since probably my behaviour was expected (since the idx file had been generated based on another grib file), but I am reporting it since it took me a bit to figure it out.

I confirm having bumped into this issue of idx file not playing nicely. I ended up adding a little function to remove the idx before doing any new downloads.
In particular what happened to me was something slightly different.
It happened to me when I was crawling through GFS files, those grib files don't have the date in the file name but the date is in the folder (i.e. https://ftpprd.ncep.noaa.gov/data/nccf/com/gfs/prod/gfs.20230923/00/atmos/). so I can have multiple files named gfs.t00z.pgrb2.0p25.f058 coming from different dates.
My workflow was: downloading - opening grib (either xarray or cfgrib)- compute whatever - delete file. When I then moved to the next date, the idx file that was present in the folder would not play nicely. Sometimes even if I used filterbykeys and the shortname I'd get the wrong variable. it was very intermittent and not consistent. It was reusing the idx file because it was there and the idx file was wrong because it had been generated for another file.
Just reporting it as a FYI. Since probably my behaviour was expected (since the idx file had been generated based on another grib file), but I am reporting it since it took me a bit to figure it out.

How did you get around to solve this? If you have to re-open the same file without any issues?

This is a cfgrib issue that we can't do much about.


The ogg plugin is missing from gstreamer in the new IOS toolchain

IOS emulator log:
...
[WARNING           ] [Audio       ] Unable to find a loader for <./lib/CPData/game/marking/good.ogg>
....
The ogg in question is a proper ogg which plays correctly in Linux, Windows and Android (kivy launcher).
If you need more info just let me know, I have a fresh Imac 10.10.2 set aside for Kivy IOS building.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/9306278-the-ogg-plugin-is-missing-from-gstreamer-in-the-new-ios-toolchain?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github).

I'll close this one as it's not under the control of Kivy itself, but more into the toolchain project. Thanks for reporting it!


Fixed bug in metadata where \n isn't recognized as a new line.

The PR fulfills these requirements: (check all the apply)

[x] It's submitted to the main branch.
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. feat: Add a button #xxx, where "xxx" is the issue number).
[x] When resolving a specific issue, the PR description includes Closes #xxx, where "xxx" is the issue number.
[x] If changes were made to ui folder, unit tests (make test) still pass.
[ ] New/updated tests are included

Hello @MattiasOz. Can you provide a further explanation about what was the actual problem you solved?

I'm on arch and couldn't run any of the examples. The make all command would write
# Generated in hatch_build.py\n__platform__ = 'linux'\n__arch__ = 'amd64'
exactly like that. The \n wasn't recognized as a new line making the whole file a comment.

Makes sense, thanks for the clarification!

Let's see if the -e option does not break Ubuntu/Win e2e tests.

As expected, it doesn't work on any of the other mainstream OSes since -e is an unknown option.

Thus reverting.


Remove unused local variables

Current coverage is 64.26% (diff: 100%)

Merging #832 into master will increase coverage by 18.85%

@@             master       #832   diff @@
==========================================
  Files           727        727
  Lines         34116      34086     -30
  Methods           0          0
  Messages          0          0
  Branches       5218       5218
==========================================
+ Hits          15494      21907   +6413
+ Misses        17249      10628   -6621
- Partials       1373       1551    +178

Powered by Codecov. Last update 237c0a8...bfed4a1


disfetch: 2.13 -> 2.14

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/q60/disfetch/releases.
meta.description for disfetch is: "Yet another *nix distro fetching program, but less complex"
meta.homepage for disfetch is: "https://github.com/q60/disfetch"
meta.changelog for disfetch is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/1n37j5v87cwn7zq84lxaqi8f4wl9p7yg-disfetch-2.14/bin/disfetch passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 2.14 in filename of file in /nix/store/1n37j5v87cwn7zq84lxaqi8f4wl9p7yg-disfetch-2.14

directory tree listing: https://gist.github.com/28a86dd3c949453e6e9543a62d166198

du listing: https://gist.github.com/9961d049d6499dc4137429e13ed3ccfb

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
disfetch

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1n37j5v87cwn7zq84lxaqi8f4wl9p7yg-disfetch-2.14 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A disfetch https://github.com/r-ryantm/nixpkgs/archive/e369479c1411e0812fbcaf5ced9f194a6a0cecf2.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1n37j5v87cwn7zq84lxaqi8f4wl9p7yg-disfetch-2.14
ls -la /nix/store/1n37j5v87cwn7zq84lxaqi8f4wl9p7yg-disfetch-2.14/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

disfetch

Maintainer pings
cc @q60 for testing.

Result of nixpkgs-review pr 135499 run on x86_64-linux 1

1 package built:

disfetch


Remove unused class TxHelper

Collaborator

Will handle this separately, in order to keep the changes strictly related to this PR.

Rather than deleting unused files one by one can we please move them to resources/deleted and then delete them all at once? sometimes there is code that you want to review / reuse and it can be hard to find when you don't know exactly where it was or when it was deleted.

Will handle this separately, in order to keep the changes strictly related to this PR.


how to live longer with hair?

any idea?

less coding

any idea?

less coding

any idea?

less coding

no coding.

hhhhhfunny.Since be a programmer,don't think about it(bushi

How to live with longer hair?

I think baldness just a sign of sub-health. There is no direct relationship between baldness and coding.

Finasteride

Does this really help?

Hair transplant?

Why not let hair free?

Use hair serum. water your hair, shine enough sunlight.


Is it possible to inference using LGPMA model on cpu?

Hi,
I would like to inference table structure using LGPMA model on CPU memory.
Is it possible to do that?
Best regards 👍

I'm not sure if this model can be run on a CPU, but I think there's must be some engineering processes that need to be done to fit the cpu since the model is a little bit large.
The framework is totally based on the mmdetection, you might find some instruction from it.

Thank you for your detailed reply.
I managed to run LGPMA model on CPU after changing inference.py.

@setotaku0515  can you show me how did you change the inference.py to run it on CPU please?


add support for zdf.de

It would be nice to be able to play videos from https://www.zdf.de

@freddii The new version 5.12.0 (requires Firefox 88+) supports ZDF.


No Accounts for customers?

Hi,
Looks like there are no accounts feature for customers. So what customers stands in admin of shoppe where we can see customers list? Any way to integrate user accounts system in shoppe?
Thanks.

How do you mean? Is this similar to #272

Hi, I want the feature where users can create account so they can see their past orders, track their purchases easily, add their addresses. I need a way where users can  sign in & sign out. Quite similar to #272 .

Hi, any update on this? Can I use devise for users login & logout. I need an feature where users can log in and see their past order history etc.

The customer model is basic so you can modify it to how you want it
On 21 Sep 2015 10:12 am, "Sumeet Mahajan"<EMAIL_ADDRESS>wrote:

Hi, any update on this? Can I use devise for users login & logout. I need
an feature where users can log in and see their past order history etc.
—
Reply to this email directly or view it on GitHub
https://github.com/tryshoppe/shoppe/issues/273#issuecomment-141919875.

You could add devise to your app and then do the following

Create a User model with Devise with a customer_id field
Add belongs_to :customer, class: Shoppe::Customer to User model
Create a Customer record on signup and use that for orders

@deanperry
Thank you so much. I will add it. One more question, after following this - whenever an account will be created it will automatically shown in the admin panel?

Yep Customers are shown in the admin. Users you create with Devise won't be however

Guys, i dont know if this helps i have been able to do this, worked around extending the Shoppe::Customer Model witrh devise.
`require_dependency Shoppe::Engine.root.join('app', 'models', 'shoppe', 'customer').to_s
module Shoppe
class Customer < ActiveRecord::Base
devise :database_authenticatable, :registerable, :recoverable, :rememberable, :trackable, :validatable
end
end`
also the migration has been tweaked to change_table and not create_table
`class DeviseCreateShoppeCustomers < ActiveRecord::Migration
def change
change_table :shoppe_customers do |t|
## Database authenticatable
t.string :encrypted_password, null: false, default: ""
  ## Recoverable
  t.string   :reset_password_token
  t.datetime :reset_password_sent_at

  ## Rememberable
  t.datetime :remember_created_at

  ## Trackable
  t.integer  :sign_in_count, default: 0, null: false
  t.datetime :current_sign_in_at
  t.datetime :last_sign_in_at
  t.inet     :current_sign_in_ip
  t.inet     :last_sign_in_ip

  ## Confirmable
  t.string   :confirmation_token
  t.datetime :confirmed_at
  t.datetime :confirmation_sent_at
  t.string   :unconfirmed_email # Only if using reconfirmable

  ## Lockable
  # t.integer  :failed_attempts, default: 0, null: false # Only if lock strategy is :failed_attempts
  # t.string   :unlock_token # Only if unlock strategy is :email or :both
  # t.datetime :locked_at
end

add_index :shoppe_customers, :email,                unique: true
add_index :shoppe_customers, :reset_password_token, unique: true
# add_index :shoppe_customers, :confirmation_token,   unique: true
# add_index :shoppe_customers, :unlock_token,         unique: true

end
end
`
This seems to work for me ..

@samueladesoga : You also need to add this line in routes.rb
devise_for :shoppe_customers, class_name: Shoppe::Customer


Convert to use aws-sdk instead of right_aws

This converts all providers to use aws-sdk.  This allows S3 to use the iam-profile and opens up other options as aws-sdk supports a much larger range of aws services.

@cwebberOps How about merging this in? aws-sdk is the right thing, and I can see the folks are already starting to fork @ubiquitousthey for this feature. I'd be happy to discuss testing.

BTW, I have run all existing tests successfully with this branch.

What do I need to do to get this merged in?

https://aws.amazon.com/blogs/aws/now-available-version-2-of-the-aws-sdk-for-ruby/ <=- it's officially out of preview. We should update the version and merge.

Any progress on getting this merged?

:+1: The current right_aws gem is no longer maintained. Please merge support for aws_sdk

@someara Thanks!

Is there any update on its merge.
Still i am getting the same error while using updated version of "aws" cookbook.
But its working fine using aws sdk,
example
aws s3 --region us-east-1 cp s3://#{node['s3']['bucket_name']}#{node['s3']['path_of_file']} #{node['your_system_path']['filename']}


Suggestion: add Apollo Server config directive to typescript-resolvers plugin

I am using typescript-resolvers and I see #3805 talks about this a little, but I have a suggestion for better Apollo Server compatibility.
It seems both of these config directives are necessary:
   config
      useIndexSignature: true.
      defaultMapper: Partial<{T}>

The defaultMapper one is needed to support resolver chains since each resolver might only return part of the final result.
My suggestion is to add a first-class useApolloServer: true config setting that combines both of these. Or at least update the doc section about Apollo Server.
What do you think? Happy to send a PR for either one.

I will update the docs, thanks @benallfree :)

Done in 1.13.3


Release v0.6.4

Release v0.6.4

Making some final passes. Will tag for build in ~30 minutes.

Release is out, so I'm going to merge this now


Edit antibodies from the "my submissions" page

Clicking on an row on the "my submissions" table opens an editor rather than the viewer.

@bandrow @masonpairish Is there any workflow we should take into account here (e.g. change of state)?
May also deleting previously submitted antibodies be relevant?

@Filippo Ledda @.***> there are no antibodies that we
delete.
we do mark them rejected, but basically keep the record
The only use case that I can think of is if we get attacked by some spam
bot.
In those cases I can always just go into the back end to remove records, we
should not have a delete function.
On Tue, Oct 25, 2022 at 8:53 AM Filippo Ledda @.***>
wrote:

@bandrow https://github.com/bandrow @masonpairish
https://github.com/masonpairish Is there any workflow we should take
into account here (e.g. change of state)?
May also deleting previously submitted antibodies be relevant?
—
Reply to this email directly, view it on GitHub
https://github.com/MetaCell/scicrunch-antibody-registry/issues/116#issuecomment-1290782266,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNSWNPNE45Z67DNT2ZAHI3WE7673ANCNFSM6AAAAAARODVNWA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
All key biological entities deserve an #RRID!
orcid.org/0000-0002-5497-0243


Add sufficient HTTP2 ballerina tests

Description:
$subject
Describe your task(s)

[ ] Initially identify the failing test cases when listener version is 2.0
[ ] Fix failing cases and add HTTP2 tests

Following test cases were disabled due to missing features:

testHttp25XXHandleError
testHttp2InvalidUrlLength
testHttp2InvalidPayloadSize
testHttp2HateoasLinks1
testHttp2HateoasLinks2
testHttp2HateoasLinksInBody


Privacy Policy

Where can I find the privacy policy for SmoothScroll?

Why does a javascript library need a "privacy policy"?!?!?
I wold be a LOT worried if it could save any user data at all!


Release <IP_ADDRESS>

Waiting for: https://github.com/aiven/prometheus-exporter-plugin-for-opensearch/issues/109
Waiting for upstream: OpenSearch 2.4.0 has been released, see release notes.
Official OpenSearch release ticket: https://github.com/opensearch-project/opensearch-build/issues/2649
Checklist:

[ ] Compatibility Matrix is updated
[ ] Release <IP_ADDRESS> tag is present in the repo
[ ] Release artifacts are available and downloadable
[ ] Release notes published
[ ] Forum announcement published

Opensearch 2.4 is available now.

@cgoeller I was offline yesterday (public holidays). I am on it now.

Done.


Document the new config system

This should not be merged before the remaining code PRs, #430 and #431. This represents the end of the config work as outlined in #410.
Feedback on this is appreciated, I think it's all pretty straightforward but I've also been thinking about it for the past month.
This also includes a few formatting tweaks in frontend.md.

Does the file based approach also handle front-end specific settings? You mention font_face and font_size which look like front-end specific settings (they don't make sense for a TUI front-end for example). If yes then how does validation work in that case? How does the core know which front-end specific settings are valid?

@eyelash good Q. My thinking is that as a convenience we will offer a basic set of front-end settings, and a client that isn't interested in these can just ignore them.
If in the future we want to support custom client settings, we  could add a new 'client' domain, and the client can provide a template which we'd use for validation?

I've updated this minorly to remove mention of xdg and to mention that xi-core handles watching/reloading.


forX with variable declaration doesn't work since 0.3.1

There's a break change  for forX since<EMAIL_ADDRESS>below is the test case:
const Sval = require('sval')

const interpreter = new Sval({ ecmaVer: 8 })

interpreter.run(`
  const x = [1,2];
  for(var xx of x) console.log(xx)
  for(var xx in x) console.log(xx)
  exports.end = true
`)

module.exports = 'End: ' + interpreter.exports.end

Expected result:
1
2
0
1

Actual result:
undefined
undefined
undefined
undefined

While below test case works great:
const Sval = require('sval')

const interpreter = new Sval({ ecmaVer: 8 })

console.log(Sval.version);

interpreter.run(`
  const x = [1,2];
  var xx;
  for(xx of x) console.log(xx)
  for(xx in x) console.log(xx)
  exports.end = true
`)

module.exports = 'End: ' + interpreter.exports.end

It looks like forX deals with VariableDeclaration incorrectly.
And this new issue is caused when fixing this issue

I will be appreciated if you could pull requests for tests.


Compensate for setTimeout lag

Resolves #114
There will still be some tossup reading speed inaccuracy from setTimeout, but it should be much less than before. I have no method to test it, though.

Looks great!


wrong Link

The link in the section:
https://docs.microsoft.com/en-us/microsoft-365/compliance/office-365-service-encryption?view=o365-worldwide#customer-key named " See Enhancing transparency and control for Office 365 customers for the original announcement." redirects to a not existent site. The right one seems to be:
https://www.microsoft.com/en-us/microsoft-365/blog/2015/04/21/enhancing-transparency-and-control-for-office-365-customers/
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7f2f2581-292d-a72b-0683-897eaac0adb0
Version Independent ID: d10014a1-164d-1aea-635f-8c63da26b44c
Content: Service Encryption - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/office-365-service-encryption.md
Service: o365-seccomp
GitHub Login: @KCCross
Microsoft Alias: krowley

@J-a-k-o-b Thank you for your feedback. We have made changes tot he document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/4818
Thanks
Sri

@J-a-k-o-b I will close the issue, after PR is merged. Reopening.

@yogkumgit - I closed the related PR because the link is updated.


Does semeru's unique logo "enableGPU" have an impact on the client or server?

There is no guarantee that it'll send tasks to the GPU and, there are minimum requirements! (Also if you're serious about a server you likely don't have a GPU, as what's the point of it on a server)


is it still running

Hello, do you know if the code still runs for iOS 10 devices?

As far as I know the Nordic version  had to be updated for iOS 10. My assumption is that this one has to as well. However this  seems to be more lightweight  to than the Nordic SDK.
Did anyone try this out as yet?

I just got my first nRF52 board and would like to know if it compiles under SDK 12 ?

Does this project need makefile?
I cloned it, but I couldn't build it.

I was able to build using Eclipse for the NRF51822, key items were:-

Use NRF SDK version 9

Use softdevice S130 version 1

Before opening the project in eclipse, edit the .projects file to update the location of the SDK files

good luck

did you get this working?


bizarre zig compilation error on commented out Go code in zig src file

This is a good one, and quite funny  !
0.14.0-dev.823+624fa8523 on Mac intel. Don't know if Mac arm64 has the same issue yet.
src/datetime.zig:1007:2: error: expected statement, found 'invalid token'
//  us := rand.Int63n(31536000000000000)
same thing happens in web socket lib
handshake.zig:55:5: error: expected statement, found 'invalid token'
//  Connection: keep-alive, Upgrade
Not a problem with your code, but ... looks like its uncovered an interesting compiler bug.
At first I thought it might be the incompatible Go code in a comment tripping up sema, but on closer inspection, looks like its the presence of a colon in a comment.
Working on a patch to Zig to fix :)

Had https://github.com/karlseguin/websocket.zig/pull/40 on websocket.zig re tab in a comment this morning.

oh - its a tab in a comment ?
that's weird. ok thanks. Still a zig bug then - good catch
Yeah, can't reproduce with a simple test app with commented out Go code

https://github.com/ziglang/zig/pull/20885
https://github.com/ziglang/zig-spec/issues/38
This is the new reality I guess.


Update alpine/drush launcher.

Just some house keeping, needs testing.

Testing log:

download this PR in buildkit. Ran docker buildx bake to create all the images locally. (make bake failed, not sure why)
set up a new isle-site-template using the automatic method. Edit .env and change ISLANDORA_TAG from main to local. Also changed the site name to test-drush.
build the drupal container and start the containers using the commands in README.template.md.

Result:
drupal container failed and exited.
Note: I think the testing was legit despite me not knowing what i'm doing, because the image used by the drupal container, named islandora.io/test-drush:local includes, in its layers,  drush version 0.10.2.
The problem I think is that drush launcher doesn't work with drush 12, which is what i'm trying to install with the starter site because it's the only one that's not facing down EOL in the next few months.

That said, I re-tested with 0.8.0 and it works great. drush runs in the ide. So this is a good bit of housekeeping though the launcher part is still a problem. Mind if i merge?


Added zwave lock state from alarm type workaround

Description:
#17386 Did not seem to fix the issues I was experiencing with my YRD220. The access_control value was set to none on my lock when events were occurring. I noticed the alarm_type value was being update on my lock. This PR adds a workaround to set lock state from the alarm_type value.
This will also probably have a conflict with #18737. I can resolve the conflict, should that PR get accepted before this one. This PR is based on the workaround implementation of both of the aforementioned pull requests.
CC: @Juggler00
Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

@ahayworth Thanks for the input. I implemented 1, but I think implementing 2 as is would cause issues with my lock, since the codes in LOCK_STATUS are more inclusive. Some states would be dropped since they are not in ALARM_TYPE_STD. It would also prevent the _lock_status property from being set around 307.
The conflicts I imagine the 2 PRs to have are more with the setup of the workarounds, rather than their specific implementations. Specifically line 34 and probably a few more between lines 233 to 250ish.

Just wondering how you got HA to identify your "Vision" YRD220 as a lock? Looking through /lib/python3.6/site-packages/python_openzwave/ozw_configmanufacturer_specific.xml, the only references to both the YRD210 and 220 are:
<Manufacturer id="0129" name="Assa Abloy">   <Product type="0001" id="0000" name="Yale Touchscreen Lever (YRL220)" config="assa_abloy/TouchLever.xml"/>   <Product type="0002" id="0000" name="Yale Touchscreen Deadbolt (YRD220)" config="assa_abloy/TouchDeadbolt.xml" />   <Product type="0004" id="0000" name="Yale Push Button Deadbolt (YRD210)" config="assa_abloy/PushButtonDeadbolt.xml"/>   <Product type="0004" id="0209" name="Yale Push Button Deadbolt (YRD210)" config="assa_abloy/PushButtonDeadbolt.xml"/>   <Product type="0004" id="aa00" name="Yale Push Button Deadbolt (YRD210)" config="assa_abloy/PushButtonDeadbolt.xml"/>
Under the Vision section, there are no locks:
<Manufacturer id="0109" name="Vision">

@Juggler00 Honestly, I have no idea. I paired the locked with my Z-Wave network, and thats what the manufacturer was. I tried removing the node and adding it again, but it still showed up as a Vision device. If I recall, it didn't show up as a lock initially. I think each node communicates its capabilities to the controller on the initial pair sequence. I also know you need the network_key key in your configuration.yaml to work properly with secure devices. I also added the network key in options.xml as well. If you post what's in your zwcfg for your lock, I can add the workaround for your device.

@adrum Very odd... here is what is in my zwcfg for my lock:
<Node id="2" name="" location="" basic="4" generic="64" specific="3" type="Secure Keypad Door Lock" listening="false" frequentListening="true" beaming="true" routing="true" max_baud_rate="40000" version="4" secured="true" query_stage="Complete">
    <Manufacturer id="109" name="Vision">
        <Product type="4" id="0" name="Yale Push Button Deadbolt (YRD210)" />
    </Manufacturer>

Let me know if you need anything further or any testing done.

There are additional Kwikset IDs that have not been covered.
For instance the 914 Converts are ID 0446.   These had been working with @mtreinish workaround at one point.

@cdkonecny hmm, the conversion kit has as a separate id? Well, it's easy enough to add that to the workaround list so those locks are covered by the device state workaround. (maybe in a separate PR since it's sorta unrelated to what @adrum is fixing here)

Yes, I tested by adding ID 0446 to the list as a custom component and it
fixed them right up.  I apologize for not posting in the best spot.   The
more applicable thread had been shut down.
On December 6, 2018 9:41:56 AM Matthew Treinish<EMAIL_ADDRESS>wrote:

@cdkonecny hmm, the conversion kit has as a separate id? Well, it's easy
enough to add that to the workaround list so those locks are covered by the
device state workaround. (maybe in a separate PR since it's sorta unrelated
to what @adrum is fixing here)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@Juggler00 I believe your device matches mine based on your zwcfg, so feel free to pull this down and test it.

@turbokongen After resolving all the merge conflicts from #18737, I believe this PR is ready for review. I know you and @ahayworth were discussing a refactor, but I'm hoping this could get merged and released before the refactor happens.

I know you and @ahayworth were discussing a refactor, but I'm hoping this could get merged and released before the refactor happens.

I had volunteered for it but won't realistically have time to work on it until after the new year. Don't let it stop you. 😄

@turbokongen Any chance you can review this?


Add Option generate files for CodeChef too.

For CodeChef structure should be - Single folder with inputs and outputs line by line, since we have to upload test cases one by one there.

I will work on it.

Sure @agarwalrounak

@aashutoshrathi Is the issue still open for work? I would like to work on this.

@prithaupadhyay Go ahead!
I was supposed to work on it but couldn't do so due to other commitments.

Sure @prithaupadhyay Go ahead!
😉

@aashutoshrathi Thank You. Already started working 👍


have some way to configure the build URL

 Issue by bacongobbler
2017-07-01 17:03:29

Right now this is hardcoded to localhost:8080 and noted as a FIXME. We need to figure out a way to configure this redirect URL and/or intelligently set this redirect URL.
I was thinking one way we could figure this out is by following the installation guide:

From your project...

Click on the settings menu item (with the gear icon)
In the left navigation, click on Webhooks
On the Webhooks screen, click Add Webhook
Complete the form:

...
We can ask the user to run "Test Webhook" after this point, which then the system could store and infer its own inbound request URL via the webhook URL.

Comment by bacongobbler
2017-07-01 17:21:23

incoming PR.


Bug 2026063: Update buildah to v1.24.2

Bump buildah from v1.22.4 to v1.24.0, pulling in updates to its dependencies as well.  There are no known release-blocking changes.

/retest

/retest

/retest

@adambkaplan Should we mark this as bug 2026063?

Linking this to https://bugzilla.redhat.com/show_bug.cgi?id=2026063 - there's a very minor CVE fix in the bump.

/retest

bump @nalind - this PR has a merge conflict
but as @adambkaplan noted we'd like to knock off a CVE with this one

/retitle Bug 2026063: Update buildah to v1.24.1

/retest

/retest

/retest

/test e2e-aws-image-ecosystem

/retest

/retest

@nalind the failure in https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_builder/282/pull-ci-openshift-builder-master-e2e-aws-cgroupsv2/1496542538092253184 could be related to this PR .... the buildah bumped may have changed what the cgroup related tests in openshift/origin may need to look for.  Probably needs some investigation by you.
at least the last image eco failure was a clusteri install fail.
On the rhel7 job failure, I'm seeing this in the log:

TASK [openshift_node : Fail if not using RHEL8 beginning with version 4.10] ****
task path: /usr/share/ansible/openshift-ansible/roles/openshift_node/tasks/version_checks.yml:19
Wednesday 23 February 2022  18:37:35 +0000 (0:00:00.068)       0:00:05.493 ****
fatal: [ip-10-0-140-125.us-east-2.compute.internal]: FAILED! => {
    "changed": false,
    "msg": "As of v4.10, RHEL nodes must be at least version 8.4"
}
fatal: [ip-10-0-174-52.us-east-2.compute.internal]: FAILED! => {
    "changed": false,
    "msg": "As of v4.10, RHEL nodes must be at least version 8.4"
}
fatal: [ip-10-0-249-203.us-east-2.compute.internal]: FAILED! => {
    "changed": false,
    "msg": "As of v4.10, RHEL nodes must be at least version 8.4"

seems contradictory to enforce rhel8 in a rhel7 test.  Note, this entire rhel7 suite has been super flakey.  We may just need to skip that if we want this to merge, though @adambkaplan wanted this test in based on prior support experiences.
Probably something to discuss live in the next scrum / office hours

@nalind the failure in https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_builder/282/pull-ci-openshift-builder-master-e2e-aws-cgroupsv2/1496542538092253184 could be related to this PR .... the buildah bump may have changed what the cgroup related tests in openshift/origin may need to look for. Probably needs some investigation by you.

Opened https://github.com/openshift/origin/pull/26885.  It looks like the runc we're pulling in is setting up a new cgroup namespace for the RUN instruction, so there's no kubepods.slice component in the paths for the memory limits.

/retest


For Pub sub cluster using the io.lettuce.core.pubsub.PubSubCommandArgs, due which does not make use of hash and does not distribute the channels across shards

Bug Report
For Pub sub cluster using the io.lettuce.core.pubsub.PubSubCommandArgs,
And in this below overridden method, which always return null key
@Override
public ByteBuffer getFirstEncodedKey() {
return null;
}
due which does not make use of hash key and does not distribute the data(channel subscription) across the shards.
Current Behavior
ClusterDistributionChannelWriter -->
ByteBuffer encodedKey = args.getFirstEncodedKey(); // this gets called to identify the encoded key, however due to PubSubCommandArgs above behaviour its always return null and complete data(channel subscription) goes to one shard only.
Expected behavior/code
Pubsub should make use of CommandArgs instead of PubSubCommandArgs Or should provide a option to override this.
Environment

Lettuce version(s): [6.2.6.RELEASE]

Possible solutions
RedisPubSubReactiveCommandsImpl -- a option can be provided to pass CommandArgs while creating cluster client  as a client options.

Care to elaborate on what you're trying to achieve? Cluster Pub/Sub messages get replicated to each cluster node so there's no point in applying key slot hashing.
On the other side, shared Pub/Sub, would benefit from such routing, see #1984

So do we have any plan to implement #1984 ?

I don't know about you, but I have a plan to eventually implement the feature. Getting support from users and the community would help to speed up the process.

Intentionally for pubsub we have dropped slot hashing, did we face any performance issue for that ?

Yes, that is by intent. Routing Pub/Sub commands to the default connection avoids creating additional connections to cluster nodes.


chore: update sample owner config file

The sample config files do not work as they are without the new required fields (service_info_api_url and service_info_api_authentication), so incorporate them in the sample config.

@7flying can you rebase to main? I'll merge afterwards

Closing 'cause I'm unable to clean history. See #248


Dart SDK compatibility

What could be improved
Remove requirement for dart:ui and therefore the Flutter SDK
Why should this be improved
Currently this package/parser is only usable with the Flutter SDK. However I believe there are valid use-cases for the Dart SDK. For example I'm currently working on a CLI to optimize tmx to only include the tiles actually used in a map in it's tilesets. This is not possible using the Flutter SDK.
Any risks?
This improvement would require the removal of all dart:ui dependencies. However there are only two in this project, which I replaced in the "#69-dart-sdk branch" on my fork:

Color, which I replaced with color_models and a little bit of code for hex parsing
Rect, which I simply replaced by dart:math's Rectangle

More information
These changes would break public contracts and therefore require some work on the flame_tiled plugin.
However I believe this would be minimal. I did not look into this further, but from what I changed this would only require converting Rect to Rectangle and casting ColorModel to Model, which is implemented in flutter_color_models.
I believe this would be worth it and I am prepared to submit PRs both here and in flame_tiled to make these rather simple changes. However if you would consider this to be too invasive, I am happy to just keep this as a fork!

Overall it sounds like a good idea, we've had this plan before but never done the implementation.
I'm was thinking that maybe it is a better idea to pull in the color and rect class directly in tiled.dart instead of pulling it from the third party library so that we don't have to make yet another breaking change if that library become unmaintained an we have to update something, but on the other hand we could just pull in those classes at that time then instead.

@kurtome @ufrshubham @jtmcdole do you have any input on this, #70 and #71?

Overall it sounds like a good idea, we've had this plan before but never done the implementation. I'm was thinking that maybe it is a better idea to pull in the color and rect class directly in tiled.dart instead of pulling it from the third party library so that we don't have to make yet another breaking change if that library become unmaintained and we have to update something, but on the other hand we could just pull in those classes at that time then instead.

Maybe I'm missing something, but nothing is ever done with both Rect and Color right?
Could Color than not just store RGBA values and thats it? There would be no need for any library at all.
(Don't know why I did not think of this before when I implemented this)
Since Rectangle comes from dart:math I think its just find to depend on it!

Oh and I forgot to mention that I needed to replace flutter_test with test but I don't think thats going to be problem since their APIs seem to be the same

Maybe I'm missing something, but nothing is ever done with both Rect and Color right? Could Color than not just store RGBA values and thats it? There would be no need for any library at all. (Don't know why I did not think of this before when I implemented this)

Ah yes, if that works it would be great!

Oh and I forgot to mention that I needed to replace flutter_test with test but I don't think thats going to be problem since their APIs seem to be the same

:+1:

My only thoughts are: make sure the Color primitives play nice with Dart's canvas methods. If I remember correctly I was having issues with the hex string being a pain to deal with, so I put the hex parsing in the tiled.dart library so we could access the dart:ui color value, but also left the hex string on the structs because (in my opinion) we want to leave let users of this library access values from the tmx file

Could Color than not just store RGBA values and thats it?
I think that works because then you can make a dart:ui Color easily, the problem is the hex string didn't have a built in parser in dart:ui so it was annoying to use

I needed to include Rect, Color, and Size in some of my Dart projects, therefore this library - pure_dart_ui - came into being.
Also, I needed to parse the TMX file to pure Dart and I am very grateful for your Tiled package. The pure_dart_ui package was included in Tiled, and all tests passed. If you don't mind this option, I will prepare 2 PRs:

To close this issue WITHOUT a breaking change.

To close the Map Export issue.

What do you mean by „WITHOUT a breaking change“? Wouldn’t changing the types be a breaking change anyway?
Also I don‘t think it‘s smart to depend on another library when all that is needed is one Type: Color
Secondly how would you propose a PR for #71 ? Did you already write an Exporter as well? I have already opened #78 with a finished implementation!

What do you mean by „WITHOUT a breaking change“? Wouldn’t changing the types be a breaking change anyway?

I mean that the types from pure_dart_ui are 100% copied from Flutter dart.ui: all the functionality remains unchanged.

Also I don‘t think it‘s smart to depend on another library when all that is needed is one Type: Color

Wrapping classes in a library is a solution that makes the library more stable, but not inflexible. These classes can be used for many other pure Dart projects and can be extended with some useful extensions, as Flame has already done.

Secondly how would you propose a PR for https://github.com/flame-engine/tiled.dart/issues/71 ? Did you already write an Exporter as well? I have already opened https://github.com/flame-engine/tiled.dart/pull/78 with a finished implementation!

Nice!

I mean that the types from pure_dart_ui are 100% copied from Flutter dart.ui: all the functionality remains unchanged.

So does my ColorData class, however both are breaking changes as conversion methods are required in order to adapt to for example dart:ui's Color that is needed by flutter/flame! Or am I missing something?

Wrapping classes in a library is a solution that makes the library more stable, but not inflexible. These classes can be used for many other pure Dart projects and can be extended with some useful extensions, as Flame has already done.

While you are correct I don't believe that is worth it for only a single class, however that is something that should be decided by this projects maintainers @spydon

So does my ColorData class, however both are breaking changes as conversion methods are required in order to adapt to for example dart:ui's Color that is needed by flutter/flame! Or am I missing something?

The pure_dart_ui contains only the copied classes from dart:ui with all dependencies to make it work. See below for details.

While you are correct I don't believe that is worth it for only a single class, however that is something that should be decided by this projects maintainers

Color, Offset, Rect, Size, and we can add all the necessary classes and functions later.
So we can unbind from dart.ui and maintain compatibility with pure Dart for all Flame packages without rewriting the code except for these lines:
* import 'dart:ui' hide Color, Offset, Rect, Size;
+ import 'package:pure_dart_ui/pure_dart_ui.dart';

While you are correct I don't believe that is worth it for only a single class, however that is something that should be decided by this projects maintainers @spydon

I agree here, especially since the class name will clash with everywhere where people are using the Flutter classes.
So let's go with the ColorData class.

So we can unbind from dart.ui and maintain compatibility with pure Dart for all Flame packages without rewriting the code except for these lines:

It will still be a breaking change even though no code except the imports need to change, the classes won't be compatible with the ones in dart:ui for our users, even though they are copies.

Ok with Color / ColorData. What about the other classes: Size, Rect., Offset?

Ok with Color / ColorData. What about the other classes: Size, Rect, Offset?

They are not used here afaik.


Expose Require command

Expose the 'require' command line option to allow single installs of vendor projects with having an existing composer.json file.
Require will create the composer.json file which install can then be executed against

Looking to change this to a library


Show only allowed widgets for a certain main parameter

What type of PR is this?
This is an extension of an existing feature . The existing feature shows all the dashboard's widgets in the dashboard page, no matter what the dashboard parameters values are.
In our extension (the code in this PR) we add the possibility to only show the allowed widgets according to a parameter value.
You can see this happening on staging in this video

Pre-Description

This has been developed :
Assuming that the dashboard parameters that we use -in our case- never share common values.

Description
After we include this code in our image of redash, we will be able to only show the widgets that we want for the dashboards that we  wish to customise.
The introduced code is a transition before the final state where we will be getting the list of allowed_widgets from our product re****_mo**** directly.
But, in order to make it work for now, we need to have certain things set:
1- The allowed_widgets_dashboardID query

The alllowed_widgets query needs to have its name in the format allowed_widgets_{dashboardID}
If a dashboard has no related alllowed_widgets query, it will simply show all the widgets without filtering.
*The query should have the following columns : main_parameter and widgets, where main_parameter will include the parameter values , for each parameter value that we want to customise, we specify the list of widgets we want to show, a widget is identified by its visualisation name

example :

select "collector123" as "main_parameter",  "['heat1', 'water2']" as "widgets"
union select "zone456" as "main_parameter",  "['heat1', 'heat2']" as "widgets"

2- The visualisations names

For every widget that we want to show when we want, we should add a name to its visualisation (so far only one ... in the futur many)

example : 'heat1' as visualization name
How is this tested?

[x] Unit tests (pytest, jest)
[x] Manually , check video mentioned above

What is next?

using the widgets tags instead of the visualisation names (but will they really be better than the visualisation names)
build the allowed_widgets query directly from our product re****_mo****
discuss the positioning of the widgets if needed

Related Tickets & Documents
Closes https://github.com/orgs/metr-systems/projects/18/views/1?pane=issue&itemId=74429589
Closes https://github.com/orgs/metr-systems/projects/18/views/1?pane=issue&itemId=74429568
Closes https://github.com/orgs/metr-systems/projects/18/views/1?pane=issue&itemId=74429622

Very cool!
Do you need help to deploy it staging? I'm super curious to see it live hehe

it is already in staging :D

@imenattatra so we do need the possibility for several widget tags in order to test properly because I would like to test with GAG. Since we have no UI yet for the tags, any way we can simulate several tags using the visualisation name is fine with me.

https://dashboard.staging.metr.systems/staging/queries/12

and something like that when the dashboard is based on controller but you also have other devices like for GAG:
with
prep_groups_controller as (
SELECT distinct device_id , --concat('''',type,number,'''') as visu_list_prep,
building_uuid
FROM core_group
where device_id in (13, 69, 155) and device_type_id = 12),
prep_groups as (
select c.building_uuid, concat('''',c.type,c.number,'''') as visu_list_prep
from core_group c
inner join prep_groups_controller p on c.building_uuid=p.building_uuid
where p.device_id in (select device_id from prep_groups_controller)
group by c.building_uuid, c.type, c.number
),
data_prep as (
SELECT building_uuid , STRING_AGG(visu_list_prep, ',') as visu_list
FROM prep_groups
GROUP BY building_uuid)
select c.device_id as main_parameter, concat('[',visu_list,']') as widgets
from data_prep p
left join prep_groups_controller c on c.building_uuid=p.building_uuid

it will get y bit trickier with OMS groups because they have the same types and number as other groups linked to bus devices but they would be different widgets. For Example refley may be monitored like a controller or with an OMS sensor. in both cases I would have a reflex1 group BUT one will have a device linked to it and the other one not.
Therefore I think we can build the list checking this as well and provided different widgets names fro bus or OMS devices.
I have not written this query but I believe it is doable. I will double check with Andreas and Chris on monday

@imenattatra, @thiagogds explained me his comments on the parameter value. Now I understand we are there speaking about the possibility to have 2 'main_parameters' to filter the dashobard and that the value of both currently would be consiedered by the allowed_widget query. I agree with @thiagogds that we can and should restrict to only the value of 1 parameter being considered. We will have only 1 main parameter and can 'hard code' the UI dynamic on that.

it will get y bit trickier with OMS groups because they have the same types and number as other groups linked to bus devices but they would be different widgets. For Example refley may be monitored like a controller or with an OMS sensor. in both cases I would have a reflex1 group BUT one will have a device linked to it and the other one not. Therefore I think we can build the list checking this as well and provided different widgets names fro bus or OMS devices.
I have not written this query but I believe it is doable. I will double check with Andreas and Chris on monday

@imenattatra I am pretty sure we will find a way for this. You can continue on the milestone


adding devtools to the appconfig dev_reqs to solve python core issue

Attempted fix to https://dev.azure.com/azure-sdk/internal/_build/results?buildId=704789&view=results brought up by reviewing #15015

/azp run python - core

/azp run python - core

/azp run python - appconfiguration - tests

/azp run python - appconfiguration - tests


add test to GroupCommand and its parser

Codecov Report

Merging #203 (2fb2249) into master (527edaf) will increase coverage by 1.67%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master     #203      +/-   ##
============================================
+ Coverage     59.35%   61.02%   +1.67%
- Complexity      655      670      +15
============================================
  Files           114      114
  Lines          2524     2530       +6
  Branches        289      291       +2
============================================
+ Hits           1498     1544      +46
+ Misses          928      881      -47
- Partials         98      105       +7

Impacted Files
Coverage Δ

...ava/seedu/address/logic/commands/GroupCommand.java
40.90% <33.33%> (+40.90%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
70.70% <0.00%> (+2.02%)
:arrow_up:

...c/main/java/seedu/address/model/person/Person.java
94.11% <0.00%> (+14.70%)
:arrow_up:

...seedu/address/logic/parser/GroupCommandParser.java
84.21% <0.00%> (+84.21%)
: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 527edaf...2fb2249. Read the comment docs.


Add lesson on how to use COB

For ontology engineers

We should prioritise creating a page about COB after Chris talk at ontology summit. @souzadevinicius can you create a page in the explanation section called cob.md, and once the Summit Videos are only we will embed Chris talk in there.


Update Node.js versions to v20 and v22

@fabnguess https://github.com/NodeSecure/ci#requirements

I correct it immediately

I think we should not see all theses changes, may you need to rebase ?


fix: Enable Log tabs visibility in AppLogReader permission set

This PR adds tab visibility settings to the AppLogReader permission set to ensure users can see the Log_List, Log_Reader, and Log_Tester tabs in the App Logs application.
Changes:

Added tab visibility settings to AppLogReader permission set
Set Log_List, Log_Reader, and Log_Tester tabs to Visible

Good catch Ryan. This looks good.


add typecode standard patters to schema

To develop a consistent and well defined schema, we should capture how to make a typecode and ensure it's easy for people to identify when to apply specific practices.
criteria for completion

[ ] add slot standards to schema
[ ] define the standards and when/how to use them. For example, sm= sample , actv = activity
[ ] define when the parent class should be included in the subclass typecode (for example wf for workflow added to the typecode for "MetatranscriptomeAnnotationActivity" wfmtan)

relates to https://github.com/microbiomedata/nmdc-metadata/issues/2


Allow to properly set a custom api_url

Potential fix for Issue #3

Thanks! I'll check this shortly and verify it before merging.

Hello,
Any updates on this ?
I'm still using my fork for now but this gem could use some more PR to add new features like ACL support or fix unwanted behavior like https://github.com/kontena/kong-client-ruby/issues/5
At my company, we could provide those PR but we need to make sure it could be merged shortly, or else we would better create a client of our own.

Sorry for delay. Looks definitely good! Thanks for contributing!

Glad to see it merged, thank you :)


jsDelivr repository

@jimaek Would it be possible to get the alertify.js repository pointed here? Not sure how that all works, or even if it is possible, but the original repository fabien-d/alertify.js is no longer actively maintained, and this fork is.

Sure you can simply send a PR to our repo and change the link. You can enable auto-updates as well if you wish. Use Github's web interface to speed things up.

Thanks!


CSRF auth key problem

Hello, just trying to start UIFD and containers seems to be block on Unable to find an existing CSRF auth key. Generating a new key
I can connect to the web interface however nothing dynamic is displayed .
And whenever I connect to web interface , UIFD log : persistent connection closed
Is it a known issue ?

Hi @adi90x,
the first log Unable to find an existing CSRF auth key. Generating a new key is an information message. However the second one persistent connection closed indicates an error.
Could you give us information about your docker version? E.g. docker info and docker version

Hello deviantony,
I will close the issue as I was unable to reproduce it with a clean docker setup.


X10 tutorial - avoid scalarized() in loops for defaultXLA

Keeping accumulated training statistics as tensors and only calling scalarized() for printing appears to speed up performance ~2x with XLA devices.  The timings below were done on Colab free instances.

seconds per epoch - CPU - XLA goes from 82 to 39
seconds per epoch - TPU - XLA goes from 11.8 to 6.5
seconds per epoch - CPU - Eager - goes from 37 to 35 (probably noise, reported for comparison)

The changes I made are:

import Foundation to time and report seconds per epoch
data in Statistics struct as Tensors so scalarized() called outside batch loops
add Statistics.init(on: device) to avoid device mismatch errors
DRY accumulating batch statistics with update() method
format print strings with fewer decimal places

There may be a simpler way to get the same impact in the training tutorial.  Also, I have seen similar Statistics code that might be a more general solution.

Great job on the profiling. I had forgotten that we'd done the same for other internal models to avoid the use of scalarized() and its performance consequences. This also simplifies the training loop code.
Would it be possible to clean the execution state of your notebook, removing the output from the cells and their execution order? I'd like for the notebook to start blank, and it would be preferable for the diff here to only be the code that was changed.

I removed the outputs and execution state.  I did not realise those got saved as part of the notebook.

Thanks for bearing with me as I add silly mistakes while hurrying commits.  I found your suggestions all very useful.


[stdlib] Utilize '#if' for switch cases in HashedCollections

_Cocoa{Set,Dictionary}{Buffer,Index,Iterator} can be completely eliminated from non-ObjC runtime environments.

@swift-ci Please test

@rintaro Why was it closed and reopened again? Is it OK to merge?

@moiseev
It was just an accidental close/reopen.
I'm OK to merge this.

If SE-0154 is going to be implemented in Swift4.0, probably we should hold off this PR.
#if for swift case feature hasn't been taken into swift-4.0-branch https://github.com/apple/swift/pull/9457

Ah, I didn't know SE-0154 has already been implemented in  https://github.com/apple/swift/pull/9030
If it's stable, I think we can merge this PR.
@natecook1000 Any concern?

No concerns on my end!

@swift-ci Please smoke test and merge


compatible_partitions between boolean mask and regular array returns True if array is sliced while it shouldn't

In the following snippet, compatible_partitions between a boolean mask and a regular array returns True while the array is sliced while it shouldn't.
import uproot
import dask_awkward as dak
from dask_awkward.lib.core import compatible_partitions

fname = "coffea/tests/samples/nano_dy.root"
events = uproot.dask({fname: "Events"})

compatible_partitions(events.Electron_pt[5:], dak.num(events.Electron_pt) == 2)

Technically these are compatibly partitioned. Only because they have the same number of partitions.
compatible_partitions first checks to see if the total number of partitions are the same. If they are not, this would fail. If we haven't failed yet, it checks to see if the divisions are the same. But here we run into the case where divisions may not be known. As things stand right now dak.num(events.Electron_pt) == 2 is not passing down divisions, so the RHS here is telling compatible_partitions that it doesn't know it's divisions, just that there is 1 partition. So this check is correctly returning True for a case where divisions are uknown for one of the parts of the comparison.
What we really need  to do is implement divisions conservation for things like __eq__

Ah my mistake, it's dak.num that is killing the known divisions, not the __eq__ call (that call is conserving known divisions)

will close this now that #303 is in, but it's possible there are still lingering cases out there (of divisions not getting conserved when they should). open another issue if you find one!


Too many connections are getting created

Hi Everyone,
I have gone through the HikariCP connection pool, I could say this is awesome as I have seen good performance. But here my concern is I think it is taking too many connections.
Scenario:
I have list object which contains 10004 records and when I ran insert query it is taking 13 seconds to complete the operation.
DB Properties:
final HikariDataSource dataSource = new HikariDataSource();
dataSource.setMaximumPoolSize(100);
dataSource.setDriverClassName("oracle.jdbc.driver.OracleDriver");
dataSource.setJdbcUrl("jdbc:oracle:thin:@g9u1769.houston.hpecorp.net:1525:ODSDBD");
dataSource.setUsername("Solid_batch1");
dataSource.setPassword("solid_batch123");
dataSource.setMaxLifetime(30000);

log:
2016-08-27 11:26:01.779 [] [] [] [Hikari connection adder (pool HikariPool-0)] DEBUG com.zaxxer.hikari.pool.HikariPool - HikariPool-0 - Added connection oracle.jdbc.driver.T4CConnection@221e8d6a
2016-08-27 11:26:04.204 [] [] [] [Hikari connection adder (pool HikariPool-0)] DEBUG com.zaxxer.hikari.pool.HikariPool - HikariPool-0 - Added connection oracle.jdbc.driver.T4CConnection@3c4ecbf
2016-08-27 11:26:06.620 [] [] [] [Hikari connection adder (pool HikariPool-0)] DEBUG com.zaxxer.hikari.pool.HikariPool - HikariPool-0 - Added connection oracle.jdbc.driver.T4CConnection@7592f187
2016-08-27 11:26:09.038 [] [] [] [Hikari connection adder (pool HikariPool-0)] DEBUG com.zaxxer.hikari.pool.HikariPool - HikariPool-0 - Added connection oracle.jdbc.driver.T4CConnection@22f125f
2016-08-27 11:26:11.455 [] [] [] [Hikari connection adder (pool HikariPool-0)] DEBUG com.zaxxer.hikari.pool.HikariPool - HikariPool-0 - Added connection oracle.jdbc.driver.T4CConnection@605f1c17
2016-08-27 11:26:13.869 [] [] [] [Hikari connection adder (pool HikariPool-0)] DEBUG com.zaxxer.hikari.pool.HikariPool - HikariPool-0 - Added connection oracle.jdbc.driver.T4CConnection@42d5b6f
2016-08-27 11:26:13.975 [] [] [] [main] WARN  c.h.i.i.d.manager.dao.DaoService - detail query : 13

If I can control the too many connections. I think I can control the performance of the query.
Please suggest if there is anything which can improve performance.
Thanks

@yaswanthbs If you set maximumPoolSize to 100, by the default HikariCP is going to create 100 connections -- because by default it operates as a fixed-size pool.  Two things...
First, unless you're running 20,000 transactions per seconds against a 32-core Oracle server, 100 connections is overkill.  Please read About pool sizing and watch the video, as it particularly applies to Oracle.  In the video Oracle clearly show supporting 20K TPS with 96 connections with 10K similated front-end users.  Unless your application is at that scale, or your hardware is capable of supporting it, you likely need something more in the neighborhood of one or two dozen connections max.
Second, if you want the pool to grow and shrink, set minimumIdle to something like 2.  This does not necessarily mean you will have a pool size of 2, just that HikariCP will try to keep two connections "in reserve".  If you application is regularly using 13 connections, the pool will probably sit around 15 connections.


display color image

How to view color images stored in HDF5?
It is always inconvenient to only view one channel, and there is also a need to adjust the D0/1/2 and FlipY.

Hi, we do support RGB images as defined by the HDF5 Image and Palette Specification. All you need is to add attribute CLASS="IMAGE" to your dataset and you should automatically see a new RGB tab in H5Web.
For an example, open H5Web's demo application, select /nD_datasets/threeD_rgb, and inspect the metadata.

Is there a reload button in the current version?

It would be more convenient to have a reload button during debugging (>_<

There is another problem, if I save the encoded image, will it not be displayed?
import h5py
import cv2

path_jpg = ""
path_h5 = path_jpg[:-4] + "_v1.h5"

with h5py.File(path_h5, "w") as h5f:
    with open(path_jpg, 'rb') as fb:
        binary = fb.read()
        data = np.frombuffer(binary, dtype='uint8')
        dset = h5f.create_dataset('image', data=data)
        dset.attrs["CLASS"] = "IMAGE"

with h5py.File(path_h5, "r") as h5f:
    data = h5f['image'][:]
    img_jpg = cv2.imdecode(data, cv2.IMREAD_COLOR)
    print("img_jpg:", img_jpg.shape)
    cv2.imwrite(path_jpg[:-4] + "_v1.jpg", img_jpg)

Currently, H5Web supports only  IMAGE_SUBCLASS="IMAGE_TRUECOLOR" as well as the following features:

n-dimensional RGB datasets (i.e. image stacks)
RGB vs BGR
transposition, dimension selection
custom axes (label+values) via the NeXus convention (cf. NXdata)

We're happy to consider supporting more of the HDF5 Image and Palette specification, but the spec is quite dense and we don't want to bloat H5Web with features that few people use, so please know that we will require careful justification when considering feature requests. If you're up for it, please open an issue on the main H5Web repository.

Regarding adding a button to reload the file, this is not trivial because this VS Code extension is just a wrapper around H5Web. I'm also a bit reluctant to add a button to the UI that is meant for debugging, especially when it's so easy to close a file and re-open it (unless there's an issue with this that I'm not seeing).
That being said, we could add a VS Code command for this, like "H5Web: Reload" — that'd be fairly straightforward.

There is another problem, if I save the encoded image, will it not be displayed?

Do you mean that you are storing a JPEG image as binary in HDF5? If so, then this is definitely not going to be supported. :wink:


[in_app_purchase] Fix crash when retrieveReceiptWithError gives an error.

It seems like the app is crashing because it can't send receiptError.userInfo over the methodchannel in a FlutterError. This PR uses the existing error mapper to parse the error to something that can be send over the methodchannel.
Fixes flutter/flutter#85943
Fixes flutter/flutter#85960
Pre-launch Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I read the Tree Hygiene wiki page, which explains my responsibilities.
[X] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[X] I signed the CLA.
[X] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[X] I listed at least one issue that this PR fixes in the description above.
[X] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[X] I updated CHANGELOG.md to add a description of the change.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[X] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

I tested and in my case it doesn't crash anymore.
Awesome, thanks for the quick fix!

@cyanglaz @LHLL @stuartmorgan Do you guys have in mind when this PR got merged and a new version is published?
I need this fix ASAP and if this needs some more time, I need to add the source branch directly into my pubspec.yaml to fix this issue

It should land shortly.

Thanks so much for fixing this! Just to quickly check, how can I know if the changes has landed?

I'm "in_app_purchases" package and having the same issue. But seems the solution would be use the "in_app_purchases_ios". Do I have to use both packages? Why can't we have the fix on "in_app_purchase" package it self ?

Hi @inshaf,
No it is not necessary to depend on both packages. The "in_app_purchase" package depends on the "in_app_purchase_ios" package using the Caret syntax (like so: in_app_purchase_ios: ^0.1.1). Meaning it will pick the latest version in the range of 0.1.1 to 0.2.0 (excluding 0.2.0). So if you would run flutter pub upgrade it will automatically pull in the latest published version of the "in_app_purchase_ios" plugin (being version 0.1.2+2 containing the fix).

Thanks so much for fixing this! Just to quickly check, how can I know if the changes has landed?

Once merged plugins will be published automatically to pub.dev if the submit queue is green. This is done by the "release" workflow that can be monitored here.

Want to add to @mvanbeusekom that is indeed already published, as you can see on pub.dev: https://pub.dev/packages/in_app_purchase_ios/changelog#0122

Many thanks @renefloor this really solved my issue!


Change the use of "optimize" to "rewrite"

https://github.com/aesara-devs/aesara/pull/1054 in Aesara replaces the use of "optimize" with "rewrite". AeHMC will need to adapt to the API change.

I don't think I saw any imports from the old "optimization" modules, so this might not involve any changes.

optimize_graph is called in utils.py.


feat: make detailed var info deterministic

The result of the SSAVar#getDetailedVarInfo method can be used in generated code, so it's important to achieve that the method produces the same result for ssaVars with the same properties.

@pubiqq thanks :+1:


Autopopulate CopySourceSSECustomerKeyMD5 for s3

This is implementing logic used for this PR: https://github.com/aws/aws-cli/pull/1487. We have it for SSECustomer members but not the copy source one which may be needed when calling CopyObject.
cc @jamesls @mtdowling @rayluo @JordonPhillips

Looks good to me :ship:

:ship:

@jamesls I updated. Let me know what you think.

:shipit:


Fix asset manifest logic to ensure uploaded files remain on Workers KV remote

This PR fixes the bug observed yesterday, where wrangler publish --site seemed to publish nothing to the Workers KV remote.
The bug was a missing patch of logic related to wrangler publish --site's calling of sync().
When sync is called, it does two things:

first, upload keys (using the asset manifest logic, file paths with their value hash appended)
second, delete keys that exist on remote but not on local.

The bug was that during the second step, the local keys did not have the file's value hash appended, and wrangler perceived the keys on the local machine as JUST their pathnames. this means that during the second delete step of sync, all the files on the remote (which had just been uploaded) were deleted!
I have now successfully deployed a site from scratch using the wrangler init and wrangler publish --site logic. I made sure to nuke my old static assets namespace and start TOTALLY from scratch to ensure it works as expected.

Ah - yes. This is a result of me not understanding what exactly the function was used for - this makes a lot of sense to me now.