ItemsDictionary - Possible NullReferenceException with .GetEnumerator()

Describe the bug
In https://github.com/aspnet/AspNetCore/blob/f854b561a5bc9efa657821d562694dcee59c5284/src/Http/Http/src/Internal/ItemsDictionary.cs#L136
there is a possible NullReferenceException if the _items collection is null when .GetEnumerator() is called. I believe the expected line of code would use a null-conditional operator (?.) similar to line 134 (just above it). https://github.com/aspnet/AspNetCore/blob/f854b561a5bc9efa657821d562694dcee59c5284/src/Http/Http/src/Internal/ItemsDictionary.cs#L134
To Reproduce
This snippet of code would most likely reproduce the error :
var dict = new Microsoft.AspNetCore.Http.ItemsDictionary();
IEnumerable en = dict as IEnumerable;
foreach(var item in en) // <-- implicit call to .GetEnumerator() will throw NullReferenceException
{
    // this code won't be reached
}

Currently, I'm trying to get the AspNetCore project built on my machine to develop a unit test.
I originally ran into this issue when using the Elmah library.
https://github.com/ElmahCore/ElmahCore/issues/51
Please see the above issue for screenshots of the issue occurring in context. Here is a screenshot of the exception occurring out of context.

Further technical details

ASP.NET Core version
3.0.0

Include the output of dotnet --info

PS D:\Projects\Main\Powershell\Ucsb.Sa.Powershell> dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   3.0.100
 Commit:    04339c3a26

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.18362
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\3.0.100\

Host (useful for support):
  Version: 3.0.0
  Commit:  7d57652f33

.NET Core SDKs installed:
  2.1.4 [C:\Program Files\dotnet\sdk]
  2.1.202 [C:\Program Files\dotnet\sdk]
  2.1.300 [C:\Program Files\dotnet\sdk]
  2.1.500 [C:\Program Files\dotnet\sdk]
  2.1.509 [C:\Program Files\dotnet\sdk]
  2.1.701 [C:\Program Files\dotnet\sdk]
  2.2.101 [C:\Program Files\dotnet\sdk]
  2.2.300 [C:\Program Files\dotnet\sdk]
  2.2.301 [C:\Program Files\dotnet\sdk]
  3.0.100 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

The IDE (VS / VS Code/ VS4Mac) you're running on, and it's version

Fixed via https://github.com/aspnet/AspNetCore/pull/16947


Fix the Svg component for Edge and IE

Since innerHTML isn't supported well on SVGElement, wrap the SVG in a span.
In IE, a Polyfill is required (could have been done), but the real problem is Edge. In Edge, <path .../> self closing tags are not well interpreted and the resulting SVG is broken. See http://codepen.io/JPLachance/pen/VKBoVV?editors=1010 (in Edge) for an example.

Current coverage is 100% (diff: 100%)

Merging #28 into master will not change coverage

@@           master   #28   diff @@
===================================
  Files           2     3     +1
  Lines          54    58     +4
  Methods        13    13
  Messages        0     0
  Branches        9    10     +1
===================================
+ Hits           54    58     +4
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 41eeca0...ad76ab1


ContextCompat.RECEIVER_NOT_EXPORTED Compilation Error in Version 1.0.2

I recently updated react-native-screenguard from version 1.0.0 to 1.0.2. After the update, I encountered a compilation error related to ContextCompat.RECEIVER_NOT_EXPORTED in the ScreenGuardColorActivity.java file. This issue did not occur in version 1.0.0.

Error Detail

Task :react-native-screenguard:compileReleaseJavaWithJavac FAILED
/Users/sungwookkim/workspace/supplies/gymboxx-app-client/node_modules/react-native-screenguard/android/src/main/java/com/screenguard/ScreenGuardColorActivity.java:95: error: cannot find symbol
registerReceiver(closeReceiver, intentFilter, ContextCompat.RECEIVER_NOT_EXPORTED);
^
symbol:   variable RECEIVER_NOT_EXPORTED
location: class ContextCompat
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
1 error

Please let me know if there is any other recommended solution or if this can be fixed in the next release.
Thank you for your assistance.

I also have this issue, for me the fix was to pin it to 1.0.0 for now. Hope it still will be fixed

hi @crazywook, working on it, will have a release soon.
You can also fix it yourself by opening Android studio, and go to react-native-screenguard/java/com/screenguard/ScreenGuardColorActivity.java, line 94, and change like this:
...
 registerReceiver(closeReceiver, intentFilter, RECEIVER_NOT_EXPORTED);
...

thanks and happy hacking !

go to react-native-screenguard/java/com/screenguard/ScreenGuardColorActivity.java, line 94,
patch the package
IntentFilter intentFilter = new IntentFilter(SCREENGUARD_COLOR_ACTIVITY_CLOSE);
    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) { // Android 13+
        registerReceiver(closeReceiver, intentFilter, Context.RECEIVER_NOT_EXPORTED);
    } else {
        registerReceiver(closeReceiver, intentFilter);
    }

I have fixed this issue by apply above patch


Provide a more accurate message backlog value

Motivation
Currently, the message backlog value is measured by the number of entries. It is not based on the number of messages. Hence it is difficult for applications to figure out the exact message count in the backlog. We should provide a mechanism to provide a more accurate message backlog value.

I've this exact problem, I need to figure out how many messages are still in the backlog but I receive a backlog count way more lower than the expected number. Finding a message backlog value way lower than excepted give the false feeling that some message has been lost "on the way" (still they are there to be read).

@diegosalvi Yes, we are working on a fix currently, the new approach is based on the message index which can provide the message-based backlogs.

Any news here? @codelipenghui

Any news on this issue @codelipenghui? It would be really helpful to detect how much "lag" an application has processing events. Currently as fallback I can only know how much "old" is the next event that I have to process but I don't know really if I have 1K events or 1M (just how many entries)

@diegosalvi
we already have the 'getPreciseBackLog' option.
Does not work for you ?
https://github.com/apache/pulsar/blob/e2fa1898f7e9a38b1743b2abf21c87eafe126ee2/pulsar-client-admin-api/src/main/java/org/apache/pulsar/client/admin/Topics.java#L1097

I already use the precise backlog but from my understanding still reports entries and not messages.
I should deactivate batches on the producer but it doesn't seem to me a performance wise choice.

@diegosalvi  would you mind working on this ? the implementation should be pretty straightforward if you take a look to the getPreciseBacklog code path.
The MessageMetadata structure has the number of messages in the Entry.

I'm looking into it :)

@eolivelli I've looked into getPreciseBacklog code (Topic.getStats -> PersistentSubscription.getStats -> MangedCursor.getNumberOfEntriesInBacklog -> ManagedCursorImpl.getNumberOfEntries).
Probably I'm missing something but from what i saw it "count" entry ranges without looking int any entry at all.
From my undersanding I should read every entry from subscription position up to lac, build theirs metadata and then sum them. It seems very inefficient to me

@diegosalvi you are correct!
we would need to read all the entries from BK, and that's not doable, because it costs too much, and also you would be going to pollute all the caches.
You can implement it, but we should warn the users that the operation has a very high cost and it should not be done frequently


[WIP] Attempt to deflake MultiMasterJournalTest and similar tests

Here, I'm not looking for not only the test to pass but for tests like MultiMasterJournalTest to not go to the second try.

@gpang did some work on getting unique ports for multi master tests, you might want to talk to him

Thanks. I'll take a look at https://github.com/Alluxio/alluxio/pull/9894/files. I just wanted to trigger some CIs to look at what the results are like and plan to close this PR.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6663/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6665/
Test FAILed.

My old PR is not the correct final solution, so there is no need to base anything off of that.


allow custom match between suggestion and typing

So I could match with tags and wildcards

Okay, I will add callback option which will allow custom check for suggestions to appear.

Added suggestMatch option in settings and published the changes. You can also check the README for more info about this. Hope it will be helpful.

@quazardous  Closing this now. You can re open this if the mentioned solution does not work for you.


getting errors with using the stylex CLI in next 15 App dir.

ISSUE

Im getting the following error while using the stylex CLI with next 15 while following the example in https://github.com/facebook/stylex/pull/618/commits/eab671c6e7f925ae2eac8f08cd531c79a6d38eae:
'stylex.create' should never be called at runtime. It should be compiled away by '@stylexjs/babel-plugin'

CODES
.stylex.json5:
{
  input: ['./source'],
  output: ['./src'],
  cssBundleName: 'stylex_bundle.css',
  useCSSLayers: true,
  babelPresets: [
    ['@babel/preset-typescript', { allExtensions: true, isTSX: true }],
    // '@babel/preset-react',
  ],
  // modules_EXPERIMENTAL: [
  //   ['@stylexjs/open-props', { ignore: ['src', '__tests__'] }],
  // ],
  // watch: true,
}

next.config.ts (default nothing changed.):
import type { NextConfig } from "next";

const nextConfig: NextConfig = {
  /* config options here */
};

export default nextConfig;

and here's my package.json incase its needed:
{
  "name": "next_cli_stylx_three_appdir",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "predev": "stylex --config ./.stylex.json5",
    "dev": "stylex --config ./.stylex.json5 -w & next dev",
    "prestart": "stylex --config ./.stylex.json5 && next build",
    "build": "next build",
    "start": "next start",
    "lint": "next lint"
  },
  "dependencies": {
    "@babel/preset-typescript": "^7.26.0",
    "@babel/runtime": "^7.26.0",
    "@stylexjs/babel-plugin": "^0.9.3",
    "@stylexjs/cli": "^0.9.3",
    "@stylexjs/stylex": "^0.9.3",
    "next": "15.0.3",
    "react": "19.0.0-rc-66855b96-20241106",
    "react-dom": "19.0.0-rc-66855b96-20241106"
  },
  "devDependencies": {
    "@types/node": "^20",
    "@types/react": "^18",
    "@types/react-dom": "^18",
    "typescript": "^5"
  }
}

and here's the code of me trying to use the stylex.create() in app/page.tsx
import "../stylex_bundle.css";
import * as stylex from "@stylexjs/stylex"

export default function Home() {
  return (
    <>

    </>
  );
}

const styles = stylex.create({
  rootDiv: {width: "100%", minHeight: "100dvh", background: "green"}
})

I have made all the changes relating to the src also.
If needed, ill put the project up on github.

I actually dont know why the whole text is messed up. Ill create a github repo and post the link here in a bit. sorry lol

I don't see anything wrong with your setup. Might be missing something small. Please put up your project on Github so I can check.

heyo im sorry I forgot to put the project on github. Ill put it up today and comment the link.

I was going to message but looks like you merged my PR already. Hopefully it's working on your machine as well.
There are a few of limitations of the CLI for now:

All imports need to be manually types to be relative imports. The @/ alias can't be used.
The globals.css file should be wrapped with @layer priority1 { ... } to ensure it doesn't override StyleX styles.
Compiling external packages that use StyleX styles has some bugs

We're working to fix all of these issues and it should be done soon.

thanks for all this help. Im having an issue of when i try to run npm run dev now, its generating a new src folder along with layout.js page.js etc. which is giving me this error:
ERROR
`Module not found: Can't resolve '..\stylex_bundle.css'
Did you mean './..\stylex_bundle.css'?
Requests that should resolve in the current directory need to start with './'.
Requests that start with a name are treated as module requests and resolve within module directories (node_modules).
If changing the source code is not an option there is also a resolve options called 'preferRelative' which tries to resolve these kind of requests in the current directory too.

1 | import "..\stylex_bundle.css";
| ^
2 | import * as stylex from "@stylexjs/stylex";
3 | import "./globals.css";
4 | export const metadata = {

https://nextjs.org/docs/messages/module-not-found`
Codes being generated from the npm run dev command:
layout.js
import "..\\stylex_bundle.css"; import * as stylex from "@stylexjs/stylex"; import "./globals.css"; export const metadata = {   title: "Create Next App",   description: "Generated by create next app" }; export default function RootLayout({   children }) {   return        {children}     ; }
page.js:
import "..\\stylex_bundle.css"; import { props, create, keyframes } from "@stylexjs/stylex"; // Although StyleX can compile @/ paths, TS can only resolve // them to one location. This will be fixed in the CLI soon. import { fonts, tokens } from "../vars.stylex"; export default function Home() {   return
Welcome to Next.js & StyleX!

         Get started by editing pages/index.tsx

     ; } const gradientColors = ["#ffadad", "#ffd6a5", "#fdffb6", "#caffbf", "#9bf6ff", "#a0c4ff", "#bdb2ff", "#ffc6ff", "#ffadad"].join(", "); const rotate = "xa503dy-B";
along with stylex_bundle.css which has alot of css classes with styles aswell as vars.stylex.js. I don't think those file contents have anything to do with the error so i wont share it but if neccessary I will.
Is there any way to stop generating a new src file when i run npm run dev?

and I still don't know why github is ruining the text formatting

(The text formatting is broken, because you're not putting a new line after the triple backticks. Try to edit your comment again and see the correct way to define codeblocks.)
Seeing the error message, there seems to be a Windows-specific bug with import paths.
To verify, can you try replacing
import "..\\stylex_bundle.css";

with
import "../stylex_bundle.css";

in the generated files and see if that fixes the problem?

Explanation: Windows uses backslash for file paths instead of forward slash like mac/unix/linux. We're using Node APIs to get relative file paths which are being returned with backslashes, but I think the import statements expect forward slashes.

If this is indeed the problem, the fix should be pretty straightforward.

If you'd like to help fix the problem (or patch it and unblock yourself), look at the last 7 lines in @stylexjs/stylex/lib/files.js and @stylexjs/stylex/lib/es/files.mjs
Patching those files to return forward slashes instead of backslashes should fix the problem.

I tried doing that, it's giving me this error.
Module not found: Can't resolve '..\stylex_bundle.css'
Did you mean './..\stylex_bundle.css'?
Requests that should resolve in the current directory need to start with './'.
Requests that start with a name are treated as module requests and resolve within module directories (node_modules).
If changing the source code is not an option there is also a resolve options called 'preferRelative' which tries to resolve these kind of requests in the current directory too.
> 1 | import "..\\stylex_bundle.css";
    | ^
  2 | import { props, create, keyframes } from "@stylexjs/stylex";
  3 | // Although StyleX can compile `@/` paths, TS can only resolve
  4 | // them to one location. This will be fixed in the CLI soon.

https://nextjs.org/docs/messages/module-not-found

Also how can i stop it from generating a new src folder everytime with a layout.js and page.js since I already have a layout.tsx and page.tsx ?

In your own comment, you stated that the generated file had:
import "..\\stylex_bundle.css";

yes i edited the
import "..\\stylex_bundle.css"

to
import "../stylex_bundle.css";

which is what you told me to try.

@Lolhp0 and Next still can't compile it? Is the stylex_bundle.css file present one folder up?

In your own comment, you stated that the generated file had:
import "..\\stylex_bundle.css";

It'll take me a bit longer to debug the windows issue as I don't have windows on me handy at the moment, but I'll work on it soon.

I am having the same issue.
It works if I do what the error says .. to prefix the import with "./"
However I can hardly rewrite each import stylex bundle statement to include that after every time I run the CLI

If you'd like to help fix the problem (or patch it and unblock yourself), look at the last 7 lines in @stylexjs/stylex/lib/files.js and @stylexjs/stylex/lib/es/files.mjs
Patching those files to return forward slashes instead of backslashes should fix the problem.

Since I was just tweaking files directly in my node_modules folder .. for me it was @stylexjs/stylex/lib/index.js
And I altered the getRelativePath function to always prepend "./" .. and that worked for me.
I'll just use patch-package to keep it that way until it's fixed.

@BjornHMS thanks for confirming that editing the file fixes the problem. I have not been able to test on Windows to reproduce this issue and was looking for some help. With this confirmation I can go and fix the bug.


click-to-copy updates to remove Flash dependency

This tees up a fix for issue 387.
Removes Zeroclipboard in favor of Clipboard:

no flash dependency
offers a nice “Press ⌘+C to copy” message (configurable) for safari
and browsers that don’t support JS setting the clipboard

Updates the click-to-copy component:

can set all the success / error messages in the tooltips
can set the trigger text the user clicks to copy
isolated css
no more anchor tag with # href that would reload the page in
browsers without Flash

Tests will fail due to various ember package updates. Thanks to
@rwjblue for the help and offering to look at these failures.
Thanks @sandersonet for the testing help!
In action:

:+1:  Nice work


Proto update and first attempt at proper backend versioning

BREAKING CHANGES

Adds api-version 2020-05-05

Uses latest version of the proto.

Adds an internal data model used for mapping the backend.

I know we thought it might be best to keep using the proto data model but it simply isn't sustainable when we can't easily adjust previously stored data to the new model in CosmosDB. This will hopefully be the last change which will require a database reset.
Now, the protos are only used in the CovidSafe.API project while the backend data model stays in CovidSafe.Entities and interactions occur in CovidSafe.DAL; the proto types for each version must contain a mapping to the internal data model. Any fields updated or removed can be adjusted in these AutoMapper mapping profiles.

Resolves #7


Update Drizzlepac satellite trail masking notebook

This notebook checklist has been made available to us by the Notebooks For All team.
Its purpose is to serve as a guide for both the notebook author and the technical reviewer highlighting critical aspects to consider when striving to develop an accessible and effective notebook.
The First Cell

[ ] The title of the notebook in a first-level heading (eg. <h1> or # in markdown).
[ ] A brief description of the notebook.
[ ] A table of contents in an ordered list (1., 2., etc. in Markdown).
[ ] The author(s) and affiliation(s) (if relevant).
[ ] The date first published.
[ ] The date last edited (if relevant).
[ ] A link to the notebook's source(s) (if relevant).

The Rest of the Cells

[ ] There is only one H1 (# in Markdown) used in the notebook.
[ ] The notebook uses other heading tags in order (meaning it does not skip numbers).

Text

[ ] All link text is descriptive. It tells users where they will be taken if they open the link.
[ ] All acronyms are defined at least the first time they are used.
[ ] Field-specific/specialized terms are used when needed, but not excessively.

Code

[ ] Code sections are introduced and explained before they appear in the notebook. This can be fulfilled with a heading in a prior Markdown cell, a sentence preceding it, or a code comment in the code section.
[ ] Code has explanatory comments (if relevant). This is most important for long sections of code.
[ ] If the author has control over the syntax highlighting theme in the notebook, that theme has enough color contrast to be legible.
[ ] Code and code explanations focus on one task at a time. Unless comparison is the point of the notebook, only one method for completing the task is described at a time.

Images

[ ] All images (jpg, png, svgs) have an image description. This could be

[ ] Alt text (an alt property)
[ ] Empty alt text for decorative images/images meant to be skipped (an alt attribute with no value)
[ ] Captions
[ ] If no other options will work, the image is decribed in surrounding paragraphs.

[ ] Any text present in images exists in a text form outside of the image (this can be alt text, captions, or surrounding text.)

Visualizations

[ ] All visualizations have an image description. Review the previous section, Images, for more information on how to add it.

[ ] Visualization descriptions include

[ ] The type of visualization (like bar chart, scatter plot, etc.)
[ ] Title
[ ] Axis labels and range
[ ] Key or legend
[ ] An explanation of the visualization's significance to the notebook (like the trend, an outlier in the data, what the author learned from it, etc.)

[ ] All visualizations and their parts have enough color contrast (color contrast checker) to be legible. Remember that transparent colors have lower contrast than their opaque versions.

[ ] All visualizations convey information with more visual cues than color coding. Use text labels, patterns, or icons alongside color to achieve this.

[ ] All visualizations have an additional way for notebook readers to access the information. Linking to the original data, including a table of the data in the same notebook, or sonifying the plot are all options.

This notebook has been updated in collaboration with Jen Mack and the Drizzlepac team. It contains many general improvements, including an added WFC3 example to go alongside the ACS example. The requirements file has been updated to show only the packages needed to run the notebook in Python3.11.

The link is still showing the old notebook.  Can someone help?
https://spacetelescope.github.io/hst_notebooks/notebooks/DrizzlePac/mask_satellite/mask_satellite.html

Is the html version generated automatically from the notebook once it’s on github, or is this something I need to do and then push? If the latter, I’m happy to do so.
--
David V. Stark
Staff Scientist II, Science Support
HST/ACS Team
Space Telescope Science Institute
From: mackjenn @.>
Reply-To: spacetelescope/hst_notebooks @.>
Date: Thursday, May 30, 2024 at 2:14 PM
To: spacetelescope/hst_notebooks @.>
Cc: David Stark @.>, Author @.***>
Subject: Re: [spacetelescope/hst_notebooks] Update Drizzlepac satellite trail masking notebook (PR #269)
The link is still showing the old notebook. Can someone help?
https://spacetelescope.github.io/hst_notebooks/notebooks/DrizzlePac/mask_satellite/mask_satellite.htmlhttps://urldefense.com/v3/__https:/spacetelescope.github.io/hst_notebooks/notebooks/DrizzlePac/mask_satellite/mask_satellite.html__;!!CrWY41Z8OgsX0i-WU-0LuAcUu2o!z3YNvryVi8G_pAkxdrKGVaaPhiSGPhhrowX5WV12BRRVi7nwHeqNQbNDGiQ0MrHn9xVfzGfBRgCwWvT92Ohk26g$
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/spacetelescope/hst_notebooks/pull/269*issuecomment-2140535129__;Iw!!CrWY41Z8OgsX0i-WU-0LuAcUu2o!z3YNvryVi8G_pAkxdrKGVaaPhiSGPhhrowX5WV12BRRVi7nwHeqNQbNDGiQ0MrHn9xVfzGfBRgCwWvT9tMjDzgw$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/ACQIOODEA736UA3CPISMYJDZE5UAVAVCNFSM6AAAAABHYSBP3SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBQGUZTKMJSHE__;!!CrWY41Z8OgsX0i-WU-0LuAcUu2o!z3YNvryVi8G_pAkxdrKGVaaPhiSGPhhrowX5WV12BRRVi7nwHeqNQbNDGiQ0MrHn9xVfzGfBRgCwWvT9ALAnCiE$.
You are receiving this because you authored the thread.Message ID: @.***>

Hey @dvstark and @mackjenn,
We perform weekly health checks on the notebooks to ensure they still run properly after a merge. If a notebook fails during the weekly execution, it may be due to an update in packages or something similar. When this occurs, you won't see any outputs. Please open a new PR for this notebook. Thanks!


Shortcuts for logic expressions?

Many if not most programming languages implement shortcuts for the logic operators or and and:

For or, if the left hand parameter is true, the right hand parameter is not evaluated as the result is already determined to be true
For and, if the left hand parameter is false, the right hand parameter is not evaluated, as the result is already determined to be false

This allows for some checking to see if for example a calculation even makes sense (think avoiding division by zero for example): foo != 0 && 3 / foo < 10).
As I understand the code (https://github.com/TomFrost/Jexl/blob/master/lib/evaluator/handlers.js#L27), this is currently not implemented, causing every "branch" of an expression being evaluated, even if it's not needed to calculate a result.
Could we consider implementing this change?

You're 100% correct. There's another issue in here that would have benefitted from that also, so I've been playing with solutions. I don't want to give up the ability for binary operators to evaluate the left and right sides in parallel (this is a massive speed improvement for users of async transforms), so I'm toying with some binaryOp-level config that will specify whether a given binary op should use parallel or synchronous processing. Stay tuned :)

I'm toying with some binaryOp-level config that will specify whether a given binary op should use parallel or synchronous processing. Stay tuned :)

Hmm, I wonder if some specific syntax for parallel processing wouldn't be better. Having "core" features of a language configurable feels very.. PHP-ish to me (personal opinion) and may lead to unexpected behaviour, as the users writing JEXL expressions may not be the same ones who configure the runtime.
Instead, for example you could have a parallelMap() and let users who want parallel processing encode || and && via any(parallelMap(f,[a,b])) and all(parallelMap(f,[a,b]))

Oh, I didn't mean expression-facing option :). I just meant a flag in the grammar. Consider the following-- this is a snippet from the grammar file as it is now:
  '<=': {
    type: 'binaryOp',
    precedence: 20,
    eval: (left, right) => left <= right
  },
  '&&': {
    type: 'binaryOp',
    precedence: 10,
    eval: (left, right) => left && right
  },

I'm simply playing with this idea:
  '<=': {
    type: 'binaryOp',
    precedence: 20,
    eval: (left, right) => left <= right
  },
  '&&': {
    type: 'binaryOp',
    precedence: 10,
    manualOperandEval: true,
    eval: async (left, right) => await left.eval() && await right.eval()
  },

I'd just expose the manualOperandEval flag as an option in addBinaryOp and addUnaryOp and we'd be all set. There'd be no such thing as a parallel-processed or configurable && and ||.

Small amendment... after some thought, for TypeScript friendliness I'll likely axe the boolean flag and instead split eval into two different keys: one for manual resolution, the other for the existing functionality. Assuming I don't go with the all-manual solution I'm still considering.

I do like that idea! :+1:

Boom, see #84 for the feature. The syntax in the grammar file isn't as nice as what I proposed above because that wouldn't have supported synchronous evaluation, but I'm happy with where it landed.
I decided to do this in two steps. Step 1 is releasing the functionality as a non-breaking change. && and || now evaluate the right operand conditionally, and jexl.addBinaryOp has a new optional argument to denote whether your custom binary operator will be responsible for calling eval on its own operands. This can be released now without a major version bump.
Step 2 will come whenever this library embraces Typescript a bit better, because changing the type of arguments depending on an unrelated boolean is type hell. So with a major version bump, I'm planning to change addBinaryOp to accept an object similar to the objects in the grammar file instead of 4 different arguments.


Bit depth is changed by Sharp

sharp(Buffer.from(image, 'base64'))
.flatten({background: 'white'})
.resize(width, height, {
fit:"cover",
background:"white"
})
.toFormat('png')
.toBuffer();
with the above code  the bit depth of an image from 2 is transformed  to 24 and I got a much bigger one (size: from 0.5 mb to 4.49 mb).
How can I do to omit this behavior or at least how can I forecast the size (or bit depth) of the rendered image with the size of the actual one?

Hi, did you see https://sharp.pixelplumbing.com/api-output#png ?
For example, setting colours to 4 will give you a bit depth of 2.
await sharp(input)
  .png({ colours: 4 })
  .toBuffer();

Hi, did you see https://sharp.pixelplumbing.com/api-output#png ?
For example, setting colours to 4 will give you a bit depth of 2.
await sharp(input)
  .png({ colours: 4 })
  .toBuffer();

Hi
yes, I have seen it. In my case, the user can upload many images. It could be jpeg or png and the bit depth could be different for every image. Is it possible to keep the bit depth as it is after rendering the image?

Perhaps try something like the following (untested):
const { paletteBitDepth } = await sharp(input).metadata();
const colours = 1 << paletteBitDepth;

const output = await sharp(input)
  .png({ colours })
  .toBuffer();

I hope this information helped. Please feel free to re-open with more details if further assistance is required.

Hello, sorry to re-open this after to long, but this is the first result on google when searching for "sharp bit depth".
It seems that in version 0.32.4 or earlier, the paletteBitDepth field was removed. I see that we now have depth but that's only the datatype, as far as I understand. How can we get the actual bit depth? Alternatively, how can I make sure my output depth matches my input depth? In my case I'm reading a 32-bit grayscale .tif file and I would like to output a similar file.

@tobloef paletteBitDepth is still present in the metadata() response for palette-based images (GIF, some PNGs) but it appears it was missing from the docs, added via https://github.com/lovell/sharp/commit/0bc79cdb953c9c0f1cf9015be65fb231fd5ec520 - however it sounds like you want depth, which will probably be ushort for 32-bit greyscale (but that's a guess). Please open a new issue if you still require help.


Updated translations with latest changes

Add new strings

It seems we've already merged a PR to update these translations. Please re-open a PR with revised changes if you'd like to make an amendment 🙂


package.json fixes

Thanks for this. At the time of PR, the analytics.js infrastructure was fundamentally incompatible with npm/browserify; we've since migrated from Duo to Browserify, so we can close this out safely.


does not work as a SPA when  the # sign is removed from menuItem Url and when useHash value is false.

Hi, the demo does not work as a SPA when i removed the # sign from the method generating the menuItem Url and set useHash value to false. it causes the browser to refresh everytime a router link is clicked. i tried using [routerLink] in the  tag but still does not work.

Hi, can you please provide your routes? Did you change anything else aside from removing the # from menu component and changing the hash setting on the router?
Is this a problem only with menu items? Have you tried adding a custom link and seeing if it causes a full reload or redirects correctly?

hi geglio, The routes are pretty much the same as the demo's, i have not changed anything else aside from removing the # from menu component and changing the hash setting on the router.  i think its a problem only with the menu items because the breadcrumbs link is working fine and redirects correctly.
this is the app.routing.ts
import { Routes, RouterModule } from '@angular/router';

export const routes: Routes = [
  { path: '', redirectTo: 'pages', pathMatch: 'full' }
];

export const routing = RouterModule.forRoot(routes, { useHash: false });

and pages.routing.ts
import { Routes, RouterModule }  from '@angular/router';
import { Pages } from './pages.component';
// noinspection TypeScriptValidateTypes
const routes: Routes = [
  {
    path: 'login',
    loadChildren: () => System.import('./login/login.module')
  },
  {
    path: 'register',
    loadChildren: () => System.import('./register/register.module')
  },
  {
    path: 'pages',
    component: Pages,
    children: [
      { path: '', redirectTo: 'dashboard', pathMatch: 'full' },
      { path: 'dashboard', loadChildren: () => System.import('./dashboard/dashboard.module') },
      { path: 'editors', loadChildren: () => System.import('./editors/editors.module') },
      //{ path: 'components', loadChildren: () => System.import('./components/components.module') }
      { path: 'charts', loadChildren: () => System.import('./charts/charts.module') },
      { path: 'ui', loadChildren: () => System.import('./ui/ui.module') },
      { path: 'forms', loadChildren: () => System.import('./forms/forms.module') },
      { path: 'tables', loadChildren: () => System.import('./tables/tables.module') },
      { path: 'maps', loadChildren: () => System.import('./maps/maps.module') }
    ]
  }
];

export const routing = RouterModule.forChild(routes);

Thank you for providing more information, I will look into that in a few
hours and give you an update soon
16.09.2016 05:44 "olacodeslayer"<EMAIL_ADDRESS>napisał(a):

hi geglio, The routes are pretty much the same as the demo's, i have not
changed anything else aside from removing the # from menu component and
changing the hash setting on the router. i think its a problem only with
the menu items because the breadcrumbs link is working fine and redirects
correctly.
this is the app.routing.ts
import { Routes, RouterModule } from '@angular/router';
export const routes: Routes = [
{ path: '', redirectTo: 'pages', pathMatch: 'full' }
];
export const routing = RouterModule.forRoot(routes, { useHash: false });
and pages.routing.ts
import { Routes, RouterModule }  from '@angular/router';
import { Pages } from './pages.component';
// noinspection TypeScriptValidateTypes
const routes: Routes = [
{
path: 'login',
loadChildren: () => System.import('./login/login.module')
},
{
path: 'register',
loadChildren: () => System.import('./register/register.module')
},
{
path: 'pages',
component: Pages,
children: [
{ path: '', redirectTo: 'dashboard', pathMatch: 'full' },
{ path: 'dashboard', loadChildren: () => System.import('./dashboard/dashboard.module') },
{ path: 'editors', loadChildren: () => System.import('./editors/editors.module') },
//{ path: 'components', loadChildren: () => System.import('./components/components.module') }
{ path: 'charts', loadChildren: () => System.import('./charts/charts.module') },
{ path: 'ui', loadChildren: () => System.import('./ui/ui.module') },
{ path: 'forms', loadChildren: () => System.import('./forms/forms.module') },
{ path: 'tables', loadChildren: () => System.import('./tables/tables.module') },
{ path: 'maps', loadChildren: () => System.import('./maps/maps.module') }
]
}
];
export const routing = RouterModule.forChild(routes);
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/akveo/ng2-admin/issues/258#issuecomment-247511636,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2SJxslr-3XNlHsh0171v5LXsqRkeczks5qqhCjgaJpZM4J-OsP
.

Hi geglio, i have been able to resolve this issue. it turns out that it was the [target]="menuItem.target" in

    {{ menuItem.title }}

causes it to refresh every time it is clicked.
it worked correctly when i removed it. i don't know yet if it has any effects but it works properly for now. thanks a bunch..

Awesome! Good job :) let's leave this issue open so a maintainer can see if
this should be pulled into the repository.
16.09.2016 06:40 "olacodeslayer"<EMAIL_ADDRESS>napisał(a):

Hi geglio, i have been able to resolve this issue. it turns out that it
was the [target]="menuItem.target" in

{{ menuItem.title }}

causes it to refresh every time it is clicked.
it worked correctly when i removed it. i don't know yet if it has any
effects but it works properly for now. thanks a bunch..
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/akveo/ng2-admin/issues/258#issuecomment-247516742,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2SJ0HcdQTGy9b_9iQ9eHGzABJa2s31ks5qqh2lgaJpZM4J-OsP
.


The hide middleware hides the floating element when the referenced element has fixed position

Describe the bug
The hide middleware hides the floating element when the referenced element has position: fixed; and it's inside an element with overflow: hidden;.
To Reproduce

https://codesandbox.io/s/hungry-saha-5fzzb5
Steps to reproduce the behavior:
Note that button 1 with fixed position does not display the tooltip, but button 2 with absolute position does.
Expected behavior
The tooltip must be displayed as long as the referenced element is also visible.
Screenshots

A real example to exemplify our use case:

Context:

OS: OS: macOS Monterey 12.1
Browser: Google Chrome
Version: 112.0.5615.49

Additional context
Add any other context about the problem here.

Seems to be caused by https://github.com/floating-ui/floating-ui/pull/2170; works in 1.2.0

Maybe I'm missing something, but it appears this is still an issue in 1.2.6.

The original sandbox got updated, it fixed the first (this) issue but seems there's another one which has been opened :')

Thanks for clarifying. I'll keep an eye on the proper one. 👀


[Bug]: Contributing markdown has npm commands

Relevant scope
angular application
Description
Setup, Development, and Build have npm commands that haven't been converted to appropriate yarn or nx commands.
Do you want to contribute with a pull request?
No

Those commands are correct. They are just aliases to the correct Nx commands. You can see this in the package.json file.
https://github.com/angular-sanctuary/angular-hub/blob/1d6e47dbf402e76446778ce9cdee2893e9dc827d/package.json#L5-L9
This is common practice so we don't need to memorize commands and stick to npm run build instead of nx build <app name>.

Ok.


[IOTDB-1181] Upgrade jetty jar to fix CVE-2020-27216

current  jetty version is 9.4.24.v20191120,  which have a high loopholes CVE-2020-27216， we can use 9.4.35.v20201120 version

@HTHou @jixuan1989 @qiaojialin  hi,  could you review this?
Thanks...

now we have many dependencies that having problem of CVEs.
It is time to upgrade them but we must guarantee the stability and the peroformance not decreasing.


Versions of libraries in the requirements.txt

Hi, I would like to know if the versions of libraries in the requirements.txt can be provided. Currently I am using the latest TF (2.11) that seems to have breaking change in the optimizer. Thus, the initialization for base class in AdamWeightDecay() (https://github.com/google-research/pix2seq/blob/main/models/model_utils.py#L104) has incorrect arguments compared to the corresponding ones in https://github.com/keras-team/keras/blob/v2.11.0/keras/optimizers/optimizer_experimental/adam.py#L86. Thanks!

I think the code should be compatible with latest (stable) version of tensorflow. AdamWeightDecay is a subclass of the default Adam optimizer in keras, so it is fine to have different input arguments for AdamWeightDecay as long as that for Adam is consistent (which it is).


print_ctrl_ids with unicode

I am using python 3.6.4 and pywinauto 0.6.5 with Google Earth Pro <IP_ADDRESS>91.  All are in 64-bit code versions.  There are some recently updates in Google Earth Pro which caused print_ctrl_ids generating unicodes (such as '\u200b') and errors when writing to a file (but not on the screen).

Any suggestions to deal with this issue.  Doing some searching, I notice that this issue (print_ctlr_ids with unicodes) was brought up in very late 2016 but don't know if this is the same or not.
Is there an option in print_ctrl_ids to save to a string variable instead of to a file to avoid this problem and be more flexible.

Thank for all the support and maintenance for a very useful software, pywinauto.
bbnluu
Capturing of errors
Python 3.6.4 (v3.6.4:d48eceb, Dec 19 2017, 06:54:40) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

import os
from pywinauto import Application, keyboard, mouse
kmlfn = os.path.abspath('rcglic.kml')
os.startfile(kmlfn)
app = Application(backend='uia').connect(path='googleearth.exe')
dlg = app['Google Earth']
dlg.maximize()
<uiawrapper.UIAWrapper - 'Google Earth Pro', Dialog,<PHONE_NUMBER>716585331>
dlg.print_ctrl_ids(filename='tmpids.txt')
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\Brian\AppData\Local\Programs\Python\Python36\lib\site-packages\pywinauto\application.py", line 668, in print_control_identifiers
print_identifiers([this_ctrl, ], log_func=log_func)
File "C:\Users\Brian\AppData\Local\Programs\Python\Python36\lib\site-packages\pywinauto\application.py", line 657, in print_identifiers
print_identifiers(ctrl.children(), current_depth + 1, log_func)
File "C:\Users\Brian\AppData\Local\Programs\Python\Python36\lib\site-packages\pywinauto\application.py", line 657, in print_identifiers
print_identifiers(ctrl.children(), current_depth + 1, log_func)
File "C:\Users\Brian\AppData\Local\Programs\Python\Python36\lib\site-packages\pywinauto\application.py", line 657, in print_identifiers
print_identifiers(ctrl.children(), current_depth + 1, log_func)
[Previous line repeated 6 more times]
File "C:\Users\Brian\AppData\Local\Programs\Python\Python36\lib\site-packages\pywinauto\application.py", line 653, in print_identifiers
log_func(output)
File "C:\Users\Brian\AppData\Local\Programs\Python\Python36\lib\site-packages\pywinauto\application.py", line 666, in log_func
log_file.write(str(msg) + os.linesep)
File "C:\Users\Brian\AppData\Local\Programs\Python\Python36\lib\encodings\cp1252.py", line 19, in encode
return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode character '\u200b' in position 114: character maps to

==================================================================

Hi @bbnluu there was an attempt to fix this issue in PR #564. Still unfinished though.


Assertions json report file is malformed

The value of attribute actualValue is missing an opening '[' causing the json to be malformed.
e.g.
"actualValue": 768.0]

Closed by 70cf2c56ba447ef2ee2d9eab25b286ac86e78a4f


Gopay status data check

Unforunately, not all fields are mandatory in the gopay status response, and I could not found in the documentation, which fields are, so I've added checking to all of them. We had problems with the card_issuer_country, card_issuer_bank and card_brand fields (undefined index notice).

Merged via https://github.com/remp2020/crm-gopay-module/commit/318a917b84823e4e790b544a3d608d7a32d6ab97, tagged as 0.16.1.


RLP encode on-chain to make deposit() simpler

I think we should consider looking into on-chain RLP encoding, assuming it's not too expensive (gas-wise). This would make deposit() significantly simpler by not requiring any user input.

Another option proposed by @DavidKnott is to include deposits as some special transaction keccak256(msg.sender, msg.value). This would increase the simplicity of deposit transactions at the cost of slightly increased complexity during the validation process. Deposit blocks should eventually be easily identifiable if we implement a "decimals for deposits" mechanism.

Closed as we're now using the mechanism proposed by David.


[Bug] Error in the console failed to load manifesto

Describe the bug
In the console there is an error:
Failed to load resource: the server responded with a status of 401 https://podkrepi.bg/site.webmanifest
To Reproduce
When loaded, the site gives the error in the console

Go to 'podkrepi.bg'
Click on 'Inspect Element' anywhere
Check the console
See error

Expected behavior
No error.
Screenshots

Desktop (please complete the following information):

OS: MacOS BigSur 11.2.3
Browser Opera GX
Version LVL2 (core: 73.0.3856.400)

This problem is caused by the http password that we still require in order to access the website.
Once the password is gone, this problem will be gone as well

This should be OK now


[quickfix]: fix exception specification

Fixes #31380

[X] Changes comply with the maintainer guide
[X] SHA512s are updated for each updated download
[X] ~The "supports" clause reflects platforms that may be fixed by this new version~
[X] ~Any fixed CI baseline entries are removed from that file. ~
[X] ~Any patches that are no longer applied are deleted from the port's directory.~
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

Hi,
I think there's a problem with the exception specification QUICKFIX_THROW. It is defined as no_except. When added in a function declaration, it means that when something inside this function throws an exception, the function will terminate instead of passing the exception to the caller. This completely change the excepted behaviour of the library.
The current version of quick fix in git hub uses an EXCEPT that just resolves to noexcept(false) which is good.


Add CONTRIBUTION CODE OF CONDUCT

I think we will need contribution code of conduct to guide everyone.

what is contribution code of conduct Philips?

Code of conduct is a guideline of manners that are tolerated amongst everyone who wishes to contribute or participate in this project. This is to avoid conflicts, hope this explanation is clear enough.

@Kellsonphilips I see. Thank you! I totally forgot about that. Adding it now.

That's right.

@Kellsonphilips I added one, so please check, and if okay, close this issue. Thank you!

That's perfect


Unable to re-start the ALVR server once it has been closed

Description

After closing the Dashboard, I'm unable to re-start the ALVR server, neither from the start menu, nor from the command line. Sometimes there's a brief flicker, suggesting that the GUI appears, just to disappear again in the next moment.
Unfortunately, there's no log or any other messages being printed to the console when directly issuing alvr_launcher from the terminal.
General Troubleshooting

[x] I carefully followed the instructions in the README and successfully completed the setup wizard
[x] I read the ALVR Wikis here and here

Environement
Hardware
Note: for Linux, an upload to the hw-probe database is preferred: hw-probe -all -upload
https://linux-hardware.org/?probe=bc0462d8e3
CPU:
GPU:
Audio:
Installation
ALVR Version:
17.0.1+1643781622+0541c19e on both client and server
SteamVR Version:
1.20.4
Install Type:

[x] Packaged (exe, deb, rpm, etc)
[ ] Portable (zip)
[ ] Source

OS Name and Version (winver on Windows or grep PRETTY_NAME /etc/os-release on most Linux distributions):

Debian GNU/Linux 11 (bullseye) (up-to-date)

i see this behaviour if steam and/or steamvr is still open, normally i start steam (revoke its window) and exit it from the menu. after this alvr is working again

i see this behaviour if steam and/or steamvr is still open, normally i start steam (revoke its window) and exit it from the menu. after this alvr is working again

Confirmed -- after stopping Steam, I can successfully re-launch ALVR! Good to know, but this somehow doesn't seem right, does it?

it's SteamVR not stopping properly
run...
ps aux | grep -iE 'steam|vr'
and kill it's runaway processes. (usually there a "reaper" process which is meant to kill this stuff but, it falls over sometimes. just killing that can usually get rid of all the other processes)

Hi Niroc,

it's SteamVR not stopping properly run... ps aux | grep -iE 'steam|vr'
and kill it's runaway processes. (usually there a "reaper" process which is meant to kill this stuff but, it falls over sometimes. just killing that can usually get rid of all the other processes)

at least in my case following subjectdenied's advice of just closing Steam seems to be sufficient. Not that I'm afraid of using the command line, but two mouse clicks are hard to beat! ;-)
Cheers --
Torsten

Is this issue still occurring?


No member named _0

CC=gcc CXX=g++ make -f minicargo.mk is failing with many errors like the following:
output/libcore.hir.o.c:156961:15: error: ‘struct s__ZN4core3str14ParseBoolError’ has no member named ‘_0’
  var0 = & arg0->_0; // Local(0) = Borrow(0, Shared, Field(0, Deref(Argument(0))))
               ^
output/libcore.hir.o.c:156962:4: error: ‘struct s__ZN4core3str14ParseBoolError’ has no member named ‘_0’
  rv._0 = (*var0); // Return = Struct(::"core"::str::ParseBoolError, {Deref(Local(0))})
    ^

Same error in Travis. It looks like this was introduced in c7e76526b54c496708411bf9afcfba2dfa8e4014.

Known issue, travis is failing with it.

Fixed in 277c6721

Confirmed fixed. Thanks!


poh-pinned-cpu-core on isolated core

i have isolated core 2 for the thread solPohTickProd using grub, but when i set --experimental-poh-pinned-cpu-core to 2, the process gets pinned to core 3. as far as i know, core_affinity creates logical core ids, and these ids do not match the physical core ids. core_affinity sees my cores as:

core 0 (id: 0)
core 1 (id: 1)
core 2 (isolated)
core 3 (id: 2)
core 4 (id: 3)

core_affinity::get_core_ids() cannot get the isolated cores.
solution
set forced core with core_affinity::set_for_current() and skip the get_core_ids:
--experimental-poh-pinned-cpu-core 2 force
workaround
#!/bin/bash

# wait to load the binary
#sleep 120

# main pid of solana-validator
solana_pid=$(pgrep -f "^solana-validator --identity")
if [ -z "$solana_pid" ]; then
    logger "set_affinity: solana_validator_404"
    exit 1
fi

# find thread id
thread_pid=$(ps -T -p $solana_pid -o spid,comm | grep 'solPohTickProd' | awk '{print $1}')
if [ -z "$thread_pid" ]; then
    logger "set_affinity: solPohTickProd_404"
    exit 1
fi

current_affinity=$(taskset -cp $thread_pid 2>&1 | awk '{print $NF}')
if [ "$current_affinity" == "2" ]; then
    logger "set_affinity: solPohTickProd_already_set"
    exit 1
else
    # set poh to cpu2
    sudo taskset -cp 2 $thread_pid
    logger "set_affinity: set_done"
     # $thread_pid
fi

I would use cgroups for that and also move hardware interrupts from that cpu away

I would use cgroups for that and also move hardware interrupts from that cpu away

You can do this with cpusets. The cpuset equivalent of isolcpus is the cpu_exclusive flag you can read about in cpuset(7).


OpenCV and C++ version

Hi,
I'm trying o run the code on visual studio and having some trouble..
I think the visual studio 2013 is not compatible with al the C++ features that are used in your code. could this be true?
in addition could you please specify what openCV version should I use?
thanks a lot,
shelef

We use the version of OpenCV that ships with ROS Melodic. There are a number of modern features of C++ that are used. As such I can't recommend an older compiler.

hi,
thank you very much for the information.
the code is working wonderfully!
Liran
On Mon, May 27, 2019 at 10:44 PM Kenneth Chaney<EMAIL_ADDRESS>wrote:

We use the version of OpenCV that ships with ROS Melodic. There are a
number of modern features of C++ that are used. As such I can't recommend
an older compiler.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/daniilidis-group/msckf_mono/issues/20?email_source=notifications&email_token=AMAGI33DPC3GQI4C42QKSADPXQ2YJA5CNFSM4HLGHSD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWKNLPY#issuecomment-496293311,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMAGI3ZPZNHKCJFSTF3CFZ3PXQ2YJANCNFSM4HLGHSDQ
.


add cluster list cmd to docs

#1298  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @BelSasha and the rest of your teammates on  Graphite


Tuesday, July 5th 2016 - Scratch Pad + Summary

bundle exec rake test TEST=test/acceptance/team_test.rb

Run one mini test...
ruby path/to/test/file_test.rb --name name_of_test_method_name
Example...
ruby test/exercism/user_test.rb --name test_user_not_a_guest

Kit says:The Mystery Guest

[The mystery guest anti-pattern] (https://robots.thoughtbot.com/mystery-guest) is where the test reader is not able to see the cause and effect between fixture and verification logic because part of it is done outside the Test Method.


bugfix / NumberMessageHandler was missing command handler registratio…

…n in chanel configuration
improvement / channel/channelType trace logging added with more fields than just UID

Thanks for contributing @smolbo !


[#5276] Update code analysis to latest VS recommendations - Fix error rules - Part 4

Addresses #5276
#minor

IMPORTANT!
Note: These changes are being address as part of enabling the .NET Analyzers library in the PR #6251.

Description
This PR removes unnecessary initialization, and forwards the CancellationToken to methods that take one.
List:

CA2016
CA1805

Specific changes

Removes unnecessary initialization variables and properties.
Forwards the CancellationToken in methods that take one, from the parent method.
Disable a few cases where forwarding the CancellationToken doesn't work.

Testing
This image shows the CI pipeline successfully running after the changes.

We will not address this at the moment.


Fix all the license headers

Originally the wrong boiler plate got copied in which suggested this
code was licensed to the Apache Software Foundation. It is not.
This also updates the date ranges to 2017 and adds missing license
headers to some of the Python files.
Closes #24

Ping @econchick


Sd

--

Codecov Report

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

@@            Coverage Diff             @@
##           master    #2286      +/-   ##
==========================================
- Coverage   78.53%   75.72%   -2.81%
==========================================
  Files          36       36
  Lines        3745     3732      -13
  Branches      343        0     -343
==========================================
- Hits         2941     2826     -115
- Misses        780      906     +126
+ Partials       24        0      -24

Impacted Files
Coverage Δ

Tests/Tests/SDWebImageDownloaderTests.m
82.84% <0%> (-14.23%)
:arrow_down:

SDWebImage/SDWebImagePrefetcher.m
79.26% <0%> (-10.98%)
:arrow_down:

SDWebImage/SDWebImageManager.m
73.8% <0%> (-10%)
:arrow_down:

Tests/Tests/SDWebImageManagerTests.m
92.13% <0%> (-7.87%)
:arrow_down:

...age/FLAnimatedImage/FLAnimatedImageView+WebCache.m
74.57% <0%> (-6.78%)
:arrow_down:

Tests/Tests/SDImageCacheTests.m
94.59% <0%> (-5.41%)
:arrow_down:

SDWebImage/SDWebImageDownloaderOperation.m
87.37% <0%> (-4.81%)
:arrow_down:

SDWebImage/SDWebImageDownloader.m
78.35% <0%> (-3.82%)
:arrow_down:

SDWebImage/SDImageCache.m
65.34% <0%> (-3.47%)
:arrow_down:

SDWebImage/NSData+ImageContentType.m
46.77% <0%> (-1.56%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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


Split autodiscovery methods into gradle builds

This commit should be faily self-explainatory. It splits the
three autodiscovery methods into their own gradle builds so
they can be plugged in and out easily. This commit doesn't
necessarily make the latter possible though.
Dependency graph should look something like this now:
  app:ui
  |
  |\app:autodiscovery:api
   \|
    |\app:autodiscovery:providersxml
    |
    |\app:autodiscovery:srvrecords
    |
     \app:autodiscovery:thunderbird

@cketti A few points for your consideration:

I think there may be a bunch of "unnecessary" changes (automatically created gradle/IDE files)
There's some weird DI arrangement possibly
Not sure how easily methods can be plugged in and out
Might be some unnecessary stuff in the new build.gradle files.

@cketti Just pushed the changes you requested :+1:


large set of changes needed to get our libs to deploy to npm

most of these changes dont need to be looked at...

let me know if you have feedback, and ill fix it tomorrow, i want to test this in prod


Embed Report is now under File menu

The page mentions:
On the Share menu, select Embed report > SharePoint Online.
However, on my Power BI Service, the Embed report submenu is found inside the File menu.

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

ID: 0fc5abbd-e082-d1e1-109d-6f6cec8854a1
Version Independent ID: f8affce3-98f2-0cc6-b4de-d9af7ced0fbe
Content: Embed a report web part in SharePoint Online - Power BI
Content Source: powerbi-docs/collaborate-share/service-embed-report-spo.md
Service: powerbi
Sub-service: pbi-collaborate-share
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hey @DesertDwarf, thanks for calling this out. I've replaced the offending image. Should be online in a few hours. Many thanks!


Fix sr-frontend issues

Describe your changes

Fix bad typing for datamine
Fix missing dependency for build

Issue or discord link

Testing/validation

Checklist before requesting a review (leave this PR as draft if any part of this list is not done.)

[x] I have commented my code, in hard-to understand areas.
[x] I have made corresponding changes to README or wiki.
[x] For front-end changes, I have updated the corresponding English translations.
[x] Ran yarn run mini-ci locally to validate format + lint.
[x] If there were format issues, I ran nx format write to resolve them automatically.

Invalidated by https://github.com/frzyc/genshin-optimizer/pull/1334


possible need for note-off and note-on prioritizing

When I was authoring some phrase like this:
1   @30 o4 V110 v100 l8 RSD20 DSD40
    Kc+Kf+
    [ fg(50g)50g^2.b4a4f4 : fg(50g)50g^1. ]2  fg(50g)50g^1f2  [r-1]8
    [ de(50e)50e^2.g4f4d4 : de(50e)50e^1. ]2  de(50e)50e^1d2  [r-1]8
    < [ ab(50b)50b^2.>e4d4<a4 : ab(50b)50b^1. ]2  ab(50b)50b^1a2

It is like, I wanted to enter each note on the tritone (possibly quadtone, after this) chords respectively and entered MML like this, making use of [r-1] to roll back. However at the end of the fourth bar (at b4a4f4 bar), only a4 is noted on.
It is because the other chord notes emit note-offs, resulting that the first note-on events (e.g. b4a4f4) are suppressed by the following note-offs of the chord note sequence (e.g. g4f4d4, note that f4 is connected).
It can be easily workarounded if the song gives q operation e.g. q1 but that's not ideal.

Solution:

(1) detect and warn about zero-length note operations. It has points because this kind of problems may happen in other use cases, but does not resolve the present issue.
(2) detect zero-length note operations and alter note-on/off operations by prioritizing them, if applicable. It should exactly resolve the problem, but complicates the MML processing rules.


Screenshots for Pebble Time

It looks like the screenshot functionality doesn't work right now for Pebble Time. I get this error:
Screenshot failed: Unrecognised image format.

For now, you can use the emulator and save its image (right click -> save image as).


CMake

Needs an automated pass-off test to earn the badge.

 It was great!
There were lots of things to watch/read, but it didn't feel like there were lots of activities to practice.
A quiz and resulting badge just like the Make module has would be good!
I personally had a hard time following the video for this module. I think it might be because the terminals were very messy. That might just be me though.
None
Cmake is much more convineint then make
This tutorial provided plenty of information on cmake, which I liked.
it was pretty good
I think it was good.
Still confused, but I will keep reviewing as I need it in my research this summer
Great resources and talk, I look forward to using this in practice.
It was good.
The beginning of the video was new and I learned something but definitely caused me to be zoned out by the time it began to cover cmake. I enjoyed the live lecture more.
none
I really liked this module.
The video was a little confusing but Dr Goeders helped clarify some things for during the lecture hour. I like Cmake more than make.
There is a lot of material to learn many things about cmake.

Add quizes to the activities.
Finish the Pass-Off Test.
Add Video Chapters for the Youtube Video.


gurunudi.ai.APIError: server_inaccessible

when i am running
from gurunudi import AI,lang
ai=AI()
corrected_text = ai.autocorrect("who is the primem inister of idnia")
print(corrected_text)
giving me gurunudi.ai.APIError: server_inaccessible error

The server is live from today. Please try again. More APIs will be added going forward and existing ones will be further updated to increase accuracy. Feel free to post any issues.

from gurunudi import AI,lang
ai=AI()
definition=ai.define("ಕನ್ನಡ",lang.KANNADA)
print(definition)
Giving below error
gurunudi.ai.APIError: status_code_404

Sever is down again!


Selective dependency upgrade

Alternative to https://github.com/joukevandermaas/saule/pull/216 without upgrading Humanizer.Core which appears to be pulling in alot of new dependencies.

Further details on the change in Humanizer.Core at https://github.com/Humanizr/Humanizer/issues/798.


fix sorries

I've fixed all sorries to build the project.
Just grepping, it looks like there are other sorries in a file in the Unused directory. Are we also supposed to fix these?

Thanks, this PR looks very good. If you fixed all "TODO-MR" comments, things are fine.
(The code in the Unused directory has not really been fixed up since porting to Lean 4. Every now and then, I would gradually fix some things... and it's also unused. So: fixing it is of course welcome, but not required.)

I decided to just merge the PR into mine. Thanks again!


cleanup(pubsub): move detach subscription example

This change is

Part of https://github.com/googleapis/google-cloud-cpp/issues/12987


Final project submission

In this project, I create a budget app that allows a user to keep track of their expenses.
Accomplishments:

Splash screen

[ ] A simple page with the name of your app (yes, you need to choose one), and links to the sign up and log in pages.

Sign up and log in pages

[ ] The user should be able to register in the app with full name, email and password (all mandatory).

[ ] The user can log into the app using email and password.

[ ] If the user is not logged in, they can't access pages that require the user to be logged in (all the pages described below).

Home page (categories page)

[ ] When the user logs in, they are presented with the categories page.

[ ] For each category, the user can see their name, icon and the total amount of all the transactions that belongs to that category.

[ ] When the user clicks (or taps) on a category item, the application navigates to the transactions page for that category.

[ ] There is a button "add a new category" at the bottom that brings the user to the page to create a new category.

Transactions page

[ ] For a given category, the list of transactions is presented, ordered by the most recent.

[ ] At the top of the page the user could see the total amount for the category (sum of all of the amounts of the transactions in that category).

[ ] There is a button "add a new transaction" at the bottom that brings the user to the page to create a new transaction.

[ ] When the user clicks on the "Back" button (<), the user navigates to the home page.

"Add a new category" page

[ ] The user fills out a form to create a new category, indicating their name and icon (both mandatory).

[ ] The user clicks (or taps) the "Save" button to create the new category, and is taken to the home page on success.

[ ] When the user clicks on the "Back" button (<), the user navigates to the home page.

[ ] "Add a new transaction" page

The user fills out a form to create a new transaction with:

[ ] name (mandatory)

[ ] amount (mandatory)

[ ] categories (mandatory at least one)

[ ] The user click (or taps) the "Save" button to create the new transaction, and is taken to the transactions page for that category.

[ ] When the user clicks on the "Back" button (<), the user navigates to the transactions page for that category.

Testing requirements

[ ] Create unit and integration tests for all the most important components of your RoR application.

Thank you @Reem-lab for the time to look at my code and the suggestions. 🙏🏽


Skyline Problem

A city's skyline is the outer contour of the silhouette formed by all the buildings in that city when viewed from a distance. Given the locations and heights of all the buildings, return the skyline formed by these buildings collectively.
The geometric information of each building is given in the array buildings where buildings[i] = [lefti, righti, heighti]:
left-i is the x coordinate of the left edge of the ith building.
right-i is the x coordinate of the right edge of the ith building.
height-i is the height of the ith building.

You may assume all buildings are perfect rectangles grounded on an absolutely flat surface at height 0.
The skyline should be represented as a list of "key points" sorted by their x-coordinate in the form [[x1,y1],[x2,y2],...]. Each key point is the left endpoint of some horizontal segment in the skyline except the last point in the list, which always has a y-coordinate 0 and is used to mark the skyline's termination where the rightmost building ends. Any ground between the leftmost and rightmost buildings should be part of the skyline's contour.
Note: There must be no consecutive horizontal lines of equal height in the output skyline. For instance, [...,[2 3],[4 5],[7 5],[11 5],[12 7],...] is not acceptable; the three lines of height 5 should be merged into one in the final output as such: [...,[2 3],[4 5],[12 7],...]
Example 1:
Input: buildings = [[2,9,10],[3,7,15],[5,12,12],[15,20,10],[19,24,8]]
Output: [[2,10],[3,15],[7,12],[12,0],[15,10],[20,8],[24,0]]
Explanation:
Figure A shows the buildings of the input.
Figure B shows the skyline formed by those buildings. The red points in figure B represent the key points in the output list.

Example 2:
Input: buildings = [[0,2,3],[2,5,3]]
Output: [[0,3],[5,0]]

Hi there!
I am the Mentor of this project.
Although this repo is not part of Hacktoberfest 2024, we have some great alternatives for you. Welcome to SkillShow, a repository focused on skill enhancement!
Guidelines:

Please adhere to the existing guidelines.
No additional rules, thanks!
Check it out here: SkillShow.

Note: Start by creating an issue. Once it’s assigned to you, feel free to open a pull request (PR). Directly opening a PR will lead to an invalid label.
Looking forward to your contributions!


[Doubt] Model options params are not set on model

The collection sets this.params with options.params and this doesn't happen in models, I think would be good to have the same approach. I am not entirely sure how common would be.
https://github.com/rendrjs/rendr/blob/master/shared/base/collection.js#L38
https://github.com/rendrjs/rendr/blob/master/shared/base/model.js#L22
My use case is, our API have sub resources associated to a resource:
GET /venues => collection of venues
GET /venues/:id => specific venues
GET /venues/:venueId/musicGenres => music genres from a specific venue
POST /venues/:venueId/musicGenres => add a music genres to a specific venue
DEL /venues/:venueId/musicGenres/:id => music genres from a specific venue
GET /musicGenres => list of available music genres

So when I execute fetch on MusicGenres collection with a venue id, I would like the options.params of the model be the same as collection, this would allow to do:
venueMusicGenresCollection.remove(id).destroy();

So since, my model url is conditional to params on model, this would execute:
DEL /venues/:venueId/musicGenres/:id => music genres from a specific venue

At the moment I will be overriding model constructor to due this after executing default implementation, but maybe would be reasonable to do it on rendr model, or not. :)

Maybe it make sense for other uses cases but not for this one anymore, I figured that is going to out of scope. Sorry! :)


JsonSerializer enum serialization differs from Newtonsoft

Describe the bug
The System.Text.Json JsonSerializer will serialize enums with their underlying type rather than the string representation, which is a breaking change from Newtonsoft's behavior.
In addition, there doesn't appear to be a JsonSerializerOption to get matching behavior.
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core 3.0.0-preview7.19365.7
Return an object containing an enum type. In my particular case, I returned GeoJSON using https://github.com/GeoJSON-Net/GeoJSON.Net, from an API controller, like so:

        [HttpGet("[action]")]
        public IActionResult Example()
        {
            var geom = new GeoJSON.Net.Geometry.Point(new GeoJSON.Net.Geometry.Position(50, 50));

            return Ok(geom);
        }

Expected behavior
This should return the following JSON:
{"type":"Point","coordinates":[50.0,50.0]}

Expected behavior
With System.Text.Json, it instead returns the following JSON:
{"type":0,"coordinates":{"altitude":null,"latitude":50,"longitude":50},"boundingBoxes":null,"crs":null}

Notice in particular that "type" is set to 0, which is invalid GeoJSON.

You could configure JsonOptions to use JsonStringEnumConverter as part of your start up to have it produce the string enum value rather than the numeric value. The library you list specifically adds converters for JSON.NET.
If you need the serialized behavior to be identical,
a) you'd  would have to write custom converters for System.Text.Json written in the same vein as the library.
b) continue using Json.Net
/cc @ahsonkhan \ @steveharter

Thanks for contacting us. We believe that the question you've raised have been answered. If you still feel a need to continue the discussion, feel free to reopen it and add your comments.

The library you list specifically adds converters for JSON.NET.

Good point; I missed that. Thanks!


LFX Mentorship is a new name for CommunityBridge

What is the current behavior?
https://github.com/github/docs/blob/main/content/github/administering-a-repository/displaying-a-sponsor-button-in-your-repository.md#about-funding-files
What changes are you suggesting?
Community Bridge is now called LFX (specifically, LFX Mentorship for the mentoring purposes) - https://lfx.linuxfoundation.org/blog/introducing-lfx-tools-for-everyone/
Please update the wording where necessary.

👋  Hi @idvoretskyi, this is a great addition to make our docs more accurate! You or anyone else are welcome to open a PR ✨
For whoever opens a PR, it would be helpful to refer to LFX Mentorship as "LFX Mentorship (formerly CommunityBridges)" since the syntax isn't changing and people may still search for CommunityBridges.

@ethanpalm great, I'm happy to submit a PR.
Is there any other place that needs to be updated except (https://github.com/github/docs/blob/main/content/github/administering-a-repository/displaying-a-sponsor-button-in-your-repository.md#about-funding-files)?

@idvoretskyi, I searched and that's the only article in the repo that mentions CommunityBridge. 👍

@ethanpalm here we go - https://github.com/github/docs/pull/3928


test that app launches

to avoid #90 and similar kerfuffles.
Will be key to ensure similar runtime conditions.
Full-on docker runtime conditions are in #93

test in #29 covers this.


Opening files with handlr does not respect defaults set in ~/.config/mimeapps.list

Hi, having a strange issue where certain files simply do not open using the correct application as set in my ~/.config/mimeapps.list.
For instance, when opening a .opus file via handlr open, I expect it to use mpv, but instead the file opens in Kid3, an audio tag editing application. Calling handlr list correctly shows that audio/opus and audio/ogg are associated with mpv.desktop. Not sure what I am misunderstanding or doing wrong here. The output of handlr list as well as my mimeapps.list are attached below.
handlrlist.txt
mimeapps.list.txt

That certainly is a strange issue. Would you mind running handlr mime on the file and posting the output?

Thank you for the nudge in the right direction. It turns out that the mimetype was recognized as audio/x-opus+ogg, which is not a mimetype in the default .desktop entry for mpv. A simple handlr set audio/x-opus+ogg mpv.desktop resolved the issue for me.

Glad I could help!
Also, you might want to consider using wildcards like audio/* if you want all mimetypes of a certain category to open in the same program. It would prevent similar problems like this from arising in the future. And you can always override it for individual mimetypes by defining handlers for those mimetypes anyways.
Although I would note that this is almost certainly incompatible with other file openers that use mimeapps.list if that is a concern.


[Documentation] steps to execute tfd planer added

Hi!
With this pull request I  updated the README documentation to execute tfd easier, providing the basic commands. I hope it would be useful for the community.
Cheers.
Signed-off-by: P4B5<EMAIL_ADDRESS>

Hi @P4B5
This repo is devoted to the use of TFD as a plugin for PlanSys2. The Planner uses it via this plugin. Maybe if you change the section title to "Manual execution of TFD" or "Testing TFD installation," it would be more appropriate.
Thanks!"

Thanks!!!


Adds one more step before restart the game

(From Target-1 Akiba)
Adds one more step before restart the "Count Up" or "Time Attack" games.
Current:

Game start
Game end
(Push button)
Game restart

After:

Game start
Game end
(Push button)
New step for waiting button
(Push button)
Game restart

This feature is implemented by commit de83c3b


Future  attached to a different loop

Hello, I've running this on my Windows 10 machine and it works fine, however when I tried running it on my Raspberry Pi 4B machine. It got an error that says
Exception: Task <Task pending name='Task-5' coro=<Event.wait() running at /usr/lib/python3.9/asyncio/locks.py:226> cb=[_release_waiter(<Future pendi...f80b8d0d0>()]>)() at /usr/lib/python3.9/asyncio/tasks.py:416]> got Future <Future pending> attached to a different loop

Any idea why this is happening?

Maybe revert this change for Python 3.10; it was about event loops: fix for 'DeprecationWarning: There is no current event loop' in Python 3.10. Then try again with Python 3.9 on Raspberry.

It now works thanks for you help
I may need to mention this but I found out its possible to pair again in case you've lost the encryption key but will lose all data in the blood pressure monitor because it needs to be reset
To reset, just press then start button until the bluetooth icon flash and press the time logo part at the upper part
Anyways, it has been a great help thank you

Thanks for the tip! I did not know how to reset the unit.


English - Basic Modules

Closes https://github.com/haskellfoundation/HaskellSchool/issues/26

I added some comments related to grammar (just for the start of the article), but I'll probably have to learn more about modules myself to make a more substantive review.

Here's a page screenshot for other reviewers.

Rebasing this PR onto development fixed the issue for me locally. I'd say try that.

@joelmccracken I was really hoping that would just work

drat, sorry. I was really hoping, too.
this version of the branch is still fine locally, so I cloned it and am gonna investigate further.

Well, i have the problem occurring on my fork of this project, so, progress?


Eslint: disallow space before fn paren

also disallow space before paren in function call

There is more stuff to tweak, but let's wait for the final code style rules.


(bug)

What happened?
when it will be finished?
OS
Android (Jailed)
Opti Loader
OptiManager (Android)
Relevant log output

Code of Conduct

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

I'm still not sure when it will be out yet since I don't have the proper resources to develop on Android 😢
I'm basically a 1 person development team on the coding end, and I do not have access to an Android phone to add support for Opti. Although, I may try to get an android phone during the holidays! Once I do, development will speed up for Opti on android.

its okay you should reach out for contributors

I will try! I do have a few others on the team and I'll let them know about this
On the other hand, I may or may not have some good news if this works

Merry Christmas!!! I finally have an Android phone to develop on, so progress will be coming soon!

Not trying to notification spam, but I have great news.

Not trying to notification spam, but I have great news.

glad to hear I'll wait for it
but would like you to suggest some things
make it different we have bunny, revenge, vendetta all same
try to makr different ui like i remember there was a project opencord which had great ui hut it got discontinued
and add in built plugin list so we won't have to insert url to install plugin  just search or scroll and install plugin


Fix bug with my previous PR 😅

Hi there, I'm back with another PR. My previous work contained a whopper of a bug and the encoding/decoding loop of the cookie session couldn't possibly work. I'm really sorry. These commits fix the bug, and ensure the roundtrip test covers the previously-faulty logic.

That PR reminds me why I have not used binary in so long.
Here are some further improvements if you care to comment on before I do a release: 5cdf9faf0440caf92391c6c89c7143dc6cc24e7e

Follow-up improvements look great. Thank you!


Failed reading the chunked-encoded stream
Closing connection 18
TLSv1.2 (IN), TLS header, Supplemental data (23):
TLSv1.2 (OUT), TLS header, Supplemental data (23):
TLSv1.3 (OUT), TLS alert, close notify (256):

Reproduction Steps
No response
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
If you use Postman, then everything is successful.
Insomnia Version
2022.7.5
What operating system are you using?
Windows
Operating System Version
Microsoft Windows [Version 10.0.19042.1526]
Installation method
https://updates.insomnia.rest/downloads/windows/latest?app=com.insomnia.app&source=website
Last Known Working Insomnia version
2022.7.5

Hi @gh-qqwweasdzxc please try to edit the description to make it a bit more readable what the issue is.
In other http clients, when this error happens:
Unrecognized content encoding type. libcurl understands deflate, gzip, br content encodings.
Failed reading the chunked-encoded stream

It might be due to a handful of cases. I would suggest you double check your request headers:

Content-Encodin: UTF-8 where Encodin should probably be Encoding
 Accept-Encoding: gzip, deflate, br might work bettter with just  Accept-Encoding: gzip, deflate
 Accept: / should probably be  Accept: */*

Thanks.
I followed Your recommendations, but
the "Could not read the fragmented encoded stream"
remains.
GET /nifi-api/templates/0f1ba51e-6bab-4836-8403-cf0c046800a5/download HTTP/1.1
Host: server1:58443
User-Agent: insomnia/2022.7.5
Cookie: __Secure-Request-Token=3e7a7893-a3b0-40d6-b668-ed637d50af72
Accept: /
Encoding: UTF-8
Accept-Encoding: gzip, deflate
TLSv1.2 (IN), TLS header, Supplemental data (23):
TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
TLSv1.2 (IN), TLS header, Supplemental data (23):
Mark bundle as not supporting multiuse
HTTP/1.1 200 OK
Date: Mon, 23 Jan 2023 21:40:07 GMT
Cache-Control: private, no-cache, no-store, no-transform
Content-Encoding: UTF-8
Content-Disposition: attachment; filename* = UTF-8''Base64EncodeContent.xml
Content-Type: application/xml
Vary: Accept-Encoding
Content-Security-Policy: frame-ancestors 'self'
Strict-Transport-Security: max-age=31536000 ; includeSubDomains
X-Content-Type-Options: nosniff
X-Frame-Options: SAMEORIGIN
X-XSS-Protection: 1; mode=block
Transfer-Encoding: chunked
Received 15.5 KB chunk
Unrecognized content encoding type. libcurl understands deflate, gzip, br content encodings.
Failed reading the chunked-encoded stream
Closing connection 27
TLSv1.2 (IN), TLS header, Supplemental data (23):
TLSv1.2 (OUT), TLS header, Supplemental data (23):
TLSv1.3 (OUT), TLS alert, close notify (256):


Fix issue with dynamicpruning filters used in converted GPU scans when S3 paths are replaced with alluxio [databricks]

Fixes #7933
This handles the scenario where CONVERT_TIME is used as the Alluxio path replacement algorithm and one of the scans being converted has a dynamic pruning filter in its partitionFilters. In this scenario, on Databricks, the dynamicpruningexpression (of type DynamicPruningSubquery) actually doesn't have the same inheritance tree as in Apache Spark, so AlluxioUtils needs to check if it is of type DynamicPruning instead to ensure that it's skipped over in this logic.
Verifying DPP is working correctly here, for NDS query 7 (referenced in #7933), the scan that would contain the dynamicpruningexpression shows the following set of output rows in store_sales for each of the following scenarios:

DPP
Alluxio Automount
Number of Output Rows

disabled
disabled
8,251,124,389

enabled
disabled
1,632,403,114

disabled
enabled
8,251,124,389

enabled
enabled
1,632,403,114

Alluxio has no effect on DPP reducing the number of output rows, so therefore it should work correctly with DPP now.

build

build

retriggered build, previous timeout due to internal networking slowness


Validating elastic failing

When I use logtrail I see this:

Looking at issues in the repo I think it's because a request to validate indexes returns false. Usking Chrome dev tools and copying the request as curl I see the below
curl 'http://localhost:5601/logtrail/validate/es'
-H 'Pragma: no-cache'
-H 'Origin: http://localhost:5601'
-H 'Accept-Encoding: gzip, deflate, br'
-H 'Accept-Language: en-US,en;q=0.8,en-GB;q=0.6'
-H 'kbn-version: 5.4.1'
-H 'User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.100 Safari/537.36'
-H 'Content-Type: application/json;charset=UTF-8'
-H 'Accept: application/json, text/plain, */*'
-H 'Cache-Control: no-cache'
-H 'Referer: http://localhost:5601/app/logtrail'
-H 'Connection: keep-alive' -H 'DNT: 1'
--data-binary '{"index":"fluentd-*"}'
--compressed

The response is:
{"ok":false,"resp":{}}

Is this the reason for the error and if so how do I fix it because Kibana works fine with this index apart from logtrail.
Thanks
I'm using Kibana 5.4.1 and Logtrail 0.1.16

Can you share your logtrail.json file? Also can you paste the response details from the Network tab of Chrome Dev Tools for validate/es request?

{
  "index_patterns" : [
    {
      "es": {
        "default_index": "fluentd-*",
        "allow_url_parameter": false
      },
      "tail_interval_in_seconds": 10,
      "es_index_time_offset_in_seconds": 0,
      "display_timezone": "local",
      "display_timestamp_format": "YYYY-MM-DD HH:mm:ss",
      "max_buckets": 500,
      "nested_objects" : true,
      "default_time_range_in_days" : 0,
      "max_hosts": 100,
      "max_events_to_keep_in_viewer": 5000,
      "fields" : {
        "mapping" : {
            "timestamp" : "ts",
            "display_timestamp" : "ts",
            "hostname" : "@log_name",
            "program": "",
            "message": "info.Message"
        },
        "message_format": "{{{ThreadId}}} {{{level}}} {{{info.Component}}} {{{info.Method}}} {{{info.Message}}}"
      }
    }
  ]
}

Will get the screenshot a bit later

ah, looking at the json, i'm not sure we have a filed called ts, will double check

ah we do

Also check if there are any errors in Kibana server logs.

Here's the screenshot of the request.  I changed the logtrail.json to make the timestamp property use the timestamp field not the ts one and it still errors:

Can you select the Response tab for this request in Network section of
chrome dev tools and get the screenshot? The attached screenshot shows only
the headers. Also check if there are any errors in Kibana server logs.
On Tue, Nov 14, 2017 at 3:19 PM, Jonathan Channon<EMAIL_ADDRESS>wrote:

Here's the screenshot of the request. I changed the logtrail.json to make
the timestamp property use the timestamp field not the ts one and it still
errors:
[image: screen shot 2017-11-14 at 09 49 04]
https://user-images.githubusercontent.com/105126/32773471-2c1e0ff0-c921-11e7-888f-3abba8672afb.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sivasamyk/logtrail/issues/201#issuecomment-344202968,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZXemKOyePN02qUwUBqEkMsasprlqy3ks5s2WJHgaJpZM4QcHq4
.

--
Regards,
Siva.

I've listed the errors in the server logs above

Looks like the fieldStats query is failing for the index. Can you check if the timestamp field is present in this index? You can also manually execute the fieldStats query in elasticsearch for this field and verify. Checkout https://www.elastic.co/guide/en/elasticsearch/reference/current/search-field-stats.html for the curl API details.

My bad I changed the json to look for timestamp not @timestamp, once i had @timestamp all was well. Thanks


Register

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/38191
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.0 -m "<description of version>" 02dc934c51dc8389b08e3dab0831915d5f6c5643
git push origin v1.0.0


Q: How to control read & write cache size limit ?

Hello,
Many thanks to s3fs contributors and very sorry for asking here.
I think s3fs might have option for fsspec maxblocks option to control cache size limit,  but I cannot find  a clue at doc and source code.
Is there any way to control max read and write cache size limit?
Actually, I have an OOM killed issue when using s3fs 0.4.2, python 3.6.9, , concurrent.futures.ProcessPoolExecutor.map()  at RedHat system.
I have applied below options, but OOM killed issued was not resolved.
    kws3fs = {
        #- instance cache
        "skip_instance_cache": False,   # default False.  True to force creating a new instance even if a matching instance exists.

        #- listing cache
        "use_listings_cache": True,     # default True. lsdir caching
        "listings_expiry_time": 60,     # default None for no expiry. seconds of  LRU caching for max_paths
        "max_paths": 128,               # defult None as listings_expiry_time is None. set 512 lists

        #- file read & write cache
        #"default_block_size": 5 * 1024 * 1024,  # deafult 5MB.  blocksize for read & write caching. fsspec maxblocks for limit cache size, how to set at s3fs?

        #- cache type: Set to “none” if no caching is desired. OOM case, https://github.com/dask/s3fs/issues/234
        "default_cache_type": 'none',
        #- note: s3fs 0.4.2, fspec 0.8.0: default_cache_type='none'  might not properly worked  OOM case when using a little larger chunksize.
        "default_fill_cache": False,
    }

Thanks,
TK.

The blocksize determined how much data is kept in memory by the file instance after passing data to the caller. If, however, you are issuing commands file f.read(big_number), then you will still have high memory usage, if there are many bytes available. Therefore, it might depend on how you are interacting with the files. Can you figure out where memory is being used up? Perhaps the stacktrace of the OOM is useful.

Hello Martindurant,  Many thanks for your input. I will try to profile memory usage and will comment here for any findings.

Hi,
Not sure where is memory leak, but catch tracemalloc output as an example and share at blow.
I imagine many text lines(1+ million) converting to dataclass converting stream processing (include decompression) might cause memory fragmentation and can consumed 30+% system memory in a 1 cpu, 2 GB memory system.
The minimum and default block_size is 5MB,  it looks like a little large for compressed file and can consume 20+MB memory (4 times of compressed size) for a block processing.
=tracemalloc=  top 10 display
Top 10 lines
#1: <frozen importlib._bootstrap_external>:487: 20625.7 KiB
#2: /usr/lib/python3.6/json/decoder.py:355: 3898.9 KiB
    obj, end = self.scan_once(s, idx)
#3: /usr/lib/python3.6/abc.py:133: 854.6 KiB
    cls = super().__new__(mcls, name, bases, namespace, **kwargs)
#4: /usr/lib/python3.6/collections/__init__.py:429: 589.9 KiB
    exec(class_definition, namespace)
#5: /usr/lib/python3.6/linecache.py:137: 511.6 KiB
    lines = fp.readlines()
#6: /mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/asn1crypto/parser.py:232: 448.6 KiB
    encoded_data[pointer:contents_end],
#7: /mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/pycparser/yacctab.py:16: 415.0 KiB
    _lr_action[_x][_k] = _y
#8: /mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/cryptography/hazmat/bindings/openssl/binding.py:93: 414.9 KiB
    setattr(conditional_lib, attr, getattr(lib, attr))
#9: <frozen importlib._bootstrap_external>:60: 283.7 KiB
#10: /usr/lib/python3.6/_weakrefset.py:37: 274.2 KiB
    self.data = set()
29037 other: 21382.4 KiB
Total allocated size: 49699.7 KiB

=tracemalloc=  top 1 traceback
20101 memory blocks: 2008.4 KiB
  File "/usr/lib/python3.6/json/decoder.py", line 355
    obj, end = self.scan_once(s, idx)
  File "/usr/lib/python3.6/json/decoder.py", line 339
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.6/json/__init__.py", line 367
    return cls(**kw).decode(s)
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/botocore/loaders.py", line 175
    return json.loads(payload, object_pairs_hook=OrderedDict)
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/botocore/loaders.py", line 420
    found = self.file_loader.load_file(possible_path)
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/botocore/loaders.py", line 132
    data = func(self, *args, **kwargs)
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/botocore/session.py", line 160
    endpoints = loader.load_data('endpoints')
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/botocore/session.py", line 919
    self._components[name] = factory()
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/botocore/session.py", line 697
    return self._internal_components.get_component(name)
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/botocore/session.py", line 824
    endpoint_resolver = self._get_internal_component('endpoint_resolver')
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/s3fs/core.py", line 293
    **self.client_kwargs)
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/s3fs/core.py", line 187
    self.s3 = self.connect()
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/fsspec/spec.py", line 55
    obj = super().__call__(*args, **kwargs)
  File "/mnt/c/projectg/sfgtl/sfgtl/s3com.py", line 87
    sfs = s3fs.S3FileSystem(anon=False, **kws3fs)
  File "/mnt/c/projectg/sfgtl/sfgtl/scan.py", line 285
    sfs = s3com.make_sfs()
  File "/mnt/c/projectg/sfgtl/sfgtl/check.py", line 121
    limit=limit)
  File "/mnt/c/projectg/sfgtl/sfgtl/gate.py", line 90
    for acheck in itcheck:
  File "/mnt/c/projectg/sfgtl/sfgtl/form.py", line 106
    for agate in itgate:
  File "/mnt/c/projectg/sfgtl/sfgtl/load.py", line 71
    for aform in itform:
  File "/mnt/c/projectg/sfgtl/sfgtl/load.py", line 118
    for aload in itload:
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/click/decorators.py", line 21
    return f(get_current_context(), *args, **kwargs)
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/click/core.py", line 610
    return callback(*args, **kwargs)
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/click/core.py", line 1066
    return ctx.invoke(self.callback, **ctx.params)
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/click/core.py", line 1259
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/mnt/c/projectg/sfgtl/.venv-linux/lib/python3.6/site-packages/click/core.py", line 782
    rv = self.invoke(ctx)

Thanks,
TK

A few times the data size seems reasonable to me - you have the original compressed bytes in the HTTP stream, data bytes, some slicing to get specific ranges, conversion to text (needed by JSON) and set of objects being built by JSON.
In your own code, you may be able to figure out where the read is happening.
The 5MB standard in the library comes from wanting to reduce the fraction of time spent establishing connections - there is a time overhead for each call, so you generally want to fetch as man bytes as to make that call worthwhile.

Thanks for your comment and will close this case.
I have  reduced concurrent.futures.ProcessPoolExecutor.map, chunksize=1 and  line-by-line converting chunking size=0.5MB  from 1MB for small system to reducing  OOM killed cases.
Thanks. TK.

Hello,
When I use fileobj.flush()  periodically, OOM issue was not happened and process memory was not increased continuously.
As the fsspec flush doc:  Write buffered data to backend store. Writes the current buffer, if it is larger than the block-size.
Thanks.
TK

Hello,
I have identify root cause of this case, it was caused by my code when using itertools.tee to split a big data file at small system.
It was resolved by implementing flow controlled, locked tee to balance data consuming in between iter tee objects.
Detail code for this implementation can be found here, https://stackoverflow.com/questions/6703594/is-the-result-of-itertools-tee-thread-safe-python/63636352#63636352
Thanks,
TK.


README.md example updates

Hi!
Many thanks for the library! I was getting it setup with your code, examples and thought I'd push the tiny changes through that I needed to make it build.

Much appreciated, thanks for pushing these changes!


fix(bap1): Update bap1

Bap1 format for date_filed was a string
but CL is expecting it as a datetime.date
object
Also - Updated tests to reflect this change.

I keep thinking that a json schema for jurisraper outputs makes a lot of sense. It'd prevent errors like this in the future (where the data returned makes sense but is wrong).


Typo

Typo fixed

@DhyanRathore : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks @DhyanRathore.


Update README.md

Added final papers in PDF format to Sessions 3, 4, 5, 6, 7.

Awesome! Great thanks for your effort 🙏


Make tmpdir configurable

Description
If the repo mirrors are big, it can be that the chef run fails because
/tmp fills up.
Now it is configurable via the default['aptly']['tmpDir'] attribute.
Issues Resolved

Check List

[ ] All tests pass. See TESTING.md for details.
[ ] New functionality includes testing.
[x ] New functionality has been documented in the README if applicable.

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

  Generated by :no_entry_sign: Danger

Again I'm going to merge this PR like #110 as the failures will be fixed shortly.

Released as: 2.4.0