⚠ Bofrost.be has degraded performance

In 72cbe3e, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2749 ms

Resolved: Bofrost.be performance has improved in efb080f.


show datapoints with 0 downloads

Dates with zero downloads are not included in the downloaded data, so
we have to insert new datapoints between the other datapoints.

Chart.js has a spanGaps option:

If true, lines will be drawn between points with no or null data. If false, points with null data will create a break in the line. Can also be a number specifying the maximum gap length to span. The unit of the value depends on the scale used.

Wouldn't that be a simpler solution?

I'm not sure that would work better, because it would create gaps instead of connecting a line to 0, and we would still have to insert data points with a null value where the gap should be. I think it's easier to insert data points with a value of 0 and not use this option.


Not working

This modules seems to not fetching data anymore

Oh sorry I forgot to tell everyone that this repo is deprecated and I replaced with MMM-HK-Transport-ETA


Bug fix on no MatchError issue

It's expecting to match "SequenceNumber" => instead of sequence_number: on the message in the function head. Closes https://paradoxai.atlassian.net/browse/TD-925

** (MatchError) no match of right hand side value: %Broadway.Message{data: "CiQ0NzBlYzA3My1hZTY3L...", metadata: %{"ApproximateArrivalTimestamp" =><PHONE_NUMBER>.392, "EncryptionType" => "KMS", "PartitionKey" => "714efd9e-45b1-4ee2-b364-185d4a648240", "SequenceNumber" => "49640695724539347397828651676380246766125300365922402722"}, acknowledger: {Broadway.CallerAcknowledger, {#PID<0.6234.0>, #Reference<0.3282250854.2604662786.39419>}, nil}, batcher: :default, batch_key: :default, batch_mode: :bulk, status: :ok}

@ryanlchandler good point. I can check how often that error has shown up.


fix-proffile

Fixed spelling error

Thank you very much. Unfortunately, I didn't checked for this opened PR. I will definitly will do in the future since I like you getting involved.
I have fixed this during my past live stream.


ability to extend Auth class

Description
Hi , I had a problem with lucia and nestjs , if I wanted to use lucia and continue to use dependency injection pattern I can't do it without work around at first the work around was to make class and object assign the what returned from lucia function to this class and then annotate the service as unknown then convert it into type Auth then I got it to work like this
in the AuthService :
@Injectable()
export class AuthService  {

    constructor(readonly db: PrismaService) {
        let auth = lucia({
            middleware: express(),
            adapter: prisma(db, {
                user: "user", // model User {}
                key: "key", // model Key {}
                session: "session" // model Session {}
            })
            , env: "DEV"
        })
        Object.assign(this, auth)

    }
}

in the AuthController :
@Controller('auth')
export class AuthController {
	auth :Auth
	constructor( AuthService:AuthService){
		this.auth = AuthService as unknown as Auth
	}
    // ***
}

it works fine and types also works but it doesn't seem right to me so I tried  to make AuthService extend  Auth class but the problem was that Auth exported as type not as a class so I modified the source code to export Auth class and it worked
the AuthService code :
@Injectable()
export class AuthService extends Auth {
    constructor(readonly db: PrismaService) {
        super({
            middleware: express(),
            adapter: prisma(db, {
                user: "user", // model User {}
                key: "key", // model Key {}
                session: "session" // model Session {}
            })
            , env: "DEV"
        })
    }
}

AuthController code :
@Controller('auth')
export class AuthController {
	constructor(readonly AuthService:AuthService){}
}

to me the ability to extend Auth class would be great but I don't know the project , so is it possible to do ?
or I am doing it wrong and there is already a solution for it ?

Unfortunately not right now, but this is addressed in v3 (currently in beta but very experimental).

after alot of search it came out that there is a way to do it using custome provider .
it would be somthing like this :
AuthModule :
const LuciaProvider = {
  provide: "Auth",
  useFactory: (db: PrismaService) => {
    return lucia({
      middleware: express(),
      adapter: prisma(db, {
        user: "user", // model User {}
        key: "key", // model Key {}
        session: "session" // model Session {}
      })
      , env: "DEV"
    })
  },
  inject: [PrismaService]
}
@Module({
  providers: [LuciaProvider],
  imports: [DatabaseModule],
  controllers: [AuthController]
})

AuthController :
import { Controller,  Inject } from '@nestjs/common';
import type { Auth } from "lucia"

@Controller('auth')
export class AuthController {
	constructor(@Inject("Auth") readonly auth: Auth) {

	}
	// ***
}


Replace Vue with React

Thanks for this great work!
Is there any way to replace Vue with React?

Should be straight forward to change the two main.ts to react dom and replace vite plugin in vite.config.ts

Best quick solution is to disable fast refresh in the react plugin:
// vite.config.ts
react({ fastRefresh: false })


ember-cli install fails

Here's the error after ember install:addon ember-data-model-fragments:
version: 0.1.5
Installing packages for tooling via npm.sh: ./node_modules/bower/bin/bower: No such file or directory
<EMAIL_ADDRESS>postinstall: `./node_modules/bower/bin/bower install`
Exit status 127
Error<EMAIL_ADDRESS>postinstall: `./node_modules/bower/bin/bower install`
Exit status 127
    at EventEmitter.<anonymous> (/Users/iradchenko/workspace/mapgeo2/dashboard/node_modules/ember-cli/node_modules/npm/lib/utils/lifecycle.js:212:16)
    at EventEmitter.emit (events.js:98:17)
    at ChildProcess.<anonymous> (/Users/iradchenko/workspace/mapgeo2/dashboard/node_modules/ember-cli/node_modules/npm/lib/utils/spawn.js:14:12)
    at ChildProcess.EventEmitter.emit (events.js:98:17)
    at maybeClose (child_process.js:743:16)
    at Process.ChildProcess._handle.onexit (child_process.js:810:5)

Love that you can't truly test these things until post-publish. Weeee!

Yeah, good thing we have the patch number in semver :+1:

Maybe use this feature: https://github.com/ember-cli/ember-cli/pull/2805
Plus something like this https://gist.github.com/stravid/a3a56d8eadc6251c0be9#file-blueprint-js maybe

That was the intention. The postinstall hook in package.json that is screwing this up has been around since the beginning, I do it with all projects.

This should be fixed in v0.2.5.


Copy/paste using long-press on text field

Currently, long press on a text field opens a menu to "edit prompt / remove response". It would be helpful to be able to paste a string, and possibly also to copy its existing contents.
Can the copy/paste options be added to the long press menu for text fields?
This issue came up recently:
https://groups.google.com/forum/#!topic/opendatakit/f9nINg8x81Y
I encountered the same issue when I needed to copy a location from Google Maps to clipboard, and paste it in an ODK question.

+1 - this would be really helpful!

+1, this would be useful when using other applications alongside ODK.


style attribute used with leafletBaseLayers

[x] Review the README
[x] Tell us what version of Leaflet, Angular, and ngx-leaflet you're using
[x] Include code that reproduces your issue

leaflet: 1.5.1
angular: 8.2.8
ngx-leaflet: 4.0.0 & 6.0.1
Using leaflet as the docs describe by setting a style height on the div breaks.
  <div leaflet style="height:540px"
    [leafletOptions]="options"
    [leafletBaseLayers]="baseLayers"
    [leafletLayersControlOptions]="controlOptions"
    (leafletMapZoomEnd)="zoomEnd( $event )"
    (leafletMapReady)="onMapReady( $event )"
  >

This works just fine:
  <div leaflet
    [leafletOptions]="options"
    [leafletBaseLayers]="baseLayers"
    [leafletLayersControlOptions]="controlOptions"
    (leafletMapZoomEnd)="zoomEnd( $event )"
    (leafletMapReady)="onMapReady( $event )"
  >

Used with
.leaflet-container {
  height: 540px;
}

I know this is a bit late of a response, but usually that error implies that there is something wrong that caused the plugin to fail to load or that there's something wrong with the 'leaflet' directive not getting applied to the element.
Are you sure there isn't a typo in your HTML that's causing it to not recognize the 'leaflet' attribute? Or, that you aren't forgetting to import the LeafletModule into your module and app module?

Closing cause no response


Applications not appearing in Start Menu search results after "Run OO Shutup" tweak

After running the abovementioned tweak, newly installed applications do not show up in the start menu search, even though Start menu entries exist.
To Reproduce

Perform clean install of Windows 10 22H2
Install 7zip from downloaded installer. Install Spotify from MS Store.
Open Start menu and search. 7zip/Spotify appear in search results as expected.
Uninstall 7-zip/Spotify.
Download winutil.
Run "Run OO Shutup" tweak.
Install applications again.
Open Start menu and search. 7zip/Spotify do NOT appear in search results.

Expected behavior
Apps installed after winutil tweaks should appear in Start Menu search results.
The configured shutup tweak either needs to not break the Start menu searching for local apps, or there should be an explicit warning stating that it does (like the warnings on the other tweaks).

The same thing happens for me. Undoing the tweaks doesn't seem to make the apps appear in the search again either.

I'm having the same issue, it would be good to know which option can be changed in O&O Shutup to fix it.

I'm also having the exact same issue

i was able to use the search function again by installing the O&O ShutUp10++ app, then undo all changes ("factory settings"), and then Apply only recommended settings.

This seems to work for me too

i was able to use the search function again by installing the O&O ShutUp10++ app, then undo all changes ("factory settings"), and then Apply only recommended settings.

That worked for me as well.

Also adding this. I can reproduce this most times.
Interestingly, does not always happen though.

Just tested the 23.07.2020 version, and this issue still persists on a clean Win 10 22H2 install.
Running OOShutup manually and turn off "Prohibit apps from running in the background" under Current User > App Privacy
Then rebooting restores proper installed app search functionality.
Note you may want to go to Windows Settings > Privacy > Background apps and uncheck individual apps. 'Settings' seems to control if Search can properly index app installations.

Thanks for double checking, I added a new commit cd586b3
that updates the o&o cfg to leave app background enabled


Service 'boilerplatebackend' failed to build

Expected: It to work.
Actual: It didn't work.
Docker version 17.06.2-ce, build cec0b72

I am not too well versed in Docker, so it may be obvious to another set of eyes. But I followed the instructions to the t, and this is actually a fresh admin account on this Mac with only a few programs and Docker installed.

You do have NodeJS installed and in the path?


Fix Array textarea reactivity

Fixes #1597.
I have no idea why exactly this fixes it but I assume it has something to do with how vue handles v-model on custom components in combination with this TextareaAutosize component.

@mktcode we can then also remove this? https://github.com/snapshot-labs/snapshot/blob/aa851ce4522c58465adae07b79f0989a02cdb5d6/src/views/SpaceSettings.vue#L168

Yep, did that.

The reason for those components ignoring the modelValue changing coming from the parent is actually simple:
if (props.modelValue) input.value = props.modelValue.join('\n');
This line is simply not reactive. It's executed once. It would need a watcher. By passing the modelValue directly to the child component now in the TextareaArray, this issue doesn't exist.
I'll leave the JSON textarea as is for now, since we don't have problems with it and the change would trigger the input being formatted whenever it becomes valid, which I think can get annoying when manually writing JSON. Otherwise it might actually be cool. Idk...

Yep, because doing it with a watcher was creating some other issue for me as far as I can remember writing this. The auto format seem kinda neat I think :D

Yep, because doing it with a watcher was creating some other issue for me as far as I can remember writing this. The auto format seem kinda neat I think tho :D

Yea, if the cursor wouldn't jump to the bottom everytime... :(


useResizeColumns broken in v7.beta.22

The columns still display the resizing styles, but dragging them has no effect.
Easily reproduced in https://codesandbox.io/s/github/tannerlinsley/react-table/tree/master/examples/column-resizing

Investigating...

FYI I think that:
https://github.com/tannerlinsley/react-table/blob/master/src/plugin-hooks/useResizeColumns.js#L64 should be ...newColumnWidths and not ...action.columnWidths, and that https://github.com/tannerlinsley/react-table/blob/master/src/plugin-hooks/useResizeColumns.js#L133 should be startX: clientX rather than just clientX.

Fixed in beta.23

yay though it's a bit weird to see the release before the code is here, and yes, I can see tha tyou release from a private repo.

Yeah, until I can nail down reliable CI release automation, it's manual for now. Usually I'll build/test/release locally, then push the commits (including the auto-incremented package version) to github, eg. The code you're referring to was available just moments after I posted that I fixed it. I also posted that before it was done publishing 😛, so there was even more delay.

In case it's not obvious, I really appreciate the amount of time you put into this library, I'm finding the v7 approach really powerful.

Thanks! That’s good to hear. Validating as well.
Tanner Linsley
On Dec 2, 2019, 5:07 PM -0700, ggascoigne<EMAIL_ADDRESS>wrote:

In case it's not obvious, I really appreciate the amount of time you put into this library, I'm finding the v7 approach really powerful.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

I hate to say this, but as far as I can tell, the column-resizing example is still broken.  I even built it locally in case it was the code sandox not doing the upgrade correctly.

I hate to say this, but as far as I can tell, the column-resizing example is still broken. I even built it locally in case it was the code sandox not doing the upgrade correctly.

Same for me. This is still definitely broken.

Just tested with versions beta.23 and beta.24 and it's working just fine. Make sure you clear your cache in codesandbox.


Adds janitorial service lights

About the PR

This PR adds wall-mounted service signs/lights and a low-brightness light bulb variant that is used in the sign itself. The sign's blinking light is a signal for roaming janitors that a department needs cleaning.
Why / Balance

The service light is a toggled visual indicator of a department's request for assistance, placed where janitors will encounter them. It is meant to improve station communication by providing an alternative to janitor calls on common radio.
Technical details

Media

https://github.com/space-wizards/space-station-14/assets/138547931/b878dbe6-0f2f-4be3-ad62-8099fed56b21

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

No changelog this time because the lights still need to be mapped. This just adds them into the game. If a changelog is wanted though, I will write one.

you should write a changelog, ill map these in

I would greatly appreciate that! Thanks for the help Emisse! Ill write a changelog right now.


Support tabular formats: jsonlines, csv

It would be handy to have direct support for jsonl and CSV/TSV formats.
The easy workaround is:
jq -s '{ "items": . }' my.jsonl jinjanate my.j2 --format json

(assuming my jinja2 iterates through a key items)
or
jq -s . my.jsonl jinjanate my.j2 --format json --format-option array-name=items

Similarly, for CSV/TSV input tools like csvjson can easily do the processing up-front
So low priority but would be handy to avoid boilerplate preprocessing

Definitely an interesting idea, and this sort of thing is the reason why plugin support was created :-)
If the preprocessing can be done by a Python package, then creating a plugin would be fairly easy to do.


Create a custom property on payloads with an item property

When you read a payload that has an item property and convert the JSON to objects in Powershell, it is sometimes difficult to access the item property. This is because the default PSCustomObject that is returned from ConvertFrom-Json already has an Item property.
So you get back an array instead. It's not clear to me what is in the first element. It appears to be an empty string.
In the second element is the response, but item is accessible on it.
In my recent testing with small JSON I couldn't replicated this, but I see it with object views all the time.
So consider adding an alias property to the response object (named data?) that is the item.
A custom property can be added like this:
PS > $newResponse = Select-Object -InputObject $response -Property data
PS > $newResponse.data = $response[1].item
PS > $newResponse.data.presentValue
0

This allows the client of the response object to just access $newReponse.object.propertyName

UPDATE: The issue that I'm seeing has to do with the response to Invoke-MetasysMethod. It's returning an array (not a string) with the second element (index 1) containing the response. This is an odd error. Then when you run it thru ConvertFrom-Json you get the issue of having to access the second element. So the root cause must be fixed. Why is an array being returned?

And the root cause: Anything you evaluate in a Powershell script goes to the output stream. Anything you write-output goes to the output stream. Any thing you return goes to the output stream. As a matter of fact these are equivalent
PS > 5
PS > return 5
PS > Write-Output 5

I believe I've fixed my core issue by changing all Write-Output to Write-Information. The prompts for the user still show up on the console, but aren't part of the output.


Two roundings in MultiplyByQuantizedMultiplier(TFLite) leads to inconsistent result with tensorflow

https://github.com/tensorflow/tensorflow/blob/ff91cd691027076e6128afbd902d3d38e3672787/tensorflow/lite/kernels/internal/common.h#L105
Per my understanding, MultiplyByQuantizedMultiplier is used to simulate floating point multiplication.
But in some cases, the two roundings in it will cause different result from floating point arithmetic, which will further lead to unpredictable tflite accuracy compared with tensorflow training pipeline.
inline int32 MultiplyByQuantizedMultiplier(int32 x, int32 quantized_multiplier, int shift)
Assume x = 7984, quantized_multiplier =<PHONE_NUMBER>, shift = -9 (the simulated floating point arithmetic is std::round(7984 * 0.0014402703931141053)). The multiplication of x and quantized_multiplier will be<PHONE_NUMBER>7296(0xB7FC6402F40), it will undergo two rounding shift(one in SaturatingRoundingDoublingHighMul and the other one in RoundingDivideByPOT), and results in number 12. But the floating point result is 11. We can find that if we combine these two rounding shifts into one, i.e. do only one rounding shift, the results will be 12, which matches with floating point result.

Hey, thanks for raising the question and explaining your logic in the example.
I think the problem is not with splitting the 31 bit shift and 9 bit shift: (a >> 31) >> 9 is the same as a >> 40, assuming there is no overflow. The subtle difference in from the "nudge" in gemmlowp (https://github.com/google/gemmlowp/blob/master/fixedpoint/fixedpoint.h#L320), that instead of (a >> 31) >> 9, it becomes ((a + delta) >> 31) >> 9. The nudge logic is added to simulate "VQRDMULH".

if one merges the >> 9 to .../(1ll<<31) to make it .../(1ll<<40), the result is still 12.
If one forces "nudge" to be zero, the result is 11.

BTW, the value just happen to be close to one of the 0.5. Without nudge, SaturatingRoundingDoublingHighMul gives 5887 and 5887/2^9 = 11.49, which rounds to 11; with nudge, SaturatingRoundingDoublingHighMul gives 5888 and 5888/2^9 = 11.5, which rounds to 12.
Do you have a measurement of the overall accuracy impact because of this rounding? Thanks.

Hi, what is this "The nudge logic is added to simulate "VQRDMULH". Why is it necessary?

Hey, thanks for raising the question and explaining your logic in the example.
I think the problem is not with splitting the 31 bit shift and 9 bit shift: (a >> 31) >> 9 is the same as a >> 40, assuming there is no overflow. The subtle difference in from the "nudge" in gemmlowp (https://github.com/google/gemmlowp/blob/master/fixedpoint/fixedpoint.h#L320), that instead of (a >> 31) >> 9, it becomes ((a + delta) >> 31) >> 9. The nudge logic is added to simulate "VQRDMULH".

if one merges the >> 9 to .../(1ll<<31) to make it .../(1ll<<40), the result is still 12.
If one forces "nudge" to be zero, the result is 11.

BTW, the value just happen to be close to one of the 0.5. Without nudge, SaturatingRoundingDoublingHighMul gives 5887 and 5887/2^9 = 11.49, which rounds to 11; with nudge, SaturatingRoundingDoublingHighMul gives 5888 and 5888/2^9 = 11.5, which rounds to 12.
Do you have a measurement of the overall accuracy impact because of this rounding? Thanks.

--> The nudge logic is added to simulate "VQRDMULH".
I think using VQRDMULH is wrong here,  because the result is not final number and you need to perform rounding-shift again.  Rounding after 31-bit shifting will cause the error on numbers such as x.499.
I think VQDMULH without rounding is the correct instruction to use.

instead of (a >> 31) >> 9, it becomes ((a + delta) >> 31) >> 9.
* if one merges the >> 9 to .../(1ll<<31) to make it .../(1ll<<40), the result is still 12.

If I understand correctly, in RoundingDivideByPOT, effectively another nudge is added, making it (((a+delta_1) >> 31) + delta_2) >> 9. (This is not implemented by adding a constant delta_2 but by checking if the 'shifted-away-bits' are larger than a threshold, and then adding 1 to the result.).
I think that's the reason why the two separate shifts are not the same as a single shift of 40, and that might be the cause of these rounding mismatches ?

@petewarden  Would it be a good idea to use this rounding on tflite-micro ?

Bit-exactness is not part of the contract here. Do whatever is friendly to your target hardware as long as it's a reasonable round-to-nearest scheme without significant bias.
It is expected that different hardware targets will run different, non-bit-exact flavors of MultiplyByQuantizedMultiplier and RoundingRightShift.
Regarding MultiplyByQuantizedMultiplier:
For example, we have seen people use 16-bit fixedpoint instead of 32-bit fixedpoint multipliers on Qualcomm HVX, and we have seen people use float32 instead of 32-bit fixedpoint multipliers on Google EdgeTPU.
Regarding RoundingRightShift:
Even on ARM NEON, our thinking has evolved a bit. We used to think that it would be important for the rounding right shift to break ties away from zero, out of a concern that breaking ties upward would be "biased". However, in practice, away-from-zero is only unbiased if the data is centered around zero, which it not always is, so there was no practical unbiasing benefit from that, so we now happily use ARM NEON's own "rounding right shift", breaking ties upward.
In the newer matrix multiplication library used by TFLite in the quantized case on ARM, we are using SQRDMULH followed by RSHL i.e. native NEON rounding shift per the previous paragraph, by default. We've retained the ability to compile to the old break-ties-away-from-zero way but might drop it soon.
Search for NATIVE_ROUNDING in this file,
https://github.com/google/ruy/blob/a0ca5e6d3d1b1771197368c5cf38de1f9fe7f1e0/ruy/kernel_arm64.cc
We hadn't bothered to update these TFLite functions because not much time was being spent there.
Do you care about the efficiency of these functions because they're significant on your CPU profiles or do you care about the clarification that bit-exactness is not required for your own ports?  In the former case, we'll happily take a patch optimizing these functions by relaxing them as in the above-linked ruy code. In the latter case, this might be more of a documentation issue? Do you still see a need for SaturatingRoundingDoublingHighMulWithShift given the non-requirement of bit-exactness ?

Thanks for the clarification, that helps a lot!
I will investigate the latency impact of this on the systems I'm interested in, and if its significant I'll submit a patch.

Do whatever is friendly to your target hardware as long as it's a reasonable round-to-nearest scheme without significant bias.

I have one more question about this: from what I understood, the scheme should always round-to-nearest, and the freedom is in how perfect ties are rounded. But in the example above, the numbers 0.25, 1.25, 2.25 etc all get rounded upwards to 1, 2, 3 respectively, which is not round-to-nearest anymore. So is there more freedom than just the ties?

Ah, I understand what you were saying now, sorry that I didn't read closely enough the first time.
It's not very well defined what exactly are the boundaries of what's allowed. As I alluded to above, some hardware targets have needed to perform these computations at lower bit depths, such as 16-bit fixedpoint multipliers on Qualcomm HVX or 23-bit-mantissa float32 on Google EdgeTPU. That in itself is more freedom than merely freedom in breaking ties. So when I said "Do whatever is friendly to your target hardware as long as it's a reasonable round-to-nearest scheme without significant bias." it had by necessity to include allowing for such deeper arithmetic differences, such as using lower precision data types, and then it seems unavoidable that we can't require off-by-one errors to be limited to ties only.  So yes, there is more freedom than just the ties.
How much freedom exactly, is not formalized at the moment.
Separate from "how much freedom do hardware ports have" is the other question "what code should TFLite carry as 'reference code'". TFlite's reference code needs to be accurate, portable, and give a reasonable starting point for porting to hardware. The current code is shown by the present issue to have a quite serious accuracy flaw, but on the other hand it's easy to port to hardware thanks to being a sequence of two distinct steps, MultiplyByQuantizedMultiplier and RoundingRightShift, each of which tends to be fairly well understood how to map to hardware. by contrast, SaturatingRoundingDoublingHighMulWithShift is better accuracy-wise but it is apparently difficult to port to most hardware, because it seems to require going through twice-wider integer (e.g. 32bit -> 64bit) to apply the nudge there, unless the architecture natively has an instruction for doing exactly that computation (it's not inherently harder to implement in hardware, it just does not fit common fixed-point-multiplication instructions). So at this point I don't know what we can usefully do about this.  Maybe at least add a summary of this discussion / a link to this issue in a comment ?

TFlite's reference code needs to be accurate, portable, and give a reasonable starting point for porting to hardware.

Thanks for the clarification, that makes a lot of sense, and the 'fix' I suggested does not satisfy those criteria.

it seems to require going through twice-wider integer (e.g. 32bit -> 64bit) to apply the nudge there

I'm not sure if that is a big problem, since the nudge was already being added to the 64-bit multiplication result, before it was right-shifted back into a 32-bit value. I do agree that the function as a whole would no longer translate to a common hardware instruction.

Maybe at least add a summary of this discussion / a link to this issue in a comment ?

That sounds good to me.
I have been running TF Micro on Cortex-M cores using CMSIS-NN, and it seems that in their effort to become TF Micro compliant, they have copied this reference rounding code, see here .
As far as I know, on the Cortex-M series these two functions do not directly translate to instructions, in which case SaturatingRoundingDoublingHighMulWithShift would actually make things faster (and more accurate).
So based on what you're saying (and if its indeed faster), it makes sense to ping the devs of CMSIS-NN and tell them that the bit-exact match is not needed, and they can use SaturatingRoundingDoublingHighMulWithShift, potentially guarded by a preprocessor flag like RUY_OPT_NATIVE_ROUNDING from the example you linked to.
(The tflite implementation of MultiplyByQuantizedMultiplier could also do this based on such a preprocessor flag, but that would only affect reference code so it should not matter so much.)

I do agree that the function as a whole would no longer translate to a common hardware instruction.

Right, that's the concern that I was highlighting here.
I agree with everything you say. I hadn't been following these developments around TFLite for microcontrollers. It does sound like some documentation or at least comments to make the conclusions of the present discussion more discoverable, would be useful.
Point taken that, SaturatingRoundingDoublingHighMulWithShift, while less friendly to NEON hardware, is more friendly to Cortex-M, so it arguably makes a better reference function to have here in TFLite code. I'd like to let others take it from here, as I haven't been very closely involved in this code for the past 2 years and I shouldn't end up owning this again. @suharshs ?

Thanks @bjacob for the detailed responses.
We cannot change the default rounding mode in TFLite due to backwards compat issues, that being said since you are targetting a microcontroller, TFLite Micro may be able to accomodate this need in their infra. Will assign to them to get their thoughts.
Thanks!

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The TFLite team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the TFLite space.
Thanks.


[dashboard] Fix URLShortLinkButton position after click anchor link

CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
in #7640 I added a change that prevent click event bubble up, so that clicking on anchor link will not trigger tab switch. But i just found this introduced a bug: if you click on tab's anchor link at the bottom of long dashboard, URLShortLinkButton can not re-position since there is no click event triggered, so it cause URLShortLinkButton show up in wired position.
Solution
Fix the click event handling in higher parent component. In click handler for Tabs component, I added logic to check event's target. It will switch tab when the event's target is not anchor link component.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:

After:

TEST PLAN
CI and manual test.
REVIEWERS
@etr2460 @williaster @michellethomas

Codecov Report

Merging #7812 into master will decrease coverage by 0.02%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master    #7812      +/-   ##
==========================================
- Coverage   65.89%   65.87%   -0.03%
==========================================
  Files         459      459
  Lines       22011    22009       -2
  Branches     2417     2418       +1
==========================================
- Hits        14505    14498       -7
- Misses       7385     7390       +5
  Partials      121      121

Impacted Files
Coverage Δ

superset/assets/src/components/AnchorLink.jsx
100% <ø> (+8.33%)
:arrow_up:

...erset/assets/src/components/URLShortLinkButton.jsx
76.47% <ø> (ø)
:arrow_up:

...s/src/dashboard/components/gridComponents/Tabs.jsx
64.61% <75%> (+0.09%)
:arrow_up:

superset/db_engine_specs/sqlite.py
45.83% <0%> (-20.84%)
:arrow_down:

superset/utils/core.py
87.93% <0%> (-0.18%)
: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 6d7701e...d35e29d. Read the comment docs.


Update: OpenJS.NodeJS to v18.2.0

Updated OpenJS.NodeJS Current to v18.2.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


GEOPM integration automation does not always add GEOPM to the LD_LIBRARY_PATH

Describe the bug
I tried to run geopmlaunch impi -n 1 -ppn 1 hostname. I expected the run to simply print the host name and exit. Instead, several errors about LD_PRELOAD were printed before the hostname.
GEOPM version
Output from geopmread --version: 1.1.0+dev1917g7f9ed5b3
Expected behavior
When running an application that is not already linked to GEOPM, the launcher should preload the GEOPM runtime.
Actual behavior
The GEOPM launcher failed to preload the GEOPM runtime.
Error messages produced
ERROR: ld.so: object 'libgeopm.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): **ignored.**

Additional context
This issue was encountered on an OpenSUSE 15.3 machine. I guess something about our automatic build configuration selects a different lib path in that case. GEOPM's python libraries are installed to <prefix>/lib in our OpenSUSE 15.3 rpm. Our C libraries are installed to <prefix>/lib64. The integration/config/run_env.sh prepends only GEOPM's lib directory to LD_LIBRARY_PATH, so LD_PRELOAD is not able to find the .so files.
Workaround
Modify run_env.sh: LD_LIBRARY_PATH_EXT=${GEOPM_INSTALL}/lib
To look like: LD_LIBRARY_PATH_EXT=${GEOPM_INSTALL}/lib:${GEOPM_INSTALL}/lib64

Just tried on 2.0.0+dev826g01470fd59, SLES15.2 (which also has lib and lib64) and everything worked fine. Also explicitly added --geopm-preload but still no repro.
This issue may be obsolete, but my re-run example is also a slightly different config. Need to investigate what else changed in the launcher in case it was fixed by something else.

Repro on 2.0.0+dev826g01470fd59 with:
./configure --prefix=$HOME/build/geopm --libdir="$HOME/build/geopm/lib64" --libexecdir="$HOME/build/geopm/lib"
make -j && make install
# libdir is /usr/lib64 on this system (rpm --eval '%{_libdir}'). libexecdir is /usr/lib. Doing it this way to mimic how our specfiles do it
# compute node:
source ~/geopm/integration/config/run_env.sh
geopmlaunch impi --geopm-ctl=application --geopm-profile=anything --geopm-preload -n 1 -ppn 1 -- hostname


[Impeller] TRect::Area can overflow

The TRect::Area() method currently uses a simply multiply to return its result which can easily overflow integer math dimensions. It should use a saturated operation on integers to return MAX_INT (or similar) when the area overflows the size of the integer coordinate type.

@flar was this fixed with your patch to migrate to the new Rect implementation?

@flar was this fixed with your patch to migrate to the new Rect implementation?

No, it was discovered at that time but seemed beyond the scope of that PR.


Serve jquery-ui stylesheet from protocol relative URL

@marcincichon Can you review? Thanks

@sparkdevelopment looks good


New API in version 39?

First of all, let me apologize if this was already explained somewhere else. My google-fu has failed me. If anyone can point me in the right direction, I'd appreciate it.
Today, I use pkg_resources.parse_version() to check if a package version number conforms to standards. Before v39, this returned a pkg_resources.SetuptoolsVersion object if the version number was fine, or a pkg_resources.SetuptoolsLegacyVersion if it was not fine.
In the version 39 release that just came out, it looks like pkg_resources.SetuptoolsLegacyVersion has moved to pkg_resources._vendor.packaging.version.LegacyVersion. The fact that it's now in _vendor makes me think that it's an internal namespace that I shouldn't be relying on.
Is there a recommended way for me to use setuptools to determine if a version number is valid? Setuptools is already doing that work, so I'd like to take advantage of the existing library, but it seems like now it's not intended for general use?

Great question. And good instincts. The good news is you can still rely on Version and LegacyVersion from the packaging.version packages, and the fact that the class is coming from the _vendor package is just an artifact of how Setuptools gets packaging. It can't depend on packaging and thus has to vendor it (due to #581).
So the best recommendation, if you wish to determine if a version number is valid, is to use the packaging. But you're also welcome to use the result from pkg_resources.parse_version.
If you wish for packaging.version to be the same class as the one returned by pkg_resources.parse_version, that's a little more difficult, but you can delete the packages in pkg_resources._vendor and pkg_resources will use packaging as installed. I wouldn't recommend that except in specialized cases where you know what you're doing.
Does that help?

Re-reading your question and thinking about it some more, here's what I recommend.
The most direct and best way to determine if a version number is valid is to simply parse it with packaging.version.Version, which will raise an exception if it's invalid.
But if you're currently relying on the interfaces supplied by setuptools/pkg_resources, the best way to determine if a version is valid may be something like this:
ver = pkg_resources.parse_version(input)
is_legacy = 'Legacy' in ver.__class__.__name__

That approach will work for old and future versions of pkg_resources for as long as I can forsee.


Added functionality to pass BertProcessor a custom BertTokenizer

Particularly helpful if you have performed additional steps of pre-training with your own vocabulary.

Many thanks for this PR @thomasvn!!
Feel free to contribute with others PRs if you find any other useful features / functionalities!


Can’t use both async and regular top level tests

Using async callbacks for all top level tests works:
import t from 'tap'

t.test('1', async t => {
  t.pass()
})

t.test('2', async t => {
  t.pass()
})

But mixing async and regular tests like this:
import t from 'tap'

t.test('1', t => {
  t.pass()
})

t.test('2', async t => {
  t.pass()
})

Causes errors like this:

Yes, there is an error in your test.  It is indeed unfinished.  (Also, it looks like both code samples are the same, maybe a typo?)
The function provided to t.test() must do one of three things:

Return a Promise.  When the promise resolves or rejects, the test is completed.
Set a planned number of assertions by calling t.plan(n).  When n assertions are filled, the test ends.  (This plan must be set before making any assertions!)
Call t.end() explicitly.  (If this is done after a plan or promise completion, it's a no-op.)

Since async functions return a promise, they're in the first category.  If you want to mix non-async functions with async functions, that's fine.  Just end them by setting a plan, or calling t.end() when they're complete.
const t = require('tap')

t.test('plan', t => {
  t.plan(1)
  t.pass('this is fine')
})

t.test('end method', t => {
  t.pass('so is this')
  t.end()
})

t.test('explicit promise', t => {
  t.pass('this is fine')
  return Promise.resolve()
})

t.test('async promise', async t => {
  t.pass('i promise that this is fine')
})

it looks like both code samples are the same, maybe a typo?

Yes, it was a typo. Fixed!

If you want to mix non-async functions with async functions, that's fine. Just end them by setting a plan, or calling t.end() when they're complete.

Thanks for clarifying how it currently works, that is helpful. Would you be open to improving that behavior though? It is not a very intuitive limitation that you need to manually end the test inside a synchronous function; that is not how other libs like AVA work:
import test from 'ava'

test('foo', t => {
  t.pass()
})

test('bar', async t => {
  const bar = Promise.resolve('bar')
  t.is(await bar, 'bar')
})

If there is some sort of technical reason not to, it would be helpful to make the docs more obvious to recommend every test use an async function, or for people with the require-await eslint rule to alternatively use t.end(). Here is what I was doing:
t.test(
  '`execute` middleware options not an object.',
  // eslint-disable-next-line require-await
  async t => {
    try {
      execute(true)
      t.fail('Expected an error.')
    } catch (error) {
      t.matchSnapshot(errorSnapshot(error), 'Creation error.')
    }
  }
)

Does t.fail() properly end the test?

No, t.fail(msg) just prints a failing assertion.  (It's the inverse of t.pass(message).)

It is not a very intuitive limitation that you need to manually end the test inside a synchronous function; that is not how other libs like AVA work

Well, that's not how ava works.  Most other test frameworks in Node.js either sniff the test function to see if it takes an argument (ie, mocha and lab) and if so, pass in a done callback, or (tap, tape, and a few others) provide an object that has to be manually ended, or don't really support asynchrony with callbacks or objects at all (mostly very old test runners, like nodeunit and a few others), so every test has to be 100% synchronous.
Automatically ending tests with sync functions would be a huge change in tap.  There's a massive number of tests out there that do things like:
t.test('foo', t => {
  getSomeFoo(a, b, c, (er, data) => {
    if (er)
      throw er
    t.match(data,  {some: 'thing'})
    t.end()
  })
})

Those tests would auto-end and cause a lot of problems if I did what you're suggesting.  Ava started life by doing t.test.cb(..) for that use case, but tap follows a different pattern.


Node stays behind approximately distance to snapshot.

I have installed I'm running a base client pointed towards my up to date, synced L1.
eth_blockNumber of my L2 stays behind for approximately the same distance to the end of the snapshots, and then keeps the pace but doesn't' advance.
For example if I point HOST_DATA_DIR to the /reth-data (empty) it starts syncing very fast but from the very beginning
If I point it to my snapshot it starts also syncing very fast
node-1       | t=2024-10-26T01:58:38+0000 lvl=info msg="Advancing bq origin" origin=0x42bccb52f4bcdfd7cb77145632f14f98e3c276c8b370d0322c5a7ec68d683231:21015340 originBehind=false
but until certain point and they kind of keeps up but always lagging behind around 3-4 days which is the distance to the end of my  snapshotted data.
I've tried running it --l1.trustrpc but doesn't seem to make any difference.
The following command as suggested by the docs:
echo Latest synced block behind by: $((($( date +%s )-
$( curl -s -d '{"id":0,"jsonrpc":"2.0","method":"optimism_syncStatus"}' -H "Content-Type: application/json" http://localhost:7545 |
jq -r .result.unsafe_l2.timestamp))/60)) minutes
Always keeps falling behind.
Any pointers would be greatly appreciated.
The errors I see in the log is one that I don't think is related
execution-1  | ts=2024-10-26T01:08:00.38459394Z level=error target=engine::jwt-validator message="Invalid JWT: Authorization header is missing or invalid"
And one that might be
node-1       | t=2024-10-26T01:51:23+0000 lvl=warn msg="Engine temporary error" err="temp: failed to sync forkchoice with engine: context deadline exceeded"
The peer count is in general around 8 and goes up to 11

i was/am still hitting same thing , can you check log of reth if synchronising in batch mode?!
OP_NODE_SYNCMODE=execution-layer
OP_NODE_L2_ENGINE_KIND=reth <== seem without this  https://docs.optimism.io/builders/node-operators/configuration/consensus-config#l2enginekind reth doesn't go into batch mode for me, and it is missing in env example

then i can go to in-sync from similar scenario  of few days , but still some performance issues unsolved(random beteween few seconds - 1hour delay some times ) on reth  (already on nvme & not seeing obvious cpu bottleneck)

@jun0tpyrc I honestly gave up trying to make reth work and now switched to geth, it seems to be syncing reasonably fast now.

I had quite hard times here. I experienced static 12h drag and struggled a lot to fix it.
OP_NODE_L2_ENGINE_KIND=reth
fixes the problem for me. I believe it should be documented in this guide as there is no mention of it, not even in .env files

I had quite hard times here. I experienced static 12h drag and struggled a lot to fix it. OP_NODE_L2_ENGINE_KIND=reth fixes the problem for me. I believe it should be documented in this guide as there is no mention of it, not even in .env files

wonder what spec you on &/ able to stay within minute of delay on reth for base-mainnet?! 🙏

peer count 8-11 is quite low for an optimized performance, but if you managed to fix the issue great to hear ! true that OP_NODE_L2_ENGINE_KIND=reth is something you might have to add on configs, but its something you dont necessarily need but can be useful to know if you encounter some problems.


Pull Terraform "plugin" into module

Solution
The builtin Terraform inventory plugin should be shipped as a Puppet module that is bundled with Bolt, instead of being hardcoded into Bolt.
This should be structured similar to the Azure plugin.

CLosed with https://github.com/puppetlabs/bolt/pull/1308


Feature: Save settings in local storage

Type of feature
🍕 Feature
Current behavior
I use the same settings every-time I generate a description. But every-time there is an update or the cookie refreshes I need to save the settings again. It would be nice if we save the user settings in local storage.
Suggested solution
localstorage is preferred, but we could potentially store a settings JSON.
"settings" : {
  "Description Source" : "Commit Message",
  "Description Type" : "Formal"
}

Additional context
No response
Code of Conduct

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

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

We are saving the settings to chrome.storage.local for it to be persistent.
https://github.com/open-sauced/ai/blob/cc76e950a33d30677e2eb37fadb3d318ceca2d5b/src/popup/pages/aiprdescription.tsx#L35-L39
https://github.com/open-sauced/ai/blob/cc76e950a33d30677e2eb37fadb3d318ceca2d5b/src/utils/ai-utils/descriptionconfig.ts#L39-L41
The problem being, the storage is reset if there's a reinstall.

@Anush008, consider using chrome.storage.sync instead. See https://developer.chrome.com/docs/extensions/reference/storage/#storage-areas. The only caveat is it can't store much, only 100KB but it will perist across all devices and shouldn't get wiped between installs (need to double check that)
Also, see https://pandaquests.medium.com/data-retention-made-easy-persist-data-with-chrome-storage-api-in-chrome-extensions-ea4f4c721a62

Right. The extension settings are being saved to the synced storage. https://github.com/open-sauced/ai/blob/f67f67339a5daca506822715ee29aa66c4b0ba6e/src/popup/pages/settings.tsx#L15-L30
We can do same for the AI description settings too.

@Anush008 do we have anything to do for this issue?


Exception while trying to run multiple services connecting to same Cassandra

I am trying to run multiple services of same type connecting to a Cassandra cluster. Looks like we are hitting a race condition when running the scripts. The server fails to come up with the following exception. Please let me know if any one has faced the same issue.
The version of the software being used is 2.2.0
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'migrationTask' defined in class path resource [org/cognitor/cassandra/migration/spring/CassandraMigrationAutoConfiguration.class]: Invocation of init method failed; nested exception is org.cognitor.cassandra.migration.MigrationException: Error during migration of script 2_Change-13418.cql while executing 'DROP TABLE conversation_message_subscriber;'    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1778)    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:593)    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:515)    at org.springframework.beans.factory.support.AbstractBeanFactory.lambda$doGetBean$0(AbstractBeanFactory.java:320)    at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:222)    at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:318)    at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:199)    at org.springframework.beans.factory.support.DefaultListableBeanFactory.preInstantiateSingletons(DefaultListableBeanFactory.java:849)    at org.springframework.context.support.AbstractApplicationContext.finishBeanFactoryInitialization(AbstractApplicationContext.java:877)    at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:549)    at org.springframework.boot.web.servlet.context.ServletWebServerApplicationContext.refresh(ServletWebServerApplicationContext.java:142)    at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:775)    at org.springframework.boot.SpringApplication.refreshContext(SpringApplication.java:397)    at org.springframework.boot.SpringApplication.run(SpringApplication.java:316)    at org.springframework.boot.SpringApplication.run(SpringApplication.java:1260)    at org.springframework.boot.SpringApplication.run(SpringApplication.java:1248)    at Application.main(Application.java:32)    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)    at java.lang.reflect.Method.invoke(Method.java:498)    at org.springframework.boot.loader.MainMethodRunner.run(MainMethodRunner.java:48)    at org.springframework.boot.loader.Launcher.launch(Launcher.java:87)    at org.springframework.boot.loader.Launcher.launch(Launcher.java:50)    at org.springframework.boot.loader.JarLauncher.main(JarLauncher.java:51)Caused by: org.cognitor.cassandra.migration.MigrationException: Error during migration of script 2_Change-13418.cql while executing 'DROP TABLE conversation_message_subscriber;'    at org.cognitor.cassandra.migration.Database.execute(Database.java:187)    at java.util.ArrayList.forEach(ArrayList.java:1257)    at org.cognitor.cassandra.migration.MigrationTask.migrate(MigrationTask.java:52)    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)    at java.lang.reflect.Method.invoke(Method.java:498)    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.invokeCustomInitMethod(AbstractAutowireCapableBeanFactory.java:1903)    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.invokeInitMethods(AbstractAutowireCapableBeanFactory.java:1846)    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1774)    ... 24 common frames omittedCaused by: com.datastax.driver.core.exceptions.InvalidQueryException: unconfigured table conversation_message_subscriber

Hi Patrick, Even the latest version gives the issue. Also the new issue I am seeing now is the migration runs successfully. But there are some tables missing in the key space.

Hi,
this is really strange and so far you are the only person who is having this issue. Are you managing all tables with cassandra migration or is something else involved in the table migration? Are you able to produce a small spring project that reproduces the issue?

Hi. i am experiencing the same issue (race condition), on version 2.2.1_v4. because of v4, i cannot upgrade to 2.3. will you port the leader-selection change over to v4 branch as well?

That is actually my plan but like everybody else I am currently working remotely and have two kids to manage as well. I will try to find some time to do this but I can currently not give you an ETA.

Hi @ysheela,
I will look into this in the next couple of days and get back to you. Maybe I can provide you with a version where the consistency level for the lead can be configured, so you can try again. As this is a race condition, I do not assume you have a reproduce-able test case?


LinkGenerator doesn't match this rule

This rule is very useful but I have some methods that wrap LinkGenerator's methods. These methods don't match the rule.
For example:

LinkGenerator.GetUriByName, that return a string

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

ID: ad8d87c0-61af-3be4-553b-444974fdf329
Version Independent ID: 4124e17c-6f49-c7b0-6481-620be6ee1168
Content: CA1055: URI return values should not be strings (code analysis) - .NET
Content Source: docs/fundamentals/code-analysis/quality-rules/ca1055.md
Product: dotnet-fundamentals
GitHub Login: @gewarren
Microsoft Alias: gewarren

This issue was moved to dotnet/roslyn-analyzers#4450


fix: Add missing dash in private-key regex

Description:
The regex for private-key rule is missing the last dash which makes the finding unusable.
Checklist:

[ ] Does your PR pass tests?
[ ] Have you written new tests for your changes?
[ ] Have you lint your code locally prior to submission?

Not sure why the test is failing


quick and dirty probe-rs-tool read capability to store in a iHex file

I know this is not clean, and not the way to do it, but it is functional so I put it here for anyone to use if needed, because the "dump flash to iHex/Bin file" is something missing in probe-rs (I tryed to make it cleaner but would take much more time than I have for it, sorry)

Thanks for this, but what exactly do you expect us to do with this PR, if it's not in a good state, and you don't intend to spend the time to improve it?

maybe just close it, and somebody in the future will find it and improve it :/

I'm closing this for now, if you or somebody finds the time to finish this, then please reopen.
It would be a nice feature to have.


Add Infoblox DNS provider

What this PR does / why we need it:
Introducing Infoblox DNS provider
Release note:
Introducing Infoblox DNS provider

Thanks @bpressure for the guidance

/ok-to-test

Unit tests are working in our environment. I think it makes sense that it breaks here since there are no credentials and host to check. Regarding the chart I'm not sure if I should do anything more.

Any updates on this?

/lint

Unit tests are passing as expected.
=== RUN   TestInfobloxPresent
--- PASS: TestInfobloxPresent (0.64s)
=== RUN   TestInfobloxCleanUp
--- PASS: TestInfobloxCleanUp (10.22s)
PASS
ok      github.com/jetstack/cert-manager/pkg/issuer/acme/dns/infobloxdns        10.900s

Thank you for this PR. We're in the process of changing how we handle DNS01 providers that should make the process easier for users and us as maintainers. You can read more about how we plan to handle these here
Unfortunately, this means we will not be reviewing and accepting this PR for the moment. Once we have the webhook implementation up and running, we'll let you know how to set up and get this running.
You can follow along with our webhook implementation at PR #1443

Hey @DanielMorsing. I'm aware. I have already talked with @munnerz about this and I'm waiting for you to implement the Webhook solution so I can start writing the provider.

First of all, thanks for this pull request and for your time working on it.
As mentioned in the above comment, we have added a DNS01 provider extension
mechanism to cert-manager that allows out-of-tree DNS01 providers to be
built, including an example repository
to act as a template for how to implement your own providers.
At this time, we don't have the bandwidth nor means to maintain and automatically
test this DNS01 provider, and so I'm going to close this PR and ask that you
port your changes over to the new webhook example format in order to allow
anyone to use your new provider without needing to have it accepted into
the cert-manager codebase.
If in future your plugin has a wide user base and sufficient testing, we
may consider accepting it into the cert-manager project. The exact process
for this is still to be worked out, and we'd like to work with the community
to determine how we should best publish plugins for users :smile:
Again, thank you for your time, and I look forward to hearing your feedback
on the new DNS provider plugin feature!

Hey @munnerz, thanks for the clarification once again. :) Currently, we are not using this and I don't have much time to focus on writing the DNS01 provider based on the new architecture. If there is demand for that, I could address it in the future.


Controlling List of Extensions

Please add a reference to this page here, I expected to find it.

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

ID: 26b29b2f-783a-63a0-a1af-e2ce4d321387
Version Independent ID: 97597ce9-dc46-960f-6ad7-047bfe936d48
Content: VM extension management with Azure Arc-enabled servers - Azure Arc
Content Source: articles/azure-arc/servers/manage-vm-extensions.md
Service: azure-arc
Sub-service: azure-arc-servers
GitHub Login: @johnmarco
Microsoft Alias: johnmarc

#assign: johnmarco

Hello @gamullen. The change you requested is in progress and should be live in the next day or so. Thank you for your suggestion. I will now close this issue.

#please-close

@gamullen
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

[like]  Gary Mullen-Schultz reacted to your message:

From: TBalaji-MSFT @.>
Sent: Thursday, May 9, 2024 4:04:00 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Mention @.>; Author @.>
Subject: Re: [MicrosoftDocs/azure-docs] Controlling List of Extensions (Issue #122358)
@gamullenhttps://github.com/gamullen
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/122358#issuecomment-2102959020 or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALDK3HBOTYBTABQYQLHZNXLZBOM7DBFKMF2HI4TJMJ2XIZLTTGBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVE4DGNRXGAZDONBSURXGC3LFVFUGC427NRQWEZLMQKSXMYLMOVS2SOBTGY3TANBYGIZ2I3TBNVS2S2DBONPWYYLCMVWIFJLWMFWHKZNJHAZTMNZQGQ4TSNFENZQW2ZNJNBQXGX3MMFRGK3ECUV3GC3DVMWVDCNBQGQ4DSNJVGYYKI3TBNVS2S2DBONPWYYLCMVWIFJLWMFWHKZNKGE3DKOJZGYYDAOBSURXGC3LFVFUGC427NRQWEZLMQKSXMYLMOVS2UMJWGU4TSNRQGA4DHJDOMFWWLKLIMFZV63DBMJSWZLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOMIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVA3TENRYGUYDENUCUR2HS4DFUVUXG43VMWSXMYLMOVS2UMRSHA3TOMBXHEYTBAVEOR4XAZNFNRQWEZLMUV3GC3DVMWUTQMZWG4YDENZUGKBKI5DZOBS2K3DBMJSWZJLWMFWHKZNJHAZTMNZQGQ4DEM4CUR2HS4DFUVWGCYTFNSSXMYLMOVS2SOBTGY3TANBZHE2IFJDUPFYGLJLMMFRGK3FFOZQWY5LFVIYTIMBUHA4TKNJWGCBKI5DZOBS2K3DBMJSWZJLWMFWHKZNKGE3DKOJZGYYDAOBSQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRRGY2TSOJWGAYDQM5HORZGSZ3HMVZKMY3SMVQXIZI.
You are receiving this email because you were mentioned.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


feat: improve dev setup

Changes:

[x] make apply using parameterize value of DAGS_PATH, OPTIMUS_PLUGINS_PATH, and OPTIMUS_SERVE_PORT
[x] provide dummy echo plugin as an example
[x] add script to load secret

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.565%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7905

Relevant Lines:
9812

💛  - Coveralls


Create a CDC/chlorine Callibration Module

At the plant, the operators check chemical flow rates by timing how long it takes for the coagulant to drain from a fixed volume calibration column. They then perform this calculation:
CoagulantDose = CallibrationColumnVolume * CoagulantStockConcentration / CallibrationColumnTimeToEmpty / PlantFlowRate
However, the only variable that the operators have to measure regularly is the CallibrationColumnTimeToEmpty. If we could help them by presetting the other variables, and measure the time with a built in stopwatch, we may be able to dramatically reduce the time it takes operators to collect dosing information. Same equations/process apply to chlorine dosing.

Sorry wrong calculCoagulantDose = CallibrationColumnVolume * PlantFlowRate * CoagulantStockConcentration / CallibrationColumnTimeToEmpty

We want to launch this functionality straight from the odk collect app. You can learn how to launch an external intent with a regular xml form here and you can have the visualizations app respond to that launched external intent by installing this plugin. Here's a rather outdated blog post on manually modifying the app to listen for intents and here's a plugin that I believe does the same thing... I think the plugin is the simpler way to go, but the post would give some more information about how it actually works.

The operators change the dose setting by moving a slider that looks like this out (away from the entrance tank) to increase coagulant flow rate (toward '100') and in to decrease the flow rate. We want to ask them to record the position of the slider every time that they enter the coagulant. This information should be sent as a separate data point when the form gets sent. Note that this data can be used to create a relationship between slider position and chemical flow rate, and then that relationship can be used to suggest a specific slider location. Another potential use is to monitor how clogged the chemical tubing is, and recommend cleaning based on those numbers.

@Shangyuanniu it would be nice to have this finished, or at the very least have it in some sort of passable state before we part ways. Is there any chance you could upload what you have so far to this repo on another branch?

@asm278 started a repo to work on this called post.dose


Remove BDALogger dependency from CassandraDAO

CassandraDAO build fails due to its dependency to BDALogger.

fixed in remove-bdalogger branch


Advanced Tabs tabbarbackgroundcolor does not work!!!

tabbarbackgroundcolor does not work!!

I am also facing the same issue and have yet to find a solution.


Update typed client registration guidance

In points 7 & 8 of "Update Blazor WebAssembly" projects you provide guidance for transient http client registration, but not specifically for typed clients eg:
builder.Services.AddHttpClient<IMyService, MyService>(
                client => client.BaseAddress = new Uri(MyServiceApi))
                .AddHttpMessageHandler<MyServiceApiAuthorizationMessageHandler>();

which would be useful for those using same

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

ID: b031caa0-c62b-37da-46c3-59043256cac2
Version Independent ID: bdf98c9a-96f6-79fd-a650-5c0385d1a62b
Content: Migrate from ASP.NET Core 3.1 to 5.0
Content Source: aspnetcore/migration/31-to-50.md
Product: aspnet-core
Technology: aspnetcore-migration
GitHub Login: @Rick-Anderson
Microsoft Alias: scaddie

Hello @johndhunter ... That's because there are no default typed clients registered in a default WASM app scenario. However, there's nothing at all wrong with calling it out. I won't be back until Monday. If you want to add that to the topic, select the Edit button and submit a PR that has "Fixes #22842" in the opening comment. Add the code to a step (1. XXXXX) in the list with triple-backticks around it ... you'll see the other examples there when you edit the topic. You can ping me for review. Otherwise, I'll add it on Monday.

At the top right of the article, select the Edit pen icon:

Select the pen icon again. Save.

@johndhunter ... Come to think of it 🤔 ... typed-client registrations didn't change between 3.1 and 5.0 generally. What are you saying changed?

@johndhunter ... I'm closing because I'm not aware of a 3.1-to-5.0 change for typed-client registrations. If you clarify it, I'll re-open this issue.


⚠ Bofrost.nl has degraded performance

In fe28028, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1254 ms

Resolved: Bofrost.nl performance has improved in cba7014.


Update container-service-swarm-mode-walkthrough.md

Fix wrong location and typos.

@alexandair : Thanks for your contribution to the Azure documentation! The author, @neilpeterson, has been notified to review your proposed change.

@alexandair - thanks for the updates. Hope to run into you at IT/Dev this week.
#sign-off


Move to mutiny-wasm

Needs to be updated to new version of mutiny-core/wasm once this is merged and released: https://github.com/MutinyWallet/mutiny-node/pull/371

Think there should be a few compile issues, when you build, I have something locally I can push up

It's probably just because of the changes I needed in https://github.com/MutinyWallet/mutiny-node/pull/371 - will get that fixed whenever we do another release.

seems like the associated methods aren't in the wasm package perhaps?

oh my bad I didn't see the comments above

something seems wrong, this parameter isn't appearing either
https://github.com/MutinyWallet/mutiny-node/blob/master/mutiny-wasm/src/lib.rs#L210

Works now: https://2811bedf.mutiny-web-poc-reckless.pages.dev/


Implement Hot Exit

#14200 Hot Exit: Support hot exit on folder workspaces
#12400 Original hot exit PR
#14055 Hot Exit: avoid direct calls to fs.readdir()/fs.readdirSync() (aab05a86f5a959accce2fa388a36fb2cbc603f2b)
#14060 Correctly clean up workspace backups when exiting (873d095ac3555c1853a1636bb052b7fbaee8ea49)
#14100 & #14067 Have TextFileEditorModel handle restores (b23229f65dd68c594915650e0e2aed722cdaa401)
#14068 Hot Exit: do not load backup files to restore sync on startup in the renderer
#14065 Hot Exit: push renderer backup logic into new backup service
#14066 Hot Exit: backupService.ts feedback
Big refactor to throw as much as possible into the now several Backup*Services and leverage events
#14079 Remove public backup()
#14074 Hot Exit: Careful when you veto the shutdown from failing backups
#14181 Introduce on model dispose events to ITextFileEditorModelManager and IUntitledEditorService and track in BackupModelService
#14180 Don't hash untitled resource names for their backup paths
#14037 Merge backup path logic from node/fileService.ts into backupService.ts
#14064 Hot Exit: backupService saves and loads a LOT - Resources are no longer tracked at all in workspace.json and instead the actual backup path is checked
#14063 Hot Exit: deal with backup workspaces that do not exist anymore
#14056 Hot Exit: backup workspace handling should all happen on main side
#14330 Hot exit: Reloading the window should restore all files in the window
#14132 Hot Exit: restoring file model issues
#13305 Hot Exit: Support multiple folder restore (5bb2cb8470e2ef00de464beb042ec1866141311c, 57c6e72e5dacd8cfa72241a07039510d4f9bf6b5)
#15134 Hot exit on Mac currently allows closing the final single window
#15130 Cannot exit empty workspace windows on Windows when hot exit is enabled
#15137 Close folder does not clear the backup workspace path

@Tyriar there are merge conflicts?

I merged.

@Tyriar looking a lot better. I like the direction. My only concern besides the comments I added is about the amount of events that are being sent now while typing. See my comment in the review on that topic.

@bpasero please take another look if you get a chance, I've gone through all the issues that I feel need to be done before the initial merge. Tomorrow unless you find some more issues that need resolving I'll probably get the team in Redmond to spend some time testing it to make sure it's solid across several different configurations.

@Tyriar first round of feedback provided

@Tyriar replied to your comments, let me know when I should review again.

@bpasero I've responded to all the comments, if you consider any of these issues to be must fix before merge please label them with important. I want to polish the PR then merge it so we can start getting some testing from Insiders and move to smaller changes that are easier to work with.

@Tyriar only few comments left. my assumption is that you extracted the bigger feedback I gave into issues to look into and that is fine so that we can continue on master in these.

Fix warnings in https://github.com/Microsoft/vscode/pull/14115/commits/97371ada3da62980cf1a844e1c9070ce97ddab17 (not sure why I can't reply to that comment).

@bpasero yes we've been tracking all the comments in issues, the important ones have are in the November milestone.

Got the go ahead from @bpasero to merge, all tests are passing and did a manual smoke test on Mac and Windows to double check nothing has regressed since our internal test the other day. Merging! :fireworks:


Determine connection drain status async

Note: haven't seen this survive node drains as expected, not ready for review yet. See https://github.com/authzed/spicedb/pull/1283
This starts a worker that periodically (or on any connection error) asks Cockroach if any of its nodes are draining. If draining nodes are found, a shared draining flag is set, and only then do we check if connections are draining on acquire.
The goal is to remove the roundtrip from the common case, while still being able to make use of it when nodes are draining.

Closing in favor of #1294


a11y: fieldset and legend usage for form elements

What Should Happen
<fieldset> and <legend> should be used to group and describe related form elements, and should not be used for other reasons.
JIRA Issues

[x] MOVE-129
[x] MOVE-106
[x] MOVE-41
[x] MOVE-40
[x] MOVE-23

Punted on a few tasks related to the use of <fieldset> and <legend> as styling elements in Vuetify inputs with the outlined prop - changing this would be a lot of either design work (if we want to change to another form element style, such as the default "underlined" style, throughout our forms) or dev work (if we want to mimic the existing outlined style without <fieldset>, which would basically require us to build these components from the ground up.)


Cleanup nodeFunction R

Some time ago we renovated the nodeFunction construction system.  This PR removes now-unused code lying around from the old system.

:+1: Thanks for cleaning this up!


VS code creates empty jupyter notebooks

vscode creates four empty notebooks when opening  one notebook.

remote ssh connection to a linux machine.
version:
1.49.3
2af051012b66169dde0c4dfae3f5ef48f787ff69
x64

Thank you for your feedback @diff7.

Could you share with us the json content of your ipynb? (To see it in vscode, right click on your file and select Open With... -> Text Editor)
Please also share the content of the Python output tab in the Panel when the issue happens.


Looking for new maintainers

This project is not maintained and is looking for new maintainers

Better not use the word dead so we don't despair people.


HAWQ-53. Fix to handling nulls in complex data types in avro via pxf

+1

+1.
Please update automation tests to reflect this fix.

@hornn I've already created a pull request for the automation test https://stash.eng.pivotal.io/projects/HDQA/repos/pxf_automation/pull-requests/21


disabled download bundle button

Hid the download bundle button as this currently doesn't work for CQL-based measures.
Pull requests into Bonnie require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[X] This pull request describes why these changes were made.
[X] JIRA ticket for this PR: https://jira.mitre.org/browse/BONNIE-1107
[x] JIRA ticket links to this PR
[X] Code diff has been done and been reviewed
[X] If UI changes have been made, google WAVE plug-in has been executed to ensure no 508 issues were introduced.
[X] Tests are included and test edge cases
[X] Tests have been run locally and pass (remember to update Gemfile when applicable)
[x] Automated regression test(s) pass

If JIRA tests were used to supplement or replace automated tests:

[X] JIRA test links: N/A
[X] Justification for using JIRA tests: N/A
[X] JIRA tests have been added to sprint

Reviewer 1:

[x] Code is maintainable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

If JIRA tests were used to supplement or replace automated tests:

[ ] JIRA tests have been run and pass
[ ] You agree with the justification for use of JIRA tests or have provided input on why you disagree

Reviewer 2:

[ ] Code is maintainable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

If JIRA tests were used to supplement or replace automated tests:

[ ] JIRA tests have been run and pass
[ ] You agree with the justification for use of JIRA tests or have provided input on why you disagree

Wasn't there discussion of disabling the endpoint in the backend?

I certainly can do the changes you two requested. I carefully added unit tests that show the button is hidden and the route is gone - you can't activate the back-end service.
Since this is a feature that is merely being temporarily disabled - we need to add the button back in - I chose the less invasive changes. Do you disagree that my tests are complete? If we are going to just rip all this code out, we need to make sure it is documented for Cypress because they will need to undo the entire PR.

(Regarding my previous comment) Maybe the files don't need to be deleted if there is a way to keep the server from sending the modal to the client.

@mayerm94 okay, removed modal from measures view; note the model is still in the users view, which seems fine since admins don't care if we have a temporarily broken button
@alexanderelliott121 does test "bundle download" in users_controller_test.rb satisfy your concern?

just rebased

Merged in cql4bonnie and fixed 3 warnings


Android Browser: has no method 'animationRequestFrame'

Note: for support questions, please use one of these channels: Chat: AngularClass.slack or Twitter: @AngularClass

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
Android Browser and IE 9 throw error '...has no method 'animationRequestFrame'

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
Just run the Angular 2 Webpack Starter on the native Android Browser on Lollipop

What is the expected behavior?
Should run

What is the motivation / use case for changing the behavior?
Customer are criticizing quality of the WebApp because it's not running on their devices.

Please tell us about your environment:

Angular version: 2.0.0-beta.X
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

I addded a solution for this.
Add this to the index.html
<script<EMAIL_ADDRESS>

thanks, I left ie-shim commented out for this reason

Where can I find it?
Greets
Smith
Von meinem iPhone gesendet

Am 19.06.2016 um 07:14 schrieb PatrickJS<EMAIL_ADDRESS>thanks, I left ie-shim commented out for this reason
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

in polyfills commented out
https://github.com/AngularClass/angular2-webpack-starter/blob/master/src/polyfills.browser.ts#L3

@gdi2290 thanks bro!


feat: use events to listen to token changes

Describe the changes you have made in this PR
listen for token changes via alby-js-sdk events
Type of change
(Remove other not matching type)

feat: New feature (non-breaking change which adds functionality)

How has this been tested?
trying oauth in extension
Checklist

[X] Self-review of changed code
[X] Manual testing
[X] Added automated tests where applicable
[X] Update Docs & Guides
For UI-related changes
[X] Darkmode
[X] Responsive layout

@rolznz any updates here?

closing in favour of #2830 as there is lots of dependency conflict in yarn.lock


session: Support protocol changes related to sidechain magic

There is a challenge in version compatibility control. After discussion we've decided to postpone sidechain magic feature after the error system's intro.

This PR not broken, though, unlike others. We can keep it, in this case.

Nevermind, it is broken too :face_exhaling:


onbindviewholder is called when list is empty

When I implement filterable and got an empty arraylist, onbindviewholder/onplacesubheaderbetweenitems are called, providing position 0 (causing indexoutofbound). It didn't happen in the default adapter. How can I solve this?

@hctibs678  Can you create some sample app which demonstrates the issue because I can't recommend anything because I don't know where the bug is.

@zhukic I have created a sample app demonstrating the problem.
https://github.com/hctibs678/testproject
Thanks!

@hctibs678 You need to call notifyDataChanged method instead of notifyDataSetChanged.
It is required to recalculate the state of sections.

@zhukic After using notifyDataChanged, everything work fine. Thanks a lot for helping!


Fix ternary

Description of the PR
Fix ternary in KlasaConsole
Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)

Semver Classification

[ ] This PR only includes documentation or non-code changes.
[x] This PR fixes a bug and does not change the (intended) framework interface.
[ ] This PR adds methods or properties to the framework interface.
[ ] This PR removes or renames methods or properties in the framework interface.

No changes


Fixed the RaisedButton shadow issue

The onHighlightChanged callback was being given "true" every time
instead of the actual value.

LGTM


Update README.md

cf cups accepts JSON format, not comma-separated list; this created some confusion

Fixed the README.md

Fixed the README.md


Using mapillary-js

From #1, @kepta lets figure out how to integrate mapillary-js for providing more context of photos to mappers.
The objective is to help a user establish the correct position and orientation of a photograph on the map.

Integration should be pretty straight forward https://github.com/mapillary/mapillary-js
moveToKey -> https://mapillary.github.io/mapillary-js/classes/viewer.html#movetokey is really the key here

We have this now.


Add Observable.fromIteratorChunkedUnsafe

Posting @deusaquilus from gitter:

I did some profiling of the Quill performance issue. What we really need is a Observeable.fromIteratorChunkedUnsafe that basically assumes the Iterator.next call will (usually) not block for a specified N elements, and that the most efficient course of action is to fetch the next N into a chunk (i.e. buffer), over and over until the end of the result set. This is basically what the JDBC ResultSet does via the FetchSize property (here's an example: https://docs.oracle.com/middleware/1212/toplink/TLJPA/q_jdbc_fetch_size.htm#TLJPA647) which I then write into an iterator.

Not sure if you still need it, but it could be useful to have

Oh yes I certainly still do :)
On Thu, Sep 24, 2020 at 12:02 PM Piotr Gawryś<EMAIL_ADDRESS>wrote:

Posting @deusaquilus https://github.com/deusaquilus from gitter:
I did some profiling of the Quill performance issue. What we really need
is a Observeable.fromIteratorChunkedUnsafe that basically assumes the
Iterator.next call will (usually) not block for a specified N elements, and
that the most efficient course of action is to fetch the next N into a
chunk (i.e. buffer), over and over until the end of the result set. This is
basically what the JDBC ResultSet does via the FetchSize property (here's
an example:
https://docs.oracle.com/middleware/1212/toplink/TLJPA/q_jdbc_fetch_size.htm#TLJPA647)
which I then write into an iterator.
Not sure if you still need it, but it could be useful to have
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/monix/monix/issues/1275, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKOLCC7Z3N2TPWS4OFLYY3SHNUQLANCNFSM4RYPOIZA
.

No problem, I'll make sure to include it in the next release though!


☂ ITP: Work out a plan for getting sufficient pair programming time

Background
Every sprint of ITP (except the very first one), every trainee is assigned an issue called "Pair Programming": Spend an hour working with a volunteer on something.
Example ticket: https://github.com/CodeYourFuture/Module-User-Focused-Data/issues/95
For most of the course, the suggested work to do is a codewars kata. For the very first module, because they haven't learnt any JavaScript yet, the suggestion is to spend the time pairing on any code-writing exercise they are doing (probably either an HTML+CSS project or a Google Sheets task).
What a volunteer needs in order to perform this task

Enough technical knowledge to support the trainee in solving a problem.
Ideally either experience in the programming language they're using, or significant (>5 years) professional experience coding.
Ideally enough mentoring skills that they will almost never tell the trainee the answer to a problem, but will help them learn.
1 hour of dedicated time in a single block.
Remote.

Ideally the volunteer would know roughly what the trainees are expected know at the stage of ITP (so they don't tell trainees they're doing something wrong because they're not using something they don't know about yet), but also, learning new more advanced techniques is sometimes appropriate and useful.
Note: There is no assessment part of this work. The goals are to help the trainees learn, and to give the trainees experience working with others.
How much do we need
1 hour per trainee per sprint.
How should trainees and volunteers match-make and coordinate
To be decided.
Probably we shouldn't have the same trainees and the same volunteers paired up regularly. We will have a mix of mentoring skill levels which we want to distribute, and also people benefit from different learning styles/interactions.
Currently we have two channels in the ITP slack: #cyf-codewars and #cyf-pair-programming. Some people have offered office hours in the codewars canvas. Others are making ad-hoc threads in #cyf-pair-programming.
How can we track this?
To be decided.
Ideally trainees would know this is expected, want it to happen, and be able to escalate when they're not getting it.
Acceptance criteria
Every sprint, 80% of ITP trainees have 1 hour of a pair programming.
In 80% of these pair programming interactions, the trainee feels like they learned something or found the experience valuable.
Volunteers are generally happy to keep doing these sessions.
Owner
??
Stakeholder
@illicitonion
Approver
@illicitonion

We discussed some of the logistics of this today, and settled on "every volunteer should have a calendly or similar" I suspect at some point we'll hit scale issues (checking 20 calendlys to find an available slot isn't ideal), but we can cross that bridge when we come to it.
I've split this into https://github.com/CodeYourFuture/Ops-Programme/issues/136 for the "recruiting" piece, https://github.com/CodeYourFuture/curriculum/pull/1077 for the guidance piece, and https://github.com/CodeYourFuture/curriculum/issues/1081 for simplifying maintainable trainee instructions. That's probably sufficient for now.

Oh I should make a pitch slide for the volunteer onboarding too.

Left a comment on https://docs.google.com/presentation/d/1rawXCCKbTj7t0Em48X1vxw3PxSYsVLYwAxDqsOgHsVU/edit#slide=id.g30abc708696_0_14 with information to include in the pitch slide.


opendal doesn't support  Azure Data Lake Storage Gen2

    let account = "account_name";
    let container = "container_name";
    let access_key = "access_key";
    let endpoint = "endpoint_url";

    let mut builder: Builder = azblob::Builder::default();

    builder.root(&env::var("OPENDAL_AZBLOB_ROOT").unwrap_or_else(|_| "/".to_string()));
    builder.container(container);
    builder.endpoint(endpoint);
    builder.account_name(account);
    builder.account_key(access_key);

    let op: Operator = Operator::new(builder.build()?);

    // Working
    let path = "some.txt";
    info!("try to read file: {}", &path);
    let _content = op.object(&path).read().await?;
    info!(
        "read file successful"
    );

    // Not working
    let path = "";
    info!("List directory at: {}", &path);
    let mut os = op.object(path).list().await?;
    while let Some(entry) = os.try_next().await? {
        let path = entry.path();
        info!(
            "Found path: {}",
            path
        );
        let mode = entry.mode();
        info!(
            "Mode for path: {}",
            mode
        );
    }

The example code for azblob via builder method and local setup using azurite is working flawlessly for list operation but with azure credentials it is failing with status code 400 only for list operation. The read operation is working without any issue.
Here is the error:
{ service: azblob, path: / } => status code: 400, headers: {"content-length": "221", "content-type": "application/json;charset=utf-8", "server": "Windows-Azure-HDFS/1.0 Microsoft-HTTPAPI/2.0", "x-ms-request-id": "919d2243-601f-0074-1beb-ff1f9f000000", "date": "Thu, 24 Nov 2022 09:59:27 GMT"}, body: "{\"error\":{\"code\":\"InvalidQueryParameterValue\",\"message\":\"Value for one of the query parameters specified in the request URI is invalid.\\nRequestId:919d2243-601f-0074-1beb-ff1f9f000000\\nTime:2022-11-24T09:59:28.0676530Z\"}}"

Thanks for the report!
Based on the error message:

"server": "Windows-Azure-HDFS/1.0 Microsoft-HTTPAPI/2.0", "x-ms-request-id": "919d2243-601f-0074-1beb-ff1f9f000000", "date": "Thu, 24 Nov 2022 09:59:27 GMT"}, body: "{"error":{"code":"InvalidQueryParameterValue","message":"Value for one of the query parameters specified in the request URI is invalid.\nRequestId:919d2243-601f-0074-1beb-ff1f9f000000\nTime:2022-11-24T09:59:28.0676530Z"}

What's the storage service are you using? Seems it's not Azure Blob storage? Maybe  Azure Data Lake Storage Gen2?
Can you provide the endpoint you use (please remove the account_id first)?

Sure the endpoint is https://account_name.dfs.core.windows.net. I believe you are correct, it is not Azure Blob storage. looking at Windows-Azure-HDFS felt i should use hdfs service, but then going through the example and documentation says otherwise, also the azure credentials I have, confused me.

Sure the endpoint is https://account_name.dfs.core.windows.net.

I believe it's https://learn.microsoft.com/en-us/rest/api/storageservices/data-lake-storage-gen2
data lake storage gen2 shares the same credential with azblob, but they have different API sets. I will add this service support next week. Please keep in tune!

Awesome. Thanks a ton!

Hi, @akd-pie, azdfs service has been implemented. Please give azdfs service a try, related docs can be found here: https://opendal.databend.rs/opendal/services/azdfs/index.html


[Bug] [connector-v2] PostgreSQL versions below 9.5 are compatible use cdc sync problem

https://github.com/apache/seatunnel/issues/5110
…h CDC

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Thanks for your contribution. It would be better if you changed the title of this pr. Is there any mention of this in the PostgreSQL documentation?

PostgreSql 9.5 version document https://www.postgresql.org/about/press/presskit95/ru/ mention

Please fix code style. Refer : https://github.com/apache/seatunnel/blob/02114db626a290502d33797dfbe9a11ee5b52b6f/docs/en/contribution/coding-guide.md

use mvn spotless:apply format code success,
add doc

@Hisoka-X


Update meme.css

New color style added in h1, background and url tags. :smile: :+1:

Update new color style in meme.css file :smile:  :+1:


[cleanup] [broker] when serverCnx disabled auto read use same implements

Motivation
https://github.com/apache/pulsar/blob/732049fc6ca1beb046deb43057be2b130736fbca/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java#L2645-L2652
L2468-L2469 make serverCnx disabled auto read, but there has a beter implements at L2723-L2728:
https://github.com/apache/pulsar/blob/732049fc6ca1beb046deb43057be2b130736fbca/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java#L2723-L2728
At the better implements, can reduce false statistics( when a serverCnx already disabled auto read, will not increment another once).
Documentation

[ ] doc-required
[x] no-need-doc
[ ] doc
[ ] doc-added

@michaeljmarshall could you take a look

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

/pulsarbot run-failure-checks

Sorry @315157973, accidentally re-requested your review.

@lhotari Could you help me, I do not know what to do.

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

/pulsarbot run-failure-checks

@gaoran10 Could you take a look

/pulsarbot run-failure-checks

@poorbarcode Please rebase to the master branch.

@poorbarcode Please rebase to the master branch.

ok


[kjobctl] Add list RayJob command.

What type of PR is this?
/kind feature

What this PR does / why we need it:
Add list RayJob command on kjobctl.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

lgtm
I've payed most attention to the tests, they look good

/approve
@trasc would you like to give the final pass?


Package sorting version list

Issue number:
Closes # n/a
Description of changes:
Adds sorting to package versions on /<lang>/os/<minor ver>/version-information/packages/<patch ver>/ in a template. See PR #280
Terms of contribution:
By submitting this pull request, I confirm that my contribution is made under
the terms of the licenses outlined in the LICENSE-SUMMARY file.

Thanks! :cake:


Issue/84

Ok, my attempt at the same thing. Sorry, commit is big due to clean-ups. What's mainly missing is ListView.selection.items. You can ignore the examples (they are just clean-up).
Fixes #84

Looking at https://github.com/Sciss/SwingPlus now to see how far we get with the new version. It seems that the different collection type for insertAll is very unfortunate, and thus it would be best to add a type alias to package.scala

I managed to build my entire desktop application tree now; from what I can see, all seems to be fine, so this PR is ready for review. Anyone? @SethTisue @benhutchison @som-snytt

created branch work from 2.0.x which can serve as basis for further development. Since versions are tagged, I don't think we need one branch per each new version. Updated README accordingly.

Agree that we should keep minimum branches, I like that there's just one branch to worry about for the future.

I probably won't be able to look until January, perhaps @lrytz could take a look

All good from me

@lrytz ok - so I removed swing.Seq; I took the opportunity, though, to move to immutable.Seq now where possible and reasonable.

If no more objection, I will merge this tonight, and then bump version to stable 2.1.0 -- who would be responsible for pushing the new artifacts to Maven Central?

I think that's great! This sounds like it should go in a major release though?

I did this since we are already on a new major version 2.1.0 (previous is 2.0.x) due to several changes.

Great! I can click the buttons to release the artifacts to maven central, just ping me.

@benhutchison are you familiar with https://github.com/scala/scala-swing/tree/work/admin#tag-driven-releasing - if so, could you bump to 2.1.0 stable, tag and publish?

Done. The releases have gone to Sonatype staging and all looks normal.
@lrytz https://github.com/lrytz  please release.
Note Im heading on post-Xmas family beach vacation (it's summer in
Australia) today, packing the car now, so will be only online
intermittently for next week.
On Thu, Dec 27, 2018 at 12:31 AM Hanns Holger Rutz<EMAIL_ADDRESS>wrote:

@benhutchison https://github.com/benhutchison are you familiar with
https://github.com/scala/scala-swing/tree/work/admin#tag-driven-releasing

if so, could you bump to 2.1.0 stable, tag and publish?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scala/scala-swing/pull/86#issuecomment-449967028, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAF05DUbbOjEqbp8ZBBDxe9VavarR1vEks5u83oWgaJpZM4ZPrCO
.

Note that the version in build.sbt that has the git-tag v2.1.0 still says "2.1.0-SNAPSHOT". So probably it needs another commit with that incremented.

Good catch Sciss. I've packed my laptop in car, please make change if you
can.
On Thu., 27 Dec. 2018, 9:32 am Hanns Holger Rutz<EMAIL_ADDRESS>wrote:

Note that the version in build.sbt that has the git-tag v2.1.0 still says
"2.1.0-SNAPSHOT". So probably it needs another commit with that
incremented.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scala/scala-swing/pull/86#issuecomment-450035675, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAF05D5w3HRJNWWIoezNkELPdtJM5bh0ks5u8_j-gaJpZM4ZPrCO
.

@lrytz Lukas, could you check in to sonatype and release the artifacts? Double check, because since I moved the v2.1.0 tag to the new commit with the stable version in build.sbt, Travis published again, so I don't know if it just overwrites the previously staged files or not.
As far as I can tell, Ben's release didn't make it to Maven Central yet:
https://search.maven.org/search?q=g:org.scala-lang.modules AND a:scala-swing_2.12&core=gav

It's no big deal, but I don't think there was any need to move the v2.1.0 tag. The version in the published artifacts is determined by the tag, overriding the version setting, so it's fine to bump that setting after publishing.
Anyway, I don't understand exactly what happened, but somehow we ended up in a confused situation where the staging repos ended up with a mix of artifacts from the two different attempts, which different checksums.
It seems likely to me that the artifacts from either attempt are fine, but the whole thing makes me nervous. So I dropped all four staging repos (1658 through 1661), deleted the v2.1.0 tag, and then re-pushed the tag.
The new Travis job is https://travis-ci.org/scala/scala-swing/builds/473049546

@SethTisue ok thank you. The problem with the alignment of the tag is also that, for example, in GitHub release you download attached sources that don't correspond with the maven sources, because version was still 2.1.0-SNAPSHOT.
I don't know what the general procedure is next time, but I think it's best to match build.sbt version with git tag. ?

I don't know what the general procedure is next time, but I think it's best to match build.sbt version with git tag. ?

It certainly can't hurt!

I would follow roughly this: https://github.com/sbt/sbt-release#release-process
So, after release, merge to master, then push another commit to have the next SNAPSHOT in work.

staging repos 1662 and 1663 closed and released, artifacts should be on their way to Maven Central

https://repo1.maven.org/maven2/org/scala-lang/modules/scala-swing_2.12/2.1.0/
https://repo1.maven.org/maven2/org/scala-lang/modules/scala-swing_2.11/2.1.0/
https://repo1.maven.org/maven2/org/scala-lang/modules/scala-swing_2.13.0-M5/2.1.0/