Closing notification takes me to the top of the page

There is a notification that says We are hiring.
When I try to close that by clicking the x on the bar, it sets the URL fragment to # which just scrolls me to the top.
Sometimes, I am at the mid-bottom of the documentation and just to get space, I close the notification bar, but it scrolls me to the top. It is annoying to scroll back to the bottom where I was reading.
File responsible for this:
/site/layouts/partials/notification.html

I've tested this and it looks like it's been resolved. @vighnesh-r, please let us know if it happens again! Thank you. :)


Program unexpected panic when multiply operation.

What version of regex are you using?
1.7.1
Describe the bug at a high level.
Program attempt to multiply with overflow, which causing a panic.
What are the steps to reproduce the behavior?
Using auto-generated fuzz target can reproduce the behavior. it was uploaded to the issue platform.
multiply_overflow.zip
Similar problems were found in the fuzzy test:

regex/src/re_set.rs:419:1
regex/src/re_trait.rs:23:23

What is the actual behavior?
Panic happens, the program crashes.
What is the expected behavior?
Multiply operation without causing a panic

OK, unlike #948 and #949, this one looks like a valid bug. Again, an actual minimal example is much clearer:
fn main() {
    let pattern = "\n\n\n\n&;*****\0\u{2}**\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n****\u{f}*\u{19}\r.\u{19}";
    let group_index =<PHONE_NUMBER>225171988;

    let re = regex::Regex::new(pattern).unwrap();
    let locs = re.capture_locations();
    dbg!(locs.get(group_index));
}

The contract of CaptureLocations::get is:

Returns the start and end positions of the Nth capture group. Returns None if i is not a valid capture group or if the capture group did not match anything. The positions returned are always byte indices with respect to the original string matched.

Obviously the offset here is quite pathological, but the correct return value here is None.
Thank you. Nice find.


expressif-esp32-c6-devkitc-1: first flash issue

after installing the toolchain as suggested by an initial buidling error with:
rustup component add rust-src --toolchain nightly-2024-03-11-x86_64-unknown-linux-gnu
I run into the below error:
Running probe-rs run --chip '${PROBE_RS_CHIP}' ../../build/bin/expressif-esp32-c6-devkitc-1/hello-world/cargo/riscv32imac-unknown-none-elf/release/hello-world
Error: The chip '${PROBE_RS_CHIP}' was not found in the database.
Caused by:
The requested chip '${PROBE_RS_CHIP}' was not found in the list of known targets.

Thanks!
This is b/c the esps don't use probe-rs, but the README.md since #195 incorrectly states that replacing the board name is enough to use the same command line for all boards.
The real fix here is to just default to probe-rs-run for boards supporting that, and dropping the use in the command line. Will get to that.


test doesn't compile on GCC

gcc (Ubuntu 10.2.0-13ubuntu1) 10.2.0

reports
$make test
gcc -Wall -O2 -o test_list test_list.c
test_list.c:7:20: error: variably modified ‘items’ at file scope
    7 | static IntListItem items[n];
      |                    ^~~~~
make: *** [Makefile:7: test] Error 1

in line 6 n is defined as a const size_t, C requires compile-time constant expressions, const in C doesn't mean that
by changing
const size_t n = 1000;

to
#define n 1000

the problem gets fixed

@dcorderoch thanks for spotting this. It seems that your GCC is using the C89 standard to compile the code. The code should not trigger an error for C99 and above.
That said, you are still right - const size_t does not declare a compile-time constant in C and the array declaration works in C99 and above since it's treated as a variable length array. And that wasn't the intention.
Added your fix in 3f4eb2d.


allow data export filtered by workflow

noting this here to collect input, start conversation...
data export by workflow feature request per Notes from Nature conversion to Panoptes:
currently within Project Builder, Data Exports there are the classification, subject, workflow and workflow contents exports, which create related CSV files with the entire project's data, and we’d like to generate the CSV with only a chosen workflow’s data (filtered on the back-end before the CSV is created).
the impetus behind this is that there’ll be multiple institutions working within NfN, each creating there own subject sets and related workflow, and it would be preferred that when they go to export data they can export their specific workflow (i.e. institution’s) data, as opposed to the the entire project’s data, which could be quite large.
currently it looks like there’s a post call made from PFE to the API with the data export type that triggers the CSV generation. i’m sure i’m horribly over-simplifying and embarrassing myself here, but i’m hoping we can add a workflow id param to the API post call, that the API can then filter the data by and then generate the CSV, but i’m sure it’s far from that simple.

There is nothing to stop us generating reports for a specific workflow. I guess we'd have to think about how to present data exports / requests per workflow in the lab pages. I know @marten is working on data exports but not sure where we're at.
@mcbouslog what's a realistic time line on requiring this? I assume it's not needed for project launch?

@camallen - we're hoping to open the project to the various science teams/institutions May 1st, could that be a realistic timeline? Full launch goal is June 1st. Though not needed by dates noted, would be nice to show if possible, but I'm admittedly ignorant when it comes to gauging what's a reasonable timeline for this.
Might be nice project for/if Adler hires a new back-end dev sooner than later? Though again I'm not sure this would be a good intro project to API and could be more pressing needs.

I think the 1st of May will be too ambitious (holidays and travel to the US). When will a science team actually start using the data, that will be a more realistic time frame. In my experience this is much later than after project launch.
@marten thoughts on this / new dev project work?

It'll need a bunch of front-end work too, might not be the best thing for a new team member to have to deal with multiple codebases immediately. I'm not opposed to doing this, and I think it makes more sense in general than lumping everything together. However I don't think it'll have a high priority for a while since it can be worked around by filtering the full export on the workflow column after downloading.

In any case, if we're going to do this, I would suggest we just decide that this is not going to be "a filter", but rather that we stop generating project classification exports, and always generate them per workflow.
Keep in mind that nothing is set up to do permissions per workflow, so all those NfN institutions will have access to all of the project anyways. Yesterday the thought of overarching "organizations" that bundle projects came up, might have been a good fit here (but timing wise it's too late for that).

@zwolf is this fixed?

Yeah, this is in production.


Bootloader connection improvements

Nitrokey Start update: improve connection process after bootloader upload:

[ ] repeat bootloader connection attempt
[ ] additional delay allowing for the bootloader to start

Connected: https://github.com/Nitrokey/nitrokey-start-firmware/issues/51
Details: https://support.nitrokey.com/t/nitrokey-start-update-funktioniert-nicht/2596/4

@daringer Hi! Could you take a look at this for the v0.4 release?

does up to 3 retries, while being chatty
delay introduced, which increases for each re-try
messages massively reworked, please see #40


⚠ TROJAN 🇸🇬 Singapore SGV 1 has degraded performance

In 5d23e3d, TROJAN 🇸🇬 Singapore SGV 1 (https://sgt-1.optnl.com/) experienced degraded performance:

HTTP code: 200
Response time: 11803 ms

Resolved: TROJAN 🇸🇬 Singapore SGV 1 performance has improved in 72123d5.


Question about promt templates

Description
Good afternoon.
How and where (you can even have a code example, for example with llama 3.1) can I create a prompt template for LLM?
And how can I see (if possible) what the default prompt template looks like (also with a code example)?
Thank you

We have the PromptTemplateTransformer which should automatically apply whatever template is embedded into the model.
If you want to see what that template looks like I think it's encoded as one of the model metadata items (I can't remember the name, but if you just print all of them it should be obvious which it is).

You may find an example of using PromtTemplateTransformer in LLama2ChatSessions
Default template in case if PromtTemplateTransformer is not used - DefaultHistoryTransform
If you would like to see chat templates for different models without downloading them, here is a good link: https://github.com/chujiezheng/chat_templates/tree/main/chat_templates


Version 2.7 Scans exiting

What happened: Git cloned a repository to run Horusec against; exited with Error: open /Users/sean/FOLDER/.horusec/2d2ff837-bb11-4b49-ae19-169ba9176bf7/SafeMart.Utilities.Pdf/Converter.cs: no such file or directory
What you expected to happen: I did not expect that error
How to reproduce it (as minimally and precisely as possible): After repository clone, download and install latest version of Horusec; Run Horusec generate; vim or nano into "./output.json" and set horusecCliPrintOutputType: "json"; increase time limits; run Horusec start
Anything else we need to know?: Error does not happen with version 2.6.9
Environment:

Horusec version (use horusec version):2.7.0
Operating System: Mac Monterrey 12.1
Network plugin / Tool and version (if this is a network-related / tool bug):
Others:

ty for you feedback, we'll try to fix it as soon as possible.
To make more easy to find your problem can you upload your config file, the exactly command you've runned and the full log after command runned?

closing since there was no response, @seanpclayton you can reopen it when you answer my questions


some content thumbnail do not have enough space

https://new-doc-page.vercel.app/start

gave it more space, i will make it dynamic if it is not yet fixed


Request time out while inserting file

Opening issue as I see the 500 request timeout error again. Here is the sample code where I am doing file insert but am getting this response. Let me know if I am doing anything wrong
Line: 152, Column: 1IBMWatsonServiceExceptions.InternalServerErrorException: { "code" : 500, "error" : "Request timed out, please try again." }
// finding PDF file uploaded to Salesforce with the name "sample_pdf"
String fileTitle = 'SalesSummit-ebrochure';
SObject[] queryResult = Database.query('Select VersionNumber, VersionData, Title, Origin, FileType, FileExtension, Description, ContentUrl, ContentSize, ContentLocation, ContentDocumentId From ContentVersion WHERE Title = :fileTitle');

IBMDiscoveryV1 discovery = new IBMDiscoveryV1('2017-11-07');

// the VersionData property has the Blob we want to use for our file
Blob fileBlob = (Blob) queryResult[0].get('VersionData');
IBMWatsonFile.FileBuilder fileBuilder = new IBMWatsonFile.FileBuilder();
fileBuilder.body(fileBlob);
IBMWatsonFile file = fileBuilder.build();

// we need to specify the filename and content type - validators have been added to check for this
IBMDiscoveryV1Models.AddDocumentOptions options = new IBMDiscoveryV1Models.AddDocumentOptionsBuilder()
  .environmentId('<<environment id>>')
  .collectionId('<<collectionid>>')
  .file(file)
  .filename('SalesSummit-ebrochure')
  .fileContentType(IBMWatsonHttpMediaType.APPLICATION_PDF)
  .build();

IBMDiscoveryV1Models.DocumentAccepted resp = discovery.addDocument(options);

Are you using the same SDK version you did in the last issue you created or the newest one?

@lpatino10  I am using '2018-02-16' version. It was working for me last time you made changes, yesterday when I retried it started breaking with the same 500 timeout issue

It has been opened for 24 days, can we get some traction here. This is delaying the schedule for our component from getting published to market place

Sorry for the delay @Nabarun. I'll take a look as soon as I get the chance.

Thanks @lpatino10

Closing as this issue should be fixed in the newest release.


Mac OpenGL broken due to OS forcing choice of Core or Legacy

It looks like OpenGL on the mac wants to use both different include files and different SDL parameters to open OpenGL Core (modern) vs. OpenGL Legacy (older). This looks like it may force two different RenderManager approaches on the mac. It may be that we can handle it by passing a flag to the routine that calls SDL, letting it know which interface to expose to the user code.
In the meantime, we have the problem that we can't compile the Core-based shader program in RenderManagerOpenGL.cpp, which looks like it is because we're not using Core in the code.
The first step will be to implement a Core-only path and be able to run the Core-profile example programs on the Mac.  We can disable building of the Legacy-profile applications on the mac for now.
http://stackoverflow.com/questions/20931528/shader-cant-be-compiled
http://stackoverflow.com/questions/11961116/opengl-3-x-context-creation-using-sdl2-on-osx-macbook-air-2012
https://www.reddit.com/r/gamedev/comments/27c93z/os_x_opengl_corecompatibility/

There is a mac-working branch that needs to be tested under Windows and, if it works, will get a pull request to fix this.

Fixed in commit 96a32f2831c63a4204146a0d835c18d91ff09eda


A crash that is bound to happen.

if data source size is zero. It collapsed.
library version :1.0.0
_

I guess no one is really maintaining this library. The FlexboxLayoutManager doesn't work at all, it keeps crashing!


Add Debugger

Look at truffle-debugger or remix-debugger.
We want to provider a tx debugger

We decided to use truffle debugger while we produce our own


Background applied to elements rather than wrapper

The background colour is inherited onto each element rather than applying to the wrapping container. This means when margins are used the background is lost.

One potential fix is to use mso-margin-alt:0 to remove the margins and apply mso-border-alt: to fake them.
The issue here is that the faux-margin would have to be the same size on all sides. Also we'd need to be aware of teh desired background colour.

Potentially the hack I've mentioned here might help a little. But ideally we need a way for it to resize the height dependent on the content


feat(notifications): sync with @carbon/react v11

Related Ticket(s)
[notifications]: sync with @carbon/react version #10120
Description
Sync with @carbon/react v11 - this includes new actionable variant
@carbon/react version here: https://react.carbondesignsystem.com/?path=/story/components-notifications-actionable--playground

Looks good @annawen1 !
Icons
This looks like it might be a bug in React as well, but I believe the warning icons need a black fill in order to pass color contrast. I see it working as expected in the toast / warning / low contrast but not some of the other variants (inline and actionable) or warning alts

Bug in React
WC inline variant does have the focus around the close button when you tab in, but React doesn't!

Storybook (small things that don't fully matter)

React uses alphabetical order for all their controls and the order of "kind". Do you think we should too?

Playground stories
For some reason the Playground stories have a period after "Subtitle goes here" and also add "Time stamp" in the toast notification, but those both don't appear in the Default stories (which match React)

Looks good @annawen1 !
Icons
This looks like it might be a bug in React as well, but I believe the warning icons need a black fill in order to pass color contrast. I see it working as expected in the toast / warning / low contrast but not some of the other variants (inline and actionable) or warning alts

Bug in React
WC inline variant does have the focus around the close button when you tab in, but React doesn't!

Storybook (small things that don't fully matter)

React uses alphabetical order for all their controls and the order of "kind". Do you think we should too?

Playground stories
For some reason the Playground stories have a period after "Subtitle goes here" and also add "Time stamp" in the toast notification, but those both don't appear in the Default stories (which match React)

@oliviaflory  I opened an a11y bug with the Carbon core team for the inline notification here: https://github.com/carbon-design-system/carbon/issues/13616 with both of the mentioned issues (close button not focusing & the warning icon fill)
I added fixes for the warning icon fill - we can remove them once the fix is updated in the Carbon core styles and made the storybook adjustments


jQuery.animate({property: 'toggle'})

This is a feature request for the implementation of a new command called toggle like it exists in jQuery.animate().
The purpose of this feature is to be able to achieve effects like this easily: http://jsbin.com/wenajojifa/2/edit?html,css,js,output
This command achieves 2 things:

Animates the value of the selected property between 0 and it's original value, even if it is auto.
Sets the value of display automatically in order to show/hide the element as appropriate, like [Fade & Slide] (http://julian.com/research/velocity/#fade)

In addition to that, if display is originally set to none, it automatically animates from 0 to the value set in CSS, even if it's auto.
This is very useful to introduce elements in/out of the interface with minimal code like [Fade & Slide] (http://julian.com/research/velocity/#fade), while still allowing full flexibility of effects. It doesn't seem to be possibly otherwise to achieve the effect demonstrated in the fiddle easily with Velocity while having height set to auto.

This feature relies on velocity to getComputedStyle on every invocation. This forces a layout thrash and isn't really desirable.
The sugar the API affords isn't worth papering over the performance losses, IMO.

@paulirish, by "every invocation" do you mean every tick? Or every invocation of velocity?

Every invocation of toggle()

It could be implemented with reverse, and something special only on the 1st invocation, depending on the display value. http://jsbin.com/jikutusuya/1/edit?css,js,output

IIRC there are many cases where the initial invocation thrashes layout, e.g. when not force-feeding, isn't it?

Fair feature suggestion. I appreciate you taking the time to write up the breakdown for us, @aristidesfl.
Half of the feature is already implemented via Velocity's UI pack. Specifically, designating "In" and "Out" prefixes on effects name (see docs if you are unfamiliar). The other half has legitimate utility, but I don't plan to implement it at this time. If I do in the near future, I will incorporate it into a larger overhaul of Velocity's animation workflow improvements.
Thanks again :)


hot reloading too fast

I'm using webstorm 2017.2.1 and when I change focus from the editor to my browser i guess my .idea/workspace.xml gets changed causing node to stop because the port is in use (i'm using port 30100). Is there a way I can exclude certain directories from being watched by the hot reloader?
This is the console after i save index.js and change window focus over to my browser:
File changed: index.js - Restarting server...

File changed: .idea/workspace.xml - Restarting server...

File changed: .idea/workspace.xml - Restarting server...
events.js:182
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE :::30100
    at Object.exports._errnoException (util.js:1026:11)
    at exports._exceptionWithHostPort (util.js:1049:20)
    at Server.setupListenHandle [as _listen2] (net.js:1305:14)
    at listenInCluster (net.js:1353:12)
    at doListen (net.js:1479:7)
    at _combinedTickCallback (internal/process/next_tick.js:105:11)
    at process._tickCallback (internal/process/next_tick.js:161:9)

Well I failed to search the closed issues before i open this and found a similar report.
This worked for me
micro-dev -p 30100 --ignore=**/.idea/**


Transporter-2

Mission, per spaceflight, is NET June (not July). On board there will be 36 payloads managed by Spaceflight Inc. See here for more information about payloads

Booster will be B1060 and this will be its eighth flight. It will be an RTLS to LZ-1. SpaceX static fired F9 at 2021-06-22 19:50:00Z.
On board there will be at least 29 payloads from ExoLaunch (nearly 100 kg) and 6 from D-Orbit (mass unknown).

ROS Briarwood just departed.

Both fairings succesfully recovered


Minecraft <-> IRC colour conversion

Colors in irc are sent by putting ^CCOLOR into the text (http://www.mirc.com/colors.html) In Minecraft, they are sent by adding §COLOR into a text.
When sending a message to irc or to minecraft, a color may be lost and displayed in a weird way on the receiving side.
It would be cool if colors would be kept from irc to Minecraft and the other way around.

There seem to be some decent docs around formatting convention in general here: http://modern.ircdocs.horse/formatting.html
Needs:

Configuration options in general and per service
Making service APIs take text components instead of strings (maybe deprecate the string function so plugins don't break, and add a TextComponent version)


SQFormDatePicker selecting same month/day in the future/past years is not acceptable

When attempting to click the same date for a future year the calendar doesn't see anything as changed and doesn't want to close.
Steps to reproduce
Open the datepicker
Change the year to a future year
Select the same month/day for the future year
Loom in consumption: https://www.loom.com/share/7d37ad4b545347a4b6eba86475c12b61
Loom in Storybook: https://www.loom.com/share/d7452011fb52465aa6c4db73a4a98c39

@Chris-Boyle Can you try passing this prop to SQFormDatePicker?
I'm curious if this works or not. I wouldn't consider this a solution to close this issue, but curious if this prop effects this.
muiFieldProps={{
  allowSameDateSelection={true}
}}

@Chris-Boyle Can you try passing this prop to SQFormDatePicker?
I'm curious if this works or not. I wouldn't consider this a solution to close this issue, but curious if this prop effects this.
muiFieldProps={{
  allowSameDateSelection={true}
}}

@Chris-Boyle Can you try passing this prop to SQFormDatePicker?
I'm curious if this works or not. I wouldn't consider this a solution to close this issue, but curious if this prop effects this.
muiFieldProps={{
  allowSameDateSelection: true
}}

Edit: Correcting my pseudo code syntax

Just tested that in storybook and it looks like that will work!  Thanks.

@Chris-Boyle Can you try passing this prop to SQFormDatePicker?
I'm curious if this works or not. I wouldn't consider this a solution to close this issue, but curious if this prop effects this.
muiFieldProps={{
  allowSameDateSelection: true
}}

Edit: Correcting my pseudo code syntax

Just tested that in storybook and it looks like that will work!  Thanks.

@SeanGroff  Should that be default to true?  It feels like that should always be true.  I can't see a scenario where we would not want to allow that.

@SeanGroff  Should that be default to true?  It feels like that should always be true.  I can't see a scenario where we would not want to allow that.

@SeanGroff Should that be default to true? It feels like that should always be true. I can't see a scenario where we would not want to allow that.

@Chris-Boyle  Nah, this should default to false.
To explain, by default if you open the date picker calendar and select Jan 18, 2021. The Calendar closes. If you open the date picker calendar again it won't let you select Jan 18, 2021, again.
 Where the library doesn't consider the same day but a different year a different day.
The next step we'll have to take is to see if this is a known issue with the library itself. https://next.material-ui-pickers.dev/api/DatePicker and sift through the GitHub issues.

@SeanGroff Should that be default to true? It feels like that should always be true. I can't see a scenario where we would not want to allow that.

@Chris-Boyle  Nah, this should default to false.
To explain, by default if you open the date picker calendar and select Jan 18, 2021. The Calendar closes. If you open the date picker calendar again it won't let you select Jan 18, 2021, again.
 Where the library doesn't consider the same day but a different year a different day.
The next step we'll have to take is to see if this is a known issue with the library itself. https://next.material-ui-pickers.dev/api/DatePicker and sift through the GitHub issues.

Issue is over a year old. We can re-open a new one if this is found to still be an issue.


Name threads used by LocalCacheManager

What changes are proposed in this pull request?
Using name threads is generally a good practice and helps system administrators understand what's going on with the system. Without explicit naming, the JDK-provided pools jave threads named pool-N-thread-T.

CLA signed & sent.

the build did not run, probably because it's my first time here.

thanks @jja725 for your review and allowing the build to run too

alluxio-bot, merge this please

@jja725 thank you for quick review & merge!

@jja725 do you have plans for Alluxio 313 release?


Cannot render list of strings Argument.

Hi! Nice library. I'm willing to use it in my project. But I have an issue:
input SomeInput {
    someListArgument: [String!]!
}

Given list of strings, I'd like to generate string someListArgument: ["value"].
The following raises pydantic validation error:
graphql_query.Argument(name='someArgument', value=['value'])

E   pydantic.error_wrappers.ValidationError: 5 validation errors for Argument
E   value
E     str type expected (type=type_error.str)
E   value
E     value is not a valid dict (type=type_error.dict)
E   value -> 0
E     value is not a valid dict (type=type_error.dict)
E   value -> 0
E     value is not a valid list (type=type_error.list)
E   value
E     value is not a valid dict (type=type_error.dict)

And this  generates object instead of array:
graphql_query.Argument(name='someArgument', value=[
    graphql_query.Argument(name='someArgumentItem', value=v)
    for v in ['value']
])

There doesn't seem to be a nice way to generate an array argument. (I have to explicitly render it as string right now)
I havent tried variables. But even if they work, I'd like to keep my query explicit.
Thanks in advance!

hello, thanks for your question!
Yes, it is problem now. We can not use an argument value with list of strings.
There will be a new version soon that includes a fix for this issue.


golang format tools

Produced via:
gofmt -s -w $(find -path './vendor' -prune -o -type f -name '*.go' -print))
goimports -w $(find -name '*.go' | grep -v vendor)
/assign ImJasonH
/cc ImJasonH

/hold
The robot needs to ignore third_party. If I close this it'll just open another one.

also, why is it back ? :stuck_out_tongue_closed_eyes:

Huh, I thought I changed this.  Maybe I didn't redeploy?
It's back because @ImJasonH asked for it!

Running it again, at a minimum I updated the PR description to match my update 🤔


Revert pr #5020

Which issue does this PR close?

Closes #5788.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Hi @xiaoyong-z and @alamb, could you help have a check?

For anyone else following along the original PR that added this change was https://github.com/apache/arrow-datafusion/pull/5020
@yahoNanJing  -- looking at the screen shot you provided,

It looks to me like the parquet file in question is being read using 2 streams (aka the file was opened by two different tasks which are reading it in parallel)
Thus while the wall clock time takes only 4 seconds it may be possible that the total cpu time is actually 7 seconds
You could potentially disable the repartition_file_scans option
https://docs.rs/datafusion/latest/datafusion/config/struct.OptimizerOptions.html#structfield.repartition_file_scans
And see if the metrics were more like what you expected
Perhaps @thinkharderdev or @tustvold  have additional thoughts they could share

Thanks @alamb. My previous comment may be misleading. The main controversial is the metrics of time_elapsed_scanning_total contains the times cost by its parent operators, which may be misleading. By the above metrics, developers may think the bottleneck is the parquet scanning. However, it should be the aggregation whose time is wrongly counted in the metrics time_elapsed_scanning_total

Thanks @alamb. My previous comment may be misleading. The main controversial is the metrics of time_elapsed_scanning_total contains the times cost by its parent operators, which may be misleading. By the above metrics, developers may think the bottleneck is the parquet scanning. However, it should be the aggregation whose time is wrongly counted in the metrics time_elapsed_scanning_total

I think the parent/child relationships of where time is being spent will be hard to capture in scalar metrics (aka any sort of "elapsed time" metrics). I think to get that level of detail, DataFusion would likely need some sort of tracing / span abstraction that can show the heirarchy


Activity selection

Have you read the Contributing Guidelines on Pull Requests?
yes
Description
Added solution of activity selection problem
with the use of Dynamic Array of pairs in c++
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#826

kindly review and merge this issue as part of hakin codes

KINDLY CHECK IT @HarshCasper

@Kajol-Kumari due to so many days there were many pr merging and for resolving conflicts i have to make chages in readme and i still see there are two competitive coding links in readme i don't know who has done that i have made changes from my side and also idented the code you can re review

Thankyou @Kajol-Kumari

i have updated readme same as it is now and added activity selection link and resolved all conflicts @Kajol-Kumari @iamrajiv  please check it and merge it thank you

@siddharth25pandey @Abhijit2505  kindly merge it

@all-contributors please add @NEEHITGOYAL for his code.


Limit DiffEqBase >=6.21 to >=1.3

This change isn't effective. It's also necessary to modify
["4.22-6.23"]
julia = "1"

Indeed, see #11397 posted the same minute as you commented :stuck_out_tongue:

Convergent analysis.

Please don't merge changes like this without testing them.

It was tested and it worked. It was just reversed by Registrator....

What Registrator does with conflicting ranges depends on Dict iteration order. I can't really follow the Pkg code but I wouldn't be surprised if the same holds there.


youtube_dl  attribute version does not exist

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running youtube-dl version 2<IP_ADDRESS>
[x] I've searched the bugtracker for similar feature requests including closed ones

Description

Hi, by PEP 396 youtube_dl.__version__ should exist and report the current version of the module.
When trying this feature I get the following error:
In [1]: import youtube_dl
In [2]: youtube_dl.__version__                                                                                ---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)                                                                                <ipython-input-2-a225f3ac3329> in <module>()                                                                                ----> 1 youtube_dl.__version__
AttributeError: 'module' object has no attribute '__version__'

Same here. My exact error.


HDDS-8370. Intermittent failure in TestOzoneFileSystem#testListStatusWithIntermediateDir

What changes were proposed in this pull request?
testListStatusWithIntermediateDir intermittently fails because it finds two entries in the root directory instead of one:
AssertionError: expected:<1> but was:<2>
  ...
  at org.apache.hadoop.fs.ozone.TestOzoneFileSystem.testListStatusWithIntermediateDir(TestOzoneFileSystem.java:645)

Including the file status list in the assertion message reveals that the unexpected extra entry is .Trash:
AssertionFailedError: Found [
  FileStatus{path=o3fs://bucket40939.volume15180/.Trash; isDirectory=true; modification_time=1680592792546; access_time=1680592792546; owner=runner; group=runner; permission=rwxrwxrwx; isSymlink=false; hasAcl=false; isEncrypted=false; isErasureCoded=false},
  FileStatus{path=o3fs://bucket40939.volume15180/object-dir; isDirectory=true; modification_time=1680592792548; access_time=1680592792548; owner=runner; group=runner; permission=rwxrwxrwx; isSymlink=false; hasAcl=false; isEncrypted=false; isErasureCoded=false}
] with layout:LEGACY, fsPaths:false ==> expected: <1> but was: <2>

Since trash is handled by background thread (cleaning up deleted items from other tests in the same class), it may or may not be present at the time of the test.  This PR changes the test to ignore it.
https://issues.apache.org/jira/browse/HDDS-8370
How was this patch tested?
No failure of this test in 100/100 runs (but two other tests are still flaky):
https://github.com/adoroszlai/hadoop-ozone/actions/runs/4669634394
Regular CI:
https://github.com/adoroszlai/hadoop-ozone/actions/runs/4681320457

Thanks @ayushtkn for the review.


[HUDI-7228] Fix eager closure of log reader input streams with log record reader (0.x branch)

Change Logs
Fix eager closure of log reader input streams with log record reader. While reading log blocks, we do make sure to close the actual content and release it for GC purposes. But the inputstream instantiated with log record reader is not closed. Fixing the same in this patch.
Impact
Fix eager closure of log reader input streams with log record reader. Memory usage when reading 100+ log files will be reduced to minimal.
Memory snapshot before this patch:
25 log files/25 log blocks were present in the file slice of interest.

Notice 25 instances of input streams open.
Memory snapshot after this patch.
Dump taken just after reading all log block headers, but before starting to deser any block for records.

Risk level (write none, low medium or high below)
If medium or high, explain what verification was done to mitigate the risks.
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

I think this also failed the master. Why just the 0.X branch?

@linliu-code : yes, we do have a fix for master here here https://github.com/apache/hudi/pull/10340


performAction to use Action-provided values

Would it be permissible to do this with Helper.performAction:
request.get(someUrl).then(request.Helper.performAction(someActionName)) // no data!
where the data comes from the Action fields, if no data was passed in?
Perhaps even an extends, where the data overwrites any Action-provided default values.

So I guess it already does this, basically. But the data is optional now in PR #2. Take a look and let me know what you think.

Thank you!  Sorry for the late response on this one.  I have merged PR #2.


Marketplace Extensions with Docker Environments

When attempting to install extensions from the marketplace in a Docker container running the US release of 17.4, the following error occurs:
Installation of extensions from the marketplace is not supported for your system configuration. For more information, see https://docs.microsoft.com/en-us/dynamics365/business-central/ui-extensions
There doesn't appear to be any information in the related MS Document explaining why this operation is forbidden.
Please update this article with Marketplace extension limitations or instructions as they relate to Docker Containers.

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

ID: 66f6ce33-2528-c87d-d097-257e598ec407
Version Independent ID: d606ddd8-17c5-0864-cc19-1489b3dbf901
Content: Installing Extensions to Customize Business Central - Business Central
Content Source: business-central/ui-extensions.md
Service: dynamics365-business-central
GitHub Login: @edupont04
Microsoft Alias: edupont

Hello,
Thank you for your feedback! Apps in the marketplace apply to Business Central online only and cannot be installed in Business Central on-premises deployments, including Docker containers. This will be clarified in the next doc update.

Thanks for the information. Is there a reason for this limitation?
I encountered this problem while attempting to enhance an App Source offering.
To make the marketplace more robust, I would imagine that MS would want developers to code unimpeded in Docker with the appropriate dependencies.  This is specially important when SaaS has Unit Testing does not grant developers the same test tool libraries as are available in Docker via the BcContainerHelper.
Sincerely,
Kieran Sweeney

From: Eva Dupont<EMAIL_ADDRESS>Sent: Tuesday, March 2, 2021 9:13:16 AM
To: MicrosoftDocs/dynamics365smb-docs<EMAIL_ADDRESS>Cc: Kieran Sweeney<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/dynamics365smb-docs] Marketplace Extensions with Docker Environments (#507)
Hello,
Thank you for your feedback! Apps in the marketplace apply to Business Central online only and cannot be installed in Business Central on-premises deployments, including Docker containers. This will be clarified in the next doc update.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/dynamics365smb-docs/issues/507#issuecomment-788937511, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASKFG4QKNGVR6CIYKWFKEJTTBTW7ZANCNFSM4YKQP6YA.

Hi Kieran,
Thanks for the follow-up! I can't answer questions about product strategy here, but I've been told that there are practical reasons for not allowing AppSource apps on-premises. I suggest that you join our community in Yammer if you haven't already, and you might want to submit a feature request here: https://aka.ms/businesscentralfeedback

Someone else has already posted a feature request that you can vote up here: https://experience.dynamics.com/ideas/idea/?ideaid=42616aad-587b-eb11-8fed-0003ff4582f9

Now live.


Doesn't seem to be the acr claim anymore that holds the policy used to get a token,.

At least with a token I got from b2c, the policy name is encoded in the tfp claim rather than the acr claim

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

ID: b05bb7df-98fc-1342-640d-02063f6d8a11
Version Independent ID: f6ac357f-f846-83c5-f19f-91e1bc0954e3
Content: Overview of tokens - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/tokens-overview.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@ModernRonin  Thanks for reaching out. We will investigate and update the thread accordingly.

@ModernRonin , this is expected. As already documented here,

Name
Claim
Example value
Description

Authentication context class reference
acr
Not applicable
Used only with older policies.

Trust framework policy
tfp
b2c_1_signupsignin1
The name of the policy that was used to acquire the ID token.

We will now proceed to close this issue, but if there are any further questions, please tag me in your reply and will be happy to continue the conversation.
#please-close

Shouldn't the docs be updated to reflect this default behavior?

@SayakMukhopadhyay , the docs says:

Claim representing policy ID - This property identifies the claim type into which the policy name used in the token request is populated. The default value is tfp. The value of acr is only provided for backward-compatibility.

Apart from that it also has a table as I pointed out in my reply above.
If you are suggesting any other change required for better clarity, please feel free to elaborate a bit. I would be happy to make the suggested change.

Yeah, I think that the following paragraph should be updated

To determine which policy was used to sign a token (and where to go to request the metadata), you have two options. First, the policy name is included in the acr claim in the token. You can parse claims out of the body of the JWT by base-64 decoding the body and deserializing the JSON string that results. The acr claim is the name of the policy that was used to issue the token. The other option is to encode the policy in the value of the state parameter when you issue the request, and then decode it to determine which policy was used. Either method is valid.

in https://docs.microsoft.com/en-us/azure/active-directory-b2c/tokens-overview#validate-signature
The above paragraph gives a wrong indication that the acr claim is the valid claim for this info. I suggest changing it to include tfp (since its the default) something like

To determine which policy was used to sign a token (and where to go to request the metadata), you have two options. First, the policy name is included in the acr or tfp, as configured, claim in the token. You can parse claims out of the body of the JWT by base-64 decoding the body and deserializing the JSON string that results. The acr or tfp claim is the name of the policy that was used to issue the token. The other option is to encode the policy in the value of the state parameter when you issue the request, and then decode it to determine which policy was used. Either method is valid.

(emphasis mine)

thanks, @SayakMukhopadhyay  - I knew there had been something that had misled me originally, but then when I saw the first reaction here, I thought "strange that I didn't see that". Your comment just made it clear :-)

Thanks @SayakMukhopadhyay for pointing that out. Will publish a change shortly.

Update has been published.


Styled strings merging.

In the merging process, StyleArray of BaseStringPool is populated from the base package only.
The other modules should participate.

It looks just perfect!
Let's close this.


download models from models dir

Identified via: https://github.com/containers/ai-lab-recipes/actions/runs/8885812868/job/24397987802?pr=388#step:7:1
Models should be downloaded from models dir.

That is the problem when code has changed while the CI filtered some workflows which could have shown that the code changes broke CI


Enable usage of PANYNJ API for trip times

Summary
This change enables the optional usage of the PANYNJ API for retrieving realtime arrival times. This is a simple JSON endpoint that is used for the PANYNJ website's schedule: https://www.panynj.gov/path/en/index.html
Motivation
Recently, the path-data API used to power this project has been having frequent data issues (more info can be found here: https://github.com/mrazza/path-data/issues/21).  Though I am hopeful this will be resolved, as it is a more feature-complete API then what is used here, having an alternative could be useful if these issues are long term.
Design
A new client, PaNyNjClient, is added which implements the SourceClient interface. This allows it to be a drop-in replacement for the existing HTTP/gRPC clients. Below is how each method is implemented:
GetStationToStopId
A static map of station IDs to GTFS stop IDs is defined in gtfsmapping.go and returned. No attempt to match specific platforms is made here, and instead the the stops of the format 26xxx are always used. More granular platform mapping could potentially be done, but there is no GTFS information exposed in the API to assist with this.
GetRouteToRouteId
A static map of route IDs to GTFS route IDs is defined in gtfsmapping.go and returned. These should not frequently, if ever, change, so having them checked in should be OK. There is no GTFS routing information exposed by the API.
GetTrainsAtStation
This API returns all station data with every call. Because of this, there is no need to keep calling it for every station in the same update loop. To avoid an interface change, the data from the API is cached for 10 seconds and a R/W lock is used to manage reads and updates to the cache. Further optimizations are possible here, but in testing this is already quite fast.
Further mapping between the data returned from the API and the source API types (route, direction, etc.) are also done here using static maps.
Opting into the new API
A new flag use_panynj_api is added, which will use this new API. A minimum update period of 15 seconds is enforced, which is how frequently the PANYNJ website refreshes its data.
Testing

Tests using a mocked HTTP client are defined in panynj_test.go. This reads in a mock JSON file that represents data returned by the API and validates that each station is correctly transformed to the source API types.
Some manual testing with Transiter

This PR is awesome! The idea of using the PANYNJ’s website’s json data is really good.
Ever since I started this project I’ve been conscious that it’s fragile by design: it relies on a tricky undocumented PATH API, Matt Razza’s personal project being maintained and this personal project being maintained. There’s lots of failure points in the data pipeline. Just relying on this json blob from the port authority website means the data pipeline is a little less likely to break. The single con I see is that because the json api is so simple, it may be easier for the port authority to change it.
So right now my feeling is that we should merge this in behind the flag you added, but in the long run potentially remove support for the Razza API entirely? This would dramatically simplify the code. In the meantime the performance of the new code can be measured using some of the Prometheus metrics I believe.
Of course we are very grateful to Matt Razza for working on his project years ago when this data was not available in an easier format, and for maintaining his project since then.

Two organizational notes:
I am traveling until early January without a computer and will generally be slow to respond. I think I can still do most things on GitHub using the UI but will be unable to deploy new code on path.transitdata.nyc until I get home.
I just added you as a collaborator on the repo so you can merge code yourself directly, as with the Transiter and GTFS repos. :) I made a few comments on this PR but nothing blocking submission really, which you should feel free to do.

LGTM! I’ll let you do the squash merge yourself in case there are final tweaks you want to make, and of course feel free to make changes after. Thanks Sam it’s a great PR!

FYI, the JSON file at https://www.panynj.gov/bin/portauthority/ridepath.json lags behind mrazza's API by ~80 seconds. I blogged about it here.

Thanks for the additional context @kerrickstaley! I don't think the longterm plan is to remove mrazza's API, and @jamespfennell very generously hosts feeds derived from both mrazza's API and the PANYNJ data on https://path.transitdata.nyc/gtfsrt and https://path2.transitdata.nyc/gtfsrt, respectively.


[External Request] - Report Vendor/Item Purchases - variable ResetItemTotal

could ResetItemTotal be made protected var? I need to add more values from value entry based on that variable from onbeforeonaftergetrecord event in an report extension.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 12340.


Ipfs add broken on main

I was doing some checks on #603 to see if a hamt directory would appear on the gateway. It does not. But it seems that directory adding on main is also broken.
I created a directory and tried to add it on main.
iroh on  main [$?] via 🦀 v1.65.0
❯ cargo run -p iroh -- add --recursive --offline testdir_many_files/nohamt1
    Finished dev [unoptimized + debuginfo] target(s) in 0.21s
     Running `target/debug/iroh add --recursive --offline testdir_many_files/nohamt1`
[1/2] Calculating size...
[2/2] Importing content 0 B...
/ipfs/bafybeieopdpzqhyyxeuuipsdxnxlgo5q5ontpwe2j7xdojql3zk6du3jzy

Requesting this from the gateway does not give a directory as expected:

Requesting this via iroh get also gives an error:
iroh on  main [$?] via 🦀 v1.65.0
❯ cargo run -p iroh -- get /ipfs/bafybeieopdpzqhyyxeuuipsdxnxlgo5q5ontpwe2j7xdojql3zk6du3jzy
    Finished dev [unoptimized + debuginfo] target(s) in 0.21s
     Running `target/debug/iroh get /ipfs/bafybeieopdpzqhyyxeuuipsdxnxlgo5q5ontpwe2j7xdojql3zk6du3jzy`
Error: path contains non-relative component

Reverting to 31b1acdcd99352cbbbd973c5c751d60ab44c50de, iroh add produces a different hash, which works.
iroh on  main [$?⇣] via 🦀 v1.65.0 took 31s
❯ cargo run -p iroh -- add --recursive --offline testdir_many_files/nohamt1
    Finished dev [unoptimized + debuginfo] target(s) in 0.21s
     Running `target/debug/iroh add --recursive --offline testdir_many_files/nohamt1`
[1/2] Calculating size...
[2/2] Importing content 0 B...
/ipfs/bafybeidunulhcn73uqvcchm75qiktchxk2ksww27gkrvdzgbtvq6x5fqzm

hm, likely introduced in https://github.com/n0-computer/iroh/pull/585 then


v1: Only return composes of the last 14 days

chm, source is not prepared for some reason. :thinking:

chm, source is not prepared for some reason. thinking

source prepare checks that you include the last spec composer spec now, which makes sense


Pin FAST foundation and element versions until new versions vetted

Pull Request
🤨 Rationale
Unit tests on Firefox seem to be sporadically timing out when we update to the latest exports of fast-foundation (2.50.0) and fast-element (1.14.0). We want to pin our dependency versions to the prior versions until we are confident that the new versions do not introduce real problems for clients.
👩💻 Implementation
Pinned fast-element to 1.12.0 and fast-foundation to 2.49.6 in the three packages that reference them.
🧪 Testing
N/A
✅ Checklist

[x] I have updated the project documentation to reflect my changes or determined no changes are needed.

Forgot to update the lock file. Will push soon.


cargo login should ensure ~/.cargo/config is not world-readable

I have a preexisting ~/.cargo/config, which I created to define paths. When running cargo login, it happily added my token to that file even though it has the mode 0644. cargo login should either change the mode to 0600 and issue a warning telling me it did so, or it should do nothing and print an error telling me I need to fix the mode on the file.

I started trying to look into fixing this, but I'm not sure how to do this in a cross-platform way given what I could find out about the state of implementing file permissions for Windows in the standard library.
The docs for Permissions currently say:

This module only currently provides one bit of information, readonly, which is exposed on all currently supported platforms. Unix-specific functionality, such as mode bits, is available through the os::unix::PermissionsExt trait.

... which sounds like not much is available for windows. Sooo I wasn't sure whether to try and fix this just for unix systems for now, orrrr if there's a crate that implements more granular permissions for windows that cargo would be willing to have as a dependency orrrr something else...?
Some more relevant discussions:

https://github.com/rust-lang/rfcs/issues/939
https://github.com/rust-lang/rfcs/pull/1044

@carols10cents yeah to the best of my knowledge there's not a canonical "easy way" to make a file not-world-readable on Windows like there is on Unix. I think that an implementation of this for now would be fine just calling chmod directly (e.g. just using libc::chmod)

Also, the entire ~/.cargo, in which cargo creates various files and directories, must be (in POSIX terminology) 0700 to avoid symlink race attack.

I think this is handled by https://github.com/rust-lang/cargo/pull/3978. Now we store credentials in a separate file with correct permissions.
If you think that this issue is not fixed, feel free to reopen!


No small caps with OTF font

Description
Small caps are not selected even if the OTF font provides such a style.
Check/indicate

[ ] Relevant for XeTeX
[X] Relevant for LuaTeX
[X] Issue tracker has been searched for similar issues?
[ ] Links to <tex.stackexchange.com> discussion if appropriate

Minimal example demonstrating the issue
\documentclass[paper=a4]{scrreprt}

\usepackage{fontspec}
\setmainfont{Minion Pro}

\usepackage{polyglossia}
\setmainlanguage{german}

\begin{document}
\textsc{No small caps}
\end{document}

Further details
The Minion Pro OTF font includes small caps:
$ otfinfo -f ~/.fonts/Minion\ Pro/MinionPro-Regular.otf
aalt    Access All Alternates
c2sc    Small Capitals From Capitals
case    Case-Sensitive Forms
cpsp    Capital Spacing
dlig    Discretionary Ligatures
dnom    Denominators
fina    Terminal Forms
frac    Fractions
hist    Historical Forms
kern    Kerning
liga    Standard Ligatures
lnum    Lining Figures
numr    Numerators
onum    Oldstyle Figures
ordn    Ordinals
ornm    Ornaments
pnum    Proportional Figures
salt    Stylistic Alternates
sinf    Scientific Inferiors
size    Optical Size
smcp    Small Capitals
sups    Superscript
swsh    Swash
tnum    Tabular Figures
xx01    <unknown feature>
zero    Slashed Zero

In contrast, fontspec does not recognise the style:
LaTeX Font Warning: Font shape `TU/MinionPro(1)/m/sc' undefined
(Font)              using `TU/MinionPro(1)/m/n' instead on input line 15.

[1{/usr/local/texlive/2021/texmf-var/fonts/map/pdftex/updmap/pdftex.map}]
(./test.aux)

LaTeX Font Warning: Some font shapes were not available, defaults substituted.

Thanks for logging the issue. This must be due to something changing outside of fontspec — it’s not possible that this has been broken for ever…

This is not a bug in fontspec, it's an odd font configuration. Minion Pro has the small caps feature for the default language of the Latin script, but it has a special feature table for german which overwrites the default in german documents. The german table only supports the discretionary ligatures feature, not small caps.

Thank you very much for your explanation.


Trigger event in Mixpanel whenever a user connects to a dapp via WalletConnect

Overview
As a dao stakeholder, i want to know how frequently users are connecting to dapps via WalletConnect as well as which dapps they're connecting to on which chains.
References and additional details
n/a
Acceptance Criteria
AC:

When a user connects to a dapp on mixpanel, an event "Dapp Connected" is triggered
Properties of the event include Dapp: [ie. Uniswap] and Network: [ie. Arbitrum]

Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

this is already implemented https://github.com/shapeshift/web/blob/develop/src/plugins/walletConnectToDapps/components/DappRegistryGrid.tsx#L52

we do trigger an event when users click a dapp on the /dapps page, but we don't currently trigger an event when a user actually uses WalletConnect to connect to a dapp. this issue is to give us insight into if/how users are using the WalletConnect integration.


Setup PyPI Token for cdevents-bot

In preparation for releasing the python SDK, and following the release similar to java SDK, we need to setup an account for the cdevents-bot on the PyPI site so we can use github actions to automate the release of the SDK with each release. @afrittoli do you mind following this link to setup an account for the bot and then setup an API token! Similar to the secrets.GH_BOT_TOKEN used in the changelog github action, a new token like secrets.PYPI_BOT_TOKEN could be used. Then I can create the automation and work on creating our first release using github actions. Source of how upload our python package.
Also there is no rush on this, with the CD Con next week, please take all the time you need! I'm still working on a few automation features while my teammate is working on the pydantic enhancements. Hope you have a great start to May!

@elmsdata I provisioned the cdevents-bot account, configured 2FA and added the details to 1password.
I provisioned an API token and stored it in the PYPI_BOT_TOKEN secret, so that it's available to GitHub actions.
As I'm sure you're aware, the link you provided pointed to the test instance of pypi. Once the release process is confirmed to be working fine we shall create an account in the production instance of pypi and update the API token.

Will now work on the github automation to push to the pypi!


ROW_FORMAT not respected in alter statements

Describe the bug
Adding ROW_TYPE as part of alter statement gives error while parsing SQL:
To Reproduce
Steps to reproduce the behavior:
ALTER TABLE basic_test_case ADD COLUMN display_name varchar(512) NOT NULL DEFAULT '' AFTER name, ADD KEY test_case_status (test_case_status), add KEY display_name (display_name), ROW_FORMAT=DYNAMIC;
Encountered unexpected token: "ROW_FORMAT" <S_IDENTIFIER>
at line 16, column 391.
Was expecting one of:
"ADD"
"ALGORITHM"
"ALTER"
"CHANGE"
"COMMENT"
"DROP"
"MODIFY"
"RENAME"
at net.sf.jsqlparser.parser.CCJSqlParser.generateParseException(CCJSqlParser.java:25031)
at net.sf.jsqlparser.parser.CCJSqlParser.jj_consume_token(CCJSqlParser.java:24875)
at net.sf.jsqlparser.parser.CCJSqlParser.AlterExpression(CCJSqlParser.java:15646)
at net.sf.jsqlparser.parser.CCJSqlParser.AlterTable(CCJSqlParser.java:15683)
at net.sf.jsqlparser.parser.CCJSqlParser.SingleStatement(CCJSqlParser.java:159)
at net.sf.jsqlparser.parser.CCJSqlParser.Statements(CCJSqlParser.java:578)
at net.sf.jsqlparser.parser.CCJSqlParserUtil.parseStatements(CCJSqlParserUtil.java:149)
... 21 common frames omitted

Expected behavior
ROW_FORMAT should be recognized in alter, like it is in create table.
System

Database you are using: MySQL
Java Version: 1.8.0_231
JSqlParser version: 3.2

ROW_FORMAT don't has a keyword defination in jsqlparser.jjt, so it is accidently supportted in create table.

Greetings!
The issue is not about the particular ROW_FORMAT but rather about the supported ALTER ... operations. There was a lack of support for altering the table properties.
I have added a new UNSPECIFIC operation which simply will route through all tokens, when no different ALTER ... operation has matched before. This solution is very flexible and your statement will parse, but if your are interested in the content of the UNSPECIFIC operation you will need to process the optionalSpecifier of the AlterExpression by yourself.


⚠ HawkeUno has degraded performance

In 038cfd7, HawkeUno ($UNO) experienced degraded performance:

HTTP code: 403
Response time: 2142 ms

Resolved: HawkeUno performance has improved in e813a81 after 6 minutes.


destroyed menubar

With beta 2.0.0 (81) on macOS 14.5 the Menubar sometimes destroys if the "time left" setting is on

I know about the issue but I have no idea what's the fix :(

Unfortunately, I can't help there either.
 Perhaps it would be possible to make the font smaller and display the current wattage in another line, for example.


Gender-neutral responses

Some messages are gender-specific, like:
I'm sorry; I'm not sure I understand what you're trying to say sir.
Try typing "help" or "request"

Can (Should) we make them gender-neutral?

+1
Don't see added value in having a gendered answer, and do see potential for cases to detract from the bot's value for some users.

+1
It's a good thing to strive for.

What about asking in the beginning for the sex and then he goes off of that. We could have a gender neutral option too
Sent from my Huawei Nexus 6P using FastHub

We can get the gender from the User Profile API. 🎉
Just like we're getting the name here, we can get more info about the user.

Safest bet seems to be using name, if that's available.

The gender is available in the API, names could include names that are applicable to both genders.
Sent from my Samsung SM-G930W8 using FastHub

@pt2pham - I guess I'm suggesting we skip gender. Just refer to people by name.

@leegenes ah I see, had a misunderstanding. That seems to be the best solution I'd agree.
Sent from my Samsung SM-G930W8 using FastHub

I agree with avoiding the concept of gender altogether. Some users may be equally turned off it were only presented as a binary option, and it seems to add little value to include gendered answers.

Is anyone actively working on this? Looks like the general consensus is to avoid using genders, and to refer to the user by name instead. I'd love to try my hand at this. It's my first time working with this kind of project, so I can't make any promises, but I'm willing to give it a shot.

Based on the above discussions, the consensus is to avoid the concept of gender altogether.


Counter was not increased for every flow

Counter was not increased since collection messages always give one flow.

Yey!

Codecov Report

Merging #224 into master will increase coverage by 0.07%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #224      +/-   ##
==========================================
+ Coverage   51.16%   51.23%   +0.07%
==========================================
  Files          33       33
  Lines        3915     3917       +2
==========================================
+ Hits         2003     2007       +4
+ Misses       1715     1713       -2
  Partials      197      197

Impacted Files
Coverage Δ

cmd/remoteenforcer/remotecollector.go
75% <0%> (-10.72%)
:arrow_down:

monitor/linuxmonitor/linuxMonitor.go
72.54% <0%> (+3.92%)
: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 66b0a1e...3cb1f70. Read the comment docs.


Enable using "--etcd-prefix" for reusing external ETCD cluster with multiple tenants/K3S cluster

Hi
do you plan to implement the possibility to reuse an existing external ETCD cluster for multiple tenants/K3S cluster?
This is done by having the embedded Kubernetes use the startup argument "--etcd-prefix ", which is used as prefix within ETCD environment to separate the registry tree.
Thanks
Dejan

I think this should work fine if you use --datastore-endpoint=https://X with --kube-apiserver-arg=etcd-prefix=foo. I haven't tried that myself though - reply here if you run into any problems?

Hi,
yes, this works, got this option completely out of my mind. Thx, Dejan


reqwest::Error { kind: Decode, source: Error("missing field whois", line: 585, column: 9) }

When i typing ip, and pressing enter i getting 400 error and 'reqwest::Error { kind: Decode, source: Error("missing field whois", line: 585, column: 9) }'

Thanks for reporting, definitely a poorly handled error. Can you share the IP that didn't work for you?

Hi,
I am also getting a similar error:

It seems to behave differently depending on the IP I am looking at. For example, <IP_ADDRESS> doesn't respond with anything.
I've tried a number of IP addresses. I've double-checked my API keys and they are correct.
Any help would be really appreciated. This looks amazing.

Appreciate the additional information, I'll get this resolved as soon as I can.

This should be resolved. Please let me know if you are still encountering this issue after updating. Thank you again!

ip is my :) yeah, error is fixed, thank!!

Glad to hear it! Let me know if you are into any other bugs or potential improvements by filing an issue.


feature: the kubectl plugins should be based on proper arg parsing

Feature Description
The kubectl plugins, aside from the syncer-gen one, are currently bash scripts that do some generic parsing of kubectl flags.  This is more complex programming than should be done in bash and, more importantly, is ignorant of what kubectl flags really are.
Proposed Solution
These should be rewritten to rely on a standard library that actually understands kubectl flags, such as https://github.com/kubernetes/cli-runtime .
Alternative Solutions
🤷
Want to contribute?

[ ] I would like to work on this issue.

Additional Context
No response

@MikeSpreitzer can you point me at an example plugin that uses https://github.com/kubernetes/cli-runtime?  I need to learn by example before trying to modify any of our plugins.

I am working on converting the get_log script first.  I hope to have something working shortly.

the scripts I am targeting for conversion, in no particular order, are:
kubectl-kubestellar-deploy
kubectl-kubestellar-ensure-location
kubectl-kubestellar-ensure_wmw
kubectl-kubestellar-get_external_kubeconfig
kubectl-kubestellar-get_internal_kubeconfig
kubectl-kubestellar-get_log
kubectl-kubestellar-prep_for_cluster
kubectl-kubestellar-prep_for_syncer
kubectl-kubestellar-remove-location
kubectl-kubestellar-remove-wmw
I am not sure if there is a need to support:
kubectl-workspace-ensure

For example, the KubeStellar scripts do not know which flags take an argument vs. which do not.

@namasl is covering this starting with #1294

we will pick this up in the new year

With the recent architecture change to use the space framework, PR #1294 has been closed, and a new PR will contain similar Bash script to Go conversions for the latest kubectl-kubestellar* commands.
As this issue's description is generic enough to cover the requisite changes under the new architecture, I figured I'd add this comment here rather than make a new issue.


File missing

Hello,
It's me or a file is missing?
VehicleEditorPage.xaml ?

Let me take a look.
Callum Moffat
App Developer
Windows 10 Enthusiast

Keep track of the costs of your vehicles with Petrolhead!
https://www.microsoft.com/en-us/store/apps/petrolhead/9nblggh1rjqt?cid=phmailsig
On Wed, Mar 23, 2016 at 7:19 AM, QuentinSc<EMAIL_ADDRESS>wrote:

Hello,
It's me or a file is missing?
VehicleEditorPage.xaml ?
[image: image]
https://cloud.githubusercontent.com/assets/16439299/13962822/01c26f0a-f063-11e5-8540-9626e38bb2bd.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/SupernovaApps/Petrolhead/issues/8

Yes, it would appear so. I'll correct this today.
Callum Moffat
App Developer
Windows 10 Enthusiast

Keep track of the costs of your vehicles with Petrolhead!
https://www.microsoft.com/en-us/store/apps/petrolhead/9nblggh1rjqt?cid=phmailsig
On Wed, Mar 23, 2016 at 8:10 AM, Callum Moffat<EMAIL_ADDRESS>wrote:

Let me take a look.
Callum Moffat
App Developer
Windows 10 Enthusiast

Keep track of the costs of your vehicles with Petrolhead!
https://www.microsoft.com/en-us/store/apps/petrolhead/9nblggh1rjqt?cid=phmailsig
On Wed, Mar 23, 2016 at 7:19 AM, QuentinSc<EMAIL_ADDRESS>wrote:

Hello,
It's me or a file is missing?
VehicleEditorPage.xaml ?
[image: image]
https://cloud.githubusercontent.com/assets/16439299/13962822/01c26f0a-f063-11e5-8540-9626e38bb2bd.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/SupernovaApps/Petrolhead/issues/8

I have corrected this now. Thanks for reporting!


Agent reconnect functionality

In this PR I've added the bare minimum amount of changes to enable the agents to gracefully reconnect to rabbitmq in the event of a connection failure.
On the raspberry pi specifically, I ran into a few issues where the network wouldn't really be up by the time the agent wanted to start, which meant that it couldn't access RMQ. With the existing agent code, this meant the agent just outright quit, and required a manual restart.
There are a number of things I want to do to the rabbitmq.go file to reflect some of the improvements that @vCabbage made in other patches, but for now I'm focused on fixing this particular issue. A future PR will expand on these changes and improve the entire file.

@vCabbage Updated the error message with the right verb. Were you finished with your review, my good sir?

Other than the comments I made, lgtm 👍

Awesome, thank you sir! Even though this was a band-aid and not a full refactor, you (as always) made it that much better! Waiting on Travis to give the thumbs up then I'll merge.
I should have more time to work on stuff like this after Interop! :smile: (famous last words)
Thanks again!


Doesn't work or install on windwos

Hi,
follows errors on build
# github.com/golang-ui/nuklear/nk
In file included from src\github.com\golang-ui\nuklear\nk\cgo_helpers.go:11:0:
./nuklear_glfw_gl2.h:16:24: fatal error: GLFW/glfw3.h: No such file or directory
compilation terminated.

C:\goworkspace>go get -x github.com/golang-ui/nuklear/nk
WORK=C:\Users\Marc\AppData\Local\Temp\go-build622038790
mkdir -p $WORK\github.com\golang-ui\nuklear\nk\_obj\
mkdir -p $WORK\github.com\golang-ui\nuklear\
cd c:\goworkspace\src\github.com\golang-ui\nuklear\nk
CGO_LDFLAGS="-g" "-O2" "-lglfw3" "-lopengl32" "-lgdi32" "-lm" "-Wl,--allow-multiple-definition" "C:\\Go\\pkg\\tool\\windows_amd64\\cgo.exe" -objdir "C:\\Users\\Marc\\AppData\\Local\\Temp\\go-build622038790\\github.com\\golang-ui\\nuklear\\nk\\_obj\\" -importpath github.com/golang-ui/nuklear/nk -- -I "C:\\Users\\Marc\\AppData\\Local\\Temp\\go-build622038790\\github.com\\golang-ui\\nuklear\\nk\\_obj\\" -g -O2 -DNK_INCLUDE_FIXED_TYPES -DNK_INCLUDE_STANDARD_IO -DNK_INCLUDE_DEFAULT_ALLOCATOR -DNK_INCLUDE_FONT_BAKING -DNK_INCLUDE_DEFAULT_FONT -Wno-implicit-function-declaration -DNK_INCLUDE_FIXED_TYPES -DNK_INCLUDE_STANDARD_IO -DNK_INCLUDE_DEFAULT_ALLOCATOR -DNK_INCLUDE_FONT_BAKING -DNK_INCLUDE_DEFAULT_FONT -Wno-implicit-function-declaration -DNK_INCLUDE_FIXED_TYPES -DNK_INCLUDE_STANDARD_IO -DNK_INCLUDE_DEFAULT_ALLOCATOR -DNK_INCLUDE_FONT_BAKING -DNK_INCLUDE_DEFAULT_FONT -DNK_INCLUDE_VERTEX_BUFFER_OUTPUT -D_GLFW_WIN32 -DNK_INCLUDE_FIXED_TYPES -DNK_INCLUDE_STANDARD_IO -DNK_INCLUDE_DEFAULT_ALLOCATOR -DNK_INCLUDE_FONT_BAKING -DNK_INCLUDE_DEFAULT_FONT -Wno-implicit-function-declaration -DNK_INCLUDE_FIXED_TYPES -DNK_INCLUDE_STANDARD_IO -DNK_INCLUDE_DEFAULT_ALLOCATOR -DNK_INCLUDE_FONT_BAKING -DNK_INCLUDE_DEFAULT_FONT -Wno-implicit-function-declaration cgo_helpers.go color.go const.go font.go impl_glfw.go nk.go types.go
# github.com/golang-ui/nuklear/nk
In file included from src\github.com\golang-ui\nuklear\nk\cgo_helpers.go:11:0:
./nuklear_glfw_gl2.h:16:24: fatal error: GLFW/glfw3.h: No such file or directory
compilation terminated.

MinGW-W64 is installed
Using built-in specs.

On c:\go\pkg\tool\windows_amd64\cgo.exe: is nothing this cgo.exe
Whats the problem?

@Fruchtgummi
You're using some very old version of the repo, there is no nuklear_glfw_gl2.h and any references to it in the master branch. I implemented drawing backends in pure Go back to early January 2017 ;)
Try to go-get update it.

Indeed!
Yes, now I have the following error! :)
# nuke
c:\go\pkg\tool\windows_amd64\link.exe: running gcc failed: exit status<PHONE_NUMBER>

Fehler: Prozess beendet mit Rückgabewert 2.

@Fruchtgummi Awesome!
By the way, this is very specific error, can I see the source of nuke?
That's not one of my examples, could you first check with https://github.com/golang-ui/nuklear/tree/master/cmd/nk-example so we can be sure
go get github.com/golang-ui/nuklear/cmd/nk-example

Yes, it is your example!
Here a error from LiteIDE and the process of build.
# github.com/golang-ui/nuklear/cmd/nk-example
c:\go\pkg\tool\windows_amd64\link.exe: running gcc failed: exit status<PHONE_NUMBER>

Fehler: Prozess beendet mit Rückgabewert 2.

after "get force rebuilding" and "get update network"
go build github.com/go-gl/glfw/v3.2/glfw: c:\go\pkg\tool\windows_amd64\cgo.exe: exit status 2
go build github.com/go-gl/gl/v3.2-core/gl: c:\go\pkg\tool\windows_amd64\cgo.exe: exit status 2
Fehler: Prozess beendet mit Rückgabewert 1.

after force build
go build runtime/cgo: c:\go\pkg\tool\windows_amd64\cgo.exe: exit status 2
Fehler: Prozess beendet mit Rückgabewert 1.

I need C-Compiler?
I have installed
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=C:/mingw-w64/x86_64-5.3.0-posix-seh-rt_v4-rev0/mingw64/bin/../libexec/gcc/x86_64-w64-mingw32/5.3.0/lto-wrapper.exe
Target: x86_64-w64-mingw32

....

Thread model: posix
gcc version 5.3.0 (x86_64-posix-seh-rev0, Built by MinGW-W64 project)

@Fruchtgummi it seems you have wrong/broken MinGW environment, I'm not sure what's wrong with it, but I never got these issues under Win7/10 both 32/64 bits.
Also you certainly should be able to build
go build github.com/go-gl/glfw/v3.2/glfw
go build github.com/go-gl/gl/v3.2-core/gl

so it seems that the environment is broken, no the nuklear package itself.
Try to use MSYS (see https://github.com/golang-ui/nuklear#desktop-installation)
P.S. also found this https://sourceforge.net/p/mingw-w64/mailman/message/31834115/

It's Running! Thx


New option for minutes step

With the current timepicker the user need to press for every minuted he want to change.
Add an option to allow minuted steps to make the user click less.
options='step: 5'

Update to latest version.
set minuteStep to 5


add gradio app

added a gradio app that uses ai-gradio python package to easily deploy a chat app in a few lines of code

Thank you for your contribution. Gradio is a highly popular component, and its deployment process is well-understood by the community. To maintain the brevity and clarity of our documentation, as is common with other open-source frameworks, we have opted not to include detailed deployment or usage code here. Therefore, we will respectfully decline this merge request. We truly appreciate your effort and understanding!


Minor fixes for BoxTrapper and CPanel MTAs according to detection document

This fixes some minor mistakes for BoxTrapper and CPanel and makes the code more readable.
Check out bullet point no. 2 in the detection document here.

Thanks @walling, Precedence and Preference confused me for the second time !


Add delete group api

resolve https://github.com/skiptests/astraea/issues/497 Add delete group api

@chia7712 有空的話 再幫忙review 一下


Not an issue - is there a way to change the prop values

Hi there,
This is not an issue but was wondering if there was a way to change the prop values after selecting the component?

Hi Thanks for raising the issue ans closing it. Can you share your anwser/analysis on this so thatothers can get benefitted.
Thanks.

Sorry, realized that this plugin doesn't have this option so I'm going to either find a way to do it or check another plugin.

Can you raise a feature request (issue) on the same? Let me find out a possibility for this plugin. Thanks.

@wing5822 I appreciate the ticket. Would you be able to provide some more information? Any use cases (sample codes) will help me to dig deep.

hey @abhinaba-ghosh
I'll use your example
`const MyComponent = ({ someBooleanProp }) => (
My Component {someBooleanProp ? 'show this' : ''}
);
const App = () => (

);
ReactDOM.render(, document.getElementById('root'));`
We get the react component like so
cy.getReact('MyComponent', { someBooleanProp: 'true' })
.getProps('someBooleanProp')
But what I want set the prop value to another value without touching the UI
cy.getReact('MyComponent', { someBooleanProp: 'true' })
.setProps('someBooleanProp', 'false')

@wing5822 Thanks for the details. You will get an update by upcoming Monday.

Hi @wing5822 , after careful analysis, I found out this library can be best suited for E2E testing. Changing prop and state require re-rendering of the React components. So, your tests should have access to source code directly.
The best way to efficiently testing the react component using both the library:

cypress-react-selector
cypress-react-unit-test

You can find an existing example of using these two libraries hand-in-hand.
Although I am still validating enhancing this library to handle all possible use cases. You can track here:  https://github.com/abhinaba-ghosh/cypress-react-selector/issues/23

Its main idea is to model the entire process of user-application interaction as near to real-life as possible. A user cannot manipulate a component's props and state directly, so the cypress-react-selector is also not supposed to do this. Instead, you can emulate a user action that should lead to intended props/state modification and then check how it affects a page state after re-render.
Also, direct modification of React component props in the real environment in the middle of the React lifecycle can cause unexpected side-effects, make a test unstable and lead to hard-to-locate bugs.

That I understand. All good =)


linux file listing methods and linux file copy method

Implemented a few linux specific methods in common_items/LinuxUtils.cpp:

GetAllFilesInFolder
GetAllFilesInFolderRecursive
TryCopyFile
Implemented TryCopyFile with sendfile because this copies the file in kernel space, which is more efficient
That will cause compatability issues for linux kernels pre version 2.2 / glibc 2.1 (released 2004).
If this is a problem, I will reimplement it with POSIX read/write functions.

Added a few helper functions for use in LinuxUtils methods only
Reorganized methods so that helper functions are declared before the others
As per previous comment, couldn't find the credit list, so I'll add to that next time :)

Tested functionality on Debian 8 and Linux Mint 17.2 (Ubuntu 14.04)

Sorry for not providing more detail. The credits list is in Vic2ToHoI4/Data_Files/ReadMe.txt.
I am fully in favor of techniques to speed up file operations. When I've measured, the copy operations are (by far) the most time consuming parts of converting.
Also, if function signatures can be improved, feel free to change them. I can make the Windows code match an improved interface.

It is dang exciting to see it get further and further in conversion:
2017-05-12 17:47:43    [INFO] Converter version 0.2
2017-05-12 17:47:43    [INFO] Built on May 10 2017 at 19:58:26
2017-05-12 17:47:43    [INFO] Current directory is /mnt/d/My Projects/paradoxGameConverters - Vic2ToHoI4/Vic2ToHoI4/Release-Linux
2017-05-12 17:47:43    [INFO] Using input file The_Roman_Empire1926_01_01.v2
2017-05-12 17:47:43    [INFO] Reading configuration file
2017-05-12 17:47:43   [DEBUG]     Victoria 2 install path is /mnt/d/Paradox Interactive/Victoria 2
2017-05-12 17:47:43   [DEBUG]     HoI4 path install path is /mnt/d/SteamLibrary/SteamApps/common/Hearts of Iron IV
2017-05-12 17:47:43   [DEBUG]     HoI4 documents directory is /mnt/c/Users/Stephen/Documents/Paradox Interactive/Hearts of Iron IV
2017-05-12 17:47:43    [INFO] Double-checking Vic2 mods
2017-05-12 17:47:43   [DEBUG]     Expecting a mod with name TheEmpireStrikesBack
2017-05-12 17:47:43    [INFO] Using output name The_Roman_Empire1926_01_01
2017-05-12 17:47:43    [INFO] * Importing V2 save *
2017-05-12 17:47:43    [INFO] Parsing save
2017-05-12 17:47:43   [ERROR] convert8859_15ToUTF16() has been stubbed out in LinuxUtils.cpp.
I'm sure you're seeing similar results on your system.


Line Height in the "Folder" section of the sidebar.

It would be nice to be able customizing the line height or separation between the folders and file names of the sidebar. There are plenty and awesome themes out there, but the "folders view" is still to tight in comparition with the editor which we can tweak using the line height.

Somewhat a duplicate of https://github.com/Microsoft/vscode/issues/519
But let's keep this open for now since it is about line height

Time to make some noize on this PR https://github.com/Microsoft/vscode/pull/63602

#63602 is already closed due to merge conflicts.
I am also waiting for this line height option

There are Millions of developers out there who want to personalize their VSCode experience to meet their unique needs. This is precisely what makes open-source software so powerful - the ability to shape and mold a tool to fit your individual workflow.
However, I understand that it can be disheartening to feel like our voice is not being heard in the VSCode community. Our opinion as a user is incredibly worthless to this community. This community are constantly striving to improve and evolve VSCode based on the feedback you receive from users like us.

So ignoring our opinion without saying any reason is not the best choice.

Strange workflow:

"This feature request received a sufficient number of community upvotes and we moved it to our backlog"
@isidorn isidorn added the [*out-of-scope]

This feature is still pretty much needed. Please consider adding this 🙏


Update ggcr and use the new tarball.WithCompressCaching

This simplifies the logic for creating layers and relies on the new option instead of pre-gzipping (and gunzip being faster).  The net effect of this should be the elimination of an extra gunzip, but it should not materially impact the memory pressure because we were ALREADY caching the gzipped layers in-memory, we were just doing it on the caller side, which provided less benefit!

The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #263      +/-   ##
==========================================
- Coverage   37.06%   36.89%   -0.18%
==========================================
  Files          33       33
  Lines        1465     1461       -4
==========================================
- Hits          543      539       -4
  Misses        833      833
  Partials       89       89

Impacted Files
Coverage Δ

pkg/build/gobuild.go
57.66% <100.00%> (-0.56%)
:arrow_down:

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 06ebf03...c6b2a04. Read the comment docs.

@googlebot I signed it!

@googlebot rescan

@googlebot rescan


Claims api

Refactored claims API to include:

default logging of all API actions via rack middleware
added source field to claims api so that we can tell if the request came from the API or web

my attempt to squash seems to have picked up too many previous commits making this seem bigger than it is.. damn git is annoying sometimes.
I'm not going to have a chance to seriously look at this again to Wednesday so either someone else will have to fix, or just go for the merge


(docs): add nft sections

What does this PR do ?

Resolves #402

Steps to test

Run yarn build
Run yarn serve
Go to localhost:3000
Go to Developers Guidelines > Learn EOSIO > NFTs on EOSIO ecosystem
Go to Developers Guidelines > Learn EOSIO > How to mint?

CheckList

[ ] Follow proper Markdown format
[ ] The content is adequate
[ ] The content is available in both english and spanish
[ ] I Ran a spell check

See LGTM suggestions


Removing MPD and audio engine

There may be a time for this in the future, but right now it's just making the project messier and thus harder to maintain

/update

/update

/update

/update


[typo] (Broken link)  Fix add-in commands link

Fixes #3075

Docs Build status updates of commit d8c8e9d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/add-in-manifests.md
:white_check_mark:Succeeded
View

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 the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Write a Python program to check alphabet

Description
Write a Python program to check alphabet
Check that the given character is an alphabet or not (a-z, A-Z).
Input  : a
Output : Alphabet

How to contribute

Save the solution in check-alphabet.py file
Add check-alphabet.py file in check-alphabet folder

can you please assign this issue to me, i want to solve this.

!assign


livelinessProbe and readinessProbe

Hi, is it possible to add those 2 probe so I can deploy on my cluster? It's the required configs for istio to health check

Hello,
you can add it via helm values. Here is the chart part related to all plugins - link. There you can notice that each .Values.plugin  is place into $container variable, and then lower in the code we use $container.livenessProbe and $container.readinessProbe to define the probes.
This means that you can set the probes with these helm values:
plugin:
  generic-crd-plugin:
    livenessProbe:
      ...
    readinessProbe:
      ...

I guess it would be nice to have these in the example plugin.yaml file that is in this repo. We are open to PRs ;)
Also, newest vcluster version - v0.14.0 comes with a feature with which we intend to succeed this plugin, so check it out as well - Docs.

hi, I mean which value should I set for those probes? It's not clear about the port and endpoints that I should use

You can use the same readiness probe as is used in the "syncer" container of vcluster, as that one won't start until plugins are connected.
I am not sure if there is a good way to implement a proper livenessProbe right now :thinking:
Temporary hacky workaround could be something like:
livenessProbe:
  exec:
    command:
    - "true"
  initialDelaySeconds:<PHONE_NUMBER>
  periodSeconds:<PHONE_NUMBER>


fix: show draft banner on all draft content (resolves #1299)

[x] Resolves #1299

Bonus:

[x] Resolves #764
[x] Resolves #920
[x] Resolves #937

Prerequisites
If this PR changes PHP code or dependencies:

[x] I've run composer format and fixed any code formatting issues.
[x] I've run composer analyze and addressed any static analysis issues.
[x] I've run php artisan test and ensured that all tests pass.
[x] I've run composer localize to update localization source files and committed any changes.

If this PR changes CSS or JavaScript code or dependencies:

[x] I've run npm run lint and fixed any linting issues.
[x] I've run npm run build and ensured that CSS and JavaScript assets can be compiled.

@greatislander it doesn't appear that the FRO's page has been updated to address #764 and although it's a single step, it might be more consistent to also add in the horizontal rules as mentioned in #920.


DisableAcceleratorUsageMetrics has been removed

https://kubernetes.io/docs/concepts/cluster-administration/system-metrics/#disable-accelerator-metrics
The DisableAcceleratorUsageMetrics feature gate disables metrics collected by the kubelet, with a timeline for enabling this feature by default.

According to the documentation the "DisableAcceleratorUsageMetrics" feature gate has already been removed, so the section "Disable accelerator metrics" needs to be updated.

/assign @pegasas


wine-discord-ipc-bridge on steam games

in wine discord ipc bridge repository (https://github.com/0e4ef622/wine-discord-ipc-bridge/) there is a shell script that allows it to work on proton. however I haven't had any luck using it on nixos. is there any way to get the bridge working on steam games?

got it to work properly, all you need to do is set the bridge path to where winediscordipcbridge.exe is. and by default it checks in the same directory. I think you can just paste the script in the same directory and it will work.

In that case we could include the script in wine-discord-ipc-bridge/bin.


Add missing call to base class in ReactiveUserControl.OnDataContextChanged

What does the pull request do?
Adds the missing call to base.OnDataContextChanged
What is the current behavior?
the control does not call base.OnDataContextChanged, so DataContextChanged - event is not raised
What is the updated/expected behavior with this PR?
the control does call base.OnDataContextChanged
How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues
Fixes #7636

You can test this PR using the following package version. 0.10.999-cibuild0018846-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


⚠ Spryr has degraded performance

In 69520e4, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3700 ms

Resolved: Spryr performance has improved in e948ded after 7 minutes.


No module named 'cyber.python'

hello, test the cyber_example_listener and cyber_example_listener is ok, and cyber_monitor is ok,
but get error by using the cyber_channel list :
➜  build git:(v8.0.0) source setup.zsh
➜  build git:(v8.0.0) cyber_channel list
Traceback (most recent call last):
  File "/home/mini/researches/cyber_rt_8/CyberRT/build/bin/cyber_channel", line 23, in <module>
    from cyber.python.cyber_py3 import cyber_time
ModuleNotFoundError: No module named 'cyber.python'

What can I do? Thanks a lot.

/home/mini/researches/cyber_rt_8/CyberRT

hi @plumewind , Please check the Python version in CyberRT/build/lib. If it is python 3.8, please try running the following command: export PYTHONPATH=$PYTHONPATH:/home/mini/researches/cyber_rt_8/CyberRT/build/lib/python3.8/site-packages

It still error.

my local environment is ubuntu20.04, should other command be setting?

According to the error message, it appears that the cyber/python directory was not found in the build/lib/python3.8/site-packages/ path. Please verify that your environment and the steps you followed are correct. Here is my local environment:

It is a unhappy thing.I want to cry for a while.

It is a unhappy thing.I want to cry for a while.

You can try it with a clean Docker environment.


WIP: Cached OVA from a Pre-Build VMX

Background
more detail in RAC-4360
PPT in https://rackhd.atlassian.net/wiki/display/RAC1/RackHD+CI%3A+packer+build+enhancement+oppotunity
the OVA packer build is too long for 1 hour. but in 80% time, the actions are the same for each build.
so we use 2-way packer build
(1) build a template VMX
(2) install rackhd into above VMX, then pack to OVA

it requires on-build-config change as well.
jobs/build_vagrant/build_vagrant.sh
the box name become:packer_virtualbox-ovf_virtualbox.box
BOX="$PACKERDIR/packer_virtualbox-iso_virtualbox.box"
if [ -e "$BOX" ]; then
  mv "$BOX" "$PACKERDIR/rackhd-${OS_VER}-${RACKHD_VERSION}.box"
fi

@anhou

:+1:

comments fixed.
@anhou  @iceiilin


Truffle decomposition not effecting non-splash potions

When drinking a potion as a truffle you still gain effects from potion, im not sure if this is a bug or not but when reading the effects list, I assumed that the decomposition attribute would also affect drinking potions.

This would be an Extra Origins issue, not an Apoli issue.
But as to answer your question, this is intended behavior.


Update TypeScript Instructions

Closes https://github.com/zeit/next.js/issues/8176

Only thing that shouldn't be in Git would be .next, the rest should be tracked 👍


Resign first responder on iPad when dismissing kb (when using an external keyboard)

Bug Report Dump (Auto-generated)

Version 1.11.0 (2101)
Device: iPhone 7 Plus (iOS 11.0.3)
TestFlight: true

Which view? I’m confused how this is a thing. Doesn’t resign happen when you dismiss the kb?
Sent with GitHawk

@BasThomas is this what you're talking about?

Kind of, @heshamsalman. I’ll try to post a video to explain the issue.
Sent with GitHawk

Oh, so maybe we need to add resignFirstResponder when the view disappears?

Yep, that would be a way of fixing it 👍
Sent with GitHawk


svg height should be fixed, not equal to the width. fixes #24

for some reason the height was set to tw (total width)

:+1: I'm looking forward to this being accepted.

thanks!


Sort-of handle terraform apply

This doesn't try and get too clever with the parser, but it does handle terraform apply relatively well.
I think this approach could be extended a bit with some refactoring to pass-through the progress output from terraform commands, and only parse & redisplay the "meat" of the plan itself.
This implements the second strategy mentioned in https://github.com/coinbase/terraform-landscape/issues/53#issuecomment-392192553
I've done some local testing with the binaries on the shell, and the rspec test seems to cover the happy path.

All done and ready to go.


Add link to curl in blog

and fix the  comment again

Let's roll forward the website when this is submitted.


Instant bean validation range support

Using the Past annotation on an Instant doesn't seem to work.
There should be a InstantRandomizer extending from "RangeRandomizer" registered in the TimeRandomizerRegistry.

Fixed in #383 .
@stevenvanophem Thank you very much for reporting the issue and contributing a fix!


Improving terraform taggable resources list

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
As #374 described, this pr change the static taggable resources list to a calculated list by using terraform-aws-schema / terraform-azurerm-schema / terraform-google-schema.
This pr also implemented the following logics:

Once a taggable resource was deprecated in the later major version (e.g.: azurerm_data_lake_analytics_account was deprecated and deleted in AzureRM provider v3.0), is should still be taggable for backward compatibility.
Once a taggable resource's tags argument was deprecated and deleted (e.g.: azurerm_log_analytics_linked_service's tags has been removed by this pr), it should not be taggable.
A resource which contains tags but with different type (map(string) for aws and azurerm, set(string) for google) should not be taggable. (e.g.: azurerm_api_management_property)
Resources in structure.unsupportedTerraformBlocks should not be taggable.

I was unable to generate schemas for awsv0, awsv1, azurermv0, azurermv1, googlev0, googlev1, so the lowest major version for these three clouds is v2. For now it supports awsv2, awsv3, awsv4, awsv5, azurermv2, azurermv3, googlev2, googlev3, googlev4. Once a new major version of provider has been released, there won't be new corresponding schema tag, we need update the schema repo to generate new package version for schema repo, and upgrade yor's code to adapt with the new schema package.
This pr contains some new unit tests to verify these logics. @nimrodkor Would you please give it a review? Thanks!

It looks like bug in the latest go-git version (v5.7.0), I'll try to downgrade go-git version for schema repo and try again tomorrow.

I was unable to downgrade the go-git version... It seems like the only way is solve this go-git issue.

please keep it open

I have upgraded go-git version to 5.8.0 in this pr to get tests passed. @nimrodkor I think this pr is ready for your review finally.

@gruebel A kindly ping, is there anything I can do to get this pr be merged?

Hi @gruebel a kindly ping, I've rebased this pr with the latest main branch and upgraded the module's version. I saw security step failed once but I cannot see the detail. Would you please try this pr again and guide me how to fix the security scan issue if it failed again? Thanks!

Hello @nimrodkor @gruebel a kindly ping, I think this pr is ready for your review, and the update it brought is very important for some community users. Could we merge it? Thanks!

@rotemavni @gruebel @omryMen PTAL 🙏


Remove is_err() from linera-core/src/unit_tests/worker_tests.rs

The goal of this cleanup task is to replace is_err by assert!(matches!(...)) as we did in linera-core/src/unit_tests/client_tests.rs already.

I did a little bit of digging and here is my findings. I found total of 14 instances of is_err(). Below is my finding (errors previously defined) on how to perform the cleanup (Just a rough mapping for discussing)

Bad Signature                                -> InvalidSignature
Non Positive Amount                    -> InsufficientFunding
Timestamp too far in the future   -> InvalidTimestamp
Timestamp older than previous  -> InvalidBlockTimestamp
Missing earlier blocks                   -> MissingEarlierBlocks
Insufficient funding -> InsufficientFunding
Exceed Balance -> InsufficientBalance
Sender is checked -> InvalidSigner(Owner)
certificate_unknown_sender -> CertificateSignatureVerificationFailed
Unknown Sender -> UnexpectedMessage
With Chaining -> Couldn't find any
admin chain has an anticipated message -> Couldn't find any

I couldn't get hold of the point 11, function name is run_test_handle_block_proposal_with_chaining. Is ordering the reason?
Also I couldn't understand this comment admin chain has an anticipated message
Thank you!

Hi @ashu26jha, Thanks for inquiring. The methodology here is to figure out each expected error by executing the test: typically replacing each assert!(xxxxx.is_err()); by xxxxx.unwrap(); successfully.

Hey thanks for the reply @ma2bd
Just to be sure, here is an example of what I'm planning to do :
Previous :
// Insufficient funding

assert!(worker.handle_block_proposal(block_proposal).await.is_err());

Now :
// Insufficient funding

assert!(matches!(worker
      .handle_block_proposal(block_proposal).await,
      Err(SystemExecutionError::InsufficientFunding { .. })
));

Is this the way to move forward?

Yes, look at the examples in client_test.rs to be sure.


Create Omeda+IdentityX Rapid Identification Router

Can be used by a website to mount a JSON endpoint for rapidly identifying the active IdentityX user (if present).
If the Omeda ID for the configured brand is already set on the IdentityX user, the value will simply be passed through. If not set, the "rapid identification dance" is initiated.

 Omeda integration


Added example and documentation for EiffelTestCaseFinishedEvent.

Addresses issue #29.

:+1:

Typo: failued -> failed


Terminate peacefully on connection failure

The current behavior of Pegasus on SSH connection failure is to panic, and then abort. It'll be great if the async tasks handling connections just return and the scheduling loop detects this (or perhaps don't even start before all connections are successfully established) and terminates Pegasus peacefully.

Implemented in v1.2.0.


another attempt to get the windows ci builds working

simplifying things this time (hopefully)

@ptosco I think I have something working here. I will tweak a bit more tomorrow to confirm

@ptosco I think this is now ready to merge.

@greglandrum Thanks for letting me know!


Palettes with 256 fixed entries from an image don't work

Trying to generate a palette with fixed entries from a 256x1 image gives an [error] Too many fixed colors for palette. Reducing the size of the image to 255x1 works, but leaves a palette slot unusable.