Track: FRR Quagga fork

https://github.com/FRRouting/frr
Just to track the steps this project goes. It's a very active fork of Quagga and currently implementing EIGRP, NHRP and ISIS. Some limitations on BSD, some only for OpenBSD.
pfSense will replace their mixup with this.
ATM I don't see any reason to go for it but it seems worth to track.
Changelog FRR 3.0:
https://github.com/FRRouting/frr/wiki/FRR-2.0-→-FRR-3.0-Changelog

Given support from the Linux foundation, a more open dev model with well-documented governance, and (less absolutely positive but I like) Github-based development, it seems reasonable to give this serious consideration.
FRR already has a number of interesting improvements over Quagga (BGP AddPath, BGP unnumbered, 32-bit route tags, ldpd, etc.), and development velocity seems pretty fantastic ATM: https://github.com/FRRouting/frr/pulse

pfSense, as it stands right now, has frr as a package which is mutually exclusive with the quagga-based routing packages. Given quagga is not in core, perhaps a "middle way" with less risk would be to allow both a quagga-based routing package and an frr-based one?

I would think having both would be less beneficial in the future.  We should pick the one that gives us the best results in the mid and especially long term...
@fabianfrz and @mimugmail should decide, they work on the plugin regularly.

@fichtner I think we cannot support both but if we switch we will have to be careful not to break old installations. this means we would have to move the plugin to frr and provide a meta package called quagga with a newer version which installs the frr plugin as a dependency.
If the vtysh output and the config file format has not changed, we can simply switch, otherwise we will have to adjust the plugin code (especially the regular expressions in quagga.rb).

I ran through the migration with a few non-trivial configs on a linux box (BGP, OSPF, didn't try a RIP config). Config files were not changed and worked as expected, and vtysh output didn't change.
Cumulus has a doc on the transition on their OS which provides useful context: https://docs.cumulusnetworks.com/display/DOCS/Upgrading+from+Quagga+to+FRRouting
The commit activity on frr is astounding. On its own, that's not necessarily a good thing (tm), but most of it seems to not be window dressing - real outstanding bugfixes and cleanup that have been long-standing on Quagga are being resolved.

Recently there was a discussion on the Quagga ML:
https://lists.quagga.net/pipermail/quagga-users/2017-August/014824.html
Seems there are some global players not interested in any changes since it runs fine for them :)

I read that differently - that entrenched interests are not super interested in a continuously improving open source routing engine competing against their wares :)

frr will be in packages starting with 17.7.3. Do you guys want to keep discussing or should we close this ticket in favour of action-based follow up tickets?

Love it. I'm open to closing this up, and look forward to testing 17.7.3. Do you think it will drop by end of September? I have a deployment starting testing the last week of September that I'd like to try frr out on, based on some initial solid results I've had with frr in linux-land.

17.7.3 is scheduled for Tuesday if all goes well. Worst case Wednesday. 😉 I can't speak for pulling the old switcheroo with the plugin itself (using the quagga configuration logic with frr underneath), but the vanilla package will certainly work just as good as quagga in its place.

On 17. Sep 2017, at 22:10, Michael OBrien<EMAIL_ADDRESS>wrote:
Love it. I'm open to closing this up, and look forward to testing 17.7.3. Do you think it will drop by end of September? I have a deployment starting testing the last week of September that I'd like to try frr out on, based on some initial solid results I've had with frr in linux-land.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Awesome, thanks :) Looking forward to playing with it. No need for the switcheroo, this is a pretty minimal config.

I have to look how much time I have but I may be able to give it a try.

Installed frr via pkg install, replaced quagga, configured manually in /usr/local/etc/frr/ files, and OSPF is working beautifully against 2 other OPNSense hosts running quagga.

@fichtner I take back my previous comment, and add a couple reports - if you want to switch this to separate action-based issues please feel free!

The previous 'success' was the box rememering Quagga routes.
When I try to start ospfd using the same conf file Quagga used, I get, for each interface: MPLS-TE(initialize_linkparams) Could not find corresponding OSPF Interface for (e.g.) igb0
I suspect this might be an issue with perms for the frr user, given the following:
vtysh is running as frr user, so 'copy run start' fails in vtysh (though I don't know if that's critical, as I don't know if OPNSense's Quagga stuff will use vtysh), as it cannot write to the root:wheel owned /usr/local/etc/frr/* files. chowning that dir -R as frr:wheel solved this problem, but may cause many others :)

@obrienmd the current quagga files are owned by quagga - this is done by the setup script.
VTYSH is used by quagga.rb for printing out the routing tables etc.

Thanks for the context - so, my manual chown of the frr config files should have worked. I'll dig into the OSPF error message.

Still trying to figure this one out. It can clearly see the interfaces in 'show interface' on vtysh, but when I try to start the ospf service, it just pukes with that error. Odd one.

Franco, would it be possible to package a clone of the os-quagga UI plugin for frr, so I can see if GUI config just implements something I'm missing?
Not sure how big an ask this is, i.e. if it's just pointing it to a different config directory, or something more complex. Happy if it's broken, not-qc, and I'm just a guinea pig :)

I wouldn't want a full clone yet, maybe we can build a type of QUAGGA_IS_FRR switch into the current quagga port?

On the plugins.git level, not from the GUI. /CC @fabianfrz @mimugmail

@fichtner there are also some path adjustments so it will not completely work. 'quagga' -> 'frr'
However the models and controllers will be probably untouched which means in worst case we have to add some changes to the config files - but I will probably not work on it now.

@fichtner @fabianfrz I'd need some guidance, but I'd be happy to help!

@fichtner @fabianfrz Note, I have a fork of plugins with os-frr alongside os-quagga (not installed at same time), and it works as well as manual config did (i.e. still has MPLS-TE(initialize_linkparams) Could not find corresponding OSPF Interface for (e.g.) igb0 error on all ints).
I'm digging in with frr on slack on that right now.

Ha! Apparently those MPLS-TE log messages are warnings. With my find/replace version of os-quagga installed as os-frr, OSPFv2 is working as expected. Awesome!
Fork is here: https://github.com/obrienmd/plugins
Let me know if you want a pull request - I suspect based on previous comments probably not :) Hope the fork is helpful for experimentation regardless, if only just to save a few minutes of mundane case-sensitive find/replace.

Nice, we can put the changes in a branch and try to integrate them so they expand on plugin build/install. That way we can keep and test both and decide about a replacement timeline. This can take a few months, but come 18.1 we should have a clear decision / strategy of what we want to use in the future. Sounds good?
All I’m saying is we should not carry both. A similar decision may be OpenSSL/LibreSSL in the long term... :)

On 27. Sep 2017, at 18:29, Michael OBrien<EMAIL_ADDRESS>wrote:
Ha! Apparently those MPLS-TE log messages are warnings. With my find/replace version of os-quagga installed as os-frr, OSPFv2 is working as expected. Awesome!
Fork is here: https://github.com/obrienmd/plugins
Let me know if you want a pull request - I suspect based on previous comments probably not :) Hope the fork is helpful for experimentation regardless, if only just to save a few minutes of mundane case-sensitive find/replace.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I'd wait till v3 .. no matter if 18.1 is behind or ahead :)

Fair enough. All the more reason to integrate FRR into the current port so that new features benefit both flavours. I’ll look at this in detail in October.

On 27. Sep 2017, at 18:46, Michael<EMAIL_ADDRESS>wrote:
I'd wait till v3 .. no matter if 18.1 is behind or ahead :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Makes sense. Let me know if I can help in any way!

I think @mimugmail is right - there is a RC available at the moment:
https://github.com/FRRouting/frr/releases
So for integration waiting for stable is probably the best idea before any changes are done.

Cool, makes sense.

I'm going to love FRR and hate Quagga. All my 4 core routers were affected by this:
https://lists.quagga.net/pipermail/quagga-users/2017-September/014830.html
All iBGPs were flapping and 2 of the 4 cores crashed. If 4 were crashed the complete AS would be gone.
FFR fixed this in code ... and Quagga will stay with it I believe. :/
https://github.com/donaldsharp/frr/commit/084002351fbfd6b4e2d9c4c218288b2324ad81cd

@mimugmail you can ask @fichtner if he can build an rc package of frr 3.0 for you and then you can start porting your stuff. Just keep everything except those things which have be changed (paths, filenames etc.) to keep the plugin functional and don't loose the configuration (model refers to the same XML element).

@fichtner I'd be interested in that rc package of frr 3.0 as well!

3.0 seems to have been released: https://github.com/FRRouting/frr/releases/tag/frr-3.0
I'm working on a port update at the moment. :)

Done, packages tomorrow... https://github.com/opnsense/ports/commit/5f90b819d
I can't find a 3.0 release announcement, looks like this either isn't final or still being assembled...

We have pushed FRR to FreeBSD ports as well. It looks like it's working ;)

FRR will be version 3.0.2 in 17.7.8. If anyone wants to start migrating/copying the quagga plugin, that would be a good time to start it on our way to 18.1 ;)
From the model perspective, maybe we can keep "quagga" naming, but only change the internal service usage.

Plugin exists, probably hitting 1.0 with 18.1.

@fichtner if this is released, os-frr should be released at the same time.

@fabianfrz yes, this will both automatically land on stable/18.1. maybe os-frr should be bumped right now to 1.0 as well?

sure

great, thanks.. see https://github.com/opnsense/plugins/commit/7a8fe4a6


Migrate to Radix v4

Describe the solution you'd like

When Radix v4 is out of Beta, Data Source should be migrated to v4.
Additional context

https://github.com/mediocregopher/radix

I think Radix v4 is out of beta. I see no indication that v4 is in beta, but v3 is still the default branch.

Hi @leandro-deveikis no worries, I kept this issue open in case it became apparent that there was a compelling reason to upgrade from Radix 3->4. AFAIK there isn't. Looks like there are some quality-of-life bits and some things around the edges - but I don't see anything, as of this writing, that stands out to justify the effort/risk of revving a major version.
Just so you know, totally open to being wrong about this, I'm conversant in golang but not fluent - if there's a really compelling reason to do it please by all means lay it out for me and I can definitely consider moving forward with the upgrade.
Another possible path (if a change is necessary and if the upgrade is more than a simple version change) is to switch over to the official Redis golang client (go-redis).
Atm the client seems fine and as far as I know is still receiving some patches (though the current version is a year old at this point). So the old adage "if it ain't broke don't fix it" resonates with me a lot here - at least in the interim.

Hi @slorello89  Sorry for the super late response.
For a little bit of more context, I am working on PDC (Private datasource connect - link to docs) on Grafana, and I would like to contribute to enabling PDC on the Redis Datasource, to allow grafana customers to securely connect to their remote Redis instances.
The problem is that RadixV3 doesn't allow configuring a custom dialer, making it impossible to implement PDC. However, if you move to RadixV4, it would be really easy to implement (setting NetDialer here).
I would happily pick up that task and open a PR once V4 is already implemented if you like to.

Hi! Any update on this?

Hello, I'm interested to contribute to make this move forward :)
Can you confirm that if I open a MR, someone will be able to review it?


Add blog post regarding 2.0.0 GA release

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind documentation
[skip ci]
What does does this PR do / why we need it:
Adds a blog post which will go live on September 23rd regarding the
2.0.0 release.
Which issue(s) this PR fixes:
N/A
PR acceptance criteria:

[X] Unit test

[X] Integration test

[X] Documentation

[X] I have read the test guidelines

How to test changes / Special notes to the reviewer:
N/A
Signed-off-by: Charlie Drage<EMAIL_ADDRESS>

I am not sure but i would also add the odo slack channel odo-users info in here somwhere so that anyone reading can comment if they have questions or anything to say.


Try to store history data and latest data in a single collection

Based on discussion with Benjamin and this blog, lets try to keep history data for all models inside the single document.

Implemented.


Confidence intervals from error estimation from backtest

🚀 Feature Request
We can build confidence intervals for generative models or models like Arima out of the box.
But we miss this opportunity for Regression models.
We should build intervals from backtest results for such models.
Motivation
Confidence intervals are helpfull for risk estimation.
Proposal

Pipeline method forecast has  parameter confidence_interval by default False
Pipeline __init__ has parameters interval_width by default 0.95 and  confidence_interval_cv by default 3
While forecast -- default implementation of confidence intervals is used, if it's not available -- we use another procedure

Estimating errors on crossvalidation results using backtest
confidence interva is given by (y_{i} - t_{\alpha} * SE,  y_{i} + t_{\alpha} * SE) - SE -- standart error estimation

Test cases
No response
Alternatives
No response
Additional context
No response
Checklist

[X] Added feature request
[X] Added motivation
[X] Added proposal

If model already has confidence interval support then use model's method, otherwise use Pipeline's


[feature] Implement Map with insertion order

🚀 Feature request
Implement Map to hold the order of elements, using the order of insertion to keep track of it.
Current Behavior
Currently, Map's implementation does not have any typeclass properties such as sort and traverse on the instance.
We can see that the implementation of Map does not keep track of the order, as the toArray needs and Ord<A> to give it an order:
export declare const toArray: <K>(O: Ord<K>) => <A>(m: Map<K, A>) => [K, A][]

Desired Behavior
Map should have many of the methods that the Array typeclass has, because Map's are ordered, as seen in the link provided under "Additional context".
The toArray method should look like this:
export declare const toArray: <K, A>(m: Map<K, A>) => [K, A][]

Suggested Solution
Add each item in this non-exhaustive list of methods and typeclasses to the Map and ReadonlyMap typeclass:

[ ] TraversableWithIndex2<URI>
[ ] FunctorWithIndex2<URI>
[ ] Compactable2<URI>
[ ] Monad2<URI>

I would love to hear some feedback on the viability of the list above, as I haven't tried to implement these myself yet.

Who does this impact? Who is this for?
All users who work with map.
This looks like a breaking change if done to the current Map typeclasses.
Instead, we could add OrderedMap and ReadonlyOrderedMap.

Describe alternatives you've considered

Creating a new URI for "OrderedMap" and "ReadonlyOrderedMap", to avoid breaking changes.
Replacing the implementations of Map with the implementation mentioned in this issue, which is a breaking change.
getWitherable<A>(Ord<A>) looked promising, but it requires an Ord to work, which it shouldn't really need.

Additional context
Key Order

The keys in Map are ordered. Thus, when iterating over it, a Map object returns keys in order of insertion.

Go to the Objects vs. Maps section of the page:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map

Your environment

Software
Version(s)

fp-ts
^2.5.4

TypeScript
^3.8.3

using the order of insertion

The "order of insertion" is not actually an order, you can't define an Ord for it.
If you don't sort the keys with an Ord, functions like toArray are not a pure
const m1 = new Map([
  ['a', 1],
  ['b', 2]
])

console.log(...m1.keys()) // a b

const m2 = new Map([
  ['b', 2],
  ['a', 1]
])

console.log(...m2.keys()) // b a

m1 is equal to m2 but a b is not equal to b a (i.e. keys is not pure)...
...unless all keys are equal. Indeed you can get what you want by passing the trivial Ord
import * as M from 'fp-ts/lib/Map'
import { Ord, fromCompare } from 'fp-ts/lib/Ord'

const trivial: Ord<unknown> = fromCompare(() => 0)

console.log(M.keys(trivial)(m1)) // [ 'a', 'b' ]
console.log(M.keys(trivial)(m2)) // [ 'b', 'a' ]

but that's a hack.

m1 is equal to m2 but a b is not equal to b a

I see. We learn something new every day! Thank you for your explanation and response.


Table: support range selection

With this PR, you can press shift to enable range selection. My team is asking for this feature for a while, and I believe others would love this as well.

[x] Make sure you follow Element's contributing guide (中文 | English | Español).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

Thanks for your PR! It seems awesome feature!
Could you supplement some test cases for that? We will add this feature in next major release.

Okay I added a test case. Also, improved range selection by supporting range un-select (exact behavior of Gmail). This is how it looks like:


Cant do a lot of things after tweaks

I cant restore my pc, take screenshots, search and i cant open my settings anymore either.
When i try to restore my settings/system it gives the error: VSS Encountered Problems while sending events to writers. (0x80042315
I would love to know wether i need to reset my pc to have normal access to it again.

So,
Just looking at and reading your "Issue" it sounds like you ran the script on a live system you require use of without first assessing the possible risks involved with running this application and it's various automated scripts.
My personal suggestion to you would be to simply rollback to a system state where your system was functioning (if you have backups enabled and actively being kept up-to-date).
OR If you did not have backups enabled, or, your backups were not up-to-date then I suggest that you move what files you can to a secondary drive and then do a reinstall of your exact Windows OS (Windows 11 or Windows 10 depending on which OS you are running).
NOTE FOR FUTURE REFERENCE
In the future I suggest that you research the risks involved with blindly running scripts on your system without first being prepared with a system-state backup and being well aware of the fact that this utility is not a commercial tool and so relies on the community to ensure that it is bug-free.
Unfortunately because of the tools limited user base only so many people will donate time to this project to help keep it working fluidly, if you want to see the project improve then offer up what free time you have to help improve it for everyone that uses it, after all it is a community effort with Chris as the driving force at the front!

try running the system corruption scan in the config tab

or you could

Click on Start, type Services in Start Search. When the Services page is open, check for Volume Shadow Copy service.

Right click on Volume Shadow Copy service (VSS) and Check if it is started.

If it is stopped select Start or Restart.

VSS by default is set to manual and isn't changed. Simply running the tool should fix this as I updated it in march to set these to defaults.


Always generate a valid parse tree, even on incomplete input

Since 5c8a8d5eecfab642581b66d00efe9c21e8a46bf2, a parse error no longer causes the parser to exit but rather to continue parsing.
Still, all places where errors are registered should be scrutinized, to ensure 1) the parser always terminates 2) the AST is always valid, and optionally, the error recovery strategy is "sensible".


Fix attribute redirects.

When these pages were moved during the PR changes, I forgot to update the redirects.

Too bad the linkchecker couldn't see those links; but still, they're fixed for now.


"Access to Information" page does not show up in Uber app, while doing native SSO authorisation

Hello There,
I have integrated rides-ios-sdk for SSO authorisation in my native iOS application, It is working fine until I launch Uber app and open left menu.
Here is the use case:

Launched Uber application and open left menu in Uber app.
Performed SSO authorisation from my native iOS application.

Result:
Access to Information that is authorisation page does not show up, it just launches the Uber application.

Uber application version : v3.299.10002
Uber SDK version : 0.9.0
iOS version: 10.3.2
Deeplink: uberauth://connect?third_party_app_name=my-app-name&callback_uri_string=com.XXXX.YYYY://oauth/consumer&client_id=XXXXXXXXXXXXXX&scope=all_trips&sdk=ios&sdk_version=0.9.0

This is fixed in iOS version 3.300.x; you'll see it released soon!

How was your report on 3.299? Is that a typo? 3.299 isn't on the App Store yet.

Are there any "states" within the Uber app for which a native SSO request is ignored? Specifically, the app was in the background after a ride, with the ride feedback request at the botto, and a native SSO request from our app didn't result in showing the "Access to Information" page - the Uber app went to the foreground and was showing the option to give feedback at the bottom. Going back to our app and then attempting native SSO again resulted in the same behaviour multiple times.
This is on the iPhone X (iOS 12.1)with Uber version v3.325.10003 and uber-rides SDK 0.10.
Thanks!


No matching manifest entry

Describe the problem
I had to restart the server today, which caused the automatic update to trigger. Since some players are still on MC 1.19.4 I wanted to downgrade the server using the "VERSION" environment variable.
At first it did nothing. After updating the container itself it now gives me errors in the init.
I checked the user id and group ID those are right and the previous .jar file is still there.
Container definition
mc:
    image: itzg/minecraft-server
    ports:
      - 25565:25565
    environment:
      - EULA=true
      - MEMORY=4G
      - VERSION=1.19.4
    tty: true
    stdin_open: true
    restart: unless-stopped
    volumes:
      # attach a directory relative to the directory containing this compose file
      - ./minecraft-data:/data
    networks:
      - services

Container logs
[init] Running as uid=1000 gid=1000 with /data as 'drwxrwxr-x 7 1000 1000 4096 Jun 12 05:06 /data'
[init] Resolved version given "1.19.4" into "1.19.4" and major version "1.19
[init] Resolving type given VANILLA
[init] ERROR: couldn't find a matching manifest entry for "1.19.4"

It was my own mistake. I forgot to rebuild the container. In the process of trying to fix it, I tried quotes around the version, which the container didn't like


Memoize

Adds memoize functions and tests for it.
(Benchmars will be later)

current changes are merged to release/2.0.0, for other improvements please open new pr.


rpc trying to pay same invoice twice should return preimage as well

If you pay a bolt11 invoice twice, first time you get the preimage in the json, but the second time you only get
{'status': 'invoice is already paid'}
Could we receive the preimage as well?

If you do TrackPayment, you'll get the last update which'll include the pre-image if the payment has completed: https://api.lightning.community/#trackpaymentv2

@yyforyongyu I think this issue is already resolved ?

@yyforyongyu Is this issue already resolved ?

I think it's correct that an error is returned, that way app developers can be certain that the payment didn't succeed. If we went ahead and added a new status (for example INVOICE_ALREADY_PAID) then API users that aren't aware of the new code would simply show the payment as succeeded.
@saubyk since the proper way is to check using TrackPayment I would vote for closing the issue.
@mohamedawnallah the --json flag only affects the successful output (to switch from the live-updated table view to a JSON rendered output). But you got an error message, which in lncli is always formatted that way.

Agree with the assessment here. Doesn't make sense to break the existing rpc behavior to return pre-image, when we have an rpc to get the pre-image, if required.
developers can catch the error status 'invoice already paid' and invoke trackpayment to get the pre-image.
closing this issue


feat(raccoon): fix buf lint issues

Following changes are done -

Standardised raccoon files with respect to other odpf projects.
Fixed lint issues

Signed-off-by: ramey<EMAIL_ADDRESS>

@ramey Let's stick with v1beta1 for API version.

@ramey Let's stick with v1beta1 for API version.

@ravisuhag we already use v1 in raccoon. That's why made it v1. I think we should be consistent with both and if we make it v1beta1 then It will be a breaking change in Raccoon.

@ramey can you illustrate where we are using v1 right now?

@ravisuhag v1 is in API path that clients use to connect to Raccoon raccoon.gojekapi.com/v1/events. Agree, we are introducing versioning for the first time but we already have v1 APIs. Won't that be a little confusing or it doesn't matter?

I think until our APIs are not stable we should not release v1. One way to deal with raccoon hard coded path is to add alias for now. And give a timeline to EOL.

Currently we have two sets of APIs, the old EventRequest and EventResponse data models which are stable and that's why we were using v1 and the new gRPC APIs which are not, so it makes sense for them to be under v1beta1.  The new gRPC API's are dependent on the old data models. So we can do two things here, either create two directories under raccon for data models and service and both have their own semantic versioning or we leave it the way it was previously and just do other field level changes? WDYT @ravisuhag

@ramey - Can we keep the SendEventRequest/SendEvent/SendEventResponse data models outside the raccoon.proto? These models are not coupled with gRPC and they are used by the http APIs as well. We are not generating the http paths from the service protos either.
We could refactor and merge the service/model protos when we release v1 version of the rpc protos.
Meaning, today:
odpf/raccon/ ==> All models go here
odpf/v1beta1/ ==> gRPC service proto goes here
when gRPC moves to a stable version, we would have:
/odpf/raccoon/v1/raccoon.proto ==> at this point: we will have http path generated as well here!

@chakravarthyvp Any reason to put models in v1beta1?

@chakravarthyvp Any reason to put models in v1beta1?
We shouldn't put the models in v1beta1.
I think @ramey has included this in v1beta1 as part of raccoon.proto (service proto). I'm proposing that this should be moved out of this version and be at higher level as the models are reused by http APIs & gRPC.

https://github.com/odpf/proton/pull/126#issuecomment-1110540829
Do you agree with this approach @ravisuhag ?

@chakravarthyvp Any reason to put models in v1beta1?

We shouldn't put the models in v1beta1. I think @ramey has included this in v1beta1 as part of raccoon.proto (service proto). I'm proposing that this should be moved out of this version and be at higher level as the models are reused by http APIs & gRPC.
#126 (comment)
Do you agree with this approach @ravisuhag ?

I meant not to put in V1beta1.

@ravisuhag - The models (which are ideally payloads) are used by the http APIs and re-used by RPCs. This has to be agnostic of which API/RPC uses it. Clients should be able to use these models along with the API they choose and needn't tie with the RPC versions.

@chakravarthyvp v1beta1 version we want to add in the proton is not specific to just the RPC but for all proto messages actually. So, I would recommend attaching a version to models as well. We can for sure think if the versioning of models and GRPC service should be maintained separately or not.


Db not working

Hi,
I just updated to the latest version and now the server seems to not expose my data (as if the json file wasn't found, except it says it loaded it).
`sbayeta@DESKTOP-6NQVI8N:~/repos/QFrontend$ json-server mockdb.json
{^_^}/ hi!
Loading mockdb.json
Done
Resources
http://localhost:3000/posts
http://localhost:3000/comments
http://localhost:3000/profile
Home
http://localhost:3000
Type s + enter at any time to create a snapshot of the database
GET /posts 200 9.393 ms - 77
GET /phones 404 2.239 ms - 2`

Sorry guys, apparently the server over wrote my db file and I didn't notice.


Invalid security certificate error

When i tried downloading the dataset manually from the website, http://www.cs.utoronto.ca/~kriz/cifar.html , I get an invalid security certificate error.

Try https://www.cs.toronto.edu/~kriz/cifar.html
They probably have a certificate that is valid for *.toronto.edu but not *.utoronto.ca (they point to the same thing).

Yeah. Got it. Thanks! :+1:

If we refer to the other URL somewhere, a PR would be welcome.


Include workload field as metadata

Signed-off-by: Raul Sevilla<EMAIL_ADDRESS>Description
Including the workload name as metadata field will be useful to perform comparisons. The value of this field, namely WORKLOAD, is already set by the different benchmarks of this repo.

/rerun all

This is going to be very useful on uperf dashboards

LGTM

Results for e2e-benchmarking CI Tests

Workload
Test
Result
Runtime

router-perf-v2
./ingress-performance.sh
PASS
00:05:25

etcd-perf
./run_etcd_tests_fromgit.sh
PASS
00:03:06

scale-perf
./run_scale_fromgit.sh
PASS
00:13:14

network-perf
./run.sh
PASS
00:05:20

kube-burner
WORKLOAD=cluster-density ./run.sh
PASS
00:05:28

kube-burner
WORKLOAD=node-density ./run.sh
PASS
00:05:58

kube-burner
WORKLOAD=node-density-heavy ./run.sh
PASS
00:06:09

kube-burner
WORKLOAD=pod-density-heavy ./run.sh
PASS
00:06:01

kube-burner
WORKLOAD=max-services ./run.sh
PASS
00:04:48

kube-burner
WORKLOAD=max-namespaces ./run.sh
PASS
00:05:18

upgrade-perf
./run_upgrade_fromgit.sh
PASS
00:00:58


Interest in ownership?

Hey Internetz,
Do any of you want to own this library?  I'd love to transfer it to a responsible owner.
Please respond to this thread with any interest.
Thanks,
Jake

Hi Jake, I'll take it over, I'm planning on using it for a small website and am willing to maintain the repository.
Please let me know how you would like to transfer ownership.

@couetilc , what's your npm username?  I want to add you there so you can get the 5.0 release published when you feel it's ready.

Hi @jaketrent, my npm username is https://www.npmjs.com/~couetilc, thank you. I'll get 5.0 published once the github action for publishing is set up


[BUG] Dask DataFrame init has an error when there are None values

Minimal Code To Reproduce
from fugue_sql import FugueSQLWorkflow
from fugue_dask import DaskExecutionEngine

# fugue.dask.dataframe.default.partitions determines the default partitions for a new DaskDataFrame
engine = DaskExecutionEngine({"fugue.dask.dataframe.default.partitions":4})

# Creating sample data
data = [
    ["A", "2020-01-01", 10],
    ["A", "2020-01-02", None],
    ["B", "2020-01-01", None],
    ["B", "2020-01-02", 40]
]
schema = "id:str,date:date,value:int"
with FugueSQLWorkflow(engine) as dag:
    df1 = dag.df(data, schema)
    dag("""
    SELECT id, date, value FROM df1
    PRINT
    """)

Describe the bug
There is a DataFrame Init Error
pyarrow.lib.ArrowTypeError: ("Expected bytes, got a 'int' object", 'Conversion failed for column value with type object')
Expected behavior
It should work
Environment (please complete the following information):

This happens in the dev container environment.

The issue is double can handle values but int cannot for the DaskExecutionEngine.


Add the docker database to docker compose

The database will not be duplicated in the backend, this was a backup solution.


Cannot execute with UPDATE PUT/ADD/INCREMENT new executor: ADD members = #xxxx:xxx DB name="XXXX"

OrientDB Version:
Java Version:
OS:
windows 10
Expected behavior
Add a new link target to exist LINKSET by SQL like:
update #13:358 add members = #151:160
Actual behavior
Oritent DB Ver3.0.23 popup error message: "Cannot execute with UPDATE PUT/ADD/INCREMENT new executor: ADD roles = #xxxx:xxx"
Steps to reproduce
<An SQL script to reproduce the problem or a JUnit test case will increase A LOT the chance to have a quick fix>
class property MyTeam.members defined as LINKSET, could not use SQL:
update <recordid: #xxx.xx> ADD members = <userid: #xxx.xxx>.
but insteadof 'ADD', 'SET/REMOVE' work well

how to use isSet() with this query , to add only if it does not exist in the array ?


ITE drivers/timer: fix timer

1.We don't need to convert the free run clock count,
that will be converted by the kernel
(base on CONFIG_SYS_CLOCK_TICKS_PER_SEC),
so we should return the HW register count value directly.
2.ITE RTOS timer HW frequency is fixed at 32768Hz, because this
clock source is always active in any EC mode (running/doze/deep doze).
3.When run the test_mutex_lock_timeout(), we need more time (>300us) for the testing
thread to finish the job, or we will get a fail (we didn't run the test before).
Because our event timer doesn't handle the float part, and 32768 is divisible by 8192 which
is closest to kernel tick default 10000, I add CONFIG_SYS_CLOCK_TICKS_PER_SEC = 8192
for it8xxx2_evb.
If the CONFIG_SYS_CLOCK_TICKS_PER_SEC = 32768, the 10 tick of timeout = 300us.
If the CONFIG_SYS_CLOCK_TICKS_PER_SEC = 8192, the 10 tick of timeout = 1200us.
So we can get more time to finish the job.
fixes https://github.com/zephyrproject-rtos/zephyr/issues/43513
fixes https://github.com/zephyrproject-rtos/zephyr/issues/42847
Signed-off-by: Ruibin Chang<EMAIL_ADDRESS>

You added CONFIG_SYS_CLOCK_TICKS_PER_SEC definition but didn't describe why you need that now but didn't before. Guessing the change to not convert system clock cycles to ticks in the other file is linked to this.
The commit message does not tell me anything about what is changing or why. Please change the commit message to describe the change.

Done

Update commit message (delete redundant space)

Rebase the commits


remove unnecessary parameters

Make sure set the target branch to develop
What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

conflict.. better to update your develop branch to the lastest, and try to create a new PR.

conflict.. better to update your develop branch to the lastest, and try to create a new PR.

ok

conflict.. better to update your develop branch to the lastest, and try to create a new PR.

ok


Fix azure keyvault managed identity docs cause AAD Pod Identity fields are now case sensitive

Describe the issue
Fix Azure Key Vault managed identity docs cause AAD Pod Identity fields are now case sensitive.
More info about this change here
URL of the docs
https://docs.dapr.io/operations/components/setup-secret-store/supported-secret-stores/azure-keyvault-managed-identity/
Expected content
Correct configuration for AzureIdentity and AzureIdentityBinding must read:
apiVersion: "aadpodidentity.k8s.io/v1"
kind: AzureIdentity
metadata:
  name: [you managed identity name]
spec:
  type: 0
  resourceID: [you managed identity id]
  clientID: [you managed identity Client ID]
---
apiVersion: "aadpodidentity.k8s.io/v1"
kind: AzureIdentityBinding
metadata:
  name: [you managed identity name]-identity-binding
spec:
  azureIdentity: [you managed identity name]
  selector: [you managed identity selector]

@orizohar since #989 was merged I'm closing the issue.


WLED not working in V1.12

Home Assistant Adaptive Lighting Issue Template
Bug Reports
Before submitting a bug report, please follow these troubleshooting steps:
Please confirm that you have completed the following steps:

[x] I have updated to the latest Adaptive Lighting version available in HACS.
[x] I have reviewed the Troubleshooting Section in the README.
[ ] (If using Zigbee2MQTT) I have read the Zigbee2MQTT troubleshooting guide in the README.
[x] I have checked the V2 Roadmap and open issues to ensure my issue isn't a duplicate.

Required information for bug reports:
Please include the following information in your issue.
Issues missing this information may not be addressed.

Debug logs captured while the issue occurred. See here for instructions on enabling debug logging:

Debug logs for version 1.11 (that works): home-assistant_adaptive_lighting_2023-05-25T18-28-38.253Z.log
Debug logs for version 1.12: home-assistant_adaptive_lighting_2023-05-25T18-33-37.614Z.log

Your Adaptive Lighting configuration:

lights:
  - light.wled_sink
prefer_rgb_color: true
include_config_in_attributes: true
initial_transition: 0
sleep_transition: 1
transition: 0
transition_until_sleep: false
interval: 10
min_brightness: 1
max_brightness: 45
min_color_temp: 2000
max_color_temp: 5500
sleep_brightness: 1
sleep_rgb_or_color_temp: color_temp
sleep_color_temp: 1000
sleep_rgb_color:
  - 255
  - 56
  - 0
sunrise_time: null
max_sunrise_time: null
sunrise_offset: 0
sunset_time: null
min_sunset_time: null
sunset_offset: 0
only_once: false
take_over_control: false
detect_non_ha_changes: false
separate_turn_on_commands: false
send_split_delay: 0
adapt_delay: 0
autoreset_control_seconds: 0
name: Sink

(If using Zigbee2MQTT), provide your configuration files (remove all personal information before posting):

devices.yaml
groups.yaml
configuration.yaml ⚠; Warning REMOVE ALL of the PERSONAL INFORMATION BELOW before posting ⚠;

mqtt: server:
mqtt: user:
mqtt: password:
advanced: pan_id:
advanced: network_key:
anything in log_syslog if you use this

Brand and model number of problematic light(s)

Describe the bug and how to reproduce it:
After upgrading to V1.12, some of my WLED lights do not adapt anymore.

Steps to reproduce the behavior:
Upgraded to the newest version, no clue how to reproduce is, but restarting or reconfiguring does not seem to change anything. Downgrading to 1.11 makes them work again.

Hello, i have the same problem here.

What's the brand and model number of your lights?

This particular light is controlled via a QuinLED DigUno controller board, running WLED version 0.13.3.
This LED strips itself are a SK6812 RGBW ledstrip from BTFLight.

What's the brand and model number of your lights?

Hello, it's a strip led controled with Shelly RGBW2

Controller: Shelly RGBW2 (V. 1.13.1)
LED strip: Paulmann MaxLED 250 - PN 79867

same problem with WLED and some SK6812 , went back to 1.11 to solve it.

I have the same problem, the shelly rgbw2 lights don't adapt... I've tried 1.12, 1.13 and 1.14, none of them resolved it, with 1.11, which is where I'm staying, they work perfectly

Same here..

Same issue for me also using 1.12, 1.13, and 1.14. I am using SK6812 running directly off an ESP8266 in both instances

Came here to confirm and also post something else I found out. When you have a split the lightstrips in multiple segments, you get a Master light. That one does adapt with this integration still somehow (brightness). The Master only only has brightness, but that does still adapt.

This also isn't working for me on 1.14 with an Athom LS-5P (an ESP2866 running WLED). I get changes using 1.11, but the color is way warmer than it should be.

good morning, do you know if this already works in the .15? Until the .14 I could go back to the .11, but when I left the .15 it only appears to go back to the .12 that doesn't work... how could I make a backup of the .11 to test if it works or not, and if not, to be able to go back to the .11 ..

Still not fixed in .15

Unfortunately, this hasn't been addressed yet. The code introducing the bug is likely here: https://github.com/basnijholt/adaptive-lighting/compare/1.11.0...1.12.0#diff-2ae6ce86e459ad0e8703be113d8bd00c7fb232c57bc6133c2f3e5a680fd008b2

I am looking into this.
For the lights that do not work, what supported_features is reported in the Developer Tools?
For example, my light here is 44:

I have opened a PR #637 where I attempt to fix this issue.
Can anyone give it a try?

i installed the newest adaptive lighting version on my HA version 2023.7.2 and patched the switch.py as you suggested.
adaptive lighting on my wled works now, however it breaks a couple of my mqtt lights. this is what i get when i try to turn on my light.kinderzimmer_decke_light light (from zigbee2mqtt):
Logger: homeassistant.components.websocket_api.http.connection
Source: custom_components/adaptive_lighting/switch.py:310
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 22:03:52 (1 occurrences)
Last logged: 22:03:52

[140533052812944] find_switch_for_lights: Light(s) ['light.kinderzimmer_decke_light'] not found in any switch's configuration. You must either include the light(s) that is/are in the integration config, or pass a switch under 'entity_id'.
Traceback (most recent call last):
  File "/home/homeassistant/homeassistant_p3.11/lib/python3.11/site-packages/homeassistant/components/websocket_api/commands.py", line 205, in handle_call_service
    await hass.services.async_call(
  File "/home/homeassistant/homeassistant_p3.11/lib/python3.11/site-packages/homeassistant/core.py", line 1965, in async_call
    response_data = await coro
                    ^^^^^^^^^^
  File "/home/homeassistant/homeassistant_p3.11/lib/python3.11/site-packages/homeassistant/core.py", line 2005, in _execute_service
    return await cast(
           ^^^^^^^^^^^
  File "/home/homeassistant/.homeassistant/custom_components/adaptive_lighting/hass_utils.py", line 46, in service_func_proxy
    await intercept_func(call, data)
  File "/home/homeassistant/.homeassistant/custom_components/adaptive_lighting/switch.py", line 1810, in _service_interceptor_turn_on_handler
    adaptive_switch = find_switch_for_lights(self.hass, [entity_id])
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/homeassistant/.homeassistant/custom_components/adaptive_lighting/switch.py", line 310, in find_switch_for_lights
    raise ValueError(
ValueError: find_switch_for_lights: Light(s) ['light.kinderzimmer_decke_light'] not found in any switch's configuration. You must either include the light(s) that is/are in the integration config, or pass a switch under 'entity_id'.

@diplix, thanks for reporting back. That issue should also be fixed now.
Please install v1.16.0 😄
Thanks all for your patience and support! 🙌

For what it's worth, I worked around this issue by creating a template light, which also let me insert a blueprint "Use RGBW as CT light" so that I could calibrate the color temperature of my WLED lights to match my other lights.


RTL: Table: Lack of right alignment and other issues

Columns

Column content doesn't right-align
Date header doesn't right-align, but others do

Search

Icon doesn't switch to right of text entry

Column controls
Text in column elements doesn't right-align

Filters
Dropdown option menu doesn't right-align

Expanding rows

Content in expanded row doesn't right-align
Labels don't shift to the right of values

Overflow menu

No padding btw text and icon

Multi-select

Actions don't move to left, items selected doesn't move to right (in the table toolbar, those things do switch)
Divider line btw Cancel and other actions disappears

Page control

Items per page element doesn't right-align and overlaps with other elements
Divider lines aren't in the right place
Next / Back arrows point wrong way

Dumb table & minitable
Content and headers don't right align

Nested table rows

Indent in nested rows offsets to the right, should indent to the left

No data & custom empty state

"no data" texts aligns left, not right

Horizontal scroll - full width

the end of the right-most column can't be horizontally scrolled to, which means you'll lose pieces of primary information when they are right-aligned.  The reader will lost the beginning of everything. (this is actually a problem in LTR as well. submitted as issue #448)

[x] Column content doesn't right-align
[x] Date header doesn't right-align, but others do

Columns

[x] Column content doesn't right-align
[x] Date header doesn't right-align, but others do

Search

[x] Icon doesn't switch to right of text entry

Column controls

[x]Text in column elements doesn't right-align

Filters

[x] Dropdown option menu doesn't right-align

Expanding rows

[x] Content in expanded row doesn't right-align

~Labels don't shift to the right of values~ - this is user provided content

Overflow menu

[x] No padding btw text and icon

Multi-select

[x] Actions don't move to left, items selected doesn't move to right (in the table toolbar, those things do switch)
[x] Divider line btw Cancel and other actions disappears

Page control

[x] Items per page element doesn't right-align and overlaps with other elements
[x] Divider lines aren't in the right place
[x] Next / Back arrows point wrong way

Dumb table & minitable
~Content and headers don't right align~ - resolved in other PR

Nested table rows

[x] Indent in nested rows offsets to the right, should indent to the left

No data & custom empty state

~"no data" texts aligns left, not right~ - this design has changed and defect is no longer present

Horizontal scroll - full width
*~ the end of the right-most column can't be horizontally scrolled to, which means you'll lose pieces of primary information when they are right-aligned.  The reader will lost the beginning of everything. (this is actually a problem in LTR as well. submitted as issue #448)~ - resolved in other PR

:tada: This issue has been resolved in version 2.78.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Path to virtual environment script

Sorry for the noob question, but may I ask what path should I fill in the Script section for the Virtual Environment task? I tried absolute/relative paths of my Python/Shell scripts, but the task report keeps telling me that the file does not exist.

The path should be relative to the workspace directory. However, if you're trying to run a script outside of your workspace, I realize it may not be constructing the path correctly -- I'll fix that. If you are trying to run a script inside your workspace, I guess I'd check that you've set a Python Interpreter in the Python extension settings for your workspace?

You'll also want to make sure the script is executable. The task tries to run it directly, not running it as an argument to python or sh or anything.

You'll also want to make sure the script is executable. The task tries to run it directly, not running it as an argument to python or sh or anything.

Thanks a lot! I can now run my script by making it executable.


Recommended example for :backdrop

Hey 👋
Might be cool to have some kind of support for :backdrop behavior or a heads-up on how you might implement that yourself.
Could be in something like:
[popover].:open:before { ...backdropStyles }
@supports ([popover]:backdrop) { ...backdropStyles }

Although using the pseudoelement runs into the issue of getting the :backdrop beneath the content. Might be nice to give some kind of guided snippet for those wanting that behavior 👍
@jh3y

One solution, if your browser supports :has:
::backdrop, body:has(.\:popover-open)::before {
  z-index: 1;
  content: '';
  width: 100%;
  height: 100%;
  position: fixed;
  left: 0;
  top: 0;

  background: #0004;
  backdrop-filter: blur(2px);
}

.\:popover-open {
  z-index: 2;
}

[popup]:popover-open {
  z-index: 2;
}

@mirisuzanne @keithamus What do you think about this?

I agree these are the same issue - so we could close the newer one as a duplicate. Overall, I think it's a good idea to provide a solution here. I'd have to play around with the different options to know what solution I like best. It seems to me like providing an additional wrapping element might be the most robust solution?

I closed the newer issue. @mirisuzanne Is this something you'd have time to look at? Or maybe @stacyk?

Looking again, I think the body:has() trick from @joshnuss is pretty promising. Could simplify a bit, maybe:
/* This could be built into the polyfill */
body:has(.\:popover-open)::before {
  content: '';
  position: fixed;
  inset: 0;
}

/* This is the snippet for authors to use */
::backdrop,
body:has(.\:popover-open)::before {
  /* backdrop styles */
}

I don't think we need the z-index stuff, since both parts will be positioned, and body::before should default to the bottom of the stack. I also replaced the combo top + left + width + height with just inset: 0.
But someone should probably test that.

If we don't want to require :has() support, we could probably just inject an element at the start of the body - and do something similar with it?

Another solution for avoiding :has could be to add a class to the <body> element when the popover is open.


POWERMON-199: Use correct kepler namespace in must-gather

Use labels to get correct namespace name.

On a related topic, perhaps status.deploymentNamespace can be added to kepler.


Ensure MINIO_PORT env variable is injected as string

Fixing issue created in https://github.com/sorry-cypress/charts/pull/65
Ran into the following issue https://github.com/kubernetes/kubernetes/issues/82296 where integer environment variables need to be quoted
Use the quote template function to ensure that minio.service.port is injected as a string

@tico24 Does this sounds all good to you?


fixed line chart with negative values

:+1: thanks, exactly what I needed

@nnnick this sounds like a basic feature that is requested in multiple issues. Can we have this in core?

We've recently announced a feature complete status for v1.x and will only be accepting bug fixes for v1.x. Be sure to take a look at v2.0-dev and it's new features that are either finished or currently under development.  If you see your feature is still not implemented, feel free to submit another PR to the v2.0-dev (or later) branch for your feature.


Test shallow clone for the ngitpuller links

For the nbgitpuller url, it appears that https://github.com/jupyterhub/nbgitpuller/pull/67 has been merged.
This should in principle let us add something like depth=1 to the url for the nbgitpuller link on both the Julia and Datascience material and get the shallow clone.
Remains to be tested, though.

Just tested this and it works. We add &depth=1 to the end of the URL.

SHould we add this to the launch bar?  @cc7768  @sglyon  lets discuss at some point.

The nbgitpuller docs recommend only adding it if we see performance hits, and the full clone is fairly quick (at least on the VSE jupyterhub, where I tested this.)
That said, I don't see the harm in it.

OK, lets hold off then.


Obfuscation of eval expressions

I should implement obfuscation of eval expressions.
Idea: convert all content of eval into AST-tree before main obfuscation stage and then convert it back to eval expression after obfuscation stage.

out in<EMAIL_ADDRESS>


Fix inclusion with readonly classes

Description:
See https://github.com/ballerina-platform/ballerina-spec/issues/649#issuecomment-728617706

Fixed in #27391, merged via #27652.

Fixed in #27391, merged via #27652.


Update Docker images used in test

Desired Outcome
Fix failing Jenkins build. Logs:
Error response from daemon: manifest for vault:latest not found: manifest unknown: manifest unknown

failed to solve: phusion/baseimage:master-amd64: docker.io/phusion/baseimage:master-amd64: not found

Implemented Changes

vault is deprecated in favor of hashicorp/vault
phusion/baseimage master tag is obsolete, using jammy-1.0.1 (Ubuntu 22.04)

Connected Issue/Story
N/A
Definition of Done
At least 1 todo must be completed in the sections below for the PR to be
merged.
Changelog

[ ] The CHANGELOG has been updated, or
[x] This PR does not include user-facing changes and doesn't require a
CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code
changes, or
[x] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: [insert issue ID]
[x] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[x] No behavior was changed with this PR

Security

[ ] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[x] There are no security aspects to these changes

Wrong repo 🙃


Inconsistent casing of Distribution target types

The page refers to distribution target types sometimes in lowerCamelCase and sometimes in UpperCamelCase.
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/image-builder-json#distribute-managedimage has them in lowerCamelCase in both the heading and the sample JSON.
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/image-builder-json#distribute-sharedimage has the heading in lowerCamelCase but the sample JSON is in UpperCamelCase.
I think it needs to be UpperCamelCase based on testing? We got the following error when using sharedImage :
Deployment failed. Correlation ID: 8d1f356f-6687-4859-b267-6ecef2f3a110. {
  "code": "ValidationFailed",
  "message": "Validation failed: 'ImageTemplate.properties': Field 'distribute[0].type' has a bad value: ''. Please review http://aka.ms/azvmimagebuildertmplref for details on fields requirements in the Image Builder Template."
}

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

ID: 9ef2d683-1557-7097-0415-ea67a0350718
Version Independent ID: 20667a9c-cd22-ce77-d4d6-61ac10d380c8
Content: Create an Azure Image Builder template (preview) - Azure Linux Virtual Machines
Content Source: articles/virtual-machines/linux/image-builder-json.md
Service: virtual-machines-linux
Sub-service: imaging
GitHub Login: @danielsollondon
Microsoft Alias: danis

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

Thanks for the feedback, I'll take a look.

I've issued a PR to change this. Thanks for being vigilant! I am now closing this issue. Have a good weekend.


test(client): prototype referential integrity tests

Closing as we keep things into one place here https://github.com/prisma/prisma/pull/14221


starter-cloud9/README.md has broken images

Hello,
It looks like there is an issue where the images are hosted.  Someone might want to take a look.
Regards,
David


Tab title not set for Ubuntu windows

Windows Terminal version
1.6.10571.0
Windows build number
10.0.22000.527
Other Software
Ubuntu 20.04.3 LTS
Steps to reproduce

Renamed Ubuntu profile from the default name to just "Ubuntu"
Executed wt new-tab --profile "Ubuntu" --title "My Title" --suppressApplicationTitle from a command prompt

Expected Behavior
I expected to see the tab title as "My Title"
Actual Behavior
The tab title was "My Title" for a fraction of a second and then switched to "jonathanbyrne@DESKTOP-2N2UM3V: /mnt/c/Users/Jonathan Byrne"

This is something that's controlled by the bash PS1 environment variable. The default value is setup in your ~/.bashrc file. Regarding the title in particular, look for a line that is something like this:
PS1="\[\e]0;${debian_chroot:+($debian_chroot)}\u@\h: \w\a\]$PS1"

That's what changes the title to show the current directory and user/host name.
Alternatively, there's is an option in the terminal's advanced profile section to suppress title changes. That will prevent all applications from altering the title.

Thank you!  Suppressing the title change int he advanced profile section fixed it for me.

The --suppressApplicationTitle command line argument was added in Terminal 1.8. If you upgrade your terminal version (the latest is 1.12) the command line argument should start working.

That explains it.  Thank you!


Trail, world origin

Godot version
4.0 beta 16
System information
Operating System: Arch Linux KDE Plasma Version: 5.26.5 KDE Frameworks Version: 5.102.0 Qt Version: 5.15.8 Kernel Version: 5.15.90-1-lts (64-bit) Graphics Platform: Wayland Processors: 8 × AMD Ryzen 5 2500U with Radeon Vega Mobile Gfx Memory: 14,6 GiB of RAM Graphics Processor: AMD Radeon Vega 8 Graphics Manufacturer: HP Product Name: HP Laptop
Issue description

This might be a duplicate issue.
Not sure if this intended since I believe you need to enable trails in particleprocessmaterial but if you only enable trail support in material and run a particle simulation, some verticies seem to go to world origin.
Steps to reproduce

create a parent node and a gpuparticle3D node as a child node. Set gpuparticle3D position to (2,0,0)
create a draw pass that uses a trail mesh
create material and enable "Use Particle trails"
rotate parent node on a y-axis.

Minimal reproduction project
TrailParticleTest.zip

Just played around a bit more and noticed that when changing trail ribbon mesh's section from 5 to 7 and enabling trails, same result.

Possible related to https://github.com/godotengine/godot/issues/70546

Based on issue's #70546 video, it does seem to relate.

Still reproducible on 4.2.dev 42425baa5:
https://github.com/godotengine/godot/assets/180032/2d71793f-37fa-4bf6-82ea-48e6fc2102d8

I ran into this issue on 4.2 release windows.


Fidget

Fidget is not working, even though a client is already attached to a buffer.

Can you provide more details on what is happening vs what you are expecting?  Are there any error messages?

If i open a lua file, there is no fidget notification on bottom right corner about loading workspace or anything, even though lua-ls is attached to buffer. There is no error message that shows up.

Are you using lazy.nvim?  If so, can you run the :Lazy command and see if fidget is installed and in the 'Loaded' list?  If there were any install issues it would show the error there.  If it wasn't loaded it would be in the 'Not Loaded' list at the bottom.  I tried opening a Lua file with my config and it does show things like 'Loading Workspace' in the bottom right corner with fidget.  Are you using my Starter Kit as a starting point or did you just take specific parts of it?  Can you provide your nvim-lspconfig plugin configuration?

I'm using only your starter kit python branch only but still it doesn't show fidget, but with my old configuration the fidget is showing up. so i'm confused what is wrong.

I just noticed that when i open a lua file outside of ~/.config/nvim/ directory, fidget is showing up, but it's fidget is not working inside ~/.config/nvim/ directory.

Is there any way you can share your config?

im  only using your default starter kit python sir.

What version of Neovim are you using?  All of my setup was done with 0.9.5.  Also, when you first set everything up, if you want to use the exact versions of the plugins that I am using, you can do ':Lazy restore' to reset all plugins to the same versions that I have locked in the repo.  If you never ran this command it would try to install the latest versions of each and there are likely new conflicts introduced.

I'm using neovim v0.10 on linux. i'll try to do :Lazy restore later as i'm on my phone. thank you

If that doesn't work, it could be an incompatibility with Neovim v0.10.0.  Just my opinion, but I never take a .0 release of anything.  I always wait for something more stable.

hi sir, your latest commit fixed the issue { "folke/neodev.nvim", opts = {} }, thanks for your time sir


Update index.mdx

more precise instruction
Overview

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests / types / typos

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Use cases and why

One use case

Another use case

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

thanks @neonpie !


Clean uploaded Form file to remove < or > without trailing or preceding spaces

Summary | Résumé
Cleans uploaded form files to ensure opening/closing angle brackets have preceding or trailing spaces so they're not mistaken for HTML tags. ie, <something> becomes < something >

@dsamojlenko I fixed up the backspace issue.
Let's get CI fixed up and write some tests for this PR.


Look into SM bug

https://app.roll20.net/forum/permalink/9698823/
It does have issues with SM in the tool, however. If I have a SM -3 monster attack a SM 0 PC, it doesn't get a +3 bonus unless I pretend the PC is SM +3.

Not an issue. Rules as Written, only apply the targets size modifier, not the difference between attacker and target.

Not an issue. Rules as Written, only apply the targets size modifier, not the difference between attacker and target.


Cannot enter text into search box on iOS home assistant app

Every time i try to type into the textbox, my keyboard comes up then immediately goes back down. No matter where i put the card it seems to do this.

Hi,
I indeed have the same problem on android (and even via the browser). As I mostly had enough time to type, I didn't focus on that bug.
I now checked it as I understand it might be annoying. Beta version is available on branch develop. I still need a few days to test it, but you already can try it. Be aware you also need the beta version (also on branch develop) of the Integration 'Kodi Media Sensor'


seamail:  add avatars to 'header' and make subject smaller

Sounds perfect.   We could also add the subject to the list of participants you get when long pressing on the header.

Done.


Incrementing default gas limit to homestead era

This PR simply brings the block gas limit up to the Homestead release

Please rebase.

Closing in favor of #91


Fix: Remove Unnecessary Vertical Scrollbar

Issue Summary
There is an unnecessary vertical scrollbar appearing on the main page, even when the content fits within the viewport.
https://github.com/user-attachments/assets/31101719-2c47-4a9d-b71e-f1a3f3b6bc38
Steps to Reproduce
just go to main page and scroll vertically
Ghost Version
1.26.1
Node.js Version
v20.15.0
How did you install Ghost?
ghost install
Database type
MySQL 5.7
Browser & OS version
Chrome <IP_ADDRESS> on Windows 10
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

Hi @prajwaln07 ,
I noticed the issue regarding the unnecessary scrollbar on the main page and I would love to work on fixing this. Could you please assign this issue to me?
Thank you!
savit0r

so you want the scroll bar to be completely removed from main page?

@Sourabh-km13 YES

Hi @prajwaln07 ,
I noticed the issue regarding the unnecessary scrollbar on the main page and I would love to work on fixing this. Could you please assign this issue to me?
Thank you! savit0r

Hi @prajwaln07 ,
I noticed the issue regarding the unnecessary scrollbar on the main page and I would love to work on fixing this. Could you please assign this issue to me?
Thank you! savit0r

@savit0r NO,sorry


chore: Update Bloop to 2.0.2

Btw. I managed to improve the situation in https://github.com/scalacenter/bloop/pull/2450


Automerge doesn't respect branch deletion rule

Description
If your repository is configured to automatically delete a PR branch when it is merged but you use (for instance) "Fast Forward Only (after checks succeed)" to schedule an automerge of the branch after branch protection rules pass, then the PR branch will not be automatically deleted when the PR is merged.
Gitea Version
1.22.2
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
Being run from rootless docker image
Database
PostgreSQL

Not sure if this is related, I've noticed after upgrading to 1.22.3 (probably happened in an earlier release, we didn't use branches often before) that the merge PR does not delete the branch like it's supposed to.  My question on the related side of things, is I've noticed I'm unable to delete a branch in any way it seems, did something change with a later release that would be preventing this?  For example when going to the branches list on a repo, there used to be a delete button and it's just not there for me anymore.  I can't even delete the branches remotely. This is true even on repos with no branch branches at all (as far as the button being missing from the branches page at least).

Not sure if this is related, I've noticed after upgrading to 1.22.3 (probably happened in an earlier release, we didn't use branches often before) that the merge PR does not delete the branch like it's supposed to. My question on the related side of things, is I've noticed I'm unable to delete a branch in any way it seems, did something change with a later release that would be preventing this? For example when going to the branches list on a repo, there used to be a delete button and it's just not there for me anymore. I can't even delete the branches remotely. This is true even on repos with no branch branches at all (as far as the button being missing from the branches page at least).

Can you reproduce this on demo site?

Seems the demo site is down/unreachable for me at the moment.  I have narrowed this down to something having changed in branch protections, we have a '**' protection in place to require signed commits and that breaks Gitea doing branch deletion on merge along with the ability to delete them from the GUI.  Seems branch deletion got thrown in there without a new option being added to branch protections to allow for that?


Need more cases of ObjectDisposedException

I just spent several hours fighting bugs in Menubar and ContextMenu only to discover that they are passing around views (and themsevles) that have been disposed.
We have a lot of this going on and we need to squash it.
This issue is to track adding a bunch of calls like this in key places:
#if DEBUG_IDISPOSABLE
            if (WasDisposed)
            {
                throw new ObjectDisposedException (GetType ().FullName);
            }
#endif

Visible get/set
Enabled get/st
HasFocus get/set
Mouse event handling

Just a reminder:
Don't forget to use the handy-dandy ObjectDisposedException.ThrowIf helper instead of manually constructing and throwing, too!
Better static analysis and optimization, with a little less code, so a win all around!
The code in the initial post would become:
ObjectDisposedException.ThrowIf(WasDisposed,typeof(whateverClassThisIsIn));

Don't use the overload that takes an object instance, as it uses reflection to get the type name (which is all the second argument is for). It calls GetType() at run-time on whatever you passed. typeof(SomeType) is a compile-time constant, which avoids that and is more efficient, to boot. 🥳

Also, it's typically not thrown from property accessor methods, unless the accessor itself accesses a different disposable object that has been disposed, and it is supposed to leave the throwing of that up to that object.
Methods, however, are expected to throw, ideally with a more detailed InnerException if possible/relevant.

A couple of heads ups before you write a whole bunch of code, just in case I didn't beat this particular horse enough or properly last time we did exception work. 😄

Oooo
And for methods that return IDisposables, and any types that are themselves IDisposable, it's friendly to include the [MustDisposeResource(bool)] attribute, with true if the caller is expected to dispose it or false if it doesn't matter who disposes it.

Here's the issue I found that led to this exercise:
https://github.com/gui-cs/Terminal.Gui/issues/3678


Support .jscsrc file outwith project tree

I would like to share the same .jscrc configuration between multiple projects. However, searching for a .jscrc stops at the project root.
Could support for a global default config file or scanning beyond the project root be supported? For example, the Brackets JSHint extension supports scanning the complete parent tree to find a configuration file.

:+1:  This would be huge!

Being able to place a jshintrc and a jscsrc in the same folder (shared by multiple children projects) would be great, either by scanning the same way as jshint or with a config setting.

The extension v0.6.0 now supports walking up directories to look for config files. It will look for the nearest config file from the file you're currently editing.
However, 0.6.0 requires that you have JSCS installed globally via:
npm install -g jscs


Can access the accelerometer api int Android-SDK?

Can use accelerometer api int Android-SDK?Thank you !

We are working on it.

IS this feature available in the Android SDK now?

@FernandoNos not yet, we will be releasing this week power modes.


Is Oidc.UserManager singleton or can have multiple objects

we are trying to hook multiple policies using Oidc.UserManager by creating multiple objects by passing the different set of settings.
We are working on Azure B2C where we can create multiple policies like Sign-in, Edit Profile, Sign -in and Sign-up polices. Each policy has its own settings to make them work we are trying to create multiple objects for Oidc.UserManager.
For that, we want to know

Oidc.UserManager act as a singleton object or can have multiple objects.
If I create multiple objects for Oidc.UserManager and hook events for all the objects differently, when Oidc trying to fire addAccessTokenExpiring eventwill  it will fire for all the objects events at a time or based on the expires_in time it will fire specific object event.
Example: Object A have expires_in have 180 seconds and Object B have expires_in have 120, if clock reaches 60seconds it will fire Object B only or it will fire Object A also?

1: You can create many instances of UserManager, but it stores things and the key is based on the combination of the authority and the client id. So if B2C uses the same authority and client id for its multiple endpoints, then this will be a problem. As a side note, I'm quite confused by that design -- i thought they used a custom "p" param to the one authorization endpoint to trigger the different profiles? Anyway...
2: The events and timers will be per-instance, per-user.

BTW, you should check to see if B2C supports CORS on its metadata endpoint. Normal AAD does not.

hmm yes, currently B2C also not supports CORS. We put metadata manually and it is working fine.
Even we added manually metadata we kept using authority and clientid same across multiple usermanager objects, because of that it not worked for us.
Now we modified all the authorities to accept different URLs and SPA is working fine 👍
Can we assume if we add metadata and keys manually, authority endpoint will not be useful in this case?

Can we assume if we add metadata and keys manually, authority endpoint will not be useful in this case?

Authority as a value is still needed to uniquely identify the OP/STS/IdP you're using. You will still need to set it, and it still needs to be unique for each OP you're using.

Regarding "You can create many instances of UserManager, but it stores things and the key is based on the combination of the authority and the client id":
I bumped into the same question - I want to use multiple user managers for different audiences (not with IdSrv but another authorization service).
I think it is solved by passing a userStore with a custom prefix  to the UserManager constructor:
new UserManager({
...,
userStore: new WebStorageStateStore({ prefix: oidc.${userManagerName}., store: sessionStorage })
})
So, I have  different UserManager objects, each identified by some kind of name, and using different session storage entries.
This seems to work, unless I'm overlooking something?


flake8 exceptions raising on regular basis when defining docstring

the moment I open a docstring triple quote, I get flake8 exceptions
I'm using the most up to date version of the linter package

Does flake8 work on the command line?
I wouldn't be surprised if you upgraded pep257 to version 0.7.0. flake8 crashes with anything newer than 0.6.0 (and older versions to, I think 0.3.0 and below, but I'm not sure.

Actually, closing as a duplicate of #96, please respond with what you find there as neither of the other two people reporting this have responded yet.

every time I open a docstring I get the flake8 errors on the right of the screen
flake8 DOES work on the command line (using version 2.4.1)
I'm using python 3.4


[SOFT-294] Create telemetry makefile + get Travis working

Testing Travis atm, don't approve

should be good to go


How to add the static json file in generate function in order to create the dynamic routes from Json file

generate: {     routes: function () {       return axios.get("./static/subcategories.json")       .then((res) => {         return res.data.map((items) => {           return '/items/' + items.id         })       })     }   },

This question is available on Nuxt community (#c9560)

Trying using require() instead of axios.get() for this case, since the file seems to be readily available.

@galvez Thanks for your response . can you please share the code if possible


python37: 3.7.13 -> 3.7.14

https://pythoninsider.blogspot.com/2022/09/python-releases-3107-3914-3814-and-3714.html
Fixes: CVE-2020-10735
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

python37.pkgs.setuptools-scm is broken, because it requires importlib-metadata, which in turn requires setuptools-scm, which is why python37.tests are broken.


Error when using tabs-reopen.sh

Hello,
when using the tabs-reopen.sh script, I get the following output and nothing happens on the phone.
Using unsafe HTTP verb GET to invoke /json/new. This action supports only PUT verb.
This message is repeated for every tab that should be reopend.
I had no issues saving the tabs from my old phone.
I'm running the script on Ubuntu 23.04, PHP 8.1.12ubuntu4.3, Android Debug Bridge version 1.0.41
I'm trying to reopen the tabs on a Pixel 8 Pro running Android 14.
If you need any more logs or information, I'm happy to provide them.
Best regards

The method was changed back in march this year:

Internet Archive - March 06 / 2023
Internet Archive - March 12 / 2023

The current version of the docs does indeed state PUT as the endpoint method.

I just merged #24 to fix this.


[Intellisense TS] Poor Intellisense support when overriding methods/properties/getters/setters

From @Elarcis on October 14, 2016 18:13

VSCode Version: Code 1.6.1 (9e4e44c19e393803e2b05fe2323cf4ed7e36880e, 2016-10-13T16:21:53.542Z)
OS Version: Windows_NT ia32 10.0.14393
Typescript version: 2.0.3
Extensions: abierbaum.vscode-file-peek-1.0.1, alefragnani.Bookmarks-0.9.2, be5invis.vscode-icontheme-nomo-dark-1.1.4, codezombiech.gitignore-0.2.0, dbaeumer.vscode-eslint-1.0.8, eg2.tslint-0.5.39, gerane.Theme-TomorrowNight-0.0.2, jakethashi.vscode-angular2-emmet-1.0.9, joelday.docthis-0.3.5, johnpapa.Angular2-1.0.2, ms-vscode.csharp-1.4.1, ms-vscode.typescript-javascript-grammar-0.0.4, msjsdiag.debugger-for-chrome-1.2.0, ryu1kn.partial-diff-0.1.0

Intellisense falls a bit short when overriding a class's properties and methods using inheritance.
Steps to Reproduce:

In a Typescript file, start with this base code:

abstract class Parent {
    /**
     * Function that can be overriden.
     */
    public abstract toOverride(param1, param2): number;

    public canBeOverridden(param1) { }
}

class Child extends Parent {

}

In the Child class, try to implement the function toOverride : no autocompletion is proposed, yet TS shows the error Non-abstract class 'Child' does not implement inherited abstract member 'toOverride' from class 'Parent'. when hovering the class name until we actually write a function with the same signature, which means it knows the missing function's signature.
When the function has been implemented in Child, the parent function's doc isn't reflected in the overriding function, even if it has no doc. I can imagine why it is so, but that means we have to copy paste the doc for it to actually show up when we use Child.toOverride().
It doesn't work either with non-abstract functions, nor getter/setters, etc.

Full code:
abstract class Parent {
    /**
     * This doc will only show up for `Parent` instances,
     * or objets treated as `Parent`.
     */
    public abstract toOverride(param1, param2): number;

    public canBeOverridden(param1) { }
}

class Child extends Parent {
    // No completion has been provided while implementing this function
    public toOverride(p, p2): number {
        return 2;
    }
}

new Child().toOverride(1, 2); // The doc doesn't show up when hovering toOverride.
(new Child() as Parent).toOverride(1, 2); // The doc shows up when hovering toOverride.

Copied from original issue: Microsoft/vscode#13779

@Elarcis thank you for opening this issue. This is an excellent find. This issue is with the JavaScript language service. Migrating to that repo.

Note that I have confirmed this with the latest bits VS Code Insiders and TS 2.0.6

@waderyan I think this is also tracked in #7158

Closed as duplicated of #7158


Update jwt.helper.js to increase Github JWT Token duration

Summary:
Currently Github users are logged out after 5 minutes. This makes it impossible to save changes without refreshing and losing all unsaved changes. As far as I can tell, refresh tokens have not been implemented currently. Increasing the JWT token duration to 1 day should make it more practical to edit Github models. A longer term solution is to finish implementation of refresh tokens.
Description for the changelog:
Update access token and refresh token duration.
Access token updated to 1 day from 5 minutes.
Refresh token updated to 7 days from 24 hours.
Other info:
None

there is a failing unit test because it is searching for '5m', did you want to fix that up? otherwise I can modify this

@jgadsden The unit test should be updated now.


Crossgen doesnt build (rc4)

Project with multiple TargetFrameworks doesnt work
Affect rc3, rc4
repro

dotnet new console -lang f#
set targetframeworks as net451;netcoreapp1.0
dotnet restore -r win7-x64
dotnet build -r win7-x64

Expected

build both net451 and netcoreapp1.0

Actual
Fails:
C:\dotnetcli\dotnet-dev-win-x<IP_ADDRESS>-rc4-004771\sdk\1.0.0-rc4-004771\Microsoft.Common.CurrentVersion.targets(2620,7): error MSB4057: The target "CreateManifestResourceNames" does not exist in the project. [e:\temp3\rc4t\rc4t.fsproj]

version info

FSharp.NET.Sdk: 1.0.0-beta-060000

dotnet --info

.NET Command Line Tools (1.0.0-rc4-004771)

Product Information:
 Version:            1.0.0-rc4-004771
 Commit SHA-1 hash:  4228198f0e

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.14393
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\dotnetcli\dotnet-dev-win-x<IP_ADDRESS>-rc4-004771\sdk\1.0.0-rc4-004771

fixed, was a bad target name reenabled test in suite https://github.com/dotnet/netcorecli-fsc/commit/3949528fa237f404eb1a8e95f9587181335d67c9


Verify php session id is regenerated after successful login to prevent session hijacking.

Jordan, I'll assign it to you. If you need help understanding my changes to how sessions are handled just ask.


CI is "successful" even if Overwrite FS fails

Fixed by the latest commit utilizing the arcgis overwrite API


[feature] Option to filter files

It would be nice to be able to do:
$ erdtree --filter <glob_pattern>
$ erdtree -f <glob_pattern>

For example,
$ erdtree -f !*.snap

Would filter all insta snapshot files, and
$ erdtree -f *.snap

Would only display the snapshot files.
It seems like the ignore crate already supports adding ignore overrides.
Would you accept a PR that implements this?

Idea: cribbing the CLI from ripgrep might reduce friction for newcomers, since there's a high chance they'll already be familiar with that. For reference, it uses a -g/--glob flag:
    -g, --glob <GLOB>...
            Include or exclude files and directories for searching that match the given
            glob. This always overrides any other ignore logic. Multiple glob flags may be
            used. Globbing rules match .gitignore globs. Precede a glob with a ! to exclude
            it. If multiple globs match a file or directory, the glob given later in the
            command line takes precedence.

            As an extension, globs support specifying alternatives: *-g ab{c,d}* is
            equivalet to *-g abc -g abd*. Empty alternatives like *-g ab{,c}* are not
            currently supported. Note that this syntax extension is also currently enabled
            in gitignore files, even though this syntax isn't supported by git itself.
            ripgrep may disable this syntax extension in gitignore files, but it will
            always remain available via the -g/--glob flag.

            When this flag is set, every file and directory is applied to it to test for
            a match. So for example, if you only want to search in a particular directory
            'foo', then *-g foo* is incorrect because 'foo/bar' does not match the glob
            'foo'. Instead, you should use *-g 'foo/**'*.

        --glob-case-insensitive
            Process glob patterns given with the -g/--glob flag case insensitively. This
            effectively treats --glob as --iglob.

            This flag can be disabled with the --no-glob-case-insensitive flag.

@jprochazk sure that sounds great. If possible I would like to go with @ErichDonGubler's suggestion for option/argument naming. Would also appreciate if you included a test here: https://github.com/solidiquis/erdtree/blob/master/src/fs/erdtree/tree/test.rs


frontend: move activeScanQR into ReceiverAddressInput sub-component

After #2870
In case #2870 is merged we can remove a lot of complexity from the open/closed state management of the scan QR dialog.

rebased / also amended the commit message to explain the change from #2870

@thisconnect tested webdev and anroid, CI lint fail is unrelated could you please run CI again ?

@NicolaLS CI passed 🎉


Static files should be accessed from the public folder instead of './static'

Describe the Bug
At least one of the use cases for this project is to use React Email to build, preview and then send transactional (dynamic) emails within existing (almost certainly NextJS) projects.
Right now, accessing static (image) assets in both the React-Email preview and when sending the email via your app requires you to duplicate these assets in the repo because React-Email requires them to be in a static folder within the root, while NextJS - since version 9.0.6, so as far as I can tell, before even the creation of this project - requires them be within the public folder (and you'd have to put your files within an unnecessary static folder inside public too just to match the preview).
This just seems a pointless level of complication and redundancy when you could mirror the public folder into the .react-email public folder instead.
Not only this, but it's additionally confusing to new users in general as there is no documentation about where to put your files so that they are accessible within the .react-email preview app and the demo in the repository incorrectly stores these files within the emails folder (which achieves nothing in reality, and would of course still require duplicating these assets to be available in your public folder anyway even if it did).
Please just simplify things by mirroring the ./public folder into .react-email/public, or at the very least having a configuration file that defines where your static assets are so both ./static and ./public can work as expected.
Which package is affected (leave empty if unsure)
No response
Link to the code that reproduces this issue
https://github.com/resendlabs/react-email
To Reproduce

Place image files in your public folder so your email can include them when sent.
run npx react-email dev
Broken images in preview build because the files are not copied into .react-email/public

Expected Behavior

Place image files in your public folder so your email can include them when sent.
run npx react-email dev
Images work in preview because ./public is mirrored into .react-email/public

What's your node version? (if relevant)
No response

I am also confused as to where to place images.

I am so confused about where to place images.

really should be something that is documented

Our intentions are to be agnostic in relation to what way the user will use for hosting their images, if they will be hosting on their Next App or if they will be hosting somewhere else is not something we can account for perfectly.
That being said, we can't force users to use static files inside of public/static. A possible better solution is for us to have some kind of flag over which users can provide the location of the static files for it to work properly on any case and by default work as is.
Sorry for taking so long to answer this, I'll take a look into getting this in a better shape. I also have the docs in mind and have a few plans of PRs to make as of to improve them in this sense as well.


Updating state with shallow false makes multiple reenders

When changing the search param with shallow: false, makes multiple reenders on client and blocks the RSC to refetch when the client component is heavy.

I think that is a Nextjs problem.
 But I have a reproduction showing some reendes in useSearchParams()
https://search-param-35vu.vercel.app/search-param
https://github.com/juliomuhlbauer/search-param/
Try to click in the links to change the search params. For now the Select does nothing.
I will expand the reproduction to show when the UI is heavy.

Now I made an example with the next-usequerystate and it causes multiple reenders.
This example doesn't block the UI. But when it is with heavy UI it blocks.
https://search-param-nqx9ne9vw-juliomuhlbauer.vercel.app/use-query
Check de console
https://github.com/juliomuhlbauer/search-param/

Made an directory with a heavy UI. Well is not that heavy, but it blocks the render.
https://search-param-wine.vercel.app/use-query-with-select

A workround kind of works: https://search-param-wine.vercel.app/use-query-with-select-in-layout-custom
But is does give an error of React and it does not get the value SSR.

Thanks, I was able to reproduce on my end, will take a look.
I found on the playground that there are two renders on the first query update after a hard reload of the page, but only one for subsequent ones, not sure why.
Also, in development React will use strict mode which performs two render passes per actual render to fish out incorrect useEffect usage, so checking for renders should be done in a production build.

Try removing the useSearchParam hook. I think Nextjs makes a reender whithin the Suspense boundary and causes the UI to reender multiple times.
I created a alternative hook for useSearchParam, but it only works client side. So need to import dynamically without ssr.

Maybe this involves the behavior of useSearchParams: https://nextjs.org/docs/app/api-reference/functions/use-search-params#behavior
I dont't know why its different from the useParams hook.

Thanks! I opened an issue to the Ark UI library: https://github.com/chakra-ui/ark/issues/1738
But I think it is a Nextjs bug.

I opened an issue on Nextjs: https://github.com/vercel/next.js/issues/58684

Closing as unrelated to nuqs (behaviour is reproducible without it).


Rule builder dropdown goes outside viewport

Description
When you edit rules in the rules builder the dropdown goes outside the viewport of the browser, making it impossible to scroll through all the options.
This problem is also visible in the permissions dropdown for users.
Possible Solution
The dropdown should open up where there is space, for example upwards.
Additional context

Hi @Zyles,
thanks for reporting this issue.
This was a general problem which happened to every dropdown menu that was inside a container with limited space. It will be fixed by making the dropdown menus more dynamic, like you already suggested. When not enough space is there it will open up in the opposite direction.
The fix is in code review internally and will be in the next 6.1 patch release. You can track the ticket status here:
https://issues.shopware.com/issues/NEXT-6200
Best regards
Tobias

Hi @Zyles,
the issue should be fixed in the current release. Dropdown menus now open up in the opposite direction when no space is available.

Best regards
Tobias


Add logger in order to see progress

I am not sure whether sharing progress between threads is good idea in context of Embulk, but it fits well for our case.
I've added you as a contributor and a gem owner. I am looking forward to your contribution :smile:

I am not sure too.
In other plugin implementation, logging is separated by each thread.
To log by each thread is maybe better in context of Embulk.

Hmm, because Embulk can run in distributed system, per-thread logging is more consistent, but I do not care.
This implementation is more informative for us, so how about pick this for now and wait for another pull requests from sophisticated embulkers?

how about pick this for now and wait for another pull requests from sophisticated embulkers?

I agree. 👌
it is no probrem for us.

I'll merge now and release soon.


pass detail from widget failures

The detail should be passed to the LANA log.
https://github.com/adobecom/dc/blob/c6c5f3e318dffcd78ebb77905af92eb06e189315/acrobat/blocks/dc-converter-widget/dc-converter-widget.js#L339-L350


TN-2039 Extract OverallStatus strings for translation

Properly extract strings from OverallStatus model

Hello @ivan-c! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file portal/models/i18n.py:

Line 60:80: E501 line too long (86 > 79 characters)

Closing in favor of #3239


Training mamba model with Huggingface transformers from scratch

Hi everyone,
I want to create new mamba model to use in my project for generating new sequences. I have my own dataset based on MIDI files so I'm not keen on using pretrained model and I want to play with number of parameters. For ease of coding, I wanted to use Hugging Face transformers as I saw that it is possible to fine tune model from checkpoint. Is it possible to do it from scratch? I noticed that MambaForCausalLM inherits from MambaPreTrainedModel which inherits from PreTrainedModel in transformers code, and everyone was just fine tuning the model and loading using from_pretrained method and never training it from scratch.
Thanks for answers!

I'm not familiar with the implementation in HF.
In any case it's just some model code so I think training from scratch should work.

Hi @FIlipHand,
I'm interested in training the Mamba model from scratch using Hugging Face transformers. Have you had success with this approach?
Thanks!

Hello @yangzhao1230
Yes, I managed to train a mamba model with huggigface. It is actually quite easy. You can take a look at #294 where someone wrote a sample code that implemented training from a checkpoint. The only difference is that instead of passing model checkpoint as string you have to pass MambaConfig object with your parameters. The rest is the same. I personally haven't been able to scale it properly on my hardware and train it for LM purpose, but I have friends who used smaller mamba models for purposes like phonemizer, and it was successful.


chore(CI): ignore example.tfvars and helpers/foundation-deployer in example-foundation-int tests

Doesn't appear the Integration tests cover foundation-deployer, so I've opened PR https://github.com/terraform-google-modules/terraform-example-foundation/pull/1218 for a dedicated workflow.

applied


String input

This is the on screen keyboard I was working on a while back with all the bugs fixed (i think).
there is also a small dependency checker function used by the keyboard to allow it to run with or without lib_spec_char.ksm and by the example script (a login interface) to determine if you have previously created an account and deny you access to the terminal if you get the user name or password wrong (when used as a boot script).

Ok fixed and retested (since i was modifying the script).

Looks good from here; I'll let you merge this one if you feel it's ready


CBMC: Understand and use the correct shape of stub-harnesses for spec-proofs

When we verify specs, we still need to provide stub harnesses invoking the function-under-proof. Morally, it seems that those harnesses should be irrelevant, but https://github.com/pq-code-package/mlkem-c-aarch64/pull/306 (specifically this commit https://github.com/pq-code-package/mlkem-c-aarch64/commit/a95a2ce56cf11567242a8ff80d6219986cc2851c) suggest that this may be wrong: In this commit, the spec for poly_add() does not provide any bound on the r buffer, and yet the proof goes through. This is likely because the harness allocates the data on the stack, which may inadvertently add to the pre-conditions of poly_add -- it's likely that an uninitialized poly* should have been passed.

[ ] Understand what the correct shape of harnesses for spec-proofs is
[ ] Apply needed changes throughout
[ ] Adjust the CBMC proof guide accordingly

@rod-chapman Please comment

I agree that the harness should infer a little information as possible, so passing an unitialized pointer rather than a pointer to a stack-allocated local object feels OK to me, assuming the proofs all succeed. We need to confirm this advice with CBMC team, then update the Proof Guide accordingly.

The CBMC team has confirmed this.

@rod-chapman Let me know when you have updated the proof guide, and then we can close this.


Inaccurate comment about GovMaxSquareSize being superseded

Context
https://github.com/celestiaorg/celestia-app/blob/b70048b275a0e16e0509634c2175c99e2aad7bd1/x/blob/README.md?plain=1#L64-L70
Problem
GovMaxSquareSize isn't superseded by MaxSquareSize. Maybe this sentence was trying to say that GovMaxSquareSize is upper bounded by MaxSquareSize but that also isn't the case (see https://github.com/celestiaorg/celestia-app/issues/2185).
Proposal
Remove this sentence

This value is superseded by the MaxSquareSize, which is hardcoded and cannot change without hardforking the chain.

Hii can i work on this ..??

@rootulp i want to work on that issue!

Sorry @sumanthsumu1418 already has a PR open https://github.com/celestiaorg/celestia-app/pull/2245


Partial chromosome parallelization

Increase parallelization by breaking chromosomes into pieces

This actually dramatically slows the processing down as the pipeline then has to process a lot more pieces. After splitting by chr, this parallelization is no longer necessary.


need a way to pass in additional options to the underlying commands

For example, I was getting some strange GPG error
ERROR: Unable to fetch GPG key from keyserver.
There are a couple of ways to investigate it - by passing the debug flag to the lxc-create command or by passing the --no-validate.
As far as I could tell, dev-lxc wrapper doesn't allow options to be passed to underlying commands.

Thanks for the suggestion! Resolved by https://github.com/jeremiahsnapp/dev-lxc/commit/72f069b785cc9c6eca46e43909ac11fbcaf38e46