Offset of sliced Array is 0

Describe the bug
In Python with pyarrow, we can check that an array has been sliced by checking its offset:
import pyarrow as pa
arr = pa.array([1, 2, 3, 4])
arr.offset
# 0
sliced = arr.slice(2, 2)
sliced.offset
# 2

However it seems that in arrow-rs offset is hard-coded to be 0:
https://github.com/apache/arrow-rs/blob/80ed7128510bac114c6feec08c34ef3beed3a44a/arrow-array/src/array/primitive_array.rs#L1112-L1114
So that means this simple test fails:
#[cfg(test)]
mod test {
    use arrow::datatypes::Int64Type;
    use arrow_array::{Array, PrimitiveArray};

    #[test]
    fn test_offset() {
        let arr: PrimitiveArray<Int64Type> = PrimitiveArray::from(vec![1, 2, 3, 4]);
        let offset = 2;
        let sliced = arr.slice(offset, 2);
        assert_eq!(sliced.offset(), offset);
    }
}

assertion `left == right` failed
  left: 0
 right: 2

This seems like a bug, or am I missing something?
To Reproduce
Above test.
Expected behavior
Expected array.offset() to reflect the array's slice.
Additional context
This was found while researching https://github.com/apache/arrow-rs/issues/6151, and is somewhat related.

I'm guessing the reason is we slice the underlying buffer here:
https://github.com/apache/arrow-rs/blob/80ed7128510bac114c6feec08c34ef3beed3a44a/arrow-array/src/array/primitive_array.rs#L747-L748
It does make me wonder if we should just remove offset from the Array and expose APIs on the buffers themselves for seeing if it's sliced. Is there a use case you have for wanting to know this offset?

In a general sense, it can be useful to know "is the array at hand sliced". In the specific task at hand, in https://github.com/kylebarron/arro3 I'm trying to match core pyarrow APIs using arrow-rs. And so I'm trying to match pyarrow's offset attribute.

And so I'm trying to match pyarrow's offset attribute

I don't think this is possible, arrow-rs handles slicing at the granularity of buffers not arrays

It does make me wonder if we should just remove offset from the Array

IIRC it gets used for BooleanArray


Add MaterialBottomSheet control

https://m3.material.io/components/bottom-sheets/overview

Hi, would be really happy to see it also🔥

Hi @SnowPowerCore , we're currently developing an initial, basic iteration of this control, set to be included in the forthcoming release of the NuGet package.

That is amazing to hear!


[9.x] Reconnect when redis throw read error exception

ref to issue https://github.com/laravel/framework/issues/41651, and adds a match for 'read error on connection' to trigger a reconnection.

Can we have this fix for Laravel 8.X as well? @taylorotwell

@leexin Laravel v8 isn't maintained anymore. Please upgrade to Laravel v9.


website: render approved GEPs in chronological order

What type of PR is this?

/kind documentation
What this PR does / why we need it:
This will order the navigation menu for approved GEPs.
Fix is using the support from the already installed mkdocs-awesome-pages-plugin
configured as it is given here:  https://github.com/lukasgeiter/mkdocs-awesome-pages-plugin#customize-navigation
Now it order the nav items in chronological order:

Which issue(s) this PR fixes:

Fixes https://github.com/kubernetes-sigs/gateway-api/issues/1428
Does this PR introduce a user-facing change?:

NONE

Thanks for finding a way to do this @Amila-Rukshan!
I tried experimenting with using the order and sort_type: natural options from the docs you linked (instead of listing them manually), but it seemed to not work as expected (four-digit GEPs with a leading 1 sorted as less than three-digit GEPs) unless I added a leading zero to the GEP numbers in filenames 😕

@mikemorris I also tried sort_type: natural and didn't notice what you have noticed though.


Error to create myResouceGroup myCluster

Hi, everybody!
When I tried to create a ResourceGroup running the following command it's happening this error.
Please see the file .txt in attachment.
Thanks for advance.
Error to create myResouceGroup.txt

After new try I got it.

Other errors related:
Azure:/
PS Azure:> az aks list –o table
az: error: unrecognized arguments: –o table
usage: az [-h] [--verbose] [--debug]
[--output {json,jsonc,table,tsv,yaml,none}] [--query JMESPATH]
{aks} ...
PS Azure:> az aks install-cli
Downloading client to "/usr/local/bin/kubectl" from "https://storage.googleapis.com/kubernetes-release/release/v1.17.0/bin/linux/amd64/kubectl"
Connection error while attempting to download client ([Errno 13] Permission denied: '/usr/local/bin/kubectl')

@Azure/aks-pm please take a look!

@JasonVolney Apologies for the late reply. This issue is open for quite sometime now. Please let us know if you need any further assistance on this. Awaiting your reply.

@JasonVolney Apologies for the late reply. This issue is open for quite sometime now. Please let us know if you need any further assistance on this. Awaiting your reply.

Hi, @navba-MSFT Thanks you so much. But I don't need support for that no more.

@JasonVolney Thanks for your reply. We will now proceed with closure of this github issue. If you need any further assistance on this issue in future, please feel free to reopen this thread. We would be happy to help.


Spell checker support

It would be great to have spell checker support for proto files. A stack overflow answer makes it look like support must explicitly added to the plugin.

@kshchepanovskyi It would be great to have basic spellchecking in proto files, it's a bit confusing that it's absent


refactor: cleanup export flags

This PR cleans up all export flags so that:

The code is more readable as it's more broken up by function.
We could move these options to YAML one day and update them at runtime instead of at startup.

Changes:

Ensures all flag values have their own struct.
Move flag validation and defaulting to their own methods.
Ensure all flag structs have only serializable values (e.g. ExporterOpts.Lease was moved out).
Use URLs with kingpin (flag library) to avoid needing to parse strings manually.

I also broke up the monstrosity that was the setup package exporter constructor which returned a function that would create an exporter. The reason this was done was because first we setup flags and then returned a method that would create the exporter once the flags were parsed. Now you can just call SetupFlags and then call exporter.New.
Tested a Prometheus version built with these changes with e2e tests, and they pass.

For ease of making this PR easier to review, I split up the rule-evaluator refactors to this PR: https://github.com/GoogleCloudPlatform/prometheus-engine/pull/1058


feat: cleanup some Nat lemmas

Depends on #182
Potentially breaking changes:

Changed Nat.mul_lt_mul (1 use in Mathlib). See comments below.
Made two parameters implicit in Nat.lt_add_right (4 uses in Mathlib)
Made parameters implicit for Nat.add_le_add_iff_left (no uses in Mathlib)
Made parameters implicit for Nat.add_le_add_iff_right (no uses in Mathlib)
Made parameters implicit for Nat.add_lt_add_iff_left (no uses in Mathlib)
Made parameters implicit for Nat.add_lt_add_iff_right (no uses in Mathlib)
Made parameters implicit for Nat.dvd_iff_mod_eq_zero (16 uses in Mathlib)
Renamed Nat.max_le (no uses in Mathlib) to Nat.max_le_iff and added new Nat.max_le (Mathlib conformance)
Renamed Nat.le_min (no uses in Mathlib)  to Nat.le_min_iff  and added new Nat.le_min (Mathlib conformance)
Protected Nat.zero_le (unknown uses in Mathlib; at least one hiding clause)
Protected Nat.one_pos (unknown uses in Mathlib)
Protected Nat.two_pos (no uses in Mathlib)
Deprecated Nat.lt_min (no uses in Mathlib)
Deprecated Nat.add_le_to_le_sub (no uses in Mathlib)
Deprecated Nat.succ_add_eq_succ_add (4 uses in Mathlib)
Deprecated Nat.mul_le_mul_of_nonneg_left (4 uses in Mathlib)
Deprecated Nat.mul_le_mul_of_nonneg_right (3 uses in Mathlib)
Deprecated Nat.mul_lt_mul' (2 uses in Mathlib)
Deprecated Nat.zero_max (1 use in Mathlib)
Deprecated Nat.zero_min (no uses in Mathlib)
Deprecated Nat.max_zero (no uses in Mathlib)
Deprecated Nat.min_zero (no uses in Mathlib)

Deprecated lemmas all have recommended replacements.

This is a good stopping point. I did only a few small additions. The main one are some custom recursors in Std.Data.Nat.Basic. I'm happy to hear reviews from anyone who has time.

Regarding #118: I'm tempted to rename Nat.mul_lt_mul₀ to just Nat.mul_lt_mul. There are some issues:

On the one hand, Mathlib has mul_lt_mul₀ in Mathlib.Algebra.Order.WithZero so it makes sense to use the same name. On the other hand, like every theorem in that file, the subscript zero indicates WithZero which doesn't make much sense as a distinction in Nat.
On the one hand, there is a deprecated Nat.mul_lt_mul with one use in Mathlib. But the old Nat.mul_lt_mul was a bad theorem with unnecessarily strong hypotheses, so it's a good riddance.

I finally opted for Nat.mul_lt_mul over Nat.mul_lt_mul₀. What tipped the scale for me is that the old Nat.mul_lt_mul was just a bad theorem. Existing uses can be replaced by Nat.mul_lt_mul_of_lt_of_le after adapting the hypotheses.

Now broken up into smaller pieces.


Fix #47

Hi. Fix for the above issue. I hope the code and comments are reasonably clear.
Cause. The main decode loop cannot differentiate between an invalid decode input and a destination buffer that is too small. When it encounters an invalid input it will exponentially malloc until it can malloc no more, at which point it fails with the dreaded malloc: Cannot allocate memory error.
Fix. We fast fail zero-length inputs, which are always invalid. This obliviates the allocation of zero destination buffers. We can now interpret a zero lzfse_decode_buffer return value as a decode failure. The API docs are a bit vague in regards to invalid inputs: but as we can see here, lzfse_decode_buffer will return a zero value in the case of failure.

Is this related to #44 ?

On further inspection, this patch introduces an error when decoding zero-length files. There is another workaround, but it's more involved.


Added configurable drupal_username_prefix

Thanks! Two things needed here:

add a test for this functionality
fix up code to meet PSR-2 standards (lns 56-58)

Will take a look at this as soon as I get some spare time - thanks again.

Thnx, will do it soon
On 29/01/15 15:29, Paul Byrne wrote:

Thanks! Two things needed here:

add a test for this functionality
fix up code to meet PSR-2 standards (lns 56-58)

Will take a look at this as soon as I get some spare time - thanks again.
—
Reply to this email directly or view it on GitHub
https://github.com/pfaocle/codeception-module-drupal-user-registry/pull/8#issuecomment-72033164.

Met vriendelijke groet,
Ralph van den Houdt
(werkdagen: di, wo, do)
--
ezCompany BV
Tivolistraat 2
5017 HP  Tilburg
T +31 13 5322878
F +31 13 5322879
www.ezcompany.nl

I'm taking a look at the moment - I've fixed up PSR-2 and the current tests - replacing the constant for the prefix with a variable broke a test, but that part needed looking at anyway!
Will see if I get chance this evening to add a test myself for this and I'll merge. Many thanks!

Hi - I am closing this in favour of PR #9
Feel free to contribute towards that PR but otherwise I'm happy to merge that shortly.


Dark mode is not working

Something changed in libadawata that made dark mode not work again

Can confirm dark mode is working now post fix on my machine.


Fixed incorrect path returning on file save

Removed "File Successfully Saved " from result.success
Fixed incorrect path returning in some mobile phones(Ex. Xiaomi devices)

Thank You so much for your hard work, it seems you're pro in native development, can you help me save the file in downloads folder in android 11?

Sure @incrediblezayed


Weird artifact on some hardwares

See https://codepen.io/anon/pen/yZpXZE?editors=1010 which results in something like

(noticed that weird final red line segment) on some (macOS?) hardware.
See https://github.com/plotly/plotly.js/issues/3522 for more details.

That can be related with wrong handling of tail vertices. I noticed similar artifact too in related component gl-waveform.
I'd estimate that 3-4 hours of work.

Let me know if there's anything else I can do to help test it out!


Replace XML toolbox/block definitions with JSON equivalents

The Data toolbox category in particular uses XML definitions, and the general toolbox definition in scratch-gui also uses XML. These should be converted to the modernized JSON definitions.

If you can obsolete make-toolbox-xml.js, I'll owe you cookies!  I'd owe you MORE cookies, I guess.
As long as project compatibility is maintained and extensions can still be loaded, I'm happy with XML or JSON, whichever works for you. If you have a strong argument for YAML or Protocol Buffers or whatever, that's probably OK too :)
This does feel like it could break compatibility for some folks using our VM, but we're just earning that major version bump, right?
Would this mean that ConvertedBlockInfo (extension-manager.js) could store only JSON instead of both XML and JSON?


remove AddonUpsell model and table (fixes #965)

fixes #965

Nice discovery, it's really not used anywhere! r+ :smiley:


Add "Get Followed Streams" feature

New feature implemented by Twitch the 2021‑04‑20

Pull Request Test Coverage Report for Build 772143421

9 of 11 (81.82%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 86.659%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

streams.go
9
11
81.82%

Totals

Change from base Build 768426408:
-0.06%

Covered Lines:
799

Relevant Lines:
922

💛  - Coveralls


master: linux: Create 5.19 kernel based recipe for sa8155p adp

Create 5.19 kernel based recipe for sa8155p adp
board. The rational behind the same is that
master support for sa8155p adp board would always
try to support the latest upstream kernel version
tested on the board.
So, while by default the releases for adp will still
be based on '5.15 + OE dunfell', the newer snapshots will
also be available based on '5.19 + OE master'.
Signed-off-by: Bhupesh Sharma<EMAIL_ADDRESS>

Self NACK'ing this one. This will be used to point to qcom as a reference for writing sample recipes though.


[NRCE] Actor Node API

I'm interested to hear your thoughts about the current API.
I will describe it here for the single nodes. This could later be used for the documentation.

the actor node accepts the output of the input node (#113 )

further, it accepts "DEL" and "LRN" messages. to start the learn and delete mode. in this mode, it will accept teach-in telegrams and will either add the new telegram if it does not exist or delete it if it does.
duration is the time in seconds the node will listen for incoming teach-in telegrams
{
    "type": "LRN",
    "duration": 3
}

{
    "type": "DEL",
    "duration": 3
}

you can also use "LRN" and "DEL" to manually teach in sensors:
{
    "type": "LRN",
    "senderId": "002a1d7e",
    "eep": "f6-02-03",
    "name": "Button5"
}

output
it emits messages with decoded data of known sensors. a known sensor is a sensor that was previously taught in.
{
  "payload": {
    "LRNB": {
      "name": "LRN Bit",
      "unit":"",
      "rawValue":1,
      "value":"1",
      "description": "Data telegram"
    },
    "HUM": {
      "name": "Humidity",
      "rawValue":106,
      "value": 42.400000000000006,
      "range": {
        "min": "0",
        "max": "250"
      },
      "scale": {
        "min": "0",
        "max": "100"
      },
      "unit": "%"
    },
    "TMP": {
      "name": "Temperature",
      "rawValue": 135,
      "value": 23.200000000000003,
      "range": {
        "min": "0",
        "max": "250"
      },
      "scale": {
        "min": "-20",
        "max": "+60"
      },
      "unit": "°C"
    },
    "TSN": {
      "name": "T-Sensor",
      "unit": "",
      "rawValue": 1,
      "value": "1",
      "description":"available"
    }
  },
  "meta":{
    "type": "radio-erp1",
    "rorg":165,
    "senderId":"01a47f23",
    "destinationId":"ffffffff",
    "subTelNum":1,
    "rssi":57,
    "timestamp":1557720491493,
    "eep":"a5-04-02",
    "payload":"006a870a",
    "name":"Sensor2",
    "actorName":""
  },
  "_msgid":"dea90ee.4b9f5f"
}

the output is generated according to the eep specification

the API is frozen, but the values returned in the field descriptions might change when I find bugs.
The values in the "range" and "scale" field are strings but really should be numbers (#56)
so do not assume types yet.


 Please describe.
there is a "live player cam" a camera of the player with a "sad" seal image on the top left
it makes me frustrated cuz this bitch depicting me as a seal 😡😡😡
Describe the solution you'd like
pls remove it
Describe alternatives you've considered
there are no alternatives, i cannot ignore this seal....
Additional context
image of da bitch sealy:
https://i.imgur.com/89Obzuf.png

[ ]

You have conflicting with beatrun addons and you closed the warning


Fix major startup lag, by using Animated.loop instead of giant duration

This was always a silly-looking hack: what we really want is for
this animation to loop indefinitely, and we do that by saying it
runs for 1000 seconds.

Specifically, in my desktop Android emulator (which is typically
1x-2x the speed of my physical Pixel 4), it takes about 500ms.
Worse: it blocks the entire JS thread while it does that.  So
the app's whole UI is stalled.
Because we use this at startup (in LoadingIndicator), it's been
adding that 500ms of lag (likely more like 1000ms on many devices)
to our startup time since forever.  Moreover, we use
LoadingIndicator in the "Connecting..." banner... which there's
four of, one on each of the app's main-screen tabs.  So when we
start loading -- which is promptly after each startup, if you're
already logged in -- the UI hangs for another 2000ms, or more
depending on device.
There are a number of things we could do here.  It seems like a
performance bug in RN's Animated.  Also we could shorten the
1000 seconds to like 100 seconds, or 50 seconds -- better that
you occasionally see a stopped spinner, if you've already waited
a long time, than that you find the app completely stuck for a
full second or more every time you start it.
But in fact a very simple solution is available: if you take a look
at the upstream docs for Animated:
https://reactnative.dev/docs/animated
there is built-in support for just explicitly having a loop.
There's no need for this hack at all.  Even in RN v0.47.2, the
version we were using when this hack was introduced in 2017 in
6594f8ebc, there was an Animated.loop and there was no need
for this hack.
So use that.

Wow!! Thanks for catching this! LGTM, please merge at will, since it sounds like you've tested it manually :)

Thanks for the review!
I was curious (for the sake of writing the user-facing changelog) whether this affected iOS too, so I just did a bit of testing.
I think the severity seems to be milder on physical devices -- both my physical Pixel 4, and a physical iPhone 7 and iPhone XR -- than in the emulator (so that's good, I guess), but the effect is still visible when I watch for it.
And it looks like this bug does indeed appear on iOS: in a simulated iPhone 12 on my rather underpowered Mac, where everything including the iOS system UI is slow, the difference with and without this fix becomes conspicuous. The way the issue manifests is:

After the blue background of the loading screen appears (and after the "Downloading" banner has completed and gone away, in a debug build), there's a delay of something like 300ms before the spinner and the Zulip logo appear.
Once the loading screen is replaced by the main screen (with white background, and the bottom-tab icons), there's a hang of something like 1000ms while the top part of the UI appears.

With or without this fix, there's an actually pretty weird-looking animation when the main screen first appears: those icons at the top slide in from the top of the screen. On a physical device it's a lot faster and never stood out to me before, but now that I've seen it I notice it there too.
The hang often happens in the middle of that animation. In the simulator on my slow Mac, this often means that row of icons is hanging out in the status-bar area, with the time and battery level and sensor notch and so on occluding parts of it, for like a second.

Aha, and in fact when I looked at the Animated implementation to try to spot the problem, it was pretty quick to find:
https://github.com/facebook/react-native/blob/v0.64.3/Libraries/Animated/animations/TimingAnimation.js#L80-L84
    const frameDuration = 1000.0 / 60.0;
    const frames = [];
    const numFrames = Math.round(this._duration / frameDuration);
    for (let frame = 0; frame < numFrames; frame++) {
      frames.push(this._easing(frame / numFrames));

Right in the JS code (shared between platforms), it's generating an array with a precomputed value for each frame. I guess with 60,000 frames that ends up taking a while. (Plus it's then sending that information over to the native side, and that may take some additional time.)

Oh, and:

I think the severity seems to be milder on physical devices -- both my physical Pixel 4, and a physical iPhone 7 and iPhone XR -- than in the emulator (so that's good, I guess)

I think what this comparison really means may be between release builds and debug builds. All my tests of this issue so far have been in release builds on physical devices, or debug builds on emulators (and fixed versions only in debug builds on emulators), so those variables are confounded.


Bower install does not work

When trying to install via "bower install html5-device-mockups" I get this output:
"Unexpected token } in JSON at position 565"
bower.json is valid

worked fine for me on bower 1.7.9


feat(ci): Allow testing a different getsentry branch

This is useful for getsentry PRs to test this script but pointing to the branch making changes.

We don't test checking out getsentry repo.


add CMakeLists.txt build script

thanks for your MR! :-)
testing your MR i found an issue in the source which prevented the test case from succeeding.
i'll fix it myself and push a commit.
obviously i had not yet tried to run the test program. :-)


Post fix

Coverage decreased (-0.04%) to 91.096% when pulling dab2daa6ae91d71e30e2be7c68b783acaf99cd30 on post-fix into<PHONE_NUMBER>24c25cc9bab793923ad76f4ee9f04f on server-staging.


Rethink and/or document how Connections are shared between Server and API Root

This has caused a lot of confusion and problems. See my comment for more information.

@gtback, the approach you stated in https://github.com/oasis-open/cti-taxii-client/pull/36#issuecomment-391764789 makes sense to me.

Kind of tangentially related but I realized that adding proxy support to the Server class in yesterday's pull request didn't carry over to the api_roots method.
In the short term, for those struggling with proxy issues such as myself, a solution is to add self._conn = conn at line 687 of init.py and conn = self._conn at line 733.  If the proxy is specified correctly, this will work.  Is this worth another pull request or is there a better long term solution?

I am okay with removing verify from the endpoint constructors.  But changing the behavior of Server depending on whether it was initialized with user/pass or a connection object seems rather... subtle.  How about a special keyword arg to the Server constructor which controls connection reuse?  This would directly address the issue, and decouple it from where the connection comes from.  Maybe we should default it to true (reuse), since I guess that's how most people want it to work (but that wouldn't be backward-compatible, if that matters).  E.g. you could pass username/password and have reuse, and... well I guess with the current design, it wouldn't be easy to not have reuse if a connection was passed (because it may not be straightforward to get the username/password out of the current connection object.  But would be easy to add a couple properties).

Perhaps this calls for a "factory".  That might be a dirty word to some, but it could be used to encapsulate connection reuse behavior.  I've heard/seen at least three different reuse schemes so far:

Use the same connection for everything
Use a different connection per host (actually the suggestion was URL, but see below)
Use a different connection for different servers and api roots (regardless of their hosts), and collection/status uses their parent api root connections.

The third option is roughly how the taxii client works now (if ApiRoots are obtained from Server objects, and the latter are created via user/password instead of passing in a connection).
The second option might be roughly what is suggested/intended in PR #34 .  That suggestion was essentially to reuse per URL (if I understood correctly), but that would need to be clarified.  For example, every object has its own URL (of the form /<api-root>/collections/<id>/objects/<object-id>/, see TAXII 2.0 spec section 5.5), but surely we don't want a different connection for each one!  I think we need coarser-grained reuse, e.g. per host.
And others are possible too.  Am I over-thinking this? :)

HTTPConnection is just a wrapper around a requests.Session plus a few helper methods (valid_content_type could be made into a static method). A Session seems to handle multiple domains just fine, so I don't think that's a concern. It's mostly about different parameters (namely authentication credentials) that are used for different domains. Many of the more recent enhancements (proxies, verify) are likely to apply to all domains, and would not need to be changed either. The original implementation assumed that a lot of Server Discovery results would include API roots on other domains, and I don't think that is very common in practice. I would support automatically passing the same connection from Server to API Root and API Root to Collection, as long as there is some way to "override" or "customize" the connection afterwards.
One wrinkle is that an APIRoot object is created automatically from accessing the Server Discovery endpoint, but only its url property is set until other properties are accessed. This lazy-loading makes it strange to update the connection after the object has already been created, but it's really the only way given the current design.
I'll say (again, I think) that it should ideally be rare for applications to frequently traverse Server->API Root and API Root->Collection relationships. Once Collections have been discovered, they can be instantiated explicitly, and none of this connection-sharing really matters at all.

Regarding handling multiple domains: yes, Session handles them, but you can only close all at once.  You can't close the connection to one server and leave the rest open.  One important aspect of the reasoning behind creating new "connections" was their independence.  We picked what we thought was a reasonable design, but apparently people want to do it other ways.  Requiring different credentials is another reason to use multiple connections.  Credentials may typically vary on a per-host basis, but not necessarily.
Post-processing a connection after the endpoint class is created sounds like it would require either (a) subclassing the endpoint class, or (b) letting users pass in post-processing callbacks.  (a) sounds like too much work, and (b) feels awkward.  Perhaps a factory achieves a better separation of concerns, leaving the endpoint classes responsible only for communicating with the services, not dealing with funny customization hooks.
It may be ideally rare for users to traverse server->apiroot->collection, but it seems like the premise of this issue is that people are doing it and stumbling over it.  Seems like the ideal and reality are different :)
I haven't got a complete proof-of-concept implementation yet, but I'm thinking that each endpoint class constructor could be changed to accept a factory, and maybe user/password if we still want to support that.  The endpoint class would just call factory.get_connection() to get a connection, and pass in the URL and perhaps the endpoint type (server/apiroot/collection/etc).  Whatever we think the criteria should be for choosing one.  In the endpoint classes, I think usage should be pretty simple.  No commitments though until I've done more experimentation :)

Currently, the Server class will propagate its configuration into its API Root objects as the current design. Feel free to reopen if this needs to be revisited.


Podman/Docker

In this previous issue we discussed holding off on Podman/Docker for now: https://github.com/demergent-labs/kybra/issues/467
Implementing this I hope would reduce initial compilation time even more, we're at about 5 minutes on my machine right now. But especially on Mac Podman might not be much better.
If we decide not to hold off, we should address the following as we address Podman:

[ ] We could just precompile the stdlib, store it as an asset in the Kybra release, do this kind of like the images in Azle...and just include it using include_bytes macro...this would get rid of the stable memory problem and the need to explain the hashing problem if the stdlib is not compiled in...major question is just if this is going to compile quickly...actually we should probably just do this when we do Podman, because we'll have to do a release and store the stdlib somewhere anyway, might as well just do it there

We should never do this, instead we should implement the Wasm binary manipulation like Azle has, or we should encourage dynamic canister method registration.


boot/zephyr: Kconfig changes to use RSA boot signature for thingy91

The bootloader that is factory-programmed on thing91 does not support ECDSA
signature type. Hence this commit sets the signature type to RSA for
applications built for thing91.
Fixes CIA-259
Signed-off-by: Balaji Srinivasan<EMAIL_ADDRESS>

Should this be a [nrf noup]commit ?
To my knowledge, the BOARD_THINGY91_NRF9160 and BOARD_THINGY91_NRF52840 boards are not available upstream.
Code changes looks good.

Updated the commit message as suggested.


NoClassDefFoundError on 1.14.4

On the version, I found this error can somebody explain it?

Learn java


Fix assert generating dependency log

FirstMarkLogStrategy.VisitLogNodes contains a HashSet,
combinedNodesReporting, that ensures we only report duplicate combined
dependencies once. We were checking the HashSet didn't already contain
an entry but failing to add the entry after visiting the node.

@davidwrighton Can you have a look?


pull request for e2e test at 12/21/2016 18:26:21

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix typo in GitLab integration docs

Summary
This PR changes the mentioned file name requirement.txt to requirements.txt (add "s") to be consistent.
Test Plan
n/a

Thanks!


PGSync tries to find FK between unrelated tables.

PGSync version:
v2.1.1
Postgres version:
v11.1
Elasticsearch version:
v7.11.2
Redis version:
v5
Python version:
v3.8
Problem Description:
Using this schema, while bootstrap and pgsync commands run without issues, I am getting the following error when actually create a Tenant database record. Look like pgsync has mistakenly tried to correlate the UserRoles medium table with Role's foreign key entity District?
Error Message (if any):
Exception in thread Thread-23:
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/threading.py", line 932, in _bootstrap_inner
    self.run()
  File "/usr/local/lib/python3.8/threading.py", line 870, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/local/lib/python3.8/site-packages/pgsync/sync.py", line 860, in poll_redis
    self.on_publish(payloads)
  File "/usr/local/lib/python3.8/site-packages/pgsync/sync.py", line 927, in on_publish
    self.sync_payloads(_payload)
  File "/usr/local/lib/python3.8/site-packages/pgsync/sync.py", line 829, in sync_payloads
    for doc in self._payloads(payloads):
  File "/usr/local/lib/python3.8/site-packages/pgsync/sync.py", line 645, in _payloads
    filters = self._insert(
  File "/usr/local/lib/python3.8/site-packages/pgsync/sync.py", line 339, in _insert
    foreign_keys = self.query_builder._get_foreign_keys(
  File "/usr/local/lib/python3.8/site-packages/pgsync/querybuilder.py", line 45, in _get_foreign_keys
    for key, value in get_foreign_keys(through, node_b).items():
  File "/usr/local/lib/python3.8/site-packages/pgsync/base.py", line 961, in get_foreign_keys
    for table, columns in _get_foreign_keys(
  File "/usr/local/lib/python3.8/site-packages/pgsync/base.py", line 920, in _get_foreign_keys
    raise ForeignKeyError(
pgsync.exc.ForeignKeyError: 'No foreign key relationship between "public.saas_api_user_roles" and "public.saas_api_district"'

There needs to be a direct or indirect relationship between all tables in your schemas.
Can you please provide more details of your database schema?

There needs to be a direct or indirect relationship between all tables in your schemas.
Can you please provide more details of your database schema?

I've hyperlinked the schema source code in the issue description, or this link: https://gist.github.com/namoshizun/3f86d97cd0b4ce72b779ed913a4fa849

Sorry I meant the database schema itself. You can send this to me directly if you have any reservations.
@namoshizun

I appear to be running into this as well. Initial bootstrap and sync work, but adding a new record on my Book<->Author through table errors with pgsync.exc.ForeignKeyError: 'No foreign key relationship between "public.BookAuthor" and "public.BookAuthor"'. Adding just a new Book works. Adding just a new Author works. It's adding a Book with an Author that is failing. The op causing it is  DEBUG:pgsync.sync: tg_op: INSERT table: public.BookAuthor. The last bit of the stack trace is  File "/Users/me/opt/anaconda3/lib/python3.9/site-packages/pgsync/sync.py", line 824, in _payloads     filters = self._insert_op(   File "/Users/me/opt/anaconda3/lib/python3.9/site-packages/pgsync/sync.py", line 491, in _insert_op     foreign_keys = self.query_builder._get_foreign_keys(   File "/Users/me/opt/anaconda3/lib/python3.9/site-packages/pgsync/querybuilder.py", line 68, in _get_foreign_keys     for key, value in get_foreign_keys(through_node, node_b).items():   File "/Users/me/opt/anaconda3/lib/python3.9/site-packages/pgsync/base.py", line 890, in get_foreign_keys     for table, columns in _get_foreign_keys(   File "/Users/me/opt/anaconda3/lib/python3.9/site-packages/pgsync/base.py", line 850, in _get_foreign_keys     raise ForeignKeyError( I'll try to dig into the code to see what it's expecting but would be happy to hear suggestions or workarounds.

this fixed it for me: https://github.com/toluaina/pgsync/pull/332

Sorry for taking a while to address this.
Can you please try against the main branch.
This should be resolved now.


chore(main): release 1.2.0

:robot: I have created a release beep boop
1.2.0 (2024-09-18)
Features

add ability to pass args as function instead of table (#188) (c5df84c)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.2.0
:sunflower:


implement AddBefore and AddAfter Methods

this is good for an extensible pipeline through plugins

If this is specific to plugins and anybody wants this maybe they could just inherit:
public class PluginPipeline<TParamter> : Pipeline<TParamter>
{
    public PluginPipeline(IMiddlewareResolver middlewareResolver) : base(middlewareResolver)
    {
    }

    public void AddBefore<TMiddleware1, TMiddleware2>()
    {
        int index = MiddlewareTypes.IndexOf(typeof(TMiddleware1));
        if (index != -1)
        {
            MiddlewareTypes.Insert(index, typeof(TMiddleware2));
        }
    }
}

?
Would actually be nice if someone made a pull request and made all public methods on Pipeline<TParameter>/AsyncPipeline<TParameter>/ResponsibilityChain<TParameter>/AsyncResponsibilityChain<TParameter> virtual.


POST request got 401 Unauthorized

I deployed my database to heroku and everything works with Postman (even with CURL like in the example).
However I fail to make it work on the actual frontend when using axios. Here's my code
`axios({
    method: 'post',
    baseURL: 'abcdef.herokuapp.com',
    url: '/users',
    data: {
      "access_token": "123456789",
      "email"<EMAIL_ADDRESS>      "password": "123456"
    },
    headers: {
      'content-type': 'application/x-www-form-urlencoded',
    }
  }).then(data => {
    console.log(data)
  })`

When I tested things in Postman it works perfectly but I couldn't make the same thing happen with axios. I assume access_token is not where it's supposed to be but I'm not sure where else to put it. Can you help me with this or is it too specific in my case?

hi
remove  'content-type': 'application/x-www-form-urlencoded',
it works without this header

Thanks for your fast reply. Got this to work now.
I think I've tried a lot of different ways without it working (I didn't have that line originally but very likely I had some extra wrong lines) that I forgot that one simple line. Closing it now.


SMAA not working/strong enough?

Tried the mod but the AA doesnt seem to be doing enough, if anything, assuming we just have to extract in game folder and launch the game and wait for it to configure and not change anything else for SMAA to start working.
First pic is with mod, second without. Will probably not be very clear in the pictures so letting you know this is the 3rd scene in the game so you can check it out. Chose this scene specifically since its early and its very clear that blue and pink haired in the back are very aliased, especially pink haired's face is a mess since she is very back.

Applies no SMAA to mirror reflections also


[86bxzrr3y][base-trigger] Fixed state for filter trigger

Motivation and Context
Added state prop for FilterTrigger for correct view invalid (or valid) states.

How has this been tested?
Manually

Screenshots (if appropriate):

Types of changes

[X] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[X] Nice improve.

Checklist:

[X] My code follows the code style of this project.
[X] I have updated the documentation accordingly or it's not required.
[X] Unit tests are not broken.
[X] I have added changelog note to corresponding CHANGELOG.md file with planned publish date.
[ ] I have added new unit tests on added of fixed functionality.

We won't do this request, because requesters use FilterTrigger in a wrong way.


How can I use Waterfall with Cauldron?

I want to set up a network with both spigot and cauldron servers, how can i do that?

You probably want Travertine, as opposed to waterfall, in order to support a 1.7.10 server; beyond that, it's configured the exact same as a spigot server is, albeit, if you're not using the spigot fork, you won't have proper bungee support (IP and UUID forwarding).

My spigot servers are all 1.8 and cauldron is 1.7.10 (as all the mods i need in my server) when using bungeecord, i get this error: >11:54:16 [WARNING] [FlyrkyGameplays] <-> ServerConnector [dbw] - overflow in packet detected! Cannot receive byte array longer than 169 (got 6178 bytes)
I was asking if i use waterfall, is possible or not to use spigot 1.8 linked with 1.7.10 cauldron

Waterfall does not support 1.7.10. You'll want to use Travertine, for Waterfall with 1.7 support.

Does Travertine support 1.8 as well?

Travertine supports all the versions as Waterfall does, with additional support for 1.7.10.

One question, how do I configure Travertine? Is it the same configuration process as Waterfall?

exact same as waterfall, exact same as bungee

Ok, now I don't get that error, instead, when I jump in the portal to the Dragon Block server (cauldron) I just get disconnected, it just says "Dissconnected from server" on my screen, no errors on console. Only in the cauldron console, it appears this: >[12:50:48 INFO]: FlyrkyGameplays lost connection: Internal Exception: java.io.IOException: Broken pipe

https://github.com/WaterfallMC/Travertine/issues/63


Building angular using webpack is not importing modules correctly

Command
other
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
I have build a small example app to show off this issue. It seems when importing modules in the module declaration it does import the modules declarations. For example when I import the browser module it does not import NgIf, NgFor etc etc and when I import NgSelect it does not import any of its Schema or components. I am building the app without using angular-cli as I am currently migrating an app from angularjs to angular and moving it to use angular-cli would be an issue.
I have set up the webpack config like
const path = require('path'),
AngularWebpackPlugin = require('@ngtools/webpack'),
webpack = require('webpack'),
HtmlWebpackPlugin = require('html-webpack-plugin'),
WebpackDevServer = require('webpack-dev-server');

module.exports = {
    entry: {
        'test': { import: "./app/app.module.ts", filename: 'js/[name].bundle.min.js' },
    },
    mode: 'none',
    output: {
        path: path.resolve(__dirname, "dist"),
        clean: true
    },
    resolve: {
        //extensions: ['.ts', '...'],
        extensions: ['.tsx', '.ts', '.js', '.mjs', '.json']
    },
    optimization: {
    },
    plugins: [
        new webpack.ProvidePlugin({
            moment: 'moment'
        }),
        new HtmlWebpackPlugin({
            title: 'Test',
            template: './app/index.hbs',
            cache: false,
            hash: true,
            inject: false
        }),
        new AngularWebpackPlugin.AngularWebpackPlugin({
            'tsconfig': 'tsconfig.json',
            //"compilerOptions": {},
            "sourceMap": true,
            'jitMode': true
        })
    ],
    module: {
        rules:
            [
                {
                    test: /\.hbs$/i,
                    use: "handlebars-loader",
                    exclude: /node_modules/,
                },
                {
                    test: /\.[jt]sx?$/,
                    loader: '@ngtools/webpack',
                     include: /app[\\/]/,
                    exclude: [ /node_modules/ ]
                },
                {
                    test: /\.[cm]?js$/,
                    use: {
                      loader: 'babel-loader',
                      options: {
                        cacheDirectory: true,
                        compact: false,
                        plugins: ['@angular/compiler-cli/linker/babel'],
                      },

                    },
                },
                {
                    test: /\.template\.html$/,
                    exclude: /node_modules/,
                    use:
                        [
                            {
                                loader: 'raw-loader',
                                options:
                                {
                                    esModule: false,
                                },
                            }
                        ],
                },
                {
                    test: /\.css$/i,
                    exclude: /node_modules/,
                    use:
                        [
                            {
                                loader: 'css-loader',
                                options: {
                                    import: false,
                                    url: false,
                                    modules: false
                                }
                            }
                        ]
                }
            ],
        },
        devServer: {
            static: {
              directory: path.join(__dirname, 'dist'),
              watch: true
            },
            host: "localhost",
            compress: true,
            port: 5000,
            server: 'https',
            open: true,
            liveReload: false,
            hot: true,
            client: {
                overlay: true,
                progress: true,
            }
          }
}

and the angular test app is
app.module.ts

import 'zone.js';

import { NgSelectModule } from '@ng-select/ng-select';
import { BrowserModule } from '@angular/platform-browser';
import { NgModule } from '@angular/core';
import { AppComponent } from './app.component';
import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';

@NgModule({
    declarations: [
        AppComponent
    ],
    imports: [
        BrowserModule,
        NgSelectModule,
    ],
    providers: [
    ],
    bootstrap: [AppComponent]
})
export class AppModule {
}

platformBrowserDynamic().bootstrapModule(AppModule)
  .catch(err => console.error(err));

app.component.ts
import { Component, Input, OnInit, Output, EventEmitter } from '@angular/core';

@Component({
    selector: 'app-root',
    template: require('./app.template.html'),
})
export class AppComponent implements OnInit {
    public source = [ { id: 1, name: "moo" }]
    public selected = null;

    ngOnInit(): void {
        console.log("YAY!");
    }
}

app.template.html

<!DOCTYPE html>
<html lang="en">
<head>
    <title ng-bind="title">T.I.E.R</title>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <meta content="IE=edge, chrome=1" http-equiv="X-UA-Compatible" />
</head>
<body>
    <app-root></app-root>
    <script defer src="js/test.bundle.min.js?904b25da0a6807f141ae"></script>
</body></html>

and my tsconfig is:
{
    "compileOnSave": false,
    "compilerOptions": {
      "baseUrl": "./",
      "outDir": "./dist/out-tsc",
      "resolveJsonModule": true,
      "forceConsistentCasingInFileNames": true,
      "strict": false,
      "noImplicitOverride": false,
      "noPropertyAccessFromIndexSignature": false,
      "noImplicitReturns": true,
      "noFallthroughCasesInSwitch": true,
      "sourceMap": true,
      "declaration": true,
      "emitDecoratorMetadata": true,
      "downlevelIteration": true,
      "experimentalDecorators": true,
      "moduleResolution": "Node",
      "importHelpers": true,
      "target": "ES2022",
      "module": "ES2022",
      "useDefineForClassFields": false,
      "lib": [
        "ES2022",
        "dom"
      ]
    },
    "angularCompilerOptions": {
      "enableI18nLegacyMessageIdFormat": false,
      "strictInjectionParameters": true,
      "strictInputAccessModifiers": true,
      "strictTemplates": true,
      "compilationMode": "partial"
    }
  }

As far as I can tell I am building the Ivy partials but I am getting the errors:
NG0304: 'ng-select' is not a known element (used in the 'AppComponent' component template):
If 'ng-select' is an Angular component, then verify that it is a part of an @NgModule where this component is declared.
If 'ng-select' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.

and
NG0303: Can't bind to 'ngModel' since it isn't a known property of 'ng-select' (used in the 'AppComponent' component template).
If 'ng-select' is an Angular component and it has the 'ngModel' input, then verify that it is a part of an @NgModule where this component is declared.
If 'ng-select' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.
To allow any property add 'NO_ERRORS_SCHEMA' to the<EMAIL_ADDRESS>of this component.

as if it is not imported. Apart from that angular is working fine. If I put NgIf in imports I can use it but it should be imported from BrowserModule or CommonModule.
Minimal Reproduction

Create webpack.config.js with above config
Run webpack
Run webpack dev server
Check console.

No response

Hi, thanks for opening this issue but custom webpack setups are not supported the by Angular team.


Fix button position

Details
Using flexGrow1 sets the flex properties to flex-grow: 1, flex-shrink: 1 and flex-basis: auto which is different than flex1 which sets the properties to flex-grow: 1, flex-shrink: 1 and flex-basis: 0 and allows the flex container to redistribute  the space.
Fixed Issues
$ https://github.com/Expensify/App/issues/38497
Tests
Precondition: User is an admin of Collect workspace

Go to Profile > Workspaces > Collect workspace
Go to More features
Toggle on Taxes
Tap Add rate
Tap Name
Tap on the input field
Verify that the button is not hidden behind the virtual keyboard

[x] Verify that no errors appear in the JS console

Offline tests
N/A
QA Steps
Same as tests

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

hmm this PR didn't seem to solve the issue. So it must be something else. I have to prioritize other issues at the moment, so I'm gonna close this PR. @filip-solecki would you be able to investigate the issue further? Additionally, I noticed that NamePage uses state for name, which is not necessary when using FormProvider. Could you also please refactor that? Here's an example

Sure, I can take a look on Tuesday as we are OOO on Monday!


Is the client_name really necessary?

Description
It should be possible to remove the client_name parameter entirely.
Since we are using RSA keys for authentication, it should be safe to assume they are unique. This means that the key in use actually does identify the user (a technique used by github, gitlab, etc as well).
Of course this is no small task, the signing protocol needs to be adjusted probably to remove the requiredness of the client_name in the signed body.
This would make knife configuration distribution easier
Chef Version
12.15
Platform Version
N/A

Without some kind of ID we would need to check every single public key in the system to know which the signature matches. If you mean we could use the public key itself as the client and node name, then sure but it would make debugging much less pleasant.

As mentioned on #5655, we don't really use GitHub issues for tracking these kinds of questions. If you are interested in improving the authentication protocol, an RFC would also be the place to start.
As for the comparison to GitHub and Gitlab, the SSH protocol does in fact include the user name for this purpose as you can see in RFC 4252 otherwise it would be computationally infeasible for the same reasons as here.

https://feedback.chef.io/forums/301644-chef-product-feedback/suggestions/17460205-remove-client-names-the-rsa-key-should-identify-t

@coderanger, come on, you can easily build a hash from pubkey to clientname on the server side to look up the client name. :)
Pretty sure github has more users than most chef deployments, and they do just fine.

on github the username is always git@

just try issuing: ssh<EMAIL_ADDRESS>it will tell your github username.

The name that GitHub cares about isn't the ssh protocol name, it's the pub key blob in the authentication negotiation, it's included before an auth method is agreed on and maps back to a user in GitHub's database.
Original Quoted context has been removed as per policy


iobroker.worx - mqtt goto home

Hi! I got the iobroker.worx adapter working now with my S300 mower and Mqtt adapter.
I figured out how to send start(iobroker/worx/0/2019xxxxxxxxxxxxxxxCD/mower/state=true)
and pause(iobroker/worx/0/2019xxxxxxxxxxxxCD/mower/pause=1) commands but what is the goto home command? iobroker/worx/0/2019xxxxxxxxxxxxxxxxxxxxxCD/mower/????

Found it ! :-) It simply state=false

home(iobroker/worx/0/2019xxxxxxxxxxxxxxxCD/mower/state=false)


Fix redundancy in CRToastManager regarding showNotification:

Resolves #163
Allows easier hooking of showing notifications.

Good catch :+1:

Thanks :star:


Cats for v1-5-pruned-emaonly

The extension is appending the keyword 'cat' for the plain v1-5-pruned-emaonly.safetensors checkpoint...

user/custom mappings are confirmed empty, but I can't find any direct match for the hash either...


Feature/dsl graph settings menu

Adding a settings menu for users to opt-out of the workflowGraph experimental feature (currently enabled by default).
As the DSL graph is an experimental feature, we need a way for users to easily disable in case there are serious bugs or side effects discovered.
When the workflowGraph setting is disabled, this will hide the "Show Graph" button and the view associated with it. The timeline view will still be accessible.
Screenshots
Menu
Enabled (by default)

Disabled

Feature flag disabled (feature removed from menu)

workflowGraph enabled

workflowGraph disabled

This refactor will not be needed in the future when things move to using Vuex. Closing PR.


some released image of Eventing don't have tag

some released image of Eventing don't have tag,such as sugar_controller
installed by operator
version: v0.21.0
gcr.io/knative-releases/knative.dev/eventing/cmd/sugar_controller@sha256:d13785d2532cc6726aa89de04e185f1a1d40dae1de60e095ba07556825beab64

while other released images have tag eventing-controller
https://console.cloud.google.com/gcr/images/knative-releases/GLOBAL/knative.dev/eventing/cmd/controller@sha256:13963599dfd793dae68f3affb48c4ade86533b007be76d620de5cea6164b70bb/details?tab=info

cc @lionelvillard @markusthoemmes

cc @n3wscott

And v0.23.0 also, here is the image without tag

@knative/knative-release-leads @knative/eventing-wg-leads

might be a prow setting? We don't have anything that tags in our release. Perhaps it is just a ko flag we are missing?

it looks like ultimately the function that does this is tag_images_in_yamls from https://github.com/knative/hack/blob/88c69cd1eca7b1d655778b5ed51acde428c88ff8/release.sh#L47-L62
And this is gated by setting PUBLISH_RELEASE. This is not set by serving or eventing... this might be also set by passing the flag --publish but again, this is not set by serving or eventing. looks like --auto-release is also a flag that sets this.
So. I am thinking this problem is in a prow config difference for eventing.
Still looking.

Interestingly it looks like domain mapping (which is part of the serving build obviously) isn't tagged either 🤔

chizhg  30 minutes ago
From the log it looks these are for different platforms - https://prow.knative.dev/view/gs/knative-prow/logs/ci-knative-eventing-auto-release/1394553523051433984#1:build-log.txt%3A12275 ? But only the “default” platform has the version tag?
chizhg  28 minutes ago
Haven’t touched the script for quite a while, so not sure if there are other changes that can cause it..
n3wscott  < 1 minute ago
oh interesting, only the top level multi-arch image is tagged?

slack thread: https://knative.slack.com/archives/CCSNR4FCH/p1622147240028400

Oh seems it’s non-determined.. https://prow.knative.dev/view/gs/knative-prow/logs/ci-knative-eventing-auto-release/1394553523051433984#1:build-log.txt%3A12658 will iterate all the images and add the tag, since the tag is unique, only the last image will have the tag

@chizhg Aren't tags unique to repo-path/image? So technically say some-repo/image1 and some-repo/image2 could have the same version tag. Which is what I would expect

@chizhg is this still an issue?


Webpack's sourcemap paths are not real paths

It seems like webpack changed the way it puts the paths in the sourcemap file, but the paths are all wrong. They all look like:
webpack://whatever/~/more-stuff.js
I was working around webpack's previous weird way of writing paths in the sourcemap, but the real question is: why is webpack writing the mapped paths this way? I need to be able to go request the actual file from the server from these paths, and webpack's non-standard paths are blocking me from doing this.

Old thread, but thanks @sokra  - this config was helpful for getting vscode debugger working with sourcemaps in my node app.


编译cmake ..出错

已安装octomap和pcl，并且将cmake_modules/octomap-config.cmake文件中的四个路径修改为我安装octomap的路径，然后运行./build.sh 出错，错误信息如下
CMake Error at src/CMakeLists.txt:10 (FIND_PACKAGE):
By not providing "Findoctomap.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "octomap", but
CMake did not find one.
Could not find a package configuration file provided by "octomap" with any
of the following names:
octomapConfig.cmake
octomap-config.cmake

Add the installation prefix of "octomap" to CMAKE_PREFIX_PATH or set
"octomap_DIR" to a directory containing one of the above files.  If
"octomap" provides a separate development package or SDK, be sure it has
been installed.
-- Configuring incomplete, errors occurred!
See also "/home/hydro/octomap_tutor/build/CMakeFiles/CMakeOutput.log".

系统ubuntu18.04

hello do you solve this problem ？ Now ,I have the same error, but in Ros noetic
CMake Error at /opt/ros/noetic/share/catkin/cmake/catkinConfig.cmake:83 (find_package):
  Could not find a package configuration file provided by "octomap_ros" with
  any of the following names:

    octomap_rosConfig.cmake
    octomap_ros-config.cmake


NewServiceClient failed when ServiceProvider is not exist

Whne I startup a serviceclient, the client will exit with an error "no provider". That is different with "CPP".
What is your idea about this？Why do you call LookupService when NewServiceClient?

Hello, i finally got the time to analyze this issue; the reference CPP implementation does this:

a service client can be created before a service provider is created
a service client requests the list of available providers during every request, allowing providers to connect and disconnect dinamically

Therefore, the behavior of this library will be changed accordingly.

done


Cleanup: removes roles not required by controller/watcher

On top of #876
Changes

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

Codecov Report

Merging #889 (f988512) into main (3e05049) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #889      +/-   ##
==========================================
- Coverage   63.66%   63.56%   -0.10%
==========================================
  Files          79       79
  Lines        5072     5045      -27
==========================================
- Hits         3229     3207      -22
- Misses       1536     1537       +1
+ Partials      307      301       -6

Impacted Files
Coverage Δ

pkg/provider/github/status.go
76.42% <0.00%> (+0.90%)
:arrow_up:

pkg/provider/bitbucketserver/acl.go
63.15% <0.00%> (-2.36%)
:arrow_down:

pkg/reconciler/controller.go
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

cc @ramessesii2 🙃

https://github.com/openshift-pipelines/pipelines-as-code/actions/runs/3220108437


⚠ Bofrost.es has degraded performance

In 1a672f3, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1658 ms

Resolved: Bofrost.es performance has improved in 5deb1dd after 6 minutes.


problem of train cnn model

I set up a  cnn model to classify mnist data.the model as follow:

i train the model firstly successful,the result as follow:

when i changed the characters ,if the max_epochs greater than  step_size, the train go wrong.

learning rate schedulers are created for easy support to learning rate modification.  we will have checks to prevent a smaller step_size than the max_epochs when lr_schedulers are used.

learning rate schedulers are created for easy support to learning rate modification. we will have checks to prevent a smaller step_size than the max_epochs when lr_schedulers are used.

when I trained the tinyyolo, the same problem occurred.the code from https://go.documentation.sas.com/?cdcId=pgmsascdc&cdcVersion=9.4_3.4&docsetId=casdlpg&docsetTarget=p1np8zbnoyd0brn1dhehthuuxj4q.htm&locale=zh-CN#p1dgtuox2j70uen1sg8qpg203n9z
it is about yolo's object detection

thanks for pointing out. I will be looking into this today.

@banbsyip seems like there might be an issue with the connection or such. would you please set "swat.options.cas.debug_responses = True " and then re-run your code. I would like to see what the server sends back

eems like there might be an issue with the connection or such. would you please set "swat.options.cas.debug.responses = True" and then re-run your code. I would like to see what the server sends back

ok,i will run the code as your suggestion. I have a question about the label,i use the voc2007 data,the label of "person" have some parts such as "head","left","hand"and so on. I label all the parts as "person", is it a correct way to label?

Depends on what your task is. If you want to detect the parts, then label them for sure. If you think any human parts should use person as label, then label them as person.

Depends on what your task is. If you want to detect the parts, then label them for sure. If you think any human parts should use person as label, then label them as person.

thanks for your help.

@banbsyip seems like there might be an issue with the connection or such. would you please set "swat.options.cas.debug.responses = True" and then re-run your code. I would like to see what the server sends back
the result as follow:

@banbsyip thanks for sharing the log message. I am communicating with swat folks to see if they can help resolve this issue.

Hello, I know this is an old forum, but I am having the same error message. Does anyone know the solution? @banbsyip @ethem-kinginthenorth Thanks in advance.


Fix pcluster update with "cluster_resource_bucket"

Before this PR, when user does not specify bucket when create, but specifies bucket=[bucket we generate] when updating. In this case we will falsely detect that the bucket value has changed, when in reality it doesn’t.
After this PR,  we give user more visibility on what’s going on instead of just ignoring that parameter. When user provide the S3 bucket through "cluster_resource_bucket", the pcluster update will detect the bucket name, if user doesn't provide the bucket name, the pcluster update will not show the bucket name.
The following cases are tested:

Creation and update with same config: [cluster] not provide "cluster_resource_bucket"  --->The creation and update succeed.

Creation and update with same config: [cluster] "cluster_resource_bucket"= "bucket1" --->The creation and update succeed.

Creation with [cluster] not provide "cluster_resource_bucket", Update with [cluster] "cluster_resource_bucket"= "bucket1" ---> Creation succeed, update failed with:

#    parameter                old value    new value
---  -----------------------  -----------  ---------------
     [cluster default]
01*  cluster_resource_bucket  -            bucket1

Creation with [cluster] "cluster_resource_bucket"= "bucket1" , Update with [cluster] "cluster_resource_bucket"= "bucket2"---> Creation succeed, update failed with:

#    parameter                old value       new value
---  -----------------------  --------------  ---------------
     [cluster default]
01*  cluster_resource_bucket  bucket1           bucket2

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closed in favor of https://github.com/aws/aws-parallelcluster/pull/2235


turbolinks-cache-control no cache, form post and back button

Basically I have a form that when submitted, it delete and create objects into a database.
I've set the turbolinks-cache-control to disable cache on the form page. This is mainly to handle a user pressing on the back button. If that happens, the form use objects ids that have already been deleted, so when you click back ... the old ids are populated rather than the new ones. That's the main purpose of disabling cache in that form.
Now I can confirm if I enable turbolinks-cache-control no-cache widely, it works as expected on any other link. When I click back... the pages are reloaded from the server. But with any other form, I can't achieve the same behavior. I looked into different places but couldn't find an answer.
I could achieve it setting request headers, but I'd like to handle this with turbolinks if possible
Should I be using a different mechanism?
Thanks in advance
Juan

I manually call Turbolinks.clearCache() on the form's ajax:success event.

Hi @juanbrein, as @obnijnil mentions, if you have a form that is likely to affect the content of other pages, you can call Turbolinks.clearCache() when the form submission completes (See Following Redirects for more on this. If there is a particular page which changes a lot, then use the no-cache directive on that page (rather than on the form page). Hope that helps.


SSR problem: layout shift

In Nuxt.js from server I get plain html without base styles (like display: none; if content is collapsed), and styles applied only on mounted. It will be great if proper styles will come from server so no layout shift will be there.

Hi @zhaivoronok, you're right. This is because I made the mistake to not setup the playground on Nuxt (like I did for my other packages).
Expect a fix in the afternoon.


exiftool not found & gifsicle not found

When I use AnimateDiff, there will be the following error in the console, I think it is the lack of related files, may I ask where can I download from, what version to download, and which folder？exiftool not found & gifsicle not found
2024-04-01 19:19:45,037 - AnimateDiff - WARNING - exiftool not found, required for infotext with optimized GIF palette, try: apt install libimage-exiftool-perl or https://exiftool.org/
2024-04-01 19:19:45,054 - AnimateDiff - WARNING - gifsicle not found, required for optimized GIFs, try: apt install gifsicle

I have the same issue, someone here said to download exiftool from the https://exiftool.org/ site and put it in your PATH. I tried that, putting both the .exe and the folder download in both sections of the environment variables, and I'm still getting the error.

THX！！

gifsicle install

cmd
git clone https://github.com/kohler/gifsicle.git


[Event Request] Table 39 - Purchase Line - OnBeforeValidateJobLineAmountLCY

Dear ALAppExtensions team,
through this issue we would like to request adding event publisher OnBeforeValidateJobLineAmountLCY in Table 39 Purchase Line with the following signature:
[IntegrationEvent(false, false)]
local procedure OnBeforeValidateJobLineAmountLCY(var PurchaseLine: Record "Purchase Line"; var TempJobJournalLine: Record "Job Journal Line" temporary; var IsHandled: Boolean)
begin
end;

which would be called as follows in the OnValidate trigger of field 1010 - Job Line Amount (LCY):
trigger OnValidate()
// >>>> New >>>>>>>>>>>>>>>>
var
    IsHandled: Boolean;
// <<<< New <<<<<<<<<<<<<<<<
begin
    // >>>> New >>>>>>>>>>>>>>>>
    OnBeforeValidateJobLineAmountLCY(Rec, TempJobJnlLine, IsHandled);
    if IsHandled then
        exit;
    // <<<< New <<<<<<<<<<<<<<<<

    TestField("Receipt No.", '');
    if "Document Type" = "Document Type"::Order then
        TestField("Quantity Received", 0);
    if JobTaskIsSet then begin
        CreateTempJobJnlLine(false);
        TempJobJnlLine.Validate("Line Amount (LCY)", "Job Line Amount (LCY)");
        UpdateJobPrices();
    end;
end;

Thanks in advance!

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


Ledger: Fail to send token: Data is invalid : Unrecognized error code

After updating the cosmos app on the ledger i can't send through IBC.
I throws: Fail to send token: Data is invalid : Unrecognized error code
To Reproduce
Steps to reproduce the behavior:
Update your ledger device with the latest cosmos app
From keprl go to osmosis chain
Select an IBC token (eg. Juno) and sent it to another Osmosis address (don't use IBC transfer, click on the asset!)
You'll get Fail to send token: Data is invalid : Unrecognized error code

Before the update of the cosmos app on ledger it worked.
Anyhow only the IBC transfer doesn't work the the updated cosmos app on ledger.
Expected behavior
To send the token to the other address
So something has changed either from the cosmos ledger app or the keprl.
Thank you
Reported the same issue on Keprl's github they think it's your issue:
https://github.com/chainapsis/keplr-wallet/issues/368
Thanks

I also have this issue with Terra IBC tokens (SCRT, OSMO, ATOM) when sending between to Terra wallets.

Talk to Ledgerwallet Support on the Live support chat so that your issue can be looked into.

The problem is that we can not revert bug until the fix it!

The issue of IBC token send impossible
It's definitely something to do with the Cosmos App update.
https://github.com/chainapsis/keplr-wallet/issues/368#issuecomment-1145385768

Can anybody fix it please ?  Devs seems not looking the "issue" list!
Thank you

This bug makes it impossible, to supply any IBC assets to Kava lend or to withdraw. So my Atom is stuck there. Please fix.

Cameron from Kava Labs has been saying that they have posted a PR to fix this over a month ago but it has not been accepted yet. I have not been able to find the alleged PR but he claims the Kava team has been in contact with the Ledger team about it. It seems like there is some seriously poor communication between the teams in resolving this issue.  @Kava-Labs @LedgerHQ @Zondax

@ftheirs @Saltari  Anyone of you alive or dead ?
Thank you :)

PUSH! Any news about this?

Cameron from Kava Labs has been saying that they have posted a PR to fix this over a month ago but it has not been accepted yet. I have not been able to find that PR but he claims the Kava team has been in contact with the Ledger team about it. It seems like there is some seriously poor communication between the teams in resolving this issue. Please get it together. @Kava-Labs @LedgerHQ @Zondax

I have been waiting for a solution for a month now. Can someone pick this up please?
My ATOM tokens are stuck on Kava due to this issue.

I have been waiting for a solution for a month now. Can someone pick this up please? My ATOM tokens are stuck on Kava due to this issue.

dude, i'm not a dev here, but can advice you to do IBC transfer instead of normal transfer
it works for me when i send atoms in osmosis

Hi everyone! We've made some fixes for IBC and the release is published here: https://github.com/cosmos/ledger-cosmos/releases/tag/v2.34.5
You can install Zondax release only in NanoS and NanoS+:

Download installer_nanos.sh (or nanos_plus depending on your device)
Give execution permission (chmod +x installer_nanos.sh)
Install version 2.34.5 (./installer_nanos.sh load) ---> This will uninstall previous version and then install 2.34.5

We are working to publish a new Ledger official release!

THANKKKKK YOOOUUUUUUUU!!! Finallyyyyyyyyyyyy!!!! 💯 🥳

Hi everyone! We've made some fixes for IBC and the release is published here: https://github.com/cosmos/ledger-cosmos/releases/tag/v2.34.5
You can install Zondax release only in NanoS and NanoS+:

Download installer_nanos.sh (or nanos_plus depending on your device)

Give execution permission (chmod +x installer_nanos.sh)

Install version 2.34.5 (./installer_nanos.sh load) ---> This will uninstall previous version and then install 2.34.5

We are working to publish a new Ledger official release!

Thank you, looking forward the public release. Do you have an ETA?
Regards

I will close this issue since it's duplicated. Conversation can be followed here https://github.com/cosmos/ledger-cosmos/issues/66
However, it's a matter of time to get a new cosmos release published by Ledger.


FastApi StaticFiles with mount not working

Describe the bug
Our team is struggling to get fastapi-versioning working in our environment since we are serving static assets for our frontend and docs. This seems to be an issue with mount when using StaticFiles.
To Reproduce
Steps to reproduce the behavior:

Create the file test.py

from fastapi import FastAPI
from fastapi.staticfiles import StaticFiles
from fastapi_versioning import VersionedFastAPI, version

app = FastAPI(title="My App")

@app.get("/")
@version(1, 0)
def greet_with_hello():
    return "Hello"

@app.get("/")
@version(1, 1)
def greet_with_hi():
    return "Hi"

app.mount("/", StaticFiles(directory="/"), name="static")

app = VersionedFastAPI(app)

Run the server uvicorn test:app
Error from running application

Traceback (most recent call last):
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/bin/uvicorn", line 8, in <module>
    sys.exit(main())
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/uvicorn/main.py", line 362, in main
    run(**kwargs)
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/uvicorn/main.py", line 386, in run
    server.run()
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/uvicorn/server.py", line 49, in run
    loop.run_until_complete(self.serve(sockets=sockets))
  File "uvloop/loop.pyx", line 1494, in uvloop.loop.Loop.run_until_complete
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/uvicorn/server.py", line 56, in serve
    config.load()
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/uvicorn/config.py", line 308, in load
    self.loaded_app = import_from_string(self.app)
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/uvicorn/importer.py", line 20, in import_from_string
    module = importlib.import_module(module_str)
  File<EMAIL_ADDRESS>line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 790, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "./test.py", line 22, in <module>
    app = VersionedFastAPI(app)
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/fastapi_versioning/versioning.py", line 42, in VersionedFastAPI
    version_routes = [
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/fastapi_versioning/versioning.py", line 43, in <listcomp>
    version_to_route(route, default_version) for route in app.routes
  File "/Users/killswitch-gui/Library/Caches/pypoetry/virtualenvs/portal-pNH9Wcd1-py3.9/lib/python3.9/site-packages/fastapi_versioning/versioning.py", line 24, in version_to_route
    version = getattr(api_route.endpoint, "_api_version", default_version)
AttributeError: 'Mount' object has no attribute 'endpoint'

Expected behavior
A versioned API without having to version mounts.
Additional context
N/A

Just got blocked by this same issue.

Getting this same error but not using StaticFiles.

I've solved this issue by using the app.mount after the app = VersionedFastAPI(app)

@aminelemaizi  thanks for the workaround! I would like to know if this is an acceptable way of doing this or if there needs to be a decorator to exclude mounts for a more official fix?

@killswitch-GUI , tbh I don't know if it is an acceptable way of doing things. I didn't see what does the code under the hood to confirm that.

Following.


Change server.host to <IP_ADDRESS> for DEVELOPER_GUIDE

Description
Small update to developer guide to fix issue with the value of server.host.
Category
[Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation]
Documentation
Check List

[ ] New functionality includes testing
[ ] New functionality has been documented
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1350 (3c76c91) into main (bb443e0) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main    #1350   +/-   ##
=======================================
  Coverage   71.78%   71.78%
=======================================
  Files          88       88
  Lines        2027     2027
  Branches      269      269
=======================================
  Hits         1455     1455
  Misses        509      509
  Partials       63       63

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Espresso: q2r and matdyn jobs

Summary of Changes
End of the implementation of the phonon jobs, q2r.x is used to transform the force constant matrix from reciprocal space to real space, matdyn.x can then interpolate it to any q-point.
This is needed to perform phonon-dos with Espresso.
I also added a flow that does all of that (Note to self: test is missing for this flow)
EDIT: I think moving the sanity checks from espresso.utils into the template will be needed at some point, there are a lot of self attributes that this function should access.
Checklist

[x] I have read the "Guidelines" section of the contributing guide. Don't lie! 😉
[x] My PR is on a custom branch and is not named main.
[x] I have used black, isort, and ruff as described in the style guide.
[ ] I have added relevant, comprehensive unit tests.

Notes

Your PR will likely not be merged without proper and thorough tests.
If you are an external contributor, you will see a comment from @buildbot-princeton. This is solely for the maintainers.
When your code is ready for review, ping one of the active maintainers.

Can one of the admins verify this patch?

@Andrew-S-Rosen I have seen some of your messages about tests taking time, can you try to set "export OMP_NUM_THREADS=1" or maybe 2?
At some point I have seen something like 8 in the test outputs, which I suspect might slow things down on this particular setup?

@Andrew-S-Rosen I have seen some of your messages about tests taking time, can you try to set "export OMP_NUM_THREADS=1" or maybe 2?
At some point I have seen something like 8 in the test outputs, which I suspect might slow things down on this particular setup?

Thanks! Good suggestion. I noticed that on my local machine which has a very large amount of threads, the tests take a lot longer than on CI for that reason. I think the proper way to do this would be monkeypatch.setenv("OMP_NUM_THREADS", 2) in the relevant tests. I can try that separately from this PR to see if/where it helps.

@Andrew-S-Rosen I have seen some of your messages about tests taking time, can you try to set "export OMP_NUM_THREADS=1" or maybe 2?
At some point I have seen something like 8 in the test outputs, which I suspect might slow things down on this particular setup?

Thanks! Good suggestion. I noticed that on my local machine which has a very large amount of threads, the tests take a lot longer than on CI for that reason. I think the proper way to do this would be monkeypatch.setenv("OMP_NUM_THREADS", 2) in the relevant tests. I can try that separately from this PR to see if/where it helps.

I've just tried here with your suggestion, also adding parallel info to have 2 processes, let's see

Before:
After:

Not bad! Only phonons and bands test have been setup here, I will fix it back and open a separate PR, when applied on every tests we should be able to get that to 3 min or so

@Andrew-S-Rosen
Before:
After:
Not bad! Only phonons and bands test have been setup here, I will fix it back and open a separate PR, when applied on every tests we should be able to get that to 3 min or so

Thank you very much, @tomdemeyere! That is great!

This should be ok to merge!

Thank you very much, @tomdemeyere!


Broken link to Admin demo on ReadMe

The link to the Admin demo on the GitHub's ReadMe page doesn't work:
https://mochahost.demo.mixcore.org/portal

Thanks @RedFeet , let me double check.

@RedFeet  Fixed server IP. Kindly check again.


Auto refresh

Please implement the auto-refresh feature.

This feature is already there, but not working on android 8 and above

But why? I think it isn´t that complex. Make a timer and point on your refresh function. :)

Android 8 introduced some pretty big changes regarding background services: https://developer.android.com/about/versions/oreo/background
I'm happy to accept PR's if someone is up to the task.


Training procedure cropping and resizing for semantic segmentation

Hi,
Thanks for your great paper. For the semantic segmentation model on ADE20K, you state the following:
"""Images are resized to 520 pixels side length.
We use random horizontal flipping and random rescaling in
the range ∈ (0.5, 2.0) for data augmentation. We train on
square random crops of size 480."""
I feel I must not understand the procedure as randomly scaling a 520-pixel length image between the range (.5, 2.0) will result in some images of side-length less than your random crop size of 480. Could you please clarify the order of operations and any missing detail here? Thank you!

We used pytorch-encoding as our training framework. See here for the transform that is applied to the input during training: https://github.com/zhanghang1989/PyTorch-Encoding/blob/331ecdd5306104614cb414b16fbcd9d1a8d40e1e/encoding/datasets/base.py#L64
We used this class with base_size=520 and crop_size=480.

Thanks!


Add asdf license

Is this a licensing information PR?
If this is a PR that is adding licensing information for formula, then please make sure that your PR is only adding or editing a single formula's licensing information. If possible, a link to where you found the formula's licensing information would be good to include for tracking purposes.
Please note that you must use the use only (SPDX identifiers)[https://spdx.org/licenses/]
Formula Name
asdf
License
MIT
License URL (optional)
https://github.com/asdf-vm/asdf/blob/master/LICENSE
Is this a feature or bugfix PR?
Summary
Rationale
How did you test this?

Thanks so much for contributing this license info @igas !


Why has yiiform JS file got a messages param in beforeValidate?

Right here: https://github.com/yiisoft/yii2/blob/master/framework/assets/yii.activeForm.js#L32 Are there really messages to be had in before validate?
It always logs as empty to me

it would be logical if one could add some messages there to do some custom validation.

Ah ok, probably not what I am looking for though, but sounds reasonable


LLVM thread(s) hang after fork from the parent process

Setup: I am using gperftools 2.11 for heap profiling of tensorflow 2.11 training jobs on a RHEL 7.9 machine.
Observation: When tensorflow ops are being compiled, the main process is creating an llvm thread group and using them for parallel compilation of the ops. In my setup, I observed that the only child process being created via fork is hanging when tcmalloc+heap profiling is enabled.
The back trace for the parent process is shown below
#0  0x00007fa2ca2bba35 in pthread_cond_wait@@GLIBC_2.3.2 () from /usr/lib64/libpthread.so.0
#1  0x00007fa2c9582aec in std::condition_variable::wait(std::unique_lock<std::mutex>&) () from /usr/lib64/libstdc++.so.6
#2  0x00007fa2a7f2339b in llvm::ThreadPool::wait(llvm::ThreadPoolTaskGroup&) ()
   from /opt/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so
#3  0x00007fa2a73a1e6c in mlir::detail::OpToOpPassAdaptor::runOnOperationAsyncImpl(bool) ()

The back trace for the child process is shown below
#0  0x00007f5f54e7ae29 in syscall () from /usr/lib64/libc.so.6
#1  0x00007f5f562b8cb0 in base::internal::SpinLockDelay (w=w@entry=0x7f5f5667adb0 <heap_lock>, value=2, loop=loop@entry=31771) at ./src/base/spinlock_linux-inl.h:86
#2  0x00007f5f562b8b67 in SpinLock::SlowLock (this=this@entry=0x7f5f5667adb0 <heap_lock>) at src/base/spinlock.cc:134
#3  0x00007f5f562b3f4a in SpinLock::Lock (this=0x7f5f5667adb0 <heap_lock>) at src/base/spinlock.h:71
#4  SpinLockHolder::SpinLockHolder (l=0x7f5f5667adb0 <heap_lock>, this=<synthetic pointer>) at src/base/spinlock.h:123
#5  RecordAlloc (skip_count=0, bytes=16, ptr=0x4a06b960) at src/heap-profiler.cc:319
#6  NewHook (ptr=0x4a06b960, size=16) at src/heap-profiler.cc:341
#7  0x00007f5f562aec02 in MallocHook::InvokeNewHookSlow (p=p@entry=0x4a06b960, s=s@entry=16) at src/malloc_hook.cc:314
#8  0x00007f5f562bafa4 in MallocHook::InvokeNewHook (s=16, p=0x4a06b960) at src/malloc_hook-inl.h:133
#9  tcmalloc::do_allocate_full<tcmalloc::cpp_throw_oom> (size=16) at src/tcmalloc.cc:1808
#10 tcmalloc::allocate_full_cpp_throw_oom (size=16) at src/tcmalloc.cc:1818
#11 0x00007f5ef29028b1 in arrow::util::(anonymous namespace)::AfterForkState::AfterFork() ()
   from /opt/site-packages/pyarrow/libarrow.so.900
#12 0x00007f5f54e47c4e in fork () from /usr/lib64/libc.so.6
#13 0x00007f5f54e70830 in __spawni () from /usr/lib64/libc.so.6
#14 0x00007f5f54e707b0 in posix_spawnp@@GLIBC_2.15 () from /usr/lib64/libc.so.6
#15 0x00007f5f3389bf6d in tsl::SubProcess::Start() ()
   from /opt/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so
#16 0x00007f5f33546975 in stream_executor::CompileGpuAsm(int, int, char const*, stream_executor::GpuAsmOpts) ()
   from /opt/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so

So this is issue with fork and pthread_atfork interaction. I will save all the details and complexities of atfork stuff, but here is what matters specifically in your case:
a) TF is calling (the only) right API: posix_spawn to spawn child process. But sadly for some unknown reason glibc until 2.24 had "broken" implementation that forked (instead of vfork or rather clone_vfork it should have used and which is now used). This lack of "properness" of posix_spawn in your older version of glibc is what is triggering all the mess.
b) pthread_atfork is itself super tricky thing in many cases. Google's internal policy for example is to never use it. There is internal paper with details, but somehow I am not able to find public version. Some of paper's arguments are imho not right, but main point is sound. That is: different libraries/modules lock & state nestings won't always match nestings of atfork handlers established at runtime causing deadlocks. For some reason that arrow thingy choose to employ atfork stuff. Perhaps it is occasionally used in forked settings rather than threaded settings (I guess because python is only able to offer parallelism with fork is what addes demand to this questionable feature). Then their atfork "after" handler calls into tcmalloc. You're likely using libtcmalloc LD_PRELOAD-ed (not much else makes sense). And do some atfork business but only for our main locks (yes, despite some arguments that maybe we shouldnt). So it would have worked, but don't do usual atfork dance for heap profiler's heap_lock thingy. And this is where child's after handler finds heap_lock arbitrarily "broken" and hangs.
So we could have our atfork handling amended to also do the locking around heap profiler lock. Alternatively, you can avoid all the trouble by having libc which has right posix_spawn implementation. I have checked that RHEL 8 does. If upgrading to rhel8 or later isn't an option, then consider "stealing" right posix_spawn implementation from either modern glibc or from musl.

Thanks for the insight @alk


Update service-broker-with-always-on-availability-groups-sql-server.md

Changed the endpoint in example to match the endpoint in the CREATE ENDPOINT statement to keep it from being confusing and to match up to the example.

Thanks, @dbaduck