Restart of daily production

Hi @mawoka-myblock!
Referring to https://github.com/SonnenladenGmbH/APsystems-EZ1-API-HomeAssistant/issues/34 I now made a patch for the HA integration. Nevertheless, after being done I thought, probably this should better be fixed already in the API, i.e. in this repo. Would you agree/accept a patch for the get_ouput_data  function?
Only issue is that the API is then not stateless any more versus some strange code in the HA integration that does not seem to belong there.
Please let me know your thoughts then I can prepare a patch accordingly. I'm still on the old repo due to this missing functionality. Thanks!

Thanks for bringing that back up! The thing is that hardly any user seems to be facing this issue. I'd accept a PR when the debounce is/would be opt-in.
def __init__([...], debounce_values: bool = False):

I'd accept and appreciate that! I really appreciate the caution and thoughts you put into this library!

I really don't know where it's coming from. My inverter handles it without a problem.

I honestly don't consider this to be required as the (Home assistant integration](https://github.com/home-assistant/core/blob/dev/homeassistant/components/apsystems/sensor.py) doesn't use it and I've never heard anyone complain about that problem. I honestly really think your inverter is faulty. And just setting it in the init of the home assistant integration to true isn't a huge problem at all. Please just create the PR here and only debounce the daily values when the option is set in the init function.

I really don't know where it's coming from. My inverter handles it without a problem.

Not sure, can you deep zoom-in on 3rd of May, especially morning?

I honestly don't consider this to be required as the Home assistant integration doesn't use it.

Sorry, I lost track, what do you mean by 'it' that is not used by HA?
In my opinion it is a principle issue (not saying it is dramatic :wink:) with the inverter. The inverter turns of all communication (WiFi, BT) when there is not enough power from solar for some minutes and it resets daily energy value. Thus, whenever the solar power drops so the inverter turns off the issue occurs. Obviously, this is unlikely during day, especially summer times. No doubt.
I might see the issue more easily because I have very low power panels connected, but as described above it is an inherent topic due to the inverter functionality. Please correct me if I got something wrong.


Avoid sidebar scrollbar hiding behind navbar

Description
For sidebars that are long enough to scroll (e.g. Astro Docs or on smaller screens), the scrollbar currently spans the entire viewport height, meaning its top part is behind the nav bar.
Instead of using padding-top to push sidebar content down into view, this PR uses inset to position the sidebar directly at that point, so the the scrollbar is fully visible.
I think this closes #1350 if I understood correctly.

size-limit report 📦

Path
Size

/index.html
5.21 KB (-0.04% 🔽)

/_astro/*.js
21.42 KB (0%)

/_astro/*.css
13.05 KB (+0.03% 🔺)

Awesome, thanks for checking @TheOtterlord!


fix: handle manual suspension

❓ Is this PR still relevant our should we close it @eze-kiel ?

let's close it


How to show vertical scrollbar on mat-ree

What is the expected behavior?
What is the current behavior?
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Is there anything else we should know?

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.


GH-36120: [C#] Support schema metadata through the C API

What changes are included in this PR?
Import and export of field- and schema-level metadata via the C API.
Are these changes tested?
Yes

Closes: #36120

@wjones127 has offered to review ;)


Status: -5

Hello,
I'm sorry for the question, but what is my mistake when "Status: -5" is returned?
Best wishes
zKingFabzz

It means that the sensor you're using is not an MPU-9250.

Oh, thank you


Question: is UAC supposed to break clipboard and drag&drop?

Hello,
In the previous months I received few reports from users complaining about my app not being able to handle files pasted and dropped from Explorer and others.
Now, after a quite long research I figured out that this happens when UAC is disabled for logged in user.
Some of the reporters sustain that they always had UAC turned off, but that this problem started to happen only at some point, but of course I can't confirm this.
Now my question is: is this the intended behavior?
From my point of view this just creates a lot of confusion users have absolutely no hint about this, and the app is just broken for them.

Clarifying: "UAC disabled" means changing the UAC slider to "never notify"?

@ieburk: are you aware of any issues with file paste or drop with UAC prompts disabled?

I was not – tagging owners: do you know more about this issue @Chaz<EMAIL_ADDRESS>@Ram<EMAIL_ADDRESS>From: Steve<EMAIL_ADDRESS>Sent: Monday, October 26, 2020 11:34 AM
To<EMAIL_ADDRESS>Cc: Ieva<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [microsoft/ProjectReunion] Question: is UAC supposed to break clipboard and drag&drop? (#253)
@ieburkhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fieburk&data=04|01|Ieva.Burk%40microsoft.com|8d800625a1864521a74908d879ddbd0c|72f988bf86f141af91ab2d7cd011db47|0|0|637393340548912733|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=F9V0Lx8vWrYvry8mvL4hPrK%2BvGkVSSUJQr0d%2BZVyB%2BE%3D&reserved=0: are you aware of any issues with file paste or drop with UAC prompts disabled?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FProjectReunion%2Fissues%2F253%23issuecomment-716744498&data=04|01|Ieva.Burk%40microsoft.com|8d800625a1864521a74908d879ddbd0c|72f988bf86f141af91ab2d7cd011db47|0|0|637393340548912733|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=NLCldiDsCzGgowzAnt7IbjikPRgliv13pdyIpXuG44w%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOJ3IMYVBKVMUPZBDXBU2BDSMW6KLANCNFSM4S6E7QFQ&data=04|01|Ieva.Burk%40microsoft.com|8d800625a1864521a74908d879ddbd0c|72f988bf86f141af91ab2d7cd011db47|0|0|637393340548922729|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ZWmhEaleuYZcHOOp7DR6Y5HYhEvVffN2hFZzAoxpIrw%3D&reserved=0.

Hi, yes, I meant "never notify"!

I forgot to mention that this is about UWP, as I'm used to post in microsoft-ui-xaml repository...

Not sure if this is exactly helpful as I don't have repro steps at the moment (but can try to come up with them) but I have experienced issues with Clipboard working across UWP/Win32 in general a lot where things copied from UWP won't paste to the non-UWP world or vice versa. It's as if there's multiple clipboards.

Actually I'm not even sure that UAC is the root cause, but it seems to be at least one of them.
The problem is that dealing with final users it is super hard for me to get more details and being more specific/able to offer repro steps.
This is what I usually receive as report: video.
The interesting part is that the 🚫 doesn't look generated by the UWP platform (as it would be a white tooltip with red crossed circle).
Usually users have UAC disabled, and often (not always) enabling it allows them to drop and paste files after.


feat(back-to-top): pf-back-to-top element

What I did

Added <pf-back-to-top> element

Todo

[x] Determine accessibility improvements A href attribute and using a <a> in shadowdom is warranted as expected.  Can still use button version but there is caveat of owning accessibility in implementation, such as setting focus on element scrolled to.
[x] Figure out improvements for how a user hooks up element to "returning to top".  PFv4 doesn't make assumptions on how the button is hooked up.  We may opt for a href attribute and use a link instead of a button when present, and definitely have to add a event when button is clicked otherwise for implementors to hook into. See above, href implementation
[x] Improve documentation
[x] Add tests
[x] Add changeset

Testing Instructions

View Deploy Preview

Notes to Reviewers

Merging ahead of 2.5.0. @nikkimk @hellogreg please open new issues for anything that comes up


Removing all hard coded 18 decimals from fixedRateExchange

Fixes #664
Changes proposed in this PR:

Removing all hard coded 18 decimals from fixedRateExchange

lgtm, but we can test it on goerli first ?

lgtm, but we can test it on goerli first ?

Yeah good idea, makes sense


Release Proposal v0.10.2

What this PR does / why we need it:
Special notes for your reviewer:
Does this PR introduce a user-facing change?

LGTM


SW-1860 Truncate and Show More List of Strings

create a component that shows a list of string
given a max length, truncate the list
display a link button indicating the remaining count
clicking the link button shows a tooltip with the full list

Super cool!

do we know if there's any npm package that provides similar functionality?

@karthikbtf I did look around a bit, but the libraries I found worked with a string instead of a list of strings, where based on the ticket we don't want to break in the middle of a word (I implemented an exception for when the first word is really long)


Android targeting sdk version 29 not working with version 0.6.3+1

Hi,
if I switch to targetSdkVersion 29, the example starts but when I search for devices it does not found any.
Step to reproduce:

use Flutter 1.9.1+hotfix.6 • channel stable
download example
add implementation "androidx.core:core:1.1.0" in build.gradle so we can use flutter stable channel
switch to targetSdkVersion 29.
Same behaviour if I use flutter master channel

Do you happen to using a device running Android 10? I ran into this issue a few weeks back and made a pull request that fixed it on my end: #383

@rjcasson Yes, thanks! I'm running Android 10.
Waiting for the update I'll compile with targetSdkVersion 28.

@rjcasson Yes, thanks! I'm running Android 10.
Waiting for the update I'll compile with targetSdkVersion 28.


UI landing page

Proposed changes
This pull request will add a mobile-first landing page to the project. A custom logo was made and added for this project as well.
Types of changes

[X] New feature (non-breaking change which adds functionality)

Checklist

[X] I have read the CONTRIBUTING doc
[X] I have read the BRANCHING MODEL doc
[X] Lint and unit tests pass locally with my changes

Further comments
This pull request also addresses the Signup and Login forms not being exactly mobile friendly. User inputs should now stack one on top of the other where appropriate as opposed to being displayed side by side on a mobile phone.
This pull request also Fixes #11

This should be pushed to the develop branch first.
My mistake. Closing this pull request and reopening it into the develop branch.


Added functionality for Poi to pin the first message automatically in thread in #events

Added functionality for Poi to pin the first message automatically in a thread inside the events channel (configured per guild server settings). Only the first message from the thread author will be pinned.
Suggested by David in #suggestions
Possible todo: remove the pinned system message

Also a thing I would like to look into more is that when a message gets pinned, discord gives a ratelimit warning right away... Maybe something got spammed? I don't know, needs more investigation by me...


Don't Merge! PR real time test for webhook case

Docs Build status updates of commit 56ae981:
:clock10: Pending: waiting for processors (24 builds ahead of you)
:warning: Docs Build is busy, currently there are 24 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 56ae981:
:clock10: Full build: the average full build time is 7 min(s) 21 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 56ae981:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


interpolate overlay on the fly

This will reduce the size of the overlay a lot for functional overlays displayed on anatomical scans. This is critical to support 4D data viewers.

this also needs to be exposed as a parameter in nilearn's API (either 2D or 3D interpolation)


createReducer not responding to outside actions

Hey there,
I'm not able to get my reducer made using createReducer to respond to outside actions. I'm not sure if I'm doing something wrong. Here's my code:
//./caretPosition.js
import { createAction, createReducer } from 'redux-act'

// ------------------------------------
// Actions
// ------------------------------------
export const clearCaret = createAction('clearCaret')
export const updateCaret = createAction('updateCaret')

var { updateSelectionLayer } = require('./selectionLayer'); //important, keep this here in the order of the file and let it use the commonjs syntax!
// ------------------------------------
// Reducer
// ------------------------------------
export default createReducer({
  [updateSelectionLayer]: () => {
  	//clear the caret if the selection layer is updated!
    return -1
  },
  'redux-form/INITIALIZE': (state) => {
    debugger;
    return state
  },
  ['@@INIT']: (state) => {
    debugger;
    return state
  },
  [clearCaret]: () => {
    return -1
  },
  [updateCaret]: ({start}, payload) => {
    return payload
  },
}, 0)

in the above code, neither 'redux-form/INITIALIZE' nor '@@INIT' are triggering the debugger, but the [clearCaret] works fine.
Am I missing something here?
Thanks!
Thomas

Version 0.4.0 right?

Yikes,
I'm behind a version!
On Wed, Feb 24, 2016 at 1:32 AM, Paul Dijou<EMAIL_ADDRESS>wrote:

Version 0.4.0 right?
—
Reply to this email directly or view it on GitHub
https://github.com/pauldijou/redux-act/issues/16#issuecomment-188161468.

Okay, I bumped my version to the latest (0.4.0) and I'm still seeing this issue

My tests are all fine using string keys... I must be missing something. Sorry to ask, but any chance you could setup a super small project reproducing the problem? I would investigate asap.

Yup I'll work on that now
On Wed, Feb 24, 2016, 2:39 AM Paul Dijou<EMAIL_ADDRESS>wrote:

My tests are all fine using string keys... I must be missing something.
Sorry to ask, but any chance you could setup a super small project
reproducing the problem? I would investigate asap.
—
Reply to this email directly or view it on GitHub
https://github.com/pauldijou/redux-act/issues/16#issuecomment-188187233.

Hmm I'm not able to reproduce this on a minimal set-up... I must be doing something wrong in my main app. I'll dig deeper into that. Thanks for the help

And... all of a sudden it's working in my main app as well. Very mysterious...


fix(@embark/engine): ensure deployment hook logs don't produce unexpected output

Since https://github.com/embark-framework/embark/commit/ee56f3771389a00e019eadce595dcf30468f0afe, any deployment hook that
was using simple console.log() statements to output information, would result in
unexpected [[object object], [object object]].
Unfortunately, the reason for that was that we switched from non-arrow functions to
arrow functions here. Usually switching from non-arrow functions to arrow functions solves
a lot of lexical scope issues where the current reference of this isn't pointing at the
right thing.
We're tempering with the global console.log inside Embark, which then, combined with
proper lexical scope, causes the output described above.
Generally there are a few ways to go about this:

Ensure that our custom log functions doesn't turn every log statement into an array output
Tell users not to use console.log inside deployment hooks and instead rely on deps.logger
Revert the changes made in the mentioned commit and use non-arrow functions inside interceptLogs

Option 2) is not really a solution as we can't simply tell our users that they can't use one of
the most used functions in JavaScript. Option 1) requires finding out why we're formatting logs
by default in an array shapre in the first place. On top of that, we might be relying on this
inside Embark, so it could break certain output.
Option 3) seems to be the most pragmatic solution for now as it doesn't introduce any of the
downsides mentioned above at the cost of using non-arrow functions.

The real problem here had nothing do with this, it's more subtle and surprising. Try this in a Node.js REPL:
> function intercept(x, y) { (() => { console.log(arguments); })(1,2,3); }
undefined
> intercept({}, {})
[Arguments] { '0': {}, '1': {} }
undefined
> intercept()
[Arguments] {}

What's going on with that? The first thing to realize is that the arguments object is not defined for arrow functions; try this in a REPL:
> foo_1 = function () { console.log(arguments); }
[Function: foo_1]
> foo_2 = () => { console.log(arguments); }
[Function: foo_2]
> foo_1(1,2,3)
[Arguments] { '0': 1, '1': 2, '2': 3 }
undefined
> foo_2(1,2,3)
Thrown:
ReferenceError: arguments is not defined
    at foo_2 (repl:1:29)

So in the case of the toy intercept function above and engine's interceptLogs, it's the outer function's arguments object that ends up being referenced in the arrow function/s.
@jrainville I agree that arguments is sometimes useful, but I think in many cases it's possible to use rest parameters instead and get much of the same benefit. Here's the refactor I suggest should be take the place of these changes (I realize the PR is merged):
function interceptLogs(consoleContext, logger) {
  const context: any = {};
  context.console = consoleContext;

  context.console.log = (...args) => {
    logger.info(normalizeInput(...args));
  };
  context.console.warn = (...args) => {
    logger.warn(normalizeInput(...args));
  };
  context.console.info = (...args) => {
    logger.info(normalizeInput(...args));
  };
  context.console.debug = (...args) => {
    // TODO: ue JSON.stringify
    logger.debug(normalizeInput(...args));
  };
  context.console.trace = (...args) => {
    logger.trace(normalizeInput(...args));
  };
  context.console.dir = (...args) => {
    logger.dir(normalizeInput(...args));
  };
}

Also, I think normalizeInput could maybe use some touching-up, but I'd need to spend more time looking at all usages of it before making changes.
My apologies for not catching this problem during the refactor work in #2126. I was aware that the arguments object and arrow functions don't mix well, but I didn't look closely enough while working to quiet tslint.

@michaelsbradleyjr wow thanks, you're absolutely right. Thanks for correcting my verdict!
I'll send another PR that introduces your proposed changes.


Weird output

Hi
I ran the code, it is giving me final output that is too weird irrespective of changing the image.  I am attaching it. Can you explain what it is?

Thanks

Sorry for the delay, but can do let me know, from which layer did you extract the output?
Regards,

Hi @uakarsh
Thanks for your response.
I tried below code
fp = "img.jpeg"
tokenizer = BertTokenizerFast.from_pretrained("bert-base-uncased")
encoding = dataset.create_features(fp, tokenizer, add_batch_dim=True)
feature_extractor = modeling.ExtractFeatures(config)
docformer = modeling.DocFormerEncoder(config)
v_bar, t_bar, v_bar_s, t_bar_s = feature_extractor(encoding)
output = docformer(v_bar, t_bar, v_bar_s, t_bar_s)  # shape (1, 512, 768)
then I visualized the output.

HI,
Actually, we know that the output is (512, 768), now, this output results from the attention of three different entities:

Image feature of (512, 768)
Language Feature of (512, 768)
Spatial Dimension of (512, 768)

Now, when we perform any downstream task, we have an encoded version of these three modalities, so the diagram (which you have plotted) would be helpful for the model to know, which encoding to attend to when performing the downstream task.
The same can be seen in Pg No. 15, Figure 11. B of DocFormer Paper. Hope it helps

Thanks for your info.  How can I do entity level classification like in FUNSD dataset?

@uakarsh

I have almost finished the training script for RVL-CDIP (Document Classification), and have started working on FUNSD for token classification.
You can visit my cloned repo, and in the examples/docformer_pl, you can get the

Data visualizing
Dataset making
MLM with Pytorch Lightning
Document Classification with DocFormer (would be uploaded soon)
And next would be NER with FUNSD.

Would update you soon!!

@uakarsh Hello,
Any update on  NER with FUNSD using docformer?


Changing the resolution results in "frozen" screenshots - D3D doesn't update to current screen

When I change the resolution of my laptop after running my Python code with D3DShot, screenshot_to_disk_every() stops returning the current screen and instead returns the screen before resolution change. I don't know if this is an issue with Window's Desktop Duplication or this library.
My code:
import d3dshot
import time
import matplotlib.pyplot as plt
d = d3dshot.create()
d.screenshot_to_disk_every(10)

This issue occurs even with using other screenshot functions.

I think you will have issues with most screen capturing solutions, especially those that are closer to the metal and try to be efficient with resources when changing resolution once the context has been created.
The problem here is that by changing the resolution of a display, you invalidate the DXGI Output handle that has been acquired by D3DShot to capture stuff on that display. The handle is still valid for the desktop duplication API to use, but it's effectively a "zombie" display at that point and no new information can be returned (which you observed). The resolution-changed display effectively lives on as a new DXGI Output that you would have to detect and swap to seamlessly. This is not easy at all; not even sure it's possible.
Unfortunately, I don't think this is a reasonable issue to try and tackle. Did you accidentally stumble upon the behavior or is it part of your use case to change resolutions while capturing? Just curious.

I accidentally stumbled upon it. I expected it to be an issue with Desktop
Duplication.
My main reason for posting the issue was to provide documentation for it so
others wouldn't need to waste 3 days like I did :).
On Tue, Aug 13, 2019, 12:01 PM Nicholas Brochu<EMAIL_ADDRESS>wrote:

I think you will have issues with most screen capturing solutions,
especially those that are closer to the metal and try to be efficient with
resources when changing resolution once the context has been created.
The problem here is that by changing the resolution of a display, you
invalidate the DXGI Output handle that has been acquired by D3DShot to
capture stuff on that display. The handle is still valid for the desktop
duplication API to use, but it's effectively a "zombie" display at that
point and no new information can be returned (which you observed). The
resolution-changed display effectively lives on as a new DXGI Output that
you would have to detect and swap to seamlessly. This is not easy at all;
not even sure it's possible.
Unfortunately, I don't think this is a reasonable issue to try and tackle.
Did you accidentally stumble upon the behavior or is it part of your use
case to change resolutions while capturing? Just curious.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SerpentAI/D3DShot/issues/6?email_source=notifications&email_token=AB4TUMLOJ2BQXLTOU6JVVADQELSH3A5CNFSM4IJKRMYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4GJYHI#issuecomment-520920093,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB4TUMITEUFVMVRBCIM7TVTQELSH3ANCNFSM4IJKRMYA
.

I'll close this one. It'll remain searchable to help people stumbling upon a similar situation.


(bug) - Honour custom insync in noop mode

Prior to this PR, a custom insync message was not honoured when applying a manifest in noop mode. This was later tracked down to the offending noop method found in the resource harness class, which did not call the change_to_s method to retrieve the custom insync message.
I've updated the method to match that of the sync method below, ensuring that the functionality of the custom isync provider feature is not any different whether running puppet in noop or no-noop mode.
Can see the two methods here
Additional Context
Have opted out of adding a space between (noop) and audit_message to match the three other implementations of this in the sync and noop methods.

jenkins please test this

Thanks for your patience @jordanbreen28! This will be release in the next 8.9.0 release.


Avoid transitive dependencies in pom when publish to maven central

fixed with https://github.com/dpsoft/ap-agent/commit/18933bd383d8cb58fc00f82fb713e444a221c234


Fixes wrong alignment of shader binding table entries

// How the hell did this work before?

Will open another pull request with further changes.


Implement feature flagging system season

Duck season

Flaky Test
Failure Summary
Logs

[chromium] should display the about us section correctly.
The "About" section was not displayed within 5 seconds, and the title was not found.
Logs ↗

[chromium] should return list of users correctly.
The API call to get the list of users failed with a 404 Not Found error.
Logs ↗

[chromium] should login with credentials correctly correctly.
The login request to the server failed with a 404 Not Found error.
Logs ↗

[chromium] should display error message for invalid login correctly.
The expected HTTP status code 401 was not received, instead a function was returned.
Logs ↗

... and 4 more

View Full Report ↗ ⋅ Docs


Test to corrupt segments mid-WAL, repair and check we can read the correct number of records.

The test fails against master, as the underlying segmentReader has already moved on to the next segment.
Also included - the fix.  Make the segmentReader pad short segments with zeros, and only advance to next segment on the call to Read after completely consuming this one; previously it would increment curr on the call to Read which failed, causing the returns CorruptionErr to reference the wrong segment.
Signed-off-by: Tom Wilkie<EMAIL_ADDRESS>

I think this is an improvement, but I'm trying to think if there are still corrupt WALs which could fail repair - for instance, a WAL where a short segment causes the io.ReadFull to consume into the next segment.  Given the io.ReadFull is bounded by a page, the only case I can think of are segments containing records spanning pages - which shouldn't happen, but does.
To make this more robust (defense in depth), I think we should:

Change repair to not depend on the the index of the WAL segment from the error, but rebuild the entire WAL (which would be slow)
Get rid of repair all together, and when we find a corrupt WAL delete the entire thing and write out a new checkpoint (after all, we've just replayed the WAL into the head).
Change the WAL code to not append to the existing segment, but always start a new segment.

WDYT?

Thanks @krasi-georgiev!

a WAL where a short segment causes the io.ReadFull to consume into the next segment. Given the io.ReadFull is bounded by a page, the only case I can think of are segments containing records spanning pages - which shouldn't happen, but does.

I don't think this is actually true. I suspect the corruptions we were seeing (both torn pages and failed crc checks) were because we were reading data at incorrect offsets in the file because the previous corrupt segment hadn't been padded with 0's.

I don't think this is actually true. I suspect the corruptions we were seeing (both torn pages and failed crc checks) were because we were reading data at incorrect offsets in the file because the previous corrupt segment hadn't been padded with 0's.

We definitely have a segment with records spanning pages (and not just from concatenating segments, which isn't something the LiveReader does).  But I have no idea how we got into this state, and its kinda unrelated.

Thanks for the feedback @cstyan @krasi-georgiev !  I think I've covered it all, left me know if I've missed anything.

To make this more robust (defense in depth), I think we should:

Change repair to not depend on the the index of the WAL segment from the error, but rebuild the entire WAL (which would be slow)

Get rid of repair all together, and when we find a corrupt WAL delete the entire thing and write out a new checkpoint (after all, we've just replayed the WAL into the head).

this one sounds as the best solution, but maybe do it in a separate PR if it happens that the current implementation causes other problems.

Change the WAL code to not append to the existing segment, but always start a new segment.

this one sounds as the best solution, but maybe do it in a separate PR if it happens that the current implementation causes other problems.

For sure!  I was thinking aloud there.  I'll follow up with a ticket for this.


Status code set as 403 even if cloudfare check bypassed using retryOnBlocked

Which package is this bug report for? If unsure which one to select, leave blank
None
Issue description
Using retryOnBlocked to bypass cloudfare
However getting status code as 403 instead of new 200 status code once bypassed on retry.
I understand that 403 is first http response status code, which is when it was blocked by cloudfare, however ideally it should get updated to 200 once bypassed.
Code sample
No response
Package version
3.5.0
Node.js version
18
Operating system
No response
Apify platform

[ ] Tick me if you encountered this issue on the Apify platform

I have tested this on the next release
No response
Other context
No response

Thanks @B4nan


Google Refresh Token

To install this integration you will need a Google Refresh Token which you can get by following the instructions at: https://www.angelod.com/onhubauthtool
There are no instructions there on what in the hell I'm supposed to be doing.

Doesn't seem like the best approach to get help.

Doesn't seem like the best approach to get help.

So your choice, instead of offering clear, concise directions, is to close the issue rather than correct it.


Smarter recognition of time patterns

For example: https://www.cs.cmu.edu/afs/cs.cmu.edu/Web/People/benhdj/Publications/Published/TIME_2006.pdf

Another alternative: https://github.com/mojombo/chronic


Import-Excel relative path needs resolving

Was using the module in version 5.1.0 up to yesterday.
With the update to 6.2.1 relative paths stopped working for function Import-Excel.
Was introduced with a change to Version 6.
Example:
PS c:\>Import-Excel -Path ".\someDocument.xlsx"
Error:
New-Object : Exception calling ".ctor" with "4" argument(s): "Could not find a part of the path 'C:\Users\#USERNAME#\someDocument.xlsx'."
At C:\Program Files\WindowsPowerShell\Modules\ImportExcel\6.2.1\ImportExcel.psm1:349 char:23
+ ...   $stream = New-Object -TypeName System.IO.FileStream -ArgumentList $ ...
+

Reason System.IO.FileStream object needs a fully defined path.
Section of the code that was removed with from Version 5.1.0 on line 326
$Path = (Resolve-Path $Path).ProviderPath

Please reintegrate into ImportExcel.psm1, before line 349, for a fixe I have now integrated the line above into my code when calling the function.

https://github.com/dfinke/ImportExcel/releases/tag/v6.2.2


Manual engine roll 31cd2dfca22a...429e56e97427

Manual roll to update tests for https://github.com/flutter/engine/pull/13795
git@github.com:flutter/engine.git/compare/31cd2dfca22a...429e56e97427
git log 31cd2dfca22a..429e56e97427 --no-merges --oneline
2019-11-12<EMAIL_ADDRESS>Roll src/third_party/dart 7ebc820ffc..5cce1e4acd (1 commits) (flutter/engine#13798)
2019-11-12<EMAIL_ADDRESS>Document the coordinate space of points in FlutterPointerEvent. (flutter/engine#13782)
2019-11-12<EMAIL_ADDRESS>implement radial gradient in canvaskit backend (flutter/engine#13796)
2019-11-12<EMAIL_ADDRESS>Adds missing comma in EngineParagraphStyle.toString() (flutter/engine#13795)
2019-11-12<EMAIL_ADDRESS>Roll src/third_party/skia ad21d47cfa8d..d860a78fd60c (2 commits) (flutter/engine#13794)
2019-11-12<EMAIL_ADDRESS>Roll src/third_party/dart 7408ed4730..7ebc820ffc (20 commits)
2019-11-12<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from TnTbF... to tZdOC... (flutter/engine#13791)
2019-11-12<EMAIL_ADDRESS>[web] Implement TextStyle.shadows (flutter/engine#13769)
2019-11-12<EMAIL_ADDRESS>Expose asset lookup from plugin binding. (#42019) (flutter/engine#13743)
2019-11-12<EMAIL_ADDRESS>Roll src/third_party/skia 3de645cbca78..ad21d47cfa8d (15 commits) (flutter/engine#13790)
2019-11-12<EMAIL_ADDRESS>Move Path and PathMetrics from canvas.dart into their own files. No delta (flutter/engine#13772)
2019-11-12<EMAIL_ADDRESS>Take devicePixelRatio into account when drawing shadows (flutter/engine#13786)
2019-11-12<EMAIL_ADDRESS>Add ImageFilter and BackdropFilter to CanvasKit backend (flutter/engine#13768)
2019-11-12<EMAIL_ADDRESS>Revert "Reland "Guarding EAGLContext used by Flutter #13314" (#13759)" (flutter/engine#13788)
2019-11-12<EMAIL_ADDRESS>Add Helvetica and sans-serif as fallback font families (flutter/engine#13784)
2019-11-12<EMAIL_ADDRESS>Roll src/third_party/skia 0fd4f01b9b83..3de645cbca78 (1 commits) (flutter/engine#13776)
2019-11-12<EMAIL_ADDRESS>Add line boundary information to LineMetrics. (flutter/engine#13727)
2019-11-11<EMAIL_ADDRESS>Fix RendererContextSwitch result check in Rasterizer::MakeRasterSnapshot (flutter/engine#13785)
2019-11-11<EMAIL_ADDRESS>[web] Fix path to svg for drrect (flutter/engine#13779)
2019-11-11<EMAIL_ADDRESS>Roll src/third_party/dart d45c3d15cb..7408ed4730 (20 commits) (flutter/engine#13778)
2019-11-11<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from RrTFI... to TnTbF... (flutter/engine#13775)
2019-11-10<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from 4JJNo... to RrTFI... (flutter/engine#13774)
2019-11-10<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from xC_an... to 4JJNo... (flutter/engine#13773)
2019-11-09<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from eq_Rz... to xC_an... (flutter/engine#13771)
2019-11-09<EMAIL_ADDRESS>Roll src/third_party/skia 4135cf0b57c2..0fd4f01b9b83 (1 commits) (flutter/engine#13770)
2019-11-09<EMAIL_ADDRESS>Reland "Guarding EAGLContext used by Flutter #13314" (flutter/engine#13759)
2019-11-09<EMAIL_ADDRESS>Point old plugin registry accessors to new embedding plugin accessors. (#44225) (flutter/engine#13739)
2019-11-09<EMAIL_ADDRESS>Revert "[fuchsia] Temporarily disable intl provider (#13696)" (flutter/engine#13721)
2019-11-09<EMAIL_ADDRESS>Turn on RasterCache based on view hierarchy (flutter/engine#13762)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

What is the point of this manual engine roll until either the tests are updated or the extra data in the ParagraphStyle.toString() is removed?
Note, that the comma in https://github.com/flutter/engine/pull/13795 is not the issue - the shadow string itself is the problem whether or not it has a comma.

Yeah, realized this, I have a PR to fix the web Paragraph style

Waiting on https://github.com/flutter/engine/pull/13805 to go green, land, and build

Commit causing failing cirrus was reverted. Landing.


Strange behavior of function overloading and union type

Bug Report
Strange behavior of function overloading and union type
🔎 Search Terms
function overloading union type
🕗 Version & Regression Information

This changed between versions 3.5 (good) and 3.6 (bad)

⏯ Playground Link
Playground link with relevant code
💻 Code
import { Canvas, CanvasRenderingContext2D as NodeCanvasRenderingContext2D } from 'canvas'

let ctx: CanvasRenderingContext2D | NodeCanvasRenderingContext2D | null = null

function test(canvas: HTMLCanvasElement | Canvas) {
    ctx = canvas.getContext('2d')
    // error??
}
function test2(canvas: HTMLCanvasElement | Canvas) {
    // works!!
    if (canvas instanceof HTMLCanvasElement) {
        ctx = canvas.getContext('2d')
    } else {
        ctx = canvas.getContext('2d')
    }
}

🙁 Actual behavior
Type 'NodeCanvasRenderingContext2D | RenderingContext | null' is not assignable to type 'CanvasRenderingContext2D | NodeCanvasRenderingContext2D | null'.

🙂 Expected behavior
No error

Function type merging produces a single signature in this case, and that single signature doesn't keep around enough information.
The only way to process this "correctly" would be combinatorially explosive, so that doesn't happen.

but why does it work on TS 3.5? 🤔


Update dependencies in OpenTelemetry Java sample

Summary
Update OpenTelemetry Java sample:

Paketo OpenTelemetry 2.0.0 (just released with new agent: https://github.com/paketo-buildpacks/opentelemetry/releases/tag/v2.0.0)
Spring Boot 3.3
Java 17

Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

hello @paketo-buildpacks/content-maintainers could you please merge? thanks!


Copy edit Badging article.

https://github.com/GoogleChrome/web.dev/issues/1291

:+1:

:tada: This has been pushed live to https://developers.google.com/web/


Remove the need to restart VS Code when typescriptHero.resolver.ignorePatterns setting is changed.

Using the typescriptHero.resolver.ignorePatterns option is a bit cumbersome because VS Code needs to be restarted for accepting the changes.
Please add a watcher to settings so their values are getting caught immediately when saving the settings.json file.

Actually, there is an event for that. Good point! Thanks

Excellent! Thanks :+1: :cake:


Cannot hide screening invitation pop-up

When a tutor/student opens the settings tab in the user area (user section / "Userbereich") she will see a popup with an invitation to a screening interview. But the later ("Später") button to hide that popup no longer works. The popup will hide but immediately re-appears. The later button should hide the screening popup as long as the user is on the selected tab (settings tab).
Please see the following as an example of the bug:

This is primarily strange due to the following fact: When a tutor/student registers for Corona School, she will get an email with the request to verify her email address. If she clicks on that link, her email address gets verified and she will get redirected to the settings tab of the user area ("Userbereich"). That user area then renders entirely unusable if the register and cannot immediately go to the screening.

We have gotten a couple of support mails regarding this issue, this should be fixed as soon as possible.

@FunnyNameHaha is already working on that!


Building with clang 8.0 fails

Officially we don't support clang but it would still be nice if it worked.
Also trying it out (by doing export CC=clang CXX=clang++ before the cmake … make) shows a few areas where clang seems convinced that our use of constexpr isn't valid C++ and that's a little disconcerting. Also there seem to be a couple of unused variables.

I've created the #230 PR to track these fixes but I'm getting stuck on IndexMetaDataImpl::_isMmapBased not working as a constexpr. The current build log can be found here
build_log.txt

This was fixed with the just merged #230.Clang still shows some warnings that we don't get on GCC but it builds and the resulting binary seems to work fine.


Rebase for merge.

mh ... /usr/sbin/update-motd: not found

Anything we need to install in the Docker Container to get it running?


[Contract Interaction] Parameter validation issue

Report 1
Contract Address: 0xfff1b37123e452f3adc7965f899daaf6bd4eb7cc
ABI:
[
  {
    "constant": false,
    "inputs": [
      {
        "internalType": "address",
        "name": "balancerPoolAddress",
        "type": "address"
      },
      {
        "internalType": "address[]",
        "name": "tokens",
        "type": "address[]"
      }
    ],
    "name": "bigGulp",
    "outputs": [],
    "stateMutability": "nonpayable",
    "type": "function",
    "signature": "0x2c6e1288"
  }
]

I run the bigGulp function with following inputs:
balancerPoolAddress (address): 0x2feb4a6322432cbe44dc54a4959ac141ece53d7c
tokens (address[]):
[ "0x4DA9b813057D04BAef4e5800E36083717b4a0341", "0x625aE63000f46200499120B906716420bd059240", "0x6Ee0f7BB50a54AB5253dA0667B0Dc2ee526C30a8", "0x71fc860F7D3A592A4a98740e39dB31d25db65ae8", "0x9bA00D6856a4eDF4665BcA2C2309936572473B7E", "0xfC1E690f61EFd961294b3e1Ce3313fBD8aa4f85d" ]
When I click "Review", nothing happens.
In the Chome console, I see the following:
Uncaught (in promise) Error: expected array value (arg="tokens", coderType="array", value=undefined)
Report 2
Similar issue reported here:
If I click “review” simply nothing happens. On the console I see this error message.
This is the value I tried for the byte32:
5a58dc24756514ef6c512ca97e0af0def8430860bbd0ec7356887bb93d07d3be

Report 3
We are using gnosis safe for our governance contracts and we noticed the Contract Interaction in the ui is currently not working. The review button seems always be  disabled  but entered contract address and abi is correct.

@lukasschor, @mikheevm I've been working on improving the error handling and form button's status. That should solve the scenarios pointed in Report 2 and Report 3.
Also, I found an issue with the data passed to the Review screen that I'm planning to fix in the same PR (#974)
What I'll prefer is to move the fix for the Report 1 to a different PR/issue. As it's strictly related to the fields and its validation/parsing, and not strictly related to the other two scenarios.
This way I'll keep the PR small and makes it easier to review and give feedback.


Link Missing

Metadata file
This can be considered your “load file” that contains metadata associated with the files you exported. For a list of fields available in the metadata file, see [link]. This file can typically be ingested by 3rd party tools downstream.
see [link] ?

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

ID: 2290d327-7e3c-bc52-ca1a-2e6b60d502c0
Version Independent ID: 4b775396-e48c-717a-fccf-61705776f2ba
Content: Export documents from a review set - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/export-documents-from-review-set.md
Service: o365-seccomp
GitHub Login: @markjjo
Microsoft Alias: markjjo

@MartinMurdy Thanks for the heads up. I'll fix this link. The link will point to this file: https://docs.microsoft.com/en-us/microsoft-365/compliance/document-metadata-fields-in-advanced-ediscovery
See fields in the "Exported field name" column.


generate_vs_solution.bat does not work on the dev2.0 branch.

CMakeError.log
CMakeOutput.log

Turns out the submodules did not get cloned properly. My bad.


chore: Reserve memory for native shuffle writer per partition

Which issue does this PR close?

Closes #887.
Rationale for this change

What changes are included in this PR?

How are these changes tested?

I've copied the tests on my branch to this PR and the test hangs.
running 6 tests
test execution::datafusion::shuffle_writer::test::test_slot_size ... ok
test execution::datafusion::shuffle_writer::test::test_pmod ... ok
test execution::datafusion::shuffle_writer::test::test_insert_larger_batch ... ok
test execution::datafusion::shuffle_writer::test::test_insert_smaller_batch ... ok
test execution::datafusion::shuffle_writer::test::test_large_number_of_partitions has been running for over 60 seconds
test execution::datafusion::shuffle_writer::test::test_large_number_of_partitions_spilling has been running for over 60 seconds
^C

It is possibly caused by the deadlock on buffered_partitions.lock().

I've copied the tests on my branch to this PR and the test hangs:
running 6 tests
test execution::datafusion::shuffle_writer::test::test_slot_size ... ok
test execution::datafusion::shuffle_writer::test::test_pmod ... ok
test execution::datafusion::shuffle_writer::test::test_insert_larger_batch ... ok
test execution::datafusion::shuffle_writer::test::test_insert_smaller_batch ... ok
test execution::datafusion::shuffle_writer::test::test_large_number_of_partitions has been running for over 60 seconds
test execution::datafusion::shuffle_writer::test::test_large_number_of_partitions_spilling has been running for over 60 seconds
^C

It is possibly caused by deadlocking on buffered_partitions.lock() when spilling is triggered.

Thanks. I knew the cause of the deadlocks. I'm going to revamp some codes.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 33.97%. Comparing base (c3023c5) to head (e678cb0).
Report is 25 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #988      +/-   ##
============================================
- Coverage     34.03%   33.97%   -0.07%
+ Complexity      875      857      -18
============================================
  Files           112      112
  Lines         43289    43426     +137
  Branches       9572     9622      +50
============================================
+ Hits          14734    14752      +18
- Misses        25521    25630     +109
- Partials       3034     3044      +10

Flag
Coverage Δ

?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Hmm, these tests for large partition number shuffle fail on MacOS runners only. And no stack trace...But I cannot reproduce it locally.

I'm testing this PR out now, in conjunction with some other PRs because I currently have a reproducible deadlock caused by memory pool issues, as far as I can tell.

Thanks @andygrove @Kontinuation


Ghidra Headless Analysis Failure: No JSON object could be decoded

Good day. There is a problem as described above. Here is the log from Ida Pro.
  bytes   pages size description
--------- ----- ---- --------------------------------------------
   262144    32 8192 allocating memory for b-tree...
    65536     8 8192 allocating memory for virtual array...
   262144    32 8192 allocating memory for name pointers...
-----------------------------------------------------------------
   589824            total memory allocated

Loading processor module C:\Program Files\IDA 7.2\procs\i5164.dll for 8051...OK
Loading type libraries...
Autoanalysis subsystem has been initialized.
Database for file '307.bin' has been loaded.
============================================================
GhIDA Decompiler v0.1
Andrea Marcelli<EMAIL_ADDRESS>Cisco Talos, June 2019
GhIDA Decompiler shortcut key is Ctrl-Alt-D
============================================================
GhIDA:: [DEBUG] Reading GhIDA configuration
('GHIDA_CONF.load_save_cached_code', False)
('GHIDA_CONF.load_save_cached_comments', False)
GhIDA:: [DEBUG] code_cache_path: c:\users\user\appdata\local\temp\decompiled_cache_FE46C519975BC6E312BB1719D93A0A64.json
GhIDA:: [DEBUG] comments_cache_path: c:\users\user\appdata\local\temp\comments_cache_FE46C519975BC6E312BB1719D93A0A64.json
GhIDA:: [DEBUG] Registering handlers
GhIDA [DEBUG] ScreenEAHook initialized
---------------------------------------------------------------------------------------------
Python 2.7.18 (v2.7.18:8d21aa21f2, Apr 20 2020, 13:25:05) [MSC v.1500 64 bit (AMD64)]
IDAPython 64-bit v1.7.0 final (serial 0) (c) The IDAPython Team<EMAIL_ADDRESS>---------------------------------------------------------------------------------------------
GhIDA:: [DEBUG] DisasmsHandler HELLO
('GHIDA_CONF.global_settings', True)
('GHIDA_CONF.use_ghidra_server', False)
('GHIDA_CONF.ghidra_install_path', 'E:\\ghidra_9.1.2_PUBLIC_20200212\\ghidra_9.1.2_PUBLIC')
('GHIDA_CONF.ghidra_server_url', 'http://localhost:8080/ghidra/api')
('GHIDA_CONF.show_settings', True)
('GHIDA_CONF.load_save_cached_code', False)
('GHIDA_CONF.load_save_cached_comments', False)
GhIDA:: [INFO] Configuration saved to c:\users\user\appdata\local\temp\ghida_config.json
GhIDA:: [DEBUG] EXPORT_XML_FILE: True
GhIDA:: [DEBUG] Exporting IDA project into XML format

XML Exporter v5.0.1 : SDK 720
-----------------------------------------------------------
Exporting XML <PROGRAM> document ....

GhIDA:: [INFO] Ghidra analysis completed!
GhIDA:: [!] No JSON object could be decoded
GhIDA:: [!] Ghidra headless analysis failed
GhIDA:: [!] Decompilation interrupted.

With what it can be connected ? Incorrect compiler or Something is missing from my installation. The thing is that if I upload it directly through the hydra, then the file will decompile, but you must explicitly indicate where the code is in this file.

@jimmy-sonny Thanks for your awesome work on this plugin.
I came across the same issue "Ghidra Headless Analysis Failure: No JSON object could be decoded".  I wanted to decompile the mips binary with this plugin, and the related environment is：win10, IDA Pro 7.0, python 2.7, and with a local installation of Ghidra。
Finally I solved the issue in my case, here are some steps I have tried. @Zorro9999 Maybe you can learn from it.

run the FunctionDecompile.py in Ghidra directly  --->  it works well
run the ghidra related cmd (analyzeHeadless.bat xxx) in console directly  ---> it failed, according to the error, I added the -processor <languageID> option and made it work in my case
change subprocess.Popen() in ghidra_plugin/lib.py to subprocess.call()and the pass the cmd args as a string  ---> it works

I found the call to subprocess.Popen.poll(p) will set returncode immediately even if the child process doesn't finish in the original way.  As a result, no json file will be generated, which may be the root casue of this issue.
After looking the docs of subprocess, I found the folllow description:

The shell argument (which defaults to False) specifies whether to use the shell as the program to execute. If shell is True, it is recommended to pass args as a string rather than as a sequence.

Since my env is win10, the shell is set to True. So I pass the cmd args as s string and it works finally.
https://github.com/Cisco-Talos/GhIDA/blob/d153e0dddf437b96dbbcd9be23774a538f614317/ghida_plugin/lib.py#L239-L252
In summay, I make the following changes to make it work on my case.

add -processor <languageID> option
wrap path in quotes in case it contains spaces
pass the cmd args as a string if shell=True

cmd = ['"%s"' % ghidra_headless_path,
        ".",
        "Temp",
        "-import",
        '"%s"' % xml_file_path,
        '-readOnly',
        '-scriptPath',
        '"%s"' % ghidra_plugins_path,
        '-postScript',
        'FunctionDecompile.py',
        address,
        '"%s"' % output_path,
        "-noanalysis",
        "-deleteProject",
        "-processor",
        '"%s"' % processor]

# Options to 'safely' terminate the process
if os.name == 'posix':
    kwargs = {
        'preexec_fn': os.setsid
    }
else:
    kwargs = {
        'creationflags': subprocess.CREATE_NEW_PROCESS_GROUP,
        'shell': True
    }
    cmd = ' '.join(cmd)

p = subprocess.Popen(cmd,
                        stdout=subprocess.PIPE,
                        stderr=subprocess.STDOUT,
                        **kwargs)

Good evening. Changed according to your settings. I began to create an XML file and a code file in the folder, but then stop.
""
Total XML Elements:             666
Database exported to: A8E703B551892D73C6BB9E989C60_YuZGb.xml
GhIDA:: [DEBUG] found 0 symbols
GhIDA:: [INFO] XML exporting completed
GhIDA:: [DEBUG] decompiled cache miss (a50)
GhIDA:: [DEBUG] Decompiling a50
GhIDA:: [DEBUG] EXPORT_XML_FILE: False
GhIDA:: [!] global name 'processor' is not defined
GhIDA:: [!] Ghidra headless analysis failed
GhIDA:: [!] Decompilation interrupted.
""

@Zorro9999 If the arch of your binary is x86 or x64 (supported by GhIDA Plugin in default), you can remove the added -processor option. Otherwise, you have to provide the proper processor arg, which is not handled by GhIDA plugin in default.
In my case, I only handle the mips arch automatically. Maybe you can specify it manually first to test if it works. Here is the related information copied from the ghidra_x.x/support/analyzeHeadlessREADME.html#processor.

In my case, the processor arg is MIPS:BE:32:default. You have to find your own.

@cq674350529 cq674350529 Thank you for responding to my problem. I’ll tell you what I did, as you wrote above, I edited the lib.py file according to the example. The plugin began to swear that it does not know this type of processor, where should I indicate it in the code? I don’t understand this. In the output xml file it looks like this:

@Zorro9999 To test it quickly, you can manually specify the processor arg like the following。
cmd = ['"%s"' % ghidra_headless_path,
        ".",
        "Temp",
        "-import",
        '"%s"' % xml_file_path,
        '-readOnly',
        '-scriptPath',
        '"%s"' % ghidra_plugins_path,
        '-postScript',
        'FunctionDecompile.py',
        address,
        '"%s"' % output_path,
        "-noanalysis",
        "-deleteProject",
        "-processor",
        "<processor here>"]    // <=== here

According to the output.xml above, I'm not sure if the ghidra supports it or not. I can't find the 8051:BE:32:default, maybe you can try 8051:BE:16:default. You can try to load your binary with ghidra first. If it works, then you can get the processor info in ghidra dialog box.
By the way, I can test it if it's possible for you to upload the binary.

@cq674350529 You are the best .. Thank you so much. Earned and decompiled correctly.

Dear @cq674350529 and @Zorro9999, I'm sorry for the very late answer. Thanks for your contributions on how to improve the usage of subprocess and how to support other architectures. I'll make some additional tests and I'll include them in a future release.


Find-PSResource returns tags in wrong format, when searching for scripts

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.

Steps to reproduce
Find-PSResource send-passwordnotification -Repository PSGallery | select -ExpandProperty tags

Expected behavior
Find-Script send-passwordnotification -Repository PSGallery | select -ExpandProperty tags

password
e-mail
email
notification
Windows
PSEdition_Desktop
PSEdition_Core
PSScript

Actual behavior
Find-PSResource send-passwordnotification -Repository PSGallery | select -ExpandProperty tags

password,
e-mail,
email,
notification,
Windows,
PSEdition_Desktop,
PSEdition_Core

Find-PSResource send-passwordnotification -Repository Local | select -ExpandProperty tags

password

e-mail
email
notification
Windows
PSEdition_Desktop
PSEdition_Core
PSScript

Error details
no error
Environment data
Powershell: 5.1 and 7.3.5
PSResourceGet: 0.5.22
Visuals
No response

This problem is also probably caused by comma as separator in script metadata field Tags.

And afterwards Find-PSResource is not able to get this package metadata, like in #1220

Thanks so much for all the testing @peetrike we really appreciate it! This is resolved by #1236


Suggestion: Put the documentation on Types

As angular does, is so easy to see documentation of a library when the author puts all the documentation directly in the code about how to use it and what you can do with each option.
For presets, suites, etc, it will be easier to people to learn more about your package.
Writing a documentation website is a possibility too but takes much more time than just writing some comments on the exported code.
I'm creating this because I needed to dig into the source code to know what chunkSize and compareSize means.

Good idea! I will add inline docs and also extend the README.

@yamiteru If you want to try to automate the doc generation, I once use @microsoft/api-documenter, I create a custom markdown generator: https://github.com/H4ad/serverless-adapter/blob/main/scripts/libs/CustomMarkdownDocumenter.ts, what it does is read the TS source files and then extract comments, types, interfaces, etc, and use it to generate markdown files.
Takes a lot of time to configure but the output was very good: https://serverless-adapter.viniciusl.com.br/docs/api/Adapters/AWS/ApiGatewayV2Adapter

@yamiteru Oh, I never see that plugin, is the same thing, the only difference is that I have more control over how it will be parsed and generated but is the same idea.
The plugin will be good only if you pretend to expose your docs with docusaurus, I was thinking that you want to put all in the README.md.
Anyway if you want to put it inside the README or as docusaurus maybe I can try to help build the parser to generate the documentation with special code tags, I currently don't have the time but once I finish my current university exams I can work on it, I'm excited to use this library in my performance reviews.

Thanks for the info.
Auto-generating markdown docs in README would also be pretty neat!
I'll do some research.


Very Long String in source results in cryptic error message.

Hello
When you have a very long string in the scala source, scala.js produces a very cryptic error message:
sbt:long-string> withSjs/compile
[info] Compiling 2 Scala sources to /home/dpoliakas/Dev/personal/long-string/withSjs/target/scala-2.13/classes ...
[error] encoded string too long: 525213 bytes
[error] one error found
[error] (withSjs / Compile / compileIncremental) Compilation failed
[error] Total time: 0 s, completed 18 Aug 2020, 11:12:18

The same error without scalajs produces a slightly better error message
sbt:long-string> noSjs/compile
[info] Compiling 2 Scala sources to /home/dpoliakas/Dev/personal/long-string/noSjs/target/scala-2.13/classes ...
[error] Error while emitting example/Main$
[error] UTF8 string too large
[error] one error found
[error] (noSjs / Compile / compileIncremental) Compilation failed
[error] Total time: 0 s, completed 18 Aug 2020, 11:12:11

Minimal reproduction https://github.com/disblader/long-string
While this is not a big problem per se, it can definitely lead one on a wild goose chase. It would be good if this could be improved.

Thanks for filing the issue :)

So I understand correctly that the best we can do here is improve the error message, right? If so, should we do this upstream?

Yes, we can only improve the error message.
Even if we do it upstream, apparently this error comes from the back-end, so the Scala.js back-end kicks in first and issues its own (worse) message, preventing the JVM back-end to run and issue its (better) message.


Revert recursive behaviour to allow non-breaking v8.2.x release

Hello, I've just run into the breaking change of #238 and later reported as #246.
The proposed fixes in #246 are still breaking changes.
This PR performs a clean revert of the two commits related to #238 and #239 to allow a non-breaking v8.2.x release without preventing a second attempt in a later breaking v9.0.0 release to include this feature (e.g. by cherry-picking the commits back in again).

The proposed fixes in #246 are still breaking changes

Could you clarify? I'm not opposed to reverting at all, just feel like I'm missing something if the fix you're referring to is to only include top-level files by default. Why would that be breaking?

Hi Vincent, thanks for the quick reply.
The proposed fix to modify the default include-regex sadly won't help modules that specify a value for it e.g. sharp.

@lovell Mind if we pull you in for review in a future PR that reintroduces the feaure?

@vweevers With pleasure, thank you.

8.2.1


Koolo.exe auto closes

Was running fine for a week. Paused and shut down this morning to check stash. Then tried to run the bot up again to no prevail.
Right click Koolo.exe run as administrator and it opens for a millisecond then automatically closes the CMD. Please help!

Most likely something wrong in your config file. Try running it with an older version of your config or a freshly downloaded one.
Any small typo has caused that for me.

Narrowed the issue down to rare pickit. If i delete the // in front of the rare circlets, the CMD refuses to run and auto closes.

maybe wrong syntax? share the line causing the crash

So if I delete the // in front of [type] for all 3 lines, koolo.exe refuses to open and run.
// rare circlets
//[type] == circlet 		&& [quality] == rare && [flag] != ethereal # [sockets] >= 2 && [itemaddclassskills] >= 2 && [fcr] >= 20 && ([dexterity]+[strength] >= 10 || [maxhp] >= 20) && (([itemtohitpercentperlevel] >= 1) || ([dexterity]+[strength] >= 20) || ([maxhp] >= 35) || ([enhanceddamage] >= 15) || ([fhr] >= 10) || ([frw] >= 30) || ([coldresist]+[fireresist]+[lightresist]+[poisonresist] >= 40))
//[type] == circlet 		&& [quality] == rare && [flag] != ethereal # [sockets] >= 2 && [amazonskills] >= 2 	   && [frw] >= 30 && ([dexterity]+[strength] >= 10 || [maxhp] >= 20) && (([itemtohitpercentperlevel] >= 1) || ([dexterity]+[strength] >= 20) || ([maxhp] >= 35) || ([enhanceddamage] >= 15) || ([fhr] >= 10) || ([fcr] >= 20) || ([coldresist]+[fireresist]+[lightresist]+[poisonresist] >= 40))
//[type] == circlet 		&& [quality] == rare && [flag] != ethereal # [sockets] >= 2 && [paladinskills] >= 2 	   && [itemtohitpercentperlevel] >= 1 && (([dexterity]+[strength] >= 10) + ([dexterity]+[strength] >= 20) + ([maxhp] >= 20) + ([maxhp] >= 35) + ([enhanceddamage] >= 15) + ([fhr] >= 10) + ([frw] >= 30) + ([fcr] >= 20) + ([coldresist]+[fireresist]+[lightresist]+[poisonresist] >= 40)) >= 2


While training on Tox21 data: "NameError: name 'io_utils' is not defined"

Describe the bug
Hi,
I am encountering a bug while training the GNN model with Tox21 data. After the model training finishes, I get an error as below. The checkpoint is not getting saved as well. What should I do?
Train  accuracy: 0.9083333333333333
Validation  accuracy: 0.8970588235294118
Test  accuracy: 0.9105392156862745
Best val result:  {'epoch': 700, 'loss': tensor(0.1380, device='cuda:0', grad_fn=<DivBackward0>), 'acc': 0.9044117647058824}
Test result:  {'prec': 0.8022022213711222, 'recall': 0.72627360451945, 'acc': 0.9105392156862745, 'epoch': 999}
Traceback (most recent call last):
  File "train.py", line 1179, in <module>
    main()
  File "train.py", line 1156, in main
    benchmark_task(prog_args, writer=writer)
  File "train.py", line 932, in benchmark_task
    writer=writer,
  File "train.py", line 247, in train
    plt.savefig(io_utils.gen_train_plt_name(args), dpi=600)
  File "C:\Users\kacpe\Desktop\Github\gnn-model-explainer\utils\io_utils.py", line 667, in gen_train_plt_name
    return "results/" + io_utils.gen_prefix(args) + ".png"
NameError: name 'io_utils' is not defined

To Reproduce
Steps to reproduce the behavior:

Train the model on Tox data

Platform (please complete the following information):

Windows 10
Python 3.7

Submitted a PR @ https://github.com/RexYing/gnn-model-explainer/pull/20


Timetable for next release?

Been keeping an eye out for any showstopper bugs in the last release, but just haven't seen any in the last 5 months.  I wanted to give it about 5-6 months before I asked, but how long to a new bug fix release?
I was hoping to push a the new official release as a SublimeText dependency (for plugins that use it), and I wanted to have the fix for this https://github.com/waylan/Python-Markdown/issues/539 as I skipped pushing out the last version because of this.
No rush; just curious.

I hadn't remembered that there were any notable changes since the last release. I should be able to get one out later this week or next.

Oh, and thanks for the reminder.

np and thanks!

I understand you are probably just busy and probably haven't forgotten, but I figured I 'd just send a reminder due to what was last discussed. Thanks.

Yup, it completely slipped my mind.

Figured I'd go ahead and send a new reminder :slightly_smiling_face: .

Finally got the release out for 2.6.9. Sorry for the long delay.


Update to correct method name in local example

This is to help #1713

Can one of the admins verify this patch?

Thanks!

Thanks!


it says it doesnt have permission

429 = need wait 20 seconds. Discord has rate limit

If it gives missing access it means the bot is not on the server/you didn't give it oauth2 rights to applications.commands


Instrument ecs-logging-java to set service name

When ecs-logging-java is used in an application, we should instrument the constructor of the formatter/encoder to set the service name that has been configured for the agent if it has not been explicitly set in the logging configuration.

@felixbarny have the PRs referenced (https://github.com/elastic/ecs-logging-java/pull/168 and https://github.com/elastic/apm-agent-java/pull/2296) implemented what you intended for this, or is there more to do?

Done for log4j, but not yet other logging frameworks


Added onRouteUpdate in versioned_docs 👨💻

Issue Reference: #3399
onRouteUpdate documented and Added to version-2.0.0-alpha.63

Motivation
Excited for first time contribution to Facebook's repository.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
@slorber Please review
No code has been changed. But documented the life cycle events.

Related PRs
No

@slorber I have signed the Facebook's CLA and after that only I created this pull request. But its showing I have not. 🥺🥺

@slorber  help  in reviewing the pr

@saadpasta Thanks for letting me know that fix. 😊😊 I have updated the pull request.
@slorber please review PR. Bit excited for this 😁

@lex111 I have removed the changes in version_docs and updated the changes on master docs.
Please review. @lex111 @saadpasta  @slorber

@lex111 @saadpasta I have made changes based on end user perspective. Please review

Hi
Will be back from holiday on Monday 28 and catch up with PRs/issues soon
😉
Le mer. 16 sept. 2020 à 12:41, Abir<EMAIL_ADDRESS>a écrit :

@lex111 https://github.com/lex111 @saadpasta
https://github.com/saadpasta I have made changes based on end user
perspective. Please review
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/docusaurus/pull/3444#issuecomment-693350178,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFW6PQIBPUA5QGDCZ5L3ODSGCP57ANCNFSM4RJFQ3HA
.

Hi,
I'm sorry @imabp but I think you did not understand the purpose of this feature, and this doc is not good documentation for it. Unfortunately, I come back from holiday and have a lot of issues to check, so can't dedicate time to help you do this contribution. I'm going to close this PR.
@saadpasta looks like you understand the feature better, do you want to send a PR?
(Note: I don't think it's worth documenting onRouteUpdateDelayed as it's used nowhere, if nobody complains I'd like to get it removed).


Fix error ctor setup bug

Purpose
Fix error constructor creating unintended error types due to wrong error constructor being setup into error type.
Steps to reproduce the bug:

Clone/Download https://github.com/wso2/ballerina-integrator/tree/56c06535cb80672fef9c17a00f092b6ae6086691
Build docs/learn/integration-tutorials/transforming-message-content/ project.

Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

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

[ ] Increased Test Coverage
[ ] Added necessary documentation

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

Can we add test cases as well?

Can we add test cases as well?

Issue https://github.com/ballerina-platform/ballerina-lang/issues/18256 is created as it's bit difficult to reproduce the issue into an unit test


Cannot build with Vite and latest sentry plugin version

Environment
"vite": "4.4.10" (same with 5.x)
"@sentry/vite-plugin": "2.18.0",
vite.config.ts
import { defineConfig, loadEnv } from 'vite';
import react from "@vitejs/plugin-react";
import { sentryVitePlugin } from '@sentry/vite-plugin'
import { createHtmlPlugin } from 'vite-plugin-html';
import VitePluginReactRemoveAttributes from 'vite-plugin-react-remove-attributes';

// https://vitejs.dev/config/
export default defineConfig(({ mode }) => {
  const env = loadEnv(mode, process.cwd());
  const plugins = [react()];

  plugins.push(
    sentryVitePlugin({
      authToken: env.VITE_SENTRY_KEY,
      org: env.VITE_SENTRY_ORG,
      project: env.VITE_SENTRY_PROJECT,
      telemetry: false,
      sourcemaps: {
        assets: './build/assets',
        ignore: ['node_modules']
      }
    })
  );

  return {
    build: {
      outDir: 'build',
      commonjsOptions: {
        include: [/node_modules/, /types/],
      },
      sourcemap: true,
    },
    worker: {
      format: 'es'
    },
    server: {
      host: '<IP_ADDRESS>',
      port: 3000
    },
    preview: {
      host: true,
      port: 3000,
      strictPort: true
    },
    plugins: [
      ...plugins,
      {
        name: 'configure-response-headers',
        configureServer: (server) => {
          server.middlewares.use((_req, res, next) => {
            res.setHeader('Access-Control-Allow-Origin', '*');
            res.setHeader('Strict-Transport-Security', 'max-age=31536000');
            res.setHeader('X-Content-Type-Options', 'nosniff');
            res.setHeader('Referrer-Policy', 'same-origin');
            next();
          });
        },
      },
      createHtmlPlugin({
        inject: {
          data: {
            RECAPTCHA_SITE_KEY: env.VITE_RECAPTCHA_SITE_KEY,
          },
        },
      }),
      ...(env.VITE_E2E_ENVIRONMENT === 'true' ? [] : [
        VitePluginReactRemoveAttributes({
          attributes: ['data-testid'],
        }),
      ]),
    ],
    publicDir: 'public',
    optimizeDeps: {
      include: ['stb-common'],
    },
  };
});

Steps to Reproduce

Run build with tsc --build tsconfig.build.json && vite build

Expected Result
Build is completed, release id is created and sourcemaps are sent.
Actual Result
client:build: cache miss, executing ed08942129d3851b
client:build:
client:build: ><EMAIL_ADDRESS>build /home/project/apps/client
client:build: > tsc --build tsconfig.build.json && vite build
client:build:
client:build: vite v4.4.10 building for production...
client:build: transforming...
client:build:
client:build:  WARN  src/constants.ts (1:8) Error when using sourcemap for reporting an error: Can't resolve original location of error.
client:build:
client:build: ✓ 4322 modules transformed.
client:build: ✓ built in 26.62s
client:build:
client:build:  ERROR  "Subscriptions" is not exported by "../common/types/dist/index.js", imported by "src/constants.ts".
client:build: file: /home/project/apps/client/src/constants.ts:1:8
client:build: 1: import {Subscriptions} from "stb-common";
client:build:            ^
client:build: 2: import {NotificationsType} from "./store/Notifications";
client:build: 3: export const AXIOS_REQUEST_TIMEOUT = 1e4;
client:build:
client:build:
client:build:  ERROR  error during build:
client:build: RollupError: "Subscriptions" is not exported by "../common/types/dist/index.js", imported by "src/constants.ts".
client:build:     at error (file:///home/project/apps/node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:2287:30)
client:build:     at Module.error (file:///home/project/apps/node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:13745:16)
client:build:     at Module.traceVariable (file:///home/project/apps/node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:14175:29)
client:build:     at ModuleScope.findVariable (file:///home/project/apps/node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:12615:39)
client:build:     at MemberExpression.bind (file:///home/project/apps/node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:9539:49)
client:build:     at Property.bind (file:///home/project/apps/node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:5892:23)
client:build:     at ObjectExpression.bind (file:///home/project/apps/node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:5888:28)
client:build:     at VariableDeclarator.bind (file:///home/project/apps/node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:5892:23)
client:build:     at VariableDeclaration.bind (file:///home/project/apps/node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:5888:28)
client:build:     at ExportNamedDeclaration.bind (file:///home/project/apps/node_modules/.pnpm/rollup@3.29.4/node_modules/rollup/dist/es/shared/node-entry.js:10673:27)
client:build:
client:build:  ELIFECYCLE  Command failed with exit code 1.
client:build: ERROR: command finished with error: command (/home/project/apps/client) pnpm run build exited (1)

I'm trying to move from 0.4.0 to the 2.18.0 With previous version build is completed. Now I got this error. Am I missing something with the configuration? Without the plugin the build is complete without any issues...

I don't see how this error can be related to the Sentry bundler plugins. Would you mind sharing a minimal reproduction example we can pull? Thanks!

What I just discovered is that the build breaks from version 0.6.0. Version 0.5.1 is working. Was there any significant changes between these two?

These are the changes: https://github.com/getsentry/sentry-javascript-bundler-plugins/compare/0.5.1...0.6.0
Looks very much unrelated to this error.

We do not consider the value passed from the plugin to the SDK as public API. The plugin automatically passes the release value to the init call, or you can also use an ordinary env var.
All I can say is that import.meta.env.SENTRY_RELEASE?.id is completely wrong. import.meta.env will never contain objects.

When you provide a release value to the plugin, it automatically injects it into your bundle.
I don't think we can debug this further unless we get reproduction.

I've just noticed, that removing
commonjsOptions: {
  include: [/node_modules/, /types/],
},

breaks the build as well. Which is expected as it informs rollup commonjs plugin to transform cjs to ejs. So is it possible that sentry plugin is overriding this option or not using it at all?

Hey @pippo111, thanks for the reproduction. We're taking a look and will get back to you.

I'm also running in to the same issue, enablingsentryVitePlugin when we have optimizeDeps.include: ["@internal/units"] breaks the build.
 RollupError: "default" is not exported by "../../packages/units/lib/index.js", imported by "../../packages/shared/src/utils/units.ts".
at error (app/node_modules/rollup/dist/es/shared/node-entry.js:2287:30)
at Module.error (app/node_modules/rollup/dist/es/shared/node-entry.js:13745:16)
at Module.traceVariable (app/node_modules/rollup/dist/es/shared/node-entry.js:14175:29)
at ModuleScope.findVariable (app/node_modules/rollup/dist/es/shared/node-entry.js:12615:39)
at Identifier.bind (app/node_modules/rollup/dist/es/shared/node-entry.js:8319:40)
at CallExpression.bind (app/node_modules/rollup/dist/es/shared/node-entry.js:5892:23)
at CallExpression.bind (app/node_modules/rollup/dist/es/shared/node-entry.js:9890:15)
at VariableDeclarator.bind (app/node_modules/rollup/dist/es/shared/node-entry.js:5892:23)
at VariableDeclaration.bind (app/node_modules/rollup/dist/es/shared/node-entry.js:5888:28)
at Program.bind (app/node_modules/rollup/dist/es/shared/node-entry.js:5888:28)

Hello, sorry we are still looking into this.
@kYem do you have a reproduction for that, some github repo or a stackblitz would help a lot.

@pippo111 thanks again for the reproduction, it helped us narrowing down the problem. It looks like when our plugin injects the release into the bundle it trips up @rollup/plugin-commonjs. We have a potential solution in mind, but still need to ensure it doesn't break anything else.
In the meantime, to unblock you, you can set release: { inject: false } in the plugin options:
// vite.config.ts
...
      sentryVitePlugin({
        authToken: 'sntrys_',
        org: 'org',
        project: 'project',
        telemetry: false,
        release: { inject: false }
      }),

And then manually set a release in Sentry.init as written in the docs.


⚠ ClamAV has degraded performance

In ee4dfe6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 78 ms

Resolved: ClamAV performance has improved in f0779c2.


Create a REST api that can be read by guests and CRUD by users

Category
REST API
Amplify CLI Version
4.32.0
AWS Services

Cognito
DynamoDB

Description
I would like to create a DynmoDB table and access it through a REST API. I want this API to be read by guests, but only CRUD by users. I believe I walked though the CLI to set this up. I added a specific use case to the lambda function:
// blogLambda/src/app.js
...
/**
 * Getting the basic blog details.
 *
 * This is called to get the basics of the blog's details. This returns the
 * number of users and posts.
 */
app.get( `/blog`, function ( request, response ) {
  let params = {
    TableName: tableName,
    Key: {
      'PK': `#BLOG`,
      'SK': `#BLOG`
    }
  }
  dynamodb.get( params, ( error, result ) => {
    if ( error ) {
      response.json( { statusCode: 500, error: error.message } )
    } else {
      response.json(
        {
          statusCode: 200,
          url: request.url,
          body: result.Item
        }
      )
    }
  } )
} )

When I attempt to call this api using the SDK,
API.get( `blogAPI`, `/blog/`, {} )
  .then( blogResponse => console.log( `response`, { blogResponse } ) )
  .catch( error => console.log( `error`, { error } ) )

I get the following errors:
Origin http://localhost:8000 is not allowed by Access-Control-Allow-Origin.
XMLHttpRequest cannot load https://<endpoint>/blog/ due to access control checks.

It works correctly when the post api is called. I can even use Postman to GET the data:

What do I need to do in order to get the GET to work through the front end? I've added the Amplify CLI steps I took to produce this. I'd also like to change the auth attributes used, but I'd like to tackle this one first.
Process
tylernorlund@Tylers-MacBook-Pro Blog % amplify add api
? Please select from one of the below mentioned services: REST
? Provide a friendly name for your resource to be used as a label for this category in the project: blogAPI
? Provide a path (e.g., /book/{isbn}): /
? Choose a Lambda source Create a new Lambda function
? Provide a friendly name for your resource to be used as a label for this category in the project: blogLambda
? Provide the AWS Lambda function name: blogLambda
? Choose the runtime that you want to use: NodeJS
? Choose the function template that you want to use: CRUD function for DynamoDB (Integration with API Gateway)
? Choose a DynamoDB data source option Create a new DynamoDB table

Welcome to the NoSQL DynamoDB database wizard
This wizard asks you a series of questions to help determine how to set up your NoSQL database table.

? Please provide a friendly name for your resource that will be used to label this category in the project: blogDB
? Please provide table name: blogDB

... <adding columns and defining PK/SK/GSI>

? Do you want to add a Lambda Trigger for your Table? Yes
? Select from the following options Create a new function
Successfully added resource blogDBTrigger731a3221 locally
Successfully updated resource blogDBTrigger731a3221 locally
? Do you want to edit the local blogDBTrigger731a3221 lambda function now? No
Successfully added DynamoDb table locally
? Do you want to access other resources in this project from your Lambda function? Yes
? Select the category api, auth, function, storage
No resources found for api
No resources found for auth
Function category has a resource called blogDBTrigger731a3221
? Select the operations you want to permit for blogDBTrigger731a3221 create, read, update, delete
Storage category has a resource called blogDB
? Select the operations you want to permit for blogDB create, read, update, delete

You can access the following resource attributes as environment variables from your Lambda function
        ENV
        FUNCTION_BLOGDBTRIGGER731A3221_NAME
        REGION
        STORAGE_BLOGDB_ARN
        STORAGE_BLOGDB_NAME
? Do you want to invoke this function on a recurring schedule? No
? Do you want to configure Lambda layers for this function? No
? Do you want to edit the local lambda function now? No

So it looks like when this is created, it sets the authorization to AWS_IAM. Is this correct?


Port new site to staging and production

The new site for Filecoin Foundation, which was being worked on in a downstream fork of this repo, needs to be merged upstream.
Aside from the merge, the CI needs to be configured to match the new Nuxt app's config.

Thanks!


change match/fold extension for bools to named parameters

Hi!
I just found out, that match (and fold) for booleans exist. Which is nice and I prefer it over the ternary operator. What I have troubles with, is to distinct between true and false. Sure, you get used to it, if used more often, but at first, I didn't know which is which, since there are actually parameterless functions to provide as paramters.
So my suggestion would be, use named parameters.
Like so:
true.match(onFalse: () => 'it is false', onTrue: () => 'it is true');
I use the match function, if I want to continue with the return value for more functional stuff.
Like so:
result.match(() => something(), () => somethingdifferent()).morestuff()
Feels better then this:
(result ? something() : somethingdifferent()).morestuff()
BTW, a very nice library, coming from F# to flutter/dart, it feels just like home 😄
cheers, Thomas

They're both required parameters though, and it mimics the other interfaces that require callbacks for both "sides" of a class.  I'd rather not type more.  I can also use aBool.match(whenFalse,whenTrue) if I have the function names, which is a nicely compact notation, similar to anEither(whenLeft,whenRight) or anOption(whenNone,whenSome).

Hi @thomrad
As @RandalSchwartz says, the match extension for boolean mimics the same definition of types like Option and Either. This is mostly a convention which makes writing code more concise.
Furthermore, pattern matching is coming. This will give dart a native way of matching over patterns, possibly causing match in fpdart to become deprecated.
Closing this then, let me know if you have any further question.


Refactoring IndexCatalogObject

First step of #362
We also need to deal with 2 raw pointers. Due to compatibility reason, this is not easy.

Coverage decreased (-0.004%) to 66.276% when pulling<PHONE_NUMBER>ad07866d14375e375c25647c3196b5 on AngLi-Leon:PR into 9b11d92dd5a2a0b32d5c39a9ba0b1aa45fe0b654 on cmu-db:master.


Technical documentation (Onboarding)

Hey guys, is there any technical documentation about this project? I cannot find any docs besides the image in readme.
Because I just noticed Aspire is required, not only Docker, possibly RabbitMQ, configure DB (Postgresql), Redis Image, so, there are more stuff that needs to be explained before launching for the first time the app.

same here.. Would l like someone to make a walk through on video

ok, it looks actually it is not big issue, however I will try to add some additional information to the setup in order to make it a bit more clear, possibly a simple onboarding doc.


Chart Labels Are Not Getting Translated

Chart labels are not getting translated - see screenshot.
County selector is missing translation, and there are some issues with tab labels, but these are covered by a separate ticket.

Is this the same as issue 112?

Since I did not create the issue I need to trace it back to Teams for the testers

Tab labels issue (rogue / ns etc) is already covered and done, but I think we are missing some chart label translations.
I'll add auto-translations for now

Thanks @AlanSl .  @mboskin to respond to your question - it's a different issue.

We should now be all up to date here except the new Positivity % chart which will be updated in the build after we get the updated translations sheet


Optimizations

Codecov Report

Merging #8 (c9a7f0c) into master (819de75) will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   90.73%   90.86%   +0.12%
==========================================
  Files          11       11
  Lines         907      865      -42
==========================================
- Hits          823      786      -37
+ Misses         84       79       -5

Impacted Files
Coverage Δ

src/db_entries.rs
0.00% <ø> (ø)

src/lib.rs
88.72% <100.00%> (-0.57%)
:arrow_down:

src/model.rs
95.30% <100.00%> (+0.82%)
:arrow_up:

src/note.rs
95.83% <100.00%> (+1.05%)
:arrow_up:

Continue to review full report at Codecov.

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


Option to change the placeholder from "Select..." to anything else

@Barlevin you can use overrideStrings prop to customize as per your needs https://github.com/hc-oss/react-multi-select-component/blob/master/docs/recipes/localization.md


add_summary won't accept the tensor returned by tensor_summary

Related Problem
https://github.com/tensorflow/tensorflow/issues/6778
Environment Info
Operating System: Kubuntu 16.04 LTS
Installed TensorFlow version:
No CUDA acceleration. Compiled from source.
commit hash:
4ac9c09d5ca57a03b8daa5fb9e295947b1619854
bazel version:
Build label: 0.4.4
Build target: bazel-out/local-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Wed Feb 1 18:54:21 2017<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

Example
I restored a graph from a checkpoint and get the variables. Then I tried to write the summary to the log but got an error.
sess = tf.Session()
saver = tf.train.import_meta_graph('my-model.meta')
saver.restore(sess, tf.train.latest_checkpoint('./'))
all_vars = tf.get_collection(tf.GraphKeys.GLOBAL_VARIABLES)
writer = tf.summary.FileWriter('./many/log',sess.graph)
writer.add_summary(tf.summary.tensor_summary("conv", all_vars[0]))

>>> writer.add_summary(tf.summary.tensor_summary('conv',all_vars[0]))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/summary/writer/writer.py", line 107, in add_summary
    event = event_pb2.Event(summary=summary)
TypeError: Parameter to MergeFrom() must be instance of same class: expected tensorflow.Summary got Tensor.

Official API Description
According to the official API (https://www.tensorflow.org/api_docs/python/tf/summary/FileWriter),
tf.summary.tensor_summary

Returns:
A scalar Tensor of type string. The serialized Summary protocol buffer.

add_summary(summary, global_step=None)

You can pass the result of evaluating any summary op, using tf.Session.run or tf.Tensor.eval, to this function. Alternatively, you can pass a tf.Summary protocol buffer that you populate with your own data. The latter is commonly done to report evaluation results in event files.

So I believe this is a bug.

Also https://github.com/tensorflow/models/issues/764

Sorry I realized this issue is not about the functions but my understanding of tensorflow.

@KaraRyougi can you please provide your solution since I'm stuck on the same problem :(

@KaraRyougi @IdoHakimi - were you able to resolve this problem ?


Add healthcheck to dockerfile

Merge after #1087

[x] test on staging before merging

Deployment failed, need to review this.

Ok, this is working now


2023

Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in _run_code
File "/home/zubyl/Desktop/whatsapp-play/wplay/main.py", line 10, in
from wplay import online_tracker
File "/home/zubyl/Desktop/whatsapp-play/wplay/online_tracker.py", line 5, in
from wplay.utils import browser_config
File "/home/zubyl/Desktop/whatsapp-play/wplay/utils/browser_config.py", line 37, in
from wplay.utils.SessionManager import SessionManager
File "/home/zubyl/Desktop/whatsapp-play/wplay/utils/SessionManager.py", line 7, in
from whaaaaat import Separator, prompt
File "/home/zubyl/Desktop/whatsapp-play/.venv_new/lib/python3.11/site-packages/whaaaaat/init.py", line 6, in
from prompt_toolkit.token import Token
File "/home/zubyl/Desktop/whatsapp-play/.venv_new/lib/python3.11/site-packages/prompt_toolkit/init.py", line 16, in
from .interface import CommandLineInterface
File "/home/zubyl/Desktop/whatsapp-play/.venv_new/lib/python3.11/site-packages/prompt_toolkit/interface.py", line 19, in
from .application import Application, AbortAction
File "/home/zubyl/Desktop/whatsapp-play/.venv_new/lib/python3.11/site-packages/prompt_toolkit/application.py", line 8, in
from .key_binding.bindings.basic import load_basic_bindings
File "/home/zubyl/Desktop/whatsapp-play/.venv_new/lib/python3.11/site-packages/prompt_toolkit/key_binding/bindings/basic.py", line 9, in
from prompt_toolkit.renderer import HeightIsUnknownError
File "/home/zubyl/Desktop/whatsapp-play/.venv_new/lib/python3.11/site-packages/prompt_toolkit/renderer.py", line 11, in
from prompt_toolkit.styles import Style
File "/home/zubyl/Desktop/whatsapp-play/.venv_new/lib/python3.11/site-packages/prompt_toolkit/styles/init.py", line 8, in
from .from_dict import *
File "/home/zubyl/Desktop/whatsapp-play/.venv_new/lib/python3.11/site-packages/prompt_toolkit/styles/from_dict.py", line 9, in
from collections import Mapping
ImportError: cannot import name 'Mapping' from 'collections' (/usr/lib/python3.11/collections/init.py)

Change line 9 in /home/zubyl/Desktop/whatsapp-play/.venv_new/lib/python3.11/site-packages/prompt_toolkit/styles/from_dict.py to
try:
    import collections from Mapping
except ImportError:
    import collections.abc from Mapping

@kiddersmxj  Theres a mistake in your code snippet.
try:     from collections import Mapping except ImportError:     from collections.abc import Mapping
This works fine.


Feat implement backup to storage node js

Could not generate an in-memory file to be sent to the storage API
The function generates the CSV file on disk and sends it to the storage API.

THANK YOU! All changes merged 🥳
Please reach out to me on our Discord server if you would like to claim your Appwrite swags! As a way of saying thank you, we would also love to invite you to join the Appwrite organization on GitHub. Please share your GitHub username with us on Discord.
You can accept the invite by visiting https://github.com/orgs/appwrite/invitation. By joining our team, you will officially be an Appwrite maintainer on GitHub.
You can change your membership visibility settings, so your new Appwrite team membership badge will show up on your personal GitHub profile.
Please feel free to look for more PRs you might be interested in helping with on our long list of Hacktoberfest friendly issues and help make Appwrite better :)


Dirty and Destroy, fixes #14, #16

hey @rubenstolk, thanks for the PR!! I'll review it tonight, although after a quick skim it looks great. Very useful addition.

Hi @jakubfiala, yeah things are getting better, just added a fix for #17 as well!


Status bar not rendering correctly in Mintty on Bash on Windows

Messing around with bash on windows with Mintty wsl bridge and the status bar isn't loading properly.
https://github.com/mintty/wsltty
Everything is currently green on it, even though my agnoster theme is looking fine and I've got the powerline fonts installed an linked in the registry the way you specified in the readme.

Works for me.
Things you need to verify:

make sure you're running Windows 10 Creators update
make sure you're running Ubuntu 16.04 ($cat /etc/issue inside WSL)
make sure you kill all tmux process and didn't alter ~/.tmux.conf nor ~/.tmux.conf.local

The tmux.conf is completely untouched, tmux.conf.local is the same as it is
on my Mac and my Ubuntu computer. I am starting to see some other problems
with wsltty so I think it's a wsltty or Windows issue. I'll check the dist
version tonight after work. But honestly, probably just gonna move to using
a vm so I can mount my Ubuntu disks in it rather than install ext fs
drivers on Windows and not have to mess with the silly windows registry.
On Jun 2, 2017 2:42 AM, "Gregory Pakosz"<EMAIL_ADDRESS>wrote:

Works for me.
Things you need to verify:

make sure you're running Windows 10 Creators update
make sure you're running Ubuntu 16.04 ($cat /etc/issue inside WSL)
make sure you kill all tmux process and didn't alter ~/.tmux.conf
nor ~/.tmux.conf.local

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gpakosz/.tmux/issues/83#issuecomment-305703554, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACmFnuUfPoIqmiInNKdra9Hgb5gChp5jks5r_67QgaJpZM4Ntx6K
.

Was on Ubuntu 14.04, good call. I updated, working properly now. I thought I had done the distupgrade... guess not.

@AXington good to know! Please pay attention to the Troubleshooting paragraph in README. Until I do something automatic you have to edit your ~/.tmux.conf.local copy to remove attributes like bold or blink


fix(es/minifier): Fix react-scroll

Description:

BREAKING CHANGE:

Related issue (if exists):

https://github.com/vercel/next.js/discussions/30237#discussioncomment-2053528
Closing as already fixed.


Added mechanism to pass arguments to client.

The first endpoint to use is boards, specifically actionsresource that accept various arguments. If approved, will add other endpoints that accept arguments.
Example
var args []*trello.Argument
args = append(args, trello.NewArgument("filter", "updateCard:idList"))
args = append(args, trello.NewArgument("since", "2016-02-01"))
args = append(args, trello.NewArgument("before", "2016-02-05"))

actions, err := board.Actions(args...) // query string will be ?before=2016-02-05&filter=updateCard%3AidList&since=2016-01-28

Resolves #15

This is really good addition, @gedex. What would you think about the following API?

type Arg map[string]string
type Args []Arg

args := trello.Args{"filter": "updateCard:idList", "before": "2016-02-05"}
actions, err := board.Actions(args...)

actions, err := board.Actions(trello.Arg{"filter": "updateCard:idList"})

This is great, and something I was just thinking of adding.  I merged this into my own fork and am using it.  I fully support merging this or something like it.

@gedex any input on my last comment?

Why Trello.Arg instead of a map[string]string?

Accepting a list of argument keypairs is a good idea that it can be of the form of variadic arguments, instead of requiring a nil for all calls for which you don't want to send arguments.

Why trello.Arg instead of just map[string]string?

The only benefit would be the possibility of defining methods on Args without breaking the API. For example, we could add validation/normalization methods on Args etc. It's map[string]string anyway behind the hood.

Accepting a list of argument keypairs is a good idea that it can be of the form of variadic arguments, instead of requiring a nil for all calls for which you don't want to send arguments.

Yes, it will be variadic and optional for sure.

I'm gonna merge and rename to Arg. Thank you for this contribution!

Apologies for late reply, been busy with works. Thanks for the merge @VojtechVitek.


UI improvements for errors on migrations run and redo

Hi!
Now output contains path to migration file if run/redo errored, like
Running migration<PHONE_NUMBER>0245
Executing migration script /path/to/migration/up.sql
Failed with: unrecognized token: "{"

refs #688

@Eijebong, any new comments? Or merge it? =)

Any news here?

Thanks :)


Fix future effective amalgamation alert

*Issue:*https://github.com/bcgov/entity/issues/23873
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

Temporary Url for review: https://business-dashboard-dev--pr-65-3iyrc4pd.web.app